Repository navigation
F182 B6b: i18n sweep — Help glossary + final sweep audit (sweep complete) - #252
Merged
Merged
Conversation
added 2 commits
July 27, 2026 22:17
Task 33 (B6b): the 37-row term/definition table plus its closing repo-note, en+es+tl. Extends the help namespace PR 6a created for the rest of HelpPage; the id="glossary" scroll-spy anchor stays byte-identical. English values are unchanged from the prior hardcoded copy (Task 34 handles GLOSSARY.md reconciliation separately); es/tl are machine-drafted pending native review, matching this pack's existing loanword precedent.
Final-audit cleanup for the #182 string sweep: correct per-namespace comments in en.ts/enums.ts that still claimed a namespace was English-only / deliberately not in TRANSLATED_NAMESPACES / deferred to a native-speaker pass. All 29 namespaces now ship machine-drafted es/tl and are listed in TRANSLATED_NAMESPACES, so those assertions were false. Comments only — no keys, values, or behavior changed.
90 tasks done
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
B6b — Help page: glossary + final sweep audit. The final PR of the #182 SPA i18n string sweep. Externalizes the in-app Glossary table (the last hardcoded copy in the app), then closes the sweep with a catalog-wide audit.
What's in this PR
Glossary table →
helpns (translated inline)The
<h3 id="glossary">+ its 37 term/definition rows + the closing repo-note, externalized into the existinghelpnamespace (76 keys, mirrored en/es/tl):'/"→ real characters; not rewritten).id="glossary"kept byte-identical (scroll-spy rail target).<Trans>keeping<code>specs/product/GLOSSARY.md</code>literal (path not translated) in all three languages.// machine-drafted (#182) — pending native reviewmarker.Final sweep audit (Task 34)
web/src+index.html: zero real hardcoded user-facing strings remain. The 3 residuali18n:scanhits are scanner false positives (a boolean chain in DailyEntryPage, a CSSwidth + "%"in StockPage, areturnstatement in ErrorBoundary);index.html<title>Cluckwork</title>is the intentional brand string.TRANSLATED_NAMESPACES/ deferred to native pass" comment blocks in en.ts + 1 in enums.ts (comments-only — 110/110 balanced, no key/value/code changed). Every namespace is translated now, so those notes were false.test:coverageabove floors (stmts 86.95 / br 76.85 / fn 80.64 / ln 90.31);buildclean.The sweep is now code-complete
Every client-authored SPA string is in the catalog, and — under the translate-now switch — es/tl are shipped inline for all 29 namespaces (machine-drafted, pending native review).
npm run test1122 green (incl. catalogParity);npm run typecheckclean;npm run buildsucceeds.Reviews: Task 33 (glossary) per-task review — READY; whole-branch review (opus comprehensive) — READY, zero findings (glossary parity re-derived, comment-cleanup confirmed comments-only, no regression).
Stays open on #182 (not this PR)