i18nize remaining React UI strings#2577
Conversation
|
Warning Review limit reached
Your plan includes 5 reviews of capacity. Refill in 32 minutes and 48 seconds. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more review capacity refills, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than trial, open-source, and free plans. In all cases, review capacity refills continuously over time. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (103)
Comment |
- Add home.usageExhausted{Title,Body,Cta} to en (already in non-English locales)
- Mirror remaining missing keys from en chunks into all 12 locales so the
i18n coverage gate has 0 missing/extra/drifted entries
- Update MemoryDebugPanel test to assert on the translated key (mock returns
keys as-is)
- Update Skills composio-catalog test: button label is now "Try again"
- Update AIPanel test: i18n value uses "..." not "…"
Summary
scripts/i18n-react-audit.tsandpnpm i18n:react:checkto detect newly introduced hardcoded UI strings in TSX.t(...)instead of embedding English literals.Problem
Solution
aria-label,placeholder,title, andlabel.Submission Checklist
diff-cover) meet the gate enforced by.github/workflows/coverage.yml. Runpnpm test:coverageandpnpm test:rustlocally; PRs below 80% on changed lines will not merge. N/A: not run locally in this session; merge gate will enforce coverage in CI.docs/TEST-COVERAGE-MATRIX.mdreflect this change (orN/A: behaviour-only change) N/A: behaviour-only i18n/tooling change.## RelatedN/A: no coverage-matrix feature rows changed.docs/RELEASE-MANUAL-SMOKE.md) N/A: no release-manual-smoke doc changes required for copy/i18n plumbing.Closes #NNNin the## Relatedsection N/A: no linked GitHub issue was provided for this request.Impact
pnpm i18n:react:check.Related
AI Authored PR Metadata (required for Codex/Linear PRs)
Linear Issue
Commit & Branch
codex/fix-react-i18n-hardcoded-stringsde77b6c7c6076a55edab4c0e58ed9e97762f9715Validation Run
pnpm --filter openhuman-app format:checkpnpm typecheckpnpm debug unit src/components/settings/panels/__tests__/DevicesPanel.test.tsx src/components/settings/panels/devices/PairPhoneModal.test.tsx src/components/channels/mcp/McpCatalogBrowser.test.tsx src/pages/onboarding/steps/SkillsStep.test.tsxpnpm debug unit src/components/channels/mcp/InstallDialog.test.tsx src/components/channels/mcp/InstalledServerList.test.tsx src/components/channels/mcp/InstalledServerDetail.test.tsx src/components/channels/mcp/McpServersTab.test.tsx src/components/channels/mcp/McpCatalogBrowser.test.tsxpnpm debug unit src/components/settings/panels/TeamInvitesPanel.test.tsxpnpm debug unit src/components/settings/panels/TeamMembersPanel.test.tsxpnpm debug unit src/pages/ios/PairScreen.test.tsx src/pages/ios/MascotScreen.test.tsx src/components/commands/__tests__/Kbd.test.tsxpnpm --filter openhuman-app rust:checkvia pre-push hookcargo fmt --manifest-path app/src-tauri/Cargo.toml --all --checkviapnpm --filter openhuman-app format:checkValidation Blocked
command:N/Aerror:N/Aimpact:N/ABehavior Changes
Parity Contract
Duplicate / Superseded PR Handling