claude-code-session-manager 0.98.0 → 0.101.0

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

@@ -1,153 +0,0 @@
-// PRD 832 — unique per-project PRD numbers + dependsOn eligibility.
-// User decision 2026-07-31: duplicate NN must be impossible; ordering is
-// expressed via dependsOn frontmatter, never via shared numbers.
-import { describe, it, expect } from 'vitest';
-import fs from 'node:fs';
-import os from 'node:os';
-import path from 'node:path';
-
-// HOME stubbed BEFORE requiring scheduler.cjs (PRD 812 fix): the last
-// describe block below calls scheduler.reconcile(), which internally calls
-// ensureDirs() → fs.mkdirSync on PRDS_DIR/RUNS_DIR — paths baked from
-// os.homedir() at require time. Without this stub that mkdirSync lands in
-// the real `~/.claude/session-manager/scheduled-plans/` tree.
-const tmpHome = fs.mkdtempSync(path.join(os.tmpdir(), 'sm-832-home-'));
-process.env.HOME = tmpHome;
-
-const prdParser = require('../scheduler/prdParser.cjs');
-const { pickForProject } = require('../lib/schedulerBatch.cjs');
-
-function tmpdir() {
-  return fs.mkdtempSync(path.join(os.tmpdir(), 'sm-832-'));
-}
-
-const job = (slug, status, group, extra = {}) => ({
-  slug, status, parallelGroup: group, cwd: '/proj', ...extra,
-});
-
-describe('allocateParallelGroup extraFloor (PRD 832)', () => {
-  it('never allocates at or below the cross-dir max', async () => {
-    const dir = tmpdir();
-    // Bookkeeping dir itself is empty; another dir in the project already
-    // used numbers up to 830.
-    const nn = await prdParser.allocateParallelGroup(dir, { extraFloor: 830 });
-    expect(nn).toBe(831);
-    // And the high-water mark persists the raised floor for later calls.
-    const nn2 = await prdParser.allocateParallelGroup(dir);
-    expect(nn2).toBe(832);
-  });
-
-  it('local scan still wins when higher than extraFloor', async () => {
-    const dir = tmpdir();
-    fs.writeFileSync(path.join(dir, '900-existing.md'), '---\ntitle: x\n---\n# Goal\n');
-    const nn = await prdParser.allocateParallelGroup(dir, { extraFloor: 10 });
-    expect(nn).toBe(901);
-  });
-});
-
-describe('parsePrdRaw dependsOn (PRD 832)', () => {
-  it('parses bracket and comma forms, rejects junk', async () => {
-    const dir = tmpdir();
-    const fp = path.join(dir, '840-x.md');
-    fs.writeFileSync(fp, '---\ntitle: x\ncwd: /proj\ndependsOn: [833-epics-review-fixes, 834-guard]\n---\n# Goal\nbody\n');
-    const p = await prdParser.parsePrdRaw(fp);
-    expect(p.dependsOn).toEqual(['833-epics-review-fixes', '834-guard']);
-
-    const fp2 = path.join(dir, '841-y.md');
-    fs.writeFileSync(fp2, '---\ntitle: y\ncwd: /proj\ndependsOn: 833-a, ../evil/path, 834-b\n---\n# Goal\nbody\n');
-    const p2 = await prdParser.parsePrdRaw(fp2);
-    expect(p2.dependsOn).toEqual(['833-a', '834-b']);
-
-    const fp3 = path.join(dir, '842-z.md');
-    fs.writeFileSync(fp3, '---\ntitle: z\ncwd: /proj\n---\n# Goal\nbody\n');
-    const p3 = await prdParser.parsePrdRaw(fp3);
-    expect(p3.dependsOn).toEqual([]);
-  });
-});
-
-describe('pickForProject dependsOn eligibility (PRD 832)', () => {
-  it('holds a job while its dep row is pending/running; releases when completed', () => {
-    const dep = job('850-base', 'running', 850);
-    const dependent = job('851-next', 'pending', 851, { dependsOn: ['850-base'] });
-
-    // Dep running (tracked in queue rows, not runningSet) — not eligible.
-    let res = pickForProject([dep, dependent], new Set(), 3);
-    expect(res.batch.map((j) => j.slug)).not.toContain('851-next');
-
-    // Dep completed — eligible.
-    res = pickForProject([{ ...dep, status: 'completed' }, dependent], new Set(), 3);
-    expect(res.batch.map((j) => j.slug)).toContain('851-next');
-  });
-
-  it('a dep slug with NO queue row counts as done when found in history/archive (PRD 1122)', () => {
-    const dependent = job('853-next', 'pending', 853, { dependsOn: ['852-archived-long-ago'] });
-    // Caller (scheduler.cjs's computeDepHistorySatisfaction) precomputes this
-    // Set from history.jsonl/prds-archived — pickForProject never does I/O.
-    const satisfiedSlugs = new Set(['852-archived-long-ago']);
-    const res = pickForProject([dependent], new Set(), 3, new Map(), satisfiedSlugs);
-    expect(res.batch.map((j) => j.slug)).toContain('853-next');
-  });
-
-  it('a dep slug with NO queue row and NO history/archive record HOLDS as unresolved (PRD 1122)', () => {
-    const dependent = job('853-next', 'pending', 853, { dependsOn: ['852-archived-long-ago'] });
-    const res = pickForProject([dependent], new Set(), 3);
-    expect(res.batch).toEqual([]);
-    expect(res.reason).toMatch(/unresolvable/);
-    expect(res.reason).toContain('853-next');
-    expect(res.reason).toContain('852-archived-long-ago');
-  });
-
-  it('a FAILED dep holds the dependent with an explicit depends-gate reason', () => {
-    const dep = job('854-base', 'failed', 854);
-    const dependent = job('855-next', 'pending', 855, { dependsOn: ['854-base'] });
-    const res = pickForProject([dep, dependent], new Set(), 3);
-    expect(res.batch).toEqual([]);
-    expect(res.reason).toMatch(/depends-gate/);
-    expect(res.reason).toContain('855-next');
-    expect(res.reason).toContain('854-base');
-  });
-
-  // Contract change: parallelGroup no longer gates batch membership at all
-  // (it was a wave number; PRD 832 made it unique per PRD, which silently
-  // reduced every batch to one job). Legacy same-NN rows are simply eligible
-  // like anything else with no blocking dependsOn — the 828 row is no longer
-  // held back merely for carrying a higher number.
-  it('legacy same-NN pending jobs are eligible alongside higher-numbered ones', () => {
-    // Predates the per-project job cap (PRD 1066, default 2); raise it so
-    // this test keeps exercising parallelGroup semantics, not the cap.
-    const prevCap = process.env.SM_PROJECT_JOB_CAP;
-    process.env.SM_PROJECT_JOB_CAP = '10';
-    try {
-      const a = job('827-one', 'pending', 827);
-      const b = job('827-two', 'pending', 827);
-      const later = job('828-three', 'pending', 828);
-      const res = pickForProject([a, b, later], new Set(), 3);
-      expect(res.batch.map((j) => j.slug).sort()).toEqual(['827-one', '827-two', '828-three']);
-    } finally {
-      if (prevCap === undefined) delete process.env.SM_PROJECT_JOB_CAP;
-      else process.env.SM_PROJECT_JOB_CAP = prevCap;
-    }
-  });
-});
-
-describe('reconcile retires archived-twin pending rows (post-832 cleanup)', () => {
-  it('drops a pending row whose PRD lives in prds-archived, keeps a normal invisible pending row', async () => {
-    const scheduler = require('../scheduler.cjs');
-    const root = tmpdir();
-    const opsDir = path.join(root, 'session-manager-operations', 'scheduler');
-    fs.mkdirSync(path.join(opsDir, 'prds'), { recursive: true });
-    fs.mkdirSync(path.join(opsDir, 'prds-archived'), { recursive: true });
-    fs.writeFileSync(path.join(opsDir, 'prds-archived', '833-landed-inline.md'), '---\ntitle: x\n---\n# Goal\n');
-
-    const state = {
-      jobs: [
-        job('833-landed-inline', 'pending', 833, { cwd: root }),
-        job('900-still-hidden', 'pending', 900, { cwd: root }),
-      ],
-    };
-    await scheduler.reconcile(state);
-    const slugs = state.jobs.map((j) => j.slug);
-    expect(slugs).not.toContain('833-landed-inline');
-    expect(slugs).toContain('900-still-hidden');
-  });
-});

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

@@ -1,169 +0,0 @@
-/**
- * usageSingleFlight.test.cjs — usage.cjs's fetchUsage() is the single owner
- * of caching, single-flight de-duplication, and the shared usageCircuit for
- * the /api/oauth/usage meter. Before this, the renderer's billing:fetch IPC
- * handler owned the 30s OK-cache alone and the scheduler's pollLoop called
- * fetchUsage() directly with no cache/dedup — two independent request
- * streams against a rate-limiting endpoint, and a success on one path never
- * reset the other's failure streak. This suite proves: (1) concurrent
- * fetchUsage() callers coalesce into exactly one network fetch, (2) a
- * success resets the shared circuit's streak regardless of which "caller"
- * triggered the preceding failures, and (3) a 429's Retry-After suppresses
- * the next request until it elapses.
- *
- * Never hits the live billing endpoint — global.fetch is stubbed throughout,
- * and HOME is pointed at a scratch dir so no real credentials/cache file on
- * this machine is ever read.
- *
- * usage.cjs (and config.cjs/credentials.cjs, which it requires) are loaded
- * via Node's own require() — this is a .cjs test, not transformed ESM — so
- * vi.mock()/vi.resetModules() do NOT intercept them the way they would for
- * an ESM import graph; both also bake os.homedir() into module-level
- * constants at first require (config.cjs's allowedRoots/WRITE_PREFIXES,
- * usage.cjs's CACHE_PATH, credentials.cjs's CREDS_PATH). So this suite
- * follows historyRollup.test.cjs's pattern instead: swap $HOME to a fresh
- * tmp dir with a valid, far-from-expiry fake credentials file, purge
- * Node's require.cache for every affected module, and re-require fresh
- * each test.
- *
- * Run: timeout 120 npx vitest run src/main/__tests__/usageSingleFlight.test.cjs
- */
-
-'use strict';
-
-import { test, expect, vi, beforeEach, afterEach } from 'vitest';
-import fs from 'node:fs';
-import os from 'node:os';
-import path from 'node:path';
-
-function jsonResponse(status, body, headers = {}) {
-  return {
-    status,
-    ok: status >= 200 && status < 300,
-    text: async () => JSON.stringify(body),
-    json: async () => body,
-    headers: { get: (k) => headers[k.toLowerCase()] ?? null },
-  };
-}
-
-const usagePayload = { five_hour: { utilization: 10, resets_at: null }, seven_day: { utilization: 10, resets_at: null }, seven_day_sonnet: null, seven_day_opus: null, extra_usage: null };
-
-const MODULES_TO_RELOAD = [
-  '../usage.cjs',
-  '../config.cjs',
-  '../lib/credentials.cjs',
-  '../lib/usageCircuit.cjs',
-];
-
-function purgeRequireCache() {
-  for (const m of MODULES_TO_RELOAD) {
-    try { delete require.cache[require.resolve(m)]; } catch { /* not loaded yet */ }
-  }
-}
-
-let realHome;
-let tmpHome;
-
-beforeEach(() => {
-  realHome = process.env.HOME;
-  tmpHome = fs.mkdtempSync(path.join(os.tmpdir(), 'usage-single-flight-test-'));
-  process.env.HOME = tmpHome;
-  fs.mkdirSync(path.join(tmpHome, '.claude'), { recursive: true });
-  fs.writeFileSync(
-    path.join(tmpHome, '.claude', '.credentials.json'),
-    JSON.stringify({
-      claudeAiOauth: {
-        accessToken: 'test-access-token',
-        refreshToken: 'test-refresh-token',
-        expiresAt: Date.now() + 1_000 * 60 * 60 * 24, // 24h out — never "expiring soon"
-        subscriptionType: 'max',
-        rateLimitTier: 'default_claude_max_20x',
-      },
-    }),
-  );
-  purgeRequireCache();
-});
-
-afterEach(() => {
-  process.env.HOME = realHome;
-  vi.unstubAllGlobals();
-  vi.restoreAllMocks();
-  vi.useRealTimers();
-  purgeRequireCache();
-  // maxRetries: a fire-and-forget cache write inside usage.cjs can land in .claude/ mid-rm
-  // (ENOTEMPTY) when the machine is loaded; this is cleanup robustness, not a test retry.
-  fs.rmSync(tmpHome, { recursive: true, force: true, maxRetries: 10, retryDelay: 50 });
-});
-
-test('two concurrent fetchUsage() calls produce exactly one HTTP fetch', async () => {
-  const { fetchUsage } = require('../usage.cjs');
-  let fetchCalls = 0;
-  vi.stubGlobal('fetch', vi.fn(async () => {
-    fetchCalls++;
-    // Slow enough that both callers are guaranteed to be in-flight together.
-    await new Promise((resolve) => setTimeout(resolve, 20));
-    return jsonResponse(200, usagePayload);
-  }));
-
-  const [a, b] = await Promise.all([fetchUsage(), fetchUsage()]);
-
-  expect(fetchCalls).toBe(1);
-  expect(a.kind).toBe('ok');
-  expect(b.kind).toBe('ok');
-});
-
-test('an ok result resets the shared circuit streak built up by prior failures', async () => {
-  // The breaker's open->half_open transition (usageCircuit.cjs's state())
-  // compares real Date.now() against a jittered 15-30s backoff deadline. On
-  // a loaded machine, real wall-clock time can drift far enough between the
-  // 3rd failure and the assertions below for that deadline to pass, flipping
-  // 'open' to 'half_open' out from under this test. Freeze time so the
-  // open/closed assertions are decided by call sequence, not the real clock.
-  vi.useFakeTimers();
-  vi.setSystemTime(Date.now());
-
-  const usage = require('../usage.cjs');
-  const circuit = usage.__usageCircuitForTest;
-
-  let call = 0;
-  vi.stubGlobal('fetch', vi.fn(async () => {
-    call++;
-    if (call <= 3) return jsonResponse(500, { error: 'boom' });
-    return jsonResponse(200, usagePayload);
-  }));
-
-  // Three consecutive transient failures — simulating the scheduler's poller
-  // failing repeatedly — trip the circuit open.
-  await usage.fetchUsage();
-  await usage.fetchUsage();
-  await usage.fetchUsage();
-  expect(circuit.state()).toBe('open');
-
-  // A later success — as if it came from the renderer's independent
-  // billing:fetch call, but reached through the SAME shared fetchUsage() —
-  // must reset the streak the scheduler's calls built up, since both
-  // callers share one circuit instance.
-  const ok = await usage.fetchUsage();
-  expect(ok.kind).toBe('ok');
-  expect(circuit.state()).toBe('closed');
-});
-
-test('a 429 with Retry-After suppresses the next request until it elapses', async () => {
-  const usage = require('../usage.cjs');
-
-  let fetchCalls = 0;
-  vi.stubGlobal('fetch', vi.fn(async () => {
-    fetchCalls++;
-    return jsonResponse(429, { error: { type: 'rate_limit_error' } }, { 'retry-after': '60' });
-  }));
-
-  const first = await usage.fetchUsage();
-  expect(first.kind).toBe('meter_rate_limited');
-  expect(first.retryAfterMs).toBe(60_000);
-
-  const second = await usage.fetchUsage();
-  expect(second.kind).toBe('meter_rate_limited');
-  expect(second.suppressed).toBe(true);
-  // No second network call — the suppression window hasn't elapsed.
-  expect(fetchCalls).toBe(1);
-});

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

@@ -1,84 +0,0 @@
-/**
- * validationSentinels.test.cjs — unit tests for parseValidationSentinels
- * (PRD 1407): a plan-level validator job ends its run with one
- * `VALIDATION: <slug> VERIFIED` or `VALIDATION: <slug> REFUTED — <reason>`
- * line per PRD; this module extracts those lines from the job's result text.
- *
- * Run: timeout 300 npx vitest run src/main/__tests__/validationSentinels.test.cjs
- */
-
-'use strict';
-
-import { test, expect } from 'vitest';
-const { parseValidationSentinels } = require('../lib/validationSentinels.cjs');
-
-test('parses a VERIFIED line with no reason', () => {
-  expect(parseValidationSentinels('VALIDATION: 123-foo VERIFIED')).toEqual([
-    { slug: '123-foo', verdict: 'verified', reason: '' },
-  ]);
-});
-
-test('parses a REFUTED line with an em-dash reason', () => {
-  expect(parseValidationSentinels('VALIDATION: 123-foo REFUTED — missing test coverage')).toEqual([
-    { slug: '123-foo', verdict: 'refuted', reason: 'missing test coverage' },
-  ]);
-});
-
-test('parses a REFUTED line with an en-dash reason', () => {
-  expect(parseValidationSentinels('VALIDATION: 123-foo REFUTED – bad wiring')).toEqual([
-    { slug: '123-foo', verdict: 'refuted', reason: 'bad wiring' },
-  ]);
-});
-
-test('parses a REFUTED line with a hyphen reason', () => {
-  expect(parseValidationSentinels('VALIDATION: 123-foo REFUTED - wrong file touched')).toEqual([
-    { slug: '123-foo', verdict: 'refuted', reason: 'wrong file touched' },
-  ]);
-});
-
-test('parses a REFUTED line with no reason', () => {
-  expect(parseValidationSentinels('VALIDATION: 123-foo REFUTED')).toEqual([
-    { slug: '123-foo', verdict: 'refuted', reason: '' },
-  ]);
-});
-
-test('ignores lowercase verified/refuted', () => {
-  expect(parseValidationSentinels('VALIDATION: 123-foo verified')).toEqual([]);
-  expect(parseValidationSentinels('VALIDATION: 123-foo refuted')).toEqual([]);
-});
-
-test('ignores every other line in a multi-line transcript', () => {
-  const text = [
-    'Some preamble text',
-    'Running gate...',
-    'VALIDATION: 111-alpha VERIFIED',
-    'random noise mentioning VALIDATION: not-a-real-line',
-    'VALIDATION: 222-beta REFUTED — the fix was incomplete',
-    'trailing notes',
-  ].join('\n');
-
-  expect(parseValidationSentinels(text)).toEqual([
-    { slug: '111-alpha', verdict: 'verified', reason: '' },
-    { slug: '222-beta', verdict: 'refuted', reason: 'the fix was incomplete' },
-  ]);
-});
-
-test('last line per slug wins, order of first appearance kept', () => {
-  const text = [
-    'VALIDATION: 111-alpha VERIFIED',
-    'VALIDATION: 222-beta REFUTED — first reason',
-    'VALIDATION: 111-alpha REFUTED — actually broken',
-    'VALIDATION: 222-beta VERIFIED',
-  ].join('\n');
-
-  expect(parseValidationSentinels(text)).toEqual([
-    { slug: '111-alpha', verdict: 'refuted', reason: 'actually broken' },
-    { slug: '222-beta', verdict: 'verified', reason: '' },
-  ]);
-});
-
-test('returns [] for null, undefined, and empty input', () => {
-  expect(parseValidationSentinels(null)).toEqual([]);
-  expect(parseValidationSentinels(undefined)).toEqual([]);
-  expect(parseValidationSentinels('')).toEqual([]);
-});

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

@@ -1,89 +0,0 @@
-/**
- * workTypeLibrary.test.cjs — cross-file consistency tests for the WorkType
- * union. Epic tag and PRD tag are one concept (WorkType); this test enforces
- * that every declaration site agrees, since the two module systems involved
- * (renderer .ts, main .cjs, and the preload .d.ts that can import neither)
- * can't enforce it via a shared import. Modelled on scheduleJobSchema.test.cjs's
- * JOB_STATUSES cross-file assertions.
- *
- * Run: timeout 300 npx vitest run src/main/__tests__/workTypeLibrary.test.cjs
- */
-
-'use strict';
-
-import { test, expect } from 'vitest';
-const fs = require('node:fs');
-const path = require('node:path');
-const { WORK_TYPES, WorkTypeSchema } = require('../lib/workTypeLibrary.cjs');
-
-const REPO_ROOT = path.resolve(__dirname, '../../..');
-const TAG_LIBRARY_PATH = path.join(REPO_ROOT, 'src/renderer/lib/tagLibrary.ts');
-const API_D_TS_PATH = path.join(REPO_ROOT, 'src/preload/api.d.ts');
-const IPC_SCHEMAS_PATH = path.join(REPO_ROOT, 'src/main/ipcSchemas.cjs');
-const SCHEDULER_MCP_SERVER_PATH = path.join(REPO_ROOT, 'scripts/scheduler-mcp-server.cjs');
-
-test('WORK_TYPES matches the expected 6 canonical ids in order', () => {
-  expect(WORK_TYPES).toEqual([
-    'feature',
-    'bug',
-    'discussion',
-    'build',
-    'project-home-builder',
-    'bilko-host-publisher',
-  ]);
-});
-
-test('WorkTypeSchema accepts every WORK_TYPES value and rejects an unknown one', () => {
-  for (const value of WORK_TYPES) {
-    expect(WorkTypeSchema.safeParse(value).success).toBe(true);
-  }
-  expect(WorkTypeSchema.safeParse('not-a-real-tag').success).toBe(false);
-});
-
-test('WORK_TYPES matches the tag ids declared in tagLibrary.ts TAG_LIBRARY, in order', () => {
-  const src = fs.readFileSync(TAG_LIBRARY_PATH, 'utf8');
-  const tagMatches = [...src.matchAll(/^\s*tag:\s*'([^']+)',/gm)].map((m) => m[1]);
-  expect(tagMatches).toEqual(WORK_TYPES);
-});
-
-test('api.d.ts tag unions both contain exactly the same 6 values', () => {
-  const src = fs.readFileSync(API_D_TS_PATH, 'utf8');
-  const unionMatches = [...src.matchAll(/tag\?:\s*((?:'[^']+'\s*\|?\s*)+);/g)].map((m) =>
-    m[1]
-      .split('|')
-      .map((s) => s.trim().replace(/^'|'$/g, ''))
-      .filter(Boolean)
-  );
-
-  expect(unionMatches.length).toBeGreaterThanOrEqual(2);
-  for (const union of unionMatches) {
-    expect(union).toEqual(WORK_TYPES);
-  }
-});
-
-test('ipcSchemas.cjs no longer hardcodes the full-taxonomy tag z.enum literal', () => {
-  const src = fs.readFileSync(IPC_SCHEMAS_PATH, 'utf8');
-  // The old scheduleRetagPrd literal was missing 'bilko-host-publisher'
-  // (5 of the 6 WORK_TYPES values) — assert that exact stale shape is gone.
-  expect(src).not.toMatch(/z\.enum\(\['feature',\s*'bug',\s*'discussion',\s*'build',\s*'project-home-builder'\]\)/);
-  // Epic tag (schedulerCreateEpic-style routes) draws on the full 6-value
-  // WorkType union via EpicTagSchema, which is itself WorkTypeSchema
-  // (promptSessionSchema.cjs). PRD tag fields (schedulerCreatePrd,
-  // adminPrdFrontmatterPatch) draw on the narrower 5-value PrdWorkTypeSchema
-  // — independent values, same shared vocabulary (PRD 1041).
-  expect(src).toMatch(/tag:\s*EpicTagSchema/);
-  const prdTagMatches = [...src.matchAll(/tag:\s*PrdWorkTypeSchema/g)];
-  expect(prdTagMatches.length).toBeGreaterThanOrEqual(2);
-});
-
-test('scheduler-mcp-server.cjs requires PRD_WORK_TYPES rather than hardcoding a tag enum literal, and imports it exactly twice (scheduler_create_prd + scheduler_update_prd)', () => {
-  const { PRD_WORK_TYPES } = require('../lib/workTypeLibrary.cjs');
-  const src = fs.readFileSync(SCHEDULER_MCP_SERVER_PATH, 'utf8');
-  expect(src).toMatch(/require\(['"].*workTypeLibrary\.cjs['"]\)/);
-  const occurrences = [...src.matchAll(/enum:\s*PRD_WORK_TYPES/g)].length;
-  // scheduler_create_prd's inputSchema.tag.enum and scheduler_update_prd's
-  // frontmatter.tag.enum — exactly 2 sites, not the Epic-tag (6-value)
-  // enum on feedback_open_session's tag property.
-  expect(occurrences).toBe(2);
-  expect(PRD_WORK_TYPES).toEqual(['feature', 'bug', 'build', 'project-home-builder', 'bilko-host-publisher']);
-});