modelcontextprotocol/typescript-sdk
Watch
1
0
Fork
You've already forked typescript-sdk
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Merge pull request #65 from modelcontextprotocol/justin/stderr-improvements

Browse source 
Add way to get child process stderr pipe
This commit is contained in:
Justin Spahr-Summers 2024-11-20 21:29:20 +00:00 • committed by GitHub
commit 25d3123ac3
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 13 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

2
package.json
View file

@ -1,6 +1,6 @@
{
"name": "@modelcontextprotocol/sdk",
"version": "0.6.1",
"version": "0.7.0",
"description": "Model Context Protocol implementation for TypeScript",
"license": "MIT",
"author": "Anthropic, PBC (https://anthropic.com)",

15
src/client/stdio.ts
View file

@ -1,4 +1,4 @@
import { ChildProcess, spawn } from "node:child_process";
import { ChildProcess, IOType, spawn } from "node:child_process";
import process from "node:process";
import { ReadBuffer, serializeMessage } from "../shared/stdio.js";
import { JSONRPCMessage } from "../types.js";
@ -28,7 +28,7 @@ export type StdioServerParameters = {
*
* The default is "inherit", meaning messages to stderr will be printed to the parent process's stderr.
*/
stderr?: "inherit" | "ignore" | Stream;
stderr?: IOType | Stream | number;
};
/**
@ -123,7 +123,7 @@ export class StdioClientTransport implements Transport {
// (i.e. via `electron-forge start`) vs a production environment
// (i.e. YourApp.exe).
detached: process.platform === "win32" && isElectron(),
}
},
);
this._process.on("error", (error) => {
@ -161,6 +161,15 @@ export class StdioClientTransport implements Transport {
});
}
/**
* The stderr stream of the child process, if `StdioServerParameters.stderr` was set to "pipe" or "overlapped".
*
* This is only available after the process has been started.
*/
get stderr(): Stream | null {
return this._process?.stderr ?? null;
}
private processReadBuffer() {
while (true) {
try {