@theronap/agnoclast-mcp 0.9.223 → 0.9.225

package/lib/series_render.mjs

@@ -0,0 +1,30 @@
+// update_series replies — pure, so every rendering has a test. The tool and its engine
+// (web/lib/engine/series_update.ts) exist because a series' container type could only be changed with
+// SQL until 2026-10-08 (Agnoclast page "Agent SQL gap").
+
+const OWES = {
+  meeting: 'owes a transcript or a note; closes 15 minutes after it ends',
+  work_block: 'owes a session in the window; closes when it ends',
+  deadline: 'owes the artifact; closes when it is due',
+  appointment: 'owes nothing (a class, a visit); admits a transcript or a note',
+}
+
+const line = (s) => `• ${s.title}  [${s.seriesKey}]  ${s.containerType ?? 'no container (a marker)'}${s.brain ? ` · ${s.brain}` : ''}`
+
+/** PATCH /api/series/<ref> 200 → text. */
+export function formatSeriesUpdated(r) {
+  if (r.from === r.to) return `${r.series.title} is already ${r.to} — nothing changed.`
+  return [
+    `${r.series.title} is now ${r.to} (was ${r.from}): ${OWES[r.to] ?? ''}.`,
+    `${r.containersRetyped} instance container(s) retyped${r.openContainersRetimed ? `, ${r.openContainersRetimed} still open with their close time recomputed` : ''}; instances made later open as ${r.to}.`,
+    'Containers that already closed keep the time they closed at.',
+  ].join('\n')
+}
+
+/** A refusal body (404 / 409 / 400) → text that lets the agent pick again. */
+export function formatSeriesRefusal(j) {
+  const L = [j?.message ?? j?.error ?? 'Not changed.']
+  if (j?.candidates?.length) L.push(j.error === 'ambiguous' ? 'Which one? Pass its key:' : 'Your series:', ...j.candidates.map(line))
+  else if (j?.error === 'not_found') L.push('You own no series. (A series is a recurring class or meeting made in Agnoclast.)')
+  return L.join('\n')
+}

package/lib/server.mjs

@@ -28,6 +28,7 @@ import { renderConversationOffer, renderMaterializeBatch } from './intake_batch_
 import { obligationsForUnits, renderFilingObligations, filingObligationsBrief, OBLIGATIONS_UNANSWERED_TEXT } from './filing_obligations.mjs'
 import { renderRecord, READ_RECORD_DEFAULT_CHARS, READ_RECORD_PARTS } from './record_render.mjs'
 import { formatCreated, formatDryRun, formatSources, homeFromArgs, pickSource, renderHomeProblem } from './custom_sources.mjs'
+import { formatSeriesRefusal, formatSeriesUpdated } from './series_render.mjs'
 import { formatSends } from './sends_render.mjs'
 
 // PAGE tiers (ADR-0071). Six for a PAGE; a RECORD keeps three (`privacy` on log_session and
@@ -4986,6 +4987,44 @@ function renderNudge(payload) {
     },
   )
 
+  // update_series — the user-parity sweep, 2026-10-08 (Theron: "things I think are totally working …
+  // that regular users can't do because all of my agents are querying the SQL directly"). A series'
+  // container type could only be changed with owner-connection SQL (2026-09-03, ENT 425). The sweep's
+  // other gap, editing a custom source's mapping, shipped in parallel as set_custom_source_mapping (#1438).
+  // Agnoclast page "Agent SQL gap".
+  server.registerTool(
+    'update_series',
+    {
+      title: 'Change a recurring series\' container type',
+      description: 'Change what a recurring class or meeting series (one made in Agnoclast) COLLECTS and OWES — its container type — for the whole series: the series itself and the container of every instance already made. meeting = owes a transcript or a note · work_block = owes a work session in the window · deadline = owes the artifact · appointment = owes nothing (a class, a visit) and still admits a transcript or a note. Use it when a series was typed wrong, e.g. a 3-hour lecture typed work_block, which flags every class as "nothing happened". Containers that already closed keep their close time. Name the series by its exact title, its key, or its id; an unknown or ambiguous name answers with your series to choose from.',
+      inputSchema: {
+        series: z.string().describe('the series — its exact title ("ENT 425"), its key (with or without "series:"), or its id'),
+        container_type: z.enum(['meeting', 'work_block', 'deadline', 'appointment']).describe('what it should collect and owe'),
+      },
+    },
+    async ({ series, container_type }) => {
+      const ref = String(series).trim().replace(/^series:/i, '')
+      let res
+      try {
+        res = await fetchCortex(`${BASE}/api/series/${encodeURIComponent(ref)}`, {
+          method: 'PATCH',
+          headers: { Authorization: `Bearer ${TOKEN}`, 'Content-Type': 'application/json' },
+          body: JSON.stringify({ containerType: container_type }),
+        })
+      } catch (e) {
+        return toolError(`Not changed: ${e.message}`)
+      }
+      if (!res.ok) {
+        const body = await res.text()
+        let j = null
+        try { j = JSON.parse(body) } catch { /* not JSON */ }
+        if (j && [400, 404, 409].includes(res.status)) return toolError(formatSeriesRefusal(j))
+        return toolError(`Not changed: ${classify(res.status, res.headers.get('content-type'), body, res.headers.get('x-vercel-id')).message}`)
+      }
+      return { content: [{ type: 'text', text: formatSeriesUpdated(await res.json()) }] }
+    },
+  )
+
   server.registerTool(
     'my_day',
     {
@@ -5231,13 +5270,14 @@ function renderNudge(payload) {
     'set_routing_identifier',
     {
       title: 'Claim a routing identifier on a page',
-      description: 'Declare that a page is a Gate 4 attach HOME for an identifier (repo:owner/name or file:owner/name:path). Body [[repo:…]] stamps are navigation only and do NOT drive attach — use this instead. Prefer the narrowest id: mother pages own repo:; feature pages own file: paths. Do not copy every body mention into a routing claim.',
+      description: 'Declare that a page is a Gate 4 attach HOME for an identifier (repo:owner/name or file:owner/name:path). Body [[repo:…]] stamps are navigation only and do NOT drive attach — use this instead. Prefer the narrowest id: mother pages own repo:; feature pages own file: paths. Do not copy every body mention into a routing claim. ' +
+        'PRIVATE CONTACT CLAIMS (ADR-0079): on a PERSON page you may claim `phone:<number>` (normalized to E.164, US default — `phone:(801) 636-7505` stores `phone:+18016367505`) or `email:<address>` (lowercased). These are visible ONLY to you — never in the page body, grep, or to other brain members — and they make your captured texts / mail from that contact propose that person page. This is the right place for a contact\'s number; never put it in page text.',
       inputSchema: {
         kind: z.enum(['project', 'person', 'org', 'user']).describe('the page kind'),
         name: z.string().optional().describe('page title (or pass ref)'),
         ref: z.string().optional().describe('node ref from read_page — prefer over name when available'),
         brain: z.string().optional().describe('brain label when the title is ambiguous across brains'),
-        identifier: z.string().describe('canonical identifier, e.g. repo:theronap/cortex or file:theronap/cortex:web/lib/engine/github_intake.ts'),
+        identifier: z.string().describe('canonical identifier, e.g. repo:theronap/cortex, file:theronap/cortex:web/lib/engine/github_intake.ts, or on a person page phone:+18015550123 / email:someone@example.com (private to you)'),
       },
     },
     async ({ kind, name, ref, brain, identifier }) => {
@@ -5253,7 +5293,12 @@ function renderNudge(payload) {
         return toolError(`Could not set routing identifier: ${e.message}`)
       }
       const out = await res.json().catch(() => null)
-      if (!res.ok) return toolError(`Could not set routing identifier: ${out?.error ?? res.status}`)
+      if (!res.ok) {
+        // A per-identifier refusal arrives in `errors[]` with a top-level 400 — name it, not just the status.
+        const why = out?.error ?? out?.errors?.map((e) => e.error).join(', ') ?? res.status
+        const hint = /phone_claims_person_pages_only/.test(String(why)) ? ' A phone: claim goes on the PERSON page for that contact (kind: person).' : ''
+        return toolError(`Could not set routing identifier: ${why}.${hint}`)
+      }
       const set = out?.set?.join(', ') ?? identifier
       // ⚠ SAY WHAT ALREADY HAPPENED, NOT ONLY WHAT WILL. The claim heals history in the same
       // statement, so by the time this renders, existing records are ALREADY attached. The previous
@@ -5267,7 +5312,11 @@ function renderNudge(payload) {
         : healed > 0
           ? ` ${healed} existing record(s) ALREADY attached to this page as a result.`
           : ' 0 existing records attached — either nothing carries this identifier yet, or records do but belong to another owner (an identity claim only attaches your own).'
-      return { content: [{ type: 'text', text: `Routing identifier set on document ${out?.documentId ?? '?'}: ${set}.${healedLine} Future matching events will attach here too (body mentions alone will not).` }] }
+      const isContact = /^(phone|email):/.test(String(out?.set?.[0] ?? identifier).toLowerCase())
+      const privacyLine = isContact
+        ? ' This contact claim is PRIVATE to you: no other brain member can see it, and it is never rendered in the page. Your captured texts/mail from this contact will now propose this page.'
+        : ''
+      return { content: [{ type: 'text', text: `Routing identifier set on document ${out?.documentId ?? '?'}: ${set}.${healedLine} Future matching events will attach here too (body mentions alone will not).${privacyLine}` }] }
     },
   )
 

package/package.json

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