akm-cli 0.9.0-beta.6 → 0.9.0-rc.0

package/dist/commands/improve/improve-profiles.js

@@ -7,10 +7,14 @@ import profileDefault from "../../assets/profiles/default.json" with { type: "js
 import profileFrequent from "../../assets/profiles/frequent.json" with { type: "json" };
 import profileGraphRefresh from "../../assets/profiles/graph-refresh.json" with { type: "json" };
 import profileMemoryFocus from "../../assets/profiles/memory-focus.json" with { type: "json" };
+import profileProactiveMaintenance from "../../assets/profiles/proactive-maintenance.json" with { type: "json" };
 import profileQuick from "../../assets/profiles/quick.json" with { type: "json" };
+import profileRecombineOnly from "../../assets/profiles/recombine-only.json" with { type: "json" };
+import profileReflectDistill from "../../assets/profiles/reflect-distill.json" with { type: "json" };
+import profileSynthesize from "../../assets/profiles/synthesize.json" with { type: "json" };
 import profileThorough from "../../assets/profiles/thorough.json" with { type: "json" };
 import { parseAssetRef } from "../../core/asset/asset-ref.js";
-import { warn } from "../../core/warn.js";
+import { ConfigError } from "../../core/errors.js";
 /** Profile name used as the final fallback when nothing else resolves. */
 const FALLBACK_PROFILE_NAME = "default";
 // Built-in default allowed types per process
@@ -31,6 +35,10 @@ const BUILTIN_PROFILES = {
     frequent: profileFrequent,
     consolidate: profileConsolidate,
     catchup: profileCatchup,
+    synthesize: profileSynthesize,
+    "reflect-distill": profileReflectDistill,
+    "proactive-maintenance": profileProactiveMaintenance,
+    "recombine-only": profileRecombineOnly,
 };
 /**
  * Default enabled-state for known improve processes when neither the user
@@ -54,6 +62,18 @@ const IMPROVE_PROCESS_DEFAULTS = {
     // proposal-queue triage drains the standing backlog. Opt-in (default off),
     // like `validation` — needs an explicit `enabled: true`.
     triage: false,
+    // Layer 2 proactive-maintenance selector. Opt-in (default off) — surfaces
+    // stale high-value assets on a schedule. Enable per-profile with an explicit
+    // `processes.proactiveMaintenance.enabled: true`.
+    proactiveMaintenance: false,
+    // #609 recombine / synthesize pass — whole-corpus cross-episodic
+    // generalization. Opt-in (default off); enable per-profile with an explicit
+    // `processes.recombine.enabled: true`.
+    recombine: false,
+    // #615 procedural-compilation pass — detects recurring successful ordered
+    // action sequences and compiles them into workflow proposals. Opt-in (default
+    // off); enable per-profile with an explicit `processes.procedural.enabled: true`.
+    procedural: false,
 };
 /**
  * Compute the effective enabled-state for a named improve process.
@@ -96,14 +116,18 @@ export function resolveImproveProfile(name, config) {
         FALLBACK_PROFILE_NAME;
     const hasBuiltin = requestedName in BUILTIN_PROFILES;
     const hasUserDefined = !!config.profiles?.improve?.[requestedName];
-    let effectiveName = requestedName;
-    if (!hasBuiltin && !hasUserDefined && requestedName !== FALLBACK_PROFILE_NAME) {
-        warn(`[akm] Improve profile "${requestedName}" not found in built-ins or config. ` +
-            `Falling back to "${FALLBACK_PROFILE_NAME}".`);
-        effectiveName = FALLBACK_PROFILE_NAME;
+    // An unknown profile name is a HARD error. Silently falling back to the
+    // default (proactive-off) profile is the −96% incident class: a cron pinned
+    // to `--profile reflect-distill` ran the default for weeks because the name
+    // only existed in one host's config and the resolver swallowed the miss.
+    if (!hasBuiltin && !hasUserDefined) {
+        const valid = [
+            ...new Set([...Object.keys(BUILTIN_PROFILES), ...Object.keys(config.profiles?.improve ?? {})]),
+        ].sort();
+        throw new ConfigError(`Improve profile "${requestedName}" not found. Valid profiles: ${valid.join(", ")}.`, "UNKNOWN_IMPROVE_PROFILE");
     }
-    const builtin = BUILTIN_PROFILES[effectiveName] ?? BUILTIN_PROFILES[FALLBACK_PROFILE_NAME];
-    const userOverride = config.profiles?.improve?.[effectiveName] ?? {};
+    const builtin = BUILTIN_PROFILES[requestedName] ?? BUILTIN_PROFILES[FALLBACK_PROFILE_NAME];
+    const userOverride = config.profiles?.improve?.[requestedName] ?? {};
     return deepMerge(builtin, userOverride);
 }
 export function shouldSkipRef(ref, processName, profile) {

package/dist/commands/improve/improve-result-file.js

@@ -27,7 +27,8 @@
  */
 import crypto from "node:crypto";
 import path from "node:path";
-import { openStateDatabase, recordImproveRun } from "../../core/state-db.js";
+import { withStateDb } from "../../core/state-db.js";
+import { recordImproveRun } from "../../storage/repositories/improve-runs-repository.js";
 /**
  * Build a stable run-id for a single improve invocation.
  *
@@ -72,10 +73,16 @@ export function relativeImproveResultPath(runId) {
  * dry-run column is indexed so productivity audits can filter cleanly
  * (closes the dry-run/real-run artifact-trap recorded in MEMORY.md
  * `feedback_akm_dryrun_artifact_trap`).
+ *
+ * @param profile - The `--profile` value passed to this invocation (e.g.
+ * `quick`, `reflect-distill`), or `null`/`undefined` when no profile was
+ * given. Mirrors {@link recordTerminatedImproveRun}'s `ctx.profile`
+ * convention so successful and terminated runs are equally queryable by
+ * profile. Previously hardcoded to `null` here, which meant only
+ * abnormally-terminated runs recorded their profile in state.db.
  */
-export function writeImproveResultFile(stashDir, runId, result, startedAt) {
-    const db = openStateDatabase();
-    try {
+export function writeImproveResultFile(stashDir, runId, result, startedAt, profile) {
+    withStateDb((db) => {
         const completedAt = new Date().toISOString();
         // startedAt is the ISO timestamp captured at process launch (passed from the
         // CLI entry point). If omitted, fall back to the run-id's embedded timestamp
@@ -88,22 +95,14 @@ export function writeImproveResultFile(stashDir, runId, result, startedAt) {
             completedAt,
             stashDir,
             dryRun: Boolean(result.dryRun),
-            profile: null,
+            profile: profile ?? null,
             scopeMode: result.scope?.mode ?? "all",
             scopeValue: result.scope?.value ?? null,
             guidance: result.guidance ?? null,
             ok: Boolean(result.ok),
             result,
         });
-    }
-    finally {
-        try {
-            db.close();
-        }
-        catch {
-            // best-effort
-        }
-    }
+    });
     return relativeImproveResultPath(runId);
 }
 /**
@@ -138,8 +137,7 @@ export function recordTerminatedImproveRun(stashDir, runId, startedAt, reason, c
             ...(ctx?.errorMessage ? { errorMessage: ctx.errorMessage } : {}),
         },
     };
-    const db = openStateDatabase();
-    try {
+    withStateDb((db) => {
         recordImproveRun(db, {
             id: runId,
             startedAt,
@@ -160,13 +158,5 @@ export function recordTerminatedImproveRun(stashDir, runId, startedAt, reason, c
                 },
             },
         });
-    }
-    finally {
-        try {
-            db.close();
-        }
-        catch {
-            // best-effort
-        }
-    }
+    });
 }

package/dist/commands/improve/improve-session.js

@@ -0,0 +1,58 @@
+// 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/.
+/** Signal -> {exit code, reason, ack message}. */
+export const SIGNAL_TABLE = {
+    SIGTERM: { code: 143, reason: "SIGTERM", ack: true },
+    SIGINT: { code: 130, reason: "SIGINT", ack: true },
+    SIGHUP: { code: 129, reason: "SIGHUP", ack: false },
+};
+/**
+ * Drive the improve-run lifecycle: register signal handlers, await `runWork`,
+ * and on clean completion remove the handlers and resolve with the result. The
+ * caller (improve-cli.ts) owns success-path result persistence + the final
+ * `exit(0)`; this function returns the work result so that choreography stays
+ * in the CLI exactly as before.
+ *
+ * Abnormal paths:
+ *   - A registered signal fires -> persist terminated row (watchdog-guarded) ->
+ *     optional ack -> `exit(code)`.
+ *   - `runWork` rejects -> handlers are removed and the rejection propagates so
+ *     the CLI's existing catch can persist `"exception"` and rethrow.
+ */
+export async function runImproveSession(opts, deps) {
+    const { signalSource, exit, onTerminate, ack } = deps;
+    const makeSignalHandler = (sig) => () => {
+        const { code, reason, ack: shouldAck } = SIGNAL_TABLE[sig];
+        // Hard-exit fallback: if the synchronous persist ever hangs (e.g. a stuck
+        // sqlite lock under contention), the watchdog still exits with the correct
+        // code instead of leaving a zombie process. .unref() keeps the timer from
+        // holding the loop open on the normal (fast) path.
+        const watchdog = setTimeout(() => exit(code), 2000);
+        if (typeof watchdog.unref === "function")
+            watchdog.unref();
+        try {
+            onTerminate(reason);
+        }
+        finally {
+            clearTimeout(watchdog);
+        }
+        if (shouldAck)
+            ack?.(`received ${sig}`);
+        exit(code);
+    };
+    const sigtermHandler = makeSignalHandler("SIGTERM");
+    const sigintHandler = makeSignalHandler("SIGINT");
+    const sighupHandler = makeSignalHandler("SIGHUP");
+    signalSource.once("SIGTERM", sigtermHandler);
+    signalSource.once("SIGINT", sigintHandler);
+    signalSource.once("SIGHUP", sighupHandler);
+    try {
+        return await opts.runWork();
+    }
+    finally {
+        signalSource.removeListener("SIGTERM", sigtermHandler);
+        signalSource.removeListener("SIGINT", sigintHandler);
+        signalSource.removeListener("SIGHUP", sighupHandler);
+    }
+}