Skip to content

claude-code-hermit: less aggressive compact defaults#130

Merged
gtapps merged 3 commits into
mainfrom
worktree-fix-remove-max-thinking-tokens
May 23, 2026
Merged

claude-code-hermit: less aggressive compact defaults#130
gtapps merged 3 commits into
mainfrom
worktree-fix-remove-max-thinking-tokens

Conversation

@gtapps
Copy link
Copy Markdown
Owner

@gtapps gtapps commented May 23, 2026

Summary

  • claude-code-hermit: less aggressive compact defaults (CLAUDE_AUTOCOMPACT_PCT_OVERRIDE 50→65, COMPACT_THRESHOLD 50→75)

Default auto-compact was firing at 50% context — well before the quality-degradation zone (~73%) — discarding usable context without benefit. Bumps the trigger to 65%, a conservative move that stays well under the upstream ~83% clamp ceiling (anthropics/claude-code#31806).

COMPACT_THRESHOLD was firing the tool-call-based /compact nudge mid-session for any non-trivial work. Bumps to 75; the real signal (context_usage > 60% in suggest-compact.js) continues to drive substantive nudges.

CHANGELOG [Unreleased] adds executable Upgrade Instructions for hermit-evolve with an only-if-still-50 guard so operators who tuned to a custom value keep their choice.

Verification

  • Tests: passbash plugins/claude-code-hermit/tests/run-all.sh (84/84 contract tests + full hooks/sessions/auto-close suites) ran green during implementation. state/last-test.json not recorded in this worktree (no hermit config), so no cached HEAD-SHA pass available.
  • Grep cleanup: zero stale 50 references survive in plugins/ outside the intentional guard conditions inside the new CHANGELOG migration text.

Test plan

  • On a sandbox project with env.CLAUDE_AUTOCOMPACT_PCT_OVERRIDE == "50" and env.COMPACT_THRESHOLD == "50": run /claude-code-hermit:hermit-evolve and confirm both bump to 65/75 in config.json and settings.local.json.
  • On a project with a custom value (e.g. CLAUDE_AUTOCOMPACT_PCT_OVERRIDE: "75"): run /claude-code-hermit:hermit-evolve and confirm the value is left untouched with a "custom override preserved" message.

gtapps added 2 commits May 23, 2026 20:46
…ACT_PCT_OVERRIDE 50→65, COMPACT_THRESHOLD 50→75)

Default auto-compact was firing at 50% context — well before the
quality-degradation zone (~73%) — discarding usable context without
benefit. Bumps the trigger to 65%, conservative move that stays well
under the upstream ~83% clamp ceiling.

COMPACT_THRESHOLD was firing the tool-call-based /compact nudge mid-
session for any non-trivial work. Bumps to 75; the real signal
(context_usage > 60% in suggest-compact.js) continues to drive
substantive nudges.

CHANGELOG [Unreleased] adds executable Upgrade Instructions for
hermit-evolve with an only-if-still-50 guard so operators who tuned
to a custom value keep their choice.
…ighten architecture.md cell

PR review pointed out the `config-reference.md` description correction from
the previous commit was unmentioned in CHANGELOG, and architecture.md still
called the threshold a generic "Compaction suggestion threshold." Adds a
third `### Changed` bullet flagging the description correction, and tightens
the architecture.md env-defaults table cell to "Tool-call-count fallback for
compact suggestion" — matching the now-corrected config-reference entry.
@gtapps
Copy link
Copy Markdown
Owner Author

gtapps commented May 23, 2026

@copilot resolve the merge conflicts in this pull request

@gtapps gtapps merged commit 907083a into main May 23, 2026
1 check passed
Copy link
Copy Markdown
Contributor

Copilot AI commented May 23, 2026

@copilot resolve the merge conflicts in this pull request

Resolved by merging origin/main into this branch and pushing merge commit 86bbd56.

Copilot stopped work on behalf of gtapps due to an error May 23, 2026 20:10
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.

2 participants