@aliou/pi-processes 0.10.3 → 0.10.5

package/extensions/processes/message-renderer.ts

@@ -2,11 +2,17 @@ import type {
   ExtensionAPI,
   MessageRenderOptions,
   Theme,
+  ThemeColor,
 } from "@earendil-works/pi-coding-agent";
 import { type Component, Container, Text } from "@earendil-works/pi-tui";
 import { truncateForDisplay } from "../shared/display-text";
 import { MESSAGE_TYPE_PROCESS_NOTIFICATION } from "./constants";
-import type { ProcessNotificationDetails } from "./notifications/types";
+import type {
+  Attention,
+  ProcessNotificationDetails,
+  ProcessNotificationKind,
+} from "./notifications/types";
+import { formatPatternForDisplay, quoteFilter } from "./tools/components";
 
 export function registerProcessNotificationRenderer(pi: ExtensionAPI): void {
   pi.registerMessageRenderer<ProcessNotificationDetails>(
@@ -25,11 +31,17 @@ export function renderProcessNotificationMessage(
     return undefined;
   }
 
+  // outputPad is only populated on Pi 0.82.1+ (see MessageRenderOptions);
+  // fall back to 0 so this renderer doesn't break on older Pi builds.
+  const outputPad = options.outputPad ?? 0;
+
   const container = new Container();
-  container.addChild(new Text(formatSummary(details, theme), 0, 0));
+  container.addChild(new Text(formatSummary(details, theme), outputPad, 0));
 
   if (options.expanded) {
-    container.addChild(new Text(formatDetailLine(details, theme), 0, 1));
+    container.addChild(
+      new Text(formatDetailLine(details, theme), outputPad, 0),
+    );
   }
 
   return container;
@@ -66,17 +78,100 @@ function formatDetailLine(
   details: ProcessNotificationDetails,
   theme: Theme,
 ): string {
-  const parts = [
-    `id: ${details.processId}`,
-    `attention: ${details.attention}`,
-    details.status ? `status: ${details.status}` : null,
-    details.exitCode === undefined || details.exitCode === null
-      ? null
-      : `exit: ${details.exitCode}`,
-    details.endReason ? `reason: ${details.endReason}` : null,
-  ].filter(Boolean);
-
-  return theme.fg("muted", parts.join("  "));
+  const parts: string[] = [
+    field("id", details.processId, theme),
+    field("notify", formatAttention(details.attention, theme), theme),
+  ];
+
+  if (details.logMatch) {
+    const pattern = quoteFilter(
+      formatPatternForDisplay(details.logMatch.pattern),
+      details.logMatch.mode,
+    );
+    parts.push(
+      field("pattern", theme.fg("accent", pattern), theme),
+      field("stream", theme.fg("muted", `[${details.logMatch.stream}]`), theme),
+    );
+  } else {
+    if (details.status) {
+      parts.push(
+        field(
+          "status",
+          theme.fg(statusTone(details.kind), details.status),
+          theme,
+        ),
+      );
+    }
+
+    if (details.exitCode !== undefined && details.exitCode !== null) {
+      parts.push(
+        field(
+          "exit",
+          theme.fg(statusTone(details.kind), String(details.exitCode)),
+          theme,
+        ),
+      );
+    }
+
+    if (details.endReason) {
+      parts.push(field("reason", theme.fg("muted", details.endReason), theme));
+    }
+
+    if (details.signal) {
+      const number =
+        details.signal.number === null
+          ? ""
+          : ` (${details.signal.number}, ${details.signal.description})`;
+      parts.push(
+        field(
+          "signal",
+          theme.fg("muted", `${details.signal.name}${number}`),
+          theme,
+        ),
+      );
+    }
+  }
+
+  return parts.join("  ");
+}
+
+/** `label:` in muted, matching `buildField` in tools/utils.ts. */
+function field(label: string, value: string, theme: Theme): string {
+  return `${theme.fg("muted", `${label}:`)} ${value}`;
+}
+
+/**
+ * Plain-language attention level, reusing the tone mapping from
+ * `attentionTone` in tools/components/watch.ts (turn/context keep their
+ * color; ignore switches from muted to dim to read as "quieter").
+ */
+function formatAttention(attention: Attention, theme: Theme): string {
+  switch (attention) {
+    case "turn":
+      return theme.bold(theme.fg("warning", "now"));
+    case "context":
+      return theme.bold(theme.fg("success", "next turn"));
+    default:
+      return theme.bold(theme.fg("dim", "silent"));
+  }
+}
+
+/**
+ * Same tone family as `statusColor` in shared/ui.ts: success for a clean
+ * exit, error for failure/crash, dim for killed.
+ */
+function statusTone(kind: ProcessNotificationKind): ThemeColor {
+  switch (kind) {
+    case "success":
+      return "success";
+    case "failure":
+    case "crash":
+      return "error";
+    case "killed":
+      return "dim";
+    default:
+      return "muted";
+  }
 }
 
 function summaryVerb(details: ProcessNotificationDetails): string {

package/extensions/processes/notifications/log-matchers.ts

@@ -1,4 +1,5 @@
 import { compileLineMatcher } from "../../../src/utils/match-line";
+import { plainTextForDisplay } from "../../shared/display-text";
 import type { NotifyConfig } from "./registry";
 import type { Attention } from "./types";
 
@@ -130,8 +131,12 @@ function splitAppendedIntoLines(
   for (const entry of appended) {
     const split = entry.text.split("\n");
     for (const line of split) {
-      if (line.length === 0) continue;
-      lines.push({ type: entry.type, text: line });
+      // Match the same per-line visible text the log views render. This keeps
+      // CR-overwritten progress and invisible escape bytes from firing
+      // watches, without making watches depend on viewport width.
+      const displayLine = plainTextForDisplay(line);
+      if (displayLine.length === 0) continue;
+      lines.push({ type: entry.type, text: displayLine });
     }
   }
 

package/extensions/processes/tools/components/index.ts

@@ -1,4 +1,8 @@
 export { ProcessActionHeader } from "./process-action-header";
 export { ProcessActionTitle } from "./process-action-title";
 export { ToolLayout } from "./tool-layout";
-export { buildMatcherLine, formatPatternForDisplay } from "./watch";
+export {
+  buildMatcherLine,
+  formatPatternForDisplay,
+  quoteFilter,
+} from "./watch";

package/extensions/processes/tools/components/watch.ts

@@ -15,6 +15,8 @@ export function buildMatcherLine(
   const attentionTone =
     on === "turn" ? "warning" : on === "context" ? "success" : "muted";
 
+  const pattern = quoteFilter(formatPatternForDisplay(matcher.pattern), mode);
+
   return new Text(
     [
       prefix,
@@ -22,13 +24,22 @@ export function buildMatcherLine(
       "  ",
       theme.bold(theme.fg(attentionTone, `${on}${repeat}`)),
       "  ",
-      theme.fg("accent", formatPatternForDisplay(matcher.pattern)),
+      theme.fg("accent", pattern),
     ].join(""),
     0,
     0,
   );
 }
 
+export function quoteFilter(
+  pattern: string,
+  mode: "literal" | "regex",
+): string {
+  if (mode === "regex") return `/${pattern}/`;
+  if (pattern.includes('"')) return `'${pattern}'`;
+  return `"${pattern}"`;
+}
+
 export function formatPatternForDisplay(pattern: string): string {
   let formatted = "";
 

package/extensions/processes/tools/output/render.ts

@@ -2,7 +2,8 @@ import type { Theme } from "@earendil-works/pi-coding-agent";
 import { Container, Spacer, Text } from "@earendil-works/pi-tui";
 
 import { sanitizeForDisplay } from "../../../shared/display-text";
-import { ProcessActionHeader } from "../components";
+import { truncateToWidth } from "../../../shared/truncate";
+import { ProcessActionHeader, quoteFilter } from "../components";
 import type { ProcessesParamsType } from "../schema";
 import { buildField } from "../utils";
 import type { OutputDetails } from ".";
@@ -73,9 +74,23 @@ export function buildCollapsed(
     ),
   );
 
+  if (details.pattern) {
+    const quoted = quoteFilter(details.pattern, details.mode);
+    const value = truncateToWidth(sanitizeForDisplay(quoted), 80);
+    container.addChild(
+      new Text(
+        `${theme.fg("muted", "filter:")} ${theme.fg("accent", value)}`,
+        0,
+        0,
+      ),
+    );
+  }
+
   const bodyLines = extractOutputBody(contentText, details);
   const preview = bodyLines.slice(-2).map(sanitizeForDisplay).join("\n");
 
+  container.addChild(new Spacer(1));
+
   if (preview) {
     container.addChild(new Text(theme.fg("muted", preview), 0, 0));
   } else {
@@ -119,9 +134,8 @@ function buildOutputMeta(details: OutputDetails, theme: Theme): Container {
   container.addChild(buildField("stream", details.stream, theme));
 
   if (details.pattern) {
-    const modeTag = details.mode === "regex" ? " (regex)" : "";
     container.addChild(
-      buildField("filter", `${details.pattern}${modeTag}`, theme),
+      buildField("filter", quoteFilter(details.pattern, details.mode), theme),
     );
   }
 

package/extensions/processes-dock/components/log-dock-component.ts

@@ -3,9 +3,9 @@ import type { Theme } from "@earendil-works/pi-coding-agent";
 import { type Component, visibleWidth } from "@earendil-works/pi-tui";
 import type { ProcessInfo } from "../../../src/types";
 import { LIVE_STATUSES } from "../../../src/types";
-import { renderProcessTab } from "../../process-tabs";
 import { sanitizeForDisplay } from "../../shared/display-text";
 import { displayTextOf, renderLogLine } from "../../shared/log-line";
+import { renderProcessTab } from "../../shared/process-tabs";
 import { truncateToWidth } from "../../shared/truncate";
 import {
   clampNameColumn,

package/extensions/processes-logs/components/log-file-viewer.ts

@@ -1,8 +1,15 @@
 import type { Theme } from "@earendil-works/pi-coding-agent";
 import { visibleWidth } from "@earendil-works/pi-tui";
-import { sanitizeForDisplay } from "../../shared/display-text";
+import {
+  plainTextForDisplay,
+  sanitizeForDisplay,
+} from "../../shared/display-text";
 import { trimToBudget } from "../../shared/line-buffer";
-import { type LogLineEmphasis, renderLogLine } from "../../shared/log-line";
+import {
+  displayTextOf,
+  type LogLineEmphasis,
+  renderLogLine,
+} from "../../shared/log-line";
 import { truncateToWidth } from "../../shared/truncate";
 import type { ProcessLogLine } from "../logs-client";
 
@@ -122,8 +129,9 @@ export class LogFileViewer {
    * priority (search current match > search match > notify match > stream).
    */
   addNotifyMatch(match: { line: string }): void {
-    // Stored sanitized so it can be compared against sanitized buffer lines.
-    const line = sanitizeForDisplay(match.line);
+    // Stored as plain visible text so invisible escape bytes cannot be what
+    // makes a notification marker match.
+    const line = plainTextForDisplay(match.line);
     if (line) this.notifyLines.add(line);
   }
 
@@ -192,7 +200,8 @@ export class LogFileViewer {
           ? "search-current"
           : matchSet.has(visibleIndex)
             ? "search"
-            : this.notifyLines.has(line.text)
+            : this.notifyLines.has(line.text) ||
+                this.notifyLines.has(displayTextOf(line))
               ? "notify"
               : "none";
       return renderLogLine(line, { theme: this.theme, width, emphasis });

package/extensions/processes-logs/components/log-overlay-component.ts

@@ -17,8 +17,8 @@ import {
 import { LIVE_STATUSES, type ProcessInfo } from "../../../src/types";
 import { formatRuntime } from "../../../src/utils/format";
 import { isRecord } from "../../../src/utils/is-record";
-import { renderProcessTab } from "../../process-tabs";
 import { truncateForDisplay } from "../../shared/display-text";
+import { renderProcessTab } from "../../shared/process-tabs";
 import { truncateToWidth } from "../../shared/truncate";
 import { LineComponent, LinesComponent, RuleComponent } from "../../shared/ui";
 import { requestProcessList } from "../client";

package/extensions/shared/display-text.ts

@@ -13,10 +13,17 @@ const ESC = String.fromCodePoint(0x001b);
 const BEL = String.fromCodePoint(0x0007);
 const ST = String.fromCodePoint(0x009c);
 const RESET = `${ESC}[0m`;
+const C1_DCS = String.fromCodePoint(0x0090);
+const C1_SOS = String.fromCodePoint(0x0098);
+const C1_OSC = String.fromCodePoint(0x009d);
+const C1_PM = String.fromCodePoint(0x009e);
+const C1_APC = String.fromCodePoint(0x009f);
+const C1_STRING_INTRODUCERS = [C1_DCS, C1_SOS, C1_OSC, C1_PM, C1_APC];
 
-// Control characters a single display row must never contain. Tabs are handled
-// separately; newlines are dropped because they would shift the whole frame.
-// C1 controls are included: a raw \u009b is an alias for CSI on some terminals.
+// Control characters a single display row must never contain. Tabs and
+// carriage returns are handled separately; newlines are dropped because they
+// would shift the whole frame. C1 controls are included: a raw \u009b is an
+// alias for CSI on some terminals.
 // biome-ignore lint/suspicious/noControlCharactersInRegex: this regex intentionally targets terminal control characters.
 const DISPLAY_CONTROL_CHARS = /[\u0000-\u0008\u000a-\u001f\u007f-\u009f]/gu;
 
@@ -41,7 +48,9 @@ const TAB_WIDTH = 8;
  * into the rest of the frame.
  */
 export function sanitizeForDisplay(text: string): string {
-  if (!text.includes(ESC)) return cleanPlainText(text, 0).text;
+  if (!text.includes(ESC) && !hasC1StringIntroducer(text)) {
+    return cleanPlainText(text, 0).text;
+  }
 
   let out = "";
   let cursor = 0;
@@ -49,21 +58,34 @@ export function sanitizeForDisplay(text: string): string {
   let keptSgr = false;
 
   while (cursor < text.length) {
-    const escapeAt = text.indexOf(ESC, cursor);
-    if (escapeAt === -1) {
-      out += cleanPlainText(text.slice(cursor), column).text;
+    const sequenceAt = findNextSequenceStart(text, cursor);
+    if (sequenceAt === -1) {
+      const chunk = cleanPlainText(text.slice(cursor), column);
+      if (chunk.resetsRow) {
+        out = "";
+        keptSgr = false;
+      }
+      out += chunk.text;
       break;
     }
-    const chunk = cleanPlainText(text.slice(cursor, escapeAt), column);
+    const chunk = cleanPlainText(text.slice(cursor, sequenceAt), column);
+    if (chunk.resetsRow) {
+      out = "";
+      keptSgr = false;
+    }
     out += chunk.text;
     column = chunk.column;
 
-    const sequence = readEscapeSequence(text, escapeAt);
-    if (sequence.isSgr) {
-      out += text.slice(escapeAt, sequence.end);
-      keptSgr = true;
+    if (text[sequenceAt] === ESC) {
+      const sequence = readEscapeSequence(text, sequenceAt);
+      if (sequence.isSgr) {
+        out += text.slice(sequenceAt, sequence.end);
+        keptSgr = true;
+      }
+      cursor = sequence.end;
+    } else {
+      cursor = readC1StringSequence(text, sequenceAt).end;
     }
-    cursor = sequence.end;
   }
 
   if (!keptSgr || out.endsWith(RESET)) return out;
@@ -79,6 +101,22 @@ export function truncateForDisplay(text: string, width: number): string {
   return closeSgr(truncateToWidth(sanitizeForDisplay(text), width, "…"));
 }
 
+/**
+ * Return the unstyled text a user can actually read after terminal controls
+ * have been interpreted/dropped. Use this for comparisons such as search,
+ * notify markers, and log watches so invisible escape bytes cannot match.
+ */
+export function plainTextForDisplay(text: string): string {
+  return stripSgr(sanitizeForDisplay(text));
+}
+
+const SGR = new RegExp(`${ESC}\\[[0-9;:]*m`, "gu");
+
+/** Drop the SGR sequences `sanitizeForDisplay` kept for rendering colors. */
+export function stripSgr(text: string): string {
+  return text.replace(SGR, "");
+}
+
 /**
  * Re-close colors after truncation. `sanitizeForDisplay` ends a colored string
  * with a reset, but truncating can cut that reset off and let the color bleed
@@ -97,21 +135,42 @@ export function closeSgr(text: string): string {
 function cleanPlainText(
   text: string,
   column: number,
-): { text: string; column: number } {
-  const clean = text.replace(DISPLAY_CONTROL_CHARS, "");
+): { text: string; column: number; resetsRow: boolean } {
+  const carriageReturn = text.lastIndexOf("\r");
+  const resetsRow = carriageReturn !== -1;
+  const visibleText = resetsRow ? text.slice(carriageReturn + 1) : text;
+  const startColumn = resetsRow ? 0 : column;
+  const clean = visibleText.replace(DISPLAY_CONTROL_CHARS, "");
   if (!clean.includes("\t")) {
-    return { text: clean, column: column + visibleWidth(clean) };
+    return {
+      text: clean,
+      column: startColumn + visibleWidth(clean),
+      resetsRow,
+    };
   }
 
   const parts = clean.split("\t");
   let out = parts[0] ?? "";
-  let col = column + visibleWidth(out);
+  let col = startColumn + visibleWidth(out);
   for (const part of parts.slice(1)) {
     const spaces = TAB_WIDTH - (col % TAB_WIDTH);
     out += " ".repeat(spaces) + part;
     col += spaces + visibleWidth(part);
   }
-  return { text: out, column: col };
+  return { text: out, column: col, resetsRow };
+}
+
+function findNextSequenceStart(text: string, cursor: number): number {
+  let next = text.indexOf(ESC, cursor);
+  for (const introducer of C1_STRING_INTRODUCERS) {
+    const index = text.indexOf(introducer, cursor);
+    if (index !== -1 && (next === -1 || index < next)) next = index;
+  }
+  return next;
+}
+
+function hasC1StringIntroducer(text: string): boolean {
+  return C1_STRING_INTRODUCERS.some((introducer) => text.includes(introducer));
 }
 
 /**
@@ -151,6 +210,12 @@ function readEscapeSequence(
   return { end: start + 2, isSgr: false };
 }
 
+function readC1StringSequence(text: string, start: number): { end: number } {
+  const introducer = text[start];
+  const allowBel = introducer === C1_OSC || introducer === C1_APC;
+  return { end: findStringTerminator(text, start + 1, allowBel) };
+}
+
 /** Index just past the terminator of a string sequence, else end of input. */
 function findStringTerminator(
   text: string,

package/extensions/shared/log-line.ts

@@ -10,7 +10,12 @@
 import type { Theme } from "@earendil-works/pi-coding-agent";
 import { visibleWidth } from "@earendil-works/pi-tui";
 
-import { closeSgr, sanitizeForDisplay } from "./display-text";
+import {
+  closeSgr,
+  plainTextForDisplay,
+  sanitizeForDisplay,
+  stripSgr,
+} from "./display-text";
 import { truncateToWidth } from "./truncate";
 
 export interface DisplayLogLine {
@@ -60,7 +65,7 @@ export function renderLogLine(
 
 /** Text of a log line as the views display it, for match comparisons. */
 export function displayTextOf(line: DisplayLogLine): string {
-  return sanitizeForDisplay(line.text);
+  return plainTextForDisplay(line.text);
 }
 
 function toneLogText(
@@ -75,10 +80,3 @@ function toneLogText(
   if (type === "stderr") return theme.fg("warning", text);
   return text;
 }
-
-const SGR = new RegExp(`${String.fromCodePoint(0x001b)}\\[[0-9;:]*m`, "gu");
-
-/** Drop the SGR sequences `sanitizeForDisplay` kept. */
-function stripSgr(text: string): string {
-  return text.replace(SGR, "");
-}

package/extensions/{process-tabs.ts → shared/process-tabs.ts}

@@ -1,8 +1,8 @@
 import type { Theme } from "@earendil-works/pi-coding-agent";
 
-import type { ProcessInfo } from "../src/types";
-import { truncateForDisplay } from "./shared/display-text";
-import { MAX_TAB_NAME, statusDot } from "./shared/ui";
+import type { ProcessInfo } from "../../src/types";
+import { truncateForDisplay } from "./display-text";
+import { MAX_TAB_NAME, statusDot } from "./ui";
 
 export { MAX_TAB_NAME, statusDot as renderProcessTabDot };
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aliou/pi-processes",
-  "version": "0.10.3",
+  "version": "0.10.5",
   "license": "MIT",
   "type": "module",
   "private": false,
@@ -23,7 +23,8 @@
     "skills": [
       "./skills/pi-processes"
     ],
-    "video": "https://assets.aliou.me/pi-extensions/demos/pi-processes.mp4"
+    "image": "https://assets.aliou.me/github/aliou/pi-processes/banner-v0.10.x.png",
+    "video": "https://assets.aliou.me/pi-extensions/demos/processes/v0.10.0/agent-starts-processes.mp4"
   },
   "publishConfig": {
     "access": "public"
@@ -56,9 +57,9 @@
     "@aliou/biome-plugins": "^0.8.1",
     "@biomejs/biome": "^2.4.15",
     "@changesets/cli": "^2.27.11",
-    "@earendil-works/pi-ai": "0.80.3",
-    "@earendil-works/pi-coding-agent": "0.80.3",
-    "@earendil-works/pi-tui": "0.80.3",
+    "@earendil-works/pi-ai": "0.84.1",
+    "@earendil-works/pi-coding-agent": "0.84.1",
+    "@earendil-works/pi-tui": "0.84.1",
     "@golevelup/ts-vitest": "^4.0.0",
     "@types/node": "^25.0.10",
     "husky": "^9.1.7",
@@ -66,20 +67,24 @@
     "ts-json-schema-generator": "^2.9.0",
     "typebox": "1.1.38",
     "typescript": "^5.9.3",
-    "vitest": "^4.1.5"
+    "vitest": "^4.1.5",
+    "vitest-evals": "0.15.0"
   },
   "scripts": {
     "typecheck": "tsc --noEmit",
+    "typecheck:evals": "tsc --noEmit -p tests/evals/tsconfig.json",
     "lint": "biome check",
     "format": "biome check --write",
     "gen:schema": "ts-json-schema-generator --path extensions/processes/config/types.ts --type ProcessConfig --no-type-check -o schema.json",
     "check:schema": "ts-json-schema-generator --path extensions/processes/config/types.ts --type ProcessConfig --no-type-check -o /tmp/pi-processes-schema-check.json && diff -q schema.json /tmp/pi-processes-schema-check.json",
+    "check:changesets": "node scripts/check-changesets.ts",
     "check:lockfile": "pnpm install --frozen-lockfile --ignore-scripts",
     "prepare": "[ -d .git ] && husky || true",
     "changeset": "changeset",
     "version": "changeset version",
     "test": "vitest run",
     "test:e2e": "vitest run --config vitest.e2e.config.ts",
+    "test:evals": "vitest run --config tests/evals/vitest.config.ts",
     "release": "pnpm changeset publish"
   },
   "packageManager": "pnpm@11.5.1",