Skip to content

fix(chat): let the compaction error above the composer be dismissed (#784) - #1109

Open
agegr wants to merge 1 commit into
mainfrom
fix/compact-error-dismiss
Open

agegr wants to merge 1 commit into
mainfrom
fix/compact-error-dismiss

Conversation

@agegr

@agegr agegr commented Oct 8, 2026

Copy link
Copy Markdown
Owner

A failed or refused compaction (for example "Nothing to compact (session too small)") left its red banner above the input until the next compaction started, with no way to close it (point 3 of #784, see its screenshot).

The banner now carries the same close button as the other inline errors (DismissButton). The hook exposes dismissCompactError; ChatInput shows the button only when it is given that handler.

  • hooks/useAgentSession.ts, components/ChatWindow.tsx, components/ChatInput.tsx
  • i18n: chat.dismissCompactError (en, zh-CN, zh-TW)
  • test: components/ChatInput.test.mjs

Checks: tsc --noEmit, eslint on the changed files, npm test (2678 pass).

Refs #784

🤖 Generated with Claude Code

…784)

A failed or refused compaction ("Nothing to compact (session too small)")
left its red banner above the input until the next compaction started, with
no way to close it. The banner now has the same close button as other inline
errors.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@agegr agegr mentioned this pull request Oct 8, 2026

This branch has not been deployed

No deployments
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.

1 participant