@zhushanwen/pi-subagent-workflow 8.6.0 → 8.7.0

package/src/execution/__tests__/parent-child-matrix.test.ts

@@ -1,336 +0,0 @@
-// src/execution/__tests__/parent-child-matrix.test.ts
-//
-// SP-4: 父子联动测试（真实 SubagentService + index.ts 事件接线）。
-//
-// [M2/M7 重写] 旧套件在测试内手写 tryTransition/completeRecord/recentlyCascaded 模拟
-// （平行复刻生产逻辑），生产代码任何回归恒绿。现改为两层真实驱动：
-//   Block 1: 真实 SubagentService.onParentFork / onParentNew / disposeAllRecords
-//            → 断言 record 转 closed + closedReason + archive + pending:unregister
-//   Block 2: 真实 index.ts factory 注册的 session_before_fork / session_before_switch
-//            handler 驱动同一真实 Service → 断言事件接线与 reason 门控
-//
-// 事件语义依据 SDK 0.82.1（core/agent-session-runtime.js + core/agent-session.js）：
-//   /fork          → session_before_fork(entryId, position) → teardown("fork")
-//   /new           → session_before_switch(reason:"new") → teardown("new")
-//                    （/new 从不触发 session_before_tree——旧实现误挂即 M2 bug）
-//   /resume /import → session_before_switch(reason:"resume")
-//   /tree          → session_before_tree（同 session 内分支导航，无 session 替换）
-//                    + session_tree（导航后）——均无级联关闭语义
-
-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 type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
-
-// mock logger（import 链需要 getLogger 存在；对齐 subagent-service-parent-guard.test.ts。
-// setPiHandle 是 index.ts factory 顶层调用（注入 pi handle 给全局 logger），需一并 stub）
-const { loggerMock } = vi.hoisted(() => ({
-  loggerMock: { debug: vi.fn(), warn: vi.fn(), error: vi.fn(), info: vi.fn() },
-}));
-vi.mock("@zhushanwen/pi-extension-logger", () => ({
-  getLogger: () => loggerMock,
-  setPiHandle: vi.fn(),
-}));
-
-// mock session-runner（import 链需要 runSpawn/killAllSpawnedChildren/getChildByRecord；
-// 避免真实 kill，且 dispose 收割路径可断言）
-const { killAllSpawnedChildrenMock } = vi.hoisted(() => ({
-  killAllSpawnedChildrenMock: vi.fn(),
-}));
-vi.mock("../session-runner.ts", () => ({
-  runSpawn: vi.fn(),
-  killAllSpawnedChildren: killAllSpawnedChildrenMock,
-  getChildByRecord: vi.fn(() => undefined),
-}));
-
-import { createRecord } from "../execution-record.ts";
-import { ModelConfigService } from "../model-config-service.ts";
-import { RecordStore } from "../record-store.ts";
-import { SubagentService, setSubagentService } from "../subagent-service.ts";
-import type { PiLike } from "../subagent-service.ts";
-import type { ExecutionRecord } from "../types.ts";
-import subagentsExtension from "../../index.ts";
-
-// ── helpers ──
-
-function makeTmpAgentDir(): string {
-  return fs.mkdtempSync(path.join(os.tmpdir(), "parent-child-matrix-"));
-}
-
-/** 最小 mock PiLike（appendEntry/events.emit/sendMessage 全部可断言）。 */
-function makePi(): { pi: PiLike; eventsEmit: ReturnType<typeof vi.fn> } {
-  const eventsEmit = vi.fn();
-  const pi = {
-    appendEntry: vi.fn(),
-    events: { emit: eventsEmit },
-    sendMessage: vi.fn(),
-  } as unknown as PiLike;
-  return { pi, eventsEmit };
-}
-
-/** 暴露私有字段供测试注入 record（对齐 subagent-service-parent-guard.test.ts 模式）。 */
-interface ServiceInternals {
-  store: RecordStore;
-}
-
-/** 构造一个 running 状态的 ExecutionRecord（内存态，register 进真实 store）。 */
-function makeRunningRecord(id: string, overrides: Partial<ExecutionRecord> = {}): ExecutionRecord {
-  return createRecord(id, {
-    agent: "general-purpose",
-    model: "test/model",
-    mode: "background",
-    task: `task for ${id}`,
-    slug: `slug-${id}`,
-    startedAt: Date.now(),
-    rootSessionId: "root-session",
-    controller: new AbortController(),
-    ...overrides,
-  });
-}
-
-/** 重置进程级 SubagentService 单例槽（setSubagentService 不接受 null，测试清理用）。
- *  key 与生产 getServiceSlot 的 SERVICE_SLOT_KEY（subagent-service.ts）一致。 */
-function resetServiceSlot(): void {
-  const slot = Reflect.get(globalThis, Symbol.for("@zhushanwen/pi-subagents.service")) as
-    | { current: SubagentService | null }
-    | undefined;
-  if (slot) slot.current = null;
-}
-
-/** 创建真实 SubagentService（tmp agentDir + mock pi），返回 service 与内部 store。 */
-function setupRealService(): { service: SubagentService; store: RecordStore } {
-  const agentDir = makeTmpAgentDir();
-  const modelService = new ModelConfigService({ agentDir });
-  const service = new SubagentService({ cwd: agentDir, modelService });
-  service.initSession({ pi: makePi().pi, sessionId: "root-session" });
-  const store = (service as unknown as ServiceInternals).store;
-  return { service, store, agentDir };
-}
-
-// ============================================================
-// Block 1: 真实 SubagentService 级联关闭
-// ============================================================
-
-describe("SP-4 级联关闭（真实 SubagentService）", () => {
-  let agentDir: string;
-  let service: SubagentService;
-  let store: RecordStore;
-  let eventsEmit: ReturnType<typeof vi.fn>;
-
-  beforeEach(() => {
-    agentDir = makeTmpAgentDir();
-    const modelService = new ModelConfigService({ agentDir });
-    const { pi, eventsEmit: emit } = makePi();
-    eventsEmit = emit;
-    service = new SubagentService({ cwd: agentDir, modelService });
-    service.initSession({ pi, sessionId: "root-session" });
-    store = (service as unknown as ServiceInternals).store;
-  });
-
-  afterEach(() => {
-    service.dispose();
-    fs.rmSync(agentDir, { recursive: true, force: true });
-  });
-
-  it("onParentFork：running record 转 closed + closedReason=parent-fork，并从内存归档", () => {
-    const record = makeRunningRecord("sa-fork-1");
-    store.register(record);
-
-    const count = service.onParentFork();
-
-    expect(count).toBe(1);
-    expect(record.status).toBe("closed");
-    expect(record.closedReason).toBe("parent-fork");
-    expect(record.endedAt).toBeDefined();
-    // completeRecord 冻结合成 AgentResult（disposeAllRecords 构造，无在途执行）
-    expect(record.error).toBe("closed due to parent-fork");
-    // archive 生效：终态 record 立即从内存移除（读时从 session.jsonl 重建）
-    expect(store.getMutable("sa-fork-1")).toBeUndefined();
-    expect(store.listAllActive()).toHaveLength(0);
-  });
-
-  it("onParentNew：running record 转 closed + closedReason=parent-new", () => {
-    const record = makeRunningRecord("sa-new-1");
-    store.register(record);
-
-    const count = service.onParentNew();
-
-    expect(count).toBe(1);
-    expect(record.status).toBe("closed");
-    expect(record.closedReason).toBe("parent-new");
-    expect(store.getMutable("sa-new-1")).toBeUndefined();
-  });
-
-  it("多个 running record 全部级联关闭，返回 count 与 record 数一致", () => {
-    const records = [
-      makeRunningRecord("sa-multi-1"),
-      makeRunningRecord("sa-multi-2"),
-      makeRunningRecord("sa-multi-3"),
-    ];
-    for (const r of records) store.register(r);
-
-    const count = service.onParentNew();
-
-    expect(count).toBe(3);
-    for (const r of records) {
-      expect(r.status).toBe("closed");
-      expect(r.closedReason).toBe("parent-new");
-    }
-    expect(store.listAllActive()).toHaveLength(0);
-  });
-
-  it("级联关闭对每个被关 record 发 pending:unregister（reason=closed）", () => {
-    const record = makeRunningRecord("sa-pending-1");
-    store.register(record);
-
-    service.onParentFork();
-
-    expect(eventsEmit).toHaveBeenCalledWith("pending:unregister", {
-      id: "sa-pending-1",
-      reason: "closed",
-    });
-  });
-
-  it("幂等：无活跃 record 返回 0；级联后再次调用返回 0（record 已归档）", () => {
-    expect(service.onParentNew()).toBe(0);
-
-    const record = makeRunningRecord("sa-idem-1");
-    store.register(record);
-    expect(service.onParentFork()).toBe(1);
-    expect(service.onParentFork()).toBe(0); // 已 closed + archived，不重复关
-  });
-});
-
-// ============================================================
-// Block 2: index.ts 事件接线（真实 factory + 真实 Service）
-// ============================================================
-
-describe("SP-4 index.ts 事件接线", () => {
-  type PiHandler = (...args: unknown[]) => void | Promise<void>;
-  const registered = new Map<string, PiHandler>();
-  let processOnSpy: ReturnType<typeof vi.spyOn>;
-  let agentDir: string;
-  let service: SubagentService;
-  let store: RecordStore;
-
-  /** 最小 mock ExtensionAPI：捕获 pi.on 注册的 handler（对齐 process-shutdown-hook.test.ts 模式）。 */
-  function createMockPi(): ExtensionAPI {
-    const noop = (): void => {
-      /* mock */
-    };
-    return {
-      registerTool: vi.fn(),
-      registerCommand: vi.fn(),
-      registerMessageRenderer: vi.fn(),
-      on: (event: string, handler: PiHandler) => {
-        registered.set(event, handler);
-      },
-      appendEntry: vi.fn(),
-      events: { emit: vi.fn(), on: vi.fn() },
-    } as unknown as ExtensionAPI;
-  }
-
-  beforeEach(() => {
-    // 捕获 process.on 注册（不真实注册，防 listener 跨用例泄漏）——factory 顶层注册
-    // SIGTERM/SIGINT/beforeExit 三个 process hook（见 process-shutdown-hook.test.ts）。
-    processOnSpy = vi.spyOn(process, "on").mockImplementation((() => process) as never);
-
-    registered.clear();
-    killAllSpawnedChildrenMock.mockReset();
-
-    const ctx = setupRealService();
-    agentDir = ctx.agentDir;
-    service = ctx.service;
-    store = ctx.store;
-    setSubagentService(service);
-
-    subagentsExtension(createMockPi());
-  });
-
-  afterEach(() => {
-    processOnSpy.mockRestore();
-    service.dispose();
-    fs.rmSync(agentDir, { recursive: true, force: true });
-    resetServiceSlot();
-  });
-
-  it("factory 注册 session_before_fork 与 session_before_switch handler", () => {
-    expect(registered.get("session_before_fork")).toBeTypeOf("function");
-    expect(registered.get("session_before_switch")).toBeTypeOf("function");
-  });
-
-  it("[M2] 不注册 session_before_tree——/tree 同 session 分支导航不得级联杀 subagent", () => {
-    // SDK 中 session_before_tree 只由 navigateTree（/tree）触发，与 /new 无关。
-    // 旧实现误挂此事件当 /new 级联入口 → 普通 /tree 导航误杀全部活跃 subagent。
-    // 本断言是 M2 回归守卫：该事件不得再注册级联 handler。
-    expect(registered.has("session_before_tree")).toBe(false);
-  });
-
-  it("[M2] session_before_switch(reason:'new') 触发 /new 级联：record 转 closed + parent-new", () => {
-    const record = makeRunningRecord("sa-wire-new-1");
-    store.register(record);
-    const onParentNewSpy = vi.spyOn(service, "onParentNew");
-
-    registered.get("session_before_switch")!(
-      { type: "session_before_switch", reason: "new" },
-      undefined,
-    );
-
-    expect(onParentNewSpy).toHaveBeenCalledTimes(1);
-    expect(record.status).toBe("closed");
-    expect(record.closedReason).toBe("parent-new");
-    expect(store.getMutable("sa-wire-new-1")).toBeUndefined();
-  });
-
-  it("[M2] session_before_switch(reason:'resume') 不级联：/resume 回到已有 session，record 保持 running", () => {
-    const record = makeRunningRecord("sa-wire-resume-1");
-    store.register(record);
-    const onParentNewSpy = vi.spyOn(service, "onParentNew");
-
-    registered.get("session_before_switch")!(
-      {
-        type: "session_before_switch",
-        reason: "resume",
-        targetSessionFile: "/tmp/other-session.jsonl",
-      },
-      undefined,
-    );
-
-    expect(onParentNewSpy).not.toHaveBeenCalled();
-    expect(record.status).toBe("running");
-    expect(store.getMutable("sa-wire-resume-1")).toBe(record);
-  });
-
-  it("session_before_fork 触发 /fork 级联：record 转 closed + parent-fork", () => {
-    const record = makeRunningRecord("sa-wire-fork-1");
-    store.register(record);
-    const onParentForkSpy = vi.spyOn(service, "onParentFork");
-
-    registered.get("session_before_fork")!(
-      { type: "session_before_fork", entryId: "entry-1", position: "before" },
-      undefined,
-    );
-
-    expect(onParentForkSpy).toHaveBeenCalledTimes(1);
-    expect(record.status).toBe("closed");
-    expect(record.closedReason).toBe("parent-fork");
-    expect(store.getMutable("sa-wire-fork-1")).toBeUndefined();
-  });
-
-  it("service 单例未初始化（getSubagentService()=null）时 handler 安全 no-op", () => {
-    resetServiceSlot();
-    expect(() => {
-      registered.get("session_before_switch")!(
-        { type: "session_before_switch", reason: "new" },
-        undefined,
-      );
-      registered.get("session_before_fork")!(
-        { type: "session_before_fork", entryId: "e", position: "before" },
-        undefined,
-      );
-    }).not.toThrow();
-  });
-});

package/src/execution/__tests__/path-encoding.test.ts

@@ -1,104 +0,0 @@
-// src/__tests__/path-encoding.test.ts
-//
-// 锁定 encodeCwd 契约：session-runner 与 session-file-gc 共用此编码，
-// 漂移会导致同一 cwd 落到两个不同目录（见 path-encoding.ts 顶部注释）。
-import * as path from "node:path";
-
-import { describe, expect, it } from "vitest";
-
-import { encodeCwd, getSubagentRecordsDir, getSubagentSessionDir } from "../path-encoding.ts";
-
-describe("encodeCwd", () => {
-  it("encodes a normal unix absolute path", () => {
-    expect(encodeCwd("/Users/x/proj")).toBe("--Users-x-proj--");
-  });
-
-  it("strips a single leading backslash", () => {
-    expect(encodeCwd("\\foo")).toBe("--foo--");
-  });
-
-  it("encodes Windows drive letter (colon + backslash)", () => {
-    // C:\proj → 去掉无前导分隔符 → C:\proj → : 和 \ 都替换为 - → C--proj
-    expect(encodeCwd("C:\\proj")).toBe("--C--proj--");
-  });
-
-  it("encodes empty string to bare delimiter pair", () => {
-    expect(encodeCwd("")).toBe("----");
-  });
-
-  it("encodes relative path with no leading separator", () => {
-    expect(encodeCwd("relative/path")).toBe("--relative-path--");
-  });
-
-  it("collapses consecutive separators (forward slash)", () => {
-    expect(encodeCwd("/a//b")).toBe("--a--b--");
-  });
-
-  it("collapses consecutive separators (backslash)", () => {
-    expect(encodeCwd("a\\b\\c")).toBe("--a-b-c--");
-  });
-
-  it("encodes mixed separators and colon", () => {
-    // /x:y\\z → 去前导 / → x:y\z → : \ → - - → x-y-z
-    expect(encodeCwd("/x:y\\z")).toBe("--x-y-z--");
-  });
-});
-
-describe("getSubagentSessionDir", () => {
-  it("returns agentDir/subagents/<encodedCwd>/sessions", () => {
-    // [MF#1] 既有布局：subagents/<enc>/sessions/（不改，避免升级用户既有数据 orphan）
-    const result = getSubagentSessionDir("/home/user/.pi/agent", "/home/user/project");
-    expect(result).toBe(
-      path.join("/home/user/.pi/agent", "subagents", "--home-user-project--", "sessions")
-    );
-  });
-
-  it("uses mainCwd encoding (not effectiveCwd)", () => {
-    // D-004: 用主 cwd 编码，保证同一主 cwd 下所有 subagent 存同一目录
-    const mainCwd = "/Users/zhushanwen/Code/my-project";
-    const result = getSubagentSessionDir("~/.pi/agent", mainCwd);
-    const encoded = encodeCwd(mainCwd);
-    expect(result).toBe(path.join("~/.pi/agent", "subagents", encoded, "sessions"));
-  });
-
-  it("produces consistent path for same inputs", () => {
-    const a = getSubagentSessionDir("/agent", "/cwd");
-    const b = getSubagentSessionDir("/agent", "/cwd");
-    expect(a).toBe(b);
-  });
-
-  it("produces different path for different mainCwd", () => {
-    const a = getSubagentSessionDir("/agent", "/cwd1");
-    const b = getSubagentSessionDir("/agent", "/cwd2");
-    expect(a).not.toBe(b);
-  });
-});
-
-describe("getSubagentRecordsDir", () => {
-  it("returns agentDir/subagents/<encodedCwd>/records", () => {
-    const result = getSubagentRecordsDir("/home/user/.pi/agent", "/Users/x/proj");
-    expect(result).toBe(
-      path.join("/home/user/.pi/agent", "subagents", "--Users-x-proj--", "records")
-    );
-  });
-
-  it("shares the same <enc> segment with getSubagentSessionDir for same cwd", () => {
-    // F1：records 与 sessions 在同一 <enc> 段下，仅尾段不同（records vs sessions）。
-    // worktree 场景三者恒等的前提就靠这个共享 enc。
-    const agentDir = "/home/user/.pi/agent";
-    const cwd = "/Users/x/proj";
-    const sessions = getSubagentSessionDir(agentDir, cwd);
-    const records = getSubagentRecordsDir(agentDir, cwd);
-    // 共享 enc：去掉尾段后剩同一路径前缀
-    expect(path.dirname(sessions)).toBe(path.dirname(records));
-    // 尾段不同
-    expect(path.basename(sessions)).toBe("sessions");
-    expect(path.basename(records)).toBe("records");
-  });
-
-  it("produces different path for different mainCwd", () => {
-    const a = getSubagentRecordsDir("/agent", "/cwd1");
-    const b = getSubagentRecordsDir("/agent", "/cwd2");
-    expect(a).not.toBe(b);
-  });
-});

package/src/execution/__tests__/pi-invocation.test.ts

@@ -1,134 +0,0 @@
-// src/__tests__/pi-invocation.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 { getPiInvocation } from "../pi-invocation.ts";
-import { RELAY_ENV_NODE, RELAY_ENV_SCRIPT, RELAY_ENV_SOCKET } from "../relay-env.ts";
-
-describe("getPiInvocation", () => {
-  const originalArgv = process.argv;
-  const originalExecPath = process.execPath;
-  let tmpScript: string;
-
-  afterEach(() => {
-    Object.defineProperty(process, "argv", { value: originalArgv, configurable: true });
-    Object.defineProperty(process, "execPath", { value: originalExecPath, configurable: true });
-    if (tmpScript && fs.existsSync(tmpScript)) fs.unlinkSync(tmpScript);
-  });
-
-  it("真实脚本路径存在 → node <script> <userArgs>", () => {
-    // 创建真实临时脚本文件（避免 ESM spy 限制）
-    tmpScript = path.join(os.tmpdir(), `pi-inv-test-${Date.now()}.mjs`);
-    fs.writeFileSync(tmpScript, "// test");
-    Object.defineProperty(process, "argv", { value: ["node", tmpScript], configurable: true });
-    Object.defineProperty(process, "execPath", { value: "/usr/bin/node", configurable: true });
-
-    const result = getPiInvocation(["--mode", "json", "Task: x"]);
-    expect(result.command).toBe("/usr/bin/node");
-    expect(result.args).toEqual([tmpScript, "--mode", "json", "Task: x"]);
-  });
-
-  it("bun 虚拟脚本（/$bunfs/root/）→ 退化到 pi-in-PATH", () => {
-    tmpScript = "";
-    const virtualScript = "/$bunfs/root/pi";
-    Object.defineProperty(process, "argv", { value: ["bun", virtualScript], configurable: true });
-    Object.defineProperty(process, "execPath", { value: "/usr/bin/bun", configurable: true });
-
-    const result = getPiInvocation(["--mode", "json"]);
-    expect(result.command).toBe("pi");
-    expect(result.args).toEqual(["--mode", "json"]);
-  });
-
-  it("非通用 runtime（pi standalone binary）→ 直接 execPath", () => {
-    tmpScript = "";
-    Object.defineProperty(process, "argv", { value: ["/usr/bin/pi", "/nonexistent"], configurable: true });
-    Object.defineProperty(process, "execPath", { value: "/usr/local/bin/pi-binary", configurable: true });
-
-    const result = getPiInvocation(["--mode", "json"]);
-    expect(result.command).toBe("/usr/local/bin/pi-binary");
-    expect(result.args).toEqual(["--mode", "json"]);
-  });
-
-  it("node 通用 runtime + 脚本不存在 → pi-in-PATH", () => {
-    tmpScript = "";
-    Object.defineProperty(process, "argv", { value: ["node", "/nonexistent"], configurable: true });
-    Object.defineProperty(process, "execPath", { value: "/usr/bin/node", configurable: true });
-
-    const result = getPiInvocation(["--mode", "json"]);
-    expect(result.command).toBe("pi");
-    expect(result.args).toEqual(["--mode", "json"]);
-  });
-
-  it("空 userArgs 合法（仅 command + 空 args）", () => {
-    tmpScript = "";
-    Object.defineProperty(process, "argv", { value: ["node"], configurable: true });
-    Object.defineProperty(process, "execPath", { value: "/usr/bin/node", configurable: true });
-
-    const result = getPiInvocation([]);
-    expect(result.command).toBe("pi");
-    expect(result.args).toEqual([]);
-  });
-});
-
-describe("getPiInvocation relay 分支（E 方案 §5.1）", () => {
-  const originalArgv = process.argv;
-  const originalExecPath = process.execPath;
-  const RELAY_ENVS = [RELAY_ENV_SOCKET, RELAY_ENV_NODE, RELAY_ENV_SCRIPT] as const;
-  /** 三 env 齐备的基准值（改 argv/execPath 隔离直连分支，只测 relay 判定本身）。 */
-  const RELAY_FULL: Record<string, string> = {
-    [RELAY_ENV_SOCKET]: "/tmp/relay-test.sock",
-    [RELAY_ENV_NODE]: "/usr/bin/relay-node",
-    [RELAY_ENV_SCRIPT]: "/opt/relay/relay.mjs",
-  };
-
-  afterEach(() => {
-    Object.defineProperty(process, "argv", { value: originalArgv, configurable: true });
-    Object.defineProperty(process, "execPath", { value: originalExecPath, configurable: true });
-    for (const key of RELAY_ENVS) delete process.env[key];
-  });
-
-  /** 形态 1：三 env 齐备 + 默认 opts → relay invocation（command=NODE、args[0]=SCRIPT）。 */
-  it("三 env 齐备（无 opts）→ 代理形态 <NODE> <SCRIPT> <userArgs>", () => {
-    Object.assign(process.env, RELAY_FULL);
-    const result = getPiInvocation(["--mode", "rpc"]);
-    expect(result.command).toBe(RELAY_FULL[RELAY_ENV_NODE]);
-    expect(result.args).toEqual([RELAY_FULL[RELAY_ENV_SCRIPT], "--mode", "rpc"]);
-  });
-
-  /** 形态 1 变体：opts.relay === true 显式开启，行为与缺省一致。 */
-  it("三 env 齐备 + opts.relay=true → 同缺省（显式开启非必填）", () => {
-    Object.assign(process.env, RELAY_FULL);
-    const result = getPiInvocation(["--mode", "rpc"], { relay: true });
-    expect(result.command).toBe(RELAY_FULL[RELAY_ENV_NODE]);
-    expect(result.args[0]).toBe(RELAY_FULL[RELAY_ENV_SCRIPT]);
-  });
-
-  /** 形态 2：三缺一逐个枚举 → 回落现状直连（全有或全无，E-TUI 零回归）。 */
-  it("三缺一（逐个枚举）→ 现状直连分支", () => {
-    // 直连分支落到「node + 不存在脚本 → pi-in-PATH」可稳定断言（不依赖测试机 execPath 形态）
-    Object.defineProperty(process, "argv", { value: ["node", "/nonexistent"], configurable: true });
-    Object.defineProperty(process, "execPath", { value: "/usr/bin/node", configurable: true });
-    for (const missing of RELAY_ENVS) {
-      const partial = { ...RELAY_FULL };
-      delete partial[missing];
-      for (const key of RELAY_ENVS) delete process.env[key];
-      Object.assign(process.env, partial);
-      const result = getPiInvocation(["--mode", "rpc"]);
-      expect(result.command).toBe("pi"); // 分支 3：通用 runtime + 脚本不存在 → PATH
-      expect(result.args).toEqual(["--mode", "rpc"]);
-    }
-  });
-
-  /** 形态 3：probe 排除——显式 relay:false 时即便三 env 齐备也直连。 */
-  it("relay:false 强制直连（probe 语义：探 pi 本体，不探 runtime 健康）", () => {
-    Object.assign(process.env, RELAY_FULL);
-    Object.defineProperty(process, "argv", { value: ["node", "/nonexistent"], configurable: true });
-    Object.defineProperty(process, "execPath", { value: "/usr/bin/node", configurable: true });
-    const result = getPiInvocation(["--version"], { relay: false });
-    expect(result.command).toBe("pi");
-    expect(result.args).toEqual(["--version"]);
-  });
-});