akm-cli 0.9.17-alpha.2 → 0.9.17-alpha.4

package/dist/output/render-registry.js

@@ -1,41 +1,12 @@
 // 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/.
-/**
- * Per-command `md` / `html` renderer registries (D7).
- *
- * Mirrors `src/output/text/registry.ts`: a command may register a bespoke
- * renderer for a document format, and anything unregistered falls back to the
- * generic rendering of its shaped envelope (`./generic-render`). The fallback
- * is what makes all six `--format` values universal; the registry is what lets
- * a command that has something better to say — `akm health` and its report
- * templates — say it without the output pipeline knowing that command exists.
- *
- * Kept separate from the module that dispatches through it so per-command
- * renderer modules can import `registerMdRenderer` / `registerHtmlRenderer`
- * without a cycle back into the pipeline, exactly as the text registry does.
- *
- * Returning `null` from a handler means "I have nothing special for this
- * payload" and falls through to the generic renderer — `akm health` uses that
- * to keep its bespoke tables for the shapes that have them while still
- * rendering everything else.
- */
-import { createCommandRegistry } from "./command-registry.js";
-const MD_RENDERER_REGISTRY = createCommandRegistry();
-const HTML_RENDERER_REGISTRY = createCommandRegistry();
+const mdRenderers = new Map();
 /** Register a Markdown renderer for a command name. */
 export function registerMdRenderer(command, handler) {
-    MD_RENDERER_REGISTRY.register(command, handler);
+    mdRenderers.set(command, handler);
 }
 /** Look up a registered Markdown renderer, or `undefined` when unregistered. */
 export function getMdRendererHandler(command) {
-    return MD_RENDERER_REGISTRY.get(command);
-}
-/** Register an HTML renderer for a command name. */
-export function registerHtmlRenderer(command, handler) {
-    HTML_RENDERER_REGISTRY.register(command, handler);
-}
-/** Look up a registered HTML renderer, or `undefined` when unregistered. */
-export function getHtmlRendererHandler(command) {
-    return HTML_RENDERER_REGISTRY.get(command);
+    return mdRenderers.get(command);
 }

package/dist/output/shapes/helpers.js

@@ -284,40 +284,14 @@ export function shapeSearchHit(hit, detail) {
     // `ref` is included at `brief` so agents can run `akm show <ref>` without
     // needing --detail full or --shape agent (REC-03).
     if (detail === "brief") {
-        return pickFields(hit, [
-            "type",
-            "name",
-            "ref",
-            "action",
-            "estimatedTokens",
-            "keys",
-            "selectedRef",
-            "parentRef",
-            "fragmentOrdinal",
-            "fragmentCount",
-            "parentEstimatedTokens",
-        ]);
+        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`.
-        // `matchStage` (issue #856) surfaces which stage of the progressive
-        // AND->OR lexical ladder produced the hit; cheap compact enum, worth
-        // showing without requiring `--detail full`.
-        const shaped = capDescription(pickFields(hit, [
-            "type",
-            "name",
-            "description",
-            "action",
-            "score",
-            "estimatedTokens",
-            "warnings",
-            "quality",
-            "matchStage",
-            ...FRAGMENT_PROVENANCE_FIELDS,
-        ]), NORMAL_DESCRIPTION_LIMIT);
+        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;
@@ -338,12 +312,6 @@ export function shapeSearchHitForAgent(hit) {
         "score",
         "estimatedTokens",
         "keys",
-        // Issue #856: which stage of the progressive AND->OR lexical ladder
-        // produced this hit. Agents need this to gauge how much to trust a
-        // hit (a strict-AND match is stronger signal than an OR-fallback
-        // recovery match) without going to `--detail full`.
-        "matchStage",
-        ...FRAGMENT_PROVENANCE_FIELDS,
     ]);
     if (picked.editable !== false)
         delete picked.editHint;

package/dist/output/shapes/passthrough.js

@@ -8,15 +8,6 @@
 // envelope. Never overwrites an existing `shape`, `schemaVersion`, or `ok`;
 // a command whose exit code grades the outcome emits through
 // `outputWithExitCode` (src/cli/shared.ts), which sets `ok` from that code.
-//
-// Builds a shallow copy rather than mutating `result` in place: several
-// command results (e.g. `akm task sync --dry-run`'s `SchedulerPlanPreview`,
-// see src/tasks/scheduler-sync-preview.ts) are deliberately `Object.freeze`d
-// by their producer as an immutability guarantee, and an in-place `obj.shape
-// = …` assignment throws ("Attempting to define property on object that is
-// not extensible") the moment it hits one. Copying tolerates both frozen and
-// mutable inputs uniformly, and `output()` never uses the result's identity
-// past this call, so a copy is safe here.
 function makeStampHandler(command) {
     return (result) => {
         if (result === null || result === undefined)
@@ -39,6 +30,7 @@ const PASSTHROUGH_COMMANDS = [
     "add",
     "agent-result",
     "bundle-create",
+    "bundle-rename",
     "bundle-show",
     "clone",
     "command-dry-run",

package/dist/{indexer/search/ranking-types.js → output/text/bundle-rename.js}

@@ -1,4 +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/.
-export {};
+import { formatBundleRenamePlain } from "./helpers.js";
+export const bundleRenameFormatters = [
+    { command: "bundle-rename", handler: (r) => formatBundleRenamePlain(r) },
+];

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

@@ -270,32 +270,10 @@ export function formatSearchPlain(r, detail) {
         // Optional v1 spec §4.2 quality marker (e.g. "curated" / "proposed").
         if (typeof hit.quality === "string" && hit.quality)
             lines.push(`  quality: ${hit.quality}`);
-        // Issue #856: which stage of the progressive AND->OR lexical ladder
-        // produced this hit ("exact" | "prefix" | "relaxed").
-        if (typeof hit.matchStage === "string" && hit.matchStage)
-            lines.push(`  matchStage: ${hit.matchStage}`);
         // Surface optional hit-level warnings (v1 spec §4.2).
         if (Array.isArray(hit.warnings) && hit.warnings.length > 0) {
             lines.push(`  warnings: ${hit.warnings.join("; ")}`);
         }
-        const graph = typeof hit.graph === "object" && hit.graph !== null ? hit.graph : undefined;
-        if (graph) {
-            const entities = Array.isArray(graph.entities) ? graph.entities : [];
-            if (entities.length > 0) {
-                const matched = entities
-                    .filter((entity) => String(entity.kind ?? "") === "matched")
-                    .map((entity) => String(entity.name ?? "?"));
-                const neighbors = entities
-                    .filter((entity) => String(entity.kind ?? "") !== "matched")
-                    .map((entity) => String(entity.name ?? "?"));
-                lines.push(`  graph: ${[
-                    matched.length > 0 ? `query match=${matched.join(", ")}` : undefined,
-                    neighbors.length > 0 ? `neighbors=${neighbors.join(", ")}` : undefined,
-                ]
-                    .filter(Boolean)
-                    .join("; ")}`);
-            }
-        }
         if (detail === "full") {
             if (hit.path)
                 lines.push(`  path: ${String(hit.path)}`);
@@ -556,7 +534,8 @@ export function formatAddPlain(r) {
     const index = r.index;
     const scanned = index?.directoriesScanned ?? 0;
     const total = index?.totalEntries ?? 0;
-    const lines = [`Installed ${r.ref} (${scanned} directories scanned, ${total} total assets indexed)`];
+    const bundleSuffix = typeof r.bundleId === "string" && r.bundleId.length > 0 ? ` as bundle "${r.bundleId}"` : "";
+    const lines = [`Installed ${r.ref}${bundleSuffix} (${scanned} directories scanned, ${total} total assets indexed)`];
     const warnings = index?.warnings;
     if (Array.isArray(warnings) && warnings.length > 0) {
         lines.push(`Warnings (${warnings.length}):`);
@@ -589,6 +568,64 @@ export function formatRemovePlain(r) {
     const ok = r.ok !== false ? "OK" : "FAILED";
     return `remove: ${target} ${ok}`;
 }
+export function formatBundleRenamePlain(r) {
+    const oldId = String(r.oldId ?? "?");
+    const newId = String(r.newId ?? "?");
+    const applied = r.applied === true;
+    const config = r.config ?? {};
+    const state = r.state ?? {};
+    const index = r.index ?? {};
+    const schedulerRefs = Array.isArray(config.schedulerRefs) ? config.schedulerRefs.length : 0;
+    const proposalRefs = Number(state.proposalRefs ?? 0);
+    const proposalTargets = Number(state.proposalTargets ?? 0);
+    const taskHistoryRefs = Number(state.taskHistoryRefs ?? 0);
+    const entries = Number(index.entries ?? 0);
+    const contentRefs = Array.isArray(r.contentRefs) ? r.contentRefs : [];
+    const nativeSchedulerRows = Array.isArray(r.nativeSchedulerRows) ? r.nativeSchedulerRows : [];
+    const verb = applied ? "Renamed" : "Would rename";
+    const lines = [
+        `${verb} bundle "${oldId}" to "${newId}" (${entries} index entries, ${schedulerRefs} scheduler ref(s), ` +
+            `${proposalRefs + proposalTargets} proposal reference(s), ${taskHistoryRefs} task-history row(s)).`,
+    ];
+    if (!applied) {
+        lines.push("Re-run without --dry-run to apply. It also re-syncs native scheduler bindings under the new name.");
+        if (nativeSchedulerRows.length > 0) {
+            lines.push(`Native scheduler rows naming "${oldId}//" that the sync would replace:`);
+            for (const row of nativeSchedulerRows)
+                lines.push(`  - ${String(row)}`);
+        }
+    }
+    else {
+        const taskSync = r.taskSync;
+        const syncResult = taskSync?.result;
+        if (syncResult) {
+            const installed = Array.isArray(syncResult.installed) ? syncResult.installed.length : 0;
+            const updated = Array.isArray(syncResult.updated) ? syncResult.updated.length : 0;
+            const removed = Array.isArray(syncResult.removed) ? syncResult.removed.length : 0;
+            const failures = Array.isArray(syncResult.failures)
+                ? syncResult.failures
+                : [];
+            if (failures.length === 0) {
+                lines.push(`Native scheduler bindings re-synced under the new name (${installed} installed, ${updated} updated, ${removed} removed).`);
+            }
+            else {
+                lines.push(`Native scheduler sync after rename: ${installed} installed, ${updated} updated, ${removed} removed, ${failures.length} failed.`);
+                for (const failure of failures)
+                    lines.push(`  - ${failure.ref ?? failure.path}: ${failure.reason}`);
+                lines.push("Run `akm task sync` to retry.");
+            }
+        }
+        else if (taskSync?.ok === false && taskSync.error) {
+            lines.push(`Re-syncing native scheduler bindings failed: ${taskSync.error}. Run \`akm task sync\` to retry.`);
+        }
+    }
+    if (contentRefs.length > 0) {
+        lines.push(`Content still spelling "${oldId}//" (not rewritten — edit these by hand):`);
+        for (const file of contentRefs)
+            lines.push(`  - ${String(file)}`);
+    }
+    return lines.join("\n");
+}
 export function formatUpdatePlain(r) {
     // R-015: `processed` alone conflated three cases into one empty array —
     // a true no-op (--all with nothing configured), a source this call flatly

package/dist/output/text/helpers.js

@@ -15,7 +15,7 @@
  *
  * No registry imports — no circular dependencies.
  */
-export { formatAddPlain, formatBundleShowPlain, formatClonePlain, formatConfigPlain, formatCuratePlain, formatEnvCreatePlain, formatEnvExportPlain, formatEnvListPlain, formatEnvRemovePlain, formatEventLine, formatEventsPlain, formatFeedbackPlain, formatImportPlain, formatIndexPlain, formatInfoPlain, formatInitPlain, formatListPlain, formatModelsListPlain, formatRegistryAddPlain, formatRegistryListPlain, formatRegistryRemovePlain, formatRegistrySearchPlain, formatRememberPlain, formatRemovePlain, formatSearchPlain, formatSyncPlain, formatUpdatePlain, formatUpgradePlain, } from "./command-format.js";
+export { formatAddPlain, formatBundleRenamePlain, formatBundleShowPlain, formatClonePlain, formatConfigPlain, formatCuratePlain, formatEnvCreatePlain, formatEnvExportPlain, formatEnvListPlain, formatEnvRemovePlain, formatEventLine, formatEventsPlain, formatFeedbackPlain, formatImportPlain, formatIndexPlain, formatInfoPlain, formatInitPlain, formatListPlain, formatModelsListPlain, formatRegistryAddPlain, formatRegistryListPlain, formatRegistryRemovePlain, formatRegistrySearchPlain, formatRememberPlain, formatRemovePlain, formatSearchPlain, formatSyncPlain, formatUpdatePlain, formatUpgradePlain, } from "./command-format.js";
 export { formatHealthPlain } from "./health-format.js";
 export { formatLintPlain } from "./lint-format.js";
 export { formatGateDecisionSummary, formatProposalAcceptPlain, formatProposalDiffPlain, formatProposalDrainPlain, formatProposalListPlain, formatProposalProducerPlain, formatProposalRejectPlain, formatProposalShowPlain, } from "./proposal-format.js";

package/dist/output/text/migrate.js

@@ -18,27 +18,18 @@ export function formatMigratePlain(result) {
     if (typeof plan.status !== "string")
         return null;
     const lines = [`${planGlyph(plan.status)} ${plan.status}`];
-    if (plan.taskV3Migration) {
-        const tasks = plan.taskV3Migration;
-        lines.push(`    task-v2->v3: ${tasks.changed} change, ${tasks.skipped} current, ${tasks.blocked} blocked`);
-    }
-    if (plan.taskV4Migration) {
-        const tasks = plan.taskV4Migration;
-        lines.push(`    task-v3->v4: ${tasks.changed} change, ${tasks.skipped} current, ${tasks.blocked} blocked`);
+    if (plan.taskFiles) {
+        const tasks = plan.taskFiles;
+        lines.push(`    task files: ${tasks.changed} change, ${tasks.skipped} current, ${tasks.blocked} blocked`);
     }
     if (plan.blockers?.length) {
         lines.push("", "blockers:", ...plan.blockers.map((blocker) => `  - ${blocker}`));
     }
     if (plan.backupPath) {
-        lines.push("", `backup (v2->v3): ${plan.backupPath}`);
+        lines.push("", `backup: ${plan.backupPath}`);
     }
     if (plan.applied !== undefined)
-        lines.push(`applied (v2->v3): ${plan.applied}`);
-    if (plan.taskV4BackupPath) {
-        lines.push(`backup (v3->v4): ${plan.taskV4BackupPath}`);
-    }
-    if (plan.taskV4Applied !== undefined)
-        lines.push(`applied (v3->v4): ${plan.taskV4Applied}`);
+        lines.push(`applied: ${plan.applied}`);
     return lines.join("\n");
 }
 export const migrateFormatters = [

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

@@ -184,7 +184,6 @@ export function formatProposalRejectPlain(r) {
     return `Rejected proposal ${String(r.id)} (${String(r.ref)})${reason}`;
 }
 export function formatProposalDrainPlain(r) {
-    const policy = String(r.policy ?? "?");
     const applyMode = String(r.applyMode ?? "queue");
     const promoted = Array.isArray(r.promoted) ? r.promoted : [];
     const rejected = Array.isArray(r.rejected) ? r.rejected : [];
@@ -194,7 +193,7 @@ export function formatProposalDrainPlain(r) {
     const failed = Array.isArray(r.failed) ? r.failed : [];
     const prefix = r.dryRun === true ? "[dry-run] " : "";
     const lines = [
-        `${prefix}Drained proposal queue (strategy=${String(r.strategy ?? "?")}, policy=${policy}, applyMode=${applyMode})`,
+        `${prefix}Drained proposal queue (strategy=${String(r.strategy ?? "?")}, applyMode=${applyMode})`,
         `  promoted: ${promoted.length}`,
         `  rejected: ${rejected.length}`,
         `  deferred: ${deferred.length}`,

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

@@ -108,13 +108,6 @@ export function formatWorkflowStatusPlain(result) {
             const attempts = typeof unit.attempts === "number" ? unit.attempts : undefined;
             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 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`);
-            }
             if (typeof unit.failureReason === "string" && unit.failureReason.trim()) {
                 lines.push(`    failure_reason: ${unit.failureReason}`);
             }
@@ -126,27 +119,8 @@ export function formatWorkflowStatusPlain(result) {
             }
         }
     }
-    // Review C2: the check-in `continue` directive must survive plain-text
-    // rendering — JSON consumers saw `checkin` but the text path dropped it.
-    const checkinLine = formatWorkflowCheckinLine(result);
-    if (checkinLine) {
-        lines.push("");
-        lines.push(checkinLine);
-    }
     return lines.join("\n");
 }
-/**
- * Render the stalled-run check-in directive (#506) when present on a
- * workflow-status result. Returns null when the run is healthy.
- */
-function formatWorkflowCheckinLine(result) {
-    const checkin = typeof result.checkin === "object" && result.checkin !== null
-        ? result.checkin
-        : undefined;
-    if (!checkin || typeof checkin.directive !== "string" || !checkin.directive.trim())
-        return null;
-    return checkin.directive.trim();
-}
 /**
  * One `! lowering[<severity>] <code> (<adapter>[; <field>]): <message>` line
  * from a `{code, severity, adapter, field?, message}` lowering notice, or
@@ -346,12 +320,6 @@ export function formatWorkflowPlanPlain(result) {
     for (const [index, step] of steps.entries()) {
         renderPlanStepLines(step, index + 1, lines);
     }
-    const sourceReadSet = Array.isArray(result.sourceReadSet) ? result.sourceReadSet : [];
-    if (sourceReadSet.length > 0) {
-        lines.push("read set:");
-        for (const entry of sourceReadSet)
-            lines.push(`  ${String(entry)}`);
-    }
     const notices = Array.isArray(result.notices) ? result.notices : [];
     if (notices.length > 0) {
         lines.push("notices:");

package/dist/output/text.js

@@ -4,6 +4,7 @@
 import { writeStdout } from "./stdout.js";
 import { addFormatters } from "./text/add.js";
 import { bundleCreateFormatters } from "./text/bundle-create.js";
+import { bundleRenameFormatters } from "./text/bundle-rename.js";
 import { bundleShowFormatters } from "./text/bundle-show.js";
 import { cloneFormatters } from "./text/clone.js";
 import { configFormatters } from "./text/config.js";
@@ -40,6 +41,7 @@ import { workflowFormatters } from "./text/workflow.js";
 // compile instead of silently disappearing at runtime.
 const BUILT_IN_TEXT_FORMATTERS = [
     ...bundleCreateFormatters,
+    ...bundleRenameFormatters,
     ...bundleShowFormatters,
     ...indexFormatters,
     ...showFormatters,

package/dist/registry/factory.js

@@ -1,30 +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/.
-/**
- * Registry provider factory map.
- *
- * Maps registry provider type identifiers (e.g. "static-index", "skills-sh")
- * to factory functions that create RegistryProvider instances.
- *
- * "Registry" here refers to the stash discovery registries (static index files,
- * skills.sh API) — not to be confused with the source provider factory map in
- * `sources/provider-factory.ts` or the installed-source operations in
- * `installed-stashes.ts`.
- *
- * Factories use the `RegistryProviderFactory` type owned by
- * `src/registry/providers/types.ts`.
- */
-import { createProviderRegistry } from "./create-provider-registry.js";
 // ── Factory map ─────────────────────────────────────────────────────────────
-const registry = createProviderRegistry();
+const factories = new Map();
 export function registerRegistryProvider(type, factory) {
-    registry.register(type, factory);
+    factories.set(type, factory);
 }
 export function resolveRegistryProviderFactory(type) {
-    return registry.resolve(type);
+    return factories.get(type) ?? null;
 }
 /** Test-only seam: removes a registration made with {@link registerRegistryProvider}. */
 export function unregisterRegistryProvider(type) {
-    registry.unregister(type);
+    factories.delete(type);
 }