@iowarp/clio-coder 0.3.9 → 0.4.1

package/src/entry/panes-activation.ts

@@ -0,0 +1,31 @@
+/**
+ * Whether this boot activates the panes extension.
+ *
+ * The decision is made here, before any panes module loads, because the whole
+ * point of the activation model is that a plain boot never pays for the
+ * extension: no socket probe, no guest-mode detection, and none of the mux
+ * domain's code in the import closure. The orchestrator dynamically imports
+ * `src/entry/with-panes.ts` only when this resolves to an active rung, and a
+ * built-graph contract test (tests/contracts/instant-shell-import-graph.test.ts)
+ * pins that the default boot chunk carries no mux domain code.
+ *
+ * Precedence: the command-line flag wins in both directions, then the
+ * `panes.enabled` setting, then the shipped default of `off`.
+ */
+
+/** Mirrors `MuxEnablement` in src/domains/mux/detect.ts without importing it. */
+export type PanesEnablement = "auto" | "embedded" | "off";
+
+export function resolvePanesEnablement(
+	flag: "with" | "without" | undefined,
+	setting: PanesEnablement | undefined,
+): PanesEnablement {
+	if (flag === "without") return "off";
+	if (flag === "with") {
+		// `--with-panes` on a settings file that names `embedded` honors the
+		// stronger setting; on `off` or absent settings it activates guest
+		// detection, which is the flag's promise.
+		return setting === "embedded" ? "embedded" : "auto";
+	}
+	return setting ?? "off";
+}

package/src/entry/with-panes.ts

@@ -0,0 +1,20 @@
+/**
+ * The panes extension's composition surface.
+ *
+ * This is the one module that statically imports panes-only code: the mux
+ * domain (socket client, contract, yazi session machinery) and the interactive
+ * glue built on it. The orchestrator imports it dynamically, and only when
+ * `resolvePanesEnablement` (src/entry/panes-activation.ts) says the extension
+ * is active, so a plain `clio-coder` boot never loads any of it. The built
+ * import graph is pinned by tests/contracts/instant-shell-import-graph.test.ts:
+ * the default boot chunk must carry no mux domain code.
+ *
+ * Everything re-exported here is a value the orchestrator wires; types cross
+ * the seam as type-only imports at the call sites and cost nothing.
+ */
+
+export { createMuxDomainModule } from "../domains/mux/index.js";
+export { createMuxBridge } from "../interactive/mux-bridge.js";
+export { createPanesRuntime } from "../interactive/panes-runtime.js";
+export { createWatchPaneController } from "../interactive/watch-pane.js";
+export { createYaziBridge } from "../interactive/yazi-bridge.js";

package/src/interactive/application-controller.ts

@@ -6,32 +6,32 @@ import { isEscapeKey, type OverlayState, overlayOwnsInput } from "./overlay-life
 export const APPLICATION_DOUBLE_TAP_MS = 500;
 
 const CLOSED_ACTION_ORDER = [
-	"clio.notifications.dismiss",
-	"clio.tool.expand",
-	"clio.tool.expandAll",
-	"clio.tool.liveOutput",
-	"clio.editor.external",
-	"clio.message.followUp",
-	"clio.message.interrupt",
-	"clio.message.dequeue",
-	"clio.thinking.expand",
-	"clio.thinking.expandAll",
+	"clio-coder.notifications.dismiss",
+	"clio-coder.tool.expand",
+	"clio-coder.tool.expandAll",
+	"clio-coder.tool.liveOutput",
+	"clio-coder.editor.external",
+	"clio-coder.message.followUp",
+	"clio-coder.message.interrupt",
+	"clio-coder.message.dequeue",
+	"clio-coder.thinking.expand",
+	"clio-coder.thinking.expandAll",
 ] as const satisfies ReadonlyArray<ClioKeybinding>;
 
 const GLOBAL_ACTION_ORDER = [
-	"clio.status.toggle",
-	"clio.thinking.cycle",
-	"clio.session.tree",
-	"clio.dispatchBoard.toggle",
-	"clio.tasks.open",
-	"clio.decisions.open",
-	"clio.dispatch.background",
-	"clio.model.select",
+	"clio-coder.status.toggle",
+	"clio-coder.thinking.cycle",
+	"clio-coder.session.tree",
+	"clio-coder.dispatchBoard.toggle",
+	"clio-coder.tasks.open",
+	"clio-coder.decisions.open",
+	"clio-coder.dispatch.background",
+	"clio-coder.model.select",
 	// Backward stays ahead of forward so prefix-overlapping user bindings keep
 	// the same more-specific-first behavior as the original input listener.
-	"clio.model.cycleBackward",
-	"clio.model.cycleForward",
-	"clio.exit",
+	"clio-coder.model.cycleBackward",
+	"clio-coder.model.cycleForward",
+	"clio-coder.exit",
 ] as const satisfies ReadonlyArray<ClioKeybinding>;
 
 export type CtrlCAction = "cancel-stream" | "close-overlay" | "clear-editor" | "arm-shutdown" | "shutdown";

package/src/interactive/bus-notices.ts

@@ -125,7 +125,7 @@ function workerEscalationAxis(
 	return null;
 }
 
-export function middlewareBudgetWarningKey(payload: unknown): string | null {
+function middlewareBudgetWarningKey(payload: unknown): string | null {
 	if (!isMiddlewareHookFailedPayload(payload)) return null;
 	if (payload.kind !== "budget_exceeded") return null;
 	return `${payload.registrationId}\u0000${payload.hook}`;
@@ -137,7 +137,7 @@ export function middlewareBudgetWarningKey(payload: unknown): string | null {
  * either way; this warn notice is the operator's only interactive signal
  * that a guard or assessor is misbehaving.
  */
-export function middlewareHookFailedNotice(
+function middlewareHookFailedNotice(
 	payload: unknown,
 	options: MiddlewareHookFailedNoticeOptions = {},
 ): BusNotice | null {

package/src/interactive/chat-loop.ts

@@ -215,7 +215,7 @@ export interface ChatSubmitOptions {
  * the turn runtime's own local fallback, so a `/btw` round against a local
  * server authenticates exactly the way a turn against it does.
  */
-const LOCAL_SIDE_QUESTION_API_KEY = "clio-local-target";
+const LOCAL_SIDE_QUESTION_API_KEY = "clio-coder-local-target";
 
 /** Closing notice an operator interrupt leaves in the transcript and the ledger. */
 const INTERRUPT_CANCEL_REASON = "[Clio Coder] run interrupted by operator; delivering the new message now.";
@@ -372,7 +372,7 @@ export interface CreateChatLoopDeps {
 	/**
 	 * Whitelist of target ids that the chat-loop is allowed to drive. The
 	 * orchestrator composes this from `providers.list()` so an unknown
-	 * `settings.orchestrator.target` surfaces a configuration error before
+	 * `settings.chat.target` surfaces a configuration error before
 	 * the agent is constructed.
 	 */
 	knownTargets: () => ReadonlySet<string>;
@@ -527,7 +527,7 @@ export interface CreateChatLoopDeps {
 	registerPrewarmEndpointSlot?: (runtime: AgentRuntime) => (() => void) | null;
 }
 
-export function reloadProtectedArtifactsForSession(
+function reloadProtectedArtifactsForSession(
 	protectedArtifacts: NonNullable<CreateChatLoopDeps["protectedArtifacts"]>,
 	readSessionEntries: (() => ReadonlyArray<SessionEntry>) | undefined,
 ): void {
@@ -547,7 +547,7 @@ export function createChatLoop(deps: CreateChatLoopDeps): ChatLoop {
 	const sideQuestionRound = deps.runSideQuestion ?? runSideQuestion;
 	const handoffRound = deps.runHandoffRound ?? runHandoffRound;
 	const middlewareToolChoice = deps.middlewareToolChoice ?? createMiddlewareToolChoiceControl();
-	const state = createTurnState(deps.getSettings().orchestrator.thinkingLevel ?? "off");
+	const state = createTurnState(deps.getSettings().chat.thinkingLevel ?? "off");
 	const toolStartTimes = new Map<string, number>();
 
 	const preparationListeners = new Set<(phase: TurnPreparationPhase) => void>();
@@ -639,13 +639,13 @@ export function createChatLoop(deps: CreateChatLoopDeps): ChatLoop {
 	 * collapsing them sent a client with a configured target to the wrong fix.
 	 */
 	const nullRuntimeAdmissionReason = (): string => {
-		const orchestrator = deps.getSettings().orchestrator;
+		const orchestrator = deps.getSettings().chat;
 		const target = orchestrator.target?.trim() ?? "";
 		const model = orchestrator.model?.trim() ?? "";
 		return target.length > 0 && model.length === 0 ? "model-not-configured" : "orchestrator-not-configured";
 	};
 
-	const retrySettings = (): RetrySettings => normalizeRetrySettings(deps.getSettings().retry);
+	const retrySettings = (): RetrySettings => normalizeRetrySettings(deps.getSettings().chat.retry);
 
 	const currentToolInvokeOptions = (): Partial<ToolInvokeOptions> => {
 		const options: Partial<ToolInvokeOptions> = {};
@@ -1088,7 +1088,7 @@ export function createChatLoop(deps: CreateChatLoopDeps): ChatLoop {
 			// 4. Preflight overflow check, before the user turn is committed.
 			// A blocked request must not leave a dangling user entry that the
 			// next replay would treat as an unanswered turn.
-			const compactionThreshold = deps.getSettings().compaction?.threshold ?? null;
+			const compactionThreshold = deps.getSettings().context.compaction?.threshold ?? null;
 			const captureTurnSnapshot = (turnId: string): ContextSnapshot =>
 				context.captureRuntimeContextSnapshot(agentRuntime, turnId, compactionThreshold, {
 					promptSegments: compiledPrompt
@@ -1179,7 +1179,7 @@ export function createChatLoop(deps: CreateChatLoopDeps): ChatLoop {
 				runtimeId: agentRuntime.runtimeId,
 				runtimeKind: agentRuntime.runtimeResolution.runtimeKind,
 				wireModelId: agentRuntime.wireModelId,
-				autonomy: deps.getSettings().autonomy,
+				autonomy: deps.getSettings().safety.autonomy,
 				compiledPromptHash: promptHash,
 				staticCompositionHash: promptHash,
 				promptSignature: promptHash,

package/src/interactive/chat-panel.ts

@@ -384,7 +384,7 @@ export interface ChatPanel extends Component {
 
 export interface ChatPanelOptions {
 	/**
-	 * Resolves the user-visible key string for the `clio.tool.expand`
+	 * Resolves the user-visible key string for the `clio-coder.tool.expand`
 	 * action, which folds and unfolds the newest tool call or worker block.
 	 * Returning a non-empty string surfaces a dim ` (<key>)` hint on the one
 	 * surface the key would act on: the latest finished collapsed tool subline,

package/src/interactive/clio-editor.ts

@@ -1,4 +1,12 @@
-import { Editor, stripTerminalSequences, type TUI, truncateToWidth, visibleWidth } from "../engine/tui.js";
+import {
+	Editor,
+	getKeybindings,
+	stripTerminalSequences,
+	type TUI,
+	truncateToWidth,
+	visibleWidth,
+} from "../engine/tui.js";
+import { guardPastedEditorOperator } from "./editor-bash.js";
 import { fitHintEntries } from "./overlay-frame.js";
 import { type PermissionInspectionHint, permissionHintEntries } from "./permission-hint.js";
 import type { ClioTheme } from "./theme/index.js";
@@ -154,7 +162,50 @@ function findBottomRail(lines: readonly string[], width: number): number {
 	return -1;
 }
 
+function cursorEndsDirectoryPath(editor: Editor): boolean {
+	const cursor = editor.getCursor();
+	const line = editor.getLines()[cursor.line] ?? "";
+	return line.slice(0, cursor.col).endsWith("/");
+}
+
+function remapPastedBangOffsets(
+	before: string,
+	after: string,
+	offsets: ReadonlySet<number>,
+	markInsertedAsPasted: boolean,
+): Set<number> {
+	let prefix = 0;
+	while (prefix < before.length && prefix < after.length && before[prefix] === after[prefix]) prefix += 1;
+	let beforeEnd = before.length;
+	let afterEnd = after.length;
+	while (beforeEnd > prefix && afterEnd > prefix && before[beforeEnd - 1] === after[afterEnd - 1]) {
+		beforeEnd -= 1;
+		afterEnd -= 1;
+	}
+
+	const next = new Set<number>();
+	const shift = afterEnd - prefix - (beforeEnd - prefix);
+	for (const offset of offsets) {
+		if (offset < prefix) next.add(offset);
+		else if (offset >= beforeEnd) next.add(offset + shift);
+	}
+	if (markInsertedAsPasted) {
+		for (let offset = prefix; offset < afterEnd; offset += 1) {
+			if (after[offset] === "!") next.add(offset);
+		}
+	}
+	return next;
+}
+
+function startsWithPastedOperator(text: string, pastedBangOffsets: ReadonlySet<number>): boolean {
+	const first = text.search(/\S/u);
+	return first >= 0 && text[first] === "!" && pastedBangOffsets.has(first);
+}
+
 export class ClioEditor extends Editor {
+	private pastedBangOffsets = new Set<number>();
+	private bracketedPasteActive = false;
+
 	constructor(
 		tui: TUI,
 		private readonly chrome: EditorChrome,
@@ -213,6 +264,15 @@ export class ClioEditor extends Editor {
 		return lines;
 	}
 
+	/**
+	 * Preserve paste provenance for every immediate-send binding, including the
+	 * alternate paths that invoke the submit controller without an Editor Enter.
+	 */
+	getTextForSubmit(): string {
+		const text = this.getExpandedText();
+		return startsWithPastedOperator(this.getText(), this.pastedBangOffsets) ? guardPastedEditorOperator(text) : text;
+	}
+
 	/**
 	 * pi-tui's bracketed-paste handling inserts pasted text (including a
 	 * trailing newline the paste carried) without submitting: a pasted
@@ -222,8 +282,46 @@ export class ClioEditor extends Editor {
 	 * through the same handler the typed-Enter path uses.
 	 */
 	override handleInput(data: string): void {
+		const openedPaste = data.includes("\x1b[200~");
 		const closedPaste = data.includes("\x1b[201~");
+		const pasteMutation = this.bracketedPasteActive || openedPaste;
+		const textBeforeInput = this.getText();
+		const keybindings = getKeybindings();
+		// Pi expands and trims the buffer immediately before onSubmit. Envelope a
+		// pasted bang draft for that synchronous handoff so the Bash parser can
+		// distinguish it from a typed operator; the submit controller unwraps it
+		// before sending the literal prompt onward.
+		if (
+			startsWithPastedOperator(textBeforeInput, this.pastedBangOffsets) &&
+			keybindings.matches(data, "tui.input.submit")
+		) {
+			const pastedText = this.getTextForSubmit();
+			this.pastedBangOffsets.clear();
+			this.bracketedPasteActive = false;
+			super.setText(pastedText);
+			super.handleInput(data);
+			return;
+		}
+		const completingDirectory =
+			this.isShowingAutocomplete() &&
+			(keybindings.matches(data, "tui.input.tab") || keybindings.matches(data, "tui.select.confirm"));
+		const textBeforeCompletion = completingDirectory ? this.getText() : "";
 		super.handleInput(data);
+		if (completingDirectory && this.getText() !== textBeforeCompletion && cursorEndsDirectoryPath(this)) {
+			// Directory rows are submenus on the same provider. Re-open immediately
+			// after acceptance so ↑/↓ continues in the child tree without requiring
+			// a second Tab. Pi's provider request remains the only completion path.
+			super.handleInput("\t");
+		}
+		const textAfterInput = this.getText();
+		this.pastedBangOffsets = remapPastedBangOffsets(
+			textBeforeInput,
+			textAfterInput,
+			this.pastedBangOffsets,
+			pasteMutation,
+		);
+		if (openedPaste) this.bracketedPasteActive = true;
+		if (closedPaste) this.bracketedPasteActive = false;
 		if (!closedPaste) return;
 		const text = this.getText();
 		if (!text.endsWith("\n")) return;
@@ -232,4 +330,10 @@ export class ClioEditor extends Editor {
 		this.setText("");
 		this.onSubmit?.(command);
 	}
+
+	override setText(text: string): void {
+		this.pastedBangOffsets.clear();
+		this.bracketedPasteActive = false;
+		super.setText(text);
+	}
 }

package/src/interactive/context-activity.ts

@@ -6,6 +6,7 @@ import {
 	type ContextActivityStatus,
 } from "../core/bus-events.js";
 import type { SafeEventBus } from "../core/event-bus.js";
+import { wrapTextWithAnsi } from "../engine/tui.js";
 import { type ClioTheme, clioTheme, formatCompactMs, frame, GLYPH, padAnsi, spinnerFrame } from "./theme/index.js";
 
 export interface ContextActivitySnapshot {
@@ -152,17 +153,18 @@ export function formatContextActivityIslandLines(
 	const percent = `${Math.round(activityProgress(activity) * 100)}%`.padStart(4);
 	const progressLine = `${progressBar(theme, activity, barWidth)} ${theme.fg("dim", percent)}`;
 	const message = theme.fg(activity.status === "failed" ? "error" : "muted", activity.message);
-	// Every row here can outrun a narrow island: at 40 columns the trail stopped
-	// at "sta" and the message at "refreshed pr", each reading as the whole
-	// value. padAnsi's default marker is empty, so the marker is passed
-	// explicitly. The progress bar is sized to fit and never cuts.
+	// The trail is compact navigation chrome and marks its cut. The message and
+	// detail are the only explanation of this operation, so they wrap in full.
+	// The progress bar is sized to fit and never cuts.
 	const body = [
 		padAnsi(topLine, bodyWidth, GLYPH.ellipsis),
 		padAnsi(progressLine, bodyWidth),
 		phaseTrail(theme, activity, bodyWidth),
-		padAnsi(message, bodyWidth, GLYPH.ellipsis),
+		...wrapTextWithAnsi(message, bodyWidth).map((line) => padAnsi(line, bodyWidth)),
 	];
-	if (activity.detail) body.push(padAnsi(theme.fg("dim", activity.detail), bodyWidth, GLYPH.ellipsis));
+	if (activity.detail) {
+		body.push(...wrapTextWithAnsi(theme.fg("dim", activity.detail), bodyWidth).map((line) => padAnsi(line, bodyWidth)));
+	}
 	return frame(theme, "Context", body, width);
 }
 

package/src/interactive/context-meter.ts

@@ -23,7 +23,7 @@ export const CONTEXT_CATEGORY_TOKEN: Readonly<Record<ContextLedgerCategory, Clio
 	streaming: "accent",
 };
 
-export function contextCategoryGlyph(category: ContextLedgerCategory): string {
+function contextCategoryGlyph(category: ContextLedgerCategory): string {
 	const filled = visibleSingle(GLYPH.contextFull) ? GLYPH.contextFull : GLYPH.barFull;
 	const free = visibleSingle(GLYPH.contextFree) ? GLYPH.contextFree : GLYPH.barEmpty;
 	// The autocompact reserve is held-back headroom, not consumed context; it
@@ -187,7 +187,7 @@ export type WorkerContextSeverity = "healthy" | "warn" | "critical";
 export const WORKER_CONTEXT_WARN_PCT = 80;
 export const WORKER_CONTEXT_CRITICAL_PCT = 95;
 
-export function workerContextSeverity(pct: number): WorkerContextSeverity {
+function workerContextSeverity(pct: number): WorkerContextSeverity {
 	if (pct >= WORKER_CONTEXT_CRITICAL_PCT) return "critical";
 	if (pct >= WORKER_CONTEXT_WARN_PCT) return "warn";
 	return "healthy";
@@ -211,10 +211,7 @@ export interface WorkerContextView {
  * usage has been observed yet. The percentage is capped at 100: a provider
  * report past the window is a full window, not a >100% claim.
  */
-export function workerContextView(
-	usedTokens: number,
-	contextWindow: number | undefined | null,
-): WorkerContextView | null {
+function workerContextView(usedTokens: number, contextWindow: number | undefined | null): WorkerContextView | null {
 	if (contextWindow === undefined || contextWindow === null || contextWindow <= 0) return null;
 	if (!Number.isFinite(usedTokens) || usedTokens <= 0) return null;
 	const pct = Math.min(100, Math.round((usedTokens / contextWindow) * 100));

package/src/interactive/context-overlay.ts

@@ -166,7 +166,7 @@ function renderWorkingSetLines(view: WorkingSetView, config: WorkingSetConfigVie
 	];
 }
 
-export function renderContextLedgerLines(
+function renderContextLedgerLines(
 	ledger: ContextLedger,
 	contentWidth: number,
 	workingSet?: WorkingSetView | null,
@@ -330,6 +330,7 @@ export function openContextOverlay(
 	const handle = showClioOverlayFrame(tui, text, {
 		anchor: "center",
 		width: CONTEXT_OVERLAY_WIDTH,
+		markerId: "context-view",
 		title: "Context Window",
 		footerHint: buildHint([]),
 	});

package/src/interactive/cost-overlay.ts

@@ -271,7 +271,7 @@ function modelBlock(row: CostRow): string[] {
 	]);
 }
 
-export function formatCostOverlayBodyLines(
+function formatCostOverlayBodyLines(
 	totalCost: CostAggregate,
 	totalTokens: number,
 	rows: ReadonlyArray<CostRow>,
@@ -374,6 +374,8 @@ export function openCostOverlay(
 	const handle = showClioOverlayFrame(tui, body, {
 		anchor: "center",
 		width: COST_OVERLAY_WIDTH,
+		// Not derived from the title: that one carries the session id.
+		markerId: "cost",
 		title: sessionId && sessionId.length > 0 ? `Session usage (${sessionId})` : "Session usage",
 		footerHint: buildHint([]),
 	});

package/src/interactive/council-grid.ts

@@ -14,6 +14,7 @@
  */
 
 import { truncateToWidth, wrapTextWithAnsi } from "../engine/tui.js";
+import { COUNCIL_SYNTHESIS_LABEL } from "./council.js";
 import {
 	type ClioTheme,
 	type ClioToken,
@@ -38,9 +39,6 @@ export const COUNCIL_COLUMN_GUTTER = 2;
 /** Rows of a member's bounded answer tail a column shows before it defers to `/view`. */
 export const COUNCIL_ANSWER_ROWS = 4;
 
-/** The label the coordinator gives the synthesis run of every council group. */
-export const COUNCIL_SYNTHESIS_LABEL = "synthesis";
-
 /** One council run as the board draws it: the roster facts plus the run's presentation. */
 export interface CouncilMemberView {
 	runId: string;
@@ -80,7 +78,7 @@ export type CouncilLayout = { mode: "grid"; columnWidth: number } | { mode: "sta
  * moment one column would fall under {@link COUNCIL_COLUMN_MIN_WIDTH}: a grid
  * where only some columns are readable is worse than no grid at all.
  */
-export function councilGridLayout(memberCount: number, contentWidth: number): CouncilLayout {
+function councilGridLayout(memberCount: number, contentWidth: number): CouncilLayout {
 	const count = Math.max(1, memberCount);
 	const usable = contentWidth - COUNCIL_COLUMN_GUTTER * (count - 1);
 	const columnWidth = Math.floor(usable / count);
@@ -126,7 +124,7 @@ function answerRows(theme: ClioTheme, member: CouncilMemberView, width: number,
  * answer. Every line is clipped to the column so a long model id or a long
  * answer line can never push a neighbouring column out of position.
  */
-export function councilMemberLines(
+function councilMemberLines(
 	theme: ClioTheme,
 	member: CouncilMemberView,
 	width: number,

package/src/interactive/council.ts

@@ -26,6 +26,17 @@ export type CouncilSynthesisMode = (typeof COUNCIL_SYNTHESIS_MODES)[number];
 /** Rounds the dispatch tool admits. The command enforces the same bound where the operator typed it. */
 export const COUNCIL_MAX_ROUNDS = 3;
 
+/**
+ * The label the coordinator gives the synthesis run of every council group.
+ *
+ * It lives here rather than beside the board layout that draws it because it is
+ * council vocabulary, not presentation: the share path compares against it to
+ * decide whether a run seals the whole council or one voice, and that path must
+ * stay off the render module graph. See the header of
+ * `src/interactive/slash-commands.ts` for the closure budget that depends on it.
+ */
+export const COUNCIL_SYNTHESIS_LABEL = "synthesis";
+
 export function isCouncilSynthesisMode(value: string): value is CouncilSynthesisMode {
 	return COUNCIL_SYNTHESIS_MODES.some((mode) => mode === value);
 }