@north-light/crouter 0.3.190 → 0.3.191

package/dist/core/runtime/broker/node-named.d.ts

@@ -0,0 +1,14 @@
+/** What crtrd stored, as it stored it — the payload of a `node_named` frame. */
+export interface NodeNamedNotice {
+    description: string;
+    icon: string;
+    editorLabel: string;
+}
+/** Register the broker's fan-out. One broker hosts one pi process, so the last
+ *  registration wins; a second broker in one process is not a thing that
+ *  exists. */
+export declare function onNodeNamed(fn: (notice: NodeNamedNotice) => void): void;
+/** Announce a generated name crtrd's guarded commit APPLIED. A no-op when no
+ *  broker is listening — the same extension code also runs under test harnesses
+ *  that host no client registry. */
+export declare function publishNodeNamed(notice: NodeNamedNotice): void;

package/dist/core/runtime/broker/node-named.js

@@ -0,0 +1,27 @@
+// node-named.ts — the in-process hop from the naming extension to the broker's
+// client fan-out.
+//
+// A node's generated name is committed by the naming extension
+// (`pi-extensions/broker-local.ts`), which runs INSIDE the broker's own pi
+// process but holds no reference to the broker's client registry: pi loads
+// extensions by path, so there is no constructor seam to pass one through. This
+// module is that seam — one module-level slot, both ends resolving the same
+// file in the same process.
+//
+// The frame it carries (`node_named`) exists because the name lands seconds
+// AFTER the turn that triggered it: a consumer settling its display state on
+// turn-end reads an unnamed node and has nothing left to re-read on. The
+// committing side pushes instead.
+let listener;
+/** Register the broker's fan-out. One broker hosts one pi process, so the last
+ *  registration wins; a second broker in one process is not a thing that
+ *  exists. */
+export function onNodeNamed(fn) {
+    listener = fn;
+}
+/** Announce a generated name crtrd's guarded commit APPLIED. A no-op when no
+ *  broker is listening — the same extension code also runs under test harnesses
+ *  that host no client registry. */
+export function publishNodeNamed(notice) {
+    listener?.(notice);
+}

package/dist/core/runtime/broker-protocol.d.ts

@@ -1,6 +1,6 @@
 import type { AgentSession, AgentSessionEvent, RpcExtensionUIRequest, RpcExtensionUIResponse, RpcSessionState, SessionEntry, SessionInfo, SettingsConfig, SessionStats } from '@earendil-works/pi-coding-agent';
 import type { Api, ImageContent, Model } from '@earendil-works/pi-ai';
-import type { BrokerErrorFrame } from '../../api/dto/broker.js';
+import type { BrokerErrorFrame, BrokerNodeNamedFrame } from '../../api/dto/broker.js';
 import type { ToolGroupSummary } from '../../shared/tool-groups.js';
 /** pi's own RPC types, re-exported unchanged. `broker-protocol` stays the one
  *  import site for the wire union; the shapes themselves are pi's, versioned by
@@ -386,6 +386,15 @@ export interface ModelChangedFrame {
  *  promise instead of hanging it — absent on uncorrelated errors (engine drive
  *  errors, command-op failures, frame_overflow). There is NO `retryable` field. */
 export type ErrorFrame = BrokerErrorFrame;
+/** The `node_named` control frame. SOURCED from the shared
+ *  `@north-light/crouter-api` contract for the same reason `error` is: Northlight
+ *  Core reads this frame off its own attach, so the wire shape here and the shape
+ *  it consumes are one compiler-checked type.
+ *
+ *  Broadcast when the naming extension's GUARDED commit applied — a name a human
+ *  already set never produces one. The in-process hop from the extension to this
+ *  fan-out is `broker/node-named.ts`. */
+export type NodeNamedFrame = BrokerNodeNamedFrame;
 /** Result of a command op (§1.3): `for` echoes the op name, `ok` the
  *  outcome, `detail` an optional human-readable note. */
 export interface AckFrame {
@@ -589,7 +598,7 @@ export interface ToolGroupSummaryFrame {
 /** Everything the broker can send. Live `AgentSessionEvent`s are relayed
  *  verbatim (the broker adds nothing); the broker's own control frames carry
  *  non-colliding `type` discriminants. */
-export type BrokerToClient = WelcomeFrame | WorkingActivityFrame | ModelChangedFrame | ErrorFrame | AckFrame | BrokerDataFrame | BashStartFrame | BashOutputFrame | BashEndFrame | ExtensionUIRequestFrame | ExtensionUIDismissFrame | ToolGroupSummaryFrame | AgentSessionEvent;
+export type BrokerToClient = WelcomeFrame | WorkingActivityFrame | ModelChangedFrame | ErrorFrame | NodeNamedFrame | AckFrame | BrokerDataFrame | BashStartFrame | BashOutputFrame | BashEndFrame | ExtensionUIRequestFrame | ExtensionUIDismissFrame | ToolGroupSummaryFrame | AgentSessionEvent;
 /** Encode one frame as a single newline-terminated JSON line. */
 export declare function encodeFrame(frame: ClientToBroker | BrokerToClient): string;
 /** Byte bounds for a {@link FrameDecoder} (C5). */

package/dist/core/runtime/broker.js

@@ -42,6 +42,7 @@ import { visibleMessages } from './session-visibility.js';
 import { isUnknownModel } from './model-selection.js';
 import { BrokerClientRegistry } from './broker/client-registry.js';
 import { ToolGroupTracker } from './broker/tool-groups.js';
+import { onNodeNamed } from './broker/node-named.js';
 import { FaultRetry } from './broker/fault-retry.js';
 import { EventProjection } from './broker/event-projection.js';
 import { createFrameDispatchContext, handleFrame } from './broker/frame-dispatch.js';
@@ -275,6 +276,11 @@ export async function runBroker(nodeId, startupAt = process.hrtime.bigint(), sta
     const registry = new BrokerClientRegistry(nodeId);
     const pendingDialogs = new Map();
     const toolGroups = new ToolGroupTracker(nodeId, (frame) => registry.broadcast(frame));
+    // The naming extension commits the node's generated name a few seconds into
+    // its first turn and hands it here (broker/node-named.ts). Every attached
+    // client learns it on the pipe it already holds instead of re-reading the node
+    // on some later, unrelated trigger.
+    onNodeNamed((notice) => registry.broadcast({ type: 'node_named', ...notice }));
     let disposed = false;
     let disposePending = false;
     let server;

package/dist/core/runtime/naming.d.ts

@@ -34,9 +34,9 @@ export declare function sanitizeIcon(raw: string): string;
  *  to a sanitized SessionName, or `{name:'',icon:''}` on timeout, a missing tool
  *  call, or a malformed emission so the caller can fall back to a local slug.
  *
- *  Exported (not just for tests): the canvas-coupled persistence wrapper
- *  generateAndPersistName lives in naming-persist.ts to keep this module
- *  canvas-free, and imports this pure helper. */
+ *  This module stays canvas-free so CLI leaves can reach it. The canvas-coupled
+ *  commit path lives in pi-extensions/broker-local.ts, which imports this pure
+ *  helper and writes through crtrd's guarded generated-name operation. */
 export declare function headlessName(body: string, sourceCap?: number): Promise<SessionName>;
 /** Generate a name from a bounded conversation source rather than a single
  *  kickoff message. The caller owns selecting representative conversation text;

package/dist/core/runtime/naming.js

@@ -185,9 +185,9 @@ export function sanitizeIcon(raw) {
  *  to a sanitized SessionName, or `{name:'',icon:''}` on timeout, a missing tool
  *  call, or a malformed emission so the caller can fall back to a local slug.
  *
- *  Exported (not just for tests): the canvas-coupled persistence wrapper
- *  generateAndPersistName lives in naming-persist.ts to keep this module
- *  canvas-free, and imports this pure helper. */
+ *  This module stays canvas-free so CLI leaves can reach it. The canvas-coupled
+ *  commit path lives in pi-extensions/broker-local.ts, which imports this pure
+ *  helper and writes through crtrd's guarded generated-name operation. */
 export async function headlessName(body, sourceCap = PROMPT_CAP) {
     const empty = { name: '', icon: '' };
     const route = nameModelRequest();

package/dist/daemon/api/handlers/broker-ops.js

@@ -350,7 +350,12 @@ function handleGeneratedName(ctx) {
                 description: request.description,
                 ...(request.icon.trim() === '' ? {} : { icon: request.icon }),
             });
-            return { applied: true, editorLabel: editorLabel(updated) };
+            return {
+                applied: true,
+                editorLabel: editorLabel(updated),
+                description: updated.description ?? '',
+                icon: updated.icon ?? '',
+            };
         }
         const expected = request.expected;
         if (node.name !== expected.name || (node.description ?? '') !== expected.description
@@ -360,7 +365,12 @@ function handleGeneratedName(ctx) {
             description: request.description,
             icon: request.icon.trim() === '' ? undefined : request.icon,
         });
-        return { applied: true, editorLabel: editorLabel(updated) };
+        return {
+            applied: true,
+            editorLabel: editorLabel(updated),
+            description: updated.description ?? '',
+            icon: updated.icon ?? '',
+        };
     });
     return { status: 200, body: result };
 }

package/dist/pi-extensions/broker-local.js

@@ -9,6 +9,7 @@ import { join } from 'node:path';
 import { CRTR_DIR_NAME } from '../types.js';
 import { REVIVE_KICKOFF_SENTINEL } from '../shared/generated-context.js';
 import { brokerExtensionState, commitBrokerGeneratedName, } from '../core/runtime/broker/daemon-ops.js';
+import { publishNodeNamed } from '../core/runtime/broker/node-named.js';
 import { headlessName, headlessNameFromConversation, sanitizeSessionName, slugFromPrompt, } from '../core/runtime/naming.js';
 export { REVIVE_KICKOFF_SENTINEL };
 function contextDir(nodeId) {
@@ -58,8 +59,20 @@ export function generateAndCommitInitialName(nodeId, prompt, onNamed) {
         }));
     })
         .then((result) => {
-        if (result?.applied && result.editorLabel !== undefined)
-            onNamed?.(result.editorLabel);
+        if (!result?.applied || result.editorLabel === undefined)
+            return;
+        // Push the committed name down the attach stream BEFORE updating the live
+        // session label: both are the same delivery — the label reaches this
+        // process's own editor chip, the frame reaches every attached client
+        // (Northlight Core titles the conversation from it). The name lands
+        // seconds after the turn that triggered it, so a consumer with no frame
+        // has nothing left to re-read on.
+        publishNodeNamed({
+            description: result.description ?? '',
+            icon: result.icon ?? '',
+            editorLabel: result.editorLabel,
+        });
+        onNamed?.(result.editorLabel);
     })
         .catch(() => { });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@north-light/crouter",
-  "version": "0.3.190",
+  "version": "0.3.191",
   "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.190",
+  "version": "0.3.191",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@north-light/crouter",
-      "version": "0.3.190",
+      "version": "0.3.191",
       "hasInstallScript": true,
       "license": "MIT",
       "dependencies": {

package/dist/core/runtime/naming-persist.d.ts

@@ -1,16 +0,0 @@
-import type { NodeMeta } from '../canvas/index.js';
-/** Asynchronously generate a name for `prompt` and persist it to the node's
- *  meta as `description` + `icon` — only if the node has none yet (so a later
- *  message never clobbers it). Non-blocking: safe to call from inside a live pi
- *  event loop. Best-effort; swallows all errors. The slug fallback carries no
- *  icon: a locally derived name has no judgment behind it to pick a glyph from.
- *
- *  `onNamed` (optional) fires with the freshly-persisted meta the moment the
- *  name lands — the canvas-goal-capture naming hook passes a callback that calls
- *  pi.setSessionName(editorLabel(meta)) so the LIVE editor label updates in the
- *  same session, instead of waiting for the next revive/cycle. */
-export declare function generateAndPersistName(nodeId: string, prompt: string, onNamed?: (meta: NodeMeta) => void): void;
-/** Re-name an automatically titled node from its current conversation. The caller
- *  supplies the guard that distinguishes the original generated editor title from
- *  a title a person changed while this headless request was in flight. */
-export declare function regenerateAndPersistName(nodeId: string, conversation: string, isStillAutomatic: () => boolean, onNamed?: (meta: NodeMeta) => void): void;

package/dist/core/runtime/naming-persist.js

@@ -1,68 +0,0 @@
-// Session naming — canvas-coupled persistence seam.
-//
-// The pure naming machinery (headless pi call + slug fallbacks) lives in
-// naming.ts and stays canvas-free so CLI leaves can reach it without dragging
-// in canvas state. This one function is the only naming path that writes to a
-// node's meta, so it lives here — broker-side (canvas-goal-capture), never on a
-// CLI leaf's import graph.
-import { headlessName, headlessNameFromConversation, sanitizeSessionName, slugFromPrompt } from './naming.js';
-import { getNode, updateNode } from '../canvas/index.js';
-/** Asynchronously generate a name for `prompt` and persist it to the node's
- *  meta as `description` + `icon` — only if the node has none yet (so a later
- *  message never clobbers it). Non-blocking: safe to call from inside a live pi
- *  event loop. Best-effort; swallows all errors. The slug fallback carries no
- *  icon: a locally derived name has no judgment behind it to pick a glyph from.
- *
- *  `onNamed` (optional) fires with the freshly-persisted meta the moment the
- *  name lands — the canvas-goal-capture naming hook passes a callback that calls
- *  pi.setSessionName(editorLabel(meta)) so the LIVE editor label updates in the
- *  same session, instead of waiting for the next revive/cycle. */
-export function generateAndPersistName(nodeId, prompt, onNamed) {
-    const body = (prompt ?? '').trim();
-    if (body === '')
-        return;
-    const persist = (generated) => {
-        try {
-            const meta = getNode(nodeId);
-            if (meta === null)
-                return;
-            if ((meta.description ?? '').trim() !== '')
-                return; // already named
-            const clean = sanitizeSessionName(generated.name);
-            const description = clean !== '' ? clean : slugFromPrompt(body);
-            const icon = generated.icon.trim();
-            const updated = updateNode(nodeId, { description, ...(icon !== '' ? { icon } : {}) });
-            onNamed?.(updated);
-        }
-        catch {
-            // best-effort
-        }
-    };
-    void headlessName(body).then(persist);
-}
-/** Re-name an automatically titled node from its current conversation. The caller
- *  supplies the guard that distinguishes the original generated editor title from
- *  a title a person changed while this headless request was in flight. */
-export function regenerateAndPersistName(nodeId, conversation, isStillAutomatic, onNamed) {
-    const body = (conversation ?? '').trim();
-    if (body === '')
-        return;
-    const persist = (generated) => {
-        try {
-            if (!isStillAutomatic())
-                return;
-            const meta = getNode(nodeId);
-            if (meta === null)
-                return;
-            const clean = sanitizeSessionName(generated.name);
-            const description = clean !== '' ? clean : slugFromPrompt(body);
-            const icon = generated.icon.trim();
-            const updated = updateNode(nodeId, { description, icon: icon !== '' ? icon : undefined });
-            onNamed?.(updated);
-        }
-        catch {
-            // best-effort
-        }
-    };
-    void headlessNameFromConversation(body).then(persist);
-}