@tiens.nguyen/gu-cli 1.0.795 → 1.0.797

package/gu-repl.mjs

@@ -138,7 +138,8 @@ import { unapprovedProjectServers } from "./mcp-command.mjs";
 import { readHooks } from "./hooks-config.mjs";
 import { readSkillSource, sourceLabel, SKILL_SOURCE_KEY } from "./skill-source.mjs";
 import { codingBannerLabel, isOllamaCloudUrl, SECRET_KEY_HINT } from "./coding-key-prompt.mjs";
-import { thoughtPrintSeq } from "./terminal-layout.mjs";
+import { questionEchoText, thoughtPrintSeq } from "./terminal-layout.mjs";
+import { embedNotice, embedState } from "./embed-model-command.mjs";
 import {
   MODELFALLBACK_PREFIX, fallbackAfterTurn, fallbackHistoryLine, fallbackNotice, keyMissingNotice,
   parseModelFallback,
@@ -160,7 +161,7 @@ import {
   registeredRange,
   selectionKeyAction,
 } from "./terminal-copy.mjs";
-import { createHoverController, applyToggleShift } from "./terminal-hover.mjs";
+import { createHoverController, applyToggleShift, rowsDiffer } from "./terminal-hover.mjs";
 // The required pair, imported rather than re-typed: the startup "no models" message names them,
 // and a second copy of these strings would drift from the check that enforces them.
 import { ACCEPTED_EMBED_MODELS, REQUIRED_AGENT_MODEL, REQUIRED_AGENT_PREFIX } from "./ollama-setup.mjs";
@@ -587,13 +588,15 @@ if (cliVerdict.kind === "run") {
 } else if (argv.includes("--help") || argv.includes("-h")) {
   console.log(
     "gu — terminal agent REPL for the current folder\n\n" +
-      "Usage: gu [-v|--verbose] [--license <pro token>]\n" +
+      "Usage: gu [-v|--verbose] [--log] [--license <pro token>]\n" +
       "       gu status                 # worker + a table of the running model servers\n" +
       "       gu start|stop|restart|logs # manage the background worker\n" +
       "       gu logs <port|pid> [-f]   # tail a running model server's log\n" +
       "       gu doctor|login|set        # models / account (same as `gu-cli …`)\n" +
       "       gu rag                    # is RAG working? tests the embedder end-to-end\n\n" +
       "  -v, --verbose   show RAG index-file activity in the bottom bar (each shard written)\n" +
+      "  --log           log this run in detail (~/.gonext/worker.log). Without it only starts,\n" +
+      "                  stops and errors are logged. Ends when this gu closes.\n" +
       "  --license <token>  pro license: a first install ASKS how this machine runs (hosted\n" +
       "                     client / MLX host / private). Without it, a first install is\n" +
       "                     private by default — the whole stack on this machine.\n" +
@@ -613,6 +616,17 @@ if (cliVerdict.kind === "run") {
 // could not deliver, because agent LOG events stay on the worker's console and never enter
 // the job stream. It now drives a real, streamable signal.
 const REPL_VERBOSE = argv.includes("-v") || argv.includes("--verbose");
+// --log (task #352): THIS RUN logs in detail. By default the worker writes only starts, stops and
+// errors; a run started with --log takes a lease (its pid, in ~/.gonext/log-detail.json) that the
+// worker reads, and gives it back on exit. Per run by the owner's rule — nothing is saved, and a
+// run that is killed leaves a lease for a dead process, which is no lease.
+const REPL_LOG = argv.includes("--log");
+if (REPL_LOG) {
+  const ll = await import("./log-level.mjs");
+  await ll.takeLease().catch(() => {});
+  process.on("exit", () => ll.releaseLeaseSync());
+  console.log(dim("  --log: detailed logging is on for this run (~/.gonext/worker.log); back to minimal when gu closes."));
+}
 if (REPL_VERBOSE) {
   process.env.GONEXT_REPL_VERBOSE = "1";
 }
@@ -847,7 +861,7 @@ if (!workerKey || !apiBase) {
       console.error(red(`gu: setup failed: ${e instanceof Error ? e.message : e}`));
       if (e && e.code === "PAIRING_UNSUPPORTED") {
         console.error(
-          "Configure manually: gu-cli set <workerKey> --api-base https://chat-api-v2.gomarsic.cc"
+          "Configure manually: gu-cli set <workerKey> --api-base https://chat-api-v2.getmarsic.cc"
         );
       }
       if (e && (e.code === "KEY_REJECTED" || e.code === "KEY_UNCHECKED")) {
@@ -862,7 +876,7 @@ if (!workerKey || !apiBase) {
     console.error(
       red("gu: no worker key / API base found.") +
         `\nRun ${accent("gu-cli login")} to sign in, or configure manually:` +
-        "\n  gu-cli set <workerKey> --api-base https://chat-api-v2.gomarsic.cc"
+        "\n  gu-cli set <workerKey> --api-base https://chat-api-v2.getmarsic.cc"
     );
     process.exit(1);
   }
@@ -1633,11 +1647,14 @@ let agentMissing = false; // no agent model chosen → agent falls back to the c
 // question to. Seeded from the startup payload; re-checked before a question is refused.
 let noModelAtAll = false;
 let embedMissing = false; // no embed model chosen → RAG is off until /embed-model picks one
+// #347: WHY it is off — "unset" (none chosen) or "off" (one IS chosen, RAG is switched off) — and
+// the chosen model's name, so the row never says "no embed model" about a model that is set.
+let embedWhy = { state: "unset", model: "" };
 // One footer row; the two halves are independent so a model that IS set is never named.
 // ONE SHORT ROW EACH (#317): joined, they ran past 80 columns and were cut mid-word.
 const missingNoticeRows = () => [
   ...(agentMissing ? ["⚠ agent model not set — using the coding model · /agent-model"] : []),
-  ...(embedMissing ? ["⚠ no embed model — RAG is off · /embed-model"] : []),
+  ...(embedMissing ? [embedNotice(embedWhy) || "⚠ no embed model — RAG is off · /embed-model"] : []),
 ];
 
 // ANOTHER MODEL STOOD IN (task #343). The agent reports it as a __MODELFALLBACK__:: step; it is
@@ -1662,6 +1679,30 @@ const fallbackNoticeRows = () => {
   ];
 };
 
+/**
+ * Set the embed warning from what the settings SAY (task #347) — the one place it is decided
+ * after startup's first guess.
+ *
+ * `repair`: an embedder is chosen but RAG is off. On a Local-mode machine the only way to choose
+ * one is /embed-model or setup, and until this fix /embed-model never switched RAG on — so that
+ * state is this bug's leftover, and it is finished here instead of asking the person to choose
+ * again. In Cloud mode the web Settings has its own RAG switch; "off" may be a decision, so it is
+ * reported, not changed.
+ */
+async function syncEmbedWarning({ repair = false } = {}) {
+  const settings = await fetchWorkerSettings();
+  if (!settings) return; // offline: keep what the bar already says
+  let st = embedState(settings);
+  if (st.state === "off" && repair && isClientExpert()) {
+    const md = await import("./model-doctor.mjs");
+    const saved = await md.saveWorkerSettings(apiBase, workerKey, { ragEnabled: true }).catch(() => false);
+    if (saved) st = { ...st, state: "on" };
+  }
+  embedWhy = st;
+  embedMissing = st.state !== "on";
+  if (barPinned()) paintBar();
+}
+
 // #307: the calm note after a quick change of mind — dim, not the ⚠ orange of a warning, and in the
 // bar rather than the transcript (which the whole point is to leave untouched).
 let stopNotice = "";
@@ -2418,10 +2459,10 @@ async function runUpdate() {
   if (cli) rows.push({ id: "cli", label: `Update gu to ${cli}`, note: "takes effect after you restart gu" });
   if (!rows.length) return;
   console.log("");
-  console.log(dim(`  Update  (${PICK_HINT}):`));
+  console.log(dim(`   Update  (${PICK_HINT}):`));
   const idx = await pickFromList(rows.map((r) => r.label + dim(`  · ${r.note}`)), 0);
   if (idx < 0) {
-    console.log(dim("  (cancelled)\n"));
+    console.log(dim("   (cancelled)\n"));
     paintBar();
     return;
   }
@@ -2448,7 +2489,7 @@ async function runUpdate() {
             r.stderr ? ` — ${String(r.stderr).trim().split("\n").slice(-1)[0]}` : ""}`);
         }
       });
-      console.log(accent(`  ✓ API updated to ${api}\n`));
+      console.log(accent(` ✓ API updated to ${api}\n`));
       availableUpdates = { ...availableUpdates, api: "", any: Boolean(availableUpdates.cli) };
     } else {
       await withPendingStatus(`updating gu to ${cli}`, async () => {
@@ -2478,12 +2519,12 @@ async function runUpdate() {
       });
       // NOT "updated" — the files changed, THIS process did not. Claiming otherwise is the one
       // way this feature could lie to someone.
-      console.log(accent(`  ✓ gu ${cli} installed`) +
+      console.log(accent(` ✓ gu ${cli} installed`) +
                   dim(` — ${updateRestartNote()}\n`));
       availableUpdates = { ...availableUpdates, cli: "", any: Boolean(availableUpdates.api) };
     }
   } catch (e) {
-    console.log(red(`  update failed: ${e?.message ?? e}\n`));
+    console.log(red(`   update failed: ${e?.message ?? e}\n`));
   }
   paintBar();
 }
@@ -2577,12 +2618,18 @@ const tokenLineText = () => {
   );
   const total = dim(" · ") + dim("total ") + accent(`↓ ${fmtTokens(Math.max(globalOutputTokens, lastTurnTotalOutput))}`);
   // #323: what the last turn's prompt cost, LAST on the row after a "|" — "~" when the cached share
-  // was assumed; nothing when the model has no price (never a misleading "| $0"). It survives a
+  // was assumed; "no price" when the model has none (never a misleading "| $0"). It survives a
   // narrow window: "total ↓" is dropped before it is (tokenRowWithPrice).
   return tokenRowWithPrice({
     withTotal: head + total,
     withoutTotal: head,
-    price: lastTurnCost?.usd != null ? dim(" | ") + accent(fmtCost(lastTurnCost)) : "",
+    // #349: a model with NO price says so ("no price", dim) — the blank it used to leave read as
+    // the cost feature being broken. Still never "$0": unpriced is not free.
+    price: lastTurnCost?.usd != null
+      ? dim(" | ") + accent(fmtCost(lastTurnCost))
+      : lastTurnCost
+        ? dim(" | no price")
+        : "",
     room: (process.stdout.columns || 80) - 1 - visibleLen(GUTTER),
   });
 };
@@ -3348,6 +3395,7 @@ const hoverCtl = createHoverController({
   pinned: () => barPinned(),
   transcriptRows: () => transcriptRows(),
   blockAtRow: (row) => thoughtBlockAtRow(row),
+  canToggle: (b) => blockCanToggle(b),
   paintRows: (b, on) => paintThoughtRows(b, on),
   replaceRange: (from, to, rows) => transcript.replaceRange(from, to, rows),
   repaintSeq: (b) => thoughtBlockRepaintSeq(b),
@@ -3646,9 +3694,20 @@ function copyTargetAtRow(row) {
   return { block, renderedLines: lines.slice(from, to + 1) };
 }
 
+/**
+ * Would a click on this block change anything? (#350) — its collapsed and its expanded rows, laid
+ * out for THIS window, are not the same. Asked by the hover (may it light up?), by the click and
+ * by Ctrl+T, so the three can never disagree. See rowsDiffer in terminal-hover.mjs.
+ */
+function blockCanToggle(block) {
+  if (!block) return false;
+  return rowsDiffer(paintThoughtRows(block, false, true), paintThoughtRows(block, false, false));
+}
+
 function toggleThoughtAtRow(row) {
   const block = thoughtBlockAtRow(row);
-  return block ? toggleThoughtBlock(block) : false;
+  // A block shown whole has no other form: it did not light up, and the click is not for it.
+  return block && blockCanToggle(block) ? toggleThoughtBlock(block) : false;
 }
 
 /**
@@ -3718,8 +3777,12 @@ function toggleThoughtBlock(block) {
 
 /** Expand/collapse the most recently printed thought block. The keyboard path (Ctrl+T). */
 function toggleNewestThought() {
-  const block = thoughtBlocks[thoughtBlocks.length - 1];
-  if (!barPinned() || !block) return false;
+  if (!barPinned()) return false;
+  // The newest block that HAS another form (#350): Ctrl+T on a thought already shown whole did
+  // nothing visible, while an expandable one sat just above it.
+  let block = null;
+  for (let i = thoughtBlocks.length - 1; i >= 0 && !block; i--) if (blockCanToggle(thoughtBlocks[i])) block = thoughtBlocks[i];
+  if (!block) return false;
   return toggleThoughtBlock(block);
 }
 
@@ -4254,8 +4317,8 @@ if (process.stdin.isTTY) {
       // PROCESS (reported: "PromptCancelled: cancelled … Node.js v22"). Cancel is caught here; any
       // other failure is reported and the session goes on — a keypress must not kill the REPL.
       replaceCodingKey().catch((e) => {
-        if (wasPromptCancelled(e)) console.log(dim("  (cancelled)\n"));
-        else console.log(red(`  ctrl+k failed: ${e?.message ?? e}\n`));
+        if (wasPromptCancelled(e)) console.log(dim("   (cancelled)\n"));
+        else console.log(red(`   ctrl+k failed: ${e?.message ?? e}\n`));
         paintBar();
       });
       return;
@@ -4617,13 +4680,13 @@ const ask = async (q, { prefill = "", hint = [], atomic = false, noCancel = fals
       rl.line = String(prefill);
       rl.cursor = rl.line.length;
     }
-    // ELLIPSIS, so a truncation is VISIBLE. Silently cutting "…[https://ollama2.gomarsic.cc/v1]:"
+    // ELLIPSIS, so a truncation is VISIBLE. Silently cutting "…[https://ollama2.getmarsic.cc/v1]:"
     // to "…[https:/" hides the default Enter would accept, which is worse than not showing it —
     // the reader cannot tell whether the value is short or the row lied. Callers should keep the
     // final line short and put the long part above; this is the guard, not the plan.
     // "… " with the space: the label normally ends in ": ", and a truncation that eats the
     // separator leaves the typed text jammed against the last surviving word
-    // ("agent + embed ser…ollama2.gomarsic.cc"). Two columns for the ellipsis and its space.
+    // ("agent + embed ser…ollama2.getmarsic.cc"). Two columns for the ellipsis and its space.
     inlinePrompt =
       (visibleLen(label) > budget ? clipVisible(label, budget - 2) + "… " : label) || PROMPT;
     try {
@@ -5603,10 +5666,10 @@ async function runModelDoctor() {
   const dc = await import("./doctor-check.mjs");
   const settings = await withPendingStatus("reading your model settings", fetchWorkerSettings);
   if (!settings) {
-    console.log(red("  couldn't reach the API to read your model settings.\n"));
+    console.log(red("   couldn't reach the API to read your model settings.\n"));
     return;
   }
-  console.log(dim("  Asking each model to answer — the real request, not just a ping."));
+  console.log(dim("   Asking each model to answer — the real request, not just a ping."));
   const roles = dc.rolesFromSettings(settings);
   const results = [];
   for (const r of roles) {
@@ -5649,7 +5712,7 @@ async function reconfigureOllamaServer() {
 
   const settings = await withPendingStatus("reading your model settings", fetchWorkerSettings);
   if (!settings) {
-    console.log(red("  couldn't reach the API to read your current settings.\n"));
+    console.log(red("   couldn't reach the API to read your current settings.\n"));
     return;
   }
 
@@ -5678,7 +5741,7 @@ async function reconfigureOllamaServer() {
     console.log(dim(`    ${hint()}`));
     // askSecret: masked, kept out of history; Esc throws PromptCancelled → /model is cancelled
     // with nothing changed, which is what "skip all" means here.
-    const k = (await askSecret(`  API key for ${host} (Enter for none): `)).trim();
+    const k = (await askSecret(`   API key for ${host} (Enter for none): `)).trim();
     typedKeys.set(host, k);
     console.log(GUTTER + dim(ui.keyConfirmation(host, k)));
     return k;
@@ -5694,7 +5757,7 @@ async function reconfigureOllamaServer() {
       // instruction, leaving the reader unsure whether their own address needs it. It does not
       // — normalizeOllamaUrl adds it — and displayUrl round-trips, so what is shown can be fed
       // straight back as the default.
-      console.log(dim(`  ${label} — Enter keeps ${os.displayUrl(dflt)}`));
+      console.log(dim(`   ${label} — Enter keeps ${os.displayUrl(dflt)}`));
       console.log(dim(`    ${hint()}`));
       // THE ROW MUST SAY WHICH SERVER IT IS ASKING FOR. It said a bare "server:" — chosen to
       // fit the frozen row, which traded away the one thing the label was for. Two questions
@@ -5704,13 +5767,13 @@ async function reconfigureOllamaServer() {
       const typed = (await ask(`  ${short}: `)).trim();
       // "s" is the advertised skip (#301); the older words still work.
       if (ui.readStepAnswer(typed).kind === "skip") return null;
-      // EVERY READING OF A BARE HOSTNAME, in order. Typing "ollama2.gomarsic.cc" means
+      // EVERY READING OF A BARE HOSTNAME, in order. Typing "ollama2.getmarsic.cc" means
       // https://…:443 behind a proxy, while "192.168.1.40" means http://…:11434 — one guess
       // is right for one of them and produces a timeout that looks like a dead server for the
       // other. See urlCandidates; the load decides, not the heuristic.
       const candidates = os.urlCandidates(typed || dflt);
       if (!candidates.length) {
-        console.log(dim("  that is not a server address — try 192.168.1.40 or a full URL"));
+        console.log(dim("   that is not a server address — try 192.168.1.40 or a full URL"));
         continue;
       }
       let n = null;
@@ -5725,7 +5788,7 @@ async function reconfigureOllamaServer() {
           try {
             models = await load(cand, key);
             n = cand;
-            if (candidates.length > 1) console.log(dim(`  using ${os.displayUrl(cand.url)}`));
+            if (candidates.length > 1) console.log(dim(`   using ${os.displayUrl(cand.url)}`));
             return;
           } catch (err) {
             lastErr = err;
@@ -5735,31 +5798,31 @@ async function reconfigureOllamaServer() {
       await tryAll();
       // Refused for want of credentials: ask now, whatever the address looked like, and list again.
       if (!n && ui.needsKeyAfterStatus(lastErr?.httpStatus) && !key) {
-        console.log(yellow(`  ${candidates[0].host} needs an API key to list its models.`));
+        console.log(yellow(`   ${candidates[0].host} needs an API key to list its models.`));
         typedKeys.delete(candidates[0].host);
         key = await askKeyFor(candidates[0].host);
         if (key) await tryAll();
       }
       if (!n && ui.needsKeyAfterStatus(lastErr?.httpStatus)) {
-        console.log(red(`  ${candidates[0].host} refused the request (HTTP ${lastErr.httpStatus})${key ? " — that key was not accepted" : ""}.`));
+        console.log(red(`   ${candidates[0].host} refused the request (HTTP ${lastErr.httpStatus})${key ? " — that key was not accepted" : ""}.`));
         continue;
       }
       if (!n) {
         if (lastErr?.httpStatus === 404) {
           // IT ANSWERED — it is simply not an Ollama (#43). "Couldn't reach … is Ollama listening"
           // sent people to debug a server that was working.
-          for (const l of am.describeNotOllama(candidates[0].host, role)) console.log(yellow(`  ${l}`));
+          for (const l of am.describeNotOllama(candidates[0].host, role)) console.log(yellow(`   ${l}`));
           continue;
         }
         // One address typed, one failure reported — naming every candidate would be two errors
         // for one mistake, and neither would be the address they gave.
-        console.log(red(`  couldn't reach ${candidates[0].host}: ${os.networkReason(lastErr)}`));
-        console.log(dim("  check the address, that Ollama is listening on the network, and /proxy."));
+        console.log(red(`   couldn't reach ${candidates[0].host}: ${os.networkReason(lastErr)}`));
+        console.log(dim("   check the address, that Ollama is listening on the network, and /proxy."));
         continue;
       }
       const usable = os.usableModelList(models);
       if (!usable.ok) {
-        for (const l of os.describeUnusableList(usable.why, n.host)) console.log(dim(`  ${l}`));
+        for (const l of os.describeUnusableList(usable.why, n.host)) console.log(dim(`   ${l}`));
         continue;
       }
       if (require) {
@@ -5768,13 +5831,13 @@ async function reconfigureOllamaServer() {
         // rather than the server.
         const need = os.checkRequired(models);
         if (!need.ok) {
-          for (const l of os.describeNotReady(n.host, need.missing)) console.log(yellow(`  ${l}`));
-          console.log(dim("  (or type s to skip — you can set it later with /model)"));
+          for (const l of os.describeNotReady(n.host, need.missing)) console.log(yellow(`   ${l}`));
+          console.log(dim("   (or type s to skip — you can set it later with /model)"));
           continue;
         }
-        console.log(dim(`  ✓ ${n.host} has the ${os.REQUIRED_AGENT_PREFIX} family and the embedder`));
+        console.log(dim(` ✓ ${n.host} has the ${os.REQUIRED_AGENT_PREFIX} family and the embedder`));
       }
-      console.log(dim(`  found ${models.length} model${models.length === 1 ? "" : "s"} on ${n.host}`));
+      console.log(dim(`   found ${models.length} model${models.length === 1 ? "" : "s"} on ${n.host}`));
       return { normalized: n, models };
     }
     return null;
@@ -5785,17 +5848,17 @@ async function reconfigureOllamaServer() {
   // ask again. Keep it, say where it is changed, and go on to the coder.
   const agentOnProvider = am.agentIsOnProvider(settings);
   const curAgent =
-    String(settings.agentModelUrl ?? "").trim() || "https://ollama1.gomarsic.cc/v1";
+    String(settings.agentModelUrl ?? "").trim() || "https://ollama1.getmarsic.cc/v1";
   const curCode = am.coderServerDefault(settings);
 
   let agent = null;
   if (agentOnProvider) {
     const host = hostOf(String(settings.agentModelUrl ?? ""));
-    console.log(dim(`  The agent model is on ${host} (OpenAI-compatible) — kept as it is; change it with /agent-model.`));
-    console.log(dim("  Enter keeps the current address."));
+    console.log(dim(`   The agent model is on ${host} (OpenAI-compatible) — kept as it is; change it with /agent-model.`));
+    console.log(dim("   Enter keeps the current address."));
   } else {
-    console.log(dim("  TWO servers may be used: the agent + embedder on one, the coder on another."));
-    console.log(dim("  Enter keeps the current address."));
+    console.log(dim("   TWO servers may be used: the agent + embedder on one, the coder on another."));
+    console.log(dim("   Enter keeps the current address."));
     agent = await askServer("Agent + embedding server", curAgent, {
       require: true,
       short: "agent + embed server",
@@ -5815,7 +5878,7 @@ async function reconfigureOllamaServer() {
   }
 
   if (!agent && !coder) {
-    console.log(dim("  nothing set — models stay as they are.\n"));
+    console.log(dim("   nothing set — models stay as they are.\n"));
     return;
   }
 
@@ -5838,13 +5901,13 @@ async function reconfigureOllamaServer() {
         const size = am.formatSize(m.sizeBytes);
         return `${m.model}${size ? dim(`  ${size}`) : ""}${m.model === agentDefault ? dim("  (current)") : ""}`;
       });
-      console.log(dim(`  AGENT model — routes every turn, writes the plain replies  (${PICK_HINT}):`));
+      console.log(dim(`   AGENT model — routes every turn, writes the plain replies  (${PICK_HINT}):`));
       const pick = await pickFromList(
         labels,
         Math.max(0, cands.findIndex((m) => m.model === agentDefault))
       );
       if (pick < 0) {
-        console.log(dim("  (cancelled — nothing changed)\n"));
+        console.log(dim("   (cancelled — nothing changed)\n"));
         return;
       }
       chat = cands[pick].model;
@@ -5863,10 +5926,10 @@ async function reconfigureOllamaServer() {
       const kind = am.isLikelyEmbedder(m) ? dim("  · embedder") : "";
       return `${m.model}${size ? dim(`  ${size}`) : ""}${kind}${m.model === roles.code ? dim("  (current)") : ""}`;
     });
-    console.log(dim(`  CODING model — writes the code; worth waiting for  (${PICK_HINT}):`));
+    console.log(dim(`   CODING model — writes the code; worth waiting for  (${PICK_HINT}):`));
     const idx = await pickFromList(labels, Math.max(0, coder.models.findIndex((m) => m.model === roles.code)));
     if (idx < 0) {
-      console.log(dim("  (cancelled — nothing changed)\n"));
+      console.log(dim("   (cancelled — nothing changed)\n"));
       return;
     }
     code = coder.models[idx].model;
@@ -5881,7 +5944,7 @@ async function reconfigureOllamaServer() {
           .flatMap((b) => (Array.isArray(b.models) ? b.models : []))
       );
       console.log(
-        dim(`  Also offer these in /model?  (up to ${MAX_ALT_CODERS} · ${PICK_MANY_HINT}):`)
+        dim(`   Also offer these in /model?  (up to ${MAX_ALT_CODERS} · ${PICK_MANY_HINT}):`)
       );
       const ticks = await pickManyFromList(
         rest.map((m) => `${m.model}${am.formatSize(m.sizeBytes) ? dim(`  ${am.formatSize(m.sizeBytes)}`) : ""}`),
@@ -5890,7 +5953,7 @@ async function reconfigureOllamaServer() {
       );
       // null is Escape and cancels the command; an EMPTY list is a legitimate "offer none".
       if (ticks === null) {
-        console.log(dim("  (cancelled — nothing changed)\n"));
+        console.log(dim("   (cancelled — nothing changed)\n"));
         return;
       }
       extras = ticks.map((i) => rest[i].model);
@@ -5936,7 +5999,7 @@ async function reconfigureOllamaServer() {
     md.saveWorkerSettings(apiBase, workerKey, patch)
   );
   if (!saved) {
-    console.log(red("  couldn't save that to this machine — nothing changed."));
+    console.log(red("   couldn't save that to this machine — nothing changed."));
     console.log(GUTTER + dim("if this machine's local API is old, restart it with `gu api restart`.\n"));
     return;
   }
@@ -5956,13 +6019,13 @@ async function reconfigureOllamaServer() {
         const url = used.find((u) => hostnameOf(u) === host);
         const found = ps.providerRowForUrl(ps.existingBackendRows(fresh ?? {}), url);
         if (!found) {
-          console.log(yellow(`  no server row holds ${host} yet — add its key with ctrl+k in /model.`));
+          console.log(yellow(`   no server row holds ${host} yet — add its key with ctrl+k in /model.`));
           continue;
         }
         const ok = await withPendingStatus("saving the key", () => addCodingKey(host, k, found.id));
         console.log(ok
           ? GUTTER + dim(`✓ key stored for ${host} — the server's default key`)
-          : red(`  couldn't store the key for ${host} — add it with ctrl+k.`));
+          : red(`   couldn't store the key for ${host} — add it with ctrl+k.`));
       }
     }
   }
@@ -5980,11 +6043,11 @@ async function reconfigureOllamaServer() {
       for (const u of cloudUrls) {
         const found = ps.providerRowForUrl(ps.existingBackendRows(fresh ?? {}), u);
         if (found && !am.boundKeyFor(keys, found.id)) {
-          console.log(yellow(`  ${hostOf(u)} is the Ollama cloud — it needs its own API key (the model list worked without one).`));
+          console.log(yellow(`   ${hostOf(u)} is the Ollama cloud — it needs its own API key (the model list worked without one).`));
           // The shared key prompt (task #245); the server's first key becomes its default.
           const { name, secret } = await askServerKey({ host: hostOf(u) });
           if (secret && !(await withPendingStatus("saving the key", () => addCodingKey(name, secret, found.id)))) {
-            console.log(red("  couldn't save the key — add it with ctrl+k.\n"));
+            console.log(red("   couldn't save the key — add it with ctrl+k.\n"));
           }
         }
       }
@@ -6020,7 +6083,7 @@ async function reconfigureOllamaServer() {
     if (g !== null) globalOutputTokens = g;
   }
   paintBar(); // …and show all of it now, not at the next keystroke
-  console.log(accent("  ✓ models saved"));
+  console.log(accent(" ✓ models saved"));
   if (agent) {
     console.log(dim(`      chat   ${chat}  @ ${agent.normalized.host}`));
     console.log(dim(`      RAG    ${os.defaultEmbedModel?.(agent.models) ?? os.REQUIRED_EMBED_MODEL}  @ ${agent.normalized.host}`));
@@ -6038,9 +6101,9 @@ async function reconfigureOllamaServer() {
   // falsely scream "changed" while the seeded value is exactly what settings now hold.
   const seededEmbed = agent ? (os.defaultEmbedModel?.(agent.models) ?? os.REQUIRED_EMBED_MODEL) : "";
   if (agent && String(settings.ragEmbedModel ?? "") !== seededEmbed) {
-    console.log(dim("    the embedder changed — existing RAG indexes need rebuilding (/rag)."));
+    console.log(dim("   the embedder changed — existing RAG indexes need rebuilding (/rag)."));
   }
-  console.log(dim("    takes effect on your next question.\n"));
+  console.log(dim("   takes effect on your next question.\n"));
 }
 
 // ---------- fetch the ready-to-run agent payload from user settings ----------
@@ -6091,7 +6154,7 @@ async function reconfigureOllamaServer() {
  */
 async function pickExtraModels(rest, preChecked = []) {
   if (rest.length === 0) return [];
-  console.log(dim(`  Also offer these in /model?  (up to ${MAX_ALT_CODERS} · ${PICK_MANY_HINT}):`));
+  console.log(dim(`   Also offer these in /model?  (up to ${MAX_ALT_CODERS} · ${PICK_MANY_HINT}):`));
   const already = new Set(preChecked);
   const ticks = await pickManyFromList(
     rest,
@@ -6137,7 +6200,7 @@ async function chooseCodingModelFrom(ps, all, question, current) {
     const idx = await pickFromList(rows, at >= 0 ? at : 0);
     if (idx < 0) return null;
     if (idx === list.length) {
-      const typed = (await ask("  model name: ")).trim();
+      const typed = (await ask("   model name: ")).trim();
       return typed || null;
     }
     if (idx === list.length + 1) {
@@ -6165,11 +6228,11 @@ async function chooseCodingModelFrom(ps, all, question, current) {
 async function chooseFromModelList(all, question, current = 0) {
   const rows = [...all, "type a model name…"];
   const at = typeof current === "string" ? all.indexOf(current) : Number(current) || 0;
-  console.log(dim(`  ${question}  (${PICK_HINT})`));
+  console.log(dim(`   ${question}  (${PICK_HINT})`));
   const idx = await pickFromList(rows, at >= 0 ? at : 0);
   if (idx < 0) return null;
   if (idx === all.length) {
-    const typed = (await ask("  model name: ")).trim();
+    const typed = (await ask("   model name: ")).trim();
     return typed || null;
   }
   return all[idx];
@@ -6179,7 +6242,7 @@ async function manageCodingProviders() {
   const ps = await import("./coding-provider-setup.mjs");
   const settings = await withPendingStatus("reading your providers", fetchWorkerSettings);
   if (!settings) {
-    console.log(red("  couldn't reach the API to read your current settings.\n"));
+    console.log(red("   couldn't reach the API to read your current settings.\n"));
     return;
   }
   const rows = ps.existingBackendRows(settings);
@@ -6196,10 +6259,10 @@ async function manageCodingProviders() {
   );
   const ADD = labels.length;
   labels.push("add another provider — Claude API, OpenAI, Kimi…");
-  console.log(dim(`  Providers — pick one to change it, or add another  (${PICK_HINT}):`));
+  console.log(dim(`   Providers — pick one to change it, or add another  (${PICK_HINT}):`));
   const idx = await pickFromList(labels, 0);
   if (idx < 0) {
-    console.log(dim("  (cancelled)\n"));
+    console.log(dim("   (cancelled)\n"));
     return;
   }
   if (idx === ADD) {
@@ -6214,7 +6277,7 @@ async function manageCodingProviders() {
     1
   );
   if (action <= 0) {
-    console.log(dim("  (nothing changed)\n"));
+    console.log(dim("   (nothing changed)\n"));
     return;
   }
   if (action === 1) await changeProviderModel(ps, rows, p);
@@ -6273,12 +6336,12 @@ async function changeProviderModel(ps, rows, p) {
   if (models.length) {
     model = await chooseCodingModelFrom(ps, models, `Which model should ${p.host} use?`, p.model);
     if (model === null) {
-      console.log(dim("  (nothing changed)\n"));
+      console.log(dim("   (nothing changed)\n"));
       return;
     }
   } else {
     console.log(GUTTER + dim(`Enter keeps ${p.model || "it unset"}`));
-    model = (await ask("  model name: ")).trim();
+    model = (await ask("   model name: ")).trim();
   }
   if (!model) {
     console.log(GUTTER + dim("(nothing changed)\n"));
@@ -6302,7 +6365,7 @@ async function changeProviderModel(ps, rows, p) {
     })
   );
   if (!saved) {
-    console.log(red("  couldn't save that — nothing changed.\n"));
+    console.log(red("   couldn't save that — nothing changed.\n"));
     return;
   }
   // THE FOLDER'S PICK NAMES THE OLD MODEL. Leaving it would send a name this provider no longer
@@ -6316,14 +6379,14 @@ async function changeProviderModel(ps, rows, p) {
     await syncActiveCoderWhere();
     console.log(GUTTER + dim("this folder's pick named the old model — cleared it."));
   }
-  console.log(accent(`  ✓ ${p.host} → ${model}`) + dim("  ·  pick it in /model\n"));
+  console.log(accent(` ✓ ${p.host} → ${model}`) + dim("  ·  pick it in /model\n"));
 }
 
 /** Point a provider at a different URL, keeping its id so saved picks still resolve. */
 async function changeProviderUrl(ps, rows, p) {
   const md = await import("./model-doctor.mjs");
   console.log("\n" + GUTTER + dim(`Enter keeps ${p.url}`));
-  const typed = (await ask("  provider URL: ")).trim();
+  const typed = (await ask("   provider URL: ")).trim();
   if (!typed || typed === p.url) {
     console.log(GUTTER + dim("(nothing changed)\n"));
     return;
@@ -6348,12 +6411,12 @@ async function changeProviderUrl(ps, rows, p) {
     })
   );
   if (!saved) {
-    console.log(red("  couldn't save that — nothing changed.\n"));
+    console.log(red("   couldn't save that — nothing changed.\n"));
     return;
   }
   // The KEY still belongs to this row, because the row id did not change — which is the whole
   // reason an id is not derived from the URL.
-  console.log(accent(`  ✓ ${p.host} → ${r.host}`) + dim("  ·  its key still applies\n"));
+  console.log(accent(` ✓ ${p.host} → ${r.host}`) + dim("  ·  its key still applies\n"));
 }
 
 /** Remove a provider, and the keys that would be stranded by its removal. */
@@ -6386,7 +6449,7 @@ async function removeCodingProvider(ps, rows, p) {
     })
   );
   if (!saved) {
-    console.log(red("  couldn't save that — nothing was removed.\n"));
+    console.log(red("   couldn't save that — nothing was removed.\n"));
     return;
   }
   // THE ROW GOES FIRST. If a key delete fails afterwards the key is merely orphaned, which is
@@ -6410,7 +6473,7 @@ async function removeCodingProvider(ps, rows, p) {
     console.log(GUTTER + dim("this folder was using it — cleared the pick.\n"));
   }
   console.log(
-    accent(`  ✓ removed ${p.host}`) +
+    accent(` ✓ removed ${p.host}`) +
       (stranded.length ? dim(`  ·  ${deleted} key${deleted === 1 ? "" : "s"} deleted`) : "") +
       "\n"
   );
@@ -6451,7 +6514,7 @@ async function addServer({ role = "coding", dialect = "openai", url: presetUrl =
 
   const settings = await withPendingStatus("reading your model settings", fetchWorkerSettings);
   if (!settings) {
-    console.log(red("  couldn't reach the API to read your current settings.\n"));
+    console.log(red("   couldn't reach the API to read your current settings.\n"));
     return;
   }
   // EVERY ROW THAT IS ALREADY THERE, including the synthesized one a pre-#123 account keeps in
@@ -6477,10 +6540,10 @@ async function addServer({ role = "coding", dialect = "openai", url: presetUrl =
     // without Claude, which serves no embeddings).
     const presets = ps.providerPresets(role);
     const rowsShown = [...presets.map((p) => `${p.name}${dim(`  · ${p.url}`)}`), "another URL…"];
-    console.log(dim(`  Which provider?  (${PICK_HINT})`));
+    console.log(dim(`   Which provider?  (${PICK_HINT})`));
     const pi = await pickFromList(rowsShown, 0);
     if (pi < 0) {
-      console.log(dim("  (nothing saved)\n"));
+      console.log(dim("   (nothing saved)\n"));
       return;
     }
     if (pi < presets.length) target = as.serverTarget("openai", presets[pi].url);
@@ -6488,12 +6551,12 @@ async function addServer({ role = "coding", dialect = "openai", url: presetUrl =
   if (!target && !presetUrl) {
     const dflt = as.defaultServerUrl(role, dialect, settings, existing);
     console.log(dim(dialect === "ollama"
-      ? "  An Ollama server: this machine, a box on your network, or https://ollama.com (needs a key)."
-      : "  Any OpenAI-compatible endpoint: OpenRouter, Together, vLLM, LM Studio, …"));
+      ? "   An Ollama server: this machine, a box on your network, or https://ollama.com (needs a key)."
+      : "   Any OpenAI-compatible endpoint: OpenRouter, Together, vLLM, LM Studio, …"));
     // The address goes ABOVE, in full, and Enter's meaning is said out loud (46d1524e).
     console.log(dim(dflt ? `  Enter uses ${dflt}  ·  Esc cancels\n` : "  Esc cancels.\n"));
     for (let attempt = 0; attempt < 3 && !target; attempt++) {
-      const typed = await ask("  server URL: ");
+      const typed = await ask("   server URL: ");
       if (typed === null || as.isCancelWord(typed)) break;
       const raw = typed.trim() || dflt;
       if (!raw) break;
@@ -6510,7 +6573,7 @@ async function addServer({ role = "coding", dialect = "openai", url: presetUrl =
     }
   }
   if (!target) {
-    console.log(dim("  (nothing saved)\n"));
+    console.log(dim("   (nothing saved)\n"));
     return;
   }
   const host = target.host;
@@ -6532,7 +6595,7 @@ async function addServer({ role = "coding", dialect = "openai", url: presetUrl =
   // THE ROW CAP BEFORE ANY QUESTION about a new server: exceeding it is a silent 200 with no row,
   // and asking for a key and a model first would be the worst order to discover that in.
   if (!found && existing.length >= ps.MAX_BACKENDS) {
-    console.log(yellow(`  ${ps.MAX_BACKENDS} servers is the limit — remove one in /model → manage providers first.\n`));
+    console.log(yellow(`   ${ps.MAX_BACKENDS} servers is the limit — remove one in /model → manage providers first.\n`));
     return;
   }
   const need = as.keyRequirement(target);
@@ -6577,7 +6640,7 @@ async function addServer({ role = "coding", dialect = "openai", url: presetUrl =
     let probe = await list();
     // THE STORED KEY WAS REFUSED — ask for the right one once, rather than saving nothing.
     if ((probe.status === 401 || probe.status === 403) && !typedKey && key) {
-      console.log(yellow(`  ${host} refused the stored key (HTTP ${probe.status}) — enter the right one.`));
+      console.log(yellow(`   ${host} refused the stored key (HTTP ${probe.status}) — enter the right one.`));
       if (!(await askKey()) || !key) return;
       probe = await list();
     }
@@ -6612,11 +6675,11 @@ async function addServer({ role = "coding", dialect = "openai", url: presetUrl =
         ? await chooseCodingModelFrom(ps, models, R.question, 0)
         : await chooseFromModelList(models, R.question, Math.max(0, models.indexOf(current)));
       if (model === null) {
-        console.log(dim("  (cancelled — nothing was saved)\n"));
+        console.log(dim("   (cancelled — nothing was saved)\n"));
         return;
       }
     } else {
-      model = String((await ask("  model name: ")) ?? "").trim();
+      model = String((await ask("   model name: ")) ?? "").trim();
     }
     if (!model) {
       console.log(GUTTER + dim("(cancelled — nothing was saved)\n"));
@@ -6633,7 +6696,7 @@ async function addServer({ role = "coding", dialect = "openai", url: presetUrl =
       if (!sw.changed) return;
       plan = sw.plan;
     } else if (as.serverTarget(target.dialect, String(settings.ragEmbedUrl ?? "")).root === target.root && !typedKey) {
-      console.log(dim(`  already embedding with ${model} on ${host} — nothing changed.\n`));
+      console.log(dim(`   already embedding with ${model} on ${host} — nothing changed.\n`));
       return;
     }
   }
@@ -6662,7 +6725,7 @@ async function addServer({ role = "coding", dialect = "openai", url: presetUrl =
   } else {
     const r = as.rowForSave(role, { existing: found.row, model });
     if (r.full) {
-      console.log(yellow(`  ${host} already offers ${ps.MAX_MODELS_PER_SERVER} models — remove one in /model → manage providers first.\n`));
+      console.log(yellow(`   ${host} already offers ${ps.MAX_MODELS_PER_SERVER} models — remove one in /model → manage providers first.\n`));
       return;
     }
     if (r.changed) rows = existing.map((b) => (b === found.row ? r.row : b));
@@ -6670,7 +6733,7 @@ async function addServer({ role = "coding", dialect = "openai", url: presetUrl =
   if (rows !== existing) {
     const ok = await withPendingStatus(`saving ${host}`, () => md.saveWorkerSettings(apiBase, workerKey, { agentCodingBackends: rows }));
     if (!ok) {
-      console.log(red(`  couldn't save ${host} — nothing changed.`));
+      console.log(red(`   couldn't save ${host} — nothing changed.`));
       console.log(GUTTER + dim("if this machine's local API is old, restart it with `gu api restart`.\n"));
       return;
     }
@@ -6679,7 +6742,7 @@ async function addServer({ role = "coding", dialect = "openai", url: presetUrl =
   if (typedKey) {
     const id = await withPendingStatus("saving the key", () => addCodingKey(keyName, key, rowId));
     if (!id) {
-      console.log(yellow(`  ${host} was saved, but its key was not — nothing else changed.`));
+      console.log(yellow(`   ${host} was saved, but its key was not — nothing else changed.`));
       console.log(GUTTER + dim("try again, or add the key with ctrl+k.\n"));
       return;
     }
@@ -6697,18 +6760,20 @@ async function addServer({ role = "coding", dialect = "openai", url: presetUrl =
       console.log(GUTTER + dim("the banner shows the new model next time gu starts.\n"));
     } else if (rows !== existing || typedKey) {
       // The SERVER landed — say so, or the user re-enters a key that is already stored.
-      console.log(yellow(`  ${host} was saved, but the agent model was not pointed at it.`));
+      console.log(yellow(`   ${host} was saved, but the agent model was not pointed at it.`));
       console.log(GUTTER + dim("run /agent-model to finish, or check the API is reachable.\n"));
     }
     return;
   }
   if (role === "embed") {
     // ragEmbedKeyId "" — the server's own keys decide now (its default, or this folder's pick).
-    const fields = { ragEmbedUrl: target.roleUrl("embed"), ragEmbedModel: model, ragEmbedKeyId: "" };
+    // …and ragEnabled: true (#347): choosing an embedder IS turning RAG on. Without it the choice
+    // was stored and never took effect, and the "no embed model" warning came back at every start.
+    const fields = ec.embedSaveFields({ url: target.roleUrl("embed"), model });
     if (!(await saveEmbedSettings(fields))) return;
     if (plan) announceEmbedSaved({ chosen: model, plan, url: target.roleUrl("embed") });
     else {
-      console.log(accent(`  ✓ embedding server → ${host}`));
+      console.log(accent(` ✓ embedding server → ${host}`));
       console.log(dim(`    ${model} did not change, so no knowledge base rebuilds — the same model just answers from ${host}.\n`));
     }
     return;
@@ -6736,7 +6801,7 @@ async function useCoderInThisFolder({ ps, rows, beforeChoices, url, host, locali
     (c) => c.model === model && (c.backendId ? c.backendId === backendId : sameHost(c.url, url))
   );
   if (!offered) {
-    console.log(yellow(`  ${host} was saved, but this API is not offering ${model} yet.`));
+    console.log(yellow(`   ${host} was saved, but this API is not offering ${model} yet.`));
     console.log(GUTTER + dim("run /model — if it is missing there too, the API may be older than"));
     console.log(GUTTER + dim("this feature; restart it with `gu api restart`.\n"));
     return;
@@ -6750,7 +6815,7 @@ async function useCoderInThisFolder({ ps, rows, beforeChoices, url, host, locali
     ])
   );
   if (lost.length) {
-    console.log(yellow(`  ⚠ this API dropped ${lost.length === 1 ? "a provider" : `${lost.length} providers`} when saving:`));
+    console.log(yellow(` ⚠ this API dropped ${lost.length === 1 ? "a provider" : `${lost.length} providers`} when saving:`));
     for (const u of lost) console.log(GUTTER + dim(`    ${u}`));
     if (ps.apiKeysBackendsByKind(after?.codingChoices)) {
       console.log(GUTTER + dim("this API identifies a backend by its KIND, so it holds ONE"));
@@ -6774,8 +6839,8 @@ async function useCoderInThisFolder({ ps, rows, beforeChoices, url, host, locali
   // The key question for this coder is answered — leaving the flag set would warn about a key
   // that was just entered, which reads as the save having failed.
   codingKeyMissingKnown = false;
-  console.log(accent(`  ✓ coding model → ${model}`) + dim(`  · ${host}`));
-  if (key) console.log(dim(`    API key ${keyName || "saved"} ····${key.slice(-4)}  ·  ${host}'s key\n`));
+  console.log(accent(` ✓ coding model → ${model}`) + dim(`  · ${host}`));
+  if (key) console.log(dim(`   API key ${keyName || "saved"} ····${key.slice(-4)}  ·  ${host}'s key\n`));
   // A HOSTED SERVER NEEDS ITS WINDOW SET: with nothing set gu assumes 24,000 prompt tokens and
   // compacts there — on a large hosted model that throws history away every turn.
   if (ps.shouldMentionContextWindow(locality)) {
@@ -6795,7 +6860,7 @@ async function chooseModel() {
       fetchAgentPayload([{ role: "user", content: "ping" }])
     );
   } catch (err) {
-    console.log(red(`  ${describeApiFailure(err, "couldn't load models")}\n`));
+    console.log(red(`   ${describeApiFailure(err, "couldn't load models")}\n`));
     return;
   }
   const allowed = Array.isArray(probe?.codingAllowed) ? probe.codingAllowed : [];
@@ -6816,7 +6881,7 @@ async function chooseModel() {
     // choices while still occupying its URL, so sending this branch straight to the ADD flow
     // would meet "already configured here" and dead-end. The manager fixes such a row, and falls
     // through to add when there is genuinely nothing there.
-    console.log(dim("  No coding model configured."));
+    console.log(dim("   No coding model configured."));
     const go = await pickFromList(
       [dim("leave it"), "add or change a provider — Claude API, OpenAI, Kimi…", "change the Ollama server this machine uses…"],
       1
@@ -6834,7 +6899,7 @@ async function chooseModel() {
   // symptom of being pointed at the wrong box, and the old message sent people to a web app a
   // Client Expert machine does not have.
   if (choices.length === 1) {
-    console.log(dim(`  Only one coding model is available: ${choices[0].model}.`));
+    console.log(dim(`   Only one coding model is available: ${choices[0].model}.`));
     // APPENDED AFTER the existing entry, here and below, so that entry keeps the index anyone
     // who has used it already reaches for.
     const go = await pickFromList(
@@ -6882,10 +6947,10 @@ async function chooseModel() {
   // enough in name to have already been mixed up. The fix is to say which one THIS is — not to
   // print the other two here: the agent model and the embedder are independent settings with
   // their own commands, and listing them together implies a relationship that does not exist.
-  console.log(dim(`  Choose the CODING model — the one that writes code  (${PICK_HINT}):`));
+  console.log(dim(`   Choose the CODING model — the one that writes code  (${PICK_HINT}):`));
   const idx = await pickFromList(labels, activeIdx >= 0 ? activeIdx : 0);
   if (idx < 0) {
-    console.log(dim("  (cancelled)\n"));
+    console.log(dim("   (cancelled)\n"));
     return;
   }
   // The appended entries sit past the last model, so they are the only indices without a choice.
@@ -6915,7 +6980,7 @@ async function chooseModel() {
   const where = showTag && !activeCoderWhere ? ` (${tags[idx]})` : "";
   console.log(
     accent(
-      `  ✓ coding model → ${coderDisplayName(chosen.model, activeCoderWhere)}${where}${chosen.id === def ? " (default)" : ""}\n`
+      ` ✓ coding model → ${coderDisplayName(chosen.model, activeCoderWhere)}${where}${chosen.id === def ? " (default)" : ""}\n`
     )
   );
   // DOES THIS ONE NEED AN API KEY? (task #181) Reported live on Windows 11 / Client Expert:
@@ -6968,7 +7033,7 @@ async function chooseModel() {
       host: ks.provider,
     });
     console.log(
-      dim("    API key ") + accent(line.text) +
+      dim("   API key ") + accent(line.text) +
         (line.note ? orange(`  ·  ${line.note}`) : "") +
         dim(`  ·  ctrl+k to ${line.verb} it`)
     );
@@ -6977,7 +7042,7 @@ async function chooseModel() {
     // the more common one — "how do I change the model" — with no visible answer at all. Reported
     // three times from exactly this view, twice after being told where the picker entry was.
     // /provider is hidden (see HIDDEN_COMMANDS): point at the row of /model that does the same.
-    console.log(dim("    to change this provider's model or URL, or remove it: /model → add or change a provider\n"));
+    console.log(dim("   to change this provider's model or URL, or remove it: /model → add or change a provider\n"));
   }
   // Task #125: the GLOBAL lifetime total is per-backend, so re-fetch it for the newly
   // selected backend NOW — otherwise the footer would keep showing the previous backend's
@@ -7331,13 +7396,13 @@ async function runSwitchSkillCommand(line) {
     }
   } else {
     const options = ["claude", "gu"];
-    console.log(dim(`  Whose skills, MCP servers and hooks this folder uses  (${PICK_HINT}):`));
+    console.log(dim(`   Whose skills, MCP servers and hooks this folder uses  (${PICK_HINT}):`));
     const idx = await pickFromList(
       options.map((o) => sourceLabel(o) + dim(`  · ${where(o)}`) + (o === current ? dim("  · in use") : "")),
       Math.max(0, options.indexOf(current))
     );
     if (idx < 0) {
-      console.log(dim("  (cancelled)\n"));
+      console.log(dim("   (cancelled)\n"));
       paintBar();
       return;
     }
@@ -7355,7 +7420,7 @@ async function runSwitchSkillCommand(line) {
   _skillScan = { at: 0, skills: [] }; // the "/" menu and Tab follow at once
   const say = (l) => console.log(GUTTER + l);
   say(accent(`✓ ${pick === current ? "this folder uses" : "this folder now uses"} ${sourceLabel(pick)}`));
-  say(dim(`  skills, MCP servers and hooks are read from ${where(pick)}`));
+  say(dim(`   skills, MCP servers and hooks are read from ${where(pick)}`));
   // What was found there — and what was not, said rather than shown as an empty list.
   try {
     const sk = discoverSkills({ cwd: process.cwd(), reserved: COMMAND_NAMES });
@@ -7367,20 +7432,20 @@ async function runSwitchSkillCommand(line) {
     say(dim(`  ${count(sk.skills.length, "skill")}${sk.skills.length ? ` (${sk.skills.map((x) => "/" + x.name).join(", ")})` : ""} · ${count(running, "MCP server")} · ${count(nHooks(hk.user) + (hk.projectApproved ? nHooks(hk.project) : 0), "hook")}`));
     // Nothing there at all is a normal state, not a fault: say so, and that gu needs none of it.
     if (!sk.skills.length && !mcp.servers.length && !nHooks(hk.user) && !nHooks(hk.project) && !hk.notes.length) {
-      say(dim("  Nothing is set up there yet — gu works the same without any of them."));
+      say(dim("   Nothing is set up there yet — gu works the same without any of them."));
     }
     for (const w of [...sk.warnings, ...mcp.warnings, ...hk.warnings]) say(yellow("  " + w));
     const unsupported = mcp.servers.filter((x) => x.error);
     if (unsupported.length) say(yellow(`  ${count(unsupported.length, "MCP server")} cannot run here: ${unsupported.map((x) => `${x.name} (${x.error})`).join("; ")}`));
     const waiting = mcp.servers.filter((x) => x.scope === "project" && !x.approved && !x.error);
     if (waiting.length) say(yellow(`  ${count(waiting.length, "project MCP server")} waiting for your approval: ${waiting.map((x) => x.name).join(", ")} — /mcp approve <name>`));
-    if (hk.projectPending) say(yellow(`  this folder's ${hk.projectFile.replace(/^\.\//, "")} has hooks that will not run until you approve them — /hooks approve`));
+    if (hk.projectPending) say(yellow(`   this folder's ${hk.projectFile.replace(/^\.\//, "")} has hooks that will not run until you approve them — /hooks approve`));
     if (hk.notes.length) say(yellow(`  ${count(hk.notes.length, "hook")} in Claude's settings ${hk.notes.length === 1 ? "is" : "are"} NOT run by gu — /hooks says which and why`));
-    if (pick === "claude" && (nHooks(hk.user) || nHooks(hk.project))) say(dim("  Claude's hooks get Claude's event and tool names from gu, so they record here as they do there."));
+    if (pick === "claude" && (nHooks(hk.user) || nHooks(hk.project))) say(dim("   Claude's hooks get Claude's event and tool names from gu, so they record here as they do there."));
   } catch (err) {
-    say(yellow(`  (could not read it back: ${err.message})`));
+    say(yellow(`   (could not read it back: ${err.message})`));
   }
-  say(dim(`  It holds from your next question. ${pick === "claude" ? "/switch-skill gu" : "/switch-skill claude"} switches back.`));
+  say(dim(`   It holds from your next question. ${pick === "claude" ? "/switch-skill gu" : "/switch-skill claude"} switches back.`));
   console.log("");
   if (barPinned()) paintBar();
 }
@@ -7529,7 +7594,7 @@ async function runUsePresetsCommand(line) {
       console.log(GUTTER + dim("/save-preset pastes one in — copy it from another computer's /open-presets.") + "\n");
       return;
     }
-    console.log(dim(`  Presets  (${ed.PICKER_HINT}):`));
+    console.log(dim(`   Presets  (${ed.PICKER_HINT}):`));
     const pick = await pickFromList(found, 0, { keys: ed.PICKER_KEYS });
     if (!pick || pick.index < 0) {
       console.log(GUTTER + dim("nothing changed.") + "\n");
@@ -7608,7 +7673,7 @@ async function runSaveSettingsCommand(line) {
   // have to type is a name — and Enter keeps gu-settings.json, as before.
   let typed = settingsArg(line, "/save-settings");
   if (!typed) {
-    const answer = await ask(`  file name in ${home}  [${st.DEFAULT_SETTINGS_NAME}]: `);
+    const answer = await ask(`   file name in ${home}  [${st.DEFAULT_SETTINGS_NAME}]: `);
     if (answer === null) {
       console.log(GUTTER + dim("  (cancelled — nothing was written)\n"));
       return;
@@ -7668,7 +7733,7 @@ async function runLoadSettingsCommand(line) {
   if (!typed) {
     const found = st.settingsFilesIn(listJsonFiles(home));
     if (found.length > 1) {
-      console.log(dim(`  Settings files in ${home}  (${PICK_HINT}):`));
+      console.log(dim(`   Settings files in ${home}  (${PICK_HINT}):`));
       const i = await pickFromList(found, 0);
       if (i < 0) {
         console.log(GUTTER + dim("  (cancelled — nothing changed)\n"));
@@ -7849,6 +7914,7 @@ async function noteTurnCost(inputTokens, breakdown, coder = null) {
     const c = mp.turnCost({ input: inputTokens, ...cache, price });
     if (!c) {
       lastTurnCost = { usd: null, model, host: mp.hostOf(url) };
+      if (barPinned()) paintBar(); // #349: "no price" is shown, so it is painted like a cost is
       return;
     }
     lastTurnCost = { ...c, input: inputTokens, cacheReported: cache.cacheReported, model, host: mp.hostOf(url) };
@@ -7891,7 +7957,7 @@ async function runPricesCommand(line = "/prices") {
   console.log(GUTTER + dim(pricePublishedAt ? `from gu's tables published ${new Date(pricePublishedAt).toLocaleString()}` : "gu's built-in tables (none published applied yet)"));
   if (priceUpdate) console.log(GUTTER + accent(`${mp.priceUpdateLabel(priceUpdate.publishedAt)} — click it below, or /prices update`));
   if (!mp.findPrice(rows, url, model)) {
-    console.log(GUTTER + yellow(`no price for ${model || "the coder"} at ${mp.hostOf(url) || "its server"} — no cost is shown for its turns.`));
+    for (const r of wrapBlock(mp.noPriceLine({ model, host: mp.hostOf(url) }), GUTTER, width)) console.log(GUTTER + yellow(r));
   }
   if (lastTurnCost?.usd != null) {
     const share = lastTurnCost.input ? Math.round((100 * lastTurnCost.cachedTokens) / lastTurnCost.input) : 0;
@@ -8017,11 +8083,11 @@ async function runClearLogs() {
     const question = (q) => {
       const hint = `(${PICK_HINT}):`;
       console.log("");
-      if (2 + q.length + 2 + hint.length < width) console.log(dim(`  ${q}  ${hint}`));
-      else console.log(dim(`  ${q}\n  ${hint}`));
+      if (2 + q.length + 2 + hint.length < width) console.log(dim(`   ${q}  ${hint}`));
+      else console.log(dim(`   ${q}\n  ${hint}`));
     };
     const cancelled = () => {
-      console.log(dim(`  (stopped — nothing ${freed.length ? "further " : ""}removed)\n`));
+      console.log(dim(`   (stopped — nothing ${freed.length ? "further " : ""}removed)\n`));
       paintBar();
     };
 
@@ -8122,7 +8188,7 @@ async function saveEmbedSettings(fields) {
   const saved = await withPendingStatus("saving your embedder", () =>
     md.saveWorkerSettings(apiBase, workerKey, fields)
   );
-  if (!saved) console.log(red("  couldn't save that to your account — nothing changed.\n"));
+  if (!saved) console.log(red("   couldn't save that to your account — nothing changed.\n"));
   return saved;
 }
 
@@ -8140,7 +8206,7 @@ async function confirmEmbedSwitch(ec, current, chosen) {
   }
   const plan = ec.embedChangePlan({ current, chosen, indexes });
   if (!plan.changed) {
-    console.log(dim(`  already using ${current} — nothing changed.\n`));
+    console.log(dim(`   already using ${current} — nothing changed.\n`));
     return { changed: false, plan: null };
   }
   // "Are you sure?" with nothing behind it is a prompt people learn to dismiss; the four projects
@@ -8148,7 +8214,7 @@ async function confirmEmbedSwitch(ec, current, chosen) {
   for (const l of ec.describeEmbedChange(plan)) console.log(GUTTER + dim(l));
   const go = await pickFromList([`switch to ${chosen}`, `keep ${current}`], plan.stale.length ? 1 : 0);
   if (go !== 0) {
-    console.log(dim("  (kept)\n"));
+    console.log(dim("   (kept)\n"));
     return { changed: false, plan: null };
   }
   return { changed: true, plan };
@@ -8158,13 +8224,13 @@ async function confirmEmbedSwitch(ec, current, chosen) {
 // shared recognition step. Returns { changed, chosen, plan }.
 async function pickEmbedModel(ec, current) {
   const rows = ec.embedChoiceRows(current);
-  console.log(dim(`  Choose the embedder (${PICK_HINT}):`));
+  console.log(dim(`   Choose the embedder (${PICK_HINT}):`));
   const labels = rows.map(
     (r) => `${r.model}${r.dims ? dim(` · ${r.dims}d`) : ""}${dim(` · ${r.note}`)}` + (r.current ? accent(" (current)") : "")
   );
   const idx = await pickFromList(labels, Math.max(0, rows.findIndex((r) => r.current)));
   if (idx < 0) {
-    console.log(dim("  (cancelled)\n"));
+    console.log(dim("   (cancelled)\n"));
     return { changed: false, chosen: "", plan: null };
   }
   const chosen = rows[idx].model;
@@ -8178,21 +8244,21 @@ async function pickEmbedModel(ec, current) {
 // a model that was never pulled fails at the first search, not here. Say it once rather than let
 // it surface as "RAG is broken".
 function announceEmbedSaved({ chosen, plan, url }) {
-  console.log(accent(`  ✓ embedding model → ${chosen}` + (url ? `  on ${hostOf(url)}` : "")));
+  console.log(accent(` ✓ embedding model → ${chosen}` + (url ? `  on ${hostOf(url)}` : "")));
   if (plan?.stale.length) {
     console.log(
       dim(`    ${plan.stale.length} knowledge base${plan.stale.length === 1 ? "" : "s"} ` +
           "will rebuild on the next search — the first one will be slower.")
     );
   }
-  console.log(dim(`    Make sure it is pulled there:  ollama pull ${chosen}\n`));
+  console.log(dim(`   Make sure it is pulled there:  ollama pull ${chosen}\n`));
 }
 
 async function chooseEmbedModel() {
   const ec = await import("./embed-model-command.mjs");
   const settings = await withPendingStatus("reading your embedder", fetchWorkerSettings);
   if (!settings) {
-    console.log(red("  couldn't reach the API to read your current embedder.\n"));
+    console.log(red("   couldn't reach the API to read your current embedder.\n"));
     return;
   }
   const current = ec.currentEmbedModel(settings);
@@ -8205,11 +8271,11 @@ async function chooseEmbedModel() {
   const keyRequired = isOllamaCloudUrl(currentUrl) || /\/v1\/?$/i.test(currentUrl) ||
     Boolean(String(settings.ragEmbedKeyId ?? "").trim());
   const menu = ec.embedMenuOptions({ current, host, keyRequired });
-  console.log(dim(`  Embedding model (currently ${current} on ${host || currentUrl})`));
+  console.log(dim(`   Embedding model (currently ${current} on ${host || currentUrl})`));
   const mi = await pickFromList(menu.map((o) => o.label), 0);
   const action = mi >= 0 ? menu[mi].action : "cancel";
   if (action === "cancel") {
-    console.log(dim("  (cancelled)\n"));
+    console.log(dim("   (cancelled)\n"));
     return;
   }
 
@@ -8250,12 +8316,12 @@ async function saveAgentModel(agentUrl, model, host) {
     md.saveWorkerSettings(apiBase, workerKey, { agentModelUrl: agentUrl, agentModelName: model, agentModel: model })
   );
   if (!ok) {
-    console.log(red(`  couldn't save the agent model — nothing changed.`));
+    console.log(red(`   couldn't save the agent model — nothing changed.`));
     console.log(GUTTER + dim("if this machine's local API is old, restart it with `gu api restart`.\n"));
     return false;
   }
-  console.log(accent(`  ✓ agent model → ${model}`) + dim(`  · ${host}`));
-  console.log(dim("    takes effect on your next question.\n"));
+  console.log(accent(` ✓ agent model → ${model}`) + dim(`  · ${host}`));
+  console.log(dim("   takes effect on your next question.\n"));
   return true;
 }
 
@@ -8268,7 +8334,7 @@ async function askKeyName(prompt = "  key name: ") {
   for (let attempt = 0; attempt < 2; attempt++) {
     const name = (await ask(prompt)).trim();
     if (!am.looksLikeSecret(name)) return name;
-    console.log(yellow("  that looks like the API key itself — the NAME is shown on screen, so it must not be the key."));
+    console.log(yellow("   that looks like the API key itself — the NAME is shown on screen, so it must not be the key."));
     console.log(GUTTER + dim('type a short label like "work", or press Enter for none — the key comes next.'));
   }
   return "";
@@ -8289,7 +8355,7 @@ async function askServerKey({ host = "", nameHint = '"work", "personal"', option
   console.log("\n" + GUTTER + dim(`A name just tells keys apart — ${nameHint}.`));
   const name = (await askKeyName()).trim();
   console.log(GUTTER + dim(SECRET_KEY_HINT));
-  const secret = (await askSecret(`  API key${host ? ` for ${host}` : ""}${name ? ` (${name})` : ""}: `)).trim();
+  const secret = (await askSecret(`   API key${host ? ` for ${host}` : ""}${name ? ` (${name})` : ""}: `)).trim();
   if (!secret) {
     console.log(GUTTER + dim(optional ? "(no key — the server will be called anonymously)" : "(cancelled — nothing was saved)\n"));
     return { name, secret: "" };
@@ -8324,15 +8390,15 @@ async function chooseAgentProvider() {
   const am = await import("./agent-model-command.mjs");
   const settings = await withPendingStatus("reading your providers", fetchWorkerSettings);
   if (!settings) {
-    console.log(red("  couldn't reach the API to read your providers.\n"));
+    console.log(red("   couldn't reach the API to read your providers.\n"));
     return;
   }
   const choices = am.agentProviderChoices(settings);
   if (choices.length === 1) return addServer({ role: "agent", dialect: "openai" });
-  console.log(dim(`  Which OpenAI-compatible provider should the agent use?  (${PICK_HINT})`));
+  console.log(dim(`   Which OpenAI-compatible provider should the agent use?  (${PICK_HINT})`));
   const idx = await pickFromList(choices.map((c) => c.label), 0);
   if (idx < 0) {
-    console.log(dim("  (cancelled)\n"));
+    console.log(dim("   (cancelled)\n"));
     return;
   }
   const pick = choices[idx];
@@ -8354,7 +8420,7 @@ async function chooseAgentModel() {
   const am = await import("./agent-model-command.mjs");
   const settings = await withPendingStatus("reading your agent model", fetchWorkerSettings);
   if (!settings) {
-    console.log(red("  couldn't reach the API to read your current agent model.\n"));
+    console.log(red("   couldn't reach the API to read your current agent model.\n"));
     return;
   }
   const current = am.currentAgentModel(settings);
@@ -8440,12 +8506,12 @@ async function chooseAgentModel() {
   // WHAT TO CHANGE, FIRST (#44). The ways to use an Ollama server or another provider used to sit
   // after every model on this one — 130 rows down on api.openai.com — which read as "OpenAI models
   // are all you can choose". A short menu of what to change comes before any list.
-  console.log(dim(`  Agent model — the one that TALKS: ${current || "unset"} on ${listHost}`));
+  console.log(dim(`   Agent model — the one that TALKS: ${current || "unset"} on ${listHost}`));
   const menu = am.agentMenuOptions({ host: listHost, modelCount: models.length, needsKey });
   const first = await pickFromList(menu.map((o) => o.label), 0);
   const firstAction = first >= 0 ? menu[first].action : "cancel";
   if (firstAction === "cancel") {
-    console.log(dim("  (cancelled)\n"));
+    console.log(dim("   (cancelled)\n"));
     return;
   }
   if (firstAction === "ollama") return addServer({ role: "agent", dialect: "ollama" });
@@ -8459,7 +8525,7 @@ async function chooseAgentModel() {
   }
 
   const rows = am.agentModelRows(models, current);
-  console.log(dim(`  Agent model — the one that TALKS  (currently ${current || "unset"})`));
+  console.log(dim(`   Agent model — the one that TALKS  (currently ${current || "unset"})`));
   // Strip whichever list endpoint was asked, so the header names the SERVER rather than the path.
   console.log(
     dim(`  ${models.length} on ${url.replace(/\/api\/tags$/, "").replace(/\/models$/, "")}  (${PICK_HINT}):`)
@@ -8488,12 +8554,12 @@ async function chooseAgentModel() {
   // where they can be seen, rather than after every model.
   const idx = await pickFromList(labels, Math.max(0, rows.findIndex((r) => r.current)));
   if (idx < 0) {
-    console.log(dim("  (cancelled)\n"));
+    console.log(dim("   (cancelled)\n"));
     return;
   }
   const chosen = rows[idx].model;
   if (chosen === current) {
-    console.log(dim(`  already using ${current} — nothing changed.\n`));
+    console.log(dim(`   already using ${current} — nothing changed.\n`));
     return;
   }
   if (rows[idx].embedder) {
@@ -8504,7 +8570,7 @@ async function chooseAgentModel() {
     console.log(GUTTER + yellow("produce nonsense rather than an error."));
     const go = await pickFromList([`use it anyway`, `pick something else`], 1);
     if (go !== 0) {
-      console.log(dim("  (kept)\n"));
+      console.log(dim("   (kept)\n"));
       return;
     }
   }
@@ -8517,15 +8583,15 @@ async function chooseAgentModel() {
     agentModel: chosen,
   });
   if (!saved) {
-    console.log(red(`  couldn't save that to your account — still using ${current || "unset"}.`));
+    console.log(red(`   couldn't save that to your account — still using ${current || "unset"}.`));
     // The likeliest cause by far, and it is invisible otherwise: this field only became writable
     // through the worker route in task #169, so an older local API silently ignores it.
     console.log(GUTTER + dim("if this machine's local API predates /agent-model, restart it with"));
     console.log(GUTTER + dim("`gu api restart` to pick up the current version.\n"));
     return;
   }
-  console.log(accent(`  ✓ agent model → ${chosen}`));
-  console.log(dim("    takes effect on your next question.\n"));
+  console.log(accent(` ✓ agent model → ${chosen}`));
+  console.log(dim("   takes effect on your next question.\n"));
 }
 
 // Probe whether SSH KEY auth already works for a server (so we can tell the user to run
@@ -8550,7 +8616,7 @@ async function checkServerKeyAuth(s) {
     if (ok) console.log(dim("    ✓ key auth works — deploys will connect without a password."));
     else
       console.log(
-        yellow("    key auth not set up yet.") +
+        yellow("   key auth not set up yet.") +
           dim(` Run once in your terminal:  `) +
           `ssh-copy-id ${s.user}@${s.host}`
       );
@@ -8750,20 +8816,20 @@ async function runS3Command(line) {
   // and, for the secret, does not have to know it.
   const keep = (typed, was) => (String(typed ?? "").trim() ? String(typed).trim() : was);
   const location = keep(
-    await ask(`  Bucket (ARN, s3://… or name)${current.location ? ` [${current.location}]` : ""}: `),
+    await ask(`   Bucket (ARN, s3://… or name)${current.location ? ` [${current.location}]` : ""}: `),
     current.location
   );
   const region = keep(
-    await ask(`  Region [${current.region || "ap-southeast-1"}]: `),
+    await ask(`   Region [${current.region || "ap-southeast-1"}]: `),
     current.region || "ap-southeast-1"
   );
   const accessKeyId = keep(
-    await ask(`  AWS access key id${current.accessKeyId ? ` [${s3.maskKeyId(current.accessKeyId)}]` : ""}: `),
+    await ask(`   AWS access key id${current.accessKeyId ? ` [${s3.maskKeyId(current.accessKeyId)}]` : ""}: `),
     current.accessKeyId
   );
   // askSecret, never ask: the echo is muted and the value never reaches the scrollback (#127).
   const typedSecret = String(
-    await askSecret(`  AWS secret${current.secretConfigured ? " [Enter keeps the stored one]" : ""}:`)
+    await askSecret(`   AWS secret${current.secretConfigured ? " [Enter keeps the stored one]" : ""}:`)
   ).trim();
 
   if (!location || !accessKeyId || !(typedSecret || current.secretConfigured)) {
@@ -8960,16 +9026,16 @@ async function chooseServer() {
     );
     const data = await res.json().catch(() => ({}));
     if (!res.ok) {
-      console.log(red(`  couldn't load servers: ${data?.error || `HTTP ${res.status}`}\n`));
+      console.log(red(`   couldn't load servers: ${data?.error || `HTTP ${res.status}`}\n`));
       return;
     }
     servers = Array.isArray(data?.servers) ? data.servers : [];
   } catch (err) {
-    console.log(red(`  ${describeApiFailure(err, "couldn't load servers")}\n`));
+    console.log(red(`   ${describeApiFailure(err, "couldn't load servers")}\n`));
     return;
   }
   if (servers.length === 0) {
-    console.log(dim("  No deployment servers yet — add them in the web app → Servers.\n"));
+    console.log(dim("   No deployment servers yet — add them in the web app → Servers.\n"));
     return;
   }
   // Arrow-key picker: ↑/↓ to move, Enter to choose (no number typing). The list ends with
@@ -8977,21 +9043,21 @@ async function chooseServer() {
   const labels = servers.map((s) => `${s.name}  (${s.user}@${s.host})`);
   labels.push("none (clear selection)");
   const activeIdx = servers.findIndex((s) => s.host === selectedServer?.host);
-  console.log(dim(`  Choose a deployment server  (${PICK_HINT}):`));
+  console.log(dim(`   Choose a deployment server  (${PICK_HINT}):`));
   const idx = await pickFromList(labels, activeIdx >= 0 ? activeIdx : 0);
   if (idx < 0) {
-    console.log(dim("  (cancelled)\n"));
+    console.log(dim("   (cancelled)\n"));
     return;
   }
   if (idx === servers.length) {
     // the trailing "none" row
     selectedServer = null;
-    console.log(dim("  ✓ deployment server cleared.\n"));
+    console.log(dim(" ✓ deployment server cleared.\n"));
     return;
   }
   const s = servers[idx];
   selectedServer = { name: s.name, host: s.host, user: s.user };
-  console.log(accent(`  ✓ deploy target → ${s.name} (${s.user}@${s.host})`));
+  console.log(accent(` ✓ deploy target → ${s.name} (${s.user}@${s.host})`));
   await checkServerKeyAuth(selectedServer);
   console.log("");
 }
@@ -9083,10 +9149,10 @@ async function showOtherAgents() {
     return;
   }
   console.log("");
-  console.log(dim(`  Your other running agents — pick one to act on  (${PICK_HINT}):`));
+  console.log(dim(`   Your other running agents — pick one to act on  (${PICK_HINT}):`));
   const which = await pickFromList(items.map((i) => i.label), 0);
   if (which < 0) {
-    console.log(dim("  (cancelled)\n"));
+    console.log(dim("   (cancelled)\n"));
     paintBar();
     return;
   }
@@ -9109,10 +9175,10 @@ async function showOtherAgents() {
     { id: "stop", label: "Stop it", note: "ends the run — its unsaved work is lost" },
   ];
   console.log("");
-  console.log(dim(`  ${shortName}  (${PICK_HINT}):`));
+  console.log(dim(`   ${shortName}  (${PICK_HINT}):`));
   const act = await pickFromList(actions.map((a) => a.label + dim(`  · ${a.note}`)), 0);
   if (act < 0) {
-    console.log(dim("  (cancelled)\n"));
+    console.log(dim("   (cancelled)\n"));
     paintBar();
     return;
   }
@@ -9410,9 +9476,9 @@ async function promptNewCodingKey(backendId = "", provider = "") {
   const id = await addCodingKey(name, secret, backendId);
   console.log(
     id
-      ? accent(`  ✓ added ${name || "Unknown"}`) +
+      ? accent(` ✓ added ${name || "Unknown"}`) +
           (backendId && provider ? dim(`  · for ${provider}`) : "") + "\n"
-      : red("  couldn't save the key — try again, or set it in web Settings → Agent\n")
+      : red("   couldn't save the key — try again, or set it in web Settings → Agent\n")
   );
   return id;
 }
@@ -9505,9 +9571,9 @@ async function replaceCodingKey() {
   console.log(
     ok
       ? clearing
-        ? accent(`  ✓ ${ks.model} API key removed`) + dim(" — coding falls back to your local model\n")
-        : accent(`  ✓ ${ks.model} API key saved\n`)
-      : red("  couldn't save the key — try again, or set it in web Settings → Agent\n")
+        ? accent(` ✓ ${ks.model} API key removed`) + dim(" — coding falls back to your local model\n")
+        : accent(` ✓ ${ks.model} API key saved\n`)
+      : red("   couldn't save the key — try again, or set it in web Settings → Agent\n")
   );
   paintBar();
 }
@@ -9537,7 +9603,7 @@ async function manageServerKeys({ role = "coding", backendId, host, model = "" }
     backendId ? fetchServerKeys(backendId) : fetchCodingKeys()
   );
   if (!keys) {
-    console.log(red(`  couldn't read the keys for ${host} — is this machine's API up to date? (\`gu api restart\`)\n`));
+    console.log(red(`   couldn't read the keys for ${host} — is this machine's API up to date? (\`gu api restart\`)\n`));
     paintBar();
     return "";
   }
@@ -9582,7 +9648,7 @@ async function manageServerKeys({ role = "coding", backendId, host, model = "" }
     const fresh = await fetchServerKeys(backendId);
     const added = fresh?.find((k) => k.id === id);
     if (added?.default) {
-      console.log(accent(`  ✓ ${added.name} is ${host}'s default key\n`));
+      console.log(accent(` ✓ ${added.name} is ${host}'s default key\n`));
       paintBar();
       return id;
     }
@@ -9594,7 +9660,7 @@ async function manageServerKeys({ role = "coding", backendId, host, model = "" }
   if (!key.backendId && backendId) {
     const ok = await withPendingStatus(`tying ${key.name} to ${host}`, () => bindKeyToServer(key.id, backendId));
     if (!ok) {
-      console.log(red(`  couldn't tie ${key.name} to ${host} — nothing changed.\n`));
+      console.log(red(`   couldn't tie ${key.name} to ${host} — nothing changed.\n`));
       paintBar();
       return "";
     }
@@ -9613,10 +9679,10 @@ async function applyKeyPick({ role, backendId, host, key }) {
   let scope = "folder";
   if (!backendId) {
     await saveFolderKeyPick(role, key.id);
-    console.log(accent(`  ✓ ${folderLabel()}'s ${roleLabel} → ${key.name}`) + dim(`  ····${key.tail}\n`));
+    console.log(accent(` ✓ ${folderLabel()}'s ${roleLabel} → ${key.name}`) + dim(`  ····${key.tail}\n`));
   } else if (key.default) {
     await saveFolderKeyPick(role, "");
-    console.log(accent(`  ✓ ${folderLabel()}'s ${roleLabel} → ${key.name}`) + dim(`  ····${key.tail}  ·  ${host}'s default\n`));
+    console.log(accent(` ✓ ${folderLabel()}'s ${roleLabel} → ${key.name}`) + dim(`  ····${key.tail}  ·  ${host}'s default\n`));
   } else {
     const choices = sk.scopeChoices({ host, role });
     console.log("\n" + GUTTER + dim(`Use ${key.name} for…`));
@@ -9630,16 +9696,16 @@ async function applyKeyPick({ role, backendId, host, key }) {
     if (scope === "default") {
       const ok = await withPendingStatus(`making ${key.name} the default for ${host}`, () => setServerDefaultKey(backendId, key.id));
       if (!ok) {
-        console.log(red("  couldn't change the default — nothing changed.\n"));
+        console.log(red("   couldn't change the default — nothing changed.\n"));
         paintBar();
         return "";
       }
       // This folder follows the default now, rather than pinning a copy of it.
       await saveFolderKeyPick(role, "");
-      console.log(accent(`  ✓ ${key.name} is now ${host}'s default`) + dim(`  ····${key.tail}  ·  every folder without its own pick\n`));
+      console.log(accent(` ✓ ${key.name} is now ${host}'s default`) + dim(`  ····${key.tail}  ·  every folder without its own pick\n`));
     } else {
       await saveFolderKeyPick(role, key.id);
-      console.log(accent(`  ✓ ${folderLabel()}'s ${roleLabel} → ${key.name}`) + dim(`  ····${key.tail}  ·  this folder only\n`));
+      console.log(accent(` ✓ ${folderLabel()}'s ${roleLabel} → ${key.name}`) + dim(`  ····${key.tail}  ·  this folder only\n`));
     }
   }
   if (role === "coding") {
@@ -9693,7 +9759,7 @@ async function deleteServerKeyFlow({ role, key, host, folderKeyId, lastOfServer
   }
   const out = await deleteCodingKey(key.id);
   if (!out.ok) {
-    console.log(red("  couldn't delete the key — try again, or remove it in web Settings → Agent\n"));
+    console.log(red("   couldn't delete the key — try again, or remove it in web Settings → Agent\n"));
     paintBar();
     return;
   }
@@ -9702,7 +9768,7 @@ async function deleteServerKeyFlow({ role, key, host, folderKeyId, lastOfServer
     const mine = r === "agent" ? sessionAgentKeyId : r === "embed" ? sessionEmbedKeyId : sessionCodingKeyId;
     if (mine === key.id) await saveFolderKeyPick(r, "");
   }
-  console.log(accent(`  ✓ deleted ${out.name || key.name}`) + dim(`  ····${key.tail}  ·  ${out.remaining} key(s) left\n`));
+  console.log(accent(` ✓ deleted ${out.name || key.name}`) + dim(`  ····${key.tail}  ·  ${out.remaining} key(s) left\n`));
   paintBar();
 }
 
@@ -9970,7 +10036,7 @@ async function approvalPrompt(command) {
       ok = (await runS3Command("/s3")) === true;
     } catch (e) {
       if (!wasPromptCancelled(e)) throw e;
-      console.log(dim("  (cancelled — the file is saved on this computer)\n"));
+      console.log(dim("   (cancelled — the file is saved on this computer)\n"));
     } finally {
       following = wasFollowing;
     }
@@ -11088,6 +11154,7 @@ async function runAgentTurn(history, { attachJobId = "", ownJob = false } = {})
   // whether the user has clicked to expand it. Both reset when the step's Thought changes.
   let fullLiveThought = "";
   let thoughtExpanded = false;
+  let liveExpandable = false; // the live thought line has more to show than fits (set each tick)
   // Task #148: the CURRENT step's raw model output, accumulated in full (fenceThought stops at
   // 400 chars — enough for a one-line label, not for the paragraphs we now print). Bounded so a
   // runaway generation cannot grow without limit; the prose we keep is far shorter anyway.
@@ -11298,6 +11365,9 @@ async function runAgentTurn(history, { attachJobId = "", ownJob = false } = {})
     // "almost done" > "Thinking". When clickable we drive line 1 from the SAME fuller string, so
     // the grey expansion CONTINUES where line 1 was cut off instead of repeating the head.
     const clickable = !!fullLiveThought && !runningCmd && fullLiveThought.length > max;
+    // #350: the click and Ctrl+T ask the SAME question the highlight does. They used to accept
+    // any thought at all, so a line that was already whole took the click and redrew for nothing.
+    liveExpandable = clickable;
     let label, rest;
     if (clickable) {
       // Split at a WORD boundary near the line-1 width so nothing is cut mid-word: the last
@@ -11450,14 +11520,14 @@ async function runAgentTurn(history, { attachJobId = "", ownJob = false } = {})
     if (row < top || row > top + statusContentLines - LEAD_ROWS - WORD_GAP - 2) return;
     // Only meaningful when there IS a fuller thought to reveal; otherwise ignore the click
     // so we don't flicker an empty expand.
-    if (!thoughtExpanded && !(fullLiveThought && !runningCmd)) return;
+    if (!thoughtExpanded && !liveExpandable) return;
     toggleThought();
   };
   // Same expand/collapse, reachable without the mouse: Ctrl+T while a turn runs (see
   // toggleActiveThought). Ignored when there's nothing more to reveal, so it never flickers an
   // empty expansion.
   const toggleThought = () => {
-    if (!thoughtExpanded && !(fullLiveThought && !runningCmd)) return;
+    if (!thoughtExpanded && !liveExpandable) return;
     thoughtExpanded = !thoughtExpanded;
     clearStatus(); // next tick (≤120ms) redraws at the new size; keeps cursor math correct
   };
@@ -12457,7 +12527,10 @@ async function main() {
     // reaches the model over HTTP), so it runs HERE, where the files are, and calls the
     // model machine for tokens. That needs a worker here like any other machine. The earlier
     // build suppressed the daemon entirely, which is why this comment exists.
-    if (!status.running) {
+    // GU_NO_WORKER_AUTOSTART=1 (task #351): the terminal UI tests play a scripted turn from a fake
+    // API — no worker is needed, and one started for a throwaway home outlived the test (on
+    // Windows it would be a scheduled task). Nothing a user sets; without it, as always:
+    if (!status.running && process.env.GU_NO_WORKER_AUTOSTART !== "1") {
       // NO LONGER A QUESTION (task #159, user 2026-08-09: "it need to run otherwise cannot work,
       // so, just notify"). It used to ask, and a "no" was then remembered per folder — but the
       // only workable answer was yes: without a worker nothing claims your turns, so a question
@@ -12471,7 +12544,7 @@ async function main() {
       // meaningfully refuse.
       const steps = await import("./first-run-steps.mjs");
       const step = steps.stepById("daemon");
-      console.log(dim(`  ${step.doing}…`));
+      console.log(dim(`   ${step.doing}…`));
       let started = false;
       let why = "";
       try {
@@ -12479,7 +12552,7 @@ async function main() {
         started = !!r.started;
         localDaemonRunning = started || localDaemonRunning;
         if (started) {
-          console.log(dim(`  started background worker (pid ${r.pid}) · logs ${ctl.LOG_FILE}`));
+          console.log(dim(`   started background worker (pid ${r.pid}) · logs ${ctl.LOG_FILE}`));
         } else {
           why = r.reason || "the worker did not report a pid";
         }
@@ -12493,7 +12566,7 @@ async function main() {
       await recordStepOutcome("daemon", started, why);
       if (!started) {
         console.log(
-          yellow("  the background worker did not start") +
+          yellow("   the background worker did not start") +
             dim(` — ${why}\n  ${step.why}.\n  Try \`gu start\` to see the error in full.\n`)
         );
       }
@@ -12613,7 +12686,12 @@ async function main() {
     // private mode's local API can itself be unconfigured.
     if (p.agentMissing === true) agentMissing = true;
     noModelAtAll = nothingToAnswerWith(p);
-    if (p.ragEnabled !== true) embedMissing = true;
+    if (p.ragEnabled !== true) {
+      embedMissing = true;
+      // …but "off" has two causes, and only the settings can tell them apart (#347). In the
+      // background: startup never waits on it, and the bar is repainted when the answer is in.
+      void syncEmbedWarning({ repair: true });
+    }
     // A RESTORED PICK MUST STILL EXIST. The stored id names a model on the box this folder was
     // last pointed at — and the box can change (`/model` → change the Ollama server), a model
     // can be removed, or the folder can be opened on another machine entirely. Sending a name
@@ -12797,7 +12875,10 @@ async function main() {
         // check used to sit INSIDE the "chat model is down" branch, so a client pointing at
         // a healthy remote model was never offered them and every turn died with
         // "No module named 'smolagents'". Independent now.
-        if (!(await md.hasAgentLibs())) {
+        // GU_NO_AGENT_DEPS_PROMPT=1 (task #351): the terminal UI tests run no agent — on a machine
+        // without the framework every test's throwaway home answered "yes" and sat through a pip
+        // install (found on the Ubuntu server: 21 seconds of downloads, then a timeout).
+        if (process.env.GU_NO_AGENT_DEPS_PROMPT !== "1" && !(await md.hasAgentLibs())) {
           const skipDeps = await loadSessionSkip(resolve(process.cwd()), "skipDepsPrompt");
           if (!skipDeps) {
             console.log(
@@ -12838,7 +12919,7 @@ async function main() {
             } else {
               await saveSessionSkip(resolve(process.cwd()), "skipDepsPrompt", true);
               console.log(
-                dim("  Skipped — install later with `gu doctor`") +
+                dim("   Skipped — install later with `gu doctor`") +
                   dim(clientModeBanner ? " --client" : "") +
                   dim("; I won't ask again in this folder.\n")
               );
@@ -12872,7 +12953,7 @@ async function main() {
               );
           if (skipSetup) {
             console.log(
-              dim("  Set-up skipped (remembered for this folder) — `gu doctor` runs it ") +
+              dim("   Set-up skipped (remembered for this folder) — `gu doctor` runs it ") +
                 dim("any time.\n")
             );
           }
@@ -12900,7 +12981,7 @@ async function main() {
             await ctl2.setClientMode(true);
             const st = await ctl2.daemonStatus();
             console.log(
-              dim("  Skipped — this machine won't SERVE models; the agent still runs here and\n") +
+              dim("   Skipped — this machine won't SERVE models; the agent still runs here and\n") +
                 dim("  calls your model machine over the network. Point the Agent model URL at\n") +
                 dim("  that machine (not 127.0.0.1) — `gu status` shows what it resolves to.\n") +
                 dim("  `gu doctor` sets models up here · `/host-mode` undoes this.\n")
@@ -13078,8 +13159,8 @@ async function main() {
         const started = await tryStartLocalApi();
         console.error(
           started
-            ? dim("  Started it — run `gu` again.\n")
-            : dim("  Start it with:  gu-cli api start") +
+            ? dim("   Started it — run `gu` again.\n")
+            : dim("   Start it with:  gu-cli api start") +
                 dim("\n  If that fails, check MongoDB is running and see ~/.gonext/local-api.log\n")
         );
       }
@@ -13338,7 +13419,12 @@ async function main() {
       // TRANSCRIPT: the echo is written exactly once per submitted line and never repainted,
       // so there is nothing here for a blank to accumulate against.
       // As typed, not as sent: "[Image #1]", never the file path the agent needs (#51).
-      const echo = PROMPT_PAST + imageMarkersForDisplay(line) + "\n\n";
+      // Every row under the first one's text (#348): a pasted multi-line question used to be
+      // echoed with only its first line indented, the rest hard against the left edge.
+      const echo = questionEchoText(imageMarkersForDisplay(line), {
+        width: process.stdout.columns || 80,
+        paintPrompt: () => PROMPT_PAST,
+      }) + "\n\n";
       if (stopNotice) {
         stopNotice = "";
         if (stopNoticeTimer) clearTimeout(stopNoticeTimer);
@@ -13418,7 +13504,10 @@ async function main() {
     }
     if (line === "/embed-model") {
       await chooseEmbedModel();
-      embedMissing = false; // Task #14: an embed model was just chosen → clear its warning
+      // Read back what is SAVED (#347). The warning used to be cleared here whatever happened —
+      // a cancelled menu or a failed save removed it for the session, and it returned at the
+      // next start as if the choice had been forgotten.
+      await syncEmbedWarning({ repair: false });
       continue;
     }
     if (line === "/server") {
@@ -13520,7 +13609,7 @@ async function main() {
       const direct = line === "/rag-local" ? "local" : line === "/rag-cloud" ? "both" : null;
       let chosen = direct;
       if (!chosen) {
-        console.log(dim(`  Where this folder's RAG knowledge base lives  (${PICK_HINT}):`));
+        console.log(dim(`   Where this folder's RAG knowledge base lives  (${PICK_HINT}):`));
         const idx = await pickFromList(
           RAG_STORAGE.map(
             (r) =>
@@ -13531,7 +13620,7 @@ async function main() {
           Math.max(0, RAG_STORAGE.findIndex((r) => r.id === sessionRagStorage))
         );
         if (idx < 0) {
-          console.log(dim("  (cancelled)\n"));
+          console.log(dim("   (cancelled)\n"));
           continue;
         }
         chosen = RAG_STORAGE[idx].id;
@@ -13547,7 +13636,7 @@ async function main() {
       // switching to cloud leaves the local shards on disk unread, and switching to local
       // leaves the S3 ones untouched — either way the new store is built by the next index.
       console.log(
-        accent(`  ✓ RAG storage → ${row.label}`) +
+        accent(` ✓ RAG storage → ${row.label}`) +
           dim(`\n    ${row.note}`) +
           dim("\n    Existing shards are not moved — the new store fills as this folder is re-indexed.\n")
       );
@@ -13654,7 +13743,7 @@ async function main() {
       if (!wasPromptCancelled(e)) throw e;
       // The same words the pickers print, for the same event — a cancel should not read as one
       // thing when the answer was typed and another when it was arrowed to.
-      console.log(dim("  (cancelled)\n"));
+      console.log(dim("   (cancelled)\n"));
       continue;
     }
     // SWITCH TO ANOTHER AGENT (task #211) — chosen with the mouse in showOtherAgents, delivered