@volter/supercode-interchange 0.1.15 → 0.1.16

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@volter/supercode-interchange",
-  "version": "0.1.15",
+  "version": "0.1.16",
   "type": "module",
   "description": "Read-only harness metadata, transcript codecs and session projections for Supercode",
   "license": "MIT OR Apache-2.0",

package/session-codex-display.mjs

@@ -15,6 +15,9 @@ export function fromCodexDisplayStr(text, messageLimit) {
   if (!Number.isSafeInteger(messageLimit) || messageLimit < 0) throw new RangeError('invalid Codex display message limit');
   const session = createSession('codex'), { meta, messages } = session, precedingUsers = [], canonicalTexts = new Set();
   const retain = Math.min(Number.MAX_SAFE_INTEGER, Math.max(messageLimit, 1) + 64);
+  // The item ids and event texts the kept messages may hold: a record that names neither replaces nothing and looks
+  // through no message, so a read with no bound on its messages (a display anchor's) is linear in what it reads.
+  const displayItems = new Set(), eventTexts = new Set();
   let paginated = false, recordCount = 0, total = 0;
   for (const line of nonEmptyLines(text)) {
     recordCount++;
@@ -24,12 +27,13 @@ export function fromCodexDisplayStr(text, messageLimit) {
     else if (kind === 'event_msg' && paginated && type === 'item_completed') {
       const item = field(payload, 'item'), id = stringField(item, 'id');
       if (id != null) {
-        const position = messages.findIndex((message) => message.metadata.codex_display_item === id), projected = [];
+        const held = displayItems.has(id), position = held ? messages.findIndex((message) => message.metadata.codex_display_item === id) : -1, projected = [];
         pushCodexDisplayItem(item, projected);
         for (const message of projected) message.metadata.codex_display_item = id;
+        if (projected.length) displayItems.add(id);
         stampCodexMessages(projected, 0, timestamp);
         let removed = 0;
-        for (let index = messages.length - 1; index >= 0; index--) if (messages[index].metadata.codex_display_item === id) { messages.splice(index, 1); removed++; }
+        if (held) for (let index = messages.length - 1; index >= 0; index--) if (messages[index].metadata.codex_display_item === id) { messages.splice(index, 1); removed++; }
         total = Math.max(0, total - removed) + projected.length;
         const at = Math.min(position === -1 ? messages.length : position, messages.length);
         for (let index = 0; index < projected.length; index++) messages.splice(at + index, 0, projected[index]);
@@ -47,7 +51,7 @@ export function fromCodexDisplayStr(text, messageLimit) {
       const isAssistant = type === 'message' && stringField(payload, 'role') === 'assistant';
       const assistantText = isAssistant ? extractTextContent(payload, 'content') : null;
       if (assistantText != null && nativeTrim(assistantText) !== '') {
-        const duplicate = messages.findLastIndex((message) => Object.hasOwn(message.metadata, 'codex_event_message') && message.content === assistantText);
+        const duplicate = eventTexts.has(assistantText) ? messages.findLastIndex((message) => Object.hasOwn(message.metadata, 'codex_event_message') && message.content === assistantText) : -1;
         if (duplicate !== -1) { messages.splice(duplicate, 1); total = Math.max(0, total - 1); }
         canonicalTexts.add(nativeTrim(assistantText));
       }
@@ -62,7 +66,7 @@ export function fromCodexDisplayStr(text, messageLimit) {
       const text = extractTextContent(payload, 'message');
       if (nativeTrim(text) !== '' && !canonicalTexts.has(nativeTrim(text))) {
         const before = messages.length; pushCodexAssistant(messages, text); total++;
-        messages.at(-1).metadata.codex_event_message = 'true';
+        messages.at(-1).metadata.codex_event_message = 'true'; eventTexts.add(messages.at(-1).content);
         const phase = stringField(payload, 'phase'); if (phase != null) messages.at(-1).metadata.phase = phase;
         stampCodexMessages(messages, before, timestamp);
       }