@north-light/crouter 0.3.173 → 0.3.175

package/dist/core/keybindings/__tests__/inbox-affordance.test.js

@@ -1,38 +0,0 @@
-// Run with: node --import tsx/esm --test src/core/keybindings/__tests__/inbox-affordance.test.ts
-//
-// The shared inbox-affordance formatter is the single source of truth for how
-// every pending-interaction surface (attach footer, `human ask`/`review`
-// follow-up + help effects, `surface node focus` error) names the inbox shortcut. These
-// guard that it resolves the live binding — default, custom, multi-gesture, and
-// disabled — instead of hardcoding a symbol that can drift from the real key.
-import { test } from 'node:test';
-import assert from 'node:assert/strict';
-import { resolveKeybindings } from '../index.js';
-import { inboxOpenHint, inboxHint, inboxOpenInstruction, inboxShortcut, } from '../inbox.js';
-const custom = () => resolveKeybindings({ 'crtr.tmux.inbox.toggle': ['alt+j'] });
-const multi = () => resolveKeybindings({ 'crtr.tmux.inbox.toggle': ['alt+i', 'alt+j'] });
-const disabled = () => resolveKeybindings({ 'crtr.tmux.inbox.toggle': [] });
-test('inboxShortcut resolves default, custom, multi-gesture, and disabled', () => {
-    assert.equal(inboxShortcut(resolveKeybindings()), 'Alt+I');
-    assert.equal(inboxShortcut(custom()), 'Alt+J');
-    assert.equal(inboxShortcut(multi()), 'Alt+I / Alt+J');
-    assert.equal(inboxShortcut(disabled()), null);
-});
-test('inboxOpenHint mirrors the attach footer contract', () => {
-    assert.equal(inboxOpenHint(resolveKeybindings()), 'Alt+I inbox');
-    assert.equal(inboxOpenHint(custom()), 'Alt+J inbox');
-    assert.equal(inboxOpenHint(multi()), 'Alt+I / Alt+J inbox');
-    assert.equal(inboxOpenHint(disabled()), 'crtr human list');
-});
-test('inboxHint is a bare parenthetical token, CLI form when disabled', () => {
-    assert.equal(inboxHint(resolveKeybindings()), 'Alt+I');
-    assert.equal(inboxHint(custom()), 'Alt+J');
-    assert.equal(inboxHint(multi()), 'Alt+I / Alt+J');
-    assert.equal(inboxHint(disabled()), '`crtr human list`');
-});
-test('inboxOpenInstruction is an imperative clause, CLI-only when disabled', () => {
-    assert.equal(inboxOpenInstruction(resolveKeybindings()), 'press Alt+I in a node viewer (or run `crtr human list` for the queue)');
-    assert.equal(inboxOpenInstruction(custom()), 'press Alt+J in a node viewer (or run `crtr human list` for the queue)');
-    assert.equal(inboxOpenInstruction(multi()), 'press Alt+I / Alt+J in a node viewer (or run `crtr human list` for the queue)');
-    assert.equal(inboxOpenInstruction(disabled()), 'run `crtr human list` for the queue');
-});

package/dist/core/review/__tests__/project.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/core/review/__tests__/project.test.js

@@ -1,71 +0,0 @@
-import assert from 'node:assert/strict';
-import { mkdtempSync, readFileSync, rmSync, writeFileSync } from 'node:fs';
-import { test } from 'node:test';
-import { join } from 'node:path';
-import { projectReviewResult, buildReviewResult } from '../project.js';
-import { ticketDir } from '../../human/root.js';
-import { readTicketResult, submitReview } from '../../human/tickets.js';
-test('approved review results with unresolved comments round-trip and project idempotently', () => {
-    const home = mkdtempSync(join('/tmp', 'crtr-review-project-'));
-    const previousHome = process.env['CRTR_HOME'];
-    process.env['CRTR_HOME'] = home;
-    try {
-        const source = join(home, 'source.md');
-        writeFileSync(source, '# Source\nsecond\nthird\n');
-        const bridge = submitReview({
-            dir: ticketDir('review-bridge'),
-            review: { file: source, title: 'Review source', subtitle: 'Review source.', source: {} },
-        });
-        const review = {
-            review_id: 'review-result-test',
-            origin_kind: 'ticket',
-            origin_node_id: 'origin-node',
-            bridge_node_id: 'review-bridge',
-            companion_node_id: 'companion-node',
-            baseline_sha256: 'baseline',
-            output_path: null,
-            file: source,
-        };
-        const at = '2025-01-04T00:00:00.000Z';
-        const result = buildReviewResult(review, {
-            comments: [
-                { id: 'c1', line: 1, endLine: 1, lineText: '# Source', comment: 'first', createdAt: at },
-                { id: 'c2', line: 2, endLine: 2, lineText: 'second', comment: 'second', createdAt: at },
-                { id: 'c3', line: 3, endLine: 3, lineText: 'third', comment: 'third', createdAt: at },
-            ],
-            total: 3,
-            unresolved: 2,
-        }, 'approved', at);
-        projectReviewResult(review, result);
-        const responsePath = join(bridge.dir, 'response.json');
-        const firstResponse = readFileSync(responsePath);
-        const decoded = readTicketResult(bridge.dir);
-        assert.equal(decoded?.kind, 'review');
-        assert.equal(decoded?.kind === 'review' ? decoded.result.approved : false, true);
-        assert.equal(decoded?.kind === 'review' ? decoded.result.comments.length : 0, 3);
-        assert.deepEqual(decoded?.kind === 'review' ? decoded.result : undefined, {
-            file: source,
-            submitted: true,
-            approved: true,
-            comments: result.comments,
-            submittedAt: at,
-            savedAt: at,
-            reviewId: 'review-result-test',
-            changed: true,
-            baselineSha256: 'baseline',
-            approvedSha256: 'approved',
-            companionNodeId: 'companion-node',
-            commentsTotal: 3,
-            commentsUnresolved: 2,
-        });
-        projectReviewResult(review, result);
-        assert.deepEqual(readFileSync(responsePath), firstResponse);
-    }
-    finally {
-        if (previousHome === undefined)
-            delete process.env['CRTR_HOME'];
-        else
-            process.env['CRTR_HOME'] = previousHome;
-        rmSync(home, { recursive: true, force: true });
-    }
-});

package/dist/core/runtime/__tests__/command-expansion.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/core/runtime/__tests__/command-expansion.test.js

@@ -1,25 +0,0 @@
-import assert from 'node:assert/strict';
-import test from 'node:test';
-import { expandDeterministicCommand, isDeterministicCommandExpansion, stripMarkdownComments } from '../command-expansion.js';
-test('slash-command expansion omits Markdown comments while preserving literal code examples', () => {
-    const body = [
-        '<!-- author-only $ARGUMENTS -->',
-        'Use $ARGUMENTS.',
-        '```html',
-        '<!-- literal example -->',
-        '```',
-        '`<!-- inline literal -->`',
-    ].join('\n');
-    assert.equal(expandDeterministicCommand({ kind: 'memory-slash', commandName: 'example', body }, 'topic'), 'Use topic.\n```html\n<!-- literal example -->\n```\n`<!-- inline literal -->`');
-    assert.equal(stripMarkdownComments('before <!-- hidden --> after'), 'before  after');
-    assert.equal(stripMarkdownComments('before\n<!-- hidden -->\nafter'), 'before\nafter');
-});
-test('deterministic expansion metadata accepts only the exact inert wire shape', () => {
-    assert.equal(isDeterministicCommandExpansion({ kind: 'memory-slash', commandName: 'dev', body: 'do it' }), true);
-    assert.equal(isDeterministicCommandExpansion({ kind: 'memory-slash', commandName: 'dev', body: 'do it', extra: true }), false);
-    assert.equal(isDeterministicCommandExpansion({ kind: 'memory-slash', commandName: 'dev', body: 1 }), false);
-    assert.equal(isDeterministicCommandExpansion({ kind: 'other', commandName: 'dev', body: 'do it' }), false);
-    assert.equal(isDeterministicCommandExpansion(Object.create({ kind: 'memory-slash', commandName: 'dev', body: 'do it' })), false);
-    assert.equal(isDeterministicCommandExpansion({ get kind() { return 'memory-slash'; }, commandName: 'dev', body: 'do it' }), false);
-    assert.equal(isDeterministicCommandExpansion(new Proxy({}, { getPrototypeOf() { throw new Error('hostile extension'); } })), false);
-});

package/dist/core/runtime/__tests__/session-visibility.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/core/runtime/__tests__/session-visibility.test.js

@@ -1,119 +0,0 @@
-// Run with: node --import tsx/esm --test src/core/runtime/__tests__/session-visibility.test.ts
-//
-// This table keeps the review companion transcript boundary a pure, cheap
-// regression surface: ordinary snapshots are untouched, a persisted marker
-// wins from its first occurrence, and markerless companions expose nothing.
-import { test } from 'node:test';
-import assert from 'node:assert/strict';
-import { boundaryRootedTree, REVIEW_BOUNDARY_CUSTOM_TYPE, visibleMessages } from '../session-visibility.js';
-function message(fields) {
-    return fields;
-}
-function contentOf(item) {
-    return item.content;
-}
-const boundary = message({
-    role: 'custom',
-    customType: REVIEW_BOUNDARY_CUSTOM_TYPE,
-    content: 'review boundary',
-    display: true,
-    timestamp: 2,
-});
-const cycleDivider = message({
-    role: 'custom',
-    customType: 'crtr-cycle-divider',
-    content: 'cycle 2 begins',
-    display: true,
-    timestamp: 3,
-});
-const cases = [
-    {
-        name: 'first marker wins even when it is not first, and preserves later cycle dividers',
-        messages: [
-            message({ role: 'user', content: 'inherited origin context', timestamp: 0 }),
-            message({ role: 'assistant', content: 'inherited answer', timestamp: 1 }),
-            boundary,
-            message({ role: 'user', content: 'review message', timestamp: 2 }),
-            cycleDivider,
-            message({ role: 'assistant', content: 'later cycle message', timestamp: 4 }),
-        ],
-        reviewCompanion: true,
-        assertResult: (_input, result) => {
-            assert.deepEqual(result, [boundary, message({ role: 'user', content: 'review message', timestamp: 2 }), cycleDivider, message({ role: 'assistant', content: 'later cycle message', timestamp: 4 })]);
-            assert.equal(result[0], boundary);
-            assert.ok(!result.some((item) => contentOf(item) === 'inherited origin context'));
-            assert.ok(!result.some((item) => contentOf(item) === 'inherited answer'));
-        },
-    },
-    {
-        name: 'ordinary nodes return the exact input reference without a marker',
-        messages: [
-            message({ role: 'user', content: 'ordinary message', timestamp: 0 }),
-            cycleDivider,
-        ],
-        reviewCompanion: false,
-        assertResult: (input, result) => {
-            assert.strictEqual(result, input);
-        },
-    },
-    {
-        name: 'a markerless companion replaces only a compaction summary with a synthetic boundary',
-        messages: [
-            message({
-                role: 'compactionSummary',
-                content: 'summary may include inherited origin context',
-                timestamp: 7,
-            }),
-            message({ role: 'user', content: 'post-compaction review message', timestamp: 8 }),
-        ],
-        reviewCompanion: true,
-        assertResult: (_input, result) => {
-            assert.deepEqual(result, [
-                message({
-                    role: 'custom',
-                    customType: REVIEW_BOUNDARY_CUSTOM_TYPE,
-                    content: 'Earlier conversation with the origin is not shown.',
-                    display: true,
-                    timestamp: 7,
-                }),
-                message({ role: 'user', content: 'post-compaction review message', timestamp: 8 }),
-            ]);
-            assert.ok(!result.some((item) => contentOf(item) === 'summary may include inherited origin context'));
-        },
-    },
-];
-for (const testCase of cases) {
-    test(`visibleMessages: ${testCase.name}`, () => {
-        const input = testCase.messages;
-        const result = visibleMessages(input, { reviewCompanion: testCase.reviewCompanion });
-        testCase.assertResult(input, result);
-    });
-}
-function treeNode(id, entry, children = []) {
-    return { entry: { id, ...entry }, children };
-}
-test('visibleMessages: every markerless non-compaction shape remains synthetic-only', () => {
-    const result = visibleMessages([message({ role: 'user', content: 'inherited context', timestamp: 5 })], { reviewCompanion: true });
-    assert.equal(result.length, 1);
-    assert.equal(result[0]?.customType, REVIEW_BOUNDARY_CUSTOM_TYPE);
-});
-test('boundaryRootedTree drops inherited ancestors and preserves only boundary descendants', () => {
-    const tree = [
-        treeNode('origin', { type: 'message', message: { role: 'user', content: 'PREBOUNDARY_CANARY' } }, [
-            treeNode('inherited-reply', { type: 'message', message: { role: 'assistant', content: 'inherited reply' } }, [
-                treeNode('boundary', { type: 'custom_message', customType: REVIEW_BOUNDARY_CUSTOM_TYPE, content: 'review boundary', display: true }, [
-                    treeNode('review-user', { type: 'message', message: { role: 'user', content: 'review finding' } }),
-                    treeNode('review-assistant', { type: 'message', message: { role: 'assistant', content: 'review resolution' } }),
-                ]),
-            ]),
-        ]),
-    ];
-    const result = boundaryRootedTree(tree);
-    assert.deepEqual(result.map((node) => node.entry.id), ['boundary']);
-    assert.deepEqual(result[0]?.children.map((node) => node.entry.id), ['review-user', 'review-assistant']);
-    assert.equal(JSON.stringify(result).includes('PREBOUNDARY_CANARY'), false);
-});
-test('boundaryRootedTree returns no picker tree for a markerless companion', () => {
-    const tree = [treeNode('origin', { type: 'message', message: { role: 'user', content: 'PREBOUNDARY_CANARY' } })];
-    assert.deepEqual(boundaryRootedTree(tree), []);
-});

package/dist/core/runtime/__tests__/spawn-at-given-id.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/core/runtime/__tests__/spawn-at-given-id.test.js

@@ -1,34 +0,0 @@
-// spawn-at-given-id.test.ts — regression for the reviewed opaque-id-contract
-// violation (finding 3): the old validator required >=2 hyphen segments and
-// rejected a hyphen-less id outright, contradicting the design's "opaque,
-// never parsed for structure" ID scheme. Duplicate-rejection is covered by
-// spawn-worktree-id-conflict.test.ts (the actual reviewed defect); it is not
-// re-asserted here to avoid restating already-covered feature behavior.
-import { test, before, after, beforeEach } from 'node:test';
-import assert from 'node:assert/strict';
-import { mkdtempSync, rmSync } from 'node:fs';
-import { closeDb } from '../../canvas/db.js';
-import { getNode } from '../../canvas/canvas.js';
-import { spawnNode } from '../nodes.js';
-let home;
-before(() => {
-    home = mkdtempSync('/tmp/crtr-spawn-at-id-');
-    process.env['CRTR_HOME'] = home;
-});
-beforeEach(() => {
-    closeDb();
-    rmSync(home, { recursive: true, force: true });
-});
-after(() => {
-    closeDb();
-    rmSync(home, { recursive: true, force: true });
-});
-test('spawnNode({ nodeId }): a hyphen-less opaque id (no minimum segment count) is valid and spawns at the exact given id', () => {
-    const meta = spawnNode({ kind: 'general', cwd: '/tmp/work', nodeId: 'remote42' });
-    assert.equal(meta.node_id, 'remote42');
-    assert.notEqual(getNode('remote42'), null);
-});
-test('spawnNode({ nodeId }): a charset-unsafe or oversized id is still rejected with a usage error before the canvas is touched', () => {
-    assert.throws(() => spawnNode({ kind: 'general', cwd: '/tmp/work', nodeId: 'BadID!' }), /invalid --node-id/);
-    assert.throws(() => spawnNode({ kind: 'general', cwd: '/tmp/work', nodeId: 'a'.repeat(129) }), /invalid --node-id/);
-});