@semalt-ai/code 1.8.4 → 1.8.5

package/lib/ui/chat-history.js

@@ -3,6 +3,7 @@
 const { RST, DIM, BOLD, FG_CYAN, FG_GREEN, FG_YELLOW, FG_RED, FG_DARK, FG_GRAY } = require('./ansi');
 const { getCols, stripAnsi } = require('./utils');
 const { UI_THEME, UI_ICONS, TOOL_CATEGORIES } = require('./theme');
+const { summarizeToolResult } = require('./format');
 const writer = require('./writer');
 
 
@@ -147,7 +148,24 @@ class ChatHistory {
       // scrollback via endActivity — when a caller supplies an empty
       // `content` they're signalling that the header is already present
       // and only the expandable output body should render here.
-      if (content) {
+      //
+      // History-loaded tool messages arrive with the raw stored payload as
+      // `content` and no `output` — collapse them through summarizeToolResult
+      // to a single live-activity-style line. Callers that pass `output`
+      // (debug blocks, live-activity error pass-through) keep the legacy
+      // header chrome.
+      if (msg.content && !msg.output) {
+        const summary = safeContent(summarizeToolResult(msg.content));
+        if (summary) {
+          const indicator = msg.isError
+            ? `${UI_THEME.error}${UI_ICONS.error}${RST}`
+            : `${UI_THEME.success}${UI_ICONS.success}${RST}`;
+          const sep = ` ${DIM}·${RST} `;
+          const styled = summary.split(' · ').map((p) => _dimPaths(p)).join(sep);
+          out = `  ${indicator} ${styled}\n`;
+          lineCount = 1;
+        }
+      } else if (content) {
         const indicator = msg.isError
           ? `${UI_THEME.error}${UI_ICONS.error}${RST}`
           : `${UI_THEME.success}${UI_ICONS.success}${RST}`;

package/lib/ui/format.js

@@ -95,11 +95,28 @@ function _categoryLabel(tag) {
   return cat.length >= CATEGORY_WIDTH ? cat.slice(0, CATEGORY_WIDTH) : cat.padEnd(CATEGORY_WIDTH);
 }
 
-function _truncate(text, max) {
+// Display-only normalizer for tool argument text. Collapses every run of
+// whitespace (including \n from heredocs and `\<NL>` line continuations)
+// to a single space and trims the ends, so the summary line never spills
+// across multiple physical rows. Pure: never mutates the value used for
+// execution or sent to the model.
+function normalizeCmdForDisplay(text) {
   if (text == null) return '';
-  const s = String(text);
+  return String(text).replace(/\s+/g, ' ').trim();
+}
+
+// Truncate to fit `max` visible chars, normalizing first so a multi-line
+// command becomes a single visual line. When the cut would land mid-word,
+// back up to the nearest space — but only if it doesn't sacrifice more
+// than ~30% of the available width (otherwise prefer the harder cut).
+function _truncate(text, max) {
+  const s = normalizeCmdForDisplay(text);
   if (s.length <= max) return s;
-  return s.slice(0, Math.max(0, max - 1)) + '…';
+  const cap = Math.max(0, max - 1);
+  let cut = s.slice(0, cap);
+  const lastSpace = cut.lastIndexOf(' ');
+  if (lastSpace > cap * 0.7) cut = cut.slice(0, lastSpace);
+  return cut + '…';
 }
 
 // Verb + target string. Never more than ~80 chars visible — longer URLs,
@@ -199,6 +216,7 @@ function formatToolLine(args) {
     durationMs,
     meta,
     error,
+    noDuration,
   } = args || {};
 
   let glyph;
@@ -218,7 +236,6 @@ function formatToolLine(args) {
   const catColor = (UI_THEME.categories && UI_THEME.categories[_normalizeTag(tag) && (TOOL_CATEGORIES[_normalizeTag(tag)] || 'tool')]) || UI_THEME.accent;
 
   const op = _operation(tag, arg, attrs);
-  const durStr = formatDuration(durationMs || 0) + (status === 'pending' ? '…' : '');
   const metaParts = _metaParts(tag, meta, error);
 
   // Segment-by-segment styling. Each fragment carries its own ANSI codes
@@ -232,16 +249,73 @@ function formatToolLine(args) {
   const segments = [];
   segments.push(`  ${glyphColor}${glyph}${RST} ${catColor}${cat}${RST}`);
   segments.push(`${UI_THEME.default}${op}${RST}`);
-  segments.push(`${durColor}${durStr}${RST}`);
+  if (!noDuration) {
+    const durStr = formatDuration(durationMs || 0) + (status === 'pending' ? '…' : '');
+    segments.push(`${durColor}${durStr}${RST}`);
+  }
   for (const m of metaParts) {
     if (m) segments.push(`${metaColor}${m}${RST}`);
   }
   return segments.join(sep);
 }
 
+// Collapse a stored tool-result string to a single-line summary in the same
+// shape as the live activity bubble ("net · GET https://x · 200 · 256 KB").
+// Pure: no ANSI, no IO, no allocation beyond the returned string. Idempotent
+// — a value that already looks like a summary is returned untouched so the
+// helper survives double-application once a separate display field lands in
+// storage.
+function summarizeToolResult(content) {
+  if (typeof content !== 'string' || !content) return '';
+  const trimmed = content.trim();
+  if (!trimmed) return '';
+
+  if (!trimmed.includes('\n') && trimmed.includes(' · ') && trimmed.length < 200) {
+    return trimmed;
+  }
+
+  // HTTP: agent.js formats as `HTTP <VERB> <url> (<status>):\n<body>`.
+  const httpMatch = content.match(/^HTTP\s+(\w+)\s+(\S+)\s+\((\d+)\)/);
+  if (httpMatch) {
+    const parts = ['net', `${httpMatch[1]} ${httpMatch[2]}`, httpMatch[3]];
+    const bytes = formatBytes(Buffer.byteLength(content, 'utf8'));
+    if (bytes) parts.push(bytes);
+    return parts.join(' · ');
+  }
+
+  // Exec: `Command \`<cmd>\`:\nExit code: <N>\n<output>`. The cmd may span
+  // multiple lines if the user passed a heredoc — non-greedy capture stops
+  // at the first `\`:` boundary, then we collapse whitespace for the preview.
+  const cmdMatch = content.match(/^Command `([\s\S]+?)`:/);
+  const exitMatch = content.match(/^Exit code: (-?\d+)$/m);
+  if (cmdMatch && exitMatch) {
+    const cmd = cmdMatch[1].replace(/\s+/g, ' ').trim();
+    const preview = cmd.length > 60 ? cmd.slice(0, 59) + '…' : cmd;
+    return `exec · ${preview} · exit ${exitMatch[1]}`;
+  }
+
+  const lines = content.split('\n');
+  if (lines.length <= 3 && /^(Wrote|Read|Created|Deleted|Moved|Renamed)\b/.test(lines[0])) {
+    return lines[0];
+  }
+
+  if (lines.length > 1 || content.length > 120) {
+    const firstLine = lines[0] || '';
+    const preview = firstLine.length > 100 ? firstLine.slice(0, 99) + '…' : firstLine;
+    const parts = ['tool', preview];
+    const bytes = formatBytes(Buffer.byteLength(content, 'utf8'));
+    if (bytes) parts.push(bytes);
+    return parts.join(' · ');
+  }
+
+  return trimmed;
+}
+
 module.exports = {
   formatDuration,
   formatBytes,
   formatHttpErrorTag,
   formatToolLine,
+  summarizeToolResult,
+  normalizeCmdForDisplay,
 };

package/lib/ui/terminal.js

@@ -7,6 +7,7 @@
 // the process dies.
 
 const writer = require('./writer');
+const dbg = require('../debug');
 
 let _registered = false;
 
@@ -30,20 +31,24 @@ function registerTerminalCleanup() {
 
   // Normal exit + process.exit(): fires synchronously, last thing Node does.
   // Catches every path that doesn't already manually call teardown.
-  process.on('exit', () => { try { writer.teardown(); } catch {} });
+  process.on('exit', () => {
+    try { writer.teardown(); } catch {}
+    try { dbg.close(); } catch {}
+  });
 
   // Signals that should terminate the app. Cleanup first, then exit with
   // the conventional 128+signum code. In TUI raw mode, Ctrl+C is consumed
   // at the byte level and SIGINT is not delivered, so this handler only
   // trips in non-raw contexts (one-shot commands, readline prompts, etc.).
-  process.on('SIGINT',  () => { try { writer.teardown(); } catch {} process.exit(130); });
-  process.on('SIGTERM', () => { try { writer.teardown(); } catch {} process.exit(143); });
-  process.on('SIGHUP',  () => { try { writer.teardown(); } catch {} process.exit(129); });
+  process.on('SIGINT',  () => { try { writer.teardown(); } catch {} try { dbg.close(); } catch {} process.exit(130); });
+  process.on('SIGTERM', () => { try { writer.teardown(); } catch {} try { dbg.close(); } catch {} process.exit(143); });
+  process.on('SIGHUP',  () => { try { writer.teardown(); } catch {} try { dbg.close(); } catch {} process.exit(129); });
 
   // Last-chance net: if something throws outside a try/catch, still
   // restore terminal state before the stack trace prints.
   process.on('uncaughtException', (err) => {
     try { writer.teardown(); } catch {}
+    try { dbg.close(); } catch {}
     // audit: allowed — crash handler stderr after writer teardown.
     try { console.error(err && err.stack ? err.stack : err); } catch {}
     process.exit(1);
@@ -51,6 +56,7 @@ function registerTerminalCleanup() {
 
   process.on('unhandledRejection', (reason) => {
     try { writer.teardown(); } catch {}
+    try { dbg.close(); } catch {}
     // audit: allowed — crash handler stderr after writer teardown.
     try { console.error(reason && reason.stack ? reason.stack : reason); } catch {}
     process.exit(1);

package/lib/ui/writer.js

@@ -29,6 +29,7 @@
 // can't interleave with another task's writes.
 
 const { stripAnsi, termWidth, truncateVisible } = require('./utils');
+const dbg = require('../debug');
 
 let _queue = Promise.resolve();
 let _liveLines = [];
@@ -228,16 +229,13 @@ function setLive(lines, caret) {
     // The input row's visible position is relative to the bottom of the
     // viewport; it stays still only while the live region keeps a stable
     // height. If a code path sneaks in a different-height setLive call the
-    // input jumps one row. Log under DEBUG_TUI so future drift is visible
-    // without leaking into the TUI itself.
-    if (process.env.DEBUG_TUI &&
-        _previousLiveLineCount !== undefined &&
+    // input jumps one row. Log to the debug file (extended trace) so future
+    // drift is visible without leaking into the TUI itself.
+    if (_previousLiveLineCount !== undefined &&
         _previousLiveLineCount !== _liveLines.length) {
-      try {
-        process.stderr.write(
-          `live region height changed: ${_previousLiveLineCount} → ${_liveLines.length}\n`
-        );
-      } catch {}
+      dbg.logExtended(
+        `[writer] live region height changed: ${_previousLiveLineCount} → ${_liveLines.length}`
+      );
     }
     _previousLiveLineCount = _liveLines.length;
     _writeSync(_HIDE + eraseSeq + _drawLiveSeq() + _positionCaretSeq() + _caretShowSeq());

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@semalt-ai/code",
-  "version": "1.8.4",
+  "version": "1.8.5",
   "description": "Self-hosted AI Coding Assistant CLI",
   "main": "index.js",
   "bin": {