Skip to content

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

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

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

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 Claude automatic compaction default from 350k to 829.8k.
    • Applied the updated label across all supported languages, including English, German, French, Japanese, Korean, Russian, Turkish, Simplified Chinese, and Traditional Chinese.
    • Clarified the default value in the Claude Code settings.

@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: 2a9d9a67-ac99-4f58-8bf0-8323e9bf70d3

📥 Commits

Reviewing files that changed from the base of the PR and between ddd9ee8 and 4e79c51.

📒 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; 0 remain after this review.


📝 Walkthrough

Walkthrough

The change updates the documented and localized Claude auto-compaction default from 350k to 829.8k. Runtime option behavior remains unchanged.

Changes

Claude auto-compaction display

Layer / File(s) Summary
Update documented and localized default labels
gui/src/pages/ClaudeCode.tsx:107, gui/src/i18n/*.ts
The page comment and German, English, French, Japanese, Korean, Russian, Turkish, Traditional Chinese, and Chinese labels now display 829.8k instead of 350k. Locale-specific default markers remain unchanged.

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

Merge Risk: ⚪ Minimal · up to 4e79c

This localized copy-only change aligns the displayed auto-compact default with the existing runtime value and does not alter behavior; no actionable merge-blocking risk remains after normal checks and review.

Suggested reviewers: lidge-jun

<fixed_issue_severity>Low</fixed_issue_severity>

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
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 the Claude auto-compact default label and matches the main changeset.
✨ Finishing Touches 💡 2
📝 Generate docstrings 💡
  • Create stacked PR
  • Commit on current branch
🛠️ Fix failing CI checks 💡
  • 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-6z730s

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

@github-actions github-actions Bot added bug Something isn't working intake: hygiene-blocked Deterministic PR hygiene checks failed labels 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 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 07:39
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