pi-subagents 0.68.0 → 0.70.0

package/src/missions/types.ts

@@ -1,190 +0,0 @@
-export const MISSION_STATUSES = [
-	"planned",
-	"active",
-	"waiting",
-	"needs_decision",
-	"completed",
-	"failed",
-	"cancelled",
-] as const;
-
-export type MissionStatus = typeof MISSION_STATUSES[number];
-export type MissionRunMode = "single" | "parallel" | "chain" | "workflow" | "scheduled" | "external";
-export type MissionArtifactKind = "status" | "output" | "patch" | "manifest" | "review" | "note" | "other";
-export type MissionReceiptKind = "pull_request" | "ci" | "deployment" | "release";
-export type MissionReceiptStatus = "pending" | "ready" | "succeeded" | "failed";
-export type MissionGoalStatus = "active" | "paused" | "budget-exhausted";
-
-export interface MissionGoal {
-	status: MissionGoalStatus;
-}
-
-export interface MissionTokenBudget {
-	tokens: number;
-}
-
-export interface MissionTokenUsage {
-	tokens: number;
-}
-
-export interface MissionRunLink {
-	runId: string;
-	asyncDir?: string;
-	childIndex?: number;
-	agent?: string;
-	mode: MissionRunMode;
-	status?: string;
-	startedAt?: string;
-	completedAt?: string;
-	usage?: MissionTokenUsage;
-}
-
-export interface MissionDecision {
-	id: string;
-	status: "open" | "resolved";
-	title: string;
-	prompt?: string;
-	options?: string[];
-	recommendation?: string;
-	createdAt: string;
-	resolvedAt?: string;
-	resolution?: string;
-}
-
-export interface MissionChildHeartbeat {
-	updatedAt: string;
-	status?: string;
-	phase?: string;
-	message?: string;
-}
-
-export interface MissionWorkflowChild {
-	workflowRunId: string;
-	key: string;
-	status: string;
-	startedAt: string;
-	updatedAt: string;
-	runId?: string;
-	agent?: string;
-	task?: string;
-	label?: string;
-	phase?: string;
-	completedAt?: string;
-	sessionPath?: string;
-	artifactPaths: string[];
-	heartbeat?: MissionChildHeartbeat;
-}
-
-export type MissionWorkflowChildUpdate = Pick<MissionWorkflowChild, "workflowRunId" | "key" | "status"> & Partial<Omit<MissionWorkflowChild, "workflowRunId" | "key" | "status" | "startedAt" | "updatedAt" | "artifactPaths" | "heartbeat">> & {
-	startedAt?: string;
-	artifactPaths?: string[];
-	heartbeat?: Omit<MissionChildHeartbeat, "updatedAt"> & { updatedAt?: string };
-};
-
-export interface MissionArtifact {
-	kind: MissionArtifactKind;
-	path: string;
-	description?: string;
-}
-
-export interface MissionReceipt {
-	kind: MissionReceiptKind;
-	status: MissionReceiptStatus;
-	title: string;
-	url: string;
-	createdAt: string;
-	description?: string;
-}
-
-export interface MissionRecord {
-	schemaVersion: 1;
-	id: string;
-	title: string;
-	objective: string;
-	goal?: MissionGoal;
-	budget?: MissionTokenBudget;
-	usage?: MissionTokenUsage;
-	status: MissionStatus;
-	createdAt: string;
-	updatedAt: string;
-	cwd?: string;
-	ownerSessionId?: string;
-	runs: MissionRunLink[];
-	workflowChildren: MissionWorkflowChild[];
-	decisions: MissionDecision[];
-	artifacts: MissionArtifact[];
-	receipts: MissionReceipt[];
-	summary?: string;
-	acceptance?: unknown;
-	labels?: string[];
-}
-
-export interface MissionIndexEntry {
-	schemaVersion: 1;
-	missionId: string;
-	projectRoot: string;
-	recordPath: string;
-	title: string;
-	status: MissionStatus;
-	updatedAt: string;
-	lastRunId?: string;
-}
-
-export interface MissionStoreConfig {
-	enabled?: boolean;
-	directory?: string;
-	globalIndex?: boolean;
-	globalIndexDir?: string;
-	retainTerminal?: number;
-}
-
-export interface MissionStoreLocation {
-	projectRoot: string;
-	missionDir: string;
-	globalIndexDir: string;
-	writeGlobalIndex: boolean;
-	retainTerminal?: number;
-}
-
-export interface MissionListResult {
-	records: MissionRecord[];
-	warnings: string[];
-}
-
-export interface GlobalMissionIndexRecord extends MissionIndexEntry {
-	stale: boolean;
-	staleReason?: string;
-}
-
-export interface GlobalMissionListResult {
-	entries: GlobalMissionIndexRecord[];
-	warnings: string[];
-}
-
-export interface MissionCreateInput {
-	title: string;
-	objective: string;
-	goal?: boolean;
-	budget?: MissionTokenBudget;
-	status?: MissionStatus;
-	labels?: string[];
-	ownerSessionId?: string;
-}
-
-export interface MissionUpdateInput {
-	title?: string;
-	objective?: string;
-	goal?: MissionGoal | false;
-	budget?: MissionTokenBudget;
-	usage?: MissionTokenUsage;
-	status?: MissionStatus;
-	summary?: string;
-	labels?: string[];
-	acceptance?: unknown;
-	addRuns?: MissionRunLink[];
-	upsertWorkflowChildren?: MissionWorkflowChildUpdate[];
-	addArtifacts?: MissionArtifact[];
-	addDecisions?: Array<Omit<MissionDecision, "id" | "status" | "createdAt">>;
-	resolveDecision?: { id: string; resolution: string };
-	addReceipts?: Array<Omit<MissionReceipt, "createdAt">>;
-}

package/src/missions/workflow-state.ts

@@ -1,281 +0,0 @@
-import { execFileSync } from "node:child_process";
-import { randomUUID } from "node:crypto";
-import * as fs from "node:fs";
-import * as path from "node:path";
-import { writePrivateAtomicJson } from "../shared/atomic-json.ts";
-import { DEFAULT_FILE_SYSTEM_RETRY_DELAYS_MS, isRetryableFileSystemError, waitForFileSystemRetry } from "../shared/file-system-retry.ts";
-import { assertWorkflowJsonValue } from "../workflows/scripted-workflow.ts";
-import type { MissionStoreLocation } from "./types.ts";
-import { validateMissionId } from "./store.ts";
-
-const STATE_KEY_PATTERN = /^[A-Za-z0-9][A-Za-z0-9._-]{0,127}$/;
-const STATE_LOCK_STALE_MS = 60_000;
-export const MISSION_STATE_MAX_BYTES = 256 * 1024;
-
-export interface MissionWorkflowState {
-	path: string;
-	get(key: string): unknown;
-	set(key: string, value: unknown): void;
-}
-
-export interface MissionWorkflowStateOptions {
-	isProcessAlive?: (pid: number) => boolean;
-	getProcessStartKey?: (pid: number) => string | undefined;
-	retryDelaysMs?: readonly number[];
-}
-
-export function missionStatePath(location: MissionStoreLocation, missionId: string): string {
-	return path.join(location.missionDir, validateMissionId(missionId), "state.json");
-}
-
-function isProcessAlive(pid: number): boolean {
-	try {
-		process.kill(pid, 0);
-		return true;
-	} catch (error) {
-		return (error as NodeJS.ErrnoException).code === "EPERM";
-	}
-}
-
-interface StateLockOwner {
-	pid: number;
-	token: string;
-	createdAt: number;
-	processKey?: string;
-}
-
-function linuxProcessStartKey(pid: number): string | undefined {
-	try {
-		const raw = fs.readFileSync(`/proc/${pid}/stat`, "utf-8");
-		const tail = raw.slice(raw.lastIndexOf(")") + 2).trim().split(/\s+/);
-		return tail[19] ? `linux:${tail[19]}` : undefined;
-	} catch {
-		return undefined;
-	}
-}
-
-function psProcessStartKey(pid: number): string | undefined {
-	try {
-		const raw = execFileSync("ps", ["-p", String(pid), "-o", "lstart="], { encoding: "utf-8", stdio: ["ignore", "pipe", "ignore"], timeout: 1000 }).trim();
-		return raw ? `ps:${raw}` : undefined;
-	} catch {
-		return undefined;
-	}
-}
-
-function windowsProcessStartKey(pid: number): string | undefined {
-	try {
-		const raw = execFileSync("powershell.exe", ["-NoProfile", "-Command", `(Get-CimInstance Win32_Process -Filter \"ProcessId=${pid}\").CreationDate`], { encoding: "utf-8", stdio: ["ignore", "pipe", "ignore"], timeout: 1000, windowsHide: true }).trim();
-		return raw ? `win:${raw}` : undefined;
-	} catch {
-		return undefined;
-	}
-}
-
-let currentProcessKey: string | undefined | null = null;
-
-function processStartKey(pid: number): string | undefined {
-	// Foreign PIDs can be reused while this process lives, so resolve them afresh.
-	if (pid !== process.pid) return computeProcessStartKey(pid);
-	if (currentProcessKey === null) currentProcessKey = computeProcessStartKey(pid);
-	return currentProcessKey;
-}
-
-function computeProcessStartKey(pid: number): string | undefined {
-	if (process.platform === "linux") return linuxProcessStartKey(pid) ?? psProcessStartKey(pid);
-	if (process.platform === "win32") return windowsProcessStartKey(pid);
-	return undefined;
-}
-
-type StateLockOptions = Required<MissionWorkflowStateOptions>;
-
-function readStateLockOwner(lockPath: string): StateLockOwner | undefined {
-	try {
-		const owner = JSON.parse(fs.readFileSync(path.join(lockPath, "owner.json"), "utf-8")) as { pid?: unknown; token?: unknown; createdAt?: unknown; processKey?: unknown };
-		if (Number.isSafeInteger(owner.pid) && (owner.pid as number) > 0 && typeof owner.token === "string" && owner.token && Number.isSafeInteger(owner.createdAt)) {
-			return {
-				pid: owner.pid as number,
-				token: owner.token,
-				createdAt: owner.createdAt as number,
-				...(typeof owner.processKey === "string" && owner.processKey ? { processKey: owner.processKey } : {}),
-			};
-		}
-	} catch {
-		return undefined;
-	}
-	return undefined;
-}
-
-function stateLockIsStale(lockPath: string, options: StateLockOptions, now = Date.now()): boolean {
-	const owner = readStateLockOwner(lockPath);
-	if (owner) {
-		if (!options.isProcessAlive(owner.pid)) return true;
-		if (owner.processKey) {
-			const observedProcessKey = options.getProcessStartKey(owner.pid);
-			if (observedProcessKey) return owner.processKey !== observedProcessKey;
-			if (owner.pid === process.pid) return true;
-		}
-		return false;
-	}
-	try {
-		return now - fs.statSync(lockPath).mtimeMs > STATE_LOCK_STALE_MS;
-	} catch (error) {
-		if ((error as NodeJS.ErrnoException).code === "ENOENT") return false;
-		throw error;
-	}
-}
-
-function removeOwnedStateLock(lockPath: string, owner: StateLockOwner): void {
-	const current = readStateLockOwner(lockPath);
-	if (current?.token !== owner.token) return;
-	fs.rmSync(lockPath, { recursive: true, force: true });
-}
-
-function staleDirectoryExists(dirPath: string, now = Date.now()): boolean {
-	try {
-		return now - fs.statSync(dirPath).mtimeMs > STATE_LOCK_STALE_MS;
-	} catch (error) {
-		if ((error as NodeJS.ErrnoException).code === "ENOENT") return false;
-		throw error;
-	}
-}
-
-function tryMakeDirectory(dirPath: string, mode: number): boolean {
-	try {
-		fs.mkdirSync(dirPath, { mode });
-		return true;
-	} catch (error) {
-		if ((error as NodeJS.ErrnoException).code === "EEXIST") return false;
-		throw error;
-	}
-}
-
-function waitForStateLock(delayMs: number | undefined, lockPath: string): void {
-	if (delayMs === undefined) throw new Error(`Timed out acquiring mission state lock '${lockPath}'.`);
-	waitForFileSystemRetry(delayMs);
-}
-
-function reclaimStaleStateLock(lockPath: string, reclaimPath: string, options: StateLockOptions): boolean {
-	if (!stateLockIsStale(lockPath, options)) return false;
-	if (!tryMakeDirectory(reclaimPath, 0o700)) return false;
-	try {
-		if (!stateLockIsStale(lockPath, options)) return false;
-		fs.rmSync(lockPath, { recursive: true, force: true });
-		return true;
-	} finally {
-		fs.rmSync(reclaimPath, { recursive: true, force: true });
-	}
-}
-
-function withStateFileLock<T>(filePath: string, operation: () => T, options: StateLockOptions): T {
-	fs.mkdirSync(path.dirname(filePath), { recursive: true });
-	const lockPath = `${filePath}.lock`;
-	const reclaimPath = `${lockPath}.reclaim`;
-	let owner: StateLockOwner | undefined;
-	for (let attempt = 0; ; attempt++) {
-		if (fs.existsSync(reclaimPath)) {
-			if (staleDirectoryExists(reclaimPath)) {
-				fs.rmSync(reclaimPath, { recursive: true, force: true });
-				continue;
-			}
-			waitForStateLock(options.retryDelaysMs[attempt], lockPath);
-			continue;
-		}
-		let acquired = false;
-		try {
-			acquired = tryMakeDirectory(lockPath, 0o700);
-		} catch (error) {
-			if (isRetryableFileSystemError(error)) {
-				waitForStateLock(options.retryDelaysMs[attempt], lockPath);
-				continue;
-			}
-			throw new Error(`Failed to acquire mission state lock '${lockPath}': ${error instanceof Error ? error.message : String(error)}`);
-		}
-		if (!acquired) {
-			if (reclaimStaleStateLock(lockPath, reclaimPath, options)) continue;
-			waitForStateLock(options.retryDelaysMs[attempt], lockPath);
-			continue;
-		}
-		const key = options.getProcessStartKey(process.pid);
-		owner = { pid: process.pid, token: randomUUID(), createdAt: Date.now(), ...(key ? { processKey: key } : {}) };
-		try {
-			fs.writeFileSync(path.join(lockPath, "owner.json"), JSON.stringify(owner), { encoding: "utf-8", mode: 0o600 });
-		} catch (error) {
-			fs.rmSync(lockPath, { recursive: true, force: true });
-			owner = undefined;
-			throw error;
-		}
-		break;
-	}
-	try {
-		return operation();
-	} finally {
-		if (owner) removeOwnedStateLock(lockPath, owner);
-	}
-}
-
-function validateStateKey(value: unknown): string {
-	if (typeof value !== "string" || !STATE_KEY_PATTERN.test(value)) {
-		throw new Error("state key must be 1-128 characters using letters, numbers, '.', '_' or '-', and start with a letter or number.");
-	}
-	return value;
-}
-
-export function createMissionWorkflowState(location: MissionStoreLocation, missionId: string, options: MissionWorkflowStateOptions = {}): MissionWorkflowState {
-	const filePath = missionStatePath(location, missionId);
-	const lockOptions: StateLockOptions = {
-		isProcessAlive: options.isProcessAlive ?? isProcessAlive,
-		getProcessStartKey: options.getProcessStartKey ?? processStartKey,
-		retryDelaysMs: options.retryDelaysMs ?? DEFAULT_FILE_SYSTEM_RETRY_DELAYS_MS,
-	};
-	let loaded = false;
-	let values: Record<string, unknown> = Object.create(null) as Record<string, unknown>;
-
-	const readStateFile = (): Record<string, unknown> => {
-		let raw: string;
-		try {
-			raw = fs.readFileSync(filePath, "utf-8");
-		} catch (error) {
-			if ((error as NodeJS.ErrnoException).code === "ENOENT") return Object.create(null) as Record<string, unknown>;
-			throw new Error(`Failed to read mission state '${filePath}': ${error instanceof Error ? error.message : String(error)}`);
-		}
-		const bytes = Buffer.byteLength(raw);
-		if (bytes > MISSION_STATE_MAX_BYTES) throw new Error(`Mission state file '${filePath}' exceeds the 256 KiB limit (${bytes} bytes).`);
-		try {
-			const parsed: unknown = JSON.parse(raw);
-			if (!parsed || typeof parsed !== "object" || Array.isArray(parsed)) throw new Error("root must be a JSON object");
-			assertWorkflowJsonValue(parsed, "mission state");
-			return Object.assign(Object.create(null) as Record<string, unknown>, parsed);
-		} catch (error) {
-			throw new Error(`Invalid mission state file '${filePath}': ${error instanceof Error ? error.message : String(error)}`);
-		}
-	};
-
-	const load = (): Record<string, unknown> => {
-		if (loaded) return values;
-		values = readStateFile();
-		loaded = true;
-		return values;
-	};
-
-	return {
-		path: filePath,
-		get(key) {
-			const validKey = validateStateKey(key);
-			const current = load();
-			return Object.hasOwn(current, validKey) ? current[validKey] : undefined;
-		},
-		set(key, value) {
-			const validKey = validateStateKey(key);
-			assertWorkflowJsonValue(value, `state.set('${validKey}') value`);
-			withStateFileLock(filePath, () => {
-				const next = Object.assign(Object.create(null) as Record<string, unknown>, readStateFile(), { [validKey]: value });
-				const bytes = Buffer.byteLength(JSON.stringify(next, null, 2));
-				if (bytes > MISSION_STATE_MAX_BYTES) throw new Error(`Mission state exceeds the 256 KiB limit (${bytes} bytes; maximum ${MISSION_STATE_MAX_BYTES} bytes).`);
-				writePrivateAtomicJson(filePath, next);
-				values = next;
-				loaded = true;
-			}, lockOptions);
-		},
-	};
-}

package/src/policy/authority.ts

@@ -1,50 +0,0 @@
-export const AUTHORITY_ACTIONS = [
-	"discardWorktree",
-	"destructiveCleanup",
-	"spawnBudgetGrant",
-	"scheduleCreate",
-	"stopRun",
-	"steerRun",
-	"inspectorOpen",
-	"projectOpen",
-] as const;
-
-export type AuthorityAction = typeof AUTHORITY_ACTIONS[number];
-export type AuthorityDecision = "auto" | "confirm" | "forbid";
-export type AuthorityPolicyConfig = Partial<Record<AuthorityAction, AuthorityDecision>>;
-
-const DEFAULT_AUTHORITY_POLICY: Record<AuthorityAction, AuthorityDecision> = {
-	discardWorktree: "confirm",
-	destructiveCleanup: "confirm",
-	spawnBudgetGrant: "confirm",
-	scheduleCreate: "auto",
-	stopRun: "auto",
-	steerRun: "auto",
-	inspectorOpen: "auto",
-	projectOpen: "confirm",
-};
-
-export function resolveAuthorityDecision(input: {
-	action: AuthorityAction;
-	policy?: AuthorityPolicyConfig;
-}): AuthorityDecision {
-	return input.policy?.[input.action] ?? DEFAULT_AUTHORITY_POLICY[input.action];
-}
-
-export function validateAuthorityPolicy(value: unknown, label = "config.authorityPolicy"): AuthorityPolicyConfig | undefined {
-	if (value === undefined) return undefined;
-	if (!value || typeof value !== "object" || Array.isArray(value)) {
-		throw new Error(`${label} must be a JSON object`);
-	}
-	const policy = value as Record<string, unknown>;
-	const allowedActions = new Set<string>(AUTHORITY_ACTIONS);
-	for (const [action, decision] of Object.entries(policy)) {
-		if (!allowedActions.has(action)) {
-			throw new Error(`${label}.${action} is unknown; expected one of ${AUTHORITY_ACTIONS.join(", ")}`);
-		}
-		if (decision !== "auto" && decision !== "confirm" && decision !== "forbid") {
-			throw new Error(`${label}.${action} must be "auto", "confirm", or "forbid"`);
-		}
-	}
-	return policy as AuthorityPolicyConfig;
-}