@bli-cockpit/cli 0.2.256 → 0.2.257

package/dist/commands/agent-session-root-scope.js

@@ -16,4 +16,23 @@ import { OUTSIDE_APPROVED_ROOT_REASONS } from "./session-sync-health.js";
  */
 export function rootScopeForReason(reason) {
     return OUTSIDE_APPROVED_ROOT_REASONS.has(reason) ? "outside" : "inside";
+}
+/**
+ * The terminal attribution states: attribution finished and named no repo. They
+ * are labels, never a reason to withhold (session-first collection).
+ */
+const TERMINAL_ATTRIBUTION_STATES = new Set(["ambiguous", "unattributed", "skipped"]);
+/**
+ * Reasons a session carries no workspace of its own that are NOT an outside-root
+ * fact, so `rootScopeForReason` keeps calling them "inside". Today: a transcript
+ * with no cwd, workspace or origin signal at all (`no_repo_signals`).
+ */
+export const NO_WORKSPACE_REASONS = new Set(["no_repo_signals"]);
+/**
+ * Does this session still lack a workspace the upload can stand in? Every
+ * terminal state without a worktree rides the collector's synthetic workspace,
+ * whatever its reason; attribution stays exactly as scored.
+ */
+export function needsTransportWorkspace(result) {
+    return !result.worktree && TERMINAL_ATTRIBUTION_STATES.has(result.state);
 }

package/dist/commands/backfill-outside-roots.js

@@ -24,7 +24,7 @@ import fs from "node:fs/promises";
 import path from "node:path";
 import { getCollectorRuntimePaths } from "../local-state.js";
 import { repoFingerprintFromLocalRoot, stableWorktreeFingerprint, } from "../repo-identity.js";
-import { rootScopeForReason } from "./agent-session-root-scope.js";
+import { NO_WORKSPACE_REASONS, needsTransportWorkspace, rootScopeForReason } from "./agent-session-root-scope.js";
 /**
  * The label the sync path writes for a report with no real worktree
  * (`session-sync-record.ts`). Reused verbatim: one label, not a second one.
@@ -64,18 +64,22 @@ export function isOutsideRootCandidate(candidate) {
 export function labelOutsideRootCandidates(candidates, workspace) {
     let codex = 0;
     let claude = 0;
+    let noRepoSignals = 0;
     const labelled = candidates.map((candidate) => {
-        if (candidate.worktree || !isOutsideRootCandidate(candidate))
+        if (!needsTransportWorkspace(candidate))
             return candidate;
         if (candidate.source === "codex")
             codex += 1;
         else
             claude += 1;
+        if (NO_WORKSPACE_REASONS.has(candidate.reason))
+            noRepoSignals += 1;
         return { ...candidate, worktree: workspace };
     });
     console.error("[backfill] outside-root sessions labelled for upload", JSON.stringify({
-        root_scope: "outside",
+        reason: "unattributed_sessions_labelled_for_upload",
         count: codex + claude,
+        no_repo_signals_count: noRepoSignals,
         codex_count: codex,
         claude_count: claude,
         candidate_count: candidates.length,

package/dist/commands/public-root.js

@@ -21,7 +21,7 @@ export async function runCockpitCli(argv, io) {
   }
 
   if (command === "--version" || command === "-V" || command === "version") {
-    writeLine(io?.stdout ?? process.stdout, "0.2.256");
+    writeLine(io?.stdout ?? process.stdout, "0.2.257");
     return 0;
   }
 

package/dist/commands/session-sync-outside-roots.js

@@ -47,7 +47,7 @@ import { isRedactionStubObjectKey } from "../cursors/raw-evidence-stub-entry.js"
 import { describeError } from "../health-detail.js";
 import { getCollectorRuntimePaths } from "../local-state.js";
 import { isMissingFileError, writeJsonFileAtomic, writeJsonFile } from "../local-state-files.js";
-import { rootScopeForReason } from "./agent-session-root-scope.js";
+import { NO_WORKSPACE_REASONS, needsTransportWorkspace, rootScopeForReason } from "./agent-session-root-scope.js";
 import { ensureOutsideRootsWorkspace, isOutsideRootsWorkspace, outsideRootsWorkspace, } from "./backfill-outside-roots.js";
 /** Most outside-root sessions that upload bytes in one tick. 4 ticks an hour, so up to 3,840 a day. */
 export const OUTSIDE_ROOT_SESSIONS_PER_TICK = 40;
@@ -112,7 +112,7 @@ export function widenForCatchUp(sinceMinutes, catchUpPending, normalMinutes) {
  * than the horizon (that is backfill's).
  */
 export function outsideRootEntryOwesUpload(entry, now = new Date()) {
-    if (rootScopeForReason(entry.reason) !== "outside")
+    if (rootScopeForReason(entry.reason) !== "outside" && !NO_WORKSPACE_REASONS.has(entry.reason))
         return false;
     if (entry.uploaded_object_key && !isRedactionStubObjectKey(entry.uploaded_object_key))
         return false;
@@ -120,9 +120,13 @@ export function outsideRootEntryOwesUpload(entry, now = new Date()) {
         return false;
     return now.getTime() - entry.file_mtime_ms <= OUTSIDE_ROOT_HORIZON_MINUTES * 60_000;
 }
-/** Is this scan result a session outside every approved root with no worktree of its own? */
+/**
+ * Is this scan result a session with no worktree of its own that attribution
+ * left unattributed (outside every approved root, no repo signals, or any other
+ * terminal state)? All of them ride the synthetic workspace: a label is never a gate.
+ */
 function isUnclaimedOutsideRootResult(result) {
-    return !result.worktree && rootScopeForReason(result.reason) === "outside";
+    return needsTransportWorkspace(result);
 }
 /** Already durable remotely with the bytes the scan just read: it needs no upload. */
 function isDurableAndUnchanged(result, entry) {
@@ -224,7 +228,8 @@ export async function admitOutsideRootSessions(options) {
         },
         outsideRoots: { workspace, holds, counts },
     };
-    console.error("[session-sync] outside-root raw upload", JSON.stringify({ reason: "outside_root_sessions_admitted", ...counts, ...budget }));
+    const noRepoSignals = admitted.filter((sighting) => NO_WORKSPACE_REASONS.has(sighting.result.reason)).length;
+    console.error("[session-sync] outside-root raw upload", JSON.stringify({ reason: "outside_root_sessions_admitted", no_repo_signals_riding: noRepoSignals, ...counts, ...budget }));
     return labelled;
 }
 /** Session ids and queue turns only; no source paths or transcript content. */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bli-cockpit/cli",
-  "version": "0.2.256",
+  "version": "0.2.257",
   "private": false,
   "type": "module",
   "bin": {