@iowarp/clio-coder 0.4.0 → 0.4.1

package/src/interactive/overlays/side-question.ts

@@ -37,7 +37,7 @@ export interface OpenSideQuestionOverlayOptions {
 const SIDE_QUESTION_OVERLAY_MIN_WIDTH = 44;
 const SIDE_QUESTION_OVERLAY_MAX_WIDTH = 100;
 
-export function sideQuestionOverlayWidth(columns: number): number {
+function sideQuestionOverlayWidth(columns: number): number {
 	return Math.max(SIDE_QUESTION_OVERLAY_MIN_WIDTH, Math.min(SIDE_QUESTION_OVERLAY_MAX_WIDTH, columns - 4));
 }
 
@@ -46,7 +46,7 @@ export function sideQuestionOverlayWidth(columns: number): number {
  * the question in dim above a rule, then the answer, then one status line that
  * says whether the round is still running, cancelled, or failed.
  */
-export function formatSideQuestionBody(
+function formatSideQuestionBody(
 	question: string,
 	phase: SideQuestionOverlayPhase,
 	width: number,

package/src/interactive/overlays/skills-hub.ts

@@ -113,7 +113,7 @@ interface LibraryTabContext {
  * approval preview instead, so this returns null for one and the caller routes
  * it there.
  */
-export function libraryUseInvocation(entry: Pick<LibraryEntry, "kind" | "name">): string | null {
+function libraryUseInvocation(entry: Pick<LibraryEntry, "kind" | "name">): string | null {
 	if (entry.kind === "fleet") return null;
 	if (entry.kind === "agent") return `/run ${entry.name} `;
 	if (entry.kind === "prompt") return `/${entry.name} `;
@@ -148,7 +148,7 @@ function skillBody(content: string): string {
 }
 
 /** @internal exported for contract tests */
-export function buildInstalledItems(list: ResourceList<Skill>): ListOverlayItem[] {
+function buildInstalledItems(list: ResourceList<Skill>): ListOverlayItem[] {
 	return list.items.map((skill) => {
 		const theme = clioTheme();
 		const flagged = list.diagnostics.some((diag) => diagnosticTouchesSkill(diag.path, skill));
@@ -182,7 +182,7 @@ export function buildInstalledItems(list: ResourceList<Skill>): ListOverlayItem[
 }
 
 /** @internal exported for contract tests */
-export function buildMarketplaceItems(
+function buildMarketplaceItems(
 	skills: ReadonlyArray<MarketplaceSkill>,
 	installed: ReadonlySet<string>,
 ): ListOverlayItem[] {
@@ -215,7 +215,7 @@ export function buildMarketplaceItems(
 }
 
 /** @internal exported for contract tests */
-export function buildDiagnosticItems(
+function buildDiagnosticItems(
 	list: ResourceList<Skill>,
 	marketplaceDiagnostics: ReadonlyArray<string> = [],
 ): ListOverlayItem[] {
@@ -252,7 +252,7 @@ export function buildDiagnosticItems(
 }
 
 /** @internal exported for contract tests */
-export function libraryRowId(entry: Pick<LibraryEntry, "kind" | "name">): string {
+function libraryRowId(entry: Pick<LibraryEntry, "kind" | "name">): string {
 	return `${LIBRARY_ROW_PREFIX}${libraryEntryRef(entry)}`;
 }
 
@@ -267,7 +267,7 @@ export function libraryRowId(entry: Pick<LibraryEntry, "kind" | "name">): string
  *
  * @internal exported for contract tests
  */
-export function buildLibraryItems(kind: LibraryEntryKind, context: LibraryTabContext): ListOverlayItem[] {
+function buildLibraryItems(kind: LibraryEntryKind, context: LibraryTabContext): ListOverlayItem[] {
 	const theme = clioTheme();
 	return context.discovery.entries
 		.filter((entry) => entry.kind === kind)
@@ -318,7 +318,7 @@ export function buildLibraryItems(kind: LibraryEntryKind, context: LibraryTabCon
  *
  * @internal exported for contract tests
  */
-export function buildLibraryDiagnosticItems(discovery: LibraryDiscoveryResult): ListOverlayItem[] {
+function buildLibraryDiagnosticItems(discovery: LibraryDiscoveryResult): ListOverlayItem[] {
 	const theme = clioTheme();
 	return discovery.diagnostics
 		.filter((message) => message !== MARKETPLACE_UNCONFIGURED)
@@ -360,7 +360,7 @@ export interface LibraryInstallRunner {
  *
  * @internal exported for contract tests
  */
-export async function runLibraryInstall(
+async function runLibraryInstall(
 	entry: LibraryEntry,
 	runner: LibraryInstallRunner,
 	options: { withRequirements: boolean },

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

@@ -100,7 +100,7 @@ function fallbackPreview(node: TreeSnapshotNode): string {
  * of turns and the discarded branch had no route back. Returns null for a
  * session that was not forked.
  */
-export function forkParentLine(snapshot: TreeSnapshot, theme: ClioTheme): string | null {
+function forkParentLine(snapshot: TreeSnapshot, theme: ClioTheme): string | null {
 	const parentSessionId = snapshot.meta.parentSessionId;
 	if (typeof parentSessionId !== "string" || parentSessionId.length === 0) return null;
 	const parentTurnId = snapshot.meta.parentTurnId;
@@ -155,7 +155,7 @@ function flattenTreeSnapshot(snapshot: TreeSnapshot): TreeRow[] {
 const ROW_PREVIEW_BUDGET = 55;
 
 /** @internal */
-export function formatTreeRow(row: TreeRow, opts: { showTimestamps: boolean; width: number }): string {
+function formatTreeRow(row: TreeRow, opts: { showTimestamps: boolean; width: number }): string {
 	const theme = clioTheme();
 	const indent = "  ".repeat(row.depth);
 	// The active tip is where the next message lands; it gets its own glyph so

package/src/interactive/panes-runtime.ts

@@ -35,6 +35,7 @@ import {
 	type PanesWatchController,
 	type PanesYaziController,
 	type PanesYaziStatus,
+	type PanesZoomResult,
 } from "../domains/mux/operations.js";
 import { resolveBinary } from "../tools/executables.js";
 import { type PaneWatchSource, paneWatchDecision } from "./pane-policy.js";
@@ -55,6 +56,18 @@ export interface PanesRuntimeDeps {
 
 const UNAVAILABLE = "the pane layer is not available in this session";
 
+/** Fuzzy operator addressing shared by `close` and `zoom`: id, label, purpose; newest first. */
+function matchOwnedPane(owned: ReadonlyArray<MuxPaneRecord>, target: string): MuxPaneRecord | null {
+	const needle = target.trim().toLowerCase();
+	const newestFirst = [...owned].reverse();
+	return (
+		newestFirst.find((record) => record.ref.paneId.toLowerCase() === needle) ??
+		newestFirst.find((record) => record.label.toLowerCase().includes(needle)) ??
+		newestFirst.find((record) => record.purpose === needle) ??
+		null
+	);
+}
+
 function inventory(records: ReadonlyArray<MuxPaneRecord>): ReadonlyArray<PanesInventoryEntry> {
 	return records.map((record) => ({
 		paneId: record.ref.paneId,
@@ -154,7 +167,8 @@ export function createPanesRuntime(deps: PanesRuntimeDeps): PanesOperations {
 	return {
 		status(): PanesStatus {
 			const detection = deps.mux.detection();
-			const panes = deps.getSettings().panes;
+			const settings = deps.getSettings();
+			const panes = settings.interface.panes;
 			return {
 				mode: deps.mux.mode,
 				available: deps.mux.available(),
@@ -164,10 +178,16 @@ export function createPanesRuntime(deps: PanesRuntimeDeps): PanesOperations {
 				settings: {
 					enabled: panes.enabled,
 					notifications: panes.notifications,
-					journal: panes.journal,
-					yazi: { ...panes.yazi },
+					layout: panes.layout,
+					journal: settings.fleet.history.journal,
+					yazi: { ...panes.files },
 				},
 				yazi: yaziController?.status() ?? closedYaziStatus(),
+				docks: deps.mux.docks().map((dock) => ({
+					slot: dock.slot,
+					paneId: dock.paneId,
+					targetShare: dock.targetShare,
+				})),
 				panes: [...inventory(deps.mux.list()), ...pendingOpens.values()],
 			};
 		},
@@ -186,7 +206,7 @@ export function createPanesRuntime(deps: PanesRuntimeDeps): PanesOperations {
 					return { status: "refused", reason: `unknown preset: ${request.preset}` };
 				}
 				if (preset.id === "yazi") {
-					if (!deps.getSettings().panes.yazi.enabled) {
+					if (!deps.getSettings().interface.panes.files.enabled) {
 						return { status: "refused", reason: "the files pane is disabled by panes.yazi.enabled" };
 					}
 					if (!yaziController) {
@@ -253,6 +273,17 @@ export function createPanesRuntime(deps: PanesRuntimeDeps): PanesOperations {
 			}
 		},
 
+		async zoom(target: string): Promise<PanesZoomResult> {
+			if (!deps.mux.available()) return { status: "unavailable", reason: UNAVAILABLE };
+			const match = matchOwnedPane(deps.mux.list(), target.trim().length === 0 ? "watch" : target);
+			if (!match) return { status: "not-found", target: target.trim().length === 0 ? "watch" : target };
+			const changed = await deps.mux.zoomPane(match.ref.paneId, "toggle");
+			if (!changed) {
+				return { status: "unavailable", reason: "the pane host does not support zoom (protocol below pane.zoom)" };
+			}
+			return { status: "zoomed", paneId: match.ref.paneId, label: match.label };
+		},
+
 		async close(target: string): Promise<PanesCloseResult> {
 			if (!deps.mux.available()) return { status: "unavailable", reason: UNAVAILABLE };
 			const owned = deps.mux.list();
@@ -264,12 +295,7 @@ export function createPanesRuntime(deps: PanesRuntimeDeps): PanesOperations {
 				}
 				return { status: "closed", closed: labels.length, labels };
 			}
-			const needle = target.trim().toLowerCase();
-			const newestFirst = [...owned].reverse();
-			const match =
-				newestFirst.find((record) => record.ref.paneId.toLowerCase() === needle) ??
-				newestFirst.find((record) => record.label.toLowerCase().includes(needle)) ??
-				newestFirst.find((record) => record.purpose === needle);
+			const match = matchOwnedPane(owned, target);
 			if (!match) return { status: "not-found", target };
 			const closed = await deps.mux.closePane(match.ref.paneId);
 			return closed ? { status: "closed", closed: 1, labels: [match.label] } : { status: "not-found", target };

package/src/interactive/permission-overlay.ts

@@ -203,7 +203,7 @@ function wrapArtifactLine(value: string, max: number): string[] {
 	return lines;
 }
 
-export function permissionDecisionPresentation(view: ApprovalRequestView): DecisionPresentation {
+function permissionDecisionPresentation(view: ApprovalRequestView): DecisionPresentation {
 	return classifyDecisionPresentation(
 		decisionFactsForPermission({
 			tool: view.tool,
@@ -307,7 +307,7 @@ function permissionInspectionLines(
  * 80-column one saw the command being authorized end mid-argument, both with
  * nothing marking the cut.
  */
-export function permissionOverlayLines(view: ApprovalRequestView, width: number): string[] {
+function permissionOverlayLines(view: ApprovalRequestView, width: number): string[] {
 	const content = Math.max(8, Math.floor(width));
 	const presentation = permissionDecisionPresentation(view);
 	// The parked call is awaiting a decision, not blocked: the raw rejection

package/src/interactive/renderers/retry-status.ts

@@ -1,13 +1,21 @@
 import type { RetryStatusPayload } from "../chat-loop.js";
 import { styleTaggedNotice } from "./notice.js";
 
-function shorten(value: string, limit: number): string {
-	if (value.length <= limit) return value;
-	return `${value.slice(0, Math.max(0, limit - 3))}...`;
+const RETRY_ERROR_MAX_CHARS = 2_000;
+const RETRY_ERROR_TRUNCATION_MARKER = "… [retry error truncated]";
+
+function boundedRetryError(value: string): string {
+	const characters = Array.from(value);
+	if (characters.length <= RETRY_ERROR_MAX_CHARS) return value;
+	return `${characters.slice(0, RETRY_ERROR_MAX_CHARS).join("").trimEnd()}${RETRY_ERROR_TRUNCATION_MARKER}`;
 }
 
 function rawRetryStatus(status: RetryStatusPayload): string {
-	const suffix = status.errorMessage ? `: ${shorten(status.errorMessage, 120)}` : "";
+	// Both live and replay renderers wrap this notice to their terminal width, so
+	// ordinary errors keep their complete remedy. Raw provider failures can also
+	// be an HTML proxy page, though; keep a generous ceiling so one retry does not
+	// turn that page into hundreds of transcript rows.
+	const suffix = status.errorMessage ? `: ${boundedRetryError(status.errorMessage)}` : "";
 	if (status.phase === "waiting") {
 		return `[retry] attempt ${status.attempt}/${status.maxAttempts} in ${status.seconds ?? 0}s${suffix}`;
 	}

package/src/interactive/renderers/tool-execution.ts

@@ -426,7 +426,7 @@ function ledgerTail(finished: ToolExecutionFinished): { facts: string; offload:
 		if (details?.timedOut === true) parts.push("timed out");
 		if (details?.outputCapped === true) parts.push("output capped");
 	}
-	if (finished.excludeFromContext === true) parts.push("excluded from context");
+	if (finished.excludeFromContext === true) parts.push("not sent to model");
 	if (finished.evictedReason !== undefined) parts.push("evicted", finished.evictedReason);
 	const offloadPath = executed ? offloadPathOf(finished) : null;
 	return {
@@ -994,7 +994,7 @@ function toolUsageFact(result: unknown): string | null {
 function outputFacts(finished: ToolExecutionFinished): string[] {
 	const parts: string[] = [];
 	if (isNonExecutedOutcome(finished.outcome)) {
-		if (finished.excludeFromContext === true) parts.push("excluded from context");
+		if (finished.excludeFromContext === true) parts.push("not sent to model");
 		return parts;
 	}
 	const exitCode = structuredExitCode(finished) ?? (finished.toolName === "bash" && !finished.isError ? "0" : null);
@@ -1024,7 +1024,7 @@ function outputFacts(finished: ToolExecutionFinished): string[] {
 		if (added.length > 0) parts.push(`added ${added.join(", ")}`);
 	}
 	if (isPlainObject(finished.result) && finished.result.terminate === true) parts.push("terminal result");
-	if (finished.excludeFromContext === true) parts.push("excluded from context");
+	if (finished.excludeFromContext === true) parts.push("not sent to model");
 	if (finished.evictedReason !== undefined) parts.push("evicted", finished.evictedReason);
 	return parts;
 }
@@ -1379,7 +1379,7 @@ export function renderBashTranscriptExecution(
 	const shownBytes = Buffer.byteLength(execution.output, "utf8");
 	const totalBytes = execution.totalBytes ?? shownBytes;
 	const args: Record<string, unknown> = { command: execution.command };
-	if (execution.excludeFromContext === true) args.context = "excluded from model context";
+	if (execution.excludeFromContext === true) args.context = "not sent to model";
 	const details = {
 		resultSize: {
 			bytes: totalBytes,

package/src/interactive/session-transcript.ts

@@ -7,7 +7,7 @@ import { buildModelReplayAgentMessagesFromTurns } from "./model-session-replay.j
 
 type SessionOwner = Pick<SessionContract, "create" | "current">;
 type SessionChat = Pick<ChatLoop, "resetForSession">;
-type RoutingSettings = Pick<ClioSettings, "orchestrator">;
+type RoutingSettings = Pick<ClioSettings, "chat">;
 
 export interface SessionTranscriptDeps {
 	session?: SessionOwner;
@@ -74,8 +74,8 @@ export function createSessionTranscript(deps: SessionTranscriptDeps): SessionTra
 		const input: { cwd: string; target?: string; model?: string } = {
 			cwd: deps.getCwd?.() ?? process.cwd(),
 		};
-		if (settings?.orchestrator.target) input.target = settings.orchestrator.target;
-		if (settings?.orchestrator.model) input.model = settings.orchestrator.model;
+		if (settings?.chat.target) input.target = settings.chat.target;
+		if (settings?.chat.model) input.model = settings.chat.model;
 		deps.session.create(input);
 	};