jcarr
/
gemini-cli
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix MCP under seatbelt, improve error handling (#301)

This commit is contained in:
Olcan 2025-05-09 09:02:14 -07:00 committed by GitHub
parent b35a3856a2
commit c58f879026
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
packages/cli/src/utils/sandbox-macos-minimal.sb
View File

@ -10,6 +10,7 @@
(subpath (param "TMP_DIR")) (subpath (param "TMP_DIR"))
(subpath (string-append (param "HOME_DIR") "/.gemini")) (subpath (string-append (param "HOME_DIR") "/.gemini"))
(subpath (string-append (param "HOME_DIR") "/.npm")) (subpath (string-append (param "HOME_DIR") "/.npm"))
(subpath (string-append (param "HOME_DIR") "/.cache"))
(literal "/dev/stdout") (literal "/dev/stdout")
(literal "/dev/stderr") (literal "/dev/stderr")
(literal "/dev/null") (literal "/dev/null")

1
packages/cli/src/utils/sandbox-macos-strict.sb
View File

@ -68,6 +68,7 @@
(subpath (param "TMP_DIR")) (subpath (param "TMP_DIR"))
(subpath (string-append (param "HOME_DIR") "/.gemini")) (subpath (string-append (param "HOME_DIR") "/.gemini"))
(subpath (string-append (param "HOME_DIR") "/.npm")) (subpath (string-append (param "HOME_DIR") "/.npm"))
(subpath (string-append (param "HOME_DIR") "/.cache"))
(literal "/dev/stdout") (literal "/dev/stdout")
(literal "/dev/stderr") (literal "/dev/stderr")
(literal "/dev/null") (literal "/dev/null")

10
packages/server/src/tools/tool-registry.ts
View File

@ -187,7 +187,15 @@ export class ToolRegistry {
command: mcpServerCmd, command: mcpServerCmd,
stderr: 'pipe', stderr: 'pipe',
}); });
await this.mcpClient.connect(transport); try {
await this.mcpClient.connect(transport);
} catch (error) {
console.error(
'failed to start or connect to MCP server using ' +
`command '${mcpServerCmd}'; \n${error}`,
);
throw error;
}
this.mcpClient.onerror = (error) => { this.mcpClient.onerror = (error) => {
console.error('MCP ERROR', error.toString()); console.error('MCP ERROR', error.toString());
}; };