chatroom-cli 1.2.7 → 1.4.0

package/dist/index.js

@@ -10280,10 +10280,6 @@ function ensureMachineRegistered() {
     harnessVersions: config.harnessVersions
   };
 }
-function getMachineId() {
-  const config = loadMachineConfig();
-  return config?.machineId ?? null;
-}
 var CHATROOM_DIR2, MACHINE_FILE = "machine.json";
 var init_storage2 = __esm(() => {
   init_detection();
@@ -10756,6 +10752,9 @@ var init_pi_agent_service = __esm(() => {
       const pid = childProcess.pid;
       const context = options.context;
       const entry = this.registerProcess(pid, context);
+      const roleTag = context.role ?? "unknown";
+      const chatroomSuffix = context.chatroomId ? `@${context.chatroomId.slice(-6)}` : "";
+      const logPrefix = `[pi:${roleTag}${chatroomSuffix}`;
       const outputCallbacks = [];
       if (childProcess.stdout) {
         const reader = new PiRpcReader(childProcess.stdout);
@@ -10767,7 +10766,7 @@ var init_pi_agent_service = __esm(() => {
           for (const line of textBuffer.split(`
 `)) {
             if (line)
-              process.stdout.write(`[pi text] ${line}
+              process.stdout.write(`${logPrefix} text] ${line}
 `);
           }
           textBuffer = "";
@@ -10778,7 +10777,7 @@ var init_pi_agent_service = __esm(() => {
           for (const line of thinkingBuffer.split(`
 `)) {
             if (line)
-              process.stdout.write(`[pi thinking] ${line}
+              process.stdout.write(`${logPrefix} thinking] ${line}
 `);
           }
           thinkingBuffer = "";
@@ -10811,15 +10810,38 @@ var init_pi_agent_service = __esm(() => {
         reader.onAgentEnd(() => {
           flushText();
           flushThinking();
-          process.stdout.write(`[pi agent_end]
+          process.stdout.write(`${logPrefix} agent_end]
 `);
         });
         reader.onToolCall((name) => {
           flushText();
           flushThinking();
-          process.stdout.write(`[pi tool: ${name}]
+          process.stdout.write(`${logPrefix} tool: ${name}]
 `);
         });
+        if (childProcess.stderr) {
+          childProcess.stderr.pipe(process.stderr, { end: false });
+          childProcess.stderr.on("data", () => {
+            entry.lastOutputAt = Date.now();
+            for (const cb of outputCallbacks)
+              cb();
+          });
+        }
+        return {
+          pid,
+          onExit: (cb) => {
+            childProcess.on("exit", (code2, signal) => {
+              this.deleteProcess(pid);
+              cb({ code: code2, signal, context });
+            });
+          },
+          onOutput: (cb) => {
+            outputCallbacks.push(cb);
+          },
+          onAgentEnd: (cb) => {
+            reader.onAgentEnd(cb);
+          }
+        };
       }
       if (childProcess.stderr) {
         childProcess.stderr.pipe(process.stderr, { end: false });
@@ -11298,6 +11320,15 @@ async function registerAgent(chatroomId, options, deps) {
         harnessVersions: machineInfo.harnessVersions,
         availableModels
       });
+      try {
+        await d.backend.mutation(api.machines.recordAgentRegistered, {
+          sessionId,
+          chatroomId,
+          role,
+          agentType: "remote",
+          machineId: machineInfo.machineId
+        });
+      } catch {}
       console.log(`✅ Registered as remote agent for role "${role}"`);
       console.log(`   Machine: ${machineInfo.hostname} (${machineInfo.machineId})`);
       console.log(`   Working directory: ${process.cwd()}`);
@@ -11387,24 +11418,23 @@ Two requirements:
 
 ⚠️  WHEN THE PROCESS IS TERMINATED OR TIMED OUT
 
-\`\`\`
-@startuml
-start
-:Command terminated unexpectedly;
-if (Urgent pending work?) then (yes)
-  :Finish urgent work;
-  :Reconnect with get-next-task;
-else (no)
-  :Reconnect immediately;
-  note right: Team cannot reach you without it
-endif
-stop
-@enduml
+\`\`\`mermaid
+flowchart TD
+    A([Start]) --> B[Command terminated unexpectedly]
+    B --> C{Urgent pending work?}
+    C -->|yes| D[Finish urgent work]
+    D --> E[Reconnect with get-next-task]
+    C -->|no| E
+    E --> F([Stop])
 \`\`\`
 
 \uD83D\uDCCB BACKLOG TASKS
   chatroom backlog list --chatroom-id=<chatroomId> --role=<role> --status=backlog
-  chatroom backlog --help`;
+  chatroom backlog --help
+
+\uD83D\uDCCB CONTEXT RECOVERY (after compaction/summarization)
+  If your context was compacted, run: chatroom get-system-prompt --chatroom-id=<id> --role=<role>
+  to reload your full system and role prompt.`;
 }
 var init_reminder = () => {};
 // ../../services/backend/prompts/config/index.ts
@@ -11454,6 +11484,7 @@ var init_utils = __esm(() => {
 // ../../services/backend/prompts/base/shared/getting-started-content.ts
 var init_getting_started_content = __esm(() => {
   init_utils();
+  init_reminder();
 });
 
 // ../../services/backend/prompts/cli/index.ts
@@ -11791,18 +11822,6 @@ async function getNextTask(chatroomId, options) {
       harnessVersions: machineInfo.harnessVersions,
       availableModels
     });
-    const agentType = options.agentType ?? (machineInfo.availableHarnesses.length > 0 ? machineInfo.availableHarnesses[0] : undefined);
-    if (agentType) {
-      const workingDir = process.cwd();
-      await client2.mutation(api.machines.updateAgentConfig, {
-        sessionId,
-        machineId: machineInfo.machineId,
-        chatroomId,
-        role,
-        agentType,
-        workingDir
-      });
-    }
   } catch (machineError) {
     if (!silent) {
       console.warn(`⚠️  Machine registration failed: ${sanitizeUnknownForTerminal(machineError.message)}`);
@@ -12484,7 +12503,6 @@ async function listBacklog(chatroomId, options, deps) {
     "pending",
     "acknowledged",
     "in_progress",
-    "queued",
     "backlog",
     "backlog_acknowledged",
     "completed",
@@ -12845,8 +12863,6 @@ function getStatusEmoji(status) {
       return "\uD83D\uDCEC";
     case "in_progress":
       return "\uD83D\uDD35";
-    case "queued":
-      return "\uD83D\uDFE1";
     case "backlog":
       return "⚪";
     case "backlog_acknowledged":
@@ -13791,6 +13807,17 @@ var init_on_daemon_shutdown = __esm(() => {
   init_api3();
 });
 
+// src/infrastructure/machine/stop-reason.ts
+function resolveStopReason(code2, signal, wasIntentional) {
+  if (wasIntentional)
+    return "intentional_stop";
+  if (signal !== null)
+    return "process_terminated_with_signal";
+  if (code2 === 0)
+    return "process_exited_with_success";
+  return "process_terminated_unexpectedly";
+}
+
 // src/commands/machine/daemon-start/handlers/start-agent.ts
 async function executeStartAgent(ctx, args) {
   const { chatroomId, role, agentHarness, model, workingDir, reason } = args;
@@ -13905,15 +13932,24 @@ async function executeStartAgent(ctx, args) {
   });
   spawnResult.onExit(({ code: code2, signal }) => {
     const wasIntentional = ctx.deps.stops.consume(chatroomId, role);
+    const stopReason = resolveStopReason(code2, signal, wasIntentional);
     ctx.events.emit("agent:exited", {
       chatroomId,
       role,
       pid,
       code: code2,
       signal,
+      stopReason,
       intentional: wasIntentional
     });
   });
+  if (spawnResult.onAgentEnd) {
+    spawnResult.onAgentEnd(() => {
+      try {
+        ctx.deps.processes.kill(-pid, "SIGTERM");
+      } catch {}
+    });
+  }
   let lastReportedTokenAt = 0;
   spawnResult.onOutput(() => {
     const now = Date.now();
@@ -14166,6 +14202,9 @@ async function refreshModels(ctx) {
   }
   if (!ctx.config)
     return;
+  const freshConfig = ensureMachineRegistered();
+  ctx.config.availableHarnesses = freshConfig.availableHarnesses;
+  ctx.config.harnessVersions = freshConfig.harnessVersions;
   const totalCount = Object.values(models).flat().length;
   try {
     await ctx.deps.backend.mutation(api.machines.register, {
@@ -14182,6 +14221,41 @@ async function refreshModels(ctx) {
     console.warn(`[${formatTimestamp()}] ⚠️  Model refresh failed: ${error.message}`);
   }
 }
+function evictStaleDedupEntries(processedCommandIds, processedPingIds) {
+  const evictBefore = Date.now() - AGENT_REQUEST_DEADLINE_MS;
+  for (const [id, ts] of processedCommandIds) {
+    if (ts < evictBefore)
+      processedCommandIds.delete(id);
+  }
+  for (const [id, ts] of processedPingIds) {
+    if (ts < evictBefore)
+      processedPingIds.delete(id);
+  }
+}
+async function dispatchCommandEvent(ctx, event, processedCommandIds, processedPingIds) {
+  const eventId = event._id.toString();
+  if (event.type === "agent.requestStart") {
+    if (processedCommandIds.has(eventId))
+      return;
+    processedCommandIds.set(eventId, Date.now());
+    await onRequestStartAgent(ctx, event);
+  } else if (event.type === "agent.requestStop") {
+    if (processedCommandIds.has(eventId))
+      return;
+    processedCommandIds.set(eventId, Date.now());
+    await onRequestStopAgent(ctx, event);
+  } else if (event.type === "daemon.ping") {
+    if (processedPingIds.has(eventId))
+      return;
+    processedPingIds.set(eventId, Date.now());
+    handlePing();
+    await ctx.deps.backend.mutation(api.machines.ackPing, {
+      sessionId: ctx.sessionId,
+      machineId: ctx.machineId,
+      pingEventId: event._id
+    });
+  }
+}
 async function startCommandLoop(ctx) {
   let heartbeatCount = 0;
   const heartbeatTimer = setInterval(() => {
@@ -14220,40 +14294,11 @@ Listening for commands...`);
   }, async (result) => {
     if (!result.events || result.events.length === 0)
       return;
-    const evictBefore = Date.now() - AGENT_REQUEST_DEADLINE_MS;
-    for (const [id, ts] of processedCommandIds) {
-      if (ts < evictBefore)
-        processedCommandIds.delete(id);
-    }
-    for (const [id, ts] of processedPingIds) {
-      if (ts < evictBefore)
-        processedPingIds.delete(id);
-    }
+    evictStaleDedupEntries(processedCommandIds, processedPingIds);
     for (const event of result.events) {
-      const eventId = event._id.toString();
       try {
         console.log(`[${formatTimestamp()}] \uD83D\uDCE1 Stream command event: ${event.type}`);
-        if (event.type === "agent.requestStart") {
-          if (processedCommandIds.has(eventId))
-            continue;
-          processedCommandIds.set(eventId, Date.now());
-          await onRequestStartAgent(ctx, event);
-        } else if (event.type === "agent.requestStop") {
-          if (processedCommandIds.has(eventId))
-            continue;
-          processedCommandIds.set(eventId, Date.now());
-          await onRequestStopAgent(ctx, event);
-        } else if (event.type === "daemon.ping") {
-          if (processedPingIds.has(eventId))
-            continue;
-          processedPingIds.set(eventId, Date.now());
-          handlePing();
-          await ctx.deps.backend.mutation(api.machines.ackPing, {
-            sessionId: ctx.sessionId,
-            machineId: ctx.machineId,
-            pingEventId: event._id
-          });
-        }
+        await dispatchCommandEvent(ctx, event, processedCommandIds, processedPingIds);
       } catch (err) {
         console.error(`[${formatTimestamp()}] ❌ Stream command event failed: ${err.message}`);
       }
@@ -14271,6 +14316,7 @@ var MODEL_REFRESH_INTERVAL_MS;
 var init_command_loop = __esm(() => {
   init_api3();
   init_client2();
+  init_machine();
   init_on_daemon_shutdown();
   init_on_request_start_agent();
   init_on_request_stop_agent();
@@ -14337,8 +14383,9 @@ class DaemonEventBus {
 
 // src/events/daemon/agent/on-agent-exited.ts
 function onAgentExited(ctx, payload) {
-  const { chatroomId, role, pid, code: code2, signal, intentional } = payload;
+  const { chatroomId, role, pid, code: code2, signal, stopReason, intentional } = payload;
   const ts = formatTimestamp();
+  console.log(`[${ts}] Agent stopped: ${stopReason} (${role})`);
   if (intentional) {
     console.log(`[${ts}] ℹ️  Agent process exited after intentional stop ` + `(PID: ${pid}, role: ${role}, code: ${code2}, signal: ${signal})`);
   } else {
@@ -14351,6 +14398,8 @@ function onAgentExited(ctx, payload) {
     role,
     pid,
     intentional,
+    stopReason,
+    stopSignal: stopReason === "process_terminated_with_signal" ? signal ?? undefined : undefined,
     exitCode: code2 ?? undefined,
     signal: signal ?? undefined
   }).catch((err) => {
@@ -14446,11 +14495,7 @@ function createDefaultDeps16() {
     }
   };
 }
-async function initDaemon() {
-  if (!acquireLock()) {
-    process.exit(1);
-  }
-  const convexUrl = getConvexUrl();
+function validateAuthentication(convexUrl) {
   const sessionId = getSessionId();
   if (!sessionId) {
     const otherUrls = getOtherSessionUrls();
@@ -14467,18 +14512,10 @@ Run: chatroom auth login`);
     releaseLock();
     process.exit(1);
   }
-  const machineId = getMachineId();
-  if (!machineId) {
-    console.error(`❌ Machine not registered`);
-    console.error(`
-Run any chatroom command first to register this machine,`);
-    console.error(`for example: chatroom auth status`);
-    releaseLock();
-    process.exit(1);
-  }
-  const client2 = await getConvexClient();
-  const typedSessionId = sessionId;
-  const validation = await client2.query(api.cliAuth.validateSession, { sessionId: typedSessionId });
+  return sessionId;
+}
+async function validateSession(client2, sessionId, convexUrl) {
+  const validation = await client2.query(api.cliAuth.validateSession, { sessionId });
   if (!validation.valid) {
     console.error(`❌ Session invalid: ${validation.reason}`);
     console.error(`
@@ -14486,32 +14523,34 @@ Run: chatroom auth login`);
     releaseLock();
     process.exit(1);
   }
+}
+function setupMachine() {
+  ensureMachineRegistered();
   const config3 = loadMachineConfig();
-  const openCodeService = new OpenCodeAgentService;
-  const piService = new PiAgentService;
-  const agentServices = new Map([
-    ["opencode", openCodeService],
-    ["pi", piService]
-  ]);
+  return config3;
+}
+async function registerCapabilities(client2, sessionId, config3, agentServices) {
+  const { machineId } = config3;
   const availableModels = await discoverModels(agentServices);
-  if (config3) {
-    try {
-      await client2.mutation(api.machines.register, {
-        sessionId: typedSessionId,
-        machineId,
-        hostname: config3.hostname,
-        os: config3.os,
-        availableHarnesses: config3.availableHarnesses,
-        harnessVersions: config3.harnessVersions,
-        availableModels
-      });
-    } catch (error) {
-      console.warn(`⚠️  Machine registration update failed: ${error.message}`);
-    }
+  try {
+    await client2.mutation(api.machines.register, {
+      sessionId,
+      machineId,
+      hostname: config3.hostname,
+      os: config3.os,
+      availableHarnesses: config3.availableHarnesses,
+      harnessVersions: config3.harnessVersions,
+      availableModels
+    });
+  } catch (error) {
+    console.warn(`⚠️  Machine registration update failed: ${error.message}`);
   }
+  return availableModels;
+}
+async function connectDaemon(client2, sessionId, machineId, convexUrl) {
   try {
     await client2.mutation(api.machines.updateDaemonStatus, {
-      sessionId: typedSessionId,
+      sessionId,
       machineId,
       connected: true
     });
@@ -14524,6 +14563,45 @@ Run: chatroom auth login`);
     releaseLock();
     process.exit(1);
   }
+}
+function logStartup(ctx, availableModels) {
+  console.log(`[${formatTimestamp()}] \uD83D\uDE80 Daemon started`);
+  console.log(`   CLI version: ${getVersion()}`);
+  console.log(`   Machine ID: ${ctx.machineId}`);
+  console.log(`   Hostname: ${ctx.config?.hostname ?? "unknown"}`);
+  console.log(`   Available harnesses: ${ctx.config?.availableHarnesses.join(", ") || "none"}`);
+  console.log(`   Available models: ${Object.keys(availableModels).length > 0 ? `${Object.values(availableModels).flat().length} models across ${Object.keys(availableModels).join(", ")}` : "none discovered"}`);
+  console.log(`   PID: ${process.pid}`);
+}
+async function recoverState(ctx) {
+  console.log(`
+[${formatTimestamp()}] \uD83D\uDD04 Recovering agent state...`);
+  try {
+    await recoverAgentState(ctx);
+  } catch (e) {
+    console.log(`   ⚠️  Recovery failed: ${e.message}`);
+    console.log(`   Continuing with fresh state`);
+  }
+}
+async function initDaemon() {
+  if (!acquireLock()) {
+    process.exit(1);
+  }
+  const convexUrl = getConvexUrl();
+  const sessionId = validateAuthentication(convexUrl);
+  const client2 = await getConvexClient();
+  const typedSessionId = sessionId;
+  await validateSession(client2, typedSessionId, convexUrl);
+  const config3 = setupMachine();
+  const { machineId } = config3;
+  const openCodeService = new OpenCodeAgentService;
+  const piService = new PiAgentService;
+  const agentServices = new Map([
+    ["opencode", openCodeService],
+    ["pi", piService]
+  ]);
+  const availableModels = await registerCapabilities(client2, typedSessionId, config3, agentServices);
+  await connectDaemon(client2, typedSessionId, machineId, convexUrl);
   const deps = createDefaultDeps16();
   deps.backend.mutation = (endpoint, args) => client2.mutation(endpoint, args);
   deps.backend.query = (endpoint, args) => client2.query(endpoint, args);
@@ -14538,21 +14616,8 @@ Run: chatroom auth login`);
     agentServices
   };
   registerEventListeners(ctx);
-  console.log(`[${formatTimestamp()}] \uD83D\uDE80 Daemon started`);
-  console.log(`   CLI version: ${getVersion()}`);
-  console.log(`   Machine ID: ${machineId}`);
-  console.log(`   Hostname: ${config3?.hostname ?? "Unknown"}`);
-  console.log(`   Available harnesses: ${config3?.availableHarnesses.join(", ") || "none"}`);
-  console.log(`   Available models: ${Object.keys(availableModels).length > 0 ? `${Object.values(availableModels).flat().length} models across ${Object.keys(availableModels).join(", ")}` : "none discovered"}`);
-  console.log(`   PID: ${process.pid}`);
-  console.log(`
-[${formatTimestamp()}] \uD83D\uDD04 Recovering agent state...`);
-  try {
-    await recoverAgentState(ctx);
-  } catch (e) {
-    console.log(`   ⚠️  Recovery failed: ${e.message}`);
-    console.log(`   Continuing with fresh state`);
-  }
+  logStartup(ctx, availableModels);
+  await recoverState(ctx);
   return ctx;
 }
 var init_init2 = __esm(() => {
@@ -15292,7 +15357,7 @@ program2.command("report-progress").description("Report progress on current task
   });
 });
 var backlogCommand = program2.command("backlog").description("Manage task queue and backlog");
-backlogCommand.command("list").description("List tasks in a chatroom").requiredOption("--chatroom-id <id>", "Chatroom identifier").requiredOption("--role <role>", "Your role").requiredOption("--status <status>", "Filter by status (pending|in_progress|queued|backlog|completed|cancelled|active|pending_review|archived|all)").option("--limit <n>", "Maximum number of tasks to show (required for --status=all)").option("--full", "Show full task content without truncation").action(async (options) => {
+backlogCommand.command("list").description("List tasks in a chatroom").requiredOption("--chatroom-id <id>", "Chatroom identifier").requiredOption("--role <role>", "Your role").requiredOption("--status <status>", "Filter by status (pending|in_progress|backlog|completed|cancelled|active|pending_review|archived|all)").option("--limit <n>", "Maximum number of tasks to show (required for --status=all)").option("--full", "Show full task content without truncation").action(async (options) => {
   if (options.status === "all" && !options.limit) {
     console.error("❌ When using --status=all, you must specify --limit=<n>");
     console.error("   Example: chatroom backlog list --chatroom-id=<id> --role=builder --status=all --limit=50");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "chatroom-cli",
-  "version": "1.2.7",
+  "version": "1.4.0",
   "description": "CLI for multi-agent chatroom collaboration",
   "type": "module",
   "bin": {