@bli-cockpit/mcp 0.1.28 → 0.1.29

package/README.md

@@ -34,7 +34,7 @@ about what an agent can reach.
 
 <!-- BEGIN GENERATED verb census — `npm run mcp:readme` -->
 
-**77 of 80 Tower verbs have an MCP twin.**
+**81 of 84 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
@@ -78,7 +78,11 @@ three tables below.
 | `cockpit mail search` | `mail_search` | `GET /api/mail/search` |
 | `cockpit mail send` | `mail_send` | `POST /api/mail/send` |
 | `cockpit mail sync` | `mail_sync` | `POST /api/mail/accounts/[id]/sync` |
+| `cockpit memory forget` | `forget_memory` (`bli-memory`) | `POST /api/memory/forget` |
 | `cockpit memory log` | `memory_experience` | `POST /api/memory/experience` |
+| `cockpit memory save` | `save_memory` (`bli-memory`) | `POST /api/memory/save` |
+| `cockpit memory search` | `search_memory` (`bli-memory`) | `POST /api/memory/search` |
+| `cockpit memory update` | `update_memory` (`bli-memory`) | `POST /api/memory/update` |
 | `cockpit model set` | `model_set` | `POST /api/settings/jarvis-model` |
 | `cockpit model show` | `model_show` | `GET /api/settings/jarvis-model` |
 | `cockpit models compare` | `models_compare` | `GET /api/models/cards?compare=` |

package/dist/readme-census.js

@@ -26,7 +26,10 @@ export function renderCensusMarkdown() {
         .filter((verb) => MCP_TWINS[verb])
         .map((verb) => {
         const twin = MCP_TWINS[verb];
-        return `| ${spell(verb)} | \`${twin.tool}\` | \`${twin.door}\` |`;
+        // BLI-4047: two MCP servers ship. A tool cell that did not say which one
+        // registers it would send a reader to `bli-tower` for `search_memory`.
+        const server = twin.server && twin.server !== "bli-tower" ? ` (\`${twin.server}\`)` : "";
+        return `| ${spell(verb)} | \`${twin.tool}\`${server} | \`${twin.door}\` |`;
     });
     const owedRows = verbs
         .filter((verb) => AWAITING_TWIN[verb])

package/dist/verb-census.d.ts

@@ -39,11 +39,31 @@ export interface TowerVerb {
 export declare function towerNouns(): string[];
 /** Every Tower verb a person may type today. */
 export declare function towerVerbs(): TowerVerb[];
+/**
+ * The tools the SECOND MCP server registers (BLI-4047).
+ *
+ * BLI Memory ships its own server, `bli-memory`
+ * (`packages/cockpit-memory-mcp`), because its hooks and its session handling
+ * are its own; the four memory tools live there and not on `bli-tower`. This
+ * package declares no dependency on it, so — exactly like the collector scan
+ * above — the names are read out of its source. A text scan is the weaker
+ * check, and it is still strictly better than a census that calls four verbs
+ * twinned against tools nothing verifies exist.
+ */
+export declare const MEMORY_MCP_SERVER_FILE: string;
+export declare function memoryMcpToolNames(): string[];
 export interface McpTwin {
     /** The MCP tool name. Must actually be registered — the suite checks. */
     tool: string;
     /** The API door it goes through: the SAME one the CLI verb calls. */
     door: string;
+    /**
+     * Which MCP server registers it. Absent means `bli-tower`, this package.
+     * BLI-4047: `bli-memory` is the second server we ship, and a twin naming a
+     * tool on it while the suite only ever read `bli-tower`'s registry would be
+     * a claim nothing checks.
+     */
+    server?: "bli-tower" | "bli-memory";
 }
 /** A CLI verb and the MCP tool that does the same thing, over the same door. */
 export declare const MCP_TWINS: Record<string, McpTwin>;

package/dist/verb-census.js

@@ -138,7 +138,10 @@ function resolveShapeAlias(variant, unionSource) {
 export function towerVerbs() {
     const nouns = new Set(towerNouns());
     const aliases = actionAliases();
-    const verbs = [{ noun: "memory", action: "log", spelling: "memory log" }];
+    const verbs = [
+        { noun: "memory", action: "log", spelling: "memory log" },
+        ...memoryStoreVerbs(aliases),
+    ];
     const seenNouns = new Set();
     for (const variant of commandVariants()) {
         const kind = /kind:\s*"([a-z-]+)"/.exec(variant)?.[1];
@@ -169,6 +172,41 @@ export function towerVerbs() {
     }
     return verbs.sort((a, b) => a.spelling.localeCompare(b.spelling));
 }
+/**
+ * `cockpit memory search|save|update|forget` (BLI-4047).
+ *
+ * `memory` is a COLLECTOR noun — `parseMemoryArgs` lives in
+ * `local-args-collector-status.ts` and is not re-exported from
+ * `local-args-tower.ts`, because `install` and `status` act on THIS MACHINE.
+ * Four of its verbs act on Tower, so the mechanical noun scan cannot see them
+ * and this reads them off `MemoryStoreAction` instead. Deliberately NOT a
+ * hardcoded list of four: a fifth store verb still has to face the census and
+ * name its MCP twin.
+ */
+function memoryStoreVerbs(aliases) {
+    const actions = aliases.get("MemoryStoreAction") ?? [];
+    if (actions.length === 0) {
+        throw new Error("MemoryStoreAction not found in the collector's local-args-tower-* files — "
+            + "`cockpit memory`'s Tower verbs would drop out of the census silently.");
+    }
+    return actions.map((action) => ({ noun: "memory", action, spelling: `memory ${action}` }));
+}
+/**
+ * The tools the SECOND MCP server registers (BLI-4047).
+ *
+ * BLI Memory ships its own server, `bli-memory`
+ * (`packages/cockpit-memory-mcp`), because its hooks and its session handling
+ * are its own; the four memory tools live there and not on `bli-tower`. This
+ * package declares no dependency on it, so — exactly like the collector scan
+ * above — the names are read out of its source. A text scan is the weaker
+ * check, and it is still strictly better than a census that calls four verbs
+ * twinned against tools nothing verifies exist.
+ */
+export const MEMORY_MCP_SERVER_FILE = resolve(HERE, "../../cockpit-memory-mcp/src/server.ts");
+export function memoryMcpToolNames() {
+    const source = readFileSync(MEMORY_MCP_SERVER_FILE, "utf8");
+    return [...source.matchAll(/registerTool\(\s*"([a-z_]+)"/g)].map((match) => match[1]);
+}
 /** A CLI verb and the MCP tool that does the same thing, over the same door. */
 export const MCP_TWINS = {
     "docs list": { tool: "docs_list", door: "GET /api/docs/documents" },
@@ -222,6 +260,12 @@ export const MCP_TWINS = {
     "notes unshare": { tool: "notes_unshare", door: "POST /api/notes/share (share=false)" },
     "notes move": { tool: "notes_move", door: "POST /api/notes/move" },
     "memory log": { tool: "memory_experience", door: "POST /api/memory/experience" },
+    // BLI-4047: the memory STORE at a terminal. The twins live on the second
+    // server, `bli-memory`, over the same four doors these CLI verbs call.
+    "memory search": { tool: "search_memory", door: "POST /api/memory/search", server: "bli-memory" },
+    "memory save": { tool: "save_memory", door: "POST /api/memory/save", server: "bli-memory" },
+    "memory update": { tool: "update_memory", door: "POST /api/memory/update", server: "bli-memory" },
+    "memory forget": { tool: "forget_memory", door: "POST /api/memory/forget", server: "bli-memory" },
     "ops status": { tool: "ops_status", door: "GET /api/ops/status" },
     "ops recompile": { tool: "ops_recompile", door: "POST /api/ops/recompile" },
     "slack coverage": { tool: "slack_coverage", door: "GET /api/ops/slack/coverage" },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bli-cockpit/mcp",
-  "version": "0.1.28",
+  "version": "0.1.29",
   "private": false,
   "description": "bli-tower \u2014 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",