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

package/dist/output/shapes/helpers.js

@@ -11,11 +11,12 @@ export function shapeProposalProducerOutput(result, detail) {
             ...(result.ref !== undefined ? { ref: result.ref } : {}),
             ...(result.type !== undefined ? { type: result.type } : {}),
             ...(result.name !== undefined ? { name: result.name } : {}),
+            ...(result.engine !== undefined ? { engine: result.engine } : {}),
             ...(result.exitCode !== undefined ? { exitCode: result.exitCode } : {}),
         };
         if (detail === "full") {
             return {
-                schemaVersion: result.schemaVersion ?? 1,
+                schemaVersion: result.schemaVersion,
                 ...base,
                 ...(result.stdout !== undefined ? { stdout: result.stdout } : {}),
                 ...(result.stderr !== undefined ? { stderr: result.stderr } : {}),
@@ -23,16 +24,16 @@ export function shapeProposalProducerOutput(result, detail) {
         }
         return base;
     }
-    const proposal = result.proposal ?? {};
+    const proposal = result.proposal;
     const base = {
-        ok: true,
+        ok: result.ok,
         ref: result.ref,
-        ...(result.agentProfile !== undefined ? { agentProfile: result.agentProfile } : {}),
+        ...(result.engine !== undefined ? { engine: result.engine } : {}),
         ...(typeof result.durationMs === "number" ? { durationMs: result.durationMs } : {}),
         proposal: shapeProposalEntry(proposal, detail === "brief" ? "normal" : detail),
     };
     if (detail === "full") {
-        return { schemaVersion: result.schemaVersion ?? 1, ...base };
+        return { schemaVersion: result.schemaVersion, ...base };
     }
     return base;
 }
@@ -42,8 +43,7 @@ export function shapeProposalEntry(entry, detail) {
     }
     if (detail === "normal") {
         // `confidence` and `gateDecision` (#577) explain why a proposal is pending,
-        // so they are projected at `normal` for `akm proposal list/show` — both are
-        // optional and absent on legacy proposals.
+        // so they are projected at `normal` for `akm proposal list/show` when present.
         return pickFields(entry, [
             "id",
             "ref",
@@ -73,74 +73,54 @@ export function shapeProposalEntry(entry, detail) {
     ]);
 }
 export function shapeProposalListOutput(result, detail) {
-    const proposals = Array.isArray(result.proposals) ? result.proposals : [];
+    const proposals = result.proposals;
     const shaped = proposals.map((p) => shapeProposalEntry(p, detail));
     const base = {
-        totalCount: result.totalCount ?? shaped.length,
+        totalCount: result.totalCount,
         proposals: shaped,
     };
     if (detail === "full") {
-        return { schemaVersion: result.schemaVersion ?? 1, ...base };
+        return { schemaVersion: result.schemaVersion, ...base };
     }
     return base;
 }
 export function shapeProposalShowOutput(result, detail) {
-    const proposal = result.proposal ?? {};
+    const proposal = result.proposal;
     const validation = result.validation;
     const base = {
         proposal: shapeProposalEntry(proposal, detail === "brief" ? "normal" : detail),
         ...(validation ? { validation } : {}),
     };
     if (detail === "full") {
-        return { schemaVersion: result.schemaVersion ?? 1, ...base };
+        return { schemaVersion: result.schemaVersion, ...base };
     }
     return base;
 }
 export function shapeProposalAcceptOutput(result, detail) {
-    const proposal = result.proposal ?? {};
+    const proposal = result.proposal;
     const base = {
-        ok: result.ok ?? true,
+        ok: result.ok,
         id: result.id,
         ref: result.ref,
         assetPath: result.assetPath,
         proposal: shapeProposalEntry(proposal, detail === "brief" ? "normal" : detail),
     };
     if (detail === "full") {
-        return { schemaVersion: result.schemaVersion ?? 1, ...base };
+        return { schemaVersion: result.schemaVersion, ...base };
     }
     return base;
 }
 export function shapeProposalRejectOutput(result, detail) {
-    const proposal = result.proposal ?? {};
+    const proposal = result.proposal;
     const base = {
-        ok: result.ok ?? true,
+        ok: result.ok,
         id: result.id,
         ref: result.ref,
         ...(result.reason !== undefined ? { reason: result.reason } : {}),
         proposal: shapeProposalEntry(proposal, detail === "brief" ? "normal" : detail),
     };
     if (detail === "full") {
-        return { schemaVersion: result.schemaVersion ?? 1, ...base };
-    }
-    return base;
-}
-export function shapeDistillOutput(result, detail) {
-    const proposal = result.proposal;
-    if (detail === "brief") {
-        return pickFields(result, ["ok", "outcome", "inputRef", "lessonRef", "proposalId", "message"]);
-    }
-    const base = {
-        ok: result.ok ?? true,
-        outcome: result.outcome,
-        inputRef: result.inputRef,
-        lessonRef: result.lessonRef,
-        ...(result.proposalId !== undefined ? { proposalId: result.proposalId } : {}),
-        ...(result.message !== undefined ? { message: result.message } : {}),
-        ...(Array.isArray(result.findings) && result.findings.length > 0 ? { findings: result.findings } : {}),
-        ...(proposal ? { proposal: shapeProposalEntry(proposal, detail) } : {}),
-    };
-    if (detail === "full") {
-        return { schemaVersion: result.schemaVersion ?? 1, ...base };
+        return { schemaVersion: result.schemaVersion, ...base };
     }
     return base;
 }
@@ -153,7 +133,7 @@ export function shapeProposalDiffOutput(result, detail) {
         ...(result.targetPath !== undefined ? { targetPath: result.targetPath } : {}),
     };
     if (detail === "full") {
-        return { schemaVersion: result.schemaVersion ?? 1, ...base };
+        return { schemaVersion: result.schemaVersion, ...base };
     }
     return base;
 }
@@ -163,8 +143,14 @@ export function shapeEventsOutput(result, detail) {
     const base = {
         ...(result.ref !== undefined ? { ref: result.ref } : {}),
         ...(result.type !== undefined ? { type: result.type } : {}),
+        ...(result.run !== undefined ? { run: result.run } : {}),
         ...(result.since !== undefined ? { since: result.since } : {}),
         ...(typeof result.sinceOffset === "number" ? { sinceOffset: result.sinceOffset } : {}),
+        // D-38: echo `--limit` through the shaped envelope too — `akmEventsList`
+        // already conditionally includes it in its raw result (see
+        // src/commands/events.ts) only when the caller passed it, so this mirrors
+        // that same "present only if requested" convention.
+        ...(typeof result.limit === "number" ? { limit: result.limit } : {}),
         totalCount: result.totalCount ?? shapedEvents.length,
         events: shapedEvents,
     };
@@ -189,45 +175,6 @@ export function shapeEventEntry(entry, detail) {
     // full: project everything the reader emits.
     return pickFields(entry, ["id", "schemaVersion", "eventType", "ref", "ts", "metadata"]);
 }
-export function shapeHistoryOutput(result, detail) {
-    const entries = Array.isArray(result.entries) ? result.entries : [];
-    const shapedEntries = entries.map((entry) => shapeHistoryEntry(entry, detail));
-    if (detail === "full") {
-        return {
-            schemaVersion: result.schemaVersion ?? 1,
-            ...(result.ref !== undefined ? { ref: result.ref } : {}),
-            ...(result.since !== undefined ? { since: result.since } : {}),
-            totalCount: result.totalCount ?? shapedEntries.length,
-            entries: shapedEntries,
-            // `sources` lists the event sources included in this response.
-            // Always contains "usage_events"; also "events.jsonl" when
-            // --include-proposals was specified.
-            ...(Array.isArray(result.sources) ? { sources: result.sources } : {}),
-            ...(Array.isArray(result.warnings) && result.warnings.length > 0 ? { warnings: result.warnings } : {}),
-        };
-    }
-    return {
-        ...(result.ref !== undefined ? { ref: result.ref } : {}),
-        ...(result.since !== undefined ? { since: result.since } : {}),
-        totalCount: result.totalCount ?? shapedEntries.length,
-        entries: shapedEntries,
-        ...(Array.isArray(result.sources) ? { sources: result.sources } : {}),
-        ...(Array.isArray(result.warnings) && result.warnings.length > 0 ? { warnings: result.warnings } : {}),
-    };
-}
-export function shapeHistoryEntry(entry, detail) {
-    if (detail === "brief") {
-        // signal is load-bearing for feedback rows (positive/negative) so we
-        // project it even at brief — without it the entry is ambiguous.
-        // source lets callers verify filter correctness (e.g. --source user).
-        return pickFields(entry, ["eventType", "ref", "signal", "source", "createdAt"]);
-    }
-    if (detail === "normal") {
-        return pickFields(entry, ["eventType", "ref", "signal", "source", "query", "createdAt"]);
-    }
-    // full: return everything the reader emits.
-    return pickFields(entry, ["id", "eventType", "ref", "entryId", "query", "signal", "source", "metadata", "createdAt"]);
-}
 export function shapeSearchOutput(result, detail, shape = "human") {
     const forAgent = shape === "agent";
     const hits = Array.isArray(result.hits) ? result.hits : [];
@@ -248,11 +195,10 @@ export function shapeSearchOutput(result, detail, shape = "human") {
     if (detail === "full") {
         return {
             schemaVersion: result.schemaVersion,
-            stashDir: result.stashDir,
+            bundleDir: result.bundleDir,
             source: result.source,
             hits: shapedHits,
             ...(shapedRegistryHits.length > 0 ? { registryHits: shapedRegistryHits } : {}),
-            ...(result.semanticSearch ? { semanticSearch: result.semanticSearch } : {}),
             ...(result.tip ? { tip: result.tip } : {}),
             ...(result.warnings ? { warnings: result.warnings } : {}),
             ...(result.timing ? { timing: result.timing } : {}),
@@ -330,7 +276,21 @@ export function shapeSearchHit(hit, detail) {
 }
 /** Agent-optimized search hit: only fields an LLM agent needs to decide and act */
 export function shapeSearchHitForAgent(hit) {
-    const picked = pickFields(hit, ["name", "ref", "type", "description", "action", "score", "estimatedTokens", "keys"]);
+    const picked = pickFields(hit, [
+        "name",
+        "ref",
+        "type",
+        "path",
+        "editable",
+        "editHint",
+        "description",
+        "action",
+        "score",
+        "estimatedTokens",
+        "keys",
+    ]);
+    if (picked.editable !== false)
+        delete picked.editHint;
     return capDescription(picked, NORMAL_DESCRIPTION_LIMIT);
 }
 export function capDescription(hit, limit) {
@@ -349,9 +309,13 @@ export function truncateDescription(description, limit) {
 }
 export function shapeShowOutput(result, detail, shape = "human") {
     if (shape === "agent") {
-        return pickFields(result, [
+        const shaped = pickFields(result, [
             "type",
             "name",
+            "ref",
+            "path",
+            "editable",
+            "editHint",
             "description",
             "action",
             "content",
@@ -371,11 +335,18 @@ export function shapeShowOutput(result, detail, shape = "human") {
             "keys",
             "related",
         ]);
+        if (shaped.editable !== false)
+            delete shaped.editHint;
+        return shaped;
     }
     if (shape === "summary") {
         return pickFields(result, [
             "type",
             "name",
+            // ref is present on every show shape (human/summary/agent) — it is the
+            // canonical identity of the asset, not an agent-only convenience field
+            // (R-020).
+            "ref",
             "description",
             "tags",
             "parameters",
@@ -390,6 +361,8 @@ export function shapeShowOutput(result, detail, shape = "human") {
     const base = pickFields(result, [
         "type",
         "name",
+        // ref is always projected, same as path/editable below (R-020).
+        "ref",
         "origin",
         "action",
         "description",
@@ -410,8 +383,9 @@ export function shapeShowOutput(result, detail, shape = "human") {
         "activeRun",
         "keys",
         "related",
-        // path and editable are always projected so JSON consumers can locate and
-        // edit the asset without needing --detail full (QA #7).
+        // ref, path, and editable are always projected — at every --detail level,
+        // not just --detail full — so JSON consumers can locate and edit the
+        // asset without needing --detail full (QA #7 / D-14).
         "path",
         "editable",
     ]);

package/dist/output/shapes/passthrough.js

@@ -21,33 +21,20 @@ function makeStampHandler(command) {
 const PASSTHROUGH_COMMANDS = [
     "add",
     "agent-result",
+    "bundle-create",
+    "bundle-show",
     "clone",
     "config",
-    "disable",
-    "enable",
     "env-create",
     "env-export",
     "env-remove",
-    "env-set",
-    "env-unset",
     "feedback",
-    "graph-entities",
-    "graph-entity",
-    "graph-export",
-    "graph-orphans",
-    "graph-related",
-    "graph-relations",
-    "graph-summary",
-    "graph-update",
     "extract",
     "health",
     "improve",
-    "improve-canary",
-    "lessons-coverage",
     "import",
     "index",
     "info",
-    "init",
     "lint",
     "list",
     "proposal-accept-batch",
@@ -55,47 +42,32 @@ const PASSTHROUGH_COMMANDS = [
     "proposal-reject-batch",
     "proposal-revert",
     "registry-add",
-    "registry-build-index",
     "registry-list",
     "registry-remove",
     "remember",
     "remove",
-    "save",
     "secret-set",
-    "secret-remove",
     "setup",
-    "tasks-add",
-    "tasks-disable",
-    "tasks-doctor",
-    "tasks-enable",
-    "tasks-history",
-    "tasks-init",
-    "tasks-list",
-    "tasks-remove",
-    "tasks-run",
-    "tasks-show",
-    "tasks-sync",
+    "sync",
+    "task-add",
+    "task-doctor",
+    "task-history",
+    "task-run",
+    "task-sync",
     "update",
     "upgrade",
-    "wiki-create",
-    "wiki-ingest",
-    "wiki-lint",
-    "wiki-list",
-    "wiki-pages",
-    "wiki-register",
-    "wiki-remove",
-    "wiki-show",
-    "wiki-stash",
     "workflow-abandon",
+    "workflow-brief",
     "workflow-complete",
     "workflow-complete-rejected",
     "workflow-create",
     "workflow-list",
     "workflow-next",
+    "workflow-report",
     "workflow-resume",
+    "workflow-run",
     "workflow-start",
     "workflow-status",
-    "workflow-validate",
 ];
 export const passthroughShapes = PASSTHROUGH_COMMANDS.map((command) => ({
     command,

package/dist/output/shapes/proposal/producer.js

@@ -1,12 +1,20 @@
 // This Source Code Form is subject to the terms of the Mozilla Public
 // 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 shape registration for `akm reflect` and `akm propose` (#226).
-// Both share the proposal-producer envelope shape (success carries a proposal
-// entry; failure carries an AgentFailureReason discriminant).
+// Output shape registration for `akm proposal new` (#226; formerly the
+// top-level `akm propose`, moved under `proposal` in the 0.9 CLI overhaul, S8).
+// Shares the proposal-producer envelope shape (success carries a proposal
+// entry; failure carries an AgentFailureReason discriminant) with the
+// internal `akmReflect` result shape — but `reflect` has no standalone CLI
+// verb and never reaches the output() chokepoint (reflect.ts is called
+// directly by the improve loop, not through this registry), so only
+// `proposal new` is registered here.
+//
+// Deliberately phrased WITHOUT the literal call spelling: the registry
+// completeness scanner (tests/integration/output-shape-registry-completeness.ts)
+// greps sources for that exact pattern to find commands needing a shape, and
+// it does not skip comments — so writing the spelling here to explain the
+// absence would re-summon the very registration this comment says is gone.
 import { shapeProposalProducerOutput } from "../helpers.js";
 const handler = (result, detail) => shapeProposalProducerOutput(result, detail);
-export const proposalProducerShapes = [
-    { command: "reflect", handler },
-    { command: "propose", handler },
-];
+export const proposalProducerShapes = [{ command: "proposal-new", handler }];

package/dist/output/shapes/registry.js

@@ -1,7 +1,15 @@
 // This Source Code Form is subject to the terms of the Mozilla Public
 // 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/.
-const OUTPUT_SHAPE_REGISTRY = new Map();
+/**
+ * Output shape registry.
+ *
+ * Separated from `shapes.ts` so per-command shape modules can import
+ * `registerOutputShape` without creating a circular dependency back into
+ * the main shapes module.
+ */
+import { createCommandRegistry } from "../command-registry.js";
+const OUTPUT_SHAPE_REGISTRY = createCommandRegistry();
 /**
  * Register an output shape handler for a command name.
  *
@@ -13,7 +21,7 @@ const OUTPUT_SHAPE_REGISTRY = new Map();
  * ```
  */
 export function registerOutputShape(command, handler) {
-    OUTPUT_SHAPE_REGISTRY.set(command, handler);
+    OUTPUT_SHAPE_REGISTRY.register(command, handler);
 }
 /**
  * Register a batch of {@link OutputShapeEntry} definitions in iteration order.
@@ -23,15 +31,13 @@ export function registerOutputShape(command, handler) {
  * with no reliance on module import order.
  */
 export function registerOutputShapes(entries) {
-    for (const { command, handler } of entries) {
-        OUTPUT_SHAPE_REGISTRY.set(command, handler);
-    }
+    OUTPUT_SHAPE_REGISTRY.registerAll(entries);
 }
 /**
  * Remove a previously-registered output shape. Test-only utility.
  */
 export function deregisterOutputShape(command) {
-    OUTPUT_SHAPE_REGISTRY.delete(command);
+    OUTPUT_SHAPE_REGISTRY.deregister(command);
 }
 /**
  * Look up a registered output shape handler by command name.

package/dist/output/shapes.js

@@ -19,10 +19,8 @@
  */
 import { UsageError } from "../core/errors.js";
 import { curateShapes } from "./shapes/curate.js";
-import { distillShapes } from "./shapes/distill.js";
 import { envListShapes } from "./shapes/env-list.js";
 import { eventsShapes } from "./shapes/events.js";
-import { historyShapes } from "./shapes/history.js";
 import { passthroughShapes } from "./shapes/passthrough.js";
 import { proposalAcceptShapes } from "./shapes/proposal/accept.js";
 import { proposalDiffShapes } from "./shapes/proposal/diff.js";
@@ -35,11 +33,6 @@ import { registrySearchShapes } from "./shapes/registry-search.js";
 import { searchShapes } from "./shapes/search.js";
 import { secretListShapes } from "./shapes/secret-list.js";
 import { showShapes } from "./shapes/show.js";
-// Re-export helpers so existing imports from `shapes.ts` keep working.
-export { capDescription, NORMAL_DESCRIPTION_LIMIT, pickFields, shapeAssetHit, shapeDistillOutput, shapeEventEntry, shapeEventsOutput, shapeHistoryEntry, shapeHistoryOutput, shapeProposalAcceptOutput, shapeProposalDiffOutput, shapeProposalEntry, shapeProposalListOutput, shapeProposalProducerOutput, shapeProposalRejectOutput, shapeProposalShowOutput, shapeRegistrySearchOutput, shapeSearchHit, shapeSearchHitForAgent, shapeSearchOutput, shapeShowOutput, truncateDescription, } from "./shapes/helpers.js";
-// Re-export registry API so callers can use this module as the single entry
-// point (backward compat).
-export { deregisterOutputShape, registerOutputShape } from "./shapes/registry.js";
 // ── Explicit built-in shape assembly ──────────────────────────────────────────
 // Each entry below is a pure exported `OutputShapeEntry[]` from a per-command
 // module. The set is registered ONCE, deterministically, with no reliance on
@@ -51,7 +44,6 @@ const BUILT_IN_OUTPUT_SHAPES = [
     ...curateShapes,
     ...registrySearchShapes,
     ...showShapes,
-    ...historyShapes,
     ...eventsShapes,
     ...proposalListShapes,
     ...proposalShowShapes,
@@ -59,7 +51,6 @@ const BUILT_IN_OUTPUT_SHAPES = [
     ...proposalRejectShapes,
     ...proposalDiffShapes,
     ...proposalProducerShapes,
-    ...distillShapes,
     ...envListShapes,
     ...secretListShapes,
     // Passthrough commands are registered last so an explicit dedicated handler

package/dist/output/text/{init.js → bundle-create.js}

@@ -2,4 +2,6 @@
 // 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/.
 import { formatInitPlain } from "./helpers.js";
-export const initFormatters = [{ command: "init", handler: (r) => formatInitPlain(r) }];
+export const bundleCreateFormatters = [
+    { command: "bundle-create", handler: (r) => formatInitPlain(r) },
+];

package/dist/output/text/bundle-show.js

@@ -0,0 +1,7 @@
+// This Source Code Form is subject to the terms of the Mozilla Public
+// 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/.
+import { formatBundleShowPlain } from "./helpers.js";
+export const bundleShowFormatters = [
+    { command: "bundle-show", handler: (r) => formatBundleShowPlain(r) },
+];