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

Merge pull request #739 from nsheaps/main

Browse source 
fix: correct stdout buffering issues in spawned processes
This commit is contained in:
Ola Hungerford 2025-09-01 09:22:32 -07:00 • committed by GitHub
commit e1d311c6e1
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 8 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

8
cli/src/cli.ts
View file

@ -106,6 +106,10 @@ async function runWebClient(args: Args): Promise<void> {
await spawnPromise("node", [inspectorClientPath, ...startArgs], { await spawnPromise("node", [inspectorClientPath, ...startArgs], {
signal: abort.signal, signal: abort.signal,
echoOutput: true, echoOutput: true,
// pipe the stdout through here, prevents issues with buffering and
// dropping the end of console.out after 8192 chars due to node
// closing the stdout pipe before the output has finished flushing
stdio: "inherit",
}); });
} catch (e) { } catch (e) {
if (!cancelled || process.env.DEBUG) throw e; if (!cancelled || process.env.DEBUG) throw e;
@ -151,6 +155,10 @@ async function runCli(args: Args): Promise<void> {
env: { ...process.env, ...args.envArgs }, env: { ...process.env, ...args.envArgs },
signal: abort.signal, signal: abort.signal,
echoOutput: true, echoOutput: true,
// pipe the stdout through here, prevents issues with buffering and
// dropping the end of console.out after 8192 chars due to node
// closing the stdout pipe before the output has finished flushing
stdio: "inherit",
}); });
} catch (e) { } catch (e) {
if (!cancelled || process.env.DEBUG) { if (!cancelled || process.env.DEBUG) {