Skip to content

fix(monitor-ci): add time-based floor to circuit breaker - #107

Open
nartc wants to merge 2 commits into
mainfrom
ct/reasonable-circuit-breaker-monitor-ci
Open

fix(monitor-ci): add time-based floor to circuit breaker#107
nartc wants to merge 2 commits into
mainfrom
ct/reasonable-circuit-breaker-monitor-ci

Conversation

@nartc

@nartc nartc commented Mar 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • circuit breaker now requires both 5+ no-progress polls AND 20+ minutes of stagnation
  • track no-progress streak start timestamp (noProgressSince) alongside count
  • prevent premature circuit breaker trips when polls are fast

🤖 Generated with Claude Code

- circuit breaker now requires both 5+ no-progress polls AND 20+ minutes of stagnation
- track no-progress streak start timestamp (noProgressSince) alongside count
- prevent premature circuit breaker trips when polls are fast

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@nartc nartc self-assigned this Mar 20, 2026
@nx-cloud-snapshot

nx-cloud-snapshot Bot commented Mar 20, 2026

Copy link
Copy Markdown
Contributor

View your CI Pipeline Execution ↗ for commit 649df9c

Command Status Duration Result
nx run-many -t lint test build typecheck e2e-ci ✅ Succeeded 3s View ↗
nx-cloud record -- npx nx format:check --base=r... ✅ Succeeded 1s View ↗
nx sync-artifacts-verification ✅ Succeeded 12s View ↗
nx validate-cursor-plugin ✅ Succeeded 2m 10s View ↗
nx-cloud record -- claude plugin validate .clau... ✅ Succeeded <1s View ↗
nx-cloud record -- claude plugin validate .clau... ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2026-03-20 20:04:40 UTC

@nx-cloud-snapshot nx-cloud-snapshot Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nx Cloud is proposing a fix for your failed CI:

We synced the generated artifacts to fix the sync-artifacts-verification failure caused by the PR's circuit breaker changes not being propagated to all output files. Running npx nx sync-artifacts regenerated the 10 out-of-sync files across generated/.agents/, generated/.gemini/, generated/.github/, and generated/.opencode/, bringing them in line with the updated source.

Tip

We verified this fix by re-running @org/source:sync-artifacts-verification.

Warning

The suggested diff is too large to display here, but you can view it on Nx Cloud ↗


🔔 Heads up, your workspace has pending recommendations ↗ to auto-apply fixes for similar failures.

Apply fix via Nx Cloud  Reject fix via Nx Cloud


Or Apply changes locally with:

npx nx-cloud apply-locally KGRE-mkqK

Apply fix locally with your editor ↗   View interactive diff ↗



🎓 Learn more about Self-Healing CI on nx.dev

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