Replies: 2 comments
|
Independently hit this and can corroborate the exact fix proposed here: added the one line to python/sdk-runtime/package.json, verify-runtime-closure passes (closed graph, 110 packages), and the rebuilt node-mode runtime runs a composition with dsh-llm-pi-ai (custom provider route) + dsh-mcp-client (stdio server, ~96 tools) + dsh-tool-fs end to end through the Python SDK — tools discovered, registered, and called by the model. Until it ships in the wheel, every MCP-carrying embedder has to maintain this same one-line fork. Fuller writeup of how this interacts with the boot race in #2273. |
|
This is the same deploy-root omission as #2273 — a branch already exists with the exact one-line fix.
Cross-refs: #2273 / #1240 / #2115 all report the same |
Uh oh!
There was an error while loading. Please reload this page.
Summary
The Python SDK's bundled runtime cannot mount MCP servers: the deploy root
(
python/sdk-runtime/package.json, whose dependency closure IS both the exe'splugin set and the node carrier's tree) does not list
@deepseek-ai/dsh-mcp-client.packaged-bin.jsresolves bare plugins only fromthat closed tree, so any Cordis composition carrying an mcp-client row fails to
load on both bundled carriers — and MCP is the standard way for a host
application to hand its own tools to the agent.
The sdk-runtime README
describes exactly the intended fix path:
Why it matters
For an embedder, the SDK runtime without an MCP client is an agent that can only
use the plugins baked at build time — no host-provided tools at all. We currently
work around this by maintaining our own npm deploy-root closure (same
packaged-bin.js, our ownpackage.jsonwith the extra line), which works butmeans the official wheels are unusable for any MCP-carrying composition.
dsh-mcp-clientis already published to npm at the same0.1.0-rc.xwave asthe rest of the runtime, so this appears to be exactly the advertised
one-line-plus-rebuild change.
Ask
Add
"@deepseek-ai/dsh-mcp-client"topython/sdk-runtime/package.json'sdependencies and include it in the next runtime wheel build. A composition that
doesn't reference it pays nothing; one that does currently cannot boot at all on
the bundled carriers.
Environment
deepseek-harness-sdk/deepseek-harness-runtime-bin0.1.0rc6(PyPI);verified against the npm-published
0.1.0-rc.6packages that an mcp-client rowloads and serves
mcp__<server>__<tool>tools once the package is present inthe closure.
All reactions