Skip to content

fix: prefer compatible session MCP servers - #534

Open
emilk wants to merge 1 commit into
agentclientprotocol:mainfrom
emilk:fix/session-mcp-precedence
Open

emilk wants to merge 1 commit into
agentclientprotocol:mainfrom
emilk:fix/session-mcp-precedence

Conversation

@emilk

@emilk emilk commented Sep 22, 2026

Copy link
Copy Markdown

Summary

  • let ACP session MCP definitions override same-named configured servers when both use the same transport
  • continue filtering incompatible HTTP/stdio conflicts to avoid invalid deep-merged definitions
  • preserve unrelated configured MCP servers

Testing

  • npm test — 742 passed, 26 skipped
  • npm run typecheck
  • npm run build

Disclosure

This change was implemented with assistance from an LLM (OpenAI Codex).

@emilk
emilk marked this pull request as ready for review September 22, 2026 13:28

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.

1 participant