@north-light/crouter 0.3.229 → 0.3.230

package/dist/core/__tests__/canvas-inbox-watcher-hold.test.js

@@ -250,7 +250,7 @@ describe('canvas inbox watcher — hold + idle delivery', () => {
         pi.fire('agent_start', { type: 'agent_start' }, { isIdle: () => false });
         await wait(TICK_MS + 100);
         appendInbox(meta.node_id, { from: 'child-x', tier: 'normal', kind: 'update', label: 'handoff at yield' });
-        await wait(SETTLE_MS);
+        await waitForFirstInjection(pi);
         assert.equal(pi.injected.length, 1, 'the active conversation received the handoff before yield');
         // Yield discards that conversation before it can act on the queued handoff.
         // Its settle must leave the entry durable for the fresh revive's kickoff.
@@ -267,7 +267,7 @@ describe('canvas inbox watcher — hold + idle delivery', () => {
         // No agent_start fired → watcher treats the node as idle.
         await wait(TICK_MS + 100);
         appendInbox('node-idle', { from: 'child-3', tier: 'normal', kind: 'final', label: 'done while idle' });
-        await wait(SETTLE_MS);
+        await waitForFirstInjection(pi);
         assert.equal(pi.injected.length, 1);
         assert.equal(pi.injected[0].deliverAs, undefined, 'idle → sendUserMessage triggers a turn, no deliverAs');
     });
@@ -295,7 +295,7 @@ describe('canvas inbox watcher — hold + idle delivery', () => {
             assert.equal(pi.injected.length, 0, 'a paused profile delivers nothing to its live broker');
             assert.equal(readCursor('node-paused-profile'), undefined, 'the held entry stays unconsumed');
             resumeProfile(profile.profileId);
-            await wait(SETTLE_MS);
+            await waitForFirstInjection(pi);
             assert.equal(pi.injected.length, 1, 'resume delivers the entry that waited');
             assert.match(pi.injected[0].content, /sent while paused/);
         }
@@ -348,7 +348,7 @@ describe('canvas inbox watcher — action-gated startup (session_start arm)', ()
         // now must pick up the already-buffered pre-bind entry.
         actionsReady = true;
         pi.fire('session_start', { type: 'session_start' }, undefined);
-        await wait(SETTLE_MS);
+        await waitForFirstInjection(pi);
         assert.equal(sendAttempts, 1, 'exactly one action attempt once session_start arms the watcher');
         assert.equal(pi.injected.length, 1, 'exactly one handoff once actions are bound');
         assert.equal(readCursor('node-action-gate'), undefined, 'still uncommitted until agent_settled');
@@ -404,7 +404,7 @@ describe('canvas inbox watcher — durable cursor (at-least-once)', () => {
         pi1.fire('agent_start', { type: 'agent_start' }, { isIdle: () => false });
         await wait(TICK_MS + 100);
         appendInbox('node-cursor-crash', { from: 'child-1', tier: 'normal', kind: 'update', label: 'still working' });
-        await wait(SETTLE_MS);
+        await waitForFirstInjection(pi1);
         assert.equal(pi1.injected.length, 1, 'delivered mid-turn as a followUp');
         assert.equal(pi1.injected[0].deliverAs, 'followUp');
         // The durability boundary: no settle has fired, so the on-disk cursor must
@@ -420,7 +420,7 @@ describe('canvas inbox watcher — durable cursor (at-least-once)', () => {
         // modeling crash-then-revive rather than a clean stop.
         const pi2 = makeFakePi();
         disposers.push(registerWatcher(pi2));
-        await wait(SETTLE_MS);
+        await waitForFirstInjection(pi2);
         assert.equal(pi2.injected.length, 1, 'a fresh watcher re-reads and re-delivers the same uncommitted entry');
         assert.match(pi2.injected[0].content, /still working/);
     });
@@ -431,7 +431,7 @@ describe('canvas inbox watcher — durable cursor (at-least-once)', () => {
         pi1.fire('agent_start', { type: 'agent_start' }, { isIdle: () => false });
         await wait(TICK_MS + 100);
         const entry = appendInbox('node-cursor-committed', { from: 'child-2', tier: 'normal', kind: 'update', label: 'still working' });
-        await wait(SETTLE_MS);
+        await waitForFirstInjection(pi1);
         assert.equal(pi1.injected.length, 1);
         assert.equal(readCursor('node-cursor-committed'), undefined, 'not yet committed before the settle');
         // The turn settles with pi's queue drained (the followUp was consumed) —
@@ -450,7 +450,7 @@ describe('canvas inbox watcher — durable cursor (at-least-once)', () => {
         // No agent_start fired → watcher treats the node as idle.
         await wait(TICK_MS + 100);
         const entry = appendInbox('node-cursor-idle', { from: 'child-3', tier: 'normal', kind: 'update', label: 'idle update' });
-        await wait(SETTLE_MS);
+        await waitForFirstInjection(pi);
         assert.equal(pi.injected.length, 1);
         assert.equal(pi.injected[0].deliverAs, undefined, 'idle delivery has no deliverAs');
         // The idle send triggers a turn, but its void return is a handoff, not proof
@@ -477,7 +477,7 @@ describe('canvas inbox watcher — durable cursor (at-least-once)', () => {
         assert.equal(readCursor('node-cursor-failsend'), undefined, 'durable cursor not advanced past a re-buffered, never-delivered entry');
         const pi2 = makeFakePi();
         disposers.push(registerWatcher(pi2));
-        await wait(SETTLE_MS);
+        await waitForFirstInjection(pi2);
         assert.equal(pi2.injected.length, 1, 'a fresh watcher re-reads and re-delivers the never-committed entry');
         assert.match(pi2.injected[0].content, /never delivered/);
     });
@@ -501,7 +501,7 @@ describe('canvas inbox watcher — durable cursor (at-least-once)', () => {
         assert.equal(readCursor('node-cursor-critical'), undefined, 'abort-generated settle does not commit the pending critical entry before its retry');
         const pi2 = makeFakePi();
         disposers.push(registerWatcher(pi2));
-        await wait(SETTLE_MS);
+        await waitForFirstInjection(pi2);
         assert.equal(pi2.injected.length, 1, 'a fresh watcher re-delivers the critical entry');
         assert.match(pi2.injected[0].content, /critical work/);
     });
@@ -517,7 +517,7 @@ describe('canvas inbox watcher — durable cursor (at-least-once)', () => {
         pi1.fire('agent_start', { type: 'agent_start' }, { isIdle: () => false });
         await wait(TICK_MS + 60);
         const a = appendInbox('node-cursor-batchB', { from: 'child-a', tier: 'normal', kind: 'update', label: 'batch A' });
-        await wait(280); // > 200ms debounce → A flushes and is delivered mid-turn (handoff)
+        await waitForFirstInjection(pi1);
         assert.equal(pi1.injected.length, 1, 'batch A delivered mid-turn');
         assert.equal(pi1.injected[0].deliverAs, 'followUp');
         const b = appendInbox('node-cursor-batchB', { from: 'child-b', tier: 'normal', kind: 'update', label: 'batch B' });
@@ -529,7 +529,7 @@ describe('canvas inbox watcher — durable cursor (at-least-once)', () => {
         assert.notEqual(readCursor('node-cursor-batchB'), b.entry_id, 'commit does not cross the still-buffered batch B');
         const pi2 = makeFakePi();
         disposers.push(registerWatcher(pi2));
-        await wait(SETTLE_MS + 40);
+        await waitForFirstInjection(pi2);
         assert.equal(pi2.injected.length, 1, 'a fresh watcher re-reads from A and re-delivers B');
         assert.match(pi2.injected[0].content, /batch B/);
         assert.doesNotMatch(pi2.injected[0].content, /batch A/, 'A is already committed, only B is re-delivered');
@@ -548,7 +548,7 @@ describe('canvas inbox watcher — durable cursor (at-least-once)', () => {
         pi1.fire('agent_start', { type: 'agent_start' }, { isIdle: () => false });
         await wait(TICK_MS + 100);
         const entry = appendInbox('node-cursor-failhold', { from: 'child-1', tier: 'normal', kind: 'update', label: 'queued across a failure' });
-        await wait(SETTLE_MS);
+        await waitForFirstInjection(pi1);
         assert.equal(pi1.injected.length, 1, 'delivered mid-turn as a followUp');
         assert.equal(pi1.injected[0].deliverAs, 'followUp');
         // A failure/aborted agent_end is never a commit point regardless of queue
@@ -583,7 +583,7 @@ describe('canvas inbox watcher — durable cursor (at-least-once)', () => {
         pi.fire('agent_start', { type: 'agent_start' }, { isIdle: () => false });
         await wait(TICK_MS + 100);
         appendInbox(meta.node_id, { from: 'child-x', tier: 'normal', kind: 'update', label: 'handoff at shutdown' });
-        await wait(SETTLE_MS);
+        await waitForFirstInjection(pi);
         assert.equal(pi.injected.length, 1, 'the active conversation received the handoff before refresh');
         setIntent(meta.node_id, 'refresh');
         await shutdown(pi, { pending: false });
@@ -596,7 +596,7 @@ describe('canvas inbox watcher — durable cursor (at-least-once)', () => {
         pi1.fire('agent_start', { type: 'agent_start' }, { isIdle: () => false });
         await wait(TICK_MS + 100);
         const entry = appendInbox('node-cursor-shutdownhold', { from: 'child-1', tier: 'normal', kind: 'update', label: 'still queued at shutdown' });
-        await wait(SETTLE_MS);
+        await waitForFirstInjection(pi1);
         assert.equal(pi1.injected.length, 1, 'delivered mid-turn as a followUp');
         // session_shutdown fires with no prior agent_settled, and the context
         // reports an unconsumed queued send — the hold declines the commit.
@@ -606,7 +606,7 @@ describe('canvas inbox watcher — durable cursor (at-least-once)', () => {
         // A fresh watcher re-reads and re-delivers, then a drained settle commits.
         const pi2 = makeFakePi();
         disposers.push(registerWatcher(pi2));
-        await wait(SETTLE_MS);
+        await waitForFirstInjection(pi2);
         assert.equal(pi2.injected.length, 1, 'a fresh watcher re-reads and re-delivers the never-committed entry');
         assert.match(pi2.injected[0].content, /still queued at shutdown/);
         await settle(pi2);

package/dist/core/__tests__/integration/spawn-root.test.js

@@ -8,7 +8,6 @@ import { tmpdir } from 'node:os';
 import { join } from 'node:path';
 import { createServer } from 'node:net';
 import { spawn as spawnProcess } from 'node:child_process';
-import { findDaemonPids, stopDaemonProcess } from '../../../daemon/manage.js';
 import { createNode, getNode, getRow, subscribersOf } from '../../canvas/canvas.js';
 import { closeDb } from '../../canvas/db.js';
 import { nodeDir } from '../../canvas/paths.js';
@@ -21,6 +20,7 @@ import { piSessionsRoot } from '../../runtime/pi-vendored.js';
 import { emptyContextExposureState, exposureTarget, loadContextExposureState, registerExposure, saveContextExposureState, } from '../../substrate/injected-store.js';
 import { ROOT_PROFILE_ID } from '../../profiles/manifest.js';
 let home;
+let previousPidfile;
 function spawner(id) {
     return {
         node_id: id,
@@ -38,15 +38,25 @@ before(() => {
     // canvas home short because root readiness now opens a real view.sock.
     home = mkdtempSync('/tmp/crtr-spawn-root-');
     process.env['CRTR_HOME'] = home;
+    previousPidfile = process.env['CRTR_PIDFILE'];
+    process.env['CRTR_PIDFILE'] = join(home, 'crtrd.pid');
 });
 beforeEach(() => {
     closeDb();
     rmSync(home, { recursive: true, force: true });
+    mkdirSync(home, { recursive: true });
+    // These tests exercise direct spawn behavior, not daemon autostart. Keep
+    // ensureDaemon() from launching a background daemon whose startup verifier
+    // would outlive the test's next CRTR_HOME reset.
+    writeFileSync(join(home, 'crtrd.pid'), `${process.pid}\n`);
 });
-after(async () => {
+after(() => {
     closeDb();
-    await Promise.all(findDaemonPids().map((pid) => stopDaemonProcess(pid)));
     rmSync(home, { recursive: true, force: true });
+    if (previousPidfile === undefined)
+        delete process.env['CRTR_PIDFILE'];
+    else
+        process.env['CRTR_PIDFILE'] = previousPidfile;
 });
 test('managed child: parent on the spine + active subscription + provenance', () => {
     createNode(spawner('A'));

package/dist/core/human/__tests__/integration/inbox-core.test.js

@@ -17,10 +17,11 @@ writeFileSync(pageSource, 'export default () => (\n  <Page title="A page" subtit
 const page = (id, emittedAt) => submitPage({ dir: ticketDir(id), sourceFile: pageSource, source: { emittedAt, nodeId: 'node-1' }, delivery: { placement: 'inline', inbox: true, reply: true } });
 const answer = { go: { text: 'yes', edited: true } };
 async function waitFor(paths) {
-    for (let attempts = 0; !paths.every(existsSync); attempts++) {
-        if (attempts > 1_000)
+    const deadline = Date.now() + 10_000;
+    while (!paths.every(existsSync)) {
+        if (Date.now() >= deadline)
             throw new Error(`workers did not reach barrier: ${paths.join(', ')}`);
-        await new Promise((resolvePromise) => setTimeout(resolvePromise, 1));
+        await new Promise((resolvePromise) => setTimeout(resolvePromise, 10));
     }
 }
 page('z-last', '2025-01-01T00:00:00.000Z');

package/package.json

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