Repository navigation
Conversation
- Set up i18next and react-i18next infrastructure - Extract UI strings from Help, Auth, and Settings components - Add English locale files with proper structure - Implement styledText utility for interpolation - Convert Help tests to use snapshots per Jacob's feedback - Add direct component styling verification This is a pure refactoring with zero behavioral changes. All tests pass and code functionality remains identical.
Summary of ChangesHello @jethac, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request, the second in a series of four for the Gemini CLI's internationalization (i18n) system, focuses on integrating language selection into the user settings and optimizing the discovery and loading of locale-specific assets. It introduces a new experimental language setting, enhances the asset management by dynamically scanning for translation files, and improves UI responsiveness through caching. Additionally, it refines the structure of i18n namespaces and provides a robust solution for rendering styled text within translated content, laying further groundwork for a fully internationalized command-line interface. Highlights
Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request introduces a robust internationalization (i18n) system, adding a language toggle in settings, dynamically loading locale files, and caching language options for improved UI performance. No vulnerabilities were found in the provided code changes. The implementation is of high quality, clean, well-tested, and includes a new renderStyledText utility for styled translations, with no issues of high or critical severity identified. This is a great addition to the Gemini CLI.
b036ea3 to
c095d46
Compare
aaa5c91 to
16f48d4
Compare
16f48d4 to
5941718
Compare
5941718 to
2ac679b
Compare
|
Hi there! Thank you for your contribution to Gemini CLI. We really appreciate the time and effort you've put into this pull request. To keep our backlog manageable and ensure we're focusing on current priorities, we are closing pull requests that haven't seen maintainer activity for 30 days. Currently, the team is prioritizing work associated with 🔒 maintainer only or help wanted issues. If you believe this change is still critical, please feel free to comment with updated details. Otherwise, we encourage contributors to focus on open issues labeled as help wanted. Thank you for your understanding! |
Summary
This is the second of four pull requests to implement the i18n system for Gemini CLI. This PR adds the language toggle to the Settings dialog and optimizes how bundled locale files are discovered and loaded.
Stacked on PR #18171
What's Included
experimental.languagesetting into the schema, allowing users to switch between "Auto", "English", and "Japanese".manifest.jsonfile in each locale folder to determine display names.PR Stack
Testing
npm run build npm test -w packages/cli -- src/config/settings.test.ts src/i18n/index.test.ts src/ui/components/SettingsDialog.test.tsx