Make compression threshold editable in the UI.#12317
Conversation
|
Size Change: +81 B (0%) Total Size: 20.3 MB ℹ️ View Unchanged
|
|
/gemini review |
There was a problem hiding this comment.
Code Review
This pull request successfully introduces a new, editable setting model.compressionThreshold to control chat history compression, replacing the old, less flexible model.chatCompression.contextPercentageThreshold. The changes are well-reflected in the documentation and tests. However, I've identified a couple of issues: a consistent typo in the new setting's name throughout the code, and a lack of validation for the new setting's value, which could lead to unexpected behavior. Addressing these will improve the code's correctness and maintainability.
821d5b5 to
ac39bd4
Compare
NTaylorMullen
left a comment
There was a problem hiding this comment.
Should address each comment BUT approving to unblock
ac39bd4 to
0109b4d
Compare
0109b4d to
4e05db8
Compare
|
/patch preview |
|
✅ Patch workflow(s) dispatched successfully! 📋 Details:
🔗 Track Progress: |
|
🚀 Patch PR Created! 📋 Patch Details:
📝 Next Steps:
🔗 Track Progress: |
|
🚀 Patch Release Started! 📋 Release Details:
⏳ Status: The patch release is now running. You'll receive another update when it completes. 🔗 Track Progress: |
|
✅ Patch Release Complete! 📦 Release Details:
🎉 Status: Your patch has been successfully released and published to npm! 📝 What's Available:
🔗 Links: |
|
/patch stable |
|
✅ Patch workflow(s) dispatched successfully! 📋 Details:
🔗 Track Progress: |
|
/patch stable |
|
✅ Patch workflow(s) dispatched successfully! 📋 Details:
🔗 Track Progress: |
|
Fixed an error where patch was failing due to git hook husky linting validation |
|
/patch stable |
|
✅ Patch workflow(s) dispatched successfully! 📋 Details:
🔗 Track Progress: |
# Conflicts: # packages/cli/src/config/config.test.ts # packages/core/src/services/chatCompressionService.test.ts # packages/core/src/services/chatCompressionService.ts
|
🚀 Patch PR Created! 📋 Patch Details:
📝 Next Steps:
🔗 Track Progress: |
Summary
Introduce a new setting
model.compressionThresholdwhich can be edited with/settings.Details
The old settings
model.chatCompression.contextPercentageThresholdhad two problems:Rather than fixing it I thought it would be easier to just create a new one especially since it is unlikely that we will have many other compression properties.
Related Issues
For https://github.com/google-gemini/maintainers-gemini-cli/issues/973
How to Validate
Change your settings!
Pre-Merge Checklist