@bytetrue/byspace 0.15.0 → 0.16.0

package/node_modules/@bytetrue/client/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bytetrue/client",
-  "version": "0.15.0",
+  "version": "0.16.0",
   "description": "BySpace client SDK package",
   "files": [
     "dist",
@@ -39,8 +39,8 @@
     "test": "vitest run"
   },
   "dependencies": {
-    "@bytetrue/protocol": "0.15.0",
-    "@bytetrue/relay": "0.15.0"
+    "@bytetrue/protocol": "0.16.0",
+    "@bytetrue/relay": "0.16.0"
   },
   "devDependencies": {
     "@types/node": "^20.9.0",

package/node_modules/@bytetrue/highlight/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bytetrue/highlight",
-  "version": "0.15.0",
+  "version": "0.16.0",
   "files": [
     "dist",
     "!dist/**/*.map",

package/node_modules/@bytetrue/protocol/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bytetrue/protocol",
-  "version": "0.15.0",
+  "version": "0.16.0",
   "description": "BySpace shared protocol schemas and wire types",
   "files": [
     "dist",

package/node_modules/@bytetrue/relay/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bytetrue/relay",
-  "version": "0.15.0",
+  "version": "0.16.0",
   "description": "BySpace relay for bridging daemon and client connections",
   "files": [
     "dist",

package/node_modules/@bytetrue/server/dist/server/server/byspace-env.d.ts

@@ -1,3 +1,4 @@
+export declare const PI_TERMINAL_HOOK_OWNER_PID_ENV_KEY = "BYSPACE_PI_TERMINAL_HOOK_OWNER_PID";
 export type BySpaceNodeEnv = "development" | "production" | "test";
 export type ProcessEnvRecord = Record<string, string | undefined>;
 export type ExternalProcessEnv = NodeJS.ProcessEnv & Record<string, string>;

package/node_modules/@bytetrue/server/dist/server/server/byspace-env.js

@@ -1,5 +1,7 @@
 const BYSPACE_NODE_ENV = "BYSPACE_NODE_ENV";
 const ELECTRON_RUN_AS_NODE = "ELECTRON_RUN_AS_NODE";
+// Shared with the installed Pi extension (which cannot import from here).
+export const PI_TERMINAL_HOOK_OWNER_PID_ENV_KEY = "BYSPACE_PI_TERMINAL_HOOK_OWNER_PID";
 const RUNTIME_CONTROL_ENV_KEYS = [
     BYSPACE_NODE_ENV,
     "BYSPACE_DESKTOP_MANAGED",
@@ -7,6 +9,13 @@ const RUNTIME_CONTROL_ENV_KEYS = [
     ELECTRON_RUN_AS_NODE,
     "ELECTRON_NO_ATTACH_CONSOLE",
     "ESBUILD_BINARY_PATH",
+    // Written into `process.env` by the Pi terminal-activity extension so child Pi
+    // processes do not register a second reporter for the same terminal. If it
+    // survives into a fresh terminal or agent, that Pi process sees a marker whose
+    // value is a stale pid, concludes it is an inherited child, and registers no
+    // handlers at all — the terminal then never reports running/idle. It is
+    // per-process state, never inherited. See docs/terminal-activity.md.
+    PI_TERMINAL_HOOK_OWNER_PID_ENV_KEY,
 ];
 function buildInternalProcessEnv(baseEnv) {
     return { ...baseEnv };

package/node_modules/@bytetrue/server/dist/server/terminal/agent-hooks/pi/pi-extension.d.ts

@@ -1,2 +1,2 @@
-export declare const PI_TERMINAL_ACTIVITY_EXTENSION_SOURCE = "// byspace.pi-terminal-activity\nconst ownerPidKey = \"BYSPACE_PI_TERMINAL_HOOK_OWNER_PID\";\n\n// Keep one request in flight and only the latest state waiting behind it.\nexport function createReportQueue(sendReport) {\n  let pendingReport;\n  let reporting;\n\n  async function flushReports() {\n    while (pendingReport) {\n      const next = pendingReport;\n      pendingReport = undefined;\n      try {\n        await sendReport(next);\n      } catch {}\n    }\n    reporting = undefined;\n  }\n\n  return {\n    enqueue(next) {\n      pendingReport = next;\n      reporting ??= Promise.resolve().then(flushReports);\n      return reporting;\n    },\n    wait() {\n      return reporting ?? Promise.resolve();\n    },\n  };\n}\n\nasync function sendReport(next) {\n  await fetch(next.url, {\n    method: \"POST\",\n    headers: { \"content-type\": \"application/json\" },\n    body: JSON.stringify({\n      terminalId: next.terminalId,\n      token: next.token,\n      state: next.state,\n    }),\n    signal: AbortSignal.timeout(1_000),\n  });\n}\n\nconst reportQueue = createReportQueue(sendReport);\n\nfunction report(state) {\n  const url =\n    process.env.BYSPACE_TERMINAL_ACTIVITY_URL;\n  const terminalId = process.env.BYSPACE_TERMINAL_ID;\n  const token = process.env.BYSPACE_ACTIVITY_TOKEN;\n  if (!url || !terminalId || !token) return reportQueue.wait();\n\n  return reportQueue.enqueue({ url, terminalId, token, state });\n}\n\nexport default function byspaceTerminalActivity(pi) {\n  const ownerPid = process.env[ownerPidKey];\n  if (ownerPid && ownerPid !== String(process.pid)) return;\n  process.env[ownerPidKey] = String(process.pid);\n\n  pi.on(\"agent_start\", () => void report(\"running\"));\n  pi.on(\"ui_prompt_start\", () => void report(\"needs-input\"));\n  pi.on(\"ui_prompt_end\", () => void report(\"running\"));\n  pi.on(\"agent_settled\", () => void report(\"idle\"));\n  pi.on(\"session_shutdown\", () => report(\"idle\"));\n}\n";
+export declare const PI_TERMINAL_ACTIVITY_EXTENSION_SOURCE = "// byspace.pi-terminal-activity\nconst ownerPidKey = \"BYSPACE_PI_TERMINAL_HOOK_OWNER_PID\";\n\n// Keep one request in flight and only the latest state waiting behind it.\nexport function createReportQueue(sendReport) {\n  let pendingReport;\n  let reporting;\n\n  async function flushReports() {\n    while (pendingReport) {\n      const next = pendingReport;\n      pendingReport = undefined;\n      try {\n        await sendReport(next);\n      } catch {}\n    }\n    reporting = undefined;\n  }\n\n  return {\n    enqueue(next) {\n      pendingReport = next;\n      reporting ??= Promise.resolve().then(flushReports);\n      return reporting;\n    },\n    wait() {\n      return reporting ?? Promise.resolve();\n    },\n  };\n}\n\nasync function sendReport(next) {\n  await fetch(next.url, {\n    method: \"POST\",\n    headers: { \"content-type\": \"application/json\" },\n    body: JSON.stringify({\n      terminalId: next.terminalId,\n      token: next.token,\n      state: next.state,\n    }),\n    signal: AbortSignal.timeout(1_000),\n  });\n}\n\nconst reportQueue = createReportQueue(sendReport);\n\n// The marker is inherited by Pi child processes so they do not register a second\n// reporter for the same terminal. A marker left by a process that is gone is\n// stale, not an owner: treating it as an owner makes this Pi report nothing at\n// all, which looks exactly like \"the terminal stopped showing running\".\nfunction ownerIsLive(ownerPid) {\n  const pid = Number(ownerPid);\n  if (!Number.isInteger(pid) || pid <= 0) return false;\n  try {\n    process.kill(pid, 0);\n    return true;\n  } catch (error) {\n    return error?.code === \"EPERM\";\n  }\n}\n\nfunction report(state) {\n  const url =\n    process.env.BYSPACE_TERMINAL_ACTIVITY_URL;\n  const terminalId = process.env.BYSPACE_TERMINAL_ID;\n  const token = process.env.BYSPACE_ACTIVITY_TOKEN;\n  if (!url || !terminalId || !token) return reportQueue.wait();\n\n  return reportQueue.enqueue({ url, terminalId, token, state });\n}\n\nexport default function byspaceTerminalActivity(pi) {\n  const ownerPid = process.env[ownerPidKey];\n  if (ownerPid && ownerPid !== String(process.pid) && ownerIsLive(ownerPid)) return;\n  process.env[ownerPidKey] = String(process.pid);\n\n  pi.on(\"agent_start\", () => void report(\"running\"));\n  pi.on(\"ui_prompt_start\", () => void report(\"needs-input\"));\n  pi.on(\"ui_prompt_end\", () => void report(\"running\"));\n  pi.on(\"agent_settled\", () => void report(\"idle\"));\n  pi.on(\"session_shutdown\", () => report(\"idle\"));\n}\n";
 //# sourceMappingURL=pi-extension.d.ts.map

package/node_modules/@bytetrue/server/dist/server/terminal/agent-hooks/pi/pi-extension.js

@@ -44,6 +44,21 @@ async function sendReport(next) {
 
 const reportQueue = createReportQueue(sendReport);
 
+// The marker is inherited by Pi child processes so they do not register a second
+// reporter for the same terminal. A marker left by a process that is gone is
+// stale, not an owner: treating it as an owner makes this Pi report nothing at
+// all, which looks exactly like "the terminal stopped showing running".
+function ownerIsLive(ownerPid) {
+  const pid = Number(ownerPid);
+  if (!Number.isInteger(pid) || pid <= 0) return false;
+  try {
+    process.kill(pid, 0);
+    return true;
+  } catch (error) {
+    return error?.code === "EPERM";
+  }
+}
+
 function report(state) {
   const url =
     process.env.BYSPACE_TERMINAL_ACTIVITY_URL;
@@ -56,7 +71,7 @@ function report(state) {
 
 export default function byspaceTerminalActivity(pi) {
   const ownerPid = process.env[ownerPidKey];
-  if (ownerPid && ownerPid !== String(process.pid)) return;
+  if (ownerPid && ownerPid !== String(process.pid) && ownerIsLive(ownerPid)) return;
   process.env[ownerPidKey] = String(process.pid);
 
   pi.on("agent_start", () => void report("running"));

package/node_modules/@bytetrue/server/dist/server/terminal/terminal.js

@@ -786,6 +786,12 @@ export async function createTerminal(options) {
         if (!commandFinished) {
             return true;
         }
+        // The interactive shell only emits this when its foreground job ends. A
+        // terminal agent that exits without reporting idle (SIGKILL, crash, a
+        // suspend that never resumes) leaves `working` set forever, so treat the
+        // shell regaining the prompt as proof that the agent is gone. `interrupt`
+        // only clears `working`; a finished-attention idle state is untouched.
+        activityTracker.interrupt();
         for (const listener of Array.from(commandFinishedListeners)) {
             try {
                 listener(commandFinished);