@modeltoolsprotocol/mtpcli 0.3.1 → 0.3.2

package/README.md

@@ -20,7 +20,7 @@ LLM agents need to discover and use tools. Right now there are two worlds:
 | **Scriptable without an LLM** | Yes, that's the whole point of CLIs | Not really, designed for LLM interaction |
 | **Adoption cost** | One flag/decorator | New protocol, server scaffolding, SDK |
 
-The gap is discovery. `mtpcli` fills it.
+The gap for CLIs is discovery. `mtpcli` fills it.
 
 ## Install
 

package/dist/index.js

@@ -8738,7 +8738,7 @@ function argToJsonSchema(arg) {
   return schema;
 }
 function commandToMcpTool(toolName, cmd) {
-  const mcpName = cmd.name === "_root" ? toolName : `${toolName}__${cmd.name}`;
+  const mcpName = cmd.name === "_root" ? toolName : `${toolName}__${cmd.name.replace(/ /g, "_")}`;
   const properties = {};
   const required = [];
   for (const arg of cmd.args) {
@@ -8772,7 +8772,7 @@ function commandToMcpTool(toolName, cmd) {
 function buildCliCommand(toolName, commandName, arguments_, commandSchema) {
   const cmd = [toolName];
   if (commandName !== "_root")
-    cmd.push(commandName);
+    cmd.push(...commandName.split(" "));
   const argSchemas = new Map;
   for (const a of commandSchema.args) {
     argSchemas.set(a.name.replace(/^-+/, ""), a);
@@ -8826,29 +8826,57 @@ function invokeCliTool(toolName, commandName, arguments_, commandSchema, authEnv
       stdio: [stdinStr !== undefined ? "pipe" : "ignore", "pipe", "pipe"],
       env: { ...process.env, ...authEnv }
     });
+    const timer = setTimeout(() => {
+      child.kill();
+      resolve({
+        content: [{ type: "text", text: `Error: tool timed out after ${TOOL_TIMEOUT_MS / 1000}s` }],
+        isError: true
+      });
+    }, TOOL_TIMEOUT_MS);
     if (stdinStr !== undefined && child.stdin) {
       child.stdin.write(stdinStr);
       child.stdin.end();
     }
     let stdout = "";
     let stderr = "";
-    child.stdout?.on("data", (d) => stdout += d);
-    child.stderr?.on("data", (d) => stderr += d);
+    let stdoutBytes = 0;
+    let stderrBytes = 0;
+    let truncated = false;
+    child.stdout?.on("data", (d) => {
+      stdoutBytes += d.length;
+      if (stdoutBytes <= MAX_OUTPUT_BYTES) {
+        stdout += d;
+      } else if (!truncated) {
+        truncated = true;
+        child.kill();
+      }
+    });
+    child.stderr?.on("data", (d) => {
+      stderrBytes += d.length;
+      if (stderrBytes <= MAX_OUTPUT_BYTES) {
+        stderr += d;
+      }
+    });
     child.on("error", (e) => {
+      clearTimeout(timer);
       resolve({
         content: [{ type: "text", text: `Error: ${e.message}` }],
         isError: true
       });
     });
     child.on("close", (code) => {
+      clearTimeout(timer);
       const content = [];
+      if (truncated) {
+        content.push({ type: "text", text: `[truncated: output exceeded ${MAX_OUTPUT_BYTES / 1024 / 1024}MB]` });
+      }
       if (stdout.trim())
         content.push({ type: "text", text: stdout.trim() });
       if (stderr.trim())
         content.push({ type: "text", text: `[stderr] ${stderr.trim()}` });
       if (content.length === 0)
         content.push({ type: "text", text: "(no output)" });
-      resolve({ content, isError: code !== 0 });
+      resolve({ content, isError: code !== 0 || truncated });
     });
   });
 }
@@ -8948,13 +8976,14 @@ async function run(toolNames) {
     }
   }
 }
-var execFileAsync7, VERSION = "0.1.0";
+var execFileAsync7, VERSION = "0.1.0", TOOL_TIMEOUT_MS = 60000, MAX_OUTPUT_BYTES;
 var init_serve = __esm(() => {
   init_auth();
   init_mcp();
   init_models();
   init_search2();
   execFileAsync7 = promisify9(execFile9);
+  MAX_OUTPUT_BYTES = 1024 * 1024 * 1024;
 });
 
 // src/mcp-oauth.ts
@@ -10564,7 +10593,7 @@ function cleanJson(obj) {
 }
 
 // src/index.ts
-var VERSION3 = "0.3.1";
+var VERSION3 = "0.3.2";
 function selfDescribe() {
   const schema = {
     name: "mtpcli",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@modeltoolsprotocol/mtpcli",
-  "version": "0.3.1",
+  "version": "0.3.2",
   "license": "Apache-2.0",
   "type": "module",
   "bin": {