@dotdrelle/wiki-manager 0.14.12 → 0.14.16

package/src/shell/RightPane.tsx

@@ -1,5 +1,5 @@
 /** @jsxImportSource @opentui/solid */
-import { createMemo, Index, Show } from 'solid-js';
+import { createMemo, createSignal, Index, Show } from 'solid-js';
 import { filterRuntimeLogs } from '../core/runtimeLog.js';
 import { fit } from './textFit';
 
@@ -19,7 +19,6 @@ type LogLineParts = { time: string | null; message: string };
 // 4 slots (was 6): items can now span up to 5 lines each (wrapped label +
 // wrapped status/error), so fewer, readable entries beat more, truncated ones.
 const ACTIVITY_SLOTS = Array.from({ length: 4 }, (_, index) => index);
-const LOG_SLOTS = Array.from({ length: 24 }, (_, index) => index);
 const PLAN_VIEWPORT_ROWS = 12;
 
 function wrapLine(value: string, width: number) {
@@ -70,13 +69,59 @@ function queueColor(status: string) {
   return activityColor(status);
 }
 
-function activityLine(activity: any) {
-  const detail = activity?.progress?.detail ?? null;
-  const phase = activity?.progress?.step ?? activity?.progress?.phase ?? activity?.progress?.currentStep ?? null;
-  // On failure the error message is the information that matters — surface
-  // it instead of repeating the bare status the label color already conveys.
+function activityDetailText(activity: any) {
   const error = activityErrorText(activity);
-  return [activity?.status ?? 'unknown', error ?? detail ?? phase].filter(Boolean).join(' · ');
+  if (error) return error;
+  const progress = activity?.progress ?? {};
+  const stepIndex = Number(progress.stepIndex);
+  const stepTotal = Number(progress.stepTotal);
+  const batchIndex = Number(progress.batchIndex);
+  const batchCount = Number(progress.batchCount);
+  const instructionCount = Number(progress.instructionCount);
+  const taskIndex = Number(progress.taskIndex);
+  const taskTotal = Number(progress.taskTotal);
+  const taskProgress = Number.isFinite(taskIndex) && Number.isFinite(taskTotal) && taskTotal > 0
+    ? `Task ${Math.max(1, taskIndex)}/${taskTotal}`
+    : null;
+  const stepProgress = Number.isFinite(stepIndex) && Number.isFinite(stepTotal) && stepTotal > 0
+    ? `Step ${Math.max(1, stepIndex)}/${stepTotal}`
+    : null;
+  const batchProgress = Number.isFinite(batchIndex) && Number.isFinite(batchCount) && batchCount > 0
+    ? `Batch ${Math.min(batchCount, Math.max(1, batchIndex + 1))}/${batchCount}`
+    : null;
+  const instructions = Number.isFinite(instructionCount) && instructionCount > 0
+    ? `${instructionCount} instructions`
+    : null;
+  const detail = batchProgress && /^batch\s+\d+\/\d+/i.test(String(progress.detail ?? ''))
+    ? null
+    : progress.detail;
+  const stabilize = [
+    ['kept', progress.stabilizeKept],
+    ['merged', progress.stabilizeMerged],
+    ['inserted', progress.stabilizeInserted],
+    ['removed', progress.stabilizeRemoved],
+  ].filter(([, value]) => Number.isFinite(Number(value)))
+    .map(([label, value]) => `${label} ${Number(value)}`)
+    .join(', ');
+  const values = [
+    taskProgress,
+    stepProgress,
+    batchProgress,
+    progress.label,
+    detail,
+    instructions,
+    stabilize || null,
+    progress.currentStep ?? progress.step ?? progress.phase,
+  ].map((value) => String(value ?? '').trim()).filter(Boolean);
+  return [...new Set(values)].join(' · ');
+}
+
+function activityDisplayLabel(activity: any) {
+  const businessLabel = activity?._aggregated ? activity?.progress?.label : null;
+  const raw = String(businessLabel ?? activity?.label ?? `${activity?.source ?? 'mcp'} ${activity?.kind ?? 'job'}`);
+  // The percentage has its own right-aligned badge; do not repeat it in the
+  // label (aggregate activities often arrive as "knowledge.update - 85 %").
+  return raw.replace(/\s*[-–—·]\s*\d+\s*%\s*$/, '').trim();
 }
 
 function activityErrorText(activity: any): string | null {
@@ -95,18 +140,17 @@ function activityPercentBadge(activity: any): { text: string; bg: string; fg: st
   return { text: ` ${pct}% `, bg: '#1e3a5f', fg: '#93c5fd' };
 }
 
-function updatedLine(activity: any) {
-  const source = activity?.source ?? 'mcp';
-  const id = activity?.id ? `#${activity.id}` : null;
-  if (!activity?.updatedAt) return [source, id].filter(Boolean).join(' ');
-  const age = Math.max(0, Math.round((Date.now() - Date.parse(activity.updatedAt)) / 1000));
-  return [source, id, `${age}s ago`].filter(Boolean).join(' · ');
-}
+// Single source of truth for terminal-status aliases so a plan step's icon and
+// its color can never disagree (e.g. a 'succeeded' step showing done-green but a
+// pending icon). Shared by planStepColor and PlanPanel's icon().
+const DONE_STATUSES = ['done', 'complete', 'completed', 'success', 'succeeded'];
+const FAILED_STATUSES = ['failed', 'error'];
 
 function planStepColor(step: PlanStep, firstPendingStep: number | null) {
-  if (step.status === 'done') return '#8BD5CA';
-  if (step.status === 'failed') return '#F38BA8';
-  if (step.status === 'running') return '#89B4FA';
+  const status = String(step.status ?? '').toLowerCase();
+  if (DONE_STATUSES.includes(status)) return '#8BD5CA';
+  if (FAILED_STATUSES.includes(status)) return '#F38BA8';
+  if (status === 'running') return '#89B4FA';
   if (step.step === firstPendingStep) return '#89B4FA';
   return '#7F8C8D';
 }
@@ -135,8 +179,12 @@ export function PlanPanel(props: { plan: PlanStep[]; width: number; jobName?: st
   // Keep one column for the native vertical scrollbar when the plan is long.
   const lineWidth = () => Math.max(8, props.width - 3);
   const firstPending = () => props.plan.find((s) => s.status === 'pending')?.step ?? null;
-  const icon = (status: string) =>
-    status === 'done' ? '[✓]' : status === 'failed' ? '[✗]' : status === 'running' ? '[…]' : '[ ]';
+  const icon = (rawStatus: string) => {
+    const status = String(rawStatus ?? '').toLowerCase();
+    if (DONE_STATUSES.includes(status)) return '[✓]';
+    if (FAILED_STATUSES.includes(status)) return '[✗]';
+    return status === 'running' ? '[…]' : '[ ]';
+  };
   const visualRows = createMemo(() => props.plan.reduce((total, step) =>
     total + wrapLine(`${icon(step.status)} ${step.step}. ${step.description}`, lineWidth()).slice(0, 2).length, 0));
   const title = () => {
@@ -149,6 +197,7 @@ export function PlanPanel(props: { plan: PlanStep[]; width: number; jobName?: st
       <text width={lineWidth()} fg="#D6DEE8" content={fit(title(), lineWidth())} />
       <scrollbox
         height={viewportRows()}
+        focusable={false}
         scrollY={true}
         scrollX={false}
         stickyStart="top"
@@ -178,12 +227,13 @@ export function PlanPanel(props: { plan: PlanStep[]; width: number; jobName?: st
 export function ActivityPanel(props: { activities: any[]; width: number }) {
   const lineWidth = () => Math.max(8, props.width - 2);
   const visible = () => props.activities.slice(-ACTIVITY_SLOTS.length).reverse();
+  const visibleSlots = () => visible().map((_activity, index) => index);
   const activityAt = (index: number) => visible()[index] ?? null;
   return (
-    <box flexShrink={0} flexDirection="column" padding={1}>
+    <box flexShrink={0} flexDirection="column" paddingX={1}>
       <text width={lineWidth()} fg="#D6DEE8" content="Activity" />
       <Show when={visible().length > 0} fallback={<text width={lineWidth()} fg="#7F8C8D" content="no active jobs" />}>
-        <Index each={ACTIVITY_SLOTS}>
+        <Index each={visibleSlots()}>
           {(slot) => {
             const activity = () => activityAt(slot());
             // Wrap instead of hard-truncating: a 40-column pane cut labels to
@@ -193,8 +243,8 @@ export function ActivityPanel(props: { activities: any[]; width: number }) {
             const labelLines = () => {
               const item = activity();
               if (!item) return [''];
-              const label = String(item.progress?.label ?? item.label ?? `${item.source ?? 'mcp'} ${item.kind ?? 'job'}`);
-              const lines = wrapLine(label, lineWidth()).slice(0, 2);
+              const label = activityDisplayLabel(item);
+              const lines = wrapLine(label, Math.max(8, lineWidth() - badgeLen())).slice(0, 2);
               if (lines.length === 2 && String(label).length > lines[0].length + lines[1].length) {
                 lines[1] = fit(lines[1], lineWidth());
               }
@@ -202,23 +252,19 @@ export function ActivityPanel(props: { activities: any[]; width: number }) {
             };
             const badge = () => activityPercentBadge(activity());
             const badgeLen = () => badge()?.text.length ?? 0;
-            const statusLines = () => {
+            const detailLines = () => {
               const item = activity();
-              if (!item) return [''];
-              return wrapLine(activityLine(item), Math.max(8, lineWidth() - badgeLen())).slice(0, 2);
+              if (!item) return [];
+              return wrapLine(activityDetailText(item), lineWidth()).slice(0, 2);
             };
             const statusColor = () => (activityErrorText(activity()) ? '#F38BA8' : '#AAB7C4');
             return (
-              <box flexDirection="column" marginTop={slot() === 0 ? 1 : 0}>
-                <text width={lineWidth()} fg={activityColor(activity()?.status)} content={labelLines()[0]} />
-                <Show when={labelLines()[1]}>
-                  <text width={lineWidth()} fg={activityColor(activity()?.status)} content={labelLines()[1]} />
-                </Show>
+              <box flexDirection="column">
                 <box height={1} flexDirection="row">
                   <text
                     width={lineWidth() - badgeLen()}
-                    fg={statusColor()}
-                    content={activity() ? fit(statusLines()[0], lineWidth() - badgeLen()) : ''}
+                    fg={activityColor(activity()?.status)}
+                    content={fit(labelLines()[0], lineWidth() - badgeLen())}
                   />
                   <text
                     width={badgeLen()}
@@ -227,10 +273,12 @@ export function ActivityPanel(props: { activities: any[]; width: number }) {
                     content={badge()?.text ?? ''}
                   />
                 </box>
-                <Show when={statusLines()[1]}>
-                  <text width={lineWidth()} fg={statusColor()} content={fit(statusLines()[1], lineWidth())} />
+                <Show when={labelLines()[1]}>
+                  <text width={lineWidth()} fg={activityColor(activity()?.status)} content={labelLines()[1]} />
                 </Show>
-                <text width={lineWidth()} fg="#7F8C8D" content={activity() ? fit(updatedLine(activity()), lineWidth()) : ''} />
+                <Index each={detailLines()}>
+                  {(line) => <text width={lineWidth()} fg={statusColor()} content={fit(line(), lineWidth())} />}
+                </Index>
               </box>
             );
           }}
@@ -254,9 +302,9 @@ function logMessageColor(message: string): string {
 }
 
 function logRenderLines(logs: string[], width: number): LogSegment[][] {
-  // Newest entry FIRST (descending): the freshest information belongs at the
-  // top of the pane. Wrapped continuation lines stay attached below their
-  // entry's first line.
+  // Runtime history arrives oldest-first. Reverse complete entry blocks once
+  // so the newest event is at the top while wrapped continuation lines remain
+  // attached below their own entry.
   const blocks: LogSegment[][][] = [];
   for (const raw of logs) {
     blocks.push(logEntryLines(raw, width));
@@ -269,11 +317,12 @@ function logEntryLines(raw: string, width: number): LogSegment[][] {
   for (const item of [raw]) {
     const rawLine = item;
     const sourceMatch = String(rawLine).match(/^(runtime)\s+(.*)$/);
-    const source = sourceMatch ? sourceMatch[1] : null;
     const rest = sourceMatch ? sourceMatch[2] : String(rawLine);
     const parts = logLineParts(rest);
     const prefix: LogSegment[] = [];
-    if (source) prefix.push({ text: `${source} `, fg: '#C6A0F6' });
+    // This pane already contains runtime logs exclusively. Repeating the word
+    // "runtime" on every row consumed ~8 columns and squeezed the useful
+    // message into the middle of narrow terminals.
     if (parts.time) prefix.push({ text: `${parts.time} `, fg: '#89B4FA' });
     const prefixLength = prefix.reduce((total, segment) => total + segment.text.length, 0);
     const messageColor = logMessageColor(parts.message);
@@ -291,17 +340,44 @@ function logEntryLines(raw: string, width: number): LogSegment[][] {
 }
 
 export function LogPanel(props: { logs: string[]; width: number; filter?: string }) {
+  const [activeLogTab, setActiveLogTab] = createSignal<'flow' | 'agent-status'>('flow');
   const lineWidth = () => Math.max(8, props.width - 2);
-  const filteredLogs = () => filterRuntimeLogs(props.logs, props.filter ?? '');
-  const visibleLines = createMemo(() => logRenderLines(filteredLogs(), lineWidth()).slice(0, 24));
-  const segmentsAt = (index: number): LogSegment[] => visibleLines()[index] ?? [];
+  const isAgentStatus = (line: string) => /agent[_ -]?status/i.test(line);
+  // Filtering preserves the runtime history order. logRenderLines performs
+  // the single block-level reversal shared by both tabs.
+  const filteredLogs = () => filterRuntimeLogs(props.logs, props.filter ?? '')
+    .filter((line) => activeLogTab() === 'agent-status' ? isAgentStatus(line) : !isAgentStatus(line));
+  const allLines = createMemo(() => logRenderLines(filteredLogs(), lineWidth()));
   return (
-    <box flexGrow={1} flexDirection="column" padding={1}>
-      <text width={lineWidth()} fg="#D6DEE8" content="Logs / Trace" />
-      <box flexGrow={1} flexDirection="column" overflow="hidden">
-        <Index each={LOG_SLOTS}>
-          {(slot) => {
-            const segments = () => segmentsAt(slot());
+    <box flexGrow={2} flexDirection="column" paddingX={1} focusable={false}>
+      <text width={lineWidth()} fg="#4B5563" content={'─'.repeat(lineWidth())} />
+      <box height={1} flexDirection="row">
+        <text
+          fg={activeLogTab() === 'flow' ? '#0B1020' : '#D6DEE8'}
+          bg={activeLogTab() === 'flow' ? '#89B4FA' : undefined}
+          content=" Runtime "
+          onMouseUp={() => setActiveLogTab('flow')}
+        />
+        <text fg="#4B5563" content=" " />
+        <text
+          fg={activeLogTab() === 'agent-status' ? '#0B1020' : '#D6DEE8'}
+          bg={activeLogTab() === 'agent-status' ? '#FBBF24' : undefined}
+          content=" Agent status "
+          onMouseUp={() => setActiveLogTab('agent-status')}
+        />
+      </box>
+      <scrollbox
+        flexGrow={1}
+        focusable={false}
+        scrollY={true}
+        scrollX={false}
+        stickyStart="top"
+        viewportCulling={true}
+        verticalScrollbarOptions={{ visible: allLines().length > 0 }}
+      >
+        <Index each={allLines()}>
+          {(line) => {
+            const segments = () => line();
             const segmentAt = (position: number): LogSegment => segments()[position] ?? { text: '', fg: '#AAB7C4' };
             const usedWidth = (upTo: number) => segments().slice(0, upTo).reduce((total, segment) => total + segment.text.length, 0);
             const widthFor = (position: number) => {
@@ -313,7 +389,7 @@ export function LogPanel(props: { logs: string[]; width: number; filter?: string
                 : Math.min(segment.text.length, lineWidth() - usedWidth(position));
             };
             return (
-              <box height={1} flexDirection="row" overflow="hidden">
+              <box height={1} flexShrink={0} flexDirection="row" overflow="hidden">
                 <text width={widthFor(0)} fg={segmentAt(0).fg} content={segmentAt(0).text} />
                 <text width={widthFor(1)} fg={segmentAt(1).fg} content={segmentAt(1).text} />
                 <text width={widthFor(2)} fg={segmentAt(2).fg} content={segmentAt(2).text} />
@@ -321,7 +397,7 @@ export function LogPanel(props: { logs: string[]; width: number; filter?: string
             );
           }}
         </Index>
-      </box>
+      </scrollbox>
     </box>
   );
 }
@@ -397,14 +473,31 @@ export function RightPane(props: {
   queueInfo: QueueInfo;
   activeTab: 'plan' | 'queue';
   logFilter?: string;
+  pendingApprovals: any[];
+  onApprove: () => void;
   onTabClick: (tab: 'plan' | 'queue') => void;
 }) {
   const planJobName = () => activityJobName(
     [...props.activities].reverse().find((activity) => !activity.terminal) ?? props.activities.at(-1),
   );
   return (
-    <box width={props.width} height="100%" flexDirection="column" gap={1} padding={1} overflow="hidden">
+    <box
+      width={props.width}
+      height="100%"
+      flexDirection="column"
+      paddingLeft={1}
+      paddingTop={1}
+      paddingBottom={1}
+      overflow="hidden"
+      focusable={false}
+    >
       <TabHeader active={props.activeTab} queueCount={props.queueInfo.active} width={props.width} onTabClick={props.onTabClick} />
+      <Show when={props.pendingApprovals.length > 0}>
+        <box height={2} flexDirection="column" border={['left']} borderStyle="heavy" borderColor="#FBBF24" paddingX={1}>
+          <text fg="#FBBF24" content={`${props.pendingApprovals.length} approbation(s) requise(s)`} />
+          <text fg="#0B1020" bg="#FBBF24" content=" Approuver le run " onMouseUp={props.onApprove} />
+        </box>
+      </Show>
       <Show when={props.activeTab === 'queue'} fallback={(
         <>
           <Show when={props.plan && props.plan.length > 0}>

package/src/shell/StartupScreen.tsx

@@ -7,9 +7,7 @@ export type StartupAction =
   | 'open-workspace'
   | 'run-workflow'
   | 'init-workspace'
-  | 'retry-preflight'
-  | 'start-services'
-  | 'open-logs';
+  | 'retry-preflight';
 
 type StartupItem = {
   action: StartupAction;
@@ -115,7 +113,7 @@ export function StartupScreen(props: {
     if (!props.hasWorkspace) {
       return [initWorkspaceItem('Create and configure the default workspace')];
     }
-    const base = [
+    const base: StartupItem[] = [
       {
         action: 'new-conversation',
         label: 'Start a new conversation',
@@ -124,7 +122,7 @@ export function StartupScreen(props: {
       {
         action: 'open-workspace',
         label: 'Open workspace',
-        detail: 'Select, load, then start agents and services',
+        detail: 'Select and load a workspace',
       },
       initWorkspaceItem('Create and configure a new workspace'),
       {
@@ -135,8 +133,6 @@ export function StartupScreen(props: {
     ];
     if (props.preflight?.status === 'degraded') {
       base.push({ action: 'retry-preflight', label: 'Retry pending checks', detail: 'Recheck Docker, Internet, containers and MCP' });
-      base.push({ action: 'start-services', label: 'Start services', detail: 'Start agents and workspace containers, then recheck' });
-      base.push({ action: 'open-logs', label: 'Open diagnostics', detail: 'Show service logs and detailed MCP status' });
     }
     return base;
   });

package/src/shell/renderer.ts

@@ -25,6 +25,7 @@ export function renderPlainMarkdown(value: string) {
     .replace(/\*\*([^*]+)\*\*/g, '$1')
     .replace(/__([^_]+)__/g, '$1')
     .replace(/`([^`]+)`/g, '$1')
+    .replace(/<((?:https?):\/\/[^>\s]+)>/g, '$1')
     .replace(/\[([^\]]+)\]\(([^)]+)\)/g, '$1 ($2)')
     // collapse consecutive blank lines left by removed elements
     .replace(/\n{3,}/g, '\n\n')

package/src/shell/repl.js

@@ -822,73 +822,8 @@ function rememberProductionActivity(session, payload) {
   return true;
 }
 
-const TERMINAL_PLAN_STATUSES = new Set(['done', 'completed', 'succeeded', 'success', 'failed', 'error', 'cancelled', 'canceled']);
-
-function planTaskKey(task, index) {
-  return String(task?.id ?? task?.step ?? `task-${index + 1}`);
-}
-
-function errorDescription(value) {
-  if (!value) return null;
-  if (typeof value === 'string') return value.trim() || null;
-  if (value instanceof Error) return value.message;
-  if (typeof value === 'object') {
-    return errorDescription(value.message ?? value.reason ?? value.detail ?? value.description ?? value.error)
-      ?? (() => { try { return JSON.stringify(value); } catch { return String(value); } })();
-  }
-  return String(value);
-}
-
-function planTaskError(task, state) {
-  const activityKey = task?.activityKey ?? task?._activityKey ?? task?.ownerActivityKey;
-  const activity = (state?.activities ?? []).find((item) =>
-    (activityKey && String(item?.key) === String(activityKey))
-    || (task?.id != null && String(item?.taskId ?? '') === String(task.id)));
-  return errorDescription(
-    task?.error
-      ?? task?.result?.error
-      ?? task?.result?.reason
-      ?? activity?.error
-      ?? activity?.detail
-      ?? (['failed', 'error'].includes(String(state?.status ?? '').toLowerCase()) ? state?.evaluation?.reason : null),
-  );
-}
-
-export function appendRuntimePlanCompletionMessages(session, state) {
-  const plan = Array.isArray(state?.plan) ? state.plan : [];
-  const runId = String(state?.runId ?? 'runtime');
-  const current = new Map(plan.map((task, index) => [planTaskKey(task, index), String(task?.status ?? 'pending').toLowerCase()]));
-  if (session._runtimePlanRunId !== runId || !(session._runtimePlanStatuses instanceof Map)) {
-    session._runtimePlanRunId = runId;
-    session._runtimePlanStatuses = current;
-    return 0;
-  }
-
-  let appended = 0;
-  plan.forEach((task, index) => {
-    const key = planTaskKey(task, index);
-    const status = current.get(key);
-    const previous = session._runtimePlanStatuses.get(key);
-    if (!TERMINAL_PLAN_STATUSES.has(status) || TERMINAL_PLAN_STATUSES.has(previous)) return;
-    const label = String(task?.label ?? task?.description ?? task?.name ?? `Job ${key}`);
-    const failed = status === 'failed' || status === 'error';
-    const cancelled = status === 'cancelled' || status === 'canceled';
-    const error = failed ? planTaskError(task, state) : null;
-    const content = failed
-      ? `✗ Job terminé en erreur : ${label}\nStatus: ${status}${error ? `\nErreur: ${error}` : ''}`
-      : cancelled
-        ? `○ Job terminé : ${label}\nStatus: ${status}`
-        : `✓ Job terminé : ${label}\nStatus: ${status}`;
-    conversationMessages(session).push({ role: 'command', content });
-    appended += 1;
-  });
-  session._runtimePlanStatuses = current;
-  return appended;
-}
-
 export function applyRuntimeStateToShellSession(session, state) {
   if (!state || typeof state !== 'object') return false;
-  appendRuntimePlanCompletionMessages(session, state);
   const displayState = sanitizeRuntimeStateForDisplay(state);
   session.agentProjection = {
     conversation: Array.isArray(displayState.conversation) ? displayState.conversation.map((message) => ({ ...message })) : [],
@@ -937,21 +872,9 @@ export function applyRuntimeStateToShellSession(session, state) {
 
 export function sanitizeRuntimeStateForDisplay(state) {
   if (!state || typeof state !== 'object') return state;
-  const status = String(state.status ?? 'idle').toLowerCase();
-  const visible = ['running', 'queued', 'pending', 'waiting', 'pending_approval', 'error', 'failed']
-    .includes(status);
-  if (visible) return state;
-  return {
-    ...state,
-    conversation: [],
-    chain: [],
-    plan: [],
-    activities: [],
-    workflow: null,
-    logs: [],
-    summary: null,
-    planPatches: [],
-  };
+  // Keep the last terminal run visible for post-run inspection. A new
+  // run_started event replaces these fields when the next run begins.
+  return state;
 }
 
 // In agent mode, Donna receives every free-text turn and decides whether to
@@ -1467,7 +1390,10 @@ export async function runLine(line, { agent, packageJson, session, onUpdate, onS
       onUpdate?.();
       return { exit: false, oneShotAgent: true };
     }
-    conversationMessages(session).push({ role: 'user', content: oneShotAgentInput });
+    // The runtime persists and returns this same user message through /state.
+    // Mark the optimistic local copy so useSession can bind that runtime entry
+    // to it instead of appending a duplicate after `/agent <instruction>`.
+    conversationMessages(session).push({ role: 'user', content: oneShotAgentInput, _pending: true });
     onUpdate?.();
     const outcome = await submitRuntimeRun(oneShotAgentInput, { runtime, session });
     applyRuntimeOutcome(session, outcome, (msg) => onStep?.(msg));