@north-light/crouter 0.3.172 → 0.3.174

package/dist/pi-extensions/canvas-recap.js

@@ -38,11 +38,11 @@
 //
 // Plain TS-with-types — no imports from @earendil-works/* so this compiles
 // inside crouter's own tsc build without a dep on the pi packages.
-import { getNode, editorLabel } from '../core/canvas/index.js';
-import { regenerateAndPersistName } from '../core/runtime/naming-persist.js';
+import { brokerExtensionState } from '../core/runtime/broker/daemon-ops.js';
 import { generateRecap } from '../core/runtime/recap.js';
-import { readRoadmap } from '../core/runtime/roadmap.js';
 import { isReviewBoundary } from '../core/runtime/session-visibility.js';
+import { editorLabelForBrokerNode } from '../core/runtime/broker-extension-render.js';
+import { generateAndCommitRecapName, readRoadmap } from './broker-local.js';
 // ---------------------------------------------------------------------------
 // Tuning constants
 // ---------------------------------------------------------------------------
@@ -171,13 +171,14 @@ function buildNamingConversation(sm, assistantPartial, reviewId) {
  * Returns immediately (inert) when CRTR_NODE_ID is absent — a plain pi session
  * or legacy job agent loads it as a no-op.
  */
-export function registerCanvasRecap(pi, recapGenerator = generateRecap, nameRegenerator = regenerateAndPersistName) {
+export function registerCanvasRecap(pi, recapGenerator = generateRecap, nameRegenerator) {
     const nodeId = process.env['CRTR_NODE_ID'];
     if (nodeId === undefined || nodeId.trim() === '')
         return; // not a canvas node
     // Captured on session_start; used by the timer and the message handlers.
     let ui;
     let sessionManager;
+    let brokerNode;
     /** Exact provider/model that most recently ran this conversation successfully.
      *  The recap subprocess prefers it over a hard-pinned provider that may be in
      *  cooldown (the failure mode that made the old recap silently disappear). */
@@ -248,23 +249,42 @@ export function registerCanvasRecap(pi, recapGenerator = generateRecap, nameRege
     const refreshAutomaticName = (conversation) => {
         if (pi.getSessionName === undefined || pi.setSessionName === undefined)
             return;
-        const meta = getNode(nodeId);
-        if (meta === null)
-            return;
-        // `/rename` persists a person's explicit handle on the node as well as in
-        // the pi session. Such a handle remains authoritative even when its live
-        // session text exactly matches editorLabel(meta).
-        if (meta.name.trim() !== '' && meta.name !== meta.kind)
-            return;
-        const original = editorLabel(meta);
-        if (pi.getSessionName() !== original)
+        // Keep the injectable test seam independent of the daemon. The production
+        // regenerator below is the broker-state closure; it captures its own
+        // compare-and-set projection before asking the headless namer.
+        if (nameRegenerator !== undefined) {
+            const original = pi.getSessionName();
+            if (original === undefined)
+                return;
+            nameRegenerator(nodeId, conversation, () => pi.getSessionName?.() === original, (label) => {
+                try {
+                    pi.setSessionName?.(label);
+                }
+                catch { /* best-effort */ }
+            });
             return;
-        nameRegenerator(nodeId, conversation, () => pi.getSessionName?.() === original, (named) => {
-            try {
-                pi.setSessionName?.(editorLabel(named));
-            }
-            catch { /* best-effort */ }
-        });
+        }
+        void brokerExtensionState(nodeId).then(({ node }) => {
+            // `/rename` persists a person's explicit handle on the node as well as in
+            // the pi session. Such a handle remains authoritative even when its live
+            // session text exactly matches the generated editor label.
+            if (node.name.trim() !== '' && node.name !== node.kind)
+                return;
+            const original = editorLabelForBrokerNode(node);
+            if (pi.getSessionName?.() !== original)
+                return;
+            generateAndCommitRecapName(nodeId, conversation, {
+                name: node.name,
+                description: node.description ?? '',
+                icon: node.icon ?? '',
+                kind: node.kind,
+            }, () => pi.getSessionName?.() === original, (label) => {
+                try {
+                    pi.setSessionName?.(label);
+                }
+                catch { /* best-effort */ }
+            });
+        }).catch(() => { });
     };
     /** Start one briefing generation. A manual request refreshes an untouched
      *  generated title as a separate best-effort task from the broader conversation. */
@@ -272,7 +292,10 @@ export function registerCanvasRecap(pi, recapGenerator = generateRecap, nameRege
         if (!canPaint() || sessionManager === undefined || generating)
             return;
         const assistantPartial = liveAssistantPartial;
-        const reviewId = getNode(nodeId)?.review_binding?.review_id;
+        const reviewId = brokerNode?.review_binding?.review_id
+            ?? sessionManager.getEntries()
+                .map((entry) => entry.details?.reviewId)
+                .find((id) => typeof id === 'string');
         const conversation = buildConversation(sessionManager, assistantPartial, reviewId);
         if (conversation === '') {
             recapInputUnavailable = true;
@@ -328,6 +351,10 @@ export function registerCanvasRecap(pi, recapGenerator = generateRecap, nameRege
         ui = ctx.ui;
         sessionManager = ctx.sessionManager;
         captureLiveModel(ctx);
+        brokerNode = undefined;
+        void brokerExtensionState(nodeId).then((state) => {
+            brokerNode = state.node;
+        }).catch(() => { });
         // Fresh session / hot-swap: start the idle clock now and drop any stale
         // recap that bled through a /reload.
         lastMessageAt = Date.now();

package/dist/pi-extensions/canvas-review-boundary.js

@@ -4,7 +4,7 @@
 // inherited fork context and before canvas-context-intro appends fork bearings.
 // The display projection starts at this marker while the model keeps the full
 // inherited context.
-import { getNode } from '../core/canvas/index.js';
+import { brokerExtensionState } from '../core/runtime/broker/daemon-ops.js';
 import { isReviewBoundary, REVIEW_BOUNDARY_CUSTOM_TYPE } from '../core/runtime/session-visibility.js';
 // ---------------------------------------------------------------------------
 // Collapsed-by-default rendering
@@ -47,12 +47,18 @@ export function renderReviewBoundary(_message, _options, theme) {
  */
 export function registerCanvasReviewBoundary(pi) {
     pi.registerMessageRenderer(REVIEW_BOUNDARY_CUSTOM_TYPE, renderReviewBoundary);
-    pi.on('session_start', (_event, ctx) => {
+    pi.on('session_start', async (_event, ctx) => {
         const nodeId = process.env['CRTR_NODE_ID'];
         if (!nodeId || nodeId.trim() === '')
             return;
-        const node = getNode(nodeId);
-        if (node?.review_binding == null)
+        let node;
+        try {
+            node = (await brokerExtensionState(nodeId)).node;
+        }
+        catch {
+            return;
+        }
+        if (node.review_binding == null)
             return;
         // The active branch, not the whole session file, is the context the next
         // turn sees. Check for an existing boundary marker on this branch.

package/dist/pi-extensions/canvas-stophook.d.ts

@@ -1,9 +1,4 @@
 type PiEvents = 'agent_start' | 'turn_end' | 'agent_end' | 'agent_settled' | 'session_shutdown' | 'session_start' | 'input' | 'tool_execution_start' | 'tool_execution_end' | 'tool_execution_update' | 'message_update';
-/** Universal upper bound for an unfocused broker parked without runnable work. */
-export declare const UNATTENDED_PARK_MS: number;
-/** Production cadence for the bounded-release clock. A shorter test bound also
- * shortens the cadence, so the full-tier regression need not sleep for a minute. */
-export declare const PARK_POLL_MS: number;
 interface PiLike {
     on: (event: PiEvents, handler: (event: any, ctx: any) => void | Promise<void>) => void;
     sendUserMessage: (content: string, options?: {