claude-code-session-manager 0.98.0 → 0.101.0

package/src/main/__tests__/agentOverlayWrite.test.cjs

@@ -1,95 +0,0 @@
-/**
- * agentOverlayWrite.test.cjs — Agent Library's per-project OVERRIDE write path
- * (savePersona with `projectName`): frontmatter-only overlay, removal on clear,
- * full-body refusal, and end-to-end resolution for a scheduled PRD.
- *
- * Run: timeout 120 npx vitest run src/main/__tests__/agentOverlayWrite.test.cjs
- */
-
-'use strict';
-
-import { test, expect, afterEach } from 'vitest';
-const fs = require('node:fs');
-const fsp = require('node:fs/promises');
-const os = require('node:os');
-const path = require('node:path');
-const { savePersona, listPersonas, getPersonaBody } = require('../agentLibrary.cjs');
-const { resolvePrdPersonaForSpawn } = require('../lib/agentModelResolve.cjs');
-
-const tmpDirs = [];
-afterEach(async () => {
-  while (tmpDirs.length) await fsp.rm(tmpDirs.pop(), { recursive: true, force: true });
-});
-
-function setup() {
-  const cwd = fs.mkdtempSync(path.join(os.tmpdir(), 'sm-ov-cwd-'));
-  const globalDir = fs.mkdtempSync(path.join(os.tmpdir(), 'sm-ov-global-'));
-  tmpDirs.push(cwd, globalDir);
-  const globalFile = path.join(globalDir, 'dev-lead.md');
-  fs.writeFileSync(globalFile, '---\nname: dev-lead\ndescription: d\nmodel: sonnet\n---\nGlobal body.\n');
-  const overlayFile = path.join(cwd, '.claude', 'agents', 'dev-lead.md');
-  const deps = { globalDir, loadSessions: async () => ({ tabs: [{ cwd }] }), validatePath: (p) => p,
-    writeTextAtomic: async (p, t) => { await fsp.mkdir(path.dirname(p), { recursive: true }); await fsp.writeFile(p, t); } };
-  const projectName = path.basename(cwd);
-  const save = (over) => savePersona({ name: 'dev-lead', projectName, description: '', tools: [], color: '', tags: [], body: '', ...over, ...deps });
-  return { cwd, globalFile, overlayFile, deps, save, projectName };
-}
-
-test('setting a model writes a frontmatter-only overlay with only that key; global untouched', async () => {
-  const { overlayFile, globalFile, save } = setup();
-  const before = fs.readFileSync(globalFile, 'utf8');
-  await save({ model: 'claude-opus-4-6', effort: 'inherit' });
-  expect(fs.readFileSync(overlayFile, 'utf8')).toBe('---\nmodel: claude-opus-4-6\n---\n');
-  expect(fs.readFileSync(globalFile, 'utf8')).toBe(before);
-});
-
-test('adding effort adds only that key', async () => {
-  const { overlayFile, save } = setup();
-  await save({ model: 'claude-opus-4-6' });
-  await save({ model: 'claude-opus-4-6', effort: 'high' });
-  expect(fs.readFileSync(overlayFile, 'utf8')).toBe('---\nmodel: claude-opus-4-6\neffort: high\n---\n');
-});
-
-test('clearing both removes the overlay file', async () => {
-  const { overlayFile, save } = setup();
-  await save({ model: 'claude-opus-4-6', effort: 'high' });
-  const r = await save({ model: 'inherit', effort: 'inherit' });
-  expect(r.removed).toBe(true);
-  expect(fs.existsSync(overlayFile)).toBe(false);
-});
-
-test('a full-body overlay is refused, not rewritten', async () => {
-  const { overlayFile, save } = setup();
-  fs.mkdirSync(path.dirname(overlayFile), { recursive: true });
-  const full = '---\nname: dev-lead\nmodel: haiku\n---\nProject body.\n';
-  fs.writeFileSync(overlayFile, full);
-  await expect(save({ model: 'opus' })).rejects.toThrow(/full-body/);
-  expect(fs.readFileSync(overlayFile, 'utf8')).toBe(full);
-});
-
-test('unknown project is rejected', async () => {
-  const { save } = setup();
-  await expect(save({ projectName: 'nope', model: 'opus' })).rejects.toThrow(/project not open/);
-});
-
-test('listPersonas counts a frontmatter-only overlay and reports global → project values', async () => {
-  const { save, deps, projectName } = setup();
-  await save({ model: 'claude-opus-4-6' });
-  const [p] = await listPersonas(deps);
-  expect(p.model).toBe('sonnet');
-  expect(p.overridingProjects).toEqual([projectName]);
-  expect(p.overrideDetails[0]).toMatchObject({ fields: ['model'], values: { model: 'claude-opus-4-6' }, bodyOverridden: false });
-});
-
-test('scheduled-PRD model resolution returns the overlay model; global persona stays sonnet', async () => {
-  const { cwd, save, deps } = setup();
-  await save({ model: 'claude-opus-4-6' });
-  const r = await resolvePrdPersonaForSpawn({
-    cwd, agentType: 'dev-lead', fallbackModel: 'fallback',
-    deps: { getPersonaBody: (a) => getPersonaBody({ ...a, ...deps }) },
-  });
-  expect(r.model).toBe('claude-opus-4-6');
-  expect(r.systemPrompt).toContain('Global body.');
-  const [g] = await listPersonas(deps);
-  expect(g.model).toBe('sonnet');
-});

package/src/main/__tests__/bilkoHost-deriveSlug.test.cjs

@@ -1,26 +0,0 @@
-// bilkoHost.cjs's deriveSlug — pure kebab-case transform used as the
-// default static-path slug when Stage A prepares a project's bundle.
-const assert = require('node:assert/strict');
-const { deriveSlug } = require('../bilkoHost.cjs');
-
-test('deriveSlug: kebab-cases a plain npm package name', () => {
-  assert.equal(deriveSlug('claude-code-session-manager'), 'claude-code-session-manager');
-});
-
-test('deriveSlug: lowercases and collapses non-alphanumeric runs', () => {
-  assert.equal(deriveSlug('My Cool App!! v2'), 'my-cool-app-v2');
-});
-
-test('deriveSlug: trims leading/trailing separators', () => {
-  assert.equal(deriveSlug('--Hello--'), 'hello');
-});
-
-test('deriveSlug: falls back to "project" for an empty/unusable name', () => {
-  assert.equal(deriveSlug('   '), 'project');
-  assert.equal(deriveSlug('!!!'), 'project');
-});
-
-test('deriveSlug: caps length at 64 characters', () => {
-  const long = 'a'.repeat(100);
-  assert.equal(deriveSlug(long).length, 64);
-});

package/src/main/__tests__/bilkoHost-integration.test.cjs

@@ -1,118 +0,0 @@
-// bilkoHost.cjs — integration test against a real temp project dir: adding
-// and removing documents, and that Prepare Bundle wholesale-rebuilds dist/
-// so a removed document can never linger on disk (the local half of the
-// "delete propagates on next Publish" guarantee — see bilkoHost-integration.md).
-const assert = require('node:assert/strict');
-const fs = require('node:fs');
-const path = require('node:path');
-const os = require('node:os');
-const config = require('../config.cjs');
-const bilkoHost = require('../bilkoHost.cjs');
-
-function makeProject() {
-  const root = fs.mkdtempSync(path.join(os.tmpdir(), 'bilko-host-'));
-  config.addAllowedRoot(root);
-  fs.mkdirSync(path.join(root, 'session-manager-operations', 'project-pages'), { recursive: true });
-  fs.writeFileSync(
-    path.join(root, 'session-manager-operations', 'project-pages', 'home.html'),
-    '<html><title>Demo</title>Home</html>',
-  );
-  fs.writeFileSync(path.join(root, 'feature.html'), '<html><title>Demo</title>Feature</html>');
-  fs.writeFileSync(path.join(root, 'package.json'), JSON.stringify({ name: 'demo-app', version: '1.2.3' }));
-  return root;
-}
-
-test('prepareBundle seeds a root document and writes dist/index.html + manifest.json', async () => {
-  const root = makeProject();
-  const result = await bilkoHost.prepareBundle({ cwd: root, slug: 'demo-app' });
-  assert.equal(fs.existsSync(path.join(result.distPath, 'index.html')), true);
-  assert.equal(fs.readFileSync(path.join(result.distPath, 'index.html'), 'utf8'), '<html><title>Demo</title>Home</html>');
-  assert.equal(result.manifest.slug, 'demo-app');
-  assert.equal(result.manifest.bundle.fileCount, 1);
-  fs.rmSync(root, { recursive: true, force: true });
-});
-
-test('addDocument then prepareBundle writes a sub-path document at the expected location', async () => {
-  const root = makeProject();
-  await bilkoHost.prepareBundle({ cwd: root, slug: 'demo-app' });
-  await bilkoHost.addDocument({
-    cwd: root,
-    subpath: 'special-doc/01',
-    title: 'Feature deep-dive',
-    source: { kind: 'file', path: 'feature.html' },
-  });
-  const result = await bilkoHost.prepareBundle({ cwd: root, slug: 'demo-app' });
-  assert.equal(
-    fs.readFileSync(path.join(result.distPath, 'special-doc', '01', 'index.html'), 'utf8'),
-    '<html><title>Demo</title>Feature</html>',
-  );
-  assert.equal(result.manifest.bundle.fileCount, 2);
-  fs.rmSync(root, { recursive: true, force: true });
-});
-
-test('a non-home project-page-lens document throws the retired-lens error (no output/ fallback)', async () => {
-  const root = makeProject();
-  await bilkoHost.prepareBundle({ cwd: root, slug: 'demo-app' });
-  await bilkoHost.addDocument({
-    cwd: root,
-    subpath: 'special-doc/01',
-    title: 'Feature deep-dive',
-    source: { kind: 'project-page-lens', lens: 'feature' },
-  });
-  await assert.rejects(
-    () => bilkoHost.prepareBundle({ cwd: root, slug: 'demo-app' }),
-    /retired "feature" Project Page lens; only the single home\.html page is generated now/,
-  );
-  fs.rmSync(root, { recursive: true, force: true });
-});
-
-test('removeDocument + prepareBundle wholesale-rebuilds dist/, leaving no trace of the removed file', async () => {
-  const root = makeProject();
-  await bilkoHost.prepareBundle({ cwd: root, slug: 'demo-app' });
-  const added = await bilkoHost.addDocument({
-    cwd: root,
-    subpath: 'special-doc/01',
-    title: 'Feature deep-dive',
-    source: { kind: 'file', path: 'feature.html' },
-  });
-  await bilkoHost.prepareBundle({ cwd: root, slug: 'demo-app' });
-  const docId = added.documents.find((d) => d.subpath === 'special-doc/01').id;
-
-  await bilkoHost.removeDocument({ cwd: root, id: docId });
-  // Not yet rebuilt — the stale file is still on disk (get() should flag this via bundleStale).
-  const distBefore = await bilkoHost.get({ cwd: root });
-  assert.equal(distBefore.bundleStale, true);
-
-  const result = await bilkoHost.prepareBundle({ cwd: root, slug: 'demo-app' });
-  assert.equal(fs.existsSync(path.join(result.distPath, 'special-doc')), false);
-  assert.equal(result.manifest.bundle.fileCount, 1);
-
-  const after = await bilkoHost.get({ cwd: root });
-  assert.equal(after.bundleStale, false);
-  fs.rmSync(root, { recursive: true, force: true });
-});
-
-test('removeDocument refuses to remove the root document', async () => {
-  const root = makeProject();
-  await bilkoHost.prepareBundle({ cwd: root, slug: 'demo-app' });
-  const before = await bilkoHost.get({ cwd: root });
-  const rootId = before.documents.find((d) => d.subpath === '').id;
-  await assert.rejects(() => bilkoHost.removeDocument({ cwd: root, id: rootId }), /cannot remove the root document/);
-  fs.rmSync(root, { recursive: true, force: true });
-});
-
-test('addDocument refuses a duplicate sub-path', async () => {
-  const root = makeProject();
-  await bilkoHost.prepareBundle({ cwd: root, slug: 'demo-app' });
-  await bilkoHost.addDocument({
-    cwd: root,
-    subpath: 'doc',
-    title: 'A',
-    source: { kind: 'file', path: 'feature.html' },
-  });
-  await assert.rejects(
-    () => bilkoHost.addDocument({ cwd: root, subpath: 'doc', title: 'B', source: { kind: 'file', path: 'feature.html' } }),
-    /already uses subpath/,
-  );
-  fs.rmSync(root, { recursive: true, force: true });
-});

package/src/main/__tests__/bilkoHostCore.test.cjs

@@ -1,72 +0,0 @@
-// bilkoHostCore.cjs — pure multi-document bundle logic for the "Host on
-// Bilko.run" tab (subpath validation, document-set invariants, path/URL math).
-const assert = require('node:assert/strict');
-const {
-  validateSubpath,
-  validateDocumentSet,
-  documentDistRelPath,
-  documentUrl,
-} = require('../bilkoHostCore.cjs');
-
-test('validateSubpath: "" (root) is always valid', () => {
-  assert.equal(validateSubpath('').ok, true);
-});
-
-test('validateSubpath: accepts lowercase kebab-case, single or multi-segment', () => {
-  assert.equal(validateSubpath('special-doc').ok, true);
-  assert.equal(validateSubpath('special-doc/01').ok, true);
-  assert.equal(validateSubpath('a/b/c').ok, true);
-});
-
-test('validateSubpath: rejects leading/trailing slash', () => {
-  assert.equal(validateSubpath('/foo').ok, false);
-  assert.equal(validateSubpath('foo/').ok, false);
-});
-
-test('validateSubpath: rejects uppercase, spaces, dot-segments', () => {
-  assert.equal(validateSubpath('Foo').ok, false);
-  assert.equal(validateSubpath('foo bar').ok, false);
-  assert.equal(validateSubpath('..').ok, false);
-  assert.equal(validateSubpath('foo/../bar').ok, false);
-});
-
-test('validateSubpath: rejects excessive depth or length', () => {
-  assert.equal(validateSubpath(Array(10).fill('a').join('/')).ok, false);
-  assert.equal(validateSubpath('a'.repeat(300)).ok, false);
-});
-
-test('validateDocumentSet: requires exactly one root document', () => {
-  assert.equal(validateDocumentSet([]).ok, false);
-  assert.equal(validateDocumentSet([{ id: '1', subpath: 'x' }]).ok, false);
-  assert.equal(validateDocumentSet([{ id: '1', subpath: '' }]).ok, true);
-  assert.equal(
-    validateDocumentSet([{ id: '1', subpath: '' }, { id: '2', subpath: '' }]).ok,
-    false,
-  );
-});
-
-test('validateDocumentSet: rejects duplicate subpaths', () => {
-  const res = validateDocumentSet([
-    { id: '1', subpath: '' },
-    { id: '2', subpath: 'doc' },
-    { id: '3', subpath: 'doc' },
-  ]);
-  assert.equal(res.ok, false);
-  assert.match(res.error, /duplicate subpath/);
-});
-
-test('validateDocumentSet: propagates a bad segment error with the document id', () => {
-  const res = validateDocumentSet([{ id: '1', subpath: '' }, { id: 'bad-doc', subpath: 'Not Ok' }]);
-  assert.equal(res.ok, false);
-  assert.match(res.error, /bad-doc/);
-});
-
-test('documentDistRelPath: root -> index.html, sub-path -> <subpath>/index.html', () => {
-  assert.equal(documentDistRelPath(''), 'index.html');
-  assert.equal(documentDistRelPath('special-doc/01'), 'special-doc/01/index.html');
-});
-
-test('documentUrl: root has no sub-path segment, sub-path is nested under the slug', () => {
-  assert.equal(documentUrl('my-app', ''), 'https://bilko.run/projects/my-app/');
-  assert.equal(documentUrl('my-app', 'special-doc/01'), 'https://bilko.run/projects/my-app/special-doc/01/');
-});

package/src/main/__tests__/broadcastCoalescer.test.cjs

@@ -1,122 +0,0 @@
-/**
- * broadcastCoalescer.test.cjs — unit tests for src/main/lib/broadcastCoalescer.cjs,
- * the trailing-edge debounce used by scheduler.cjs's broadcast().
- *
- * Run: timeout 300 npx vitest run src/main/__tests__/broadcastCoalescer.test.cjs
- */
-
-'use strict';
-
-import { test, expect, vi, beforeEach, afterEach } from 'vitest';
-
-const { createBroadcastCoalescer } = require('../lib/broadcastCoalescer.cjs');
-
-beforeEach(() => {
-  vi.useFakeTimers();
-});
-
-afterEach(() => {
-  vi.useRealTimers();
-});
-
-function makeCoalescer(delayMs = 200) {
-  const sent = [];
-  let counter = 0;
-  const getPayload = vi.fn(() => `state-${counter}`);
-  const send = vi.fn((payload) => sent.push(payload));
-  const coalescer = createBroadcastCoalescer({ delayMs, send, getPayload });
-  return {
-    coalescer,
-    sent,
-    send,
-    getPayload,
-    bump: () => { counter += 1; },
-  };
-}
-
-test('5 rapid schedule() calls within the window produce exactly 1 send', async () => {
-  const { coalescer, sent, bump } = makeCoalescer(200);
-
-  bump(); coalescer.schedule();
-  bump(); coalescer.schedule();
-  bump(); coalescer.schedule();
-  bump(); coalescer.schedule();
-  bump(); coalescer.schedule();
-
-  await vi.advanceTimersByTimeAsync(200);
-
-  expect(sent.length).toBe(1);
-});
-
-test('the trailing send reflects the LATEST state, not the state at the first schedule() call', async () => {
-  const { coalescer, sent, bump } = makeCoalescer(200);
-
-  bump(); // state-1
-  coalescer.schedule();
-  bump(); // state-2
-  coalescer.schedule(); // no-op — timer already armed
-  bump(); // state-3
-  coalescer.schedule(); // no-op
-
-  await vi.advanceTimersByTimeAsync(200);
-
-  expect(sent).toEqual(['state-3']);
-});
-
-test('flush() cancels the pending timer and sends immediately with the current payload', async () => {
-  const { coalescer, sent, bump } = makeCoalescer(200);
-
-  bump();
-  coalescer.schedule();
-
-  // Flush before the 200ms window elapses.
-  await vi.advanceTimersByTimeAsync(50);
-  await coalescer.flush();
-
-  expect(sent).toEqual(['state-1']);
-
-  // The window's original timer must not fire a second, stale send.
-  await vi.advanceTimersByTimeAsync(200);
-  expect(sent).toEqual(['state-1']);
-});
-
-test('a schedule() call after a quiet period still arrives (not dropped)', async () => {
-  const { coalescer, sent, bump } = makeCoalescer(200);
-
-  bump();
-  coalescer.schedule();
-  await vi.advanceTimersByTimeAsync(200);
-  expect(sent).toEqual(['state-1']);
-
-  bump();
-  coalescer.schedule();
-  await vi.advanceTimersByTimeAsync(200);
-  expect(sent).toEqual(['state-1', 'state-2']);
-});
-
-test('flush() with no pending timer still sends (no-arm case)', async () => {
-  const { coalescer, sent, bump } = makeCoalescer(200);
-  bump();
-  await coalescer.flush();
-  expect(sent).toEqual(['state-1']);
-});
-
-test('a getPayload() that throws does not wedge the coalescer — the next schedule() still sends', async () => {
-  const sent = [];
-  const send = vi.fn((payload) => sent.push(payload));
-  const getPayload = vi.fn()
-    .mockRejectedValueOnce(new Error('boom'))
-    .mockResolvedValue('state-recovered');
-  const coalescer = createBroadcastCoalescer({ delayMs: 200, send, getPayload });
-
-  coalescer.schedule();
-  await vi.advanceTimersByTimeAsync(200);
-  expect(sent).toEqual([]);
-
-  coalescer.schedule();
-  await vi.advanceTimersByTimeAsync(200);
-  expect(sent).toEqual(['state-recovered']);
-  expect(getPayload).toHaveBeenCalledTimes(2);
-});
-
-console.log('broadcastCoalescer tests: PASS');

package/src/main/__tests__/chat-cancel-terminal.test.cjs

@@ -1,120 +0,0 @@
-/**
- * chat-cancel-terminal.test.cjs — regression for the "stuck thinking" hang.
- *
- * Bug (2026-07-05): cancelling an active chat run SIGTERM'd the child, but the
- * exit handler's `if (!gotResult && !killed)` guard skipped every broadcast
- * because `killed === true`. No terminal IPC event ever reached the renderer,
- * so the chat store stayed `running:true` forever — disabled textarea, endless
- * "running…" spinner, unresponsive chat UI.
- *
- * Fix: every run emits EXACTLY ONE terminal event (complete/needs-input/error),
- * and the exit path emits a fallback `chat:run:error` when a run settled
- * without one (cancel, crash, silent exit). This test drives the REAL spawn
- * path (via SM_CLAUDE_BIN pointing at a stub process) and asserts a terminal
- * event is broadcast on cancel.
- *
- * Run: timeout 300 npx vitest run src/main/__tests__/chat-cancel-terminal.test.cjs
- */
-
-'use strict';
-
-delete process.env.SM_CHAT_CONCURRENCY;
-
-import { test, afterAll } from 'vitest';
-const assert = require('node:assert/strict');
-const fs = require('node:fs');
-const os = require('node:os');
-const path = require('node:path');
-
-// A stub "claude" that ignores stdin/args and stays alive until killed —
-// mimics a long-running headless run so cancel() has something to SIGTERM.
-// Shebang + chmod makes it directly spawnable as a single executable (spawn
-// does not split a path on spaces, so SM_CLAUDE_BIN must be one binary).
-const stubPath = require('../../../tests/helpers/claudeStub.cjs').writeClaudeStub({
-  body: 'setInterval(() => {}, 1000);',
-});
-process.env.SM_CLAUDE_BIN = stubPath;
-
-const cr = require('../chatRunner.cjs');
-
-// Scratch project dir under os.tmpdir() — never the real repo root (opsErrorLog would pollute it).
-const scratchCwd = fs.mkdtempSync(path.join(os.tmpdir(), 'sm-chat-cwd-'));
-
-const tick = () => new Promise((r) => setImmediate(r));
-
-test('cancelling an ACTIVE run broadcasts a terminal event (unsticks the UI)', async () => {
-  const events = [];
-  // Fake BrowserWindow that captures every broadcast.
-  cr.attachWindow({
-    isDestroyed: () => false,
-    webContents: {
-      isDestroyed: () => false,
-      send: (channel, payload) => events.push({ channel, payload }),
-    },
-  });
-
-  cr.run({ tabId: 'T', sessionId: 'S', prompt: 'hello', cwd: scratchCwd, resume: false });
-
-  // Let the spawn happen and started event fire.
-  for (let i = 0; i < 6; i++) await tick();
-  assert.ok(
-    events.some((e) => e.channel === 'chat:run:started'),
-    'started event should fire once the run launches',
-  );
-
-  cr.cancel('T');
-
-  // Wait for SIGTERM → child exit → fallback terminal broadcast.
-  for (let i = 0; i < 40 && !events.some((e) => isTerminal(e.channel)); i++) {
-    await new Promise((r) => setTimeout(r, 25));
-  }
-
-  const terminal = events.filter((e) => isTerminal(e.channel));
-  assert.equal(terminal.length, 1, 'exactly one terminal event is emitted');
-  assert.equal(terminal[0].channel, 'chat:run:error', 'cancel surfaces as an error turn');
-  assert.match(terminal[0].payload.message, /cancel/i, 'message names the cancellation');
-});
-
-test('cancel() resolves only AFTER the terminal broadcast fires, not immediately on SIGTERM (PRD 718)', async () => {
-  // Order-of-events assertion (not timing-based — a real SIGTERM can be fast
-  // enough on a lightweight stub child that a fixed-delay race would flake).
-  const order = [];
-  cr.attachWindow({
-    isDestroyed: () => false,
-    webContents: {
-      isDestroyed: () => false,
-      send: (channel, payload) => {
-        if (isTerminal(channel) && payload.tabId === 'T2') order.push('terminal');
-      },
-    },
-  });
-
-  cr.run({ tabId: 'T2', sessionId: 'S2', prompt: 'hello', cwd: scratchCwd, resume: false });
-
-  for (let i = 0; i < 6; i++) await tick();
-
-  await cr.cancel('T2').then(() => order.push('resolved'));
-
-  assert.deepEqual(order, ['terminal', 'resolved'], 'cancel() resolves strictly after the terminal broadcast');
-});
-
-test('cancel() on a tab with no in-flight or waiting run resolves immediately (no-op)', async () => {
-  cr.attachWindow({
-    isDestroyed: () => false,
-    webContents: { isDestroyed: () => false, send: () => {} },
-  });
-  await cr.cancel('no-such-tab'); // must resolve, not hang
-});
-
-function isTerminal(channel) {
-  return (
-    channel === 'chat:run:complete' ||
-    channel === 'chat:run:needs-input' ||
-    channel === 'chat:run:error'
-  );
-}
-
-afterAll(() => {
-  try { fs.unlinkSync(stubPath); } catch { /* already gone */ }
-  delete process.env.SM_CLAUDE_BIN;
-});

package/src/main/__tests__/chat-dead-channels.test.cjs

@@ -1,63 +0,0 @@
-/**
- * chat-dead-channels.test.cjs — regression guard for the removed
- * chat:probe-context / chat:context-usage channels (PRD 897).
- *
- * These IPC channels were deliberately deleted (handler, broadcast, preload
- * and zod schema all removed) rather than left half-wired. This test walks
- * the source tree and fails if either literal string reappears anywhere
- * under src/main, src/preload, or src/renderer — such a reintroduction must
- * be wired end-to-end (handler + preload + zod schema + broadcast) or not
- * added at all.
- *
- * Run: timeout 120 npx vitest run src/main/__tests__/chat-dead-channels.test.cjs
- */
-
-'use strict';
-
-const fs = require('fs');
-const path = require('path');
-
-import { test, expect } from 'vitest';
-
-const REPO_ROOT = path.resolve(__dirname, '..', '..', '..');
-const SCAN_DIRS = ['src/main', 'src/preload', 'src/renderer'];
-const FORBIDDEN_STRINGS = ['chat:probe-context', 'chat:context-usage'];
-const SELF_PATH = path.resolve(__filename);
-
-function walk(dir, files) {
-  for (const entry of fs.readdirSync(dir, { withFileTypes: true })) {
-    if (entry.name === 'node_modules') continue;
-    if (entry.isSymbolicLink()) continue;
-    const full = path.join(dir, entry.name);
-    if (entry.isDirectory()) {
-      walk(full, files);
-    } else {
-      files.push(full);
-    }
-  }
-}
-
-test('chat:probe-context / chat:context-usage channels stay removed', () => {
-  const files = [];
-  for (const dir of SCAN_DIRS) {
-    walk(path.join(REPO_ROOT, dir), files);
-  }
-
-  const hits = [];
-  for (const file of files) {
-    if (path.resolve(file) === SELF_PATH) continue;
-    const contents = fs.readFileSync(file, 'utf8');
-    for (const needle of FORBIDDEN_STRINGS) {
-      if (contents.includes(needle)) {
-        hits.push(`${path.relative(REPO_ROOT, file)}: contains "${needle}"`);
-      }
-    }
-  }
-
-  expect(
-    hits,
-    'chat:probe-context / chat:context-usage were deliberately removed as a dead/half-wired ' +
-      'channel (PRD 897). If reintroducing this feature, wire it end-to-end (handler, preload, ' +
-      'zod schema, broadcast) in the same change — do not add just one side.\n' + hits.join('\n'),
-  ).toEqual([]);
-});