feat(context): add remote configuration for tool output masking thresholds#18553
feat(context): add remote configuration for tool output masking thresholds#18553abhipatel12 merged 1 commit intomainfrom
Conversation
Summary of ChangesHello @abhipatel12, 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 transitions the tool output masking thresholds from hardcoded values to a remotely configurable system using the Experiments framework. This enhancement allows for dynamic optimization of the context window without requiring client-side updates, improving flexibility and responsiveness to changing model requirements. 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
|
|
Size Change: +1.19 kB (0%) Total Size: 23.9 MB
ℹ️ View Unchanged
|
There was a problem hiding this comment.
Code Review
This pull request transitions the tool output masking thresholds to a remote configuration system, a positive step towards dynamic optimization. However, a high-severity security vulnerability has been identified in the offloading mechanism (writing masked outputs to disk) due to insecure temporary file handling with default permissions and predictable filenames, which could expose sensitive tool outputs and enable symlink attacks. Additionally, a critical issue in config.ts involves non-robust parsing of remote configuration values, potentially leading to NaN values and unpredictable behavior.
854fb63 to
252e86c
Compare
Summary
Transitioned previously hardcoded tool output masking thresholds to a remotely configurable system using the Experiments framework. This allows for dynamic context window optimization without requiring client updates.
Details
45758817,45758818, and45758819toflagNames.ts.Config.tsto fetch these values asynchronously from the experiments service.ToolOutputMaskingServiceto await the resolved configuration before processing.Related Issues
Related to tool output masking and context optimization.
How to Validate
npm test -w @google/gemini-cli-core -- src/services/toolOutputMaskingService.test.tsshould respect remote configuration overridescorrectly simulates and validates dynamic threshold changes.npm run preflightto confirm overall workspace health.Pre-Merge Checklist