fix: wrap hardcoded alert() strings with _() for i18n localization

[vanshika2720]

fix: localize hardcoded alert() strings for full i18n coverage

What

Wraps all hardcoded English alert() strings with _() so they participate in Music Blocks’ i18n system and are translatable for non-English users.

This completes a targeted audit of user-facing alert dialogs across the JavaScript codebase.

---

Files Changed (5)

• js/blocks/ProgramBlocks.js
• js/activity.js
• js/utils/synthutils.js
• js/widgets/sampler.js
• js/loader.js

---

Notable Implementation Detail

loader.js Bootstrap Guard

loader.js executes during early RequireJS bootstrap, before _() is guaranteed to exist.

Each alert site uses an inline safety guard:

const t_ = typeof _ === "function" ? _ : s => s;
alert(t_("..."));

This is intentional because:

• These alerts are inside separate RequireJS callbacks.
• They fire only during bootstrap failure paths.
• Each site is independently safe and self-contained.
• Avoids restructuring early initialization code in a surgical fix branch.

There are no functional behavior changes — only localization support added.

Intentionally Not Changed

• js/SaveInterface.js — alert strings appear inside dynamically generated HTML source text, not runtime UI alerts.
• js/blocks/ActionBlocks.js — alert(xmlHttp.responseText) is fully dynamic server content (no static English string to localize).
• Alerts already wrapped in _() were left untouched.

Scope

• No refactors
• No structural changes
• No behavior changes
• Localization-only fix
  This keeps the PR small, reviewable, low-risk, and focused strictly on improving i18n coverage.
• Bug Fix

[github-actions]

✅ All Jest tests passed! This PR is ready to merge.

[vanshika2720]

@walterbender

[Ashutoshx7]

@vanshika2720 tried it locally it seems okay to me !
great works btw

[walterbender]

Looks good but there seem to be unrelated changes in sampler.js. Please remove those changes.

[github-actions]

❌ Some Jest tests failed. Please check the logs and fix the issues before merging.

Failed Tests:

GraphicsBlocks.test.js

[github-actions]

❌ Some Jest tests failed. Please check the logs and fix the issues before merging.

Failed Tests:

GraphicsBlocks.test.js

[github-actions]

❌ Some Jest tests failed. Please check the logs and fix the issues before merging.

Failed Tests:

GraphicsBlocks.test.js

[github-actions]

✅ All Jest tests passed! This PR is ready to merge.

[github-actions]

✅ All Jest tests passed! This PR is ready to merge.

[github-actions]

✅ All Jest tests passed! This PR is ready to merge.

[github-actions]

❌ Some Jest tests failed. Please check the logs and fix the issues before merging.

Failed Tests:

GraphicsBlocks.test.js

[Ashutoshx7]

@vanshika2720 there is still lint faling in sampler.js
take a look

[vanshika2720]

@Ashutoshx7 YES i am working on it.

[github-actions]

✅ All Jest tests passed! This PR is ready to merge.

[vanshika2720]

Looks good but there seem to be unrelated changes in sampler.js. Please remove those changes.

@walterbender Done