@iowarp/clio-coder 0.4.6 → 0.4.8

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 } : {}),
 	});
@@ -488,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: () => {
@@ -514,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();

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

@@ -1,16 +1,18 @@
+import type { ExtensionPanel } from "../domains/extensions/public-api.js";
 import type { LibraryEntryKind, ResourcesContract } from "../domains/resources/index.js";
-import { installSkill as installMarketplaceSkill } from "../domains/resources/skills/marketplace.js";
 import type { TUI } from "../engine/tui.js";
 import type { ClioEditor } from "./clio-editor.js";
 import type { ClioKeybindingManager } from "./keybinding-manager.js";
 import type { OverlayTransitions } from "./overlay-transitions.js";
-import { openAgentsOverlay } from "./overlays/agents.js";
+import { openExtensionPanel } from "./overlays/extension-panel.js";
 import { openExtensionsOverlay } from "./overlays/extensions.js";
 import { openHelpOverlay } from "./overlays/help-reference.js";
 import { openInteropOverlay } from "./overlays/interop.js";
-import { openPromptsOverlay } from "./overlays/prompts.js";
-import { openSkillsHub } from "./overlays/skills-hub.js";
-import type { SlashCommandContext } from "./slash-commands.js";
+import { openLibraryOverlay } from "./overlays/library.js";
+import { createLibraryLifecycle, libraryRefreshHost } from "./overlays/library-lifecycle.js";
+import type { LibraryBrowseRequest, SlashCommandContext } from "./slash-commands.js";
+
+export type { LibraryBrowseRequest as LibraryOpenRequest } from "./slash-commands.js";
 
 export interface OverlayResourceOpenersDeps {
 	tui: TUI;
@@ -21,80 +23,90 @@ export interface OverlayResourceOpenersDeps {
 	resources?: Pick<ResourcesContract, "skills">;
 	closeOverlay: () => void;
 	openHelpOverlay?: typeof openHelpOverlay;
-	openAgentsOverlay?: typeof openAgentsOverlay;
-	openSkillsHub?: typeof openSkillsHub;
-	openPromptsOverlay?: typeof openPromptsOverlay;
+	openSkillsHub?: typeof openLibraryOverlay;
 	openExtensionsOverlay?: typeof openExtensionsOverlay;
 	openInteropOverlay?: typeof openInteropOverlay;
-	installSkill?: typeof installMarketplaceSkill;
 }
 
 export interface OverlayResourceOpeners {
+	openExtensionPanelState(owner: string, panel: ExtensionPanel, valid: () => boolean): boolean;
 	openHelpOverlayState(query?: string): void;
-	openAgentsOverlayState(): void;
-	openSkillsHubState(tab?: LibraryEntryKind): void;
-	openPromptsOverlayState(): void;
+	openSkillsHubState(request?: LibraryBrowseRequest | LibraryEntryKind): void;
 	openExtensionsOverlayState(): void;
 	openInteropOverlayState(): void;
 }
 
+/** The session's own resource reload, or nothing when this host has none wired. */
+function reloadResources(ctx: SlashCommandContext): (() => { generation: number }) | undefined {
+	const reload = ctx.reloadPlugins;
+	return reload ? () => reload() : undefined;
+}
+
 export function createOverlayResourceOpeners(deps: OverlayResourceOpenersDeps): OverlayResourceOpeners {
 	const openHelp = deps.openHelpOverlay ?? openHelpOverlay;
-	const openAgents = deps.openAgentsOverlay ?? openAgentsOverlay;
-	const openSkills = deps.openSkillsHub ?? openSkillsHub;
-	const openPrompts = deps.openPromptsOverlay ?? openPromptsOverlay;
+	const openLibrary = deps.openSkillsHub ?? openLibraryOverlay;
 	const openExtensions = deps.openExtensionsOverlay ?? openExtensionsOverlay;
 	const openInterop = deps.openInteropOverlay ?? openInteropOverlay;
-	const installSkill = deps.installSkill ?? installMarketplaceSkill;
 
 	const openHelpOverlayState = (query?: string): void => {
 		if (deps.transitions.state !== "closed") return;
 		deps.transitions.state = "help";
-		deps.transitions.handle = openHelp(deps.tui, deps.keybindings, deps.closeOverlay, query);
+		const ctx = deps.getSlashContext();
+		deps.transitions.handle = openHelp(
+			deps.tui,
+			deps.keybindings,
+			deps.closeOverlay,
+			query,
+			ctx.operatorExtensions?.commands(ctx.listPrompts().items.map((prompt) => prompt.name)),
+		);
 		deps.tui.requestRender();
 	};
 
-	const openAgentsOverlayState = (): void => {
+	const openInteropOverlayState = (): void => {
 		if (deps.transitions.state !== "closed") return;
-		deps.transitions.state = "agents";
-		deps.transitions.handle = openAgents(deps.tui, deps.getSlashContext(), deps.closeOverlay);
+		deps.transitions.state = "interop";
+		deps.transitions.handle = openInterop(deps.tui, deps.getSlashContext(), deps.closeOverlay);
 		deps.tui.requestRender();
 	};
 
-	const openSkillsHubState = (tab?: LibraryEntryKind): void => {
+	const openSkillsHubState = (request?: LibraryBrowseRequest | LibraryEntryKind): void => {
 		if (deps.transitions.state !== "closed") return;
+		const open: LibraryBrowseRequest = typeof request === "string" ? { tab: request } : (request ?? {});
 		deps.transitions.state = "skills-hub";
-		deps.transitions.handle = openSkills(deps.tui, {
-			...(tab ? { initialTab: tab } : {}),
+		const ctx = deps.getSlashContext();
+		deps.transitions.handle = openLibrary(deps.tui, {
+			...(open.tab ? { initialTab: open.tab } : {}),
+			...(open.focus ? { focus: open.focus } : {}),
+			...(open.intent ? { intent: open.intent } : {}),
+			...(open.importSource ? { importSource: open.importSource } : {}),
+			...(open.scope ? { initialScope: open.scope } : {}),
+			// The session's own resource reload is the refresh a committed change
+			// asks for. It is reported separately from the write and can be retried
+			// on its own, so a refresh failure never restates a successful install.
+			lifecycle: createLibraryLifecycle(libraryRefreshHost(reloadResources(ctx))),
 			// A fleet's `use` is its approval preview, which is a surface of its own.
-			// The hub closes first so the preview owns the overlay slot, exactly as
-			// `/fleet run <name>` typed into the composer would.
+			// The Library closes first so the preview owns the overlay slot, exactly
+			// as `/fleet run <name>` typed into the composer would.
 			openFleetRun: (name) => {
 				deps.closeOverlay();
 				deps.getSlashContext().startFleetRun?.(name, {});
 			},
-			listSkills: () => deps.resources?.skills(process.cwd()) ?? { items: [], diagnostics: [] },
+			// Discovery is explicit. Opening the Library never sweeps another
+			// agent's home; this is the only route that does, and only on `o`.
+			openImport: () => {
+				deps.closeOverlay();
+				openInteropOverlayState();
+			},
 			setEditorText: (text) => {
 				deps.editor.setText(text);
 				deps.tui.requestRender();
 			},
 			notice: (level, text) => deps.getSlashContext().notice(level, text),
-			installSkill: async (name) => {
-				const result = installSkill({ source: name, scope: "project" });
-				return { name: result.name, path: result.path, warnings: result.warnings };
-			},
 			onClose: deps.closeOverlay,
 		});
 		deps.tui.requestRender();
 	};
 
-	const openPromptsOverlayState = (): void => {
-		if (deps.transitions.state !== "closed") return;
-		deps.transitions.state = "prompts";
-		deps.transitions.handle = openPrompts(deps.tui, deps.getSlashContext(), deps.closeOverlay);
-		deps.tui.requestRender();
-	};
-
 	const openExtensionsOverlayState = (): void => {
 		if (deps.transitions.state !== "closed") return;
 		deps.transitions.state = "extensions";
@@ -102,18 +114,16 @@ export function createOverlayResourceOpeners(deps: OverlayResourceOpenersDeps):
 		deps.tui.requestRender();
 	};
 
-	const openInteropOverlayState = (): void => {
-		if (deps.transitions.state !== "closed") return;
-		deps.transitions.state = "interop";
-		deps.transitions.handle = openInterop(deps.tui, deps.getSlashContext(), deps.closeOverlay);
-		deps.tui.requestRender();
-	};
-
 	return {
+		openExtensionPanelState(owner, panel, valid) {
+			if (deps.transitions.state !== "closed") return false;
+			deps.transitions.state = "extensions";
+			deps.transitions.handle = openExtensionPanel(deps.tui, owner, panel, valid, deps.closeOverlay);
+			deps.tui.requestRender();
+			return true;
+		},
 		openHelpOverlayState,
-		openAgentsOverlayState,
 		openSkillsHubState,
-		openPromptsOverlayState,
 		openExtensionsOverlayState,
 		openInteropOverlayState,
 	};