@crouton-kit/crouter 0.3.21 → 0.3.23

package/dist/commands/skill/shared.js

@@ -1,56 +1,7 @@
 import { usage } from '../../core/errors.js';
-import { SCOPE_SKILL_PLUGIN } from '../../types.js';
-import { listSkillSiblings, listSkillChildren, } from '../../core/resolver.js';
 import { resolveScopeArg, projectScopeRoot } from '../../core/scope.js';
 // ---------------------------------------------------------------------------
-// Neighbors section (ported from old impl)
-// ---------------------------------------------------------------------------
-export function formatNeighborQualifier(s) {
-    return s.plugin === SCOPE_SKILL_PLUGIN
-        ? `${s.scope}/${s.name}`
-        : `${s.plugin}/${s.name}`;
-}
-export function formatNeighborKeywords(s) {
-    const kw = s.frontmatter.keywords;
-    if (!kw || kw.length === 0)
-        return '';
-    return ` — [${kw.join(', ')}]`;
-}
-export function buildNeighborsSection(skill) {
-    const siblings = listSkillSiblings(skill);
-    const children = listSkillChildren(skill);
-    if (siblings.length === 0 && children.length === 0)
-        return null;
-    const lines = [
-        '## Neighbors',
-        '*Auto-discovered from filesystem. Run `crtr memory read <name>` for full description + body.*',
-        '',
-    ];
-    if (siblings.length > 0) {
-        lines.push('**Siblings:**');
-        for (const s of siblings) {
-            lines.push(`- \`${formatNeighborQualifier(s)}\`${formatNeighborKeywords(s)}`);
-        }
-        if (children.length > 0)
-            lines.push('');
-    }
-    if (children.length > 0) {
-        lines.push('**Nested:**');
-        for (const s of children) {
-            lines.push(`- \`${formatNeighborQualifier(s)}\`${formatNeighborKeywords(s)}`);
-        }
-    }
-    return lines.join('\n');
-}
-export function appendNeighbors(skill, body) {
-    const section = buildNeighborsSection(skill);
-    if (section === null)
-        return body;
-    const sep = body.endsWith('\n') ? '\n' : '\n\n';
-    return body + sep + `<neighbors>\n${section}\n</neighbors>\n`;
-}
-// ---------------------------------------------------------------------------
-// Resolve scope for enable/disable/scaffold
+// Resolve scope for scaffold
 // ---------------------------------------------------------------------------
 export function resolveWriteScope(scopeStr) {
     if (scopeStr !== undefined) {
@@ -73,19 +24,7 @@ export function renderCatalogSection(label, sources, descriptions, out) {
     const count = sources.reduce((n, s) => n + s.roots.length, 0);
     out.push('');
     out.push(`${label} (${count})`);
-    const sentinel = sources.filter((s) => s.plugin === SCOPE_SKILL_PLUGIN);
-    const named = sources
-        .filter((s) => s.plugin !== SCOPE_SKILL_PLUGIN)
-        .sort((a, b) => a.plugin.localeCompare(b.plugin));
-    for (const s of sentinel) {
-        if (s.roots.length > CATALOG_T) {
-            out.push(`  (scope skills)  ${s.roots.length} skills`);
-        }
-        else {
-            for (const n of s.roots)
-                out.push(`  ${n}`);
-        }
-    }
+    const named = [...sources].sort((a, b) => a.plugin.localeCompare(b.plugin));
     if (named.length === 0)
         return;
     const classified = named.map((s) => {
@@ -123,6 +62,13 @@ export function renderCatalogSection(label, sources, descriptions, out) {
     })
         .reduce((m, p) => Math.max(m, p.plugin.length + 1), 0);
     for (const p of classified) {
+        // Bare native/builtin skills (no plugin namespace) carry sourceKey ''.
+        // Render them as flat lines, never under a stray '/' header.
+        if (p.plugin === '') {
+            for (const r of [...p.roots].sort())
+                out.push(`  ${r}`);
+            continue;
+        }
         const direct = p.subcats.size + p.bare.length;
         if (direct > CATALOG_T) {
             out.push(`  ${(p.plugin + '/').padEnd(inlineW)}  ${p.roots.length} skills${descSuffix(p.plugin)}`);

package/dist/commands/skill.js

@@ -1,22 +1,21 @@
 // `crtr skill` subtree handlers.
-// Sub-branches: author {guide, scaffold}, state {enable, disable}.
+// Sub-branch: author {guide, scaffold}.
 // Discovery and reading are now `crtr memory` verbs.
 import { defineBranch } from '../core/command.js';
 import { authorBranch } from './skill/author.js';
-import { stateBranch } from './skill/state.js';
 export function registerSkill() {
     return defineBranch({
         name: 'skill',
         rootEntry: {
             concept: 'a SKILL.md you author and manage',
             desc: 'author and manage skills',
-            useWhen: 'authoring a new skill (`crtr skill author`) or toggling a skill on/off (`crtr skill state`). To discover and read skills use `crtr memory` — `crtr memory read <name>`, `crtr memory find`, `crtr memory list`.',
+            useWhen: 'authoring a new skill (`crtr skill author`). To discover and read skills use `crtr memory` — `crtr memory read <name>`, `crtr memory find`, `crtr memory list`.',
         },
         help: {
             name: 'skill',
-            summary: 'author and manage skill state — use `crtr memory` to discover and read skills',
-            model: '`author` when you are writing a new skill — it carries the template workflow and the scaffolder. `state` when a skill should be hidden from discovery without being removed. Discovery and reading are `crtr memory` verbs: `crtr memory list`, `crtr memory find`, `crtr memory read`. Append `-h` at any branch or leaf for its full schema.',
+            summary: 'author skills — use `crtr memory` to discover and read them',
+            model: '`author` when you are writing a new skill — it carries the template workflow and the scaffolder. Discovery and reading are `crtr memory` verbs: `crtr memory list`, `crtr memory find`, `crtr memory read`. Visibility is governed by the substrate (INDEX/rung/gate), not a per-skill on/off. Append `-h` at any branch or leaf for its full schema.',
         },
-        children: [authorBranch, stateBranch],
+        children: [authorBranch],
     });
 }

package/dist/commands/sys/doctor.js

@@ -1,54 +1,16 @@
 import { join } from 'node:path';
 import { defineLeaf } from '../../core/command.js';
 import { readConfig, updateConfig } from '../../core/config.js';
-import { scopeRoot, listScopes, builtinSkillsRoot, marketplacesDir, pluginsDir } from '../../core/scope.js';
-import { listInstalledPlugins, listSkillsInPlugin } from '../../core/resolver.js';
+import { scopeRoot, listScopes, marketplacesDir, pluginsDir } from '../../core/scope.js';
 import { readMarketplaceManifest, readPluginManifest } from '../../core/manifest.js';
-import { parseFrontmatter } from '../../core/frontmatter.js';
-import { pathExists, listDirs, removePath, readText, writeText } from '../../core/fs-utils.js';
+import { pathExists, listDirs, removePath } from '../../core/fs-utils.js';
 import { lsRemote } from '../../core/git.js';
-import { SKILL_TYPES, isSkillType } from '../../types.js';
 function pass(scope, name, message) {
     return { scope, name, status: 'pass', message };
 }
 function failCheck(scope, name, message, remediation) {
     return { scope, name, status: 'fail', message, ...(remediation ? { remediation } : {}) };
 }
-function warnCheck(scope, name, message, remediation) {
-    return { scope, name, status: 'warn', message, ...(remediation ? { remediation } : {}) };
-}
-/**
- * Surgically replace a single frontmatter scalar field's value in a SKILL.md
- * file. Preserves the rest of the file (key order, comments, extra fields,
- * body) exactly. Returns true on success, false if no frontmatter or no such
- * field was found.
- */
-function editFrontmatterField(filePath, field, newValue) {
-    let src;
-    try {
-        src = readText(filePath);
-    }
-    catch {
-        return false;
-    }
-    const fmMatch = src.match(/^(---\s*\r?\n)([\s\S]*?)(\r?\n---\s*\r?\n?)/);
-    if (!fmMatch)
-        return false;
-    const [, head, body, tail] = fmMatch;
-    const fieldRe = new RegExp(`^(\\s*${field}\\s*:\\s*)(.*)$`, 'm');
-    if (!fieldRe.test(body))
-        return false;
-    const quoted = /[:#\-\[\]{},&*?|<>=!%@`]/.test(newValue) || /^\s/.test(newValue) || /\s$/.test(newValue);
-    const formatted = quoted ? `"${newValue.replace(/"/g, '\\"')}"` : newValue;
-    const newBody = body.replace(fieldRe, `$1${formatted}`);
-    try {
-        writeText(filePath, head + newBody + tail + src.slice(fmMatch[0].length));
-        return true;
-    }
-    catch {
-        return false;
-    }
-}
 /**
  * Apply a remediation. Returns true if applied successfully. Idempotent for
  * the supported kinds (re-applying a config-key removal that's already gone
@@ -79,11 +41,6 @@ function applyRemediation(rem) {
                 removePath(rem.path);
                 return true;
             }
-            case 'edit_frontmatter': {
-                if (!rem.filePath || !rem.field || rem.value === undefined)
-                    return false;
-                return editFrontmatterField(rem.filePath, rem.field, rem.value);
-            }
             default:
                 return false;
         }
@@ -92,46 +49,7 @@ function applyRemediation(rem) {
         return false;
     }
 }
-function readRawTypeField(skillPath) {
-    const content = readText(skillPath);
-    const { raw } = parseFrontmatter(content);
-    if (!raw)
-        return undefined;
-    const m = raw.match(/^type:\s*(.+?)\s*$/m);
-    if (!m)
-        return undefined;
-    let v = m[1].trim();
-    if ((v.startsWith('"') && v.endsWith('"')) || (v.startsWith("'") && v.endsWith("'"))) {
-        v = v.slice(1, -1);
-    }
-    return v;
-}
-function runChecksForBuiltin() {
-    const root = builtinSkillsRoot();
-    const plugins = listInstalledPlugins('builtin');
-    if (plugins.length === 0) {
-        return [failCheck('builtin', 'builtin:crtr:root', `builtin-skills root missing or has no valid plugin.json: ${root}`)];
-    }
-    const results = [
-        pass('builtin', 'builtin:crtr:root', `builtin-skills root present: ${root}`),
-    ];
-    for (const plugin of plugins) {
-        results.push(pass('builtin', `builtin:${plugin.name}:manifest`, `manifest valid`));
-        const skills = listSkillsInPlugin(plugin);
-        for (const skill of skills) {
-            if (!skill.frontmatter.name) {
-                results.push(failCheck('builtin', `builtin:${plugin.name}:skill:${skill.name}:frontmatter`, `frontmatter missing or name field empty`));
-            }
-            else {
-                results.push(pass('builtin', `builtin:${plugin.name}:skill:${skill.name}:frontmatter`, `frontmatter valid`));
-            }
-        }
-    }
-    return results;
-}
 function runChecksForScope(scope, opts) {
-    if (scope === 'builtin')
-        return runChecksForBuiltin();
     const results = [];
     const root = scopeRoot(scope);
     if (!root)
@@ -253,62 +171,6 @@ function runChecksForScope(scope, opts) {
             else {
                 seenPluginNames.set(name, dir);
             }
-            // Check: skills frontmatter name. Convention: frontmatter `name:` holds
-            // the leaf segment only (e.g. "cli-design"); the full discovered name
-            // ("interface/cli-design") is derived from the path automatically.
-            const plugin = listInstalledPlugins(scope).find((p) => p.name === name);
-            if (plugin) {
-                const skills = listSkillsInPlugin(plugin);
-                for (const skill of skills) {
-                    const checkName = `plugin:${name}:skill:${skill.name}:frontmatter`;
-                    const segments = skill.name.split('/');
-                    const baseName = segments[segments.length - 1];
-                    if (skill.frontmatter.name === baseName) {
-                        results.push(pass(scope, checkName, `frontmatter valid`));
-                    }
-                    else if (skill.frontmatter.name === '') {
-                        const remediation = {
-                            kind: 'edit_frontmatter',
-                            description: `Set frontmatter "name: ${baseName}" (discovered name "${skill.name}" is auto-derived from the directory path; frontmatter holds the base segment only)`,
-                            filePath: skill.path,
-                            field: 'name',
-                            value: baseName,
-                        };
-                        if (opts.fix && applyRemediation(remediation)) {
-                            results.push({ scope, name: checkName, status: 'fail', message: `frontmatter name was missing — set to "${baseName}"`, fixed: true, remediation });
-                        }
-                        else {
-                            results.push(failCheck(scope, checkName, `frontmatter missing or name field empty`, remediation));
-                        }
-                    }
-                    else {
-                        const remediation = {
-                            kind: 'edit_frontmatter',
-                            description: `Replace frontmatter "name: ${skill.frontmatter.name}" with "name: ${baseName}" (discovered name "${skill.name}" is auto-derived from the directory path; frontmatter holds the base segment only)`,
-                            filePath: skill.path,
-                            field: 'name',
-                            value: baseName,
-                        };
-                        if (opts.fix && applyRemediation(remediation)) {
-                            results.push({ scope, name: checkName, status: 'warn', message: `frontmatter name updated from "${skill.frontmatter.name}" to "${baseName}"`, fixed: true, remediation });
-                        }
-                        else {
-                            results.push(warnCheck(scope, checkName, `name mismatch: frontmatter says "${skill.frontmatter.name}", expected base name "${baseName}" (discovered as "${skill.name}")`, remediation));
-                        }
-                    }
-                    const typeCheckName = `plugin:${name}:skill:${skill.name}:type`;
-                    const rawType = readRawTypeField(skill.path);
-                    if (rawType === undefined) {
-                        results.push(warnCheck(scope, typeCheckName, `missing type field — add one of: ${SKILL_TYPES.join(' | ')}`));
-                    }
-                    else if (!isSkillType(rawType)) {
-                        results.push(failCheck(scope, typeCheckName, `invalid type "${rawType}" — valid: ${SKILL_TYPES.join(' | ')}`));
-                    }
-                    else {
-                        results.push(pass(scope, typeCheckName, `type: ${rawType}`));
-                    }
-                }
-            }
             // Git remote check (slow, opt-in)
             if (opts.remote && manifest.source) {
                 const res = lsRemote(manifest.source);
@@ -326,23 +188,23 @@ function runChecksForScope(scope, opts) {
 export const sysDoctorLeaf = defineLeaf({
     name: 'doctor',
     description: 'diagnose installation health',
-    whenToUse: 'something in your crtr install looks off and you want it diagnosed — a plugin or marketplace manifest is missing, a config entry points at a directory that no longer exists, or skill frontmatter has drifted from its filename. Reports each problem with a structured remediation, and can apply the repairs for you.',
+    whenToUse: 'something in your crtr install looks off and you want it diagnosed — a plugin or marketplace manifest is missing, or a config entry points at a directory that no longer exists. Reports each problem with a structured remediation, and can apply the repairs for you.',
     help: {
         name: 'sys doctor',
-        summary: 'diagnose missing manifests, broken config entries, and skill frontmatter drift',
+        summary: 'diagnose missing manifests and broken config entries',
         params: [
             { kind: 'flag', name: 'scope', type: 'enum', choices: ['user', 'project'], required: false, constraint: 'Scope to check. Default: all scopes.' },
             { kind: 'flag', name: 'fix', type: 'bool', required: false, constraint: 'Apply each non-pass check\'s remediation and report what was fixed.' },
             { kind: 'flag', name: 'remote', type: 'bool', required: false, constraint: 'Check git remotes with ls-remote (slow — makes network calls).' },
         ],
         output: [
-            { name: 'checks', type: 'object[]', required: true, constraint: 'Each: {scope, name, status, message, fixed?, remediation?}. status: pass | fail | warn. remediation (when present) is {kind, description, ...payload} where kind is remove_config_key | rm_path | edit_frontmatter. Sorted by scope then name.' },
+            { name: 'checks', type: 'object[]', required: true, constraint: 'Each: {scope, name, status, message, fixed?, remediation?}. status: pass | fail. remediation (when present) is {kind, description, ...payload} where kind is remove_config_key | rm_path. Sorted by scope then name.' },
             { name: 'ok', type: 'boolean', required: true, constraint: 'True when no unresolved fail checks remain.' },
         ],
         outputKind: 'object',
         effects: [
             'Read-only unless --fix is passed.',
-            'With --fix: applies each non-pass check\'s `remediation` — removes stale config entries, deletes dangling plugin/marketplace directories, edits frontmatter name fields to the base-name convention.',
+            'With --fix: applies each non-pass check\'s `remediation` — removes stale config entries and deletes dangling plugin/marketplace directories.',
             'Each non-pass result carries a structured `remediation` describing the fix action (absolute paths, exact config keys) so callers can apply it directly without --fix.',
         ],
     },

package/dist/core/__tests__/broker-dormant-wake.test.js

@@ -1,51 +1,88 @@
 // Run with: node --import tsx/esm --test src/core/__tests__/broker-dormant-wake.test.ts
 //
-// Acceptance item 3 (DORMANT) of the headless-broker migration. Split out of
-// broker-lifecycle.test.ts (see its header for the full file map); the test is
-// the original acceptance gate, unchanged, on its own isolated harness.
+// HEADLESS RETARGET (foundation-spec §C.5 + §E). Acceptance item 3 (DORMANT) of
+// the headless-broker migration, fabricated DIRECTLY in the canvas — no real
+// tmux session, no pane chrome, and NO real broker boot. The wake decision is
+// pure daemon logic, so the test never pays the ~5s SDK-boot cost.
+//
+// (1) BUG LOCKED — Invariant D (idle-release dormancy). A node that naturally
+//     stops while awaiting a LIVE subscription idle-RELEASES its broker (the
+//     broker process exits); the daemon's SECOND PASS must revive it (RESUME)
+//     the moment an UNSEEN inbox entry appears. Without the second pass a dormant
+//     orchestrator never wakes on a worker's message and the graph deadlocks.
+//
+// (2) WHY MODEL-LEVEL, NOT TMUX CHROME — every step is canvas/daemon state: the
+//     dormant state is status=idle / intent=idle-release / pi_pid dead (a
+//     released broker is a dead pid, the headless analog of a closed pane), and
+//     superviseTick's second pass gates on status===idle && intent===idle-release
+//     && !isPidAlive(pi_pid) && readInboxSince>cursor → reviveNode(resume). No
+//     pane is read anywhere; a broker is supervised purely by pid, so a
+//     fabricated row reproduces the exact dormant state with no process at all.
+//
+// (3) HOW THE FABRICATED DRIVE STILL FAILS IF THE BUG REGRESSES — pass 1: a tick
+//     with NO unseen inbox must NOT revive (cycles unchanged) — the idle-release
+//     early-return. Then we appendInbox an unseen entry and tick again; pass 2's
+//     reviveNode bumps cycles + transition('revive') BEFORE the (detached,
+//     unawaited) broker spawn, so the wake is observable instantly via cycles +
+//     status. If the second-pass reviveNode is removed, the post-inbox tick never
+//     bumps cycles → the "revived" asserts go RED. Verified by commenting the
+//     second-pass reviveNode(resume) call (see bug-injection report).
 import { test, before, after } from 'node:test';
 import assert from 'node:assert/strict';
-import { createHarness, hasTmux } from './helpers/harness.js';
-import { subscribe } from '../canvas/canvas.js';
+import { spawnSync } from 'node:child_process';
+import { createHarness } from './helpers/harness.js';
 import { appendInbox } from '../feed/inbox.js';
-import { isPidAlive } from '../canvas/pid.js';
 let h;
-let root;
 before(async () => {
-    if (!hasTmux())
-        return;
-    h = await createHarness({ sessionPrefix: 'crtr-brkdorm' });
-    // An active resident root: the spawn parent AND the live node a child can hold
-    // an active subscription to (so its natural stop classifies 'awaiting').
-    root = h.spawnRoot('broker-dormant suite root');
+    h = await createHarness({ headless: true, sessionPrefix: 'crtr-brkdorm' });
 });
 after(async () => {
     if (h !== undefined)
         await h.dispose();
 });
+// A pid that is reaped (dead) by the time spawnSync returns — a released
+// broker's exited process.
+function deadPid() {
+    const r = spawnSync('true', [], { stdio: 'ignore' });
+    return r.pid ?? 0x7ffffffe;
+}
 // ===========================================================================
-// Item 3 (DORMANT) — a natural stop while awaiting a live subscription idle-
-// RELEASES the broker (it exits); the daemon's second pass revives it (RESUME)
-// on the next unseen inbox entry.
+// Item 3 (DORMANT) — a node idle-RELEASED (its broker exited) is NOT revived by
+// a bare tick, but the daemon's second pass revives it (RESUME) on the next
+// unseen inbox entry.
 // ===========================================================================
-test('DORMANT wake — idle-release → broker exits → daemon pass-2 reviveNode(resume) on inbox', { skip: !hasTmux() }, async () => {
-    const id = await h.spawnHeadlessChild(root, 'headless worker — dormant wake');
-    const pid = h.node(id).pi_pid;
-    // Give the (terminal) child an ACTIVE subscription to the LIVE root, so the
-    // stop-guard classifies its natural stop as 'awaiting' (a dormant orchestrator
-    // awaiting a worker) → the stophook idle-releases it (paneless → no focus).
-    subscribe(id, root, true);
-    await h.stop(id, 'stop');
-    await h.waitForStatus(id, 'idle');
-    assert.equal(h.node(id).intent, 'idle-release', 'awaiting + unfocused → idle-release');
-    await h.waitFor(() => !isPidAlive(pid), { label: 'released broker process exited' });
-    // Daemon owns wake-on-message for a dormant (pi-dead) broker: an unseen inbox
-    // entry + a tick → pass 2 → reviveNode(resume:true) → a fresh broker boots.
+test('DORMANT wake — idle-release broker (pi dead) → daemon pass-2 reviveNode(resume) on unseen inbox', async () => {
+    // Fabricate the post-release dormant state directly: status=idle,
+    // intent=idle-release, a DEAD supervised pid, pi_session_id set (so the revive
+    // RESUMES the saved session). No boot, no stop, no pane.
+    const id = h.fabricateBrokerNode({
+        status: 'idle',
+        intent: 'idle-release',
+        pi_pid: deadPid(),
+        pi_session_id: 'sess-dorm',
+    });
+    {
+        const n = h.node(id);
+        assert.equal(n.host_kind, 'broker', 'broker-hosted (paneless)');
+        assert.equal(n.status, 'idle', 'idle (released)');
+        assert.equal(n.intent, 'idle-release', 'intent=idle-release');
+        assert.equal(n.cycles ?? 0, 0, 'no revive yet');
+    }
+    // PASS 1 — a tick with NO unseen inbox must NOT revive: idle-release is
+    // dormancy by choice; the daemon leaves it until an inbox entry arrives.
+    await h.tick();
+    assert.equal(h.node(id).cycles ?? 0, 0, 'no unseen inbox → NOT revived (idle-release left dormant)');
+    assert.equal(h.node(id).status, 'idle', 'still idle after the empty tick');
+    // PASS 2 — an unseen inbox entry + a tick → the daemon owns wake-on-message for
+    // a dormant (pi-dead) broker: second pass → reviveNode(resume:true).
     appendInbox(id, { from: 'tester', tier: 'normal', kind: 'message', label: 'resume-me', data: { body: 'work after release' } });
     await h.tick();
-    const boot2 = await h.awaitBoot(id, { minCount: 2 });
-    assert.equal(boot2.resuming, true, 'the dormant revive RESUMES the saved session');
-    const newPid = h.node(id).pi_pid;
-    assert.ok(newPid != null && isPidAlive(newPid), 'the revived broker pid is alive');
-    assert.notEqual(newPid, pid, 'the revived broker is a fresh process');
+    // reviveNode bumps cycles + transition('revive') BEFORE the (detached,
+    // unawaited) broker spawn, so the wake is observable instantly.
+    {
+        const n = h.node(id);
+        assert.equal(n.cycles ?? 0, 1, 'unseen inbox + tick → daemon pass-2 REVIVED (cycles bumped to 1)');
+        assert.equal(n.status, 'active', 'revive → active (transition(revive))');
+        assert.equal(n.intent ?? null, null, 'intent cleared by the revive');
+    }
 });

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

@@ -13,11 +13,13 @@ import registerCanvasInboxWatcher from '../../pi-extensions/canvas-inbox-watcher
 import { appendInbox } from '../feed/inbox.js';
 import { createNode, setIntent } from '../canvas/canvas.js';
 import { closeDb } from '../canvas/db.js';
-// Mirror the watcher's internal cadence (TICK_MS=800, DEBOUNCE_MS=1000): allow a
-// resolve+seed tick, a read tick, and the debounce window before asserting.
-const TICK_MS = 800;
-const DEBOUNCE_MS = 1000;
-const SETTLE_MS = TICK_MS * 2 + DEBOUNCE_MS + 500;
+// Drive the watcher's injectable cadence seam (CRTR_WATCHER_TICK_MS /
+// CRTR_WATCHER_DEBOUNCE_MS) at a fast tempo so the test sleeps milliseconds, not
+// seconds. SETTLE_MS still allows a resolve+seed tick, a read tick, and the
+// debounce window before asserting, exactly as against the real 800/1000 cadence.
+const TICK_MS = 20;
+const DEBOUNCE_MS = 25;
+const SETTLE_MS = TICK_MS * 2 + DEBOUNCE_MS + 30;
 let origHome;
 let origNode;
 const homes = [];
@@ -40,9 +42,15 @@ function makeFakePi() {
     };
 }
 const wait = (ms) => new Promise((r) => setTimeout(r, ms));
+let origTick;
+let origDebounce;
 before(() => {
     origHome = process.env['CRTR_HOME'];
     origNode = process.env['CRTR_NODE_ID'];
+    origTick = process.env['CRTR_WATCHER_TICK_MS'];
+    origDebounce = process.env['CRTR_WATCHER_DEBOUNCE_MS'];
+    process.env['CRTR_WATCHER_TICK_MS'] = String(TICK_MS);
+    process.env['CRTR_WATCHER_DEBOUNCE_MS'] = String(DEBOUNCE_MS);
 });
 afterEach(() => {
     while (disposers.length > 0)
@@ -57,6 +65,14 @@ after(() => {
         delete process.env['CRTR_NODE_ID'];
     else
         process.env['CRTR_NODE_ID'] = origNode;
+    if (origTick === undefined)
+        delete process.env['CRTR_WATCHER_TICK_MS'];
+    else
+        process.env['CRTR_WATCHER_TICK_MS'] = origTick;
+    if (origDebounce === undefined)
+        delete process.env['CRTR_WATCHER_DEBOUNCE_MS'];
+    else
+        process.env['CRTR_WATCHER_DEBOUNCE_MS'] = origDebounce;
     for (const h of homes) {
         try {
             rmSync(h, { recursive: true, force: true });

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

@@ -13,11 +13,13 @@ import { tmpdir } from 'node:os';
 import { join } from 'node:path';
 import registerCanvasInboxWatcher from '../../pi-extensions/canvas-inbox-watcher.js';
 import { appendInbox } from '../feed/inbox.js';
-// Mirror the watcher's internal cadence (TICK_MS=800, DEBOUNCE_MS=1000): allow a
-// resolve+seed tick, a read tick, and the debounce window before asserting.
-const TICK_MS = 800;
-const DEBOUNCE_MS = 1000;
-const SETTLE_MS = TICK_MS * 2 + DEBOUNCE_MS + 500;
+// Drive the watcher's injectable cadence seam (CRTR_WATCHER_TICK_MS /
+// CRTR_WATCHER_DEBOUNCE_MS) at a fast tempo so the test sleeps milliseconds, not
+// seconds. SETTLE_MS still allows a resolve+seed tick, a read tick, and the
+// debounce window before asserting, exactly as against the real 800/1000 cadence.
+const TICK_MS = 20;
+const DEBOUNCE_MS = 25;
+const SETTLE_MS = TICK_MS * 2 + DEBOUNCE_MS + 30;
 let origHome;
 let origNode;
 const homes = [];
@@ -40,9 +42,15 @@ function makeFakePi() {
     };
 }
 const wait = (ms) => new Promise((r) => setTimeout(r, ms));
+let origTick;
+let origDebounce;
 before(() => {
     origHome = process.env['CRTR_HOME'];
     origNode = process.env['CRTR_NODE_ID'];
+    origTick = process.env['CRTR_WATCHER_TICK_MS'];
+    origDebounce = process.env['CRTR_WATCHER_DEBOUNCE_MS'];
+    process.env['CRTR_WATCHER_TICK_MS'] = String(TICK_MS);
+    process.env['CRTR_WATCHER_DEBOUNCE_MS'] = String(DEBOUNCE_MS);
 });
 afterEach(() => {
     while (disposers.length > 0)
@@ -57,6 +65,14 @@ after(() => {
         delete process.env['CRTR_NODE_ID'];
     else
         process.env['CRTR_NODE_ID'] = origNode;
+    if (origTick === undefined)
+        delete process.env['CRTR_WATCHER_TICK_MS'];
+    else
+        process.env['CRTR_WATCHER_TICK_MS'] = origTick;
+    if (origDebounce === undefined)
+        delete process.env['CRTR_WATCHER_DEBOUNCE_MS'];
+    else
+        process.env['CRTR_WATCHER_DEBOUNCE_MS'] = origDebounce;
     for (const h of homes) {
         try {
             rmSync(h, { recursive: true, force: true });