@bli-cockpit/cli 0.2.282 → 0.2.283

package/dist/commands/local-args-tower-notes.js

@@ -55,6 +55,7 @@ const NOTES_ACTIONS = new Set([
     "bot-upcoming",
     "record",
     "app",
+    "review",
 ]);
 /** Actions whose first positional is the note it acts on. */
 const NOTES_ACTIONS_NEEDING_A_NOTE = new Set([
@@ -110,6 +111,9 @@ const NOTES_ALLOWED_FLAGS = [
     // transcript was stored cut short. A FLAG, not a verb: it is the same sync
     // door with a different request.
     "--repair-truncated",
+    // BLI-22350. `review <note-id> --submit` reads `{ raw_hash, answers }` from stdin; `--accept-all` sends the defaults.
+    "--submit",
+    "--accept-all",
 ];
 const NOTES_VALUE_FLAGS = [
     "--home",
@@ -150,6 +154,7 @@ export function parseNotesArgs(args) {
         throw new Error("notes rename needs --name <name>.");
     const { to, clearShelf } = readNotesShelfTarget(action, values);
     const repairTruncated = readRepairTruncated(action, values);
+    const review = readReviewRequest(action, values, rest);
     const { limit, month, offset } = readNotesWindow(action, values);
     return {
         kind: "notes",
@@ -184,6 +189,7 @@ export function parseNotesArgs(args) {
         ...(month ? { month } : {}),
         ...(offset === undefined ? {} : { offset }),
         ...(repairTruncated ? { repairTruncated: true } : {}),
+        ...review,
         yes,
         json,
     };
@@ -193,7 +199,7 @@ function readNotesAction(first) {
     // almost always wants — same reasoning as bare `cockpit` running convergence.
     const action = (first === undefined ? "list" : first);
     if (!NOTES_ACTIONS.has(action)) {
-        throw new Error(`Unknown notes command: ${first}. Try list, show, shelf, shelves, upload, paste, share, unshare, summarize, move, place, accounts, connect, detach, sync, sync-status, recordings, recording, webhook, granola-status, or transcription-stats (and record start|stop|status|cancel, app install|uninstall|status).`);
+        throw new Error(`Unknown notes command: ${first}. Try list, show, shelf, shelves, upload, paste, share, unshare, summarize, move, place, accounts, connect, detach, sync, sync-status, recordings, recording, webhook, granola-status, transcription-stats, or review (and record start|stop|status|cancel, app install|uninstall|status).`);
     }
     return action;
 }
@@ -220,6 +226,23 @@ function readAppRequest(rest) {
         throw new Error(`notes app ${verb} does not take "${rest[1]}".`);
     return { appAction: verb };
 }
+/** BLI-22350: `review [<note-id>] [--submit|--accept-all]`. Bare lists mine; an id shows one; a flag answers it. */
+function readReviewRequest(action, values, rest) {
+    const flag = values.booleans.has("--submit") ? "--submit" : values.booleans.has("--accept-all") ? "--accept-all" : null;
+    if (action !== "review") {
+        if (flag)
+            throw new Error(`${flag} belongs to \`cockpit notes review <note-id> ${flag}\`.`);
+        return {};
+    }
+    if (rest.length > 1)
+        throw new Error(`notes review takes one note id, not ${rest.length}.`);
+    const noteId = optionalNonEmpty(rest[0]);
+    if (values.booleans.has("--submit") && values.booleans.has("--accept-all"))
+        throw new Error("notes review takes --submit or --accept-all, not both.");
+    if (flag && !noteId)
+        throw new Error(`notes review ${flag} needs a note id: cockpit notes review <note-id> ${flag}.`);
+    return { ...(noteId ? { noteId } : {}), ...(flag === "--submit" ? { submit: true } : {}), ...(flag === "--accept-all" ? { acceptAll: true } : {}) };
+}
 function readRepairTruncated(action, values) {
     const repairTruncated = values.booleans.has("--repair-truncated");
     if (repairTruncated && action !== "sync") {
@@ -328,6 +351,7 @@ function readNotesPaths(action, rest) {
     const takesNoPositional = action !== "paste"
         && action !== "record"
         && action !== "app"
+        && action !== "review"
         && action !== "connect"
         && action !== "sync"
         && action !== "sync-status"

package/dist/commands/local-help-notes.js

@@ -3,7 +3,7 @@ export const NOTES_HELP = [
     [
         "notes",
         [
-            "Usage: cockpit notes [list|show <id>|shelf|shelves|folders|mkdir <path>|rmdir <path>|rename <path>|upload <paths...> [--wait]|paste|share <id>|unshare <id>|summarize <id>|move <id>|accounts|connect|detach <id>|sync <id>|sync-status [<id>]|recordings <note-id>|recording <id>|record start|stop|status|cancel|app install|uninstall|status] [flags]",
+            "Usage: cockpit notes [list|show <id>|shelf|shelves|folders|mkdir <path>|rmdir <path>|rename <path>|upload <paths...> [--wait]|paste|share <id>|unshare <id>|summarize <id>|move <id>|accounts|connect|detach <id>|sync <id>|sync-status [<id>]|recordings <note-id>|recording <id>|record start|stop|status|cancel|app install|uninstall|status|review [<note-id>] [--submit|--accept-all]] [flags]",
             "Audio uploads: MP3, M4A, WAV, WebM and OGG, up to 1 GB. Over 4 MB (a whole meeting) the file goes straight to storage and is transcribed in the background; --wait then polls for up to 4 hours and prints progress, otherwise for at most 330 seconds.",
             "",
             "record start|stop [<session-id>]|status|cancel — record a meeting in a room from this machine's microphone (Apple Silicon macOS and Windows; Linux says it is not supported). On Windows the microphone needs Settings > Privacy & security > Microphone on, including Let desktop apps access your microphone. Nothing leaves the machine while recording; the recorder stops by itself after 8 hours (under the 1 GB upload limit). `stop` uploads the whole original recording, silence included, and Tower writes a note with Speaker 1, Speaker 2... lines, or says no speech was detected (add --wait to watch the transcription). `cancel` stops and KEEPS the file; `status` lists what is recording and every session waiting for upload (`needs_upload`); `stop <session-id>` uploads a cancelled, crashed or auto-stopped session. The BLI Notetaker app uploads waiting sessions by itself. The recording also stays on this machine.",
@@ -42,6 +42,7 @@ export const NOTES_HELP = [
             "bot-upcoming [--json] — your own Google Meet meetings today that the bot will join, with claim state. Decline or mark private to opt out.",
             "granola-status [--limit <n>] [--json] — the Granola cron's own last several runs (ticket 10718): reason, counts, model, spend and which lookback it used. Granola runs on a shared server-side key, not a connected account, so this is its own read rather than `accounts`.",
             "transcription-stats [--month YYYY-MM] [--json] — what meeting transcription cost and did (BLI-18003): minutes, dollars (list and real), requests, failures and fallbacks per provider for the UTC month (default: this one), the spend against the monthly cap and how close the 80% message and the hold are, how many meetings are waiting for budget, which provider keys the deploy holds (by name), and how old the numbers are. Switch providers or move the cap with `cockpit settings switches set transcription.provider|transcription.fallback_chain|transcription.monthly_cap_usd <value>`.",
+            "review [<note-id>] [--submit|--accept-all] [--json] — the meeting reviews waiting for you (BLI-22350). A recorded meeting stays with its attendees until one of them clears one review screen: who's who, lines the raw transcript may have put under the wrong speaker, unclear words, spots where the clean-up may have changed the meaning, sensitive moments. Bare `review` lists yours (waiting, being applied, cleared in the last 7 days). `review <note-id>` shows one in that order with every default answer; --json prints the raw read (raw_hash, items, defaults). `--accept-all` submits every default (names as guessed, lines kept, best-guess readings, the words as said where the meaning may have changed, sensitive moments to attendees only) and prints what it accepted. `--submit` reads { \"raw_hash\": ..., \"answers\": [...] } from stdin: edit the defaults from --json and pipe them in. A refusal says to reload when the review changed (stale_raw_hash, stale_guess) or names what is wrong with the answers. Same door as the web screen (/meeting-notes/<note-id>/review) and the notes_review_* MCP tools.",
             "A large note is read by a model on the server and can take a couple of minutes; the terminal says so before it waits.",
             "--json writes one machine-readable object to stdout; every reason, receipt and progress line stays on stderr.",
             "Sharing and moving need a signed-in session the database can see. If this deployment cannot mint one, they are refused as needs_signed_in_session rather than done with no permission check — and reads say when they came back narrower than the browser's.",

package/dist/commands/notes-review.js

@@ -0,0 +1,204 @@
+/**
+ * `cockpit notes review [<note-id>] [--submit|--accept-all] [--json]` (BLI-22350): the attendee's meeting review in
+ * the terminal. Sibling of `notes.ts`, named in its header. Owner doc: `docs/runbooks/meeting-review-screen.md`.
+ *
+ *   review                       the reviews waiting for me (GET /api/notes/review)
+ *   review <id>                  one review, in the screen's order (GET /api/notes/review/<id>); --json is the raw read
+ *   review <id> --submit         `{ raw_hash, answers }` JSON from stdin (POST /api/notes/review/<id>)
+ *   review <id> --accept-all     read, then submit every item's default answer, and print what it accepted
+ *
+ * The same doors the web screen and the `notes_review_*` MCP tools call; the database decides who is a reviewer. This
+ * file holds no rule about answers: the door checks them and its refusal (reason and sentence) is relayed verbatim.
+ */
+import { isInteractiveStdin, readPipedText, writeLine } from "./cli-io.js";
+import { ask, emit, errorText, fail, READ_DEADLINE_MS } from "./notes-door.js";
+const WRITE_DEADLINE_MS = 60_000;
+/** A review's answers are a few dozen short objects; this is far above any real one. */
+const SUBMIT_MAX_CHARS = 512_000;
+/** The dashboard's `review-words.ts` sentence, repeated here: this package does not import the dashboard. */
+const AS_TRANSCRIBED_LINE = "This meeting is kept as transcribed; only the speakers need confirming.";
+/** The web screen's plain label per meaning-check issue, repeated here for the same reason. */
+const MEANING_ISSUE_WORDS = {
+    number: "A number may have changed",
+    negation_or_pronoun: "A 'not' or a 'we/you' may have flipped",
+    hedge_or_correction: "A 'maybe' or a correction may have been dropped",
+    added_words: "Words may have been added",
+    unclear_made_sensible: "An unclear bit may have been made to sound sensible",
+    speaker_moved: "A line may have been moved to another speaker",
+    passage_left_out: "A passage may have been left out",
+    other: "The meaning may have changed",
+};
+const STATE_WORDS = {
+    queued: "waiting for Tower to prepare it",
+    preparing: "being prepared (transcript cleaned and checked)",
+    awaiting_attendee: "waiting for your answers",
+    applying: "answered; Tower is applying the answers",
+    cleared: "cleared",
+    failed: "stuck; an engineer has been told",
+};
+const reviewPath = (noteId) => `/api/notes/review/${encodeURIComponent(noteId)}`;
+export async function runNotesReview(command, door) {
+    if (!command.noteId)
+        return listReviews(door);
+    if (command.submit)
+        return submitFromStdin(command.noteId, door);
+    if (command.acceptAll)
+        return acceptAll(command.noteId, door);
+    return showReview(command.noteId, door);
+}
+async function listReviews(door) {
+    const answer = await ask(door, { path: "/api/notes/review", method: "GET", label: "notes review", timeoutMs: READ_DEADLINE_MS });
+    if (!answer.ok)
+        return fail(door, answer.reason, answer.detail);
+    const body = answer.body;
+    if (door.json)
+        return emit(door, { ok: true, ...body });
+    const out = door.io.stdout;
+    if (body.reviews.length === 0) {
+        writeLine(out, "No meeting reviews for you: nothing waits, and nothing was cleared in the last 7 days.");
+        return 0;
+    }
+    writeLine(out, `${body.waiting} meeting(s) wait for your review.`);
+    for (const row of body.reviews) {
+        writeLine(out, `${row.note_id}  ${row.meeting_date}  ${row.item_count} item(s)  ${STATE_WORDS[row.state] ?? row.state}  ${row.note_title ?? "(untitled meeting)"}`);
+    }
+    writeLine(out, `Open one: cockpit notes review <note-id>, or ${door.dashboardUrl}/meeting-notes/<note-id>/review in a browser.`);
+    return 0;
+}
+async function readOne(noteId, door) {
+    return ask(door, { path: reviewPath(noteId), method: "GET", label: "notes review", timeoutMs: READ_DEADLINE_MS });
+}
+function lines(value) {
+    if (value === null || value === undefined)
+        return [];
+    if (typeof value === "string")
+        return value.trim() ? [value.trim()] : [];
+    if (Array.isArray(value))
+        return value.flatMap((entry) => (typeof entry === "string" ? [entry] : [JSON.stringify(entry)]));
+    return [JSON.stringify(value)];
+}
+/**
+ * The review in the screen's order: who's who, lines under the wrong speaker, unclear words, meaning checks,
+ * sensitive moments.
+ */
+/** A meaning check's stretch is one or more transcript lines: the label on the first, the rest indented under it. */
+function stretch(label, text) {
+    const [first = "", ...rest] = text.split(/\r?\n/);
+    const pad = " ".repeat(label.length + 5);
+    return [`    ${label} ${first}`, ...rest.map((line) => `${pad}${line}`)];
+}
+export function renderReview(body, dashboardUrl) {
+    const out = [];
+    out.push(`${body.note_title ?? "Meeting"}${body.meeting_date ? `, ${body.meeting_date}` : ""}: ${STATE_WORDS[body.state] ?? body.state}.`);
+    const of = (kind) => body.items.filter((item) => item.kind === kind);
+    const speakers = of("speaker");
+    if (speakers.length)
+        out.push("", "Who's who (the name you confirm or correct):");
+    for (const item of speakers) {
+        out.push(`  ${item.label}: ${item.guess || "(no guess; leave empty to keep the label)"} [${item.id}]`, `    why: ${item.reasoning}`);
+        for (const quote of item.quotes ?? [])
+            out.push(`    ${quote.timestamp} "${quote.text}"`);
+    }
+    const moved = of("speaker_line");
+    if (moved.length)
+        out.push("", "Lines that may be under the wrong speaker (default: keep):");
+    for (const item of moved)
+        out.push(`  ${item.timestamp} under ${item.guess || "(no speaker)"}: "${item.quote.text}" [${item.id}]`);
+    const unclear = of("unclear");
+    if (unclear.length)
+        out.push("", "Unclear words (default: the best guess):");
+    for (const item of unclear) {
+        out.push(`  ${item.timestamp} "${item.words}" -> ${item.guess || "(keep marked unclear)"}${item.changesMeaning ? "  CHANGES THE MEANING" : ""} [${item.id}]`, `    context: ${item.context}`);
+    }
+    const meaning = of("meaning_check");
+    if (meaning.length)
+        out.push("", "Did the clean-up change the meaning? (default: keep the words as said; answer \"raw\" or \"cleaned\"):");
+    for (const item of meaning) {
+        out.push(`  ${item.timestamp} ${MEANING_ISSUE_WORDS[item.issue] ?? MEANING_ISSUE_WORDS.other} [${item.id}]`);
+        if (item.said || item.written)
+            out.push(`    "${item.said}" -> "${item.written}"`);
+        out.push(...stretch("as said:      ", item.raw), ...stretch("as cleaned up:", item.cleaned));
+    }
+    const sensitive = of("sensitive");
+    if (sensitive.length)
+        out.push("", "Sensitive moments (default: attendees only):");
+    for (const item of sensitive)
+        out.push(`  ${item.timestamp} ${item.what} [${item.id}]`);
+    if (body.as_transcribed)
+        out.push("", AS_TRANSCRIBED_LINE);
+    else {
+        for (const [heading, value] of [["Summary", body.summary], ["Decisions", body.decisions], ["Action items", body.action_items]]) {
+            const said = lines(value);
+            if (said.length)
+                out.push("", `${heading}:`, ...said.map((line) => `  ${line}`));
+        }
+    }
+    if (body.state === "cleared" && body.attendee_notes)
+        out.push("", "Kept for the attendees only:", ...body.attendee_notes.split("\n").map((line) => `  ${line}`));
+    if (body.state === "awaiting_attendee") {
+        out.push("", `Answer: cockpit notes review ${body.note_id} --accept-all (every default above), or edit the defaults from`, `cockpit notes review ${body.note_id} --json and pipe { "raw_hash", "answers" } to cockpit notes review ${body.note_id} --submit.`, `Or open ${dashboardUrl}/meeting-notes/${body.note_id}/review.`);
+    }
+    return out;
+}
+async function showReview(noteId, door) {
+    const answer = await readOne(noteId, door);
+    if (!answer.ok)
+        return fail(door, answer.reason, answer.detail);
+    if (door.json)
+        return emit(door, answer.body);
+    for (const line of renderReview(answer.body, door.dashboardUrl))
+        writeLine(door.io.stdout, line);
+    return 0;
+}
+async function post(noteId, door, body, accepted) {
+    const answer = await ask(door, { path: reviewPath(noteId), method: "POST", label: "notes review submit", timeoutMs: WRITE_DEADLINE_MS, body });
+    if (!answer.ok)
+        return fail(door, answer.reason, answer.detail);
+    const done = answer.body;
+    if (door.json)
+        return emit(door, { ok: true, ...done, ...(accepted ? { accepted } : {}) });
+    writeLine(door.io.stdout, `Submitted ${done.answers} answer(s). Tower is applying them; the meeting is released to the team once its privacy check passes.`);
+    return 0;
+}
+async function submitFromStdin(noteId, door) {
+    if (isInteractiveStdin(door.io)) {
+        return fail(door, "nothing_piped", `Pipe { "raw_hash": ..., "answers": [...] } in: cockpit notes review ${noteId} --json, edit the defaults, then pipe them to cockpit notes review ${noteId} --submit.`);
+    }
+    let parsed;
+    try {
+        parsed = JSON.parse(await readPipedText(door.io.stdin, { maxChars: SUBMIT_MAX_CHARS, overflowMessage: "The answers piped in are far larger than any review." }));
+    }
+    catch (error) {
+        return fail(door, "invalid_body", `The answers piped in are not JSON (${errorText(error)}). Send { "raw_hash": ..., "answers": [...] }.`);
+    }
+    return post(noteId, door, parsed);
+}
+function acceptedWords(answer) {
+    const value = answer.answer;
+    if (answer.kind === "speaker")
+        return `speaker ${answer.guess || "(no guess)"} -> ${value === "" ? "keep the label" : String(value)}`;
+    if (answer.kind === "speaker_line")
+        return "line stays where it is";
+    if (answer.kind === "unclear") {
+        const reading = value.reading;
+        return reading === null ? "unclear word stays marked" : `unclear word -> ${reading}`;
+    }
+    if (answer.kind === "meaning_check")
+        return value === "cleaned" ? "meaning check -> keep the clean-up's words" : "meaning check -> keep the words as said";
+    return `sensitive moment -> ${String(value) === "keep_in" ? "kept in" : "attendees only"}`;
+}
+async function acceptAll(noteId, door) {
+    const read = await readOne(noteId, door);
+    if (!read.ok)
+        return fail(door, read.reason, read.detail);
+    const review = read.body;
+    if (review.state !== "awaiting_attendee") {
+        return fail(door, "review_not_awaiting_attendee", `This review is ${STATE_WORDS[review.state] ?? review.state}, so there is nothing to answer.`);
+    }
+    if (!door.json) {
+        writeLine(door.io.stdout, `Accepting ${review.defaults.length} default answer(s):`);
+        for (const answer of review.defaults)
+            writeLine(door.io.stdout, `  ${answer.item_id}: ${acceptedWords(answer)}`);
+    }
+    return post(noteId, door, { raw_hash: review.raw_hash, answers: review.defaults }, review.defaults);
+}

package/dist/commands/notes.js

@@ -40,6 +40,8 @@
  *    Notetaker app on this Mac or Windows PC (Mac LaunchAgent: `notes-app-plist.ts`; Windows: `notes-app-windows.ts`).
  *  - `notes-transcription-stats.ts` — BLI-18003: what meeting transcription
  *    cost and did per provider for a month, and the spend against the cap.
+ *  - `notes-review.ts` — BLI-22350: the meeting reviews waiting for me, one
+ *    review, and answering it (`--submit` from stdin, `--accept-all`).
  *
  * Every public name is still importable from this file.
  */
@@ -52,6 +54,7 @@ import { armNotesWebhook, connectNotetaker, detachNotetaker, listNotesAccounts,
 import { showBotUpcoming } from "./notes-bot-upcoming.js";
 import { showGranolaStatus } from "./notes-granola-status.js";
 import { showTranscriptionStats } from "./notes-transcription-stats.js";
+import { runNotesReview } from "./notes-review.js";
 import { listRecordings, showRecording } from "./notes-recordings.js";
 import { defaultRecordDeps, runNotesRecord } from "./notes-record.js";
 import { runNotesApp } from "./notes-app.js";
@@ -94,6 +97,7 @@ const NOTES_VERBS = {
     "bot-upcoming": showBotUpcoming,
     "granola-status": showGranolaStatus,
     "transcription-stats": showTranscriptionStats,
+    review: runNotesReview,
 };
 export async function runNotes(command, io) {
     // BLI-22979: recording is local until `stop` uploads, so only the upload needs the paired session.

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bli-cockpit/cli",
-  "version": "0.2.282",
+  "version": "0.2.283",
   "private": false,
   "type": "module",
   "bin": {
@@ -28,7 +28,7 @@
   },
   "dependencies": {
     "@bli-cockpit/memory-mcp": "0.1.88",
-    "@bli-cockpit/mcp": "0.1.171",
+    "@bli-cockpit/mcp": "0.1.172",
     "@bli-cockpit/telemetry-core": "0.1.94",
     "unpdf": "1.8.1"
   },