Skip to content

fix: bound client suite concurrency - #509

Open
DorianChn wants to merge 1 commit into
modelcontextprotocol:mainfrom
DorianChn:codex/bound-client-suite-concurrency
Open

DorianChn wants to merge 1 commit into
modelcontextprotocol:mainfrom
DorianChn:codex/bound-client-suite-concurrency

Conversation

@DorianChn

Copy link
Copy Markdown

Summary

  • bound client-suite execution concurrency for timing-sensitive SSE checks
  • keep the existing scenario assertions and lifecycle behavior intact

Why

Issue #500 identified that the client suite could create timing interference when too many checks ran concurrently. A bounded runner makes the test behavior more stable without changing production code.

Validation

  • pre-push formatting check passed
  • pre-push test passed: 48 files, 629 tests
  • npm run typecheck
  • npm run build
  • npm run lint
  • git diff --check

Fixes #500

This branch has not been deployed

No deployments
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.

Bound client-suite concurrency for timing-sensitive SSE checks

1 participant