@fitchmultz/pi-subagents 0.49.1 → 0.50.0

package/src/shared/session-entries.ts

@@ -1,108 +0,0 @@
-import type { SessionManager, SessionEntry } from "@earendil-works/pi-coding-agent";
-
-type EntrySource = Pick<SessionManager, "getEntries" | "getSessionId"> &
-  Partial<Pick<SessionManager, "getSessionFile" | "getEntry" | "getLeafId" | "getEntryCount">>;
-interface Snapshot {
-  readonly manager: EntrySource;
-  readonly sessionId: string;
-  readonly file: string | null | undefined;
-  readonly count: number | undefined;
-}
-
-/** Follow append-only parent links; rescan after replacement or append off the active branch. */
-export class SessionEntryCursor {
-  private manager?: EntrySource;
-  private sessionId?: string;
-  private file?: string | null;
-  private readonly known = new Map<string, SessionEntry>();
-  private anchor?: SessionEntry;
-  reset(): void {
-    this.manager = undefined;
-    this.known.clear();
-    this.anchor = undefined;
-  }
-  private changed(snapshot: Snapshot): boolean {
-    if (
-      this.manager !== snapshot.manager ||
-      this.sessionId !== snapshot.sessionId ||
-      this.file !== snapshot.file
-    ) {
-      return true;
-    }
-    if (snapshot.count !== undefined && snapshot.count < this.known.size) {
-      return true;
-    }
-    return (
-      this.anchor !== undefined &&
-      snapshot.manager.getEntry !== undefined &&
-      snapshot.manager.getEntry(this.anchor.id) !== this.known.get(this.anchor.id)
-    );
-  }
-  private appendedBranch(
-    manager: EntrySource,
-  ): { entries: SessionEntry[]; tail: string | null } | undefined {
-    if (manager.getEntry === undefined || manager.getLeafId === undefined) {
-      return undefined;
-    }
-    const entries: SessionEntry[] = [];
-    const visited = new Set<string>();
-    let id = manager.getLeafId();
-    while (id !== null && id !== "" && !this.known.has(id) && !visited.has(id)) {
-      visited.add(id);
-      const entry = manager.getEntry(id);
-      if (!entry) {
-        break;
-      }
-      entries.push(entry);
-      id = entry.parentId;
-    }
-    return { entries, tail: id };
-  }
-  private appended(snapshot: Snapshot): SessionEntry[] | undefined {
-    if (snapshot.count === undefined) {
-      return undefined;
-    }
-    const branch = this.appendedBranch(snapshot.manager);
-    if (!branch) {
-      return undefined;
-    }
-    const { entries, tail } = branch;
-    const connects = tail === null || tail === "" || this.known.has(tail);
-    return snapshot.count === this.known.size + entries.length && connects
-      ? entries.reverse()
-      : undefined;
-  }
-  private rescan(
-    manager: EntrySource,
-    reset: boolean,
-  ): { entries: SessionEntry[]; reset: boolean } {
-    const all = manager.getEntries();
-    const added = all.filter((entry) => !this.known.has(entry.id));
-    const replaced =
-      all.length !== this.known.size + added.length ||
-      all.some((entry) => this.known.has(entry.id) && this.known.get(entry.id) !== entry);
-    return { entries: reset || replaced ? all : added, reset: reset || replaced };
-  }
-  read(manager: EntrySource): { entries: SessionEntry[]; reset: boolean } {
-    const snapshot: Snapshot = {
-      manager,
-      sessionId: manager.getSessionId(),
-      file: manager.getSessionFile?.(),
-      count: manager.getEntryCount?.(),
-    };
-    const changed = this.changed(snapshot);
-    const appended = changed ? undefined : this.appended(snapshot);
-    const result = appended ? { entries: appended, reset: false } : this.rescan(manager, changed);
-    if (result.reset) {
-      this.known.clear();
-    }
-    for (const entry of result.entries) {
-      this.known.set(entry.id, entry);
-    }
-    this.anchor = result.entries.at(-1) ?? (result.reset ? undefined : this.anchor);
-    this.manager = manager;
-    this.sessionId = snapshot.sessionId;
-    this.file = snapshot.file;
-    return result;
-  }
-}

package/src/shared/session-identity.ts

@@ -1,25 +0,0 @@
-interface SessionIdentityManager {
-  readonly getSessionFile: () => string | null | undefined;
-  readonly getSessionId: () => string | null | undefined;
-}
-
-// Only delegation supplies this identity; Pi's ordinary fork/clone ancestry does not.
-export function resolveRootSessionId(
-  sessionManager: SessionIdentityManager,
-  env: Readonly<NodeJS.ProcessEnv> = process.env,
-): string {
-  const rootId = env.PI_SUBAGENT_CHILD === "1" ? env.PI_SUBAGENT_ROOT_SESSION_ID : undefined;
-  const sessionId = rootId !== undefined && rootId !== "" ? rootId : sessionManager.getSessionId();
-  if (sessionId === null || sessionId === undefined || sessionId === "") {
-    throw new Error("Root session identity is unavailable.");
-  }
-  return sessionId;
-}
-
-export function resolveCurrentSessionId(sessionManager: SessionIdentityManager): string {
-  const sessionId = sessionManager.getSessionId();
-  if (sessionId === null || sessionId === undefined || sessionId === "") {
-    throw new Error("Current session identity is unavailable.");
-  }
-  return sessionId;
-}

package/src/shared/settings.ts

@@ -1,196 +0,0 @@
-/** Chain step traversal, behavior resolution, and instruction composition. */
-import * as path from "node:path";
-import type { AgentConfig } from "./types/config.ts";
-import type {
-  ChainStep,
-  ParallelStep,
-  DynamicParallelStep,
-  ParallelTaskItem,
-  ResolvedStepBehavior,
-  StepOverrides,
-} from "./types/workflow.ts";
-import type { ReadonlyInput } from "./types/inputs.ts";
-import { normalizeSkillInput } from "../agents/skills.ts";
-export type {
-  ChainStep,
-  ParallelStep,
-  DynamicParallelStep,
-  SequentialStep,
-  ParallelTaskItem,
-  ResolvedStepBehavior,
-  StepOverrides,
-  DynamicExpandSpec,
-  DynamicParallelTemplate,
-  DynamicCollectSpec,
-} from "./types/workflow.ts";
-export {
-  createChainDir,
-  removeChainDir,
-  cleanupOldChainDirs,
-  writeInitialProgressFile,
-} from "./chain-files.ts";
-export {
-  resolveTaskTextForFileUpdatePolicy,
-  taskDisallowsFileUpdates,
-  suppressProgressForReadOnlyTask,
-} from "./task-file-policy.ts";
-export type { ParallelTaskResult } from "../runs/shared/parallel-utils.ts";
-
-export function isParallelStep(
-  step: ReadonlyInput<ChainStep>,
-): step is ReadonlyInput<ParallelStep> {
-  return "parallel" in step && Array.isArray(step.parallel);
-}
-export function isDynamicParallelStep(
-  step: ReadonlyInput<ChainStep>,
-): step is ReadonlyInput<DynamicParallelStep> {
-  return (
-    "expand" in step && "collect" in step && "parallel" in step && !Array.isArray(step.parallel)
-  );
-}
-export function getStepAgents(step: ReadonlyInput<ChainStep>): string[] {
-  if (isParallelStep(step)) {
-    return step.parallel.map((task) => task.agent);
-  }
-  if (isDynamicParallelStep(step)) {
-    return [step.parallel.agent];
-  }
-  return [step.agent];
-}
-export function collectInvocationAgentNames(
-  params: ReadonlyInput<{ agent?: string; tasks?: Array<{ agent: string }>; chain?: ChainStep[] }>,
-): string[] {
-  const names: string[] = [];
-  if (params.agent !== undefined && params.agent.length > 0) {
-    names.push(params.agent);
-  }
-  for (const task of params.tasks ?? []) {
-    names.push(task.agent);
-  }
-  for (const step of params.chain ?? []) {
-    names.push(...getStepAgents(step));
-  }
-  return names;
-}
-export type ResolvedTemplates = (string | string[])[];
-export function resolveChainTemplates(steps: readonly ChainStep[]): ResolvedTemplates {
-  return steps.map((step, index) => {
-    if (isParallelStep(step)) {
-      return step.parallel.map((task) =>
-        task.task !== undefined && task.task.length > 0 ? task.task : "{previous}",
-      );
-    }
-    if (isDynamicParallelStep(step)) {
-      return step.parallel.task ?? "{previous}";
-    }
-    if (step.task !== undefined && step.task.length > 0) {
-      return step.task;
-    }
-    return index === 0 ? "{task}" : "{previous}";
-  });
-}
-function normalizeOutputOverride(output: string | false | undefined): string | false | undefined {
-  return output === "false" ? false : output;
-}
-function resolveStepSkills(
-  agentSkills: readonly string[] | undefined,
-  overrideSkills: readonly string[] | false | undefined,
-  chainSkills: readonly string[] | false | undefined,
-): string[] | false {
-  if (chainSkills === false || overrideSkills === false) {
-    return false;
-  }
-  const skills = [...(overrideSkills ?? agentSkills ?? [])];
-  return chainSkills !== undefined && chainSkills.length > 0
-    ? [...new Set([...skills, ...chainSkills])]
-    : skills;
-}
-/** Per-field precedence: step override, agent defaults, then disabled/unset. */
-export function resolveStepBehavior(
-  agentConfig: AgentConfig,
-  stepOverrides: StepOverrides,
-  chainSkills?: readonly string[] | false,
-): ResolvedStepBehavior {
-  return {
-    output:
-      normalizeOutputOverride(stepOverrides.output) ??
-      normalizeOutputOverride(agentConfig.output) ??
-      false,
-    outputMode: stepOverrides.outputMode ?? "inline",
-    reads: stepOverrides.reads ?? agentConfig.defaultReads ?? false,
-    progress: stepOverrides.progress ?? agentConfig.defaultProgress ?? false,
-    skills: resolveStepSkills(agentConfig.skills, stepOverrides.skills, chainSkills),
-    model: stepOverrides.model ?? agentConfig.model,
-  };
-}
-function resolveChainPath(filePath: string, chainDir: string): string {
-  return path.isAbsolute(filePath) ? filePath : path.join(chainDir, filePath);
-}
-/** Compose read paths and progress instructions for a chain child. */
-export function buildChainInstructions(
-  behavior: ResolvedStepBehavior,
-  chainDir: string,
-  isFirstProgressAgent: boolean,
-): { prefix: string; suffix: string } {
-  const prefixParts: string[] = [];
-  const suffixParts: string[] = [];
-  if (behavior.reads !== false && behavior.reads.length > 0) {
-    prefixParts.push(
-      `[Read from: ${behavior.reads.map((file) => resolveChainPath(file, chainDir)).join(", ")}]`,
-    );
-  }
-  if (behavior.progress) {
-    const progressPath = path.join(chainDir, "progress.md");
-    suffixParts.push(
-      isFirstProgressAgent
-        ? `Create and maintain progress at: ${progressPath}`
-        : `Update progress at: ${progressPath}`,
-    );
-  }
-  return {
-    prefix: prefixParts.length > 0 ? `${prefixParts.join("\n")}\n\n` : "",
-    suffix: suffixParts.length > 0 ? `\n\n---\n${suffixParts.join("\n")}` : "",
-  };
-}
-export function resolveParallelBehaviors(
-  tasks: readonly ParallelTaskItem[],
-  agentConfigs: readonly AgentConfig[],
-  stepIndex: number,
-  chainSkills?: readonly string[] | false,
-): ResolvedStepBehavior[] {
-  return tasks.map((task, taskIndex) => {
-    const config = agentConfigs.find((agent) => agent.name === task.agent);
-    if (config === undefined) {
-      throw new Error(`Unknown agent: ${task.agent}`);
-    }
-    const behavior = resolveStepBehavior(
-      config,
-      {
-        output: task.output,
-        outputMode: task.outputMode,
-        reads: task.reads,
-        progress: task.progress,
-        skills: normalizeSkillInput(task.skill),
-        model: task.model,
-      },
-      chainSkills,
-    );
-    return {
-      ...behavior,
-      output: namespaceParallelOutput(behavior.output, task.agent, stepIndex, taskIndex),
-    };
-  });
-}
-export function namespaceParallelOutput(
-  output: string | false | undefined,
-  agent: string,
-  stepIndex: number,
-  taskIndex: number,
-): string | false {
-  if (output === undefined || output === false || output.length === 0) {
-    return false;
-  }
-  return path.isAbsolute(output)
-    ? output
-    : path.join(`parallel-${stepIndex}`, `${taskIndex}-${agent}`, output);
-}

package/src/shared/status-format.ts

@@ -1,232 +0,0 @@
-import { loadConfig } from "../extension/config.ts";
-import type { ReadonlyDeep } from "type-fest";
-import type {
-  AgentProcessExit,
-  ActivityState,
-  AsyncJobStep,
-  ManagementAction,
-  ManagementControl,
-  ManagementRunState,
-  SubagentLiveIntercomHealth,
-} from "./types.ts";
-
-export function formatAgentProcessExit(exit: Readonly<AgentProcessExit> | undefined): string {
-  return exit
-    ? `Agent process exited (${exit.signal !== null ? `signal ${exit.signal}` : `code ${exit.code ?? "unrecorded"}`}). Command and descendant exit are not implied.`
-    : "Agent process exit not recorded. Command exit is unconfirmed.";
-}
-
-export function formatRunAction(
-  action: ManagementAction | "questions" | "answer",
-  id: string | undefined,
-  fields: Readonly<Record<string, string | number | boolean>> = {},
-  childSafe = false,
-): string {
-  const legacyChild = childSafe && loadConfig().compactChildTools === false;
-  const name = legacyChild ? action : parentActionName(action);
-  const args = Object.entries({ action: name, ...(id !== undefined ? { id } : {}), ...fields })
-    .map(([key, value]) => `${key}: ${JSON.stringify(value)}`)
-    .join(", ");
-  const call = `${legacyChild || action === "extend" ? "subagent" : "agent_runs"}({ ${args} })`;
-  return action === "extend" && !legacyChild ? `load_subagent({}), then ${call}` : call;
-}
-
-function parentActionName(action: ManagementAction | "questions" | "answer"): string {
-  switch (action) {
-    case "status":
-      return "inspect";
-    case "resume":
-      return "continue";
-    case "interrupt":
-      return "stop";
-    case "answer":
-    case "extend":
-    case "nudge":
-    case "questions":
-    case "review":
-      return action;
-  }
-}
-
-type ManagementInput = ReadonlyDeep<{
-  state: ManagementRunState;
-  runId: string;
-  index?: number;
-  intercomTarget?: string;
-  canExtend?: boolean;
-  canResume?: boolean;
-  canNudge?: boolean;
-  canInterrupt?: boolean;
-  canReview?: boolean;
-  unavailableActions?: Partial<Record<ManagementAction, string>>;
-  revivedFromRunId?: string;
-}>;
-function managementCapabilities(input: ManagementInput): ManagementAction[] {
-  const capabilities: ManagementAction[] = ["status"];
-  if (input.state === "live") {
-    if (input.canNudge === true) {
-      capabilities.push("nudge");
-    }
-    if (input.canResume === true) {
-      capabilities.push("resume");
-    }
-    if (input.canInterrupt === true) {
-      capabilities.push("interrupt");
-    }
-  } else if (input.canResume === true) {
-    capabilities.push("resume");
-  }
-  if (input.state === "live" && input.canExtend === true) {
-    capabilities.push("extend");
-  }
-  if (input.canReview === true) {
-    capabilities.push("review");
-  }
-  return capabilities;
-}
-function managementNextAction(
-  action: ManagementAction,
-  input: ManagementInput,
-): ManagementControl["nextActions"][number] {
-  return {
-    action,
-    runId: input.runId,
-    ...(input.index !== undefined && (action === "nudge" || action === "resume")
-      ? { index: input.index }
-      : {}),
-    ...(input.intercomTarget !== undefined &&
-    input.intercomTarget !== "" &&
-    (action === "nudge" || action === "resume")
-      ? { intercomTarget: input.intercomTarget }
-      : {}),
-  };
-}
-export function buildManagementControl(input: ManagementInput): ManagementControl {
-  const capabilities = managementCapabilities(input);
-  return {
-    state: input.state,
-    runId: input.runId,
-    capabilities,
-    nextActions: capabilities.map((action) => managementNextAction(action, input)),
-    ...(input.unavailableActions ? { unavailableActions: input.unavailableActions } : {}),
-    ...(input.revivedFromRunId !== undefined && input.revivedFromRunId !== ""
-      ? { revivedFromRunId: input.revivedFromRunId, pendingReplyContextValid: false }
-      : {}),
-  };
-}
-
-type StepStatusLike = Readonly<Pick<AsyncJobStep, "status">>;
-
-function formatActivityAge(ms: number): string {
-  if (ms < 1000) {
-    return "now";
-  }
-  if (ms < 60000) {
-    return `${Math.floor(ms / 1000)}s`;
-  }
-  return `${Math.floor(ms / 60000)}m`;
-}
-
-export function formatActivityLabel(
-  lastActivityAt: number | undefined,
-  activityState?: ActivityState,
-  now = Date.now(),
-): string | undefined {
-  if (lastActivityAt === undefined) {
-    if (activityState === "needs_attention") {
-      return "needs attention";
-    }
-    return undefined;
-  }
-  const age = formatActivityAge(Math.max(0, now - lastActivityAt));
-  if (activityState === "needs_attention") {
-    return `no activity for ${age}`;
-  }
-  return age === "now" ? "active now" : `active ${age} ago`;
-}
-
-function isCompletedStepStatus(status: AsyncJobStep["status"]): boolean {
-  return status === "complete" || status === "completed";
-}
-
-export function aggregateStepStatus(steps: readonly StepStatusLike[]): AsyncJobStep["status"] {
-  if (steps.some((step) => step.status === "running")) {
-    return "running";
-  }
-  if (steps.some((step) => step.status === "failed")) {
-    return "failed";
-  }
-  if (steps.some((step) => step.status === "blocked")) {
-    return "blocked";
-  }
-  if (steps.some((step) => step.status === "paused")) {
-    return "paused";
-  }
-  if (steps.length > 0 && steps.every((step) => isCompletedStepStatus(step.status))) {
-    return "complete";
-  }
-  return "pending";
-}
-
-export function formatAgentRunningLabel(count: number): string {
-  return count === 1 ? "1 agent running" : `${count} agents running`;
-}
-
-function intercomHealth(health: ReadonlyDeep<SubagentLiveIntercomHealth> | undefined): string {
-  if (!health) {
-    return "unknown";
-  }
-  if (health.status !== "registered") {
-    return health.status.replace(/_/g, " ");
-  }
-  return `registered${health.sessionStatus !== undefined && health.sessionStatus !== "" ? `, ${health.sessionStatus}` : ""}${health.acceptsAsks !== undefined ? `, accepts_asks:${health.acceptsAsks}` : ""}${health.pendingAsks !== undefined ? `, pending_asks:${health.pendingAsks}` : ""}`;
-}
-
-export function formatLiveIntercomActionLines(
-  input: ReadonlyDeep<{
-    runId: string;
-    target: string;
-    index?: number;
-    health?: SubagentLiveIntercomHealth;
-    indent?: string;
-    childSafe?: boolean;
-  }>,
-): string[] {
-  const fields = {
-    ...(input.index !== undefined ? { index: input.index } : {}),
-    message: "What are you blocked on?",
-  };
-  const healthText = intercomHealth(input.health);
-  const indent = input.indent ?? "";
-  return [
-    `${indent}Intercom: ${healthText} (${input.target})`,
-    `${indent}Nudge (preferred live coordination): ${formatRunAction("nudge", input.runId, fields, input.childSafe)}`,
-    `${indent}Ask (blocking wait only; parent must remain alive): intercom({ action: "ask", to: "${input.target}", delivery: "steer", message: "What are you blocked on?" })`,
-  ];
-}
-
-export function formatParallelOutcome(
-  steps: readonly StepStatusLike[],
-  total: number,
-  options: { readonly showRunning?: boolean } = {},
-): string {
-  const running = steps.filter((step) => step.status === "running").length;
-  const succeeded = steps.filter((step) => isCompletedStepStatus(step.status)).length;
-  const failed = steps.filter((step) => step.status === "failed").length;
-  const paused = steps.filter((step) => step.status === "paused").length;
-  const blocked = steps.filter((step) => step.status === "blocked").length;
-  const parts = [`${succeeded}/${total} succeeded`];
-  if (options.showRunning !== false && running > 0) {
-    parts.unshift(formatAgentRunningLabel(running));
-  }
-  if (failed > 0) {
-    parts.push(`${failed} failed`);
-  }
-  if (paused > 0) {
-    parts.push(`${paused} paused`);
-  }
-  if (blocked > 0) {
-    parts.push(`${blocked} need human action`);
-  }
-  return parts.join(" · ");
-}

package/src/shared/task-file-policy.ts

@@ -1,35 +0,0 @@
-import type { ResolvedStepBehavior } from "./types/workflow.ts";
-
-export function resolveTaskTextForFileUpdatePolicy(
-  task: string | undefined,
-  originalTask?: string,
-): string | undefined {
-  if (task === undefined || task.length === 0) {
-    return originalTask;
-  }
-  return originalTask !== undefined && originalTask.length > 0
-    ? task.replaceAll("{task}", () => originalTask)
-    : task;
-}
-export function taskDisallowsFileUpdates(task: string | undefined): boolean {
-  if (task === undefined || task.length === 0) {
-    return false;
-  }
-  return (
-    /\breview[- ]only\b/i.test(task) ||
-    /\bread[- ]only\s+(?:review|audit|inspection|pass)\b/i.test(task) ||
-    /\b(?:no|without)\s+(?:file\s+)?edits?\b/i.test(task) ||
-    /\b(?:do not|don't|must not)\s+(?:edit|modify|write|touch)\b/i.test(task) ||
-    /\bleave\s+files?\s+unchanged\b/i.test(task)
-  );
-}
-export function suppressProgressForReadOnlyTask(
-  behavior: ResolvedStepBehavior,
-  task: string | undefined,
-  originalTask?: string,
-): ResolvedStepBehavior {
-  const policyTask = resolveTaskTextForFileUpdatePolicy(task, originalTask);
-  return behavior.progress && taskDisallowsFileUpdates(policyTask)
-    ? { ...behavior, progress: false }
-    : behavior;
-}