@zhushanwen/pi-subagent-workflow 8.6.0 → 8.7.0

package/src/execution/__tests__/agent-result-mapper.test.ts

@@ -1,150 +0,0 @@
-// src/execution/__tests__/agent-result-mapper.test.ts
-//
-// T3.14 (NFR-兼容性): AgentResult 映射字段对齐 —— D-A10 纯函数单测
-// T3.1  (正常): executeAndAwait 返回 content → 间接覆盖（subagent-service.test.ts 集成验证）
-//
-// 测试范围: mapToWorkflowAgentResult 所有字段映射分支（success/error/usage/toolCalls）
-
-import { describe, expect, it } from "vitest";
-
-import { mapToWorkflowAgentResult } from "../agent-result-mapper.ts";
-import type { AgentResult as SubagentsAgentResult, AgentUsageTotal, ToolCall } from "../types.ts";
-
-describe("mapToWorkflowAgentResult (D-A10)", () => {
-  const minimalResult: SubagentsAgentResult = {
-    text: "Hello",
-    turns: 3,
-    durationMs: 1200,
-    success: true,
-    sessionId: "session-123",
-    toolCalls: [],
-  };
-
-  // ── T3.14: 正常成功路径 ──
-
-  it("映射成功结果: text→content, durationMs/sessionId 透传", () => {
-    const result = mapToWorkflowAgentResult(minimalResult);
-    expect(result.content).toBe("Hello");
-    expect(result.durationMs).toBe(1200);
-    expect(result.sessionId).toBe("session-123");
-    expect(result.error).toBeUndefined();
-    expect(result.parsedOutput).toBeUndefined();
-    expect(result.usage).toBeUndefined();
-    expect(result.toolCalls).toEqual([]);
-  });
-
-  // ── sessionFile 透传（方案 A：让 workflow 继承 subagent 的 session jsonl 路径）──
-
-  it("映射 sessionFile: subagents AgentResult.sessionFile → workflow AgentResult.sessionFile 透传", () => {
-    const r: SubagentsAgentResult = {
-      ...minimalResult,
-      sessionFile: "/abs/path/.pi/agent/subagents/enc/sessions/2026-07-15T10-00-00-000Z_session-123.jsonl",
-    };
-    const result = mapToWorkflowAgentResult(r);
-    expect(result.sessionFile).toBe("/abs/path/.pi/agent/subagents/enc/sessions/2026-07-15T10-00-00-000Z_session-123.jsonl");
-  });
-
-  it("映射 sessionFile: 无 sessionFile → workflow AgentResult.sessionFile undefined（窗口期/未回填）", () => {
-    const r: SubagentsAgentResult = { ...minimalResult, sessionFile: undefined };
-    const result = mapToWorkflowAgentResult(r);
-    expect(result.sessionFile).toBeUndefined();
-  });
-
-  it("映射 parsedOutput 透传（structured-output 契约 BC-8）", () => {
-    const parsedData = { score: 0.95, label: "positive" };
-    const r: SubagentsAgentResult = { ...minimalResult, parsedOutput: parsedData };
-    const result = mapToWorkflowAgentResult(r);
-    expect(result.parsedOutput).toEqual(parsedData);
-  });
-
-  // ── T3.14: 失败路径 ──
-
-  it("映射失败: success=false 且 error → 填入 error 字段", () => {
-    const r: SubagentsAgentResult = { ...minimalResult, success: false, error: "timeout" };
-    const result = mapToWorkflowAgentResult(r);
-    expect(result.error).toBe("timeout");
-    expect(result.content).toBe("Hello"); // content 仍保留（可能有部分输出）
-  });
-
-  it("映射失败: success=false 但无 error（abort 路径）→ error 填入 fallback 而非 undefined", () => {
-    // H4: session-runner.ts abort 时 success=false, error=undefined。
-    // 旧代码 error=undefined → executeAgentCall 误判 completed。
-    // 修复后应 synthesize fallback error。
-    const r: SubagentsAgentResult = { ...minimalResult, success: false };
-    const result = mapToWorkflowAgentResult(r);
-    expect(result.error).toBeDefined();
-    expect(typeof result.error).toBe("string");
-    expect(result.error.length).toBeGreaterThan(0);
-  });
-
-  it("映射成功: success=true 且 error 存在 → error=undefined（不误填）", () => {
-    const r: SubagentsAgentResult = { ...minimalResult, success: true, error: "stale" };
-    const result = mapToWorkflowAgentResult(r);
-    expect(result.error).toBeUndefined();
-  });
-
-  // ── T3.14: usage 映射 ──
-
-  it("映射 usage: AgentUsageTotal → AgentUsage（字段形状转换）", () => {
-    const usage: AgentUsageTotal = {
-      input: 1000,
-      output: 500,
-      cacheRead: 200,
-      cacheWrite: 100,
-      total: 1800,
-      cost: 0.005,
-    };
-    const r: SubagentsAgentResult = { ...minimalResult, usage };
-    const result = mapToWorkflowAgentResult(r);
-    expect(result.usage).toEqual({
-      input: 1000,
-      output: 500,
-      cacheRead: 200,
-      cacheWrite: 100,
-      cost: 0.005,
-      contextTokens: 1800, // total → contextTokens
-      turns: 3, // 来自 AgentResult.turns
-    });
-  });
-
-  it("映射 usage: 无 usage → usage undefined", () => {
-    const result = mapToWorkflowAgentResult(minimalResult);
-    expect(result.usage).toBeUndefined();
-  });
-
-  // ── T3.14: toolCalls 映射 ──
-
-  it("映射 toolCalls: ToolCall → ToolCallEntry（name/input 形状转换）", () => {
-    const calls: ToolCall[] = [
-      { toolName: "read", args: { path: "/a.txt" } },
-      { toolName: "bash", args: { command: "ls" }, result: { content: ["ok"] }, isError: false },
-    ];
-    const r: SubagentsAgentResult = { ...minimalResult, toolCalls: calls };
-    const result = mapToWorkflowAgentResult(r);
-    expect(result.toolCalls).toBeDefined();
-    expect(result.toolCalls!).toHaveLength(2);
-    expect(result.toolCalls![0]).toEqual({ name: "read", input: '{"path":"/a.txt"}' });
-    expect(result.toolCalls![1].name).toBe("bash");
-  });
-
-  it("映射 toolCalls: args=undefined → input=''", () => {
-    const calls: ToolCall[] = [{ toolName: "list" }];
-    const r: SubagentsAgentResult = { ...minimalResult, toolCalls: calls };
-    const result = mapToWorkflowAgentResult(r);
-    expect(result.toolCalls![0].input).toBe("");
-  });
-
-  it("映射 toolCalls: 长 args 截断（>500 chars）", () => {
-    const longStr = "x".repeat(600);
-    const calls: ToolCall[] = [{ toolName: "write", args: { content: longStr } }];
-    const r: SubagentsAgentResult = { ...minimalResult, toolCalls: calls };
-    const result = mapToWorkflowAgentResult(r);
-    expect(result.toolCalls![0].input.length).toBeLessThanOrEqual(503); // 500 + "..."
-  });
-
-  it("映射 toolCalls: 无 toolCalls → undefined", () => {
-    const r: SubagentsAgentResult = { ...minimalResult, toolCalls: undefined };
-    const result = mapToWorkflowAgentResult(r);
-    expect(result.toolCalls).toBeUndefined();
-  });
-});

package/src/execution/__tests__/alive-store.test.ts

@@ -1,147 +0,0 @@
-// src/__tests__/alive-store.test.ts
-
-import * as fs from "node:fs";
-import * as os from "node:os";
-import * as path from "node:path";
-
-import { afterEach,describe, expect, it } from "vitest";
-
-import {
-  isProcessAlive,
-  readAliveMarker,
-  removeAliveMarker,
-  writeAliveMarker,
-} from "../alive-store.ts";
-import type { AliveMarker } from "../types.ts";
-
-function makeTmpDir(): string {
-  return fs.mkdtempSync(path.join(os.tmpdir(), "alive-store-test-"));
-}
-
-describe("alive-store", () => {
-  let tmpDir: string;
-
-  afterEach(() => {
-    if (tmpDir) {
-      fs.rmSync(tmpDir, { recursive: true, force: true });
-    }
-  });
-
-  // ── writeAliveMarker + readAliveMarker 往返 ──
-
-  it("write → read round-trip", () => {
-    tmpDir = makeTmpDir();
-    const sessionFile = path.join(tmpDir, "session.jsonl");
-    const marker: AliveMarker = { pid: 12345, id: "bg-1-abc", startedAt: Date.now() };
-
-    writeAliveMarker(sessionFile, marker);
-    const result = readAliveMarker(sessionFile);
-
-    expect(result).toEqual(marker);
-    // 验证 sidecar 文件是单行 JSON
-    const raw = fs.readFileSync(`${sessionFile}.alive`, "utf-8");
-    expect(raw.endsWith("\n")).toBe(true);
-    expect(JSON.parse(raw.trim())).toEqual(marker);
-  });
-
-  // ── readAliveMarker 损坏文件 ──
-
-  it("readAliveMarker returns undefined for corrupted file", () => {
-    tmpDir = makeTmpDir();
-    const sessionFile = path.join(tmpDir, "session.jsonl");
-    fs.writeFileSync(`${sessionFile}.alive`, "not-json", "utf-8");
-
-    expect(readAliveMarker(sessionFile)).toBeUndefined();
-  });
-
-  it("readAliveMarker returns undefined for structurally invalid JSON", () => {
-    tmpDir = makeTmpDir();
-    const sessionFile = path.join(tmpDir, "session.jsonl");
-    // pid 字段缺失
-    fs.writeFileSync(`${sessionFile}.alive`, '{"id":"x","startedAt":1}\n', "utf-8");
-
-    expect(readAliveMarker(sessionFile)).toBeUndefined();
-  });
-
-  it("readAliveMarker returns undefined when file does not exist", () => {
-    tmpDir = makeTmpDir();
-    const sessionFile = path.join(tmpDir, "session.jsonl");
-
-    expect(readAliveMarker(sessionFile)).toBeUndefined();
-  });
-
-  // ── removeAliveMarker ──
-
-  it("removeAliveMarker removes existing sidecar", () => {
-    tmpDir = makeTmpDir();
-    const sessionFile = path.join(tmpDir, "session.jsonl");
-    const marker: AliveMarker = { pid: 1, id: "x", startedAt: 0 };
-
-    writeAliveMarker(sessionFile, marker);
-    expect(fs.existsSync(`${sessionFile}.alive`)).toBe(true);
-
-    removeAliveMarker(sessionFile);
-    expect(fs.existsSync(`${sessionFile}.alive`)).toBe(false);
-  });
-
-  it("removeAliveMarker does not throw when file does not exist", () => {
-    tmpDir = makeTmpDir();
-    const sessionFile = path.join(tmpDir, "session.jsonl");
-
-    expect(() => removeAliveMarker(sessionFile)).not.toThrow();
-  });
-
-  // ── isProcessAlive ──
-
-  it("isProcessAlive returns true for current process", () => {
-    expect(isProcessAlive(process.pid)).toBe(true);
-  });
-
-  it("isProcessAlive returns false for non-existent pid", () => {
-    // pid 0 不存在于用户空间（kill(0,0) 在某些 OS 有特殊语义，用大数更安全）
-    expect(isProcessAlive(9999999)).toBe(false);
-  });
-
-  it("isProcessAlive returns true for EPERM (process exists but no permission)", () => {
-    // 模拟 EPERM：进程存在但无权限发信号
-    const err = new Error("EPERM") as NodeJS.ErrnoException;
-    err.code = "EPERM";
-    const originalKill = process.kill;
-    process.kill = (_pid: number, _signal?: string | number) => {
-      throw err;
-    };
-    try {
-      expect(isProcessAlive(12345)).toBe(true);
-    } finally {
-      process.kill = originalKill;
-    }
-  });
-
-  it("isProcessAlive returns false for ESRCH (no such process)", () => {
-    // 模拟 ESRCH：进程不存在
-    const err = new Error("ESRCH") as NodeJS.ErrnoException;
-    err.code = "ESRCH";
-    const originalKill = process.kill;
-    process.kill = (_pid: number, _signal?: string | number) => {
-      throw err;
-    };
-    try {
-      expect(isProcessAlive(12345)).toBe(false);
-    } finally {
-      process.kill = originalKill;
-    }
-  });
-
-  it("isProcessAlive returns false for unknown errors", () => {
-    // 模拟未知错误：保守判死
-    const originalKill = process.kill;
-    process.kill = (_pid: number, _signal?: string | number) => {
-      throw new Error("unknown error");
-    };
-    try {
-      expect(isProcessAlive(12345)).toBe(false);
-    } finally {
-      process.kill = originalKill;
-    }
-  });
-});