@north-light/crouter 0.3.181 → 0.3.182

package/dist/core/runtime/broker/inbox.d.ts

@@ -18,5 +18,9 @@ export declare function readBrokerCursor(nodeId: string): BrokerInboxEntryId | u
 export declare function readBrokerInboxSince(nodeId: string, cursor?: string): BrokerInboxEntry[];
 export declare function readBrokerInboxRange(nodeId: string, after: string | undefined, through: string): BrokerInboxEntry[];
 export declare function readBrokerCanceledEntryIds(nodeId: string): Set<BrokerInboxEntryId>;
+/** The node id a canonical report ref points at, or undefined when the ref is
+ *  absent, absolute, or not a report path. Callers use it to tell whether the
+ *  report-node projection they hold can still resolve the ref. */
+export declare function reportRefNodeId(ref: string | undefined): string | undefined;
 /** Render the existing inbox digest wording with daemon-projected report existence. */
 export declare function coalesceBrokerInbox(entries: readonly BrokerInboxEntry[], reportNodes: ReadonlyMap<string, BrokerReportNodeDTO>): string;

package/dist/core/runtime/broker/inbox.js

@@ -101,16 +101,25 @@ function clipBody(body) {
     }
     return { text: text.trimEnd(), clipped };
 }
-function inlineReport(entry, reportNodes) {
-    if (entry.ref === undefined || entry.ref.startsWith('/'))
-        return null;
-    const separator = entry.ref.indexOf(':');
+/** The node id a canonical report ref points at, or undefined when the ref is
+ *  absent, absolute, or not a report path. Callers use it to tell whether the
+ *  report-node projection they hold can still resolve the ref. */
+export function reportRefNodeId(ref) {
+    if (ref === undefined || ref.startsWith('/'))
+        return undefined;
+    const separator = ref.indexOf(':');
     if (separator <= 0)
+        return undefined;
+    const relpath = ref.slice(separator + 1);
+    if (!relpath.startsWith('reports/') || relpath.includes('..') || relpath.startsWith('/'))
+        return undefined;
+    return ref.slice(0, separator);
+}
+function inlineReport(entry, reportNodes) {
+    const nodeId = reportRefNodeId(entry.ref);
+    if (nodeId === undefined || !reportNodes.has(nodeId))
         return null;
-    const nodeId = entry.ref.slice(0, separator);
-    const relpath = entry.ref.slice(separator + 1);
-    if (!reportNodes.has(nodeId) || !relpath.startsWith('reports/') || relpath.includes('..') || relpath.startsWith('/'))
-        return null;
+    const relpath = entry.ref.slice(entry.ref.indexOf(':') + 1);
     const path = `${nodeDir(nodeId)}/${relpath}`;
     try {
         const body = readFileSync(path, 'utf8');

package/dist/pi-extensions/canvas-inbox-watcher.js

@@ -5,7 +5,7 @@
 // return is only a handoff. Exact cursor persistence at agent_settled is the
 // durable boundary; an uncommitted process exit is recovered by rereading from
 // the unchanged cursor.
-import { coalesceBrokerInbox, readBrokerCanceledEntryIds, readBrokerCursor, readBrokerInboxSince, } from '../core/runtime/broker/inbox.js';
+import { coalesceBrokerInbox, readBrokerCanceledEntryIds, readBrokerCursor, readBrokerInboxSince, reportRefNodeId, } from '../core/runtime/broker/inbox.js';
 import { advanceBrokerInboxCursor, brokerExtensionState } from '../core/runtime/broker/daemon-ops.js';
 import { emitEvent } from '../core/events/emit.js';
 import { errorClassFromError } from '../core/events/errors.js';
@@ -186,6 +186,8 @@ class CanvasInboxWatcher {
     fatalError;
     fatalEscaped = false;
     reportNodes = new Map();
+    reportNodesAsked = new Set();
+    reportNodesRefreshing = false;
     constructor(pi, cadence, generation) {
         this.pi = pi;
         this.cadence = cadence;
@@ -288,6 +290,7 @@ class CanvasInboxWatcher {
     async refreshReportNodes(nodeId) {
         if (nodeId === undefined)
             return;
+        this.reportNodesRefreshing = true;
         try {
             const state = await brokerExtensionState(nodeId);
             if (!this.canMutate())
@@ -297,6 +300,33 @@ class CanvasInboxWatcher {
         catch {
             // Report bodies remain canonical pointers until the daemon projection is available.
         }
+        finally {
+            this.reportNodesRefreshing = false;
+        }
+    }
+    /** True when delivery must wait a tick for a report-node projection refresh.
+     *  The projection captured at session_start omits every node created since —
+     *  each `crtr human ask` bridge and each child this node spawns — so their
+     *  reports would render as bare refs instead of inlined bodies. One refresh
+     *  per unknown node id keeps a genuinely missing node from stalling delivery. */
+    refreshForUnknownReportRefs() {
+        if (this.reportNodesRefreshing)
+            return true;
+        const unknown = [];
+        for (const entry of this.buffer) {
+            const refNodeId = reportRefNodeId(entry.ref);
+            if (refNodeId === undefined)
+                continue;
+            if (this.reportNodes.has(refNodeId) || this.reportNodesAsked.has(refNodeId))
+                continue;
+            unknown.push(refNodeId);
+        }
+        if (unknown.length === 0)
+            return false;
+        for (const refNodeId of unknown)
+            this.reportNodesAsked.add(refNodeId);
+        void this.refreshReportNodes(currentNodeId());
+        return this.reportNodesRefreshing;
     }
     arm() {
         if (this.armed)
@@ -343,6 +373,7 @@ class CanvasInboxWatcher {
             this.cursor = undefined;
             this.durableCursor = undefined;
             this.reportNodes.clear();
+            this.reportNodesAsked.clear();
             this.pendingCommit = undefined;
             this.nodeId = undefined;
             this.lastCtx = undefined;
@@ -509,6 +540,8 @@ class CanvasInboxWatcher {
             this.lastArrival = Date.now();
         }
         if (this.buffer.length > 0 && Date.now() - this.lastArrival >= this.cadence.debounceMs) {
+            if (this.refreshForUnknownReportRefs())
+                return;
             this.flushOrdinary(nodeId);
         }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@north-light/crouter",
-  "version": "0.3.181",
+  "version": "0.3.182",
   "description": "crtr — agent runtime with memory, plugins, and marketplaces",
   "type": "module",
   "main": "dist/index.js",

package/runtime.lock.json

@@ -1,12 +1,12 @@
 {
   "name": "@north-light/crouter",
-  "version": "0.3.181",
+  "version": "0.3.182",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@north-light/crouter",
-      "version": "0.3.181",
+      "version": "0.3.182",
       "hasInstallScript": true,
       "license": "MIT",
       "dependencies": {