claude-code-session-manager 0.97.0 → 0.100.0

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');
-});

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

@@ -1,160 +0,0 @@
-/**
- * prdUpdateDependsOn.test.cjs — PRD 1124: scheduler.remote.updatePrd accepts
- * a `dependsOn` array patch, validates it write-time against the SAME
- * resolver (depSlugResolve.cjs) scheduler_create_prd's prdCreate.cjs uses,
- * and accepts an explicit empty array to CLEAR the dependency.
- *
- * Mirrors scheduler-reconcile-quarantine.test.cjs's HOME-isolation fixture.
- *
- * Run: timeout 120 npx vitest run src/main/__tests__/prdUpdateDependsOn.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-update-prd-dependson-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);
-  // updatePrd writes through config.writeTextAtomic, which enforces
-  // validatePath's allowed-write-root boundary (os.homedir() by default).
-  // The live app widens this via pty.cjs's spawn / activeIndexMerge.cjs's
-  // boot registration as a side effect of opening the project; a test that
-  // never spawns a PTY has to register it explicitly, same as
-  // prdCreate.cjs's own createPrd does for a chat-only Epic.
-  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',
-  );
-}
-
-test('updatePrd patches dependsOn to a new valid list naming an existing bare slug', async () => {
-  const { cwd, prdsDir } = makeFixtureProject('sm-update-dependson-valid-');
-  try {
-    writePrd(prdsDir, '3-widget-base.md', cwd);
-    writePrd(prdsDir, '4-widget-follow-up.md', cwd);
-
-    const result = await scheduler.remote.updatePrd({
-      slug: '4-widget-follow-up',
-      cwd,
-      frontmatter: { dependsOn: ['widget-base'] },
-    });
-
-    expect(result.ok).toBe(true);
-    const written = fs.readFileSync(path.join(prdsDir, '4-widget-follow-up.md'), 'utf8');
-    expect(written).toMatch(/^dependsOn: \[widget-base\]$/m);
-  } finally {
-    fs.rmSync(cwd, { recursive: true, force: true });
-  }
-});
-
-test('updatePrd refuses a dependsOn entry that resolves to no existing PRD, with a near-match suggestion, and writes nothing', async () => {
-  const { cwd, prdsDir } = makeFixtureProject('sm-update-dependson-invalid-');
-  try {
-    writePrd(prdsDir, '3-widget-base.md', cwd);
-    writePrd(prdsDir, '4-widget-follow-up.md', cwd);
-    const before = fs.readFileSync(path.join(prdsDir, '4-widget-follow-up.md'), 'utf8');
-
-    const result = await scheduler.remote.updatePrd({
-      slug: '4-widget-follow-up',
-      cwd,
-      frontmatter: { dependsOn: ['wigdet-bsae'] },
-    });
-
-    expect(result.ok).toBe(false);
-    expect(result.error).toMatch(/does not resolve to any existing PRD/);
-    expect(result.error).toMatch(/widget-base/);
-    const after = fs.readFileSync(path.join(prdsDir, '4-widget-follow-up.md'), 'utf8');
-    expect(after).toBe(before);
-  } finally {
-    fs.rmSync(cwd, { recursive: true, force: true });
-  }
-});
-
-test('updatePrd patches dependsOn to an explicit empty array, clearing a previously-set dependency without validation', async () => {
-  const { cwd, prdsDir } = makeFixtureProject('sm-update-dependson-clear-');
-  try {
-    writePrd(prdsDir, '3-widget-base.md', cwd);
-    writePrd(prdsDir, '4-widget-follow-up.md', cwd, 'dependsOn: [widget-base]\n');
-
-    const result = await scheduler.remote.updatePrd({
-      slug: '4-widget-follow-up',
-      cwd,
-      frontmatter: { dependsOn: [] },
-    });
-
-    expect(result.ok).toBe(true);
-    const written = fs.readFileSync(path.join(prdsDir, '4-widget-follow-up.md'), 'utf8');
-    expect(written).not.toContain('dependsOn');
-  } finally {
-    fs.rmSync(cwd, { recursive: true, force: true });
-  }
-});
-
-test('updatePrd patching an unrelated field leaves a PRD with no dependsOn unaffected', async () => {
-  const { cwd, prdsDir } = makeFixtureProject('sm-update-dependson-unrelated-');
-  try {
-    writePrd(prdsDir, '3-standalone.md', cwd);
-
-    const result = await scheduler.remote.updatePrd({
-      slug: '3-standalone',
-      cwd,
-      frontmatter: { estimateMinutes: 25 },
-    });
-
-    expect(result.ok).toBe(true);
-    const written = fs.readFileSync(path.join(prdsDir, '3-standalone.md'), 'utf8');
-    expect(written).not.toContain('dependsOn');
-    expect(written).toMatch(/^estimateMinutes: 25$/m);
-  } finally {
-    fs.rmSync(cwd, { recursive: true, force: true });
-  }
-});