fix(cli): add dev.usePseudotranslator to config schema (v1.14)#2019
fix(cli): add dev.usePseudotranslator to config schema (v1.14)#2019cherkanovart merged 2 commits intomainfrom
Conversation
… in setup The setup.ts changes from PR #1984 correctly check dev.usePseudotranslator, but the I18nConfig schema (v1.13) has no dev field — Zod strips it during parsing, making the check always undefined. This extends the schema to v1.14 with a dev.usePseudotranslator option so the config value actually reaches the CLI runtime. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review infoConfiguration used: defaults Review profile: CHILL Plan: Pro 📒 Files selected for processing (4)
📝 WalkthroughWalkthroughIntroduces configuration version 1.14 with a new Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related issues
Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
Comment |
Summary
dev.usePseudotranslatoroptional field--pseudoflag anddev.usePseudotranslatorconfig for provider selection and auth skippingContext
PR #1984 correctly modified
setup.tsto checkctx.config?.dev?.usePseudotranslator, but theI18nConfigschema (v1.13) has nodevfield — Zod'ssafeParse()strips it during parsing, so the value is alwaysundefinedat runtime.This PR adds the missing schema definition so the config value actually reaches the CLI.
Test plan
devfield🤖 Generated with Claude Code
Summary by CodeRabbit
New Features
dev.usePseudotranslatorconfiguration option to enable pseudo translator mode directly from your config file, complementing existing CLI flags.Tests