@bli-cockpit/cli 0.2.276 → 0.2.278

package/dist/image-store-state.js

@@ -0,0 +1,78 @@
+/**
+ * Which images Tower refused to store, and until when this machine leaves them alone (BLI-23329).
+ *
+ * Kept in its OWN file, apart from the described/failed record of the describe door
+ * (`image-descriptions.json`, shared with the BLI Memory daemon). Two reasons, both found in plan review:
+ *  - the describe record's merge deletes a failure for every image that is described by now, whatever the failure
+ *    was about (`mergeImageDescriptionStates`), so a refused store of an image the describer knows would be
+ *    forgotten at the next write and retried on every tick;
+ *  - the daemon rewrites that file from its own copy; a field it does not know can be dropped.
+ * This file is written by the sync tick alone (one writer under the sync lock), so neither can touch it.
+ *
+ * Holds keys, reason labels, attempt counts and times only. Never a path, a session id or a byte.
+ */
+import { createHash } from "node:crypto";
+import path from "node:path";
+import { imageRetryDelayMs, imageStateKey } from "./image-description-state.js";
+import { isMissingFileError, readJsonFile, writeJsonFileAtomic } from "./local-state-files.js";
+export const IMAGE_STORE_FAILURES_FILENAME = "image-store-failures.json";
+export const IMAGE_STORE_FAILURES_SCHEMA = "cockpit-image-store-failures.v1";
+/** Oldest records leave first past this many. */
+export const IMAGE_STORE_FAILURES_MAX = 5_000;
+const KEY_CHARS = 32;
+const TAG = "[image-store]";
+export function imageStoreFailuresPath(stateDir) {
+    return path.join(stateDir, IMAGE_STORE_FAILURES_FILENAME);
+}
+/**
+ * The key a refusal is remembered under. Most refusals are about the IMAGE (its bytes, its type): the image's key.
+ * A refusal that is about the SESSION (a founder forgot it) must not stop another session storing the same picture,
+ * so it is remembered under a key that mixes both in.
+ */
+export function storeFailureKey(sha256, sessionId, aboutSession) {
+    if (!aboutSession)
+        return imageStateKey(sha256);
+    return createHash("sha256").update(`${sessionId}|${sha256}`).digest("hex").slice(0, KEY_CHARS);
+}
+/** The failures on record. A missing file is empty; an unreadable or damaged one is empty and said out loud. */
+export async function readStoreFailures(stateDir) {
+    let raw;
+    try {
+        raw = await readJsonFile(imageStoreFailuresPath(stateDir));
+    }
+    catch (error) {
+        if (!isMissingFileError(error)) {
+            console.error(TAG + " store failure record unreadable, starting from an empty one", JSON.stringify({ reason: "image_store_failures_unreadable" }));
+        }
+        return {};
+    }
+    const record = (raw && typeof raw === "object" ? raw : {});
+    const failed = {};
+    if (record.failed && typeof record.failed === "object" && !Array.isArray(record.failed)) {
+        for (const [key, value] of Object.entries(record.failed)) {
+            const entry = value;
+            if (key.length !== KEY_CHARS || !entry)
+                continue;
+            if (typeof entry.reason !== "string" || typeof entry.attempts !== "number" || typeof entry.retry_after !== "number")
+                continue;
+            failed[key] = { reason: entry.reason, attempts: entry.attempts, retry_after: entry.retry_after };
+        }
+    }
+    return failed;
+}
+export async function writeStoreFailures(stateDir, failures) {
+    const entries = Object.entries(failures).sort((a, b) => a[1].retry_after - b[1].retry_after);
+    const kept = Object.fromEntries(entries.slice(Math.max(0, entries.length - IMAGE_STORE_FAILURES_MAX)));
+    await writeJsonFileAtomic(imageStoreFailuresPath(stateDir), { schema_version: IMAGE_STORE_FAILURES_SCHEMA, failed: kept });
+}
+/** Remember one more refusal of this image and push its next try out along the same ladder the describe door uses. */
+export function rememberStoreFailure(failures, key, reason, permanent, now) {
+    const attempts = (failures[key]?.attempts ?? 0) + 1;
+    const record = { reason, attempts, retry_after: now + imageRetryDelayMs(attempts, permanent) };
+    failures[key] = record;
+    return record;
+}
+export function isBackingOff(failures, key, now) {
+    const record = failures[key];
+    return Boolean(record && record.retry_after > now);
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bli-cockpit/cli",
-  "version": "0.2.276",
+  "version": "0.2.278",
   "private": false,
   "type": "module",
   "bin": {
@@ -27,9 +27,9 @@
     "test": "node dist/cli.js --help && node ../../scripts/assert-public-cli-routing.mjs && node ../../scripts/assert-public-cli-verb-help.mjs && node ../../scripts/assert-public-cli-surface-doors.mjs && node ../../scripts/assert-public-cli-runtime-files.mjs && node ../../scripts/assert-public-cli-exit-contract.mjs && node ../../scripts/assert-public-cli-no-fleet-posts.mjs && node ../../scripts/assert-public-cli-doctor-json.mjs && node ../../scripts/assert-public-package-pack.mjs --workspace=@bli-cockpit/cli"
   },
   "dependencies": {
-    "@bli-cockpit/memory-mcp": "0.1.84",
-    "@bli-cockpit/mcp": "0.1.168",
-    "@bli-cockpit/telemetry-core": "0.1.91",
+    "@bli-cockpit/memory-mcp": "0.1.85",
+    "@bli-cockpit/mcp": "0.1.169",
+    "@bli-cockpit/telemetry-core": "0.1.92",
     "unpdf": "1.8.1"
   },
   "repository": {