@bli-cockpit/cli 0.2.117 → 0.2.121

package/dist/commands/session-sync-upload.js

@@ -86,17 +86,7 @@ async function syncOneWorktree(options) {
     const attributedSyntheticTarget = options.isDiscoveredWorktree
         ? null
         : worktree;
-    const contextOptions = {
-        homeDir: run.homeDir,
-        repoRoot: worktree.repo_root,
-        activeTicketId: run.activeTicketId,
-        operatorId: run.operatorId,
-        sessionId: run.sessionId,
-        ...(attributedSyntheticTarget ? {} : { branch: run.branch }),
-    };
-    const ensureContext = () => attributedSyntheticTarget
-        ? startLocalWorkContextForAttributedTarget(contextOptions, attributedSyntheticTarget)
-        : startLocalWorkContext(contextOptions);
+    const ensureContext = () => startWorktreeContext(options, attributedSyntheticTarget);
     let context = null;
     if (run.startContexts || attributedSyntheticTarget) {
         context = await ensureContext();
@@ -118,6 +108,20 @@ async function syncOneWorktree(options) {
     }
     return { worktree, context, sync };
 }
+function startWorktreeContext(options, attributedSyntheticTarget) {
+    const { run, worktree } = options;
+    const contextOptions = {
+        homeDir: run.homeDir,
+        repoRoot: worktree.repo_root,
+        activeTicketId: run.activeTicketId,
+        operatorId: run.operatorId,
+        sessionId: run.sessionId,
+        ...(attributedSyntheticTarget ? {} : { branch: run.branch }),
+    };
+    return attributedSyntheticTarget
+        ? startLocalWorkContextForAttributedTarget(contextOptions, attributedSyntheticTarget)
+        : startLocalWorkContext(contextOptions);
+}
 /**
  * Assemble what one worktree is about to upload: its sibling worktree
  * inventory, its Codex mains, its Claude mains and sidecars, and its share of

package/dist/commands/sessions.js

@@ -21,9 +21,45 @@ export async function runSessions(command, io) {
     const homeDir = command.homeDir ?? os.homedir();
     const worktrees = await discoverCommandWorktrees(command.repoRoot, { maxDepth: command.maxDepth, maxRepos: command.maxRepos, homeDir: command.homeDir }, io);
     const window = await sessionsScanWindow(command, now);
-    const wantCodex = command.source !== "claude";
-    const wantClaude = command.source !== "codex";
-    const codex = wantCodex
+    const { codex, claude } = await scanSessionSources(command, homeDir, worktrees, now, window);
+    const { codexRows, claudeRows } = sessionRows(codex, claude);
+    // BLI-3483: the scanners have always counted the folders and files they
+    // could not read, and this command has always thrown those counts away
+    // except for one field in `--json`. So "No sessions observed" was printed on
+    // a machine where the scan had been blocked from looking — the operator's
+    // one diagnostic tool answering the question with the failure removed.
+    const readFailures = countReadFailures(codex, claude);
+    if (command.json) {
+        return writeSessionJson(io, window, readFailures, codex, claude, codexRows, claudeRows);
+    }
+    return writeSessionText(io, window, readFailures, codexRows, claudeRows);
+}
+function writeSessionText(io, window, readFailures, codexRows, claudeRows) {
+    writeLine(io.stdout, "Tower sessions (read-only attribution)");
+    writeLine(io.stdout, `window: ${sessionsWindowLine(window)}`);
+    for (const row of codexRows) {
+        writeLine(io.stdout, sessionRowLine(row));
+    }
+    for (const row of claudeRows) {
+        writeLine(io.stdout, sessionRowLine(row));
+        for (const sidecar of row.sidecar_skips) {
+            writeLine(io.stdout, `    sidecar ${sidecar.file_name}: ${sidecar.reason}`);
+        }
+    }
+    if (codexRows.length === 0 && claudeRows.length === 0) {
+        writeLine(io.stdout, readFailures.total > 0
+            ? `No sessions listed, but ${readFailures.total} read failure(s) mean the scan could not see everywhere — this is not proof there were none.`
+            : "No sessions observed in the scan window.");
+    }
+    if (readFailures.total > 0) {
+        for (const line of readFailureLines(readFailures)) {
+            writeLine(io.stdout, line);
+        }
+    }
+    return 0;
+}
+async function scanSessionSources(command, homeDir, worktrees, now, window) {
+    const codex = command.source !== "claude"
         ? await scanAndAttributeCodexSessions({
             sessionsDirs: defaultCodexSessionDirs(homeDir),
             worktrees,
@@ -32,7 +68,7 @@ export async function runSessions(command, io) {
             limit: window.limit,
         })
         : null;
-    const claude = wantClaude
+    const claude = command.source !== "codex"
         ? await scanAndAttributeClaudeSessions({
             projectsDir: path.join(homeDir, ".claude", "projects"),
             worktrees,
@@ -41,6 +77,26 @@ export async function runSessions(command, io) {
             limit: window.mode === "default" ? undefined : window.limit,
         })
         : null;
+    return { codex, claude };
+}
+function writeSessionJson(io, window, readFailures, codex, claude, codexRows, claudeRows) {
+    writeLine(io.stdout, JSON.stringify({
+        window,
+        read_failures: readFailures,
+        ...(codex ? { codex: { counts: codex.counts, sessions: codexRows } } : {}),
+        ...(claude
+            ? {
+                claude: {
+                    counts: claude.counts,
+                    project_dirs_skipped: claude.project_dirs_skipped,
+                    sessions: claudeRows,
+                },
+            }
+            : {}),
+    }, null, 2));
+    return 0;
+}
+function sessionRows(codex, claude) {
     // Safe output contract (B.4 §6): id, state, reason, scores, signals, sidecar
     // skip reasons, plus the repo_label basename. Branch is intentionally omitted
     // — branch names can carry operator-authored task/customer text.
@@ -71,53 +127,7 @@ export async function runSessions(command, io) {
             reason: sidecar.skipped_reason,
         })),
     }));
-    // BLI-3483: the scanners have always counted the folders and files they
-    // could not read, and this command has always thrown those counts away
-    // except for one field in `--json`. So "No sessions observed" was printed on
-    // a machine where the scan had been blocked from looking — the operator's
-    // one diagnostic tool answering the question with the failure removed.
-    const readFailures = countReadFailures(codex, claude);
-    if (command.json) {
-        writeLine(io.stdout, JSON.stringify({
-            window,
-            read_failures: readFailures,
-            ...(codex
-                ? { codex: { counts: codex.counts, sessions: codexRows } }
-                : {}),
-            ...(claude
-                ? {
-                    claude: {
-                        counts: claude.counts,
-                        project_dirs_skipped: claude.project_dirs_skipped,
-                        sessions: claudeRows,
-                    },
-                }
-                : {}),
-        }, null, 2));
-        return 0;
-    }
-    writeLine(io.stdout, "Tower sessions (read-only attribution)");
-    writeLine(io.stdout, `window: ${sessionsWindowLine(window)}`);
-    for (const row of codexRows) {
-        writeLine(io.stdout, sessionRowLine(row));
-    }
-    for (const row of claudeRows) {
-        writeLine(io.stdout, sessionRowLine(row));
-        for (const sidecar of row.sidecar_skips) {
-            writeLine(io.stdout, `    sidecar ${sidecar.file_name}: ${sidecar.reason}`);
-        }
-    }
-    if (codexRows.length === 0 && claudeRows.length === 0) {
-        writeLine(io.stdout, readFailures.total > 0
-            ? `No sessions listed, but ${readFailures.total} read failure(s) mean the scan could not see everywhere — this is not proof there were none.`
-            : "No sessions observed in the scan window.");
-    }
-    if (readFailures.total > 0) {
-        for (const line of readFailureLines(readFailures)) {
-            writeLine(io.stdout, line);
-        }
-    }
-    return 0;
+    return { codexRows, claudeRows };
 }
 /**
  * Everything the two scanners looked at and could not read. Counts only — the

package/dist/commands/slack.js

@@ -46,15 +46,23 @@ async function runCoverageAction(command, io, tower) {
     const workspaces = Array.isArray(coverage["workspaces"])
         ? coverage["workspaces"]
         : [];
+    writeCoverageReply(command, io, body, coverage, workspaces);
+    logCoverageRead(command, io, workspaces);
+    // A stale cursor is a finding, not a failure of this command: exit 0 so a
+    // person reading coverage is not told their own read broke.
+    return 0;
+}
+function writeCoverageReply(command, io, body, coverage, workspaces) {
     if (command.json) {
         writeLine(io.stdout, JSON.stringify(body));
+        return;
     }
-    else {
-        const styled = colorEnabled(io);
-        for (const line of coverageLines(coverage, workspaces, command.staleOnly === true, (text) => dim(text, styled))) {
-            writeLine(io.stdout, line);
-        }
+    const styled = colorEnabled(io);
+    for (const line of coverageLines(coverage, workspaces, command.staleOnly === true, (text) => dim(text, styled))) {
+        writeLine(io.stdout, line);
     }
+}
+function logCoverageRead(command, io, workspaces) {
     const staleTotal = workspaces.reduce((sum, row) => sum + numberOf(row["staleChannelsTotal"]), 0);
     writeLine(io.stderr, `[slack cli] coverage read ${JSON.stringify({
         workspace: command.workspace ?? "all",
@@ -64,75 +72,74 @@ async function runCoverageAction(command, io, tower) {
         open_skips: workspaces.reduce((sum, row) => sum + numberOf(row["openSkipsTotal"]), 0),
         stale_only: command.staleOnly === true,
     })}`);
-    // A stale cursor is a finding, not a failure of this command: exit 0 so a
-    // person reading coverage is not told their own read broke.
-    return 0;
 }
 function coverageLines(coverage, workspaces, staleOnly, paint) {
     const lines = [];
     const staleAfter = numberOf(coverage["staleAfterHours"]);
     for (const workspace of workspaces) {
-        const key = String(workspace["workspace"] ?? "?");
-        const known = numberOf(workspace["channelsKnown"]);
-        const covered = numberOf(workspace["covered"]);
-        lines.push(`${key.toUpperCase()}  ${covered} of ${known} channels readable`);
-        if (!staleOnly) {
-            const byVisibility = Array.isArray(workspace["byVisibility"])
-                ? workspace["byVisibility"]
-                : [];
-            for (const entry of byVisibility) {
-                lines.push(paint(`  ${String(numberOf(entry["count"])).padStart(4)}  ${String(entry["state"] ?? "?")} — ${String(entry["meaning"] ?? "?")}`));
-            }
-            lines.push(paint(`  newest cursor: ${String(workspace["newestCursorIso"] ?? "never")}`));
-        }
-        const stale = Array.isArray(workspace["staleChannels"])
-            ? workspace["staleChannels"]
-            : [];
-        const staleTotal = numberOf(workspace["staleChannelsTotal"]);
-        if (staleTotal === 0) {
-            lines.push(paint(`  no channel is over ${staleAfter}h since its last sync`));
-        }
-        else {
-            lines.push(`  ${staleTotal} channel${staleTotal === 1 ? "" : "s"} over ${staleAfter}h since last sync:`);
-            for (const entry of stale) {
-                lines.push(paint(`    ${String(entry["name"] ?? "?")}  last ${String(entry["lastSyncedAt"] ?? "never")}`));
-            }
-            if (stale.length < staleTotal)
-                lines.push(paint(`    …and ${staleTotal - stale.length} more`));
-        }
-        const skips = Array.isArray(workspace["openSkips"])
-            ? workspace["openSkips"]
-            : [];
-        if (skips.length > 0) {
-            lines.push(`  open ingest skips:`);
-            for (const skip of skips) {
-                lines.push(paint(`    ${String(numberOf(skip["count"])).padStart(4)}  ${String(skip["reason"] ?? "?")}`));
-            }
-        }
-        lines.push("");
+        writeWorkspaceCoverageLines(lines, workspace, staleAfter, staleOnly, paint);
     }
+    writeCoverageFooter(lines, coverage, paint);
+    return lines;
+}
+function writeWorkspaceCoverageLines(lines, workspace, staleAfter, staleOnly, paint) {
+    const key = String(workspace["workspace"] ?? "?");
+    const known = numberOf(workspace["channelsKnown"]);
+    const covered = numberOf(workspace["covered"]);
+    lines.push(`${key.toUpperCase()}  ${covered} of ${known} channels readable`);
+    writeVisibilityLines(lines, workspace, staleOnly, paint);
+    writeStaleChannelLines(lines, workspace, staleAfter, paint);
+    writeOpenSkipLines(lines, workspace, paint);
+    lines.push("");
+}
+function writeCoverageFooter(lines, coverage, paint) {
     const orphanChannels = numberOf(coverage["orphanChannels"]);
     if (orphanChannels > 0) {
         lines.push(paint(`${orphanChannels} channel(s) belong to no known workspace and are not counted above.`));
     }
     if (typeof coverage["note"] === "string")
         lines.push(paint(coverage["note"]));
-    return lines;
+}
+function writeVisibilityLines(lines, workspace, staleOnly, paint) {
+    if (staleOnly)
+        return;
+    const byVisibility = Array.isArray(workspace["byVisibility"])
+        ? workspace["byVisibility"]
+        : [];
+    for (const entry of byVisibility) {
+        lines.push(paint(`  ${String(numberOf(entry["count"])).padStart(4)}  ${String(entry["state"] ?? "?")} — ${String(entry["meaning"] ?? "?")}`));
+    }
+    lines.push(paint(`  newest cursor: ${String(workspace["newestCursorIso"] ?? "never")}`));
+}
+function writeStaleChannelLines(lines, workspace, staleAfter, paint) {
+    const stale = Array.isArray(workspace["staleChannels"])
+        ? workspace["staleChannels"]
+        : [];
+    const staleTotal = numberOf(workspace["staleChannelsTotal"]);
+    if (staleTotal === 0) {
+        lines.push(paint(`  no channel is over ${staleAfter}h since its last sync`));
+        return;
+    }
+    lines.push(`  ${staleTotal} channel${staleTotal === 1 ? "" : "s"} over ${staleAfter}h since last sync:`);
+    for (const entry of stale) {
+        lines.push(paint(`    ${String(entry["name"] ?? "?")}  last ${String(entry["lastSyncedAt"] ?? "never")}`));
+    }
+    if (stale.length < staleTotal)
+        lines.push(paint(`    …and ${staleTotal - stale.length} more`));
+}
+function writeOpenSkipLines(lines, workspace, paint) {
+    const skips = Array.isArray(workspace["openSkips"])
+        ? workspace["openSkips"]
+        : [];
+    if (skips.length === 0)
+        return;
+    lines.push(`  open ingest skips:`);
+    for (const skip of skips) {
+        lines.push(paint(`    ${String(numberOf(skip["count"])).padStart(4)}  ${String(skip["reason"] ?? "?")}`));
+    }
 }
 async function runReadAction(command, io, tower) {
-    const body = {};
-    if (command.person)
-        body["person"] = command.person;
-    if (command.channel)
-        body["channel"] = command.channel;
-    if (command.query)
-        body["query"] = command.query;
-    if (command.since)
-        body["since"] = command.since;
-    if (command.until)
-        body["until"] = command.until;
-    if (command.limit !== undefined)
-        body["limit"] = command.limit;
+    const body = buildReadBody(command);
     const result = await callTower(tower, {
         path: "/api/ops/slack/read",
         method: "POST",
@@ -156,6 +163,27 @@ async function runReadAction(command, io, tower) {
     const messages = Array.isArray(read["messages"])
         ? read["messages"]
         : [];
+    writeReadReply(command, io, payload, read, messages);
+    logReadResult(command, io, read, messages);
+    return 0;
+}
+function buildReadBody(command) {
+    const body = {};
+    if (command.person)
+        body["person"] = command.person;
+    if (command.channel)
+        body["channel"] = command.channel;
+    if (command.query)
+        body["query"] = command.query;
+    if (command.since)
+        body["since"] = command.since;
+    if (command.until)
+        body["until"] = command.until;
+    if (command.limit !== undefined)
+        body["limit"] = command.limit;
+    return body;
+}
+function writeReadReply(command, io, payload, read, messages) {
     if (command.json) {
         writeLine(io.stdout, JSON.stringify(payload));
     }
@@ -179,6 +207,8 @@ async function runReadAction(command, io, tower) {
             writeLine(io.stdout, dim(read["note"], styled));
         }
     }
+}
+function logReadResult(command, io, read, messages) {
     const coverage = asRecord(read["coverage"]);
     writeLine(io.stderr, `[slack cli] read ${JSON.stringify({
         status: read["status"] ?? null,
@@ -191,7 +221,6 @@ async function runReadAction(command, io, tower) {
         has_channel: Boolean(command.channel),
         has_query: Boolean(command.query),
     })}`);
-    return 0;
 }
 function numberOf(value) {
     return typeof value === "number" && Number.isFinite(value) ? value : 0;

package/dist/commands/status.js

@@ -29,6 +29,14 @@ import { getCollectorRuntimePaths, inspectLocalCollectorStatus, readLocalCollect
 import { normalizeCollectionRoots } from "../root-normalization.js";
 export async function runStatus(command, io) {
     const backfillCursor = await inspectBackfillCursor(command.homeDir);
+    const setupReceipt = await readMachineSetupReceipt(command.homeDir, io);
+    const worktrees = await discoverCommandWorktrees(command.repoRoot, { maxDepth: command.maxDepth, maxRepos: command.maxRepos, homeDir: command.homeDir }, io);
+    if (worktrees.length > 1) {
+        return writeMultiRepoStatus(command, io, backfillCursor, setupReceipt, worktrees);
+    }
+    return writeSingleRepoStatus(command, io, backfillCursor, setupReceipt);
+}
+async function readMachineSetupReceipt(homeDir, io) {
     // BLI-3731: "is this machine actually connected?" belongs beside "is it
     // collecting?", and it is read back off the host rather than remembered.
     // Refreshing here also keeps the heartbeat's cache warm without the 15-minute
@@ -38,40 +46,43 @@ export async function runStatus(command, io) {
     // about the MACHINE, so it cannot depend on whether the folder you happen to
     // stand in holds one repo or six — and the multi-repo branch, which every
     // agent machine takes, used to return before the receipt was ever built.
-    const setupReceipt = await refreshSetupReceipt(io, {
-        ...(command.homeDir ? { homeDir: command.homeDir } : {}),
+    return refreshSetupReceipt(io, {
+        ...(homeDir ? { homeDir } : {}),
     });
-    const worktrees = await discoverCommandWorktrees(command.repoRoot, { maxDepth: command.maxDepth, maxRepos: command.maxRepos, homeDir: command.homeDir }, io);
-    if (worktrees.length > 1) {
-        const statuses = await Promise.all(worktrees.map(async (worktree) => ({
-            ...(await inspectLocalCollectorStatus({
-                homeDir: command.homeDir,
-                repoRoot: worktree.repo_root,
-            })),
-            head_sha: worktree.head_sha,
-        })));
-        if (command.json) {
-            writeLine(io.stdout, JSON.stringify({
-                mode: "multi_repo",
-                statuses,
-                backfill_cursor: backfillCursor,
-                // Beside `backfill_cursor`, and for the same reason: both are facts
-                // about this machine, so both sit at the top of the payload rather
-                // than once per repo row. Null means this run could not read the
-                // host — `refreshSetupReceipt` named the reason on stderr — never
-                // "this machine is not connected".
-                setup_receipt: setupReceipt?.receipt ?? null,
-            }, null, 2));
-            return 0;
-        }
-        writeLine(io.stdout, "Tower status — every repo below this folder");
-        writeLine(io.stdout, `Old sessions: ${backfillCursorLine(backfillCursor)}`);
-        for (const status of statuses) {
-            writeLine(io.stdout, `- ${status.repo_label ?? status.repo}/${status.worktree_label ?? "worktree"} · ${status.branch} · head:${shortSha(status.head_sha)} · ${status.upload_state}`);
-        }
-        writeConnectedBlock(io, setupReceipt);
-        return 0;
+}
+async function writeMultiRepoStatus(command, io, backfillCursor, setupReceipt, worktrees) {
+    const statuses = await Promise.all(worktrees.map(async (worktree) => ({
+        ...(await inspectLocalCollectorStatus({
+            homeDir: command.homeDir,
+            repoRoot: worktree.repo_root,
+        })),
+        head_sha: worktree.head_sha,
+    })));
+    if (command.json)
+        return writeMultiRepoJson(io, backfillCursor, setupReceipt, statuses);
+    writeLine(io.stdout, "Tower status — every repo below this folder");
+    writeLine(io.stdout, `Old sessions: ${backfillCursorLine(backfillCursor)}`);
+    for (const status of statuses) {
+        writeLine(io.stdout, `- ${status.repo_label ?? status.repo}/${status.worktree_label ?? "worktree"} · ${status.branch} · head:${shortSha(status.head_sha)} · ${status.upload_state}`);
     }
+    writeConnectedBlock(io, setupReceipt);
+    return 0;
+}
+function writeMultiRepoJson(io, backfillCursor, setupReceipt, statuses) {
+    writeLine(io.stdout, JSON.stringify({
+        mode: "multi_repo",
+        statuses,
+        backfill_cursor: backfillCursor,
+        // Beside `backfill_cursor`, and for the same reason: both are facts
+        // about this machine, so both sit at the top of the payload rather
+        // than once per repo row. Null means this run could not read the
+        // host — `refreshSetupReceipt` named the reason on stderr — never
+        // "this machine is not connected".
+        setup_receipt: setupReceipt?.receipt ?? null,
+    }, null, 2));
+    return 0;
+}
+async function writeSingleRepoStatus(command, io, backfillCursor, setupReceipt) {
     // The staging walk is opt-in (BLI-3797) and this is the surface that pays for
     // it: one person asking one question. The multi-repo branch above deliberately
     // does not — the backlog is a MACHINE fact, so N repos would walk one tree N
@@ -80,14 +91,16 @@ export async function runStatus(command, io) {
         ...command,
         includeStagingBacklog: true,
     });
-    if (command.json) {
-        writeLine(io.stdout, JSON.stringify({
-            ...status,
-            backfill_cursor: backfillCursor,
-            setup_receipt: setupReceipt?.receipt ?? null,
-        }, null, 2));
-        return 0;
-    }
+    if (command.json)
+        return writeSingleRepoJson(io, status, backfillCursor, setupReceipt);
+    writeSingleRepoLines(io, status, backfillCursor, setupReceipt);
+    return 0;
+}
+function writeSingleRepoJson(io, status, backfillCursor, setupReceipt) {
+    writeLine(io.stdout, JSON.stringify({ ...status, backfill_cursor: backfillCursor, setup_receipt: setupReceipt?.receipt ?? null }, null, 2));
+    return 0;
+}
+function writeSingleRepoLines(io, status, backfillCursor, setupReceipt) {
     writeLine(io.stdout, "Tower status");
     writeLine(io.stdout, `Installed: ${status.installed ? "yes" : "no"}`);
     writeLine(io.stdout, `Signed in: ${signedInLine(status.session_state)}`);
@@ -117,7 +130,6 @@ export async function runStatus(command, io) {
     for (const detail of status.details)
         writeLine(io.stdout, `- ${detail}`);
     writeConnectedBlock(io, setupReceipt);
-    return 0;
 }
 /**
  * The receipt block, in the words `cockpit doctor` prints, from the one reader

package/dist/commands/workbook.js

@@ -100,26 +100,9 @@ function logIndexRead(io, projects) {
 function writeDoc(command, io, payload, width) {
     const markdown = payload.markdown ?? "";
     const sections = payload.sections ?? [];
-    let selected = markdown;
-    if (command.section) {
-        const slice = sectionSlice(markdown, sections, command.section);
-        if (slice === null) {
-            const known = sections.map((section) => section.id ?? "").filter(Boolean);
-            const sentence = `No section "${command.section}" in this document (${known.join(", ") || "none"}).`;
-            writeLine(io.stderr, `[workbook cli] section missed ${JSON.stringify({
-                reason: "workbook_section_not_found",
-                sections: sections.length,
-            })}`);
-            if (command.json) {
-                writeLine(io.stdout, JSON.stringify({ ok: false, error: "workbook_section_not_found", detail: sentence }));
-            }
-            else {
-                writeLine(io.stderr, sentence);
-            }
-            return 1;
-        }
-        selected = slice;
-    }
+    const selected = selectDocumentSection(command, io, markdown, sections);
+    if (selected === null)
+        return 1;
     if (command.json) {
         writeLine(io.stdout, JSON.stringify({
             ok: true,
@@ -150,6 +133,26 @@ function writeDoc(command, io, payload, width) {
     })}`);
     return 0;
 }
+function selectDocumentSection(command, io, markdown, sections) {
+    if (!command.section)
+        return markdown;
+    const slice = sectionSlice(markdown, sections, command.section);
+    if (slice !== null)
+        return slice;
+    const known = sections.map((section) => section.id ?? "").filter(Boolean);
+    const sentence = `No section "${command.section}" in this document (${known.join(", ") || "none"}).`;
+    writeLine(io.stderr, `[workbook cli] section missed ${JSON.stringify({
+        reason: "workbook_section_not_found",
+        sections: sections.length,
+    })}`);
+    if (command.json) {
+        writeLine(io.stdout, JSON.stringify({ ok: false, error: "workbook_section_not_found", detail: sentence }));
+    }
+    else {
+        writeLine(io.stderr, sentence);
+    }
+    return null;
+}
 // ----------------------------------------------------------------- small parts
 /**
  * The wrap column: an explicit `--width` wins, otherwise the terminal's own

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bli-cockpit/cli",
-  "version": "0.2.117",
+  "version": "0.2.121",
   "private": false,
   "type": "module",
   "bin": {
@@ -28,7 +28,7 @@
   },
   "dependencies": {
     "@bli-cockpit/memory-mcp": "0.1.32",
-    "@bli-cockpit/mcp": "0.1.45",
+    "@bli-cockpit/mcp": "0.1.48",
     "@bli-cockpit/telemetry-core": "0.1.48"
   }
 }