Repository navigation
plugins: every base takes a stdio server; pi's adapter and dsh's client spawn it themselves - #180
Merged
Merged
Conversation
…nt spawn it themselves The gateway refused a package with a stdio MCP server on pi and dsh (422 unsupported_transport) on the premise that their clients take URLs only. Neither does: pi-mcp-adapter runs a `command` with `args` as a stdio transport, and dsh-mcp-client discriminates its config on `transport` with a `stdio` branch (StdioClientTransport on command, args, env, cwd). The runner now hands both the launcher it already writes for every other backend, and the per-base refusal is gone: one launcher, every client runs it. Verified on the hosted deployment across the bases. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
richard-epsilla
approved these changes
Sep 15, 2026
richard-epsilla
left a comment
richard-epsilla
left a comment
Collaborator
There was a problem hiding this comment.
Both clients spawn the launcher themselves; gateway and runner tests green in CI.
richard-epsilla
added a commit
to ZixiaoL/harnessrouter
that referenced
this pull request
Oct 3, 2026
…nt spawn it themselves (HarnessRouter#180) * plugins: every base takes a stdio server; pi's adapter and dsh's client spawn it themselves The gateway refused a package with a stdio MCP server on pi and dsh (422 unsupported_transport) on the premise that their clients take URLs only. Neither does: pi-mcp-adapter runs a `command` with `args` as a stdio transport, and dsh-mcp-client discriminates its config on `transport` with a `stdio` branch (StdioClientTransport on command, args, env, cwd). The runner now hands both the launcher it already writes for every other backend, and the per-base refusal is gone: one launcher, every client runs it. Verified on the hosted deployment across the bases. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> * ci: start checks --------- Co-authored-by: richard-epsilla <richard@epsilla.com> Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#175 refused a package with a stdio MCP server on pi and dsh (422
unsupported_transport) on the premise that their clients take URLs only. Neither does: pi-mcp-adapter runs acommandwithargsas a stdio transport, and dsh-mcp-client discriminates its config ontransportwith astdiobranch (StdioClientTransport on command, args, env, cwd). The runner now hands both the launcher it already writes for every other backend (pi: a command entry in its mcp.json; dsh: one more inserted client row), and the gateway's per-base refusal and its constant are gone. The self-hosting guide says so.Tests: gateway test_plugins.py (the refusal test became an install-on-every-base test), runner test_uhp_plugins.py (+2: pi's command entry, dsh's stdio row).
🤖 Generated with Claude Code