@bli-cockpit/cli 0.2.69 → 0.2.72

package/dist/local-state-status.js

@@ -1,5 +1,7 @@
 import os from "node:os";
 import path from "node:path";
+import { readStagingInventory } from "./disk-usage.js";
+import { describeError } from "./health-detail.js";
 import { summarizeLocalUploadSpool } from "./spool/local-spool.js";
 import { summarizeInstallEventOutbox } from "./spool/install-event-outbox.js";
 import { readRawEvidenceStagingState, summarizeStuckEvidence, } from "./raw-evidence-staging.js";
@@ -55,6 +57,10 @@ export async function inspectLocalCollectorStatus(options = {}) {
         stuck_evidence_max_attempts: stuckEvidence.max_attempts,
         stuck_evidence_oldest_failure_at: stuckEvidence.oldest_first_failed_at,
         stuck_evidence_reasons: stuckEvidence.reasons,
+        staging_uncommitted_count: reading.staging?.uncommitted_count ?? null,
+        staging_uncommitted_bytes: reading.staging?.uncommitted_bytes ?? null,
+        staging_uncommitted_reason: reading.staging?.dominant_uncommitted_reason?.reason ?? null,
+        staging_oldest_uncommitted_at: reading.staging?.oldest_uncommitted_at ?? null,
         details: describeCollectorStatus(reading),
     };
 }
@@ -76,7 +82,20 @@ async function readCollectorStatus(options, now) {
         readRawEvidenceStagingState(paths.state_dir),
     ]);
     const stuckEvidence = summarizeStuckEvidence(stagingState, now);
+    // Best-effort by construction: a status read must never fail because the
+    // staging tree could not be walked, and an absent backlog reads as "this read
+    // did not ask", never as "there is none".
+    const staging = options.includeStagingBacklog
+        ? await readStagingInventory(paths, now).catch((error) => {
+            console.error("[collector status] the staging tree could not be read; the undelivered backlog is unknown", JSON.stringify({
+                reason: "staging_inventory_unreadable",
+                ...describeError(error),
+            }));
+            return null;
+        })
+        : null;
     return {
+        staging,
         paths,
         config,
         session,
@@ -134,6 +153,18 @@ function describeCollectorStatus(reading) {
     if (healthOutbox.pending_count > 0) {
         details.push(`Collector health retry pending: ${healthOutbox.pending_count} sanitized receipt(s) queued since ${healthOutbox.oldest_created_at ?? "unknown"}.`);
     }
+    // BLI-3797: the line above only sees objects whose delivery FAILED. This one
+    // sees evidence that is on this disk and not in the bucket, however it got
+    // that way — which is the only question that answers "has everything landed?".
+    if (reading.staging && reading.staging.uncommitted_count > 0) {
+        const dominant = reading.staging.dominant_uncommitted_reason;
+        details.push(`Staged evidence not yet accepted: ${reading.staging.uncommitted_count} object(s), ${mib(reading.staging.uncommitted_bytes)} MB, oldest staged ${reading.staging.oldest_uncommitted_at ?? "unknown"}${dominant
+            ? ` — biggest reason ${dominant.reason} on ${dominant.count} object(s)`
+            : ""}. \`cockpit sync\` re-offers them each tick; \`cockpit doctor --fix\` does it now.`);
+    }
+    else if (reading.staging) {
+        details.push("Staged evidence: every object on this disk is accepted.");
+    }
     if (stuckEvidence.stuck_object_count > 0) {
         details.push(`Raw evidence stuck: ${stuckEvidence.stuck_object_count} object(s) have never been accepted (${stuckEvidence.held_object_count} waiting on backoff, worst ${stuckEvidence.max_attempts} attempt(s) since ${stuckEvidence.oldest_first_failed_at ?? "unknown"}) — reasons: ${stuckEvidence.reasons.join(", ") || "unknown"}.`);
     }
@@ -155,6 +186,9 @@ export function classifyCollectorFreshness(context, lastUploadSuccessAt, now) {
         return "stale";
     return now.getTime() - latestActivityAt <= 5 * 60 * 1000 ? "fresh" : "stale";
 }
+function mib(bytes) {
+    return (bytes / (1024 * 1024)).toFixed(1);
+}
 function latestTimestamp(values) {
     const timestamps = values
         .map((value) => Date.parse(value ?? ""))

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bli-cockpit/cli",
-  "version": "0.2.69",
+  "version": "0.2.72",
   "private": false,
   "type": "module",
   "bin": {
@@ -27,8 +27,8 @@
     "test": "node dist/cli.js --help && node ../../scripts/assert-public-cli-routing.mjs && node ../../scripts/assert-public-cli-runtime-files.mjs && node ../../scripts/assert-public-cli-no-fleet-posts.mjs && node ../../scripts/assert-public-package-pack.mjs --workspace=@bli-cockpit/cli"
   },
   "dependencies": {
-    "@bli-cockpit/memory-mcp": "0.1.9",
-    "@bli-cockpit/mcp": "0.1.10",
-    "@bli-cockpit/telemetry-core": "0.1.30"
+    "@bli-cockpit/memory-mcp": "0.1.11",
+    "@bli-cockpit/mcp": "0.1.12",
+    "@bli-cockpit/telemetry-core": "0.1.32"
   }
 }