Skip to content

fix(compaction): notify and stop when compaction fails - #1446

Merged
TheGreatAxios merged 2 commits into
mainfrom
fix/compaction-failure-notifies
Oct 15, 2026
Merged

TheGreatAxios merged 2 commits into
mainfrom
fix/compaction-failure-notifies

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Collaborator

Summary

  • A verify abort or summarizer keep-prior keeps the prior turns and notifies Compaction failed — keeping prior context, including an AbortError with no lifecycle signal ((verify), (auth, verify), (quota, verify), or (failed)).
  • A failed /handoff clears the post-compact infer so the session does not start another inference. A signaled lifecycle abort still reports only Compaction interrupted — keeping prior context.
  • Quota failures classify from quota_exhausted or HTTP 429. Statistics stub folds are unchanged.

Verification

  • Pre-commit tsc --noEmit and bun build pass
  • bun test src/agent/compaction.test.ts src/agent/director-compaction.test.ts src/session/runtime-assembly.test.ts — 113 pass, 0 fail

A verify abort or summarizer keep-prior, including an AbortError with no
lifecycle signal, keeps the prior turns and reports the failure instead of
throwing or starting another inference after /handoff.
@corbits-triage

corbits-triage Bot commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Triage: Fail

@TheGreatAxios, please address the following:

  • pull request does not reference an issue

@TheGreatAxios
TheGreatAxios merged commit 3d130f4 into main Oct 15, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant