@theronap/agnoclast-mcp 0.9.146 → 0.9.148

package/lib/capture.mjs

@@ -404,10 +404,14 @@ async function captureWork(stdinRaw) {
   // merge a deadline stated twice, reclassify whose it is. Never adds; fails open (obligation_review.mjs).
   // Only when there are candidates, which most sessions never produce.
   let obligations = extracted?.obligations ?? []
+  // The quotes the review retired (merged into a later statement, or not owed). The ONLY thing that lets
+  // the server delete an earlier read's proposal; a read that is merely silent about one deletes nothing.
+  let retiredObligations = []
   if (obligations.length) {
-    const { reviewObligations, describeReview } = await import('./obligation_review.mjs')
+    const { reviewObligations, describeReview, retiredEvidence } = await import('./obligation_review.mjs')
     const rev = reviewObligations(obligations, transcript)
     process.stderr.write(`cortex: ${stamp} ${sid} obligations ${describeReview(obligations.length, rev)}\n`)
+    retiredObligations = retiredEvidence(obligations, rev)
     obligations = rev.obligations
   }
   // `pages` is the edge's PROPOSAL of where this record belongs (ADR-0055). Free-text names, resolved
@@ -424,6 +428,8 @@ async function captureWork(stdinRaw) {
         // transcript it was shown — the server cannot re-check the quote (it never receives the
         // transcript), so these arrive already filtered. Omitted when empty, same as `pages`.
         ...(obligations.length ? { obligations } : {}),
+        // ADR-0059 §5.4: quotes the reviewer retired. Omitted when empty, so an older server sees nothing new.
+        ...(retiredObligations.length ? { retiredObligations } : {}),
       }
     : { ...common, transcript }
 

package/lib/edge_extract.mjs

@@ -167,15 +167,26 @@ const PAGES_FRAGMENT =
 // ⚠ `evidence` IS VERIFIED HERE, NOT SERVER-SIDE (ADR-0059 §4.1). The cloud receives only the derived
 // digest and never the transcript, so there is nothing server-side to compare a quote against. The
 // check is `verbatimIn()` below, run against the SAME slice the model was shown.
+//
+// ⚠ "THE 15TH" IS A STATED DAY (2026-09-11). The old wording — "otherwise null — do NOT invent or infer
+// a date … that was not said" — left a model free to read a day with no month as "not said", and one
+// run of the STRAT 421 lecture did exactly that: no due_at AND no due_phrase for "on the 15th" and
+// "by Tuesday the 22nd", while a repeat of the same run filled both. The phrase is what the server
+// resolves, so it is now REQUIRED whenever the evidence says when. The server also reads the verbatim
+// quote itself when a candidate arrives with neither (resolveDueFromQuote in web/lib/engine/due_phrase.ts),
+// so this wording lowers how often that net is needed rather than being the only thing holding the date.
 const OBLIGATIONS_FRAGMENT =
   '"obligations" (array of things someone must DO by a date, stated in this session. Each an object ' +
   'with "subject" (what must be done, one short phrase), ' +
-  '"due_at" (a DATE, YYYY-MM-DD, when only a day is stated; a date-time, YYYY-MM-DDTHH:MM, ONLY when a ' +
-  'time of day is actually said; otherwise null — do NOT invent or infer a date or a time that was not ' +
-  'said. A day with no time is kept as a bare date and treated as all-day), ' +
+  '"due_at" (a DATE, YYYY-MM-DD, when only a day is stated — and a day said without its month or ' +
+  'year, like "the 15th", IS a stated day: complete it from the record\'s own date; a date-time, ' +
+  'YYYY-MM-DDTHH:MM, ONLY when a ' +
+  'time of day is actually said; null only when no day is stated at all — never invent a day or a ' +
+  'time the evidence does not state. A day with no time is kept as a bare date and treated as all-day), ' +
   '"due_phrase" (the exact words that say WHEN it is due — "on the 15th", "by Tuesday the 22nd", ' +
-  '"Saturday", "by 5 pm tomorrow" — copied VERBATIM from the evidence sentence; null if the evidence ' +
-  'names no time. The date itself is worked out from these words, so copy them, do not rephrase), ' +
+  '"Saturday", "by 5 pm tomorrow" — copied VERBATIM from the evidence sentence. REQUIRED whenever the ' +
+  'evidence says when, even when it leaves out the month or year; null ONLY if the evidence names no ' +
+  'time at all. The date itself is worked out from these words, so copy them, do not rephrase), ' +
   '"evidence" (the sentence from the session that says so, copied EXACTLY and VERBATIM — it is ' +
   'checked against the transcript and the whole item is discarded if it does not match), ' +
   '"obligated_party" (exactly one of: "self" if the person whose session this is must do it; ' +
@@ -285,7 +296,9 @@ export function verbatimIn(needle, haystack) {
  * `obligated_party` outside the preset is malformed and goes — the server's CHECK constraint would
  * reject it anyway, and failing here means the record still lands with its other keys intact.
  */
-const normWs = (x) => x.toLowerCase().replace(/[’']/g, '').replace(/\s+/g, ' ').trim()
+// Punctuation is spacing here too: "Tuesday, the 22nd" is the same words as "Tuesday the 22nd", and a
+// phrase dropped over a comma leaves the server with only due_at — the gap a phrase exists to close.
+const normWs = (x) => x.toLowerCase().replace(/[’']/g, '').replace(/[,.;:!?"“”()]/g, ' ').replace(/\s+/g, ' ').trim()
 
 export function keepVerifiableObligations(items, shown) {
   if (!Array.isArray(items)) return []

package/lib/obligation_review.mjs

@@ -172,6 +172,29 @@ export function reviewObligations(cands, text, opts = {}) {
   return { obligations: kept, reviewed: true, changes, model }
 }
 
+/**
+ * PURE. The quotes this review RETIRED: candidates it merged into a later statement of the same item, or
+ * dropped as not an obligation. A quote it kept is never retired, even if another decision named it.
+ *
+ * 🔴 THIS IS THE ONLY THING THAT MAY DELETE AN EARLIER READ'S PROPOSAL (ADR-0059 §5.4, 2026-09-11). A read
+ * that simply does not mention a proposal says nothing about it: the model may have missed it this run, may
+ * have paraphrased its quote so the verbatim gate dropped it, or — in a session past 400,000 characters — may
+ * never have been SHOWN it (summaryWindow elides the middle). A reviewer that read the sentence in context
+ * and judged it superseded or not owed is a statement. So the server deletes exactly these, and nothing for
+ * silence. Empty when the review did not run: an unreviewed read retires nothing.
+ */
+export function retiredEvidence(cands, rev) {
+  if (!rev?.reviewed || !Array.isArray(cands)) return []
+  const kept = new Set((rev.obligations ?? []).map((o) => o?.evidence))
+  const idx = [...(rev.changes?.dropped ?? []), ...(rev.changes?.merged ?? []).map(([i]) => i)]
+  const out = []
+  for (const i of idx) {
+    const ev = cands[i]?.evidence
+    if (typeof ev === 'string' && ev && !kept.has(ev) && !out.includes(ev)) out.push(ev)
+  }
+  return out
+}
+
 /** PURE. One log line for a review outcome. */
 export function describeReview(before, rev) {
   if (!rev.reviewed) return `review skipped [${rev.reason}] — ${before} candidate(s) go on unreviewed`

package/lib/obligations_worker.mjs

@@ -25,7 +25,7 @@ import { mkdirSync, openSync, readdirSync, readFileSync, writeFileSync, renameSy
 import { homedir } from 'node:os'
 import { join, dirname } from 'node:path'
 import { fileURLToPath } from 'node:url'
-import { reviewObligations, describeReview } from './obligation_review.mjs'
+import { reviewObligations, describeReview, retiredEvidence } from './obligation_review.mjs'
 
 export const MAX_PARK_DAYS = 30
 export const FIRST_RETRY_MS = 5 * 60_000
@@ -53,19 +53,45 @@ export function classifyPost(status) {
   return 'retry'
 }
 
+/**
+ * PURE. The log line for a recorded post: what the server did with the candidates, and what DATE
+ * information each one carried.
+ *
+ * 🔴 WHY (2026-09-11). A re-run of a lecture logged "recorded 2 of 3 as proposals" while two of the three
+ * had landed with no due date — one having lost the date an earlier read stored. Nothing on this machine
+ * said so; it took a query against the table, and what the model had sent was unrecoverable. So the line
+ * now carries the server's counts and, per candidate, its SHAPE — phrase, date, both, or neither. Never the
+ * subject or the quote: capture.log is created with default permissions, unlike the 0600 parked files,
+ * and a candidate's words are the person's content.
+ */
+export function describeRecorded(body, sent) {
+  const b = body ?? {}
+  const counts = [`${b.proposed ?? '?'} new`]
+  // An older server answers without these; say nothing rather than "undefined refreshed".
+  if (b.refreshed !== undefined) counts.push(`${b.refreshed} refreshed`)
+  if (b.replaced !== undefined) counts.push(`${b.replaced} replaced`)
+  const shape = (o) => (o?.due_phrase ? (o.due_at ? 'phrase+date' : 'phrase') : o?.due_at ? 'date' : 'NO date')
+  const list = Array.isArray(sent) ? sent : []
+  let line = `recorded ${list.length} on record ${b.recordId ?? '?'}: ${counts.join(', ')} — sent ${list.map((o, i) => `#${i} ${shape(o)}`).join(', ')}`
+  if (b.fromQuote) line += `; ${b.fromQuote} dated from the quote`
+  if (b.undated) line += `; ⚠ ${b.undated} left WITHOUT a date`
+  return line
+}
+
 /** PURE. The next retry delay: 5 min, doubling, capped at 6 h. */
 export function backoffMs(attempts) {
   return Math.min(FIRST_RETRY_MS * 2 ** Math.max(0, attempts - 1), MAX_RETRY_MS)
 }
 
 /** POST the candidates. Never throws: a network failure is `retry`, like a 5xx. */
-export async function postCandidates({ base, token, target, obligations, fetchImpl }) {
+export async function postCandidates({ base, token, target, obligations, retired = [], fetchImpl }) {
   if (!token) return { action: 'retry', status: 0, body: { error: 'no CORTEX_TOKEN in env or wired config' } }
   try {
     const res = await fetchImpl(`${base}/api/obligations/candidates`, {
       method: 'POST',
       headers: { Authorization: `Bearer ${token}`, 'Content-Type': 'application/json' },
-      body: JSON.stringify({ ...target, obligations }),
+      // retiredObligations: quotes the reviewer retired (ADR-0059 §5.4). Omitted when empty.
+      body: JSON.stringify({ ...target, obligations, ...(retired.length ? { retiredObligations: retired } : {}) }),
     })
     const body = await res.json().catch(() => ({}))
     return { action: classifyPost(res.status), status: res.status, body }
@@ -113,13 +139,17 @@ export async function processJob(job, deps) {
   // drop, merge or reclassify, never add; it fails open. Optional here so a caller without it is unchanged.
   const review = deps.review ?? ((o) => ({ obligations: o, reviewed: false, reason: 'no reviewer' }))
   const before = obligations.length
+  const cands = obligations
   const rev = review(obligations, job.text)
   log(describeReview(before, rev))
   obligations = rev.obligations
-  if (obligations.length === 0) return { outcome: 'none', reviewed: true }
-  const res = await post(job.target, obligations)
+  // What the review RETIRED is the only thing that may delete an earlier read's proposal (retiredEvidence).
+  // So a read whose every candidate was retired still posts: the retirement is the whole message.
+  const retired = retiredEvidence(cands, rev)
+  if (obligations.length === 0 && retired.length === 0) return { outcome: 'none', reviewed: true }
+  const res = await post(job.target, obligations, retired)
   if (res.action === 'recorded') {
-    log(`recorded ${res.body?.proposed ?? '?'} of ${obligations.length} as proposals on record ${res.body?.recordId ?? '?'}`)
+    log(describeRecorded(res.body, obligations))
     return { outcome: 'recorded', proposed: res.body?.proposed ?? null }
   }
   if (res.action === 'drop') {
@@ -127,7 +157,7 @@ export async function processJob(job, deps) {
     return { outcome: 'dropped', status: res.status }
   }
   const t = now()
-  park({ target: job.target, title: job.title ?? null, obligations, parkedAt: t, attempts: 1, nextAttemptAt: t + backoffMs(1) })
+  park({ target: job.target, title: job.title ?? null, obligations, ...(retired.length ? { retired } : {}), parkedAt: t, attempts: 1, nextAttemptAt: t + backoffMs(1) })
   log(`parked ${obligations.length} — ${res.action === 'pending' ? 'the unit is not materialised yet' : `${res.status} ${res.body?.error ?? ''}`.trim()}; retried by later captures`)
   return { outcome: 'parked', status: res.status }
 }
@@ -165,12 +195,13 @@ export async function flushParkedObligations(deps) {
       continue
     }
     if ((entry.nextAttemptAt ?? 0) > t) continue
-    const res = await post(entry.target, entry.obligations)
+    const res = await post(entry.target, entry.obligations, entry.retired ?? [])
     if (res.action === 'recorded' || res.action === 'drop') {
       try { unlinkSync(file) } catch { /* raced with another flush; the write is idempotent */ }
       flushed += 1
       log(res.action === 'recorded'
-        ? `recorded ${res.body?.proposed ?? '?'} parked obligations for ${id} on attempt ${(entry.attempts ?? 0) + 1}`
+        ? `recorded ${res.body?.proposed ?? '?'} parked obligations for ${id} on attempt ${(entry.attempts ?? 0) + 1}` +
+          (res.body?.undated ? ` — ⚠ ${res.body.undated} left WITHOUT a date` : '')
         : `DROPPED parked obligations for ${id} — the server answered ${res.status} ${res.body?.error ?? ''}`.trim())
       continue
     }
@@ -234,7 +265,7 @@ async function realDeps(target) {
     extract: extractSession,
     lastSkip: lastEdgeSkip,
     review: (obligations, text) => reviewObligations(obligations, text),
-    post: (t, obligations) => postCandidates({ base, token, target: t, obligations, fetchImpl: fetchCortex }),
+    post: (t, obligations, retired) => postCandidates({ base, token, target: t, obligations, retired, fetchImpl: fetchCortex }),
     park: (entry) => writeParked(parkDir(), entry),
     log: logger(target),
     sleep: (ms) => new Promise((r) => setTimeout(r, ms)),

package/lib/server.mjs

@@ -933,7 +933,7 @@ function renderNudge(payload) {
       inputSchema: {
         intakeItemId: z.string().describe('intake item uuid'),
         orgId: z.string().describe('destination brain org uuid'),
-        documentIds: z.array(z.string()).optional().describe('additional brain_documents ids in orgId (deterministic homes are merged automatically)'),
+        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(),
         summary: z.string().optional(),
         source: z.string().optional(),

package/package.json

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