Coalesce consecutive tool-failure recovery audits into one count

[TheGreatAxios]

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 treats a missing count as one

Verification

• Focused tests for tool-failure intervention coalescing passed
• bun run check blocked by known pre-existing main failures

Fixes CL-7648

[linear-code]

CL-7648

[TheGreatAxios]

Review · Request changes

Unique coalesce patch arms once and writes one JSONL line when the recovery infer is issued. Do not merge until this is rebased onto current 885 (3e213d30).

Findings

• Three-dot still contains stale parent 4a695dce. Current 885 is 3e213d30 (docs + extra tests). nudge-director.test.ts conflicts on merge-tree.
• src/subagent/nudge-director.ts:405-410 — deferred flush only in applyPendingNudge after an infer is present. Cancel/destroy or compact-without-infer drops audits that used to log at arm time.
• Tests only cover fail+fail+ok → count 2. No single-failure or all-fail batch.

Notes

GitHub blocks request-changes on own PRs, so this is a comment with that verdict.

[TheGreatAxios]

Critic · Request changes

16a5570f is the unique commit. Stack still carries old 885 tip.

Findings

• Merge-base with current origin/cl-7642 is 57226a72, not 885 HEAD.
• Flush only after infer (nudge-director.ts:502-514); compact return at :414-419 skips apply.
• intervention-log.ts docs say omit count for one; writer always writes count when counter > 0.