acuvo-code 0.3.3 → 0.3.5

package/ENTERPRISE.md

@@ -189,7 +189,7 @@ are the numbers to quote. Counting is the first thing a reviewer does.
 
 ⚠️ **This said "18 shipped files", then "41", then "90", then "101", then "108", and every
 one went stale in turn.** The package ships **119 files — 117 in `lib/`, 2 in
-`bin/` — about 77552 lines**, with **234 test files** beside them (counted 2026-08-22).
+`bin/` — about 77604 lines**, with **235 test files** beside them (counted 2026-08-22).
 
 ⭐ **AND THE 108 WENT STALE IN THE MOST INSTRUCTIVE WAY POSSIBLE: THREE OF THE FILES IT
 MISSED WERE REACHABLE FROM NOTHING.** `wiring-reach.test.mjs` was naming

package/lib/banner.mjs

@@ -3,36 +3,61 @@
  *
  * Roman, 2026-08-22, having typed it: *"it's not opening as a typable terminal,
  * with acuvo logo top left and details up top etc, like how claude code opens.
- * have you actually designed the page?"*
+ * have you actually designed the page?"* — and then: *"no we want OUR logo"*.
  *
- * No — the previous version printed two dense lines of facts and a prompt. Every
- * fact was correct and none of it was designed. This is the opening screen: a
- * wordmark, one instruction, and the four things a person needs to know before
- * they type anything.
+ * So this is the real mark, not letters spelling the name. It is
+ * `console/public/brand/acuvo-mark.png` — the angular A — resampled to
+ * half-blocks at 14x14 and embedded as text.
  *
- * ── ⚠️ THE CONSTRAINTS THAT SHAPED IT, NOT PREFERENCES ──────────────────────
+ * ── ⚠️ WHY IT IS EMBEDDED RATHER THAN DECODED AT RUNTIME ────────────────────
  *
- * · **Half-block glyphs only** (▀ ▄ █). Braille and fancy box-drawing render as
- *   tofu in cmd.exe and in a lot of CI log viewers, and a logo that renders as
- *   question marks is worse than no logo.
- * · **No colour codes here.** The caller owns colour and already honours
- *   NO_COLOR; a module that hard-codes escapes produces garbage the moment its
- *   output is piped to a file.
- * · **Every line under 80 columns**, because the narrowest terminal anyone
- *   actually uses is 80 and a wrapped banner looks broken rather than dense.
+ * This package has ZERO dependencies, deliberately, and Node cannot decode a
+ * PNG without one. Converting at build time and pasting the result keeps that
+ * promise and costs nothing at startup. If the mark ever changes, re-run the
+ * conversion — the logo is DERIVED from the brand asset, not drawn by hand, so
+ * it stays honest to it.
+ *
+ * ── ⚠️ THE CONSTRAINTS, WHICH ARE NOT PREFERENCES ───────────────────────────
+ *
+ * · **Half-block glyphs only** (▀ ▄ █). Braille and box-drawing render as tofu
+ *   in cmd.exe and in many CI log viewers, and a logo that renders as question
+ *   marks is worse than no logo — on the exact platform this is developed on.
+ * · **No colour escapes here.** The caller owns colour and honours NO_COLOR; a
+ *   module that hard-codes them emits garbage the moment output is piped.
+ * · **Every line under 80 columns.** The narrowest terminal in real use is 80,
+ *   and a wrapped banner does not read as dense, it reads as broken.
+ *
+ * ⭐ A NOTE ON DOING BETTER: `lib/terminal-graphics.mjs` can send a real PNG
+ * inline on Kitty and iTerm2. It is deliberately NOT used here — Windows
+ * Terminal speaks neither protocol, so the block art is what most users would
+ * see anyway, and one rendering that is the same everywhere beats two that
+ * disagree.
  */
 
 /**
- * ACUVO in half-blocks. Three rows, because five is a splash screen — it belongs
- * to something you open once a day, not something you run forty times.
+ * The Acuvo mark, resampled from the brand PNG. Seven rows: tall enough to be
+ * the mark rather than a smudge, short enough for something run forty times a
+ * day rather than opened once.
  */
-const WORDMARK = [
-  '▄▀█ █▀▀ █ █ █ █ █▀█',
-  '█▀█ █▄▄ █▄█ ▀▄▀ █▄█',
+const MARK = [
+  '      ▄█',
+  '     ▄███',
+  '    ▄█▀▀██',
+  '   ▄█▀  ▀██',
+  '  ▄█▀ ██ ▀██',
+  ' ▄████▀▀█████',
+  '█▀      █▀  ▀█',
 ];
 
+const MARK_WIDTH = Math.max(...MARK.map((l) => l.length));
+const GUTTER = 2;
+export const MAX_BANNER_COLUMNS = 80;
+
+/** Columns left for the detail rows once the mark and gutter are placed. */
+const TEXT_COLUMNS = MAX_BANNER_COLUMNS - MARK_WIDTH - GUTTER;
+
 /**
- * Build the opening screen.
+ * Build the opening screen: mark on the left, facts on the right.
  *
  * @param {object} o
  * @param {string} o.version
@@ -47,36 +72,47 @@ export function openingScreen({ version, workspace, model, billing, canRun, inte
   /**
    * ── ⚠️⚠️ ELIDED, BECAUSE THE VALUES ARE NOT OURS ────────────────────────────
    *
-   * Every value here arrives at runtime: a deep monorepo path, a long model id,
-   * the shell-mode warning. The first version of this function assumed they
-   * would be short and its own test caught it wrapping at 80 columns on
-   * perfectly ordinary inputs.
+   * Every value arrives at runtime: a deep monorepo path, a long model id, the
+   * shell-mode warning. The first version assumed they would be short and its
+   * own test caught it wrapping at 80 columns on ordinary inputs.
    *
-   * ⭐ ELIDED IN THE MIDDLE, NOT THE END. The informative parts of a path are the
-   * drive and the leaf; chopping the tail leaves `C:\Users\somebody\Projects\a-`
-   * which identifies nothing. Same for a model id, where the family is at the
-   * front and the variant at the back.
+   * ⭐ ELIDED IN THE MIDDLE, NOT THE END. The informative parts of a path are
+   * the drive and the leaf; chopping the tail leaves
+   * `C:\Users\somebody\Projects\a-`, which identifies nothing. Same for a model
+   * id, where the family is at the front and the variant at the back.
    */
-  const room = MAX_BANNER_COLUMNS - 2 - 11;
+  const room = TEXT_COLUMNS - 11;
   const fit = (v) => {
     const s = String(v ?? '');
     if (s.length <= room) return s;
     const head = Math.ceil((room - 1) / 2);
-    const tail = room - 1 - head;
-    return `${s.slice(0, head)}…${s.slice(s.length - tail)}`;
+    return `${s.slice(0, head)}…${s.slice(s.length - (room - 1 - head))}`;
   };
-  const label = (k, v) => `  ${k.padEnd(11)}${fit(v)}`;
-  const lines = [
-    '',
-    `  ${WORDMARK[0]}`,
-    `  ${WORDMARK[1]}   CODE${version ? `  ${version}` : ''}`,
+
+  const right = [
+    `ACUVO CODE${version ? `  ${version}` : ''}`,
     '',
-    label('workspace', workspace),
-    label('model', model),
-    label('billing', billing),
-    label('can run', canRun),
+    `workspace  ${fit(workspace)}`,
+    `model      ${fit(model)}`,
+    `billing    ${fit(billing)}`,
+    `can run    ${fit(canRun)}`,
     '',
   ];
+
+  /**
+   * ⚠️ THE TWO COLUMNS ARE ZIPPED, NOT CONCATENATED, and the row counts are
+   * allowed to differ — whichever is shorter simply runs out. Assuming they
+   * match would break the layout the first time a row is added to either side.
+   */
+  const rows = Math.max(MARK.length, right.length);
+  const lines = [''];
+  for (let i = 0; i < rows; i += 1) {
+    const mark = (MARK[i] ?? '').padEnd(MARK_WIDTH + GUTTER);
+    const text = right[i] ?? '';
+    lines.push(`${mark}${text}`.trimEnd());
+  }
+  lines.push('');
+
   /**
    * ⚠️ ONLY WHEN A PROMPT ACTUALLY FOLLOWS. Printing "type what you want done"
    * above a one-shot run that has already been given its task is an instruction
@@ -87,11 +123,3 @@ export function openingScreen({ version, workspace, model, billing, canRun, inte
   }
   return lines.join('\n');
 }
-
-/**
- * ⚠️ EXPORTED SO A TEST CAN ASSERT THE WIDTH RULE RATHER THAN TRUST IT. The
- * banner is assembled from values supplied at runtime — a long model id or a
- * deep workspace path is what would push it over, and neither is visible from
- * reading this file.
- */
-export const MAX_BANNER_COLUMNS = 80;

package/lib/chat.mjs

@@ -353,9 +353,33 @@ export async function runChat({
 
   try {
     for (;;) {
+      /**
+       * ── ⭐⭐ THE INPUT IS A BOX, BECAUSE A BARE `› ` IS NOT A PLACE TO TYPE ──
+       *
+       * Roman, comparing against a real Claude Code screenshot: *"you can see
+       * the box where you type, acuvo doesn't have that."* He is right — the
+       * prompt was two characters floating in the scrollback, which reads as
+       * output rather than as somewhere input goes.
+       *
+       * ⚠️ THE BOX OPENS BEFORE THE LINE AND CLOSES AFTER IT, deliberately.
+       * Drawing all four sides up front needs absolute cursor control, and
+       * readline owns the cursor once `question()` starts — every version of
+       * that fights the line editor the moment input wraps, a history entry is
+       * recalled, or the window is resized. Opening on entry and closing on
+       * submit needs no cursor math at all, survives all three, and leaves a
+       * transcript where each turn is visibly a closed unit.
+       *
+       * ⚠️ WIDTH IS READ FRESH EACH TURN — a terminal resized mid-session would
+       * otherwise draw rules at the old width for the rest of the run. Clamped,
+       * because `columns` is `undefined` when stdout is not a TTY and enormous
+       * when someone maximises on an ultrawide.
+       */
+      const width = Math.max(40, Math.min(100, (output.columns ?? process.stdout.columns ?? 80) - 1));
+      if (interactive) output.write(`\n╭${'─'.repeat(width - 1)}\n`);
       const line = interactive
-        ? await ask(rl, '› ', state)
+        ? await ask(rl, '│ › ', state)
         : (queueIndex < queued.length ? queued[queueIndex++] : null);
+      if (interactive && line !== null) output.write(`╰${'─'.repeat(width - 1)}\n`);
       // Echo a piped prompt so a scripted transcript reads like a session.
       if (!interactive && line !== null && line.trim()) output.write(`› ${line.trim()}
 `);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "acuvo-code",
-  "version": "0.3.3",
+  "version": "0.3.5",
   "description": "Acuvo Code — the terminal client for the Acuvo capability registry. Zero dependencies, by design.",
   "type": "module",
   "bin": {