@zhushanwen/pi-subagent-workflow 8.4.0 → 8.6.0

package/src/execution/engine/engines/pi/__tests__/pi-engine.test.ts

@@ -0,0 +1,469 @@
+// src/execution/engine/engines/pi/__tests__/pi-engine.test.ts
+//
+// PiEngine 适配层测试（P1 回填）：capabilities 链路接通口径（D3）、run 委托与
+// 中立声明→ExecuteOptions 映射、interact 三 action 直通、read 三级降级、probe 形状
+// （C1：ok=false 时 error.recovery 非空）。fake PiEngineService 注入，不 spawn 真进程。
+
+import * as fs from "node:fs";
+import * as os from "node:os";
+import * as path from "node:path";
+
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
+
+import { createRecord } from "../../../../execution-record.ts";
+import type { ExecuteOptions, SubagentRecord } from "../../../../types.ts";
+import { IDENTITY_CUSTOM_TYPE } from "../../../../session-reconstructor.ts";
+import { SubagentStream } from "../../../../stream-sink.ts";
+import type { RunContext } from "../../../port.ts";
+import type { AgentTaskSpec, EngineHandle } from "../../../types.ts";
+import {
+  PI_ADAPTER_VERSION,
+  PI_ENGINE_ID,
+  PiEngine,
+  type PiEngineService,
+} from "../pi-engine.ts";
+
+// ── fake 编排服务（结构子集，记录调用） ──
+
+interface ServiceCalls {
+  executeOpts?: ExecuteOptions;
+  signal?: AbortSignal;
+  onEvent?: (event: unknown) => void;
+  stream?: unknown;
+  deliverMessage?: { recordId: string; text: string; interrupt: boolean };
+  close?: { recordId: string; force: boolean };
+  cancelId?: string;
+  getRecordId?: string;
+}
+
+/** 构造最小完整 SubagentRecord（collectRecords 返回项——resolveRecordId 只读 id/sessionFile）。 */
+function makeListedRecord(id: string, sessionFile: string | undefined): SubagentRecord {
+  return {
+    id,
+    agent: "reviewer",
+    task: "t",
+    slug: "s",
+    status: "running",
+    mode: "background",
+    startedAt: 1,
+    rootSessionId: undefined,
+    parentRecordId: undefined,
+    depth: 0,
+    endedAt: undefined,
+    turns: 0,
+    totalTokens: 0,
+    model: "p/m",
+    thinkingLevel: undefined,
+    eventLog: [],
+    displayItems: [],
+    ...(sessionFile !== undefined ? { sessionFile } : {}),
+  };
+}
+
+function makeFakeService(overrides?: {
+  executeResult?: Awaited<ReturnType<PiEngineService["executeAndAwait"]>>;
+  executeReject?: Error;
+  recordsById?: Map<string, ReturnType<typeof createRecord>>;
+  listed?: Array<{ id: string; sessionFile?: string }>;
+  cancelResult?: boolean;
+  deliverReject?: Error;
+}): { service: PiEngineService; calls: ServiceCalls } {
+  const calls: ServiceCalls = {};
+  const record = createRecord("sa-fake", {
+    agent: "reviewer",
+    model: "p/m",
+    thinkingLevel: "high",
+    mode: "background",
+    task: "t",
+    slug: "s",
+    startedAt: 1,
+    controller: new AbortController(),
+  });
+  const service: PiEngineService = {
+    executeAndAwait: async (opts, signal, onEvent, stream) => {
+      calls.executeOpts = opts;
+      calls.signal = signal;
+      calls.onEvent = onEvent;
+      calls.stream = stream;
+      if (overrides?.executeReject) throw overrides.executeReject;
+      return (
+        overrides?.executeResult ?? {
+          content: "done",
+          durationMs: 42,
+          toolCalls: [],
+        }
+      );
+    },
+    getRecordForAction: (id) => {
+      calls.getRecordId = id;
+      // recordsById 提供时严格查表（模拟「record 不存在 → throw」路径）；缺省返回默认 record
+      if (overrides?.recordsById) {
+        const found = overrides.recordsById.get(id);
+        if (!found) {
+          throw new Error(`subagent not found or not owned: ${id}`);
+        }
+        return found;
+      }
+      return record;
+    },
+    deliverMessage: async (rec, text, interrupt) => {
+      calls.deliverMessage = { recordId: rec.id, text, interrupt };
+      if (overrides?.deliverReject) throw overrides.deliverReject;
+    },
+    closeSubagent: async (rec, force) => {
+      calls.close = { recordId: rec.id, force };
+    },
+    cancel: (id) => {
+      calls.cancelId = id;
+      return overrides?.cancelResult ?? true;
+    },
+    collectRecords: () =>
+      (overrides?.listed ?? []).map(({ id, sessionFile }) => makeListedRecord(id, sessionFile)),
+  };
+  return { service, calls };
+}
+
+/** 全字段任务声明（覆盖全部泛化点）。 */
+function makeSpec(): AgentTaskSpec {
+  return {
+    task: "do the thing",
+    slug: "do-thing",
+    agent: "reviewer",
+    model: "zai-coding-cn/glm-5.2",
+    effort: "high",
+    persona: { skillPath: "/skills/r/SKILL.md", appendSystemPrompt: ["extra"] },
+    schema: { type: "object", properties: { ok: { type: "boolean" } } },
+    maxTurns: 5,
+    graceTurns: 2,
+    fork: true,
+    worktree: false,
+    cwd: "/work",
+    conversation: true,
+    idleTimeoutMs: 99,
+  };
+}
+
+function makeHandle(sessionRef: Record<string, string>): EngineHandle {
+  return { data: { v: 1, engineId: PI_ENGINE_ID, sessionRef, poolKey: "shared", adapterVersion: PI_ADAPTER_VERSION } };
+}
+
+function makeRunCtx(overrides?: Partial<RunContext>): RunContext {
+  return { taskId: "sa-test", poolKey: "shared", ...overrides };
+}
+
+// ── capabilities ──
+
+describe("PiEngine.capabilities（D3 链路接通口径）", () => {
+  it("id = 'pi'；声明本仓 subagent 链路实际接通的能力（非 RPC 理论能力）", () => {
+    const engine = new PiEngine({ getService: () => null });
+    expect(engine.id).toBe(PI_ENGINE_ID);
+    expect(engine.capabilities()).toEqual({
+      schemaEnforcement: "native", // PI_WORKFLOW_SCHEMA env 注入链路（方案 A 唯一权威）
+      steer: "unsupported", // RPC 有 steer 但 spawn 链路未接通（session-runner no-op）
+      conversation: "native", // chatMode idle 复用 + message/close/cancel 已接通
+      personaInjection: "flag", // --skill / --append-system-prompt
+      eventGranularity: "stream", // 30+ 事件流
+      sandbox: "emulated", // 无 OS sandbox；worktree 隔离
+      sessionRead: "full", // pi JSONL 完整重建
+      resume: "native", // 同进程 idle 复用 + --session 冷续写
+      interrupt: "kill-only", // 现链路 abort = SIGTERM，rpc abort 未接通
+      permissionMode: "native", // argv-mirror 镜像主进程 flag
+    });
+  });
+});
+
+// ── run ──
+
+describe("PiEngine.run", () => {
+  it("中立声明 → ExecuteOptions 映射 + signal/onEvent/stream/ctxModel 直通（行为零变化）", async () => {
+    const { service, calls } = makeFakeService();
+    const engine = new PiEngine({ getService: () => service });
+    const spec = makeSpec();
+    const ctxModel = { id: "m", name: "M", provider: "p", reasoning: false };
+    const onEvent = (e: unknown) => void e;
+    const stream = new SubagentStream("sa-test", { setWidget: () => {} });
+    const signal = new AbortController().signal;
+
+    const { handle, outcome } = await engine.run(spec, makeRunCtx({ signal, onEvent, ctxModel, stream }));
+
+    // 传参恒等（SAR 接线前的直接调用形态）
+    expect(calls.signal).toBe(signal);
+    expect(calls.onEvent).toBe(onEvent);
+    expect(calls.stream).toBe(stream);
+    expect(calls.executeOpts).toEqual({
+      task: "do the thing",
+      slug: "do-thing",
+      agent: "reviewer",
+      model: "zai-coding-cn/glm-5.2",
+      thinkingLevel: "high", // effort 恒等映射回
+      skillPath: "/skills/r/SKILL.md", // persona 还原
+      appendSystemPrompt: ["extra"],
+      schema: spec.schema,
+      schemaEnv: JSON.stringify(spec.schema), // schema 派生（D-A6 bridge）
+      maxTurns: 5,
+      graceTurns: 2,
+      ctxModel,
+      fork: true,
+      worktree: false,
+      cwd: "/work",
+      conversation: true,
+      idleTimeoutMs: 99,
+      engine: "pi", // P4 引擎留痕（D9①）：实际执行引擎 id 进 record 投影链
+    });
+
+    // outcome 字段全集映射（缺字段在此转红——SAR outcomeToRunnerResult 的完整性依赖）
+    expect(outcome).toEqual({
+      content: "done",
+      parsedOutput: undefined,
+      usage: undefined,
+      durationMs: 42,
+      error: undefined,
+      sessionId: undefined,
+      sessionFile: undefined,
+      worktreePath: undefined,
+      toolCalls: [],
+      engineId: "pi",
+    });
+
+    // handle 自描述（engineId + sessionRef + poolKey + adapter 版本）
+    expect(handle.data).toMatchObject({
+      v: 1,
+      engineId: "pi",
+      poolKey: "shared",
+      adapterVersion: PI_ADAPTER_VERSION,
+    });
+  });
+
+  it("sessionFile 回填进 handle.sessionRef；ctx.poolKey 透传（空串兜底 'shared'）", async () => {
+    const { service } = makeFakeService({
+      executeResult: { content: "x", sessionFile: "/tmp/s.jsonl", sessionId: "sess-1", toolCalls: [] },
+    });
+    const engine = new PiEngine({ getService: () => service });
+    const r1 = await engine.run({ task: "t", slug: "s" }, makeRunCtx({ poolKey: "agent-reviewer" }));
+    expect(r1.handle.data.sessionRef).toEqual({ sessionFile: "/tmp/s.jsonl" });
+    expect(r1.handle.data.poolKey).toBe("agent-reviewer");
+    expect(r1.outcome.sessionId).toBe("sess-1");
+    const r2 = await engine.run({ task: "t", slug: "s" }, makeRunCtx({ poolKey: "" }));
+    expect(r2.handle.data.poolKey).toBe("shared");
+  });
+
+  it("服务不可用（prepare 期）→ reject 且不产生 handle", async () => {
+    const engine = new PiEngine({ getService: () => null });
+    await expect(engine.run({ task: "t", slug: "s" }, makeRunCtx())).rejects.toThrow(/SubagentService unavailable/);
+  });
+
+  it("executeAndAwait throw（嵌套超限等创建期异常）→ 向上传播（SAR catch 兜底不变）", async () => {
+    const { service } = makeFakeService({ executeReject: new Error("nesting depth exceeded") });
+    const engine = new PiEngine({ getService: () => service });
+    await expect(engine.run({ task: "t", slug: "s" }, makeRunCtx())).rejects.toThrow("nesting depth exceeded");
+  });
+});
+
+// ── interact ──
+
+describe("PiEngine.interact（chatMode 交互面直通）", () => {
+  it("message → deliverMessage(record, text, false)（interrupt 中立 action 未携带，P1 恒 followUp）", async () => {
+    const { service, calls } = makeFakeService();
+    const engine = new PiEngine({ getService: () => service });
+    const res = await engine.interact(makeHandle({ recordId: "sa-fake" }), { kind: "message", payload: "next round" });
+    expect(res).toEqual({ ok: true, delivered: true });
+    expect(calls.deliverMessage).toEqual({ recordId: "sa-fake", text: "next round", interrupt: false });
+  });
+
+  it("close → closeSubagent(record, force)；payload 缺省 force=false", async () => {
+    const { service, calls } = makeFakeService();
+    const engine = new PiEngine({ getService: () => service });
+    await engine.interact(makeHandle({ recordId: "sa-fake" }), { kind: "close", payload: { force: true } });
+    expect(calls.close).toEqual({ recordId: "sa-fake", force: true });
+    await engine.interact(makeHandle({ recordId: "sa-fake" }), { kind: "close" });
+    expect(calls.close).toEqual({ recordId: "sa-fake", force: false });
+  });
+
+  it("cancel → service.cancel(recordId)", async () => {
+    const { service, calls } = makeFakeService();
+    const engine = new PiEngine({ getService: () => service });
+    const res = await engine.interact(makeHandle({ recordId: "sa-fake" }), { kind: "cancel" });
+    expect(res).toEqual({ ok: true, delivered: true });
+    expect(calls.cancelId).toBe("sa-fake");
+  });
+
+  it("cancel 返回 false → engine_interact_failed", async () => {
+    const { service } = makeFakeService({ cancelResult: false });
+    const engine = new PiEngine({ getService: () => service });
+    const res = await engine.interact(makeHandle({ recordId: "sa-fake" }), { kind: "cancel" });
+    expect(res.ok).toBe(false);
+    if (!res.ok) {
+      expect(res.code).toBe("engine_interact_failed");
+      expect(res.message).toContain("sa-fake");
+    }
+  });
+
+  it("recordId 缺席时按 sessionFile 扫描兜底定位（collectRecords 匹配）", async () => {
+    const { service, calls } = makeFakeService({ listed: [{ id: "sa-scan-hit", sessionFile: "/x/s.jsonl" }] });
+    const engine = new PiEngine({ getService: () => service });
+    await engine.interact(makeHandle({ sessionFile: "/x/s.jsonl" }), { kind: "message", payload: "hi" });
+    expect(calls.getRecordId).toBe("sa-scan-hit");
+  });
+
+  it("死 handle（无定位符 / record 不存在）→ engine_session_not_resumable（含 cold resume 指引）", async () => {
+    const { service } = makeFakeService();
+    const engine = new PiEngine({ getService: () => service });
+    const res = await engine.interact(makeHandle({}), { kind: "message", payload: "hi" });
+    expect(res.ok).toBe(false);
+    if (!res.ok) {
+      expect(res.code).toBe("engine_session_not_resumable");
+      expect(res.message).toContain("--session");
+    }
+  });
+
+  it("交互面 throw → engine_interact_failed（message 文案透传行动语言）", async () => {
+    const { service } = makeFakeService({
+      deliverReject: new Error("subagent sa-fake is not ready for a new message"),
+    });
+    const engine = new PiEngine({ getService: () => service });
+    const res = await engine.interact(makeHandle({ recordId: "sa-fake" }), { kind: "message", payload: "hi" });
+    expect(res).toEqual({
+      ok: false,
+      code: "engine_interact_failed",
+      message: "subagent sa-fake is not ready for a new message",
+    });
+  });
+
+  it("服务不可用 → reject", async () => {
+    const engine = new PiEngine({ getService: () => null });
+    await expect(
+      engine.interact(makeHandle({ recordId: "x" }), { kind: "message", payload: "hi" }),
+    ).rejects.toThrow(/SubagentService unavailable/);
+  });
+});
+
+// ── read ──
+
+describe("PiEngine.read（D6 降级链）", () => {
+  let tmpDir: string;
+
+  beforeEach(() => {
+    tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "pi-engine-read-"));
+  });
+  afterEach(() => {
+    fs.rmSync(tmpDir, { recursive: true, force: true });
+  });
+
+  it("无 sessionRef → outcome-only（turns 空）", async () => {
+    const engine = new PiEngine({ getService: () => null });
+    const view = await engine.read(makeHandle({}));
+    expect(view).toEqual({ engineId: "pi", turns: [], source: "outcome-only" });
+  });
+
+  it("sessionFile 不存在/损坏 → outcome-only（不 throw）", async () => {
+    const engine = new PiEngine({ getService: () => null });
+    const view = await engine.read(makeHandle({ sessionFile: path.join(tmpDir, "missing.jsonl") }));
+    expect(view.source).toBe("outcome-only");
+  });
+
+  it("pi JSONL 原生读取（第①级）：turns 重建 + usage 聚合 + 内部态剥离", async () => {
+    const file = path.join(tmpDir, "s.jsonl");
+    const lines: string[] = [
+      JSON.stringify({ type: "session", version: 3, id: "sess-uuid", timestamp: "2026-01-01T00:00:00.000Z", cwd: tmpDir }),
+      JSON.stringify({
+        type: "custom", id: "id-1", parentId: null, timestamp: "2026-01-01T00:00:00.000Z",
+        customType: IDENTITY_CUSTOM_TYPE,
+        data: { id: "bg-1", agent: "worker", mode: "background", task: "t", startedAt: 500 },
+      }),
+      JSON.stringify({
+        type: "message", id: "msg-1", parentId: "id-1", timestamp: new Date(1000).toISOString(),
+        message: {
+          role: "assistant",
+          content: [
+            { type: "text", text: "hello" },
+            { type: "toolCall", id: "tc-1", name: "bash", arguments: { cmd: "ls" } },
+          ],
+          usage: { input: 10, output: 20, cacheRead: 1, cacheWrite: 2, totalTokens: 33, cost: { total: 0.5 } },
+          stopReason: "stop",
+          timestamp: 1000,
+        },
+      }),
+      JSON.stringify({
+        type: "message", id: "tr-1", parentId: "msg-1", timestamp: new Date(2000).toISOString(),
+        message: {
+          role: "toolResult", toolCallId: "tc-1", toolName: "bash",
+          content: [{ type: "text", text: "ok" }], timestamp: 2000,
+        },
+      }),
+    ];
+    fs.writeFileSync(file, lines.join("\n") + "\n");
+
+    const engine = new PiEngine({ getService: () => null });
+    const view = await engine.read(makeHandle({ sessionFile: file }));
+    expect(view.source).toBe("native");
+    expect(view.engineId).toBe("pi");
+    expect(view.sessionId).toBe("bg-1");
+    expect(view.turns).toHaveLength(1);
+    expect(view.turns[0].text).toBe("hello");
+    expect(view.turns[0].closed).toBe(true);
+    // toolCall 纯净形状（无 _status/startedTs 内部态）
+    expect(view.turns[0].toolCalls[0]).toMatchObject({ toolName: "bash" });
+    expect("_status" in view.turns[0].toolCalls[0]).toBe(false);
+    // usage 聚合（turn usageDelta → AgentUsageTotal）
+    expect(view.usage).toEqual({ input: 10, output: 20, cacheRead: 1, cacheWrite: 2, cost: 0.5, total: 33 });
+  });
+});
+
+// ── probe ──
+
+describe("PiEngine.probe（D7 + C1 形状）", () => {
+  const originalArgv = process.argv;
+  const originalExecPath = process.execPath;
+  const originalPath = process.env.PATH;
+
+  afterEach(() => {
+    Object.defineProperty(process, "argv", { value: originalArgv, configurable: true });
+    Object.defineProperty(process, "execPath", { value: originalExecPath, configurable: true });
+    process.env.PATH = originalPath;
+  });
+
+  it("探针成功：invocation + version 两级 check，engineVersion 实测", async () => {
+    const probeVersion = vi.fn(async () => "pi 0.84.1");
+    const engine = new PiEngine({ getService: () => null, probeVersion });
+    const report = await engine.probe();
+    expect(report.ok).toBe(true);
+    expect(report.engineVersion).toBe("pi 0.84.1");
+    expect(report.checks.map((c) => c.name)).toEqual(["invocation", "version"]);
+    expect(report.error).toBeUndefined();
+    expect(probeVersion).toHaveBeenCalledTimes(1);
+  });
+
+  it("版本探测失败 → ok=false 且 error.recovery 非空（C1：恢复指引必填）", async () => {
+    const engine = new PiEngine({ getService: () => null, probeVersion: async () => undefined });
+    const report = await engine.probe();
+    expect(report.ok).toBe(false);
+    expect(report.error?.code).toBe("engine_probe_failed");
+    expect(report.error?.recovery.length).toBeGreaterThan(0);
+  });
+
+  it("结果缓存：二次 probe 不重跑；force 强探重跑", async () => {
+    const probeVersion = vi.fn(async () => "pi 1.0");
+    const engine = new PiEngine({ getService: () => null, probeVersion });
+    const first = await engine.probe();
+    const second = await engine.probe();
+    expect(second).toBe(first); // 缓存命中（同对象）
+    expect(probeVersion).toHaveBeenCalledTimes(1);
+    await engine.probe({ force: true });
+    expect(probeVersion).toHaveBeenCalledTimes(2);
+  });
+
+  it("invocation 不可解析（PATH 无 pi）→ 第 1 check 失败，不再尝试版本探测", async () => {
+    // bun 虚拟脚本 + 通用 runtime → 退化为 PATH 依赖形态（command === "pi"）
+    Object.defineProperty(process, "argv", { value: ["bun", "/$bunfs/root/pi"], configurable: true });
+    Object.defineProperty(process, "execPath", { value: "/usr/bin/bun", configurable: true });
+    process.env.PATH = path.join(os.tmpdir(), `no-pi-here-${Date.now()}`);
+    const probeVersion = vi.fn(async () => "should-not-run");
+    const engine = new PiEngine({ getService: () => null, probeVersion });
+    const report = await engine.probe();
+    expect(report.ok).toBe(false);
+    expect(report.checks).toHaveLength(1);
+    expect(report.checks[0].name).toBe("invocation");
+    expect(report.checks[0].ok).toBe(false);
+    expect(probeVersion).not.toHaveBeenCalled();
+  });
+});

package/src/execution/engine/engines/pi/__tests__/reader.test.ts

@@ -0,0 +1,155 @@
+// reader.test.ts —— PiEngine reader（P5）：readPiSessionView 对 session-reconstructor
+// 的投影包装。断言三件事：①合法 JSONL → SessionView（turns/toolCalls strip 内部态/
+// usage 聚合/source='native'）；②reconstructor 降级条件（文件缺失/缺 identity/无
+// assistant）→ undefined 不 throw；③不泄漏 InternalToolCall 内部态（_status/startedTs）。
+
+import * as fs from "node:fs";
+import * as os from "node:os";
+import * as path from "node:path";
+
+import { beforeAll, describe, expect, it } from "vitest";
+
+import { readPiSessionView } from "../reader.ts";
+
+let tmpDir: string;
+
+beforeAll(() => {
+  tmpDir = fs.mkdtempSync(join(os.tmpdir(), "pi-reader-"));
+});
+
+// node:path 的 join 在 ESM 测试里直接用（避免顶部 import 与 beforeAll 顺序问题）
+function join(...segs: string[]): string {
+  return path.join(...segs);
+}
+
+/** 最小合法 subagent JSONL：header + identity + 一轮 assistant（text/thinking/toolCall）+ toolResult。 */
+function writeValidSession(file: string): void {
+  const lines = [
+    JSON.stringify({ type: "session", id: "pi-sess-1", cwd: "/proj", timestamp: "2026-08-25T10:00:00Z" }),
+    JSON.stringify({
+      type: "custom",
+      customType: "subagent-identity",
+      data: { id: "bg-1-abc", agent: "reviewer", mode: "background", task: "review code", slug: "rev", startedAt: 1756000000000 },
+    }),
+    JSON.stringify({
+      type: "message",
+      timestamp: "2026-08-25T10:00:01Z",
+      message: {
+        role: "assistant",
+        timestamp: 1756000001000,
+        content: [
+          { type: "thinking", thinking: "let me look" },
+          { type: "toolCall", id: "tc-1", name: "bash", arguments: { command: "ls" } },
+        ],
+        usage: { input: 10, output: 5, cacheRead: 1, cacheWrite: 2, cost: { total: 0.1 } },
+        stopReason: "toolUse",
+      },
+    }),
+    JSON.stringify({
+      type: "message",
+      timestamp: "2026-08-25T10:00:02Z",
+      message: { role: "toolResult", toolCallId: "tc-1", toolName: "bash", content: [{ type: "text", text: "file-a" }], timestamp: 1756000002000 },
+    }),
+    JSON.stringify({
+      type: "message",
+      timestamp: "2026-08-25T10:00:03Z",
+      message: {
+        role: "assistant",
+        timestamp: 1756000003000,
+        content: [{ type: "text", text: "all good" }],
+        usage: { input: 20, output: 8, cacheRead: 0, cacheWrite: 0, cost: { total: 0.05 } },
+        stopReason: "stop",
+      },
+    }),
+  ];
+  fs.writeFileSync(file, `${lines.join("\n")}\n`, "utf8");
+}
+
+describe("readPiSessionView", () => {
+  it("projects a valid subagent JSONL to a native SessionView", async () => {
+    const file = join(tmpDir, "valid.jsonl");
+    writeValidSession(file);
+
+    const view = await readPiSessionView(file);
+
+    expect(view).toBeDefined();
+    expect(view?.engineId).toBe("pi");
+    expect(view?.source).toBe("native");
+    // 两轮 assistant message → 两个 turn（toolResult 不产 turn）
+    expect(view?.turns).toHaveLength(2);
+
+    const [turn1, turn2] = view?.turns ?? [];
+    expect(turn1?.text).toBe("");
+    expect(turn1?.thinking).toBe("let me look");
+    expect(turn1?.toolCalls).toHaveLength(1);
+    const tc = turn1?.toolCalls[0];
+    expect(tc?.toolName).toBe("bash");
+    expect(tc?.args).toEqual({ command: "ls" });
+    expect(tc?.result).toEqual({ content: [{ type: "text", text: "file-a" }] });
+    // InternalToolCall 内部态不泄漏（导出纯净形状）
+    expect(tc).not.toHaveProperty("_status");
+    expect(tc).not.toHaveProperty("startedTs");
+    expect(turn1?.closed).toBe(true);
+
+    expect(turn2?.text).toBe("all good");
+    expect(turn2?.closed).toBe(true);
+
+    // usage 聚合：两轮 usageDelta 之和（cost 浮点累加有精度噪声，用 toBeCloseTo）
+    expect(view?.usage?.input).toBe(30)
+    expect(view?.usage?.output).toBe(13)
+    expect(view?.usage?.cacheRead).toBe(1)
+    expect(view?.usage?.cacheWrite).toBe(2)
+    expect(view?.usage?.cost).toBeCloseTo(0.15, 10)
+    expect(view?.usage?.total).toBe(46);
+  });
+
+  it("returns undefined (no throw) when file is missing", async () => {
+    await expect(readPiSessionView(join(tmpDir, "nope.jsonl"))).resolves.toBeUndefined();
+  });
+
+  it("returns undefined when identity entry is missing", async () => {
+    const file = join(tmpDir, "no-identity.jsonl");
+    fs.writeFileSync(
+      file,
+      `${JSON.stringify({ type: "session", id: "s" })}\n${JSON.stringify({
+        type: "message",
+        message: { role: "assistant", content: [{ type: "text", text: "hi" }], timestamp: 1 },
+      })}\n`,
+      "utf8",
+    );
+    await expect(readPiSessionView(file)).resolves.toBeUndefined();
+  });
+
+  it("returns undefined when no assistant message exists", async () => {
+    const file = join(tmpDir, "no-assistant.jsonl");
+    fs.writeFileSync(
+      file,
+      `${JSON.stringify({ type: "session", id: "s" })}\n${JSON.stringify({
+        type: "custom",
+        customType: "subagent-identity",
+        data: { id: "bg-2", agent: "w", mode: "background", task: "t", startedAt: 1 },
+      })}\n`,
+      "utf8",
+    );
+    await expect(readPiSessionView(file)).resolves.toBeUndefined();
+  });
+
+  it("omits usage when no message carries it", async () => {
+    const file = join(tmpDir, "no-usage.jsonl");
+    fs.writeFileSync(
+      file,
+      `${JSON.stringify({ type: "session", id: "s" })}\n${JSON.stringify({
+        type: "custom",
+        customType: "subagent-identity",
+        data: { id: "bg-3", agent: "w", mode: "background", task: "t", startedAt: 1 },
+      })}\n${JSON.stringify({
+        type: "message",
+        message: { role: "assistant", content: [{ type: "text", text: "hi" }], stopReason: "stop", timestamp: 1 },
+      })}\n`,
+      "utf8",
+    );
+    const view = await readPiSessionView(file);
+    expect(view?.turns).toHaveLength(1);
+    expect(view?.usage).toBeUndefined();
+  });
+});