akm-cli 0.8.0-rc2 → 0.8.1

package/dist/output/shapes/vault-list.js

@@ -0,0 +1,19 @@
+// 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/.
+// vault-list strips `path` from each vault object (security fix M3: avoid
+// leaking absolute disk paths) then stamps the envelope (#484).
+import { registerOutputShape } from "./registry";
+registerOutputShape("vault-list", (result) => {
+    const r = result;
+    const vaults = Array.isArray(r.vaults) ? r.vaults : [];
+    return {
+        ...r,
+        shape: r.shape ?? "vault-list",
+        schemaVersion: r.schemaVersion ?? 1,
+        vaults: vaults.map((v) => {
+            const { path: _path, ...rest } = v;
+            return rest;
+        }),
+    };
+});

package/dist/output/shapes.js

@@ -1,561 +1,63 @@
+// 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/.
 /**
  * Pure shaping functions that select and trim fields from command result
  * objects according to the active detail level / agent mode.
  *
  * Every function in this module is side-effect free and operates on plain
  * `Record<string, unknown>` shapes, which makes them trivial to unit test.
+ *
+ * Output shapes are registered via `registerOutputShape` — see the per-command
+ * modules in `src/output/shapes/` for individual registrations. The central
+ * `shapeForCommand` dispatcher looks up the registry and throws for unknown
+ * commands (v1 spec §9 — exhaustive registry, no silent fallback).
  */
-const NORMAL_DESCRIPTION_LIMIT = 250;
-export function shapeForCommand(command, result, detail, forAgent = false) {
-    switch (command) {
-        case "search":
-            return shapeSearchOutput(result, detail, forAgent);
-        case "registry-search":
-            return shapeRegistrySearchOutput(result, detail);
-        case "show":
-            return shapeShowOutput(result, detail, forAgent);
-        // Output shape registration for `akm history` — paired with the textRenderer in text.ts.
-        case "history":
-            return shapeHistoryOutput(result, detail);
-        // Output shape registration for `akm events list` and `akm events tail`
-        // (#204). Both share the same envelope; the renderer in text.ts uses
-        // distinct command names so it can format streaming differently.
-        case "events-list":
-        case "events-tail":
-            return shapeEventsOutput(result, detail);
-        // Output shape registration for `akm proposal {list,show,accept,reject,diff}`
-        // (#225). Each verb gets its own arm so the registry stays exhaustive (no
-        // silent JSON.stringify fallback). The proposal payload is reshaped per
-        // detail level — `brief` omits the full content body, while some proposal
-        // shapers still retain normal-level metadata such as review details;
-        // `full`/`agent` includes everything.
-        case "proposal-list":
-            return shapeProposalListOutput(result, detail);
-        case "proposal-show":
-            return shapeProposalShowOutput(result, detail);
-        case "proposal-accept":
-            return shapeProposalAcceptOutput(result, detail);
-        case "proposal-reject":
-            return shapeProposalRejectOutput(result, detail);
-        case "proposal-diff":
-            return shapeProposalDiffOutput(result, detail);
-        // 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).
-        case "reflect":
-        case "propose":
-            return shapeProposalProducerOutput(result, detail);
-        case "improve":
-            return result;
-        // Output shape registration for `akm distill <ref>` (#228). The shape is
-        // simple — outcome + ids + optional payload — so `brief` strips the full
-        // proposal blob, `normal` keeps the headline fields, and `full` projects
-        // everything for downstream automation.
-        case "distill":
-            return shapeDistillOutput(result, detail);
-        case "graph-summary":
-        case "graph-entities":
-        case "graph-relations":
-        case "graph-related":
-        case "graph-export":
-            return result;
-        // Identity-passthrough commands — registered here so the registry stays
-        // exhaustive (v1 spec §9). Each result object is already shaped at the
-        // command boundary; the registry just confirms there's no surprise
-        // command name slipping through.
-        case "add":
-        case "clone":
-        case "config":
-        case "curate":
-        case "disable":
-        case "enable":
-        case "feedback":
-        case "import":
-        case "index":
-        case "health":
-        case "info":
-        case "init":
-        case "list":
-        case "registry-add":
-        case "registry-build-index":
-        case "registry-list":
-        case "registry-remove":
-        case "remember":
-        case "remove":
-        case "save":
-        case "update":
-        case "upgrade":
-        case "vault-create":
-        case "vault-list":
-        case "vault-set":
-        case "vault-unset":
-        case "wiki-create":
-        case "wiki-ingest":
-        case "wiki-lint":
-        case "wiki-list":
-        case "wiki-pages":
-        case "wiki-register":
-        case "wiki-remove":
-        case "wiki-show":
-        case "wiki-stash":
-        case "workflow-complete":
-        case "workflow-create":
-        case "workflow-list":
-        case "workflow-next":
-        case "workflow-resume":
-        case "workflow-start":
-        case "workflow-status":
-        case "workflow-validate":
-        case "tasks-add":
-        case "tasks-list":
-        case "tasks-show":
-        case "tasks-remove":
-        case "tasks-enable":
-        case "tasks-disable":
-        case "tasks-run":
-        case "tasks-history":
-        case "tasks-sync":
-        case "tasks-doctor":
-            return result;
-        // Output shape registration for `akm lint`, `akm setup`, and `akm consolidate`.
-        // Each result is already fully shaped at the command boundary; pass through as-is.
-        case "lint":
-        case "setup":
-        case "consolidate":
-            return result;
-        // Output shape registration for `akm agent <profile>`. The result carries
-        // structured agent-run fields; the text renderer uses the interactive/
-        // captured distinction to decide what to print.
-        case "agent-result":
-            return result;
-        default:
-            // v1 spec §9 (output-shape registry exhaustive): no silent JSON.stringify
-            // fallback. A missing case here is a registration bug — fail loudly so
-            // the caller (or its tests) sees the missing command name.
-            throw new Error(`output shape not registered for command: ${command}`);
-    }
-}
+import { UsageError } from "../core/errors";
+import { getOutputShapeHandler } from "./shapes/registry";
+// 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";
+// Re-export registry API so callers can use this module as the single entry
+// point (backward compat).
+export { deregisterOutputShape, registerOutputShape } from "./shapes/registry";
+// ── Per-command shape modules (self-register at import time) ──────────────────
+// Importing these modules triggers their `registerOutputShape(...)` calls.
+// These imports must come AFTER the registry module has been loaded (guaranteed
+// by the import order above).
+import "./shapes/search";
+import "./shapes/curate";
+import "./shapes/registry-search";
+import "./shapes/show";
+import "./shapes/history";
+import "./shapes/events";
+import "./shapes/proposal-list";
+import "./shapes/proposal-show";
+import "./shapes/proposal-accept";
+import "./shapes/proposal-reject";
+import "./shapes/proposal-diff";
+import "./shapes/proposal-producer";
+import "./shapes/distill";
+import "./shapes/env-list";
+import "./shapes/vault-list";
+import "./shapes/secret-list";
+import "./shapes/passthrough";
+// ── Dispatcher ────────────────────────────────────────────────────────────────
 /**
- * Shape the result of `akm reflect` / `akm propose`. On success we surface
- * the queued proposal entry (using the standard proposal-entry shaper so
- * detail levels behave uniformly with `akm proposal show`). On failure we
- * surface the structured failure-reason envelope as-is — the failure
- * surface is small and the reason / error text is always load-bearing.
+ * Commands whose shape handler implements the `summary` projection. For every
+ * other command, `--shape summary` is a usage error (v1 §5 — honest rejection
+ * for a soon-frozen contract, not a silent fallback to `human`).
  */
-export function shapeProposalProducerOutput(result, detail) {
-    if (result.ok === false) {
-        const base = {
-            ok: false,
-            reason: result.reason,
-            error: result.error,
-            ...(result.ref !== undefined ? { ref: result.ref } : {}),
-            ...(result.type !== undefined ? { type: result.type } : {}),
-            ...(result.name !== undefined ? { name: result.name } : {}),
-            ...(result.exitCode !== undefined ? { exitCode: result.exitCode } : {}),
-        };
-        if (detail === "full") {
-            return {
-                schemaVersion: result.schemaVersion ?? 1,
-                ...base,
-                ...(result.stdout !== undefined ? { stdout: result.stdout } : {}),
-                ...(result.stderr !== undefined ? { stderr: result.stderr } : {}),
-            };
-        }
-        return base;
-    }
-    const proposal = result.proposal ?? {};
-    const base = {
-        ok: true,
-        ref: result.ref,
-        ...(result.agentProfile !== undefined ? { agentProfile: result.agentProfile } : {}),
-        ...(typeof result.durationMs === "number" ? { durationMs: result.durationMs } : {}),
-        proposal: shapeProposalEntry(proposal, detail === "brief" ? "normal" : detail),
-    };
-    if (detail === "full") {
-        return { schemaVersion: result.schemaVersion ?? 1, ...base };
-    }
-    return base;
-}
-export function shapeProposalEntry(entry, detail) {
-    if (detail === "brief") {
-        return pickFields(entry, ["id", "ref", "status", "source", "createdAt"]);
-    }
-    if (detail === "normal" || detail === "summary") {
-        return pickFields(entry, ["id", "ref", "status", "source", "sourceRun", "createdAt", "updatedAt", "review"]);
-    }
-    // full / agent: project everything including the payload.
-    return pickFields(entry, [
-        "id",
-        "ref",
-        "status",
-        "source",
-        "sourceRun",
-        "createdAt",
-        "updatedAt",
-        "payload",
-        "review",
-    ]);
-}
-export function shapeProposalListOutput(result, detail) {
-    const proposals = Array.isArray(result.proposals) ? result.proposals : [];
-    const shaped = proposals.map((p) => shapeProposalEntry(p, detail));
-    const base = {
-        totalCount: result.totalCount ?? shaped.length,
-        proposals: shaped,
-    };
-    if (detail === "full") {
-        return { schemaVersion: result.schemaVersion ?? 1, ...base };
-    }
-    return base;
-}
-export function shapeProposalShowOutput(result, detail) {
-    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 base;
-}
-export function shapeProposalAcceptOutput(result, detail) {
-    const proposal = result.proposal ?? {};
-    const base = {
-        ok: result.ok ?? true,
-        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 base;
-}
-export function shapeProposalRejectOutput(result, detail) {
-    const proposal = result.proposal ?? {};
-    const base = {
-        ok: result.ok ?? true,
-        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 === "summary" ? "normal" : detail) } : {}),
-    };
-    if (detail === "full") {
-        return { schemaVersion: result.schemaVersion ?? 1, ...base };
-    }
-    return base;
-}
-export function shapeProposalDiffOutput(result, detail) {
-    const base = {
-        id: result.id,
-        ref: result.ref,
-        isNew: result.isNew,
-        unified: result.unified,
-        ...(result.targetPath !== undefined ? { targetPath: result.targetPath } : {}),
-    };
-    if (detail === "full") {
-        return { schemaVersion: result.schemaVersion ?? 1, ...base };
-    }
-    return base;
-}
-export function shapeEventsOutput(result, detail) {
-    const events = Array.isArray(result.events) ? result.events : [];
-    const shapedEvents = events.map((event) => shapeEventEntry(event, detail));
-    const base = {
-        ...(result.ref !== undefined ? { ref: result.ref } : {}),
-        ...(result.type !== undefined ? { type: result.type } : {}),
-        ...(result.since !== undefined ? { since: result.since } : {}),
-        ...(typeof result.sinceOffset === "number" ? { sinceOffset: result.sinceOffset } : {}),
-        totalCount: result.totalCount ?? shapedEvents.length,
-        events: shapedEvents,
-    };
-    if (typeof result.nextOffset === "number") {
-        base.nextOffset = result.nextOffset;
-    }
-    if (typeof result.reason === "string") {
-        base.reason = result.reason;
-    }
-    if (detail === "full") {
-        return { schemaVersion: result.schemaVersion ?? 1, ...base };
-    }
-    return base;
-}
-export function shapeEventEntry(entry, detail) {
-    if (detail === "brief") {
-        return pickFields(entry, ["eventType", "ref", "ts"]);
-    }
-    if (detail === "normal" || detail === "summary") {
-        return pickFields(entry, ["eventType", "ref", "ts"]);
-    }
-    // full / agent: 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.
-        return pickFields(entry, ["eventType", "ref", "signal", "createdAt"]);
-    }
-    if (detail === "normal" || detail === "summary") {
-        return pickFields(entry, ["eventType", "ref", "signal", "query", "createdAt"]);
+const SHAPE_SUMMARY_COMMANDS = new Set(["show"]);
+export function shapeForCommand(command, result, detail, shape = "human") {
+    if (shape === "summary" && !SHAPE_SUMMARY_COMMANDS.has(command)) {
+        throw new UsageError(`'--shape summary' is not supported for 'akm ${command}'. It is only available on 'akm show'.`, "INVALID_SHAPE_VALUE");
     }
-    // full / agent: return everything the reader emits.
-    return pickFields(entry, ["id", "eventType", "ref", "entryId", "query", "signal", "metadata", "createdAt"]);
-}
-export function shapeSearchOutput(result, detail, forAgent = false) {
-    const hits = Array.isArray(result.hits) ? result.hits : [];
-    const registryHits = Array.isArray(result.registryHits) ? result.registryHits : [];
-    const shapedHits = forAgent
-        ? hits.map((hit) => shapeSearchHitForAgent(hit))
-        : hits.map((hit) => shapeSearchHit(hit, detail));
-    const shapedRegistryHits = forAgent
-        ? registryHits.map((hit) => shapeSearchHitForAgent(hit))
-        : registryHits.map((hit) => shapeSearchHit(hit, detail));
-    if (forAgent) {
-        return {
-            hits: shapedHits,
-            ...(shapedRegistryHits.length > 0 ? { registryHits: shapedRegistryHits } : {}),
-            ...(result.tip ? { tip: result.tip } : {}),
-        };
-    }
-    if (detail === "full") {
-        return {
-            schemaVersion: result.schemaVersion,
-            stashDir: result.stashDir,
-            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 } : {}),
-        };
-    }
-    return {
-        hits: shapedHits,
-        ...(shapedRegistryHits.length > 0 ? { registryHits: shapedRegistryHits } : {}),
-        ...(Array.isArray(result.warnings) && result.warnings.length > 0 ? { warnings: result.warnings } : {}),
-        ...(result.tip ? { tip: result.tip } : {}),
-    };
-}
-export function shapeRegistrySearchOutput(result, detail) {
-    const hits = Array.isArray(result.hits) ? result.hits : [];
-    const assetHits = Array.isArray(result.assetHits) ? result.assetHits : [];
-    // Shape stash hits as registry type
-    const shapedKitHits = hits.map((hit) => shapeSearchHit({ ...hit, type: "registry" }, detail));
-    // Shape asset hits by detail level
-    const shapedAssetHits = assetHits.map((hit) => shapeAssetHit(hit, detail));
-    const shaped = {
-        hits: shapedKitHits,
-        ...(shapedAssetHits.length > 0 ? { assetHits: shapedAssetHits } : {}),
-        ...(Array.isArray(result.warnings) && result.warnings.length > 0 ? { warnings: result.warnings } : {}),
-    };
-    if (detail === "full") {
-        shaped.query = result.query;
-    }
-    return shaped;
-}
-export function shapeAssetHit(hit, detail) {
-    if (detail === "brief")
-        return pickFields(hit, ["assetName", "assetType", "action", "estimatedTokens"]);
-    if (detail === "normal") {
-        return capDescription(pickFields(hit, ["assetName", "assetType", "description", "stash", "action", "estimatedTokens"]), NORMAL_DESCRIPTION_LIMIT);
-    }
-    return hit;
-}
-export function shapeSearchHit(hit, detail) {
-    if (hit.type === "registry") {
-        if (detail === "brief") {
-            // RegistrySearchHit uses `title` (not `name`); always project installRef
-            // and score so callers can use the result without --detail full (QA #28).
-            const out = pickFields(hit, ["title", "name", "installRef", "score"]);
-            // Normalise: if only title exists, expose it as `name` for consistency
-            if (out.title && !out.name)
-                out.name = out.title;
-            return out;
-        }
-        if (detail === "normal") {
-            // `curated` was removed in v1 (spec §4.2). Renderers project optional
-            // hit-level `warnings` instead so providers can surface non-fatal issues.
-            const out = capDescription(pickFields(hit, ["title", "name", "description", "action", "installRef", "score", "warnings"]), NORMAL_DESCRIPTION_LIMIT);
-            if (out.title && !out.name)
-                out.name = out.title;
-            return out;
-        }
-        return hit;
-    }
-    // Stash hit (local or remote)
-    // `ref` is included at `brief` so agents can run `akm show <ref>` without
-    // needing --detail full or --for-agent (REC-03).
-    if (detail === "brief")
-        return pickFields(hit, ["type", "name", "ref", "action", "estimatedTokens", "keys"]);
-    if (detail === "normal") {
-        // `warnings` is projected at `normal` so non-fatal hit-level issues are
-        // visible without forcing callers up to `--detail full`. Optional
-        // `quality` (v1 spec §4.2) is also surfaced when present so callers
-        // can see why a `proposed` entry showed up under `--include-proposed`.
-        const shaped = capDescription(pickFields(hit, ["type", "name", "description", "action", "score", "estimatedTokens", "warnings", "quality"]), NORMAL_DESCRIPTION_LIMIT);
-        if (Array.isArray(hit.keys) && hit.keys.length > 0)
-            shaped.keys = hit.keys;
-        return shaped;
-    }
-    return hit;
-}
-/** 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"]);
-    return capDescription(picked, NORMAL_DESCRIPTION_LIMIT);
-}
-export function capDescription(hit, limit) {
-    if (typeof hit.description !== "string")
-        return hit;
-    return { ...hit, description: truncateDescription(hit.description, limit) };
-}
-export function truncateDescription(description, limit) {
-    const normalized = description.replace(/\s+/g, " ").trim();
-    if (normalized.length <= limit)
-        return normalized;
-    const truncated = normalized.slice(0, limit - 1);
-    const lastSpace = truncated.lastIndexOf(" ");
-    const safe = lastSpace >= Math.floor(limit * 0.6) ? truncated.slice(0, lastSpace) : truncated;
-    return `${safe.trimEnd()}...`;
-}
-export function shapeShowOutput(result, detail, forAgent = false) {
-    if (forAgent) {
-        return pickFields(result, [
-            "type",
-            "name",
-            "description",
-            "action",
-            "content",
-            "template",
-            "prompt",
-            "run",
-            "setup",
-            "cwd",
-            "activeRun",
-            "toolPolicy",
-            "modelHint",
-            "agent",
-            "parameters",
-            "workflowTitle",
-            "workflowParameters",
-            "steps",
-            "keys",
-            "comments",
-            "related",
-        ]);
-    }
-    if (detail === "summary") {
-        return pickFields(result, [
-            "type",
-            "name",
-            "description",
-            "tags",
-            "parameters",
-            "workflowTitle",
-            "action",
-            "run",
-            "origin",
-            "keys",
-            "comments",
-            "related",
-        ]);
-    }
-    const base = pickFields(result, [
-        "type",
-        "name",
-        "origin",
-        "action",
-        "description",
-        "tags",
-        "content",
-        "template",
-        "prompt",
-        "toolPolicy",
-        "modelHint",
-        "agent",
-        "parameters",
-        "workflowTitle",
-        "workflowParameters",
-        "steps",
-        "run",
-        "setup",
-        "cwd",
-        "activeRun",
-        "keys",
-        "comments",
-        "related",
-        // path and editable are always projected so JSON consumers can locate and
-        // edit the asset without needing --detail full (QA #7).
-        "path",
-        "editable",
-    ]);
-    if (detail !== "full") {
-        return base;
-    }
-    return {
-        schemaVersion: 1,
-        ...base,
-        ...pickFields(result, ["editHint"]),
-    };
-}
-export function pickFields(source, fields) {
-    const result = {};
-    for (const field of fields) {
-        if (source[field] !== undefined) {
-            result[field] = source[field];
-        }
+    const handler = getOutputShapeHandler(command);
+    if (handler) {
+        return handler(result, detail, shape);
     }
-    return result;
+    // v1 spec §9 (output-shape registry exhaustive): no silent JSON.stringify
+    // fallback. A missing case here is a registration bug — fail loudly so
+    // the caller (or its tests) sees the missing command name.
+    throw new Error(`output shape not registered for command: ${command}`);
 }
-export { NORMAL_DESCRIPTION_LIMIT };

package/dist/output/text/add.js

@@ -0,0 +1,6 @@
+// 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 { formatAddPlain } from "./helpers";
+import { registerTextFormatter } from "./registry";
+registerTextFormatter("add", (r) => formatAddPlain(r));

package/dist/output/text/clone.js

@@ -0,0 +1,6 @@
+// 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 { formatClonePlain } from "./helpers";
+import { registerTextFormatter } from "./registry";
+registerTextFormatter("clone", (r) => formatClonePlain(r));

package/dist/output/text/config.js

@@ -0,0 +1,6 @@
+// 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 { formatConfigPlain } from "./helpers";
+import { registerTextFormatter } from "./registry";
+registerTextFormatter("config", (r) => formatConfigPlain(r));

package/dist/output/text/curate.js

@@ -0,0 +1,6 @@
+// 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 { formatCuratePlain } from "./helpers";
+import { registerTextFormatter } from "./registry";
+registerTextFormatter("curate", (r, detail) => formatCuratePlain(r, detail));