chatroom-cli 1.13.2 → 1.14.0

package/dist/index.js

@@ -10227,7 +10227,7 @@ class BaseCLIAgentService {
   }
   checkVersion(command) {
     try {
-      const output = this.deps.execSync(`${command} --version`, {
+      const output = this.deps.execSync(`${command} --version 2>&1`, {
         stdio: ["pipe", "pipe", "pipe"],
         timeout: 5000
       }).toString().trim();
@@ -10500,7 +10500,7 @@ var init_pi_agent_service = __esm(() => {
     }
     async listModels() {
       try {
-        const output = this.deps.execSync(`${PI_COMMAND} --list-models`, {
+        const output = this.deps.execSync(`${PI_COMMAND} --list-models 2>&1`, {
           stdio: ["pipe", "pipe", "pipe"],
           timeout: 1e4
         }).toString().trim();
@@ -12552,7 +12552,7 @@ async function createDefaultDeps8() {
 }
 async function handoff(chatroomId, options, deps) {
   const d = deps ?? await createDefaultDeps8();
-  const { role, message, nextRole, attachedArtifactIds = [] } = options;
+  const { role, message, nextRole, attachedArtifactIds = [], attachedWorkflowKeys = [] } = options;
   const sessionId = d.session.getSessionId();
   if (!sessionId) {
     const otherUrls = d.session.getOtherSessionUrls();
@@ -12582,6 +12582,22 @@ async function handoff(chatroomId, options, deps) {
       process.exit(1);
     }
   }
+  const resolvedWorkflowIds = [];
+  if (attachedWorkflowKeys.length > 0) {
+    for (const key of attachedWorkflowKeys) {
+      try {
+        const result2 = await d.backend.query(api.workflows.resolveWorkflowId, {
+          sessionId,
+          chatroomId,
+          workflowKey: key
+        });
+        resolvedWorkflowIds.push(result2.workflowId);
+      } catch {
+        formatError(`Workflow "${key}" not found`, ["Check the workflow key and try again"]);
+        process.exit(1);
+      }
+    }
+  }
   let result;
   try {
     result = await d.backend.mutation(api.messages.handoff, {
@@ -12592,6 +12608,9 @@ async function handoff(chatroomId, options, deps) {
       targetRole: nextRole,
       ...attachedArtifactIds.length > 0 && {
         attachedArtifactIds
+      },
+      ...resolvedWorkflowIds.length > 0 && {
+        attachedWorkflowIds: resolvedWorkflowIds
       }
     });
   } catch (error) {
@@ -12665,6 +12684,32 @@ ${error instanceof Error ? error.message : String(error)}`);
       console.log(`   • ${id}`);
     });
   }
+  if (resolvedWorkflowIds.length > 0) {
+    for (const wfId of resolvedWorkflowIds) {
+      try {
+        const detail = await d.backend.query(api.workflows.getWorkflowDetail, {
+          sessionId,
+          chatroomId,
+          workflowId: wfId
+        });
+        const wf = detail.workflow;
+        console.log("");
+        console.log(`\uD83D\uDCCA Attached Workflow: ${wf.workflowKey} (${wf.status}, ${detail.steps.length} steps)`);
+        for (let i2 = 0;i2 < detail.steps.length; i2++) {
+          const step = detail.steps[i2];
+          const isLast = i2 === detail.steps.length - 1;
+          const prefix = isLast ? "└─" : "├─";
+          const statusEmoji = step.status === "completed" ? "✅" : step.status === "in_progress" ? "\uD83D\uDD04" : step.status === "cancelled" ? "❌" : "⏳";
+          const roleLabel = step.assigneeRole ? ` [${step.assigneeRole}]` : "";
+          const deps2 = step.dependsOn.length > 0 ? ` (depends: ${step.dependsOn.join(", ")})` : "";
+          console.log(`   ${prefix} ${step.stepKey}${roleLabel} ${statusEmoji} ${step.status}${deps2}`);
+        }
+        console.log("");
+        console.log(`   Inspect: chatroom workflow status --chatroom-id=${chatroomId} --workflow-key=${wf.workflowKey}`);
+        console.log(`   View step: chatroom workflow step-view --chatroom-id=${chatroomId} --workflow-key=${wf.workflowKey} --step-key=<key>`);
+      } catch {}
+    }
+  }
   const convexUrl = d.session.getConvexUrl();
   const cliEnvPrefix = getCliEnvPrefix(convexUrl);
   console.log(`
@@ -13428,14 +13473,14 @@ var init_file_content = () => {};
 // src/commands/workflow/index.ts
 var exports_workflow = {};
 __export(exports_workflow, {
+  viewStep: () => viewStep,
   specifyWorkflowStep: () => specifyWorkflowStep,
   parseSections: () => parseSections,
   getWorkflowStatus: () => getWorkflowStatus,
   exitWorkflow: () => exitWorkflow,
   executeWorkflow: () => executeWorkflow,
   createWorkflow: () => createWorkflow,
-  completeStep: () => completeStep,
-  cancelStep: () => cancelStep
+  completeStep: () => completeStep
 });
 async function createDefaultDeps11() {
   const client2 = await getConvexClient();
@@ -13612,10 +13657,11 @@ async function specifyWorkflowStep(chatroomId, options, deps) {
   const d = deps ?? await createDefaultDeps11();
   const sessionId = requireAuth3(d);
   validateChatroomId2(chatroomId);
-  const sections = parseSections(options.stdinContent, ["GOAL", "REQUIREMENTS"], ["WARNINGS"]);
+  const sections = parseSections(options.stdinContent, ["GOAL", "REQUIREMENTS"], ["WARNINGS", "SKILLS"]);
   const goal = sections.get("GOAL");
   const requirements = sections.get("REQUIREMENTS");
   const warnings = sections.get("WARNINGS") || undefined;
+  const skills = sections.get("SKILLS") || undefined;
   try {
     await d.backend.mutation(api.workflows.specifyStep, {
       sessionId,
@@ -13625,7 +13671,8 @@ async function specifyWorkflowStep(chatroomId, options, deps) {
       assigneeRole: options.assigneeRole,
       goal,
       requirements,
-      warnings
+      warnings,
+      skills
     });
     console.log("");
     console.log("✅ Step specified");
@@ -13732,6 +13779,9 @@ async function getWorkflowStatus(chatroomId, options, deps) {
           if (spec.goal) {
             console.log(`   \uD83D\uDCCE Goal: ${spec.goal}`);
           }
+          if (spec.skills) {
+            console.log(`   \uD83D\uDD27 Skills: ${spec.skills}`);
+          }
           if (spec.requirements) {
             console.log(`   \uD83D\uDCCE Requirements: ${spec.requirements}`);
           }
@@ -13787,58 +13837,118 @@ async function completeStep(chatroomId, options, deps) {
     return;
   }
 }
-async function cancelStep(chatroomId, options, deps) {
+async function exitWorkflow(chatroomId, options, deps) {
   const d = deps ?? await createDefaultDeps11();
   const sessionId = requireAuth3(d);
   validateChatroomId2(chatroomId);
   if (!options.reason || options.reason.trim().length === 0) {
-    console.error("❌ Reason is required when cancelling a step");
+    console.error("❌ Reason is required when exiting a workflow");
     process.exit(1);
     return;
   }
   try {
-    await d.backend.mutation(api.workflows.cancelStep, {
+    await d.backend.mutation(api.workflows.exitWorkflow, {
       sessionId,
       chatroomId,
       workflowKey: options.workflowKey,
-      stepKey: options.stepKey,
       reason: options.reason.trim()
     });
     console.log("");
-    console.log("❌ Step cancelled");
-    console.log(`   Workflow: ${options.workflowKey}`);
-    console.log(`   Step: ${options.stepKey}`);
+    console.log("❌ Workflow exited (cancelled)");
+    console.log(`   Key: ${options.workflowKey}`);
     console.log(`   Reason: ${options.reason.trim()}`);
     console.log("");
   } catch (error) {
-    console.error(`❌ Failed to cancel step: ${error.message}`);
+    console.error(`❌ Failed to exit workflow: ${error.message}`);
     process.exit(1);
     return;
   }
 }
-async function exitWorkflow(chatroomId, options, deps) {
+async function viewStep(chatroomId, options, deps) {
   const d = deps ?? await createDefaultDeps11();
   const sessionId = requireAuth3(d);
   validateChatroomId2(chatroomId);
-  if (!options.reason || options.reason.trim().length === 0) {
-    console.error("❌ Reason is required when exiting a workflow");
-    process.exit(1);
-    return;
-  }
   try {
-    await d.backend.mutation(api.workflows.exitWorkflow, {
+    const result = await d.backend.query(api.workflows.getStepView, {
       sessionId,
       chatroomId,
       workflowKey: options.workflowKey,
-      reason: options.reason.trim()
+      stepKey: options.stepKey
     });
+    const step = result.step;
+    const emoji = getStepStatusEmoji(step.status);
     console.log("");
-    console.log("❌ Workflow exited (cancelled)");
-    console.log(`   Key: ${options.workflowKey}`);
-    console.log(`   Reason: ${options.reason.trim()}`);
+    console.log("══════════════════════════════════════════════════");
+    console.log(`${emoji} STEP: ${step.stepKey}`);
+    console.log("══════════════════════════════════════════════════");
+    console.log(`Workflow: ${result.workflowKey} (${result.workflowStatus})`);
+    console.log(`Description: ${step.description}`);
+    console.log(`Status: ${step.status.toUpperCase()}`);
+    if (step.assigneeRole) {
+      console.log(`Assignee: ${step.assigneeRole}`);
+    }
+    if (step.dependsOn.length > 0) {
+      console.log(`Dependencies: ${step.dependsOn.join(", ")}`);
+    }
+    console.log(`Order: ${step.order}`);
+    if (step.completedAt) {
+      const completedDate = new Date(step.completedAt).toLocaleString("en-US", {
+        month: "short",
+        day: "numeric",
+        hour: "2-digit",
+        minute: "2-digit",
+        hour12: false
+      });
+      console.log(`Completed: ${completedDate}`);
+    }
+    if (step.cancelledAt) {
+      const cancelledDate = new Date(step.cancelledAt).toLocaleString("en-US", {
+        month: "short",
+        day: "numeric",
+        hour: "2-digit",
+        minute: "2-digit",
+        hour12: false
+      });
+      console.log(`Cancelled: ${cancelledDate}`);
+      if (step.cancelReason) {
+        console.log(`Cancel reason: ${step.cancelReason}`);
+      }
+    }
+    if (step.specification) {
+      const spec = step.specification;
+      console.log("");
+      console.log("──────────────────────────────────────────────────");
+      console.log("\uD83D\uDCCB SPECIFICATION");
+      console.log("──────────────────────────────────────────────────");
+      if (spec.goal) {
+        console.log("");
+        console.log("Goal:");
+        console.log(spec.goal);
+      }
+      if (spec.skills) {
+        console.log("");
+        console.log("Skills:");
+        console.log(spec.skills);
+      }
+      if (spec.requirements) {
+        console.log("");
+        console.log("Requirements:");
+        console.log(spec.requirements);
+      }
+      if (spec.warnings) {
+        console.log("");
+        console.log("⚠️  Warnings:");
+        console.log(spec.warnings);
+      }
+    } else {
+      console.log("");
+      console.log("⚠️  No specification set. Run `workflow specify` to add one.");
+    }
+    console.log("");
+    console.log("══════════════════════════════════════════════════");
     console.log("");
   } catch (error) {
-    console.error(`❌ Failed to exit workflow: ${error.message}`);
+    console.error(`❌ Failed to view step: ${error.message}`);
     process.exit(1);
     return;
   }
@@ -14883,7 +14993,7 @@ var init_on_request_start_agent = __esm(() => {
 
 // src/commands/machine/daemon-start/handlers/stop-agent.ts
 async function executeStopAgent(ctx, args) {
-  const { chatroomId, role, reason } = args;
+  const { chatroomId, role, reason, pid } = args;
   console.log(`   ↪ stop-agent command received`);
   console.log(`      Chatroom: ${chatroomId}`);
   console.log(`      Role: ${role}`);
@@ -14891,7 +15001,8 @@ async function executeStopAgent(ctx, args) {
   const result = await ctx.deps.agentProcessManager.stop({
     chatroomId,
     role,
-    reason
+    reason,
+    pid
   });
   const msg = result.success ? `Agent stopped (${role})` : `Failed to stop agent (${role})`;
   console.log(`   ${result.success ? "✅" : "⚠️ "} ${msg}`);
@@ -14907,7 +15018,8 @@ async function onRequestStopAgent(ctx, event) {
   await executeStopAgent(ctx, {
     chatroomId: event.chatroomId,
     role: event.role,
-    reason: event.reason
+    reason: event.reason,
+    pid: event.pid
   });
 }
 var init_on_request_stop_agent = () => {};
@@ -15848,9 +15960,7 @@ var init_crash_loop_tracker = __esm(() => {
 });
 
 // src/infrastructure/machine/stop-reason.ts
-function resolveStopReason(code2, signal, wasIntentional) {
-  if (wasIntentional)
-    return "user.stop";
+function resolveStopReason(code2, signal) {
   if (signal !== null)
     return "agent_process.signal";
   if (code2 === 0)
@@ -15866,7 +15976,6 @@ function agentKey2(chatroomId, role) {
 class AgentProcessManager {
   deps;
   slots = new Map;
-  pendingStopReasons = new Map;
   constructor(deps) {
     this.deps = deps;
   }
@@ -15890,12 +15999,18 @@ class AgentProcessManager {
     const key = agentKey2(opts.chatroomId, opts.role);
     const slot = this.slots.get(key);
     if (!slot || slot.state === "idle") {
+      const eventPid = opts.pid;
+      if (eventPid && eventPid > 0) {
+        try {
+          this.deps.processes.kill(eventPid, "SIGTERM");
+        } catch {}
+      }
       this.deps.backend.mutation(api.machines.recordAgentExited, {
         sessionId: this.deps.sessionId,
         machineId: this.deps.machineId,
         chatroomId: opts.chatroomId,
         role: opts.role,
-        pid: 0,
+        pid: eventPid ?? 0,
         stopReason: opts.reason,
         exitCode: undefined,
         signal: undefined,
@@ -15915,7 +16030,6 @@ class AgentProcessManager {
       slot.pendingOperation = undefined;
       return { success: true };
     }
-    this.pendingStopReasons.set(key, opts.reason);
     slot.state = "stopping";
     const operation = this.doStop(key, slot, pid, opts);
     slot.pendingOperation = operation;
@@ -15931,9 +16045,7 @@ class AgentProcessManager {
     if (slot.state === "stopping") {
       return;
     }
-    const intentionalReason = this.pendingStopReasons.get(key);
-    this.pendingStopReasons.delete(key);
-    const stopReason = intentionalReason ?? resolveStopReason(opts.code, opts.signal, false);
+    const stopReason = resolveStopReason(opts.code, opts.signal);
     this.deps.spawning.recordExit(opts.chatroomId);
     const harness = slot.harness;
     const model = slot.model;
@@ -17297,6 +17409,8 @@ program2.command("classify").description("Classify a task's origin message (entr
 });
 program2.command("handoff").description("Complete your task and hand off to the next role").requiredOption("--chatroom-id <id>", "Chatroom identifier").requiredOption("--role <role>", "Your role").requiredOption("--next-role <nextRole>", "Role to hand off to").option("--attach-artifact <artifactId>", "Attach artifact to handoff (can be used multiple times)", (value, previous) => {
   return previous ? [...previous, value] : [value];
+}, []).option("--attachment <type:value>", "Attach a resource (can be used multiple times). Format: type:value. Supported: workflow-key:<key>", (value, previous) => {
+  return previous ? [...previous, value] : [value];
 }, []).action(async (options) => {
   await maybeRequireAuth();
   const { decode: decode2 } = await Promise.resolve().then(() => exports_decode);
@@ -17313,12 +17427,29 @@ program2.command("handoff").description("Complete your task and hand off to the
     console.error("❌ Message is empty");
     process.exit(1);
   }
+  const attachedWorkflowKeys = [];
+  for (const att of options.attachment || []) {
+    const colonIndex = att.indexOf(":");
+    if (colonIndex === -1) {
+      console.error(`❌ Invalid attachment format: "${att}". Expected type:value (e.g., workflow-key:my-workflow)`);
+      process.exit(1);
+    }
+    const type = att.substring(0, colonIndex);
+    const value = att.substring(colonIndex + 1);
+    if (type === "workflow-key") {
+      attachedWorkflowKeys.push(value);
+    } else {
+      console.error(`❌ Unknown attachment type: "${type}". Supported: workflow-key`);
+      process.exit(1);
+    }
+  }
   const { handoff: handoff2 } = await Promise.resolve().then(() => (init_handoff(), exports_handoff));
   await handoff2(options.chatroomId, {
     role: options.role,
     message,
     nextRole: options.nextRole,
-    attachedArtifactIds: options.attachArtifact || []
+    attachedArtifactIds: options.attachArtifact || [],
+    attachedWorkflowKeys
   });
 });
 program2.command("report-progress").description("Report progress on current task (does not complete the task)").requiredOption("--chatroom-id <id>", "Chatroom identifier").requiredOption("--role <role>", "Your role").action(async (options) => {
@@ -17504,31 +17635,26 @@ workflowCommand.command("step-complete").description("Mark a workflow step as co
     stepKey: options.stepKey
   });
 });
-workflowCommand.command("step-cancel").description("Cancel a workflow step with a reason").requiredOption("--chatroom-id <id>", "Chatroom identifier").requiredOption("--role <role>", "Your role").requiredOption("--workflow-key <key>", "Workflow key").requiredOption("--step-key <stepKey>", "Step key to cancel").requiredOption("--reason <text>", "Reason for cancellation (required)").action(async (options) => {
+workflowCommand.command("exit").description("Exit (cancel) an entire workflow with a reason").requiredOption("--chatroom-id <id>", "Chatroom identifier").requiredOption("--role <role>", "Your role").requiredOption("--workflow-key <key>", "Workflow key to exit").requiredOption("--reason <text>", "Reason for exiting the workflow (required)").action(async (options) => {
   await maybeRequireAuth();
   if (!options.reason || options.reason.trim().length === 0) {
     console.error("❌ --reason is required and cannot be empty");
     process.exit(1);
   }
-  const { cancelStep: cancelStep2 } = await Promise.resolve().then(() => (init_workflow(), exports_workflow));
-  await cancelStep2(options.chatroomId, {
+  const { exitWorkflow: exitWorkflow2 } = await Promise.resolve().then(() => (init_workflow(), exports_workflow));
+  await exitWorkflow2(options.chatroomId, {
     role: options.role,
     workflowKey: options.workflowKey,
-    stepKey: options.stepKey,
     reason: options.reason
   });
 });
-workflowCommand.command("exit").description("Exit (cancel) an entire workflow with a reason").requiredOption("--chatroom-id <id>", "Chatroom identifier").requiredOption("--role <role>", "Your role").requiredOption("--workflow-key <key>", "Workflow key to exit").requiredOption("--reason <text>", "Reason for exiting the workflow (required)").action(async (options) => {
+workflowCommand.command("step-view").description("View the full details and specification of a single workflow step").requiredOption("--chatroom-id <id>", "Chatroom identifier").requiredOption("--role <role>", "Your role").requiredOption("--workflow-key <key>", "Workflow key").requiredOption("--step-key <stepKey>", "Step key to view").action(async (options) => {
   await maybeRequireAuth();
-  if (!options.reason || options.reason.trim().length === 0) {
-    console.error("❌ --reason is required and cannot be empty");
-    process.exit(1);
-  }
-  const { exitWorkflow: exitWorkflow2 } = await Promise.resolve().then(() => (init_workflow(), exports_workflow));
-  await exitWorkflow2(options.chatroomId, {
+  const { viewStep: viewStep2 } = await Promise.resolve().then(() => (init_workflow(), exports_workflow));
+  await viewStep2(options.chatroomId, {
     role: options.role,
     workflowKey: options.workflowKey,
-    reason: options.reason
+    stepKey: options.stepKey
   });
 });
 var taskCommand = program2.command("task").description("Manage tasks");

package/package.json

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