@tiens.nguyen/gu-cli 1.0.865 → 1.0.866

package/README.md

@@ -35,10 +35,11 @@ meaning the whole stack runs on this machine.
 ## The `gu` REPL
 
 ```
-gu [-v|--verbose] [--log] [--license <pro token>]
+gu [-v|--verbose] [--log] [--name <text>] [--license <pro token>]
 ```
 
 - `-v, --verbose` shows RAG index-file activity in the bottom bar.
+- `--name <text>` draws `<text>` instead of the gu logo in the startup banner (this run only). Letters, digits, space, `-` and `_` become block art; anything else, or a name over 12 characters, is shown as plain text.
 - `--log` logs this run in detail to `~/.gonext/worker.log`. Without it only starts, stops and errors are logged.
 - Ctrl-C stops following a turn.
 

package/banner-name.mjs

@@ -0,0 +1,110 @@
+/**
+ * `gu --name <text>` (#224): the startup banner draws <text> instead of the "gu" mark.
+ *
+ * Same material as the built-in mark: half-block elements only (▀ ▄ █), so it renders on any
+ * terminal without a font to install. Two rows of x-height, plus a third row used only by letters
+ * with a descender (g j p q y). Glyphs are separated by one blank column.
+ *
+ * ANYTHING THE FONT CANNOT DRAW (accents, emoji, punctuation) is shown as plain text on a single
+ * row instead of being dropped — a name must never silently turn into a different name.
+ */
+
+const G = {
+  a: ["█▀▀█", "█▀▀█"],
+  b: ["█▀▀▄", "█▄▄▀"],
+  c: ["█▀▀▀", "█▄▄▄"],
+  d: ["▄▀▀█", "▀▄▄█"],
+  e: ["█▀▀█", "█▄▄▄"],
+  f: ["█▀▀▀", "█▀▀ "],
+  g: ["█▀▀█", "█▄▄█", "▄▄▄█"],
+  h: ["█  █", "█▀▀█"],
+  i: ["▄", "█"],
+  j: ["  █", "  █", "▄▄█"],
+  k: ["█ ▄▀", "█▀▄ "],
+  l: ["█  ", "█▄▄"],
+  m: ["█▀▄▀█", "█   █"],
+  n: ["█▀▀█", "█  █"],
+  o: ["█▀▀█", "█▄▄█"],
+  p: ["█▀▀█", "█▀▀▀", "▀   "],
+  q: ["█▀▀█", "█▄▄█", "   █"],
+  r: ["█▀▀█", "█▀▀▄"],
+  s: ["▄▀▀▀", "▄▄▄▀"],
+  t: ["▀█▀", " █ "],
+  u: ["█  █", "█▄▄█"],
+  v: ["█  █", "▀▄▄▀"],
+  w: ["█   █", "█▄█▄█"],
+  x: ["▀▄▀", "▄▀▄"],
+  y: ["█  █", "▀▄▄█", "▄▄▄▀"],
+  z: ["▀▀▀█", "█▄▄▄"],
+  0: ["█▀▀█", "█▄▀█"],
+  1: ["▄█", " █"],
+  2: ["▄▀▀█", "█▄▄▄"],
+  3: ["▀▀▀█", "▄▄▄█"],
+  4: ["█  █", "▀▀▀█"],
+  5: ["█▀▀▀", "▄▄▄█"],
+  6: ["█▀▀▀", "█▄▄█"],
+  7: ["▀▀▀█", "  █ "],
+  8: ["▄▀▀▄", "▀▄▄▀"],
+  9: ["█▀▀█", "▀▀▀█"],
+  "-": ["   ", "▄▄▄"],
+  _: ["    ", "▄▄▄▄"],
+  " ": ["  ", "  "],
+};
+
+/** Longest name accepted at all; longer is cut (and the cut is reported by the caller). */
+export const NAME_MAX = 24;
+/** Most characters drawn as block art; a longer name is shown as plain text. */
+export const ART_MAX = 12;
+
+/** What the user typed → what is shown: trimmed, control chars removed, capped. */
+export function cleanName(raw) {
+  return String(raw ?? "")
+    .replace(/[\u0000-\u001f\u007f]/g, "")
+    .trim()
+    .slice(0, NAME_MAX);
+}
+
+/** Draw `name` as rows of equal visible width. Returns { rows, width, art }; null for an empty name. */
+export function logoFor(raw, maxWidth = Infinity) {
+  const name = cleanName(raw);
+  if (!name) return null;
+  const lower = name.toLowerCase();
+  const drawable = [...lower].every((ch) => G[ch]);
+  if (drawable && lower.length <= ART_MAX) {
+    const rows = ["", "", ""];
+    [...lower].forEach((ch, i) => {
+      const g = G[ch];
+      const w = Math.max(...g.map((r) => r.length));
+      for (let r = 0; r < 3; r++) {
+        rows[r] += (i ? " " : "") + (g[r] ?? "").padEnd(w);
+      }
+    });
+    // The third row only exists for descenders; drop it when it is all blanks.
+    const out = rows[2].trim() ? rows : rows.slice(0, 2);
+    const width = Math.max(...out.map((r) => r.length));
+    if (width <= maxWidth) return { rows: out.map((r) => r.padEnd(width)), width, art: true };
+  }
+  const text = name.slice(0, Math.max(1, Math.min(name.length, maxWidth)));
+  return { rows: [text], width: text.length, art: false };
+}
+
+/**
+ * Read `--name <x>` / `--name=<x>` from argv. { value } | { missing: true } | {} when absent.
+ * A following token that is itself a flag is NOT taken as the value.
+ */
+export function parseNameArg(argv) {
+  const args = Array.isArray(argv) ? argv : [];
+  for (let i = 0; i < args.length; i++) {
+    const a = String(args[i]);
+    if (a.startsWith("--name=")) {
+      const value = cleanName(a.slice(7));
+      return value ? { value } : { missing: true };
+    }
+    if (a === "--name") {
+      const next = args[i + 1];
+      const value = next === undefined || String(next).startsWith("-") ? "" : cleanName(next);
+      return value ? { value } : { missing: true };
+    }
+  }
+  return {};
+}

package/client-expert-run.mjs

@@ -14,7 +14,7 @@
  * that does not work — and that state is much harder to get out of than an unfinished setup.
  */
 import { execFile, spawn } from "node:child_process";
-import { readFile, writeFile, mkdir, rm, rename } from "node:fs/promises";
+import { readFile, writeFile, mkdir, rm, rename, cp } from "node:fs/promises";
 import { writeFileSync } from "node:fs";
 import { homedir } from "node:os";
 import { join } from "node:path";
@@ -867,6 +867,43 @@ export async function probeApiInstall(version) {
   }
 }
 
+/** Errors Windows raises while another process (antivirus, indexer) holds a handle in a folder. */
+const LOCKED = new Set(["EPERM", "EBUSY", "EACCES", "ENOTEMPTY"]);
+
+/**
+ * Put the unpacked folder at its final path. WINDOWS REFUSES TO RENAME OR DELETE A DIRECTORY
+ * while any process has a handle inside it, and the files `tar` has just written are exactly what
+ * Defender and the indexer open (issue #223: "EPERM: operation not permitted, rename ...incoming").
+ * So retry with backoff, then fall back to copy + remove, then say which folder is locked.
+ * `fs` is injectable for the tests.
+ */
+export async function moveIntoPlace(staging, dir, fs = {}) {
+  const { rename: mv = rename, rm: del = rm, cp: copy = cp, sleep = (ms) => new Promise((r) => setTimeout(r, ms)) } = fs;
+  const tries = fs.tries ?? 10;
+  let last;
+  for (let i = 0; i < tries; i++) {
+    try {
+      await del(dir, { recursive: true, force: true });
+      await mv(staging, dir);
+      return;
+    } catch (e) {
+      last = e;
+      if (!LOCKED.has(e?.code)) throw e;
+      await sleep(Math.min(250 * 2 ** i, 3000));
+    }
+  }
+  try {
+    await del(dir, { recursive: true, force: true });
+    await copy(staging, dir, { recursive: true });
+    await del(staging, { recursive: true, force: true }).catch(() => {});
+  } catch (e) {
+    throw new Error(
+      `Could not move the unpacked API into ${dir} (${e?.code ?? last?.code}): a program is holding a file in it. ` +
+      "Close gu and any terminal or editor open in that folder, pause antivirus scanning for it, and run the install again."
+    );
+  }
+}
+
 /**
  * Download and unpack the published API.
  *
@@ -908,9 +945,8 @@ export async function downloadApiPackage(manifest, { onProgress = () => {} } = {
   // ends in "gzip: stdin: unexpected end of file". A path with no drive letter works for GNU and bsdtar.
   await run("tar", ["xzf", "package.tar.gz", "-C", "."], { cwd: staging });
   await rm(tmpTar, { force: true });
-  await rm(dir, { recursive: true, force: true });
   await mkdir(join(homedir(), ".gonext", "api"), { recursive: true });
-  await rename(staging, dir);
+  await moveIntoPlace(staging, dir);
   onProgress(`installed ${manifest.version} → ${dir}`);
   return { dir, version: manifest.version };
 }

package/gu-repl.mjs

@@ -193,6 +193,7 @@ import { EditCardStream, isEditCardHeader } from "./edit-card-render.mjs";
 // Task #240: what a failed agent-payload call means, and when a rejected key may be recovered
 // from in place instead of ending the session.
 import { payloadError, recoveryPlan } from "./credential-recovery.mjs";
+import { logoFor, parseNameArg } from "./banner-name.mjs";
 import { isNoCodeRetry, splitVerboseLine, stepHasCode } from "./verbose-lines.mjs";
 import {
   createTranscript,
@@ -382,8 +383,9 @@ const vwidth = (s) => String(s).replace(/\x1b\[[0-9;]*m/g, "").length;
 // square, read as a box; with an all-caps "GU AGENT" beside it the banner shouted its name twice.
 // The product is written "gu", so the mark is too: a g with a real descender and a u — a wordmark,
 // one row shorter, and the only coloured thing on the banner.
-const LOGO = ["█▀▀█ █  █", "█▄▄█ █▄▄█", "▄▄▄█"];
-const LOGO_W = 9;
+// `let`: `gu --name <text>` (#224) swaps in that name's art once argv has been read.
+let LOGO = ["█▀▀█ █  █", "█▄▄█ █▄▄█", "▄▄▄█"];
+let LOGO_W = 9;
 // Lay the LOGO and the status rows SIDE BY SIDE (logo left, facts right), or STACK them (logo
 // then facts) when the terminal is too narrow to fit both without wrapping — so the banner is
 // never the soft-wrapped jumble the plain "· … · …" lines used to be (task #131). Side-by-side
@@ -402,7 +404,7 @@ const renderBanner = (statusRows) => {
       out.push(GUTTER + left + gap + (statusRows[i] ?? ""));
     }
   } else {
-    for (const r of LOGO) out.push(GUTTER + accent(r));
+    for (const r of LOGO) out.push(GUTTER + accent(clipVisible(r, cols - 1 - GUTTER.length)));
     for (const r of statusRows) out.push(GUTTER + clipVisible(r, cols - 1 - GUTTER.length));
   }
   return out.join("\n");
@@ -622,7 +624,7 @@ 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] [--log] [--color] [--license <pro token>]\n" +
+      "Usage: gu [-v|--verbose] [--log] [--color] [--name <text>] [--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" +
@@ -633,6 +635,8 @@ if (cliVerdict.kind === "run") {
       "                  stops and errors are logged. Ends when this gu closes.\n" +
       "  --color         colour the thinking spinner and its playful word (cycling colours).\n" +
       "                  Without it they are plain white.\n" +
+      "  --name <text>   show <text> instead of the gu logo in the startup banner\n" +
+      "                  (this run only; letters, digits, space - _ are drawn as art)\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" +
@@ -660,6 +664,21 @@ const REPL_LOG = argv.includes("--log");
 // --color (#216): the thinking spinner and its playful word cycle through colours only on a run
 // started with this flag; by default they are plain white.
 const REPL_COLOR = argv.includes("--color");
+// --name <text> (#224): the startup banner draws <text> in place of the "gu" mark. Per run only.
+{
+  const named = parseNameArg(argv);
+  if (named.missing) {
+    console.error(red("gu: --name needs a name, e.g.  gu --name acme"));
+    process.exit(2);
+  }
+  if (named.value) {
+    const l = logoFor(named.value, Math.max(8, (process.stdout.columns || 80) - 6));
+    if (l) {
+      LOGO = l.rows;
+      LOGO_W = l.width;
+    }
+  }
+}
 if (REPL_LOG) {
   const ll = await import("./log-level.mjs");
   await ll.takeLease().catch(() => {});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tiens.nguyen/gu-cli",
-  "version": "1.0.865",
+  "version": "1.0.866",
   "description": "Gu CLI — the gu terminal plus the local worker that runs agent / OCR / PDF / embedding jobs on your Mac (Ollama / MLX / OpenAI-compatible).",
   "type": "module",
   "license": "MIT",
@@ -101,6 +101,7 @@
     "rag-job-config.mjs",
     "completed-flush.mjs",
     "verbose-lines.mjs",
+    "banner-name.mjs",
     "agent-model-label.mjs",
     "coding-choice-tags.mjs",
     "coding-provider-setup.mjs",