chatroom-cli 1.57.1 → 1.57.3

package/dist/index.js

@@ -75589,9 +75589,18 @@ function getPlannerToBuilderHandoffTemplate(nativeIntegration = false) {
 **Detail bar:** Specify down to **every file** the builder will create or modify (full repo paths). Include code snippets — types, signatures, stubs, or target implementations — until a competent builder **cannot misinterpret** what to write. Vague layers ("update the backend", "fix the component") are not acceptable.
 
 \`\`\`markdown
+## Summary
+<brief context for this delegation slice — what problem it solves and where it fits in the larger task>
+
 ## Goal
 <one sentence: the outcome this slice delivers>
 
+## Key Knowledge for High Quality Bar
+<details that would move the implementation from good to excellent and delightful — domain context, user expectations, edge cases, naming, UX polish, invariants the builder must preserve>
+
+## Force Multipliers
+<choices that greatly simplify the solution while preserving long-term maintainability — reuse existing abstractions, avoid unnecessary layers, leverage platform conventions>
+
 ## Files to implement (exhaustive, file-level)
 List **every** file in this slice. For each file, state the exact change and paste the code the builder should match (no guessing).
 
@@ -75661,9 +75670,19 @@ function getPlannerToUserReportTemplate() {
 ## Summary
 <what was accomplished, in plain terms — no references to prior messages>
 
-## Proof — files changed
+## Template Disclosure Confirmation
+- [ ] I confirm that I have seen this template at the start of any planning, before working on or delegating any task to the team
+
+## Proof of Principle
+<!-- Demonstrate adherence to:
+- Organization & Maintainability: a small change in requirements should result in a small change in code in a small number of files and folders.
+- Static Evaluability and Provability: the system's behavior should be provably correct by looking at the source code, then automated tests, then manual tests, in this order.
+-->
+<how this work follows the principles above — localized changes, readable structure, correctness provable from source then tests>
+
+## Proof of Completion
 - \`path/to/file.ts\` — <what changed and why>
-<list every file you (or the builder) modified; this is the evidence of work>
+<evidence the goal was met — list every file you (or the builder) modified>
 
 ## Key Technical Decisions
 - <schema design, modules, interfaces, domain entities — what you chose and why, or "Not Applicable">
@@ -76172,7 +76191,7 @@ var claudeCapabilities;
 var init_claude_config = __esm(() => {
   claudeCapabilities = {
     runtimeKind: "cli",
-    supportsSessionResume: false,
+    supportsDaemonMemoryResume: false,
     supportsNativeIntegration: false,
     lifecycle: {
       turnCompleted: false,
@@ -76188,7 +76207,7 @@ var commandcodeCapabilities;
 var init_commandcode_config = __esm(() => {
   commandcodeCapabilities = {
     runtimeKind: "cli",
-    supportsSessionResume: false,
+    supportsDaemonMemoryResume: false,
     supportsNativeIntegration: false,
     lifecycle: {
       turnCompleted: true,
@@ -76204,7 +76223,7 @@ var copilotCapabilities;
 var init_copilot_config = __esm(() => {
   copilotCapabilities = {
     runtimeKind: "cli",
-    supportsSessionResume: false,
+    supportsDaemonMemoryResume: false,
     supportsNativeIntegration: false,
     lifecycle: {
       turnCompleted: true,
@@ -76220,7 +76239,7 @@ var cursorSdkCapabilities;
 var init_cursor_sdk_config = __esm(() => {
   cursorSdkCapabilities = {
     runtimeKind: "sdk",
-    supportsSessionResume: false,
+    supportsDaemonMemoryResume: true,
     supportsNativeIntegration: true,
     lifecycle: {
       turnCompleted: true,
@@ -76236,7 +76255,7 @@ var cursorCapabilities;
 var init_cursor_config = __esm(() => {
   cursorCapabilities = {
     runtimeKind: "cli",
-    supportsSessionResume: false,
+    supportsDaemonMemoryResume: false,
     supportsNativeIntegration: false,
     lifecycle: {
       turnCompleted: true,
@@ -76252,7 +76271,7 @@ var opencodeSdkCapabilities;
 var init_opencode_sdk_config = __esm(() => {
   opencodeSdkCapabilities = {
     runtimeKind: "sdk",
-    supportsSessionResume: false,
+    supportsDaemonMemoryResume: true,
     supportsNativeIntegration: true,
     lifecycle: {
       turnCompleted: true,
@@ -76268,7 +76287,7 @@ var opencodeCapabilities;
 var init_opencode_config = __esm(() => {
   opencodeCapabilities = {
     runtimeKind: "cli",
-    supportsSessionResume: false,
+    supportsDaemonMemoryResume: false,
     supportsNativeIntegration: false,
     lifecycle: {
       turnCompleted: false,
@@ -76284,7 +76303,7 @@ var piSdkCapabilities;
 var init_pi_sdk_config = __esm(() => {
   piSdkCapabilities = {
     runtimeKind: "sdk",
-    supportsSessionResume: false,
+    supportsDaemonMemoryResume: false,
     supportsNativeIntegration: true,
     lifecycle: {
       turnCompleted: true,
@@ -76300,7 +76319,7 @@ var piCapabilities;
 var init_pi_config = __esm(() => {
   piCapabilities = {
     runtimeKind: "cli",
-    supportsSessionResume: false,
+    supportsDaemonMemoryResume: false,
     supportsNativeIntegration: false,
     lifecycle: {
       turnCompleted: true,
@@ -76353,43 +76372,6 @@ var init_types = __esm(() => {
 });
 // ../../services/backend/prompts/review-guidelines/index.ts
 var init_review_guidelines = () => {};
-// ../../services/backend/prompts/cli/resume-session/message.ts
-function composeResumeMessage(params) {
-  const { chatroomId, role, convexUrl, supportsNativeIntegration } = params;
-  const cliEnvPrefix = getCliEnvPrefix(convexUrl);
-  const contextRead = `${cliEnvPrefix}chatroom context read --chatroom-id="${chatroomId}" --role="${role}"`;
-  if (supportsNativeIntegration) {
-    return [
-      "Your previous turn has ended. Your session stays active.",
-      "The next chatroom task will be injected automatically when ready.",
-      "",
-      "If you need context on what you were doing, run:",
-      `  ${contextRead}`
-    ].join(`
-`);
-  }
-  const getNextTask2 = getNextTaskCommand({
-    chatroomId,
-    role,
-    cliEnvPrefix
-  });
-  return [
-    "Your previous turn has ended. A pending chatroom task may already be waiting.",
-    "You MUST immediately run this command as a foreground blocking bash tool call (not in background):",
-    `  ${getNextTask2}`,
-    "",
-    "Do not reply with text only — run the command first. The harness delivers tasks only while get-next-task blocks.",
-    "",
-    "If you need context on what you were doing, run:",
-    `  ${contextRead}`
-  ].join(`
-`);
-}
-var init_message = __esm(() => {
-  init_env();
-  init_command();
-});
-
 // ../../services/backend/prompts/generator.ts
 function generateHandoffOutput(params) {
   const { role, nextRole, chatroomId, convexUrl, supportsNativeIntegration } = params;
@@ -76428,7 +76410,6 @@ var init_generator = __esm(() => {
   init_utils3();
   init_types();
   init_review_guidelines();
-  init_message();
 });
 
 // src/commands/handoff/index.ts
@@ -84916,16 +84897,6 @@ function createTurnCompletedBackend(deps) {
       machineId: deps.machineId,
       ...args2
     }),
-    emitSessionResumed: (args2) => deps.backend.mutation(api.machines.emitSessionResumed, {
-      sessionId: deps.sessionId,
-      machineId: deps.machineId,
-      ...args2
-    }),
-    emitSessionResumeFailed: (args2) => deps.backend.mutation(api.machines.emitSessionResumeFailed, {
-      sessionId: deps.sessionId,
-      machineId: deps.machineId,
-      ...args2
-    }),
     emitAgentStartFailed: (args2) => deps.backend.mutation(api.machines.emitAgentStartFailed, {
       sessionId: deps.sessionId,
       machineId: deps.machineId,
@@ -85551,9 +85522,6 @@ async function tryAbortResumeStorm(deps, input, slot) {
     return false;
   }
   deps.resumeStormTracker.reset(input.chatroomId, input.role);
-  if (slot) {
-    slot.resumeInFlight = false;
-  }
   try {
     await deps.backend.emitResumeStormAborted({
       chatroomId: input.chatroomId,
@@ -85591,9 +85559,6 @@ var RECENT_LOG_LINE_CAP2 = 100;
 
 // src/domain/agent-lifecycle/use-cases/handle-turn-completed.ts
 async function handleTurnCompleted(deps, input, slot) {
-  if (slot?.resumeInFlight) {
-    return { outcome: "skipped_duplicate" };
-  }
   if (await tryAbortResumeStorm(deps, input, slot)) {
     return { outcome: "storm_aborted" };
   }
@@ -85612,35 +85577,6 @@ async function handleTurnCompleted(deps, input, slot) {
     deps.killProcess(input.pid);
     return { outcome: "killed_terminal_provider_error" };
   }
-  if (input.supportsSessionResume && input.wantResume) {
-    if (slot) {
-      slot.resumeInFlight = true;
-    }
-    try {
-      await deps.resumeTurn(input.pid, deps.composeResumePrompt({ chatroomId: input.chatroomId, role: input.role }));
-      try {
-        await deps.backend.emitSessionResumed({
-          chatroomId: input.chatroomId,
-          role: input.role,
-          ...slot?.harnessSessionId ? { harnessSessionId: slot.harnessSessionId } : {}
-        });
-      } catch {}
-      return { outcome: "resumed" };
-    } catch (err) {
-      try {
-        await deps.backend.emitSessionResumeFailed({
-          chatroomId: input.chatroomId,
-          role: input.role,
-          reason: err.message,
-          ...slot?.harnessSessionId ? { harnessSessionId: slot.harnessSessionId } : {}
-        });
-      } catch {}
-    } finally {
-      if (slot) {
-        slot.resumeInFlight = false;
-      }
-    }
-  }
   deps.killProcess(input.pid);
   return { outcome: "killed" };
 }
@@ -86051,7 +85987,6 @@ class AgentProcessManager {
         model: slot.model,
         workingDir: slot.workingDir,
         startedAt: slot.startedAt,
-        resumeInFlight: slot.resumeInFlight,
         recentLogLines: slot.recentLogLines,
         wantResume: slot.wantResume
       });
@@ -86157,9 +86092,7 @@ class AgentProcessManager {
   }
   async runHandleAgentEnd(opts) {
     const slot = this.slots.get(agentKey3(opts.chatroomId, opts.role));
-    const service3 = this.deps.agentServices.get(opts.harness);
     const capabilities = getHarnessCapabilities(opts.harness);
-    const supportsSessionResume = capabilities.supportsSessionResume && typeof service3?.resumeTurn === "function";
     this.updateSlotsMirror(opts.chatroomId, opts.role, {
       state: slot?.state ?? "idle",
       pid: slot?.pid,
@@ -86168,11 +86101,10 @@ class AgentProcessManager {
       model: slot?.model,
       workingDir: slot?.workingDir,
       startedAt: slot?.startedAt,
-      resumeInFlight: slot?.resumeInFlight,
       recentLogLines: slot?.recentLogLines,
       wantResume: slot?.wantResume
     });
-    console.log(`[AgentProcessManager] lifecycle.turn.completed: role=${opts.role} pid=${opts.pid} harness=${opts.harness} supportsResume=${supportsSessionResume}`);
+    console.log(`[AgentProcessManager] lifecycle.turn.completed: role=${opts.role} pid=${opts.pid} harness=${opts.harness}`);
     if (capabilities.supportsNativeIntegration) {
       await this.runHandleNativeTurnEnd(opts, slot);
       return;
@@ -86181,18 +86113,6 @@ class AgentProcessManager {
       resumeStormTracker: this.deps.resumeStormTracker,
       backend: createTurnCompletedBackend(this.deps),
       now: () => this.deps.clock.now(),
-      composeResumePrompt: ({ chatroomId, role }) => composeResumeMessage({
-        chatroomId,
-        role,
-        convexUrl: this.deps.convexUrl,
-        supportsNativeIntegration: capabilities.supportsNativeIntegration
-      }),
-      resumeTurn: async (pid, prompt) => {
-        if (!service3?.resumeTurn) {
-          throw new Error("Harness does not support resumeTurn");
-        }
-        await service3.resumeTurn(pid, prompt);
-      },
       killProcess: (pid) => {
         try {
           this.deps.processes.kill(-pid, "SIGTERM");
@@ -86202,27 +86122,17 @@ class AgentProcessManager {
     }, {
       chatroomId: opts.chatroomId,
       role: opts.role,
-      pid: opts.pid,
-      supportsSessionResume,
-      wantResume: slot?.wantResume ?? true
+      pid: opts.pid
     }, slot);
-    if (result.outcome === "skipped_duplicate") {
-      console.log(`[AgentProcessManager] lifecycle.turn.completed: skipping duplicate resume for ${opts.role} (resume already in flight)`);
-    } else if (result.outcome === "storm_aborted") {
+    if (result.outcome === "storm_aborted") {
       console.log(`[AgentProcessManager] ✅ Handled rapid resume storm for ${opts.role}`);
-    } else if (result.outcome === "resumed") {
-      console.log(`[AgentProcessManager] ✅ Emitted agent.sessionResumed for ${opts.role}`);
     } else if (result.outcome === "killed") {
-      console.log(`[AgentProcessManager] lifecycle.turn.completed: killed process for ${opts.role} (resume disabled or failed)`);
+      console.log(`[AgentProcessManager] lifecycle.turn.completed: killed process for ${opts.role}`);
     } else if (result.outcome === "killed_terminal_provider_error") {
       console.log(`[AgentProcessManager] ⛔ Terminal provider error for ${opts.role} — emitted agent.startFailed`);
     }
   }
   async runHandleNativeTurnEnd(opts, slot) {
-    if (slot?.resumeInFlight) {
-      console.log(`[AgentProcessManager] lifecycle.turn.completed: skipping duplicate native turn-end for ${opts.role}`);
-      return;
-    }
     if (await tryAbortResumeStorm({
       resumeStormTracker: this.deps.resumeStormTracker,
       backend: createTurnCompletedBackend(this.deps),
@@ -86231,9 +86141,7 @@ class AgentProcessManager {
     }, {
       chatroomId: opts.chatroomId,
       role: opts.role,
-      pid: opts.pid,
-      supportsSessionResume: false,
-      wantResume: false
+      pid: opts.pid
     }, slot)) {
       console.log(`[AgentProcessManager] ✅ Handled rapid resume storm for ${opts.role}`);
       return;
@@ -86633,12 +86541,12 @@ class AgentProcessManager {
     if (!stored) {
       return null;
     }
-    const resumeFromDaemonMemory = opts.service.resumeFromDaemonMemory;
-    if (!resumeFromDaemonMemory) {
+    if (!opts.service.resumeFromDaemonMemory) {
       return null;
     }
     try {
-      const spawnResult = await resumeFromDaemonMemory({
+      await this.emitSessionResumeRequested(opts.chatroomId, opts.role, opts.agentHarness, stored.harnessSessionId);
+      const spawnResult = await opts.service.resumeFromDaemonMemory({
         workingDir: stored.workingDir,
         prompt: createSpawnPrompt(opts.initPrompt),
         systemPrompt: opts.systemPrompt,
@@ -86685,6 +86593,21 @@ class AgentProcessManager {
     }
     return null;
   }
+  async emitSessionResumeRequested(chatroomId, role, agentHarness, harnessSessionId) {
+    try {
+      await this.deps.backend.mutation(api.machines.emitSessionResumeRequested, {
+        sessionId: this.deps.sessionId,
+        machineId: this.deps.machineId,
+        chatroomId,
+        role,
+        agentHarness,
+        ...harnessSessionId ? { harnessSessionId } : {}
+      });
+      console.log(`[AgentProcessManager] ✅ Emitted agent.sessionResumeRequested for ${role}`);
+    } catch (err) {
+      console.log(`   ⚠️  Failed to emit sessionResumeRequested event: ${err.message}`);
+    }
+  }
   async emitSessionResumed(chatroomId, role, harnessSessionId) {
     try {
       await this.deps.backend.mutation(api.machines.emitSessionResumed, {
@@ -87091,7 +87014,6 @@ class AgentProcessManager {
 }
 var AGENT_EXIT_RETRY_INTERVAL_MS = 1e4;
 var init_agent_process_manager = __esm(() => {
-  init_generator();
   init_types();
   init_participant();
   init_esm();
@@ -89665,4 +89587,4 @@ program2.hook("preAction", async (_thisCommand, actionCommand) => {
 });
 program2.parse();
 
-//# debugId=CFDE8D88B080E55664756E2164756E21
+//# debugId=4A0FE03BEB3CACE864756E2164756E21