@zhushanwen/pi-subagent-workflow 8.5.0 → 8.7.0

package/src/execution/__tests__/model-resolver.test.ts

@@ -1,413 +0,0 @@
-// src/__tests__/model-resolver.test.ts
-import { afterEach, beforeEach, describe, expect, it } from "vitest";
-
-import {
-  availableThinkingLevels,
-  type ModelInfo,
-  type ModelRegistryLike,
-  resolveModel,
-} from "../model-resolver.ts";
-
-// ============================================================
-// helpers
-// ============================================================
-
-function makeModel(over: Partial<ModelInfo> = {}): ModelInfo {
-  return {
-    id: over.id ?? "sonnet-4-5",
-    name: over.name ?? "Claude Sonnet 4.5",
-    provider: over.provider ?? "anthropic",
-    reasoning: over.reasoning ?? false,
-    thinkingLevelMap: over.thinkingLevelMap,
-    contextWindow: over.contextWindow,
-  };
-}
-
-/** 构造 mock registry：registered 表 (provider/modelId) → ModelInfo；authed 集合控制鉴权。 */
-function makeRegistry(models: ModelInfo[], authed: string[] = models.map((m) => `${m.provider}/${m.id}`)): ModelRegistryLike {
-  const authSet = new Set(authed);
-  return {
-    getAvailable: () => models,
-    find: (provider, modelId) => models.find((m) => m.provider === provider && m.id === modelId),
-    hasConfiguredAuth: (m) => {
-      if (!m || typeof m !== "object") return false;
-      const mm = m as ModelInfo;
-      return authSet.has(`${mm.provider}/${mm.id}`);
-    },
-  };
-}
-
-/** 主 agent model（第三层兼底）。 */
-const ctxModel = makeModel({ id: "main-model", provider: "main" });
-
-// ============================================================
-// resolveModel — 三层优先级
-// ============================================================
-
-describe("resolveModel — three-layer priority", () => {
-  it("L1: paramOverride.model wins over agentConfig and ctxModel", () => {
-    const m1 = makeModel({ id: "explicit", provider: "p1" });
-    const reg = makeRegistry([m1]);
-    const r = resolveModel(
-      { name: "worker", systemPrompt: "", model: "main/agent-md" },
-      reg,
-      { model: "p1/explicit" },
-      ctxModel,
-    );
-    expect(r.model.id).toBe("explicit");
-  });
-
-  it("L2: agentConfig.model used when no paramOverride", () => {
-    const m = makeModel({ id: "agent-md-model", provider: "ap" });
-    const reg = makeRegistry([m]);
-    const r = resolveModel(
-      { name: "worker", systemPrompt: "", model: "ap/agent-md-model" },
-      reg,
-      undefined,
-      ctxModel,
-    );
-    expect(r.model.id).toBe("agent-md-model");
-  });
-
-  it("L3: ctxModel (main agent model) used when no override and no agentConfig.model", () => {
-    const reg = makeRegistry([]);
-    const r = resolveModel(
-      { name: "worker", systemPrompt: "" },
-      reg,
-      undefined,
-      ctxModel,
-    );
-    expect(r.model).toBe(ctxModel);
-    expect(r.thinkingLevel).toBeUndefined();
-  });
-
-  it("L3: ctxModel used even when registry is empty (no lookup needed)", () => {
-    const reg = makeRegistry([]);
-    const r = resolveModel(undefined, reg, undefined, ctxModel);
-    expect(r.model).toBe(ctxModel);
-  });
-});
-
-// ============================================================
-// 显式指定失败的错误行为
-// ============================================================
-
-describe("resolveModel — explicit override failures throw (no silent fallback)", () => {
-  it("paramOverride.model not in registry → throws (does NOT fall back to ctxModel)", () => {
-    const reg = makeRegistry([]);
-    expect(() =>
-      resolveModel(undefined, reg, { model: "x/nonexistent" }, ctxModel),
-    ).toThrow(/not found in registry/);
-  });
-
-  it("paramOverride.model found but auth missing → throws auth-specific message", () => {
-    const m = makeModel({ id: "unauthed", provider: "u" });
-    const reg = makeRegistry([m], []); // 无鉴权
-    expect(() =>
-      resolveModel(undefined, reg, { model: "u/unauthed" }, ctxModel),
-    ).toThrow(/exists but auth is not configured/);
-  });
-
-  it("agentConfig.model not in registry → throws", () => {
-    const reg = makeRegistry([]);
-    expect(() =>
-      resolveModel(
-        { name: "worker", systemPrompt: "", model: "x/missing" },
-        reg,
-        undefined,
-        ctxModel,
-      ),
-    ).toThrow(/not found in registry/);
-  });
-
-  it("no override, no agentConfig.model, no ctxModel → throws listing available", () => {
-    const m = makeModel({ id: "visible", provider: "v" });
-    const reg = makeRegistry([m]);
-    expect(() => resolveModel(undefined, reg, undefined, undefined)).toThrow(
-      /No available model.*Available models/s,
-    );
-  });
-
-  it("invalid model string (no slash) → throws", () => {
-    const reg = makeRegistry([]);
-    expect(() =>
-      resolveModel(undefined, reg, { model: "no-slash" }, ctxModel),
-    ).toThrow(/not found in registry/);
-  });
-});
-
-// ============================================================
-// thinkingLevel 解析
-// ============================================================
-
-describe("resolveModel — thinkingLevel resolution", () => {
-  it("override path: returns requested thinkingLevel when model supports it", () => {
-    const m = makeModel({
-      id: "reasoning-model",
-      provider: "rp",
-      reasoning: true,
-      thinkingLevelMap: { low: 1, high: 2, xhigh: 3 },
-    });
-    const reg = makeRegistry([m]);
-    const r = resolveModel(undefined, reg, { model: "rp/reasoning-model", thinkingLevel: "high" });
-    expect(r.thinkingLevel).toBe("high");
-  });
-
-  it("override path: clamps down to highest available when requested unsupported", () => {
-    const m = makeModel({
-      id: "limited-model",
-      provider: "lp",
-      reasoning: true,
-      thinkingLevelMap: { low: 1, medium: 2 }, // 不含 xhigh
-    });
-    const reg = makeRegistry([m]);
-    const r = resolveModel(undefined, reg, { model: "lp/limited-model", thinkingLevel: "xhigh" });
-    expect(r.thinkingLevel).toBe("medium");
-  });
-
-  it("override path: returns undefined when model.reasoning === false", () => {
-    const m = makeModel({ id: "non-reasoning", provider: "nr", reasoning: false });
-    const reg = makeRegistry([m]);
-    const r = resolveModel(undefined, reg, { model: "nr/non-reasoning", thinkingLevel: "high" });
-    expect(r.thinkingLevel).toBeUndefined();
-  });
-
-  it("ctxModel path: thinkingLevel from paramOverride.thinkingLevel (pass-through, no clamp)", () => {
-    const reg = makeRegistry([]);
-    const r = resolveModel(undefined, reg, { thinkingLevel: "high" }, ctxModel);
-    // ctxModel reasoning=false，但 ctxModel 路径不 clamp（主 agent model 直接透传）
-    expect(r.thinkingLevel).toBe("high");
-  });
-
-  it("ctxModel path: thinkingLevel from agentConfig.thinkingLevel when no paramOverride", () => {
-    const reg = makeRegistry([]);
-    const r = resolveModel(
-      { name: "worker", systemPrompt: "", thinkingLevel: "medium" },
-      reg,
-      undefined,
-      ctxModel,
-    );
-    expect(r.thinkingLevel).toBe("medium");
-  });
-
-  it("ctxModel path: thinkingLevel undefined when no override and ctxModel non-reasoning", () => {
-    const reg = makeRegistry([]);
-    // ctxModel（makeModel 默认 reasoning=false）无 override → 非 reasoning 不支持 thinking
-    const r = resolveModel(undefined, reg, undefined, ctxModel);
-    expect(r.thinkingLevel).toBeUndefined();
-  });
-
-  // ── 默认 thinking 兜底：无显式指定时用「模型最高可用档」（不落 pi 默认 medium）──
-
-  it("ctxModel path: no override + reasoning model with map → highest mapped level", () => {
-    const reg = makeRegistry([]);
-    const m = makeModel({ id: "r-main", provider: "rm", reasoning: true, thinkingLevelMap: { low: 1, high: 2, xhigh: 3 } });
-    const r = resolveModel(undefined, reg, undefined, m);
-    expect(r.thinkingLevel).toBe("xhigh");
-  });
-
-  it("ctxModel path: no override + reasoning model with map containing max → max", () => {
-    const reg = makeRegistry([]);
-    const m = makeModel({ id: "r-max", provider: "rm", reasoning: true, thinkingLevelMap: { high: 2, max: 99 } });
-    const r = resolveModel(undefined, reg, undefined, m);
-    expect(r.thinkingLevel).toBe("max");
-  });
-
-  it("ctxModel path: no override + reasoning model without map → xhigh (pi-legal ceiling)", () => {
-    const reg = makeRegistry([]);
-    // 无 map 时不能用 "max"——pi 侧 model:"max" 后缀非法会导致 model 解析失败
-    const m = makeModel({ id: "r-nomap", provider: "rm", reasoning: true });
-    const r = resolveModel(undefined, reg, undefined, m);
-    expect(r.thinkingLevel).toBe("xhigh");
-  });
-
-  it("override path: no thinkingLevel param → defaults to model's highest mapped level", () => {
-    const m = makeModel({
-      id: "default-thinking",
-      provider: "dt",
-      reasoning: true,
-      thinkingLevelMap: { minimal: 1, medium: 2, high: 3 },
-    });
-    const reg = makeRegistry([m]);
-    const r = resolveModel(undefined, reg, { model: "dt/default-thinking" });
-    expect(r.thinkingLevel).toBe("high");
-  });
-});
-
-// ============================================================
-// lookupModel 容错：剥离 ":thinkingLevel" 后缀（A）
-// ============================================================
-
-describe('resolveModel — strips ":thinkingLevel" suffix from model string (A)', () => {
-  it('resolves model passed with ":xhigh" suffix', () => {
-    const m = makeModel({ id: "ds-pro", provider: "deepseek-router", reasoning: true, thinkingLevelMap: { xhigh: 3 } });
-    const reg = makeRegistry([m]);
-    const r = resolveModel(undefined, reg, { model: "deepseek-router/ds-pro:xhigh" }, ctxModel);
-    expect(r.model.id).toBe("ds-pro");
-  });
-
-  it('resolves model passed with ":high" suffix (registry has no suffix)', () => {
-    const m = makeModel({ id: "sonnet", provider: "anthropic", reasoning: true, thinkingLevelMap: { high: 2 } });
-    const reg = makeRegistry([m]);
-    const r = resolveModel(undefined, reg, { model: "anthropic/sonnet:high" });
-    expect(r.model.id).toBe("sonnet");
-  });
-
-  it('strips ":off" suffix (off is a valid thinking level)', () => {
-    const m = makeModel({ id: "m1", provider: "p", reasoning: false });
-    const reg = makeRegistry([m]);
-    const r = resolveModel(undefined, reg, { model: "p/m1:off" });
-    expect(r.model.id).toBe("m1");
-  });
-
-  it('does NOT strip unrelated colon suffix (e.g. ":foo")', () => {
-    // ":foo" 不是合法 thinking level，不剥离 → 查不到 → 抛 not found
-    const m = makeModel({ id: "m1", provider: "p", reasoning: false });
-    const reg = makeRegistry([m]);
-    expect(() => resolveModel(undefined, reg, { model: "p/m1:foo" })).toThrow(/not found in registry/);
-  });
-
-  it('suffix-stripped resolve still respects explicit thinkingLevel param', () => {
-    const m = makeModel({ id: "ds-pro", provider: "deepseek-router", reasoning: true, thinkingLevelMap: { high: 2, xhigh: 3 } });
-    const reg = makeRegistry([m]);
-    // model 带 ":xhigh" 但 thinkingLevel param 指定 high -> thinking 取 high
-    const r = resolveModel(undefined, reg, { model: "deepseek-router/ds-pro:xhigh", thinkingLevel: "high" });
-    expect(r.model.id).toBe("ds-pro");
-    expect(r.thinkingLevel).toBe("high");
-  });
-});
-
-// ============================================================
-// not-found 错误信息：列出相近可用 model（B）
-// ============================================================
-
-describe("resolveModel — not-found error suggests similar models (B)", () => {
-  it("not-found with available registry lists similar models", () => {
-    const m = makeModel({ id: "ds-pro", provider: "deepseek-router" });
-    const reg = makeRegistry([m]);
-    let msg = "";
-    try {
-      resolveModel(undefined, reg, { model: "deepseek-router/ds-por" }, ctxModel); // 拼写错误
-    } catch (e) {
-      msg = (e as Error).message;
-    }
-    expect(msg).toMatch(/not found in registry/);
-    expect(msg).toMatch(/deepseek-router\/ds-pro/); // 建议列表含正确拼写
-  });
-
-  it("not-found with empty registry reports no available models", () => {
-    const reg = makeRegistry([]);
-    expect(() => resolveModel(undefined, reg, { model: "x/none" }, ctxModel)).toThrow(
-      /Registry has no available models/,
-    );
-  });
-
-  it("auth-missing error does NOT list models, points to models.json", () => {
-    const m = makeModel({ id: "unauthed", provider: "u" });
-    const reg = makeRegistry([m], []);
-    let msg = "";
-    try {
-      resolveModel(undefined, reg, { model: "u/unauthed" }, ctxModel);
-    } catch (e) {
-      msg = (e as Error).message;
-    }
-    expect(msg).toMatch(/auth is not configured/);
-    expect(msg).toMatch(/models\.json/);
-    expect(msg).not.toMatch(/Similar available models/); // auth 错误不列 model
-  });
-
-  it("P1: resolves model passed with ':max' suffix (THINKING_ORDER includes 'max')", () => {
-    const m = makeModel({ id: "mock-model", provider: "p1", reasoning: true, thinkingLevelMap: { max: 4 } });
-    const reg = makeRegistry([m]);
-    const r = resolveModel(
-      { name: "worker", systemPrompt: "" },
-      reg,
-      { model: "p1/mock-model:max" },
-      ctxModel,
-    );
-    expect(r.model).toBeTruthy();
-    expect(r.model.id).toBe("mock-model");
-    expect(r.thinkingLevel).toBe("max");
-  });
-});
-
-describe("availableThinkingLevels", () => {
-  it("returns [] when reasoning false", () => {
-    expect(availableThinkingLevels({ reasoning: false })).toEqual([]);
-  });
-  it("returns [] when no thinkingLevelMap", () => {
-    expect(availableThinkingLevels({ reasoning: true })).toEqual([]);
-  });
-  it("filters THINKING_ORDER to non-null map entries, ascending", () => {
-    expect(
-      availableThinkingLevels({ reasoning: true, thinkingLevelMap: { off: 0, high: 2, low: 1, xhigh: 3 } }),
-    ).toEqual(["off", "low", "high", "xhigh"]);
-  });
-
-  it("P2: includes 'max' as the last level (THINKING_ORDER SSOT derived)", () => {
-    const levels = availableThinkingLevels({ reasoning: true, thinkingLevelMap: { off: 0, low: 1, high: 2, xhigh: 3, max: 4 } });
-    expect(levels).toEqual(["off", "low", "high", "xhigh", "max"]);
-    expect(levels.slice(-3)).toEqual(["high", "xhigh", "max"]);
-  });
-});
-
-// ============================================================
-// ModelConfigService: initModel(ctxModel) → resolveModel L3 plumb-through
-// changeset 主打路径的 runtime 契约保护（EA-4）。
-// 链路：session_start initModel({ctxModel}) → _ctxModel 缓存 → resolveModel 第三层命中
-// ============================================================
-
-import * as fs from "node:fs";
-import * as os from "node:os";
-import * as path from "node:path";
-
-import { ModelConfigService } from "../model-config-service.ts";
-
-describe("ModelConfigService: ctx.model plumb-through (EA-4)", () => {
-  let tmpDir: string;
-
-  beforeEach(() => {
-    tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "subagents-resolver-test-"));
-  });
-  afterEach(() => {
-    fs.rmSync(tmpDir, { recursive: true, force: true });
-  });
-
-  /** 构造已 initModel 的 ModelConfigService，ctxModel 注入缓存。 */
-  function makeService(ctxModel: ModelInfo | undefined, registry: ModelRegistryLike = makeRegistry([])): ModelConfigService {
-    const svc = new ModelConfigService({ agentDir: tmpDir });
-    svc.initModel({
-      modelRegistry: registry,
-      sessionId: "test-session",
-      ctxModel,
-    });
-    return svc;
-  }
-
-  it("initModel caches ctxModel; resolveModel L3 returns it (no override, no agentConfig.model)", () => {
-    const main = makeModel({ id: "main-model", provider: "main" });
-    const svc = makeService(main);
-
-    // 无 override、无 agentConfig.model → 第三层命中缓存的 ctxModel
-    const r = svc.resolveModel("general-purpose", undefined);
-    expect(r.model).toBe(main);
-  });
-
-  it("resolveModel L3 hits cached ctxModel even with agent that has no model in frontmatter", () => {
-    const main = makeModel({ id: "inherited", provider: "parent" });
-    const svc = makeService(main);
-
-    // worker.md 无 model frontmatter → 跳过 L2，命中 L3 ctxModel
-    const r = svc.resolveModel("worker", undefined);
-    expect(r.model).toBe(main);
-  });
-
-  it("resolveModel still prefers explicit paramOverride over cached ctxModel (L1 > L3)", () => {
-    const main = makeModel({ id: "main", provider: "main" });
-    const explicit = makeModel({ id: "explicit", provider: "p1" });
-    const svc = makeService(main, makeRegistry([explicit]));
-
-    const r = svc.resolveModel("worker", { model: "p1/explicit" });
-    expect(r.model.id).toBe("explicit");
-  });
-});

package/src/execution/__tests__/nested-visibility-env-propagation.test.ts

@@ -1,287 +0,0 @@
-// src/execution/__tests__/nested-visibility-env-propagation.test.ts
-//
-// SP-8 嵌套可见性修复验证（TC-3：PI_SUBAGENT_ROOT_SESSION_ID env 贯穿）。
-//
-// 验证 env 身份贯穿机制：
-//   TC-3a: 根进程 initSession：无 env → sessionRootId = sessionId（自己是 ROOT）
-//   TC-3b: 子进程 initSession：有 env → sessionRootId = env 值（贯穿真 ROOT）
-//   TC-3c: 孙进程 initSession：env 链全程贯穿
-//   TC-3d: 子进程 execute 创建的 record.rootSessionId = env 贯穿的真 ROOT
-//   TC-3e: 子进程 collectRecords 用 sessionRootId 过滤
-//
-// mock 策略与 recursive-visibility-baseline.test.ts 一致。
-
-import type { PassThrough } from "node:stream";
-
-import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
-
-// ── mock modules ──
-
-vi.mock("node:child_process", async () => {
-  const { EventEmitter } = await import("node:events");
-  const { PassThrough } = await import("node:stream");
-
-  class FakeChild extends EventEmitter {
-    pid = 12345;
-    stdout = new PassThrough();
-    stderr = new PassThrough();
-    killed = false;
-    killSignal: string | undefined;
-    kill(sig?: string): boolean {
-      this.killed = true;
-      this.killSignal = sig;
-      return true;
-    }
-  }
-
-  return {
-    spawn: vi.fn(() => new FakeChild()),
-    // buildEnvBlock 用 execFile 异步取 git branch：默认 err-first 兜底（catch → branch=""）
-    execFile: vi.fn(
-      (
-        _cmd: string,
-        _args: readonly string[],
-        _opts: unknown,
-        cb: (err: Error | null, stdout?: string, stderr?: string) => void,
-      ) => cb(new Error("execFile not configured in this test")),
-    ),
-  };
-});
-
-vi.mock("node:fs", async () => {
-  const actual = await import("node:fs");
-  return {
-    default: {
-      ...actual,
-      mkdirSync: vi.fn(),
-      existsSync: vi.fn(() => false),
-      appendFileSync: vi.fn(),
-      writeFileSync: vi.fn(),
-      readdirSync: vi.fn(() => []),
-    },
-    mkdirSync: vi.fn(),
-    existsSync: vi.fn(() => false),
-    appendFileSync: vi.fn(),
-    writeFileSync: vi.fn(),
-    readdirSync: vi.fn(() => []),
-    promises: actual.promises,
-  };
-});
-
-vi.mock("../alive-store.ts", async (importOriginal) => {
-  const actual = await importOriginal<typeof import("../alive-store.ts")>();
-  return {
-    ...actual,
-    writeAliveMarker: vi.fn(),
-    removeAliveMarker: vi.fn(),
-  };
-});
-
-vi.mock("../finalized-marker.ts", () => ({
-  writeFinalized: vi.fn(),
-  readFinalized: vi.fn(() => false),
-}));
-
-vi.mock("../manifest-store.ts", () => {
-  class FakeManifestStore {
-    writeManifest = vi.fn(async () => {});
-    readManifest = vi.fn(async () => null);
-    listAllSync = vi.fn(() => []);
-    recoverTmpFiles = vi.fn(async () => []);
-  }
-  return { ManifestStore: vi.fn(function (_recordsDir: string) { return new FakeManifestStore(); }) };
-});
-
-vi.mock("../temp-prompt.ts", () => ({
-  writePromptToTempFile: vi.fn(async (agent: string) => {
-    const safeName = agent.replace(/[^\w.-]+/g, "_");
-    return { dir: `/tmp/fake-${safeName}`, filePath: `/tmp/fake-${safeName}/prompt-${safeName}.md` };
-  }),
-  cleanupTempPrompt: vi.fn(async () => {}),
-}));
-
-import { spawn } from "node:child_process";
-
-import { ModelConfigService } from "../model-config-service.ts";
-import type { ModelInfo, ModelRegistryLike } from "../model-resolver.ts";
-import type { RecordStore } from "../record-store.ts";
-import { createRecord } from "../execution-record.ts";
-import { SubagentService } from "../subagent-service.ts";
-
-const mockSpawn = vi.mocked(spawn);
-
-// ── env 名常量（与 subagent-service.ts 一致）──
-const ENV_ROOT_SESSION_ID = "PI_SUBAGENT_ROOT_SESSION_ID";
-const ENV_SELF_RECORD_ID = "PI_SUBAGENT_SELF_RECORD_ID";
-const ENV_DEPTH = "PI_SUBAGENT_DEPTH";
-const ENV_FORK_DEPTH = "PI_SUBAGENT_FORK_DEPTH";
-const ENV_ROOT_CWD = "PI_SUBAGENT_ROOT_CWD";
-
-// ── helpers ──
-
-interface FakeChild {
-  pid: number;
-  stdout: PassThrough;
-  stderr: PassThrough;
-  killed: boolean;
-  killSignal: string | undefined;
-  kill(sig?: string): boolean;
-  emit(event: string, ...args: unknown[]): boolean;
-}
-
-function lastSpawnedChild(): FakeChild {
-  const result = mockSpawn.mock.results.at(-1);
-  if (!result) throw new Error("spawn was not called yet");
-  return result.value as FakeChild;
-}
-
-function getLastSpawnEnv(): Record<string, string | undefined> {
-  return (mockSpawn.mock.calls.at(-1)?.[2]?.env as Record<string, string | undefined>) ?? {};
-}
-
-function sessionHeader(id = "env-prop-session"): Record<string, unknown> {
-  return { type: "session", id, timestamp: "2026-08-11T00-00-00-000Z", cwd: "/tmp/test" };
-}
-
-function emitStdoutLine(child: FakeChild, obj: Record<string, unknown>): void {
-  child.stdout.write(`${JSON.stringify(obj)}\n`);
-}
-
-async function driveChildToCompletion(child: FakeChild, events: Record<string, unknown>[] = []): Promise<void> {
-  emitStdoutLine(child, sessionHeader());
-  for (const e of events) emitStdoutLine(child, e);
-  child.stdout.end();
-  child.stderr.end();
-  child.emit("close", 0);
-}
-
-function makeEmptyRegistry(): ModelRegistryLike {
-  return { getAvailable: () => [], find: () => undefined, hasConfiguredAuth: () => true };
-}
-
-function makePi() {
-  return { sendMessage: vi.fn(), appendEntry: vi.fn(), events: { emit: vi.fn() } };
-}
-
-const ctxModel: ModelInfo = { id: "m", name: "M", provider: "p", reasoning: false };
-
-function setupService(sessionId: string, env?: Record<string, string>): SubagentService {
-  const agentDir = "/tmp/env-prop-it";
-  const modelService = new ModelConfigService({ agentDir });
-  modelService.initModel({
-    modelRegistry: makeEmptyRegistry(),
-    sessionId,
-    ctxModel,
-  });
-  const service = new SubagentService({
-    cwd: agentDir,
-    modelService,
-    getMainSessionFile: () => "/mock/main-session.jsonl",
-  });
-  service.initSession({ pi: makePi(), sessionId });
-  return service;
-}
-
-// ── 用例 ──
-
-describe("TC-3: PI_SUBAGENT_ROOT_SESSION_ID env 贯穿正确", () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-    for (const k of [ENV_ROOT_SESSION_ID, ENV_SELF_RECORD_ID, ENV_DEPTH, ENV_FORK_DEPTH, ENV_ROOT_CWD]) {
-      delete process.env[k];
-    }
-  });
-
-  afterEach(() => {
-    for (const k of [ENV_ROOT_SESSION_ID, ENV_SELF_RECORD_ID, ENV_DEPTH, ENV_FORK_DEPTH, ENV_ROOT_CWD]) {
-      delete process.env[k];
-    }
-    vi.restoreAllMocks();
-  });
-
-  it("TC-3a: 根进程 initSession：无 env → sessionRootId = sessionId（自己是 ROOT）", () => {
-    const service = setupService("root-it");
-    const sessionRootId = Reflect.get(service, "sessionRootId") as string;
-    expect(sessionRootId).toBe("root-it");
-  });
-
-  it("TC-3b: 子进程 initSession：有 env PI_SUBAGENT_ROOT_SESSION_ID → sessionRootId = env 值（贯穿真 ROOT）", () => {
-    process.env[ENV_ROOT_SESSION_ID] = "real-root-session";
-    const service = setupService("child-session");
-
-    const sessionRootId = Reflect.get(service, "sessionRootId") as string;
-    expect(sessionRootId).toBe("real-root-session"); // env 贯穿，不是 child-session
-  });
-
-  it("TC-3c: 孙进程 initSession：env 链全程贯穿，B 的 sessionRootId 仍为 ROOT", () => {
-    // 模拟 B 子进程收到的 env（A 在 spawn B 时注入）
-    process.env[ENV_ROOT_SESSION_ID] = "root-main-session";
-    process.env[ENV_SELF_RECORD_ID] = "sa-a-record";
-    process.env[ENV_DEPTH] = "1";
-
-    const service = setupService("b-session");
-
-    // B 的 sessionRootId 仍为 ROOT（env 贯穿，不被 B 自己的 sessionId 覆盖）
-    const sessionRootId = Reflect.get(service, "sessionRootId") as string;
-    expect(sessionRootId).toBe("root-main-session");
-
-    // B 的 execCtxBaseline 记录了 B 自己的身份（recordId + depth）
-    const baseline = Reflect.get(service, "execCtxBaseline") as { recordId: string; depth: number };
-    expect(baseline.recordId).toBe("sa-a-record");
-    expect(baseline.depth).toBe(1);
-  });
-
-  it("TC-3d: 子进程 execute 创建的 record.rootSessionId = env 贯穿的真 ROOT", async () => {
-    process.env[ENV_ROOT_SESSION_ID] = "root-main-session";
-    process.env[ENV_SELF_RECORD_ID] = "sa-parent";
-    process.env[ENV_DEPTH] = "1";
-
-    const service = setupService("child-session");
-
-    // execute 创建 record（模拟 B 内创建 C）
-    const handle = await service.execute({
-      task: "nested task",
-      ctxModel,
-    });
-    const store = Reflect.get(service, "store") as RecordStore;
-
-    // collectRecords 用 sessionRootId 过滤（子进程应看到 ROOT 整棵树）
-    const records = store.collectRecords(100, "all", "root-main-session");
-    const found = records.find((r) => r.id === handle.subagentId);
-
-    expect(found).toBeDefined();
-    expect(found!.rootSessionId).toBe("root-main-session"); // 指向 ROOT，不是 child-session
-    expect(found!.parentRecordId).toBe("sa-parent"); // 父链来自基线
-    expect(found!.depth).toBe(2); // 基线 depth=1 + 1 = 2
-  });
-
-  it("TC-3e: 子进程 collectRecords 用 sessionRootId 过滤（能看到 ROOT 整棵树）", () => {
-    process.env[ENV_ROOT_SESSION_ID] = "root-main-session";
-    process.env[ENV_SELF_RECORD_ID] = "sa-parent";
-    process.env[ENV_DEPTH] = "1";
-
-    const service = setupService("child-session");
-
-    // 手动注入一条 running record（避免需要真正 spawn）
-    const store = Reflect.get(service, "store") as RecordStore;
-    const recordC = createRecord("sa-c", {
-      agent: "worker",
-      model: "test/model",
-      mode: "background",
-      task: "deep nested",
-      startedAt: Date.now(),
-      rootSessionId: "root-main-session",
-      parentRecordId: "sa-parent",
-      depth: 2,
-    });
-    store.register(recordC);
-
-    // collectRecords（不带 filter）用 this.sessionRootId 过滤
-    const viaService = service.collectRecords(100);
-    const ids = viaService.map((r) => r.id);
-
-    // C 的 record 归 ROOT（rootSessionId=root-main-session），能被子进程查到
-    expect(ids).toContain("sa-c");
-    expect(viaService.find((r) => r.id === "sa-c")!.rootSessionId).toBe("root-main-session");
-  });
-});