@tranhoangnguyen0310/pi-flow-external 5.1.1-external.0 → 5.2.0-external.0

package/CHANGELOG.md

@@ -2,6 +2,15 @@
 
 All notable changes to pi-flow external are documented here.
 
+## [5.2.0-external.0] - 2026-10-20
+
+### Changed
+- Requires Pi 1.3: peer dependencies on `@earendil-works/pi-agent-core`, `pi-ai`, `pi-coding-agent` and `pi-tui` move from `~0.99.2` to `~1.3.0`.
+- Node 26 is supported and tested in CI. `npm run e2e` no longer passes `--experimental-transform-types`, which Node 26 removed; the sources now use erasable TypeScript syntax only and load through Node's built-in type stripping.
+
+### Fixed
+- `external_delegate` and `external_workflow` no longer disappear from a session saved before the #85 rename. Pi restores a resumed or tree-navigated session's tool loadout from its transcript, and the legacy names `Agent` and `workflow` no longer register. Each legacy name the restored loadout declares now activates its replacement. A launcher stays off when the loadout also declares its current name, or when `--tools -<name>` or `--no-tools` excludes it.
+
 ## [5.1.1-external.0] - 2026-10-20
 
 ### Changed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tranhoangnguyen0310/pi-flow-external",
-  "version": "5.1.1-external.0",
+  "version": "5.2.0-external.0",
   "description": "External Claude Code, Codex CLI, Antigravity, Grok Build CLI, Muse Code, and OpenCode delegation for pi.",
   "type": "module",
   "main": "./index.ts",
@@ -41,22 +41,22 @@
   "scripts": {
     "build": "tsc",
     "check": "tsc --noEmit && vitest run",
-    "e2e": "node --experimental-transform-types scripts/e2e/external.mjs",
+    "e2e": "node scripts/e2e/external.mjs",
     "field-report": "node scripts/field-report.mjs",
     "test": "vitest run"
   },
   "peerDependencies": {
-    "@earendil-works/pi-agent-core": "~0.99.2",
-    "@earendil-works/pi-ai": "~0.99.2",
-    "@earendil-works/pi-coding-agent": "~0.99.2",
-    "@earendil-works/pi-tui": "~0.99.2",
+    "@earendil-works/pi-agent-core": "~1.3.0",
+    "@earendil-works/pi-ai": "~1.3.0",
+    "@earendil-works/pi-coding-agent": "~1.3.0",
+    "@earendil-works/pi-tui": "~1.3.0",
     "typebox": "*"
   },
   "devDependencies": {
-    "@earendil-works/pi-agent-core": "0.99.2",
-    "@earendil-works/pi-ai": "0.99.2",
-    "@earendil-works/pi-coding-agent": "0.99.2",
-    "@earendil-works/pi-tui": "0.99.2",
+    "@earendil-works/pi-agent-core": "1.3.0",
+    "@earendil-works/pi-ai": "1.3.0",
+    "@earendil-works/pi-coding-agent": "1.3.0",
+    "@earendil-works/pi-tui": "1.3.0",
     "@types/node": "^24.12.4",
     "typebox": "1.1.38",
     "typescript": "^5.9.3",

package/src/config-modal.ts

@@ -23,7 +23,13 @@ export class ConfigModal implements Component {
   private remember = false;
   private cancelBusy?: () => void;
   focused = true;
-  constructor(private tui: TUI, private theme: Theme, private cancelled: (data:string)=>boolean = data=>matchesKey(data,Key.escape), private externalEditorKey:(data:string)=>boolean = ()=>false) {}
+  private tui: TUI;
+  private theme: Theme;
+  private cancelled: (data:string)=>boolean;
+  private externalEditorKey: (data:string)=>boolean;
+  constructor(tui: TUI, theme: Theme, cancelled: (data:string)=>boolean = data=>matchesKey(data,Key.escape), externalEditorKey:(data:string)=>boolean = ()=>false) {
+    this.tui = tui; this.theme = theme; this.cancelled = cancelled; this.externalEditorKey = externalEditorKey;
+  }
   private listTheme() {
     return { selectedPrefix: (s:string)=>this.theme.fg('accent',s), selectedText: (s:string)=>this.theme.fg('accent',s), description:(s:string)=>this.theme.fg('muted',s),scrollInfo:(s:string)=>this.theme.fg('dim',s),noMatch:(s:string)=>this.theme.fg('warning',s) };
   }

package/src/core/concurrency.ts

@@ -42,10 +42,13 @@ export class ConcurrencyLimiter {
   private active = 0;
   private readonly waiters: Waiter[] = [];
 
-  constructor(readonly max: number) {
+  readonly max: number;
+
+  constructor(max: number) {
     if (!Number.isInteger(max) || max < 1) {
       throw new Error("ConcurrencyLimiter max must be a positive integer");
     }
+    this.max = max;
   }
 
   get activeCount(): number {

package/src/core/errors.ts

@@ -8,8 +8,11 @@ export type FlowErrorCode = (typeof FLOW_ERROR_CODES)[number];
 
 /** Expected caller/configuration failure, distinct from a programming error. */
 export class ExpectedFlowError extends Error {
-  constructor(readonly code: FlowErrorCode, message: string) {
+  readonly code: FlowErrorCode;
+
+  constructor(code: FlowErrorCode, message: string) {
     super(message);
+    this.code = code;
     this.name = "ExpectedFlowError";
   }
 }

package/src/core/run-registry.ts

@@ -80,11 +80,16 @@ export class RunRegistry {
   private readonly closedSessions = new Set<string>();
   private readonly backgroundListeners = new Set<(event: BackgroundRunEvent) => void>();
 
-  constructor(private readonly completedLimit = DEFAULT_COMPLETED_LIMIT, private readonly shutdownGraceMs = 4_000) {
+  private readonly completedLimit: number;
+  private readonly shutdownGraceMs: number;
+
+  constructor(completedLimit = DEFAULT_COMPLETED_LIMIT, shutdownGraceMs = 4_000) {
     if (!Number.isInteger(completedLimit) || completedLimit < 0) {
       throw new Error("completed run limit must be a non-negative integer");
     }
     if (!Number.isInteger(shutdownGraceMs) || shutdownGraceMs < 0) throw new Error("shutdown grace must be a non-negative integer");
+    this.completedLimit = completedLimit;
+    this.shutdownGraceMs = shutdownGraceMs;
   }
 
   sessionVersion(sessionId: string): number {

package/src/external-command.ts

@@ -1090,8 +1090,11 @@ type ExternalRunsResult = { content: Array<{ type: string; text?: string }>; det
 
 /** A returned external_runs failure, raised with its code so callers recover by code, never by message. */
 class RunsActionError extends Error {
-  constructor(readonly code: string, message: string) {
+  readonly code: string;
+
+  constructor(code: string, message: string) {
     super(message);
+    this.code = code;
   }
 }
 

package/src/pi-subagent.ts

@@ -17,6 +17,7 @@ import { resolve, join } from "node:path";
 import { resolveExecutionProfile } from './execution-config.ts';
 import { mkdtemp, rm } from 'node:fs/promises';
 import { tmpdir } from 'node:os';
+import { getCurrentSystemMessage } from "@earendil-works/pi-ai";
 import { Text } from "@earendil-works/pi-tui";
 import { Type, type Static } from "typebox";
 import {
@@ -51,7 +52,7 @@ import { registerProfileCreator, startRoleInterview, startHarnessInterview } fro
 import { registerExternalCommand } from "./external-command.ts";
 import { DEFAULT_EXTERNAL_SETTINGS, loadExternalSettings, resolveCtxDefaultHarness, resolveExternalSettings, renderDefaultHarness } from "./settings.ts";
 import { EXTERNAL_HARNESSES } from "./types.ts";
-import { EXTERNAL_DELEGATE_TOOL } from "./tool-names.ts";
+import { EXTERNAL_DELEGATE_TOOL, EXTERNAL_WORKFLOW_TOOL } from "./tool-names.ts";
 import type {
   PermissionTier,
   SubagentBackend,
@@ -257,6 +258,45 @@ function isSubagentProgressNode(value: unknown): value is SubagentProgressNode {
   );
 }
 
+// Launcher names a transcript saved before the #85 rename declares, and their replacements.
+const RENAMED_LAUNCHERS: ReadonlyArray<readonly [legacy: string, current: string]> = [
+  ["Agent", EXTERNAL_DELEGATE_TOOL],
+  ["workflow", EXTERNAL_WORKFLOW_TOOL],
+];
+
+// Launchers the user kept off on the command line. Pi applies `--tools` modifiers
+// to the restored loadout but does not expose them to extensions, and a `-name`
+// modifier cannot match the legacy name it replaces. `--no-tools` turns
+// extension tools off by default. SDK `tools` options are not visible here.
+function cliExcludedLaunchers(argv: readonly string[]): Set<string> {
+  if (argv.includes("--no-tools") || argv.includes("-nt")) return new Set(RENAMED_LAUNCHERS.map(([, current]) => current));
+  let tools: string[] = [];
+  argv.forEach((arg, index) => {
+    if ((arg === "--tools" || arg === "-t") && index + 1 < argv.length) {
+      tools = argv[index + 1].split(",").map((name) => name.trim());
+    }
+  });
+  return new Set(tools.filter((name) => name.startsWith("-")).map((name) => name.slice(1)));
+}
+
+// Pi restores a resumed or tree-navigated session's tool loadout from its
+// transcript. A legacy name no longer registers, so its replacement stays off.
+// Activate the replacement for each legacy name the restored loadout declares,
+// unless the loadout also declares the replacement (it was then turned off on
+// purpose) or the command line excludes it. setActiveTools ignores a tool that
+// is not registered or not allowed.
+function restoreRenamedLaunchers(pi: ExtensionAPI, ctx: ExtensionContext, argv: readonly string[] = process.argv): void {
+  const declared = new Set(
+    (getCurrentSystemMessage(ctx.sessionManager.buildSessionProjection().messages)?.toolsAdded ?? []).map((tool) => tool.name),
+  );
+  const active = pi.getActiveTools();
+  const excluded = cliExcludedLaunchers(argv);
+  const restored = RENAMED_LAUNCHERS
+    .filter(([legacy, current]) => declared.has(legacy) && !declared.has(current) && !active.includes(current) && !excluded.has(current))
+    .map(([, current]) => current);
+  if (restored.length > 0) pi.setActiveTools([...active, ...restored]);
+}
+
 function createUsageStatusState(): SubagentUsageStatusState {
   return {
     calls: new Map(),
@@ -1019,7 +1059,9 @@ export function createSubagentExtension(options: SubagentExtensionOptions = {}):
       if (activeSessionId) scheduleWakeFlush(activeSessionId);
     });
 
+    pi.on("session_tree", (_event, ctx) => restoreRenamedLaunchers(pi, ctx));
     pi.on("session_start", (_event, ctx) => {
+      restoreRenamedLaunchers(pi, ctx);
       const sessionId = ctx.sessionManager.getSessionId();
       rememberSessionContext(ctx, sessionId);
       inAgentTurn = false;

package/src/workflow/journal.ts

@@ -48,8 +48,13 @@ export interface LoadedWorkflowJournal {
 
 /** A journal that exists but cannot be loaded; `project` is set when its own run_start line names one. */
 export class WorkflowJournalReadError extends ExpectedFlowError {
-  constructor(message: string, readonly project: string | undefined, readonly integrity: "damaged" | "unknown") {
+  readonly project: string | undefined;
+  readonly integrity: "damaged" | "unknown";
+
+  constructor(message: string, project: string | undefined, integrity: "damaged" | "unknown") {
     super("run_unavailable", message);
+    this.project = project;
+    this.integrity = integrity;
     this.name = "WorkflowJournalReadError";
   }
 }