npm - @crouton-kit/crouter - Versions diffs - 0.3.33 → 0.3.34 - Mend

@crouton-kit/crouter 0.3.33 → 0.3.34

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (566) hide show

package/dist/web-client/index.html CHANGED Viewed

@@ -5,7 +5,7 @@
     <meta name="viewport" content="width=device-width, initial-scale=1" />
     <meta name="color-scheme" content="dark light" />
     <title>crouter</title>
-    <script type="module" crossorigin src="/assets/index-D36PNBj4.js"></script>
+    <script type="module" crossorigin src="/assets/index-BRKxe-hy.js"></script>
     <link rel="stylesheet" crossorigin href="/assets/index-BZUxTkv5.css">
   </head>
   <body>

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "@crouton-kit/crouter",
-  "version": "0.3.33",
+  "version": "0.3.34",
   "description": "crtr — agent runtime with memory, plugins, and marketplaces",
   "type": "module",
   "main": "dist/index.js",
@@ -25,13 +25,17 @@
   },
   "files": [
     "dist",
-    "bin"
+    "bin",
+    "scripts/postinstall.mjs"
   ],
   "scripts": {
-    "build": "rm -rf dist && tsc && cp -R src/builtin-personas dist/builtin-personas && cp -R src/builtin-views dist/builtin-views && cp -R src/builtin-memory dist/builtin-memory && npm run build:attach && vite build --config src/clients/web/web-client/vite.config.ts",
+    "prebuild": "rm -rf node_modules/@earendil-works/pi-coding-agent/node_modules/@earendil-works/pi-tui node_modules/@earendil-works/pi-coding-agent/node_modules/@earendil-works/pi-ai node_modules/@earendil-works/pi-coding-agent/node_modules/@earendil-works/pi-agent-core",
+    "build": "rm -rf dist && tsc && cp -R src/builtin-personas dist/builtin-personas && cp -R src/builtin-views dist/builtin-views && cp -R src/builtin-memory dist/builtin-memory && cp -R src/builtin-pi-packages dist/builtin-pi-packages && npm run build:attach && vite build --config src/clients/web/web-client/vite.config.ts",
     "lint:web-px": "node scripts/lint-web-px.mjs",
     "build:attach": "esbuild src/clients/attach/attach-cmd.ts --bundle --minify --format=esm --platform=node --target=node22 --alias:@earendil-works/pi-tui=$PWD/node_modules/@earendil-works/pi-tui --outfile=dist/clients/attach/attach-cmd.js --log-level=warning --banner:js=\"import{createRequire as __cr}from'node:module';const require=__cr(import.meta.url);\"",
+    "postinstall": "node scripts/postinstall.mjs",
     "hearth:wake-proxy": "node dist/hearth/wake-proxy/main.js",
+    "hearth:control-plane": "node dist/hearth/control-plane/main.js",
     "dev": "tsx src/cli.ts",
     "link": "npm link",
     "prepublishOnly": "npm run build",
@@ -45,11 +49,11 @@
   "license": "MIT",
   "dependencies": {
     "@blaxel/core": "^0.2.94",
-    "@crouton-kit/humanloop": "^0.3.19",
-    "@earendil-works/pi-agent-core": "0.79.9",
-    "@earendil-works/pi-ai": "0.79.9",
-    "@earendil-works/pi-coding-agent": "0.79.9",
-    "@earendil-works/pi-tui": "0.79.9",
+    "@crouton-kit/humanloop": "latest",
+    "@earendil-works/pi-agent-core": "0.80.2",
+    "@earendil-works/pi-ai": "0.80.2",
+    "@earendil-works/pi-coding-agent": "0.80.2",
+    "@earendil-works/pi-tui": "0.80.2",
     "@fontsource-variable/fraunces": "^5.2.9",
     "@fontsource-variable/instrument-sans": "^5.2.8",
     "@fontsource-variable/martian-mono": "^5.2.7",

package/scripts/postinstall.mjs ADDED Viewed

@@ -0,0 +1,8 @@
+#!/usr/bin/env node
+try {
+  if (process.env.CI) process.exit(0);
+  console.log('crouter installed. Run `crtr sys setup` to install companion pi plugins (mode-switch, crtr-extensions, Claude OAuth), tmux/nvim, and your Exa key.');
+} catch {
+  // never fail install
+}
+process.exit(0);

package/dist/clients/attach/__tests__/action-parity.test.js DELETED Viewed

@@ -1,48 +0,0 @@
-// Regression: the `crtr attach` viewer's InputController wired only a subset of
-// pi's editor `app.*` actions, so several keys that work in pi were dead in the
-// attach surface — most visibly Ctrl+O (toggle tool output) and Ctrl+T (toggle
-// thinking blocks), but also Ctrl+G (external editor), Shift+Tab (cycle thinking
-// level), Shift+Ctrl+P (cycle model backward), and Alt+Up (dequeue). The broker
-// already supported every one of these frames; only the client-side onAction
-// wiring was missing. The fix wires them in input-controller's wire() and adds
-// the ChatView render toggles for the two display-only ones. This assembly lives
-// inside TUI-constructing code with no unit seam, so we guard the wiring at the
-// source level (mirrors onrequest-wired.test.ts).
-// See src/clients/attach/{input-controller,chat-view,attach-cmd}.ts.
-import { test } from 'node:test';
-import assert from 'node:assert/strict';
-import { readFileSync } from 'node:fs';
-import { fileURLToPath } from 'node:url';
-import { dirname, join } from 'node:path';
-const here = dirname(fileURLToPath(import.meta.url));
-const read = (file) => readFileSync(join(here, '..', file), 'utf8').replace(/\s+/g, ' ');
-const inputController = read('input-controller.ts');
-const chatView = read('chat-view.ts');
-const attachCmd = read('attach-cmd.ts');
-test('input-controller wires every previously-dead app.* editor action', () => {
-    for (const action of [
-        'app.tools.expand', // Ctrl+O — toggle tool output
-        'app.thinking.toggle', // Ctrl+T — toggle thinking blocks
-        'app.thinking.cycle', // Shift+Tab — cycle thinking level
-        'app.model.cycleBackward', // Shift+Ctrl+P — cycle model backward
-        'app.message.dequeue', // Alt+Up — restore queued messages
-        'app.editor.external', // Ctrl+G — external editor
-    ]) {
-        assert.match(inputController, new RegExp(`onAction\\(\\s*'${action.replace(/\./g, '\\.')}'`), `input-controller must register onAction('${action}')`);
-    }
-});
-test('cycle_model carries an explicit direction in both directions', () => {
-    assert.match(inputController, /type:\s*'cycle_model',\s*direction:\s*'forward'/);
-    assert.match(inputController, /type:\s*'cycle_model',\s*direction:\s*'backward'/);
-});
-test('ChatView exposes the Ctrl+O / Ctrl+T render toggles', () => {
-    assert.match(chatView, /toggleToolsExpanded\(\)/, 'ChatView needs toggleToolsExpanded()');
-    assert.match(chatView, /toggleThinking\(\)/, 'ChatView needs toggleThinking()');
-    // The toggles must actually fan out to the rendered children, not just flip a flag.
-    assert.match(chatView, /child\.setExpanded\(this\.toolOutputExpanded\)/);
-    assert.match(chatView, /child\.setHideThinkingBlock\(this\.hideThinking\)/);
-});
-test('attach-cmd connects the ChatView toggles to the input controller', () => {
-    assert.match(attachCmd, /onToggleToolsExpand:\s*\(\)\s*=>[\s\S]*chatView\.toggleToolsExpanded\(\)/);
-    assert.match(attachCmd, /onToggleThinking:\s*\(\)\s*=>[\s\S]*chatView\.toggleThinking\(\)/);
-});

package/dist/clients/attach/__tests__/autocomplete-and-bash-mode.test.js DELETED Viewed

@@ -1,96 +0,0 @@
-import { test } from 'node:test';
-import assert from 'node:assert/strict';
-import { chmodSync, mkdtempSync, mkdirSync, writeFileSync } from 'node:fs';
-import { tmpdir } from 'node:os';
-import { join } from 'node:path';
-import { CombinedAutocompleteProvider } from '@earendil-works/pi-tui';
-import { buildAttachAutocompleteProvider, deriveBashMode, resolveEditorFrameStyles } from '../attach-cmd.js';
-function writeFakeFd(binDir) {
-    const fdBin = join(binDir, 'fd');
-    writeFileSync(fdBin, `#!/usr/bin/env node
-const fs = require('node:fs');
-const path = require('node:path');
-if (process.argv.includes('--version')) {
-  process.stdout.write('fd 0.0.0\\n');
-  process.exit(0);
-}
-const args = process.argv.slice(2);
-const baseDirIndex = args.indexOf('--base-directory');
-const baseDir = baseDirIndex >= 0 ? args[baseDirIndex + 1] : process.cwd();
-const maxResultsIndex = args.indexOf('--max-results');
-const maxResults = maxResultsIndex >= 0 ? Number(args[maxResultsIndex + 1]) : Infinity;
-const fullPath = args.includes('--full-path');
-let query = '';
-for (let i = 0; i < args.length; i += 1) {
-  const arg = args[i];
-  if (arg === '--base-directory' || arg === '--max-results' || arg === '--type' || arg === '--exclude') {
-    i += 1;
-    continue;
-  }
-  if (arg === '--follow' || arg === '--hidden' || arg === '--full-path') continue;
-  if (!arg.startsWith('--')) query = arg;
-}
-const results = [];
-function walk(dir, rel = '') {
-  const entries = fs.readdirSync(dir, { withFileTypes: true }).sort((a, b) => a.name.localeCompare(b.name));
-  for (const entry of entries) {
-    if (entry.name === '.git') continue;
-    const nextRel = rel ? rel + '/' + entry.name : entry.name;
-    const full = path.join(dir, entry.name);
-    if (!query || nextRel.includes(query) || entry.name.includes(query)) {
-      results.push((fullPath ? nextRel : nextRel) + (entry.isDirectory() ? '/' : ''));
-      if (results.length >= maxResults) return true;
-    }
-    if (entry.isDirectory() && walk(full, nextRel)) return true;
-  }
-  return false;
-}
-walk(baseDir);
-process.stdout.write(results.join('\\n'));
-`, 'utf8');
-    chmodSync(fdBin, 0o755);
-    return fdBin;
-}
-test('#6 attach threads the resolved fd path into @-mention file completion', async () => {
-    const base = mkdtempSync(join(tmpdir(), 'crtr-at-'));
-    const binDir = mkdtempSync(join(tmpdir(), 'crtr-fd-'));
-    mkdirSync(join(base, 'nested'));
-    writeFileSync(join(base, 'alpha.txt'), 'a');
-    writeFileSync(join(base, 'nested', 'beta.md'), 'b');
-    const fdPath = writeFakeFd(binDir);
-    let seen;
-    const provider = buildAttachAutocompleteProvider(base, fdPath, [], (items, cwd, passedFdPath) => {
-        seen = { cwd, fdPath: passedFdPath, items };
-        return new CombinedAutocompleteProvider(items, cwd, passedFdPath);
-    });
-    assert.equal(seen?.cwd, base);
-    assert.equal(seen?.fdPath, fdPath);
-    assert.ok((seen?.items.length ?? 0) > 0);
-    const result = await provider.getSuggestions(['@'], 0, 1, {
-        force: false,
-        signal: new AbortController().signal,
-    });
-    assert.ok(result, 'expected @ suggestions from the file picker');
-    assert.equal(result?.prefix, '@');
-    assert.ok(result?.items.some((item) => item.label === 'alpha.txt' || item.label === 'nested/'));
-});
-test('attach-cmd derives bash mode from the leading cue and switches frame colors', () => {
-    assert.equal(deriveBashMode('plain text'), 0);
-    assert.equal(deriveBashMode('  ! echo hi'), 1);
-    assert.equal(deriveBashMode('\t!! echo hi'), 2);
-    const pal = {
-        bashMode: (s) => `green:${s}`,
-        bashModeAlt: (s) => `bright:${s}`,
-        border: (s) => `border:${s}`,
-    };
-    const off = resolveEditorFrameStyles(0, undefined, pal);
-    assert.equal(off.titleStyle('x'), '\x1b[7m x \x1b[27m');
-    assert.notEqual(off.borderColor, pal.bashMode);
-    assert.notEqual(off.borderColor, pal.bashModeAlt);
-    const single = resolveEditorFrameStyles(1, undefined, pal);
-    assert.equal(single.borderColor, pal.bashMode);
-    assert.equal(single.titleStyle('x'), '\x1b[42m\x1b[97m\x1b[1m x \x1b[0m');
-    const double = resolveEditorFrameStyles(2, undefined, pal);
-    assert.equal(double.borderColor, pal.bashModeAlt);
-    assert.equal(double.titleStyle('x'), '\x1b[102m\x1b[30m\x1b[1m x \x1b[0m');
-});

package/dist/clients/attach/__tests__/bash-bang-routing.test.js DELETED Viewed

@@ -1,58 +0,0 @@
-// Regression: typing `!cmd` in the attach viewer was sent to the engine as a
-// normal PROMPT (the LLM saw the literal `!cmd` and took a turn) instead of
-// running bash and injecting the output into context with no agent turn — pi's
-// native `!`/`!!` semantics. The fix routes a leading `!` in InputController to a
-// `bash` drive frame (and `!!` to one with excludeFromContext). These tests lock
-// the routing decision (the user-visible crux) so it can't silently revert.
-import { test } from 'node:test';
-import assert from 'node:assert/strict';
-import { mkdtempSync } from 'node:fs';
-import { tmpdir } from 'node:os';
-import { join } from 'node:path';
-import { TUI, ProcessTerminal } from '@earendil-works/pi-tui';
-import { TitledEditor } from '../titled-editor.js';
-import { InputController } from '../input-controller.js';
-import { createKeybindingsManager } from '../config-load.js';
-const ENTER = '\r';
-function build() {
-    const agentDir = mkdtempSync(join(tmpdir(), 'crtr-bang-'));
-    const km = createKeybindingsManager(agentDir);
-    const tui = new TUI(new ProcessTerminal());
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    const editor = new TitledEditor(tui, { borderColor: (s) => s, selectList: {} }, km, {
-        paddingX: 1,
-    });
-    const frames = [];
-    const hooks = {
-        onCommand: (f) => frames.push(f),
-        onDialogResponse: () => { },
-    };
-    // eslint-disable-next-line no-new -- wiring the editor.onSubmit handler is the point
-    new InputController(tui, editor, km, hooks);
-    return { editor, frames };
-}
-test('`!cmd` routes to a bash frame, not a prompt', () => {
-    const { editor, frames } = build();
-    editor.setText('!ls -la');
-    editor.handleInput(ENTER);
-    assert.deepEqual(frames, [{ type: 'bash', command: 'ls -la', excludeFromContext: false }]);
-    assert.equal(editor.getText(), '', 'editor clears after a bash submit');
-});
-test('`!!cmd` routes to a bash frame with excludeFromContext', () => {
-    const { editor, frames } = build();
-    editor.setText('!!echo secret');
-    editor.handleInput(ENTER);
-    assert.deepEqual(frames, [{ type: 'bash', command: 'echo secret', excludeFromContext: true }]);
-});
-test('a bare `!` with no command is ignored (no frame, text kept)', () => {
-    const { editor, frames } = build();
-    editor.setText('!   ');
-    editor.handleInput(ENTER);
-    assert.equal(frames.length, 0, 'a bare `!` must not emit a bash frame');
-});
-test('ordinary text still routes to a prompt frame (no regression)', () => {
-    const { editor, frames } = build();
-    editor.setText('hello world');
-    editor.handleInput(ENTER);
-    assert.deepEqual(frames, [{ type: 'prompt', text: 'hello world' }]);
-});

package/dist/clients/attach/__tests__/bundle-pi-tui-dedup.test.js DELETED Viewed

@@ -1,56 +0,0 @@
-// Regression: after the attach viewer was re-bundled (commit d72c0de), Alt+Enter
-// silently stopped inserting a newline. Root cause was NOT in the editor logic —
-// editor-newline.test.ts still passed because it runs UNBUNDLED (tsx), where
-// config-load and pi-coding-agent's CustomEditor each load their own pi-tui from
-// disk and `mirrorKeybindingsToEditor` reaches the editor's disk copy.
-//
-// In the BUNDLE, esbuild inlined TWO distinct pi-tui module instances — the
-// top-level `node_modules/@earendil-works/pi-tui` (imported by config-load) and
-// pi-coding-agent's nested `.../pi-coding-agent/node_modules/@earendil-works/pi-tui`
-// (imported by CustomEditor). `setKeybindings` ran against the first global; the
-// editor read `getKeybindings` from the second. The Alt+Enter newLine binding was
-// invisible to the editor and the key fell through — swallowed. The runtime disk
-// mirror can't help: it targets a third (on-disk) instance, not either bundled one.
-//
-// Fix: the build:attach script aliases @earendil-works/pi-tui to one absolute
-// path so esbuild collapses both imports to a SINGLE bundled instance, and
-// setKeybindings reaches the editor directly. This test bundles attach-cmd.ts
-// the way the build script does and asserts exactly one pi-tui copy is inlined.
-import { test } from 'node:test';
-import assert from 'node:assert/strict';
-import { readFileSync } from 'node:fs';
-import { fileURLToPath } from 'node:url';
-import { dirname, join } from 'node:path';
-import { build } from 'esbuild';
-const here = dirname(fileURLToPath(import.meta.url));
-const repoRoot = join(here, '..', '..', '..', '..'); // src/clients/attach/__tests__ → package root
-const entry = join(repoRoot, 'src/clients/attach/attach-cmd.ts');
-const piTui = join(repoRoot, 'node_modules/@earendil-works/pi-tui');
-/** Count distinct pi-tui package directories esbuild inlines into the bundle.
- *  Unminified output keeps each module's source path as a `// <path>` comment. */
-function piTuiCopies(code) {
-    const copies = new Set();
-    for (const m of code.matchAll(/node_modules\/(?:\.pnpm\/)?@earendil-works\/(?:pi-coding-agent\/node_modules\/@earendil-works\/)?pi-tui/g)) {
-        copies.add(m[0]);
-    }
-    return copies;
-}
-test('build:attach aliases pi-tui to one path (sanity: package.json carries the alias)', () => {
-    const pkg = JSON.parse(readFileSync(join(repoRoot, 'package.json'), 'utf8'));
-    assert.match(pkg.scripts['build:attach'], /--alias:@earendil-works\/pi-tui=/, 'build:attach must alias pi-tui so the bundle inlines a single instance');
-});
-test('attach bundle inlines exactly ONE pi-tui instance (editor + config-load share the keybindings global)', async () => {
-    const result = await build({
-        entryPoints: [entry],
-        bundle: true,
-        write: false,
-        format: 'esm',
-        platform: 'node',
-        target: 'node22',
-        alias: { '@earendil-works/pi-tui': piTui },
-        logLevel: 'silent',
-    });
-    const code = result.outputFiles[0].text;
-    const copies = piTuiCopies(code);
-    assert.equal(copies.size, 1, `bundle must inline exactly one pi-tui (two = the Alt+Enter regression). Found: ${[...copies].join(', ')}`);
-});

package/dist/clients/attach/__tests__/clipboard-image.test.js DELETED Viewed

@@ -1,76 +0,0 @@
-// Bug-regression: `crtr attach` Ctrl+V reported "no image in the clipboard" even
-// when the clipboard genuinely held an image. Root cause — the macOS reader only
-// tried `pngpaste -`, which is NOT a built-in (`brew install pngpaste`); on a
-// stock Mac its absence returned null, indistinguishable from an empty clipboard.
-//
-// These lock in the platform dispatch (clipboard-image.ts): a MISSING pngpaste
-// must FALL THROUGH to the native osascript read (not collapse to "no image"),
-// and the reader must DISTINGUISH "no clipboard tool" from "clipboard has no
-// image" so the user-facing notice is accurate. The decision logic is pure +
-// injectable precisely so it can be unit-tested without a real clipboard.
-import { test } from 'node:test';
-import assert from 'node:assert/strict';
-import { selectMacOutcome, selectLinuxOutcome, } from '../clipboard-image.js';
-const bytes = (s) => Buffer.from(s);
-test('macOS: missing pngpaste falls through to the native osascript read (the bug)', () => {
-    let nativeCalled = false;
-    const runTool = (cmd) => {
-        assert.equal(cmd, 'pngpaste');
-        return { missing: true }; // pngpaste not installed (stock Mac)
-    };
-    const readNative = () => {
-        nativeCalled = true;
-        return { image: { bytes: bytes('PNGDATA'), mimeType: 'image/png' } };
-    };
-    const out = selectMacOutcome(runTool, readNative);
-    assert.equal(nativeCalled, true, 'native osascript path must run when pngpaste is absent');
-    assert.equal(out.image?.mimeType, 'image/png');
-    assert.equal(out.image?.bytes.toString(), 'PNGDATA');
-});
-test('macOS: pngpaste present takes the fast path and skips osascript', () => {
-    let nativeCalled = false;
-    const runTool = () => ({ bytes: bytes('FASTPNG') });
-    const readNative = () => {
-        nativeCalled = true;
-        return {};
-    };
-    const out = selectMacOutcome(runTool, readNative);
-    assert.equal(nativeCalled, false, 'fast path must not invoke osascript');
-    assert.equal(out.image?.bytes.toString(), 'FASTPNG');
-});
-test('macOS: pngpaste present but clipboard empty still defers to osascript', () => {
-    // pngpaste exits non-zero on an empty clipboard → {} (ran, no image). The
-    // native path is authoritative for the empty-vs-failure distinction.
-    let nativeCalled = false;
-    const runTool = () => ({}); // ran, no bytes
-    const readNative = () => {
-        nativeCalled = true;
-        return {}; // osascript: NO_IMAGE
-    };
-    const out = selectMacOutcome(runTool, readNative);
-    assert.equal(nativeCalled, true);
-    assert.equal(out.image, undefined);
-    assert.equal(out.note, undefined, 'empty clipboard → no note (caller shows the generic notice)');
-});
-test('Linux: no clipboard tool installed surfaces a precise note, not "no image"', () => {
-    const runTool = () => ({ missing: true }); // neither xclip nor wl-paste
-    const out = selectLinuxOutcome(runTool, /* wayland */ false);
-    assert.equal(out.image, undefined);
-    assert.match(out.note ?? '', /install xclip/);
-});
-test('Linux/Wayland: no tool installed names wl-clipboard', () => {
-    const out = selectLinuxOutcome(() => ({ missing: true }), /* wayland */ true);
-    assert.match(out.note ?? '', /wl-clipboard/);
-});
-test('Linux: tool present but clipboard has no image → empty (no note)', () => {
-    const runTool = () => ({}); // xclip ran, no image of that type
-    const out = selectLinuxOutcome(runTool, /* wayland */ false);
-    assert.equal(out.image, undefined);
-    assert.equal(out.note, undefined, 'a tool that ran but found no image is genuinely empty');
-});
-test('Linux: xclip returns PNG bytes', () => {
-    const runTool = (_cmd, args) => args.includes('image/png') ? { bytes: bytes('XPNG') } : {};
-    const out = selectLinuxOutcome(runTool, /* wayland */ false);
-    assert.equal(out.image?.mimeType, 'image/png');
-    assert.equal(out.image?.bytes.toString(), 'XPNG');
-});

package/dist/clients/attach/__tests__/editor-newline.test.js DELETED Viewed

@@ -1,82 +0,0 @@
-// Regression (#11): in the attach viewer, Alt+Enter did NOTHING on an INSTALLED
-// (non-deduped) crtr — not a newline, not a submit. Root cause: pi-coding-agent's
-// CustomEditor resolves `@earendil-works/pi-tui` from its OWN node_modules, a
-// SEPARATE module instance from the one `config-load` imports. The base Editor
-// reads `getKeybindings()` from THAT instance — stuck at the default
-// `tui.input.newLine = shift+enter` unless the `mirrorKeybindingsToEditor` shim
-// (best-effort, `import.meta.resolve`-based) succeeds in registering crtr's
-// manager on it. On install layouts where that mirror silently fails, the
-// `alt+enter → newLine` override never reached the editor and Alt+Enter
-// (`\x1b[13;3u`) matched neither newLine nor submit and fell through.
-//
-// The fix makes the chord self-contained: `TitledEditor.handleInput` matches
-// newLine against crtr's OWN KeybindingsManager (the one CustomEditor already
-// uses for `app.*`) and inserts the newline directly, so it no longer depends on
-// the cross-instance mirror. The third test below builds the editor WITHOUT the
-// mirror and is the real guard for the installed-layout failure.
-import { test } from 'node:test';
-import assert from 'node:assert/strict';
-import { mkdtempSync } from 'node:fs';
-import { tmpdir } from 'node:os';
-import { join } from 'node:path';
-import { TUI, ProcessTerminal } from '@earendil-works/pi-tui';
-import { TitledEditor } from '../titled-editor.js';
-import { createKeybindingsManager, mirrorKeybindingsToEditor } from '../config-load.js';
-// Kitty CSI-u: `\x1b[<codepoint>;<1+mods>u`. enter=13, alt=2 → mods byte 3.
-const ALT_ENTER = '\x1b[13;3u';
-const ENTER = '\r';
-async function buildEditor() {
-    // Empty agent dir → no user keybindings.json, so the binding under test is the
-    // attach DEFAULT (`ATTACH_KEYBINDING_OVERRIDES` adds alt+enter to newLine).
-    const agentDir = mkdtempSync(join(tmpdir(), 'crtr-kb-'));
-    const km = createKeybindingsManager(agentDir);
-    await mirrorKeybindingsToEditor(km); // mirror still runs for general tui.* parity
-    const tui = new TUI(new ProcessTerminal());
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    return new TitledEditor(tui, { borderColor: (s) => s, selectList: {} }, km, {
-        paddingX: 1,
-    });
-}
-test('attach editor inserts a newline on Alt+Enter (cross-instance binding reaches the editor)', async () => {
-    const editor = await buildEditor();
-    let submitted = null;
-    editor.onSubmit = (t) => {
-        submitted = t;
-    };
-    editor.setText('hello');
-    editor.handleInput(ALT_ENTER);
-    assert.equal(editor.getText(), 'hello\n', 'Alt+Enter must insert a newline, not be swallowed');
-    assert.equal(submitted, null, 'Alt+Enter must not submit');
-});
-test('attach editor still submits on plain Enter', async () => {
-    const editor = await buildEditor();
-    let submitted = null;
-    editor.onSubmit = (t) => {
-        submitted = t;
-    };
-    editor.setText('hello');
-    editor.handleInput(ENTER);
-    assert.equal(submitted, 'hello', 'plain Enter must submit');
-    assert.equal(editor.getText(), '', 'editor clears after submit');
-});
-test('attach editor inserts a newline on Alt+Enter even WITHOUT the cross-instance mirror (#11)', async () => {
-    // Reproduces the installed-layout failure: the editor's pi-tui instance never
-    // received crtr's alt+enter override (mirror not run). TitledEditor.handleInput
-    // must still newline by matching crtr's own km directly.
-    const agentDir = mkdtempSync(join(tmpdir(), 'crtr-kb-'));
-    const km = createKeybindingsManager(agentDir);
-    // NOTE: deliberately NOT calling mirrorKeybindingsToEditor.
-    const tui = new TUI(new ProcessTerminal());
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    const editor = new TitledEditor(tui, { borderColor: (s) => s, selectList: {} }, km, {
-        paddingX: 1,
-    });
-    let submitted = null;
-    editor.onSubmit = (t) => {
-        submitted = t;
-    };
-    editor.setText('hello');
-    editor.handleInput(ALT_ENTER);
-    assert.equal(editor.getText(), 'hello\n', 'Alt+Enter must newline without relying on the mirror');
-    assert.equal(submitted, null, 'Alt+Enter must not submit');
-});

package/dist/clients/attach/__tests__/git-info.test.js DELETED Viewed

@@ -1,31 +0,0 @@
-// Unit coverage for the pure git-status parser that feeds the attach editor's
-// border (cwd · branch · status symbols). The shell-out is non-blocking and
-// untestable in a unit, but the porcelain-v1 --branch parse is pure and the
-// part that can silently mis-read state, so it is locked in here.
-// See src/clients/attach/git-info.ts.
-import { test } from 'node:test';
-import assert from 'node:assert/strict';
-import { parseGitStatus } from '../git-info.js';
-test('clean tree on a tracked branch', () => {
-    const info = parseGitStatus('repo', '## main...origin/main\n');
-    assert.deepEqual(info, { dir: 'repo', branch: 'main', dirty: false, ahead: 0, behind: 0 });
-});
-test('dirty tree with ahead/behind counts', () => {
-    const info = parseGitStatus('repo', '## feature...origin/feature [ahead 2, behind 3]\n M src/a.ts\n?? new.ts\n');
-    assert.equal(info.branch, 'feature');
-    assert.equal(info.dirty, true);
-    assert.equal(info.ahead, 2);
-    assert.equal(info.behind, 3);
-});
-test('untracked-only branch (no upstream) is still dirty, no counts', () => {
-    const info = parseGitStatus('repo', '## main\n?? scratch.md\n');
-    assert.equal(info.branch, 'main');
-    assert.equal(info.dirty, true);
-    assert.equal(info.ahead, 0);
-    assert.equal(info.behind, 0);
-});
-test('detached HEAD reports no branch', () => {
-    const info = parseGitStatus('repo', '## HEAD (no branch)\n');
-    assert.equal(info.branch, undefined);
-    assert.equal(info.dirty, false);
-});

package/dist/clients/attach/__tests__/onrequest-wired.test.js DELETED Viewed

@@ -1,20 +0,0 @@
-// Regression: the `crtr attach` viewer built its InputController WITHOUT the
-// `onRequest` hook, so the correlated read-op channel was never wired even
-// though ViewSocketClient.request() was fully implemented. Every native picker
-// (/model, /resume, /fork, /tree, /settings, /scoped-models) therefore degraded
-// to the "isn't available in this viewer" notice — /model couldn't open the
-// model picker. The fix wires `onRequest: (frame) => socket.request(frame)` in
-// attach-cmd. This assembly lives inside a big TUI-constructing function with no
-// seam to unit-test, so this guards the exact wiring at the source level.
-// See src/clients/attach/{attach-cmd,input-controller,slash-commands}.ts.
-import { test } from 'node:test';
-import assert from 'node:assert/strict';
-import { readFileSync } from 'node:fs';
-import { fileURLToPath } from 'node:url';
-import { dirname, join } from 'node:path';
-const src = readFileSync(join(dirname(fileURLToPath(import.meta.url)), '..', 'attach-cmd.ts'), 'utf8');
-test('attach-cmd wires onRequest to socket.request so native pickers work', () => {
-    // The InputController must receive the read-op channel; without it
-    // openPicker() short-circuits to the "isn't available in this viewer" notice.
-    assert.match(src.replace(/\s+/g, ' '), /onRequest:\s*\(frame\)\s*=>\s*socket\.request\(frame\)/, 'attach-cmd must pass onRequest: (frame) => socket.request(frame) to InputController');
-});

package/dist/clients/attach/__tests__/reconnect-giveup.test.js DELETED Viewed

@@ -1,30 +0,0 @@
-// Regression: the `crtr attach` viewer used to ALWAYS teardown ("broker gone")
-// on a socket close, so a resident broker node's yield→revive cycle dropped the
-// viewer to a shell. The fix holds the pane and re-dials the SAME view.sock
-// while the node is still alive, giving up only when it is genuinely gone. This
-// locks the give-up/keep-reconnecting decision (the bug-prone core) without a
-// socket or tmux. See src/clients/attach/{view-socket,attach-cmd}.ts.
-import { test } from 'node:test';
-import assert from 'node:assert/strict';
-import { reconnectShouldGiveUp } from '../view-socket.js';
-const rowWith = (status) => ({
-    node_id: 'n',
-    status,
-    host_kind: 'broker',
-    intent: null,
-    pi_pid: null,
-});
-test('keeps reconnecting while the node is still alive (active = mid yield/revive)', () => {
-    // A yield leaves status='active' (intent='refresh') — must NOT give up.
-    assert.equal(reconnectShouldGiveUp(rowWith('active')), false);
-    // idle-release revives on the next inbox wake — keep trying (bounded by the
-    // supervisor's own ~30s deadline), do not give up.
-    assert.equal(reconnectShouldGiveUp(rowWith('idle')), false);
-});
-test('gives up the instant the node is genuinely gone', () => {
-    assert.equal(reconnectShouldGiveUp(rowWith('done')), true);
-    assert.equal(reconnectShouldGiveUp(rowWith('dead')), true);
-    assert.equal(reconnectShouldGiveUp(rowWith('canceled')), true);
-    // A reaped row (null) → give up.
-    assert.equal(reconnectShouldGiveUp(null), true);
-});

package/dist/clients/attach/__tests__/slash-quit-copy.test.js DELETED Viewed

@@ -1,42 +0,0 @@
-// Regression: `/quit` in the attach viewer only sent a `bye` frame + a notice.
-// The broker answers `bye` with socket.end(), which the viewer's close handler
-// treats as a dropped connection and RECONNECTS — so `/quit` never detached (it
-// silently reconnected). The fix routes `/quit` through an `onQuit` hook wired to
-// teardown('detach'). Likewise `/copy` was a dead "not available" stub; it now
-// runs through an `onCopy` hook. This guards the dispatch contract for both.
-// See src/clients/attach/{slash-commands,input-controller,attach-cmd}.ts.
-import { test } from 'node:test';
-import assert from 'node:assert/strict';
-import { dispatchSlashCommand } from '../slash-commands.js';
-function ctx(overrides = {}) {
-    const sent = [];
-    const notices = [];
-    return {
-        send: (f) => sent.push(f),
-        notify: (m) => notices.push(m),
-        sent,
-        notices,
-        ...overrides,
-    };
-}
-test('/quit detaches via onQuit and does NOT send a bare bye (reconnect-loop bug)', () => {
-    let quit = 0;
-    const c = ctx({ onQuit: () => quit++ });
-    const handled = dispatchSlashCommand('/quit', c);
-    assert.equal(handled, true);
-    assert.equal(quit, 1, '/quit must invoke onQuit (teardown), not just send bye');
-    assert.equal(c.sent.length, 0, '/quit must not send a bare bye frame when onQuit is wired');
-});
-test('/quit falls back to a bye frame when onQuit is unwired', () => {
-    const c = ctx();
-    dispatchSlashCommand('/quit', c);
-    assert.deepEqual(c.sent, [{ type: 'bye' }]);
-});
-test('/copy runs through onCopy instead of the dead "not available" stub', () => {
-    let copy = 0;
-    const c = ctx({ onCopy: () => copy++ });
-    const handled = dispatchSlashCommand('/copy', c);
-    assert.equal(handled, true);
-    assert.equal(copy, 1, '/copy must invoke onCopy');
-    assert.equal(c.notices.length, 0, '/copy must not fall through to a notice when onCopy is wired');
-});