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

package/CHANGELOG.md

@@ -4,6 +4,24 @@ 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
+
+- Preserved and exposed available partial assistant output on failure, cancellation, and timeout across all backend adapters (`codex`, `claude`, `agy`, and in-process `pi`). Unsuccessful terminal receipts retain the failure status and error diagnostic while appending a bounded `Interrupted output:` preview.
+- Recovered interrupted assistant output in run evidence inspection (`external_runs(action: "inspect", view: "output")`) from completed non-done summary documents, including Pi SDK child runs.
+- `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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tranhoangnguyen0310/pi-flow-external",
-  "version": "2.1.0-external.1",
+  "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/agy.ts

@@ -1,6 +1,6 @@
 import { spawn, type ChildProcess } from "node:child_process";
 import type { ExtensionContext } from "@earendil-works/pi-coding-agent";
-import { assistantOutput, createProgressEmitter, textResult, type AgentToolResult } from "./progress.ts";
+import { assistantOutput, createProgressEmitter, formatInterruptedOutputPreview, textResult, type AgentToolResult } from "./progress.ts";
 import {
   createBoundedBuffer,
   MAX_STDERR_CHARS,
@@ -486,7 +486,8 @@ export async function spawnAgySubagent(params: {
     if (child) abortChildTree(child);
     const message = error instanceof Error ? error.message : String(error);
     const status = params.signal?.aborted ? "aborted" : "error";
-    const output = assistantOutput(assistantMessages, "interrupted");
+    const candidate = terminalResult ? textFromAgyResult(terminalResult)?.trim() : undefined;
+    const output = assistantOutput(assistantMessages, "interrupted", candidate);
     params.onUsage(latestUsage);
     if (progress) {
       progress.status = status;
@@ -495,7 +496,8 @@ export async function spawnAgySubagent(params: {
       progress.assistantOutput = output;
       progress.endedAt = Date.now();
     }
-    return textResult(`Subagent "${params.description}" (${subagentType}) ${status === "aborted" ? "aborted" : "failed"}: ${message}`, {
+    const preview = formatInterruptedOutputPreview(output);
+    return textResult(`Subagent "${params.description}" (${subagentType}) ${status === "aborted" ? "aborted" : "failed"}: ${message}${preview}`, {
       description: params.description,
       subagentType,
       backend: params.profile.backend,

package/src/core/claude.ts

@@ -3,6 +3,7 @@ import type { ExtensionContext } from "@earendil-works/pi-coding-agent";
 import {
   assistantOutput,
   createProgressEmitter,
+  formatInterruptedOutputPreview,
   textResult,
   type AgentToolResult,
 } from "./progress.ts";
@@ -242,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)
@@ -570,7 +574,7 @@ export async function spawnClaudeSubagent(params: {
     if (child) abortChildTree(child);
     const message = error instanceof Error ? error.message : String(error);
     const status = params.signal?.aborted ? "aborted" : "error";
-    const output = assistantOutput(assistantMessages, "interrupted");
+    const output = assistantOutput(assistantMessages, "interrupted", resultText);
     params.onUsage(latestUsage);
     if (progress) {
       progress.status = status;
@@ -580,7 +584,8 @@ export async function spawnClaudeSubagent(params: {
       progress.endedAt = Date.now();
     }
     const verb = status === "aborted" ? "aborted" : "failed";
-    return textResult(`Subagent "${params.description}" (${subagentType}) ${verb}: ${message}`, {
+    const preview = formatInterruptedOutputPreview(output);
+    return textResult(`Subagent "${params.description}" (${subagentType}) ${verb}: ${message}${preview}`, {
       description: params.description,
       subagentType,
       backend: params.profile.backend,

package/src/core/codex.ts

@@ -6,6 +6,7 @@ import type { ExtensionContext } from "@earendil-works/pi-coding-agent";
 import {
   assistantOutput,
   createProgressEmitter,
+  formatInterruptedOutputPreview,
   textResult,
   type AgentToolResult,
 } from "./progress.ts";
@@ -533,7 +534,7 @@ export async function spawnCodexSubagent(params: {
     if (child) abortChildTree(child);
     const message = error instanceof Error ? error.message : String(error);
     const status = params.signal?.aborted ? "aborted" : "error";
-    const output = assistantOutput(assistantMessages, "interrupted");
+    const output = assistantOutput(assistantMessages, "interrupted", resultText);
     params.onUsage(latestUsage);
     if (progress) {
       progress.status = status;
@@ -543,7 +544,8 @@ export async function spawnCodexSubagent(params: {
       progress.endedAt = Date.now();
     }
     const verb = status === "aborted" ? "aborted" : "failed";
-    return textResult(`Subagent "${params.description}" (${subagentType}) ${verb}: ${message}`, {
+    const preview = formatInterruptedOutputPreview(output);
+    return textResult(`Subagent "${params.description}" (${subagentType}) ${verb}: ${message}${preview}`, {
       description: params.description,
       subagentType,
       backend: params.profile.backend,

package/src/core/progress.ts

@@ -51,6 +51,19 @@ 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");
+  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 }> {
   return messages.flatMap((value) => {
     const message = value as { id?: unknown; role?: unknown; content?: unknown };
@@ -88,7 +101,7 @@ function getFirstTextLine(text: string): string {
   return text.split("\n").find((line) => line.trim()) ?? text;
 }
 
-function extractTextContent(content: unknown): string {
+export function extractTextContent(content: unknown): string {
   if (typeof content === "string") {
     return content;
   }

package/src/core/run-inspection.ts

@@ -5,6 +5,7 @@ import { join } from "node:path";
 import { agyActivityFromEvent } from "./agy.ts";
 import { claudeActivityFromEvent, extractClaudeFinalText } from "./claude.ts";
 import { codexActivityFromEvent, extractCodexFinalText } from "./codex.ts";
+import { extractTextContent } from "./progress.ts";
 
 const CURSOR_VERSION = 1;
 const DEFAULT_PAGE_BYTES = 32 * 1024;
@@ -117,7 +118,7 @@ export async function inspectRun({
   }
 
   if (decoded?.kind === "inspect" && decoded.source === "summary") throw new Error("Cursor source does not match inspection view");
-  if (view === "output" && summary.document && terminalStatus === "done") {
+  if (view === "output" && summary.document && (terminalStatus === "done" || assistantItems(terminal).length > 0)) {
     if (decoded?.kind === "inspect" && decoded.source === "events") throw staleCursorError();
     const observation = await readObservation(eventsPath, runId);
     const items = terminalOutputItems(terminal);
@@ -347,7 +348,7 @@ function summaryProjection(runId: string, summary: SummaryState, observation: Ru
   };
 }
 
-function outputFromEvent(event: EvidenceEvent): ({ kind: "claude" | "codex" | "agy"; id?: string; text: string }) | undefined {
+function outputFromEvent(event: EvidenceEvent): ({ kind: "claude" | "codex" | "agy" | "pi"; id?: string; text: string }) | undefined {
   if (event.type !== "backend_event") return undefined;
   const envelope = asRecord(event.data);
   const backendEvent = asRecord(envelope?.event);
@@ -368,6 +369,14 @@ function outputFromEvent(event: EvidenceEvent): ({ kind: "claude" | "codex" | "a
     const id = asString(update?.step_id);
     return text ? { kind: "agy", ...(id ? { id } : {}), text } : undefined;
   }
+  if (envelope?.backend === "pi" && backendEvent.type === "message_end") {
+    const message = asRecord(backendEvent.message);
+    if (message?.role === "assistant") {
+      const text = extractTextContent(message.content);
+      const id = asString(message.id) ?? asString(message.responseId);
+      return text ? { kind: "pi", ...(id ? { id } : {}), text } : undefined;
+    }
+  }
   return undefined;
 }
 

package/src/core/spawn.ts

@@ -13,6 +13,7 @@ import {
   extractFinalAssistantText,
   extractAssistantMessages,
   assistantOutput,
+  formatInterruptedOutputPreview,
   getFinalAssistantFailure,
   getSubagentUsage,
   textResult,
@@ -223,8 +224,7 @@ function rewriteTimeoutResult(
 ): AgentToolResult {
   const details = markSubagentTimedOut(result.details as SubagentToolDetails, params.timeoutMs);
   const message = details.error;
-  const partial = details.assistantOutput?.messages.at(-1)?.text;
-  const preview = partial ? `\n\nInterrupted output:\n${partial.slice(-4_000)}` : "";
+  const preview = formatInterruptedOutputPreview(details.assistantOutput);
   return textResult(`Subagent "${params.description}" (${params.profile.name}) aborted: ${message}${preview}`, {
     ...details,
     description: params.description,
@@ -345,11 +345,13 @@ export async function spawnSubagent(params: SpawnSubagentParams): Promise<AgentT
       const details = result.details as SubagentToolDetails;
       if (details.status === "aborted") {
         const reason = params.signal.reason instanceof Error ? params.signal.reason.message : String(params.signal.reason);
-        const previous = details.error;
         details.error = reason;
         if (details.progress) details.progress.error = reason;
         const first = result.content[0];
-        if (first?.type === "text") first.text = previous ? first.text.replace(previous, reason) : `${first.text}: ${reason}`;
+        if (first?.type === "text") {
+          const preview = formatInterruptedOutputPreview(details.assistantOutput);
+          first.text = `Subagent "${params.description}" (${params.profile.name}) aborted: ${reason}${preview}`;
+        }
       }
     }
     if (params.context) {
@@ -829,7 +831,8 @@ async function spawnSubagentRuntime(params: SpawnSubagentRuntimeParams): Promise
       if (thinkingClamped) progress.thinkingClamped = thinkingClamped;
     }
     const verb = status === "aborted" ? "aborted" : "failed";
-    return textResult(`Subagent "${description}" (${subagentType}) ${verb}: ${message}`, {
+    const preview = formatInterruptedOutputPreview(output);
+    return textResult(`Subagent "${description}" (${subagentType}) ${verb}: ${message}${preview}`, {
       description,
       subagentType,
       backend: profile.backend,

package/src/workflow/script-worker.ts

@@ -63,6 +63,8 @@ class ChildRunError extends Error {
     this.outcome = error.outcome;
     this.outputRef = error.outputRef;
     this.diagnosticsRef = error.diagnosticsRef;
+    this.assistantOutput = error.assistantOutput;
+    this.partialOutput = error.partialOutput;
   }
 }
 
@@ -116,6 +118,8 @@ function postError(error) {
         message: error.message,
         ...(error.outputRef ? { outputRef: error.outputRef } : {}),
         ...(error.diagnosticsRef ? { diagnosticsRef: error.diagnosticsRef } : {}),
+        ...(error.assistantOutput ? { assistantOutput: error.assistantOutput } : {}),
+        ...(error.partialOutput ? { partialOutput: error.partialOutput } : {}),
       } } : {}),
     });
   } finally {

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";
@@ -306,12 +307,15 @@ export function createWorkflowTool(
         }
         if (resultDetails.status !== "done") {
           const runId = resultDetails.runId ?? call.runRecord?.runId ?? childId;
+          const partialOutput = resultDetails.assistantOutput?.messages.map((m) => m.text.trim()).filter(Boolean).join("\n\n");
           throw new ChildRunError({
             runId,
             outcome: childOutcome(resultDetails),
             message: resultDetails.error ?? "subagent failed",
             outputRef: { runId, view: "output" },
             diagnosticsRef: { runId, view: "diagnostics" },
+            ...(resultDetails.assistantOutput ? { assistantOutput: resultDetails.assistantOutput } : {}),
+            ...(partialOutput ? { partialOutput } : {}),
           });
         }
         if (externalOutputSchema) {
@@ -528,8 +532,11 @@ export function createWorkflowTool(
             agent.endedAt = Date.now();
           }
         }
+        const partial = error instanceof ChildRunError && error.partialOutput
+          ? `\n\nInterrupted child output (${error.runId}):\n${error.partialOutput.slice(-OUTPUT_PREVIEW_CHARS)}`
+          : "";
         return workflowResult(
-          `Workflow "${metaName}" ${aborted ? "aborted" : "failed"}: ${message}${formatRecentLogs(snapshot.logs)}`,
+          `Workflow "${metaName}" ${aborted ? "aborted" : "failed"}: ${message}${partial}${formatRecentLogs(snapshot.logs)}`,
           cloneSnapshot(snapshot),
         );
       } finally {

package/src/workflow/types.ts

@@ -1,6 +1,7 @@
 import type { ParentContextMessages, ParentContextReceipt } from "../core/parent-context.ts";
 import type { ConcurrencyLimiter } from "../core/concurrency.ts";
 import type { RunRecord } from "../core/run-record.ts";
+import type { SubagentAssistantOutput } from "../types.ts";
 
 export type ChildRunOutcome = "failed" | "cancelled" | "timed_out";
 
@@ -15,6 +16,8 @@ export interface SerializedChildRunError {
   message: string;
   outputRef?: ChildRunReference;
   diagnosticsRef?: ChildRunReference;
+  assistantOutput?: SubagentAssistantOutput;
+  partialOutput?: string;
 }
 
 export class ChildRunError extends Error {
@@ -22,6 +25,8 @@ export class ChildRunError extends Error {
   readonly outcome: ChildRunOutcome;
   readonly outputRef?: ChildRunReference;
   readonly diagnosticsRef?: ChildRunReference;
+  readonly assistantOutput?: SubagentAssistantOutput;
+  readonly partialOutput?: string;
 
   constructor(error: SerializedChildRunError) {
     super(error.message);
@@ -30,6 +35,8 @@ export class ChildRunError extends Error {
     this.outcome = error.outcome;
     this.outputRef = error.outputRef;
     this.diagnosticsRef = error.diagnosticsRef;
+    this.assistantOutput = error.assistantOutput;
+    this.partialOutput = error.partialOutput;
   }
 }
 
@@ -40,6 +47,8 @@ export function childRunErrorData(error: ChildRunError): SerializedChildRunError
     message: error.message,
     ...(error.outputRef ? { outputRef: error.outputRef } : {}),
     ...(error.diagnosticsRef ? { diagnosticsRef: error.diagnosticsRef } : {}),
+    ...(error.assistantOutput ? { assistantOutput: error.assistantOutput } : {}),
+    ...(error.partialOutput ? { partialOutput: error.partialOutput } : {}),
   };
 }