Skip to content

Make compression threshold editable in the UI.#12317

Merged
scidomino merged 1 commit into
mainfrom
tomm_setting
Oct 30, 2025
Merged

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

Conversation

@scidomino

@scidomino scidomino commented Oct 30, 2025

Copy link
Copy Markdown
Collaborator

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

github-actions Bot commented Oct 30, 2025

Copy link
Copy Markdown

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

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

Copy link
Copy Markdown
Contributor

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.

Comment thread packages/cli/src/config/settingsSchema.ts Outdated
Comment thread packages/cli/src/config/settingsSchema.ts Outdated
@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

@NTaylorMullen NTaylorMullen left a comment

Copy link
Copy Markdown
Collaborator

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

Comment thread docs/get-started/configuration.md Outdated
Comment thread packages/cli/src/config/settings.ts
@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
Comment thread packages/core/src/config/config.ts
@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

size/m A medium sized PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants