@ricsam/r5d-worker 0.0.67 → 0.0.68

package/README.md

@@ -38,4 +38,6 @@ worker exec macos docker compose up -d
 worker exec macos bun test
 ```
 
+Agent shell commands receive no stdin by default: prompts read immediate EOF instead of hanging. A command started with `interactive: true` is spawned with a writable stdin pipe, and the agent's `shell_write` tool delivers input through the `exec_stdin` protocol message. The worker acknowledges each write with an `operation_result` carrying the delivered byte count, closes stdin on an explicit `eof`, on cancellation, on timeout, and at process exit, and advertises this support through the `execStdinV1` capability in its hello message. Servers refuse interactive runs and stdin writes for workers without the capability, so this is a protocol-compatible release that uses the ordinary worker update flow. Delivered stdin is audited server-side in the run's `process-runs/<runId>/stdin.log`.
+
 When the connected `r5d-browser` requests a port forward, the worker opens each relayed connection only to `127.0.0.1` on the requested worker port. Browser-side and worker-side ports may differ. The worker never opens a public listener, and a disconnected worker leaves the browser's long-lived mapping unavailable until the same worker label reconnects.

package/dist/cjs/main.cjs

@@ -1960,8 +1960,12 @@ async function executeStreamingCommand(input) {
       activePlanId: input.message.env?.R5D_ACTIVE_PLAN_ID
     });
     const cwd = resolveCommandCwd(input.resolvedTarget.rootPath, input.message.cwd);
+    const interactive = input.message.interactive === true;
     const subprocess = Bun.spawn(input.message.argv, {
       cwd,
+      // Without an explicit stdin the process reads /dev/null and interactive
+      // prompts see immediate EOF; "pipe" keeps stdin open for exec_stdin.
+      ...interactive ? { stdin: "pipe" } : {},
       stdout: "pipe",
       stderr: "pipe",
       detached: true,
@@ -1983,7 +1987,8 @@ async function executeStreamingCommand(input) {
       argv: input.message.argv,
       command: input.message.command,
       cwd,
-      startedAt: (/* @__PURE__ */ new Date()).toISOString()
+      startedAt: (/* @__PURE__ */ new Date()).toISOString(),
+      ...interactive ? { interactive: true, stdin: subprocess.stdin } : {}
     });
     started = true;
     sendWorkerMessage(input.ws, {
@@ -1997,6 +2002,7 @@ async function executeStreamingCommand(input) {
     if (input.message.timeoutMs) {
       timeout = setTimeout(() => {
         timedOut = true;
+        closeProcessStdin(activeProcesses.get(input.message.runId));
         void (0, import_process_tree.terminateProcessTree)(subprocess).catch((error) => {
           process.stderr.write(
             `[r5d-worker] failed to terminate timed-out process ${input.message.runId}: ${error instanceof Error ? error.message : String(error)}
@@ -2056,10 +2062,21 @@ async function executeStreamingCommand(input) {
     if (timeout) {
       clearTimeout(timeout);
     }
+    closeProcessStdin(activeProcesses.get(input.message.runId));
     activeProcesses.delete(input.message.runId);
     cancelledProcessRuns.delete(input.message.runId);
   }
 }
+function closeProcessStdin(active) {
+  if (!active?.stdin) {
+    return;
+  }
+  try {
+    void active.stdin.end();
+  } catch {
+  }
+  active.stdin = void 0;
+}
 function sendWorkerMessage(ws, message) {
   const target = currentWorkerSocket ?? ws;
   try {
@@ -2083,7 +2100,8 @@ function buildActiveProcessReports() {
     argv: active.argv,
     command: active.command,
     ...active.cwd ? { cwd: active.cwd } : {},
-    startedAt: active.startedAt
+    startedAt: active.startedAt,
+    ...active.interactive ? { interactive: true } : {}
   }));
 }
 function sendActiveProcessReport(ws) {
@@ -2889,7 +2907,8 @@ async function startWorker(options) {
           browserPortForwarding: true,
           globalWorkspaceSyncLeaseV1: true,
           eventualWorkspaceSyncV1: true,
-          workspaceIncidentCandidatePinV1: true
+          workspaceIncidentCandidatePinV1: true,
+          execStdinV1: true
         },
         projectRoot: projectsRoot,
         artifactRoot,
@@ -3332,6 +3351,7 @@ async function startWorker(options) {
         let cancelMessage;
         if (active) {
           try {
+            closeProcessStdin(active);
             await (0, import_process_tree.terminateProcessTree)(active.process);
             cancelMessage = `Stopped ${active.command}`;
           } catch (error) {
@@ -3353,6 +3373,51 @@ async function startWorker(options) {
         );
         return;
       }
+      if (message.type === "exec_stdin") {
+        const sendAck = (payload) => ws.send(
+          JSON.stringify({
+            type: "operation_result",
+            requestId: message.requestId,
+            ...payload
+          })
+        );
+        const active = activeProcesses.get(message.runId);
+        if (!active) {
+          sendAck({ error: `Process run ${message.runId} is not active on this worker` });
+          return;
+        }
+        if (!active.interactive || !active.stdin) {
+          sendAck({
+            error: `Process run ${message.runId} has no open stdin. Start a new shell command with "interactive": true to write to its stdin.`
+          });
+          return;
+        }
+        try {
+          let bytesWritten = 0;
+          if (message.data !== void 0 && message.data.length > 0) {
+            bytesWritten = await active.stdin.write(message.data);
+            await active.stdin.flush();
+          }
+          if (message.eof === true) {
+            await active.stdin.end();
+            active.stdin = void 0;
+          }
+          if (targetMayMutateVisibleWorkspace(active.target)) {
+            markWorkspaceDirty({ type: "shell_inline", detail: `exec stdin ${message.runId}` });
+          }
+          sendAck({
+            result: {
+              type: "exec_stdin",
+              runId: message.runId,
+              bytesWritten,
+              eof: message.eof === true
+            }
+          });
+        } catch (error) {
+          sendAck({ error: error instanceof Error ? error.message : String(error) });
+        }
+        return;
+      }
       if (message.type === "pty_open") {
         if (cliUpdateInProgress) {
           sendWorkerMessage(ws, {

package/dist/cjs/package.json

@@ -1,5 +1,5 @@
 {
   "name": "@ricsam/r5d-worker",
-  "version": "0.0.67",
+  "version": "0.0.68",
   "type": "commonjs"
 }

package/dist/mjs/main.mjs

@@ -1931,8 +1931,12 @@ async function executeStreamingCommand(input) {
       activePlanId: input.message.env?.R5D_ACTIVE_PLAN_ID
     });
     const cwd = resolveCommandCwd(input.resolvedTarget.rootPath, input.message.cwd);
+    const interactive = input.message.interactive === true;
     const subprocess = Bun.spawn(input.message.argv, {
       cwd,
+      // Without an explicit stdin the process reads /dev/null and interactive
+      // prompts see immediate EOF; "pipe" keeps stdin open for exec_stdin.
+      ...interactive ? { stdin: "pipe" } : {},
       stdout: "pipe",
       stderr: "pipe",
       detached: true,
@@ -1954,7 +1958,8 @@ async function executeStreamingCommand(input) {
       argv: input.message.argv,
       command: input.message.command,
       cwd,
-      startedAt: (/* @__PURE__ */ new Date()).toISOString()
+      startedAt: (/* @__PURE__ */ new Date()).toISOString(),
+      ...interactive ? { interactive: true, stdin: subprocess.stdin } : {}
     });
     started = true;
     sendWorkerMessage(input.ws, {
@@ -1968,6 +1973,7 @@ async function executeStreamingCommand(input) {
     if (input.message.timeoutMs) {
       timeout = setTimeout(() => {
         timedOut = true;
+        closeProcessStdin(activeProcesses.get(input.message.runId));
         void terminateProcessTree(subprocess).catch((error) => {
           process.stderr.write(
             `[r5d-worker] failed to terminate timed-out process ${input.message.runId}: ${error instanceof Error ? error.message : String(error)}
@@ -2027,10 +2033,21 @@ async function executeStreamingCommand(input) {
     if (timeout) {
       clearTimeout(timeout);
     }
+    closeProcessStdin(activeProcesses.get(input.message.runId));
     activeProcesses.delete(input.message.runId);
     cancelledProcessRuns.delete(input.message.runId);
   }
 }
+function closeProcessStdin(active) {
+  if (!active?.stdin) {
+    return;
+  }
+  try {
+    void active.stdin.end();
+  } catch {
+  }
+  active.stdin = void 0;
+}
 function sendWorkerMessage(ws, message) {
   const target = currentWorkerSocket ?? ws;
   try {
@@ -2054,7 +2071,8 @@ function buildActiveProcessReports() {
     argv: active.argv,
     command: active.command,
     ...active.cwd ? { cwd: active.cwd } : {},
-    startedAt: active.startedAt
+    startedAt: active.startedAt,
+    ...active.interactive ? { interactive: true } : {}
   }));
 }
 function sendActiveProcessReport(ws) {
@@ -2860,7 +2878,8 @@ async function startWorker(options) {
           browserPortForwarding: true,
           globalWorkspaceSyncLeaseV1: true,
           eventualWorkspaceSyncV1: true,
-          workspaceIncidentCandidatePinV1: true
+          workspaceIncidentCandidatePinV1: true,
+          execStdinV1: true
         },
         projectRoot: projectsRoot,
         artifactRoot,
@@ -3303,6 +3322,7 @@ async function startWorker(options) {
         let cancelMessage;
         if (active) {
           try {
+            closeProcessStdin(active);
             await terminateProcessTree(active.process);
             cancelMessage = `Stopped ${active.command}`;
           } catch (error) {
@@ -3324,6 +3344,51 @@ async function startWorker(options) {
         );
         return;
       }
+      if (message.type === "exec_stdin") {
+        const sendAck = (payload) => ws.send(
+          JSON.stringify({
+            type: "operation_result",
+            requestId: message.requestId,
+            ...payload
+          })
+        );
+        const active = activeProcesses.get(message.runId);
+        if (!active) {
+          sendAck({ error: `Process run ${message.runId} is not active on this worker` });
+          return;
+        }
+        if (!active.interactive || !active.stdin) {
+          sendAck({
+            error: `Process run ${message.runId} has no open stdin. Start a new shell command with "interactive": true to write to its stdin.`
+          });
+          return;
+        }
+        try {
+          let bytesWritten = 0;
+          if (message.data !== void 0 && message.data.length > 0) {
+            bytesWritten = await active.stdin.write(message.data);
+            await active.stdin.flush();
+          }
+          if (message.eof === true) {
+            await active.stdin.end();
+            active.stdin = void 0;
+          }
+          if (targetMayMutateVisibleWorkspace(active.target)) {
+            markWorkspaceDirty({ type: "shell_inline", detail: `exec stdin ${message.runId}` });
+          }
+          sendAck({
+            result: {
+              type: "exec_stdin",
+              runId: message.runId,
+              bytesWritten,
+              eof: message.eof === true
+            }
+          });
+        } catch (error) {
+          sendAck({ error: error instanceof Error ? error.message : String(error) });
+        }
+        return;
+      }
       if (message.type === "pty_open") {
         if (cliUpdateInProgress) {
           sendWorkerMessage(ws, {

package/dist/mjs/package.json

@@ -1,5 +1,5 @@
 {
   "name": "@ricsam/r5d-worker",
-  "version": "0.0.67",
+  "version": "0.0.68",
   "type": "module"
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ricsam/r5d-worker",
-  "version": "0.0.67",
+  "version": "0.0.68",
   "type": "module",
   "main": "./dist/cjs/main.cjs",
   "module": "./dist/mjs/main.mjs",