feat(i18n): spec 019 foundation: language packs, checker, problem+json, dashboard runtime

[ValgulNecron]

Before: Gameplane has no translation system. Dashboard text, API errors and notifications are English-only, and dates and numbers are hard-coded to en-US.

After: Gameplane has one shared catalog and its checks, still behind an English-only catalog. A new i18n/ Go module holds the language packs, an ICU MessageFormat formatter, a pack validator and the i18n-check CI tool. The API returns RFC 9457 problem details with a messageId only to clients that ask for application/problem+json. The dashboard has a react-intl runtime: language resolution, a no-flash <html lang> boot snippet and locale-aware formatting. Nothing visible changes in English, and plain-text API error bodies stay byte-identical.

This is the foundation (Phases 1–2) of spec 019, tasks T001–T034 plus T048. The pickers and the per-story work come in later PRs.

How

• i18n/: icu (parser for the ICU subset, CLDR plurals via golang.org/x/text), pack (load, validate with the finding codes from contracts/check-tool.md, completeness, 80% listing rule), catalog (embedded packs, regional → base → English fallback, invalid translations dropped), cmd/i18n-check and one seeded defective fixture per finding code (SC-007). Coverage gate 90; the module is added to go.work, GO_MODULES, the CI matrices and dependabot.
• API: api/internal/messages registry (server.* ids with translator notes). httperr.Message and WriteMessage. Content negotiation parses Accept q-values, and wildcards never select problem details. 5xx responses never echo error text or args. make i18n-sync writes the registry into i18n/locales/en/app.json.
• Web: src/lib/i18n/ (provider, resolution order FR-004, storage, format helpers, server-message mapping). APIError parses problem bodies but keeps err.body as the English detail, so existing callers are unchanged. New scripts i18n:extract, i18n:compile and i18n:manifest. The formatjs ESLint rules are scoped to migrated paths. The Dockerfile builds the language manifest in a Go stage.
• CI: the web job builds the manifest and runs i18n-check, which checks extraction drift and server-message drift and writes before/after completeness to the job summary. It also self-tests the checker on the fixtures. make check-i18n joins make lint.

Spec notes

• --registry <file> was added to i18n-check for the two server flags, and contracts/check-tool.md is updated to match.
• go:embed lives in the root i18n package because embed paths cannot use ...

Only compile checks and the generators were run locally (go build, tsc --noEmit, npm run build, make i18n-sync). Tests and lint run here in CI.

🤖 Generated with Claude Code

---

Generated by Claude Code

[coderabbitai]

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

• Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[github-actions]

CI Report - 2ec9b39

run 38323667797 - attempt 1

1 failed - 29 passed - 0 skipped - 97m44s elapsed

Failing

Job	Log
go	logs

Coverage

Module	Measured	Gate	Margin
web	L95.61 S94.17 B88.9 F91.92	L92 S92 B82 F76	+2.17

Changed

Area	Files	+/-
i18n/	75	+6034/-0
api/	55	+3410/-209
operator/	34	+3109/-40
web/	47	+2887/-103
(root)	7	+2521/-119
assets/	36	+2197/-114
gp-module/	20	+1283/-8
test/	8	+708/-8
specs/	8	+216/-82
docs/	6	+276/-2
charts/	18	+164/-18
.github/	3	+74/-2
hack/	2	+53/-4
.devcontainer/	1	+1/-1

Slowest jobs

Job	Duration
e2e telemetry / amd64 (kind)	23m13s
e2e telemetry / arm64 (kind)	22m3s
e2e capture overhead (kind)	13m29s
build e2e images	12m19s
e2e multicluster / amd64 (kind x2)	9m36s

e2e buckets run

api-agent, api-auth, api-mods, api-rbac, api-roles, bot-fast, multicluster, operator, ratelimit, telemetry, upgrade

[ValgulNecron]

web e2e (mock) fails in web/e2e/specs/api-cache.spec.ts:68. The test expects API reads to send exactly Accept: application/json, but this PR sends Accept: application/json, application/problem+json, which spec 019 contracts/server-messages.md §1 requires. The test exists to check that API reads never ask for HTML, and that still holds. Only the exact header string changed. Updating the test's expected value needs the maintainer's sign-off, which has been asked for in the project thread. Until then this check stays red.

---

Generated by Claude Code

[ValgulNecron]

web e2e (mock) fails on c0f0aba in two tests: serverDetail.spec.ts:35 and settingsSubTabs.spec.ts:50. Both assert that no console errors appear, and both see Failed to load resource: ... 502 (Bad Gateway).

Cause: this PR's AppI18nProvider requests GET /auth/locale. The browser MSW handlers in web/src/test/handlers.ts have no route for it, so the request falls through to the dev proxy, which has no backend and returns 502. The fix is a one-line mock route in the default and screenshot handler sets. That changes test setup, so it waits for the maintainer's sign-off. No assertion changes.

---

Generated by Claude Code

[ValgulNecron]

docs and repository checks fails on 45c29e4 at make check-coverage-gaps. The error is specs/013-expand-test-coverage/coverage-gaps.md:3: Summary closed count is 3, table has 4 (and open 18 vs 17).

This isn't caused by this PR: it doesn't touch that file, and master 2155568 has the same mismatch. #672 and #674 each flipped a gap against the same old summary line. No fix exists yet. I've asked the spec 013 thread, which owns the file, to fix it on master. I'll merge master here once the fix lands.

---

Generated by Claude Code

[ValgulNecron]

web is red on b1b2508 because CaptureWidget.test.tsx › "enables download button only for Completed captures" failed at line 798.

The test races the capabilities query. It asserts the Completed row's download button is enabled right after the list renders, but canDownload stays false until capabilities resolves. This PR's only change to CaptureWidget.tsx swaps toLocaleString() for fmt.number(), which doesn't touch the disable logic. The same test passed on this PR's earlier runs.

A fix already exists on #677 (a8ffe53, which wraps the assertion in waitFor). It isn't ported here because this repo needs explicit sign-off before any test edit. It will reach this PR when #677 merges and master is merged in.

---

Generated by Claude Code

[ValgulNecron]

go (operator) is red on both architectures because TestGameServer_ReleasedAddressBecomesAvailable times out (gameserver_envtest_test.go:3805).

The cause is in this PR. The new TestGameServer_ConditionArgsAddressInUse holds 203.0.113.20, the same address the existing test uses. Address conflicts are checked cluster-wide, and envtest never deletes namespaces, so the leftover holder keeps the other test reporting AddressInUse.

Fixing it means editing the new test. This repo needs explicit sign-off for test edits, and one option (a different IP) was declined. The fix is waiting on a decision in the project thread.

---

Generated by Claude Code