claude-code-session-manager 0.98.0 → 0.100.0

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

@@ -1,119 +0,0 @@
-/**
- * reconcileFlatPrdSweep.test.cjs — end-to-end proof that `reconcile()` itself
- * (not just `consolidateAllFlatPrds()` in isolation) never turns a
- * hand-written flat-dir PRD into a queue job, regardless of which of
- * reconcile's several callers (tickQueue's poll, job completion, the
- * schedule:state/schedule:rescan IPC handlers, rescheduleTimer) triggered
- * the pass.
- *
- * A code-review pass on this PRD flagged that gating only `tickQueue()` left
- * every OTHER caller of `reconcile()` free to still turn a freshly
- * hand-written flat-dir PRD into a `pending` job — the fix moved the sweep
- * inside `reconcile()` itself instead. This test exercises the real
- * `reconcile()` (not a stand-in) against a real on-disk flat-dir PRD with no
- * pre-existing queue row (exactly the state of a file dropped there moments
- * ago) and asserts it never appears in `state.jobs` afterward.
- *
- * HOME-isolated (mirrors scheduler-reconcile-invalid-repair.test.cjs): stub
- * HOME to a mkdtemp dir before requiring scheduler.cjs, and register the
- * fixture project as "active" via a fake ~/.claude/projects transcript —
- * reconcile's PRD discovery (prdLocations.cjs) only scans cwds it can find
- * that way.
- *
- * Run: timeout 120 npx vitest run src/main/__tests__/reconcileFlatPrdSweep.test.cjs
- */
-
-'use strict';
-
-import { test, expect, beforeAll, afterAll } from 'vitest';
-const fs = require('node:fs');
-const os = require('node:os');
-const path = require('node:path');
-
-let tmpHome;
-let originalHome;
-let scheduler;
-let bustCwdCache;
-
-beforeAll(() => {
-  originalHome = process.env.HOME;
-  tmpHome = fs.mkdtempSync(path.join(os.tmpdir(), 'sm-reconcile-flatsweep-home-'));
-  process.env.HOME = tmpHome;
-
-  scheduler = require('../scheduler.cjs');
-  ({ bustCwdCache } = require('../lib/queueStore.cjs'));
-
-  if (!scheduler.PRDS_DIR.startsWith(tmpHome)) {
-    throw new Error(`refusing to run: PRDS_DIR (${scheduler.PRDS_DIR}) is not under the temp HOME (${tmpHome})`);
-  }
-});
-
-afterAll(() => {
-  process.env.HOME = originalHome;
-  fs.rmSync(tmpHome, { recursive: true, force: true });
-});
-
-function registerActiveProject(cwd) {
-  const projectsDir = path.join(tmpHome, '.claude', 'projects');
-  const slugDir = path.join(projectsDir, `fake-project-slug-${path.basename(cwd)}`);
-  fs.mkdirSync(slugDir, { recursive: true });
-  fs.writeFileSync(path.join(slugDir, 'transcript.jsonl'), JSON.stringify({ cwd }) + '\n');
-  // Each test registers a NEW project mid-file; without busting queueStore's
-  // stateCwds cache (CACHE_MS 120s), a reconcile() call from an EARLIER test
-  // in this file can leave a stale cwd list cached that this test's freshly
-  // registered project is invisible to — reconcile() then never discovers
-  // its PRD at all (confirmed: this is what made
-  // 'reconcile() still onboards a PRD written to the canonical Epic-scoped
-  // dir' intermittently fail under full-suite load, PRD 1414).
-  bustCwdCache();
-}
-
-function makeFixtureProject(prefix) {
-  const cwd = fs.mkdtempSync(path.join(os.tmpdir(), prefix));
-  registerActiveProject(cwd);
-  const opsRoot = path.join(cwd, 'session-manager-operations');
-  const flatPrdsDir = path.join(opsRoot, 'scheduler', 'prds');
-  const stateDir = path.join(opsRoot, 'scheduler', 'state');
-  fs.mkdirSync(flatPrdsDir, { recursive: true });
-  fs.mkdirSync(stateDir, { recursive: true });
-  return { cwd, flatPrdsDir, stateDir, archiveDir: path.join(opsRoot, 'scheduler', 'prds-archived') };
-}
-
-test('reconcile() never turns a hand-written flat-dir PRD into a job, no matter which caller triggers it', async () => {
-  const { cwd, flatPrdsDir, archiveDir } = makeFixtureProject('sm-reconcile-flatsweep-proj-');
-  const filename = '1500-hand-written-bypass.md';
-  fs.writeFileSync(
-    path.join(flatPrdsDir, filename),
-    `---\ntitle: Hand-written bypass\ncwd: ${cwd}\nestimateMinutes: 10\n---\n\n# Goal\n\nSomething.\n`,
-    'utf8',
-  );
-  // No queue.json at all — the exact state of a PRD dropped on disk moments
-  // ago, before any prior reconcile() pass has scanned it into a job.
-
-  const state = { jobs: [], invalidJobs: [], paused: null };
-  await scheduler.reconcile(state);
-
-  const found = state.jobs.find((j) => j.slug === '1500-hand-written-bypass');
-  expect(found).toBeUndefined();
-  expect(fs.existsSync(path.join(flatPrdsDir, filename))).toBe(false);
-  expect(fs.existsSync(path.join(archiveDir, filename))).toBe(true);
-});
-
-test('reconcile() still onboards a PRD written to the canonical Epic-scoped dir (the sweep only touches the flat dir)', async () => {
-  const { cwd } = makeFixtureProject('sm-reconcile-flatsweep-epic-proj-');
-  const epicId = 'epic-abc';
-  const epicPrdsDir = path.join(cwd, 'session-manager-operations', 'scheduler', 'epics', epicId, 'prds');
-  fs.mkdirSync(epicPrdsDir, { recursive: true });
-  fs.writeFileSync(
-    path.join(epicPrdsDir, '1501-canonical.md'),
-    `---\ntitle: Canonical\ncwd: ${cwd}\nestimateMinutes: 10\nsourcePromptId: ${epicId}\ncreatedVia: scheduler-api\nissuedAt: 2026-08-07T00:00:00.000Z\n---\n\n# Goal\n\nSomething.\n`,
-    'utf8',
-  );
-
-  const state = { jobs: [], invalidJobs: [], paused: null };
-  await scheduler.reconcile(state);
-
-  const found = state.jobs.find((j) => j.slug === '1501-canonical');
-  expect(found).toBeDefined();
-  expect(found.status).toBe('pending');
-});

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

@@ -1,135 +0,0 @@
-/**
- * reconcileTiming.test.cjs — proves reconcile() (src/main/scheduler.cjs)
- * records a per-phase timing breakdown and emits exactly one warn-level
- * `logs.writeLine` when a pass exceeds RECONCILE_SLOW_PASS_MS
- * (src/main/lib/schedulerConfig.cjs), and stays silent on a normal-speed
- * pass. Before this, diagnosing a slow reconcile pass (the
- * "schedule.state timed out after 5000ms" toast) required hand-timing the
- * filesystem scans from a throwaway script.
- *
- * HOME-isolated (mirrors reconcileFlatPrdSweep.test.cjs): stub HOME to a
- * mkdtemp dir before requiring scheduler.cjs, and register the fixture
- * project as "active" via a fake ~/.claude/projects transcript — reconcile's
- * PRD discovery (prdLocations.cjs) only scans cwds it can find that way.
- *
- * Run: timeout 120 npx vitest run src/main/__tests__/reconcileTiming.test.cjs
- */
-
-'use strict';
-
-import { test, expect, beforeAll, afterAll, afterEach, vi } from 'vitest';
-const fs = require('node:fs');
-const os = require('node:os');
-const path = require('node:path');
-
-let tmpHome;
-let originalHome;
-let scheduler;
-let queueHistory;
-let logs;
-let RECONCILE_SLOW_PASS_MS;
-
-beforeAll(() => {
-  originalHome = process.env.HOME;
-  tmpHome = fs.mkdtempSync(path.join(os.tmpdir(), 'sm-reconcile-timing-home-'));
-  process.env.HOME = tmpHome;
-
-  scheduler = require('../scheduler.cjs');
-  queueHistory = require('../lib/queueHistory.cjs');
-  logs = require('../logs.cjs');
-  ({ RECONCILE_SLOW_PASS_MS } = require('../lib/schedulerConfig.cjs'));
-
-  if (!scheduler.PRDS_DIR.startsWith(tmpHome)) {
-    throw new Error(`refusing to run: PRDS_DIR (${scheduler.PRDS_DIR}) is not under the temp HOME (${tmpHome})`);
-  }
-});
-
-afterAll(() => {
-  process.env.HOME = originalHome;
-  fs.rmSync(tmpHome, { recursive: true, force: true });
-});
-
-afterEach(() => {
-  vi.restoreAllMocks();
-});
-
-function registerActiveProject(cwd) {
-  const projectsDir = path.join(tmpHome, '.claude', 'projects');
-  const slugDir = path.join(projectsDir, `fake-project-slug-${path.basename(cwd)}`);
-  fs.mkdirSync(slugDir, { recursive: true });
-  fs.writeFileSync(path.join(slugDir, 'transcript.jsonl'), JSON.stringify({ cwd }) + '\n');
-}
-
-function makeFixtureProject(prefix) {
-  const cwd = fs.mkdtempSync(path.join(os.tmpdir(), prefix));
-  registerActiveProject(cwd);
-  return cwd;
-}
-
-function writeCanonicalPrd(cwd, epicId, slug) {
-  const epicPrdsDir = path.join(cwd, 'session-manager-operations', 'scheduler', 'epics', epicId, 'prds');
-  fs.mkdirSync(epicPrdsDir, { recursive: true });
-  fs.writeFileSync(
-    path.join(epicPrdsDir, `${slug}.md`),
-    `---\ntitle: Timing fixture\ncwd: ${cwd}\nestimateMinutes: 10\nsourcePromptId: ${epicId}\ncreatedVia: scheduler-api\nissuedAt: 2026-08-07T00:00:00.000Z\n---\n\n# Goal\n\nSomething.\n`,
-    'utf8',
-  );
-}
-
-function warnCallsMatching(spy) {
-  return spy.mock.calls
-    .map(([payload]) => payload)
-    .filter((payload) => payload?.level === 'warn' && payload?.scope === 'scheduler' && /reconcile\(\) pass took/.test(payload?.message ?? ''));
-}
-
-test('a normal-speed reconcile() pass logs nothing', async () => {
-  const cwd = makeFixtureProject('sm-reconcile-timing-fast-proj-');
-  writeCanonicalPrd(cwd, 'epic-fast', '1600-fast');
-
-  const writeLineSpy = vi.spyOn(logs, 'writeLine');
-  const state = { jobs: [], invalidJobs: [], paused: null };
-  await scheduler.reconcile(state);
-
-  expect(warnCallsMatching(writeLineSpy)).toHaveLength(0);
-});
-
-test('a slow reconcile() pass emits exactly one warn with the per-phase breakdown', async () => {
-  const cwd = makeFixtureProject('sm-reconcile-timing-slow-proj-');
-  writeCanonicalPrd(cwd, 'epic-slow', '1601-slow');
-
-  const realHistoryTerminalBySlug = queueHistory.historyTerminalBySlug.bind(queueHistory);
-  vi.spyOn(queueHistory, 'historyTerminalBySlug').mockImplementation(async (...args) => {
-    await new Promise((resolve) => { setTimeout(resolve, RECONCILE_SLOW_PASS_MS + 200); });
-    return realHistoryTerminalBySlug(...args);
-  });
-  const writeLineSpy = vi.spyOn(logs, 'writeLine');
-
-  const state = { jobs: [], invalidJobs: [], paused: null };
-  await scheduler.reconcile(state);
-
-  const warnCalls = warnCallsMatching(writeLineSpy);
-  expect(warnCalls).toHaveLength(1);
-  const { meta } = warnCalls[0];
-  expect(meta.totalMs).toBeGreaterThan(RECONCILE_SLOW_PASS_MS);
-  expect(meta.phaseMs).toHaveProperty('flatPrdSweep');
-  expect(meta.phaseMs).toHaveProperty('prdDirResolve');
-  expect(meta.phaseMs).toHaveProperty('parseLoop');
-  expect(meta.phaseMs).toHaveProperty('historyLookup');
-  expect(meta.phaseMs).toHaveProperty('queueWrite');
-  expect(meta.phaseMs.historyLookup).toBeGreaterThanOrEqual(RECONCILE_SLOW_PASS_MS);
-  expect(typeof meta.prdFileCount).toBe('number');
-  expect(typeof meta.resolvedDirCount).toBe('number');
-  expect(meta.prdFileCount).toBeGreaterThanOrEqual(1);
-}, 15_000);
-
-test('reconcile() surfaces its own error and does not swallow it while timing', async () => {
-  vi.spyOn(queueHistory, 'partitionJobs').mockImplementation(() => {
-    throw new Error('boom: injected failure for timing-swallow test');
-  });
-
-  const cwd = makeFixtureProject('sm-reconcile-timing-throw-proj-');
-  writeCanonicalPrd(cwd, 'epic-throw', '1602-throw');
-
-  const state = { jobs: [], invalidJobs: [], paused: null };
-  await expect(scheduler.reconcile(state)).rejects.toThrow('boom: injected failure for timing-swallow test');
-});