@bli-cockpit/mcp 0.1.50 → 0.1.52

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` |
@@ -57,7 +59,7 @@ three tables below.
 | `cockpit cal week` | `cal_week` | `GET /api/cal/week` |
 | `cockpit careers decide` | `careers_decide` | `POST /api/careers/applications/[id]/decide` |
 | `cockpit careers grade` | `careers_grade` | `POST /api/careers/applications/[id]/grade` |
-| `cockpit careers invite` | `careers_invite` | `POST /api/careers/applications/[id]/invite` |
+| `cockpit careers send-takehome` | `careers_send_takehome` | `POST /api/careers/applications/[id]/invite` |
 | `cockpit careers list` | `careers_list` | `GET /api/careers/applications` |
 | `cockpit careers rescreen` | `careers_rescreen` | `POST /api/careers/applications/[id]/rescreen` |
 | `cockpit careers settings set` | `careers_settings_set` | `PUT /api/careers/settings` |

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/careers-tools.js

@@ -26,14 +26,24 @@ export function registerCareersTools(server, deps) {
             const answer = await callAgentDoor(session, deps.fetchImpl, action === 'show' ? 'GET' : 'POST', `/api/careers/applications/${encodeURIComponent(String(args.id))}${action === 'rescreen' ? '/rescreen' : ''}`, undefined, 60_000);
             return answer.ok ? textResult('Application', answer.body) : errorResult(doorFailureText(`careers_${action}`, answer));
         }));
-    register('careers_invite', {
-        title: 'Send the take-home',
-        description: 'Super-admin only. Sends that role\'s take-home invitation from the configured mailbox, whatever the screening scored. Only a row still at the `screened` stage; a row already invited is refused.',
-        inputSchema: { id: z.string().uuid() },
-    }, async (args) => withSession(deps, async (session) => {
-        const answer = await callAgentDoor(session, deps.fetchImpl, 'POST', `/api/careers/applications/${encodeURIComponent(String(args.id))}/invite`, undefined, 60_000);
-        return answer.ok ? textResult('Invitation sent', answer.body) : errorResult(doorFailureText('careers_invite', answer));
-    }));
+    // Two names, ONE door (BLI-4460 follow-up). Tower's button says "Send
+    // take-home" and this tool said `invite`, so an agent reading the screen and
+    // an agent reading the tool list were looking for different words for the
+    // same act. `careers_send_takehome` is the spelling the button uses and the
+    // one the census and the README teach; `careers_invite` stays registered
+    // because anything that already learned it keeps working.
+    for (const [tool, description] of [
+        ['careers_send_takehome', 'Send the take-home invitation to one applicant (the button labelled Send take-home). Same door as careers_invite. Super-admin only. Sends that role\'s take-home from the configured mailbox, whatever the screening scored. Only a row still at the `screened` stage; a row already invited is refused.'],
+        ['careers_invite', 'Super-admin only. Sends that role\'s take-home invitation from the configured mailbox, whatever the screening scored. Only a row still at the `screened` stage; a row already invited is refused. The same door as careers_send_takehome, which is the name the button uses.'],
+    ])
+        register(tool, {
+            title: 'Send the take-home',
+            description,
+            inputSchema: { id: z.string().uuid() },
+        }, async (args) => withSession(deps, async (session) => {
+            const answer = await callAgentDoor(session, deps.fetchImpl, 'POST', `/api/careers/applications/${encodeURIComponent(String(args.id))}/invite`, undefined, 60_000);
+            return answer.ok ? textResult('Invitation sent', answer.body) : errorResult(doorFailureText(tool, answer));
+        }));
     register('careers_grade', {
         title: 'Re-queue a take-home for grading',
         description: 'Super-admin only. Puts one submitted or graded application back in the take-home grader\'s queue. The grader runs on Railway every 15 minutes because it clones a repository and reads a video; `now` is answered honestly rather than run here.',

package/dist/verb-census.js

@@ -212,7 +212,12 @@ export const MCP_TWINS = {
     "careers list": { tool: "careers_list", door: "GET /api/careers/applications" },
     "careers show": { tool: "careers_show", door: "GET /api/careers/applications/[id]" },
     "careers rescreen": { tool: "careers_rescreen", door: "POST /api/careers/applications/[id]/rescreen" },
-    "careers invite": { tool: "careers_invite", door: "POST /api/careers/applications/[id]/invite" },
+    // The parser's action is `invite`; what a person types is `careers
+    // send-takehome`, the words on Tower's own button, and the tool is named to
+    // match (TYPED_SPELLING below carries that spelling into this row of the
+    // README). `cockpit careers invite` and `careers_invite` are the older words
+    // for the identical door, and both still work.
+    "careers invite": { tool: "careers_send_takehome", door: "POST /api/careers/applications/[id]/invite" },
     "careers decide": { tool: "careers_decide", door: "POST /api/careers/applications/[id]/decide" },
     "careers grade": { tool: "careers_grade", door: "POST /api/careers/applications/[id]/grade" },
     "careers takehome-show": { tool: "careers_takehome_show", door: "GET /api/careers/takehomes/[role]" },
@@ -261,6 +266,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" },
@@ -346,6 +355,9 @@ export const MCP_TWINS = {
  * shorter this map is, the closer the census keys are to the product.
  */
 export const TYPED_SPELLING = {
+    // The action is `invite`; the button, the help wall and the docs all say
+    // send-takehome, so that is what a stranger is told to type.
+    "careers invite": "careers send-takehome",
     "careers takehome-show": "careers takehome show",
     "careers takehome-set": "careers takehome set",
     "careers settings-show": "careers settings show",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bli-cockpit/mcp",
-  "version": "0.1.50",
+  "version": "0.1.52",
   "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",