@zhushanwen/pi-subagent-workflow 7.1.0 → 7.3.0

package/src/interface/subagent-actions.ts

@@ -226,7 +226,21 @@ export async function cancelHandler(
 
   // step 1: id 不存在（findRecord 只查内存 running record，不从 session.jsonl 重建）
   const rec = service.findRecord(id);
-  if (!rec) throw new Error(`No subagent record with id "${id}". It may have finished — use action:'list' with includeFinished:true to verify.`);
+  if (!rec) {
+    // [S-19] MF-1 全树可见后，list/completion 可能列出其他进程（父/兄弟）的 running record
+    //（collectRecords 扫共享 sessionsDir 按 rootSessionId 过滤，跨进程互相可见），而 cancel
+    // 只作用于本进程内存 record。区分两种失败，避免「may have finished」误导（该 record 正
+    // 被列出且未 finished，只是不属于本进程内存）。仅文案区分，不改 cancel 作用域。
+    const treeRec = service.collectRecords(DEFAULT_LIST_LIMIT, "all").find((r) => r.id === id);
+    if (treeRec && treeRec.status === "running") {
+      throw new Error(
+        `Subagent record "${id}" is running but owned by another process in the tree ` +
+          `(it was spawned by a different subagent process) — this process cannot cancel it; ` +
+          `cancel only works for subagents spawned by the current process.`,
+      );
+    }
+    throw new Error(`No subagent record with id "${id}". It may have finished — use action:'list' with includeFinished:true to verify.`);
+  }
   // step 2: controller 检查（controller 为 undefined 表示 record 已终态或未启动）
   if (rec.mode !== "background") {
     throw new Error(`Cannot cancel subagent ${id} (unsupported mode: ${rec.mode})`);

package/src/interface/subagents.ts

@@ -32,7 +32,7 @@ export function registerSubagentsCommand(pi: ExtensionAPI): void {
         try {
           const service = getSubagentService();
           if (!service) return null;
-          // collectRecords 合并内存(running) + 磁盘重建 record，按 session 过滤。
+          // collectRecords 合并内存(running) + 磁盘重建 record，按 rootSessionId 过滤。
           // cancel 只对 running 有效，但全部列出便于用户辨认（终态 record 会被 service 拒绝）。
           const records = service.collectRecords(LIST_LIMIT);
           if (records.length === 0) return null;

package/src/orchestration/__tests__/jsonl-run-store-session-file.test.ts

@@ -175,3 +175,55 @@ describe("W2: RunStore.stateFilePath 暴露 run 状态文件路径", () => {
     expect(result).toBe(path.join(tmpDir, "workflow-state", "run-xyz.jsonl"));
   });
 });
+
+// W3: save 兜底容错——run 工作目录被并发清理时 mkdir 抛 ENOENT，save 静默返回。
+//
+// 防的 bug（PR #166 CI 回归）：review-fix-loop-e2e 等 runAndWait 测试中，
+// handleReturn 的 run.transition("done") 同步改 status 后，runAndWait 轮询发现 done
+// 并 resolve，测试 afterEach 随即 rmSync 删除 sessionDir；此时 handleReturn 内 in-flight
+// 的 await save 尚未完成，mkdir 遇到目录链被并发删除 → ENOENT。原实现 await save 让错误
+// 冒泡为 unhandled promise rejection（worker-host onMessage 无 catch），CI exit 1。
+// 修复：save 仅容错 ENOENT（run 已终态，状态不再变化，持久化无意义也无法完成）→ silent return；
+// 非 ENOENT 错误（EACCES/ENOSPC 等真实磁盘问题）仍重新抛出，不掩盖。
+//
+// 注：真正的 ENOENT 只在 rmSync 与 mkdir 并发时出现（串行 rmSync 后 mkdir {recursive:true}
+// 会重建目录而非抛 ENOENT），故用 mock 直接锁定 save 的容错判定逻辑，不依赖竞态时序复现。
+describe("W3: JsonlRunStore.save 兜底容错（run 工作目录被并发清理时的竞态）", () => {
+  let tmpDir: string;
+  let store: JsonlRunStore;
+
+  beforeEach(() => {
+    tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "wf-store-enoent-"));
+    store = new JsonlRunStore({ sessionDir: tmpDir });
+  });
+
+  afterEach(() => {
+    fs.rmSync(tmpDir, { recursive: true, force: true });
+    vi.restoreAllMocks();
+  });
+
+  it("mkdir 抛 ENOENT（sessionDir 被并发清理）→ save 静默返回，不抛 unhandled rejection", async () => {
+    const run = makeRunWithDoneCall();
+    const spy = vi
+      .spyOn(fs.promises, "mkdir")
+      .mockRejectedValueOnce(
+        Object.assign(new Error("ENOENT: no such file or directory, mkdir"), {
+          code: "ENOENT",
+        }),
+      );
+    // run 已终态 + 工作目录消失 → save 放弃持久化，resolve undefined（不抛）
+    await expect(store.save(run)).resolves.toBeUndefined();
+    expect(spy).toHaveBeenCalled();
+  });
+
+  it("mkdir 抛非 ENOENT 错误（EACCES）→ save 重新抛出，不掩盖真实磁盘问题", async () => {
+    const run = makeRunWithDoneCall();
+    const spy = vi
+      .spyOn(fs.promises, "mkdir")
+      .mockRejectedValueOnce(
+        Object.assign(new Error("permission denied"), { code: "EACCES" }),
+      );
+    await expect(store.save(run)).rejects.toThrow("permission denied");
+    expect(spy).toHaveBeenCalled();
+  });
+});

package/src/orchestration/jsonl-run-store.ts

@@ -241,7 +241,26 @@ export class JsonlRunStore {
  */
   async save(run: WorkflowRun): Promise<void> {
     const filePath = this.filePathFor(run.runId);
-    await fs.promises.mkdir(path.dirname(filePath), { recursive: true });
+    // 兜底容错：run 工作目录（sessionDir）已被清理时，mkdir 抛 ENOENT，save 放弃。
+    // 竞态场景（review-fix-loop-e2e 等 runAndWait 测试）：handleReturn 内
+    // run.transition("done") 同步改 status 后，runAndWait 轮询发现 done 并 resolve，
+    // 测试 afterEach 随即 rmSync 删除 sessionDir；此时本方法 in-flight 的 mkdir
+    // 遇到目录链已删除 → ENOENT（{recursive:true} 在并发 rmSync 下仍可抛 ENOENT）。
+    // run 既已终态（状态不再变化），持久化无意义也无法完成 → silent return。
+    // 仅容错 ENOENT，重新抛出其他错误（EACCES/ENOSPC 等真实磁盘问题不掩盖）。
+    try {
+      await fs.promises.mkdir(path.dirname(filePath), { recursive: true });
+    } catch (err) {
+      if (
+        typeof err === "object" &&
+        err !== null &&
+        "code" in err &&
+        (err as { code: unknown }).code === "ENOENT"
+      ) {
+        return;
+      }
+      throw err;
+    }
     const snapshot = serializeRun(run);
     await fs.promises.writeFile(filePath, JSON.stringify(snapshot) + "\n", "utf8");
     if (this.pi) {