Skip to content

Exempt pending poll batches from doom-loop guard - #895

Merged
TheGreatAxios merged 1 commit into
mainfrom
cl-7624-parent-and-workers-still-run-the-reactor-doom-loop-guard-at
Sep 11, 2026
Merged

Exempt pending poll batches from doom-loop guard#895
TheGreatAxios merged 1 commit into
mainfrom
cl-7624-parent-and-workers-still-run-the-reactor-doom-loop-guard-at

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Collaborator

Summary

  • Pending-only wait_agents and shell_collect batches reset the doom-loop streak while targets stay live
  • Non-poll, mixed, and settled batches still count toward the guard
  • Poll tool copy names the exemption so a timeout reads as liveness, not a crash

Verification

  • bun test src/subagent/poll-exempt.test.ts vendor/intx-inference/src/reactor.test.ts tests/unit/vendor-patch-ledger.test.ts src/subagent/agent-fleet.test.ts src/agent/background-shell-tool.test.ts passes (276 pass, 0 fail)
  • bun run lint passes with 0 warnings and 0 errors
  • bun run build passes
  • bun run typecheck passes except the pre-existing untracked src/tui/upgrade-notes.test.ts error, untouched by this branch
  • bin/vendor-patch-diff maps to the new ledger entry and the ledger test passes

Fixes CL-7624

@linear-code

linear-code Bot commented Sep 11, 2026

Copy link
Copy Markdown

CL-7624

Legitimate waits repeat identical polls while work is still in flight. Reset the streak only when every polled target is still pending. Mixed and settled batches still count toward the guard.
@TheGreatAxios
TheGreatAxios force-pushed the cl-7624-parent-and-workers-still-run-the-reactor-doom-loop-guard-at branch from 5edc182 to c951619 Compare September 11, 2026 03:11
@TheGreatAxios
TheGreatAxios merged commit 586a21f into main Sep 11, 2026
9 checks passed
@TheGreatAxios
TheGreatAxios deleted the cl-7624-parent-and-workers-still-run-the-reactor-doom-loop-guard-at branch September 11, 2026 03:19
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