akm-cli 0.9.0-rc.13 → 0.9.0-rc.15

package/dist/output/text/workflow-format.js

@@ -5,33 +5,14 @@
  * Plain-text renderers for `akm workflow *` commands.
  *
  * Split out of `helpers.ts` (formerly 1418 lines / 59 fns) as its own
- * sibling module: the workflow run/step/status/brief renderers form a
+ * sibling module: the workflow list/status/run/create/resume renderers form a
  * cohesive, self-contained cluster (they call only each other, never
  * formatters from other domains).
  */
-/**
- * Plain-text rendering for a step-completion that was rejected by the
- * summary-validation gate (#506): the step stays pending and the agent gets
- * corrective feedback on what to finish/fix.
- */
-export function formatWorkflowCompleteRejectedPlain(r) {
-    const stepId = String(r.stepId ?? "?");
-    const feedback = typeof r.feedback === "string" ? r.feedback : "";
-    const missing = Array.isArray(r.missing) ? r.missing.map((m) => String(m)) : [];
-    const lines = [`workflow complete: rejected — step "${stepId}" does not meet its completion criteria`];
-    if (feedback)
-        lines.push(`  feedback: ${feedback}`);
-    if (missing.length > 0) {
-        lines.push("  outstanding:");
-        for (const m of missing)
-            lines.push(`    - ${m}`);
-    }
-    return lines.join("\n");
-}
 export function formatWorkflowListPlain(result) {
     const runs = Array.isArray(result.runs) ? result.runs : [];
     if (runs.length === 0) {
-        return "No workflow runs in the current working scope. Start one with `akm workflow next workflows/<name>` or author one with `akm workflow create <name>`.";
+        return "No workflow runs in the current working scope. Start one with `akm workflow run workflows/<name>` or author one with `akm workflow create <name>`.";
     }
     return runs
         .map((run) => {
@@ -89,8 +70,8 @@ export function formatWorkflowStatusPlain(result) {
             const suffix = attempts !== undefined && attempts > 1 ? `, attempt ${attempts}` : "";
             lines.push(`  - ${id} [${node}] (${status}${suffix})`);
             // Codex round-3 finding B: a `running` claim gone silent past the check-in
-            // window — the driver likely died. Surface it (with the claim holder) so a
-            // human can reclaim/re-run the unit, matching what `brief` reports.
+            // window — the process holding it likely died. Surface it (with the claim
+            // holder) so a human can reclaim/re-run the unit.
             if (unit.stale === true) {
                 const holder = typeof unit.claimHolder === "string" && unit.claimHolder.trim() ? ` claimed by ${unit.claimHolder}` : "";
                 lines.push(`    stale: claim went silent past the check-in window${holder} — its driver may have died`);
@@ -117,7 +98,7 @@ export function formatWorkflowStatusPlain(result) {
 }
 /**
  * Render the stalled-run check-in directive (#506) when present on a
- * workflow-next/status result. Returns null when the run is healthy.
+ * workflow-status result. Returns null when the run is healthy.
  */
 function formatWorkflowCheckinLine(result) {
     const checkin = typeof result.checkin === "object" && result.checkin !== null
@@ -127,52 +108,16 @@ function formatWorkflowCheckinLine(result) {
         return null;
     return checkin.directive.trim();
 }
-export function formatWorkflowNextPlain(result) {
-    const base = formatWorkflowStatusPlain(result);
-    const step = typeof result.step === "object" && result.step !== null ? result.step : undefined;
-    if (!step)
-        return base;
-    const lines = base ? [base, "", "next:"] : ["next:"];
-    lines.push(`  ${String(step.title ?? "Untitled step")} [${String(step.id ?? "unknown")}]`);
-    if (typeof step.instructions === "string" && step.instructions.trim()) {
-        const instrLines = step.instructions.trim().split("\n");
-        lines.push(`  instructions: ${instrLines[0]}`);
-        for (const instrLine of instrLines.slice(1))
-            lines.push(`    ${instrLine}`);
-    }
-    const completion = Array.isArray(step.completionCriteria) ? step.completionCriteria : [];
-    if (completion.length > 0) {
-        lines.push("  completion:");
-        for (const criterion of completion) {
-            lines.push(`    - ${String(criterion)}`);
-        }
-    }
-    // T2-3: surface run-id as labeled field
-    const run = typeof result.run === "object" && result.run !== null ? result.run : undefined;
-    const runId = typeof run?.id === "string" ? run.id : null;
-    const stepId = typeof step?.id === "string" ? step.id : null;
-    if (runId) {
-        lines.push("");
-        lines.push(`runId: ${runId}`);
-    }
-    // T1-6: complete command
-    if (runId && stepId) {
-        lines.push("");
-        lines.push("COMPLETE THIS STEP:");
-        lines.push(`  akm workflow complete '${runId}' --step '${stepId}'`);
-    }
-    else if (runId) {
-        lines.push("");
-        lines.push("COMPLETE THIS STEP:");
-        lines.push(`  akm workflow complete '${runId}' --step '<step-id>'`);
-    }
-    return lines.join("\n");
-}
 export function formatWorkflowRunPlain(result) {
     const run = typeof result.run === "object" && result.run !== null ? result.run : undefined;
     if (!run)
         return null;
     const lines = [`run: ${String(run.id ?? "unknown")}`, `status: ${String(run.status ?? "unknown")}`];
+    // Creation-time notices (e.g. the implicit engine fallback) must survive the
+    // text renderer: JSON/YAML pass them through, and dropping them here would
+    // hide the announcement from the DEFAULT output mode — where it matters most.
+    for (const warning of Array.isArray(result.warnings) ? result.warnings : [])
+        lines.push(`! ${String(warning)}`);
     const executed = Array.isArray(result.executed) ? result.executed : [];
     if (executed.length === 0) {
         lines.push("executed: (no steps — run was already done or blocked)");
@@ -197,126 +142,15 @@ export function formatWorkflowRunPlain(result) {
         for (const item of missing)
             lines.push(`  missing: ${String(item)}`);
     }
+    if (result.aborted === true) {
+        lines.push(result.timedOut === true
+            ? "workflow timed out; the run remains resumable."
+            : "workflow interrupted; the run remains resumable.");
+    }
     if (result.done === true)
         lines.push("workflow completed.");
     return lines.join("\n");
 }
-export function formatWorkflowBriefPlain(result) {
-    const run = typeof result.run === "object" && result.run !== null ? result.run : undefined;
-    if (!run)
-        return null;
-    const lines = [];
-    lines.push(`# Workflow brief: ${String(run.id ?? "unknown")}`);
-    lines.push(`workflow: ${String(run.workflowRef ?? "?")}  (${String(run.workflowTitle ?? "")})`);
-    lines.push(`status: ${String(run.status ?? "?")}`);
-    if (typeof result.message === "string")
-        lines.push(result.message);
-    const lease = typeof result.engineLease === "object" && result.engineLease !== null
-        ? result.engineLease
-        : undefined;
-    if (lease) {
-        const live = lease.live === true ? "LIVE" : "expired";
-        lines.push(`engine lease: ${String(lease.holder ?? "?")} (until ${String(lease.until ?? "?")}) [${live}]`);
-    }
-    const warnings = Array.isArray(result.warnings) ? result.warnings : [];
-    for (const w of warnings)
-        lines.push(`! ${String(w)}`);
-    if (result.done === true) {
-        lines.push("");
-        lines.push("This run is completed — no work remains.");
-        return lines.join("\n");
-    }
-    const step = typeof result.step === "object" && result.step !== null ? result.step : undefined;
-    if (!step) {
-        return lines.join("\n");
-    }
-    const gate = typeof step.gate === "object" && step.gate !== null ? step.gate : undefined;
-    lines.push("");
-    lines.push(`## Active step: ${String(step.stepId ?? "?")} — ${String(step.title ?? "")}  [${String(step.kind ?? "execute")}]`);
-    if (gate) {
-        const criteria = Array.isArray(gate.criteria) ? gate.criteria : [];
-        lines.push(`gate: loop ${String(gate.currentLoop ?? 1)} of max ${String(gate.maxLoops ?? 1)}` +
-            (criteria.length > 0 ? `; criteria: ${criteria.map(String).join("; ")}` : "; no completion criteria") +
-            (gate.judgesArtifact === true ? " (artifact-judged)" : ""));
-    }
-    if (step.outputSchema !== undefined)
-        lines.push("outputSchema: declared (see JSON output)");
-    const feedback = typeof result.gateFeedback === "object" && result.gateFeedback !== null
-        ? result.gateFeedback
-        : undefined;
-    if (feedback) {
-        lines.push(`gate feedback (previous loop rejected): ${String(feedback.feedback ?? "")}`);
-        const missing = Array.isArray(feedback.missing) ? feedback.missing : [];
-        for (const m of missing)
-            lines.push(`  - missing: ${String(m)}`);
-    }
-    const route = typeof result.route === "object" && result.route !== null ? result.route : undefined;
-    if (route) {
-        lines.push("");
-        lines.push(`## Route contract on ${String(route.input ?? "?")}`);
-        const when = typeof route.when === "object" && route.when !== null ? route.when : {};
-        for (const [value, target] of Object.entries(when))
-            lines.push(`  when "${value}" → ${String(target)}`);
-        if (route.defaultStepId !== undefined)
-            lines.push(`  default → ${String(route.defaultStepId)}`);
-        const decision = typeof route.decision === "object" && route.decision !== null
-            ? route.decision
-            : undefined;
-        if (decision)
-            lines.push(`  decision NOW: value "${String(decision.value)}" → ${String(decision.selected)}`);
-        else if (typeof route.decisionError === "string")
-            lines.push(`  decision error: ${route.decisionError}`);
-        else
-            lines.push("  decision: pending this step's output (evaluated after units complete)");
-    }
-    const workList = typeof result.workList === "object" && result.workList !== null
-        ? result.workList
-        : undefined;
-    const units = workList && Array.isArray(workList.units) ? workList.units : [];
-    if (workList?.error) {
-        lines.push("");
-        lines.push(`work-list error: ${String(workList.error)}`);
-    }
-    else if (units.length > 0) {
-        lines.push("");
-        lines.push(`## Units (${units.length})`);
-        for (const u of units) {
-            const model = u.model ? ` model=${String(u.model)}` : "";
-            const journaled = typeof u.journaled === "object" && u.journaled !== null ? u.journaled : undefined;
-            const jstatus = journaled ? ` [journaled: ${String(journaled.status)}]` : "";
-            lines.push(`- ${String(u.unitId)}  (runner=${String(u.runner)}${model})${jstatus}`);
-            const env = Array.isArray(u.env) ? u.env : [];
-            if (env.length > 0)
-                lines.push(`    env (names): ${env.map(String).join(", ")}`);
-            if (u.outputSchema !== undefined)
-                lines.push("    outputSchema: declared (see JSON output)");
-            const resolved = typeof u.resolved === "object" && u.resolved !== null ? u.resolved : undefined;
-            if (resolved && resolved.ok === false)
-                lines.push(`    RESOLUTION ERROR: ${String(resolved.error)}`);
-            if (typeof u.report === "string")
-                lines.push(`    report: ${u.report}`);
-        }
-    }
-    // Finding D: a non-dispatching step (route-only / empty / all-unresolvable)
-    // advances via the `--settle` verb, surfaced so a driver can advance the spine.
-    if (typeof result.settleCommand === "string") {
-        lines.push("");
-        lines.push("## Advance (no reportable units)");
-        lines.push(`settle: ${result.settleCommand}`);
-    }
-    const guidance = typeof result.reportGuidance === "object" && result.reportGuidance !== null
-        ? result.reportGuidance
-        : undefined;
-    if (guidance) {
-        lines.push("");
-        lines.push("## Reporting");
-        if (typeof guidance.checkin === "string")
-            lines.push(`heartbeat: ${guidance.checkin}`);
-        if (typeof guidance.failure === "string")
-            lines.push(`on failure: ${guidance.failure}`);
-    }
-    return lines.join("\n");
-}
 export function formatWorkflowCreatePlain(r) {
     // `workflow create --print`: no file is written, just the template text
     // (dropped `workflow template`'s output, moved onto this shape).

package/dist/output/text/workflow.js

@@ -2,17 +2,12 @@
 // License, v. 2.0. If a copy of the MPL was not distributed with this
 // file, You can obtain one at https://mozilla.org/MPL/2.0/.
 // Output text formatters for all `akm workflow *` commands.
-import { formatWorkflowBriefPlain, formatWorkflowCompleteRejectedPlain, formatWorkflowCreatePlain, formatWorkflowListPlain, formatWorkflowNextPlain, formatWorkflowResumePlain, formatWorkflowRunPlain, formatWorkflowStatusPlain, } from "./helpers.js";
+import { formatWorkflowCreatePlain, formatWorkflowListPlain, formatWorkflowResumePlain, formatWorkflowRunPlain, formatWorkflowStatusPlain, } from "./helpers.js";
 export const workflowFormatters = [
-    { command: "workflow-start", handler: (r) => formatWorkflowStatusPlain(r) },
     { command: "workflow-status", handler: (r) => formatWorkflowStatusPlain(r) },
-    { command: "workflow-complete", handler: (r) => formatWorkflowStatusPlain(r) },
-    { command: "workflow-complete-rejected", handler: (r) => formatWorkflowCompleteRejectedPlain(r) },
-    { command: "workflow-next", handler: (r) => formatWorkflowNextPlain(r) },
     { command: "workflow-list", handler: (r) => formatWorkflowListPlain(r) },
     { command: "workflow-create", handler: (r) => formatWorkflowCreatePlain(r) },
     { command: "workflow-resume", handler: (r) => formatWorkflowResumePlain(r) },
     { command: "workflow-abandon", handler: (r) => formatWorkflowStatusPlain(r) },
     { command: "workflow-run", handler: (r) => formatWorkflowRunPlain(r) },
-    { command: "workflow-brief", handler: (r) => formatWorkflowBriefPlain(r) },
 ];