@north-light/crouter 0.3.318 → 0.3.320

package/dist/daemon/api/__tests__/broker-settle-park.test.js

@@ -1,175 +0,0 @@
-// Run with: node --conditions=crtr-src --import tsx/esm --test src/daemon/api/__tests__/broker-settle-park.test.ts
-//
-// The park clock arms one pending marker, then an isolated parking turn reports
-// completion through its own operation. Main-session settlement must not consume
-// that marker: a user turn can settle while the shadow turn still runs. The
-// completion re-evaluates whether parking still applies exactly once.
-import { after, before, beforeEach, test } from 'node:test';
-import assert from 'node:assert/strict';
-import { mkdtempSync, rmSync, writeFileSync, utimesSync } from 'node:fs';
-import { join } from 'node:path';
-import { createNode, getNode } from '../../../core/canvas/canvas.js';
-import { closeDb, migrate, openDb } from '../../../core/canvas/db.js';
-import { resetSession } from '../../../core/canvas/sessions.js';
-import { beginExecution, liveExecutionForNode } from '../../../core/canvas/executions.js';
-import { enqueueNodeMail } from '../../../core/feed/messages.js';
-import { openTurn } from '../../../core/canvas/recovery.js';
-import { beginPendingPark, isPendingPark } from '../../park-pending.js';
-import { brokerOperationRoutes } from '../handlers/broker-ops.js';
-let home = '';
-let originalHome;
-const settleRoute = brokerOperationRoutes.find((route) => route.method === 'POST' && route.pattern === '/v1/nodes/:id/broker/settle');
-const parkCompleteRoute = brokerOperationRoutes.find((route) => route.method === 'POST' && route.pattern === '/v1/nodes/:id/broker/park-complete');
-const parkActivityRoute = brokerOperationRoutes.find((route) => route.method === 'POST' && route.pattern === '/v1/nodes/:id/broker/park-activity');
-before(() => {
-    originalHome = process.env['CRTR_HOME'];
-    home = mkdtempSync('/tmp/crtr-settle-park-');
-    process.env['CRTR_HOME'] = home;
-});
-beforeEach(() => {
-    closeDb();
-    rmSync(home, { recursive: true, force: true });
-    migrate(openDb());
-});
-after(() => {
-    closeDb();
-    rmSync(home, { recursive: true, force: true });
-    if (originalHome === undefined)
-        delete process.env['CRTR_HOME'];
-    else
-        process.env['CRTR_HOME'] = originalHome;
-});
-/** A live resident with an isolated parking turn in flight. */
-function residentMidParkingTurn(id) {
-    const meta = {
-        node_id: id,
-        name: id,
-        created: new Date().toISOString(),
-        cwd: home,
-        kind: 'general',
-        mode: 'base',
-        lifecycle: 'resident',
-        status: 'active',
-    };
-    createNode(meta);
-    beginExecution(id, { daemonEpoch: 'test' });
-    beginPendingPark(id, Date.now());
-}
-async function settle(id, over = {}) {
-    const ctx = {
-        method: 'POST',
-        path: `/v1/nodes/${id}/broker/settle`,
-        params: { id },
-        query: new URLSearchParams(),
-        body: {
-            expected_execution_id: liveExecutionForNode(id).executionId,
-            stopReason: 'stop',
-            backgroundJobsRunning: false,
-            pushedFinal: false,
-            requestedHumanReply: false,
-            ...over,
-        },
-    };
-    return (await settleRoute.handler(ctx)).body;
-}
-async function completePark(id) {
-    const ctx = {
-        method: 'POST',
-        path: `/v1/nodes/${id}/broker/park-complete`,
-        params: { id },
-        query: new URLSearchParams(),
-        body: { expected_execution_id: liveExecutionForNode(id).executionId, outcome: 'completed' },
-    };
-    return (await parkCompleteRoute.handler(ctx)).body;
-}
-async function recordParkActivity(id) {
-    const ctx = {
-        method: 'POST',
-        path: `/v1/nodes/${id}/broker/park-activity`,
-        params: { id },
-        query: new URLSearchParams(),
-        body: { expected_execution_id: liveExecutionForNode(id).executionId },
-    };
-    return (await parkActivityRoute.handler(ctx)).body;
-}
-test('a completed park turn with nothing new parks the node: done + intent=parked, no latch', async () => {
-    residentMidParkingTurn('summarized');
-    const directive = await completePark('summarized');
-    assert.deepEqual(directive, { action: 'shutdown' });
-    const parked = getNode('summarized');
-    assert.equal(parked.status, 'done');
-    assert.equal(parked.intent, 'parked');
-    assert.equal(parked.final_report ?? null, null, 'a park never sets the finalization latch');
-    assert.equal(isPendingPark('summarized'), false, 'the pending park is consumed');
-});
-test('a human ask raised during the parking turn does not consume the pending marker', async () => {
-    residentMidParkingTurn('asked');
-    const directive = await settle('asked', { requestedHumanReply: true });
-    assert.deepEqual(directive, { action: 'stay_dormant' });
-    assert.equal(getNode('asked').status, 'active', 'the node stays live awaiting its answer');
-    assert.equal(getNode('asked').intent ?? null, null);
-    assert.equal(isPendingPark('asked'), true, 'main-session settlement leaves completion to the shadow turn');
-});
-test('a main turn racing shadow completion keeps the broker live', async () => {
-    residentMidParkingTurn('busy-main');
-    const executionId = beginExecution('busy-main', { daemonEpoch: 'test' });
-    openTurn(executionId, { reason: 'agent-start' });
-    const directive = await completePark('busy-main');
-    assert.deepEqual(directive, { action: 'stay_dormant' });
-    assert.equal(getNode('busy-main').status, 'active');
-    assert.equal(getNode('busy-main').intent ?? null, null);
-    assert.equal(isPendingPark('busy-main'), false, 'shadow completion consumes the marker after releasing the main turn');
-});
-test('wake-worthy mail arriving during the parking turn flips the verdict back to release', async () => {
-    residentMidParkingTurn('mailed');
-    enqueueNodeMail('mailed', { kind: 'message', label: 'x', bodyJson: { body: 'work' }, urgency: 'normal', wake: 1 });
-    const directive = await completePark('mailed');
-    assert.equal(directive.action, 'stay_dormant');
-    assert.equal(getNode('mailed').status, 'active', 're-evaluation saw the new obligation and did not park');
-    assert.equal(getNode('mailed').intent ?? null, null);
-    assert.equal(isPendingPark('mailed'), false);
-});
-test('a message already consumed by the main turn still releases the parking completion', async () => {
-    residentMidParkingTurn('consumed-mail');
-    enqueueNodeMail('consumed-mail', { kind: 'message', label: 'work', bodyJson: { body: 'work' }, urgency: 'normal', wake: 1 });
-    const directive = await completePark('consumed-mail');
-    assert.deepEqual(directive, { action: 'stay_dormant' });
-    assert.equal(getNode('consumed-mail').status, 'active');
-    assert.equal(getNode('consumed-mail').intent ?? null, null);
-    assert.equal(isPendingPark('consumed-mail'), false);
-});
-test('a bound but absent main-session file is not activity', async () => {
-    residentMidParkingTurn('empty-session');
-    resetSession('empty-session', { sessionId: 'empty-session', transcriptPath: join(home, 'not-created-yet.jsonl') });
-    const directive = await completePark('empty-session');
-    assert.deepEqual(directive, { action: 'shutdown' });
-    assert.equal(getNode('empty-session').status, 'done');
-});
-test('accepted broker input atomically keeps the shadow completion live', async () => {
-    residentMidParkingTurn('admitted-input');
-    assert.deepEqual(await recordParkActivity('admitted-input'), { activity: 'recorded' });
-    const directive = await completePark('admitted-input');
-    assert.deepEqual(directive, { action: 'stay_dormant' });
-    assert.equal(getNode('admitted-input').status, 'active');
-    assert.equal(isPendingPark('admitted-input'), false);
-});
-test('main-session activity already settled before completion releases parking', async () => {
-    residentMidParkingTurn('interactive-mail');
-    const sessionFile = join(home, 'interactive-mail.jsonl');
-    writeFileSync(sessionFile, 'main turn');
-    const afterParkStart = new Date(Date.now() + 1000);
-    utimesSync(sessionFile, afterParkStart, afterParkStart);
-    resetSession('interactive-mail', { sessionId: 'interactive-mail', transcriptPath: sessionFile });
-    const directive = await completePark('interactive-mail');
-    assert.deepEqual(directive, { action: 'stay_dormant' });
-    assert.equal(getNode('interactive-mail').status, 'active');
-    assert.equal(getNode('interactive-mail').intent ?? null, null);
-    assert.equal(isPendingPark('interactive-mail'), false);
-});
-test('a final pushed during the parking turn does not consume the pending marker', async () => {
-    residentMidParkingTurn('finished');
-    const directive = await settle('finished', { pushedFinal: true });
-    assert.deepEqual(directive, { action: 'shutdown' });
-    assert.equal(getNode('finished').intent ?? null, null, 'a pushed final is not a park');
-    assert.equal(isPendingPark('finished'), true, 'main-session settlement does not claim the shadow turn marker');
-});

package/dist/daemon/api/__tests__/canvas-snapshot-fields.test.d.ts

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

package/dist/daemon/api/__tests__/canvas-snapshot-fields.test.js

@@ -1,216 +0,0 @@
-import { after, before, test } from 'node:test';
-import assert from 'node:assert/strict';
-import { createServer } from 'node:net';
-import { mkdirSync, mkdtempSync, rmSync, writeFileSync } from 'node:fs';
-import { join } from 'node:path';
-import { closeDb, migrate, openDb } from '../../../core/canvas/db.js';
-import { closeScopedStores } from '../../../core/scoped-state/db.js';
-import { initializeFreshScopedStore } from '../../../core/scoped-state/migrate.js';
-import { userScopeRoot } from '../../../core/scope.js';
-import { createNode, getNode } from '../../../core/canvas/canvas.js';
-import { contextDir, viewSocketPath } from '../../../core/canvas/paths.js';
-import { beginExecution, recordExecutionPid } from '../../../core/canvas/executions.js';
-import { openTurn, recordFault } from '../../../core/canvas/recovery.js';
-import { headlessBrokerHost } from '../../../core/runtime/host.js';
-import { resetFleetForTests } from '../../../core/runtime/fleet.js';
-import { bindTestFleet } from '../../../core/__tests__/helpers/fleet.js';
-import { handleCreate } from '../handlers/nodes.js';
-import { canvasRoutes } from '../handlers/canvas.js';
-let home = '';
-const previousHome = process.env['CRTR_HOME'];
-const previousUserHome = process.env['HOME'];
-const brokers = [];
-const launchedBrokers = new Map();
-let originalLaunch;
-before(() => {
-    // macOS Unix-domain sockets have a short path limit; keep the canvas home short.
-    home = mkdtempSync('/tmp/crtr-snapshot-fields-');
-    process.env['CRTR_HOME'] = home;
-    // The launch-config worker resolves the user scope from HOME, so it must point
-    // at this fixture's throwaway home and that store must exist. Do NOT chdir:
-    // the worker is forked with the parent's cwd, and outside the repo it cannot
-    // resolve the tsx loader the source test lane runs under.
-    process.env['HOME'] = home;
-    initializeFreshScopedStore({ kind: 'user', root: userScopeRoot() });
-    migrate(openDb());
-    // Every launcher claims a slot from the ledger before it reaches the host, so
-    // a spawn in-process needs a bound fleet (D-2).
-    bindTestFleet();
-    originalLaunch = headlessBrokerHost.launch;
-    headlessBrokerHost.launch = (nodeId) => {
-        const broker = createServer();
-        brokers.push(broker);
-        launchedBrokers.set(nodeId, broker);
-        broker.listen(viewSocketPath(nodeId));
-        return { pid: process.pid, exited: new Promise(() => { }) };
-    };
-});
-after(async () => {
-    headlessBrokerHost.launch = originalLaunch;
-    resetFleetForTests();
-    await Promise.all(brokers.map((broker) => new Promise((resolve) => {
-        if (broker.listening)
-            broker.close(() => resolve());
-        else
-            resolve();
-    })));
-    closeDb();
-    closeScopedStores();
-    rmSync(home, { recursive: true, force: true });
-    if (previousHome === undefined)
-        delete process.env['CRTR_HOME'];
-    else
-        process.env['CRTR_HOME'] = previousHome;
-    if (previousUserHome === undefined)
-        delete process.env['HOME'];
-    else
-        process.env['HOME'] = previousUserHome;
-});
-function meta(nodeId, overrides = {}) {
-    return {
-        node_id: nodeId,
-        name: nodeId,
-        created: new Date().toISOString(),
-        cwd: home,
-        kind: 'general',
-        mode: 'base',
-        lifecycle: 'resident',
-        status: 'idle',
-        host_kind: null,
-        profile_id: null,
-        parent: null,
-        ...overrides,
-    };
-}
-function request(method, path, body) {
-    return {
-        method,
-        path,
-        params: {},
-        query: new URLSearchParams(),
-        body,
-    };
-}
-function snapshotRoute() {
-    const route = canvasRoutes.find((entry) => entry.method === 'GET' && entry.pattern === '/v1/canvas/snapshot');
-    assert.ok(route, 'snapshot route exists');
-    return route;
-}
-/** A live broker is the pid + view socket pair used by isBrokerLive(). */
-async function listenFor(nodeId) {
-    const broker = createServer();
-    brokers.push(broker);
-    await new Promise((resolve, reject) => {
-        broker.once('error', reject);
-        broker.listen(viewSocketPath(nodeId), resolve);
-    });
-    return broker;
-}
-function activeJob(nodeId) {
-    const id = `${Date.now()}-snapshot-test`;
-    const dir = join(contextDir(nodeId), 'jobs', id);
-    mkdirSync(dir, { recursive: true });
-    writeFileSync(join(dir, 'cmd.sh'), 'printf live');
-    writeFileSync(join(dir, 'job.bg'), '');
-}
-test('one canvas snapshot carries title, brief, jobs, busy, latch, fault, and controller facts', async () => {
-    // Use the public create request for the brief: A2's birth contract is the
-    // source of the description that A3 must expose without a follow-up read.
-    const created = await handleCreate(request('POST', '/v1/nodes', {
-        kind: 'general',
-        root: true,
-        profile: null,
-        cwd: home,
-        prompt: '',
-        name: 'Live outcome',
-        description: 'Review the live outcome',
-    }));
-    const liveJobId = created.body.node_id;
-    activeJob(liveJobId);
-    const busyId = 'snapshot-busy';
-    createNode(meta(busyId, {
-        name: 'Busy outcome',
-        parent: liveJobId,
-        host_kind: 'broker',
-    }));
-    // The mid-turn cue is an open turn on the node's live execution, which the
-    // node row projects as `streaming` once its recorded pid is alive. pi_pid is a
-    // projection of the running execution, so the pid is stamped there.
-    const busyExecution = beginExecution(busyId, { daemonEpoch: null });
-    recordExecutionPid(busyExecution, busyId, process.pid);
-    openTurn(busyExecution, { reason: 'agent-start' });
-    const finalId = 'snapshot-final';
-    const finalizedAt = new Date().toISOString();
-    createNode(meta(finalId, {
-        name: 'Final outcome',
-        parent: liveJobId,
-        lifecycle: 'terminal',
-        status: 'done',
-        final_report: 'final.md',
-        finalized_at: finalizedAt,
-    }));
-    const faultId = 'snapshot-fault';
-    createNode(meta(faultId, {
-        name: 'Fault outcome',
-        parent: liveJobId,
-        status: 'dead',
-    }));
-    recordFault(faultId, {
-        link: 'viewer↔broker',
-        op: 'snapshot-test',
-        kind: 'connection',
-        retry: { disposition: 'manual', by: 'client' },
-        message: 'forced snapshot fault',
-    });
-    const controllerId = 'snapshot-controller';
-    createNode(meta(controllerId, { name: 'Controller', parent: liveJobId }));
-    const waitingId = 'snapshot-waiting';
-    createNode(meta(waitingId, {
-        name: 'Waiting outcome',
-        parent: liveJobId,
-        waiting_for: controllerId,
-    }));
-    const liveBroker = launchedBrokers.get(liveJobId);
-    assert.ok(liveBroker, 'created node has a live broker');
-    const busyBroker = await listenFor(busyId);
-    try {
-        const result = await snapshotRoute().handler(request('GET', '/v1/canvas/snapshot'));
-        const body = result.body;
-        const byId = new Map(body.nodes.map((node) => [node['node_id'], node]));
-        const live = byId.get(liveJobId);
-        assert.ok(live);
-        assert.equal(live['title'], 'Live outcome');
-        assert.equal(live['description'], 'Review the live outcome');
-        assert.equal(live['name'], 'Live outcome Review the live outcome');
-        assert.equal(live['jobs_active'], 1);
-        assert.equal(live['streaming'], false);
-        assert.equal(live['waiting_for'], null);
-        const busy = byId.get(busyId);
-        assert.ok(busy);
-        assert.equal(busy['title'], 'Busy outcome');
-        assert.equal(busy['jobs_active'], 0);
-        assert.equal(busy['streaming'], true);
-        const final = byId.get(finalId);
-        assert.ok(final);
-        assert.equal(final['title'], 'Final outcome');
-        assert.equal(final['final_report'], 'final.md');
-        assert.equal(final['finalized_at'], finalizedAt);
-        const fault = byId.get(faultId);
-        assert.ok(fault);
-        const hanging = fault['hanging'];
-        assert.equal(hanging.link, 'viewer↔broker');
-        assert.equal(hanging.op, 'snapshot-test');
-        assert.equal(hanging.kind, 'connection');
-        assert.deepEqual(hanging.retry, { disposition: 'manual', by: 'client' });
-        assert.equal(hanging.message, 'forced snapshot fault');
-        const waiting = byId.get(waitingId);
-        assert.ok(waiting);
-        assert.equal(waiting['title'], 'Waiting outcome');
-        assert.equal(waiting['waiting_for'], controllerId);
-        assert.equal(waiting['jobs_active'], 0);
-    }
-    finally {
-        await Promise.all([liveBroker, busyBroker].map((broker) => new Promise((resolve) => broker.close(() => resolve()))));
-    }
-    assert.ok(getNode(liveJobId));
-});

package/dist/daemon/api/__tests__/error-seam.test.d.ts

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

package/dist/daemon/api/__tests__/error-seam.test.js

@@ -1,83 +0,0 @@
-import assert from 'node:assert/strict';
-import { test } from 'node:test';
-import { mkdtempSync, rmSync } from 'node:fs';
-import { tmpdir } from 'node:os';
-import { join } from 'node:path';
-import { toErrorBody } from '../map.js';
-import { ApiError } from '../../../api/errors.js';
-import { apiErrorToCliError, InputError } from '../../../core/io.js';
-import { brokerLaunchFailed } from '../../../core/errors.js';
-import { createManagedWorktree } from '../../../core/worktree.js';
-// Regression for crouter#521: choosing the root profile on a virgin machine
-// died with a bare `<error code="internal">\ninternal error` and ZERO cause —
-// the front-door `createNode({root:true})` boot failure was thrown as a plain
-// Error deep in spawnChild, collapsed to a 500 by the daemon (which stripped
-// the message AND logged nothing), and rebuilt client-side as an opaque
-// internal. These tests pin the two halves of the fix at the seam they cross.
-/** Reproduce crtrd's server → wire → client round-trip exactly: the daemon maps
- *  a thrown value to a wire body (`toErrorBody`), the client rebuilds an
- *  `ApiError` from that body the way `client.ts:parse` does, and
- *  `apiErrorToCliError` turns it into the CLI-facing error a caller sees. */
-function roundTrip(thrown) {
-    const { status, body } = toErrorBody(thrown);
-    const client = new ApiError(status, body.error.code, body.error.message, body.error.details);
-    const cli = apiErrorToCliError(client);
-    assert.ok(cli instanceof InputError, 'a 4xx/5xx round-trips to an InputError the CLI renders');
-    return cli;
-}
-test('a named broker-launch failure round-trips verbatim, never a bare internal', () => {
-    const cli = roundTrip(brokerLaunchFailed('the root broker mrxz-1 could not start — resolveLaunchModel: no model configured', 'Resolve the cause named above, then retry.'));
-    // The specific code, curated message, and actionable next all survive the API
-    // boundary — the caller learns exactly what failed and what to do.
-    assert.equal(cli.payload.error, 'broker_launch_failed');
-    assert.match(cli.payload.message, /could not start — resolveLaunchModel: no model configured/);
-    assert.equal(cli.payload.next, 'Resolve the cause named above, then retry.');
-    // The exact failure mode #521 reported must NOT recur.
-    assert.notEqual(cli.payload.error, 'internal');
-    assert.notEqual(cli.payload.message, 'internal error');
-});
-test('an unanticipated 500 surfaces its cause instead of a causeless "internal error"', () => {
-    const cli = roundTrip(new Error('sqlite: database is locked'));
-    assert.equal(cli.payload.error, 'internal');
-    // The real cause reaches the caller (previously stripped to "internal error").
-    assert.match(cli.payload.message, /database is locked/);
-    assert.notEqual(cli.payload.message, 'internal error');
-    // ...and the next-hint is the crtr-bug guidance, not "adjust the call".
-    assert.match(cli.payload.next, /crtr bug/);
-});
-// Regression for crouter#811: a `crtr node new --worktree` against a cwd that is
-// not inside a git repository reached the caller as `500 internal`, hiding the
-// domain error the worktree layer already raised for exactly that input.
-test('a non-git cwd round-trips the worktree layer\'s own code + next-hint, never a 500', () => {
-    const outside = mkdtempSync(join(tmpdir(), 'crtr-nongit-'));
-    try {
-        let thrown;
-        try {
-            createManagedWorktree(outside, 'node-that-needs-a-repo');
-        }
-        catch (err) {
-            thrown = err;
-        }
-        assert.ok(thrown !== undefined, 'a worktree create outside a repository must fail');
-        assert.equal(toErrorBody(thrown).status, 400);
-        const cli = roundTrip(thrown);
-        assert.equal(cli.payload.error, 'not_git_repo');
-        assert.match(cli.payload.next, /--cwd/);
-    }
-    finally {
-        rmSync(outside, { recursive: true, force: true });
-    }
-});
-test('a 500 body carries only the bounded first line, never a multi-line stack', () => {
-    const withStack = new Error('top cause of the failure');
-    withStack.stack = 'Error: top cause of the failure\n    at foo (bar.js:1:1)\n    at baz (qux.js:2:2)';
-    // A real thrown Error surfaces its .message (single line by construction); the
-    // guard additionally clamps any embedded newline so a stack can never leak.
-    const multiline = new Error('first line\n    at leak (secret.js:9:9)');
-    const { status, body } = toErrorBody(multiline);
-    assert.equal(status, 500);
-    assert.equal(body.error.message, 'first line');
-    assert.equal(body.error.details, undefined, 'a 500 still omits structured details for hygiene');
-    // sanity: an ordinary message passes through untouched
-    assert.equal(toErrorBody(withStack).body.error.message, 'top cause of the failure');
-});

package/dist/daemon/api/__tests__/model-config.test.d.ts

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

package/dist/daemon/api/__tests__/model-config.test.js

@@ -1,73 +0,0 @@
-import { after, afterEach, beforeEach, test } from 'node:test';
-import assert from 'node:assert/strict';
-import { mkdirSync, mkdtempSync, rmSync, writeFileSync } from 'node:fs';
-import { tmpdir } from 'node:os';
-import { join } from 'node:path';
-import { ApiError } from '../../../api/errors.js';
-import { readRawScopeConfig, updateRawConfigAtomically } from '../../../core/config.js';
-import { resetScopeCache } from '../../../core/scope.js';
-import { initializeFreshScopedStore } from '../../../core/scoped-state/migrate.js';
-import { modelConfigRoutes } from '../handlers/model-config.js';
-const previousHome = process.env.HOME;
-const previousAgentDir = process.env['PI_CODING_AGENT_DIR'];
-let home = '';
-const payload = {
-    config_version: 'core-doc-digest',
-    modelRoutes: {
-        northlight: {
-            family: 'cloud', credentialSource: 'northlight', providerId: 'northlight',
-            models: { ultra: 'northlight/u', strong: 'northlight/s:high', medium: 'northlight/m', light: 'northlight/l' },
-        },
-    },
-    modelRouting: { familyOrder: ['cloud'], strengthFallback: ['strong', 'medium'] },
-    kinds: { developer: { model: 'cloud/medium' } },
-};
-beforeEach(() => {
-    home = mkdtempSync(join(tmpdir(), 'crtr-model-config-'));
-    process.env.HOME = home;
-    const agentDir = join(home, 'agent');
-    mkdirSync(agentDir);
-    process.env['PI_CODING_AGENT_DIR'] = agentDir;
-    writeFileSync(join(agentDir, 'models.json'), JSON.stringify({
-        providers: { northlight: { baseUrl: 'http://127.0.0.1:1', api: 'openai-completions', apiKey: 'test', models: ['u', 's', 'm', 'l'].map((id) => ({ id, name: id })) } },
-    }));
-    initializeFreshScopedStore({ kind: 'user', root: join(home, '.crouter') });
-    resetScopeCache();
-});
-afterEach(() => {
-    rmSync(home, { recursive: true, force: true });
-    resetScopeCache();
-});
-after(() => {
-    if (previousHome === undefined)
-        delete process.env.HOME;
-    else
-        process.env.HOME = previousHome;
-    if (previousAgentDir === undefined)
-        delete process.env['PI_CODING_AGENT_DIR'];
-    else
-        process.env['PI_CODING_AGENT_DIR'] = previousAgentDir;
-    resetScopeCache();
-});
-function putRoute() {
-    const route = modelConfigRoutes.find((entry) => entry.method === 'PUT' && entry.pattern === '/v1/model-config');
-    assert.ok(route, 'PUT /v1/model-config is registered');
-    return route;
-}
-function context(body) {
-    return { method: 'PUT', path: '/v1/model-config', params: {}, query: new URLSearchParams(), body };
-}
-test('PUT atomically replaces Core-owned blocks without touching unrelated config', async () => {
-    updateRawConfigAtomically('user', () => ({ completion_bell: false, kinds: { developer: { tools: ['bash'], model: 'old/strong', orchestratorModel: 'old/ultra' } } }));
-    const result = await putRoute().handler(context(payload));
-    assert.deepEqual(result, { status: 200, body: { config_version: 'core-doc-digest', delivered: true } });
-    const written = readRawScopeConfig('user');
-    assert.equal(written.completion_bell, false);
-    assert.deepEqual(written.kinds?.['developer'], { tools: ['bash'], model: 'cloud/medium', orchestratorModel: 'cloud/medium' });
-    assert.deepEqual(written.modelRoutes, payload.modelRoutes);
-});
-test('PUT rejects invalid payloads and absent baked refs before writing', async () => {
-    await assert.rejects(async () => putRoute().handler(context({ ...payload, modelRoutes: { ...payload.modelRoutes, northlight: { ...payload.modelRoutes.northlight, models: { strong: 'northlight/s' } } } })), (error) => error instanceof ApiError && error.status === 400 && error.code === 'invalid_request');
-    await assert.rejects(async () => putRoute().handler(context({ ...payload, modelRoutes: { ...payload.modelRoutes, northlight: { ...payload.modelRoutes.northlight, models: { ...payload.modelRoutes.northlight.models, ultra: 'northlight/missing' } } } })), (error) => error instanceof ApiError && error.status === 422 && error.code === 'unprocessable');
-    assert.equal(readRawScopeConfig('user'), null);
-});

package/dist/daemon/api/__tests__/node-create-description.test.d.ts

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