sweet-search 2.5.7 → 2.5.9

package/core/indexing/indexer-utils.js

@@ -123,10 +123,10 @@ export function isVerboseMode() {
 const BAR_WIDTH = 30;
 const LABEL_COL = 17;           // pad "Label:" to this width so every bar's [ ] aligns
 const SUB_BLOCKS = ['', '▏', '▎', '▍', '▌', '▋', '▊', '▉']; // eighth-block partial fills
-const CLEAR_EOL = '\x1b[K';
 const liveBars = new Map();     // label -> { current, total }; insertion order = display order
 let regionLines = 0;            // bar lines currently pinned at the bottom (TTY)
 let lastLoggedPercent = {};
+let deferredLogs = [];          // lines held back while parallel bars run (flushed on commit)
 
 function renderBar(current, total, label) {
   const ratio = total > 0 ? Math.max(0, Math.min(1, current / total)) : 1;
@@ -140,21 +140,34 @@ function renderBar(current, total, label) {
   return `${colors.cyan}${head}[${bar}${empty}] ${pct}% (${current}/${total})${colors.reset}`;
 }
 
-function drawRegion() {
-  let out = regionLines > 0 ? `\x1b[${regionLines}A\r` : '\r';
-  for (const [label, b] of liveBars) out += renderBar(b.current, b.total, label) + CLEAR_EOL + '\n';
-  process.stdout.write(out);
-  regionLines = liveBars.size;
+// (Re)draw the live region in place (the `log-update` pattern). Invariant: the
+// cursor enters and leaves at the END of the last bar line — NO trailing newline
+// — so a redraw never pushes a stale copy of a bar into scrollback. Each redraw
+// moves up to the first region line and erases to end-of-screen (\x1b[J) before
+// rewriting. `aboveLine`, if given, scrolls one permanent line above the bars.
+function regionEscape(aboveLine) {
+  const bars = [...liveBars].map(([l, b]) => renderBar(b.current, b.total, l));
+  let out = '';
+  if (regionLines > 1) out += `\x1b[${regionLines - 1}A`; // up to the first region line
+  out += '\r\x1b[J';                                       // col 0, erase region + everything below
+  if (aboveLine != null) out += aboveLine + '\n';          // permanent line above the bars
+  out += bars.join('\n');                                   // bars — no trailing newline
+  regionLines = bars.length;
+  return out;
 }
 
 export function log(message, color = 'reset') {
   if (quietMode) return;
   const line = `${colors[color]}${message}${colors.reset}`;
   if (regionLines > 0 && process.stdout.isTTY) {
-    // Print the line above the pinned bars, then redraw the bars below it.
-    let out = `\x1b[${regionLines}A\r${line}${CLEAR_EOL}\n`;
-    for (const [label, b] of liveBars) out += renderBar(b.current, b.total, label) + CLEAR_EOL + '\n';
-    process.stdout.write(out);
+    if (liveBars.size > 1) {
+      // Parallel bars live: defer the line so it can't disturb the region. Any
+      // mid-region print scrolls a stale bar-pair into scrollback. Flushed once
+      // every bar in the region finishes.
+      deferredLogs.push(line);
+      return;
+    }
+    process.stdout.write(regionEscape(line)); // single bar: line above, bar redrawn below
   } else {
     console.log(line);
   }
@@ -172,16 +185,22 @@ export function logProgress(current, total, label) {
     }
     return;
   }
-  // Interactive TTY: update this bar in the live region and redraw.
+  // Interactive TTY: update this bar in the live region and redraw in place.
   liveBars.set(label, { current, total });
-  drawRegion();
+  process.stdout.write(regionEscape());
   // Once every live bar is complete, commit the region (leave it on screen).
   let allDone = true;
   for (const b of liveBars.values()) if (b.current < b.total) { allDone = false; break; }
   if (allDone) {
+    process.stdout.write('\n');             // move below the finished bars (cursor was at their end)
     for (const k of liveBars.keys()) lastLoggedPercent[k] = 0;
     liveBars.clear();
     regionLines = 0;
+    // Flush lines deferred while parallel bars ran — now below the finished bars.
+    if (deferredLogs.length) {
+      for (const l of deferredLogs) console.log(l);
+      deferredLogs = [];
+    }
   }
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sweet-search",
-  "version": "2.5.7",
+  "version": "2.5.9",
   "description": "Sweet Search - SOTA Hybrid Code Search Engine with WASM CatBoost Query Router, Semantic/Lexical/Structural Search, and Multilingual Support",
   "type": "module",
   "main": "core/search/sweet-search.js",
@@ -163,12 +163,12 @@
   },
   "optionalDependencies": {
     "usearch": "^2.21.4",
-    "@sweet-search/native-darwin-arm64": "2.5.7",
-    "@sweet-search/native-darwin-x64": "2.5.7",
-    "@sweet-search/native-linux-arm64-gnu": "2.5.7",
-    "@sweet-search/native-linux-arm64-gnu-cuda": "2.5.7",
-    "@sweet-search/native-linux-x64-gnu": "2.5.7",
-    "@sweet-search/native-linux-x64-gnu-cuda": "2.5.7"
+    "@sweet-search/native-darwin-arm64": "2.5.9",
+    "@sweet-search/native-darwin-x64": "2.5.9",
+    "@sweet-search/native-linux-arm64-gnu": "2.5.9",
+    "@sweet-search/native-linux-arm64-gnu-cuda": "2.5.9",
+    "@sweet-search/native-linux-x64-gnu": "2.5.9",
+    "@sweet-search/native-linux-x64-gnu-cuda": "2.5.9"
   },
   "engines": {
     "node": ">=18.0.0"

package/scripts/postinstall-banner.js

@@ -2,33 +2,55 @@
 /**
  * postinstall — print a short "what next" message after install.
  *
- * Deliberately plain text: during `npm install` npm writes its own progress
- * spinner to the terminal CONCURRENTLY with this script, which would interrupt
- * any graphics/animation escape sequence mid-stream and leak its payload as
- * garbage text. So the rich animated banner is reserved for `sweet-search init`
- * and `sweet-search index` (where we own the TTY); install just prints a clean,
- * escape-light pointer. Best-effort; always exits 0 so it can't fail an install.
+ * npm pipes postinstall stdout (and swallows it for `-g`), so we write to the
+ * controlling terminal (/dev/tty) directly — same reason the message vanished
+ * when we used process.stdout. It is deliberately PLAIN TEXT (no graphics /
+ * animation): during `npm install` npm writes its own spinner to the same
+ * terminal concurrently, which would corrupt a large chunked escape sequence
+ * (the base64 garbage we saw) — a short text line is atomic and safe. The rich
+ * animated banner is reserved for `sweet-search init` / `index`, where we own
+ * the TTY. Best-effort; never throws.
  */
 import process from 'node:process';
+import { openSync, writeSync, closeSync } from 'node:fs';
 
 function run() {
   const env = process.env;
   if (env.NO_BANNER || env.SWEET_SEARCH_NO_BANNER) return;
+
+  // Choose a real-terminal sink: stdout if it's already a TTY (foreground
+  // scripts), otherwise the controlling terminal. Windows has no /dev/tty →
+  // the banner shows on `init`/`index` instead.
+  let fd = -1;
+  const useStdout = !!process.stdout.isTTY;
+  if (!useStdout) {
+    if (process.platform === 'win32') return;
+    try { fd = openSync('/dev/tty', 'w'); } catch { return; } // no controlling terminal → skip
+  }
+
+  const c = (n, s) => `\x1b[${n}m${s}\x1b[0m`;
+  // SWEET SEARCH half-block wordmark (kept in sync with core/search/cli-decoration.js).
+  const L1 = '█▀▀ █ █ █ █▀▀ █▀▀ ▀█▀  █▀▀ █▀▀ ▄▀▄ █▀▄ █▀▀ █▄█';
+  const L2 = '▄▄█ ▀▄█▄▀ ██▄ ██▄  █   ▄▄█ ██▄ █▀█ ██▄ █▄▄ █▀█';
+  const msg = [
+    '',
+    `  ${c('1;38;5;213', L1)}`,
+    `  ${c('1;38;5;213', L2)}`,
+    '',
+    `  ${c('1', 'Get started:')}`,
+    `    ${c('36', 'sweet-search init')}        set up the current project`,
+    `    ${c('36', 'sweet-search index')}       build the search index`,
+    `    ${c('36', 'sweet-search "query"')}     search your code`,
+    `  ${c('2', '(installed locally? prefix with')} ${c('2;36', 'npx')}${c('2', ')')}`,
+    '',
+    '',
+  ].join('\n');
+
   try {
-    const c = (n, s) => (process.stdout.isTTY ? `\x1b[${n}m${s}\x1b[0m` : s);
-    const lines = [
-      '',
-      `  ${c('1;38;5;213', 'sweet-search')} installed ${c('2', '— SOTA hybrid code search')}`,
-      '',
-      `  ${c('1', 'Get started:')}`,
-      `    ${c('36', 'sweet-search init')}        set up the current project`,
-      `    ${c('36', 'sweet-search index')}       build the search index`,
-      `    ${c('36', 'sweet-search "query"')}     search your code`,
-      `  ${c('2', '(installed locally? prefix with')} ${c('2;36', 'npx')}${c('2', ', e.g. `npx sweet-search init`)')}`,
-      '',
-    ];
-    process.stdout.write(lines.join('\n') + '\n');
-  } catch { /* never break an install */ }
+    if (useStdout) process.stdout.write(msg);
+    else { writeSync(fd, msg); }
+  } catch { /* best-effort */ }
+  finally { if (fd >= 0) { try { closeSync(fd); } catch { /* noop */ } } }
 }
 
 run();