@iowarp/clio-coder 0.4.0 → 0.4.2

package/src/interactive/council-grid.ts

@@ -78,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);
@@ -124,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/dispatch-board.ts

@@ -6,6 +6,7 @@ import {
 	type DispatchProgressPayload,
 	type DispatchRunIdentity,
 } from "../core/bus-events.js";
+import { resolveDispatchFailureStatus } from "../core/dispatch-outcome.js";
 import type { SafeEventBus } from "../core/event-bus.js";
 import { rawDurationMs } from "../core/timers.js";
 import type { AgentAudience } from "../domains/agents/spec.js";
@@ -37,7 +38,7 @@ import {
 	type ObservabilityNotice,
 	type ObservabilitySnapshot,
 } from "../domains/observability/index.js";
-import { type CostProvenance, foregroundStreamUsage } from "../domains/providers/index.js";
+import { type CostProvenance, foregroundStreamUsage, resolveCostProvenance } from "../domains/providers/index.js";
 import { sanitizeCallTargetText } from "../domains/safety/call-target.js";
 import { type Component, truncateToWidth, visibleWidth, wrapTextWithAnsi } from "../engine/tui.js";
 import { formatWorkerContextMeter } from "./context-meter.js";
@@ -121,7 +122,7 @@ export interface DispatchBoardRow {
 	gate?: { role: string; cycle: number };
 	council?: { group: string; label: string; color?: string; round: number };
 	/** Orchestrator-executed declared verification status. */
-	hostVerification?: "verified" | "rejected" | "skipped";
+	hostVerification?: "verified" | "rejected" | "skipped" | "not_implicated";
 	/**
 	 * The canonical trust projection of the sealed receipt, read back from
 	 * disk and authenticated against the ledger row. Absent while the run is
@@ -182,10 +183,7 @@ export interface DispatchBoardPhase {
  * the column lines up across rows; a long step id is clipped with a marker
  * rather than pushing the row's other facts out of position.
  */
-export function formatDispatchPhaseCell(
-	phase: DispatchBoardPhase | undefined,
-	width = PHASE_COLUMN_WIDTH,
-): string | null {
+function formatDispatchPhaseCell(phase: DispatchBoardPhase | undefined, width = PHASE_COLUMN_WIDTH): string | null {
 	if (phase === undefined) return null;
 	return truncateToWidth(`w${phase.wave} ${phase.stepId}`, Math.max(1, width), "…", false);
 }
@@ -294,9 +292,7 @@ export interface AgentAudiencePresentation {
  * machinery. Returns null for base/custom agents, which the operator asked for
  * directly and which therefore render like any other row.
  */
-export function agentAudiencePresentation(
-	row: Pick<DispatchBoardRow, "agentAudience">,
-): AgentAudiencePresentation | null {
+function agentAudiencePresentation(row: Pick<DispatchBoardRow, "agentAudience">): AgentAudiencePresentation | null {
 	if (row.agentAudience === "shadow") return { glyph: GLYPH.subProcess, token: "muted" };
 	if (row.agentAudience === "internal") return { glyph: GLYPH.subProcess, token: "dim" };
 	return null;
@@ -439,10 +435,7 @@ function latestEvidenceFailureReason(notices: readonly ObservabilityNotice[], ru
  * summary with an evidence bundle is ready; the latest error-level evidence
  * notice for the run marks it failed.
  */
-export function deriveRunEvidenceState(
-	snapshot: ObservabilitySnapshot | undefined,
-	runId: string,
-): RunEvidencePresentation {
+function deriveRunEvidenceState(snapshot: ObservabilitySnapshot | undefined, runId: string): RunEvidencePresentation {
 	if (!snapshot || runId.length === 0) return { state: "none", viewFilter: `dispatch:${runId}` };
 	if (snapshot.pendingEvidenceBuildRunIds.includes(runId)) {
 		return { state: "pending", viewFilter: `evidence:${runId}` };
@@ -497,6 +490,16 @@ function cardUnitsLine(theme: ClioTheme, key: string, units: ReadonlyArray<strin
 	return visibleWidth(full) <= contentWidth ? full : fitUnits(theme, prefix, units, contentWidth);
 }
 
+/** A prose card value, wrapped inside the columns left by its one-time key. */
+function cardWrappedValueLines(theme: ClioTheme, key: string, value: string, contentWidth: number): string[] {
+	const prefix = cardKvKey(theme, key);
+	const prefixWidth = visibleWidth(prefix);
+	const valueWidth = Math.max(1, contentWidth - prefixWidth);
+	return wrapTextWithAnsi(value, valueWidth).map(
+		(line, index) => `${index === 0 ? prefix : " ".repeat(prefixWidth)}${line}`,
+	);
+}
+
 // The `proof` row: the colored state marker, an optional failure reason, then
 // the dim `/view` filter an operator would type.
 function evidenceCardLine(theme: ClioTheme, evidence: RunEvidencePresentation, contentWidth: number): string | null {
@@ -509,7 +512,7 @@ function evidenceCardLine(theme: ClioTheme, evidence: RunEvidencePresentation, c
 }
 
 /** Sanitize lifecycle task text at the terminal boundary and collapse it to one bounded line. */
-export function sanitizeDispatchTaskSummary(value: unknown): string | undefined {
+function sanitizeDispatchTaskSummary(value: unknown): string | undefined {
 	if (typeof value !== "string") return undefined;
 	const sanitized = sanitizeCallTargetText(value);
 	if (sanitized.length === 0) return undefined;
@@ -622,7 +625,7 @@ function trustCardLines(theme: ClioTheme, row: DispatchBoardRow, contentWidth: n
 	);
 }
 
-export function renderDispatchCard(
+function renderDispatchCard(
 	row: DispatchBoardRow,
 	width: number,
 	evidence?: RunEvidencePresentation,
@@ -715,24 +718,22 @@ export function renderDispatchCard(
 			contentWidth,
 		),
 		targetLine,
-		...(row.taskSummary
-			? [truncateToWidth(`${cardKvKey(theme, "task")}${theme.fg("muted", row.taskSummary)}`, contentWidth, "…", false)]
-			: []),
+		...(row.taskSummary ? cardWrappedValueLines(theme, "task", theme.fg("muted", row.taskSummary), contentWidth) : []),
 		cardUnitsLine(theme, "status", statusUnits, contentWidth),
 		...trustCardLines(theme, row, contentWidth),
 		...(row.budget !== undefined
 			? [
-					truncateToWidth(
-						`${cardKvKey(theme, "policy")}${theme.fg("muted", `${formatBudgetPolicy(row.budget)}; requested ${formatBudgetRequest(row.budget)}`)}`,
+					...cardWrappedValueLines(
+						theme,
+						"policy",
+						theme.fg("muted", `${formatBudgetPolicy(row.budget)}; requested ${formatBudgetRequest(row.budget)}`),
 						contentWidth,
-						"…",
-						false,
 					),
-					truncateToWidth(
-						`${cardKvKey(theme, "budget")}${theme.fg("muted", `${formatEffectiveBudget(row.budget)}; reason ${formatBudgetReasons(row.budget)}`)}`,
+					...cardWrappedValueLines(
+						theme,
+						"budget",
+						theme.fg("muted", `${formatEffectiveBudget(row.budget)}; reason ${formatBudgetReasons(row.budget)}`),
 						contentWidth,
-						"…",
-						false,
 					),
 				]
 			: []),
@@ -797,7 +798,7 @@ export function renderDispatchCard(
 		const proofLine = evidenceCardLine(theme, evidence, contentWidth);
 		if (proofLine !== null) bodyLines.push(proofLine);
 	}
-	if (detail !== null) bodyLines.push(`${cardKvKey(theme, "detail")}${theme.fg("dim", detail)}`);
+	if (detail !== null) bodyLines.push(...cardWrappedValueLines(theme, "detail", theme.fg("dim", detail), contentWidth));
 
 	return frame(theme, cardTitle, bodyLines, width, { rightMeta: elapsed });
 }
@@ -956,10 +957,7 @@ export type DispatchBoardItem = { kind: "run"; row: DispatchBoardRow } | { kind:
  * its first row and its remaining rows are consumed, so the board never shows a
  * member both inside its group and beside it.
  */
-export function dispatchBoardItems(
-	rows: ReadonlyArray<DispatchBoardRow>,
-	selectedRunId?: string | null,
-): DispatchBoardItem[] {
+function dispatchBoardItems(rows: ReadonlyArray<DispatchBoardRow>, selectedRunId?: string | null): DispatchBoardItem[] {
 	const byGroup = new Map<string, DispatchBoardRow[]>();
 	for (const row of rows) {
 		const group = row.council?.group;
@@ -984,14 +982,14 @@ export function dispatchBoardItems(
 }
 
 /** The framed council card: the member grid or stack, with the synthesis run full width under it. */
-export function renderCouncilCard(group: CouncilGroupView, width: number): string[] {
+function renderCouncilCard(group: CouncilGroupView, width: number): string[] {
 	const theme = clioTheme();
 	const contentWidth = Math.max(0, width - 4);
 	const title = `${theme.fg("info", "council")} ${theme.paint(group.group, { bold: true })}`;
 	return frame(theme, title, councilGroupBody(theme, group, contentWidth), width, { rightMeta: group.elapsed });
 }
 
-export function formatDispatchBoardLines(
+function formatDispatchBoardLines(
 	rows: ReadonlyArray<DispatchBoardRow>,
 	width = 76,
 	observability?: ObservabilitySnapshot,
@@ -1288,12 +1286,6 @@ function resolveAgentEndStatus(rawMessages: unknown): DispatchBoardStatus | null
 	return null;
 }
 
-function resolveFailedStatus(reason: unknown): DispatchBoardStatus {
-	if (reason === "dead" || reason === "stalled") return "dead";
-	if (reason === "interrupted" || reason === "canceled") return "aborted";
-	return "failed";
-}
-
 function resolveFailureDetail(
 	payload: Partial<DispatchFailedPayload>,
 	fallback: string | null | undefined,
@@ -1604,7 +1596,7 @@ export function createDispatchBoardStore(
 			entry.durationMs = parseFiniteNumber(payload.durationMs, Math.max(0, rawDurationMs(entry.startedAtMs, now)));
 			entry.tokenCount = parseFiniteNumber(payload.tokenCount, entry.tokenCount);
 			entry.costUsd = parseFiniteNumber(payload.costUsd, entry.costUsd);
-			entry.costProvenance = payload.costProvenance ?? "unknown";
+			entry.costProvenance = resolveCostProvenance(payload.costProvenance, entry.costProvenance ?? "unknown");
 			entry.outcomeDetail = null;
 			if (payload.hostVerification !== undefined) entry.hostVerification = payload.hostVerification;
 			settleFromReceipt(entry);
@@ -1624,7 +1616,7 @@ export function createDispatchBoardStore(
 			const payload: Partial<DispatchFailedPayload> = raw ?? {};
 			const runId = parseRunId(payload.runId);
 			const previousStatus = runId !== null ? entries.get(runId)?.status : undefined;
-			const resolvedStatus = resolveFailedStatus(payload.reason);
+			const resolvedStatus = resolveDispatchFailureStatus(payload.reason);
 			const status = previousStatus === "dead" && resolvedStatus === "failed" ? "dead" : resolvedStatus;
 			const entry = upsertBase(payload, status, now);
 			if (!entry) return;
@@ -1633,7 +1625,7 @@ export function createDispatchBoardStore(
 			entry.durationMs = parseFiniteNumber(payload.durationMs, Math.max(0, rawDurationMs(entry.startedAtMs, now)));
 			entry.tokenCount = parseFiniteNumber(payload.tokenCount, entry.tokenCount);
 			entry.costUsd = parseFiniteNumber(payload.costUsd, entry.costUsd);
-			entry.costProvenance = payload.costProvenance ?? "unknown";
+			entry.costProvenance = resolveCostProvenance(payload.costProvenance, entry.costProvenance ?? "unknown");
 			entry.outcomeDetail = resolveFailureDetail(payload, entry.outcomeDetail);
 			if (payload.hostVerification !== undefined) entry.hostVerification = payload.hostVerification;
 			settleFromReceipt(entry);
@@ -1722,7 +1714,7 @@ export function createDispatchBoardStore(
 					entry.ttftMs = Math.round(performance.now() - entry.startedAtClockMs);
 				}
 			}
-			if (type === "clio_write_record_downgraded") {
+			if (type === "clio_coder_write_record_downgraded") {
 				const rawPayload = (workerEvent as { payload?: unknown }).payload;
 				const payload =
 					typeof rawPayload === "object" && rawPayload !== null ? (rawPayload as Record<string, unknown>) : null;
@@ -1740,7 +1732,7 @@ export function createDispatchBoardStore(
 			// board says about what a worker is saying or touching comes from here,
 			// so the board and the transcript block cannot tell two stories.
 			entry.progress.observe(payload.event);
-			if (type === "clio_steer_received") {
+			if (type === "clio_coder_steer_received") {
 				const steerPayload = (workerEvent as { payload?: { chars?: unknown } }).payload;
 				entry.steerAcknowledgement = {
 					receivedAtMs: Date.now(),

package/src/interactive/editor-bash.ts

@@ -6,7 +6,28 @@ export interface EditorBashCommand {
 	excludeFromContext: boolean;
 }
 
+/**
+ * Invisible one-shot envelope used only between ClioEditor and its submit
+ * callback. Pi trims submitted text before invoking that callback, so a
+ * whitespace guard would disappear and re-arm a pasted `!` command. The
+ * controller removes this envelope before routing the text as an ordinary
+ * prompt; it is never persisted or sent to a model.
+ */
+export const PASTED_EDITOR_OPERATOR_GUARD = "\u2063";
+
+export function guardPastedEditorOperator(text: string): string {
+	return `${PASTED_EDITOR_OPERATOR_GUARD}${text}`;
+}
+
+export function unguardPastedEditorOperator(text: string): string {
+	return text.startsWith(PASTED_EDITOR_OPERATOR_GUARD) ? text.slice(PASTED_EDITOR_OPERATOR_GUARD.length) : text;
+}
+
 export function parseEditorBashCommand(text: string): EditorBashCommand | null {
+	// Commands are an editor-line operator, not a shell-script paste surface.
+	// The paste guard also makes a bracketed-pasted one-liner literal until it
+	// has crossed the editor submit boundary and been routed as a normal prompt.
+	if (text.startsWith(PASTED_EDITOR_OPERATOR_GUARD) || text.includes("\n") || text.includes("\r")) return null;
 	if (!text.startsWith("!")) return null;
 	const excludeFromContext = text.startsWith("!!");
 	const command = (excludeFromContext ? text.slice(2) : text.slice(1)).trim();

package/src/interactive/editor-steer.ts

@@ -4,8 +4,8 @@
  * `!` intercept in the editor submit path.
  *
  * The target token is deliberately a bare word (letters, digits, underscore,
- * hyphen). Inline file references (`@package.json`, `@src/x.ts`) contain dots
- * or slashes and never match, so prompt file expansion keeps working.
+ * hyphen). File completion consults the same predicate and quotes an otherwise
+ * ambiguous root-level extensionless reference before it reaches this parser.
  */
 
 export interface EditorSteerMention {
@@ -23,11 +23,16 @@ export type SteerTargetResolution =
 	| { kind: "ambiguous"; candidates: RunningDispatchRef[] }
 	| { kind: "none" };
 
-const STEER_MENTION = /^@([A-Za-z0-9][A-Za-z0-9_-]*)\s+(\S[\s\S]*)$/;
+const STEER_TARGET_TOKEN = /^[A-Za-z0-9][A-Za-z0-9_-]*$/;
+const STEER_MENTION = /^@(\S+)\s+(\S[\s\S]*)$/;
+
+export function isEditorSteerTargetToken(value: string): boolean {
+	return STEER_TARGET_TOKEN.test(value);
+}
 
 export function parseEditorSteerMention(text: string): EditorSteerMention | null {
 	const match = STEER_MENTION.exec(text.trim());
-	if (!match?.[1] || !match[2]) return null;
+	if (!match?.[1] || !match[2] || !isEditorSteerTargetToken(match[1])) return null;
 	return { target: match[1], text: match[2].trim() };
 }
 

package/src/interactive/editor-submit.ts

@@ -6,7 +6,7 @@ import type { SessionContract, SessionEntry } from "../domains/session/index.js"
 import { toolPresentationPolicy } from "../tools/presentation.js";
 import type { ChatLoop } from "./chat-loop.js";
 import type { ChatPanel } from "./chat-panel.js";
-import { bashExecutionEntryInput, parseEditorBashCommand } from "./editor-bash.js";
+import { bashExecutionEntryInput, parseEditorBashCommand, unguardPastedEditorOperator } from "./editor-bash.js";
 import {
 	formatSteerCandidates,
 	parseEditorSteerMention,
@@ -57,6 +57,8 @@ export interface EditorSubmitExpansion {
 
 export interface EditorSubmitEditor {
 	getText(): string;
+	/** Draft spelling for an immediate send, including any editor provenance envelope. */
+	getTextForSubmit(): string;
 	setText(text: string): void;
 	addToHistory(text: string): void;
 }
@@ -318,18 +320,19 @@ export function createEditorSubmitController(deps: EditorSubmitDeps): EditorSubm
 		submitEditorSteerMention(mention) !== "unhandled";
 
 	const submitEditorText = (text: string): void => {
-		const trimmed = text.trim();
+		const literalText = unguardPastedEditorOperator(text);
+		const trimmed = literalText.trim();
 		if (trimmed.length === 0) return;
 		deps.collapseLaunchpadBeforeSubmit?.();
 		if (parseEditorBashCommand(text)) {
 			if (!deps.chat.isStreaming() && !activeEditorBash) {
-				deps.editor.addToHistory(text);
+				deps.editor.addToHistory(literalText);
 				deps.editor.setText("");
 			} else {
 				// Pi Editor clears the buffer before invoking onSubmit. A command
 				// refused by either admission guard is still a draft, so put it back
 				// for the operator instead of silently discarding it.
-				deps.editor.setText(text);
+				deps.editor.setText(literalText);
 			}
 			if (runEditorBash(text)) deps.ui.requestRender();
 			return;
@@ -339,12 +342,12 @@ export function createEditorSubmitController(deps: EditorSubmitDeps): EditorSubm
 			const submission = submitEditorSteerMention(steerMention);
 			if (submission !== "unhandled") {
 				if (submission === "accepted") {
-					deps.editor.addToHistory(text);
+					deps.editor.addToHistory(literalText);
 					deps.editor.setText("");
 				} else {
 					// Resolution and dispatch failures are correctable rejections. Pi
 					// has already cleared the editor, so explicitly restore the draft.
-					deps.editor.setText(text);
+					deps.editor.setText(literalText);
 				}
 				deps.ui.requestRender();
 				return;
@@ -352,18 +355,18 @@ export function createEditorSubmitController(deps: EditorSubmitDeps): EditorSubm
 		}
 		const command = parseSlashCommand(trimmed);
 		if (isRejectedCommand(command)) {
-			deps.editor.setText(text);
+			deps.editor.setText(literalText);
 			deps.dispatchCommand(trimmed);
 		} else if (command.kind === "unknown-command") {
 			const result = deps.dispatchCommand(trimmed);
 			if (result === "rejected") {
-				deps.editor.setText(text);
+				deps.editor.setText(literalText);
 			} else {
-				deps.editor.addToHistory(text);
+				deps.editor.addToHistory(literalText);
 				deps.editor.setText("");
 			}
 		} else {
-			deps.editor.addToHistory(text);
+			deps.editor.addToHistory(literalText);
 			deps.editor.setText("");
 			deps.dispatchCommand(trimmed);
 		}
@@ -388,7 +391,8 @@ export function createEditorSubmitController(deps: EditorSubmitDeps): EditorSubm
 	};
 
 	const admitCapturedText = async (text: string, signal?: AbortSignal): Promise<void> => {
-		const trimmed = text.trim();
+		const literalText = unguardPastedEditorOperator(text);
+		const trimmed = literalText.trim();
 		if (trimmed.length === 0) return;
 		signal?.throwIfAborted();
 		deps.collapseLaunchpadBeforeSubmit?.();
@@ -396,7 +400,7 @@ export function createEditorSubmitController(deps: EditorSubmitDeps): EditorSubm
 			while (deps.chat.isStreaming()) await awaitWithAbort(deps.chat.whenSettled(), signal);
 			while (activeEditorBashSettlement) await awaitWithAbort(activeEditorBashSettlement, signal);
 			signal?.throwIfAborted();
-			deps.editor.addToHistory(text);
+			deps.editor.addToHistory(literalText);
 			runEditorBash(text);
 			deps.ui.requestRender();
 			return;
@@ -408,7 +412,7 @@ export function createEditorSubmitController(deps: EditorSubmitDeps): EditorSubm
 				if (submission === "rejected") {
 					throw new Error("queued boot steer was not admitted; preserving it for recovery");
 				}
-				deps.editor.addToHistory(text);
+				deps.editor.addToHistory(literalText);
 				deps.ui.requestRender();
 				return;
 			}
@@ -421,7 +425,7 @@ export function createEditorSubmitController(deps: EditorSubmitDeps): EditorSubm
 			deps.ui.requestRender();
 			throw new Error("queued boot command needs correction; preserving it for recovery");
 		}
-		deps.editor.addToHistory(text);
+		deps.editor.addToHistory(literalText);
 		signal?.throwIfAborted();
 		if (deps.dispatchCommandAsync) await deps.dispatchCommandAsync(trimmed, signal);
 		else deps.dispatchCommand(trimmed);
@@ -429,9 +433,10 @@ export function createEditorSubmitController(deps: EditorSubmitDeps): EditorSubm
 	};
 
 	const queueFollowUpFromEditor = (): void => {
-		const text = deps.editor.getText().trim();
+		const streaming = deps.chat.isStreaming();
+		const text = (streaming ? deps.editor.getText() : deps.editor.getTextForSubmit()).trim();
 		if (text.length === 0) return;
-		if (!deps.chat.isStreaming()) {
+		if (!streaming) {
 			deps.editor.setText("");
 			submitEditorText(text);
 			deps.ui.requestRender();
@@ -458,9 +463,10 @@ export function createEditorSubmitController(deps: EditorSubmitDeps): EditorSubm
 	};
 
 	const interruptFromEditor = (): void => {
-		const text = deps.editor.getText().trim();
+		const streaming = deps.chat.isStreaming();
+		const text = (streaming ? deps.editor.getText() : deps.editor.getTextForSubmit()).trim();
 		if (text.length === 0) return;
-		if (!deps.chat.isStreaming()) {
+		if (!streaming) {
 			deps.editor.setText("");
 			submitEditorText(text);
 			deps.ui.requestRender();

package/src/interactive/external-editor.ts

@@ -50,7 +50,7 @@ export function resolveExternalEditor(
 
 export function editTextExternally(initialText: string, command: string | null): ExternalEditResult {
 	if (!command) return { ok: false, error: "no external editor configured; set VISUAL or EDITOR" };
-	const tmpFile = join(tmpdir(), `clio-editor-${process.pid}-${randomUUID()}.md`);
+	const tmpFile = join(tmpdir(), `clio-coder-editor-${process.pid}-${randomUUID()}.md`);
 	try {
 		writeFileSync(tmpFile, initialText, "utf8");
 		const result = runEditor(command, tmpFile);