akm-cli 0.9.11 → 0.9.12

package/dist/integrations/lockfile.js

@@ -9,14 +9,24 @@ import { createLockPayload, probeLock, reclaimStaleLock, releaseLock, tryAcquire
 import { acquireMaintenanceBarrier } from "../core/maintenance-barrier.js";
 import { classifyPathAccess, describeInaccessiblePath } from "../core/path-access.js";
 import { getDataDir, getLockfileLockPath, getLockfilePath } from "../core/paths.js";
+import { warn } from "../core/warn.js";
 // ── Lock sentinel ────────────────────────────────────────────────────────────
-const LOCK_MAX_RETRIES = 3;
-const LOCK_RETRY_DELAY_MS = 100;
+const LOCK_ACQUIRE_TIMEOUT_MS = 30_000;
+const LOCK_RETRY_INITIAL_DELAY_MS = 50;
+const LOCK_RETRY_MAX_DELAY_MS = 1_000;
+let lockAcquireTimeoutMsForTests;
+export function _setLockAcquireTimeoutMsForTests(ms) {
+    lockAcquireTimeoutMsForTests = ms;
+}
 async function acquireLockSentinel() {
     const sentinelPath = getLockfileLockPath();
     // Ensure the directory exists before attempting to create the sentinel.
     fs.mkdirSync(path.dirname(sentinelPath), { recursive: true });
-    for (let attempt = 0; attempt < LOCK_MAX_RETRIES; attempt++) {
+    const timeoutMs = lockAcquireTimeoutMsForTests ?? LOCK_ACQUIRE_TIMEOUT_MS;
+    const deadline = Date.now() + timeoutMs;
+    let delayMs = LOCK_RETRY_INITIAL_DELAY_MS;
+    let announced = false;
+    for (;;) {
         const releaseBarrier = acquireMaintenanceBarrier();
         try {
             const ownership = tryAcquireLockSync(sentinelPath, createLockPayload());
@@ -31,12 +41,17 @@ async function acquireLockSentinel() {
         finally {
             releaseBarrier();
         }
-        // Another process holds the lock — wait briefly before retrying.
-        if (attempt < LOCK_MAX_RETRIES - 1) {
-            await new Promise((resolve) => setTimeout(resolve, LOCK_RETRY_DELAY_MS));
+        // Another process holds the lock.
+        if (Date.now() >= deadline) {
+            throw new ConfigError(`Could not acquire lockfile sentinel at ${sentinelPath} after ${(timeoutMs / 1000).toFixed(1)}s; refusing to write without exclusive ownership.`, "INVALID_CONFIG_FILE");
+        }
+        if (!announced) {
+            announced = true;
+            warn("[akm] Waiting for another akm process to release the lockfile...");
         }
+        await new Promise((resolve) => setTimeout(resolve, delayMs));
+        delayMs = Math.min(delayMs * 2, LOCK_RETRY_MAX_DELAY_MS);
     }
-    throw new ConfigError(`Could not acquire lockfile sentinel at ${sentinelPath}; refusing to write without exclusive ownership.`, "INVALID_CONFIG_FILE");
 }
 // ── Read / Write ────────────────────────────────────────────────────────────
 export function readLockfile() {

package/dist/llm/client.js

@@ -447,3 +447,19 @@ export async function probeLlmReachable(config) {
         return { reachable: false, error: err instanceof Error ? err.message : String(err) };
     }
 }
+/**
+ * Reachability probe for `akm health` (#914): one GET against the
+ * OpenAI-compatible `/models` route, bounded by `timeoutMs`. Any HTTP
+ * response counts as reachable — the question is whether the endpoint
+ * answers, not whether the route exists or the credential is right — so a
+ * cold local server is never asked to load a model just to be checked.
+ */
+export async function probeLlmEndpoint(config, timeoutMs = 3_000) {
+    try {
+        await fetch(`${config.endpoint.replace(/\/+$/, "")}/models`, { signal: AbortSignal.timeout(timeoutMs) });
+        return { reachable: true };
+    }
+    catch (err) {
+        return { reachable: false, error: err instanceof Error ? err.message : String(err) };
+    }
+}

package/dist/llm/index-passes.js

@@ -1,7 +1,7 @@
 // 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 { ConfigError } from "../core/errors.js";
+import { warn } from "../core/warn.js";
 import { cloneExecutionJsonObject } from "../execution/json.js";
 import { lowerResolvedExecutionRequest } from "../integrations/agent/execution-lowering.js";
 import { prepareInlineExecution } from "../integrations/agent/inline-execution.js";
@@ -48,7 +48,8 @@ export function resolveIndexPassExecution(passName, config) {
     });
     const lowered = lowerResolvedExecutionRequest(prepared.request, prepared.config);
     if (lowered.runner.kind !== "llm") {
-        throw new ConfigError(`Index pass ${JSON.stringify(passName)} requires an LLM engine; ${JSON.stringify(selectedEngine)} is not one.`, "INVALID_CONFIG_FILE");
+        warn("[akm] Index pass %s requires an LLM engine; %s is not one. Skipping this pass.", passName, selectedEngine);
+        return Object.freeze({ runner: undefined, notices: NO_LOWERING_NOTICES });
     }
     return Object.freeze({ runner: lowered.runner, notices: lowered.notices });
 }

package/dist/output/shapes/passthrough.js

@@ -3,7 +3,11 @@
 // file, You can obtain one at https://mozilla.org/MPL/2.0/.
 // #484: stamp schemaVersion + shape discriminator on passthrough envelopes so
 // third-party consumers can pin a schema version and dispatch on shape uniformly.
-// Idempotent — never overwrites an existing schemaVersion or shape field.
+// #918: also stamp `ok: true` so a caller branching on `.ok` sees the same
+// field on every success envelope that it sees on the `{ok:false,...}` error
+// envelope. Never overwrites an existing `shape`, `schemaVersion`, or `ok`;
+// a command whose exit code grades the outcome emits through
+// `outputWithExitCode` (src/cli/shared.ts), which sets `ok` from that code.
 //
 // Builds a shallow copy rather than mutating `result` in place: several
 // command results (e.g. `akm task sync --dry-run`'s `SchedulerPlanPreview`,
@@ -20,9 +24,11 @@ function makeStampHandler(command) {
         if (typeof result !== "object" || Array.isArray(result))
             return result;
         const obj = result;
-        if (obj.shape !== undefined && obj.schemaVersion !== undefined)
-            return obj;
         return {
+            // `ok` first so it heads the printed envelope instead of trailing a
+            // (possibly large) dump — the spread below still wins with `obj`'s own
+            // `ok` value when present, only its position is fixed here.
+            ok: obj.ok ?? true,
             ...obj,
             shape: obj.shape ?? command,
             schemaVersion: obj.schemaVersion ?? 1,

package/dist/output/shapes.js

@@ -17,7 +17,7 @@
  * and throws for unknown commands (v1 spec §9 — exhaustive registry, no silent
  * fallback).
  */
-import { UsageError } from "../core/errors.js";
+import { warnOnce } from "../core/warn.js";
 import { curateShapes } from "./shapes/curate.js";
 import { envListShapes } from "./shapes/env-list.js";
 import { eventsShapes } from "./shapes/events.js";
@@ -67,12 +67,14 @@ registerOutputShapes(BUILT_IN_OUTPUT_SHAPES);
  */
 const SHAPE_SUMMARY_COMMANDS = new Set(["show"]);
 export function shapeForCommand(command, result, detail, shape = "human") {
+    let effectiveShape = shape;
     if (shape === "summary" && !SHAPE_SUMMARY_COMMANDS.has(command)) {
-        throw new UsageError(`'--shape summary' is not supported for 'akm ${command}'. It is only available on 'akm show'.`, "INVALID_SHAPE_VALUE");
+        warnOnce(`shape-summary-unsupported:${command}`, `[output] '--shape summary' is not supported for 'akm ${command}' (only 'akm show' has a summary projection); falling back to 'agent'.`);
+        effectiveShape = "agent";
     }
     const handler = getOutputShapeHandler(command);
     if (handler) {
-        return handler(result, detail, shape);
+        return handler(result, detail, effectiveShape);
     }
     // v1 spec §9 (output-shape registry exhaustive): no silent JSON.stringify
     // fallback. A missing case here is a registration bug — fail loudly so

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

@@ -172,7 +172,14 @@ export function formatWorkflowRunPlain(result) {
     const run = typeof result.run === "object" && result.run !== null ? result.run : undefined;
     if (!run)
         return null;
-    const lines = [`run: ${String(run.id ?? "unknown")}`, `status: ${String(run.status ?? "unknown")}`];
+    const lines = [];
+    // #919: `workflow run <ref>` silently resuming an existing active run (the
+    // #485 concurrency guard) is now announced up front, with the escape hatch.
+    if (result.resumed === true) {
+        lines.push(`resuming existing run ${String(run.id ?? "unknown")} for ${String(run.workflowRef ?? "this ref")}; ` +
+            `pass --new to start a fresh run`);
+    }
+    lines.push(`run: ${String(run.id ?? "unknown")}`, `status: ${String(run.status ?? "unknown")}`);
     // Creation-time notices (e.g. the implicit engine fallback) must survive the
     // text renderer: JSON/YAML pass them through, and dropping them here would
     // hide the announcement from the DEFAULT output mode — where it matters most.