Skip to content

Make compression threshold editable in the UI.#12317

Merged
scidomino merged 1 commit intomainfrom
tomm_setting
Oct 30, 2025
Merged

Make compression threshold editable in the UI.#12317
scidomino merged 1 commit intomainfrom
tomm_setting

Conversation

@scidomino
Copy link
Copy Markdown
Collaborator

@scidomino scidomino commented Oct 30, 2025

Summary

Introduce a new setting model.compressionThreshold which can be edited with /settings.

Details

The old settings model.chatCompression.contextPercentageThreshold had two problems:

  • it wasn't a first class property so it couldn't be edited
  • it had "percentage" in the name making it sounds like a value between 0-100.

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

  • Updated relevant documentation and README (if needed)
  • Added/updated tests (if needed)
  • Noted breaking changes (if any)
  • Validated on required platforms/methods:
    • MacOS
      • npm run
      • npx
      • Docker
      • Podman
      • Seatbelt
    • Windows
      • npm run
      • npx
      • Docker
    • Linux
      • npm run
      • npx
      • Docker

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 30, 2025

Size Change: +81 B (0%)

Total Size: 20.3 MB

ℹ️ View Unchanged
Filename Size Change
./bundle/gemini.js 20.3 MB +81 B (0%)
./bundle/sandbox-macos-permissive-closed.sb 1.03 kB 0 B
./bundle/sandbox-macos-permissive-open.sb 890 B 0 B
./bundle/sandbox-macos-permissive-proxied.sb 1.31 kB 0 B
./bundle/sandbox-macos-restrictive-closed.sb 3.29 kB 0 B
./bundle/sandbox-macos-restrictive-open.sb 3.36 kB 0 B
./bundle/sandbox-macos-restrictive-proxied.sb 3.56 kB 0 B

compressed-size-action

@scidomino
Copy link
Copy Markdown
Collaborator Author

/gemini review

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@scidomino scidomino changed the title Make compression threshold editable in the UI. Make compression threshhold editable in the UI. Oct 30, 2025
@scidomino scidomino marked this pull request as ready for review October 30, 2025 22:19
@scidomino scidomino requested a review from a team as a code owner October 30, 2025 22:19
Copy link
Copy Markdown
Collaborator

@NTaylorMullen NTaylorMullen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should address each comment BUT approving to unblock

@scidomino scidomino changed the title Make compression threshhold editable in the UI. Make compression threshold editable in the UI. Oct 30, 2025
@scidomino scidomino enabled auto-merge October 30, 2025 22:46
@scidomino scidomino added this pull request to the merge queue Oct 30, 2025
@scidomino scidomino removed this pull request from the merge queue due to a manual request Oct 30, 2025
@scidomino scidomino added this pull request to the merge queue Oct 30, 2025
Merged via the queue into main with commit 3332703 Oct 30, 2025
22 checks passed
@scidomino scidomino deleted the tomm_setting branch October 30, 2025 23:14
@abhipatel12
Copy link
Copy Markdown
Contributor

/patch preview

@github-actions
Copy link
Copy Markdown

Patch workflow(s) dispatched successfully!

📋 Details:

  • Channels: preview
  • Commit: 3332703fca4c8e754a82c0818d62ea5ee386f9e2
  • Workflows Created: 1

🔗 Track Progress:

@github-actions
Copy link
Copy Markdown

🚀 Patch PR Created!

📋 Patch Details:

📝 Next Steps:

  1. Review and approve the hotfix PR: #12334
  2. Once merged, the patch release will automatically trigger
  3. You'll receive updates here when the release completes

🔗 Track Progress:

@github-actions
Copy link
Copy Markdown

🚀 Patch Release Started!

📋 Release Details:

  • Environment: prod
  • Channel: preview → publishing to npm tag preview
  • Version: v0.12.0-preview.5
  • Hotfix PR: Merged ✅
  • Release Branch: release/v0.12.0-preview.5-pr-12317

⏳ Status: The patch release is now running. You'll receive another update when it completes.

🔗 Track Progress:

@github-actions
Copy link
Copy Markdown

Patch Release Complete!

📦 Release Details:

🎉 Status: Your patch has been successfully released and published to npm!

📝 What's Available:

🔗 Links:

@mattKorwel
Copy link
Copy Markdown
Collaborator

/patch stable

@github-actions
Copy link
Copy Markdown

Patch workflow(s) dispatched successfully!

📋 Details:

  • Channels: stable
  • Commit: 3332703fca4c8e754a82c0818d62ea5ee386f9e2
  • Workflows Created: 1

🔗 Track Progress:

@mattKorwel
Copy link
Copy Markdown
Collaborator

/patch stable

@github-actions
Copy link
Copy Markdown

Patch workflow(s) dispatched successfully!

📋 Details:

  • Channels: stable
  • Commit: 3332703fca4c8e754a82c0818d62ea5ee386f9e2
  • Workflows Created: 1

🔗 Track Progress:

@mattKorwel
Copy link
Copy Markdown
Collaborator

Fixed an error where patch was failing due to git hook husky linting validation

@mattKorwel
Copy link
Copy Markdown
Collaborator

/patch stable

@github-actions
Copy link
Copy Markdown

Patch workflow(s) dispatched successfully!

📋 Details:

  • Channels: stable
  • Commit: 3332703fca4c8e754a82c0818d62ea5ee386f9e2
  • Workflows Created: 1

🔗 Track Progress:

github-actions bot pushed a commit that referenced this pull request Oct 31, 2025
# Conflicts:
#	packages/cli/src/config/config.test.ts
#	packages/core/src/services/chatCompressionService.test.ts
#	packages/core/src/services/chatCompressionService.ts
@github-actions
Copy link
Copy Markdown

🚀 Patch PR Created!

📋 Patch Details:

📝 Next Steps:

  1. ⚠️ Resolve conflicts in the hotfix PR first: #12340
  2. Test your changes after resolving conflicts
  3. Once merged, the patch release will automatically trigger
  4. You'll receive updates here when the release completes

🔗 Track Progress:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants