Skip to content

fix(claude): finish compaction in ready state - #161

Open
Yoonwoo-Ha wants to merge 1 commit into
devswha:mainfrom
Yoonwoo-Ha:fix/claude-compaction-ready
Open

Yoonwoo-Ha wants to merge 1 commit into
devswha:mainfrom
Yoonwoo-Ha:fix/claude-compaction-ready

Conversation

@Yoonwoo-Ha

Copy link
Copy Markdown
Contributor

Problem

Claude writes /compact completion as a synthetic user summary followed by local-command wrapper records. ChatMux treated the wrapper output as a new user prompt, so an idle compacted session stayed RUN, and rendered the full generated summary directly in the conversation.

Changes

  • classify Claude compact-summary records as waiting_user without generating a reply-completion notification
  • ignore Claude's internal local-command wrapper records during activity detection
  • render compact summaries as a collapsed, localized disclosure
  • defer mounting the summary Markdown until the disclosure is opened
  • cover the activity sequence and collapsed/expanded rendering with regression tests

Verification

  • replayed the observed Claude transcript sequence: waiting_user, terminal outcome none
  • npm run verify
    • server tests: 1,842 passed
    • tmux/PTY tests: 50 passed
    • client tests: 731 passed
    • typecheck, lint, identity, Rust checks, and production build passed

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