@piewf/cli 5.5.0 → 5.6.1

package/dist/src/cli.js

@@ -8,7 +8,7 @@ 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, isNodeError, jsonValue, loadAgentDefinitions, registeredWorkflowFunctions, truncateWorkflowProgress, workflowCatalog, workflowSettingsPath } from "pi-extensible-workflows";
+import workflowExtension, { errorText, 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";
@@ -752,7 +752,7 @@ export async function runCli(args, options = {}, write = (text) => { process.std
             return doctorExitCode(report);
         }
         catch (error) {
-            stderr(`Error: ${error instanceof Error ? error.message : String(error)}\n`);
+            stderr(`Error: ${errorText(error)}\n`);
             return 1;
         }
     }
@@ -769,7 +769,7 @@ export async function runCli(args, options = {}, write = (text) => { process.std
             return doctorCleanupExitCode(report);
         }
         catch (error) {
-            stderr(`Error: ${error instanceof Error ? error.message : String(error)}\n`);
+            stderr(`Error: ${errorText(error)}\n`);
             return 1;
         }
     }
@@ -783,7 +783,7 @@ export async function runCli(args, options = {}, write = (text) => { process.std
             return 0;
         }
         catch (error) {
-            write(`Error: ${error instanceof Error ? error.message : String(error)}\n`);
+            write(`Error: ${errorText(error)}\n`);
             return 1;
         }
     }
@@ -797,7 +797,7 @@ export async function runCli(args, options = {}, write = (text) => { process.std
             return 0;
         }
         catch (error) {
-            write(`Error: ${error instanceof Error ? error.message : String(error)}\n`);
+            write(`Error: ${errorText(error)}\n`);
             return 1;
         }
     }
@@ -809,7 +809,7 @@ export async function runCli(args, options = {}, write = (text) => { process.std
             return args[0] === "run" ? await runWorkflowCli(args.slice(1), workflowOptions) : await exportWorkflowCli(args.slice(1), workflowOptions);
         }
         catch (error) {
-            stderr(`Error: ${error instanceof Error ? error.message : String(error)}\n`);
+            stderr(`Error: ${errorText(error)}\n`);
             return 1;
         }
     }

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 { isNodeError, jsonValue, validateModelAliases } from "pi-extensible-workflows";
+import { errorText, 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"]);
@@ -25,7 +25,6 @@ function isAgentState(value) { return typeof value === "string" && AGENT_STATES.
 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; }
 function runItem(entry, action, reason) { return { sessionId: entry.sessionId, runId: entry.runId, action, state: entry.run.state, stateMtimeMs: entry.stateMtimeMs, path: entry.store.directory, ...(reason ? { reason } : {}) }; }
@@ -63,33 +62,14 @@ function agentDefinition(value, label) { if (!object(value))
 function optionalRole(value, label) {
     if (value === undefined)
         return;
-    if (typeof value === "string") {
-        if (!value.trim())
-            throw new Error(`${label} is invalid`);
-        return;
-    }
-    if (!object(value) || typeof value.name !== "string" || !value.name.trim())
+    if (typeof value !== "string" || !value.trim())
         throw new Error(`${label} is invalid`);
-    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 && !isThinking(value.thinking))
-        throw new Error(`${label}.thinking is invalid`);
-    if (value.tools !== undefined && value.tools !== null)
-        stringList(value.tools, `${label}.tools`);
-    if (value.overrideSystemPrompt !== undefined && value.overrideSystemPrompt !== null && typeof value.overrideSystemPrompt !== "boolean")
-        throw new Error(`${label}.overrideSystemPrompt is invalid`);
-    if (value.contextFiles !== undefined && value.contextFiles !== null)
-        stringList(value.contextFiles, `${label}.contextFiles`);
-    if (value.skills !== undefined && value.skills !== null)
-        stringList(value.skills, `${label}.skills`);
-    if (value.extensions !== undefined && value.extensions !== null)
-        stringList(value.extensions, `${label}.extensions`);
 }
 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`); if (value.skills !== undefined)
     stringList(value.skills, `${label}.skills`); if (value.extensions !== undefined)
-    stringList(value.extensions, `${label}.extensions`); optionalString(value.worktreeOwner, `${label}.worktreeOwner`); optionalString(value.model, `${label}.model`); if (value.thinking !== undefined && !isThinking(value.thinking))
+    stringList(value.extensions, `${label}.extensions`); if (value.contextFiles !== undefined)
+    stringList(value.contextFiles, `${label}.contextFiles`); 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)
@@ -426,7 +406,7 @@ async function scanSession(cwd, sessionId, home, expectedLease) {
             runs.push({ sessionId, runId, store, run: loaded.run, stateMtimeMs: afterState.mtimeMs, dependencies: [...dependencies] });
         }
         catch (error) {
-            throw new Error(`Run ${runId} is corrupt or incomplete: ${textError(error)}`, { cause: error });
+            throw new Error(`Run ${runId} is corrupt or incomplete: ${errorText(error)}`, { cause: error });
         }
     }
     const after = await sessionEntries(runsPath);
@@ -461,7 +441,7 @@ async function recheckCandidate(entry, cutoffMs) {
     const before = await stat(join(entry.store.directory, "state.json")).catch(() => undefined);
     if (!before)
         return "State record disappeared before deletion";
-    const loaded = await entry.store.load().catch((error) => { throw new Error(`Candidate could not be reloaded: ${textError(error)}`); });
+    const loaded = await entry.store.load().catch((error) => { throw new Error(`Candidate could not be reloaded: ${errorText(error)}`); });
     validateRunRecord(loaded.run);
     if (loaded.run.id !== entry.runId || loaded.run.state !== entry.run.state || !TERMINAL_STATES.has(loaded.run.state))
         return "Candidate state changed before deletion";
@@ -550,7 +530,7 @@ export async function doctorCleanup(options = {}) {
         sessionIds = await storedSessionIds(cwd, home);
     }
     catch (error) {
-        addFailure(failures, "(project)", textError(error));
+        addFailure(failures, "(project)", errorText(error));
         return { cwd, cutoffMs, olderThanDays, yes, sessions, candidates, skipped, deleted, failures };
     }
     for (const sessionId of sessionIds) {
@@ -559,8 +539,8 @@ export async function doctorCleanup(options = {}) {
             initial = await scanSession(cwd, sessionId, home);
         }
         catch (error) {
-            sessions.push({ sessionId, path: join(projectSessionsDirectory(cwd, home), sessionId), status: "failed", reason: textError(error) });
-            addFailure(failures, sessionId, textError(error));
+            sessions.push({ sessionId, path: join(projectSessionsDirectory(cwd, home), sessionId), status: "failed", reason: errorText(error) });
+            addFailure(failures, sessionId, errorText(error));
             continue;
         }
         const initialPlan = planSession(initial, cutoffMs);
@@ -577,7 +557,7 @@ export async function doctorCleanup(options = {}) {
             lease = await acquireSessionLease(cwd, sessionId, home);
         }
         catch (error) {
-            const message = textError(error);
+            const message = errorText(error);
             if (/already owned|active ownership|RUN_OWNED/i.test(message)) {
                 sessions.push({ sessionId, path: initial.path, status: "skipped", reason: "Session has a live ownership lease" });
                 continue;
@@ -592,7 +572,7 @@ export async function doctorCleanup(options = {}) {
                 current = await scanSession(cwd, sessionId, home, lease);
             }
             catch (error) {
-                const message = textError(error);
+                const message = errorText(error);
                 sessions.push({ sessionId, path: initial.path, status: "failed", reason: message });
                 addFailure(failures, sessionId, message);
                 continue;
@@ -613,7 +593,7 @@ export async function doctorCleanup(options = {}) {
                         addUnique(skipped, item);
                 }
                 catch (error) {
-                    const message = textError(error);
+                    const message = errorText(error);
                     addFailure(failures, sessionId, message);
                     clean = false;
                     break;
@@ -625,7 +605,7 @@ export async function doctorCleanup(options = {}) {
                     ordered = deletionOrder(current, freshPlan.candidates);
                 }
                 catch (error) {
-                    const message = textError(error);
+                    const message = errorText(error);
                     addFailure(failures, sessionId, message);
                     clean = false;
                     break;
@@ -638,7 +618,7 @@ export async function doctorCleanup(options = {}) {
                     changed = await recheckCandidate(target, cutoffMs);
                 }
                 catch (error) {
-                    const message = textError(error);
+                    const message = errorText(error);
                     addFailure(failures, sessionId, message, target.runId);
                     clean = false;
                     break;
@@ -653,7 +633,7 @@ export async function doctorCleanup(options = {}) {
                     deleted.push(runItem(target, "deleted"));
                 }
                 catch (error) {
-                    const message = textError(error);
+                    const message = errorText(error);
                     addUnique(skipped, runItem(target, "failed", message));
                     addFailure(failures, sessionId, message, target.runId);
                     clean = false;
@@ -670,7 +650,7 @@ export async function doctorCleanup(options = {}) {
                 await lease.release();
             }
             catch (error) {
-                addFailure(failures, sessionId, textError(error));
+                addFailure(failures, sessionId, errorText(error));
             }
         }
     }

package/dist/src/doctor.js

@@ -212,7 +212,7 @@ function validateModel(value, known, available, source, diagnostics, aliases, dy
     }
     catch (error) {
         const message = errorText(error);
-        diagnostics.push(diagnostic("error", "MODEL_INVALID", message, source, message.includes("thinking") ? THINKING_HINT : "Use provider/model or provider/model:thinking."));
+        diagnostics.push(diagnostic("error", "MODEL_INVALID", message, source, message.includes("thinking") ? THINKING_HINT : "Use provider/model:thinking."));
     }
 }
 function inspectRole(path, activeTools, knownModels, availableModels, diagnostics, aliases, dynamicAliases, settingsPath, source) {
@@ -225,7 +225,7 @@ function inspectRole(path, activeTools, knownModels, availableModels, diagnostic
             diagnostics.push(legacyAgentResourceSelectorDiagnostic(path));
             return undefined;
         }
-        const message = error instanceof Error ? error.message : String(error);
+        const message = errorText(error);
         diagnostics.push(diagnostic("error", "ROLE_FRONTMATTER", source ? `${roleProvenance(source)} contains invalid role at "${path}": ${message}` : message, path, "Fix the role YAML frontmatter."));
         return undefined;
     }
@@ -270,14 +270,14 @@ async function inspectRoleSession(cwd, agentDir, roleName, definition, rolePath,
         prepared = await prepareAgentSetupForInspection(root, prompt, options, transport);
     }
     catch (error) {
-        setupDiagnostics.push(diagnostic("error", "ROLE_INSPECTION", error instanceof Error ? error.message : String(error), rolePath));
+        setupDiagnostics.push(diagnostic("error", "ROLE_INSPECTION", errorText(error), rolePath));
         diagnostics.push(...setupDiagnostics);
         return undefined;
     }
     if (prepared.failure) {
         const error = prepared.failure.error;
         const code = prepared.failure.hook ? "ROLE_SETUP_HOOK" : "ROLE_INSPECTION";
-        setupDiagnostics.push(diagnostic("error", code, `${prepared.failure.hook ? `Role setup hook ${prepared.failure.hook} failed: ` : ""}${error instanceof Error ? error.message : String(error)}`, prepared.failure.hook ?? rolePath));
+        setupDiagnostics.push(diagnostic("error", code, `${prepared.failure.hook ? `Role setup hook ${prepared.failure.hook} failed: ` : ""}${errorText(error)}`, prepared.failure.hook ?? rolePath));
         diagnostics.push(...setupDiagnostics);
         return undefined;
     }
@@ -285,7 +285,7 @@ async function inspectRoleSession(cwd, agentDir, roleName, definition, rolePath,
         return await createLocalPiSession({ ...prepared.setup.sessionInput, sessionManager: SessionManager.inMemory() });
     }
     catch (error) {
-        setupDiagnostics.push(diagnostic("error", "ROLE_INSPECTION", error instanceof Error ? error.message : String(error), rolePath));
+        setupDiagnostics.push(diagnostic("error", "ROLE_INSPECTION", errorText(error), rolePath));
         return undefined;
     } })();
     if (!session) {
@@ -304,7 +304,7 @@ async function inspectRoleSession(cwd, agentDir, roleName, definition, rolePath,
         return { role: roleName, path: rolePath, model: actualModel, tools: state?.tools.map(({ name }) => name) ?? [...prepared.setup.sessionInput.tools], resources: { selectors: { skills: [...policy.effective.skills], extensions: [...policy.effective.extensions], tools: [...(policy.effective.tools ?? [])] }, skills: policy.selectedSkills ?? resources.skills, extensions: policy.selectedExtensions ?? resources.extensions, tools: policy.selectedTools ?? prepared.setup.sessionInput.tools, unmatchedSkills: policy.unmatchedSkills, unmatchedExtensions: policy.unmatchedExtensions, unmatchedTools: policy.unmatchedTools ?? [], selectorSources: policy.selectorSources }, systemPrompt: { probe: prompt, expandedProbe: promptResult.expandedPrompt, text: promptResult.systemPrompt, ...(resources.systemPromptSource ? { source: resources.systemPromptSource } : {}) }, setup: { hooks: prepared.summary.hookNames, diagnostics: setupDiagnostics } };
     }
     catch (error) {
-        setupDiagnostics.push(diagnostic("error", "ROLE_INSPECTION", error instanceof Error ? error.message : String(error), rolePath));
+        setupDiagnostics.push(diagnostic("error", "ROLE_INSPECTION", errorText(error), rolePath));
         diagnostics.push(...setupDiagnostics);
         return undefined;
     }
@@ -353,7 +353,7 @@ export async function doctor(options = {}) {
         settingsSources = resolved.sources;
     }
     catch (error) {
-        const message = error instanceof Error ? error.message : String(error);
+        const message = errorText(error);
         const source = message.includes(projectSettingsPath) ? projectSettingsPath : settingsPath;
         if (!diagnostics.some(({ code, source: itemSource }) => code === "SETTINGS_INVALID" && itemSource === source))
             diagnostics.push(diagnostic("error", "SETTINGS_INVALID", message, source, "Fix or remove the invalid workflow settings file."));
@@ -363,7 +363,7 @@ export async function doctor(options = {}) {
         resourcePolicy = matchResourcePolicy(resolveAgentResourcePolicy(cwd, pi.trust.trusted, settingsPath), pi);
     }
     catch (error) {
-        const message = error instanceof Error ? error.message : String(error);
+        const message = errorText(error);
         const source = message.includes(projectSettingsPath) ? projectSettingsPath : settingsPath;
         if (!diagnostics.some(({ code, source: itemSource }) => code === "SETTINGS_INVALID" && itemSource === source))
             diagnostics.push(diagnostic("error", "SETTINGS_INVALID", message, source, "Fix or remove the invalid workflow settings file."));
@@ -395,7 +395,7 @@ export async function doctor(options = {}) {
     const definitions = new Map();
     const extensionScan = scanExtensionRoleFiles(registeredWorkflowRoleDirectoryRegistrations());
     for (const { registration, error } of extensionScan.errors) {
-        const message = error instanceof Error ? error.message : String(error);
+        const message = errorText(error);
         diagnostics.push(diagnostic("error", "ROLE_DIRECTORY", `${extensionLabel(registration.extension)} role directory "${registration.path}" could not be scanned: ${message}`, registration.path, "Fix or remove the registered role directory."));
     }
     for (const registration of extensionScan.empty)
@@ -500,7 +500,7 @@ export async function doctor(options = {}) {
                         diagnostics.push(...roleInspection.setup.diagnostics);
                 }
                 catch (error) {
-                    diagnostics.push(diagnostic("error", "ROLE_INSPECTION_MODEL", error instanceof Error ? error.message : String(error), activeRole.path));
+                    diagnostics.push(diagnostic("error", "ROLE_INSPECTION_MODEL", errorText(error), activeRole.path));
                 }
             }
         }

package/dist/src/session-inspector.js

@@ -4,7 +4,7 @@ import { emitKeypressEvents } from "node:readline";
 import { createInterface } from "node:readline/promises";
 import { stdin, stdout } from "node:process";
 import { highlightCode, initTheme, SessionManager, truncateToVisualLines } from "@earendil-works/pi-coding-agent";
-import { formatBudgetStatus, inspectWorkflowScript } from "pi-extensible-workflows";
+import { errorText, formatBudgetStatus, inspectWorkflowScript } from "pi-extensible-workflows";
 import { listPersistedSessionIds, listRunIds, RunStore } from "pi-extensible-workflows/persistence";
 function text(content) {
     if (typeof content === "string")
@@ -242,7 +242,7 @@ export async function loadSessionReport(path, home = homedir()) {
                 staticCalls = inspectWorkflowScript(script);
             }
             catch (error) {
-                parseError = error instanceof Error ? error.message : String(error);
+                parseError = errorText(error);
             }
         }
         workflows.push({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@piewf/cli",
-  "version": "5.5.0",
+  "version": "5.6.1",
   "description": "CLI for pi-extensible-workflows",
   "homepage": "https://vekexasia.github.io/pi-extensible-workflows/",
   "repository": {
@@ -46,7 +46,7 @@
   },
   "license": "MIT",
   "dependencies": {
-    "pi-extensible-workflows": "5.5.0"
+    "pi-extensible-workflows": "5.6.1"
   },
   "publishConfig": {
     "access": "public"