akm-cli 0.8.2 → 0.9.0-beta.0

package/dist/commands/{improve-auto-accept.js → improve/improve-auto-accept.js}

@@ -1,10 +1,10 @@
 // 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/.
-import { loadConfig } from "../core/config";
-import { appendEvent } from "../core/events";
-import { promoteProposal } from "../core/proposals";
-import { info, warn } from "../core/warn";
+import { loadConfig } from "../../core/config/config.js";
+import { appendEvent } from "../../core/events.js";
+import { info, warn } from "../../core/warn.js";
+import { promoteProposal } from "../proposal/validators/proposals.js";
 // ---------------------------------------------------------------------------
 // Gate implementation
 // ---------------------------------------------------------------------------

package/dist/commands/{improve-cli.js → improve/improve-cli.js}

@@ -3,16 +3,16 @@
 // file, You can obtain one at https://mozilla.org/MPL/2.0/.
 import path from "node:path";
 import { defineCommand } from "citty";
-import { getStringArg, parseAutoAcceptFlag, parseNonNegativeIntFlag, parsePositiveIntFlag } from "../cli/parse-args";
-import { output, runWithJsonErrors } from "../cli/shared";
-import { loadConfig } from "../core/config";
-import { UsageError } from "../core/errors";
-import { getCacheDir } from "../core/paths";
-import { clearLogFile, setLogFile } from "../core/warn";
-import { resolveSourceEntries } from "../indexer/search-source";
-import { getHyphenatedArg, getHyphenatedBoolean, parseFlagValue } from "../output/context";
-import { akmImprove } from "./improve";
-import { buildImproveRunId, recordTerminatedImproveRun, relativeImproveResultPath, writeImproveResultFile, } from "./improve-result-file";
+import { getStringArg, parseAutoAcceptFlag, parseNonNegativeIntFlag, parsePositiveIntFlag } from "../../cli/parse-args.js";
+import { output, runWithJsonErrors } from "../../cli/shared.js";
+import { loadConfig } from "../../core/config/config.js";
+import { UsageError } from "../../core/errors.js";
+import { getCacheDir } from "../../core/paths.js";
+import { clearLogFile, setLogFile } from "../../core/warn.js";
+import { resolveSourceEntries } from "../../indexer/search/search-source.js";
+import { getHyphenatedArg, getHyphenatedBoolean, parseFlagValue } from "../../output/context.js";
+import { akmImprove } from "./improve.js";
+import { buildImproveRunId, recordTerminatedImproveRun, relativeImproveResultPath, writeImproveResultFile, } from "./improve-result-file.js";
 export const improveCommand = defineCommand({
     meta: {
         name: "improve",
@@ -138,20 +138,41 @@ export const improveCommand = defineCommand({
                     process.stderr.write(`warning: failed to persist terminated improve run ${runId}: ${err instanceof Error ? err.message : String(err)}\n`);
                 }
             };
-            const sigtermHandler = () => {
-                persistTerminated("SIGTERM");
-                process.stderr.write(`[improve] received SIGTERM; recorded terminated run ${runId}\n`);
-                process.exit(143);
+            // M5 (code-health round 2): signal -> {exit code, reason, ack message}
+            // as an explicit table instead of three near-identical handlers. The
+            // persist of the terminated-run row MUST complete before process.exit so
+            // a SIGTERM'd run (e.g. cron timeout) always leaves a row in
+            // improve_runs. recordTerminatedImproveRun is fully synchronous
+            // (bun:sqlite writes are sync), so the in-line call below blocks until
+            // the row is flushed before we exit.
+            const SIGNAL_TABLE = {
+                SIGTERM: { code: 143, reason: "SIGTERM", ack: true },
+                SIGINT: { code: 130, reason: "SIGINT", ack: true },
+                SIGHUP: { code: 129, reason: "SIGHUP", ack: false },
             };
-            const sigintHandler = () => {
-                persistTerminated("SIGINT");
-                process.stderr.write(`[improve] received SIGINT; recorded terminated run ${runId}\n`);
-                process.exit(130);
-            };
-            const sighupHandler = () => {
-                persistTerminated("SIGHUP");
-                process.exit(129);
+            const makeSignalHandler = (sig) => () => {
+                const { code, reason, ack } = 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(() => process.exit(code), 2000);
+                if (typeof watchdog.unref === "function")
+                    watchdog.unref();
+                try {
+                    persistTerminated(reason);
+                }
+                finally {
+                    clearTimeout(watchdog);
+                }
+                if (ack) {
+                    process.stderr.write(`[improve] received ${sig}; recorded terminated run ${runId}\n`);
+                }
+                process.exit(code);
             };
+            const sigtermHandler = makeSignalHandler("SIGTERM");
+            const sigintHandler = makeSignalHandler("SIGINT");
+            const sighupHandler = makeSignalHandler("SIGHUP");
             process.once("SIGTERM", sigtermHandler);
             process.once("SIGINT", sigintHandler);
             process.once("SIGHUP", sighupHandler);
@@ -163,6 +184,7 @@ export const improveCommand = defineCommand({
                     dryRun,
                     target: targetArg,
                     autoAccept,
+                    ...(runId !== undefined ? { runId } : {}),
                     ...(limitRaw !== undefined ? { limit: limitRaw } : {}),
                     ...(timeoutMs !== undefined ? { timeoutMs } : {}),
                     ...(minRetrievalCount !== undefined ? { minRetrievalCount } : {}),

package/dist/commands/{improve-profiles.js → improve/improve-profiles.js}

@@ -1,13 +1,16 @@
 // 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/.
-import profileDefault from "../assets/profiles/default.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 profileQuick from "../assets/profiles/quick.json" with { type: "json" };
-import profileThorough from "../assets/profiles/thorough.json" with { type: "json" };
-import { parseAssetRef } from "../core/asset-ref";
-import { warn } from "../core/warn";
+import profileCatchup from "../../assets/profiles/catchup.json" with { type: "json" };
+import profileConsolidate from "../../assets/profiles/consolidate.json" with { type: "json" };
+import profileDefault from "../../assets/profiles/default.json" with { type: "json" };
+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 profileQuick from "../../assets/profiles/quick.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";
 /** Profile name used as the final fallback when nothing else resolves. */
 const FALLBACK_PROFILE_NAME = "default";
 // Built-in default allowed types per process
@@ -25,6 +28,9 @@ const BUILTIN_PROFILES = {
     thorough: profileThorough,
     "memory-focus": profileMemoryFocus,
     "graph-refresh": profileGraphRefresh,
+    frequent: profileFrequent,
+    consolidate: profileConsolidate,
+    catchup: profileCatchup,
 };
 /**
  * Default enabled-state for known improve processes when neither the user

package/dist/commands/{improve-result-file.js → improve/improve-result-file.js}

@@ -27,7 +27,7 @@
  */
 import crypto from "node:crypto";
 import path from "node:path";
-import { openStateDatabase, recordImproveRun } from "../core/state-db";
+import { openStateDatabase, recordImproveRun } from "../../core/state-db.js";
 /**
  * Build a stable run-id for a single improve invocation.
  *