@theronap/agnoclast-mcp 0.9.146 → 0.9.147

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/obligations_worker.mjs

@@ -53,6 +53,31 @@ 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)
@@ -119,7 +144,7 @@ export async function processJob(job, deps) {
   if (obligations.length === 0) return { outcome: 'none', reviewed: true }
   const res = await post(job.target, obligations)
   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') {
@@ -170,7 +195,8 @@ export async function flushParkedObligations(deps) {
       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
     }

package/package.json

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