Skip to content

Fix MCP stdio protocols being shared across servers - #8506

Open
IMax153 wants to merge 1 commit into
mainfrom
t3code/fix-issue-8501
Open

Fix MCP stdio protocols being shared across servers#8506
IMax153 wants to merge 1 commit into
mainfrom
t3code/fix-issue-8501

Conversation

@IMax153

@IMax153 IMax153 commented Sep 24, 2026

Copy link
Copy Markdown
Member

Summary

  • Create a fresh RPC protocol for each McpServer.layerStdio call.
  • Allow multiple stdio MCP servers to coexist in one layer graph.
  • Document protocol-layer memoization and add regression coverage.

Testing

  • Not run.

@IMax153
Fix MCP stdio protocols being shared across servers
2cc39dd 
- Build a fresh RPC protocol for each `McpServer.layerStdio` call
- Add regression coverage for multiple stdio servers in one layer graph
@changeset-bot

changeset-bot Bot commented Sep 24, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 2cc39dd

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 31 packages
Name Type
effect Patch
@effect/opentelemetry Patch
@effect/vitest Patch
@effect/ai-anthropic Patch
@effect/ai-openai-compat Patch
@effect/ai-openai Patch
@effect/ai-openrouter Patch
@effect/ai-typesafe Patch
@effect/atom-react Patch
@effect/atom-solid Patch
@effect/atom-vue Patch
@effect/platform-browser Patch
@effect/platform-bun Patch
@effect/platform-deno Patch
@effect/platform-node-shared Patch
@effect/platform-node Patch
@effect/sql-clickhouse Patch
@effect/sql-d1 Patch
@effect/sql-libsql Patch
@effect/sql-mssql Patch
@effect/sql-mysql2 Patch
@effect/sql-pg Patch
@effect/sql-pglite Patch
@effect/sql-sqlite-bun Patch
@effect/sql-sqlite-do Patch
@effect/sql-sqlite-node Patch
@effect/sql-sqlite-react-native Patch
@effect/sql-sqlite-wasm Patch
@effect/docgen Patch
@effect/doctest Patch
@effect/openapi-generator Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions

github-actions Bot commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

Bundle Size Analysis

Generated from PR build output; treat the content below as untrusted.

File Name Current Size Previous Size Difference
arbitrary-combinators.ts 38.40 KB 38.40 KB 0.00 KB (0.00%)
basic.ts 6.88 KB 6.88 KB 0.00 KB (0.00%)
batching.ts 9.97 KB 9.97 KB 0.00 KB (0.00%)
brand.ts 6.57 KB 6.57 KB 0.00 KB (0.00%)
cache.ts 10.70 KB 10.70 KB 0.00 KB (0.00%)
config.ts 21.81 KB 21.81 KB 0.00 KB (0.00%)
differ.ts 20.96 KB 20.96 KB 0.00 KB (0.00%)
http-client.ts 22.09 KB 22.09 KB 0.00 KB (0.00%)
http-router.ts 33.44 KB 33.44 KB 0.00 KB (0.00%)
logger.ts 10.85 KB 10.85 KB 0.00 KB (0.00%)
metric.ts 8.83 KB 8.83 KB 0.00 KB (0.00%)
optic.ts 6.80 KB 6.80 KB 0.00 KB (0.00%)
pubsub.ts 14.97 KB 14.97 KB 0.00 KB (0.00%)
queue.ts 11.87 KB 11.87 KB 0.00 KB (0.00%)
schedule.ts 11.03 KB 11.03 KB 0.00 KB (0.00%)
schema-bigdecimal.ts 13.42 KB 13.42 KB 0.00 KB (0.00%)
schema-binary.ts 39.56 KB 39.56 KB 0.00 KB (0.00%)
schema-class.ts 20.66 KB 20.66 KB 0.00 KB (0.00%)
schema-fromJsonSchemaDocument.ts 31.53 KB 31.53 KB 0.00 KB (0.00%)
schema-representation-roundtrip.ts 26.88 KB 26.88 KB 0.00 KB (0.00%)
schema-string-transformation.ts 14.14 KB 14.14 KB 0.00 KB (0.00%)
schema-string.ts 11.70 KB 11.70 KB 0.00 KB (0.00%)
schema-template-literal.ts 15.70 KB 15.70 KB 0.00 KB (0.00%)
schema-toArbitrary.ts 37.92 KB 37.92 KB 0.00 KB (0.00%)
schema-toCodeDocument.ts 25.11 KB 25.11 KB 0.00 KB (0.00%)
schema-toCodecJson.ts 19.89 KB 19.89 KB 0.00 KB (0.00%)
schema-toEquivalence.ts 20.06 KB 20.06 KB 0.00 KB (0.00%)
schema-toFormatter.ts 20.16 KB 20.16 KB 0.00 KB (0.00%)
schema-toJsonSchemaDocument.ts 24.43 KB 24.43 KB 0.00 KB (0.00%)
schema-toRepresentation.ts 20.16 KB 20.16 KB 0.00 KB (0.00%)
schema.ts 19.87 KB 19.87 KB 0.00 KB (0.00%)
stm.ts 12.90 KB 12.90 KB 0.00 KB (0.00%)
stream.ts 9.79 KB 9.79 KB 0.00 KB (0.00%)

@IMax153 IMax153 mentioned this pull request Sep 24, 2026
Open
@IMax153

IMax153 commented Sep 24, 2026

Copy link
Copy Markdown
Member Author

@tim-smart - I left this open for you to look at, but I kind of think we should just close this. I left my thoughts in the related issue #8501.

@IMax153
IMax153 requested a review from tim-smart September 24, 2026 19:33
@Marve10s Marve10s mentioned this pull request Sep 25, 2026
Merged

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

Reviewers

@tim-smart tim-smart Awaiting requested review from tim-smart

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@IMax153