@piewf/cli 5.1.1 → 5.3.0

package/dist/src/bundles.js

@@ -2,13 +2,50 @@ import { chmodSync, cpSync, existsSync, mkdirSync, mkdtempSync, readFileSync, re
 import { basename, dirname, join } from "node:path";
 import { spawnSync } from "node:child_process";
 import { fileURLToPath } from "node:url";
+function isJsonObject(value) {
+    return typeof value === "object" && value !== null && !Array.isArray(value);
+}
+function readJsonObject(path) {
+    try {
+        const value = JSON.parse(readFileSync(path, "utf8"));
+        return isJsonObject(value) ? value : undefined;
+    }
+    catch {
+        return undefined;
+    }
+}
+function readPackageMetadata(path) {
+    const value = readJsonObject(path);
+    if (!value)
+        return undefined;
+    const metadata = {};
+    if (typeof value.name === "string")
+        metadata.name = value.name;
+    if (typeof value.version === "string")
+        metadata.version = value.version;
+    if (typeof value.bin === "string")
+        metadata.bin = value.bin;
+    else if (isJsonObject(value.bin)) {
+        const bin = {};
+        let valid = true;
+        for (const [name, entry] of Object.entries(value.bin)) {
+            if (typeof entry !== "string") {
+                valid = false;
+                break;
+            }
+            bin[name] = entry;
+        }
+        if (valid)
+            metadata.bin = bin;
+    }
+    return metadata;
+}
 function packageJson() {
     const directory = dirname(fileURLToPath(import.meta.url));
     for (const path of [join(directory, "../package.json"), join(directory, "../../package.json")]) {
-        try {
-            return JSON.parse(readFileSync(path, "utf8"));
-        }
-        catch { /* Try the source and built layouts. */ }
+        const metadata = readPackageMetadata(path);
+        if (metadata)
+            return metadata;
     }
     return {};
 }
@@ -300,7 +337,6 @@ function workflowModule(workflow, functionSource, withRoles, aliasTargets, exten
         "  registerWorkflowExtension({",
         `    version: ${JSON.stringify("1.0.0")},`,
         `    headline: ${JSON.stringify("Portable workflow bundle")},`,
-        `    description: ${JSON.stringify(workflow.description)},`,
         ...(withRoles ? [`    roleDirectories: [new URL("./roles", import.meta.url)],`] : []),
         ...(aliases.length ? [`    modelAliases: { ${aliases.map(([name, target]) => `${JSON.stringify(name)}: { resolve: () => ${JSON.stringify(target)} }`).join(", ")} },`] : []),
         "    functions: {",
@@ -354,12 +390,9 @@ function copyResources(root, resources) {
                 throw new Error(`Bundle resource may contain credentials and cannot be selected: ${source}`);
             let name = basename(source);
             if (kind === "dependencies") {
-                try {
-                    const packageName = JSON.parse(readFileSync(join(source, "package.json"), "utf8")).name;
-                    if (typeof packageName === "string" && packageName.trim())
-                        name = packageName;
-                }
-                catch { /* Files can be dependency entry points. */ }
+                const packageName = readPackageMetadata(join(source, "package.json"))?.name;
+                if (typeof packageName === "string" && packageName.trim())
+                    name = packageName;
             }
             if (!name || name === "." || name === ".." || name.includes("\\") || name.startsWith("/"))
                 throw new Error(`Invalid bundle resource name: ${name}`);

package/dist/src/cli.js

@@ -8,17 +8,27 @@ import { ProjectTrustStore, SessionManager, SettingsManager, createAgentSessionF
 import { Value } from "typebox/value";
 import { doctor, doctorExitCode, formatDoctorReport } from "./doctor.js";
 import { doctorCleanup, doctorCleanupExitCode, formatDoctorCleanupReport } from "./doctor-cleanup.js";
-import workflowExtension, { formatWorkflowProgress, loadAgentDefinitions, registeredWorkflowFunctions, truncateWorkflowProgress, workflowCatalog, workflowSettingsPath } from "pi-extensible-workflows";
+import workflowExtension, { formatWorkflowProgress, isNodeError, jsonValue, loadAgentDefinitions, registeredWorkflowFunctions, truncateWorkflowProgress, workflowCatalog, workflowSettingsPath } from "pi-extensible-workflows";
 import { portableEngineVersion, portablePiVersion, writePortableWorkflowBundle } from "./bundles.js";
 import { runSessionInspector, transcriptFileLines } from "./session-inspector.js";
+import { isPersistedRun } from "pi-extensible-workflows/persistence";
 function object(value) { return typeof value === "object" && value !== null && !Array.isArray(value); }
 function has(value, key) { return Object.prototype.hasOwnProperty.call(value, key); }
-function clone(value) { return structuredClone(value); }
+function requiredArg(args, index) {
+    const value = args[index];
+    if (value === undefined)
+        throw new Error("Missing argument");
+    return value;
+}
+function typedOptionKey(options, key) { return Object.prototype.hasOwnProperty.call(options, key); }
+function clone(value) { const cloned = structuredClone(value); if (!jsonValue(cloned))
+    throw new Error("Invalid JSON passed to --input"); return cloned; }
 function kebabCase(value) { return value.replace(/([a-z0-9])([A-Z])/g, "$1-$2").replace(/([A-Z]+)([A-Z][a-z])/g, "$1-$2").replace(/[_\s]+/g, "-").toLowerCase(); }
+function isCliScalar(value) { return value === "string" || value === "integer" || value === "number" || value === "boolean"; }
 function scalarType(schema) {
     if (!object(schema) || typeof schema.type !== "string")
         return undefined;
-    return ["string", "integer", "number", "boolean"].includes(schema.type) ? schema.type : undefined;
+    return isCliScalar(schema.type) ? schema.type : undefined;
 }
 function schemaPlan(schema) {
     if (!object(schema) || schema.type !== "object")
@@ -40,8 +50,14 @@ function schemaPlan(schema) {
     return { fields, ...(requiredScalars.length === 1 ? { positional: requiredScalars[0] } : {}) };
 }
 function scalarLabel(type) { return type === "integer" ? "integer" : type; }
-function scalarFieldType(field) { return field.type === "array" ? field.itemType : field.type; }
-function fieldLabel(field) { return field.type === "array" ? `${field.option} <${scalarLabel(field.itemType)}>` : `${field.option}${field.type === "boolean" ? "" : ` <${scalarLabel(field.type)}>`}`; }
+function scalarFieldType(field) {
+    if (field.type !== "array")
+        return field.type;
+    if (!isCliScalar(field.itemType))
+        throw new Error("Invalid array field");
+    return field.itemType;
+}
+function fieldLabel(field) { return field.type === "array" ? `${field.option} <${scalarLabel(scalarFieldType(field))}>` : `${field.option}${field.type === "boolean" ? "" : ` <${scalarLabel(field.type)}>`}`; }
 function fieldDescription(field) {
     const description = typeof field.schema.description === "string" ? field.schema.description.trim() : "";
     const required = field.required ? "required" : "optional";
@@ -109,15 +125,19 @@ export function parseWorkflowCliArgs(schema, rawArgs) {
     let endOptions = false;
     const assign = (field, raw) => {
         if (field.type === "array") {
-            const values = Array.isArray(result[field.name]) ? result[field.name] : [];
-            values.push(coerce(raw, field.itemType, field.schema.items));
+            const current = result[field.name];
+            const values = Array.isArray(current) ? current : [];
+            const itemSchema = field.schema.items;
+            if (!object(itemSchema))
+                throw new Error("Invalid array field");
+            values.push(coerce(raw, scalarFieldType(field), itemSchema));
             result[field.name] = values;
         }
         else
             result[field.name] = coerce(raw, field.type, field.schema);
     };
     for (let index = 0; index < rawArgs.length; index += 1) {
-        const token = rawArgs[index];
+        const token = requiredArg(rawArgs, index);
         if (token === "--") {
             endOptions = true;
             continue;
@@ -147,7 +167,7 @@ export function parseWorkflowCliArgs(schema, rawArgs) {
                 if (equals >= 0)
                     assign(field, token.slice(equals + 1));
                 else if (rawArgs[index + 1] === "true" || rawArgs[index + 1] === "false")
-                    assign(field, rawArgs[++index]);
+                    assign(field, requiredArg(rawArgs, ++index));
                 else
                     result[field.name] = true;
             }
@@ -222,7 +242,7 @@ export function parseDoctorArgs(rawArgs) {
     let role;
     let prompt;
     for (let index = 0; index < rawArgs.length; index += 1) {
-        const token = rawArgs[index];
+        const token = requiredArg(rawArgs, index);
         const equals = token.indexOf("=");
         const option = equals >= 0 ? token.slice(0, equals) : token;
         if (option === "--role" || option === "--prompt") {
@@ -258,7 +278,7 @@ export function parseDoctorCleanupArgs(rawArgs) {
     let yes = false;
     let seenDays = false;
     for (let index = 0; index < rawArgs.length; index += 1) {
-        const token = rawArgs[index];
+        const token = requiredArg(rawArgs, index);
         if (token === "--yes") {
             yes = true;
             continue;
@@ -302,6 +322,8 @@ function stripTrustOptions(rawArgs) {
     }
     return { args, ...(trustOverride !== undefined ? { trustOverride } : {}) };
 }
+function isHeadlessWorkflowResult(value) { return object(value) && Array.isArray(value.content) && value.content.every((entry) => object(entry) && typeof entry.type === "string" && typeof entry.text === "string"); }
+function isHeadlessWorkflowTool(value) { return object(value) && value.name === "workflow" && typeof value.execute === "function"; }
 async function createWorkflowRuntime(options, shutdownHandlers = []) {
     const cwd = options.cwd ?? process.cwd();
     const agentDir = options.agentDir ?? getAgentDir();
@@ -324,6 +346,8 @@ async function createWorkflowRuntime(options, shutdownHandlers = []) {
             for (const handler of extension.handlers.get("project_trust") ?? []) {
                 try {
                     const result = await handler({ type: "project_trust", cwd }, projectTrustContext);
+                    if (!object(result))
+                        continue;
                     if (result.trusted === "undecided")
                         continue;
                     if (result.trusted !== "yes" && result.trusted !== "no")
@@ -363,7 +387,7 @@ async function createWorkflowRuntime(options, shutdownHandlers = []) {
         events: { emit() { } },
     };
     workflowExtension(headlessPi, homedir(), undefined, undefined, agentDir, options.skillPaths);
-    const workflowTool = tools.find((tool) => object(tool) && tool.name === "workflow");
+    const workflowTool = tools.find(isHeadlessWorkflowTool);
     if (!workflowTool)
         throw new Error("The workflow runtime could not be initialized");
     return { catalog: workflowCatalog({ cwd, projectTrusted: settingsManager.isProjectTrusted(), globalSettingsPath: workflowSettingsPath(agentDir) }), services, workflowTool, shutdownHandlers };
@@ -409,7 +433,7 @@ else { const result = spawnSync("piewf", ["run", ${JSON.stringify(workflowName)}
                 linkSync(tempPath, destination);
             }
             catch (error) {
-                if (error.code === "EEXIST")
+                if (isNodeError(error, "EEXIST"))
                     throw new Error(`Destination already exists: ${destination}; use --force to replace it`, { cause: error });
                 throw error;
             }
@@ -502,13 +526,15 @@ async function invokeWorkflow(fn, args, runtime, options, context) {
         return; announcedRunId = runId; options.stderr(`Run ID: ${runId}\n`); };
     const progress = new CliProgress(options.stderr, options.isTTY ?? process.stderr.isTTY, announceRunId);
     try {
-        const result = await runtime.workflowTool.execute(randomUUID(), { name: fn.name, script: `return await ${fn.name}(args);`, args, foreground: true }, options.signal, (update) => { if (object(update) && object(update.details) && object(update.details.run))
+        const result = await runtime.workflowTool.execute(randomUUID(), { name: fn.name, script: `return await ${fn.name}(args);`, args, foreground: true }, options.signal, (update) => { if (object(update) && object(update.details) && isPersistedRun(update.details.run))
             progress.update(update.details.run); }, context);
+        if (!isHeadlessWorkflowResult(result))
+            throw new Error("Workflow returned an invalid result");
         const details = object(result.details) ? result.details : {};
         const runId = typeof details.runId === "string" ? details.runId : undefined;
         if (runId)
             announceRunId(runId);
-        if (has(details, "value"))
+        if (has(details, "value") && jsonValue(details.value))
             return { value: details.value, ...(runId ? { runId } : {}) };
         const first = result.content[0];
         if (!first || first.type !== "text")
@@ -557,7 +583,7 @@ async function runWorkflowCli(rawArgs, options) {
         options.write(workflowUsage());
         return args.length ? 0 : 1;
     }
-    const name = args[0];
+    const name = requiredArg(args, 0);
     return withWorkflowRuntime({ ...options, ...(parsed.trustOverride !== undefined ? { trustOverride: parsed.trustOverride } : {}) }, async (runtime, context) => {
         const help = args.slice(1).some((arg) => arg === "--help" || arg === "-h");
         const fn = runtime.catalog.functions.find((candidate) => candidate.name === name);
@@ -582,13 +608,13 @@ async function exportWorkflowCli(rawArgs, options) {
         options.write(exportUsage());
         return args.length ? 0 : 1;
     }
-    const workflowName = args[0];
+    const workflowName = requiredArg(args, 0);
     return withWorkflowRuntime({ ...options, ...(parsed.trustOverride !== undefined ? { trustOverride: parsed.trustOverride } : {}) }, async (runtime) => {
         let name;
         let output;
         let force = false;
         for (let index = 1; index < args.length; index += 1) {
-            const arg = args[index];
+            const arg = requiredArg(args, index);
             if (arg === "--force") {
                 force = true;
                 continue;
@@ -640,7 +666,7 @@ async function bundleWorkflowCli(rawArgs, options) {
         options.write(bundleUsage());
         return args.length ? 0 : 1;
     }
-    const workflowName = args[0];
+    const workflowName = requiredArg(args, 0);
     return withWorkflowRuntime({ ...options, ...(parsed.trustOverride !== undefined ? { trustOverride: parsed.trustOverride } : {}) }, async (runtime) => {
         let name;
         let output;
@@ -648,7 +674,7 @@ async function bundleWorkflowCli(rawArgs, options) {
         const requirements = { roles: [], aliases: [], tools: [], commands: [], environment: [] };
         const resources = { extensions: [], skills: [], static: [], dependencies: [] };
         for (let index = 1; index < args.length; index += 1) {
-            const arg = args[index];
+            const arg = requiredArg(args, index);
             if (arg === "--force") {
                 force = true;
                 continue;
@@ -657,7 +683,7 @@ async function bundleWorkflowCli(rawArgs, options) {
             const option = equals >= 0 ? arg.slice(0, equals) : arg;
             const requirementOptions = { "--role": "roles", "--alias": "aliases", "--tool": "tools", "--command": "commands", "--environment": "environment" };
             const resourceOptions = { "--extension": "extensions", "--skill": "skills", "--resource": "static", "--dependency": "dependencies" };
-            if (option === "--name" || option === "--output" || Object.prototype.hasOwnProperty.call(requirementOptions, option) || Object.prototype.hasOwnProperty.call(resourceOptions, option)) {
+            if (option === "--name" || option === "--output" || typedOptionKey(requirementOptions, option) || typedOptionKey(resourceOptions, option)) {
                 const value = equals >= 0 ? arg.slice(equals + 1) : args[++index];
                 if (!value)
                     throw new Error(`Missing value for ${option}`);
@@ -665,9 +691,9 @@ async function bundleWorkflowCli(rawArgs, options) {
                     name = value;
                 else if (option === "--output")
                     output = value;
-                else if (Object.prototype.hasOwnProperty.call(requirementOptions, option))
+                else if (typedOptionKey(requirementOptions, option))
                     requirements[requirementOptions[option]].push(value);
-                else
+                else if (typedOptionKey(resourceOptions, option))
                     resources[resourceOptions[option]].push(value);
                 continue;
             }
@@ -758,10 +784,11 @@ export async function runCli(args, options = {}, write = (text) => { process.std
     }
     if (args[0] === "transcript" && args.length === 2) {
         try {
+            const transcript = requiredArg(args, 1);
             if (options.transcript)
-                await options.transcript(args[1]);
+                await options.transcript(transcript);
             else
-                write(`${transcriptFileLines(args[1]).join("\n")}\n`);
+                write(`${transcriptFileLines(transcript).join("\n")}\n`);
             return 0;
         }
         catch (error) {

package/dist/src/doctor-cleanup.js

@@ -4,7 +4,7 @@ import { homedir } from "node:os";
 import { join, resolve } from "node:path";
 import { validateBudget } from "pi-extensible-workflows";
 import { RUN_STATES } from "pi-extensible-workflows";
-import { jsonValue, validateModelAliases } from "pi-extensible-workflows";
+import { isNodeError, jsonValue, validateModelAliases } from "pi-extensible-workflows";
 import { validateSchema } from "pi-extensible-workflows";
 import { acquireSessionLease, hasLiveSessionLease, projectSessionsDirectory, RunStore } from "pi-extensible-workflows/persistence";
 const TERMINAL_STATES = new Set(["completed", "failed", "stopped"]);
@@ -18,6 +18,13 @@ const SCHEDULER_STATES = new Set(["queued", "running", "waiting_for_child", "pau
 const BUDGET_DIMENSIONS = new Set(["tokens", "costUsd", "durationMs", "agentLaunches"]);
 const BUDGET_EVENT_TYPES = new Set(["soft_crossed", "hard_overrun", "hard_exhausted", "adjustment_requested", "adjustment_approved", "adjustment_rejected"]);
 function object(value) { return typeof value === "object" && value !== null && !Array.isArray(value); }
+function isList(value) { return Array.isArray(value); }
+function isThinking(value) { return ["off", "minimal", "low", "medium", "high", "xhigh", "max"].some((candidate) => candidate === value); }
+function isRunState(value) { return RUN_STATES.some((candidate) => candidate === value); }
+function isAgentState(value) { return typeof value === "string" && AGENT_STATES.has(value); }
+function isSchedulerState(value) { return typeof value === "string" && SCHEDULER_STATES.has(value); }
+function isToolCallState(value) { return ["running", "completed", "failed"].some((candidate) => candidate === value); }
+function isActivityKind(value) { return ["reasoning", "tool", "text"].some((candidate) => candidate === value); }
 function textError(error) { return error instanceof Error ? error.message : String(error); }
 function positiveDays(value) { if (!Number.isSafeInteger(value) || value < 1 || !Number.isFinite(value * DAY_MS))
     throw new Error("older-than-days must be a positive integer"); return value; }
@@ -26,17 +33,17 @@ function sameNames(left, right) { return left.length === right.length && left.ev
 async function jsonFile(path) { return JSON.parse(await readFile(path, "utf8")); }
 async function requiredFile(path) { const info = await lstat(path); if (!info.isFile())
     throw new Error(`Required artifact is not a regular file: ${path}`); }
-function stringList(value, label, nonEmpty = false) { if (!Array.isArray(value) || value.some((item) => typeof item !== "string" || (nonEmpty && !item)))
+function stringList(value, label, nonEmpty = false) { if (!isList(value) || value.some((item) => typeof item !== "string" || (nonEmpty && !item)))
     throw new Error(`${label} is invalid`); }
 function optionalString(value, label) { if (value !== undefined && typeof value !== "string")
     throw new Error(`${label} is invalid`); }
-function nonNegativeInteger(value, label) { if (!Number.isSafeInteger(value) || value < 0)
+function nonNegativeInteger(value, label) { if (typeof value !== "number" || !Number.isSafeInteger(value) || value < 0)
     throw new Error(`${label} is invalid`); }
-function positiveInteger(value, label) { if (!Number.isSafeInteger(value) || value < 1)
+function positiveInteger(value, label) { if (typeof value !== "number" || !Number.isSafeInteger(value) || value < 1)
     throw new Error(`${label} is invalid`); }
 function finiteNumber(value, label) { if (typeof value !== "number" || !Number.isFinite(value) || value < 0)
     throw new Error(`${label} is invalid`); }
-function model(value, label) { if (!object(value) || typeof value.provider !== "string" || !value.provider || typeof value.model !== "string" || !value.model || (value.thinking !== undefined && !["off", "minimal", "low", "medium", "high", "xhigh", "max"].includes(value.thinking)))
+function model(value, label) { if (!object(value) || typeof value.provider !== "string" || !value.provider || typeof value.model !== "string" || !value.model || (value.thinking !== undefined && !isThinking(value.thinking)))
     throw new Error(`${label} is invalid`); }
 function accounting(value, label) { if (!object(value))
     throw new Error(`${label} is invalid`); for (const key of ["input", "output", "cacheRead", "cacheWrite", "cost"])
@@ -47,7 +54,7 @@ function resourceExclusions(value, label) { if (value === undefined)
     stringList(value.skills, `${label}.skills`); if (value.extensions !== undefined)
     stringList(value.extensions, `${label}.extensions`); }
 function agentDefinition(value, label) { if (!object(value))
-    throw new Error(`${label} is invalid`); optionalString(value.prompt, `${label}.prompt`); optionalString(value.description, `${label}.description`); optionalString(value.model, `${label}.model`); if (value.thinking !== undefined && !["off", "minimal", "low", "medium", "high", "xhigh", "max"].includes(value.thinking))
+    throw new Error(`${label} is invalid`); optionalString(value.prompt, `${label}.prompt`); optionalString(value.description, `${label}.description`); optionalString(value.model, `${label}.model`); if (value.thinking !== undefined && !isThinking(value.thinking))
     throw new Error(`${label}.thinking is invalid`); if (value.tools !== undefined)
     stringList(value.tools, `${label}.tools`); resourceExclusions(value.disabledAgentResources, `${label}.disabledAgentResources`); }
 function optionalRole(value, label) {
@@ -63,7 +70,7 @@ function optionalRole(value, label) {
     for (const key of ["model", "description"])
         if (value[key] !== undefined && value[key] !== null && typeof value[key] !== "string")
             throw new Error(`${label}.${key} is invalid`);
-    if (value.thinking !== undefined && value.thinking !== null && !["off", "minimal", "low", "medium", "high", "xhigh", "max"].includes(value.thinking))
+    if (value.thinking !== undefined && value.thinking !== null && !isThinking(value.thinking))
         throw new Error(`${label}.thinking is invalid`);
     if (value.tools !== undefined && value.tools !== null)
         stringList(value.tools, `${label}.tools`);
@@ -75,22 +82,20 @@ function optionalRole(value, label) {
         resourceExclusions(value.disabledAgentResources, `${label}.disabledAgentResources`);
 }
 function validateScheduledOptions(value, label) { if (!object(value) || typeof value.label !== "string" || !value.label || typeof value.cwd !== "string" || !value.cwd)
-    throw new Error(`${label} is invalid`); optionalString(value.requestedLabel, `${label}.requestedLabel`); optionalString(value.parentBreadcrumb, `${label}.parentBreadcrumb`); stringList(value.tools, `${label}.tools`); optionalString(value.worktreeOwner, `${label}.worktreeOwner`); optionalString(value.model, `${label}.model`); if (value.thinking !== undefined && !["off", "minimal", "low", "medium", "high", "xhigh", "max"].includes(value.thinking))
+    throw new Error(`${label} is invalid`); optionalString(value.requestedLabel, `${label}.requestedLabel`); optionalString(value.parentBreadcrumb, `${label}.parentBreadcrumb`); stringList(value.tools, `${label}.tools`); optionalString(value.worktreeOwner, `${label}.worktreeOwner`); optionalString(value.model, `${label}.model`); if (value.thinking !== undefined && !isThinking(value.thinking))
     throw new Error(`${label}.thinking is invalid`); optionalRole(value.role, `${label}.role`); if (value.schema !== undefined)
     validateSchema(value.schema, `${label}.schema`); if (value.retries !== undefined)
     nonNegativeInteger(value.retries, `${label}.retries`); if (value.timeoutMs !== undefined && value.timeoutMs !== null)
     positiveInteger(value.timeoutMs, `${label}.timeoutMs`); if (value.agentOptions !== undefined && (!object(value.agentOptions) || !jsonValue(value.agentOptions)))
     throw new Error(`${label}.agentOptions is invalid`); if (value.agentIdentity !== undefined) {
-    if (!object(value.agentIdentity) || !Array.isArray(value.agentIdentity.structuralPath) || value.agentIdentity.structuralPath.some((part) => typeof part !== "string") || typeof value.agentIdentity.callSite !== "string")
+    if (!object(value.agentIdentity) || !isList(value.agentIdentity.structuralPath) || value.agentIdentity.structuralPath.some((part) => typeof part !== "string") || typeof value.agentIdentity.callSite !== "string")
         throw new Error(`${label}.agentIdentity is invalid`);
     positiveInteger(value.agentIdentity.occurrence, `${label}.agentIdentity.occurrence`);
     optionalString(value.agentIdentity.parentBreadcrumb, `${label}.agentIdentity.parentBreadcrumb`);
     optionalString(value.agentIdentity.worktreeOwner, `${label}.agentIdentity.worktreeOwner`);
 } }
-function validateSessionReference(value, label) {
-    if (!object(value) || typeof value.transport !== "string" || !value.transport || typeof value.sessionId !== "string" || !value.sessionId || (value.locator !== undefined && !jsonValue(value.locator)))
-        throw new Error(`${label} is invalid`);
-}
+function validateSessionReference(value, label) { if (!object(value) || typeof value.transport !== "string" || !value.transport || typeof value.sessionId !== "string" || !value.sessionId || (value.locator !== undefined && !jsonValue(value.locator)))
+    throw new Error(`${label} is invalid`); }
 function validateAgentSetup(value, label) {
     if (!object(value))
         throw new Error(`${label} is invalid`);
@@ -102,7 +107,7 @@ function validateAgentSetup(value, label) {
     resourceExclusions(value.disabledAgentResources, `${label}.disabledAgentResources`);
 }
 function validateAgent(value, label) {
-    if (!object(value) || typeof value.id !== "string" || !value.id || typeof value.name !== "string" || !value.name || typeof value.path !== "string" || !value.path || typeof value.state !== "string" || !AGENT_STATES.has(value.state))
+    if (!object(value) || typeof value.id !== "string" || !value.id || typeof value.name !== "string" || !value.name || typeof value.path !== "string" || !value.path || !isAgentState(value.state))
         throw new Error(`${label} is invalid`);
     optionalString(value.systemPrompt, `${label}.systemPrompt`);
     optionalString(value.prompt, `${label}.prompt`);
@@ -118,7 +123,7 @@ function validateAgent(value, label) {
     stringList(value.tools, `${label}.tools`);
     nonNegativeInteger(value.attempts, `${label}.attempts`);
     if (value.attemptDetails !== undefined) {
-        if (!Array.isArray(value.attemptDetails))
+        if (!isList(value.attemptDetails))
             throw new Error(`${label}.attemptDetails is invalid`);
         for (const [index, attempt] of value.attemptDetails.entries()) {
             const at = `${label}.attemptDetails[${String(index)}]`;
@@ -126,8 +131,9 @@ function validateAgent(value, label) {
                 throw new Error(`${at} is invalid`);
             validateAgentSetup(attempt.setup, `${at}.setup`);
             if (attempt.session !== undefined) {
-                validateSessionReference(attempt.session, `${at}.session`);
-                if (attempt.session.transport !== attempt.transport)
+                const session = attempt.session;
+                validateSessionReference(session, `${at}.session`);
+                if (session.transport !== attempt.transport)
                     throw new Error(`${at}.session transport does not match attempt transport`);
             }
             accounting(attempt.accounting, `${at}.accounting`);
@@ -138,14 +144,14 @@ function validateAgent(value, label) {
     if (value.accounting !== undefined)
         accounting(value.accounting, `${label}.accounting`);
     if (value.toolCalls !== undefined) {
-        if (!Array.isArray(value.toolCalls))
+        if (!isList(value.toolCalls))
             throw new Error(`${label}.toolCalls is invalid`);
         for (const [index, call] of value.toolCalls.entries())
-            if (!object(call) || typeof call.id !== "string" || !call.id || typeof call.name !== "string" || !call.name || !["running", "completed", "failed"].includes(call.state))
+            if (!object(call) || typeof call.id !== "string" || !call.id || typeof call.name !== "string" || !call.name || !isToolCallState(call.state))
                 throw new Error(`${label}.toolCalls[${String(index)}] is invalid`);
     }
     if (value.activity !== undefined) {
-        if (!object(value.activity) || !["reasoning", "tool", "text"].includes(value.activity.kind) || typeof value.activity.text !== "string")
+        if (!object(value.activity) || !isActivityKind(value.activity.kind) || typeof value.activity.text !== "string")
             throw new Error(`${label}.activity is invalid`);
     }
     if (value.lastEventAt !== undefined)
@@ -155,20 +161,18 @@ function validateUsage(value, label) { if (!object(value))
     throw new Error(`${label} is invalid`); for (const key of ["tokens", "costUsd", "durationMs", "agentLaunches"])
     finiteNumber(value[key], `${label}.${key}`); }
 function validateBudgetEvents(value) { if (value === undefined)
-    return; if (!Array.isArray(value))
+    return; if (!isList(value))
     throw new Error("Persisted budget events are invalid"); for (const [index, event] of value.entries()) {
     const label = `budgetEvents[${String(index)}]`;
-    if (!object(event) || typeof event.type !== "string" || !BUDGET_EVENT_TYPES.has(event.type) || !Number.isSafeInteger(event.budgetVersion) || Number(event.budgetVersion) < 1 || !Array.isArray(event.dimensions) || event.dimensions.some((dimension) => typeof dimension !== "string" || !BUDGET_DIMENSIONS.has(dimension)) || typeof event.at !== "number" || !Number.isFinite(event.at) || event.limits === undefined)
+    if (!object(event) || typeof event.type !== "string" || !BUDGET_EVENT_TYPES.has(event.type) || !Number.isSafeInteger(event.budgetVersion) || Number(event.budgetVersion) < 1 || !isList(event.dimensions) || event.dimensions.some((dimension) => typeof dimension !== "string" || !BUDGET_DIMENSIONS.has(dimension)) || typeof event.at !== "number" || !Number.isFinite(event.at) || event.limits === undefined)
         throw new Error(`${label} is invalid`);
     validateUsage(event.usage, `${label}.usage`);
     validateBudget(event.limits);
 } }
-function validateRunRecord(run) {
-    const value = run;
-    if (!object(value) || typeof value.id !== "string" || !value.id || typeof value.workflowName !== "string" || !value.workflowName || typeof value.cwd !== "string" || !value.cwd || typeof value.sessionId !== "string" || !value.sessionId || !RUN_STATES.includes(value.state) || !Array.isArray(value.agents) || !Array.isArray(value.agentSessions) || Object.hasOwn(value, "nativeSessions"))
+function validateRunRecord(value) {
+    if (!object(value) || typeof value.id !== "string" || !value.id || typeof value.workflowName !== "string" || !value.workflowName || typeof value.cwd !== "string" || !value.cwd || typeof value.sessionId !== "string" || !value.sessionId || !isRunState(value.state) || !isList(value.agents) || !isList(value.agentSessions) || Object.hasOwn(value, "nativeSessions"))
         throw new Error("Persisted run state is invalid");
-    const agents = value.agents;
-    agents.forEach((agent, index) => { validateAgent(agent, `agents[${String(index)}]`); });
+    const agents = value.agents.map((agent, index) => { validateAgent(agent, `agents[${String(index)}]`); return agent; });
     const agentIds = new Set();
     for (const agent of agents) {
         const id = agent.id;
@@ -178,36 +182,36 @@ function validateRunRecord(run) {
     }
     for (const agent of agents) {
         const parentId = agent.parentId;
-        if (parentId !== undefined && (typeof parentId !== "string" || !agentIds.has(parentId)))
+        if (parentId !== undefined && !agentIds.has(parentId))
             throw new Error("Persisted agent has a missing parent");
         const seen = new Set();
-        let parent = typeof parentId === "string" ? parentId : undefined;
+        let parent = parentId;
         while (parent) {
             if (seen.has(parent))
                 throw new Error("Persisted agent parent cycle");
             seen.add(parent);
             const parentAgent = agents.find((candidate) => candidate.id === parent);
-            parent = typeof parentAgent?.parentId === "string" ? parentAgent.parentId : undefined;
+            parent = parentAgent?.parentId;
         }
     }
-    const sessions = value.agentSessions;
-    for (const [index, session] of sessions.entries())
+    for (const [index, session] of value.agentSessions.entries())
         validateSessionReference(session, `agentSessions[${String(index)}]`);
     optionalString(value.parentRunId, "Persisted parent run");
     optionalString(value.failedAt, "Persisted failed path");
-    if (value.retry !== undefined) {
-        if (!object(value.retry) || typeof value.retry.sourceRunId !== "string" || !value.retry.sourceRunId || typeof value.retry.lineageRootRunId !== "string" || !value.retry.lineageRootRunId)
+    const retry = value.retry;
+    if (retry !== undefined) {
+        if (!object(retry) || typeof retry.sourceRunId !== "string" || !retry.sourceRunId || typeof retry.lineageRootRunId !== "string" || !retry.lineageRootRunId)
             throw new Error("Persisted retry provenance is invalid");
-        const sourceRunId = value.retry.sourceRunId;
-        stringList(value.retry.completedPaths, "Persisted retry completed paths");
-        stringList(value.retry.incompletePaths, "Persisted retry incomplete paths");
-        stringList(value.retry.namedWorktrees, "Persisted retry named worktrees");
+        const sourceRunId = retry.sourceRunId;
+        stringList(retry.completedPaths, "Persisted retry completed paths");
+        stringList(retry.incompletePaths, "Persisted retry incomplete paths");
+        stringList(retry.namedWorktrees, "Persisted retry named worktrees");
         if (value.parentRunId !== sourceRunId)
             throw new Error("Persisted retry parent does not match its source");
     }
     optionalString(value.phase, "Persisted phase");
     if (value.phaseHistory !== undefined) {
-        if (!Array.isArray(value.phaseHistory))
+        if (!isList(value.phaseHistory))
             throw new Error("Persisted phase history is invalid");
         for (const phase of value.phaseHistory) {
             if (!object(phase) || typeof phase.phase !== "string" || !phase.phase)
@@ -215,6 +219,18 @@ function validateRunRecord(run) {
             nonNegativeInteger(phase.afterAgent, "Persisted phase history afterAgent");
         }
     }
+    if (value.phaseHistoryIndex !== undefined)
+        nonNegativeInteger(value.phaseHistoryIndex, "Persisted phase history index");
+    if (value.activeShellsByPhase !== undefined) {
+        if (!isList(value.activeShellsByPhase))
+            throw new Error("Persisted phase shell activity is invalid");
+        for (const activity of value.activeShellsByPhase) {
+            if (!object(activity) || !Number.isSafeInteger(activity.phaseIndex) || Number(activity.phaseIndex) < -1)
+                throw new Error("Persisted phase shell activity is invalid");
+            positiveInteger(activity.active, "Persisted phase shell activity count");
+            finiteNumber(activity.startedAt, "Persisted phase shell activity start");
+        }
+    }
     if (value.error !== undefined && (!object(value.error) || typeof value.error.code !== "string" || typeof value.error.message !== "string"))
         throw new Error("Persisted run error is invalid");
     validateBudget(value.budget);
@@ -224,7 +240,7 @@ function validateRunRecord(run) {
         validateUsage(value.usage, "Persisted usage");
     validateBudgetEvents(value.budgetEvents);
     if (value.events !== undefined) {
-        if (!Array.isArray(value.events))
+        if (!isList(value.events))
             throw new Error("Persisted run events are invalid");
         for (const event of value.events)
             if (!object(event) || typeof event.type !== "string" || typeof event.message !== "string")
@@ -232,7 +248,7 @@ function validateRunRecord(run) {
     }
 }
 function validateSnapshot(snapshot) {
-    if (!object(snapshot) || typeof snapshot.script !== "string" || !snapshot.script || !jsonValue(snapshot.args) || !object(snapshot.metadata) || typeof snapshot.metadata.name !== "string" || !snapshot.metadata.name || (snapshot.metadata.description !== undefined && typeof snapshot.metadata.description !== "string") || !object(snapshot.settings) || !Number.isSafeInteger(snapshot.settings.concurrency) || Number(snapshot.settings.concurrency) < 1 || Number(snapshot.settings.concurrency) > 16 || !Array.isArray(snapshot.models) || snapshot.models.some((modelName) => typeof modelName !== "string") || !Array.isArray(snapshot.tools) || snapshot.tools.some((tool) => typeof tool !== "string") || !Array.isArray(snapshot.agentTypes) || snapshot.agentTypes.some((agentType) => typeof agentType !== "string") || !Array.isArray(snapshot.schemas))
+    if (!object(snapshot) || typeof snapshot.script !== "string" || !snapshot.script || !jsonValue(snapshot.args) || !object(snapshot.metadata) || typeof snapshot.metadata.name !== "string" || !snapshot.metadata.name || (snapshot.metadata.description !== undefined && typeof snapshot.metadata.description !== "string") || !object(snapshot.settings) || !Number.isSafeInteger(snapshot.settings.concurrency) || Number(snapshot.settings.concurrency) < 1 || Number(snapshot.settings.concurrency) > 16 || snapshot.settings.backgroundWidget !== undefined && typeof snapshot.settings.backgroundWidget !== "boolean" || !isList(snapshot.models) || snapshot.models.some((modelName) => typeof modelName !== "string") || !isList(snapshot.tools) || snapshot.tools.some((tool) => typeof tool !== "string") || !isList(snapshot.agentTypes) || snapshot.agentTypes.some((agentType) => typeof agentType !== "string") || !isList(snapshot.schemas))
         throw new Error("Persisted launch snapshot is invalid");
     if (snapshot.identityVersion !== undefined)
         positiveInteger(snapshot.identityVersion, "Persisted snapshot identity version");
@@ -278,7 +294,7 @@ async function validateSystemPrompts(store) {
         await store.systemPrompts();
         return;
     }
-    if (!object(value) || value.version !== 1 || !Array.isArray(value.entries))
+    if (!object(value) || value.version !== 1 || !isList(value.entries))
         throw new Error("Persisted system prompts are invalid");
     for (const [index, entry] of value.entries.entries()) {
         const label = `system-prompts.entries[${String(index)}]`;
@@ -300,6 +316,13 @@ async function validateRunDirectory(store) {
             throw new Error(`Run artifact is not a regular file: ${join(store.directory, entry.name)}`);
     }
 }
+function validateOwnershipRecord(value, label, ownershipIds) {
+    if (!object(value) || typeof value.id !== "string" || !value.id || ownershipIds.has(value.id) || typeof value.label !== "string" || !value.label || !isSchedulerState(value.state))
+        throw new Error(`${label} is invalid`);
+    optionalString(value.parentId, `${label}.parentId`);
+    optionalString(value.prompt, `${label}.prompt`);
+    validateScheduledOptions(value.options, `${label}.options`);
+}
 async function validateRunArtifacts(store, workflowScript, state) {
     await validateRunDirectory(store);
     for (const name of REQUIRED_RUN_FILES)
@@ -307,7 +330,7 @@ async function validateRunArtifacts(store, workflowScript, state) {
     if (await readFile(join(store.directory, "workflow.js"), "utf8") !== workflowScript)
         throw new Error("Persisted workflow source does not match its launch snapshot");
     await validateSystemPrompts(store);
-    const result = await jsonFile(join(store.directory, "result.json")).catch((error) => { if (error.code === "ENOENT")
+    const result = await jsonFile(join(store.directory, "result.json")).catch((error) => { if (isNodeError(error, "ENOENT"))
         return undefined; throw error; });
     if (result === undefined && state === "completed")
         throw new Error("Completed run result is missing");
@@ -315,21 +338,17 @@ async function validateRunArtifacts(store, workflowScript, state) {
         throw new Error("Persisted workflow result is invalid");
     validateJournal(await jsonFile(join(store.directory, "journal.json")));
     const rawOwnership = await jsonFile(join(store.directory, "ownership.json"));
-    if (!Array.isArray(rawOwnership))
+    if (!isList(rawOwnership))
         throw new Error("Persisted ownership records are invalid");
-    const ownership = rawOwnership;
     const ownershipIds = new Set();
-    for (const [index, record] of ownership.entries()) {
+    const ownership = rawOwnership.map((record, index) => {
         const label = `ownership[${String(index)}]`;
-        if (!object(record) || typeof record.id !== "string" || !record.id || ownershipIds.has(record.id) || typeof record.label !== "string" || !record.label || typeof record.state !== "string" || !SCHEDULER_STATES.has(record.state))
-            throw new Error(`${label} is invalid`);
+        validateOwnershipRecord(record, label, ownershipIds);
         ownershipIds.add(record.id);
-        optionalString(record.parentId, `${label}.parentId`);
-        optionalString(record.prompt, `${label}.prompt`);
-        validateScheduledOptions(record.options, `${label}.options`);
-    }
+        return record;
+    });
     for (const record of ownership)
-        if (object(record) && record.parentId !== undefined && (typeof record.parentId !== "string" || !ownershipIds.has(record.parentId)))
+        if (record.parentId !== undefined && !ownershipIds.has(record.parentId))
             throw new Error("Persisted ownership parent is missing");
     await store.validateDeletionWorktrees();
     const borrowed = await store.borrowedWorktrees();
@@ -467,7 +486,7 @@ function planSession(scan, cutoffMs) {
     }
     return { candidates: scan.runs.filter(({ runId }) => oldTerminal.has(runId) && !protectedRuns.has(runId)), skipped };
 }
-function deletionOrder(scan, candidates) {
+function deletionOrder(_scan, candidates) {
     const candidateIds = new Set(candidates.map(({ runId }) => runId));
     const remaining = new Set(candidateIds);
     const ordered = [];
@@ -487,7 +506,7 @@ async function storedSessionIds(cwd, home) {
         entries = await sessionEntries(path);
     }
     catch (error) {
-        if (error.code === "ENOENT")
+        if (isNodeError(error, "ENOENT"))
             return [];
         throw error;
     }

package/dist/src/doctor.js

@@ -1,8 +1,8 @@
-import { readFileSync, readdirSync, realpathSync } from "node:fs";
+import { readFileSync, readdirSync, realpathSync, statSync } from "node:fs";
 import { basename, dirname, extname, join, resolve } from "node:path";
 import { InMemoryCredentialStore } from "@earendil-works/pi-ai";
 import { ModelRuntime, createAgentSessionFromServices, createAgentSessionServices, getAgentDir, hasTrustRequiringProjectResources, SessionManager, SettingsManager, } from "@earendil-works/pi-coding-agent";
-import { DEFAULT_SETTINGS, createLocalPiSession, loadSettings, prepareAgentSetupForInspection, resolveAgentResourcePolicy, resolveWorkflowSettings, resolveModelReference, parseThinking, parseRoleMarkdown, registeredWorkflowFunctions, registeredWorkflowRoleDirectoryRegistrations, workflowRoleDirectories, workflowProjectSettingsPath, workflowSettingsPath, } from "pi-extensible-workflows";
+import { DEFAULT_SETTINGS, createLocalPiSession, errorText, isNodeError, isObject, loadSettings, prepareAgentSetupForInspection, resolveAgentResourcePolicy, resolveWorkflowSettings, resolveModelReference, parseThinking, parseRoleMarkdown, registeredWorkflowFunctions, registeredWorkflowRoleDirectoryRegistrations, workflowRoleDirectories, workflowProjectSettingsPath, workflowSettingsPath, } from "pi-extensible-workflows";
 import { loadingRegistry } from "pi-extensible-workflows";
 import { disabledResources, unmatchedResourcePatterns } from "pi-extensible-workflows";
 const THINKING_HINT = "Use off, minimal, low, medium, high, xhigh, or max.";
@@ -20,16 +20,23 @@ function isDynamicModelAlias(value, aliases) {
     const name = match?.[1];
     return Boolean(name && (match[2] === undefined || parseThinking(match[2]) !== undefined) && aliases.has(name));
 }
+function isCredential(value) {
+    if (!isObject(value))
+        return false;
+    if (value.type === "api_key")
+        return (value.key === undefined || typeof value.key === "string") && (value.env === undefined || isObject(value.env) && Object.values(value.env).every((entry) => typeof entry === "string"));
+    return value.type === "oauth" && typeof value.refresh === "string" && typeof value.access === "string" && typeof value.expires === "number";
+}
 async function readCredentials(agentDir) {
     const credentials = new InMemoryCredentialStore();
     try {
         const parsed = JSON.parse(readFileSync(join(agentDir, "auth.json"), "utf8"));
-        if (typeof parsed !== "object" || parsed === null || Array.isArray(parsed))
+        if (!isObject(parsed))
             throw new Error("Pi auth.json must be an object");
-        await Promise.all(Object.entries(parsed).map(([provider, credential]) => credentials.modify(provider, async () => credential)));
+        await Promise.all(Object.entries(parsed).flatMap(([provider, credential]) => isCredential(credential) ? [credentials.modify(provider, async () => credential)] : []));
     }
     catch (error) {
-        if (error.code !== "ENOENT")
+        if (!isNodeError(error, "ENOENT"))
             throw error;
     }
     return credentials;
@@ -40,11 +47,11 @@ function savedTrust(cwd, agentDir) {
         parsed = JSON.parse(readFileSync(join(agentDir, "trust.json"), "utf8"));
     }
     catch (error) {
-        if (error.code === "ENOENT")
+        if (isNodeError(error, "ENOENT"))
             return undefined;
         throw error;
     }
-    if (typeof parsed !== "object" || parsed === null || Array.isArray(parsed))
+    if (!isObject(parsed))
         throw new Error("Pi trust.json must be an object");
     let current = canonical(cwd);
     while (current !== dirname(current)) {
@@ -106,12 +113,28 @@ async function discoverPi(cwd, agentDir) {
             process.env.PI_OFFLINE = previousOffline;
     }
 }
+function isRoleFile(dir, entry) {
+    if (extname(entry.name) !== ".md")
+        return false;
+    if (entry.isFile())
+        return true;
+    if (!entry.isSymbolicLink())
+        return false;
+    try {
+        return statSync(join(dir, entry.name)).isFile();
+    }
+    catch (error) {
+        if (isNodeError(error, "ENOENT"))
+            return false;
+        throw error;
+    }
+}
 function roleFiles(dir) {
     try {
-        return readdirSync(dir, { withFileTypes: true }).filter((entry) => entry.isFile() && extname(entry.name) === ".md").map((entry) => join(dir, entry.name)).sort();
+        return readdirSync(dir, { withFileTypes: true }).filter((entry) => isRoleFile(dir, entry)).map((entry) => join(dir, entry.name)).sort();
     }
     catch (error) {
-        if (error.code === "ENOENT")
+        if (isNodeError(error, "ENOENT"))
             return [];
         throw error;
     }
@@ -128,7 +151,7 @@ function scanExtensionRoleFiles(registrations) {
     for (const registration of registrations) {
         try {
             const entries = readdirSync(registration.path, { withFileTypes: true });
-            const roleFiles = entries.filter((entry) => entry.isFile() && extname(entry.name) === ".md");
+            const roleFiles = entries.filter((entry) => isRoleFile(registration.path, entry));
             if (!roleFiles.length)
                 empty.push(registration);
             for (const entry of roleFiles)
@@ -161,7 +184,8 @@ function validateModel(value, known, available, source, diagnostics, aliases, dy
             diagnostics.push(diagnostic("warning", "MODEL_UNAVAILABLE", `Model is valid-shaped but unavailable: ${name}`, source));
     }
     catch (error) {
-        diagnostics.push(diagnostic("error", "MODEL_INVALID", error.message, source, error.message.includes("thinking") ? THINKING_HINT : "Use provider/model or provider/model:thinking."));
+        const message = errorText(error);
+        diagnostics.push(diagnostic("error", "MODEL_INVALID", message, source, message.includes("thinking") ? THINKING_HINT : "Use provider/model or provider/model:thinking."));
     }
 }
 function inspectRole(path, activeTools, knownModels, availableModels, diagnostics, aliases, dynamicAliases, settingsPath, source) {
@@ -244,7 +268,7 @@ async function inspectRoleSession(cwd, agentDir, roleName, definition, rolePath,
         return undefined;
     }
     finally {
-        session.dispose();
+        await session.dispose();
     }
 }
 function resourcePolicySource(settingsSource) { return settingsSource; }
@@ -258,7 +282,7 @@ export async function doctor(options = {}) {
         settings = loadSettings(settingsPath);
     }
     catch (error) {
-        diagnostics.push(diagnostic("error", "SETTINGS_INVALID", error.message, settingsPath, "Fix or remove the invalid workflow settings file."));
+        diagnostics.push(diagnostic("error", "SETTINGS_INVALID", errorText(error), settingsPath, "Fix or remove the invalid workflow settings file."));
     }
     let settingsSources = { concurrency: settingsPath, modelAliases: settingsPath, disabledAgentResources: settingsPath };
     const projectSettingsPath = workflowProjectSettingsPath(cwd);
@@ -267,7 +291,7 @@ export async function doctor(options = {}) {
         pi = await (options.discoverPi ?? discoverPi)(cwd, agentDir);
     }
     catch (error) {
-        diagnostics.push(diagnostic("error", "PI_DISCOVERY", `Pi headless discovery failed: ${error.message}`, undefined, "Open and trust the project in Pi, fix extension errors, then rerun doctor."));
+        diagnostics.push(diagnostic("error", "PI_DISCOVERY", `Pi headless discovery failed: ${errorText(error)}`, undefined, "Open and trust the project in Pi, fix extension errors, then rerun doctor."));
         pi = { trust: { required: false, trusted: false, source: "discovery failed" }, activeTools: [], knownModels: [], availableModels: [], extensionErrors: [], functions: {} };
     }
     if (options.activeTools)
@@ -311,7 +335,7 @@ export async function doctor(options = {}) {
     const dynamicAliases = new Set(registeredModelAliases.map(({ name }) => name).filter((name) => !Object.prototype.hasOwnProperty.call(aliases, name)));
     const modelAliases = [
         ...Object.keys(aliases).map((name) => ({ name, kind: "static", provenance: settingsSources.modelAliases })),
-        ...registeredModelAliases.map(({ name, version, headline, extensionDescription }) => ({ name, kind: "dynamic", provenance: `extension: ${headline}`, version, headline, extensionDescription })),
+        ...registeredModelAliases.map(({ name, version, headline }) => ({ name, kind: "dynamic", provenance: `extension: ${headline}`, version, headline })),
     ].sort((left, right) => left.name.localeCompare(right.name) || left.kind.localeCompare(right.kind));
     const roles = [];
     const definitions = new Map();

package/dist/src/session-inspector.js

@@ -14,27 +14,39 @@ function text(content) {
     const parts = content;
     return parts.flatMap((part) => typeof part === "object" && part !== null && "type" in part && part.type === "text" && "text" in part && typeof part.text === "string" ? [part.text] : []).join("");
 }
+function isTranscriptPart(value) {
+    if (typeof value !== "object" || value === null || !("type" in value))
+        return false;
+    if (value.type === "text")
+        return "text" in value && typeof value.text === "string";
+    if (value.type === "thinking")
+        return "thinking" in value && typeof value.thinking === "string";
+    if (value.type === "toolCall")
+        return "name" in value && typeof value.name === "string";
+    return value.type === "image";
+}
+function isTranscriptMessage(value) {
+    if (typeof value !== "object" || value === null)
+        return false;
+    return (!(("role" in value) && value.role !== undefined) || typeof value.role === "string") && (!(("toolName" in value) && value.toolName !== undefined) || typeof value.toolName === "string") && (!(("customType" in value) && value.customType !== undefined) || typeof value.customType === "string");
+}
 function transcriptPartLines(part) {
-    if (typeof part !== "object" || part === null || !("type" in part))
+    if (!isTranscriptPart(part))
         return [];
-    const value = part;
-    if (value.type === "text" && typeof value.text === "string")
-        return value.text.split("\n");
-    if (value.type === "thinking" && typeof value.thinking === "string")
-        return ["Thinking:", ...value.thinking.split("\n")];
-    if (value.type === "toolCall" && typeof value.name === "string")
-        return [`Tool call: ${value.name}`, JSON.stringify(value.arguments, null, 2)];
-    if (value.type === "image")
-        return ["[image]"];
-    return [];
+    if (part.type === "text")
+        return part.text.split("\n");
+    if (part.type === "thinking")
+        return ["Thinking:", ...part.thinking.split("\n")];
+    if (part.type === "toolCall")
+        return [`Tool call: ${part.name}`, JSON.stringify(part.arguments, null, 2)];
+    return ["[image]"];
 }
 function transcriptMessageLines(message) {
-    if (typeof message !== "object" || message === null)
+    if (!isTranscriptMessage(message))
         return ["(invalid message)"];
-    const value = message;
-    const role = typeof value.role === "string" ? value.role : "message";
-    const label = role === "toolResult" && typeof value.toolName === "string" ? `${role}: ${value.toolName}` : role === "custom" && typeof value.customType === "string" ? `${role}: ${value.customType}` : role;
-    const content = Array.isArray(value.content) ? value.content.flatMap(transcriptPartLines) : typeof value.content === "string" ? value.content.split("\n") : [];
+    const role = message.role ?? "message";
+    const label = role === "toolResult" && message.toolName !== undefined ? `${role}: ${message.toolName}` : role === "custom" && message.customType !== undefined ? `${role}: ${message.customType}` : role;
+    const content = Array.isArray(message.content) ? message.content.flatMap(transcriptPartLines) : typeof message.content === "string" ? message.content.split("\n") : [];
     return [`[${label}]`, ...(content.length ? content : ["(empty)"])];
 }
 export function transcriptLines(entries) {
@@ -58,8 +70,11 @@ function mergedModels(groups) {
     return [...totals].map(([model, cost]) => ({ model, cost })).sort((a, b) => b.cost - a.cost || a.model.localeCompare(b.model));
 }
 const thinkingLevels = ["off", "minimal", "low", "medium", "high", "xhigh", "max"];
+function isThinkingLevel(value) {
+    return typeof value === "string" && thinkingLevels.some((level) => level === value);
+}
 function parsedThinking(value) {
-    return typeof value === "string" && thinkingLevels.includes(value) ? value : undefined;
+    return isThinkingLevel(value) ? value : undefined;
 }
 function modelName(provider, model) {
     return typeof provider === "string" && provider && typeof model === "string" && model ? `${provider}/${model}` : undefined;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@piewf/cli",
-  "version": "5.1.1",
+  "version": "5.3.0",
   "description": "CLI for pi-extensible-workflows",
   "homepage": "https://vekexasia.github.io/pi-extensible-workflows/",
   "repository": {
@@ -29,6 +29,7 @@
   "peerDependencies": {
     "@earendil-works/pi-ai": "*",
     "@earendil-works/pi-coding-agent": "*",
+    "@earendil-works/pi-tui": "*",
     "typebox": "*"
   },
   "devDependencies": {
@@ -45,7 +46,7 @@
   },
   "license": "MIT",
   "dependencies": {
-    "pi-extensible-workflows": "5.1.1"
+    "pi-extensible-workflows": "5.3.0"
   },
   "publishConfig": {
     "access": "public"