@tranhoangnguyen0310/pi-flow-external 2.1.0-external.2 → 2.1.0-external.3

package/CHANGELOG.md

@@ -4,6 +4,15 @@ All notable changes to pi-flow external are documented here.
 
 ## Unreleased
 
+## [2.1.0-external.3] - 2026-09-18
+
+### Fixed
+
+- Prevented Claude `is_error: true` result diagnostic from being mislabeled as assistant output on terminal error.
+- Supported Pi `responseId` in `inspectRun` event-stream projection and added regression coverage for live Pi event inspection.
+- Handled surrogate-pair boundaries when slicing the interrupted output tail and shared the `OUTPUT_PREVIEW_CHARS` constant across spawn and workflow receipts.
+- Pinned spawn-level timeout rewrite behavior and workflow script `assistantOutput` error propagation in automated tests.
+
 ## [2.1.0-external.2] - 2026-09-18
 
 ### Fixed
@@ -13,6 +22,8 @@ All notable changes to pi-flow external are documented here.
 - `ChildRunError` preserves `partialOutput` and `assistantOutput` for workflow scripts to inspect, and uncaught child errors surface the interrupted output preview in workflow failure receipts.
 - Hardened CI release workflow polling loop against npm registry replication latency (increased timeout to 150s).
 
+## [2.1.0-external.1] - 2026-09-16
+
 ### Fixed
 
 - Coordinator guidance now names all four harnesses (`Agent` description, `buildCoordinatorPrompt` body): registered Pi harnesses are first-class alongside Claude Code/Codex CLI/Antigravity, not "external CLIs only" leftovers of the pre-2.1.0 copy. `test/agent-contract`/`agent-rendering` pin the copy.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tranhoangnguyen0310/pi-flow-external",
-  "version": "2.1.0-external.2",
+  "version": "2.1.0-external.3",
   "description": "External Claude Code, Codex CLI, and Antigravity delegation for pi.",
   "type": "module",
   "main": "./index.ts",

package/src/core/claude.ts

@@ -243,6 +243,9 @@ function structuredTextFromClaudeValue(value: unknown): string | undefined {
 
 export function extractClaudeFinalText(event: Record<string, unknown>): string | undefined {
   if (event.type === "result") {
+    if (event.is_error === true) {
+      return undefined;
+    }
     return (
       structuredTextFromClaudeValue(event.structured_output) ??
       (typeof event.result === "string" ? event.result : undefined)

package/src/core/progress.ts

@@ -51,10 +51,17 @@ export function assistantOutput(
   return output.length ? { status, messages: output } : undefined;
 }
 
+export const OUTPUT_PREVIEW_CHARS = 4_000;
+
 export function formatInterruptedOutputPreview(output: SubagentAssistantOutput | undefined): string {
   if (!output?.messages?.length) return "";
   const text = output.messages.map((m) => m.text.trim()).filter(Boolean).join("\n\n");
-  return text ? `\n\nInterrupted output:\n${text.slice(-4_000)}` : "";
+  if (!text) return "";
+  let slice = text.slice(-OUTPUT_PREVIEW_CHARS);
+  if (slice.length > 0 && slice.charCodeAt(0) >= 0xdc00 && slice.charCodeAt(0) <= 0xdfff) {
+    slice = slice.slice(1);
+  }
+  return `\n\nInterrupted output:\n${slice}`;
 }
 
 export function extractAssistantMessages(messages: readonly unknown[]): Array<{ id?: string; text: string }> {

package/src/core/run-inspection.ts

@@ -373,7 +373,7 @@ function outputFromEvent(event: EvidenceEvent): ({ kind: "claude" | "codex" | "a
     const message = asRecord(backendEvent.message);
     if (message?.role === "assistant") {
       const text = extractTextContent(message.content);
-      const id = asString(message.id);
+      const id = asString(message.id) ?? asString(message.responseId);
       return text ? { kind: "pi", ...(id ? { id } : {}), text } : undefined;
     }
   }

package/src/workflow/tool.ts

@@ -17,6 +17,7 @@ import { createRunRecord } from "../core/run-record.ts";
 import { runRecordsDirectory } from "../core/retention.ts";
 import { captureParentContext } from "../core/parent-context.ts";
 import { RunRegistry } from "../core/run-registry.ts";
+import { OUTPUT_PREVIEW_CHARS } from "../core/progress.ts";
 import { filterExternalAgentProfiles, getSubagentProfiles, mergeSynthesizedPiProfiles, resolveExternalProfile } from "../profiles.ts";
 import { loadHarnessConfigs } from "../harnesses.ts";
 import { WORKFLOW_PROMPT_SNIPPET } from "../prompts.ts";
@@ -532,7 +533,7 @@ export function createWorkflowTool(
           }
         }
         const partial = error instanceof ChildRunError && error.partialOutput
-          ? `\n\nInterrupted child output (${error.runId}):\n${error.partialOutput.slice(-4_000)}`
+          ? `\n\nInterrupted child output (${error.runId}):\n${error.partialOutput.slice(-OUTPUT_PREVIEW_CHARS)}`
           : "";
         return workflowResult(
           `Workflow "${metaName}" ${aborted ? "aborted" : "failed"}: ${message}${partial}${formatRecentLogs(snapshot.logs)}`,