@north-light/crouter 0.3.312 → 0.3.314

package/dist/core/__tests__/integration/worktree-reap.test.js

@@ -22,10 +22,14 @@ import { headlessBrokerHost } from '../../runtime/host.js';
 import { reviveNode } from '../../runtime/revive.js';
 import { bindTestFleet } from '../helpers/fleet.js';
 import { resetFleetForTests } from '../../runtime/fleet.js';
-import { closeDb } from '../../canvas/db.js';
+import { closeDb, migrate, openDb } from '../../canvas/db.js';
+import { closeScopedStores } from '../../scoped-state/db.js';
+import { initializeFreshScopedStore } from '../../scoped-state/migrate.js';
+import { userScopeRoot } from '../../scope.js';
 let home;
 let remoteDir;
 let sourceDir;
+let previousHome;
 const scratchRoot = join(homedir(), '.crouter', 'canvas', 'scratch');
 function run(args, cwd) {
     const res = gitSync(args, cwd);
@@ -53,6 +57,8 @@ before(() => {
     // directory. Every path-ownership check must judge identity, not spelling.
     mkdirSync(scratchRoot, { recursive: true });
     home = mkdtempSync(join(scratchRoot, 'crtr-worktree-home-'));
+    previousHome = process.env['HOME'];
+    process.env['HOME'] = home;
     process.env['CRTR_HOME'] = home;
     // Bare "remote" wired up as origin so the test can prove landing NEVER pushes
     // to it (managed worktrees are based on and land onto local `main` only).
@@ -70,15 +76,23 @@ before(() => {
 });
 beforeEach(() => {
     closeDb();
+    closeScopedStores();
     rmSync(home, { recursive: true, force: true });
     process.env['CRTR_HOME'] = home;
+    migrate(openDb());
+    initializeFreshScopedStore({ kind: 'user', root: userScopeRoot() });
 });
 after(() => {
     closeDb();
+    closeScopedStores();
     rmSync(home, { recursive: true, force: true });
     rmSync(remoteDir, { recursive: true, force: true });
     rmSync(sourceDir, { recursive: true, force: true });
     delete process.env['CRTR_HOME'];
+    if (previousHome === undefined)
+        delete process.env['HOME'];
+    else
+        process.env['HOME'] = previousHome;
 });
 // Fix B (#327/#333): a managed worktree with zero commits ahead of its base
 // and nothing uncommitted has nothing to land — push-final's auto-drop path

package/dist/core/__tests__/kickoff.test.js

@@ -10,9 +10,11 @@ import { tmpdir } from 'node:os';
 import { join } from 'node:path';
 import { createNode, getNode, subscribe } from '../canvas/canvas.js';
 import { reportsDir } from '../canvas/paths.js';
-import { closeDb } from '../canvas/db.js';
+import { closeDb, migrate, openDb } from '../canvas/db.js';
+import { acknowledgeNodeMail, claimNodeMail } from '../canvas/deliveries.js';
+import { beginExecution } from '../canvas/executions.js';
 import { buildReviveKickoff, clearYieldMessage, drainBearings, readYieldMessage, writeYieldMessage, yieldMessagePath, } from '../runtime/kickoff.js';
-import { appendInbox, readCursor } from '../feed/inbox.js';
+import { enqueueNodeMail } from '../feed/messages.js';
 import { appendSituationalContext } from '../runtime/situational-context.js';
 import { buildContextBearings as buildContextIntro } from '../runtime/bearings.js';
 let home;
@@ -36,6 +38,7 @@ before(() => {
 beforeEach(() => {
     closeDb();
     rmSync(home, { recursive: true, force: true });
+    migrate(openDb());
 });
 after(() => {
     closeDb();
@@ -45,18 +48,23 @@ after(() => {
 test('drainBearings preserves the yield note until explicit boot confirmation', () => {
     const id = 'n1';
     const meta = createNode(node(id));
+    const executionId = beginExecution(id, { daemonEpoch: 'test' });
     writeYieldMessage(id, 'on wake, finish step 3');
-    appendInbox(id, { from: 'w1', tier: 'normal', kind: 'update', label: 'first', ref: '/tmp/a.md' });
-    const last = appendInbox(id, { from: 'w1', tier: 'normal', kind: 'update', label: 'second', ref: '/tmp/b.md' });
-    const b1 = drainBearings(meta);
+    enqueueNodeMail(id, { senderNodeId: 'w1', kind: 'update', label: 'first', bodyRef: '/tmp/a.md', urgency: 'normal', wake: 1 });
+    enqueueNodeMail(id, { senderNodeId: 'w1', kind: 'update', label: 'second', bodyRef: '/tmp/b.md', urgency: 'normal', wake: 1 });
+    const b1 = drainBearings(meta, executionId);
     assert.equal(b1.yieldMsg?.trim(), 'on wake, finish step 3');
     assert.ok(b1.unreadDigest !== null && b1.unreadDigest.length > 0);
     assert.equal(existsSync(yieldMessagePath(id)), true, 'yield note remains pending before confirmation');
     assert.equal(readYieldMessage(id), 'on wake, finish step 3', 'reader sees the same persisted note');
-    assert.equal(readCursor(id), last.entry_id, 'cursor advanced to the last unread entry');
-    const b2 = drainBearings(meta);
+    const claim = claimNodeMail(id, executionId, 'wake');
+    const receipt = acknowledgeNodeMail(id, executionId, claim.deliveries.map((delivery) => ({
+        deliveryId: delivery.deliveryId, attempts: delivery.attempts, disposition: 'acknowledged',
+    })));
+    assert.deepEqual(receipt.results.map((result) => result.outcome), ['acknowledged', 'acknowledged']);
+    const b2 = drainBearings(meta, executionId);
     assert.equal(b2.yieldMsg, 'on wake, finish step 3', 'a failed launch can read the same note again');
-    assert.equal(b2.unreadDigest, null, 'the independently committed feed cursor remains drained');
+    assert.equal(b2.unreadDigest, null, 'settled deliveries do not return to the next launch');
     clearYieldMessage(id);
     clearYieldMessage(id);
     assert.equal(readYieldMessage(id), null, 'confirmation clear is idempotent');
@@ -65,30 +73,19 @@ test('drainBearings preserves the yield note until explicit boot confirmation',
 test('fresh kickoff keeps each inbox card directly before its own message', () => {
     const id = 'ordered-cards';
     const meta = createNode(node(id));
-    appendInbox(id, {
-        from: 'sender',
-        tier: 'normal',
-        kind: 'message',
-        label: 'FIRST-BODY',
-        data: { body: 'FIRST-BODY', cards: ['<runtime kind="test-card">FIRST-CARD</runtime>'] },
+    const executionId = beginExecution(id, { daemonEpoch: 'test' });
+    enqueueNodeMail(id, {
+        senderNodeId: 'sender', kind: 'message', label: 'FIRST-BODY',
+        bodyJson: { body: 'FIRST-BODY', cards: ['<runtime kind="test-card">FIRST-CARD</runtime>'] }, urgency: 'normal', wake: 1,
     });
-    appendInbox(id, {
-        from: 'sender',
-        tier: 'normal',
-        kind: 'message',
-        label: 'SECOND-BODY',
-        data: { body: 'SECOND-BODY', cards: ['<runtime kind="test-card">SECOND-CARD</runtime>'] },
+    enqueueNodeMail(id, {
+        senderNodeId: 'sender', kind: 'message', label: 'SECOND-BODY',
+        bodyJson: { body: 'SECOND-BODY', cards: ['<runtime kind="test-card">SECOND-CARD</runtime>'] }, urgency: 'normal', wake: 1,
     });
     for (const body of ['GROUPED-BODY-1', 'GROUPED-BODY-2', 'GROUPED-BODY-3']) {
-        appendInbox(id, {
-            from: 'card-free-sender',
-            tier: 'normal',
-            kind: 'message',
-            label: body,
-            data: { body },
-        });
+        enqueueNodeMail(id, { senderNodeId: 'card-free-sender', kind: 'message', label: body, bodyJson: { body }, urgency: 'normal', wake: 1 });
     }
-    const kickoff = buildReviveKickoff(meta, drainBearings(meta));
+    const kickoff = buildReviveKickoff(meta, drainBearings(meta, executionId));
     const firstCard = kickoff.indexOf('FIRST-CARD');
     const firstBody = kickoff.indexOf('FIRST-BODY');
     const secondCard = kickoff.indexOf('SECOND-CARD');
@@ -109,7 +106,7 @@ test('drainBearings leaves persona drift pending until the replacement context e
         mode: 'orchestrator',
         persona_ack: oldPersona,
     }));
-    const bearings = drainBearings(meta);
+    const bearings = drainBearings(meta, beginExecution(meta.node_id, { daemonEpoch: 'test' }));
     assert.match(bearings.driftGuidance ?? '', /ORCHESTRATOR/);
     assert.deepEqual(getNode(meta.node_id)?.persona_ack, oldPersona, 'pre-session_start launch work cannot acknowledge the drift');
 });
@@ -117,7 +114,7 @@ test('the feed block frames awaiting workers as alive + auto-waking, so a fresh
     const parent = createNode(node('p1'));
     const child = createNode(node('c1')); // status 'active' by default
     subscribe(parent.node_id, child.node_id); // parent awaits child
-    const msg = buildReviveKickoff(parent, drainBearings(parent));
+    const msg = buildReviveKickoff(parent, drainBearings(parent, beginExecution(parent.node_id, { daemonEpoch: 'test' })));
     // The roster names the live child...
     assert.ok(msg.includes(child.node_id), 'awaiting roster lists the live child');
     // ...and asserts aliveness + the automatic wake at the source, so the node
@@ -143,7 +140,7 @@ test('a fresh revive is pointed at its subscriptions\' on-disk report history (c
     const newer = join(dir, '20260608T130000-final.md');
     writeFileSync(older, '---\n---\nearly progress', 'utf8');
     writeFileSync(newer, '---\n---\nthe result', 'utf8');
-    const msg = buildReviveKickoff(parent, drainBearings(parent));
+    const msg = buildReviveKickoff(parent, drainBearings(parent, beginExecution(parent.node_id, { daemonEpoch: 'test' })));
     // The feed block stays condensed: it reports that history exists and surfaces
     // the newest report ref for immediate dereference, while older refs stay one
     // command away instead of being preloaded into the revive prompt.
@@ -170,7 +167,7 @@ test('a fresh revive kickoff never carries situational context — it rides the
     const id = 'n3';
     const meta = createNode(node(id));
     appendSituationalContext(id, 'user is mid-checkout on the store applet');
-    const kickoff = buildReviveKickoff(meta, drainBearings(meta));
+    const kickoff = buildReviveKickoff(meta, drainBearings(meta, beginExecution(meta.node_id, { daemonEpoch: 'test' })));
     assert.doesNotMatch(kickoff, /kind="situational"/, 'visible kickoff never carries the ambient card');
     assert.doesNotMatch(kickoff, /mid-checkout on the store applet/, 'visible kickoff never carries the ambient text');
     const hidden = buildContextIntro(id);
@@ -180,20 +177,20 @@ test('a fresh revive kickoff never carries situational context — it rides the
 test('buildReviveKickoff is pure — building twice eats nothing', () => {
     const id = 'n2';
     const meta = createNode(node(id));
+    const executionId = beginExecution(id, { daemonEpoch: 'test' });
     writeYieldMessage(id, 'remember the invariant');
-    const last = appendInbox(id, { from: 'w', tier: 'normal', kind: 'update', label: 'r', ref: '/tmp/r.md' });
-    // Drain once (the consuming step), then build twice off the same bearings.
-    const bearings = drainBearings(meta);
-    const cursorAfterDrain = readCursor(id);
-    assert.equal(cursorAfterDrain, last.entry_id);
+    enqueueNodeMail(id, { senderNodeId: 'w', kind: 'update', label: 'r', bodyRef: '/tmp/r.md', urgency: 'normal', wake: 1 });
+    // Claim once, then build twice from the same immutable bearings.
+    const bearings = drainBearings(meta, executionId);
     const s1 = buildReviveKickoff(meta, bearings);
     const s2 = buildReviveKickoff(meta, bearings);
-    // Identical output, and the build mutated nothing: cursor and note unchanged.
+    // Identical output, and the build mutates neither delivery receipts nor note.
     assert.equal(s1, s2);
     assert.match(s1, /<yield-message>\nremember the invariant\n<\/yield-message>/, 'yield note is delivered inline');
     const contextInventory = s1.match(/<context-dir path="[^"]+">[\s\S]*?<\/context-dir>/)?.[0];
     assert.ok(contextInventory, 'context inventory is present');
     assert.doesNotMatch(contextInventory, /yield-message\.md/, 'ephemeral yield note is not addressable from inventory');
-    assert.equal(readCursor(id), cursorAfterDrain, 'build does not touch the cursor');
+    const claim = claimNodeMail(id, executionId, 'wake');
+    assert.equal(claim.deliveries.length, 1, 'build does not settle its claimed delivery');
     assert.equal(readYieldMessage(id), 'remember the invariant', 'build does not mutate the persisted note');
 });

package/dist/core/__tests__/lifecycle-terminal-reasons.test.js

@@ -4,7 +4,7 @@ import { mkdtempSync, rmSync } from 'node:fs';
 import { tmpdir } from 'node:os';
 import { join } from 'node:path';
 import { createNode, getNode } from '../canvas/canvas.js';
-import { closeDb } from '../canvas/db.js';
+import { closeDb, migrate, openDb } from '../canvas/db.js';
 import { transition } from '../runtime/lifecycle.js';
 let home;
 function seed(nodeId) {
@@ -30,6 +30,7 @@ before(() => {
 beforeEach(() => {
     closeDb();
     rmSync(home, { recursive: true, force: true });
+    migrate(openDb());
 });
 after(() => {
     closeDb();

package/dist/core/__tests__/lifecycle.test.js

@@ -11,7 +11,7 @@ import { mkdtempSync, rmSync } from 'node:fs';
 import { tmpdir } from 'node:os';
 import { join } from 'node:path';
 import { createNode, getNode, setStatus, setIntent } from '../canvas/canvas.js';
-import { closeDb } from '../canvas/db.js';
+import { closeDb, migrate, openDb } from '../canvas/db.js';
 import { isParked, transition } from '../runtime/lifecycle.js';
 let home;
 function node(id) {
@@ -39,6 +39,7 @@ before(() => {
 beforeEach(() => {
     closeDb();
     rmSync(home, { recursive: true, force: true });
+    migrate(openDb());
 });
 after(() => {
     closeDb();

package/dist/core/__tests__/live-lifecycle-prompt.test.js

@@ -6,10 +6,13 @@ import { mkdtempSync, rmSync } from 'node:fs';
 import { tmpdir } from 'node:os';
 import { join } from 'node:path';
 import { createNode, updateNode } from '../canvas/canvas.js';
-import { closeDb } from '../canvas/db.js';
+import { closeDb, migrate, openDb } from '../canvas/db.js';
+import { ensureScopeInitialized } from '../config.js';
+import { closeScopedStores } from '../scoped-state/db.js';
 import { renderPreferencesSection } from '../substrate/render-node.js';
 import { clearSessionCache } from '../substrate/session-cache.js';
 let home;
+let priorUserHome;
 function node(id) {
     return {
         node_id: id,
@@ -24,18 +27,27 @@ function node(id) {
 }
 before(() => {
     home = mkdtempSync(join(tmpdir(), 'crtr-live-lifecycle-prompt-'));
+    priorUserHome = process.env['HOME'];
+    process.env['HOME'] = home;
     process.env['CRTR_HOME'] = home;
 });
 beforeEach(() => {
     closeDb();
     clearSessionCache();
     rmSync(home, { recursive: true, force: true });
+    ensureScopeInitialized('user', join(home, '.crouter'));
+    migrate(openDb());
 });
 after(() => {
     closeDb();
     clearSessionCache();
+    closeScopedStores();
     rmSync(home, { recursive: true, force: true });
     delete process.env['CRTR_HOME'];
+    if (priorUserHome === undefined)
+        delete process.env['HOME'];
+    else
+        process.env['HOME'] = priorUserHome;
 });
 test('a cached resident prompt becomes terminal after a live lifecycle change', () => {
     createNode(node('live'));

package/dist/core/__tests__/memory-resolver-precedence.test.js

@@ -23,6 +23,7 @@ import { tmpdir } from 'node:os';
 import { join } from 'node:path';
 import { gitSync } from '../git.js';
 import { createProfile } from '../profiles/manifest.js';
+import { initializeFreshScopedStore } from '../scoped-state/migrate.js';
 import { resetScopeCache } from '../scope.js';
 import { loadMemoryStoreView, openProjectMemoryStore, resolveMemoryCandidates, resolveMemoryDoc, resolveMemoryDocInStore, resolveMemoryDocs, } from '../memory-resolver.js';
 let home;
@@ -54,6 +55,7 @@ before(() => {
     prevHome = process.env.HOME;
     prevProfile = process.env.CRTR_PROFILE_ID;
     process.env.HOME = home;
+    initializeFreshScopedStore({ kind: 'user', root: join(home, '.crouter') });
     // A wider user-scope doc whose name a project doc used to shadow.
     mkdirSync(join(home, '.crouter', 'memory'), { recursive: true });
     writeFileSync(join(home, '.crouter', 'memory', 'testing-stance.md'), '---\nkind: preference\n---\nUSER PREFERENCE BODY\n', 'utf8');