@fitchmultz/pi-subagents 0.49.1 → 0.51.0

package/src/runs/shared/pi-args.ts

@@ -1,280 +0,0 @@
-import * as fs from "node:fs";
-import * as os from "node:os";
-import * as path from "node:path";
-import { fileURLToPath } from "node:url";
-import { resolveMcpDirectToolNames } from "./mcp-direct-tool-allowlist.ts";
-import {
-  childRouteEnv,
-  SUBAGENT_EAGER_TOOL_ENV,
-  SUBAGENT_INHERITED_EXTENSIONS_JSON_ENV,
-} from "./child-route-env.ts";
-export * from "./child-route-env.ts";
-import type { BuildPiArgsInput, BuildPiArgsResult } from "./pi-launch-input.ts";
-import { isUnknownArray, isRecord as isObject } from "../../shared/unknown.ts";
-import { nonempty } from "./child-presence.ts";
-import { splitKnownThinkingSuffix } from "../../shared/model-info.ts";
-import { piSessionLaunch } from "./pi-session-launch.ts";
-import type { ChildProjectTrustPolicy, ReadonlyInput } from "../../shared/types.ts";
-import { loadConfig } from "../../extension/config.ts";
-// Managed macOS environments can SIGKILL Node when one argv entry reaches ~930 UTF-8 bytes.
-// Measure the full entry (including the `Task: ` prefix), not just the task body.
-const TASK_ARG_LIMIT_BYTES = 900;
-// Resolve sibling extensions in both layouts: TypeScript sources (tests, jiti) and compiled dist output.
-const MODULE_EXTENSION = import.meta.url.endsWith(".ts") ? ".ts" : ".js";
-const PROMPT_RUNTIME_EXTENSION_PATH = path.join(
-  path.dirname(fileURLToPath(import.meta.url)),
-  `subagent-prompt-runtime${MODULE_EXTENSION}`,
-);
-const FANOUT_CHILD_EXTENSION_PATH = path.join(
-  path.dirname(fileURLToPath(import.meta.url)),
-  "..",
-  "..",
-  "extension",
-  `fanout-child${MODULE_EXTENSION}`,
-);
-
-export function applyThinkingSuffix(
-  model: string | undefined,
-  thinking: string | undefined,
-): string | undefined {
-  if (!nonempty(model) || !nonempty(thinking)) {
-    return model;
-  }
-  if (nonempty(splitKnownThinkingSuffix(model).thinkingSuffix)) {
-    return model;
-  }
-  return `${model}:${thinking}`;
-}
-
-export function normalizeChildProjectTrustPolicy(input: unknown): ChildProjectTrustPolicy {
-  if (input === "approve" || input === "no-approve" || input === "inherit") {
-    return input;
-  }
-  if (isObject(input) && "childRuns" in input) {
-    return normalizeChildProjectTrustPolicy(input.childRuns);
-  }
-  return input === undefined ? "inherit" : "no-approve";
-}
-
-export function resolveConfiguredChildProjectTrustPolicy(
-  input: unknown,
-  argv: readonly string[] = process.argv,
-): ChildProjectTrustPolicy {
-  const inherited = findInheritedProjectTrustFlag(argv);
-  const configured = isObject(input) ? input.childRuns : input;
-  const explicitInherit = configured === "inherit";
-  if (explicitInherit) {
-    return inherited ?? "inherit";
-  }
-  if (inherited === "no-approve") {
-    return "no-approve";
-  }
-  if (input === undefined || (isObject(input) && !("childRuns" in input))) {
-    return "approve";
-  }
-  const normalized = normalizeChildProjectTrustPolicy(input);
-  return normalized === "inherit" ? "approve" : normalized;
-}
-
-function findInheritedProjectTrustFlag(
-  argv: readonly string[],
-): "approve" | "no-approve" | undefined {
-  let inherited: "approve" | "no-approve" | undefined;
-  for (const arg of argv) {
-    if (arg === "--approve" || arg === "-a") {
-      inherited = "approve";
-    }
-    if (arg === "--no-approve" || arg === "-na") {
-      inherited = "no-approve";
-    }
-  }
-  return inherited;
-}
-
-export function resolveChildProjectTrustArgs(
-  policy: ChildProjectTrustPolicy = "inherit",
-  argv: readonly string[] = process.argv,
-): string[] {
-  const inherited = findInheritedProjectTrustFlag(argv);
-  if (policy === "approve") {
-    return inherited === "no-approve" ? ["--no-approve"] : ["--approve"];
-  }
-  if (policy === "no-approve") {
-    return ["--no-approve"];
-  }
-
-  if (inherited === undefined) {
-    return [];
-  }
-  return [`--${inherited}`];
-}
-
-function inheritedRuntimeExtensionPaths(env: Readonly<NodeJS.ProcessEnv> = process.env): string[] {
-  const raw = env[SUBAGENT_INHERITED_EXTENSIONS_JSON_ENV]?.trim();
-  if (!nonempty(raw)) {
-    return [];
-  }
-  try {
-    const parsed: unknown = JSON.parse(raw);
-    if (!isUnknownArray(parsed)) {
-      return [];
-    }
-    return parsed.filter(
-      (entry): entry is string => typeof entry === "string" && entry.trim().length > 0,
-    );
-  } catch {
-    return [];
-  }
-}
-
-function isExtensionTool(tool: string): boolean {
-  return tool.includes("/") || tool.endsWith(".ts") || tool.endsWith(".js");
-}
-function builtinLoadout(
-  input: ReadonlyInput<BuildPiArgsInput>,
-  declared: readonly string[],
-  fanoutAuthorized: boolean,
-  compact: boolean,
-): string[] {
-  const builtin = [...declared];
-  if (builtin.length === 0) {
-    return builtin;
-  }
-  if (fanoutAuthorized) {
-    const additions = compact
-      ? ["subagent", "delegate", "agent_runs", "load_subagent"]
-      : ["subagent"];
-    builtin.push(...additions.filter((tool) => !builtin.includes(tool)));
-  }
-  if (input.structuredOutput && !builtin.includes("structured_output")) {
-    builtin.push("structured_output");
-  }
-  if (input.mcpDirectTools !== undefined && input.mcpDirectTools.length > 0) {
-    builtin.push(...resolveMcpDirectToolNames(input.mcpDirectTools, input.cwd));
-  }
-  return builtin;
-}
-function buildToolLoadout(input: ReadonlyInput<BuildPiArgsInput>): {
-  args: string[];
-  toolExtensionPaths: string[];
-  fanoutAuthorized: boolean;
-  eager: boolean;
-} {
-  const declared = input.tools?.filter((tool) => !isExtensionTool(tool)) ?? [];
-  const fanoutAuthorized = input.allowSubagents === true || declared.includes("subagent");
-  const compact = fanoutAuthorized && loadConfig().compactChildTools !== false;
-  const tools = builtinLoadout(input, declared, fanoutAuthorized, compact);
-  return {
-    args: tools.length > 0 ? ["--tools", tools.join(",")] : [],
-    toolExtensionPaths: input.tools?.filter(isExtensionTool) ?? [],
-    fanoutAuthorized,
-    // Preserve explicit advanced tools, never inherit them into another profile.
-    eager: declared.includes("subagent"),
-  };
-}
-
-function buildExtensionArgs(
-  input: ReadonlyInput<BuildPiArgsInput>,
-  fanoutAuthorized: boolean,
-  toolExtensionPaths: readonly string[],
-): string[] {
-  const args: string[] = [];
-  const runtimeExtensions = fanoutAuthorized
-    ? [
-        PROMPT_RUNTIME_EXTENSION_PATH,
-        FANOUT_CHILD_EXTENSION_PATH,
-        ...inheritedRuntimeExtensionPaths(),
-      ]
-    : [PROMPT_RUNTIME_EXTENSION_PATH, ...inheritedRuntimeExtensionPaths()];
-  if (input.extensions !== undefined) {
-    args.push("--no-extensions");
-    for (const extPath of new Set([
-      ...runtimeExtensions,
-      ...toolExtensionPaths,
-      ...input.extensions,
-    ])) {
-      args.push("--extension", extPath);
-    }
-  } else {
-    for (const extPath of new Set([...runtimeExtensions, ...toolExtensionPaths])) {
-      args.push("--extension", extPath);
-    }
-  }
-
-  return args;
-}
-
-function buildPromptArgs(input: ReadonlyInput<BuildPiArgsInput>): {
-  args: string[];
-  tempDir?: string;
-} {
-  const args: string[] = [];
-  if (!input.inheritSkills) {
-    args.push("--no-skills");
-  }
-  if (!input.inheritProjectContext) {
-    args.push("--no-context-files");
-  }
-
-  let tempDir: string | undefined;
-  if (input.systemPrompt !== undefined && input.systemPrompt !== null) {
-    tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "pi-subagent-"));
-    const promptPath = path.join(tempDir, "system.md");
-    fs.writeFileSync(promptPath, input.systemPrompt, { mode: 0o600 });
-    args.push(
-      input.systemPromptMode === "replace" ? "--system-prompt" : "--append-system-prompt",
-      promptPath,
-    );
-  }
-
-  const taskArg = `Task: ${input.task}`;
-  if (Buffer.byteLength(taskArg) > TASK_ARG_LIMIT_BYTES) {
-    if (!nonempty(tempDir)) {
-      tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "pi-subagent-"));
-    }
-    const taskFilePath = path.join(tempDir, "task.md");
-    fs.writeFileSync(taskFilePath, taskArg, { mode: 0o600 });
-    args.push(`@${taskFilePath}`);
-  } else {
-    args.push(taskArg);
-  }
-
-  return { args, tempDir };
-}
-
-export function buildPiArgs(input: ReadonlyInput<BuildPiArgsInput>): BuildPiArgsResult {
-  const args = [...input.baseArgs, ...resolveChildProjectTrustArgs(input.projectTrust)];
-  const session = piSessionLaunch(input);
-  args.push(...session.args);
-  const modelArg = applyThinkingSuffix(input.model, input.thinking);
-  if (nonempty(modelArg)) {
-    args.push("--model", modelArg);
-  }
-  const tools = buildToolLoadout(input);
-  args.push(
-    ...tools.args,
-    ...buildExtensionArgs(input, tools.fanoutAuthorized, tools.toolExtensionPaths),
-  );
-  const prompt = buildPromptArgs(input);
-  args.push(...prompt.args);
-  return {
-    args,
-    env: {
-      ...session.env,
-      [SUBAGENT_EAGER_TOOL_ENV]: tools.eager ? "1" : undefined,
-      ...childRouteEnv(input, tools.fanoutAuthorized),
-    },
-    tempDir: prompt.tempDir,
-  };
-}
-
-export function cleanupTempDir(tempDir: string | null | undefined): void {
-  if (!nonempty(tempDir)) {
-    return;
-  }
-  try {
-    fs.rmSync(tempDir, { recursive: true, force: true });
-  } catch {
-    // Temp cleanup is best effort.
-  }
-}

package/src/runs/shared/pi-launch-input.ts

@@ -1,48 +0,0 @@
-import type { ChildProjectTrustPolicy, JsonSchemaObject } from "../../shared/types.ts";
-import type { NestedPathEntry } from "./nested-path.ts";
-
-export interface BuildPiArgsInput {
-  readonly baseArgs: readonly string[];
-  readonly task: string;
-  readonly sessionEnabled: boolean;
-  readonly sessionDir?: string;
-  readonly sessionFile?: string;
-  readonly model?: string;
-  readonly thinking?: string;
-  readonly systemPromptMode?: "append" | "replace";
-  readonly inheritProjectContext: boolean;
-  readonly inheritSkills: boolean;
-  readonly tools?: readonly string[];
-  readonly allowSubagents?: boolean;
-  readonly extensions?: readonly string[];
-  readonly systemPrompt?: string | null;
-  readonly mcpDirectTools?: readonly string[];
-  /** Actual child spawn cwd; a new --session file inherits this directory natively. */
-  readonly cwd?: string;
-  readonly intercomSessionName?: string;
-  readonly orchestratorIntercomTarget?: string;
-  readonly rootSessionId?: string;
-  readonly runId?: string;
-  readonly childAgentName?: string;
-  readonly childIndex?: number;
-  readonly parentEventSink?: string;
-  readonly parentControlInbox?: string;
-  readonly parentRootRunId?: string;
-  readonly parentRunId?: string;
-  readonly parentChildIndex?: number;
-  readonly parentDepth?: number;
-  readonly parentPath?: readonly NestedPathEntry[];
-  readonly parentCapabilityToken?: string;
-  readonly structuredOutput?: {
-    readonly schema: JsonSchemaObject;
-    readonly schemaPath: string;
-    readonly outputPath: string;
-  };
-  readonly projectTrust?: ChildProjectTrustPolicy;
-}
-
-export interface BuildPiArgsResult {
-  args: string[];
-  env: Record<string, string | undefined>;
-  tempDir?: string;
-}

package/src/runs/shared/pi-session-launch.ts

@@ -1,94 +0,0 @@
-import * as fs from "node:fs";
-import * as path from "node:path";
-import type { ReadonlyInput } from "../../shared/types.ts";
-import type { BuildPiArgsInput, BuildPiArgsResult } from "./pi-launch-input.ts";
-import { prepareChildExecutionCwd } from "./child-execution-cwd.ts";
-import { hasErrorCode, isRecord as isObject } from "../../shared/unknown.ts";
-import { nonempty } from "./child-presence.ts";
-
-const SESSION_CWD_PRELOAD_URL = new URL(
-  `session-cwd-preload${import.meta.url.endsWith(".ts") ? ".ts" : ".js"}`,
-  import.meta.url,
-).href;
-
-// Read only the header; joining chunks before decoding preserves split UTF-8.
-function readSessionHeaderLine(file: string): string | undefined {
-  let fd: number;
-  try {
-    fd = fs.openSync(file, "r");
-  } catch (error) {
-    if (hasErrorCode(error, "ENOENT") && !fs.lstatSync(file, { throwIfNoEntry: false })) {
-      return;
-    }
-    throw error;
-  }
-  try {
-    const chunks: Buffer[] = [];
-    const buffer = Buffer.alloc(4096);
-    while (true) {
-      const length = fs.readSync(fd, buffer, 0, buffer.length, null);
-      if (length === 0) {
-        break;
-      }
-      const chunk = buffer.subarray(0, length);
-      const newline = chunk.indexOf(10);
-      chunks.push(Buffer.from(newline < 0 ? chunk : chunk.subarray(0, newline)));
-      if (newline >= 0) {
-        break;
-      }
-    }
-    return Buffer.concat(chunks).toString("utf8");
-  } finally {
-    fs.closeSync(fd);
-  }
-}
-
-function needsCwdOverride(file: string, cwd: string): boolean {
-  try {
-    const line = readSessionHeaderLine(file);
-    if (line === undefined) {
-      return false; // A new session inherits the spawn cwd natively.
-    }
-    const header: unknown = JSON.parse(line);
-    if (
-      isObject(header) &&
-      header.type === "session" &&
-      typeof header.cwd === "string" &&
-      path.isAbsolute(header.cwd)
-    ) {
-      return fs.realpathSync.native(header.cwd) !== fs.realpathSync.native(cwd);
-    }
-  } catch {
-    // Unknown headers or unavailable directories still need the SDK override.
-  }
-  return true;
-}
-
-/** Prepare persistence before spawn without modifying an existing session header or identity. */
-export function piSessionLaunch(input: ReadonlyInput<BuildPiArgsInput>): BuildPiArgsResult {
-  const args: string[] = [];
-  const env: Record<string, string | undefined> = {};
-  if (!nonempty(input.sessionFile)) {
-    if (!input.sessionEnabled) {
-      args.push("--no-session");
-    }
-    if (nonempty(input.sessionDir)) {
-      fs.mkdirSync(input.sessionDir, { recursive: true });
-      args.push("--session-dir", input.sessionDir);
-    }
-    return { args, env };
-  }
-  prepareChildExecutionCwd(input.sessionFile, input.cwd);
-  fs.mkdirSync(path.dirname(input.sessionFile), { recursive: true });
-  args.push("--session", input.sessionFile);
-  if (nonempty(input.cwd) && needsCwdOverride(input.sessionFile, input.cwd)) {
-    env.PI_SUBAGENT_SESSION_CWD = JSON.stringify({
-      sessionFile: path.resolve(input.cwd, input.sessionFile),
-      cwd: input.cwd,
-      nodeOptions: process.env.NODE_OPTIONS,
-    });
-    env.NODE_OPTIONS =
-      `${process.env.NODE_OPTIONS ?? ""} --import=${SESSION_CWD_PRELOAD_URL}`.trim();
-  }
-  return { args, env };
-}

package/src/runs/shared/pi-spawn.ts

@@ -1,76 +0,0 @@
-import * as fs from "node:fs";
-import * as path from "node:path";
-import { fileURLToPath } from "node:url";
-
-export const PI_CODING_AGENT_PACKAGE = "@earendil-works/pi-coding-agent";
-
-function isPiPackage(file: string): boolean {
-  if (!fs.existsSync(file)) {
-    return false;
-  }
-  const pkg: unknown = JSON.parse(fs.readFileSync(file, "utf-8"));
-  return (
-    typeof pkg === "object" && pkg !== null && "name" in pkg && pkg.name === PI_CODING_AGENT_PACKAGE
-  );
-}
-
-export function findPiPackageRootFromEntry(entryPoint: string): string | undefined {
-  let dir = path.dirname(entryPoint);
-  while (dir !== path.dirname(dir)) {
-    if (isPiPackage(path.join(dir, "package.json"))) {
-      return dir;
-    }
-    dir = path.dirname(dir);
-  }
-  return;
-}
-
-function findPathPiPackage(): string | undefined {
-  for (const dir of (process.env.PATH ?? "").split(path.delimiter)) {
-    try {
-      const root = findPiPackageRootFromEntry(fs.realpathSync(path.join(dir, "pi")));
-      if (root !== undefined) {
-        return root;
-      }
-    } catch {
-      // PATH entries can disappear or be wrappers rather than Pi.
-    }
-  }
-  return;
-}
-
-export function resolveInstalledPiPackageRoot(): string | undefined {
-  try {
-    return findPiPackageRootFromEntry(fileURLToPath(import.meta.resolve(PI_CODING_AGENT_PACKAGE)));
-  } catch {
-    return findPathPiPackage();
-  }
-}
-
-export function resolvePiPackageRoot(): string | undefined {
-  try {
-    const entry = process.argv.at(1);
-    return entry !== undefined ? findPiPackageRootFromEntry(fs.realpathSync(entry)) : undefined;
-  } catch {
-    // process.argv[1] probing is best-effort; callers can fall back to PATH/package resolution.
-    return;
-  }
-}
-
-export function requirePiPackageRoot(): string {
-  const configured = process.env.PI_PACKAGE_DIR;
-  const root =
-    configured !== undefined && configured.length > 0
-      ? configured
-      : (resolvePiPackageRoot() ?? resolveInstalledPiPackageRoot());
-  if (root === undefined) {
-    throw new Error(
-      "Could not locate Pi runtime APIs; Pi must be installed and available on PATH.",
-    );
-  }
-  return root;
-}
-
-export function getPiSpawnCommand(args: readonly string[]): { command: string; args: string[] } {
-  return { command: "pi", args: [...args] };
-}

package/src/runs/shared/run-history-queries.ts

@@ -1,195 +0,0 @@
-import type {
-  HistoryIndexHandle,
-  HistoryPage,
-  HistoryPageInput,
-  HistorySearchInput,
-  HistorySearchPage,
-} from "../../shared/types/history.ts";
-import type { SubagentParamsLike } from "../foreground/subagent-params.ts";
-import type {
-  OwnedRun,
-  OwnedRunView,
-  SubagentExecutionResult,
-  SubagentState,
-} from "../../shared/types.ts";
-import { runHistoryIndex } from "./history-index.ts";
-import { ownedRunView, resolveOwnedRun } from "./run-records.ts";
-
-type OwnedRunChildView = OwnedRunView["children"][number];
-
-type HistoryQuery = Readonly<
-  Pick<
-    SubagentParamsLike,
-    "id" | "runId" | "action" | "index" | "limit" | "before" | "cursor" | "query" | "sort" | "agent"
-  >
->;
-
-function selectHistoryChild(view: OwnedRunView, index?: number): OwnedRunChildView {
-  const child =
-    index === undefined && view.children.length === 1
-      ? view.children[0]
-      : view.children.find((entry) => entry.index === index);
-  if (!child) {
-    throw new Error("Choose index for a run with multiple children.");
-  }
-  return child;
-}
-
-function historyInput(
-  view: OwnedRunView,
-  child: OwnedRunChildView,
-  params: HistoryQuery,
-  signal?: AbortSignal,
-): HistoryPageInput {
-  return {
-    runId: view.runId,
-    index: child.index,
-    limit: params.limit,
-    before: params.before,
-    cursor: params.cursor,
-    signal,
-    ...(child.state !== "live"
-      ? {
-          terminalEntryId: child.result?.terminalEntryId,
-          endedAt: (child.result?.terminalEntryId ?? "") !== "" ? undefined : view.updatedAt,
-        }
-      : {}),
-  };
-}
-
-function historyResult(
-  view: OwnedRunView,
-  child: OwnedRunChildView,
-  page: HistoryPage,
-  limit?: number,
-): SubagentExecutionResult {
-  const earlier =
-    (page.previousCursor ?? "") !== ""
-      ? {
-          action: "history",
-          id: view.runId,
-          index: child.index,
-          limit: limit ?? 100,
-          cursor: page.previousCursor,
-        }
-      : undefined;
-  const source = child.sessionFile;
-  return {
-    content: [
-      {
-        type: "text",
-        text: [
-          `History: ${view.runId}, child ${child.index} (${child.agent}); ${page.entries.length} of ${page.count} indexed native entries.`,
-          `Browse index: ${page.freshness.state}. These are bounded previews, not completion or delivery receipts.`,
-          ...((page.unavailable ?? "") !== "" ? [page.unavailable] : []),
-          ...page.entries.map(
-            (entry) => `- ${entry.id} · position ${entry.sequence}\n${JSON.stringify(entry.entry)}`,
-          ),
-          ...(earlier ? [`Earlier: agent_runs(${JSON.stringify(earlier)})`] : []),
-          ...(source !== undefined && source !== ""
-            ? [
-                `Native source: ${source}. Open a selected record in Agents for validated full details.`,
-              ]
-            : []),
-        ].join("\n"),
-      },
-    ],
-    details: { mode: "management", results: [], runId: view.runId, history: page },
-  };
-}
-
-function searchInput(
-  params: HistoryQuery,
-  run?: OwnedRun,
-  signal?: AbortSignal,
-): HistorySearchInput {
-  if (params.query === undefined || params.query === "") {
-    throw new Error("Search requires query.");
-  }
-  if (params.sort !== undefined && !["relevance", "newest"].includes(params.sort)) {
-    throw new Error("Search sort must be relevance or newest.");
-  }
-  return {
-    query: params.query,
-    runId: run?.runId,
-    index: params.index,
-    limit: params.limit,
-    cursor: params.cursor,
-    sort: params.sort === "relevance" || params.sort === "newest" ? params.sort : undefined,
-    agent: params.agent,
-    signal,
-  };
-}
-
-function searchContinuation(input: HistorySearchInput, page: HistorySearchPage): string[] {
-  if ((page.nextCursor ?? "") === "") {
-    return [];
-  }
-  const next = {
-    action: "search",
-    query: input.query,
-    ...(input.runId !== undefined ? { id: input.runId } : {}),
-    ...(input.index !== undefined ? { index: input.index } : {}),
-    ...((input.agent ?? "") !== "" ? { agent: input.agent } : {}),
-    ...(input.sort !== undefined ? { sort: input.sort } : {}),
-    limit: input.limit ?? 20,
-    cursor: page.nextCursor,
-  };
-  return [`Next: agent_runs(${JSON.stringify(next)})`];
-}
-
-async function searchHistory(
-  index: HistoryIndexHandle,
-  input: HistorySearchInput,
-): Promise<SubagentExecutionResult> {
-  const page = await index.search(input);
-  return {
-    content: [
-      {
-        type: "text",
-        text: [
-          `Saved-text search: ${page.matches.length} matches in this owning session; index ${page.freshness.state}.`,
-          "Indexed excerpts may be stale. They do not establish completion, ownership, or delivery; full selected records are validated separately.",
-          ...page.matches.map(
-            (match) =>
-              `- ${match.runId}:${match.index} (${match.agent}) · entry ${match.entryId}\n${match.preview}\n${match.sessionFile}:${match.ref.start}`,
-          ),
-          ...searchContinuation(input, page),
-        ].join("\n"),
-      },
-    ],
-    details: { mode: "management", results: [], historySearch: page },
-  };
-}
-
-/** Browse observations never confer ownership or replace canonical control/receipt checks. */
-export async function ownedHistoryQuery(
-  state: SubagentState,
-  params: HistoryQuery,
-  signal?: AbortSignal,
-): Promise<SubagentExecutionResult> {
-  const requested = params.id ?? params.runId;
-  const run =
-    requested !== undefined && requested !== "" ? resolveOwnedRun(state, requested) : undefined;
-  if (requested !== undefined && requested !== "" && !run) {
-    throw new Error("Run not found in this owning session.");
-  }
-  const index = await runHistoryIndex(state, true);
-  if (params.action !== "history") {
-    if (params.action !== "search") {
-      throw new Error("Search requires query.");
-    }
-    return searchHistory(index, searchInput(params, run, signal));
-  }
-  if (!run) {
-    throw new Error("History requires an owned run ID.");
-  }
-  const view = ownedRunView(run, state, {
-    includeContinuations: false,
-    readConfiguration: false,
-    reconcile: false,
-  });
-  const child = selectHistoryChild(view, params.index);
-  const page = await index.historyPage(historyInput(view, child, params, signal));
-  return historyResult(view, child, page, params.limit);
-}