@henryqw/pi-subagent 27.10.0 → 27.10.1

package/dist/composition.js

@@ -1,12 +1,9 @@
 import { lstat, realpath } from "node:fs/promises";
-import { isAbsolute, normalize, relative, sep } from "node:path";
+import { isAbsolute, normalize } from "node:path";
 import { runProcess as defaultRunProcess } from "./process.js";
+import { isWithin } from "./worktree.js";
 const GIT_ROOT_TIMEOUT_CAP_MS = 30_000;
 const REVIEW_PROMPT_MAX_BYTES = 64 * 1024;
-function within(root, candidate) {
-    const fromRoot = relative(root, candidate);
-    return fromRoot === "" || (fromRoot !== ".." && !fromRoot.startsWith(`..${sep}`) && !isAbsolute(fromRoot));
-}
 function processFailure(result) {
     const detail = (result.stderr || result.stdout).trim().slice(0, 1_000);
     return new Error(`git rev-parse --show-toplevel failed with exit ${result.code}${detail ? `: ${detail}` : ""}`);
@@ -45,7 +42,7 @@ export function createCanonicalGitRootResolver(options = {}) {
         if (canonicalRoot !== root || !rootInfo.isDirectory()) {
             throw new Error("Git returned a non-canonical repository root.");
         }
-        if (!within(root, canonicalCwd)) {
+        if (!isWithin(root, canonicalCwd)) {
             throw new Error("Git repository root does not identify the requested working directory.");
         }
         return root;

package/dist/direct-herdr.js

@@ -5,7 +5,7 @@ import { tmpdir } from "node:os";
 import { join } from "node:path";
 import { setTimeout as sleep } from "node:timers/promises";
 import { createHerdrClient, hasHerdrErrorCode, herdrCommandFailure, startPiAgent } from "@henryqw/pi-herdr";
-import { assertPrivateLease, scanProcessLease } from "./process-lease.js";
+import { assertPrivateLease, PI_SUBAGENT_PROCESS_LEASE, scanProcessLease } from "./process-lease.js";
 const OPERATION_MS = 30_000;
 const SESSION_LIMIT = 16 * 1024 * 1024;
 const ANSWER_LIMIT = 50 * 1024;
@@ -384,7 +384,7 @@ export function createDirectHerdr(pi, cwd, idleMs) {
             let tabId;
             let paneId;
             try {
-                const response = await herdr.json(["tab", "create", "--workspace", workspaceId, "--cwd", workingDir, "--label", label, "--env", `PI_SUBAGENT_PROCESS_LEASE=${leasePath}`, "--no-focus"], options());
+                const response = await herdr.json(["tab", "create", "--workspace", workspaceId, "--cwd", workingDir, "--label", label, "--env", `${PI_SUBAGENT_PROCESS_LEASE}=${leasePath}`, "--no-focus"], options());
                 const created = object(response.result, "tab allocation result");
                 // Capture every returned ID before validation can fail. These are evidence,
                 // not owned tabs; the caller retains this intent even without either ID.

package/dist/git-runtime.js

@@ -1,5 +1,5 @@
 import { lstat, realpath } from "node:fs/promises";
-import { createChildWorktree, inspectIndexFlags, inspectWorktreeDirty, WorktreeSetupError, } from "./worktree.js";
+import { createChildWorktree, inspectIndexFlags, inspectWorktreeDirty, isMissing, WorktreeSetupError, } from "./worktree.js";
 import { prepareExactReviewEvidence } from "./review-evidence.js";
 import { sameIdentity, } from "./schema.js";
 import { withTransientLaunch, } from "./runner.js";
@@ -41,16 +41,13 @@ function cloneWorktree(info) {
         baseCommit: info.baseCommit.toLowerCase(),
     };
 }
-function pathMissing(error) {
-    return Boolean(error && typeof error === "object" && error.code === "ENOENT");
-}
 async function pathExists(path) {
     try {
         await lstat(path);
         return true;
     }
     catch (error) {
-        if (pathMissing(error))
+        if (isMissing(error))
             return false;
         throw error;
     }
@@ -457,7 +454,7 @@ export class CheckedGitRuntime {
                 return await realpath(path);
             }
             catch (error) {
-                if (pathMissing(error))
+                if (isMissing(error))
                     return path;
                 throw error;
             }

package/dist/herdr-runtime.js

@@ -9,7 +9,7 @@ import { isCleanCommitted, } from "./schema.js";
 import { buildChangesetTaskPrompt, withTransientLaunch, } from "./runner.js";
 import { runProcess as defaultRunProcess } from "./process.js";
 import { EXECUTION_BUDGET_ENV } from "./ephemeral.js";
-import { assertPrivateLease, scanProcessLease } from "./process-lease.js";
+import { assertPrivateLease, PI_SUBAGENT_PROCESS_LEASE, scanProcessLease } from "./process-lease.js";
 import { exactDirectTerminalTurn } from "./direct-herdr.js";
 import { READABLE_BRANCH_PATTERN } from "./worktree.js";
 const MIN_HERDR_VERSION = [0, 9, 3];
@@ -25,7 +25,6 @@ const DIAGNOSTIC_LIMIT = 8 * 1024;
 const SESSION_LIMIT = 16 * 1024 * 1024;
 const LEASE_MODE = 0o600;
 const DIRECTORY_MODE = 0o700;
-const PROCESS_LEASE_ENV = "PI_SUBAGENT_PROCESS_LEASE";
 const CORRELATION_TOKEN_PATTERN = /^[A-Za-z0-9_-]{16,128}$/;
 const HERDR_AGENT_NAME_PATTERN = /^[a-z][a-z0-9_-]{0,31}$/;
 const SETTLED_AGENT_STATES = new Set(["idle", "done"]);
@@ -803,7 +802,7 @@ export class HerdrHostRuntime {
         const budget = this.executionBudget?.();
         const args = [
             "tab", "create", "--workspace", allocation.workspaceId, "--cwd", allocation.worktreeCwd,
-            "--label", allocation.label, "--env", `${PROCESS_LEASE_ENV}=${allocation.leasePath}`,
+            "--label", allocation.label, "--env", `${PI_SUBAGENT_PROCESS_LEASE}=${allocation.leasePath}`,
             ...(budget ? ["--env", `${EXECUTION_BUDGET_ENV}=${JSON.stringify({ ...budget, startedAt: this.now() })}`] : []),
             "--no-focus",
         ];

package/dist/index.d.ts

@@ -5,7 +5,7 @@ export { loadRoleMcpConfig, parseRoleMcpAllowlist } from "./mcp-role.ts";
 export { addUsage, capEphemeralSubagentOutput, createEphemeralSubagentExecutor, DEFAULT_MAX_TURNS, EphemeralSubagentError, EXECUTION_BUDGET_ENV, formatDuration, type EphemeralSubagentActivityEvent, type EphemeralSubagentErrorCode, type EphemeralSubagentExecutionBudget, type EphemeralSubagentExecutor, type EphemeralSubagentExecutorOptions, type EphemeralSubagentResult, type EphemeralSubagentRunInput, type EphemeralSubagentTimeout, } from "./ephemeral.ts";
 export { createChildWorktree, finalizeChildWorktree, inspectIndexFlags, inspectWorktreeDirty, WorktreeSetupError, type WorktreeDirtyInspection, type WorktreeInfo, type WorktreePayload, } from "./worktree.ts";
 export { prepareExactReviewEvidence, REVIEW_MAX_PATCH_BYTES, REVIEW_MAX_PATHS, type PreparedReviewEvidence, type PrepareExactReviewEvidenceInput, } from "./review-evidence.ts";
-export declare const PI_SUBAGENT_PROCESS_LEASE = "PI_SUBAGENT_PROCESS_LEASE";
+export { PI_SUBAGENT_PROCESS_LEASE } from "./process-lease.ts";
 export declare const ROLE_MCP_POLICY_FLAG = "pi-subagent-role-mcps";
 export declare const ROLE_TOOL_POLICY_FLAG = "pi-subagent-role-tools";
 /** Read child MCP policy before Pi binds registered extension flag values. */

package/dist/index.js

@@ -17,7 +17,7 @@ const MULTI_CODEX_EXTENSION = fileURLToPath(import.meta.resolve("@henryqw/pi-mul
 const ROLE_MCP_EXTENSION = fileURLToPath(new URL("../extensions/role-mcp.ts", import.meta.url));
 const ROLE_TOOLS_EXTENSION = fileURLToPath(new URL("../extensions/role-tools.ts", import.meta.url));
 const GIT_READ_EXTENSION = fileURLToPath(new URL("../extensions/git-read.ts", import.meta.url));
-export const PI_SUBAGENT_PROCESS_LEASE = "PI_SUBAGENT_PROCESS_LEASE";
+export { PI_SUBAGENT_PROCESS_LEASE } from "./process-lease.js";
 export const ROLE_MCP_POLICY_FLAG = "pi-subagent-role-mcps";
 export const ROLE_TOOL_POLICY_FLAG = "pi-subagent-role-tools";
 const CODEMODE_TOOL = "codemode";

package/dist/launch-runtime.js

@@ -2,9 +2,10 @@ import { randomBytes } from "node:crypto";
 import { constants } from "node:fs";
 import { chmod, lstat, mkdtemp, open, realpath, rmdir, unlink } from "node:fs/promises";
 import { tmpdir } from "node:os";
-import { isAbsolute, join, normalize, relative, sep } from "node:path";
+import { isAbsolute, join, normalize } from "node:path";
 import { resolveConfiguredRoleLaunch } from "./index.js";
 import { registerModelTask } from "@henryqw/pi-task-models";
+import { isMissing, isWithin } from "./worktree.js";
 const PROMPT_FLAG = "--append-system-prompt";
 const DIRECTORY_MODE = 0o700;
 const PROMPT_MODE = 0o600;
@@ -14,13 +15,6 @@ export const ISOLATED_MODEL_TASK = {
     purpose: "Resolve one task-scoped isolated Role launch.",
     defaultProfile: "balanced",
 };
-function isMissing(error) {
-    return Boolean(error && typeof error === "object" && error.code === "ENOENT");
-}
-function isWithin(root, candidate) {
-    const fromRoot = relative(root, candidate);
-    return fromRoot === "" || (fromRoot !== ".." && !fromRoot.startsWith(`..${sep}`) && !isAbsolute(fromRoot));
-}
 async function removeTransientLaunch(directory, promptPath, promptRequired) {
     const failures = [];
     try {

package/dist/process-lease.d.ts

@@ -1,4 +1,5 @@
 import type { HerdrExecResult } from "@henryqw/pi-herdr";
+export declare const PI_SUBAGENT_PROCESS_LEASE = "PI_SUBAGENT_PROCESS_LEASE";
 export declare function assertPrivateLease(path: string, allowMissing: boolean): Promise<boolean>;
 /** Shared exact process-lease evidence for direct and isolated workers. */
 export declare function scanProcessLease(path: string, execute: (args: string[]) => Promise<HerdrExecResult>, pid?: number, allowMissing?: boolean): Promise<number[]>;

package/dist/process-lease.js

@@ -1,4 +1,5 @@
 import { lstat } from "node:fs/promises";
+export const PI_SUBAGENT_PROCESS_LEASE = "PI_SUBAGENT_PROCESS_LEASE";
 export async function assertPrivateLease(path, allowMissing) {
     let info;
     try {

package/dist/schema.d.ts

@@ -298,7 +298,6 @@ export type CommandEvidence = Static<typeof CommandEvidenceSchema>;
 export type CheckBatchEvidence = Static<typeof CheckBatchEvidenceSchema>;
 export type ReusedCheckSource = NonNullable<CommandEvidence["reusedFrom"]>;
 export type ReviewEvidence = Static<typeof ReviewEvidenceSchema>;
-export type WorkerTermination = Static<typeof WorkerTerminationSchema>;
 export declare const CLEANUP_KINDS: readonly ["worker_tab", "workspace", "worktree", "branch"];
 export type CleanupStep = Static<typeof CleanupStepSchema>;
 export type CleanupKind = CleanupStep["kind"];
@@ -507,18 +506,6 @@ declare const ReviewEvidenceSchema: Type.TObject<{
     passed: Type.TBoolean;
     at: Type.TInteger;
 }>;
-declare const WorkerTerminationSchema: Type.TObject<{
-    status: Type.TUnion<[Type.TLiteral<"terminating">, Type.TLiteral<"terminated">, Type.TLiteral<"unknown">]>;
-    workerId: Type.TString;
-    candidate: Type.TObject<{
-        branch: Type.TString;
-        head: Type.TString;
-        index: Type.TString;
-        tree: Type.TString;
-    }>;
-    at: Type.TOptional<Type.TInteger>;
-    failure: Type.TOptional<Type.TString>;
-}>;
 declare const CleanupStepSchema: Type.TObject<{
     kind: Type.TUnion<[Type.TLiteral<"worker_tab">, Type.TLiteral<"workspace">, Type.TLiteral<"worktree">, Type.TLiteral<"branch">]>;
     status: Type.TUnion<[Type.TLiteral<"pending">, Type.TLiteral<"running">, Type.TLiteral<"completed">]>;
@@ -1304,30 +1291,6 @@ declare const IntegrationCandidateSchema: Type.TObject<{
     worker: Type.TUnion<[Type.TLiteral<"retained">, Type.TLiteral<"release_pending">, Type.TLiteral<"released">]>;
     decision: Type.TOptional<Type.TLiteral<"rejected">>;
 }>;
-declare const IntegrationStageSchema: Type.TObject<{
-    taskId: Type.TString;
-    attempt: Type.TInteger;
-    source: Type.TObject<{
-        branch: Type.TString;
-        head: Type.TString;
-        index: Type.TString;
-        tree: Type.TString;
-    }>;
-    onto: Type.TObject<{
-        branch: Type.TString;
-        head: Type.TString;
-        index: Type.TString;
-        tree: Type.TString;
-    }>;
-    status: Type.TUnion<[Type.TLiteral<"pending">, Type.TLiteral<"staging">, Type.TLiteral<"staged">, Type.TLiteral<"conflict">]>;
-    tip: Type.TOptional<Type.TObject<{
-        branch: Type.TString;
-        head: Type.TString;
-        index: Type.TString;
-        tree: Type.TString;
-    }>>;
-    failure: Type.TOptional<Type.TString>;
-}>;
 declare const IntegrationGenerationSchema: Type.TObject<{
     number: Type.TInteger;
     status: Type.TUnion<[Type.TLiteral<"staging">, Type.TLiteral<"conflict">, Type.TLiteral<"validating">, Type.TLiteral<"validation_failed">, Type.TLiteral<"ready">, Type.TLiteral<"promoting">, Type.TLiteral<"promoted">, Type.TLiteral<"promotion_unknown">, Type.TLiteral<"promotion_failed">, Type.TLiteral<"superseded">]>;
@@ -1788,7 +1751,6 @@ export declare const IntegrationStateSchema: Type.TObject<{
     }>>;
 }>;
 export type IntegrationCandidate = Static<typeof IntegrationCandidateSchema>;
-export type IntegrationStage = Static<typeof IntegrationStageSchema>;
 export type IntegrationGeneration = Static<typeof IntegrationGenerationSchema>;
 export type IntegrationState = Static<typeof IntegrationStateSchema>;
 declare const TerminalRunStatusSchema: Type.TUnion<[Type.TLiteral<"completed">, Type.TLiteral<"final_failed">, Type.TLiteral<"superseded">, Type.TLiteral<"aborted">]>;

package/dist/store.js

@@ -1,11 +1,12 @@
 import { createHash } from "node:crypto";
 import { realpathSync } from "node:fs";
 import { lstat, mkdir, open, readdir, realpath } from "node:fs/promises";
-import { basename, dirname, isAbsolute, join, relative, resolve, sep } from "node:path";
+import { basename, dirname, isAbsolute, join, resolve } from "node:path";
 import { extensionConfigDir, readTextFileBounded, writePrivateTextFileAtomically } from "@henryqw/pi-config-store";
 import { check, lock } from "proper-lockfile";
 import { parseRunState, RUN_STATE_VERSION } from "./schema.js";
 import { discardResources } from "./discard.js";
+import { isMissing, isWithin } from "./worktree.js";
 const INITIAL_STATE_MAX_BYTES = 2 * 1024 * 1024;
 /*
  * create() caps the complete initial serialization at 2 MiB. Loaded requests
@@ -30,19 +31,12 @@ export class ProductiveRunLeaseBusyError extends Error {
         super("Another Pi Subagent productive request is active in this repository.");
     }
 }
-function isMissing(error) {
-    return Boolean(error && typeof error === "object" && error.code === "ENOENT");
-}
 function isAlreadyPresent(error) {
     return Boolean(error && typeof error === "object" && error.code === "EEXIST");
 }
 function isLocked(error) {
     return Boolean(error && typeof error === "object" && error.code === "ELOCKED");
 }
-function isWithin(root, candidate) {
-    const fromRoot = relative(root, candidate);
-    return fromRoot === "" || (fromRoot !== ".." && !fromRoot.startsWith(`..${sep}`) && !isAbsolute(fromRoot));
-}
 async function canonicalPlannedPath(path) {
     let ancestor = resolve(path);
     const suffix = [];

package/dist/worktree.d.ts

@@ -1,4 +1,6 @@
 import { type GitRunner } from "./git-process.ts";
+export declare function isMissing(error: unknown): boolean;
+export declare function isWithin(root: string, candidate: string): boolean;
 export interface WorktreeInfo {
     path: string;
     cwd: string;

package/dist/worktree.js

@@ -1,8 +1,15 @@
 import { createHash } from "node:crypto";
 import { existsSync, lstatSync } from "node:fs";
 import { appendFile, mkdir, readFile } from "node:fs/promises";
-import { dirname, isAbsolute, join, resolve } from "node:path";
+import { dirname, isAbsolute, join, relative, resolve, sep } from "node:path";
 import { runGit } from "./git-process.js";
+export function isMissing(error) {
+    return Boolean(error && typeof error === "object" && error.code === "ENOENT");
+}
+export function isWithin(root, candidate) {
+    const fromRoot = relative(root, candidate);
+    return fromRoot === "" || (fromRoot !== ".." && !fromRoot.startsWith(`..${sep}`) && !isAbsolute(fromRoot));
+}
 export const WORKTREES_DIRNAME = ".worktrees";
 const BRANCH_NAMESPACE = "pi-subagent";
 const NAME_COMPONENT_LIMIT = 24;

package/extensions/config.ts

@@ -2,10 +2,8 @@ import { formatConfigIssues, type ConfigLoad } from "@henryqw/pi-config-store";
 import type { ExecutionPolicySnapshot } from "../dist/schema.js";
 import type { SubagentConfig } from "@henryqw/pi-subagent";
 
-export type EffectiveExecutionPolicy = ExecutionPolicySnapshot;
-
 /** Resolve one immutable policy. A disabled config blocks delegation. */
-export function resolveExecutionPolicy(path: string, loaded: ConfigLoad<SubagentConfig>): EffectiveExecutionPolicy {
+export function resolveExecutionPolicy(path: string, loaded: ConfigLoad<SubagentConfig>): ExecutionPolicySnapshot {
 	if (loaded.status === "disabled") {
 		throw new Error(`${path}: ${formatConfigIssues(loaded.issues)}; delegation is blocked until this file is corrected.`);
 	}

package/extensions/isolated.ts

@@ -3,7 +3,6 @@ import type { JsonValue } from "@earendil-works/pi-ai";
 import type { ExtensionAPI, ExtensionContext } from "@earendil-works/pi-coding-agent";
 import { truncateToWidth } from "@earendil-works/pi-tui";
 import { capEphemeralSubagentOutput, utf8Prefix, type EphemeralSubagentExecutor } from "../dist/ephemeral.js";
-import type { EffectiveExecutionPolicy } from "./config.ts";
 import {
 	createCanonicalGitRootResolver,
 	createExactJudgmentExecutor,
@@ -31,6 +30,7 @@ import {
 	StatusOutputSchema,
 	type CheckBatchEvidence,
 	type ExecuteRequest,
+	type ExecutionPolicySnapshot,
 	type ModelClass,
 	type ReviewEvidence,
 	type RunState,
@@ -59,8 +59,8 @@ export type CreateIsolatedComponents = (options: {
 	pi: ExtensionAPI;
 	context(): ExtensionContext;
 	executor: EphemeralSubagentExecutor;
-	policy: EffectiveExecutionPolicy;
-	currentPolicy(): EffectiveExecutionPolicy;
+	policy: ExecutionPolicySnapshot;
+	currentPolicy(): ExecutionPolicySnapshot;
 	onStateSaved(state: RunState): void;
 }) => IsolatedExtensionComponents;
 
@@ -531,8 +531,8 @@ export interface IsolatedSurface {
 
 export interface RegisterIsolatedOptions {
 	executor: EphemeralSubagentExecutor;
-	policy: EffectiveExecutionPolicy;
-	currentPolicy(): EffectiveExecutionPolicy;
+	policy: ExecutionPolicySnapshot;
+	currentPolicy(): ExecutionPolicySnapshot;
 	/** Report new config issues, and throw when the config now blocks delegation, before each deferred Role launch. */
 	beforeRoleLaunch?(ctx: ExtensionContext): void;
 	componentsFactory?: CreateIsolatedComponents;

package/extensions/result-transport.ts

@@ -149,12 +149,12 @@ function sourceFor(entry: WorkflowTransportEntry): string | undefined {
 	if (entry.status === "rejected") {
 		return entry.partialOutput ? `${entry.failure}\nPartial output before failure (incomplete; not a success):\n${entry.partialOutput}` : entry.failure;
 	}
-	if (entry.status === "running" || entry.status === "succeeded") return entry.assistantOutput;
+	if (entry.status === "succeeded") return entry.assistantOutput;
 }
 
 function evidenceFor(entry: WorkflowTransportEntry, position: number, total: number): Evidence | undefined {
 	const source = sourceFor(entry);
-	if (source === undefined || !source) return;
+	if (!source) return;
 	const [preview, remainder] = splitEvidence(source);
 	return {
 		heading: `- [${position}/${total}] ${entry.name} · ${entry.role} · ${entry.status === "rejected" ? "failure" : "result"}:`,

package/extensions/subagent.ts

@@ -3,8 +3,8 @@ import type { ExtensionAPI, ExtensionContext, Theme } from "@earendil-works/pi-c
 import { type Component, Text, type TUI, truncateToWidth, visibleWidth } from "@earendil-works/pi-tui";
 import { availableTaskModels, loadTaskModelsConfig, modelReference, type ProfileName, registerModelTask, registerTaskModelsConfigReporting, resolveAvailableModel, resolveConfiguredTaskRoute, type ResolvedTaskRoute, taskThinkingLevels } from "@henryqw/pi-task-models";
 import { registerConfigReporting } from "@henryqw/pi-config-store";
-import { capEphemeralSubagentOutput as capOutput, createEphemeralSubagentExecutor, createSubagentConfigStore, DEFAULT_SUBAGENT_CONFIG, DELEGATE_TASK, findRole, formatDuration, loadRoleCatalog, prepareRoleLaunch, resolveRolePackageResources, resolveTaskRoute, ROLE_TOOL_POLICY_FLAG, type Role } from "@henryqw/pi-subagent";
-import { resolveExecutionPolicy, type EffectiveExecutionPolicy } from "./config.ts";
+import { capEphemeralSubagentOutput as capOutput, createEphemeralSubagentExecutor, createSubagentConfigStore, DEFAULT_SUBAGENT_CONFIG, DELEGATE_TASK, findRole, formatDuration, loadRoleCatalog, prepareRoleLaunch, resolveRolePackageResources, resolveTaskRoute, ROLE_TOOL_POLICY_FLAG, type ExecutionPolicySnapshot, type Role } from "@henryqw/pi-subagent";
+import { resolveExecutionPolicy } from "./config.ts";
 import { registerCheckoutAdmission } from "./admission.ts";
 import { registerIsolatedExtension } from "./isolated.ts";
 import { directRef, registerSubagentCommand, type DirectDisplay, type DirectTask } from "./subagent-command.ts";
@@ -204,7 +204,7 @@ export default function subagentExtension(pi: ExtensionAPI): void {
 	});
 	// The isolated runner reads the policy lazily without a context, so each read reports through the session's.
 	let latestCtx: ExtensionContext | undefined;
-	const currentPolicy = (): EffectiveExecutionPolicy => {
+	const currentPolicy = (): ExecutionPolicySnapshot => {
 		const loaded = configStore.loadSync();
 		if (latestCtx) configReporter.report(latestCtx, configStore, loaded);
 		const policy = resolveExecutionPolicy(configStore.path, loaded);
@@ -407,8 +407,8 @@ export default function subagentExtension(pi: ExtensionAPI): void {
 					const resolved = role.modelClass === undefined ? resolveConfiguredTaskRoute(ctx, DELEGATE_TASK) : resolveTaskRoute(ctx, role.modelClass);
 					route = { model: modelReference(resolved.model), thinking: resolved.thinkingLevel };
 				} catch (error) { route = { error: message(error) }; }
-				return { name: role.name, source: catalog.sources.get(role.name) ?? "unknown", modelClass: role.modelClass, route,
-					tools: role.tools ?? [], extensions: role.extensions ?? [], skills: role.skills ?? [], mcps: role.mcps ?? [] };
+				return { name: role.name, source: catalog.sources.get(role.name)!, modelClass: role.modelClass, route,
+					tools: role.tools, extensions: role.extensions, skills: role.skills, mcps: role.mcps ?? [] };
 			});
 			return { roles, errors: catalog.errors.map((error) => error.message) };
 		},

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@henryqw/pi-subagent",
-  "version": "27.10.0",
+  "version": "27.10.1",
   "description": "Delegate bounded direct work or durable checked isolated task graphs to Pi roles.",
   "keywords": [
     "pi-package",