@bli-cockpit/mcp 0.1.4 → 0.1.5

package/README.md

@@ -127,6 +127,14 @@ JARVIS itself — the same assistant Tower web chat, the Slack DM and
 | `jarvis_dispatch` | `POST /api/jarvis/cli` | The coding arm (BLI-2981). Two calls: once for the plan, once with the person's approval code. |
 | `jarvis_check` | `POST /api/jarvis/cli` | Where a dispatched coding task got to. Spends no approval. |
 
+**One conversation per call (BLI-3786).** A thread is single-writer: two turns
+in the same conversation at the same time read each other's messages, and one
+can answer the other's question. So every `jarvis_*` tool sends NO thread
+unless the caller named one, the door mints a conversation for that call alone,
+and the answer's `thread_id` is the name it used — pass that back as `thread`
+to continue deliberately. `"main"` is the shared terminal default nobody chose
+and is never sent from here.
+
 `jarvis_ask` answers with the SAME object `cockpit jarvis --json` prints —
 `answer`, `sources`, `turn_id`, `thread_id`, `trace_thread_id`, `degraded`,
 `degraded_reasons` — so a script and an agent read one contract. A worked
@@ -136,7 +144,7 @@ example:
 jarvis_ask  { "question": "what did Brandon ship yesterday?" }
   -> { "answer": "Two pull requests went in…\n\nSource: BLI-3654 — …",
        "sources": ["Source: BLI-3654 — …"],
-       "turn_id": "0f3c…", "thread_id": "main",
+       "turn_id": "0f3c…", "thread_id": "agent-6b1e…",
        "degraded": false, "degraded_reasons": [] }
 
 jarvis_trace { "turn_id": "0f3c…" }      # or "last"

package/dist/jarvis-tools.d.ts

@@ -24,6 +24,15 @@
  * `jarvis_ask` answers with `jarvis-answer-envelope.ts` — the same object
  * `cockpit jarvis --json` prints. A script and an agent read one shape.
  *
+ * ## One conversation per call (BLI-3786)
+ *
+ * A thread is single-writer: two turns in one conversation at the same time
+ * read each other's messages, and one can answer the other's question. Every
+ * tool here therefore sends NO thread unless the caller named one, and the
+ * door mints a conversation for that call alone and reports it as `thread_id`.
+ * Passing that id back is how an agent continues deliberately. `"main"` is the
+ * shared terminal default nobody chose and is never sent from here.
+ *
  * ## The approval code, and what this surface can and cannot prove
  *
  * The coding arm's gate (BLI-2981) has two locks: the code is an HMAC over the

package/dist/jarvis-tools.js

@@ -24,6 +24,15 @@
  * `jarvis_ask` answers with `jarvis-answer-envelope.ts` — the same object
  * `cockpit jarvis --json` prints. A script and an agent read one shape.
  *
+ * ## One conversation per call (BLI-3786)
+ *
+ * A thread is single-writer: two turns in one conversation at the same time
+ * read each other's messages, and one can answer the other's question. Every
+ * tool here therefore sends NO thread unless the caller named one, and the
+ * door mints a conversation for that call alone and reports it as `thread_id`.
+ * Passing that id back is how an agent continues deliberately. `"main"` is the
+ * shared terminal default nobody chose and is never sent from here.
+ *
  * ## The approval code, and what this surface can and cannot prove
  *
  * The coding arm's gate (BLI-2981) has two locks: the code is an HMAC over the
@@ -70,10 +79,12 @@ export function registerJarvisTools(server, deps) {
         title: "Ask JARVIS",
         description: "Asks JARVIS one question and returns its answer, the Source: lines behind it, this turn's "
             + "turn_id and the conversation's thread_id. Same JARVIS, same tool belt and same evidence "
-            + "rules as Tower web chat, the Slack DM and `cockpit jarvis`. Pass thread to continue an "
-            + "earlier conversation; pass the returned turn_id to jarvis_trace to see how the answer was "
-            + "reached. JARVIS speaks as the person this machine is paired to and cannot be made to "
-            + "speak as anyone else.",
+            + "rules as Tower web chat, the Slack DM and `cockpit jarvis`. A thread is SINGLE-WRITER: "
+            + "omit thread and this turn gets a FRESH conversation of its own, so two questions asked at "
+            + "the same time can never read each other's; pass the thread_id the answer returned to "
+            + "continue that conversation deliberately. Pass the returned turn_id to jarvis_trace to see "
+            + "how the answer was reached. JARVIS speaks as the person this machine is paired to and "
+            + "cannot be made to speak as anyone else.",
         inputSchema: {
             question: z.string().min(1).max(50_000).describe("What to ask, in plain words."),
             thread: z
@@ -81,7 +92,9 @@ export function registerJarvisTools(server, deps) {
                 .min(1)
                 .max(200)
                 .optional()
-                .describe('The conversation to continue. Omit for "main", the default terminal thread.'),
+                .describe("The thread_id of a conversation to continue, exactly as an earlier answer returned "
+                + 'it. Omit for a fresh conversation for this call alone. "main" is the shared '
+                + "default nobody chose, so it is treated as naming nothing here."),
             subject: z
                 .string()
                 .min(1)
@@ -105,7 +118,10 @@ export function registerJarvisTools(server, deps) {
     }, async (args) => withSession(deps, async (session) => {
         const { result } = await takeTurn(deps, session, "jarvis_ask", {
             question: String(args.question ?? ""),
-            thread: typeof args.thread === "string" ? args.thread : "main",
+            // BLI-3786: no thread means no thread. The door mints one for this
+            // call and reports it as thread_id; sending "main" here would put
+            // every agent on one conversation, which is the bug this replaced.
+            ...(typeof args.thread === "string" ? { thread: args.thread } : {}),
             ...(typeof args.subject === "string" ? { subject: args.subject } : {}),
             ...(typeof args.date === "string" ? { date: args.date } : {}),
             ...(typeof args.model === "string" ? { model: args.model } : {}),
@@ -201,7 +217,13 @@ export function registerJarvisTools(server, deps) {
                 .optional()
                 .describe("The 8-character hex confirmation code THE PERSON fetched for this proposal and handed "
                 + "to you. Omit it on the first call — that call is never given one."),
-            thread: z.string().min(1).max(200).optional().describe('Defaults to "main".'),
+            thread: z
+                .string()
+                .min(1)
+                .max(200)
+                .optional()
+                .describe("The thread_id of a conversation to continue. Omit for a fresh one — the approval "
+                + "code is bound to the PLAN, not to a conversation, so a dispatch needs no thread."),
         },
     }, async (args) => withSession(deps, async (session) => {
         const log = deps.log ?? defaultLog;
@@ -231,7 +253,7 @@ export function registerJarvisTools(server, deps) {
                 ? ["", `The person approved this plan. Confirmation code: ${code.toLowerCase()}`]
                 : []),
         ].join("\n");
-        const { result } = await takeTurn(deps, session, "jarvis_dispatch", { question, thread: typeof args.thread === "string" ? args.thread : "main" }, remember);
+        const { result } = await takeTurn(deps, session, "jarvis_dispatch", { question, ...(typeof args.thread === "string" ? { thread: args.thread } : {}) }, remember);
         return result;
     }));
     register("jarvis_check", {
@@ -246,14 +268,19 @@ export function registerJarvisTools(server, deps) {
                 .max(200)
                 .optional()
                 .describe("The task id JARVIS named when it dispatched. Omit to ask about the recent ones."),
-            thread: z.string().min(1).max(200).optional().describe('Defaults to "main".'),
+            thread: z
+                .string()
+                .min(1)
+                .max(200)
+                .optional()
+                .describe("The thread_id of a conversation to continue. Omit for a fresh one."),
         },
     }, async (args) => withSession(deps, async (session) => {
         const task = typeof args.task === "string" ? args.task.trim() : "";
         const question = task.length > 0
             ? `Check the coding task ${task} and tell me its status, its branch and its pull request if it has one.`
             : "Check the coding tasks dispatched for me recently and tell me the status of each.";
-        const { result } = await takeTurn(deps, session, "jarvis_check", { question, thread: typeof args.thread === "string" ? args.thread : "main" }, remember);
+        const { result } = await takeTurn(deps, session, "jarvis_check", { question, ...(typeof args.thread === "string" ? { thread: args.thread } : {}) }, remember);
         return result;
     }));
 }

package/package.json

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