@bli-cockpit/memory-mcp 0.1.6 → 0.1.7

package/dist/hooks/transcript.d.ts

@@ -1,5 +1,6 @@
 /**
- * The last exchange out of a Claude Code transcript (BLI-3580).
+ * The last exchange out of an agent host's transcript (BLI-3580; Codex added
+ * by BLI-3729).
  *
  * The Stop hook is handed `transcript_path`: a JSONL file, one record per line,
  * appended to for the whole session. What the save door wants is the LAST
@@ -19,6 +20,31 @@
  * A record shape that is not recognised is skipped, never guessed at. The
  * reader returns null rather than half an exchange: half a turn saved as a
  * memory is a memory that says something the person did not.
+ *
+ * ## Two hosts, two record shapes (BLI-3729)
+ *
+ * `cockpit memory install` now registers the same three hooks with Codex, and
+ * Codex hands its Stop hook a `transcript_path` too — pointing at its own
+ * rollout JSONL, whose records are shaped differently:
+ *
+ *   Claude Code  { "message": { "role": "user", "content": [{ "type": "text", … }] } }
+ *   Codex        { "type": "response_item",
+ *                  "payload": { "type": "message", "role": "user",
+ *                               "content": [{ "type": "input_text", "text": … }] } }
+ *
+ * Both are read here, by the same backwards walk, because the alternative was a
+ * Codex Stop hook that fired on every turn and reported `transcript_no_exchange`
+ * every time — a hook that does nothing and says it did nothing. The Codex
+ * shape was captured from a real rollout file on 2026-09-05 rather than read
+ * off a doc, and the fixture in `hooks.test.ts` keeps the record structure
+ * verbatim with only the `text` values replaced (the real ones are somebody's
+ * conversation).
+ *
+ * `assistant` content blocks are `output_text` and `user` blocks are
+ * `input_text`; both are accepted alongside Claude's `text`. A `reasoning`,
+ * `custom_tool_call` or `event_msg` record contributes nothing, which is the
+ * same rule Claude's tool results already follow — a memory made of tool output
+ * is the transcript dump this repo refuses.
  */
 /** How much of the file's end is read. Big enough for a long final turn. */
 export declare const TAIL_BYTES: number;

package/dist/hooks/transcript.js

@@ -1,5 +1,6 @@
 /**
- * The last exchange out of a Claude Code transcript (BLI-3580).
+ * The last exchange out of an agent host's transcript (BLI-3580; Codex added
+ * by BLI-3729).
  *
  * The Stop hook is handed `transcript_path`: a JSONL file, one record per line,
  * appended to for the whole session. What the save door wants is the LAST
@@ -19,6 +20,31 @@
  * A record shape that is not recognised is skipped, never guessed at. The
  * reader returns null rather than half an exchange: half a turn saved as a
  * memory is a memory that says something the person did not.
+ *
+ * ## Two hosts, two record shapes (BLI-3729)
+ *
+ * `cockpit memory install` now registers the same three hooks with Codex, and
+ * Codex hands its Stop hook a `transcript_path` too — pointing at its own
+ * rollout JSONL, whose records are shaped differently:
+ *
+ *   Claude Code  { "message": { "role": "user", "content": [{ "type": "text", … }] } }
+ *   Codex        { "type": "response_item",
+ *                  "payload": { "type": "message", "role": "user",
+ *                               "content": [{ "type": "input_text", "text": … }] } }
+ *
+ * Both are read here, by the same backwards walk, because the alternative was a
+ * Codex Stop hook that fired on every turn and reported `transcript_no_exchange`
+ * every time — a hook that does nothing and says it did nothing. The Codex
+ * shape was captured from a real rollout file on 2026-09-05 rather than read
+ * off a doc, and the fixture in `hooks.test.ts` keeps the record structure
+ * verbatim with only the `text` values replaced (the real ones are somebody's
+ * conversation).
+ *
+ * `assistant` content blocks are `output_text` and `user` blocks are
+ * `input_text`; both are accepted alongside Claude's `text`. A `reasoning`,
+ * `custom_tool_call` or `event_msg` record contributes nothing, which is the
+ * same rule Claude's tool results already follow — a memory made of tool output
+ * is the transcript dump this repo refuses.
  */
 import fs from "node:fs";
 import { STOP_CHUNK_CHARS } from "./contract.js";
@@ -123,8 +149,8 @@ function parseJsonlTail(raw) {
 function roleOf(record) {
     if (!record)
         return "";
-    const message = record["message"];
-    if (message && typeof message === "object" && !Array.isArray(message)) {
+    const message = messageOf(record);
+    if (message) {
         const role = message["role"];
         if (typeof role === "string")
             return role;
@@ -133,18 +159,32 @@ function roleOf(record) {
     return typeof type === "string" ? type : "";
 }
 /**
- * The text of a record, whichever of the three shapes it uses: a string
- * `content`, an array of blocks with `type: "text"`, or a bare `text`. A tool
- * result or an image block contributes nothing, on purpose — a memory made of
- * tool output is the transcript dump this repo refuses.
+ * The record's message envelope, whichever host wrote it: Claude Code's
+ * `message`, Codex's `payload`, or the record itself when it carries `role`
+ * directly. Returns null for a record with no envelope at all, which is how a
+ * Codex `reasoning` or `event_msg` line drops out without a special case.
+ */
+function messageOf(record) {
+    for (const key of ["message", "payload"]) {
+        const value = record[key];
+        if (value && typeof value === "object" && !Array.isArray(value)) {
+            return value;
+        }
+    }
+    return null;
+}
+/** The content block types that carry a person's or the model's own words. */
+const TEXT_BLOCK_TYPES = new Set(["text", "input_text", "output_text"]);
+/**
+ * The text of a record, whichever of the shapes it uses: a string `content`, an
+ * array of blocks whose `type` is one of `TEXT_BLOCK_TYPES`, or a bare `text`.
+ * A tool result or an image block contributes nothing, on purpose — a memory
+ * made of tool output is the transcript dump this repo refuses.
  */
 function textOf(record) {
     if (!record)
         return "";
-    const message = record["message"];
-    const source = message && typeof message === "object" && !Array.isArray(message)
-        ? message
-        : record;
+    const source = messageOf(record) ?? record;
     const content = source["content"] ?? source["text"];
     if (typeof content === "string")
         return content.trim();
@@ -159,7 +199,9 @@ function textOf(record) {
         if (!block || typeof block !== "object")
             continue;
         const record_ = block;
-        if (record_["type"] !== "text")
+        if (typeof record_["type"] !== "string")
+            continue;
+        if (!TEXT_BLOCK_TYPES.has(record_["type"]))
             continue;
         const text = record_["text"];
         if (typeof text === "string")

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bli-cockpit/memory-mcp",
-  "version": "0.1.6",
+  "version": "0.1.7",
   "private": false,
   "description": "BLI Memory — an MCP server for the memory layer BLI owns (save, search, update, forget).",
   "type": "module",
@@ -28,7 +28,7 @@
     "start": "node dist/index.js"
   },
   "dependencies": {
-    "@bli-cockpit/telemetry-core": "0.1.28",
+    "@bli-cockpit/telemetry-core": "0.1.29",
     "@modelcontextprotocol/sdk": "^1.29.0",
     "zod": "^4.3.6"
   },