@pi-archimedes/subagent 1.6.0 → 1.7.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-archimedes/subagent",
-  "version": "1.6.0",
+  "version": "1.7.0",
   "type": "module",
   "keywords": [
     "pi-package"
@@ -11,7 +11,7 @@
   ],
   "main": "./src/index.ts",
   "dependencies": {
-    "@pi-archimedes/core": "1.6.0"
+    "@pi-archimedes/core": "1.7.0"
   },
   "peerDependencies": {
     "@earendil-works/pi-ai": ">=0.1.0",

package/src/format.test.ts

@@ -0,0 +1,30 @@
+import { describe, it, expect } from "vitest";
+import { truncLine } from "./format.js";
+
+// ── truncLine ───────────────────────────────────────────────────────────────
+
+describe("truncLine", () => {
+  it("returns text unchanged when within limit", () => {
+    expect(truncLine("hello", 10)).toBe("hello");
+  });
+
+  it("truncates with '...' when exceeding limit", () => {
+    expect(truncLine("hello world", 8)).toBe("hello...");
+  });
+
+  it("stops at newline boundary instead of bleeding into next line", () => {
+    expect(truncLine("line one\nline two\nline three", 15)).toBe("line one...");
+  });
+
+  it("truncates first line if it itself exceeds limit", () => {
+    expect(truncLine("this is a very long first line\nsecond", 12)).toBe("this is a...");
+  });
+
+  it("handles multiple consecutive newlines", () => {
+    expect(truncLine("a\n\n\nb", 10)).toBe("a...");
+  });
+
+  it("handles text starting with newline", () => {
+    expect(truncLine("\nhello", 10)).toBe("...");
+  });
+});

package/src/format.ts

@@ -27,8 +27,16 @@ export function formatCost(cost: number): string {
  * measure ANSI display width. The benefit: the "..." is plain text so it
  * inherits any ANSI color wrapper applied around the result (pi-tui's
  * truncateToWidth appends "..." after closing color codes, leaving it unstyled).
+ *
+ * If the text contains newlines, truncates at the first newline boundary
+ * rather than bleeding into the next line.
  */
 export function truncLine(text: string, maxLen: number): string {
+  // Check for newline first — it's a hard boundary regardless of length
+  const nlIdx = text.indexOf("\n");
+  if (nlIdx !== -1) {
+    return text.slice(0, nlIdx).slice(0, maxLen - 3) + "...";
+  }
   if (text.length <= maxLen) return text;
   return text.slice(0, maxLen - 3) + "...";
 }

package/src/handlers.test.ts

@@ -0,0 +1,32 @@
+import { describe, it, expect } from "vitest";
+import { extractArgsPreview } from "./handlers.js";
+
+// ── extractArgsPreview ──────────────────────────────────────────────────────
+
+describe("extractArgsPreview", () => {
+  it("returns string args truncated", () => {
+    const long = "a".repeat(200);
+    expect(extractArgsPreview(long)).toBe("a".repeat(120));
+  });
+
+  it("replaces newlines in string args", () => {
+    expect(extractArgsPreview("hello\nworld")).toBe("hello world");
+  });
+
+  it("replaces newlines in single-key object value", () => {
+    const args = { command: "cat file.txt\ngrep -A5 \"test\"" };
+    const result = extractArgsPreview(args);
+    expect(result).not.toContain("\n");
+    expect(result).toBe("cat file.txt grep -A5 \"test\"");
+  });
+
+  it("replaces newlines in multi-key longest string value", () => {
+    const args = { short: "x", long: "hello\nworld\nfoo" };
+    expect(extractArgsPreview(args)).toBe("hello world foo");
+  });
+
+  it("handles number and boolean values", () => {
+    expect(extractArgsPreview({ count: 42 })).toBe("42");
+    expect(extractArgsPreview({ enabled: true })).toBe("true");
+  });
+});

package/src/handlers.ts

@@ -15,14 +15,17 @@ export interface JsonEvent {
  * Generic: no hardcoded tool names.
  */
 export function extractArgsPreview(args: unknown): string {
-  if (typeof args === "string") return args.slice(0, ARGS_PREVIEW_MAX);
+  if (typeof args === "string") {
+    // Replace newlines so previews stay single-line
+    return args.replace(/\n/g, " ").slice(0, ARGS_PREVIEW_MAX);
+  }
   if (args && typeof args === "object" && !Array.isArray(args)) {
     const obj = args as Record<string, unknown>;
     const keys = Object.keys(obj);
     // Single-key object: show the value directly (or serialize if complex)
     if (keys.length === 1) {
       const v = obj[keys[0]!];
-      if (typeof v === "string") return v.slice(0, ARGS_PREVIEW_MAX);
+      if (typeof v === "string") return v.replace(/\n/g, " ").slice(0, ARGS_PREVIEW_MAX);
       if (typeof v === "number" || typeof v === "boolean") return String(v);
       // Complex value (array/nested object) — serialize just this value
       const serialized = JSON.stringify(v);
@@ -35,7 +38,7 @@ export function extractArgsPreview(args: unknown): string {
         best = v;
       }
     }
-    if (best) return best.slice(0, ARGS_PREVIEW_MAX);
+    if (best) return best.replace(/\n/g, " ").slice(0, ARGS_PREVIEW_MAX);
   }
   const serialized = JSON.stringify(args);
   return serialized?.slice(0, ARGS_PREVIEW_MAX) ?? "";