claude-code-session-manager 0.97.0 → 0.100.0

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

@@ -1,244 +0,0 @@
-/**
- * docEdit.test.cjs — unit tests for docEdit.cjs's pure helpers (PRD 638).
- *
- * Run: timeout 300 npx vitest run src/main/__tests__/docEdit.test.cjs
- */
-
-'use strict';
-
-import { test, expect, describe, beforeEach, vi } from 'vitest';
-
-// docEdit.cjs requires chatRunner.cjs at module scope and calls `chatRunner.run(...)`
-// as a property access, so patching `run` on the already-required (cached) module
-// object — before docEdit.cjs's own require resolves to the same singleton — is
-// enough to intercept it, mirroring the exchanges.cjs patch pattern in chatRunner.spec.ts.
-const chatRunnerModule = require('../chatRunner.cjs');
-const runCalls = [];
-beforeEach(() => { runCalls.length = 0; });
-chatRunnerModule.run = (opts) => { runCalls.push(opts); };
-
-const { parseDocEdit, editPrompt, truncateDocumentText, MAX_DOC_CONTEXT, docEditViaSession, attachWindow } = require('../docEdit.cjs');
-const { duplicateNameFor } = require('../files.cjs');
-
-const broadcasts = [];
-attachWindow({
-  isDestroyed: () => false,
-  webContents: { isDestroyed: () => false, send: (channel, payload) => broadcasts.push({ channel, payload }) },
-});
-beforeEach(() => { broadcasts.length = 0; });
-
-describe('parseDocEdit', () => {
-  test('valid JSON', () => {
-    const result = parseDocEdit('{"after":"rewritten text"}');
-    expect(result).toEqual({ ok: true, after: 'rewritten text' });
-  });
-
-  test('JSON embedded in prose', () => {
-    const result = parseDocEdit('Sure, here you go:\n{"after":"rewritten text"}\nHope that helps!');
-    expect(result).toEqual({ ok: true, after: 'rewritten text' });
-  });
-
-  test('missing after', () => {
-    const result = parseDocEdit('{"other":"value"}');
-    expect(result.ok).toBe(false);
-    expect(result.error).toBeTruthy();
-  });
-
-  test('empty after', () => {
-    const result = parseDocEdit('{"after":""}');
-    expect(result.ok).toBe(false);
-    expect(result.error).toBeTruthy();
-  });
-
-  test('non-string after', () => {
-    const result = parseDocEdit('{"after":42}');
-    expect(result.ok).toBe(false);
-    expect(result.error).toBeTruthy();
-  });
-
-  test('over-length after', () => {
-    const result = parseDocEdit(JSON.stringify({ after: 'x'.repeat(16001) }));
-    expect(result.ok).toBe(false);
-    expect(result.error).toBeTruthy();
-  });
-
-  test('no JSON object in output at all', () => {
-    const result = parseDocEdit('not json at all, just prose garbage');
-    expect(result.ok).toBe(false);
-    expect(result.error).toBeTruthy();
-  });
-
-  test('empty/null input', () => {
-    expect(parseDocEdit('').ok).toBe(false);
-    expect(parseDocEdit(null).ok).toBe(false);
-  });
-});
-
-describe('editPrompt', () => {
-  test('omits the document block when documentText is absent', () => {
-    const prompt = editPrompt('selected text', 'make it concise');
-    expect(prompt).not.toMatch(/<document_/);
-  });
-
-  test('includes a nonce-tagged document block when documentText is provided', () => {
-    const prompt = editPrompt('selected text', 'make it concise', 'the whole document');
-    expect(prompt).toMatch(/<document_[0-9a-f]+>/);
-    expect(prompt).toContain('the whole document');
-  });
-
-  test('truncates an oversized documentText before embedding it', () => {
-    const huge = 'x'.repeat(MAX_DOC_CONTEXT + 5000);
-    const prompt = editPrompt('selected text', 'make it concise', huge);
-    expect(prompt).toContain('[...document truncated for length...]');
-    expect(prompt.length).toBeLessThan(huge.length + 2000);
-  });
-});
-
-describe('truncateDocumentText', () => {
-  test('passes short text through unchanged', () => {
-    expect(truncateDocumentText('short')).toBe('short');
-  });
-
-  test('truncates to the documented head+tail scheme rather than dropping or erroring', () => {
-    const head = 'H'.repeat(40000);
-    const tail = 'T'.repeat(20000);
-    const middle = 'M'.repeat(10000);
-    const result = truncateDocumentText(head + middle + tail);
-    expect(result).toBe(`${head}\n\n[...document truncated for length...]\n\n${tail}`);
-    expect(result).not.toContain('M');
-  });
-});
-
-describe('docEditViaSession', () => {
-  test('builds its prompt via the shared editPrompt and calls chatRunner.run with resume+silent', () => {
-    docEditViaSession({
-      tabId: 'tab-1',
-      sessionId: 'chat-session-1',
-      cwd: '/home/user/project',
-      before: 'old text',
-      instruction: 'make it punchier',
-      documentText: 'the whole document',
-      requestId: 'req-1',
-    });
-
-    expect(runCalls).toHaveLength(1);
-    const call = runCalls[0];
-    expect(call.tabId).toBe('tab-1');
-    expect(call.sessionId).toBe('chat-session-1');
-    expect(call.cwd).toBe('/home/user/project');
-    expect(call.resume).toBe(true);
-    expect(call.silent).toBe(true);
-    expect(typeof call.onSilentResult).toBe('function');
-
-    // The prompt is the shared editPrompt's output (nonce-tagged, so compare
-    // structurally rather than byte-for-byte) prefixed with the anti-injection
-    // system framing — not a second, forked prompt builder.
-    expect(call.prompt).toContain('Rewrite the SELECTION below per the INSTRUCTION');
-    expect(call.prompt).toMatch(/<selection_[0-9a-f]+>\nold text\n<\/selection_[0-9a-f]+>/);
-    expect(call.prompt).toContain('deterministic document-rewrite assistant');
-  });
-
-  test('onSilentResult parses the reply and broadcasts docedit:session-result', () => {
-    docEditViaSession({
-      tabId: 'tab-2',
-      sessionId: 'chat-session-2',
-      cwd: '/home/user/project',
-      before: 'old text',
-      instruction: 'shorten it',
-      requestId: 'req-2',
-    });
-
-    const { onSilentResult } = runCalls[runCalls.length - 1];
-    onSilentResult('{"after":"new text"}');
-    expect(broadcasts).toHaveLength(1);
-    expect(broadcasts[0]).toEqual({
-      channel: 'docedit:session-result',
-      payload: { tabId: 'tab-2', requestId: 'req-2', ok: true, after: 'new text' },
-    });
-  });
-
-  test('onSilentResult reports a parse failure as ok:false', () => {
-    docEditViaSession({
-      tabId: 'tab-3',
-      sessionId: 'chat-session-3',
-      cwd: '/home/user/project',
-      before: 'old text',
-      instruction: 'shorten it',
-      requestId: 'req-3',
-    });
-
-    const { onSilentResult } = runCalls[runCalls.length - 1];
-    onSilentResult('not json at all');
-    expect(broadcasts).toHaveLength(1);
-    expect(broadcasts[0].payload.ok).toBe(false);
-    expect(broadcasts[0].payload.error).toBeTruthy();
-  });
-
-  test('splits off the stop-signal protocol and reports it as a clarification request', () => {
-    docEditViaSession({
-      tabId: 'tab-4',
-      sessionId: 'chat-session-4',
-      cwd: '/home/user/project',
-      before: 'old text',
-      instruction: 'shorten it',
-      requestId: 'req-4',
-    });
-
-    const { onSilentResult } = runCalls[runCalls.length - 1];
-    onSilentResult('<<<SM_NEEDS_INPUT>>>\n{"questions":["which section?"]}');
-    expect(broadcasts).toHaveLength(1);
-    expect(broadcasts[0].payload.ok).toBe(false);
-    expect(broadcasts[0].payload.error).toContain('which section?');
-  });
-
-  test('times out and broadcasts a failure if chatRunner never calls onSilentResult (error/timeout/cancel/collision on the silent lane)', () => {
-    vi.useFakeTimers();
-    try {
-      docEditViaSession({
-        tabId: 'tab-5',
-        sessionId: 'chat-session-5',
-        cwd: '/home/user/project',
-        before: 'old text',
-        instruction: 'shorten it',
-        requestId: 'req-5',
-      });
-
-      // chatRunner.run() is a no-op stub in this test (never invokes
-      // onSilentResult) — nothing should broadcast until the bound elapses.
-      expect(broadcasts).toHaveLength(0);
-
-      vi.advanceTimersByTime(120_000);
-      expect(broadcasts).toHaveLength(1);
-      expect(broadcasts[0]).toEqual({
-        channel: 'docedit:session-result',
-        payload: { tabId: 'tab-5', requestId: 'req-5', ok: false, error: 'timed out waiting on the background session' },
-      });
-    } finally {
-      vi.useRealTimers();
-    }
-  });
-});
-
-describe('duplicateNameFor', () => {
-  test('fresh name — no collision', () => {
-    const result = duplicateNameFor('/tmp/dir', 'notes.txt', () => false);
-    expect(result).toEqual({ ok: true, name: 'notes-copy.txt' });
-  });
-
-  test('collision on the first candidate falls through to -copy-2', () => {
-    const taken = new Set(['/tmp/dir/notes-copy.txt']);
-    const result = duplicateNameFor('/tmp/dir', 'notes.txt', (full) => taken.has(full));
-    expect(result).toEqual({ ok: true, name: 'notes-copy-2.txt' });
-  });
-
-  test('preserves extension-less basenames', () => {
-    const result = duplicateNameFor('/tmp/dir', 'README', () => false);
-    expect(result).toEqual({ ok: true, name: 'README-copy' });
-  });
-
-  test('cap exhausted after 20 attempts returns an error', () => {
-    const result = duplicateNameFor('/tmp/dir', 'notes.txt', () => true);
-    expect(result.ok).toBe(false);
-    expect(result.error).toBeTruthy();
-  });
-});

package/src/main/__tests__/dod-batchkey.test.cjs

@@ -1,183 +0,0 @@
-/**
- * dod-batchkey.test.cjs — unit tests for definitionOfDone.cjs helpers.
- *
- * Run: timeout 300 npx vitest run src/main/__tests__/dod-batchkey.test.cjs
- */
-
-'use strict';
-
-import { test } from 'vitest';
-const assert = require('node:assert/strict');
-const os = require('node:os');
-const fs = require('node:fs');
-const path = require('node:path');
-const { batchKey, reportPathFor, reportExists } = require('../lib/definitionOfDone.cjs');
-
-// ─── helpers ──────────────────────────────────────────────────────────────────
-
-function makeTmpDir() {
-  return fs.mkdtempSync(path.join(os.tmpdir(), 'dod-batchkey-test-'));
-}
-
-function rmdir(dir) {
-  try { fs.rmSync(dir, { recursive: true, force: true }); } catch { /* */ }
-}
-
-function makeJob(slug, runId) {
-  return { slug, runId };
-}
-
-// ─── batchKey: order-independence ─────────────────────────────────────────────
-
-test('batchKey is order-independent', () => {
-  const jobsA = [
-    makeJob('101-foo', '2026-06-01T10-00-00-000Z'),
-    makeJob('102-bar', '2026-06-01T11-00-00-000Z'),
-  ];
-  const jobsB = [
-    makeJob('102-bar', '2026-06-01T11-00-00-000Z'),
-    makeJob('101-foo', '2026-06-01T10-00-00-000Z'),
-  ];
-  assert.strictEqual(batchKey(jobsA), batchKey(jobsB));
-});
-
-// ─── batchKey: adding a real job changes the key ─────────────────────────────
-
-test('adding a real job changes the batchKey', () => {
-  const base = [
-    makeJob('101-foo', '2026-06-01T10-00-00-000Z'),
-  ];
-  const extended = [
-    makeJob('101-foo', '2026-06-01T10-00-00-000Z'),
-    makeJob('102-bar', '2026-06-01T11-00-00-000Z'),
-  ];
-  assert.notStrictEqual(batchKey(base), batchKey(extended));
-});
-
-// ─── batchKey: adding a meta/dod job does NOT change the key ─────────────────
-
-test('adding a dod-prefixed slug does not change batchKey', () => {
-  const base = [makeJob('101-foo', '2026-06-01T10-00-00-000Z')];
-  const withDod = [
-    makeJob('101-foo', '2026-06-01T10-00-00-000Z'),
-    makeJob('dod-gate', '2026-06-01T12-00-00-000Z'),
-  ];
-  assert.strictEqual(batchKey(base), batchKey(withDod));
-});
-
-test('adding a dod-suffixed slug does not change batchKey', () => {
-  const base = [makeJob('101-foo', '2026-06-01T10-00-00-000Z')];
-  const withDod = [
-    makeJob('101-foo', '2026-06-01T10-00-00-000Z'),
-    makeJob('gate-dod', '2026-06-01T12-00-00-000Z'),
-  ];
-  assert.strictEqual(batchKey(base), batchKey(withDod));
-});
-
-test('adding a definition-of-done slug does not change batchKey', () => {
-  const base = [makeJob('101-foo', '2026-06-01T10-00-00-000Z')];
-  const withDod = [
-    makeJob('101-foo', '2026-06-01T10-00-00-000Z'),
-    makeJob('definition-of-done-pass', '2026-06-01T12-00-00-000Z'),
-  ];
-  assert.strictEqual(batchKey(base), batchKey(withDod));
-});
-
-test('adding an inline dod slug (dod between dashes) does not change batchKey', () => {
-  const base = [makeJob('101-foo', '2026-06-01T10-00-00-000Z')];
-  const withDod = [
-    makeJob('101-foo', '2026-06-01T10-00-00-000Z'),
-    makeJob('batch-dod-check', '2026-06-01T12-00-00-000Z'),
-  ];
-  assert.strictEqual(batchKey(base), batchKey(withDod));
-});
-
-// ─── batchKey: empty set ──────────────────────────────────────────────────────
-
-test('empty job list produces a stable key', () => {
-  assert.strictEqual(typeof batchKey([]), 'string');
-  assert.strictEqual(batchKey([]), batchKey([]));
-});
-
-// ─── batchKey: same slug different runId produces different keys ──────────────
-
-test('same slug but different runId produces different keys', () => {
-  const a = [makeJob('101-foo', '2026-06-01T10-00-00-000Z')];
-  const b = [makeJob('101-foo', '2026-06-01T11-00-00-000Z')];
-  assert.notStrictEqual(batchKey(a), batchKey(b));
-});
-
-// ─── reportPathFor ────────────────────────────────────────────────────────────
-
-test('reportPathFor returns a path ending with definition-of-done-<key>.md', () => {
-  const key = batchKey([makeJob('101-foo', '2026-06-01T10-00-00-000Z')]);
-  const p = reportPathFor(key);
-  assert.ok(p.endsWith(`definition-of-done-${key}.md`), `path: ${p}`);
-  assert.ok(p.includes('scheduled-plans/runs/'), `path: ${p}`);
-});
-
-// ─── reportExists ─────────────────────────────────────────────────────────────
-
-test('reportExists returns false when runs/ dir does not exist', () => {
-  const key = batchKey([makeJob('101-foo', '2026-06-01T10-00-00-000Z')]);
-  const tmpDir = makeTmpDir();
-  try {
-    const runsDir = path.join(tmpDir, 'runs');
-    // runsDir intentionally not created
-    assert.strictEqual(reportExists(key, runsDir), false);
-  } finally {
-    rmdir(tmpDir);
-  }
-});
-
-test('reportExists returns false when no matching file exists', () => {
-  const key = batchKey([makeJob('101-foo', '2026-06-01T10-00-00-000Z')]);
-  const tmpDir = makeTmpDir();
-  try {
-    const runsDir = path.join(tmpDir, 'runs');
-    const runDir = path.join(runsDir, '2026-06-01T10-00-00-000Z');
-    fs.mkdirSync(runDir, { recursive: true });
-    fs.writeFileSync(path.join(runDir, 'some-other-file.md'), '# other');
-    assert.strictEqual(reportExists(key, runsDir), false);
-  } finally {
-    rmdir(tmpDir);
-  }
-});
-
-test('reportExists returns true when matching file exists in any run subdir', () => {
-  const key = batchKey([makeJob('101-foo', '2026-06-01T10-00-00-000Z')]);
-  const tmpDir = makeTmpDir();
-  try {
-    const runsDir = path.join(tmpDir, 'runs');
-    const runDir = path.join(runsDir, '2026-06-01T10-00-00-000Z');
-    fs.mkdirSync(runDir, { recursive: true });
-    fs.writeFileSync(
-      path.join(runDir, `definition-of-done-${key}.md`),
-      '# DoD report'
-    );
-    assert.strictEqual(reportExists(key, runsDir), true);
-  } finally {
-    rmdir(tmpDir);
-  }
-});
-
-test('reportExists finds match in a nested run directory (not first subdir)', () => {
-  const key = batchKey([makeJob('101-foo', '2026-06-01T10-00-00-000Z')]);
-  const tmpDir = makeTmpDir();
-  try {
-    const runsDir = path.join(tmpDir, 'runs');
-    // Two subdirs; match is in the second
-    const runDir1 = path.join(runsDir, '2026-06-01T09-00-00-000Z');
-    const runDir2 = path.join(runsDir, '2026-06-01T10-00-00-000Z');
-    fs.mkdirSync(runDir1, { recursive: true });
-    fs.mkdirSync(runDir2, { recursive: true });
-    fs.writeFileSync(path.join(runDir1, 'other.md'), '# other');
-    fs.writeFileSync(
-      path.join(runDir2, `definition-of-done-${key}.md`),
-      '# DoD report'
-    );
-    assert.strictEqual(reportExists(key, runsDir), true);
-  } finally {
-    rmdir(tmpDir);
-  }
-});