fix(i18n): drop shebangs that break test collection on Windows

[k33bz]

Problem / Motivation

scripts/check-i18n-strings.mjs and scripts/i18n-translate.mjs open with
#!/usr/bin/env node. Four i18n tests import them. On Windows, Vite's CJS interop
prepends its generated requires to line 1, which leaves the shebang sitting after
executable code:
const fileURLToPath = __vite__cjsImport3_node_url["fileURLToPath"];#!/usr/bin/env node
^ Invalid Character !

Rolldown fails the parse, and the importing file collects zero tests, reporting
no tests rather than a red assertion.

Why it matters

Four gates stop gating on Windows, silently:

• unitLiterals, the number+unit literal ceiling
• localeFormatting
• untranslatedRatchet
• translateDriver
  A green local run on Windows means nothing for those four, and the failure announces
  itself as no tests rather than as an error. CI is Linux-only, where the transform
  leaves the shebang on its own line, so nothing catches it there either. Given Dev Fleet Pull + Build breaks the venv on Windows: pip cannot replace the locked kirocrew.exe and leaves the editable install stripped #2297
  and Windows flake: pipe-close test asserts a global CloseHandle count, so unrelated handles make it fail 2 == 1 #2301, Windows contributors are a real constituency.

What changed (motivation → approach → change)

Root cause: a shebang is only legal on line 1, and the CJS transform takes line 1.
Change: delete both shebangs. They were decorative. Both files are mode 100644
(not executable) and every invocation goes through node scripts/<name>.mjs from
package.json, so nothing could have executed them directly. The modules already
extracted so gates can be shared with tests, scripts/lib/*.mjs, carry none, which is
the pattern this follows.
Guard: src/test/scriptsShebang.test.ts asserts that no script under scripts/
imported from src/ opens with #!. A Linux-only CI cannot see this class of bug, so
a convention would not survive it.
Considered and not done: moving the imported exports into scripts/lib/ the way
qa-checks.mjs did. That is arguably the better end state, but it touches four test
files and two scripts to fix a two-line defect. Happy to follow up if preferred.

Tests

src/test/scriptsShebang.test.ts, two assertions:

• no imported script opens with a shebang. Mutation-checked: restoring either
  shebang fails it and names the file.
• the discovered import set is non-empty, so a future move behind scripts/lib/
  cannot make the guard vacuously green.

Manual verification

Check	Windows	Linux
4 affected suites + guard	0 collected → 119 passed	119 passed
Full src/i18n/ suite		597 passed
npm run check		864 files passed
npm run lint:i18n		normal report, exit 0
node scripts/i18n-translate.mjs		prints usage, exits clean
The last two matter most. Removing a shebang from a file that gets run is how you break
a build, so both scripts were exercised the way package.json and the docs invoke them.
One unrelated failure in the full run: unitLiterals timed out at 19.7 s under
--coverage on a 16-core box. It passes in isolation and is untouched by this diff.
Same pre-existing flake reported in #2023, #1810 and #2159.

Screenshots / video

N/A. No UI change.

Related Issues

None open for this. Adjacent Windows issues: #2297, #2301.

Checklist

• Single commit with a Conventional Commits title
• Existing tests pass and new tests added for new functionality
• Self-review completed; code follows project style guidelines
• Documentation updated (if applicable): N/A
• No secrets, credentials, or internal references in the diff

[github-actions]

UX Review (Fable 5, fork) — ✅ PASS

Advisory UX-level review of 541da895a61c600ca97c14e13efdc60c7f344915 via the fork AI-review pipeline — updated in place on each push; does not block merge.

UX-Verdict: PASS

Developer-tooling-only change — removes two script shebangs and adds a test guard; no user-facing surface, string, or rendered pixel is touched.

[UX-REVIEWED] 541da89

[github-actions]

Opus 4.8 Review (fork) — ✅ no blocking findings

Reviewed 541da895a61c600ca97c14e13efdc60c7f344915 via the fork AI-review pipeline; updated in place on each push.

Review details

No findings.

[OPUS-REVIEWED] 541da89

[github-actions]

Design Review (Fable 5, fork) — ✅ PASS

Advisory design-level review of 541da895a61c600ca97c14e13efdc60c7f344915 via the fork AI-review pipeline — updated in place on each push; does not block merge.

Design-Verdict: PASS

Root-cause fix (shebang is only legal on line 1), verified decorative (mode 644, node-invoked), with a cross-platform guard a Linux-only CI can't provide.

[DESIGN-REVIEWED] 541da89

[github-actions]

GPT 5.6 Review (fork) — ✅ no blocking findings

Reviewed 541da895a61c600ca97c14e13efdc60c7f344915 via the fork AI-review pipeline; updated in place on each push.

Review details

No findings.
[GPT-REVIEWED] 541da89