@iowarp/clio-coder 0.4.5 → 0.4.7

package/src/interactive/leader-key.ts

@@ -1,139 +1,86 @@
-import type { ClioKeybinding } from "../domains/config/keybindings.js";
-import { isKeyRelease, type KeyId, matchesKey } from "../engine/tui.js";
+import { decodePrintableKey, isKeyRelease, isKeyRepeat, type Keybinding, matchesKey } from "../engine/tui.js";
 
 export interface LeaderTarget {
 	key: string;
-	id: ClioKeybinding;
+	id: Keybinding;
+	label?: string;
+	disabledReason?: string;
 }
-
-export type LeaderKeyState = { status: "idle" } | { status: "pending"; expiresAt: number };
-
+export type LeaderKeyState = { status: "idle" } | { status: "pending"; selected: number; notice?: string };
+export const IDLE_LEADER_STATE: LeaderKeyState = { status: "idle" };
 export interface LeaderKeyRouteDeps {
-	matchesLeader: (data: string) => boolean;
+	matchesLeader(data: string): boolean;
 	leaderTargets: ReadonlyArray<LeaderTarget>;
-	dispatchAction: (id: ClioKeybinding) => boolean;
-	now: number;
-	timeoutMs?: number;
+	dispatchAction(id: Keybinding): boolean;
 	isRelease?: (data: string) => boolean;
 }
-
 export interface LeaderKeyRouteResult {
 	state: LeaderKeyState;
 	consumed: boolean;
 }
 
-export const LEADER_TIMEOUT_MS = 1500;
-export const IDLE_LEADER_STATE: LeaderKeyState = { status: "idle" };
-
-function baseLetterFromInput(data: string): string | null {
-	if (data.length === 1) {
-		const lower = data.toLowerCase();
-		return lower >= "a" && lower <= "z" ? lower : null;
-	}
-	for (let code = 97; code <= 122; code += 1) {
-		const key = String.fromCharCode(code) as KeyId;
-		if (matchesKey(data, key) || matchesKey(data, `shift+${key}` as KeyId)) return key;
-	}
-	return null;
-}
-
-function isEscapeKey(data: string): boolean {
-	return matchesKey(data, "escape") && !isKeyRelease(data);
-}
-
-/** Pure leader-key router: returns the next leader state and whether input was swallowed. */
 export function routeLeaderKey(data: string, state: LeaderKeyState, deps: LeaderKeyRouteDeps): LeaderKeyRouteResult {
-	const timeoutMs = deps.timeoutMs ?? LEADER_TIMEOUT_MS;
-	if (state.status === "pending") {
-		if (deps.now > state.expiresAt) return { state: IDLE_LEADER_STATE, consumed: true };
-		if (deps.isRelease?.(data) ?? false) return { state, consumed: true };
-		if (isEscapeKey(data)) return { state: IDLE_LEADER_STATE, consumed: true };
-		const base = baseLetterFromInput(data);
-		const target = base ? deps.leaderTargets.find((entry) => entry.key === base) : undefined;
-		if (target) {
-			deps.dispatchAction(target.id);
-			return { state: IDLE_LEADER_STATE, consumed: true };
-		}
-		return { state: IDLE_LEADER_STATE, consumed: true };
+	if (isKeyRelease(data)) return { state, consumed: state.status === "pending" };
+	if (matchesKey(data, "ctrl+c")) return { state: IDLE_LEADER_STATE, consumed: false };
+	if (state.status === "idle") {
+		return deps.matchesLeader(data) && !isKeyRepeat(data)
+			? { state: { status: "pending", selected: 0 }, consumed: true }
+			: { state, consumed: false };
 	}
-	if (deps.isRelease?.(data) ?? false) return { state, consumed: false };
-	if (!deps.matchesLeader(data)) return { state, consumed: false };
-	return { state: { status: "pending", expiresAt: deps.now + timeoutMs }, consumed: true };
-}
-
-export interface LeaderKeyTimeout {
-	unref?(): void;
+	if (isKeyRepeat(data) && !matchesKey(data, "up") && !matchesKey(data, "down")) return { state, consumed: true };
+	if (matchesKey(data, "escape") || deps.matchesLeader(data)) return { state: IDLE_LEADER_STATE, consumed: true };
+	const count = deps.leaderTargets.length;
+	if (matchesKey(data, "up") || matchesKey(data, "down")) {
+		const delta = matchesKey(data, "up") ? -1 : 1;
+		return {
+			state: { status: "pending", selected: count ? (state.selected + delta + count) % count : 0 },
+			consumed: true,
+		};
+	}
+	const key = data.includes("\x1b[200~")
+		? null
+		: (decodePrintableKey(data) ?? (data.length === 1 ? data : undefined))?.toLowerCase();
+	const target = matchesKey(data, "enter")
+		? deps.leaderTargets[state.selected]
+		: deps.leaderTargets.find(
+				(entry) =>
+					entry.key &&
+					(entry.key === key ||
+						(entry.key === "home" && matchesKey(data, "home")) ||
+						(entry.key === "end" && matchesKey(data, "end"))),
+			);
+	if (!target) return { state: { ...state, notice: "No action for this key" }, consumed: true };
+	if (target.disabledReason) return { state: { ...state, notice: target.disabledReason }, consumed: true };
+	deps.dispatchAction(target.id);
+	return { state: IDLE_LEADER_STATE, consumed: true };
 }
 
 export interface LeaderKeyControllerDeps {
-	matchesLeader: (data: string) => boolean;
-	leaderTargets: () => ReadonlyArray<LeaderTarget>;
-	dispatchAction: (id: ClioKeybinding) => boolean;
+	matchesLeader(data: string): boolean;
+	leaderTargets(): ReadonlyArray<LeaderTarget>;
+	dispatchAction(id: Keybinding): boolean;
 	isRelease: (data: string) => boolean;
-	now?: () => number;
-	timeoutMs?: number;
-	scheduleTimeout?: (callback: () => void, delayMs: number) => LeaderKeyTimeout;
-	clearScheduledTimeout?: (timeout: LeaderKeyTimeout) => void;
-	/**
-	 * Fired when the leader arms or disarms, including on timeout.
-	 *
-	 * Ctrl+G is the fallback for terminals where Alt is broken, and the frame
-	 * between it and the next key was identical to the idle frame: nothing said
-	 * the leader had taken, on the one surface whose users cannot test it with a
-	 * working Alt shortcut.
-	 */
 	onStateChange?: (pending: boolean) => void;
+	onMenuChange?: (state: LeaderKeyState, targets: ReadonlyArray<LeaderTarget>) => void;
 }
-
 export interface LeaderKeyController {
 	isPending(): boolean;
 	route(data: string): boolean;
 	reset(): void;
 	dispose(): void;
 }
-
 export function createLeaderKeyController(deps: LeaderKeyControllerDeps): LeaderKeyController {
-	const now = deps.now ?? Date.now;
-	const scheduleTimeout = deps.scheduleTimeout ?? ((callback, delayMs) => setTimeout(callback, delayMs));
-	const clearScheduledTimeout =
-		deps.clearScheduledTimeout ?? ((timeout: LeaderKeyTimeout) => clearTimeout(timeout as ReturnType<typeof setTimeout>));
 	let state: LeaderKeyState = IDLE_LEADER_STATE;
-	let timer: LeaderKeyTimeout | null = null;
-
 	const setState = (next: LeaderKeyState): void => {
-		const wasPending = state.status === "pending";
+		const changed = state.status !== next.status;
 		state = next;
-		if (timer) {
-			clearScheduledTimeout(timer);
-			timer = null;
-		}
-		if (next.status !== "pending") {
-			if (wasPending) deps.onStateChange?.(false);
-			return;
-		}
-		if (!wasPending) deps.onStateChange?.(true);
-		timer = scheduleTimeout(
-			() => {
-				state = IDLE_LEADER_STATE;
-				timer = null;
-				deps.onStateChange?.(false);
-			},
-			Math.max(0, next.expiresAt - now()),
-		);
-		timer.unref?.();
+		deps.onMenuChange?.(state, deps.leaderTargets());
+		if (changed) deps.onStateChange?.(state.status === "pending");
 	};
-
 	return {
 		isPending: () => state.status === "pending",
-		route: (data) => {
-			const result = routeLeaderKey(data, state, {
-				matchesLeader: deps.matchesLeader,
-				leaderTargets: deps.leaderTargets(),
-				dispatchAction: deps.dispatchAction,
-				now: now(),
-				...(deps.timeoutMs === undefined ? {} : { timeoutMs: deps.timeoutMs }),
-				isRelease: deps.isRelease,
-			});
+		route(data) {
+			const result = routeLeaderKey(data, state, { ...deps, leaderTargets: deps.leaderTargets() });
 			if (result.state !== state) setState(result.state);
 			return result.consumed;
 		},

package/src/interactive/leader-menu.ts

@@ -0,0 +1,45 @@
+import { type Component, type OverlayHandle, type TUI, truncateToWidth } from "../engine/tui.js";
+import type { LeaderKeyState, LeaderTarget } from "./leader-key.js";
+import { showClioOverlayFrame } from "./overlay-frame.js";
+import { clioTheme } from "./theme/index.js";
+
+/** Noncapturing presentation: controller retains the underlying cancellation owner. */
+export function createLeaderMenu(tui: TUI, scope: () => string, keyLabel: (id: LeaderTarget["id"]) => string) {
+	let handle: OverlayHandle | null = null;
+	let state: LeaderKeyState = { status: "idle" };
+	let targets: ReadonlyArray<LeaderTarget> = [];
+	const component: Component = {
+		render(width) {
+			if (state.status === "idle") return [];
+			const selected = state.selected;
+			const count = Math.max(1, Math.min(10, tui.terminal.rows - 8));
+			const start = Math.max(0, Math.min(selected - Math.floor(count / 2), targets.length - count));
+			return [
+				`${scope()} · letters select actions · close, then /help for commands`,
+				...targets.slice(start, start + count).map((entry, index) => {
+					const text = `${start + index === selected ? "›" : " "} ${entry.key || "·"}  ${entry.label ?? entry.id}  ${keyLabel(entry.id)}${entry.disabledReason ? ` (${entry.disabledReason})` : ""}`;
+					return truncateToWidth(start + index === selected ? clioTheme().fg("accent", text) : text, width);
+				}),
+				`${targets.length ? `${selected + 1}/${targets.length}` : "No actions in this scope"} · ${state.notice ?? "Up/Down select · Enter run · Esc close · Ctrl+C cancel"}`,
+			].map((line) => truncateToWidth(line, width));
+		},
+		invalidate() {},
+	};
+	return (next: LeaderKeyState, entries: ReadonlyArray<LeaderTarget>): void => {
+		state = next;
+		targets = entries;
+		if (state.status === "idle") {
+			handle?.hide();
+			handle = null;
+		} else if (!handle)
+			handle = showClioOverlayFrame(tui, component, {
+				title: "Actions",
+				markerId: "keyboard-actions",
+				anchor: "top-center",
+				width: 88,
+				nonCapturing: true,
+				footerHint: () => `${keyLabel("clio-coder.leader")} or Esc closes this menu`,
+			});
+		tui.requestRender();
+	};
+}

package/src/interactive/overlay-ask-user-lifecycle.ts

@@ -1,5 +1,10 @@
 import type { TUI } from "../engine/tui.js";
-import { type AskUserHandler, cancelledAskUserResult } from "../tools/ask-user.js";
+import {
+	type AskUserAnswer,
+	type AskUserHandler,
+	type AskUserQuestion,
+	cancelledAskUserResult,
+} from "../tools/ask-user.js";
 import type { OverlayState } from "./overlay-key-routing.js";
 import { type AskUserOverlaySession, openAskUserOverlay } from "./overlays/ask-user.js";
 
@@ -19,6 +24,8 @@ export interface OverlayAskUserLifecycleDeps {
 	 * overlay session opens, not per question, so one interview notifies once.
 	 */
 	onOperatorParked?(): void;
+	/** A round the operator answered, for the transcript record. Not fired on cancel. */
+	onRoundAnswered?(questions: ReadonlyArray<AskUserQuestion>, answers: ReadonlyArray<AskUserAnswer>): void;
 }
 
 export interface OverlayAskUserLifecycle {
@@ -84,6 +91,7 @@ export function createOverlayAskUserLifecycle(deps: OverlayAskUserLifecycleDeps)
 		if (!activeSession) return cancelledAskUserResult();
 		pendingCancel = cancel;
 		const result = await activeSession.ask(questions, invokeOptions?.decisionPresentation);
+		if (result.cancelled !== true && result.answers.length > 0) deps.onRoundAnswered?.(questions, result.answers);
 		if (result.cancelled === true || !toolBacked) {
 			if (result.cancelled === true) cancelledForTurn = true;
 			close();

package/src/interactive/overlay-frame.ts

@@ -11,6 +11,7 @@ import {
 	truncateToWidth,
 	visibleWidth,
 } from "../engine/tui.js";
+import { keyboardOwner } from "./keyboard-owner.js";
 import { enterModal, type ModalMarkerSink } from "./modal-marker.js";
 import { type ClioToken, clioTheme, padAnsi, screenTitle, selectListTheme, settingsListTheme } from "./theme/index.js";
 
@@ -43,6 +44,13 @@ export class FocusBox extends Box {
 		this.onInput = options?.onInput;
 	}
 
+	get keyboardScope() {
+		return keyboardOwner(this.inputTarget).keyboardScope;
+	}
+	undoInput(): boolean {
+		return keyboardOwner(this.inputTarget).undoInput?.() ?? false;
+	}
+
 	handleInput(data: string): void {
 		if (this.onInput) {
 			this.onInput(data);
@@ -427,6 +435,13 @@ export class ClioOverlayFrame implements Component {
 		return lines;
 	}
 
+	get keyboardScope() {
+		return keyboardOwner(this.child).keyboardScope;
+	}
+	undoInput(): boolean {
+		return keyboardOwner(this.child).undoInput?.() ?? false;
+	}
+
 	handleInput(data: string): void {
 		this.child.handleInput?.(data);
 	}

package/src/interactive/overlay-general-openers.ts

@@ -47,6 +47,7 @@ import { type ArtifactProviderDeps, createDefaultArtifactProviders } from "./vie
 import { openViewOverlay } from "./view/view-overlay.js";
 
 export interface OverlayGeneralOpenersDeps {
+	readTranscript?: ArtifactProviderDeps["readTranscript"];
 	tui: TUI;
 	transitions: OverlayTransitions;
 	observability: ObservabilityContract;
@@ -319,6 +320,7 @@ export function createOverlayGeneralOpeners(deps: OverlayGeneralOpenersDeps): Ov
 				dispatch: deps.dispatch,
 				sessionMeta: deps.getSessionMeta(),
 				readSessionEntries: deps.readSessionEntries,
+				readTranscript: deps.readTranscript,
 			}),
 			...(initialFilter ? { initialFilter } : {}),
 			notice: deps.notify,

package/src/interactive/overlay-key-routing.ts

@@ -1,6 +1,6 @@
 import type { ClioKeybinding } from "../domains/config/keybindings.js";
-import { isKeyRelease, matchesKey } from "../engine/tui.js";
-import { MUTATION_PREVIEW_KEY } from "./permission-hint.js";
+import { isKeyRelease, isKeyRepeat, matchesKey } from "../engine/tui.js";
+import { MUTATION_PREVIEW_KEY, PERMISSION_TERMS_KEY } from "./permission-hint.js";
 
 export type OverlayState =
 	| "closed"
@@ -23,8 +23,6 @@ export type OverlayState =
 	| "cwd-fallback"
 	| "ask-user"
 	| "help"
-	| "agents"
-	| "prompts"
 	| "extensions"
 	| "interop"
 	| "skills-hub"
@@ -48,12 +46,14 @@ export interface PermissionOverlayKeyDeps {
 	 * so the draft that makes Enter inert can be cleared without first denying
 	 * the call. Only the deletion keys in `isDraftEditKey` arrive here.
 	 */
-	editDraft?: (data: string) => void;
+	editDraft?: (operation: DraftEditOperation) => void;
 	/** Whether this card has a mutation the operator can read locally (issue #254). */
 	canInspectMutation?: () => boolean;
 	isInspectingMutation?: () => boolean;
 	toggleMutationInspection?: () => void;
 	scrollMutationInspection?: (delta: number) => void;
+	/** Fold or unfold the standing approval terms on the card. */
+	togglePermissionTerms?: () => void;
 }
 
 export interface DispatchBoardOverlayKeyDeps {
@@ -80,24 +80,29 @@ export interface AskUserOverlayKeyDeps {
 	cancelAskUser: () => void;
 }
 
-export interface OverlayKeyDeps extends PermissionOverlayKeyDeps, DispatchBoardOverlayKeyDeps, AskUserOverlayKeyDeps {}
+export interface OverlayKeyDeps extends PermissionOverlayKeyDeps, DispatchBoardOverlayKeyDeps, AskUserOverlayKeyDeps {
+	canToggleOwner?: () => boolean;
+}
 
 export function isEscapeKey(data: string): boolean {
 	return matchesKey(data, "escape") && !isKeyRelease(data);
 }
 
-const DRAFT_EDIT_KEYS = ["backspace", "delete", "ctrl+u", "ctrl+w", "alt+backspace", "ctrl+k"] as const;
-
-/**
- * The keys that only ever remove text from the composer. They are the one
- * class of input a pending permission prompt lets through, because they
- * cannot type a command, cannot submit, and are the way out of the inert-Enter
- * state a draft puts the prompt in.
- */
-function isDraftEditKey(data: string): boolean {
-	return !isKeyRelease(data) && DRAFT_EDIT_KEYS.some((key) => matchesKey(data, key));
-}
-
+export type DraftEditOperation =
+	| "deleteCharBackward"
+	| "deleteCharForward"
+	| "deleteToLineStart"
+	| "deleteToLineEnd"
+	| "deleteWordBackward";
+const DRAFT_EDITS: ReadonlyArray<readonly [import("../engine/tui.js").KeyId, DraftEditOperation]> = [
+	["backspace", "deleteCharBackward"],
+	["delete", "deleteCharForward"],
+	["ctrl+u", "deleteToLineStart"],
+	["ctrl+k", "deleteToLineEnd"],
+	["ctrl+w", "deleteWordBackward"],
+	["alt+backspace", "deleteWordBackward"],
+	["ctrl+backspace", "deleteWordBackward"],
+];
 /** Rows one page key moves the open mutation. Matches the window the card renders. */
 const MUTATION_PAGE_ROWS = 16;
 
@@ -119,8 +124,9 @@ function routePermissionOverlayKey(data: string, deps: PermissionOverlayKeyDeps)
 		if (!(deps.composerHasDraft?.() ?? false)) deps.confirmPermission();
 		return true;
 	}
-	if (deps.editDraft && isDraftEditKey(data)) {
-		deps.editDraft(data);
+	const edit = DRAFT_EDITS.find(([key]) => matchesKey(data, key));
+	if (deps.editDraft && edit) {
+		deps.editDraft(edit[1]);
 		return true;
 	}
 	// Reading the mutation is the one thing this dialog does that is not an
@@ -136,6 +142,14 @@ function routePermissionOverlayKey(data: string, deps: PermissionOverlayKeyDeps)
 		deps.toggleMutationInspection();
 		return true;
 	}
+	if (
+		matchesKey(data, PERMISSION_TERMS_KEY) &&
+		deps.togglePermissionTerms &&
+		!(deps.isInspectingMutation?.() ?? false)
+	) {
+		deps.togglePermissionTerms();
+		return true;
+	}
 	if ((deps.isInspectingMutation?.() ?? false) && deps.scrollMutationInspection && !isKeyRelease(data)) {
 		const delta = mutationScrollDelta(data);
 		if (delta !== 0) {
@@ -208,15 +222,6 @@ function routeCostOverlayKey(data: string, deps: CloseOverlayKeyDeps): boolean {
 	return false;
 }
 
-/** Pure overlay key router for the /model overlay. Esc closes; arrows and Enter fall through. */
-function routeModelOverlayKey(data: string, deps: CloseOverlayKeyDeps): boolean {
-	if (isEscapeKey(data)) {
-		deps.closeOverlay();
-		return true;
-	}
-	return false;
-}
-
 /** Pure overlay key router for the /fork message-picker. */
 function routeMessagePickerOverlayKey(data: string, deps: CloseOverlayKeyDeps): boolean {
 	if (isEscapeKey(data)) {
@@ -226,15 +231,6 @@ function routeMessagePickerOverlayKey(data: string, deps: CloseOverlayKeyDeps):
 	return false;
 }
 
-/** Pure overlay key router for ask_user. */
-function routeAskUserOverlayKey(data: string, deps: AskUserOverlayKeyDeps): boolean {
-	if (isEscapeKey(data)) {
-		deps.cancelAskUser();
-		return true;
-	}
-	return false;
-}
-
 export function overlayOwnsInput(overlayState: OverlayState): boolean {
 	return overlayState !== "closed";
 }
@@ -246,12 +242,17 @@ export function routeOverlayKey(
 	matches: (data: string, id: ClioKeybinding) => boolean,
 ): boolean {
 	if (overlayState === "closed") return false;
+	if (isKeyRelease(data)) return true;
+	if (data.includes("\x1b[200~") && overlayState === "permission-confirm") return true;
 	if (
-		(overlayState === "dispatch-board" && matches(data, "clio-coder.dispatchBoard.toggle")) ||
-		(overlayState === "tasks" && matches(data, "clio-coder.tasks.open")) ||
-		(overlayState === "tree" && matches(data, "clio-coder.session.tree")) ||
-		(overlayState === "model" && matches(data, "clio-coder.model.select")) ||
-		(overlayState === "help" && matches(data, "clio-coder.leader"))
+		!isKeyRepeat(data) &&
+		(deps.canToggleOwner?.() ?? true) &&
+		((overlayState === "dispatch-board" && matches(data, "clio-coder.dispatchBoard.toggle")) ||
+			(overlayState === "tasks" && matches(data, "clio-coder.tasks.open")) ||
+			(overlayState === "tree" && matches(data, "clio-coder.session.tree")) ||
+			(overlayState === "model" && matches(data, "clio-coder.model.select")) ||
+			(overlayState === "skills-hub" && matches(data, "clio-coder.library.toggle")) ||
+			false)
 	) {
 		deps.closeOverlay();
 		return true;
@@ -287,7 +288,7 @@ export function routeOverlayKey(
 	if (overlayState === "decisions") return false;
 	if (overlayState === "memory") return false;
 	if (overlayState === "view") return false;
-	if (overlayState === "model") return routeModelOverlayKey(data, deps);
+	if (overlayState === "model") return false;
 	// The scope dialog owns Esc itself: cancelling it must leave the model where
 	// it was, and a router-level close cannot say that to the caller waiting on it.
 	if (overlayState === "model-scope") return false;
@@ -299,11 +300,9 @@ export function routeOverlayKey(
 	if (overlayState === "tree") return false;
 	if (overlayState === "message-picker") return routeMessagePickerOverlayKey(data, deps);
 	if (overlayState === "cwd-fallback") return false;
-	if (overlayState === "ask-user") return routeAskUserOverlayKey(data, deps);
+	if (overlayState === "ask-user") return false;
 	if (
 		overlayState === "help" ||
-		overlayState === "agents" ||
-		overlayState === "prompts" ||
 		overlayState === "extensions" ||
 		overlayState === "interop" ||
 		overlayState === "skills-hub"

package/src/interactive/overlay-lifecycle.ts

@@ -1,12 +1,12 @@
 import type { LibraryEntryKind } from "../domains/resources/index.js";
-import { appendNotice } from "./command-output.js";
+import { appendInterviewRecord, appendNotice } from "./command-output.js";
 import { createOverlayAskUserLifecycle, type OverlayAskUserLifecycle } from "./overlay-ask-user-lifecycle.js";
 import { createOverlayAuthLifecycle } from "./overlay-auth-lifecycle.js";
 import { showClioOverlayFrame } from "./overlay-frame.js";
 import { createOverlayGeneralOpeners } from "./overlay-general-openers.js";
 import { createOverlayModelSelectors } from "./overlay-model-selectors.js";
 import { createOverlayPermissionLifecycle, type OverlayPermissionLifecycle } from "./overlay-permission-lifecycle.js";
-import { createOverlayResourceOpeners } from "./overlay-resource-openers.js";
+import { createOverlayResourceOpeners, type LibraryOpenRequest } from "./overlay-resource-openers.js";
 import { createOverlaySessionLifecycle } from "./overlay-session-lifecycle.js";
 import { createOverlayTransitions } from "./overlay-transitions.js";
 import {
@@ -124,14 +124,13 @@ export interface OverlayLifecycleRuntimeDeps {
 	openMemoryOverlay?: typeof import("./memory-overlay.js").openMemoryOverlay;
 	openViewOverlay?: typeof import("./view/view-overlay.js").openViewOverlay;
 	openHelpOverlay?: typeof import("./overlays/help-reference.js").openHelpOverlay;
-	openAgentsOverlay?: typeof import("./overlays/agents.js").openAgentsOverlay;
-	openSkillsHub?: typeof import("./overlays/skills-hub.js").openSkillsHub;
-	openPromptsOverlay?: typeof import("./overlays/prompts.js").openPromptsOverlay;
+	openSkillsHub?: typeof import("./overlays/library.js").openLibraryOverlay;
 	openExtensionsOverlay?: typeof import("./overlays/extensions.js").openExtensionsOverlay;
 	openInteropOverlay?: typeof import("./overlays/interop.js").openInteropOverlay;
 }
 
 export interface OverlayLifecycleController {
+	openExtensionPanelState: import("./overlay-resource-openers.js").OverlayResourceOpeners["openExtensionPanelState"];
 	getState(): OverlayState;
 	closeOverlay(): void;
 	finishAuthOverlay(dismiss: boolean): void;
@@ -161,9 +160,7 @@ export interface OverlayLifecycleController {
 	openTreeOverlayState(): void;
 	openMessagePickerOverlayState(): void;
 	openHelpOverlayState(query?: string): void;
-	openAgentsOverlayState(): void;
-	openSkillsHubState(tab?: LibraryEntryKind): void;
-	openPromptsOverlayState(): void;
+	openSkillsHubState(request?: LibraryOpenRequest | LibraryEntryKind): void;
 	openExtensionsOverlayState(): void;
 	openInteropOverlayState(): void;
 	toggleDispatchBoardOverlay(): void;
@@ -175,6 +172,8 @@ export interface OverlayLifecycleController {
 	isInspectingMutation(): boolean;
 	toggleMutationInspection(): void;
 	scrollMutationInspection(delta: number): void;
+	/** Fold or unfold the standing approval terms on the live permission card. */
+	togglePermissionTerms(): void;
 	cancelAskUser(): void;
 	dispose(): void;
 }
@@ -215,9 +214,7 @@ export function createOverlayLifecycle(deps: OverlayLifecycleRuntimeDeps): Overl
 		openMemoryOverlay: openMemoryOverlayFactory,
 		openViewOverlay: openViewOverlayFactory,
 		openHelpOverlay: openHelpOverlayFactory,
-		openAgentsOverlay: openAgentsOverlayFactory,
 		openSkillsHub: openSkillsHubFactory,
-		openPromptsOverlay: openPromptsOverlayFactory,
 		openExtensionsOverlay: openExtensionsOverlayFactory,
 		openInteropOverlay: openInteropOverlayFactory,
 	} = deps;
@@ -289,7 +286,13 @@ export function createOverlayLifecycle(deps: OverlayLifecycleRuntimeDeps): Overl
 				// Read per frame: the footer names what Enter does right now, and
 				// that depends on whether the composer holds a draft and on whether
 				// the mutation is open.
-				footerHint: (innerWidth) => permissionOverlayHint(innerWidth, editor.getText().length > 0, inspectionHint()),
+				footerHint: (innerWidth) =>
+					permissionOverlayHint(
+						innerWidth,
+						editor.getText().length > 0,
+						inspectionHint(),
+						body.isShowingTerms() ? "open" : "closed",
+					),
 			});
 			if (!overlayTransitions.showPermission(handle)) {
 				handle.hide();
@@ -330,6 +333,17 @@ export function createOverlayLifecycle(deps: OverlayLifecycleRuntimeDeps): Overl
 		...(deps.app.registerAskUserHandler ? { registerHandler: deps.app.registerAskUserHandler } : {}),
 		...(openAskUserOverlayFactory ? { openAskUserOverlay: openAskUserOverlayFactory } : {}),
 		...(deps.onOperatorParked ? { onOperatorParked: deps.onOperatorParked } : {}),
+		onRoundAnswered: (questions, answers) =>
+			appendInterviewRecord(
+				answers.map((answer) => ({
+					label:
+						questions.find((question) => question.question === answer.question)?.header ??
+						answer.question.split("\n").find((line) => line.trim().length > 0) ??
+						answer.question,
+					answer: answer.answer,
+				})),
+				busNoticeSink,
+			),
 	});
 
 	const overlayModelSelectors = createOverlayModelSelectors({
@@ -362,9 +376,7 @@ export function createOverlayLifecycle(deps: OverlayLifecycleRuntimeDeps): Overl
 		...(deps.app.resources ? { resources: deps.app.resources } : {}),
 		closeOverlay,
 		...(openHelpOverlayFactory ? { openHelpOverlay: openHelpOverlayFactory } : {}),
-		...(openAgentsOverlayFactory ? { openAgentsOverlay: openAgentsOverlayFactory } : {}),
 		...(openSkillsHubFactory ? { openSkillsHub: openSkillsHubFactory } : {}),
-		...(openPromptsOverlayFactory ? { openPromptsOverlay: openPromptsOverlayFactory } : {}),
 		...(openExtensionsOverlayFactory ? { openExtensionsOverlay: openExtensionsOverlayFactory } : {}),
 		...(openInteropOverlayFactory ? { openInteropOverlay: openInteropOverlayFactory } : {}),
 	});
@@ -399,6 +411,7 @@ export function createOverlayLifecycle(deps: OverlayLifecycleRuntimeDeps): Overl
 
 	const scheduling = deps.app.scheduling;
 	const overlayGeneralOpeners = createOverlayGeneralOpeners({
+		readTranscript: () => chatPanel.inspectionArtifacts(),
 		tui,
 		transitions: overlayTransitions,
 		observability: deps.app.observability,
@@ -487,10 +500,9 @@ export function createOverlayLifecycle(deps: OverlayLifecycleRuntimeDeps): Overl
 		openTreeOverlayState,
 		openMessagePickerOverlayState,
 		openHelpOverlayState: overlayResourceOpeners.openHelpOverlayState,
-		openAgentsOverlayState: overlayResourceOpeners.openAgentsOverlayState,
 		openSkillsHubState: overlayResourceOpeners.openSkillsHubState,
-		openPromptsOverlayState: overlayResourceOpeners.openPromptsOverlayState,
 		openExtensionsOverlayState: overlayResourceOpeners.openExtensionsOverlayState,
+		openExtensionPanelState: overlayResourceOpeners.openExtensionPanelState,
 		openInteropOverlayState: overlayResourceOpeners.openInteropOverlayState,
 		toggleDispatchBoardOverlay,
 		confirmPermission: () => {
@@ -513,6 +525,10 @@ export function createOverlayLifecycle(deps: OverlayLifecycleRuntimeDeps): Overl
 			permissionBody?.scrollInspect(delta);
 			tui.requestRender();
 		},
+		togglePermissionTerms: () => {
+			permissionBody?.toggleTerms();
+			tui.requestRender();
+		},
 		cancelAskUser: overlayAskUser.cancel,
 		dispose: () => {
 			overlayPermission?.dispose();