terminal-commands 0.1.1 → 0.1.3

package/README.md

@@ -18,6 +18,11 @@ approval prompts are enabled.
 
 Command-line programs still execute with your operating-system user permissions. A program can access more than the dedicated filesystem tools, so review every command approval carefully.
 
+The MCP command surface exposes process lifecycle operations separately: `start_process` starts
+an approved executable and returns a session ID, `read_process_output` reads bounded output,
+`list_processes` lists sessions, and `terminate_process` stops a session. This keeps terminal
+workflows available while making each model-callable operation explicit.
+
 ## Run
 
 ```bash

package/dist/agent.js

@@ -11894,7 +11894,7 @@ function attachSchema(issues, start, inst) {
     (_a3 = issues[i]).schema ?? (_a3.schema = inst);
   }
 }
-function finalizeIssue(iss, ctx, config2) {
+function finalizeIssue(iss, ctx, config3) {
   var _a3;
   const traits = iss.inst?._zod?.traits;
   if (traits?.has("$ZodType")) {
@@ -11904,7 +11904,7 @@ function finalizeIssue(iss, ctx, config2) {
       iss.schema = iss.inst;
   }
   const schemaError = iss.schema !== iss.inst ? iss.schema?._zod.def?.error : void 0;
-  const message = iss.message ? iss.message : unwrapMessage(iss.inst?._zod.def?.error?.(iss)) ?? unwrapMessage(schemaError?.(iss)) ?? unwrapMessage(ctx?.error?.(iss)) ?? unwrapMessage(config2.customError?.(iss)) ?? unwrapMessage(config2.localeError?.(iss)) ?? "Invalid input";
+  const message = iss.message ? iss.message : unwrapMessage(iss.inst?._zod.def?.error?.(iss)) ?? unwrapMessage(schemaError?.(iss)) ?? unwrapMessage(ctx?.error?.(iss)) ?? unwrapMessage(config3.customError?.(iss)) ?? unwrapMessage(config3.localeError?.(iss)) ?? "Invalid input";
   const full = {};
   for (const k of Object.keys(iss)) {
     if (k === "inst" || k === "schema" || k === "continue" || k === "input" || k === "__proto__")
@@ -33537,21 +33537,21 @@ var Client = class extends Protocol {
    * Handlers are silently skipped if the server doesn't advertise the corresponding listChanged capability.
    * @internal
    */
-  _setupListChangedHandlers(config2) {
-    if (config2.tools && this._serverCapabilities?.tools?.listChanged) {
-      this._setupListChangedHandler("tools", ToolListChangedNotificationSchema, config2.tools, async () => {
+  _setupListChangedHandlers(config3) {
+    if (config3.tools && this._serverCapabilities?.tools?.listChanged) {
+      this._setupListChangedHandler("tools", ToolListChangedNotificationSchema, config3.tools, async () => {
         const result = await this.listTools();
         return result.tools;
       });
     }
-    if (config2.prompts && this._serverCapabilities?.prompts?.listChanged) {
-      this._setupListChangedHandler("prompts", PromptListChangedNotificationSchema, config2.prompts, async () => {
+    if (config3.prompts && this._serverCapabilities?.prompts?.listChanged) {
+      this._setupListChangedHandler("prompts", PromptListChangedNotificationSchema, config3.prompts, async () => {
         const result = await this.listPrompts();
         return result.prompts;
       });
     }
-    if (config2.resources && this._serverCapabilities?.resources?.listChanged) {
-      this._setupListChangedHandler("resources", ResourceListChangedNotificationSchema, config2.resources, async () => {
+    if (config3.resources && this._serverCapabilities?.resources?.listChanged) {
+      this._setupListChangedHandler("resources", ResourceListChangedNotificationSchema, config3.resources, async () => {
         const result = await this.listResources();
         return result.resources;
       });
@@ -34881,12 +34881,12 @@ var CR = 13;
 var SPACE = 32;
 function noop(_arg) {
 }
-function createParser(config2) {
-  if (typeof config2 == "function")
+function createParser(config3) {
+  if (typeof config3 == "function")
     throw new TypeError(
       "`config` must be an object, got a function instead. Did you mean `createParser({onEvent: fn})`?"
     );
-  const { onEvent = noop, onError = noop, onRetry = noop, onComment, maxBufferSize } = config2, pendingFragments = [];
+  const { onEvent = noop, onError = noop, onRetry = noop, onComment, maxBufferSize } = config3, pendingFragments = [];
   let pendingFragmentsLength = 0, isFirstChunk = true, id, data = "", dataLines = 0, eventType, terminated = false;
   function feed(chunk) {
     if (terminated)
@@ -35480,7 +35480,11 @@ var import_websocket_server = __toESM(require_websocket_server(), 1);
 var wrapper_default = import_websocket.default;
 
 // src/agent-core.ts
-var destructiveTools = /* @__PURE__ */ new Set(["run_program", "write_text_file"]);
+var destructiveTools = /* @__PURE__ */ new Set([
+  "start_process",
+  "terminate_process",
+  "write_text_file"
+]);
 var requestSchema = external_exports.object({
   type: external_exports.literal("request"),
   requestId: external_exports.string().uuid(),
@@ -35488,7 +35492,10 @@ var requestSchema = external_exports.object({
     "get_system_info",
     "list_directory",
     "read_text_file",
-    "run_program",
+    "start_process",
+    "read_process_output",
+    "terminate_process",
+    "list_processes",
     "write_text_file"
   ]),
   arguments: external_exports.record(external_exports.string(), external_exports.unknown())
@@ -35505,18 +35512,18 @@ async function authorizeToolCall(tool, args, allowDestructive, approve) {
   }
 }
 async function connectOnce(options) {
-  const { config: config2, signal } = options;
+  const { config: config3, signal } = options;
   const client = new Client({
     name: "terminal-commands-cli",
     version: "0.1.0"
   });
   await client.connect(
     new StreamableHTTPClientTransport(
-      new URL(config2.MACHINE_TERMINAL_LOCAL_MCP_URL)
+      new URL(config3.MACHINE_TERMINAL_LOCAL_MCP_URL)
     )
   );
-  const token = options.getToken ? await options.getToken() : config2.MACHINE_TERMINAL_DEVICE_TOKEN;
-  const socket = new wrapper_default(config2.MACHINE_TERMINAL_GATEWAY_URL, {
+  const token = options.getToken ? await options.getToken() : config3.MACHINE_TERMINAL_DEVICE_TOKEN;
+  const socket = new wrapper_default(config3.MACHINE_TERMINAL_GATEWAY_URL, {
     headers: { Authorization: `Bearer ${token}` }
   });
   const stop2 = () => socket.close(1e3, "Client stopping.");
@@ -35525,8 +35532,8 @@ async function connectOnce(options) {
     socket.send(
       JSON.stringify({
         type: "hello",
-        deviceId: config2.MACHINE_TERMINAL_DEVICE_ID,
-        name: config2.MACHINE_TERMINAL_DEVICE_NAME,
+        deviceId: config3.MACHINE_TERMINAL_DEVICE_ID,
+        name: config3.MACHINE_TERMINAL_DEVICE_NAME,
         platform: `${os.platform()}-${os.arch()}`
       })
     );
@@ -35618,11 +35625,12 @@ var schema = external_exports.object({
   MACHINE_TERMINAL_DEVICE_ID: external_exports.string().min(1).max(128),
   MACHINE_TERMINAL_DEVICE_NAME: external_exports.string().min(1).max(128).default(os2.hostname()),
   MACHINE_TERMINAL_LOCAL_MCP_URL: external_exports.string().url().default("http://127.0.0.1:3333/mcp"),
-  MACHINE_TERMINAL_AGENT_ALLOW_DESTRUCTIVE: external_exports.enum(["0", "1"]).default("0")
+  MACHINE_TERMINAL_AGENT_ALLOW_DESTRUCTIVE: external_exports.enum(["0", "1"]).default("0"),
+  MACHINE_TERMINAL_AUTO_APPROVE: external_exports.enum(["0", "1"]).default("0")
 });
 function loadAgentConfig(environment = process.env) {
-  const config2 = schema.parse(environment);
-  const gateway = new URL(config2.MACHINE_TERMINAL_GATEWAY_URL);
+  const config3 = schema.parse(environment);
+  const gateway = new URL(config3.MACHINE_TERMINAL_GATEWAY_URL);
   if (!(/* @__PURE__ */ new Set(["wss:", "ws:"])).has(gateway.protocol)) {
     throw new Error("MACHINE_TERMINAL_GATEWAY_URL must use wss:// or ws://.");
   }
@@ -35631,7 +35639,87 @@ function loadAgentConfig(environment = process.env) {
       "Unencrypted ws:// is allowed only for loopback development."
     );
   }
-  return config2;
+  return {
+    ...config3,
+    MACHINE_TERMINAL_AGENT_ALLOW_DESTRUCTIVE: config3.MACHINE_TERMINAL_AUTO_APPROVE === "1" ? "1" : config3.MACHINE_TERMINAL_AGENT_ALLOW_DESTRUCTIVE
+  };
+}
+
+// src/approval.ts
+import { createInterface } from "node:readline/promises";
+
+// src/cli-format.ts
+var isColorEnabled = Boolean(process.stdout.isTTY) && process.env.NO_COLOR === void 0 && process.env.TERM !== "dumb";
+var CODES = {
+  reset: "\x1B[0m",
+  bold: "\x1B[1m",
+  dim: "\x1B[2m",
+  red: "\x1B[31m",
+  green: "\x1B[32m",
+  yellow: "\x1B[33m",
+  blue: "\x1B[34m",
+  cyan: "\x1B[36m",
+  gray: "\x1B[90m"
+};
+function paint(code, text) {
+  if (!isColorEnabled) return text;
+  return `${CODES[code]}${text}${CODES.reset}`;
+}
+var bold = (text) => paint("bold", text);
+var dim = (text) => paint("dim", text);
+var warn = (text) => paint("yellow", text);
+
+// src/approval.ts
+function approvalSummary(tool, args) {
+  if (tool === "write_text_file") {
+    const bytes = Buffer.byteLength(String(args.contents ?? ""));
+    return `Write ${bytes} bytes to ${String(args.path ?? "an unknown path")}${args.overwrite ? " (overwrite)" : ""}`;
+  }
+  if (tool === "start_process") {
+    const command = [
+      String(args.program ?? ""),
+      ...Array.isArray(args.args) ? args.args.map(String) : []
+    ];
+    return `Start: ${command.join(" ")}`;
+  }
+  if (tool === "terminate_process") {
+    return `Terminate process session ${String(args.sessionId ?? "")}`;
+  }
+  return `Run destructive tool: ${tool}`;
+}
+function createAutoApprover() {
+  return (tool, args) => {
+    console.log(
+      `
+${warn("Approval requested:")} ${approvalSummary(tool, args)} ${dim("(auto-approved)")}`
+    );
+    return Promise.resolve(true);
+  };
+}
+function createTerminalApprover() {
+  let queue = Promise.resolve(false);
+  return (tool, args) => {
+    queue = queue.catch(() => false).then(async () => {
+      if (!process.stdin.isTTY || !process.stdout.isTTY) return false;
+      const prompt = createInterface({
+        input: process.stdin,
+        output: process.stdout
+      });
+      try {
+        console.log(
+          `
+${warn("Approval requested:")} ${approvalSummary(tool, args)}`
+        );
+        const answer = await prompt.question(
+          `${bold("Approve this action?")} [y/N] `
+        );
+        return (/* @__PURE__ */ new Set(["y", "yes"])).has(answer.trim().toLowerCase());
+      } finally {
+        prompt.close();
+      }
+    });
+    return queue;
+  };
 }
 
 // src/agent.ts
@@ -35639,8 +35727,10 @@ var controller = new AbortController();
 var stop = () => controller.abort();
 process.on("SIGINT", stop);
 process.on("SIGTERM", stop);
+var config2 = loadAgentConfig();
 void runAgent({
-  config: loadAgentConfig(),
+  config: config2,
+  approve: config2.MACHINE_TERMINAL_AUTO_APPROVE === "1" ? createAutoApprover() : createTerminalApprover(),
   signal: controller.signal,
   log: (message) => console.error(`Terminal Commands: ${message}`)
 }).catch((error62) => {

package/dist/cli.js

@@ -10984,7 +10984,6 @@ import { spawn } from "node:child_process";
 import { fileURLToPath } from "node:url";
 import os5 from "node:os";
 import path3 from "node:path";
-import { createInterface } from "node:readline/promises";
 
 // src/agent-core.ts
 import os from "node:os";
@@ -35488,7 +35487,11 @@ var import_websocket_server = __toESM(require_websocket_server(), 1);
 var wrapper_default = import_websocket.default;
 
 // src/agent-core.ts
-var destructiveTools = /* @__PURE__ */ new Set(["run_program", "write_text_file"]);
+var destructiveTools = /* @__PURE__ */ new Set([
+  "start_process",
+  "terminate_process",
+  "write_text_file"
+]);
 var requestSchema = external_exports.object({
   type: external_exports.literal("request"),
   requestId: external_exports.string().uuid(),
@@ -35496,7 +35499,10 @@ var requestSchema = external_exports.object({
     "get_system_info",
     "list_directory",
     "read_text_file",
-    "run_program",
+    "start_process",
+    "read_process_output",
+    "terminate_process",
+    "list_processes",
     "write_text_file"
   ]),
   arguments: external_exports.record(external_exports.string(), external_exports.unknown())
@@ -35626,7 +35632,8 @@ var schema = external_exports.object({
   MACHINE_TERMINAL_DEVICE_ID: external_exports.string().min(1).max(128),
   MACHINE_TERMINAL_DEVICE_NAME: external_exports.string().min(1).max(128).default(os2.hostname()),
   MACHINE_TERMINAL_LOCAL_MCP_URL: external_exports.string().url().default("http://127.0.0.1:3333/mcp"),
-  MACHINE_TERMINAL_AGENT_ALLOW_DESTRUCTIVE: external_exports.enum(["0", "1"]).default("0")
+  MACHINE_TERMINAL_AGENT_ALLOW_DESTRUCTIVE: external_exports.enum(["0", "1"]).default("0"),
+  MACHINE_TERMINAL_AUTO_APPROVE: external_exports.enum(["0", "1"]).default("0")
 });
 function loadAgentConfig(environment = process.env) {
   const config2 = schema.parse(environment);
@@ -35639,7 +35646,103 @@ function loadAgentConfig(environment = process.env) {
       "Unencrypted ws:// is allowed only for loopback development."
     );
   }
-  return config2;
+  return {
+    ...config2,
+    MACHINE_TERMINAL_AGENT_ALLOW_DESTRUCTIVE: config2.MACHINE_TERMINAL_AUTO_APPROVE === "1" ? "1" : config2.MACHINE_TERMINAL_AGENT_ALLOW_DESTRUCTIVE
+  };
+}
+
+// src/approval.ts
+import { createInterface } from "node:readline/promises";
+
+// src/cli-format.ts
+var isColorEnabled = Boolean(process.stdout.isTTY) && process.env.NO_COLOR === void 0 && process.env.TERM !== "dumb";
+var CODES = {
+  reset: "\x1B[0m",
+  bold: "\x1B[1m",
+  dim: "\x1B[2m",
+  red: "\x1B[31m",
+  green: "\x1B[32m",
+  yellow: "\x1B[33m",
+  blue: "\x1B[34m",
+  cyan: "\x1B[36m",
+  gray: "\x1B[90m"
+};
+function paint(code, text) {
+  if (!isColorEnabled) return text;
+  return `${CODES[code]}${text}${CODES.reset}`;
+}
+var bold = (text) => paint("bold", text);
+var dim = (text) => paint("dim", text);
+var ok = (text) => paint("green", text);
+var warn = (text) => paint("yellow", text);
+var danger = (text) => paint("red", text);
+var info = (text) => paint("cyan", text);
+var muted = (text) => paint("gray", text);
+function heading(text) {
+  const rule = "\u2500".repeat(Math.max(text.length, 3));
+  return `${bold(info(text))}
+${muted(rule)}`;
+}
+function keyValueBlock(rows) {
+  const width = rows.reduce(
+    (widest, [label]) => Math.max(widest, label.length),
+    0
+  );
+  return rows.map(([label, value]) => `  ${dim(label.padEnd(width))}  ${value}`).join("\n");
+}
+
+// src/approval.ts
+function approvalSummary(tool, args) {
+  if (tool === "write_text_file") {
+    const bytes = Buffer.byteLength(String(args.contents ?? ""));
+    return `Write ${bytes} bytes to ${String(args.path ?? "an unknown path")}${args.overwrite ? " (overwrite)" : ""}`;
+  }
+  if (tool === "start_process") {
+    const command = [
+      String(args.program ?? ""),
+      ...Array.isArray(args.args) ? args.args.map(String) : []
+    ];
+    return `Start: ${command.join(" ")}`;
+  }
+  if (tool === "terminate_process") {
+    return `Terminate process session ${String(args.sessionId ?? "")}`;
+  }
+  return `Run destructive tool: ${tool}`;
+}
+function createAutoApprover() {
+  return (tool, args) => {
+    console.log(
+      `
+${warn("Approval requested:")} ${approvalSummary(tool, args)} ${dim("(auto-approved)")}`
+    );
+    return Promise.resolve(true);
+  };
+}
+function createTerminalApprover() {
+  let queue = Promise.resolve(false);
+  return (tool, args) => {
+    queue = queue.catch(() => false).then(async () => {
+      if (!process.stdin.isTTY || !process.stdout.isTTY) return false;
+      const prompt = createInterface({
+        input: process.stdin,
+        output: process.stdout
+      });
+      try {
+        console.log(
+          `
+${warn("Approval requested:")} ${approvalSummary(tool, args)}`
+        );
+        const answer = await prompt.question(
+          `${bold("Approve this action?")} [y/N] `
+        );
+        return (/* @__PURE__ */ new Set(["y", "yes"])).has(answer.trim().toLowerCase());
+      } finally {
+        prompt.close();
+      }
+    });
+    return queue;
+  };
 }
 
 // src/cli-options.ts
@@ -36021,43 +36124,6 @@ function renderHelp(command = "help") {
   return lines.join("\n");
 }
 
-// src/cli-format.ts
-var isColorEnabled = Boolean(process.stdout.isTTY) && process.env.NO_COLOR === void 0 && process.env.TERM !== "dumb";
-var CODES = {
-  reset: "\x1B[0m",
-  bold: "\x1B[1m",
-  dim: "\x1B[2m",
-  red: "\x1B[31m",
-  green: "\x1B[32m",
-  yellow: "\x1B[33m",
-  blue: "\x1B[34m",
-  cyan: "\x1B[36m",
-  gray: "\x1B[90m"
-};
-function paint(code, text) {
-  if (!isColorEnabled) return text;
-  return `${CODES[code]}${text}${CODES.reset}`;
-}
-var bold = (text) => paint("bold", text);
-var dim = (text) => paint("dim", text);
-var ok = (text) => paint("green", text);
-var warn = (text) => paint("yellow", text);
-var danger = (text) => paint("red", text);
-var info = (text) => paint("cyan", text);
-var muted = (text) => paint("gray", text);
-function heading(text) {
-  const rule = "\u2500".repeat(Math.max(text.length, 3));
-  return `${bold(info(text))}
-${muted(rule)}`;
-}
-function keyValueBlock(rows) {
-  const width = rows.reduce(
-    (widest, [label]) => Math.max(widest, label.length),
-    0
-  );
-  return rows.map(([label, value]) => `  ${dim(label.padEnd(width))}  ${value}`).join("\n");
-}
-
 // src/credential-store.ts
 import { promises as fs2 } from "node:fs";
 import { randomUUID } from "node:crypto";
@@ -36347,54 +36413,6 @@ function describeSettings(settings, deviceId) {
     ])
   ].join("\n");
 }
-function approvalSummary(tool, args) {
-  if (tool === "write_text_file") {
-    const bytes = Buffer.byteLength(String(args.contents ?? ""));
-    return `Write ${bytes} bytes to ${String(args.path ?? "an unknown path")}${args.overwrite ? " (overwrite)" : ""}`;
-  }
-  if (tool === "run_program") {
-    const command = [
-      String(args.program ?? ""),
-      ...Array.isArray(args.args) ? args.args.map(String) : []
-    ];
-    return `Run: ${command.join(" ")}`;
-  }
-  return `Run destructive tool: ${tool}`;
-}
-function createAutoApprover() {
-  return (tool, args) => {
-    console.log(
-      `
-${warn("Approval requested:")} ${approvalSummary(tool, args)} ${dim("(auto-approved)")}`
-    );
-    return Promise.resolve(true);
-  };
-}
-function createTerminalApprover() {
-  let queue = Promise.resolve(false);
-  return (tool, args) => {
-    queue = queue.catch(() => false).then(async () => {
-      if (!process.stdin.isTTY || !process.stdout.isTTY) return false;
-      const prompt = createInterface({
-        input: process.stdin,
-        output: process.stdout
-      });
-      try {
-        console.log(
-          `
-${warn("Approval requested:")} ${approvalSummary(tool, args)}`
-        );
-        const answer = await prompt.question(
-          `${bold("Approve this action?")} [y/N] `
-        );
-        return (/* @__PURE__ */ new Set(["y", "yes"])).has(answer.trim().toLowerCase());
-      } finally {
-        prompt.close();
-      }
-    });
-    return queue;
-  };
-}
 async function login(environment) {
   const auth2 = resolveAuth(environment);
   const code = await requestDeviceCode(auth2);