@pify/swarm 0.7.3 → 0.7.5

package/extensions/swarm.ts

@@ -316,7 +316,15 @@ export default function swarm(pi: ExtensionAPI) {
           try {
             const iso = createIsolationWorktree(ctx.cwd, run.runId + "-i" + (item.index + 1));
             await runItem(ctx, run.runId, def, item, context, iso.path, mailbox);
-            if (item.result !== null) item.result = `${item.result}\n\n${isolationNote(iso)}`;
+            // Remove the worktree when the item changed nothing — the cleanup
+            // the README promised but the code never performed (removeIfUnchanged
+            // was imported and never called, leaking a worktree + branch per
+            // read-only item). Kept when there is work to merge, and only then
+            // is the merge note worth showing.
+            const removed = removeIfUnchanged(ctx.cwd, iso);
+            if (item.result !== null) {
+              item.result = `${item.result}\n\n${removed ? CLEAN_WORKTREE_NOTE : isolationNote(iso)}`;
+            }
           } catch (err) {
             item.status = "error";
             item.error = err instanceof Error ? err.message : String(err);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pify/swarm",
-  "version": "0.7.3",
+  "version": "0.7.5",
   "description": "Coordinate multiple pi agents in parallel: swarm_run fan-out with per-item auto-routing, concurrency queue, aggregated reports",
   "keywords": [
     "pi-package",

package/src/widget-clamp.ts

@@ -0,0 +1,35 @@
+/**
+ * Keep a widget from overrunning the screen.
+ *
+ * pi caps string-array widgets at ten lines, but a Text-factory widget — the
+ * pattern every @pify widget uses because it caches and renders efficiently —
+ * bypasses that guard entirely (measured in pi's interactive-mode source). So
+ * the bound is ours to keep: an unbounded row count pushes the editor off
+ * screen, and a single line wider than the terminal can crash the renderer.
+ *
+ * Two rules, applied before any colour escape is added (slicing a coloured
+ * string would cut through an escape sequence):
+ *  - `clampWidth` truncates one segment's *visible* text.
+ *  - `clampRows` caps the row list and, per the suite's no-silent-caps rule,
+ *    replaces the overflow with a visible "+N more" line rather than dropping
+ *    it in silence.
+ */
+
+/** A widget line's variable segment never exceeds this many characters. */
+export const MAX_SEGMENT = 44;
+/** A widget never shows more than this many item rows. */
+export const MAX_WIDGET_ROWS = 8;
+
+export function clampWidth(text: string, max = MAX_SEGMENT): string {
+  const t = text.replace(/[\r\n]+/g, " ");
+  return t.length > max ? `${t.slice(0, max - 1)}…` : t;
+}
+
+/**
+ * Return at most `max` rows; when more were supplied, `more(hidden)` renders
+ * the summary line that stands in for the rest.
+ */
+export function clampRows(rows: string[], max: number, more: (hidden: number) => string): string[] {
+  if (rows.length <= max) return rows;
+  return [...rows.slice(0, max), more(rows.length - max)];
+}

package/src/widget.ts

@@ -1,3 +1,4 @@
+import { clampRows, clampWidth, MAX_WIDGET_ROWS } from "./widget-clamp.ts";
 import type { SwarmRun, ThemeLike } from "./types.ts";
 
 const WIDTH = 54;
@@ -29,7 +30,9 @@ export function buildWidgetLines(run: SwarmRun | null, theme: ThemeLike, now: nu
   const pad = Math.max(1, WIDTH - title.length - hint.length);
   lines.push(dim(`╭${title}${"─".repeat(pad)}${hint}╮`));
 
-  for (const item of run.items) {
+  // Cap the rows: a large swarm would otherwise push the editor off screen,
+  // since a Text-factory widget bypasses pi's ten-line guard.
+  const rows = run.items.map((item) => {
     const paint =
       item.status === "running"
         ? (s: string) => theme.fg("warning", s)
@@ -38,8 +41,10 @@ export function buildWidgetLines(run: SwarmRun | null, theme: ThemeLike, now: nu
           : item.status === "queued"
             ? dim
             : (s: string) => theme.fg("error", s);
-    const text = item.item.length > 32 ? `${item.item.slice(0, 32)}…` : item.item;
-    lines.push(`${dim("│ ")}${paint(`${icon(item.status)} ${item.agent}`)}${dim(` ${text}`)}`);
+    return `${dim("│ ")}${paint(`${icon(item.status)} ${clampWidth(item.agent, 24)}`)}${dim(` ${clampWidth(item.item, 32)}`)}`;
+  });
+  for (const row of clampRows(rows, MAX_WIDGET_ROWS, (hidden) => dim(`│ … +${hidden} more`))) {
+    lines.push(row);
   }
 
   lines.push(dim(`╰${"─".repeat(WIDTH)}╯`));