Skip to content

[WRONG BRANCH] gui: update Claude auto-compact default label to 829.8k - #314

Draft
luvs01 wants to merge 1 commit into
mainfrom
codex/fix-gui-default-label-for-auto-compact-0kzu7s
Draft

[WRONG BRANCH] gui: update Claude auto-compact default label to 829.8k#314
luvs01 wants to merge 1 commit into
mainfrom
codex/fix-gui-default-label-for-auto-compact-0kzu7s

Conversation

@luvs01

@luvs01 luvs01 commented Aug 17, 2026

Copy link
Copy Markdown
Owner

Motivation

  • The runtime default auto-compact window was raised to 829,800 tokens but the GUI still showed the empty/default select option as "350k (default)", which misleads users about the effective threshold.
  • Keep dashboard copy accurate and consistent with the runtime and manual-environment export to avoid user confusion and unexpected compaction behavior.

Description

  • Replace the stale "350k (default)" label with "829.8k (default)" in the English locale and the corresponding translated locale files (gui/src/i18n/{en,de,fr,ja,ko,ru,tr,zh,zh-TW}.ts).
  • Update the inline comment in gui/src/pages/ClaudeCode.tsx to note the effective 829.8k default shown by the empty select option.
  • This change updates visible copy only and preserves existing runtime behavior and values.

Testing

  • Ran the GUI unit tests with the repository-pinned Bun (../node_modules/.bin/bun test tests) and observed all tests pass (931 pass, 0 fail).
  • Ran bun run lint, bun run lint:i18n, and bun run build from gui/ and all checks completed successfully.
  • Validation used the repo-pinned Bun executable when the globally installed Bun was incompatible, and the focused GUI checks required by the repository i18n guidelines passed.

Codex Task

Summary by CodeRabbit

  • Updates
    • Updated the displayed default automatic compaction window for Claude from 350k to 829.8k.
    • Updated the value consistently across all supported languages.

@github-actions github-actions Bot added the intake: hygiene-blocked Deterministic PR hygiene checks failed label Aug 17, 2026
@github-actions

Copy link
Copy Markdown

⚠️ Deterministic hygiene checks failed.

  • missing_regression_test — Behavior changed under src/ or gui/src/ without a test change. Add focused coverage or obtain test-exception-approved.

@github-actions github-actions Bot added the bug Something isn't working label Aug 17, 2026
@github-actions github-actions Bot changed the title gui: update Claude auto-compact default label to 829.8k [WRONG BRANCH] gui: update Claude auto-compact default label to 829.8k Aug 17, 2026
@github-actions

github-actions Bot commented Aug 17, 2026

Copy link
Copy Markdown

⏳ DRAFT

  • wrong target branch (main); retarget to dev. UI screenshot required. hygiene: missing_regression_test.

What to do

  • Retarget this PR to dev — all contributions go to dev.
  • Add a screenshot of the UI change to the PR description.
  • Fix missing_regression_test — Behavior changed under src/ or gui/src/ without a test change. Add focused coverage or obtain test-exception-approved.

Its title has been prefixed with [WRONG BRANCH].
This pull request was already a draft. Its draft status will be preserved after every issue above is resolved.

@github-actions
github-actions Bot marked this pull request as draft August 17, 2026 10:22
@coderabbitai

coderabbitai Bot commented Aug 17, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 8a36c62a-ab16-4091-93f8-0b4bda2c4879

📥 Commits

Reviewing files that changed from the base of the PR and between 9dcecfe and c94d4ff.

📒 Files selected for processing (10)
  • gui/src/i18n/de.ts
  • gui/src/i18n/en.ts
  • gui/src/i18n/fr.ts
  • gui/src/i18n/ja.ts
  • gui/src/i18n/ko.ts
  • gui/src/i18n/ru.ts
  • gui/src/i18n/tr.ts
  • gui/src/i18n/zh-TW.ts
  • gui/src/i18n/zh.ts
  • gui/src/pages/ClaudeCode.tsx

Included review availability: Your plan includes up to 3 reviews per rolling hour; 1 remains after this review.


📝 Walkthrough

Walkthrough

The change updates the Claude auto-compaction default from 350k to 829.8k in preset documentation and nine localized labels. The option-generation logic remains unchanged.

Changes

Claude auto-compaction default

Layer / File(s) Summary
Update default labels and documentation
gui/src/pages/ClaudeCode.tsx, gui/src/i18n/*.ts
The preset comment at line 107 and the claude.autoCompactDefault labels at the listed locale lines now show 829.8k instead of 350k.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Merge Risk: ⚪ Minimal · up to c94d4

This change only corrects the displayed auto-compact default from 350k to 829.8k without changing runtime behavior. No actionable merge-blocking risk remains after normal checks and review.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the GUI change to update the Claude auto-compact default label to 829.8k, which matches the main changeset.
✨ Finishing Touches 💡 2
⚔️ Resolve merge conflicts 💡
  • Resolve merge conflict in branch codex/fix-gui-default-label-for-auto-compact-0kzu7s
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/fix-gui-default-label-for-auto-compact-0kzu7s

Comment @coderabbitai help to get the list of available commands.

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

Labels

aardvark bug Something isn't working codex intake: hygiene-blocked Deterministic PR hygiene checks failed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant