@selesai/code 0.13.39 → 0.14.0

package/dist/extensions/pi-subagents/src/intercom/native-supervisor-channel.ts

@@ -8,6 +8,7 @@ import type { ChildSupervisorMetadata } from "../runs/shared/child-runtime-confi
 import { INTERCOM_DETACH_REQUEST_EVENT, POLL_INTERVAL_MS, TEMP_ROOT_DIR, type ControlEvent, type IntercomEventBus, type SubagentState } from "../shared/types.ts";
 import { writeAtomicJson } from "../shared/atomic-json.ts";
 import { shouldUseNativeFsWatch } from "../shared/watch-strategy.ts";
+import { MODEL_ONLY_TOOL } from "../shared/extension-context.ts";
 import {
 	SUPERVISOR_REQUEST_MESSAGE_TYPE,
 	SUPERVISOR_REPLY_ENTRY_TYPE,
@@ -83,8 +84,11 @@ interface IntercomParams {
 type SupervisorWatch = (filename: fs.PathLike, listener: fs.WatchListener<string>) => fs.FSWatcher;
 
 interface NativeSupervisorChannelDeps {
+	/** Owned live/final-drain mailboxes. Only a completed poll retires the snapshot, never a demand probe. */
+	getChannelDirs?: () => { dirs: string[]; retire?: () => void };
 	/** Retained scheduled states for the current runtime owner, never foreign owners. */
 	getCurrentOwnerStates?: () => Iterable<SubagentState>;
+	parentWake?: Pick<ExtensionAPI, "sendMessage">;
 	platform?: NodeJS.Platform;
 	watch?: SupervisorWatch;
 	timers?: Pick<typeof globalThis, "setInterval" | "clearInterval" | "setImmediate" | "clearImmediate">;
@@ -257,13 +261,9 @@ export function registerNativeSupervisorClient(pi: ExtensionAPI, metadata: Child
 	if (!metadata || hasTool(pi, "contact_supervisor")) return;
 	const tool: ToolDefinition<typeof ContactSupervisorParamsSchema, Record<string, unknown>> = {
 		name: "contact_supervisor",
+		...MODEL_ONLY_TOOL,
 		label: "Contact Supervisor",
 		description: "Contact the parent/supervisor session for a blocking decision, structured interview, or progress update.",
-		...{ discovery: {
-			summary: "Contact the parent/supervisor session for decisions, interviews, or progress updates",
-			aliases: ["contact supervisor", "ask supervisor"],
-			category: "coordination",
-		} },
 		parameters: ContactSupervisorParamsSchema,
 		execute(id, params, signal) {
 			return sendSupervisorRequest(params as ContactSupervisorParams, metadata, signal, id);
@@ -291,18 +291,23 @@ function parseRequestFile(file: string, channelDir: string): PendingSupervisorRe
 	}
 }
 
-function listRequestFiles(): Array<{ channelDir: string; file: string }> {
-	let channelEntries: fs.Dirent[];
-	try {
-		channelEntries = fs.readdirSync(SUPERVISOR_CHANNEL_ROOT, { withFileTypes: true });
-	} catch (error) {
-		if ((error as NodeJS.ErrnoException).code === "ENOENT") return [];
-		throw error;
+function isMissingSupervisorDirectory(error: unknown, platform: NodeJS.Platform): boolean {
+	const code = (error as NodeJS.ErrnoException).code;
+	return code === "ENOENT" || (platform === "win32" && code === "UNKNOWN");
+}
+
+function listRequestFiles(channelDirs: string[] | undefined, platform: NodeJS.Platform): Array<{ channelDir: string; file: string }> {
+	if (!channelDirs) {
+		try {
+			channelDirs = fs.readdirSync(SUPERVISOR_CHANNEL_ROOT, { withFileTypes: true })
+				.filter(entry => entry.isDirectory()).map(entry => path.join(SUPERVISOR_CHANNEL_ROOT, entry.name));
+		} catch (error) {
+			if (isMissingSupervisorDirectory(error, platform)) return [];
+			throw error;
+		}
 	}
 	const files: Array<{ channelDir: string; file: string }> = [];
-	for (const entry of channelEntries) {
-		if (!entry.isDirectory()) continue;
-		const channelDir = path.join(SUPERVISOR_CHANNEL_ROOT, entry.name);
+	for (const channelDir of channelDirs) {
 		const requestsDir = path.join(channelDir, REQUESTS_DIR);
 		let requestEntries: fs.Dirent[];
 		try {
@@ -317,11 +322,11 @@ function listRequestFiles(): Array<{ channelDir: string; file: string }> {
 	return files;
 }
 
-function readDirectoryEntries(dir: string): fs.Dirent[] | undefined {
+function readDirectoryEntries(dir: string, platform: NodeJS.Platform): fs.Dirent[] | undefined {
 	try {
 		return fs.readdirSync(dir, { withFileTypes: true });
 	} catch (error) {
-		if ((error as NodeJS.ErrnoException).code === "ENOENT") return [];
+		if (isMissingSupervisorDirectory(error, platform)) return [];
 		return undefined;
 	}
 }
@@ -346,7 +351,7 @@ function removeEmptyDirectory(dir: string): boolean {
 	}
 }
 
-function removeStaleEmptySupervisorChannel(channelDir: string, nowMs: number): boolean {
+function removeStaleEmptySupervisorChannel(channelDir: string, nowMs: number, platform: NodeJS.Platform): boolean {
 	const requestsDir = path.join(channelDir, REQUESTS_DIR);
 	const repliesDir = path.join(channelDir, REPLIES_DIR);
 	const newestKnownMtimeMs = Math.max(
@@ -356,9 +361,9 @@ function removeStaleEmptySupervisorChannel(channelDir: string, nowMs: number): b
 	);
 	if (nowMs - newestKnownMtimeMs < STALE_EMPTY_CHANNEL_AGE_MS) return false;
 
-	const requestEntries = readDirectoryEntries(requestsDir);
+	const requestEntries = readDirectoryEntries(requestsDir, platform);
 	if (!requestEntries || requestEntries.length > 0) return false;
-	const replyEntries = readDirectoryEntries(repliesDir);
+	const replyEntries = readDirectoryEntries(repliesDir, platform);
 	if (!replyEntries || replyEntries.length > 0) return false;
 
 	if (!removeEmptyDirectory(requestsDir)) return false;
@@ -367,12 +372,12 @@ function removeStaleEmptySupervisorChannel(channelDir: string, nowMs: number): b
 	return true;
 }
 
-function cleanupStaleEmptySupervisorChannels(nowMs = Date.now()): number {
+function cleanupStaleEmptySupervisorChannels(nowMs = Date.now(), platform: NodeJS.Platform = process.platform): number {
 	let channelEntries: fs.Dirent[];
 	try {
 		channelEntries = fs.readdirSync(SUPERVISOR_CHANNEL_ROOT, { withFileTypes: true });
 	} catch (error) {
-		if ((error as NodeJS.ErrnoException).code === "ENOENT") return 0;
+		if (isMissingSupervisorDirectory(error, platform)) return 0;
 		throw error;
 	}
 
@@ -380,7 +385,7 @@ function cleanupStaleEmptySupervisorChannels(nowMs = Date.now()): number {
 	for (const entry of channelEntries) {
 		if (!entry.isDirectory()) continue;
 		try {
-			if (removeStaleEmptySupervisorChannel(path.join(SUPERVISOR_CHANNEL_ROOT, entry.name), nowMs)) removed++;
+			if (removeStaleEmptySupervisorChannel(path.join(SUPERVISOR_CHANNEL_ROOT, entry.name), nowMs, platform)) removed++;
 		} catch {
 			// Cleanup is opportunistic; active writers can race with us and will be picked up by a later pass.
 		}
@@ -485,7 +490,9 @@ function refreshPendingRequests(pending: Map<string, PendingSupervisorRequest>,
 
 function formatPendingLine(request: PendingSupervisorRequest): string {
 	const replyHint = request.expectsReply ? ` Reply: ${supervisorReplyHint(request.id)}` : "";
-	return `- ${request.id}: ${request.agent} [${request.runId}#${request.childIndex}] ${request.reason}.${replyHint}`;
+	const header = `- ${request.id}: ${request.agent} [${request.runId}#${request.childIndex}] ${request.reason}.${replyHint}`;
+	// The request notice can be missed; pending is the parent's only way to read the question again.
+	return request.message ? `${header}\n  ${request.message.replace(/\n/g, "\n  ")}` : header;
 }
 
 function requestVisibleText(request: PendingSupervisorRequest): string {
@@ -581,13 +588,9 @@ function publicPendingRequests(pending: Map<string, PendingSupervisorRequest>):
 function buildParentSupervisorTool(pi: ExtensionAPI, pending: Map<string, PendingSupervisorRequest>, state: SubagentState, onLifecycle: SupervisorRequestLifecycleObserver, discover: () => void, runState: (request: SupervisorRequest) => SubagentState): ToolDefinition<typeof IntercomParamsSchema, Record<string, unknown>> {
 	return {
 		name: NATIVE_SUPERVISOR_TOOL_NAME,
+		...MODEL_ONLY_TOOL,
 		label: "Subagent Supervisor",
 		description: "Native pi-subagents supervisor channel. Use reply/pending/status to answer child subagent requests without overriding pi-intercom.",
-		...{ discovery: {
-			summary: "Answer child subagent requests: reply, pending, or status",
-			aliases: ["supervisor channel", "reply to subagent"],
-			category: "coordination",
-		} },
 		parameters: IntercomParamsSchema,
 		async execute(_id, params) {
 			// Discover new request files even when demand-gated polling is idle.
@@ -616,9 +619,11 @@ function buildParentSupervisorTool(pi: ExtensionAPI, pending: Map<string, Pendin
 }
 
 export function createNativeSupervisorChannel(pi: ExtensionAPI, state: SubagentState, deps: NativeSupervisorChannelDeps = {}): {
+	registerTools: () => void;
 	start: () => void;
 	activateTransport: () => void;
 	findPendingAsks: (target: { runId: string; agent: string; childIndex: number }) => string[];
+	hasPendingRequests: () => boolean;
 	dispose: () => void;
 	pending: Map<string, PendingSupervisorRequest>;
 	getSupervisorRequestState: (event: ControlEvent) => SupervisorRequestState;
@@ -691,10 +696,11 @@ export function createNativeSupervisorChannel(pi: ExtensionAPI, state: SubagentS
 	let started = false;
 	let lastStaleCleanupAt = 0;
 	const platform = deps.platform ?? process.platform;
-	const useNativeWatcher = () => shouldUseNativeFsWatch("supervisor-channel", platform) && platform !== "win32";
+	const useNativeWatcher = () => !deps.getChannelDirs && shouldUseNativeFsWatch("supervisor-channel", platform) && platform !== "win32";
 	const hasTransportDemand = () => {
 		if (pending.size > 0) return true;
 		if (state.foregroundControls.size > 0) return true;
+		if (deps.getChannelDirs?.().dirs.length) return true;
 		if ([...state.asyncJobs.values()].some((job) => job.status === "queued" || job.status === "running")) return true;
 		for (const ownerState of deps.getCurrentOwnerStates?.() ?? []) {
 			for (const job of ownerState.asyncJobs.values()) {
@@ -709,11 +715,12 @@ export function createNativeSupervisorChannel(pi: ExtensionAPI, state: SubagentS
 	};
 
 	const cleanupStaleChannelsIfDue = (): void => {
+		if (deps.getChannelDirs) return; // The root owns global retention cleanup.
 		const nowMs = Date.now();
 		if (nowMs - lastStaleCleanupAt < STALE_EMPTY_CHANNEL_CLEANUP_INTERVAL_MS) return;
 		lastStaleCleanupAt = nowMs;
 		try {
-			cleanupStaleEmptySupervisorChannels(nowMs);
+			cleanupStaleEmptySupervisorChannels(nowMs, platform);
 		} catch {
 			// Supervisor delivery must not fail because best-effort temp cleanup failed.
 		}
@@ -724,7 +731,8 @@ export function createNativeSupervisorChannel(pi: ExtensionAPI, state: SubagentS
 		// Only display notifications require a live UI context, not request registration.
 		refreshPendingRequests(pending, state, observeRequestLifecycle, runState);
 		const now = Date.now();
-		for (const { channelDir, file } of listRequestFiles()) {
+		const channels = deps.getChannelDirs?.();
+		for (const { channelDir, file } of listRequestFiles(channels?.dirs, platform)) {
 			if (seenFiles.has(file)) continue;
 			const request = parseRequestFile(file, channelDir);
 			if (!request || !requestMatchesOwner(request, state)) continue;
@@ -736,15 +744,19 @@ export function createNativeSupervisorChannel(pi: ExtensionAPI, state: SubagentS
 				continue;
 			}
 			seenFiles.add(file);
-			if (request.expectsReply) {
-				rememberPendingRequest(request);
-				pending.set(request.id, request);
-				markForegroundSupervisorAttention(request, state);
+			if (!request.expectsReply) {
+				// Progress is already visible through child activity; do not inject a
+				// parent message or trigger a parent model turn.
+				removeRequestFile(request.requestFile);
+				continue;
 			}
+			rememberPendingRequest(request);
+			pending.set(request.id, request);
+			markForegroundSupervisorAttention(request, state);
 			// The ask is already queued above. A sendMessage failure (no UI, stale context) must not
 			// lose it, and must not abort the loop before the remaining asks register.
 			try {
-				pi.sendMessage({
+				(deps.parentWake ?? pi).sendMessage({
 					customType: SUPERVISOR_REQUEST_MESSAGE_TYPE,
 					content: requestVisibleText(request),
 					display: true,
@@ -759,33 +771,28 @@ export function createNativeSupervisorChannel(pi: ExtensionAPI, state: SubagentS
 						...(request.childTarget ? { childTarget: request.childTarget } : {}),
 						...(request.interview !== undefined ? { interview: request.interview } : {}),
 						requestBody: request.message,
-						...(request.expectsReply ? { replyHint: supervisorReplyHint(request.id) } : {}),
+						replyHint: supervisorReplyHint(request.id),
 					},
 				}, { triggerTurn: true });
-				// sendMessage accepts synchronously; one-way updates stay on disk until it returns.
-				if (!request.expectsReply) removeRequestFile(request.requestFile);
 			} catch (error) {
-				// Allow an existing later scan to retry an unaccepted one-way update.
-				if (!request.expectsReply) seenFiles.delete(file);
 				console.error(`Failed to surface supervisor request ${request.id} as a user turn:`, error);
 			}
-			if (request.expectsReply) {
-				(pi as { events?: IntercomEventBus }).events?.emit(INTERCOM_DETACH_REQUEST_EVENT, {
-					requestId: request.id,
-					runId: request.runId,
-					agent: request.agent,
-					childIndex: request.childIndex,
-				});
-				if (pending.has(request.id)) markForegroundSupervisorAttention(request, state);
-			}
+			(pi as { events?: IntercomEventBus }).events?.emit(INTERCOM_DETACH_REQUEST_EVENT, {
+				requestId: request.id,
+				runId: request.runId,
+				agent: request.agent,
+				childIndex: request.childIndex,
+			});
+			if (pending.has(request.id)) markForegroundSupervisorAttention(request, state);
 		}
+		channels?.retire?.();
 	};
 
 	const startPolling = (): void => {
 		if (poller) return;
 		poller = timers.setInterval(() => {
 			poll();
-			if (!useNativeWatcher() && platform === "darwin" && !hasTransportDemand()) {
+			if (!useNativeWatcher() && !hasTransportDemand()) {
 				if (poller) timers.clearInterval(poller);
 				poller = undefined;
 			}
@@ -820,7 +827,7 @@ export function createNativeSupervisorChannel(pi: ExtensionAPI, state: SubagentS
 		try {
 			channelEntries = fs.readdirSync(SUPERVISOR_CHANNEL_ROOT, { withFileTypes: true });
 		} catch (error) {
-			if ((error as NodeJS.ErrnoException).code === "ENOENT") return;
+			if (isMissingSupervisorDirectory(error, platform)) return;
 			startPolling();
 			return;
 		}
@@ -840,6 +847,7 @@ export function createNativeSupervisorChannel(pi: ExtensionAPI, state: SubagentS
 	};
 
 	return {
+		registerTools: registerParentTools,
 		activateTransport: () => {
 			if (!started) return;
 			poll();
@@ -851,7 +859,7 @@ export function createNativeSupervisorChannel(pi: ExtensionAPI, state: SubagentS
 			let files: string[];
 			try { files = fs.readdirSync(path.join(channelDir, REQUESTS_DIR)); }
 			catch (error) {
-				if ((error as NodeJS.ErrnoException).code === "ENOENT") return [];
+				if (isMissingSupervisorDirectory(error, platform)) return [];
 				throw error;
 			}
 			const now = Date.now();
@@ -862,11 +870,17 @@ export function createNativeSupervisorChannel(pi: ExtensionAPI, state: SubagentS
 					&& requestLifecycle(request, state, now, runState(request)) === "pending" ? [request.id] : [];
 			}).sort();
 		},
+		hasPendingRequests: () => {
+			if (!started) return false;
+			poll();
+			return pending.size > 0;
+		},
 		start: () => {
 			if (started) return;
 			started = true;
 			registerParentTools();
 			poll();
+			if (deps.getChannelDirs) return; // Child polling starts only after a descendant launch.
 			try {
 				fs.mkdirSync(SUPERVISOR_CHANNEL_ROOT, { recursive: true });
 				if (!useNativeWatcher()) {
@@ -911,4 +925,4 @@ export function createNativeSupervisorChannel(pi: ExtensionAPI, state: SubagentS
 		pending,
 		getSupervisorRequestState,
 	};
-}
+}

package/dist/extensions/pi-subagents/src/intercom/result-intercom.ts

@@ -31,6 +31,7 @@ export function resolveSubagentResultStatus(input: {
 	if (input.detached) return "detached";
 	if (input.stopped || input.state === "stopped") return "stopped";
 	if (input.interrupted || input.state === "paused") return "paused";
+	if (input.state === "running" || input.state === "queued") return "running";
 	if (input.success === true) return "completed";
 	if (isUnexplainedProcessSignal(input) && input.exitCode !== 0) return "stopped";
 	if (input.success === false) return "failed";
@@ -42,6 +43,7 @@ export function resolveSubagentResultStatus(input: {
 
 function countStatuses(children: SubagentResultIntercomChild[]): Record<SubagentResultStatus, number> {
 	const counts: Record<SubagentResultStatus, number> = {
+		running: 0,
 		completed: 0,
 		failed: 0,
 		paused: 0,
@@ -56,6 +58,7 @@ function countStatuses(children: SubagentResultIntercomChild[]): Record<Subagent
 
 function formatStatusCounts(counts: Record<SubagentResultStatus, number>): string {
 	const parts = [
+		counts.running ? `${counts.running} running` : undefined,
 		counts.completed ? `${counts.completed} completed` : undefined,
 		counts.failed ? `${counts.failed} failed` : undefined,
 		counts.stopped ? `${counts.stopped} stopped` : undefined,
@@ -85,6 +88,7 @@ function resolveGroupedStatus(children: SubagentResultIntercomChild[]): Subagent
 	if (counts.failed > 0) return "failed";
 	if (counts.stopped > 0) return "stopped";
 	if (counts.paused > 0) return "paused";
+	if (counts.running > 0) return "running";
 	if (counts.completed > 0) return "completed";
 	if (counts.detached > 0) return "detached";
 	return "failed";
@@ -272,7 +276,7 @@ function formatSubagentResultIntercomMessage(input: {
 	for (let index = 0; index < input.children.length; index++) {
 		const child = input.children[index]!;
 		lines.push("");
-		lines.push(`${index + 1}. ${child.agent} — process ${child.status} · output ${child.outputState ?? "unknown"}`);
+		lines.push(`${index + 1}. ${child.agent} — process ${child.status} · output ${child.outputState ?? "unknown"}${child.outputPartial ? " (partial)" : ""}`);
 		if (child.intercomTarget) lines.push(`${input.source === "async" ? "Previous intercom target" : "Run intercom target"}: ${child.intercomTarget}`);
 		if (child.artifactPath) lines.push(`Output artifact: ${child.artifactPath}`);
 		if (child.sessionPath) lines.push(`Session: ${child.sessionPath}`);
@@ -288,6 +292,7 @@ export function buildSubagentResultIntercomPayload(input: GroupedResultIntercomM
 	const children = input.children.map((child) => ({
 		...child,
 		outputState: child.outputState ?? "unknown",
+		...(child.outputPartial ? { outputPartial: true } : {}),
 		summary: child.summary.trim() || "(no output)",
 		children: compactNestedResultChildren(child.children),
 	}));
@@ -420,4 +425,4 @@ export function formatSubagentResultReceipt(input: {
 
 	lines.push("Full grouped output was sent over intercom.");
 	return lines.join("\n");
-}
+}

package/dist/extensions/pi-subagents/src/missions/goal-driver.ts

@@ -31,7 +31,14 @@ function readLinkedRun(run: MissionRunLink): MissionRunLink {
 	if (!run.asyncDir) return run;
 	const statusPath = path.join(run.asyncDir, "status.json");
 	if (!fs.existsSync(statusPath)) return run;
-	const status = JSON.parse(fs.readFileSync(statusPath, "utf-8")) as Record<string, unknown>;
+	let parsed: unknown;
+	try {
+		parsed = JSON.parse(fs.readFileSync(statusPath, "utf-8"));
+	} catch (error) {
+		throw new Error(`Failed to read linked run status '${statusPath}': ${error instanceof Error ? error.message : String(error)}`, { cause: error });
+	}
+	if (!parsed || typeof parsed !== "object" || Array.isArray(parsed)) throw new Error(`Linked run status '${statusPath}' must be an object`);
+	const status = parsed as Record<string, unknown>;
 	if (typeof status.state !== "string" || !status.state.trim()) throw new Error(`Linked run status '${statusPath}' is missing state`);
 	const tokens = tokenUsage(status.totalTokens)
 		?? (Array.isArray(status.steps)
@@ -123,42 +130,48 @@ export function collectGoalContinuationNotices(input: {
 	retainedChildren: RetainedChild[];
 	turnId: number;
 	now?: number;
+	onError?: (missionId: string, error: unknown) => void;
 }): GoalContinuationNotice[] {
 	const notices: GoalContinuationNotice[] = [];
 	const seen = new Set<string>();
+	const onError = input.onError ?? ((missionId: string, error: unknown) => console.error("Failed to evaluate goal mission", missionId, error));
 	for (const listed of listMissions(input.location).records) {
 		if (listed.ownerSessionId !== input.ownerSessionId || !listed.goal || TERMINAL_MISSION_STATUSES.has(listed.status)) continue;
-		let record = refreshGoalMission(input.location, readMission(input.location, listed.id));
-		if (!record.goal || record.goal.status !== "active" || !record.budget) continue;
-		if ((record.usage?.tokens ?? 0) >= record.budget.tokens) {
-			record = updateMission(input.location, record.id, { usage: record.usage ?? { tokens: 0 } });
-			if (record.goal?.status === "budget-exhausted") continue;
-		}
-		if (record.runs.some((run) => run.status && ACTIVE_RUN_STATUSES.has(run.status))) continue;
-		if (seen.has(record.id) || !record.budget) continue;
-		seen.add(record.id);
-		const budget = record.budget.tokens;
-		const used = record.usage?.tokens ?? 0;
-		const remaining = Math.max(0, budget - used);
-		const message = [
-			`Goal mission needs attention: ${bounded(record.title)}`,
-			`Mission: ${record.id}`,
-			`Remaining budget: ${remaining} tokens (${used}/${budget} used)`,
-			`Next ready action: ${nextReadyAction(input.location, record, input.retainedChildren)}`,
-		].join("\n");
-		notices.push({
-			missionId: record.id,
-			message,
-			event: {
-				type: "needs_attention",
-				to: "needs_attention",
-				ts: input.now ?? Date.now(),
-				runId: `goal-${record.id}-turn-${input.turnId}`,
-				agent: "goal mission",
+		try {
+			let record = refreshGoalMission(input.location, readMission(input.location, listed.id));
+			if (!record.goal || record.goal.status !== "active" || !record.budget) continue;
+			if ((record.usage?.tokens ?? 0) >= record.budget.tokens) {
+				record = updateMission(input.location, record.id, { usage: record.usage ?? { tokens: 0 } });
+				if (record.goal?.status === "budget-exhausted") continue;
+			}
+			if (record.runs.some((run) => run.status && ACTIVE_RUN_STATUSES.has(run.status))) continue;
+			if (seen.has(record.id) || !record.budget) continue;
+			seen.add(record.id);
+			const budget = record.budget.tokens;
+			const used = record.usage?.tokens ?? 0;
+			const remaining = Math.max(0, budget - used);
+			const message = [
+				`Goal mission needs attention: ${bounded(record.title)}`,
+				`Mission: ${record.id}`,
+				`Remaining budget: ${remaining} tokens (${used}/${budget} used)`,
+				`Next ready action: ${nextReadyAction(input.location, record, input.retainedChildren)}`,
+			].join("\n");
+			notices.push({
+				missionId: record.id,
 				message,
-				reason: "idle",
-			},
-		});
+				event: {
+					type: "needs_attention",
+					to: "needs_attention",
+					ts: input.now ?? Date.now(),
+					runId: `goal-${record.id}-turn-${input.turnId}`,
+					agent: "goal mission",
+					message,
+					reason: "idle",
+				},
+			});
+		} catch (error) {
+			onError(listed.id, error);
+		}
 	}
 	return notices;
-}
+}

package/dist/extensions/pi-subagents/src/missions/store.ts

@@ -562,7 +562,16 @@ export function listGlobalMissions(globalIndexDir: string): GlobalMissionListRes
 			try {
 				const record = parseMissionRecord(JSON.parse(fs.readFileSync(entry.recordPath, "utf-8")), entry.recordPath);
 				if (record.id !== entry.missionId) throw new Error(`record id '${record.id}' does not match index id '${entry.missionId}'`);
-				entries.push({ ...entry, stale: false });
+				const projection: MissionIndexEntry = {
+					...entry,
+					title: record.title,
+					status: record.status,
+					updatedAt: record.updatedAt,
+				};
+				delete projection.lastRunId;
+				const lastRunId = record.runs.at(-1)?.runId;
+				if (lastRunId) projection.lastRunId = lastRunId;
+				entries.push({ ...projection, stale: false });
 			} catch (error) {
 				if ((error as NodeJS.ErrnoException).code === "ENOENT") {
 					try {
@@ -581,4 +590,4 @@ export function listGlobalMissions(globalIndexDir: string): GlobalMissionListRes
 	}
 	entries.sort((left, right) => right.updatedAt.localeCompare(left.updatedAt));
 	return { entries, warnings };
-}
+}

package/dist/extensions/pi-subagents/src/missions/workflow-state.ts

@@ -1,9 +1,9 @@
-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 { processStartKey } from "../shared/process-identity.ts";
 import { assertWorkflowJsonValue } from "../workflows/scripted-workflow.ts";
 import type { MissionStoreLocation } from "./types.ts";
 import { validateMissionId } from "./store.ts";
@@ -44,49 +44,6 @@ interface StateLockOwner {
 	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 {

package/dist/extensions/pi-subagents/src/policy/authority.ts

@@ -5,6 +5,8 @@ export const AUTHORITY_ACTIONS = [
 	"scheduleCreate",
 	"stopRun",
 	"steerRun",
+	"inspectorOpen",
+	"projectOpen",
 ] as const;
 
 export type AuthorityAction = typeof AUTHORITY_ACTIONS[number];
@@ -18,6 +20,8 @@ const DEFAULT_AUTHORITY_POLICY: Record<AuthorityAction, AuthorityDecision> = {
 	scheduleCreate: "auto",
 	stopRun: "auto",
 	steerRun: "auto",
+	inspectorOpen: "auto",
+	projectOpen: "confirm",
 };
 
 export function resolveAuthorityDecision(input: {
@@ -43,4 +47,4 @@ export function validateAuthorityPolicy(value: unknown, label = "config.authorit
 		}
 	}
 	return policy as AuthorityPolicyConfig;
-}
+}

package/dist/extensions/pi-subagents/src/profiles/profiles.ts

@@ -2,9 +2,10 @@ import * as fs from "node:fs";
 import * as os from "node:os";
 import * as path from "node:path";
 import type { ExtensionAPI, ExtensionContext } from "@selesai/code";
-import { BUILTIN_AGENT_NAMES } from "../agents/agents.ts";
+import { BUILTIN_AGENT_NAMES, validateOptionalMachine } from "../agents/agents.ts";
 import { getPiSpawnCommand } from "../runs/shared/pi-spawn.ts";
 import { findModelInfo, getSupportedThinkingLevels, splitKnownThinkingSuffix, toModelInfo } from "../shared/model-info.ts";
+import { withSettingsFileLease } from "../shared/settings-file-lease.ts";
 import { getAgentDir } from "../shared/utils.ts";
 
 export const DEFAULT_PROVIDER_MODELS_MAX_AGE_DAYS = 7;
@@ -20,7 +21,7 @@ export type RecommendedRoleTier = "cheap" | "medium" | "strong";
 interface ProfileAgentOverride {
 	model?: string;
 	thinking?: string | false;
-	fallbackModels?: string[] | false;
+	machine?: string | false;
 }
 
 export interface SubagentProfileFile {
@@ -144,10 +145,10 @@ function validateSubagentProfile(filePath: string, parsed: Record<string, unknow
 		if (thinking !== undefined && thinking !== false && typeof thinking !== "string") {
 			throw new Error(`Profile '${filePath}' has invalid thinking for '${name}'; expected a string or false.`);
 		}
-		const fallbackModels = override.fallbackModels;
-		if (fallbackModels !== undefined && fallbackModels !== false && (!Array.isArray(fallbackModels) || fallbackModels.some((item) => typeof item !== "string"))) {
-			throw new Error(`Profile '${filePath}' has invalid fallbackModels for '${name}'; expected an array of strings or false.`);
+		if (override.machine !== undefined && override.machine !== false) {
+			override.machine = validateOptionalMachine(override.machine, `Profile '${filePath}' has invalid machine for '${name}'`);
 		}
+		if ((override as Record<string, unknown>).fallbackModels !== undefined) throw new Error(`Profile '${filePath}' uses removed field fallbackModels for '${name}'; configure one model instead.`);
 	}
 	const disableBuiltins = (subagents as Record<string, unknown>).disableBuiltins;
 	if (disableBuiltins !== undefined && typeof disableBuiltins !== "boolean") {
@@ -483,18 +484,29 @@ export function readSubagentProfile(name: string): { filePath: string; profile:
 export function applySubagentProfile(name: string): { filePath: string; settingsPath: string } {
 	const { filePath, profile } = readSubagentProfile(name);
 	const settingsPath = getUserSettingsPath();
-	const settings = readSettingsFile(settingsPath);
-	const existing = settings.subagents && typeof settings.subagents === "object" && !Array.isArray(settings.subagents)
-		? settings.subagents as Record<string, unknown>
-		: {};
-	// A profile owns the complete agent mapping, but unrelated subagent settings
-	// (notably disableBuiltins, modelScope, watchdog, etc.) survive profile switches.
-	settings.subagents = {
-		...existing,
-		...profile.subagents,
-		agentOverrides: profile.subagents.agentOverrides,
-	};
-	writeJsonFile(settingsPath, settings);
+	withSettingsFileLease(settingsPath, () => {
+		const settings = readSettingsFile(settingsPath);
+		const existing = settings.subagents && typeof settings.subagents === "object" && !Array.isArray(settings.subagents)
+			? settings.subagents as Record<string, unknown>
+			: {};
+		// A profile owns the complete agent mapping, but unrelated subagent settings
+		// (notably disableBuiltins, modelScope, watchdog, etc.) survive profile switches.
+		// Machine placement is not a model choice, so an existing pin survives a profile switch too.
+		const agentOverrides: Record<string, ProfileAgentOverride> = { ...profile.subagents.agentOverrides };
+		const existingOverrides = existing.agentOverrides && typeof existing.agentOverrides === "object" && !Array.isArray(existing.agentOverrides)
+			? existing.agentOverrides as Record<string, unknown>
+			: {};
+		for (const [name, value] of Object.entries(existingOverrides)) {
+			const machine = value && typeof value === "object" && !Array.isArray(value) ? (value as Record<string, unknown>).machine : undefined;
+			if (typeof machine === "string" && agentOverrides[name]?.machine === undefined) agentOverrides[name] = { ...agentOverrides[name], machine };
+		}
+		settings.subagents = {
+			...existing,
+			...profile.subagents,
+			agentOverrides,
+		};
+		writeJsonFile(settingsPath, settings);
+	});
 	return { filePath, settingsPath };
 }
 
@@ -659,4 +671,4 @@ export async function checkSubagentProfile(
 		});
 	}
 	return { profileName: name, filePath, results };
-}
+}