akm-cli 0.9.17-alpha.8 → 0.9.17

package/dist/llm/client.js

@@ -101,10 +101,9 @@ function retryBackoffMs() {
  * narrating about a document) does not get misclassified as a provider
  * context-limit error (#496).
  *
- * Canonical home: `graph-extract.ts` re-exports this so the index-pass
- * graph extractor and the retry classifier (`isRetryable`) share one
- * definition — retrying a context overflow cannot shrink the input, so it
- * must never be retried.
+ * Canonical home: `structured-call.ts`'s failure classifier and the retry
+ * classifier (`isRetryable`) here share one definition — retrying a context
+ * overflow cannot shrink the input, so it must never be retried.
  */
 export function isContextSizeError(message) {
     const lower = message.toLowerCase();
@@ -120,13 +119,12 @@ export function isContextSizeError(message) {
 /**
  * Codes describing a failure to reach or get a usable response from the
  * provider transport itself, as opposed to a malformed-but-received response
- * (`parse_error`) or a request-shape rejection (`rate_limited`). Shared by
- * {@link isRetryable} (which additionally requires evidence the failure is
- * transient) and the batch graph-extraction storm guard in `graph-extract.ts`
- * (which treats any of these as "the provider is down, stop retrying
- * per-asset") so the two classifications cannot drift apart.
+ * (`parse_error`) or a request-shape rejection (`rate_limited`). Used by
+ * {@link isRetryable}, which additionally requires evidence the failure is
+ * transient. Not exported — its one external caller (batched LLM
+ * entity-graph extraction) was retired in 0.9.17-alpha.9.
  */
-export function isTransportFailure(err) {
+function isTransportFailure(err) {
     return err.code === "provider_error" || err.code === "network_error" || err.code === "provider_html_error";
 }
 /**

package/dist/llm/embedders/remote.js

@@ -187,8 +187,7 @@ export function isContextExceededResponse(status, body) {
  * Resolve the effective in-flight request window for `RemoteEmbedder.embedBatch`.
  * Default (unset `embedding.concurrency`): 1 for a loopback endpoint, 2 for a
  * remote one, via the shared `defaultConcurrencyForEndpoint`
- * (`src/core/loopback.ts`), the same lowest-common-denominator rule
- * `getDefaultLlmConcurrency` (`src/indexer/indexer.ts`) uses.
+ * (`src/core/loopback.ts`).
  *
  * `embedding.concurrency` (#954) overrides this default in
  * either direction, bounded 1-16 at the config schema — added after field

package/dist/llm/feature-gate.js

@@ -7,8 +7,6 @@
  */
 const FEATURE_LOCATION = {
     memory_inference: (cfg) => cfg.index?.memory?.enabled ?? true,
-    graph_extraction: (cfg) => cfg.index?.graph?.enabled ?? true,
-    metadata_enhance: (cfg) => cfg.index?.metadataEnhance?.enabled ?? false,
     // #951: a real implementation of the dead `curate_rerank` key removed in
     // 0.8.0. Off by default — it requires a `search.curateRerank.endpoint` a
     // caller must explicitly configure.
@@ -82,9 +80,6 @@ export function isProcessEnabled(section, processName, config) {
         if (processName === "memory_inference" || processName === "memoryInference") {
             return isLlmFeatureEnabled(config, "memory_inference");
         }
-        if (processName === "graph_extraction" || processName === "graphExtraction") {
-            return isLlmFeatureEnabled(config, "graph_extraction");
-        }
     }
     return false;
 }

package/dist/output/shapes/helpers.js

@@ -56,11 +56,25 @@ export function shapeProposalProducerOutput(result, detail) {
 }
 export function shapeProposalEntry(entry, detail) {
     if (detail === "brief") {
-        return pickFields(entry, ["id", "ref", "status", "source", "createdAt"]);
+        // `proposal list`'s default detail level IS "brief" (no `--detail` flag
+        // given projects here, not "normal") — `retirement` itself is not
+        // projected until "normal", so a flagged retire proposal's risk would
+        // otherwise be invisible by default. A minimal boolean marker, not the
+        // full risk object, keeps brief's shape small while still letting a
+        // reviewer see it without an extra flag.
+        const retirement = entry.retirement;
+        return {
+            ...pickFields(entry, ["id", "ref", "status", "source", "createdAt"]),
+            ...(retirement?.continuityRisk ? { continuityRisk: true } : {}),
+        };
     }
     if (detail === "normal") {
         // `confidence` and `gateDecision` (#577) explain why a proposal is pending,
         // so they are projected at `normal` for `akm proposal list/show` when present.
+        // `retirement` (alpha.9) is the same kind of field for a consolidate
+        // `retire` proposal: without it, "list/show" carries no reason a reviewer
+        // could act on (its `payload.content` is empty by design — a retire
+        // proposal deletes, it does not write).
         return pickFields(entry, [
             "id",
             "ref",
@@ -72,6 +86,7 @@ export function shapeProposalEntry(entry, detail) {
             "confidence",
             "gateDecision",
             "review",
+            "retirement",
         ]);
     }
     // full: project everything including the payload.
@@ -87,6 +102,10 @@ export function shapeProposalEntry(entry, detail) {
         "gateDecision",
         "payload",
         "review",
+        "retirement",
+        "retiredArchive",
+        "promotionSource",
+        "promotionSourceHash",
     ]);
 }
 export function shapeProposalListOutput(result, detail) {
@@ -357,7 +376,6 @@ export function shapeShowOutput(result, detail, shape = "human") {
             "workflowParameters",
             "steps",
             "keys",
-            "related",
             "links",
             ...FRAGMENT_PROVENANCE_FIELDS,
             ...FRAGMENT_CONTEXT_FIELDS,
@@ -382,7 +400,6 @@ export function shapeShowOutput(result, detail, shape = "human") {
             "run",
             "origin",
             "keys",
-            "related",
             "links",
             ...FRAGMENT_PROVENANCE_FIELDS,
             ...FRAGMENT_CONTEXT_FIELDS,
@@ -412,7 +429,6 @@ export function shapeShowOutput(result, detail, shape = "human") {
         "cwd",
         "activeRun",
         "keys",
-        "related",
         "links",
         ...FRAGMENT_PROVENANCE_FIELDS,
         ...FRAGMENT_CONTEXT_FIELDS,

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

@@ -20,11 +20,20 @@ export function formatInfoPlain(r) {
     const lines = [];
     if (r.version)
         lines.push(`version: ${String(r.version)}`);
+    // Only present in infoCommand's own last-resort fallback (stash-cli.ts),
+    // when assembleInfo() itself threw — every field below it belongs to the
+    // normal shape and is absent there.
+    if (typeof r.error === "string")
+        lines.push(`error: ${r.error}`);
     if (r.bundleDir)
         lines.push(`bundleDir: ${String(r.bundleDir)}`);
     if (r.defaultBundle !== undefined) {
         lines.push(`defaultBundle: ${r.defaultBundle === null ? "(none)" : String(r.defaultBundle)}`);
     }
+    if (typeof r.configError === "string")
+        lines.push(`configError: ${r.configError}`);
+    if (typeof r.bundleDirError === "string")
+        lines.push(`bundleDirError: ${r.bundleDirError}`);
     if (Array.isArray(r.assetTypes) && r.assetTypes.length > 0) {
         lines.push(`assetTypes: ${r.assetTypes.join(", ")}`);
     }
@@ -407,14 +416,6 @@ export function formatIndexPlain(r) {
         for (const message of warnings)
             out += `\n  - ${String(message)}`;
     }
-    const notices = Array.isArray(indexResult.notices) ? indexResult.notices : [];
-    for (const notice of notices) {
-        const severity = notice.severity === "info" ? "info" : "warning";
-        const field = typeof notice.field === "string" ? ` field=${notice.field}` : "";
-        out +=
-            `\n  notice[${severity}] ${notice.code} adapter=${notice.adapter}${field}` +
-                (notice.message ? `: ${notice.message}` : "");
-    }
     const verification = indexResult.verification;
     if (verification?.ok === false && verification.message) {
         out += `\nVerification: ${String(verification.message)}`;

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

@@ -100,7 +100,16 @@ export function formatProposalListPlain(r) {
         // #577: surface the gate verdict inline so the queue explains itself.
         const gate = formatGateDecisionSummary(p.gateDecision);
         const gateSuffix = gate ? `  ${gate}` : "";
-        lines.push(`${id}  [${status}] ${ref}  source=${source}  ${created}${gateSuffix}`);
+        // A retire proposal's continuityRisk excludes it from bulk accept —
+        // visible in the default list, not just `proposal show`, since the owner
+        // reviews the backlog list-first. `list`'s default (brief) shape
+        // projects a `continuityRisk: true` marker instead of the full
+        // `retirement` object (only present from `--detail normal` up), so both
+        // shapes are checked here.
+        const retirement = p.retirement;
+        const flagged = p.continuityRisk === true || Boolean(retirement?.continuityRisk);
+        const continuitySuffix = flagged ? "  ⚠ continuity-risk" : "";
+        lines.push(`${id}  [${status}] ${ref}  source=${source}  ${created}${gateSuffix}${continuitySuffix}`);
     }
     return lines.join("\n").trimEnd();
 }
@@ -141,6 +150,43 @@ export function formatProposalShowPlain(r) {
         if (review.decidedAt)
             lines.push(`review.decidedAt: ${String(review.decidedAt)}`);
     }
+    // alpha.9: a consolidate retire proposal writes no content (`payload.content`
+    // is empty by design) — this is the reason a reviewer needs instead. `diff`
+    // shows the body being retired.
+    const retirement = p.retirement;
+    if (retirement) {
+        lines.push(`retire: ${String(retirement.retiredRef)} -> ${String(retirement.successorRef)}`);
+        lines.push(`retire.label: ${String(retirement.judgeLabel)} (cosine=${String(retirement.cosine)})`);
+        lines.push(`retire.reason: ${String(retirement.judgeReason)}`);
+        // Item 1 (continuity check): flagged, but still minted — never swept by a
+        // bulk accept, only acceptable by id, so a reviewer must see it here.
+        const continuityRisk = retirement.continuityRisk;
+        if (continuityRisk) {
+            const failingQueries = typeof continuityRisk.failingQueries === "number" ? continuityRisk.failingQueries : 0;
+            const unverifiedQueries = typeof continuityRisk.unverifiedQueries === "number" ? continuityRisk.unverifiedQueries : 0;
+            const summary = [];
+            if (failingQueries > 0) {
+                summary.push(`${failingQueries} of the retired asset's own quer${failingQueries === 1 ? "y" : "ies"} would not have found the successor top 10`);
+            }
+            // S2: a query the search call never ran, or that fell back to
+            // keyword-only ranking, is never silently trusted OR silently
+            // dropped — it excludes the proposal from bulk accept on its own.
+            if (unverifiedQueries > 0) {
+                summary.push(`${unverifiedQueries} quer${unverifiedQueries === 1 ? "y" : "ies"} unverified (search failed or used the keyword-only fallback)`);
+            }
+            lines.push(`retire.continuityRisk: ${summary.join("; ")} — excluded from bulk accept`);
+            // N3 / S4: the actual failing query text, not just the count — a
+            // reviewer deciding whether to accept by id needs to see what would
+            // stop resolving, not just how many queries.
+            const ranks = Array.isArray(continuityRisk.ranks) ? continuityRisk.ranks : [];
+            for (const rank of ranks) {
+                const successorRank = rank.successorRank === null || rank.successorRank === undefined
+                    ? "absent from top 10"
+                    : `#${String(rank.successorRank)}`;
+                lines.push(`  - "${String(rank.query)}": retired #${String(rank.retiredRank)}, successor ${successorRank}`);
+            }
+        }
+    }
     const validation = r.validation;
     if (validation) {
         const findings = Array.isArray(validation.findings) ? validation.findings : [];

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

@@ -54,19 +54,6 @@ export function formatShowPlain(r, detail) {
         if (r.schemaVersion !== undefined)
             lines.push(`schemaVersion: ${String(r.schemaVersion)}`);
     }
-    const related = typeof r.related === "object" && r.related !== null ? r.related : undefined;
-    const relatedHits = related && Array.isArray(related.hits) ? related.hits : [];
-    if (related) {
-        lines.push("");
-        lines.push(`related: ${String(related.total ?? relatedHits.length)}`);
-        for (const hit of relatedHits) {
-            lines.push(`  - ${String(hit.type ?? "?")}: ${formatRelatedLabel(hit)}`);
-            const shared = Array.isArray(hit.sharedEntities) ? hit.sharedEntities.map(String) : [];
-            if (shared.length > 0)
-                lines.push(`    shared: ${shared.join(", ")}`);
-            lines.push(`    relationCount: ${String(hit.relationCount ?? 0)}`);
-        }
-    }
     const links = typeof r.links === "object" && r.links !== null ? r.links : undefined;
     if (links) {
         lines.push("");
@@ -114,10 +101,3 @@ export function formatShowPlain(r, detail) {
     appendShowDirectives(lines, r);
     return lines.length > 0 ? lines.join("\n") : null;
 }
-function formatRelatedLabel(hit) {
-    const ref = typeof hit.ref === "string" ? hit.ref : undefined;
-    if (ref)
-        return ref;
-    const pathValue = typeof hit.path === "string" ? hit.path : "?";
-    return pathValue.split("/").pop() ?? pathValue;
-}