claude-code-session-manager 0.97.0 → 0.100.0

package/src/main/__tests__/dod-drain-hook.test.cjs

@@ -1,302 +0,0 @@
-/**
- * dod-drain-hook.test.cjs — drives the DoD drain handler.
- *
- * Run: timeout 300 npx vitest run src/main/__tests__/dod-drain-hook.test.cjs
- *
- * Scenarios:
- *   1. Drain with completed jobs → writes report once.
- *   2. Second drain over same completed set → no-op (idempotent).
- *   3. SM_DOD_DISABLE=1 → no report.
- *   4. state.paused set → no report.
- *   5. cancelToken.cancelled = true → no report.
- */
-
-'use strict';
-
-import { test, beforeEach, afterEach } from 'vitest';
-const assert = require('node:assert/strict');
-const os = require('node:os');
-const fs = require('node:fs');
-const path = require('node:path');
-const { runDefinitionOfDoneOnDrain } = require('../lib/dodDrainHook.cjs');
-const { readWatermark } = require('../lib/definitionOfDone.cjs');
-
-// ─── helpers ──────────────────────────────────────────────────────────────────
-
-function makeTmpDir() {
-  return fs.mkdtempSync(path.join(os.tmpdir(), 'dod-drain-hook-test-'));
-}
-
-function rmdir(dir) {
-  try { fs.rmSync(dir, { recursive: true, force: true }); } catch { /* */ }
-}
-
-/** Write a minimal PRD and return a completed-job record. */
-function writePrd(prdsDir, slug, cwd) {
-  const body = [
-    '---',
-    `title: ${slug}`,
-    `cwd: ${cwd}`,
-    'estimateMinutes: 5',
-    '---',
-    '',
-    '# Goal',
-    '',
-    'Test fixture.',
-    '',
-    '# Acceptance criteria',
-    '',
-    '- [ ] `timeout 10 node -e "process.exit(0)"` passes.',
-    '',
-    '# Out of scope',
-    '',
-    '- N/A',
-  ].join('\n');
-  fs.writeFileSync(path.join(prdsDir, `${slug}.md`), body);
-  return { slug, cwd, status: 'completed', runId: '2026-06-14T00-00-00-000Z' };
-}
-
-/** Count DoD report files under runsDir (shallow scan). */
-function countReports(runsDir) {
-  let count = 0;
-  try {
-    for (const entry of fs.readdirSync(runsDir, { withFileTypes: true })) {
-      if (!entry.isDirectory()) continue;
-      const dir = path.join(runsDir, entry.name);
-      for (const f of fs.readdirSync(dir)) {
-        if (f.startsWith('definition-of-done-')) count++;
-      }
-    }
-  } catch { /* runsDir may not exist */ }
-  return count;
-}
-
-// ─── tests ────────────────────────────────────────────────────────────────────
-
-test('drain with completed jobs writes exactly one report', async () => {
-  const tmpDir = makeTmpDir();
-  const prdsDir = path.join(tmpDir, 'prds');
-  const runsDir = path.join(tmpDir, 'runs');
-  const cwd = path.join(tmpDir, 'project');
-  fs.mkdirSync(prdsDir);
-  fs.mkdirSync(cwd);
-
-  const job = writePrd(prdsDir, '101-test', cwd);
-  const state = { paused: null, jobs: [job] };
-  const cancelToken = { cancelled: false };
-  const savedDisable = process.env.SM_DOD_DISABLE;
-  delete process.env.SM_DOD_DISABLE;
-
-  try {
-    await runDefinitionOfDoneOnDrain(state, { cancelToken, prdsDir, runsDir });
-    assert.strictEqual(countReports(runsDir), 1);
-  } finally {
-    if (savedDisable !== undefined) process.env.SM_DOD_DISABLE = savedDisable;
-    rmdir(tmpDir);
-  }
-});
-
-test('second drain over same completed set is a no-op (idempotent)', async () => {
-  const tmpDir = makeTmpDir();
-  const prdsDir = path.join(tmpDir, 'prds');
-  const runsDir = path.join(tmpDir, 'runs');
-  const cwd = path.join(tmpDir, 'project');
-  fs.mkdirSync(prdsDir);
-  fs.mkdirSync(cwd);
-
-  const job = writePrd(prdsDir, '102-test', cwd);
-  const state = { paused: null, jobs: [job] };
-  const cancelToken = { cancelled: false };
-  const savedDisable = process.env.SM_DOD_DISABLE;
-  delete process.env.SM_DOD_DISABLE;
-
-  try {
-    await runDefinitionOfDoneOnDrain(state, { cancelToken, prdsDir, runsDir });
-    const afterFirst = countReports(runsDir);
-    assert.strictEqual(afterFirst, 1, 'first call must write one report');
-
-    await runDefinitionOfDoneOnDrain(state, { cancelToken, prdsDir, runsDir });
-    const afterSecond = countReports(runsDir);
-    assert.strictEqual(afterSecond, 1, 'second call with same jobs must be no-op');
-  } finally {
-    if (savedDisable !== undefined) process.env.SM_DOD_DISABLE = savedDisable;
-    rmdir(tmpDir);
-  }
-});
-
-test('SM_DOD_DISABLE=1 → no report written', async () => {
-  const tmpDir = makeTmpDir();
-  const prdsDir = path.join(tmpDir, 'prds');
-  const runsDir = path.join(tmpDir, 'runs');
-  const cwd = path.join(tmpDir, 'project');
-  fs.mkdirSync(prdsDir);
-  fs.mkdirSync(cwd);
-
-  writePrd(prdsDir, '103-test', cwd);
-  const job = { slug: '103-test', cwd, status: 'completed', runId: '2026-06-14T00-00-00-001Z' };
-  const state = { paused: null, jobs: [job] };
-  const cancelToken = { cancelled: false };
-  const savedDisable = process.env.SM_DOD_DISABLE;
-  process.env.SM_DOD_DISABLE = '1';
-
-  try {
-    await runDefinitionOfDoneOnDrain(state, { cancelToken, prdsDir, runsDir });
-    assert.strictEqual(countReports(runsDir), 0, 'SM_DOD_DISABLE=1 must suppress the report');
-  } finally {
-    if (savedDisable !== undefined) process.env.SM_DOD_DISABLE = savedDisable;
-    else delete process.env.SM_DOD_DISABLE;
-    rmdir(tmpDir);
-  }
-});
-
-test('paused state → no report written', async () => {
-  const tmpDir = makeTmpDir();
-  const prdsDir = path.join(tmpDir, 'prds');
-  const runsDir = path.join(tmpDir, 'runs');
-  const cwd = path.join(tmpDir, 'project');
-  fs.mkdirSync(prdsDir);
-  fs.mkdirSync(cwd);
-
-  writePrd(prdsDir, '104-test', cwd);
-  const job = { slug: '104-test', cwd, status: 'completed', runId: '2026-06-14T00-00-00-002Z' };
-  const state = { paused: { reason: 'rate_limit', since: new Date().toISOString(), resumeAt: null }, jobs: [job] };
-  const cancelToken = { cancelled: false };
-  const savedDisable = process.env.SM_DOD_DISABLE;
-  delete process.env.SM_DOD_DISABLE;
-
-  try {
-    await runDefinitionOfDoneOnDrain(state, { cancelToken, prdsDir, runsDir });
-    assert.strictEqual(countReports(runsDir), 0, 'paused state must suppress the report');
-  } finally {
-    if (savedDisable !== undefined) process.env.SM_DOD_DISABLE = savedDisable;
-    rmdir(tmpDir);
-  }
-});
-
-test('first-ever drain (no watermark file) processes all completed jobs', async () => {
-  const tmpDir = makeTmpDir();
-  const prdsDir = path.join(tmpDir, 'prds');
-  const runsDir = path.join(tmpDir, 'runs');
-  const cwd = path.join(tmpDir, 'project');
-  fs.mkdirSync(prdsDir);
-  fs.mkdirSync(cwd);
-
-  const jobA = writePrd(prdsDir, '106-old', cwd);
-  jobA.finishedAt = '2020-01-01T00:00:00.000Z';
-  const jobB = writePrd(prdsDir, '107-new', cwd);
-  jobB.finishedAt = '2026-07-11T00:00:00.000Z';
-  const state = { paused: null, jobs: [jobA, jobB] };
-  const cancelToken = { cancelled: false };
-  const savedDisable = process.env.SM_DOD_DISABLE;
-  delete process.env.SM_DOD_DISABLE;
-
-  try {
-    assert.strictEqual(readWatermark(runsDir), null, 'no watermark should exist yet');
-    await runDefinitionOfDoneOnDrain(state, { cancelToken, prdsDir, runsDir });
-    assert.strictEqual(countReports(runsDir), 1);
-
-    const reportsRoot = fs.readdirSync(runsDir, { withFileTypes: true }).filter(e => e.isDirectory());
-    const reportFile = reportsRoot
-      .map(e => path.join(runsDir, e.name))
-      .flatMap(dir => fs.readdirSync(dir).filter(f => f.startsWith('definition-of-done-')).map(f => path.join(dir, f)))[0];
-    const contents = fs.readFileSync(reportFile, 'utf8');
-    assert.match(contents, /106-old/, 'first-ever drain must include old job too');
-    assert.match(contents, /107-new/);
-  } finally {
-    if (savedDisable !== undefined) process.env.SM_DOD_DISABLE = savedDisable;
-    rmdir(tmpDir);
-  }
-});
-
-test('subsequent drain with a persisted watermark only reverifies newer jobs', async () => {
-  const tmpDir = makeTmpDir();
-  const prdsDir = path.join(tmpDir, 'prds');
-  const runsDir = path.join(tmpDir, 'runs');
-  const cwd = path.join(tmpDir, 'project');
-  fs.mkdirSync(prdsDir);
-  fs.mkdirSync(cwd);
-  fs.mkdirSync(runsDir, { recursive: true });
-  fs.writeFileSync(
-    path.join(runsDir, '.dod-watermark.json'),
-    JSON.stringify({ lastFinishedAt: '2026-01-01T00:00:00.000Z' })
-  );
-
-  const jobOld = writePrd(prdsDir, '108-old', cwd);
-  jobOld.finishedAt = '2025-01-01T00:00:00.000Z';
-  const jobNew = writePrd(prdsDir, '109-new', cwd);
-  jobNew.finishedAt = '2026-07-11T00:00:00.000Z';
-  const state = { paused: null, jobs: [jobOld, jobNew] };
-  const cancelToken = { cancelled: false };
-  const savedDisable = process.env.SM_DOD_DISABLE;
-  delete process.env.SM_DOD_DISABLE;
-
-  try {
-    await runDefinitionOfDoneOnDrain(state, { cancelToken, prdsDir, runsDir });
-    assert.strictEqual(countReports(runsDir), 1);
-
-    const reportsRoot = fs.readdirSync(runsDir, { withFileTypes: true }).filter(e => e.isDirectory());
-    const reportFile = reportsRoot
-      .map(e => path.join(runsDir, e.name))
-      .flatMap(dir => fs.readdirSync(dir).filter(f => f.startsWith('definition-of-done-')).map(f => path.join(dir, f)))[0];
-    const contents = fs.readFileSync(reportFile, 'utf8');
-    assert.doesNotMatch(contents, /108-old/, 'jobs finished before the watermark must be excluded');
-    assert.match(contents, /109-new/, 'jobs finished after the watermark must be included');
-  } finally {
-    if (savedDisable !== undefined) process.env.SM_DOD_DISABLE = savedDisable;
-    rmdir(tmpDir);
-  }
-});
-
-test('watermark advances after a report so a repeat drain over the same jobs is a no-op', async () => {
-  const tmpDir = makeTmpDir();
-  const prdsDir = path.join(tmpDir, 'prds');
-  const runsDir = path.join(tmpDir, 'runs');
-  const cwd = path.join(tmpDir, 'project');
-  fs.mkdirSync(prdsDir);
-  fs.mkdirSync(cwd);
-
-  const job = writePrd(prdsDir, '110-test', cwd);
-  job.finishedAt = '2026-07-11T00:00:00.000Z';
-  const state = { paused: null, jobs: [job] };
-  const cancelToken = { cancelled: false };
-  const savedDisable = process.env.SM_DOD_DISABLE;
-  delete process.env.SM_DOD_DISABLE;
-
-  try {
-    await runDefinitionOfDoneOnDrain(state, { cancelToken, prdsDir, runsDir });
-    assert.strictEqual(countReports(runsDir), 1, 'first drain writes one report');
-    assert.strictEqual(readWatermark(runsDir), job.finishedAt, 'watermark must advance to the covered job\'s finishedAt');
-
-    // Second drain with a fresh job list containing the same already-covered job.
-    const state2 = { paused: null, jobs: [{ ...job }] };
-    await runDefinitionOfDoneOnDrain(state2, { cancelToken, prdsDir, runsDir });
-    assert.strictEqual(countReports(runsDir), 1, 'repeat drain over the same (now-old) job must be a no-op');
-  } finally {
-    if (savedDisable !== undefined) process.env.SM_DOD_DISABLE = savedDisable;
-    rmdir(tmpDir);
-  }
-});
-
-test('cancelToken.cancelled=true → no report written', async () => {
-  const tmpDir = makeTmpDir();
-  const prdsDir = path.join(tmpDir, 'prds');
-  const runsDir = path.join(tmpDir, 'runs');
-  const cwd = path.join(tmpDir, 'project');
-  fs.mkdirSync(prdsDir);
-  fs.mkdirSync(cwd);
-
-  writePrd(prdsDir, '105-test', cwd);
-  const job = { slug: '105-test', cwd, status: 'completed', runId: '2026-06-14T00-00-00-003Z' };
-  const state = { paused: null, jobs: [job] };
-  const cancelToken = { cancelled: true };
-  const savedDisable = process.env.SM_DOD_DISABLE;
-  delete process.env.SM_DOD_DISABLE;
-
-  try {
-    await runDefinitionOfDoneOnDrain(state, { cancelToken, prdsDir, runsDir });
-    assert.strictEqual(countReports(runsDir), 0, 'cancelled token must suppress the report');
-  } finally {
-    if (savedDisable !== undefined) process.env.SM_DOD_DISABLE = savedDisable;
-    rmdir(tmpDir);
-  }
-});

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

@@ -1,304 +0,0 @@
-/**
- * dod-report.test.cjs — unit tests for flagRiskySurfaces + writeReport.
- *
- * Run: timeout 300 npx vitest run src/main/__tests__/dod-report.test.cjs
- *
- * Fixtures: os.tmpdir() only — never writes into real runs/.
- */
-
-'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 { flagRiskySurfaces, writeReport } = require('../lib/definitionOfDone.cjs');
-
-// ─── helpers ──────────────────────────────────────────────────────────────────
-
-function makeTmpDir() {
-  return fs.mkdtempSync(path.join(os.tmpdir(), 'dod-report-test-'));
-}
-
-function rmdir(dir) {
-  try { fs.rmSync(dir, { recursive: true, force: true }); } catch { /* */ }
-}
-
-function writePrd(prdsDir, slug, implNotes, cwd) {
-  const body = [
-    '---',
-    `title: ${slug}`,
-    `cwd: ${cwd}`,
-    'estimateMinutes: 5',
-    '---',
-    '',
-    '# Goal',
-    '',
-    'Test fixture.',
-    '',
-    '# Acceptance criteria',
-    '',
-    '- [ ] `timeout 10 node -e "process.exit(0)"` passes.',
-    '',
-    '# Implementation notes',
-    '',
-    implNotes,
-    '',
-    '# Out of scope',
-    '',
-    '- N/A',
-  ].join('\n');
-  fs.writeFileSync(path.join(prdsDir, `${slug}.md`), body);
-  return { slug, cwd };
-}
-
-// ─── flagRiskySurfaces: money-path ────────────────────────────────────────────
-
-test('flagRiskySurfaces: flags money-path when impl notes reference payment', () => {
-  const tmpDir = makeTmpDir();
-  const prdsDir = path.join(tmpDir, 'prds');
-  const cwd = path.join(tmpDir, 'project');
-  fs.mkdirSync(prdsDir);
-  fs.mkdirSync(cwd);
-  const job = writePrd(prdsDir, '201-billing', '- Edit `src/payment-processor.js`\n- Update order schema', cwd);
-  try {
-    const flags = flagRiskySurfaces([job], { prdsDir });
-    assert.strictEqual(flags.length, 1);
-    assert.strictEqual(flags[0].slug, '201-billing');
-    assert.ok(flags[0].surfaces.includes('money-path'), `expected money-path in ${JSON.stringify(flags[0].surfaces)}`);
-  } finally {
-    rmdir(tmpDir);
-  }
-});
-
-test('flagRiskySurfaces: flags auth when impl notes reference auth/token', () => {
-  const tmpDir = makeTmpDir();
-  const prdsDir = path.join(tmpDir, 'prds');
-  const cwd = path.join(tmpDir, 'project');
-  fs.mkdirSync(prdsDir);
-  fs.mkdirSync(cwd);
-  const job = writePrd(prdsDir, '202-auth', '- Modify `src/auth-middleware.py`\n- Update token refresh logic', cwd);
-  try {
-    const flags = flagRiskySurfaces([job], { prdsDir });
-    assert.strictEqual(flags.length, 1);
-    assert.ok(flags[0].surfaces.includes('auth'));
-  } finally {
-    rmdir(tmpDir);
-  }
-});
-
-test('flagRiskySurfaces: flags migration when impl notes reference .sql file', () => {
-  const tmpDir = makeTmpDir();
-  const prdsDir = path.join(tmpDir, 'prds');
-  const cwd = path.join(tmpDir, 'project');
-  fs.mkdirSync(prdsDir);
-  fs.mkdirSync(cwd);
-  const job = writePrd(prdsDir, '203-schema', '- Run `db/migrations/0042_users.sql`', cwd);
-  try {
-    const flags = flagRiskySurfaces([job], { prdsDir });
-    assert.strictEqual(flags.length, 1);
-    assert.ok(flags[0].surfaces.includes('migration'));
-  } finally {
-    rmdir(tmpDir);
-  }
-});
-
-test('flagRiskySurfaces: clean job is not returned', () => {
-  const tmpDir = makeTmpDir();
-  const prdsDir = path.join(tmpDir, 'prds');
-  const cwd = path.join(tmpDir, 'project');
-  fs.mkdirSync(prdsDir);
-  fs.mkdirSync(cwd);
-  const job = writePrd(prdsDir, '204-clean', '- Update `src/ui/button.tsx`\n- Add tooltip component', cwd);
-  try {
-    const flags = flagRiskySurfaces([job], { prdsDir });
-    assert.strictEqual(flags.length, 0);
-  } finally {
-    rmdir(tmpDir);
-  }
-});
-
-test('flagRiskySurfaces: job with no PRD file is skipped (not thrown)', () => {
-  const tmpDir = makeTmpDir();
-  const prdsDir = path.join(tmpDir, 'prds');
-  const cwd = path.join(tmpDir, 'project');
-  fs.mkdirSync(prdsDir);
-  fs.mkdirSync(cwd);
-  try {
-    const flags = flagRiskySurfaces([{ slug: '205-missing', cwd }], { prdsDir });
-    assert.strictEqual(flags.length, 0);
-  } finally {
-    rmdir(tmpDir);
-  }
-});
-
-test('flagRiskySurfaces: mixed batch returns only risky jobs', () => {
-  const tmpDir = makeTmpDir();
-  const prdsDir = path.join(tmpDir, 'prds');
-  const cwd = path.join(tmpDir, 'project');
-  fs.mkdirSync(prdsDir);
-  fs.mkdirSync(cwd);
-  const jobRisky = writePrd(prdsDir, '206-risky', '- Modify `src/trade-engine.js`', cwd);
-  const jobClean = writePrd(prdsDir, '207-clean', '- Update `src/tooltip.tsx`', cwd);
-  try {
-    const flags = flagRiskySurfaces([jobRisky, jobClean], { prdsDir });
-    assert.strictEqual(flags.length, 1);
-    assert.strictEqual(flags[0].slug, '206-risky');
-  } finally {
-    rmdir(tmpDir);
-  }
-});
-
-// ─── writeReport: structure ────────────────────────────────────────────────────
-
-test('writeReport: all three jobs appear in AC table', () => {
-  const tmpDir = makeTmpDir();
-  const runsDir = path.join(tmpDir, 'runs');
-  const acResults = [
-    { slug: '101-pass',  status: 'pass',         code: 0,    ms: 500 },
-    { slug: '102-fail',  status: 'fail',         code: 1,    ms: 300 },
-    { slug: '103-risky', status: 'pass',         code: 0,    ms: 400 },
-  ];
-  const riskFlags = [{ slug: '103-risky', surfaces: ['money-path'] }];
-  try {
-    const reportPath = writeReport('abcd1234', { acResults, riskFlags, runsDir });
-    assert.ok(fs.existsSync(reportPath), `report missing at ${reportPath}`);
-    const content = fs.readFileSync(reportPath, 'utf8');
-    assert.ok(content.includes('101-pass'), 'missing 101-pass in table');
-    assert.ok(content.includes('102-fail'), 'missing 102-fail in table');
-    assert.ok(content.includes('103-risky'), 'missing 103-risky in table');
-  } finally {
-    rmdir(tmpDir);
-  }
-});
-
-test('writeReport: fail and risky slug both appear in needs-attention list', () => {
-  const tmpDir = makeTmpDir();
-  const runsDir = path.join(tmpDir, 'runs');
-  const acResults = [
-    { slug: '101-pass',  status: 'pass', code: 0, ms: 500 },
-    { slug: '102-fail',  status: 'fail', code: 1, ms: 300 },
-    { slug: '103-risky', status: 'pass', code: 0, ms: 400 },
-  ];
-  const riskFlags = [{ slug: '103-risky', surfaces: ['money-path'] }];
-  try {
-    const reportPath = writeReport('abcd1234', { acResults, riskFlags, runsDir });
-    const content = fs.readFileSync(reportPath, 'utf8');
-    // Both should appear in the Needs Human Attention section
-    const attentionIdx = content.indexOf('## Needs Human Attention');
-    assert.ok(attentionIdx !== -1, 'missing Needs Human Attention section');
-    const attentionSection = content.slice(attentionIdx);
-    assert.ok(attentionSection.includes('102-fail'), '102-fail missing from attention list');
-    assert.ok(attentionSection.includes('103-risky'), '103-risky missing from attention list');
-    // Clean pass should NOT be in attention list
-    const passInAttention = attentionSection.includes('101-pass');
-    assert.ok(!passInAttention, '101-pass should not appear in attention list');
-  } finally {
-    rmdir(tmpDir);
-  }
-});
-
-test('writeReport: report states review is recommended, not auto-run', () => {
-  const tmpDir = makeTmpDir();
-  const runsDir = path.join(tmpDir, 'runs');
-  try {
-    const reportPath = writeReport('abcd1234', { acResults: [], riskFlags: [], runsDir });
-    const content = fs.readFileSync(reportPath, 'utf8');
-    assert.ok(
-      content.toLowerCase().includes('recommended') && content.toLowerCase().includes('not auto-run'),
-      'report must state review is recommended, not auto-run'
-    );
-  } finally {
-    rmdir(tmpDir);
-  }
-});
-
-test('writeReport: no .tmp files left after write', () => {
-  const tmpDir = makeTmpDir();
-  const runsDir = path.join(tmpDir, 'runs');
-  const acResults = [{ slug: '101-pass', status: 'pass', code: 0, ms: 100 }];
-  try {
-    writeReport('deadbeef', { acResults, riskFlags: [], runsDir });
-    // Walk all files in runsDir recursively, check none end in .tmp-*
-    function findTmps(dir) {
-      const hits = [];
-      try {
-        for (const e of fs.readdirSync(dir, { withFileTypes: true })) {
-          const full = path.join(dir, e.name);
-          if (e.isDirectory()) hits.push(...findTmps(full));
-          else if (e.name.includes('.tmp-')) hits.push(full);
-        }
-      } catch { /* */ }
-      return hits;
-    }
-    const tmps = findTmps(runsDir);
-    assert.deepStrictEqual(tmps, [], `found leftover tmp files: ${tmps.join(', ')}`);
-  } finally {
-    rmdir(tmpDir);
-  }
-});
-
-test('writeReport: second call with same batchKey produces clean report, no tmp files', () => {
-  const tmpDir = makeTmpDir();
-  const runsDir = path.join(tmpDir, 'runs');
-  const acResults = [
-    { slug: '101-pass', status: 'pass', code: 0, ms: 100 },
-    { slug: '102-fail', status: 'fail', code: 1, ms: 200 },
-  ];
-  const riskFlags = [{ slug: '101-pass', surfaces: ['auth'] }];
-  try {
-    writeReport('cafebabe', { acResults, riskFlags, runsDir });
-    // Small delay to guarantee different timestamp dir
-    const t = Date.now() + 2;
-    while (Date.now() < t) { /* busy wait 2ms */ }
-    const report2 = writeReport('cafebabe', { acResults, riskFlags, runsDir });
-
-    // Second report must exist and be valid
-    assert.ok(fs.existsSync(report2));
-    const content = fs.readFileSync(report2, 'utf8');
-
-    // No duplicate rows in the AC table — slice only the AC table section.
-    const acStart = content.indexOf('## AC Results');
-    const riskStart = content.indexOf('## Risk Flags');
-    const acSection = content.slice(acStart, riskStart === -1 ? undefined : riskStart);
-    const passCount = (acSection.match(/101-pass/g) || []).length;
-    const failCount = (acSection.match(/102-fail/g) || []).length;
-    assert.strictEqual(passCount, 1, `101-pass appears ${passCount} times in AC table`);
-    assert.strictEqual(failCount, 1, `102-fail appears ${failCount} times in AC table`);
-
-    // No tmp files anywhere
-    function findTmps(dir) {
-      const hits = [];
-      try {
-        for (const e of fs.readdirSync(dir, { withFileTypes: true })) {
-          const full = path.join(dir, e.name);
-          if (e.isDirectory()) hits.push(...findTmps(full));
-          else if (e.name.includes('.tmp-')) hits.push(full);
-        }
-      } catch { /* */ }
-      return hits;
-    }
-    assert.deepStrictEqual(findTmps(runsDir), []);
-  } finally {
-    rmdir(tmpDir);
-  }
-});
-
-test('writeReport: report filename contains the batchKey', () => {
-  const tmpDir = makeTmpDir();
-  const runsDir = path.join(tmpDir, 'runs');
-  try {
-    const reportPath = writeReport('feed1234', { acResults: [], riskFlags: [], runsDir });
-    assert.ok(path.basename(reportPath).includes('feed1234'));
-  } finally {
-    rmdir(tmpDir);
-  }
-});
-
-test('writeReport: invalid batchKey throws', () => {
-  assert.throws(
-    () => writeReport('not-hex!!', { acResults: [], riskFlags: [] }),
-    /invalid batchKey/
-  );
-});