@bli-cockpit/cli 0.2.170 → 0.2.174

package/dist/state-dir-prune.js

@@ -0,0 +1,84 @@
+/**
+ * Bounds the two state-dir writers that had no bound (BLI-12832 audit).
+ *
+ * `read-cache/` keeps one response file per distinct Tower read (the key folds
+ * in the request body, so every different `memory search` is its own file) and
+ * overwrote only on a repeat of the SAME read. `memory-hook-samples/` keeps
+ * hourly timing samples and was "not yet pruned". Both are disposable: a cache
+ * miss re-reads from Tower, an old timing sample is history nobody consults.
+ * Neither is evidence, so deleting from them can never lose anything.
+ *
+ * Every other state-dir entry is bounded where it is written (see the table in
+ * `state-dir-audit.ts`). Runs on every sync tick, never throws, logs counts.
+ */
+import fs from "node:fs/promises";
+import path from "node:path";
+import { describeError } from "./health-detail.js";
+export const READ_CACHE_MAX_AGE_MS = 7 * 24 * 60 * 60 * 1000;
+export const READ_CACHE_MAX_BYTES = 32 * 1024 * 1024;
+export const HOOK_SAMPLES_MAX_AGE_MS = 14 * 24 * 60 * 60 * 1000;
+async function listFiles(dir) {
+    const out = [];
+    const entries = await fs.readdir(dir, { withFileTypes: true }).catch(() => []);
+    for (const entry of entries) {
+        const full = path.join(dir, entry.name);
+        if (entry.isDirectory()) {
+            out.push(...(await listFiles(full)));
+        }
+        else if (entry.isFile()) {
+            const info = await fs.stat(full).catch(() => null);
+            if (info)
+                out.push({ file: full, bytes: info.size, mtime_ms: info.mtimeMs });
+        }
+    }
+    return out;
+}
+async function removeAll(files) {
+    let removed = 0;
+    let bytes = 0;
+    for (const fact of files) {
+        const ok = await fs.rm(fact.file, { force: true }).then(() => true, () => false);
+        if (!ok)
+            continue;
+        removed += 1;
+        bytes += fact.bytes;
+    }
+    return { removed, bytes };
+}
+export async function pruneStateDirCaches(paths, now = new Date()) {
+    const result = {
+        read_cache_removed: 0,
+        read_cache_bytes_freed: 0,
+        hook_samples_removed: 0,
+        hook_samples_bytes_freed: 0,
+    };
+    try {
+        const cache = await listFiles(path.join(paths.state_dir, "read-cache"));
+        const fresh = cache.filter((fact) => now.getTime() - fact.mtime_ms <= READ_CACHE_MAX_AGE_MS);
+        const doomed = cache.filter((fact) => !fresh.includes(fact));
+        let kept = fresh.reduce((sum, fact) => sum + fact.bytes, 0);
+        for (const fact of [...fresh].sort((a, b) => a.mtime_ms - b.mtime_ms)) {
+            if (kept <= READ_CACHE_MAX_BYTES)
+                break;
+            doomed.push(fact);
+            kept -= fact.bytes;
+        }
+        const freed = await removeAll(doomed);
+        result.read_cache_removed = freed.removed;
+        result.read_cache_bytes_freed = freed.bytes;
+        const samples = await listFiles(path.join(paths.state_dir, "memory-hook-samples"));
+        const stale = samples.filter((fact) => now.getTime() - fact.mtime_ms > HOOK_SAMPLES_MAX_AGE_MS);
+        const sampleFreed = await removeAll(stale);
+        result.hook_samples_removed = sampleFreed.removed;
+        result.hook_samples_bytes_freed = sampleFreed.bytes;
+        console.error("[collector prune] state-dir caches bounded", JSON.stringify({
+            reason: freed.removed + sampleFreed.removed > 0 ? "caches_trimmed" : "caches_within_bounds",
+            ...result,
+            read_cache_kept_bytes: kept,
+        }));
+    }
+    catch (error) {
+        console.error("[collector prune] state-dir cache prune failed, nothing else is affected", JSON.stringify({ reason: "state_dir_cache_prune_failed", ...describeError(error) }));
+    }
+    return result;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bli-cockpit/cli",
-  "version": "0.2.170",
+  "version": "0.2.174",
   "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.46",
-    "@bli-cockpit/mcp": "0.1.87",
-    "@bli-cockpit/telemetry-core": "0.1.58"
+    "@bli-cockpit/memory-mcp": "0.1.48",
+    "@bli-cockpit/mcp": "0.1.90",
+    "@bli-cockpit/telemetry-core": "0.1.60"
   },
   "repository": {
     "type": "git",