@iowarp/clio-coder 0.3.9 → 0.4.0

package/src/interactive/overlays/library-tabs.ts

@@ -0,0 +1,28 @@
+/**
+ * The Skills Hub's tab table.
+ *
+ * It sits in its own module rather than inside `skills-hub.ts` because the slash
+ * registry parses `/library <kind>` against it, and the registry has to stay off
+ * the render module graph. Pulling the whole hub overlay in for four ids made
+ * `src/cli/run.ts` a second, disjoint reacher of the interactive theme and
+ * `src/engine/tui.ts`, which splits the merged chunk the Stage 0 instant shell
+ * depends on. See `tests/contracts/instant-shell-import-graph.test.ts`.
+ *
+ * Pure data plus its membership test: no rendering, no overlay state.
+ * `skills-hub.ts` re-exports both symbols, so every existing importer keeps the
+ * path it had.
+ */
+
+import type { LibraryEntryKind } from "../../domains/resources/index.js";
+
+/** The hub's tabs, in the order ←/→ walks them. `skill` is the original view. */
+export const LIBRARY_TABS: ReadonlyArray<{ id: LibraryEntryKind; label: string }> = [
+	{ id: "skill", label: "Skills" },
+	{ id: "agent", label: "Agents" },
+	{ id: "prompt", label: "Prompts" },
+	{ id: "fleet", label: "Fleets" },
+];
+
+export function isLibraryTab(value: string): value is LibraryEntryKind {
+	return LIBRARY_TABS.some((tab) => tab.id === value);
+}

package/src/interactive/overlays/list-overlay.ts

@@ -742,11 +742,20 @@ export interface ListOverlayHandle extends OverlayHandle {
 	activeTabId(): string;
 }
 
-export function openListOverlay(tui: TUI, options: ListOverlayOptions): ListOverlayHandle {
+/**
+ * The marker id rides on the opener rather than on `ListOverlayOptions` because
+ * it belongs to the mounted modal, not to the view: the view renders the same
+ * rows whether it is a modal or a body inside something else. It is named by
+ * the surface rather than derived from `title`, because a tabbed overlay's
+ * rendered title names the active tab while the modal holding the keyboard has
+ * not changed hands.
+ */
+export function openListOverlay(tui: TUI, options: ListOverlayOptions & { markerId: string }): ListOverlayHandle {
 	const view = new ListOverlayView(options, () => tui.requestRender());
 	const handle = showClioOverlayFrame(tui, view, {
 		anchor: "center",
 		width: 100,
+		markerId: options.markerId,
 		// A function, not the string: on a tabbed overlay the title names the tab
 		// and must be re-read after every switch.
 		title: () => view.title(),

package/src/interactive/overlays/message-picker.ts

@@ -149,6 +149,7 @@ export function openMessagePickerOverlay(tui: TUI, deps: OpenMessagePickerOverla
 	return showClioOverlayFrame(tui, box, {
 		anchor: "center",
 		width: MESSAGE_PICKER_OVERLAY_WIDTH,
+		markerId: "message-picker",
 		title: "Fork",
 		footerHint: rows.length > 0 ? buildHint([{ key: "Enter", verb: "select" }]) : buildHint([]),
 	});

package/src/interactive/overlays/model-scope.ts

@@ -0,0 +1,86 @@
+/**
+ * Where a `/model` swap lands: this session, or this session and settings.yaml.
+ *
+ * Smoke pass 2 (G3) swapped the live model mid-conversation and found the
+ * orchestrator role rewritten in settings.yaml with no prompt, so the next
+ * launch came up pointed at an endpoint that no longer existed. Every other
+ * durable change in the product goes through the settings center's scoped
+ * commit; this is that commit for the one path that used to skip it.
+ *
+ * Session is first and selected by default: a mid-conversation swap is a thing
+ * the operator is trying, and the cheap answer to a bad try is closing the
+ * session rather than editing a file.
+ */
+
+import type { ThinkingLevel } from "../../core/defaults.js";
+import { type OverlayHandle, type SelectItem, SelectList, type TUI } from "../../engine/tui.js";
+import { buildHint, DEFAULT_SELECT_THEME, FocusBox, showClioOverlayFrame } from "../overlay-frame.js";
+
+export const MODEL_SCOPE_OVERLAY_WIDTH = 72;
+
+export type ModelScopeChoice = "session" | "global";
+
+/** The swap awaiting a destination. `thinkingLevel` is set only when the operator named one. */
+export interface PendingModelScope {
+	target: string;
+	model: string;
+	thinkingLevel?: ThinkingLevel;
+}
+
+export interface OpenModelScopeOverlayDeps {
+	ref: PendingModelScope;
+	/** Called only for one of the two applying choices. */
+	onChoose: (scope: ModelScopeChoice) => void;
+	/** Called for the explicit Cancel row and for Esc. Nothing has changed yet. */
+	onCancel: () => void;
+}
+
+/** The swap as the operator typed it, for the dialog title. */
+export function formatPendingModelScope(ref: PendingModelScope): string {
+	const thinking = ref.thinkingLevel ? ` thinking=${ref.thinkingLevel}` : "";
+	return `${ref.target}/${ref.model}${thinking}`;
+}
+
+/** Three outcomes, least durable first, matching the settings center's order. */
+function buildModelScopeItems(): SelectItem[] {
+	return [
+		{
+			value: "session",
+			label: "Apply this session",
+			description: "route this session only; settings.yaml is untouched",
+		},
+		{
+			value: "global",
+			label: "Apply and save globally",
+			description: "also the orchestrator route the next launch starts on",
+		},
+		{
+			value: "cancel",
+			label: "Cancel",
+			description: "keep the current model",
+		},
+	];
+}
+
+export function openModelScopeOverlay(tui: TUI, deps: OpenModelScopeOverlayDeps): OverlayHandle {
+	const items = buildModelScopeItems();
+	const list = new SelectList(items, items.length, DEFAULT_SELECT_THEME);
+	list.onSelect = (item: SelectItem): void => {
+		if (item.value === "session" || item.value === "global") {
+			deps.onChoose(item.value);
+			return;
+		}
+		deps.onCancel();
+	};
+	list.onCancel = (): void => {
+		deps.onCancel();
+	};
+
+	return showClioOverlayFrame(tui, new FocusBox(list), {
+		anchor: "center",
+		width: MODEL_SCOPE_OVERLAY_WIDTH,
+		markerId: "model-scope",
+		title: `Apply ${formatPendingModelScope(deps.ref)}`,
+		footerHint: buildHint([{ key: "Enter", verb: "select" }]),
+	});
+}

package/src/interactive/overlays/model-selector.ts

@@ -1085,6 +1085,7 @@ export function openModelOverlay(tui: TUI, deps: OpenModelOverlayDeps): OverlayH
 	const handle = showClioOverlayFrame(tui, view, {
 		anchor: "center",
 		width: overlayWidth,
+		markerId: "model",
 		title: "Models",
 		footerHint: buildHint([
 			FILTER_HINT,

package/src/interactive/overlays/prompts.ts

@@ -67,6 +67,7 @@ export function openPromptsOverlay(tui: TUI, ctx: SlashCommandContext, onClose:
 	const allItems = [...items, ...diagnosticItems];
 
 	return openListOverlay(tui, {
+		markerId: "prompts",
 		title: "Prompt Templates",
 		items: allItems,
 		filterable: true,

package/src/interactive/overlays/session-selector.ts

@@ -271,6 +271,7 @@ export function openSessionOverlay(tui: TUI, deps: OpenSessionOverlayDeps): Over
 	const handle = showClioOverlayFrame(tui, box, {
 		anchor: "center",
 		width: SESSION_OVERLAY_WIDTH,
+		markerId: "resume",
 		title: "Sessions",
 		footerHint: buildHint([FILTER_HINT, { key: "Enter", verb: "resume" }]),
 	});

package/src/interactive/overlays/settings-sections.ts

@@ -0,0 +1,30 @@
+/**
+ * The Settings Center's section table.
+ *
+ * It sits in its own module rather than inside `settings.ts` because the slash
+ * registry parses `/settings <section>` and every `/targets`-style deep link
+ * against it, and the registry has to stay off the render module graph. Pulling
+ * the whole settings overlay in for eleven ids made `src/cli/run.ts` a second,
+ * disjoint reacher of the interactive theme and `src/engine/tui.ts`, which
+ * splits the merged chunk the Stage 0 instant shell depends on. See
+ * `tests/contracts/instant-shell-import-graph.test.ts`.
+ *
+ * Pure data: no imports, no rendering, no settings I/O. `settings.ts` re-exports
+ * both symbols, so every existing importer keeps the path it had.
+ */
+
+export const SETTINGS_SECTIONS = [
+	{ id: "safety", label: "Autonomy & Safety", group: "CORE" },
+	{ id: "orchestrator", label: "Orchestrator", group: "CORE" },
+	{ id: "fleet", label: "Fleet", group: "ROUTING" },
+	{ id: "targets", label: "Targets", group: "ROUTING" },
+	{ id: "models", label: "Models", group: "ROUTING" },
+	{ id: "budget", label: "Budget", group: "RUNTIME" },
+	{ id: "compaction", label: "Compaction", group: "RUNTIME" },
+	{ id: "retry", label: "Retry", group: "RUNTIME" },
+	{ id: "terminal", label: "Terminal", group: "EXPERIENCE" },
+	{ id: "watchdog", label: "Watchdog", group: "EXPERIENCE" },
+	{ id: "advanced", label: "Advanced", group: "EXPERIENCE" },
+] as const;
+
+export type SettingsSectionId = (typeof SETTINGS_SECTIONS)[number]["id"];