@theronap/agnoclast-mcp 0.9.212 → 0.9.214

package/lib/diagnose.mjs

@@ -679,7 +679,14 @@ export async function fetchCortex(url, opts = {}, policy = {}) {
 
 // Live health probe: hit /api/mcp-context with the token and classify the result.
 // Returns { ok, status, projectCount?, requestId?, diagnosis? }.
-export async function checkToken(token, base) {
+//
+// `probe: true` (the SessionStart status line) asks the server for the token check and the capture
+// notice ONLY, not the context. That hook fires at the same moment as `snapshot-context`, which fetches
+// the full context, so without it every session start assembled the context twice, concurrently, on a
+// max:1 pool: a ~1.6 s start measured ~5 s in prod (2026-10-06). doctor/setup/install keep the full
+// fetch on purpose; they exist to prove the context itself comes back. An older server ignores the
+// parameter and returns the full context, which this still parses, so the two can ship in either order.
+export async function checkToken(token, base, { probe = false } = {}) {
   if (!token) {
     return { ok: false, status: 0, diagnosis: { kind: 'config', retriable: false,
       message: 'No CORTEX_TOKEN found. Run: npx -y @theronap/cortex-mcp login (opens your browser; or `setup <token>` with a token from the console → Connect your AI)' } }
@@ -689,7 +696,7 @@ export async function checkToken(token, base) {
       message: `Token "${String(token).slice(0, 8)}…" is not a valid Agnoclast token (expected a UUID). ` +
         `Re-run setup with the token from the console.` } }
   }
-  const url = `${(base ?? CANONICAL_BASE).replace(/\/$/, '')}/api/mcp-context`
+  const url = `${(base ?? CANONICAL_BASE).replace(/\/$/, '')}/api/mcp-context${probe ? '?probe=1' : ''}`
   let res
   try {
     res = await fetchCortex(url, { headers: { Authorization: `Bearer ${token}` } })

package/lib/doctor.mjs

@@ -71,7 +71,7 @@ async function statusLine(out, { capturesStuck = false } = {}) {
     return
   }
   try {
-    const r = await checkToken(token, base)
+    const r = await checkToken(token, base, { probe: true })
     if (r.ok) {
       // ⚠ THE NOTICE REPLACES THE HAPPY LINE RATHER THAN FOLLOWING IT.
       // "connected — sessions on this machine are captured to your org" was printed truthfully to

package/lib/intake_batch_render.mjs

@@ -0,0 +1,26 @@
+// Rendering for ADR-0076 §4.5 — a conversation's texts, decided together and filed one record each.
+// Kept out of server.mjs so it can be tested on its output, not on its source text.
+
+/**
+ * A batch materialization, per id. ⚠ PARTIAL FAILURE IS REPORTED PER ID, never summarised as success — a
+ * batch that half landed is the one outcome an agent must not mistake for done (same rule as
+ * place_staged_record's batch).
+ */
+export function renderMaterializeBatch(results) {
+  const good = results.filter((r) => r.ok)
+  const bad = results.filter((r) => !r.ok)
+  return `Batch of ${results.length}: ${good.length} filed, ${bad.length} failed — one record each.\n` +
+    results.map((r) => `  ${r.ok ? '✓' : '✗'} ${r.id} — ${r.detail}`).join('\n') +
+    (bad.length ? `\n\n⚠ ${bad.length} did NOT land. They are still in the pile; fix the cause and file those ids again (a unit must be claimed first).` : '')
+}
+
+/**
+ * One settling offer that stands for a whole text conversation (the server collapsed its waiting texts into
+ * it), or null when the offer is a single arrival and renders the ordinary way.
+ */
+export function renderConversationOffer(x, why = '', looked = '') {
+  if (!x?.conversation || !(x.conversation.count > 1)) return null
+  const ids = x.conversation.ids.slice(0, 12).join(', ') + (x.conversation.ids.length > 12 ? ', …' : '')
+  return `    - 📱 ${x.conversation.count} texts in one conversation · newest ${x.hoursAgo}h ago · ${ids}${why}${looked}\n` +
+    '      decide them together: `intake_look` the newest, `intake_claim` them all, then file them in one `intake_materialize` (intakeItemIds) — one record each'
+}

package/lib/server.mjs

@@ -24,6 +24,7 @@ import { spawnObligationWorker } from './obligations_worker.mjs'
 import { renderObligations, renderPageObligations, tierReads, who } from './obligations_render.mjs'
 import { renderRemovalQuestion, renderRemoved } from './member_remove_render.mjs'
 import { renderSettings } from './settings_render.mjs'
+import { renderConversationOffer, renderMaterializeBatch } from './intake_batch_render.mjs'
 import { formatCreated, formatDryRun, formatSources, homeFromArgs, pickSource, renderHomeProblem } from './custom_sources.mjs'
 
 // PAGE tiers (ADR-0071). Six for a PAGE; a RECORD keeps three (`privacy` on log_session and
@@ -428,6 +429,9 @@ function renderSettling(payload) {
         ? ' — ⚠ NO session of yours matches this; it is here because it reached nobody'
         : ' — every session that matched it declined, so it is back to everyone'
     const looked = x.looks > 0 ? `  [looked at ${x.looks}x, still unclaimed]` : ''
+    // ADR-0076 §4.5: a conversation's waiting texts arrive one per message and are offered as ONE line.
+    const conversationLine = renderConversationOffer(x, why, looked)
+    if (conversationLine) return conversationLine
     return `    - ${x.source} · ${x.hoursAgo}h ago · ${x.intakeItemId}${why}${looked}`
   })
 
@@ -1230,6 +1234,12 @@ function renderNudge(payload) {
         '⚠ ADR-0077: a GitHub commit or PR does not finish until every changed path has an answer. If a changed file\'s deepest folder (or, in a grab-bag folder like web/lib/engine/ or db/supabase/, the file itself) has no claim and no decline, the call is REFUSED with the open questions; answer each in folderDecisions and call again.',
       inputSchema: {
         intakeItemId: z.string().describe('intake item uuid'),
+        intakeItemIds: z.array(z.string()).optional().describe(
+          'MORE units to file exactly the same way, in one call — one record EACH, never merged (ADR-0076 §4.5). ' +
+          'For a conversation\'s waiting texts that you decided together: claim them all with `intake_claim`, then pass ' +
+          'the first as intakeItemId and the rest here. title/summary/dedupeKey are ignored in a batch (each unit keeps its ' +
+          'own), and a preview covers intakeItemId only. Each id is reported separately; a failure never hides behind a success.',
+        ),
         orgId: z.string().describe('destination brain org uuid'),
         documentIds: z.array(z.string()).optional().describe('additional pages in orgId — each a page `ref:` exactly as read_page shows it, or a brain_documents id (deterministic homes are merged automatically)'),
         title: z.string().optional(),
@@ -1266,6 +1276,38 @@ function renderNudge(payload) {
       },
     },
     async (args) => {
+      // ADR-0076 §4.5 — A BATCH: one decision, N records. Theron, 2026-10-05: a conversation's waiting texts
+      // are looked up once and handled together, but "it shouldn't file all of them as one record". The
+      // loop lives here, as it does for place_staged_record's batch: the route stays one unit per call.
+      const batch = [...new Set([args.intakeItemId, ...(args.intakeItemIds ?? [])])].slice(0, 50)
+      if (batch.length > 1 && args.mode !== 'preview') {
+        const results = []
+        for (const id of batch) {
+          try {
+            const r = await fetchCortex(`${BASE}${intakeMaterializePath(undefined)}`, {
+              method: 'POST',
+              headers: { Authorization: `Bearer ${TOKEN}`, 'Content-Type': 'application/json' },
+              body: JSON.stringify({
+                intakeItemId: id,
+                orgId: args.orgId,
+                documentIds: args.documentIds ?? [],
+                record: { source: args.source, record_type: args.recordType },
+                attachmentMeta: (args.documentIds ?? []).map(() => ({ origin: args.origin ?? 'llm' })),
+                ...(args.governingPage ? { governingDocumentId: args.governingPage } : {}),
+                ...(args.searchedForOtherHomes ? { searchedForOtherHomes: true } : {}),
+                // One placement is a decision about ONE conversation: the server refuses any id whose
+                // batch differs from the first id's, so a stray id is never filed onto these pages.
+                sameBatchAs: batch[0],
+              }),
+            })
+            const o = await r.json().catch(() => null)
+            results.push({ id, ok: r.ok, detail: r.ok ? (o?.recordId ?? 'filed') : (o?.error ?? `HTTP ${r.status}`) })
+          } catch (e) {
+            results.push({ id, ok: false, detail: e.message })
+          }
+        }
+        return { content: [{ type: 'text', text: renderMaterializeBatch(results) }] }
+      }
       // A preview goes to its OWN route: a server too old to know it answers 404, where the materialize
       // route would ignore a `mode` it does not know and publish.
       const preview = args.mode === 'preview'
@@ -1312,8 +1354,10 @@ function renderNudge(payload) {
         }
         // Same redirect-not-dispatch rule as place_staged_record. A staged arrival needs no claim and
         // no unseal, so the fix is a different verb rather than a retry of this one.
-        if (res.status === 404 && (await pileHolding(intakeItemId)) === 'staged') {
-          return toolError(`${intakeItemId} is a STAGED arrival, not an intake unit — \`intake_materialize\` cannot act on it. Use \`place_staged_record\` with the pages it belongs to; staged arrivals take no lease, so there is nothing to claim or release. (\`staged_look\` reads it first, and carries the brain-ladder proposal.)`)
+        // ⚠ `args.intakeItemId`, not a bare `intakeItemId`: this handler never declared one, so a 404 here
+        // threw a ReferenceError instead of saying which verb to use (found 2026-10-05 adding batches).
+        if (res.status === 404 && (await pileHolding(args.intakeItemId)) === 'staged') {
+          return toolError(`${args.intakeItemId} is a STAGED arrival, not an intake unit — \`intake_materialize\` cannot act on it. Use \`place_staged_record\` with the pages it belongs to; staged arrivals take no lease, so there is nothing to claim or release. (\`staged_look\` reads it first, and carries the brain-ladder proposal.)`)
         }
         return toolError(body)
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@theronap/agnoclast-mcp",
-  "version": "0.9.212",
+  "version": "0.9.214",
   "description": "Connect your AI assistant to Cortex — your org's projects, activity, gaps, and directives, scoped to you.",
   "type": "module",
   "bin": {