Skip to content

Fix circuit breaker and request pool correctness - #261

Merged
Ferossgp merged 1 commit into
mainfrom
fix/circuit-breaker-request-pool-correctness
Aug 31, 2026
Merged

Fix circuit breaker and request pool correctness#261
Ferossgp merged 1 commit into
mainfrom
fix/circuit-breaker-request-pool-correctness

Conversation

@Ferossgp

@Ferossgp Ferossgp commented Aug 31, 2026

Copy link
Copy Markdown
Member

Follow-up to #233.

Summary

  • record successful calls in tripping strategies so failure counts reset and failure-rate windows include successes
  • make circuit admission, half-open probe reservation, and outcome transitions atomic with Ref.modify
  • replace the request pool's clamped counters with an atomic, bounded 100-outcome window
  • correct the ABI loader concurrency-cap comment

Tests

  • pnpm exec vitest run test/circuit-breaker-with-metrics.test.ts test/request-pool.test.ts
  • pnpm run check
  • pnpm run lint (passes with 6 pre-existing warnings)
  • pnpm run test

@vercel

vercel Bot commented Aug 31, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
loop-decoder-web Ready Ready Preview Aug 31, 2026 5:10pm

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying loop-decoder with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9a2fc11
Status: ✅  Deploy successful!
Preview URL: https://c47bb013.loop-decoder.pages.dev
Branch Preview URL: https://fix-circuit-breaker-request.loop-decoder.pages.dev

View logs

@Ferossgp
Ferossgp merged commit af1b146 into main Aug 31, 2026
7 checks passed
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