@tiens.nguyen/gu-cli 1.0.898 → 1.0.900

package/gu-repl.mjs

@@ -1053,7 +1053,6 @@ const COMMANDS = [
   // already has for exactly this — /quit, /rag-local.
   // /host-mode is gone (owner, 11-07): modes are chosen at setup; `gu clear` sets up again.
   // /undo is HIDDEN_COMMANDS now — still works when typed.
-  { name: "/server", desc: "pick a deployment server (host/user) for this session" },
   { name: "/proxy", usage: "/proxy [show|set|on|off|clear|test]", desc: "route MODEL calls through an HTTP proxy (local calls always go direct)" },
   { name: "/s3", usage: "/s3 [test]", desc: "set up YOUR S3 bucket (RAG index, and the PDFs/downloads gu makes) and check the AWS keys work — without it, files are saved on this computer" },
   { name: "/context-window", usage: "/context-window [tokens|auto]", desc: "tell gu how big the coding model's context is — hosted models cannot be probed" },
@@ -7171,7 +7170,6 @@ async function useCoderInThisFolder({ ps, rows, beforeChoices, url, host, locali
     console.log(GUTTER + dim("restart it with `gu api restart`, then add that provider again.\n"));
   }
   sessionCodingModel = offered.id;
-  setModelFallback(null); // #343
   sessionCodingLabel = model;
   sessionCodingUrl = url;
   // …and the host the bar puts in front of it (#263): left alone, the PREVIOUS coder's
@@ -7180,6 +7178,7 @@ async function useCoderInThisFolder({ ps, rows, beforeChoices, url, host, locali
   // The folder follows the SERVER's key (task #245) — its default, which a first key already is.
   sessionCodingKeyId = "";
   sessionApiKeyTail = key ? key.slice(-4) : "";
+  setModelFallback(null); // #343 — AFTER the name, server and tail: it repaints the bar (GH #397)
   keyChoicePending = false;
   await saveSessionCoding(cwd);
   await saveSessionCodingKeyId(cwd);
@@ -7202,7 +7201,12 @@ async function useCoderInThisFolder({ ps, rows, beforeChoices, url, host, locali
   // The bar still said "coding model not set · /model" under the success line of the FIRST model
   // added on a fresh install, until the next turn repainted it (GH #397).
   noModelAtAll = after?.payload ? nothingToAnswerWith(after.payload) : false;
-  if (after?.payload) agentMissing = after.payload.agentMissing === true;
+  if (after?.payload) {
+    agentMissing = after.payload.agentMissing === true;
+    // The first model added IS the account default now. Startup read the default while there was
+    // none, so picking "(default)" later left the bar with no model name at all (GH #397).
+    seedDefaultCoder(after.payload);
+  }
   if (barPinned()) paintBar();
 }
 
@@ -7462,7 +7466,6 @@ async function applyChosenCoder(chosen, { def = "", tag = "", quick = false } =
     tail: sessionApiKeyTail,
   };
   sessionCodingModel = chosen.id === def ? "" : chosen.id;
-  setModelFallback(null); // #343: the warning named the coder that was just replaced
   // Remember the friendly name for the token footer, which should never show a raw
   // "<kind>::<model>" wire id.
   sessionCodingLabel = chosen.id === def ? "" : chosen.model;
@@ -7472,6 +7475,12 @@ async function applyChosenCoder(chosen, { def = "", tag = "", quick = false } =
   // The server goes WITH the model, before any await (#263): set after the save below, the bar
   // could repaint in between and show the NEW model under the OLD server for a moment.
   activeCoderWhere = coderWhere(chosen.kind, chosen.url);
+  // The old provider's key tail must not sit beside the new model while the key is looked up (it
+  // was shown for a moment: Kimi with Qwen's tail). Blank until coderKeyState answers below.
+  sessionApiKeyTail = "";
+  // AFTER the fields above: this repaints the bar, and between the id and its name it showed the
+  // raw "<row id>::<model>" wire id for a frame (GH #397). #343: the warning named the old coder.
+  setModelFallback(null);
   // WRITTEN NOW, not at the end of the next turn: `/model` then `/exit` is how the choice is
   // made and then tested, and that path never reaches saveSession.
   await saveSessionCoding(resolve(process.cwd()));
@@ -9827,39 +9836,6 @@ async function agentModelOnServer({ back = false, start = "menu" } = {}) {
   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
-// ssh-copy-id if not). BatchMode=yes = never prompt for a password → exit!=0 if no key.
-async function checkServerKeyAuth(s) {
-  try {
-    const ok = await new Promise((resolve) => {
-      const p = spawn(
-        "ssh",
-        [
-          "-o", "BatchMode=yes",
-          "-o", "ConnectTimeout=6",
-          "-o", "StrictHostKeyChecking=accept-new",
-          `${s.user}@${s.host}`,
-          "true",
-        ],
-        { stdio: "ignore" }
-      );
-      p.on("exit", (code) => resolve(code === 0));
-      p.on("error", () => resolve(false));
-    });
-    if (ok) console.log(dim("    ✓ key auth works — deploys will connect without a password."));
-    else
-      console.log(
-        yellow("   key auth not set up yet.") +
-          dim(` Run once in your terminal:  `) +
-          `ssh-copy-id ${s.user}@${s.host}`
-      );
-  } catch {
-    /* best-effort check — never block the picker */
-  }
-}
-
-// /server — pick a deployment target (from the web app's Servers registry) for this
-// session. Host/user only, never a secret; the agent deploys here with KEY auth (#69).
 /**
  * `/proxy` — configure the outbound proxy for MODEL calls (task #160).
  *
@@ -10248,53 +10224,6 @@ async function runProxyCommand(line) {
   }
 }
 
-async function chooseServer() {
-  let servers;
-  try {
-    const res = await withPendingStatus("loading servers", () =>
-      fetch(`${apiBase}/api/worker/deploy-servers`, {
-        headers: { "X-Worker-Key": workerKey },
-        signal: AbortSignal.timeout(API_CALL_TIMEOUT_MS),
-      })
-    );
-    const data = await res.json().catch(() => ({}));
-    if (!res.ok) {
-      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`));
-    return;
-  }
-  if (servers.length === 0) {
-    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
-  // a "none" row to clear the selection. Pre-highlight the current pick if there is one.
-  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}):`));
-  const idx = await pickFromList(labels, activeIdx >= 0 ? activeIdx : 0);
-  if (idx < 0) {
-    console.log(dim("   (cancelled)\n"));
-    return;
-  }
-  if (idx === servers.length) {
-    // the trailing "none" row
-    selectedServer = null;
-    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})`));
-  await checkServerKeyAuth(selectedServer);
-  console.log("");
-}
-
 /**
  * Ask for the coding model's API key if the resolved coder needs one and none is stored.
  *
@@ -14543,6 +14472,20 @@ async function main() {
       // payload fields, which key-choice-regression.test.mjs forbids for exactly that reason.
       activeCoderWhere = whereOfActiveCoder(probe?.codingChoices, sessionCodingModel);
     }
+    // THE KEY TAIL BESIDE THE MODEL, at startup too (GH #397). It was read only when the folder had
+    // its own key choice, so the normal case — the server's default key — showed a model with no
+    // key until the first /model switch.
+    if (!sessionApiKeyTail) {
+      try {
+        const choices = Array.isArray(probe?.codingChoices) ? probe.codingChoices : [];
+        const active = choices.find((c) => c.id === sessionCodingModel) ?? choices.find((c) => c.isDefault);
+        const { codingNeedsKey } = await import("./coding-key-prompt.mjs");
+        if (active && codingNeedsKey(active.kind, active.url)) {
+          const kd = await fetchCodingKey(active.backendId || "");
+          sessionApiKeyTail = kd?.key ? kd.key.slice(-4) : "";
+        }
+      } catch { /* the bar works without the tail */ }
+    }
     // Task #133: no RAG storage question any more — the terminal ALWAYS keeps the index on
     // this machine. `/rag-cloud` opts into additionally mirroring it to the user's S3 bucket.
     // Startup banner (task #131): the gu logo on the left, compact status facts on the
@@ -15479,11 +15422,6 @@ async function main() {
       await syncEmbedWarning({ repair: false });
       continue;
     }
-    if (line === "/server") {
-      await chooseServer();
-      await saveSession(cwd, history); // remember the pick for this folder
-      continue;
-    }
     if (line === "/full-log" || line.startsWith("/full-log ")) {
       await runFullLogCommand(line);
       continue;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tiens.nguyen/gu-cli",
-  "version": "1.0.898",
+  "version": "1.0.900",
   "description": "gu: the coding agent in your terminal. It reads your repo, makes the change and runs the tests, with your own models (Ollama or any OpenAI-compatible provider). Free installs keep the agent, its API and its database on your machine. macOS, Linux and Windows.",
   "keywords": [
     "ai",