@north-light/crouter 0.3.224 → 0.3.226

package/dist/clients/inbox/page-adapter.js

@@ -1,6 +1,6 @@
+import { pageAwaitsResponse } from '../../core/human/page-schema.js';
 import { writePageProgress } from '../../core/human/tickets.js';
 import { mountPanel } from './tui/panel.js';
-import { awaitsResponse } from './tui/slots.js';
 /** Embeds the page renderer in a controller-owned rectangle. */
 export class PageAdapter {
     opts;
@@ -25,7 +25,8 @@ export class PageAdapter {
                 opts.onProgress?.(responses);
             },
             onComplete: opts.onComplete,
-            onExit: () => { if (awaitsResponse(this.manifest))
+            // Leaving a page that asks something must not settle it on an empty answer.
+            onExit: () => { if (pageAwaitsResponse(this.manifest))
                 opts.onBack();
             else
                 opts.onComplete({}); },

package/dist/clients/inbox/tui/slots.d.ts

@@ -1,4 +1,4 @@
-import type { CommentAnchor, PageManifest, PageSlot, SlotResponse, TableConfig } from '../../../core/human/page-schema.js';
+import type { CommentAnchor, PageSlot, SlotResponse, TableConfig } from '../../../core/human/page-schema.js';
 import type { TuiState } from './types.js';
 export type SelectGroup = 'option' | 'card' | 'row' | 'column';
 export type ActionRow = {
@@ -31,4 +31,3 @@ export declare function pageComplete(state: Pick<TuiState, 'manifest' | 'respons
 export declare function commentIdFor(anchor: CommentAnchor): string;
 /** The response-comment anchors this component may author: options take per-option, tables take rows only. */
 export declare function commentAnchors(slot: PageSlot): CommentAnchor[];
-export declare function awaitsResponse(manifest: PageManifest): boolean;

package/dist/clients/inbox/tui/slots.js

@@ -132,6 +132,3 @@ export function commentAnchors(slot) {
         return (slot.config.rows ?? []).map((row) => ({ kind: 'row', rowId: row.id }));
     return [];
 }
-export function awaitsResponse(manifest) {
-    return manifest.delivery.reply;
-}

package/dist/core/__tests__/seam/dormancy-release.test.js

@@ -19,7 +19,7 @@
 // they belong to the compiled seam tier; neither needs tmux.
 import { after, before, test } from 'node:test';
 import assert from 'node:assert/strict';
-import { readFileSync, writeFileSync } from 'node:fs';
+import { existsSync, readFileSync, writeFileSync } from 'node:fs';
 import { join } from 'node:path';
 import { appendInbox, readCursor, readInboxSince } from '../../feed/inbox.js';
 import { registerWarmSpare, subscribe, updateNode } from '../../canvas/index.js';
@@ -243,9 +243,15 @@ test('a connected broker viewer keeps an obligation-free resident live', { timeo
     });
     reviveNode(nodeId, { resume: true });
     const boot = await h.awaitBoot(nodeId);
+    // awaitBoot returns on the boot proof, which is written BEFORE the broker's
+    // own `persistAttachState()` lays down the truthful zero state. Writing our
+    // presence first lets that boot write clobber it, and the daemon then parks a
+    // node the test believes is attended — wait for the broker's file to exist.
+    const attachPath = join(h.home, 'nodes', nodeId, 'job', 'attach.json');
+    await h.waitFor(() => existsSync(attachPath), { timeoutMs: 20_000, intervalMs: 20, label: 'broker persists its boot attach state' });
     // This is the presence written by a helloed direct attach, which has no
     // focuses-table row because it did not pass through the focus opener.
-    writeFileSync(join(h.home, 'nodes', nodeId, 'job', 'attach.json'), JSON.stringify({ viewers: 1 }));
+    writeFileSync(attachPath, JSON.stringify({ viewers: 1 }));
     try {
         const seenAt = 15_000;
         await h.tick(seenAt);

package/dist/core/human/__tests__/page-tickets.test.js

@@ -116,7 +116,7 @@ test('readPageTicket derives pending, resolved and canceled state and carries th
     cancelTicket(canceledDir, { reason: 'no longer needed' });
     assert.equal(readPageTicket(canceledDir).state, 'canceled');
 });
-test('page scan derives response expectation from manifest delivery', () => {
+test('page scan derives response expectation from the page shape and its reply route', () => {
     const dir = submit('display', displayPage, '2026-01-04T00:00:00.000Z');
     const ticket = readPageTicket(dir);
     assert.equal(ticket.manifest.dialect, 'jsx');
@@ -135,6 +135,21 @@ test('page scan derives response expectation from manifest delivery', () => {
     assert.deepEqual(written['responses'], {});
     assert.equal(written['summary'], '');
 });
+test('an inbox page that asks something waits on the person with no reply route', () => {
+    const stamp = (emittedAt) => ({ nodeId: 'node-durable', emittedAt });
+    // The programmatic request shape: answerable from the inbox, never routed to an agent bridge.
+    const durable = submitPage({ dir: ticketDir('durable-request'), sourceFile: askPage, source: stamp('2026-01-06T00:00:00.000Z'), delivery: { placement: 'panel', inbox: true, reply: false } }).dir;
+    submitPage({ dir: ticketDir('durable-notice'), sourceFile: displayPage, source: stamp('2026-01-06T00:00:01.000Z'), delivery: { placement: 'panel', inbox: true, reply: false } });
+    const listed = scanInbox();
+    const request = listed.find((item) => item.id === 'durable-request');
+    assert.equal(request?.kind === 'page' ? request.awaitsResponse : undefined, true, 'a page with response-bearing slots waits on the person even with no bridge to reply to');
+    const notice = listed.find((item) => item.id === 'durable-notice');
+    assert.equal(notice?.kind === 'page' ? notice.awaitsResponse : undefined, false, 'a display-only notice waits on nothing');
+    const claim = claimTicket(durable);
+    const answered = completePage(durable, { rollout: { selectedOptionIds: ['canary'], comments: [] } }, claim.token);
+    assert.equal(answered.won, true, 'a request with no reply route still takes a real answer');
+    assert.deepEqual(JSON.parse(readFileSync(responsePath(durable), 'utf8'))['responses'], { rollout: { selectedOptionIds: ['canary'], comments: [] } });
+});
 test('scanPageHistory returns every page of one node, oldest first, regardless of terminal file', () => {
     submit('older', askPage, '2026-01-01T00:00:00.000Z');
     submit('other-node', askPage, '2026-01-02T00:00:00.000Z', 'node-2');

package/dist/core/human/page-schema.d.ts

@@ -219,6 +219,7 @@ export interface PageSource {
 export interface PageDelivery {
     placement: 'inline' | 'panel';
     inbox: boolean;
+    /** Routing only: whether an answer travels back to a resident agent's bridge. Never whether the page can be answered — {@link pageAwaitsResponse} decides that. */
     reply: boolean;
 }
 export interface PageManifest {
@@ -269,6 +270,8 @@ export interface PageOverviewActions {
 }
 /** Derive both overview facts from a manifest alone. Pure: no ticket state, no second read. */
 export declare function pageOverviewActions(manifest: PageManifest): PageOverviewActions;
+/** Whether this page is waiting on a person: it asks something, or an agent bridge is parked on its reply. */
+export declare function pageAwaitsResponse(manifest: PageManifest): boolean;
 /** The canonical response-bearing predicate used by every page consumer. */
 export declare function isResponseBearingSlot(slot: PageSlot): boolean;
 /** Validate a persisted manifest. Publish-time validation made its slots authoritative, so reads do not resolve a component catalog. */

package/dist/core/human/page-schema.js

@@ -149,6 +149,10 @@ export function pageOverviewActions(manifest) {
         fastAction: { kind: 'answer', responses: { [slot.id]: { selectedOptionIds: [recommendation.optionId], comments: [] } } },
     };
 }
+/** Whether this page is waiting on a person: it asks something, or an agent bridge is parked on its reply. */
+export function pageAwaitsResponse(manifest) {
+    return manifest.delivery.reply || manifest.slots.some(isResponseBearingSlot);
+}
 /** The canonical response-bearing predicate used by every page consumer. */
 export function isResponseBearingSlot(slot) {
     if (slot.display === true)

package/dist/core/human/scan.js

@@ -3,7 +3,7 @@ import { basename, join } from 'node:path';
 import { claimPath, isResolved, pageManifestPath, reviewPath } from './convention.js';
 import { readJsonOrNull } from '../fs-utils.js';
 import { validateReviewDescriptor } from './review-schema.js';
-import { pageOverviewActions } from './page-schema.js';
+import { pageAwaitsResponse, pageOverviewActions } from './page-schema.js';
 import { parsePage } from './page.js';
 import { pageTicketState, readTicketResult } from './tickets.js';
 import { ticketsRoot } from './root.js';
@@ -56,7 +56,7 @@ function pageSummary(dir, id) {
         inbox: manifest.delivery.inbox,
         steps: manifest.steps,
         slotKinds: manifest.slots.map((slot) => slot.kind),
-        awaitsResponse: manifest.delivery.reply,
+        awaitsResponse: pageAwaitsResponse(manifest),
         source: manifest.source ?? {},
         emittedAt,
         state,

package/dist/core/human/types.d.ts

@@ -64,6 +64,7 @@ export interface PageTicketSummary {
     inbox: boolean;
     steps: number;
     slotKinds: string[];
+    /** This ticket is waiting on the person: it asks something, or an agent bridge is parked on its reply. */
     awaitsResponse: boolean;
     /** Every question's recommended option, for a row that says what is suggested without opening the page. */
     recommendedOptions: import('./page-schema.js').RecommendedOption[];

package/dist/core/runtime/node-read.js

@@ -55,6 +55,19 @@ function resolveSessionFile(nodeId, metaFile) {
         return raw;
     return null;
 }
+/** The node's readable session file, or `null` when the node exists but nothing
+ *  has been written to its session yet. pi's SessionManager computes a session's
+ *  `.jsonl` path at `newSession()` and writes the file on the FIRST appended
+ *  entry, so a freshly created node carries a bound `pi_session_file` that is
+ *  not on disk until something is said. Throws only when the node itself is
+ *  unknown. */
+function readableSessionFile(nodeId) {
+    const node = getNode(nodeId);
+    if (node === null) {
+        throw new InputError({ error: 'not_found', message: `no node: ${nodeId}`, field: 'id', next: 'List nodes with `crtr node inspect list`.' });
+    }
+    return resolveSessionFile(nodeId, node.pi_session_file);
+}
 function emptyStats(sessionId, sessionFile) {
     return {
         sessionFile,
@@ -266,6 +279,19 @@ export async function readNodeMessagesPage(nodeId, { cursor, limit = 200 }) {
             next: 'Use a limit from 1 through 500.',
         });
     }
+    // A node that exists but has said nothing has an EMPTY history, not a fault:
+    // its session file is not on disk until the first entry is appended, and a
+    // transcript read is how a UI opens a new conversation. Only this paged read
+    // can answer honestly — `/snapshot` would have to invent the engine state a
+    // never-run node does not have, so it and the raw `/session` export still
+    // reject the same state through `reconstructVisibleMessages`.
+    // Same shape a session-file-backed empty read produces: `messageIds` stays
+    // aligned 1:1 with `messages` (an absent `message_ids` would claim this read
+    // cannot identify entries, which is a different statement than "there are
+    // none"), and `nextCursor: null` is the same end-of-history marker an
+    // exhausted backward walk returns.
+    if (readableSessionFile(nodeId) === null)
+        return { messages: [], messageIds: [], nextCursor: null };
     const { messages, identified } = await reconstructVisibleMessages(nodeId);
     const before = cursor === undefined ? messages.length : decodeMessagesCursor(cursor, messages.length);
     let start = Math.max(0, before - limit);

package/package.json

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