akm-cli 0.8.0-rc.6 → 0.8.0-rc.8

package/CHANGELOG.md

@@ -6,6 +6,20 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
 ## [Unreleased]
 
+## [0.8.0] - 2026-05-28
+
+### Fixed
+
+- **Consolidation `delete_failed` on stale index entries** — when consolidation
+  successfully deleted a memory file, the index DB was not re-indexed between
+  runs. Subsequent runs loaded the stale DB entry into their memory map, the LLM
+  re-proposed the deletion, and `deleteAssetFromSource` threw "not found in
+  source" — appearing as `delete_failed` in skipReasons. Fix: `loadMemoriesForSource`
+  now filters entries whose file no longer exists on disk before building chunks,
+  so phantom memories are never sent to the LLM. A secondary catch in the delete
+  handler emits `delete_already_gone` instead of `delete_failed` when the file
+  is confirmed absent.
+
 > **CI / Docker users:** the 0.8.0 storage split moved `akm.lock`, the event
 > database, and the registry cache out of `$XDG_CONFIG_HOME/akm/` into
 > `$XDG_DATA_HOME`, `$XDG_STATE_HOME`, and `$XDG_CACHE_HOME` respectively. If
@@ -65,6 +79,8 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
 ### Changed
 
+- **Rebrand**: the full name "Agent Kit Manager" is now **Agent Knowledge Management** — `akm` stands for Agent Knowledge Management going forward. The binary name, npm package (`akm-cli`), and all APIs remain unchanged.
+
 - **Config layer rewrite** — single-source-of-truth Zod schema in
   `src/core/config-schema.ts` replaces the per-field parse switch AND
   the per-shape load-time parser. Adding a new config field is now one

package/README.md

@@ -1,6 +1,6 @@
-# akm -- Agent Kit Manager
+# akm -- Agent Knowledge Management
 
-> **akm** (Agent Kit Manager) -- A package manager for AI agent skills, commands, tools, and knowledge.
+> **akm** (Agent Knowledge Management) -- A package manager for AI agent skills, commands, tools, and knowledge.
 
 [![npm version](https://img.shields.io/npm/v/akm-cli)](https://www.npmjs.com/package/akm-cli)
 [![npm downloads](https://img.shields.io/npm/dm/akm-cli)](https://www.npmjs.com/package/akm-cli)

package/dist/cli/shared.js

@@ -0,0 +1,114 @@
+// 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/.
+/**
+ * Shared CLI utilities extracted from `src/cli.ts` so that individual
+ * command modules can import them without a circular dependency.
+ *
+ * Exported: output, runWithJsonErrors, parseAllFlagValues, emitJsonError
+ */
+import { stringify as yamlStringify } from "yaml";
+import { ConfigError, NotFoundError, UsageError } from "../core/errors";
+import { getOutputMode } from "../output/context";
+import { shapeForCommand } from "../output/shapes";
+import { formatPlain, outputJsonl } from "../output/text";
+// ── Exit codes ───────────────────────────────────────────────────────────────
+const EXIT_GENERAL = 1;
+const EXIT_USAGE = 2;
+const EXIT_CONFIG = 78;
+// ── Helpers ──────────────────────────────────────────────────────────────────
+function classifyExitCode(error) {
+    if (error instanceof UsageError)
+        return EXIT_USAGE;
+    if (error instanceof ConfigError)
+        return EXIT_CONFIG;
+    if (error instanceof NotFoundError)
+        return EXIT_GENERAL;
+    return EXIT_GENERAL;
+}
+function extractHint(error) {
+    if (error instanceof Error && "hint" in error && typeof error.hint === "function") {
+        return error.hint();
+    }
+    return undefined;
+}
+/**
+ * Serialize an error to the standard JSON envelope and exit.
+ * Used in both the startup try/catch and `runWithJsonErrors`.
+ */
+export function emitJsonError(error) {
+    const message = error instanceof Error ? error.message : String(error);
+    const hint = extractHint(error);
+    const exitCode = classifyExitCode(error);
+    const code = error instanceof UsageError || error instanceof ConfigError || error instanceof NotFoundError
+        ? error.code
+        : undefined;
+    console.error(JSON.stringify({ ok: false, error: message, ...(code ? { code } : {}), hint }, null, 2));
+    process.exit(exitCode);
+}
+/**
+ * Run an async function and route any thrown error through the standard JSON
+ * error envelope so users never see a raw stack trace.
+ */
+export async function runWithJsonErrors(fn) {
+    try {
+        await fn();
+    }
+    catch (error) {
+        emitJsonError(error);
+    }
+}
+/**
+ * Render a command result according to the active output mode (json/jsonl/yaml/text).
+ */
+export function output(command, result) {
+    const mode = getOutputMode();
+    const shaped = shapeForCommand(command, result, mode.detail, mode.forAgent);
+    if (mode.format === "jsonl") {
+        outputJsonl(command, shaped);
+        return;
+    }
+    switch (mode.format) {
+        case "json":
+            console.log(JSON.stringify(shaped, null, 2));
+            return;
+        case "yaml":
+            console.log(yamlStringify(shaped));
+            return;
+        case "text": {
+            const plain = formatPlain(command, shaped, mode.detail);
+            console.log(plain ?? JSON.stringify(shaped, null, 2));
+            return;
+        }
+        case "md":
+            // `--format md` is currently only consumed by `akm health` for the
+            // per-run / window-compare table renderings. Commands that don't
+            // implement an md renderer fall back to the JSON envelope so
+            // pipelines never get an empty stdout.
+            console.log(JSON.stringify(shaped, null, 2));
+            return;
+    }
+}
+/**
+ * Collect all occurrences of a repeatable flag from process.argv.
+ * Citty's StringArgDef only exposes the last value when a flag is repeated,
+ * so for repeatable CLI args (like `--tag foo --tag bar`) we read argv directly.
+ * Supports both `--flag value` and `--flag=value` forms.
+ */
+export function parseAllFlagValues(flag) {
+    const values = [];
+    for (let i = 0; i < process.argv.length; i++) {
+        const arg = process.argv[i];
+        if (arg === flag && i + 1 < process.argv.length) {
+            values.push(process.argv[i + 1]);
+            // BUG-M4: skip the value index so `--tag --tag` (literal `--tag`
+            // value) does not double-count the second `--tag` as a separate
+            // flag occurrence.
+            i++;
+        }
+        else if (arg.startsWith(`${flag}=`)) {
+            values.push(arg.slice(flag.length + 1));
+        }
+    }
+    return values;
+}