omp-fabric 1.25.20 → 1.25.21

package/CHANGELOG.md

@@ -1,5 +1,14 @@
 # Changelog
 
+## 1.25.21
+
+### Fixed
+
+- **Truncation marker no longer leaks raw JSON into the TUI.** `[[omp-fabric:truncated]]` payloads are now parsed by a host-free `src/core/truncation-marker.ts` and rendered as a styled `output truncated` arc badge at the read, grep, path-list, bash, and non-core `renderBody` sites.
+- **Failed results carry visual weight.** Bash bodies and call titles show a `failed` arc badge with the exit code when `success === false`, in addition to the existing error-colored lines.
+- **Long bash titles stay on one screen.** The title's first command line caps at 120 characters with an ellipsis; the full line is still reachable in the expanded body.
+- **Expanded multicall rows gain a status-colored gutter.** Success, running, and failed rows each get their own gutter color; collapsed rows stay flat and all rows remain width-bounded.
+
 ## 1.25.20
 
 ### Changed

package/dist/chunks/{chunk-5SSQD7TW.js → chunk-7NWMFYD3.js}

@@ -4,8 +4,9 @@ import {
   isFabricAgentToolPreview,
   selectPreviewTextLines,
   shouldSkipWriteDiffBytes,
-  shouldSkipWriteDiffComplexity
-} from "./chunk-WQUMHDOS.js";
+  shouldSkipWriteDiffComplexity,
+  splitTruncationNotice
+} from "./chunk-ESPCTYVX.js";
 import {
   countNewlines,
   resolveShikiTheme
@@ -2744,6 +2745,17 @@ var CONTENT_LANGUAGE_DETECTION_CHARS = positiveEnvInteger(
   "CODE_PREVIEW_CONTENT_LANGUAGE_DETECTION_CHARS",
   5e4
 );
+var bashExitCode = (audit) => {
+  const value = resultDetails(audit)?.exitCode;
+  return typeof value === "number" && Number.isSafeInteger(value) && value !== 0 ? value : void 0;
+};
+var bashFailureArc = (audit, theme) => {
+  if (audit.success !== false) return null;
+  const exitCode = bashExitCode(audit);
+  return arcItemStyled(theme, theme.fg("error", exitCode === void 0 ? "failed" : `failed \xB7 exit ${exitCode}`));
+};
+var BASH_TITLE_COMMAND_CHARS = 120;
+var truncateBashTitleCommand = (line) => line.length <= BASH_TITLE_COMMAND_CHARS ? line : `${line.slice(0, BASH_TITLE_COMMAND_CHARS - 1)}\u2026`;
 var recordOf = (value) => typeof value === "object" && value !== null && !Array.isArray(value) ? value : void 0;
 var stringOf = (value) => typeof value === "string" ? value : void 0;
 var numberOf = (value) => typeof value === "number" && Number.isFinite(value) ? value : void 0;
@@ -2796,6 +2808,12 @@ var resultOutput = (audit) => {
   const record = recordOf(result);
   return stringOf(record?.output) ?? stringOf(record?.text) ?? contentOutput(record?.content);
 };
+var stripTruncationNotice = (value) => {
+  const notice = splitTruncationNotice(value);
+  if (!notice) return { output: value };
+  const trimmed = notice.text.endsWith("\n") ? notice.text.slice(0, -1) : notice.text;
+  return { output: trimmed, badge: notice.label };
+};
 var resultDetails = (audit) => {
   const preview = recordOf(audit.preview);
   const previewDetails = recordOf(preview?.details);
@@ -3409,8 +3427,9 @@ var renderDiff = (diff, filePath, theme, options, limitOverride) => {
 };
 var renderRead = (audit, theme, options) => {
   if (!options.expanded && !options.settings.readContentPreview) return null;
-  const output = resultOutput(audit);
-  if (output === void 0) return null;
+  const raw = resultOutput(audit);
+  if (raw === void 0) return null;
+  const { output, badge } = stripTruncationNotice(raw);
   const filePath = argString(audit, "path") ?? "";
   if (/^Read image file/i.test(output)) {
     return { lines: [theme.fg("dim", escapeControlChars3(output))], hidden: 0 };
@@ -3425,6 +3444,7 @@ var renderRead = (audit, theme, options) => {
   });
   if (notice) pushArcItem(rendered.lines, arcItem(theme, notice));
   else if (truncated) pushArcItem(rendered.lines, arcItem(theme, "Output truncated by read"));
+  if (badge) pushArcItem(rendered.lines, arcItem(theme, badge));
   return rendered;
 };
 var getWriteBefore = (audit) => {
@@ -3628,7 +3648,7 @@ var renderGrepRowLine = (row, highlighted, audit, theme) => {
 };
 var renderGrep = (audit, theme, options) => {
   if (!options.expanded && !options.settings.grepResultPreview) return null;
-  const output = resultOutput(audit)?.replace(/\r?\n$/, "");
+  const { output, badge } = stripTruncationNotice(resultOutput(audit) ?? "");
   if (!output || output === "No matches found") {
     return { lines: [theme.fg("muted", output || "No matches found")], hidden: 0 };
   }
@@ -3684,6 +3704,7 @@ var renderGrep = (audit, theme, options) => {
   if (skipHighlight) {
     pushArcItem(lines, arcItem(theme, "Syntax highlighting skipped for large grep output"));
   }
+  if (badge) pushArcItem(lines, arcItem(theme, badge));
   return { lines, hidden: selected.hidden };
 };
 var NERD_BY_NAME = {
@@ -3738,7 +3759,7 @@ var pathIcon = (filePath, directory, mode) => {
 var renderPathList = (audit, theme, options) => {
   const enabled2 = audit.tool === "find" ? options.settings.findResultPreview : options.settings.lsResultPreview;
   if (!options.expanded && !enabled2) return null;
-  const output = resultOutput(audit)?.replace(/\r?\n$/, "") ?? "";
+  const { output, badge } = stripTruncationNotice(resultOutput(audit) ?? "");
   const emptyMarker = audit.tool === "find" ? "No files found matching pattern" : "(empty directory)";
   if (!output || output === emptyMarker) {
     return {
@@ -3810,6 +3831,7 @@ var renderPathList = (audit, theme, options) => {
     }
   }
   flush();
+  if (badge) pushArcItem(lines, arcItem(theme, badge));
   return { lines, hidden: selected.hidden };
 };
 var renderBash = (audit, theme, options) => {
@@ -3819,19 +3841,25 @@ var renderBash = (audit, theme, options) => {
   const command = bashCommand(audit);
   const displayCommand = escapeControlChars3(command.replace(/\r\n/g, "\n"));
   const commandLines = displayCommand.split("\n");
+  const showFirstCommandLine = options.expanded;
   const highlightedCommand = command ? highlightCode(
     displayCommand,
     "bash",
     options.invalidate
   ) : null;
-  const lines = commandLines.slice(1).map(
-    (line, index) => `${theme.fg("dim", "  ")}${highlightedCommand?.[index + 1] ?? theme.fg("accent", line)}`
+  const bodyCommandLines = showFirstCommandLine ? commandLines : commandLines.slice(1);
+  const highlightOffset = showFirstCommandLine ? 0 : 1;
+  const lines = bodyCommandLines.map(
+    (line, index) => `${theme.fg("dim", "  ")}${highlightedCommand?.[index + highlightOffset] ?? theme.fg("accent", line)}`
   );
-  const output = resultOutput(audit)?.replace(/\r?\n$/, "") ?? "";
+  const { output, badge } = stripTruncationNotice(resultOutput(audit) ?? "");
   if (!output || output === "(no output)") {
     if (!output && audit.fromTrace) {
       return { lines: [...lines, theme.fg("dim", "output not retained across reload")], hidden: 0 };
     }
+    const failure2 = bashFailureArc(audit, theme);
+    if (failure2) pushArcItem(lines, failure2);
+    if (badge) pushArcItem(lines, arcItem(theme, badge));
     return {
       lines: [...lines, theme.fg("muted", output || "No output")],
       hidden: 0
@@ -3849,9 +3877,12 @@ var renderBash = (audit, theme, options) => {
     const text = theme.fg(audit.success === false ? "error" : "muted", escapeControlChars3(entry.line) || " ");
     lines.push(text);
   }
+  const failure = bashFailureArc(audit, theme);
+  if (failure) pushArcItem(lines, failure);
   if (nativeTruncated(audit)) {
     pushArcItem(lines, arcItem(theme, `Output truncated by ${audit.tool ?? "shell"}`));
   }
+  if (badge) pushArcItem(lines, arcItem(theme, badge));
   const fullOutputPath = stringOf(resultDetails(audit)?.fullOutputPath);
   if (fullOutputPath) pushArcItem(lines, arcItem(theme, `Full output: ${escapeControlChars3(fullOutputPath)}`));
   return { lines, hidden: selected.hidden };
@@ -3915,7 +3946,7 @@ var coreToolTitle = (audit, theme, options) => {
   const filePath = argString(audit, "path") ?? "";
   if (audit.tool === "bash") {
     const command = bashCommand(audit);
-    const firstLine = command.split("\n")[0] ?? "";
+    const firstLine = truncateBashTitleCommand(command.split("\n")[0] ?? "");
     const language = "bash";
     const highlighted = firstLine ? highlightCode(firstLine, language, options.invalidate)?.[0] : void 0;
     const timeout = numberOf(audit.args?.timeout);
@@ -4457,6 +4488,20 @@ var callHeadlinePreview = (audit) => {
   const args = audit.args ?? {};
   return providerCallDetail(provider, tool, args, audit.result, audit.preview) || headlineArg(args) || structuralCallDetail(provider, tool, args, audit.result);
 };
+var resultDetailsOf = (audit) => {
+  const preview = audit.preview;
+  if (typeof preview === "object" && preview !== null) {
+    const details = preview.details;
+    if (typeof details === "object" && details !== null) return details;
+  }
+  const result = audit.result;
+  if (typeof result === "object" && result !== null) {
+    const details = result.details;
+    if (typeof details === "object" && details !== null) return details;
+  }
+  return void 0;
+};
+var finiteExitCode = (value) => typeof value === "number" && Number.isSafeInteger(value) && value !== 0 ? value : void 0;
 var nestedResultTruncated = (audit) => {
   if (audit.resultTruncated === true) return true;
   const result = audit.result;
@@ -4469,7 +4514,13 @@ var nestedResultTruncated = (audit) => {
 };
 function nestedCallTitle(audit, theme, invalidate, core) {
   const title = nestedCallTitleText(audit, theme, invalidate, core);
-  return nestedResultTruncated(audit) ? `${title} ${theme.fg("warning", "\xB7 truncated")}` : title;
+  const markers = [];
+  if (audit.success === false) {
+    const exitCode = finiteExitCode(resultDetailsOf(audit)?.exitCode);
+    markers.push(theme.fg("error", exitCode === void 0 ? "\xB7 failed" : `\xB7 failed \xB7 exit ${exitCode}`));
+  }
+  if (nestedResultTruncated(audit)) markers.push(theme.fg("warning", "\xB7 truncated"));
+  return markers.length > 0 ? `${title} ${markers.join(" ")}` : title;
 }
 var nestedCallTitleText = (audit, theme, invalidate, core) => {
   const coreTitle = core ? coreToolTitle(audit, theme, { ...core, ...invalidate ? { invalidate } : {} }) : null;
@@ -4669,26 +4720,28 @@ var renderFabricMulticallPartial = (input, theme, invalidate) => {
       core: input.core,
       ...invalidate ? { invalidate } : {}
     });
+    const gutterColor = audit.success === false ? "error" : audit.success === void 0 ? "warning" : "success";
+    const gutter = theme.fg(gutterColor, "\u2502");
+    const withGutter = (row) => `${gutter} ${row}`;
     let callRow = `${glyph} ${nestedCallTitle(audit, theme, invalidate, input.core)}`;
     if (audit.success === false && audit.error) {
       callRow += ` ${theme.fg("dim", "\u203A")} ${theme.fg("error", truncateOneLine(safeTerminalText(audit.error), 240))}`;
     } else if (previewLines[0]) {
       callRow += ` ${previewLines[0]}`;
     }
-    rows.push(callRow);
+    rows.push(input.expanded ? withGutter(callRow) : callRow);
     if (audit.success !== false && input.preview?.auditIndex === auditIndex) {
-      for (const line of input.preview.body.split("\n")) rows.push(`  ${line}`);
+      for (const line of input.preview.body.split("\n")) rows.push(input.expanded ? withGutter(`  ${line}`) : `  ${line}`);
       if (input.preview.hidden > 0) {
-        rows.push(
-          theme.fg(
-            "dim",
-            `  \u2026 ${input.preview.hidden} more ${input.preview.hidden === 1 ? "line" : "lines"}`
-          )
+        const moreRow = theme.fg(
+          "dim",
+          `  \u2026 ${input.preview.hidden} more ${input.preview.hidden === 1 ? "line" : "lines"}`
         );
+        rows.push(input.expanded ? withGutter(moreRow) : moreRow);
       }
     }
     if (audit.success !== false && previewLines.length > 1) {
-      for (const line of previewLines.slice(1)) rows.push(line);
+      for (const line of previewLines.slice(1)) rows.push(input.expanded ? withGutter(line) : line);
     }
   }
   const callsHidden = input.audits.length - callsShown.length;
@@ -5030,4 +5083,4 @@ export {
   formatJsonAsYaml,
   formatFabricValue
 };
-//# sourceMappingURL=chunk-5SSQD7TW.js.map
+//# sourceMappingURL=chunk-7NWMFYD3.js.map