@selesai/code 0.3.0 → 0.3.1

package/dist/extensions/pi-subagents/src/runs/background/control-channel.ts

@@ -14,6 +14,7 @@
  * authoritative.
  */
 
+import { randomUUID } from "node:crypto";
 import * as fs from "node:fs";
 import * as path from "node:path";
 import { writeAtomicJson } from "../../shared/atomic-json.ts";
@@ -26,7 +27,7 @@ import { POLL_INTERVAL_MS } from "../../shared/types.ts";
  */
 export const INTERRUPT_SIGNAL: NodeJS.Signals = process.platform === "win32" ? "SIGBREAK" : "SIGUSR2";
 
-export type ControlChannelFs = Pick<typeof fs, "mkdirSync" | "existsSync" | "rmSync" | "watch">;
+export type ControlChannelFs = Pick<typeof fs, "mkdirSync" | "existsSync" | "rmSync" | "watch" | "readdirSync" | "readFileSync">;
 export type ControlChannelTimers = { setInterval: typeof setInterval; clearInterval: typeof clearInterval };
 type KillFn = (pid: number, signal?: NodeJS.Signals | 0) => unknown;
 
@@ -37,6 +38,25 @@ export interface InterruptRequest {
 	reason?: string;
 }
 
+export interface TimeoutRequest {
+	type: "timeout";
+	ts?: number;
+	source?: string;
+	reason?: string;
+}
+
+export interface SteerRequest {
+	type: "steer";
+	id: string;
+	ts: number;
+	message: string;
+	targetIndex?: number;
+	source?: string;
+}
+
+const STEER_REQUESTS_DIR = "steer-requests";
+const STEER_TARGETS_DIR = "steer-targets";
+
 /** Control inbox directory inside an async run dir. */
 export function controlInboxDir(asyncDir: string): string {
 	return path.join(asyncDir, "control");
@@ -47,6 +67,31 @@ export function interruptRequestPath(asyncDir: string): string {
 	return path.join(controlInboxDir(asyncDir), "interrupt.json");
 }
 
+/** Path of the portable timeout request file. */
+export function timeoutRequestPath(asyncDir: string): string {
+	return path.join(controlInboxDir(asyncDir), "timeout.json");
+}
+
+/** Directory of parent-to-runner steering requests. */
+export function steerRequestsDir(asyncDir: string): string {
+	return path.join(controlInboxDir(asyncDir), STEER_REQUESTS_DIR);
+}
+
+/** Per-child inbox consumed by the child prompt runtime inside the Pi process. */
+export function stepSteerInboxDir(asyncDir: string, index: number): string {
+	return path.join(controlInboxDir(asyncDir), STEER_TARGETS_DIR, String(index));
+}
+
+function steerRequestFileName(request: SteerRequest): string {
+	return `${String(request.ts).padStart(13, "0")}-${Buffer.from(request.id).toString("base64url")}.json`;
+}
+
+export function writeSteerRequestToDir(dir: string, request: SteerRequest): string {
+	const requestPath = path.join(dir, steerRequestFileName(request));
+	writeAtomicJson(requestPath, request);
+	return requestPath;
+}
+
 /**
  * Parent side: drop a portable interrupt request the runner's inbox watcher will
  * pick up regardless of OS. Written atomically (temp + rename), dir auto-created.
@@ -62,6 +107,87 @@ export function requestAsyncInterrupt(
 	return requestPath;
 }
 
+export function requestAsyncTimeout(
+	asyncDir: string,
+	payload: Omit<TimeoutRequest, "type"> = {},
+	deps: { now?: () => number } = {},
+): string {
+	const requestPath = timeoutRequestPath(asyncDir);
+	const request: TimeoutRequest = { ...payload, ts: payload.ts ?? deps.now?.() ?? Date.now(), type: "timeout" };
+	writeAtomicJson(requestPath, request);
+	return requestPath;
+}
+
+export function requestAsyncSteer(
+	asyncDir: string,
+	payload: { message: string; targetIndex?: number; source?: string; id?: string; ts?: number },
+	deps: { now?: () => number; randomId?: () => string } = {},
+): string {
+	const message = payload.message.trim();
+	if (!message) throw new Error("steer message must not be empty.");
+	if (payload.targetIndex !== undefined && (!Number.isInteger(payload.targetIndex) || payload.targetIndex < 0)) {
+		throw new Error("steer targetIndex must be a non-negative integer.");
+	}
+	const request: SteerRequest = {
+		type: "steer",
+		id: payload.id ?? deps.randomId?.() ?? randomUUID(),
+		ts: payload.ts ?? deps.now?.() ?? Date.now(),
+		message,
+		...(payload.targetIndex !== undefined ? { targetIndex: payload.targetIndex } : {}),
+		...(payload.source ? { source: payload.source } : {}),
+	};
+	return writeSteerRequestToDir(steerRequestsDir(asyncDir), request);
+}
+
+export function enqueueStepSteer(asyncDir: string, index: number, request: SteerRequest): string {
+	if (!Number.isInteger(index) || index < 0) throw new Error("steer child index must be a non-negative integer.");
+	return writeSteerRequestToDir(stepSteerInboxDir(asyncDir, index), { ...request, targetIndex: index, type: "steer" });
+}
+
+function parseSteerRequest(raw: unknown): SteerRequest | undefined {
+	if (!raw || typeof raw !== "object" || Array.isArray(raw)) return undefined;
+	const input = raw as Partial<SteerRequest>;
+	if (input.type !== "steer") return undefined;
+	if (typeof input.id !== "string" || !input.id.trim()) return undefined;
+	if (typeof input.ts !== "number" || !Number.isFinite(input.ts)) return undefined;
+	if (typeof input.message !== "string" || !input.message.trim()) return undefined;
+	if (input.targetIndex !== undefined && (!Number.isInteger(input.targetIndex) || input.targetIndex < 0)) return undefined;
+	return {
+		type: "steer",
+		id: input.id.trim(),
+		ts: input.ts,
+		message: input.message.trim(),
+		...(input.targetIndex !== undefined ? { targetIndex: input.targetIndex } : {}),
+		...(typeof input.source === "string" && input.source.trim() ? { source: input.source } : {}),
+	};
+}
+
+export function consumeSteerRequestsFromDir(dir: string, fsImpl: Pick<typeof fs, "existsSync" | "rmSync" | "readdirSync" | "readFileSync"> = fs): SteerRequest[] {
+	if (!fsImpl.existsSync(dir)) return [];
+	const requests: SteerRequest[] = [];
+	for (const entry of fsImpl.readdirSync(dir).filter((name) => name.endsWith(".json")).sort()) {
+		const requestPath = path.join(dir, entry);
+		let parsed: SteerRequest | undefined;
+		try {
+			parsed = parseSteerRequest(JSON.parse(fsImpl.readFileSync(requestPath, "utf-8")));
+		} catch {
+			parsed = undefined;
+		}
+		try {
+			fsImpl.rmSync(requestPath, { recursive: true });
+		} catch {
+			// Already removed by a concurrent check — do not execute it twice.
+			continue;
+		}
+		if (parsed) requests.push(parsed);
+	}
+	return requests.sort((left, right) => left.ts - right.ts || left.id.localeCompare(right.id));
+}
+
+export function consumeSteerRequests(asyncDir: string, fsImpl: Pick<typeof fs, "existsSync" | "rmSync" | "readdirSync" | "readFileSync"> = fs): SteerRequest[] {
+	return consumeSteerRequestsFromDir(steerRequestsDir(asyncDir), fsImpl);
+}
+
 /**
  * Runner side: consume a pending interrupt request. Idempotent — removes the file
  * so each distinct request fires exactly once. Returns whether one was pending.
@@ -80,6 +206,20 @@ export function consumeInterruptRequest(
 	return true;
 }
 
+export function consumeTimeoutRequest(
+	asyncDir: string,
+	fsImpl: Pick<typeof fs, "existsSync" | "rmSync"> = fs,
+): boolean {
+	const requestPath = timeoutRequestPath(asyncDir);
+	if (!fsImpl.existsSync(requestPath)) return false;
+	try {
+		fsImpl.rmSync(requestPath, { force: true, recursive: true });
+	} catch {
+		// Already removed by a concurrent check — still counts as consumed.
+	}
+	return true;
+}
+
 /**
  * Parent side: portable interrupt = authoritative file request + best-effort OS
  * signal. The signal is only a latency optimization on Unix; ENOSYS on Windows
@@ -114,6 +254,17 @@ export function deliverInterruptRequest(input: {
 	}
 }
 
+export function deliverTimeoutRequest(input: {
+	asyncDir: string;
+	pid?: number;
+	kill?: KillFn;
+	signal?: NodeJS.Signals;
+	now?: () => number;
+	source?: string;
+}): void {
+	requestAsyncTimeout(input.asyncDir, input.source ? { source: input.source } : {}, { now: input.now });
+}
+
 /**
  * Runner side: watch the control inbox and route interrupt requests into
  * `onInterrupt`. Uses `fs.watch` when available plus an interval poll as a
@@ -124,6 +275,8 @@ export function watchAsyncControlInbox(
 	asyncDir: string,
 	opts: {
 		onInterrupt: () => void;
+		onTimeout?: () => void;
+		onSteer?: (request: SteerRequest) => void;
 		pollIntervalMs?: number;
 		fs?: ControlChannelFs;
 		timers?: ControlChannelTimers;
@@ -142,7 +295,9 @@ export function watchAsyncControlInbox(
 	const check = (): void => {
 		if (disposed) return;
 		try {
+			if (consumeTimeoutRequest(asyncDir, fsImpl)) opts.onTimeout?.();
 			if (consumeInterruptRequest(asyncDir, fsImpl)) opts.onInterrupt();
+			for (const request of consumeSteerRequests(asyncDir, fsImpl)) opts.onSteer?.(request);
 		} catch {
 			// Never let inbox errors crash the runner.
 		}