@theronap/cortex-mcp 0.9.111 → 0.9.113

package/lib/server.mjs

@@ -859,7 +859,35 @@ export async function runServer(version) {
         const body = await res.text()
         return toolError(body)
       }
-      return { content: [{ type: 'text', text: JSON.stringify(await res.json(), null, 2) }] }
+      const body = await res.json()
+      // WHERE THE SESSION'S OWN WORK LANDED, in words rather than buried in the JSON dump.
+      //
+      // Three outcomes that look identical in raw JSON and mean completely different things:
+      //   null            the attach THREW — a real failure, and best-effort swallowed it
+      //   attached: []    the session authored no page in this brain — the ~85% normal case
+      //   attached: [...] filed under N pages
+      //
+      // Reporting only a count would make the first indistinguishable from the second, which is how
+      // a broken write hides inside a normal-looking result. `undefined` means the server has not
+      // shipped this field yet, so nothing is said rather than something wrong.
+      const lines = []
+      const sa = body?.sessionAttachments
+      if (sa === null) {
+        lines.push('⚠ Session-page attachment FAILED for this record — it was written, but not filed under the pages this session wrote. Re-run the reconciler.')
+      } else if (sa && Array.isArray(sa.attached)) {
+        if (sa.attached.length > 0) {
+          lines.push(`Filed under ${sa.attached.length} page(s) this session wrote.`)
+        } else {
+          lines.push('No pages filed: this session authored none in this brain (the normal case).')
+        }
+        if (sa.droppedCrossBrain > 0) {
+          lines.push(`${sa.droppedCrossBrain} page(s) this session wrote live in another brain and were skipped — a record belongs to one brain.`)
+        }
+      }
+      const text = lines.length
+        ? `${lines.join('\n')}\n\n${JSON.stringify(body, null, 2)}`
+        : JSON.stringify(body, null, 2)
+      return { content: [{ type: 'text', text }] }
     },
   )
 
@@ -2514,7 +2542,7 @@ export async function runServer(version) {
       description: 'Attach a pending record to the pages you judge correct — the point of the whole triage path. Use this when you have real context on what the work was; that judgment is better than any rule the webhook could run. Attachments are additive: existing deterministic homes (routing identifiers, your profile) stay. Pass park=true instead when you have looked and there is genuinely no good home — parking beats attaching to a page that merely shares a word.',
       inputSchema: {
         recordId: z.string().describe('record id from pending_records'),
-        documentIds: z.array(z.string()).optional().describe('page document ids to attach (from pending_records sweep, or read_page)'),
+        documentIds: z.array(z.string()).optional().describe('pages to attach. Accepts EITHER a brain_documents id (from a pending_records sweep) OR the `ref:` that read_page prints for a page — they are different columns and read_page only ever gives you the second, which is why this used to fail as no_documents on ids that looked perfectly valid. A ref resolves to that node\'s current page, accessible copy first. Anything that resolves to no page in this brain is named back to you rather than silently dropped.'),
         reason: z.string().describe('why these pages — recorded with the attachment'),
         park: z.boolean().optional().describe('no good home exists; leave it alone rather than guessing'),
         tier: z.number().int().optional().describe('2 when this came from the cheap sweep rather than your own context'),

package/package.json

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