@bli-cockpit/cli 0.2.135 → 0.2.140

package/dist/commands/local-help.js

@@ -100,7 +100,7 @@ export function localCommandHelp(command) {
         "  cockpit models [list|show <provider:model>|compare <provider:model> <provider:model> [...]] [--highlight] [--dashboard-url <url>] [--json]",
         "  cockpit scout [board|start|dismiss|undo <experiment-id>] [--days <n>] [--dashboard-url <url>] [--json]",
         "  cockpit ops [status [--job <id>] [--skips] [--memory] [--models] | recompile --person <email|name|id> [--dry-run]] [--dashboard-url <url>] [--json]",
-        "  cockpit slack [coverage [--workspace bli|blue_pearl] [--stale-only] | read [--person <p>] [--channel <c>] [--query <text>] [--since <YYYY-MM-DD>] [--until <YYYY-MM-DD>] [--limit <n>]] [--json]",
+        "  cockpit slack [coverage [--workspace bli|blue_pearl] [--stale-only] [--list] | read [--person <p>] [--channel <c>] [--query <text> [--word]] [--since <YYYY-MM-DD>] [--until <YYYY-MM-DD>] [--limit <n>] [--format compact|full]] [--json]",
         "  cockpit settings [personal [--chat-model <key>] [--brief-model <key>] | switches [set <key> <value>] | models [set --chat <key>] [--memory <id>] | cli-floor [<version>] | env list|set --project <p> --file <f> --content-stdin|delete --id <uuid> [--yes]] [--json]",
         "  cockpit team [members | invite <email> --role <role> [--team-id <uuid>] | role <userId> --role <role> [--yes] | device [list] | device revoke <id|name> --reason <label> [--note \"<text>\"] [--yes]] [--json]",
         "  cockpit workbook [<project> [<doc>]] [--section <id>] [--markdown] [--width <n>] [--dashboard-url <url>] [--json]",
@@ -117,7 +117,7 @@ export function localCommandHelp(command) {
         "  cockpit agent-rules [install|uninstall|status] [--host codex|claude|all] [--workspace <path>] [--json]",
         "  cockpit memory [search \"<question>\"|save|update <id>|forget <id>|profile|install|status|log <win|loss|noise> --store <bli|supermemory|both> \"<reason>\"] [--container <tag>] [--limit <n>] [--no-recent] [--custom-id <id>] [--extract] [--content-stdin] [--reason <label>] [--file <path>] [--dashboard-url <url>] [--dry-run] [--json]",
         "  cockpit clean [--dry-run] [--all-committed] [--reconcile] [--dashboard-url <url>] [--json]",
-        "  cockpit docs [list|tree|read <id|slug>|create --title <t>|update <id>] [--parent <id>|root|--clear-parent] [--query <text>] [--limit <n>] [--visibility org|private] [--file <path>|--body-stdin] [--allow-empty] [--dashboard-url <url>] [--json]",
+        "  cockpit docs [list|tree|read <id|slug>|create --title <t>|update <id>] [--parent <id>|root|--clear-parent] [--query <text> [--word]] [--limit <n>] [--visibility org|private] [--file <path>|--body-stdin] [--allow-empty] [--dashboard-url <url>] [--json]",
         "  cockpit msg [channels|create <name>|dm <email>|read <channel>|send <channel>|thread <id> --channel <channel>] [--private] [--members a@x,b@y] [--description <text>] [--thread <id>] [--limit <n>] [--dashboard-url <url>] [--json]",
         "  cockpit issue [list|show <BLI-id>|create --title <t>|update <BLI-id>|move <BLI-id> <state>|comment <BLI-id>|history <BLI-id>] [--state <s>] [--assignee me|unassigned|<uuid>] [--project <name|id>] [--limit <n>] [--priority 0-4] [--parent <BLI-id>] [--file <path>|--body-stdin] [--dashboard-url <url>] [--json]",
         "  cockpit mail [accounts|add-imap --address <a>|inbox|read <thread>|search \"<words>\"|send --account <id> --to <a> --subject <s>|attachment <id> --out <path>|sync <account>|detach <account>] [--account <id>] [--limit <n>] [--unread] [--label <l>] [--file <path>] [--dashboard-url <url>] [--json]",
@@ -136,6 +136,7 @@ export function localCommandHelp(command) {
         "  cockpit usage people [--by-repo | --by-topic | --by-subject | --by-task] [--min-confidence high|medium|low] [--person <email|me>] [--all] [--detail] [--since <n>d|<n>h|<iso>] [--until <iso>] [--include-automated] [--dashboard-url <url>] [--json]",
         "",
         `Default dashboard: ${DEFAULT_DASHBOARD_URL}. Omit --dashboard-url for normal production use; pass it only for staging/custom dashboards or to force a different pairing.`,
+        "--quiet on any command (or COCKPIT_QUIET=1) drops the `[tower client] responded {...}` trace line from stderr; --json already implies it. A failure still logs — only that success-path trace goes quiet (BLI-10450).",
     ].join("\n");
 }
 /**

package/dist/commands/local.js

@@ -57,6 +57,14 @@ export { assertCollectionRootPersisted } from "./collection-roots.js";
 export { runSelfUpdate, SelfUpdateError, } from "./install-update.js";
 import { runRoots } from "./roots.js";
 export async function runLocalCockpitCli(argv, io = defaultIo()) {
+    // BLI-10450: `--quiet` is a global toggle, stripped here so no per-command
+    // parser's `allowedFlags` needs to learn it, and mirrored onto
+    // `COCKPIT_QUIET` so `tower-client.ts` can react without every call site
+    // threading a flag through. `--json` already implies it (tower-client.ts).
+    if (argv.includes("--quiet")) {
+        argv = argv.filter((token) => token !== "--quiet");
+        process.env.COCKPIT_QUIET = "1";
+    }
     if (isLocalHelpRequest(argv)) {
         writeLine(io.stdout, localCommandHelp(argv[0]));
         return 0;

package/dist/commands/mail.js

@@ -177,7 +177,11 @@ function messageLine(message) {
     const paperclip = message.has_attachments ? "@" : " ";
     const from = message.from_address?.address ?? "(no sender)";
     const day = message.internal_date.slice(0, 16).replace("T", " ");
-    return `${flag}${paperclip} ${day}  ${from.padEnd(30).slice(0, 30)}  ${(message.subject ?? "(no subject)").slice(0, 60)}`;
+    // BLI-10450: a text row carried no thread id, so `mail read` needed a
+    // second call with `--json` just to learn what `search`/`inbox` already
+    // had in hand. `mail read <thread>` takes this verbatim.
+    const thread = message.thread_id ? `  [thread ${message.thread_id}]` : "";
+    return `${flag}${paperclip} ${day}  ${from.padEnd(30).slice(0, 30)}  ${(message.subject ?? "(no subject)").slice(0, 60)}${thread}`;
 }
 async function readThread(command, door) {
     const answer = await askAgentDoor(door, {

package/dist/commands/memory-store.js

@@ -175,6 +175,8 @@ export function runMemoryVerbList(io, json) {
         "The experience log:",
         "  cockpit memory log <win|loss|noise> --store <bli|supermemory|both> \"<reason>\"",
         "",
+        "Search: --limit caps the MEMORIES (default 5, at most 20). Up to 3 Tower pages and",
+        "tickets ride beside them whatever --limit says; for more, use `cockpit search`.",
         "Save accepts repeated --alias names (up to 16 names of 64 characters).",
         "Omit --container and this repository's own memory space is used — the same",
         "space the bli-memory MCP server and its hooks read and write.",
@@ -222,7 +224,15 @@ async function searchMemory(command, door) {
     })}`);
     if (door.json)
         return emitAgentDoor(door, answer.body);
-    if (results.length === 0) {
+    if (results.length === 0 && body.degraded) {
+        // BLI-10448: an empty answer from a search that lost a channel is NOT the
+        // record being silent. `semantic_timed_out` is the store answering on
+        // keywords alone after its vector search ran out of time, and saying
+        // "the record is silent" over it sends a person away from memories that
+        // exist.
+        writeLine(door.io.stdout, `No memories matched on the channels that answered, and one did not (${body.degraded}). Do not read this as the record being silent; ask again.`);
+    }
+    else if (results.length === 0) {
         // The distinction the whole store is built on: silence is not an outage.
         // A failure took the `!answer.ok` branch above and named itself there.
         writeLine(door.io.stdout, "No memories matched. The record is silent on this — it does not mean the store is down.");

package/dist/commands/msg.js

@@ -104,15 +104,26 @@ async function readChannel(command, door) {
     });
     if (!answer.ok)
         return failAgentDoor(door, TAG, answer.reason, answer.detail);
-    const messages = answer.body.messages ?? [];
+    const body = answer.body;
+    const messages = body.messages ?? [];
+    // BLI-10449: whether this channel is Tower's own or a Slack snapshot, and
+    // how fresh — the door's own sentence, printed as sent.
+    const provenance = body.provenance ?? null;
+    const limitApplied = body.limitApplied ?? null;
     if (door.json)
-        return emitAgentDoor(door, { ok: true, channelId: resolved.id, messages });
+        return emitAgentDoor(door, { ok: true, channelId: resolved.id, provenance, limitApplied, messages });
+    if (provenance?.sentence)
+        writeLine(door.io.stdout, provenance.sentence);
+    if (limitApplied !== null) {
+        writeLine(door.io.stdout, `Showing the newest ${limitApplied}: one read returns at most ${limitApplied} messages.`);
+    }
     if (messages.length === 0) {
         writeLine(door.io.stdout, "No messages.");
         return 0;
     }
     for (const message of [...messages].reverse()) {
-        const who = message.agent_label ?? message.user_id;
+        // An imported Slack message has no Tower user; its Slack name is the label.
+        const who = message.agent_label ?? message.user_id ?? message.slack_author_label ?? "someone";
         writeLine(door.io.stdout, `${message.created_at}  ${who}  ${message.content ?? ""}`);
     }
     return 0;

package/dist/commands/notes-reads.js

@@ -4,6 +4,30 @@
  */
 import { writeLine } from "./cli-io.js";
 import { ask, fail, emit, sayScope, READ_DEADLINE_MS } from "./notes-door.js";
+/**
+ * The lines `notes list` prints under the notes: every meeting record the
+ * docs door would show this caller, with the ref `docs read` takes, and which
+ * copy is canonical. Empty when there are none.
+ */
+export function meetingDocumentLines(meeting, readVerb) {
+    if (!meeting || meeting.count === 0)
+        return [];
+    const refOf = new Map(meeting.items.map((item) => [item.id, item.slug ?? item.id]));
+    const copies = meeting.copies > 0 ? `, ${meeting.copies} of them ${meeting.copies === 1 ? "a copy" : "copies"}` : "";
+    const lines = [
+        `Meeting records in Docs, not in Notes (${meeting.count}${copies}). Read one with \`${readVerb} <ref>\`:`,
+    ];
+    for (const item of meeting.items) {
+        const size = `${Math.max(1, Math.round(item.bodyChars / 1000))} KB${item.parts.length > 0 ? ` in ${item.parts.length + 1} parts` : ""}`;
+        const mark = item.canonical
+            ? item.canonicalReason && item.canonicalReason !== "only_copy" ? ` · canonical (${item.canonicalReason})` : ""
+            : ` · copy of ${refOf.get(item.duplicateOf ?? "") ?? item.duplicateOf}`;
+        lines.push(`  ${item.meetingDate ?? "undated   "}  ${refOf.get(item.id)}  ${item.title}  ${item.visibility} · ${item.source} · ${size}${mark}`);
+    }
+    if (meeting.truncated)
+        lines.push("  (the read hit its cap, so there may be more)");
+    return lines;
+}
 function libraryQuery(command) {
     const query = new URLSearchParams();
     if (command.series)
@@ -33,8 +57,11 @@ export async function listNotes(command, door) {
         return emit(door, body);
     sayScope(door, body);
     const series = body.series ?? [];
+    const inDocs = meetingDocumentLines(body.meetingDocuments, "cockpit docs read");
     if (series.length === 0) {
         writeLine(door.io.stdout, "No meeting notes matched.");
+        if (inDocs.length > 0)
+            writeLine(door.io.stdout, ["", ...inDocs].join("\n"));
         return 0;
     }
     for (const one of series) {
@@ -47,6 +74,8 @@ export async function listNotes(command, door) {
     }
     writeLine(door.io.stdout, "");
     writeLine(door.io.stdout, `${body.count ?? 0} note(s)${body.more ? ", and more exist beyond the limit" : ""}.`);
+    if (inDocs.length > 0)
+        writeLine(door.io.stdout, ["", ...inDocs].join("\n"));
     return 0;
 }
 export async function listShelves(command, door) {

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.135");
+    writeLine(io?.stdout ?? process.stdout, "0.2.140");
     return 0;
   }
 

package/dist/commands/slack.js

@@ -29,9 +29,15 @@ export async function runSlack(command, io) {
         : runReadAction(command, io, tower);
 }
 async function runCoverageAction(command, io, tower) {
-    const query = command.workspace ? `?workspace=${encodeURIComponent(command.workspace)}` : "";
+    const params = new URLSearchParams();
+    if (command.workspace)
+        params.set("workspace", command.workspace);
+    // BLI-10450: counts alone could not answer "is there a Kaiben channel?".
+    if (command.list)
+        params.set("list", "1");
+    const query = params.toString();
     const result = await callTower(tower, {
-        path: `/api/ops/slack/coverage${query}`,
+        path: `/api/ops/slack/coverage${query ? `?${query}` : ""}`,
         label: "slack-coverage",
     });
     if (!result.ok) {
@@ -58,7 +64,7 @@ function writeCoverageReply(command, io, body, coverage, workspaces) {
         return;
     }
     const styled = colorEnabled(io);
-    for (const line of coverageLines(coverage, workspaces, command.staleOnly === true, (text) => dim(text, styled))) {
+    for (const line of coverageLines(coverage, workspaces, command.staleOnly === true, command.list === true, (text) => dim(text, styled))) {
         writeLine(io.stdout, line);
     }
 }
@@ -71,27 +77,43 @@ function logCoverageRead(command, io, workspaces) {
         stale: staleTotal,
         open_skips: workspaces.reduce((sum, row) => sum + numberOf(row["openSkipsTotal"]), 0),
         stale_only: command.staleOnly === true,
+        list: command.list === true,
     })}`);
 }
-function coverageLines(coverage, workspaces, staleOnly, paint) {
+function coverageLines(coverage, workspaces, staleOnly, list, paint) {
     const lines = [];
     const staleAfter = numberOf(coverage["staleAfterHours"]);
     for (const workspace of workspaces) {
-        writeWorkspaceCoverageLines(lines, workspace, staleAfter, staleOnly, paint);
+        writeWorkspaceCoverageLines(lines, workspace, staleAfter, staleOnly, list, paint);
     }
     writeCoverageFooter(lines, coverage, paint);
     return lines;
 }
-function writeWorkspaceCoverageLines(lines, workspace, staleAfter, staleOnly, paint) {
+function writeWorkspaceCoverageLines(lines, workspace, staleAfter, staleOnly, list, 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);
+    if (list)
+        writeChannelListLines(lines, workspace, paint);
     writeStaleChannelLines(lines, workspace, staleAfter, paint);
     writeOpenSkipLines(lines, workspace, paint);
     lines.push("");
 }
+/** BLI-10450: `--list` names every channel, not just the per-state counts. */
+function writeChannelListLines(lines, workspace, paint) {
+    const channels = Array.isArray(workspace["channels"])
+        ? workspace["channels"]
+        : [];
+    if (channels.length === 0)
+        return;
+    for (const channel of channels) {
+        const meaning = String(channel["meaning"] ?? "?");
+        const line = `  #${String(channel["name"] ?? "?")}  ${meaning}`;
+        lines.push(meaning === "readable" ? line : paint(line));
+    }
+}
 function writeCoverageFooter(lines, coverage, paint) {
     const orphanChannels = numberOf(coverage["orphanChannels"]);
     if (orphanChannels > 0) {
@@ -175,6 +197,8 @@ function buildReadBody(command) {
         body["channel"] = command.channel;
     if (command.query)
         body["query"] = command.query;
+    if (command.query && command.word)
+        body["word"] = true;
     if (command.since)
         body["since"] = command.since;
     if (command.until)
@@ -183,16 +207,32 @@ function buildReadBody(command) {
         body["limit"] = command.limit;
     return body;
 }
+/**
+ * BLI-10450: every result used to print twice. `read["summary"]` is built by
+ * `apps/dashboard/src/lib/jarvis/chat-v2/tools-slack-render.ts` `renderSummary`
+ * for the JARVIS chat tool, which needs ONE self-contained string — so it
+ * already embeds a header, one bullet line per message, AND the note. This
+ * CLI then rendered the SAME messages a second time as full blocks and
+ * appended the note a THIRD time. Only the header (summary's first line) is
+ * still printed verbatim, because it is the sentence that keeps an empty
+ * answer from reading as "nobody said anything"; the messages are rendered
+ * exactly once, in the format the caller asked for, and the note prints once.
+ */
 function writeReadReply(command, io, payload, read, messages) {
     if (command.json) {
         writeLine(io.stdout, JSON.stringify(payload));
+        return;
+    }
+    const styled = colorEnabled(io);
+    const summary = typeof read["summary"] === "string" ? read["summary"] : "";
+    const header = summary.split("\n")[0] ?? "";
+    if (header)
+        writeLine(io.stdout, header);
+    if (command.format === "compact") {
+        for (const message of messages)
+            writeLine(io.stdout, compactMessageLine(message));
     }
     else {
-        const styled = colorEnabled(io);
-        // The server's own summary, first and verbatim: it is the sentence that
-        // keeps an empty answer from reading as "nobody said anything".
-        if (typeof read["summary"] === "string")
-            writeLine(io.stdout, read["summary"]);
         if (messages.length > 0)
             writeLine(io.stdout, "");
         for (const message of messages) {
@@ -202,11 +242,15 @@ function writeReadReply(command, io, payload, read, messages) {
                 writeLine(io.stdout, dim(`  ${message["permalink"]}`, styled));
             }
         }
-        if (typeof read["note"] === "string" && read["note"]) {
-            writeLine(io.stdout, "");
-            writeLine(io.stdout, dim(read["note"], styled));
-        }
     }
+    if (typeof read["note"] === "string" && read["note"]) {
+        writeLine(io.stdout, "");
+        writeLine(io.stdout, dim(read["note"], styled));
+    }
+}
+function compactMessageLine(message) {
+    return `- #${String(message["channel"] ?? "?")} — ${String(message["author"] ?? "?")} — `
+        + `${String(message["messageTs"] ?? "?")} — "${String(message["text"] ?? "")}" — ${String(message["permalink"] ?? "")}`;
 }
 function logReadResult(command, io, read, messages) {
     const coverage = asRecord(read["coverage"]);

package/dist/tower-client.js

@@ -29,6 +29,20 @@ import { normalizeDashboardUrl, readResponseJson, responseErrorMessage, } from "
 const defaultLog = (line) => {
     console.error(line);
 };
+/**
+ * BLI-10450: every call printed `[tower client] responded {...}` on stderr,
+ * which mixed into content whenever a caller redirected `2>&1` — the exact
+ * shape an agent piping `cockpit docs read ... 2>&1` hit. The line is a
+ * per-request trace, not a decision (the failure branch below is the one the
+ * repo's "never fail silently" rule actually protects), so it is the one
+ * allowed to go quiet: under `--json` (the line is never meant for a person
+ * reading machine output), a bare `--quiet` (stripped by the router in
+ * `local.ts` and mirrored onto `COCKPIT_QUIET` so this module needs no extra
+ * plumbing), or the env var directly.
+ */
+export function shouldSuppressTowerSuccessLog(argv = process.argv) {
+    return argv.includes("--json") || Boolean(process.env.COCKPIT_QUIET);
+}
 /**
  * Loads the paired device session, or explains in one sentence why it cannot.
  *
@@ -82,13 +96,21 @@ export async function towerRequest(options) {
     try {
         const response = await options.fetch(url, init);
         const respondedMs = Date.now() - startedAt;
-        log(`[tower client] responded ${JSON.stringify({
-            request: options.label,
-            http_status: response.status,
-            content_type: response.headers.get("content-type") ?? "none",
-            streamed: (response.headers.get("content-type") ?? "").includes("application/x-ndjson"),
-            elapsed_ms: respondedMs,
-        })}`);
+        // The success branch always decided something (BLI-10450 kept the
+        // "log the success branch too" rule intact) — it is just that under
+        // `--json`/`--quiet` the reader already gets that decision back as the
+        // command's own stdout object, so a second copy on stderr is noise, not
+        // signal. The failure branch below never quiets: a request that reached
+        // no response at all is the one case AGENTS.md's logging rule protects.
+        if (!shouldSuppressTowerSuccessLog()) {
+            log(`[tower client] responded ${JSON.stringify({
+                request: options.label,
+                http_status: response.status,
+                content_type: response.headers.get("content-type") ?? "none",
+                streamed: (response.headers.get("content-type") ?? "").includes("application/x-ndjson"),
+                elapsed_ms: respondedMs,
+            })}`);
+        }
         return { ok: true, response, startedAt, respondedMs };
     }
     catch (error) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bli-cockpit/cli",
-  "version": "0.2.135",
+  "version": "0.2.140",
   "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-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-package-pack.mjs --workspace=@bli-cockpit/cli"
   },
   "dependencies": {
-    "@bli-cockpit/memory-mcp": "0.1.35",
-    "@bli-cockpit/mcp": "0.1.61",
+    "@bli-cockpit/memory-mcp": "0.1.36",
+    "@bli-cockpit/mcp": "0.1.65",
     "@bli-cockpit/telemetry-core": "0.1.48"
   }
 }