Skip to content

fix(controller): per-recipe crash-loop budget#160

Open
OnlyTerp wants to merge 1 commit into
sybil-solutions:mainfrom
OnlyTerp:fix/crash-loop-budget-153
Open

fix(controller): per-recipe crash-loop budget#160
OnlyTerp wants to merge 1 commit into
sybil-solutions:mainfrom
OnlyTerp:fix/crash-loop-budget-153

Conversation

@OnlyTerp

@OnlyTerp OnlyTerp commented Jun 22, 2026

Copy link
Copy Markdown

Closes #153

Implements a per-recipe crash-loop budget so a repeatedly failing launch cannot spin indefinitely.

  • Tracks consecutive launch failures per recipe in EngineCoordinator.
  • Quarantines a recipe after 3 failures within 60 seconds.
  • Resets the budget on a successful launch or when a recipe is created/updated/deleted.
  • Blocks setActiveRecipe and ensureActive launches while quarantined.
  • Allows evict/cancel regardless of quarantine state.

Verification:

  • cd controller && bun run typecheck: passed
  • cd controller && bun run lint: passed
  • New integration tests: cd controller && bun test ../tests/controller/integration/engine-coordinator-crash-loop.test.ts: passed

…#153)

- Track consecutive launch failures per recipe in EngineCoordinator.
- Quarantine a recipe after 3 failures within 60 s.
- Reset the budget on successful launch or recipe create/update/delete.
- Block setActiveRecipe/ensureActive launches while quarantined.
- Allow evict/cancel regardless of quarantine state.
@OnlyTerp OnlyTerp requested a review from 0xSero as a code owner June 22, 2026 02:42
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.

Add per-recipe crash-loop launch budget

1 participant