claude-code-session-manager 0.98.0 → 0.101.0

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

@@ -1,74 +0,0 @@
-/**
- * prdParserHighWater.test.cjs — regression test for allocateParallelGroup's
- * high-water mark: once queueOps.cjs's autoArchiveCompleted starts moving
- * old NN-*.md files out of prds/ into prds-archived/, a bare directory scan
- * (maxParallelGroupInUse) can regress once the highest-NN files age out —
- * letting a later allocation reissue an already-used NN and collide with an
- * archived group. The high-water-mark sidecar (.max-allocated-group) must
- * keep the allocator monotonic even after every file is archived away.
- *
- * Run: timeout 300 npx vitest run src/main/__tests__/prdParserHighWater.test.cjs
- */
-
-'use strict';
-
-import { test, expect } from 'vitest';
-const fsp = require('node:fs/promises');
-const os = require('node:os');
-const path = require('node:path');
-
-const { allocateParallelGroup, maxParallelGroupInUse, _resetCache } = require('../scheduler/prdParser.cjs');
-
-function mkTmpPrdsDir() {
-  return fsp.mkdtemp(path.join(os.tmpdir(), 'sm-prd-highwater-'));
-}
-
-test('allocator stays monotonic after every NN-*.md file (and its reservation marker) is archived away', async () => {
-  const dir = await mkTmpPrdsDir();
-  _resetCache();
-
-  // Directly-authored legacy files (never allocated via allocateParallelGroup
-  // — e.g. /develop's deliberate-sibling path — so no .reserved-NN marker
-  // exists for them). This is the real gap: reservation markers alone would
-  // already protect any NN the allocator itself minted, so the regression
-  // this PRD guards against is specifically legacy/directly-authored NNs.
-  await fsp.writeFile(path.join(dir, '10-a.md'), '# a\n');
-  await fsp.writeFile(path.join(dir, '544-b.md'), '# b\n');
-
-  // A scan (via allocateParallelGroup) observes 544 and persists it as the
-  // high-water mark before returning the next free NN.
-  const first = await allocateParallelGroup(dir);
-  expect(first).toBe(545);
-
-  // Simulate autoArchiveCompleted moving every .md out of the live dir, and
-  // also strip the reservation marker the allocation above created — so
-  // nothing on disk still evidences 544/545 except the high-water sidecar.
-  await fsp.unlink(path.join(dir, '10-a.md'));
-  await fsp.unlink(path.join(dir, '544-b.md'));
-  await fsp.unlink(path.join(dir, '.reserved-545'));
-
-  const bareScanMax = await maxParallelGroupInUse(dir);
-  expect(bareScanMax).toBe(0);
-
-  _resetCache();
-  const second = await allocateParallelGroup(dir);
-  expect(second).toBeGreaterThan(545);
-  expect(second).toBe(546);
-});
-
-test('high-water mark also protects against a fresh scan that is lower than a prior allocation, without any archiving involved', async () => {
-  const dir = await mkTmpPrdsDir();
-  _resetCache();
-
-  await fsp.writeFile(path.join(dir, '5-only.md'), '# x\n');
-  const nn = await allocateParallelGroup(dir);
-  expect(nn).toBe(6);
-
-  // A slow/partial directory listing (or a caller passing a differently
-  // scoped dir) must not be able to walk the allocator backwards.
-  const rescanMax = await maxParallelGroupInUse(dir);
-  expect(rescanMax).toBe(6); // the .reserved-6 marker from the allocation above still counts
-
-  const next = await allocateParallelGroup(dir);
-  expect(next).toBe(7);
-});

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

@@ -1,65 +0,0 @@
-/**
- * prdParserSourcePromptId.test.cjs — parsePrdRaw() tolerates both presence
- * and absence of the optional `sourcePromptId` frontmatter key (PRD 749).
- * Additive-only: PRDs written before this field existed must still parse
- * identically (no required-field regression).
- *
- * Run: timeout 120 npx vitest run src/main/__tests__/prdParserSourcePromptId.test.cjs
- */
-
-'use strict';
-
-import { test, expect } from 'vitest';
-const fsp = require('node:fs/promises');
-const os = require('node:os');
-const path = require('node:path');
-
-const { parsePrdRaw, _resetCache } = require('../scheduler/prdParser.cjs');
-
-function mkTmpPrdsDir() {
-  return fsp.mkdtemp(path.join(os.tmpdir(), 'sm-prd-sourceprompt-'));
-}
-
-test('parsePrdRaw extracts sourcePromptId when present in frontmatter', async () => {
-  const dir = await mkTmpPrdsDir();
-  _resetCache();
-  const filePath = path.join(dir, '1-with-source.md');
-  await fsp.writeFile(filePath, [
-    '---',
-    'title: Has a source ticket',
-    'cwd: ~/Projects/session-manager',
-    'estimateMinutes: 10',
-    'sourcePromptId: ticket-xyz-789',
-    '---',
-    '# Goal',
-    '',
-    'Do the thing.',
-    '',
-  ].join('\n'));
-
-  const parsed = await parsePrdRaw(filePath);
-  expect(parsed.sourcePromptId).toBe('ticket-xyz-789');
-  expect(parsed.title).toBe('Has a source ticket');
-});
-
-test('parsePrdRaw returns null sourcePromptId for a PRD authored before this field existed', async () => {
-  const dir = await mkTmpPrdsDir();
-  _resetCache();
-  const filePath = path.join(dir, '2-legacy.md');
-  await fsp.writeFile(filePath, [
-    '---',
-    'title: Legacy PRD, no sourcePromptId',
-    'cwd: ~/Projects/session-manager',
-    'estimateMinutes: 5',
-    '---',
-    '# Goal',
-    '',
-    'Do the other thing.',
-    '',
-  ].join('\n'));
-
-  const parsed = await parsePrdRaw(filePath);
-  expect(parsed.sourcePromptId).toBe(null);
-  expect(parsed.title).toBe('Legacy PRD, no sourcePromptId');
-  expect(parsed.cwd).toBe(path.join(os.homedir(), 'Projects', 'session-manager'));
-});

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

@@ -1,222 +0,0 @@
-/**
- * prdSetDisposition.test.cjs — scheduler wave-disposition PRD:
- * scheduler.remote.setPrdDisposition (backing the renderer's
- * `schedule:set-prd-disposition` IPC action) is how a human changes an
- * already-written PRD's disposition from the Scheduler UI — promoting an
- * appended wave to its own head, or re-attaching a head behind another
- * chain. Validates via prdDisposition.cjs's computeDispositionRewrite
- * (cycle-safety, running/completed rows untouched) before delegating the
- * actual write to the SAME remote.updatePrd every other PRD edit path uses.
- *
- * Mirrors prdUpdateDependsOn.test.cjs's HOME-isolation fixture.
- *
- * Run: timeout 120 npx vitest run src/main/__tests__/prdSetDisposition.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 queueStore;
-let config;
-
-beforeAll(() => {
-  originalHome = process.env.HOME;
-  tmpHome = fs.mkdtempSync(path.join(os.tmpdir(), 'sm-set-disposition-home-'));
-  process.env.HOME = tmpHome;
-
-  scheduler = require('../scheduler.cjs');
-  queueStore = require('../lib/queueStore.cjs');
-  config = require('../config.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');
-}
-
-function makeFixtureProject(prefix) {
-  const cwd = fs.mkdtempSync(path.join(os.tmpdir(), prefix));
-  registerActiveProject(cwd);
-  config.addAllowedRoot(cwd);
-  const opsRoot = path.join(cwd, 'session-manager-operations');
-  const prdsDir = path.join(opsRoot, 'scheduler', 'epics', 'test-epic-1', 'prds');
-  const stateDir = path.join(opsRoot, 'scheduler', 'state');
-  fs.mkdirSync(prdsDir, { recursive: true });
-  fs.mkdirSync(stateDir, { recursive: true });
-  fs.writeFileSync(path.join(stateDir, 'queue.json'), JSON.stringify({ jobs: [] }, null, 2), 'utf8');
-  queueStore.bustCwdCache();
-  return { cwd, prdsDir, stateDir };
-}
-
-function writePrd(prdsDir, filename, cwd, extraFrontmatter = '') {
-  fs.writeFileSync(
-    path.join(prdsDir, filename),
-    `---\ntitle: ${filename}\ncwd: ${cwd}\nestimateMinutes: 10\ncreatedVia: scheduler-api\n${extraFrontmatter}---\n\n# Goal\nDo the thing.\n`,
-    'utf8',
-  );
-}
-
-/** Seeds a queue.json row for `slug` with the given status — setPrdDisposition
- *  (like updatePrd) reads job status off the live queue, not the PRD file. */
-function seedQueueJob(stateDir, slug, status) {
-  fs.writeFileSync(
-    path.join(stateDir, 'queue.json'),
-    JSON.stringify({ jobs: [{ slug, status, title: slug, cwd: '' }] }, null, 2),
-    'utf8',
-  );
-  queueStore.bustCwdCache();
-}
-
-test('append sets the expected edges', async () => {
-  const { cwd, prdsDir } = makeFixtureProject('sm-set-disposition-append-');
-  try {
-    writePrd(prdsDir, '3-wave1-tail.md', cwd);
-    writePrd(prdsDir, '4-wave2-root.md', cwd);
-
-    const result = await scheduler.remote.setPrdDisposition({
-      slug: '4-wave2-root',
-      cwd,
-      disposition: 'append',
-      dependsOn: ['3-wave1-tail'],
-    });
-
-    expect(result.ok).toBe(true);
-    const written = fs.readFileSync(path.join(prdsDir, '4-wave2-root.md'), 'utf8');
-    expect(written).toMatch(/^dependsOn: \[3-wave1-tail\]$/m);
-    expect(written).toMatch(/^disposition: append$/m);
-  } finally {
-    fs.rmSync(cwd, { recursive: true, force: true });
-  }
-});
-
-test('new-head sets none', async () => {
-  const { cwd, prdsDir } = makeFixtureProject('sm-set-disposition-new-head-');
-  try {
-    writePrd(prdsDir, '3-wave1-tail.md', cwd);
-    writePrd(prdsDir, '4-wave2-root.md', cwd, 'dependsOn: [wave1-tail]\ndisposition: append\n');
-
-    const result = await scheduler.remote.setPrdDisposition({
-      slug: '4-wave2-root',
-      cwd,
-      disposition: 'new-head',
-    });
-
-    expect(result.ok).toBe(true);
-    const written = fs.readFileSync(path.join(prdsDir, '4-wave2-root.md'), 'utf8');
-    expect(written).not.toContain('dependsOn');
-    expect(written).toMatch(/^disposition: new-head$/m);
-  } finally {
-    fs.rmSync(cwd, { recursive: true, force: true });
-  }
-});
-
-test('promoting a wave to a head removes exactly its root edges', async () => {
-  const { cwd, prdsDir } = makeFixtureProject('sm-set-disposition-promote-');
-  try {
-    writePrd(prdsDir, '3-wave1-tail.md', cwd);
-    writePrd(prdsDir, '5-other-head.md', cwd);
-    writePrd(prdsDir, '4-wave2-root.md', cwd, 'dependsOn: [wave1-tail]\ndisposition: append\n');
-
-    const result = await scheduler.remote.setPrdDisposition({
-      slug: '4-wave2-root',
-      cwd,
-      disposition: 'new-head',
-    });
-
-    expect(result.ok).toBe(true);
-    const promoted = fs.readFileSync(path.join(prdsDir, '4-wave2-root.md'), 'utf8');
-    expect(promoted).not.toContain('dependsOn');
-    // The unrelated sibling head is untouched by this call.
-    const other = fs.readFileSync(path.join(prdsDir, '5-other-head.md'), 'utf8');
-    expect(other).not.toContain('dependsOn');
-  } finally {
-    fs.rmSync(cwd, { recursive: true, force: true });
-  }
-});
-
-test('a change that would create a cycle is rejected, and nothing is written', async () => {
-  const { cwd, prdsDir } = makeFixtureProject('sm-set-disposition-cycle-');
-  try {
-    writePrd(prdsDir, '3-a.md', cwd, 'dependsOn: [b]\n');
-    writePrd(prdsDir, '4-b.md', cwd);
-    const before = fs.readFileSync(path.join(prdsDir, '4-b.md'), 'utf8');
-
-    const result = await scheduler.remote.setPrdDisposition({
-      slug: '4-b',
-      cwd,
-      disposition: 'append',
-      dependsOn: ['3-a'],
-    });
-
-    expect(result.ok).toBe(false);
-    expect(result.error).toMatch(/cycle/i);
-    const after = fs.readFileSync(path.join(prdsDir, '4-b.md'), 'utf8');
-    expect(after).toBe(before);
-  } finally {
-    fs.rmSync(cwd, { recursive: true, force: true });
-  }
-});
-
-test('running rows are untouched', async () => {
-  const { cwd, prdsDir, stateDir } = makeFixtureProject('sm-set-disposition-running-');
-  try {
-    writePrd(prdsDir, '3-wave1-tail.md', cwd);
-    writePrd(prdsDir, '4-wave2-root.md', cwd, 'dependsOn: [wave1-tail]\ndisposition: append\n');
-    seedQueueJob(stateDir, '4-wave2-root', 'running');
-    const before = fs.readFileSync(path.join(prdsDir, '4-wave2-root.md'), 'utf8');
-
-    const result = await scheduler.remote.setPrdDisposition({
-      slug: '4-wave2-root',
-      cwd,
-      disposition: 'new-head',
-    });
-
-    expect(result.ok).toBe(false);
-    expect(result.error).toMatch(/running/i);
-    const after = fs.readFileSync(path.join(prdsDir, '4-wave2-root.md'), 'utf8');
-    expect(after).toBe(before);
-  } finally {
-    fs.rmSync(cwd, { recursive: true, force: true });
-  }
-});
-
-test('completed rows are untouched', async () => {
-  const { cwd, prdsDir, stateDir } = makeFixtureProject('sm-set-disposition-completed-');
-  try {
-    writePrd(prdsDir, '3-wave1-tail.md', cwd);
-    writePrd(prdsDir, '4-wave2-root.md', cwd, 'dependsOn: [wave1-tail]\ndisposition: append\n');
-    seedQueueJob(stateDir, '4-wave2-root', 'completed');
-    const before = fs.readFileSync(path.join(prdsDir, '4-wave2-root.md'), 'utf8');
-
-    const result = await scheduler.remote.setPrdDisposition({
-      slug: '4-wave2-root',
-      cwd,
-      disposition: 'new-head',
-    });
-
-    expect(result.ok).toBe(false);
-    expect(result.error).toMatch(/completed/i);
-    const after = fs.readFileSync(path.join(prdsDir, '4-wave2-root.md'), 'utf8');
-    expect(after).toBe(before);
-  } finally {
-    fs.rmSync(cwd, { recursive: true, force: true });
-  }
-});

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

@@ -1,106 +0,0 @@
-/**
- * prdSizing.test.cjs — unit tests for sizingWarnings (PRD 1403).
- *
- * Run: timeout 120 npx vitest run src/main/__tests__/prdSizing.test.cjs
- */
-
-'use strict';
-
-import { test, expect } from 'vitest';
-const { SIZING_LIMITS, sizingWarnings } = require('../lib/prdSizing.cjs');
-
-function compliantInput(overrides = {}) {
-  return {
-    estimateMinutes: 10,
-    acceptanceCriteria: ['widget frobnicates on click', 'unit test covers the frobnication path'],
-    goal: 'Add frobnication to the widget subsystem.',
-    implementationNotes: 'See src/widget.cjs:10.',
-    ...overrides,
-  };
-}
-
-test('compliant PRD yields no warnings', () => {
-  expect(sizingWarnings(compliantInput())).toEqual([]);
-});
-
-test('warns when estimateMinutes exceeds the limit', () => {
-  const warnings = sizingWarnings(compliantInput({ estimateMinutes: SIZING_LIMITS.estimateMinutes + 1 }));
-  expect(warnings.some((w) => /estimateMinutes/.test(w))).toBe(true);
-});
-
-test('does not warn when estimateMinutes is exactly the limit', () => {
-  const warnings = sizingWarnings(compliantInput({ estimateMinutes: SIZING_LIMITS.estimateMinutes }));
-  expect(warnings.some((w) => /estimateMinutes/.test(w))).toBe(false);
-});
-
-test('warns when acceptanceCriteria has more than the limit of lines', () => {
-  const acceptanceCriteria = Array.from({ length: SIZING_LIMITS.acLines + 1 }, (_, i) => `criterion ${i}`);
-  const warnings = sizingWarnings(compliantInput({ acceptanceCriteria }));
-  expect(warnings.some((w) => /acceptanceCriteria has/.test(w))).toBe(true);
-});
-
-test('does not warn when acceptanceCriteria has exactly the limit of lines', () => {
-  const acceptanceCriteria = Array.from({ length: SIZING_LIMITS.acLines }, (_, i) => `criterion ${i}`);
-  const warnings = sizingWarnings(compliantInput({ acceptanceCriteria }));
-  expect(warnings.some((w) => /acceptanceCriteria has/.test(w))).toBe(false);
-});
-
-test('warns when an AC line exceeds the per-line character limit', () => {
-  const longLine = 'x'.repeat(SIZING_LIMITS.acLineChars + 1);
-  const warnings = sizingWarnings(compliantInput({ acceptanceCriteria: [longLine] }));
-  expect(warnings.some((w) => /exceed \d+ characters/.test(w))).toBe(true);
-});
-
-test('does not warn when an AC line is exactly at the per-line character limit', () => {
-  const line = 'x'.repeat(SIZING_LIMITS.acLineChars);
-  const warnings = sizingWarnings(compliantInput({ acceptanceCriteria: [line] }));
-  expect(warnings.some((w) => /exceed \d+ characters/.test(w))).toBe(false);
-});
-
-test('warns on an open-ended search-and-fix AC line', () => {
-  const warnings = sizingWarnings(compliantInput({
-    acceptanceCriteria: ['grep e2e specs and update expectations to match the new sort order'],
-  }));
-  expect(warnings.some((w) => /open-ended search-and-fix/.test(w))).toBe(true);
-});
-
-test('does not warn on an AC line that merely mentions grep or update in isolation', () => {
-  const warnings = sizingWarnings(compliantInput({
-    acceptanceCriteria: ['grep confirms no leftover references to the old flag'],
-  }));
-  expect(warnings.some((w) => /open-ended search-and-fix/.test(w))).toBe(false);
-});
-
-test('warns on an open-ended search-and-fix AC line whose "then"/"update" half is a second sentence', () => {
-  const warnings = sizingWarnings(compliantInput({
-    acceptanceCriteria: ['Grep the repo for every usage of the old flag. Then update each call site.'],
-  }));
-  expect(warnings.some((w) => /open-ended search-and-fix/.test(w))).toBe(true);
-});
-
-test('warns on an AC line with two or more timeout commands', () => {
-  const warnings = sizingWarnings(compliantInput({
-    acceptanceCriteria: ['timeout 120 npm run typecheck && timeout 300 npm run test:unit both pass'],
-  }));
-  expect(warnings.some((w) => /two or more `timeout/.test(w))).toBe(true);
-});
-
-test('does not warn on an AC line with a single timeout command', () => {
-  const warnings = sizingWarnings(compliantInput({
-    acceptanceCriteria: ['timeout 120 npm run typecheck passes'],
-  }));
-  expect(warnings.some((w) => /two or more `timeout/.test(w))).toBe(false);
-});
-
-test('warns when goal + implementationNotes + joined AC exceeds the body character limit', () => {
-  const warnings = sizingWarnings(compliantInput({
-    goal: 'g'.repeat(SIZING_LIMITS.bodyChars + 1),
-  }));
-  expect(warnings.some((w) => /exceeding the \d+-character sizing limit/.test(w))).toBe(true);
-});
-
-test('does not warn when the combined body is exactly at the character limit', () => {
-  const goal = 'g'.repeat(SIZING_LIMITS.bodyChars);
-  const warnings = sizingWarnings({ estimateMinutes: 10, acceptanceCriteria: [], goal, implementationNotes: '' });
-  expect(warnings.some((w) => /exceeding the \d+-character sizing limit/.test(w))).toBe(false);
-});

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

@@ -1,118 +0,0 @@
-/**
- * prdSourcePromptIdBackfill.test.cjs — PRD 830: a PRD file's location under
- * `.../scheduler/epics/<epicId>/prds/<slug>.md` already IS its Epic
- * membership, so ingest should stamp `sourcePromptId = <epicId>` whenever
- * frontmatter doesn't supply one — instead of leaving it null forever.
- *
- * Covers:
- *   - deriveEpicIdFromPrdPath: pure path-shape extraction (prdLocations.cjs)
- *   - parsePrdRaw: dir-derived stamping + frontmatter-wins precedence
- *   - reconcileSourcePromptId: pending-only backfill (scheduler.cjs)
- *
- * Run: timeout 120 npx vitest run src/main/__tests__/prdSourcePromptIdBackfill.test.cjs
- */
-
-'use strict';
-
-import { test, expect } from 'vitest';
-const fsp = require('node:fs/promises');
-const os = require('node:os');
-const path = require('node:path');
-
-const { deriveEpicIdFromPrdPath, resolveEpicPrdWriteDir } = require('../lib/prdLocations.cjs');
-const { parsePrdRaw, _resetCache } = require('../scheduler/prdParser.cjs');
-const { reconcileSourcePromptId } = require('../scheduler.cjs');
-
-async function mkProjectCwd() {
-  return fsp.mkdtemp(path.join(os.tmpdir(), 'sm-prd-backfill-'));
-}
-
-// ---------- deriveEpicIdFromPrdPath ----------
-
-test('deriveEpicIdFromPrdPath extracts the epic id from a well-formed epic PRD path', async () => {
-  const cwd = await mkProjectCwd();
-  const epicId = 'some-epic-abc123';
-  const dir = resolveEpicPrdWriteDir(cwd, epicId);
-  await fsp.mkdir(dir, { recursive: true });
-  const filePath = path.join(dir, '900-example.md');
-  expect(deriveEpicIdFromPrdPath(filePath)).toBe(epicId);
-});
-
-test('deriveEpicIdFromPrdPath returns null for the retired flat prds/ layout', () => {
-  const filePath = '/home/user/Projects/foo/session-manager-operations/scheduler/prds/900-example.md';
-  expect(deriveEpicIdFromPrdPath(filePath)).toBe(null);
-});
-
-test('deriveEpicIdFromPrdPath returns null for an unrelated path', () => {
-  expect(deriveEpicIdFromPrdPath('/tmp/whatever/900-example.md')).toBe(null);
-  expect(deriveEpicIdFromPrdPath(null)).toBe(null);
-});
-
-// ---------- parsePrdRaw: dir-derived stamping + frontmatter-wins ----------
-
-test('parsePrdRaw derives sourcePromptId from the parent epic dir when frontmatter omits it', async () => {
-  _resetCache();
-  const cwd = await mkProjectCwd();
-  const epicId = 'derive-me-epic-def456';
-  const dir = resolveEpicPrdWriteDir(cwd, epicId);
-  await fsp.mkdir(dir, { recursive: true });
-  const filePath = path.join(dir, '901-hand-authored.md');
-  await fsp.writeFile(filePath, [
-    '---',
-    'title: Hand-authored, no sourcePromptId',
-    `cwd: ${cwd}`,
-    '---',
-    '# Goal',
-    '',
-    'Do the thing.',
-    '',
-  ].join('\n'));
-
-  const parsed = await parsePrdRaw(filePath);
-  expect(parsed.sourcePromptId).toBe(epicId);
-});
-
-test('parsePrdRaw: explicit frontmatter sourcePromptId wins over the dir-derived epic id', async () => {
-  _resetCache();
-  const cwd = await mkProjectCwd();
-  const epicId = 'dir-epic-ghi789';
-  const dir = resolveEpicPrdWriteDir(cwd, epicId);
-  await fsp.mkdir(dir, { recursive: true });
-  const filePath = path.join(dir, '902-explicit.md');
-  await fsp.writeFile(filePath, [
-    '---',
-    'title: Explicit sourcePromptId',
-    `cwd: ${cwd}`,
-    'sourcePromptId: ticket-explicit-999',
-    '---',
-    '# Goal',
-    '',
-    'Do the thing.',
-    '',
-  ].join('\n'));
-
-  const parsed = await parsePrdRaw(filePath);
-  expect(parsed.sourcePromptId).toBe('ticket-explicit-999');
-});
-
-// ---------- reconcileSourcePromptId: pending-only backfill ----------
-
-test('reconcileSourcePromptId backfills a pending job whose sourcePromptId is null', () => {
-  const job = { slug: 'x', status: 'pending', sourcePromptId: null };
-  expect(reconcileSourcePromptId(job, 'epic-abc')).toBe('epic-abc');
-});
-
-test('reconcileSourcePromptId leaves a running job untouched even if its sourcePromptId is null', () => {
-  const job = { slug: 'x', status: 'running', sourcePromptId: null };
-  expect(reconcileSourcePromptId(job, 'epic-abc')).toBe(null);
-});
-
-test('reconcileSourcePromptId leaves a completed job untouched even if its sourcePromptId is null', () => {
-  const job = { slug: 'x', status: 'completed', sourcePromptId: null };
-  expect(reconcileSourcePromptId(job, 'epic-abc')).toBe(null);
-});
-
-test('reconcileSourcePromptId keeps a pending job\'s existing value if the fresh parse yields null', () => {
-  const job = { slug: 'x', status: 'pending', sourcePromptId: 'already-set' };
-  expect(reconcileSourcePromptId(job, null)).toBe('already-set');
-});