Repository navigation
feat: add opt-in translation checks during compilation - #794
everton-dgn wants to merge 4 commits into
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f5675792f6
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cbfece1855
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Problem
Projects that require every configured locale to be complete currently need a separate validation step before compiling. Paraglide's locale fallback is useful, but it can also conceal an accidentally missing translation.
Change
Add an opt-in
checkTranslationscompiler option, available throughparaglide.config, framework plugins, and the compiler API. It runs the Inlang SDK'scheckBundleagainst the bundles already loaded for compilation and reports diagnostics before writing output. The default remainsfalse, preserving existing fallback behavior.The compiler now closes the loaded project in a
finallyblock, including when validation fails. Documentation explains the checks and the boundary in development: watch integrations can retain the previous valid output after a failed compilation. Unused-message detection remains part ofinlang check.Validation
pnpm run cipassed withVITEST_MAX_FORKS=2 VITEST_MIN_FORKS=1. An earlier unrestricted run timed out in an existing declaration-emission test; it passed on the full rerun.