@tiens.nguyen/gu-cli 1.0.860 → 1.0.862

package/gu-repl.mjs

@@ -532,6 +532,10 @@ const SPINNERS = [
 // green, cyan, blue, indigo, pink, amber).
 const WAIT_COLORS_256 = [120, 87, 111, 147, 218, 221];
 const color256 = (n, s) => `\x1b[38;5;${n}m${s}\x1b[0m`;
+// The thinking/waiting spinner + playful word (#216): plain white by default, the cycling
+// WAIT_COLORS_256 colour only when this run was started with `gu --color`. White goes through
+// paint(), so NO_COLOR / TERM=dumb still turn it off.
+const waitPaint = (n, s) => (REPL_COLOR ? color256(n, s) : white(s));
 const orange = (s) => color256(208, s); // peak-token label (task #84)
 // Compact token count: 950 → "950", 12300 → "12.3k", 1500000 → "1.5M" (task #83).
 const fmtTokens = (n) => {
@@ -616,7 +620,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] [--license <pro token>]\n" +
+      "Usage: gu [-v|--verbose] [--log] [--color] [--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" +
@@ -625,6 +629,8 @@ if (cliVerdict.kind === "run") {
       "  -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" +
+      "  --color         colour the thinking spinner and its playful word (cycling colours).\n" +
+      "                  Without it they are plain white.\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" +
@@ -649,6 +655,9 @@ const REPL_VERBOSE = argv.includes("-v") || argv.includes("--verbose");
 // 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");
+// --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");
 if (REPL_LOG) {
   const ll = await import("./log-level.mjs");
   await ll.takeLease().catch(() => {});
@@ -4265,8 +4274,8 @@ function pendingStatusRows() {
   const wc = WAIT_COLORS_256[Math.floor(now / 2500) % WAIT_COLORS_256.length];
   const secs = Math.max(0, Math.round((now - _pendingSince) / 1000));
   const line =
-    markLine(color256(wc, glyph)) +
-    color256(wc, `${_pendingLabel}…`) +
+    markLine(waitPaint(wc, glyph)) +
+    waitPaint(wc, `${_pendingLabel}…`) +
     (secs >= 2 ? dim(` (${secs}s)`) : "");
   return [clipVisible(line, width - 1)].concat(
     barPinned() || !process.stdout.isTTY ? [] : footerRows()
@@ -12458,7 +12467,7 @@ async function runAgentTurn(history, { attachJobId = "", ownJob = false } = {})
     for (const wl of expLines) contentRows.push(clipRow(GUTTER + dim(wl)));
     for (let i = 0; i < WORD_GAP; i++) contentRows.push(""); // see WORD_GAP
     contentRows.push(
-      clipRow(`${markLine(color256(wc, glyph))}${color256(wc, `${thinkWord}…`)}${outLabel}`)
+      clipRow(`${markLine(waitPaint(wc, glyph))}${waitPaint(wc, `${thinkWord}…`)}${outLabel}`)
     );
     // PINNED → the block IS frozen rows; unpinned (a window too short to reserve any) there is
     // no zone, so it keeps the old in-place draw at the output position.

package/gu_browser.py

@@ -291,12 +291,42 @@ def should_offer(now=None):
 
 
 def mark_offered():
+    """→ whether the marker was written (a failed write must not mean "offer again", #217)."""
     try:
         os.makedirs(os.path.dirname(_offered_path()), exist_ok=True)
         with open(_offered_path(), "w") as f:
             f.write(str(int(time.time())))
+        return True
     except OSError:
-        pass
+        return False
+
+
+def claim_offer(now=None):
+    """Atomically decide that THIS process may open the store page (#217): True for exactly one of
+    several gu processes racing, and only if it was not opened in the last OFFER_EVERY_S. False when
+    GU_NO_OPEN_APPS is set (tests) or when the marker cannot be written — never offer without a
+    record that we did."""
+    if os.environ.get("GU_NO_OPEN_APPS"):
+        return False
+    lock = _offered_path() + ".lock"
+    try:
+        os.makedirs(os.path.dirname(lock), exist_ok=True)
+        try:
+            if time.time() - os.path.getmtime(lock) > 60:  # a crashed claimer's leftover
+                os.remove(lock)
+        except OSError:
+            pass
+        fd = os.open(lock, os.O_CREAT | os.O_EXCL | os.O_WRONLY)
+    except OSError:
+        return False  # another process holds the claim right now (or no writable folder)
+    try:
+        os.close(fd)
+        return should_offer(now) and mark_offered()
+    finally:
+        try:
+            os.remove(lock)
+        except OSError:
+            pass
 
 
 def open_store_page(url=None, platform=None):
@@ -369,9 +399,8 @@ def ensure_extension(log=None, emit=None, wait_s=120, installs=None):
     installs = find_installs() if installs is None else installs
     state = diagnose(installs)
     opened = False
-    if state == "missing" and should_offer():
-        opened = open_store_page()
-        mark_offered()
+    if state == "missing" and claim_offer():
+        opened = open_store_page()  # claim_offer already recorded it: at most once a day, never per call
     steps = setup_steps(state, installs, opened)
     say = emit or (lambda _t: None)
     for line in steps:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tiens.nguyen/gu-cli",
-  "version": "1.0.860",
+  "version": "1.0.862",
   "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",