pi-open-agents 0.1.31 → 0.1.32

package/README.md

@@ -334,9 +334,11 @@ Use `replace-all` for fully isolated agents that should not be influenced by
 workspace context — e.g., a subagent that must run identically regardless of
 the project it's invoked from.
 
-For **subagents** (child process), the agent body **is** the system prompt —
-pi's default prompt is not included. Skills are injected as an XML block after
-the body.
+For **subagents** (child process), the same modes apply: with the default
+`append`, the agent body is appended to pi's default prompt (`--append-system-prompt`);
+`replace`/`replace-all` pass the body as the whole system prompt (`--system-prompt`).
+Skills are injected as an XML block after the body, followed by the result-closing
+contract (`## Outcome` block, file + summary).
 
 ---
 
@@ -415,7 +417,8 @@ six sessions, zero chaos: that's the exact workstation setup documented in
 - ~~**Per-call model override** in `subagent()`~~ — ✅ shipped in [0.1.19](https://github.com/andrea-tomassi/pi-open-agents/releases/tag/v0.1.19) (#10)
 - ~~**Primary tool filters incl. extension tools**~~ — ✅ shipped in [0.1.30](https://github.com/andrea-tomassi/pi-open-agents/releases/tag/v0.1.30) (#18)
 - ~~**Subagent output: file + summary**~~ — ✅ shipped in [0.1.31](https://github.com/andrea-tomassi/pi-open-agents/releases/tag/v0.1.31) (#32)
-- **Parallel subagent limit** — #11
+- ~~**Parallel subagent limit**~~ — ✅ closed, see #11 for the outcome
+- **Zero open issues** — the tracker is empty; new work starts from new issues.
 
 ---
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-open-agents",
-  "version": "0.1.31",
+  "version": "0.1.32",
   "description": "Unified agent and subagent management for pi coding agent, with OpenCode-compatible agent definitions.",
   "keywords": [
     "pi-package",

package/src/subagent/prompt.ts

@@ -78,8 +78,8 @@ export function buildClosingContract(outputPath?: string): string {
     "",
     "If you produced files the caller asked for, list each on its own line as",
     "`<path>` — <one line describing the content>.",
-    "Keep the outcome block to 2-3 lines. Never paste the full output after the",
-    "marker when a file reference suffices; never hide an error behind a file reference.",
+    "Keep the outcome block to 2-3 lines. Keep the rest of your answer concise — the",
+    "outcome block is what the caller reads first; never hide an error behind brevity.",
   ];
   if (outputPath) {
     lines.push(

package/src/subagent/tool.ts

@@ -45,7 +45,7 @@ import {
   parentInheritedRules,
   type RecursionEnv,
 } from "./env.ts";
-import { preview, shortenPath, stringArg } from "../utils/format.ts";
+import { byteLength, preview, shortenPath, stringArg } from "../utils/format.ts";
 
 // ─── Types ───────────────────────────────────────────────────────────────────
 
@@ -161,11 +161,41 @@ function toToolResult(result: AgentResult) {
 
 function toProgressResult(progress: AgentProgress) {
   return {
-    content: [{ type: "text" as const, text: progress.output || "(running...)" }],
+    content: [{ type: "text" as const, text: liveProgressText(progress.output) }],
     details: progress,
   };
 }
 
+/**
+ * Cap for live progress text (issue #32 follow-up).
+ *
+ * Every progress emit used to re-send the child's ENTIRE accumulated output:
+ * over a long run the same bytes go out N times (once per event, growing).
+ * Partial updates are transient (never persisted to the transcript — verified
+ * against live session JSONL), so the cap only trims live display/RPC traffic:
+ * the tail stays (what the child is doing *now*), the head is already in the
+ * temp archive. Final results are untouched (see toToolResult).
+ */
+export const LIVE_PROGRESS_MAX_BYTES = 2048;
+
+/** Tail a string to a byte budget (halving converge, multibyte-safe). */
+export function tailTextByBytes(text: string, maxBytes: number): string {
+  if (byteLength(text) <= maxBytes) return text;
+  let take = Math.min(text.length, maxBytes);
+  let cut = text.slice(text.length - take);
+  while (byteLength(cut) > maxBytes && take > 0) {
+    take = Math.floor(take / 2);
+    cut = text.slice(text.length - take);
+  }
+  return cut;
+}
+
+function liveProgressText(output: string | undefined): string {
+  const text = output || "(running...)";
+  const tailed = tailTextByBytes(text, LIVE_PROGRESS_MAX_BYTES);
+  return tailed.length < text.length ? `[…earlier live output omitted…]\n${tailed}` : text;
+}
+
 // ─── Styled Rendering ────────────────────────────────────────────────────────
 
 function styledPathArg(args: Record<string, unknown>, theme: ThemeLike, fallback?: string): string {