Skip to content

Coalesce consecutive tool-failure recovery audits into one count - #893

Closed
TheGreatAxios wants to merge 1 commit into
mainfrom
cl-7648-coalesce-repeated-tool-failure-recovery-intervention-records
Closed

Coalesce consecutive tool-failure recovery audits into one count#893
TheGreatAxios wants to merge 1 commit into
mainfrom
cl-7648-coalesce-repeated-tool-failure-recovery-intervention-records

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Collaborator

Summary

  • Coalesces consecutive tool-failure recovery audits into a single director-memory record with a count when the pending recovery nudge is applied
  • Keeps recovery nudge text and arming behavior the same; forensics treat a missing count as one

Verification

  • Focused director tests for coalescing pass
  • bun run typecheck and bun run build pass
  • Full bun run check suite has four pre-existing failures on main (optimized-context-store torn-tail/heal, assembleChatAgent /build-dir EROFS)

Fixes CL-7648

The audit sink is append-only, so one failed-tool recovery per result
looked like distinct recoveries. Hold a burst count until the single
pending nudge is applied. Forensics treat a missing count as one.
@linear-code

linear-code Bot commented Sep 11, 2026

Copy link
Copy Markdown

CL-7648

@TheGreatAxios

Copy link
Copy Markdown
Collaborator Author

Superseded by the stacked #886 (same CL-7648 change, kept in the 885/886/887 stack). #886 additionally flushes coalesced audits on terminal paths and carries the matching test coverage.

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