Skip to content

feat(run): add unified seat health probe - #590

Merged
solomonneas merged 3 commits into
mainfrom
feat/577-unified-seat-health-probe
Jul 27, 2026
Merged

feat(run): add unified seat health probe#590
solomonneas merged 3 commits into
mainfrom
feat/577-unified-seat-health-probe

Conversation

@solomonneas

Copy link
Copy Markdown
Collaborator

Closes #577

Implements the shared seat-health coordinator from #474 and the accepted proposal in docs/proposals/2026-07-26-seat-health-probe-and-worker-failure-taxonomy.md.

  • probes requested seats and declared fallbacks in parallel
  • covers declaration, executable, auth, transport, version, model, and isolation checks through an explicit adapter matrix
  • enforces 30-second per-seat, 15-second model, and 35-second roster budgets
  • caches by redacted seat fingerprint and always recomputes isolation compatibility
  • writes atomic brigade.seat_health.v1 receipts with dual-clock timestamp handling
  • adds the shared result to roster doctor without removing existing labels

Verification: ./scripts/verify passed through brigade work verify run, receipt 20260727-000622-work-verify-600d75 (4,506 passed, 3 skipped).

Co-Authored-By: Codex <codex@openai.com>
@solomonneas solomonneas self-assigned this Jul 27, 2026
@coderabbitai

coderabbitai Bot commented Jul 27, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (1)
  • coderabbit-review

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository: escoffier-labs/coderabbit/.coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 1effaf0a-5b79-4543-a01a-a6107a9b82b1

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/577-unified-seat-health-probe

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

solomonneas and others added 2 commits July 26, 2026 20:17
Co-Authored-By: Codex <codex@openai.com>
Co-Authored-By: Codex <codex@openai.com>
@solomonneas
solomonneas marked this pull request as ready for review July 27, 2026 20:52
@solomonneas
solomonneas merged commit b966152 into main Jul 27, 2026
28 checks passed
@solomonneas
solomonneas deleted the feat/577-unified-seat-health-probe branch July 27, 2026 20:52
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.

run: implement the adapter-owned unified seat health probe

1 participant