@bli-cockpit/mcp 0.1.50 → 0.1.51

package/README.md

@@ -34,7 +34,7 @@ about what an agent can reach.
 
 <!-- BEGIN GENERATED verb census — `npm run mcp:readme` -->
 
-**101 of 106 Tower verbs have an MCP twin.**
+**103 of 108 Tower verbs have an MCP twin.**
 Each tool goes through the SAME door its CLI verb calls, with the same
 collector device token — never a second route and never a service-role
 reader. `src/verb-census.test.ts` fails when a verb is in none of the
@@ -42,10 +42,12 @@ three tables below.
 
 | CLI verb | MCP tool | Door |
 | --- | --- | --- |
+| `cockpit brief done` | `brief_done` | `POST /api/jarvis/ticks` |
 | `cockpit brief history` | `brief_history` | `GET /api/jarvis/brief?history=1` |
 | `cockpit brief read` | `brief_read` | `GET /api/jarvis/brief` |
 | `cockpit brief rewrite` | `brief_rewrite` | `POST /api/jarvis/recompile` |
 | `cockpit brief status` | `brief_status` | `GET /api/ops/brief-status` |
+| `cockpit brief undone` | `brief_undone` | `POST /api/jarvis/ticks` |
 | `cockpit cal calendars` | `cal_calendars` | `GET /api/cal/calendars` |
 | `cockpit cal create` | `cal_create` | `POST /api/cal/events` |
 | `cockpit cal detach` | `cal_detach` | `DELETE /api/cal/calendars` |

package/dist/brief-write-tools.js

@@ -175,4 +175,74 @@ export function registerBriefWriteTools(server, deps) {
             link: filed.link ?? null,
         });
     }));
+    // BLI-3605: the board's done checkbox. Two tools rather than one with a
+    // boolean, for the same reason the CLI has two verbs: "mark it done" and
+    // "put it back" are different intentions, and a model picking a flag wrong
+    // silently un-finishes somebody's work.
+    registerTick(register, deps, "brief_done", true);
+    registerTick(register, deps, "brief_undone", false);
+}
+/** The item a tick is about, in the two spellings a board row can have. */
+const TICK_ITEM = z
+    .string()
+    .min(1)
+    .max(200)
+    .describe("The board row: a ticket exactly as the board writes it (BLI-3511), or a claim id from brief_read({claims: true}) for a row with no ticket.");
+const TICKET_IDENTIFIER = /^[A-Z][A-Z0-9]{1,9}-\d+$/;
+/**
+ * One tick tool. `brief_done` and `brief_undone` differ by one boolean and
+ * nothing else, so they are built once here rather than written twice.
+ *
+ * The page is read first and the item looked for in it, exactly as
+ * `brief_correct` refuses an unknown claim id before writing: a key naming
+ * nothing would sit in the table forever, ticked against a row nobody can
+ * find. There is no `subject` argument on purpose, a tick is the caller's own
+ * working state, and the door has no way to accept one for anybody else.
+ */
+function registerTick(register, deps, name, done) {
+    register(name, {
+        title: done
+            ? "Tick one Tower DO THIS FIRST item off as done"
+            : "Put one Tower DO THIS FIRST item back on the board",
+        description: done
+            ? "Marks ONE row of the daily page's DO THIS FIRST board done, the website's checkbox and `cockpit brief done`, "
+                + "same door. A row with a ticket stays ticked across every recompile. Your own board only; a tick is private "
+                + "working state and cannot be made on anybody else's behalf."
+            : "Takes back a tick on ONE row of the DO THIS FIRST board, `cockpit brief undone`, same door. The row goes "
+                + "back to open. Your own board only.",
+        inputSchema: { item: TICK_ITEM },
+    }, async (args) => withSession(deps, async (session) => {
+        const item = String(args.item ?? "").trim();
+        const page = await readPage(deps, session, new URLSearchParams({ claims: "1" }));
+        if (!page.ok)
+            return errorResult(page.text);
+        const claims = page.body.claims ?? [];
+        const isTicket = TICKET_IDENTIFIER.test(item);
+        const line = isTicket
+            ? claims.find((candidate) => (candidate.text ?? "").includes(item))
+            : claims.find((candidate) => candidate.claimId === item);
+        if (!line) {
+            return errorResult(`Refused (unknown_item): nothing on your page is called “${item}”. `
+                + "Call brief_read with `claims: true` to see the ids, or name the ticket exactly as the board writes "
+                + "it. Nothing was recorded.");
+        }
+        const response = await callAgentDoor(session, deps.fetchImpl, "POST", "/api/jarvis/ticks", {
+            item_key: item,
+            ticket_identifier: isTicket ? item : null,
+            claim_id: line.claimId ?? null,
+            page_id: page.body.page?.pageId ?? null,
+            done,
+        }, QUEUE_DEADLINE_MS);
+        if (!response.ok)
+            return errorResult(doorFailureText(name, response));
+        const body = response.body;
+        if (!body.ok)
+            return errorResult(body.reply ?? body.error ?? "Tower did not record that.");
+        return textResult(done ? `[x] ${item} is done.` : `[ ] ${item} is back on the board.`, {
+            ok: true,
+            itemKey: body.item_key ?? item,
+            done,
+            tickedAt: body.ticked_at ?? null,
+        });
+    }));
 }

package/dist/verb-census.js

@@ -261,6 +261,10 @@ export const MCP_TWINS = {
     "brief history": { tool: "brief_history", door: "GET /api/jarvis/brief?history=1" },
     "brief status": { tool: "brief_status", door: "GET /api/ops/brief-status" },
     "brief rewrite": { tool: "brief_rewrite", door: "POST /api/jarvis/recompile" },
+    // BLI-3605: the DO THIS FIRST done checkbox. One door, two verbs, because
+    // "mark it done" and "put it back" are different intentions.
+    "brief done": { tool: "brief_done", door: "POST /api/jarvis/ticks" },
+    "brief undone": { tool: "brief_undone", door: "POST /api/jarvis/ticks" },
     correct: { tool: "brief_correct", door: "POST /api/jarvis/corrections" },
     "notes folders": { tool: "notes_folders", door: "GET /api/notes/folders" },
     "notes mkdir": { tool: "notes_mkdir", door: "POST /api/notes/folders" },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bli-cockpit/mcp",
-  "version": "0.1.50",
+  "version": "0.1.51",
   "private": false,
   "description": "bli-tower: an MCP server over BLI Cockpit's agent doors: JARVIS (jarvis_*), documents (docs_*), channels (msg_*), issues (work_*), the daily page (brief_*), meeting notes (notes_*), the ops board (ops_status/slack_*), settings/team/model, Scout and the workbook, plus the legacy event-stream tools (emit_event, get_ticket_timeline, get_active_tickets).",
   "type": "module",