akm-cli 0.10.261015-beta → 0.10.261017-beta

package/dist/indexer/usage/usage-events.js

@@ -69,20 +69,28 @@ export function insertUsageEventStrict(db, event) {
 // ── Query ────────────────────────────────────────────────────────────────────
 /**
  * User feedback counts for many assets at once, keyed by durable `entry_ref`.
- * Refs with no user feedback are absent from the map. Read in one query per
- * chunk of refs.
+ * A negative that carried an exact fix does not count once an accepted
+ * `feedback` proposal for the asset exists at or after it (the rule `akm
+ * improve` uses for a fix already acted on); reverting that proposal makes it
+ * count again. Refs with no user feedback are absent from the map. Read in one
+ * query per chunk of refs.
  */
 export function countFeedbackSignalsByRef(db, refs) {
     const counts = new Map();
     for (let i = 0; i < refs.length; i += SQLITE_CHUNK_SIZE) {
         const chunk = refs.slice(i, i + SQLITE_CHUNK_SIZE);
         const rows = db
-            .prepare(`SELECT entry_ref,
-           SUM(CASE WHEN signal = 'positive' THEN 1 ELSE 0 END) AS pos,
-           SUM(CASE WHEN signal = 'negative' THEN 1 ELSE 0 END) AS neg
-         FROM usage_events
-         WHERE event_type = 'feedback' AND source = 'user' AND entry_ref IN (${chunk.map(() => "?").join(",")})
-         GROUP BY entry_ref`)
+            .prepare(`SELECT u.entry_ref,
+           SUM(CASE WHEN u.signal = 'positive' THEN 1 ELSE 0 END) AS pos,
+           SUM(CASE WHEN u.signal = 'negative' AND NOT (
+             json_extract(u.metadata, '$.fix') IS NOT NULL AND EXISTS (
+               SELECT 1 FROM proposals p
+               WHERE p.ref = u.entry_ref AND p.source = 'feedback' AND p.status = 'accepted'
+                 AND julianday(p.created_at) >= julianday(u.created_at))
+           ) THEN 1 ELSE 0 END) AS neg
+         FROM usage_events u
+         WHERE u.event_type = 'feedback' AND u.source = 'user' AND u.entry_ref IN (${chunk.map(() => "?").join(",")})
+         GROUP BY u.entry_ref`)
             .all(...chunk);
         for (const row of rows)
             counts.set(row.entry_ref, { pos: row.pos, neg: row.neg });
@@ -150,3 +158,17 @@ export function purgeOldUsageEvents(db, retentionDays) {
         /* Table may not exist yet */
     }
 }
+/**
+ * Rewrite every `usage_events.entry_ref` naming `oldBundleId` to `newBundleId`
+ * (`akm bundle rename`). Search ranking counts feedback by `entry_ref`, so an
+ * event left under the old prefix stops counting for the renamed bundle's
+ * assets. `entry_id` is untouched: the rename keeps every entry's row id. A
+ * legacy unqualified ref is not matched by the `<old>//` prefix and is left alone.
+ * Returns the number of rows rewritten.
+ */
+export function renameUsageEventRefs(db, oldBundleId, newBundleId) {
+    const prefix = `${oldBundleId}//`;
+    return Number(db
+        .prepare(`UPDATE usage_events SET entry_ref = ? || substr(entry_ref, length(?) + 1) WHERE substr(entry_ref, 1, length(?)) = ?`)
+        .run(newBundleId, oldBundleId, prefix, prefix).changes);
+}

package/dist/indexer/walk/walker.js

@@ -112,7 +112,7 @@ function walkStashGit(stashRoot, options) {
         ? ignoredResult.stdout.toString("utf8")
         : String(ignoredResult.stdout ?? "");
     // Data-hygiene filename skips: the legacy metadata sidecar (never indexed as
-    // content — Chunk-8 folds it into the bundle format) plus git dot-files.
+    // content) plus git dot-files.
     const SKIP_FILES = new Set([".stash.json", ".gitignore", ".gitattributes"]);
     const stdout = Buffer.isBuffer(result.stdout) ? result.stdout.toString("utf8") : String(result.stdout ?? "");
     const files = [...stdout.split("\0"), ...ignoredStdout.split("\0")]

package/dist/output/renderers.js

@@ -217,8 +217,8 @@ const agentMdRenderer = {
     },
 };
 // ── 4. knowledge-md ──────────────────────────────────────────────────────────
-const KNOWLEDGE_ACTION = "Reference material - read the content below. For a large document, read one section with `akm show <ref>#fragment` " +
-    "(an unmatched #fragment lists the available slugs).";
+const KNOWLEDGE_ACTION = "Reference material - read the content below. For a large document, run `akm show <ref> --detail brief` for its " +
+    "outline (`toc`, no body), then `akm show <ref>#<slug>` for only the sections you need.";
 const knowledgeMdRenderer = {
     name: "knowledge-md",
     buildShowResponse(ctx) {

package/dist/output/shapes/helpers.js

@@ -71,7 +71,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` when present.
-        // `retirement` (alpha.9) is the same kind of field for a consolidate
+        // `retirement` 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).
@@ -278,7 +278,7 @@ export function shapeSearchHit(hit, detail) {
     }
     // Stash hit (local or remote)
     // `ref` is included at `brief` so agents can run `akm show <ref>` without
-    // needing --detail full or --detail agent (REC-03).
+    // needing --detail full or --detail agent.
     if (detail === "brief") {
         return pickFields(hit, ["type", "name", "ref", "action", "estimatedTokens", "keys"]);
     }
@@ -348,6 +348,7 @@ export function shapeShowOutput(result, detail, shape = "human") {
             "editHint",
             "description",
             "action",
+            "toc",
             "content",
             "template",
             "prompt",
@@ -375,12 +376,13 @@ 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 is always projected, same as path/editable below.
         "ref",
         "origin",
         "action",
         "description",
         "tags",
+        "toc",
         "content",
         "template",
         "prompt",

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

@@ -7,8 +7,7 @@
  * events/history, search/curate, and the stash-lifecycle verbs (init,
  * index, list, add, remove, update, upgrade, clone).
  *
- * Split out of `helpers.ts` (formerly 1418 lines / 59 fns) as its own
- * sibling module: unlike `show`/`workflow`/`proposal` (each a stateful,
+ * Unlike `show`/`workflow`/`proposal` (each a stateful,
  * multi-branch renderer with its own module), these are flat
  * field-extraction formatters with no shared state.
  */
@@ -308,7 +307,7 @@ export function formatSearchPlain(r, detail) {
         if (parts.length > 0)
             lines.push(`timing: ${parts.join(", ")}`);
     }
-    // REC-02: When stash hits exist, tell the agent the next required step so it
+    // When stash hits exist, tell the agent the next required step so it
     // doesn't skip `akm show` and write from training memory instead.
     if (hits.length >= 1) {
         // Prefer skill/command/agent type hits for the "Next:" ref — knowledge docs are
@@ -517,7 +516,7 @@ export function formatAddPlain(r) {
     // writes a desired locator to config — nothing is synced until a later
     // `akm bundle update`. Force-fitting the declarative shape's `added`/`entry`/`message`
     // fields into the "Installed <ref> (N scanned...)" wording produced
-    // "Installed undefined (0 directories scanned...)" (R-014) because it
+    // "Installed undefined (0 directories scanned...)" because it
     // implied a sync that never happened. `AddResponse` always carries an
     // `index` block (even when zero entries were scanned); `SourceAddResult`
     // never does — that is the reliable shape discriminator used below to
@@ -630,11 +629,11 @@ export function formatBundleRenamePlain(r) {
     return lines.join("\n");
 }
 export function formatUpdatePlain(r) {
-    // R-015: `processed` alone conflated three cases into one empty array —
+    // `processed` alone would conflate three cases into one empty array —
     // a true no-op (--all with nothing configured), a source this call flatly
     // never looked at (a plain source under --all), and a plain git/website
     // source that WAS just synced successfully but has no `UpdateResultItem`
-    // shape to report (no version/lock to diff). All three rendered as the
+    // shape to report (no version/lock to diff). All three would render as the
     // same misleading "nothing to update". `plainSynced` and `skipped` (see
     // sources/types.ts) cover the other two so this can report accurately.
     const processed = r.processed;

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

@@ -59,6 +59,13 @@ export function formatShowPlain(r, detail) {
         if (r.schemaVersion !== undefined)
             lines.push(`schemaVersion: ${String(r.schemaVersion)}`);
     }
+    if (Array.isArray(r.toc) && r.toc.length > 0) {
+        lines.push("");
+        lines.push("toc:");
+        for (const heading of r.toc) {
+            lines.push(`  ${"#".repeat(heading.level)} ${heading.text} (#${heading.slug}, line ${heading.startLine})`);
+        }
+    }
     const links = typeof r.links === "object" && r.links !== null ? r.links : undefined;
     if (links) {
         lines.push("");

package/dist/registry/semver.js

@@ -41,6 +41,18 @@ export function isSemverRange(input) {
 export function maxSatisfying(versions, range) {
     return semver.maxSatisfying(versions, range) ?? undefined;
 }
+/**
+ * Highest of `versions` in semver order, prereleases included (`0.10.1-beta` sorts above `0.9.30`). Invalid entries
+ * are ignored; `undefined` when none is valid. Use this to order known versions, not `maxSatisfying(versions, "*")`,
+ * which skips every prerelease.
+ */
+export function highestVersion(versions) {
+    return semver.rsort(versions.filter((version) => semver.valid(version) !== null))[0];
+}
+/** True when `version` is a valid semver prerelease (`1.2.3-beta`). */
+export function isPrerelease(version) {
+    return semver.valid(version) !== null && semver.prerelease(version) !== null;
+}
 /** True when `version` satisfies `range` (both real semver forms). False for an invalid version or range. */
 export function satisfiesRange(version, range) {
     return semver.satisfies(version, range);