sdd-agent-platform 0.4.2 → 0.5.1

package/node_modules/@sdd-agent-platform/core/src/sdd-docs/task-parser.test.ts

@@ -1,523 +1,467 @@
-import test from 'node:test';
-import assert from 'node:assert/strict';
-import { mkdir, mkdtemp, rm, writeFile } from 'node:fs/promises';
-import { tmpdir } from 'node:os';
-import path from 'node:path';
-
-import { validTaskMarkdown } from '../test-support/fixtures.js';
-import { hashNormalizedMarkdownContent, hashTasksContract, normalizeMarkdownContentV1 } from './document-hashes.js';
-import { inspectSddTask } from './task-inspection.js';
-import { parseSddBranch, parseSddTasksMarkdown } from './task-parser.js';
-import { renderTaskGapReport } from './task-rendering.js';
-
-test('parseSddTasksMarkdown extracts task metadata and companion sections', () => {
-  const markdown = `# Tasks
-
-### T1: Parser foundation
-
-\`\`\`sdd-task
-id: T1
-status: pending
-wave: 1
-depends_on: []
-affected_files:
-  - packages/core/src/index.ts
-change_surface: backend_only
-implementation_wave: wave-1
-validation_batch: backend-unit-wave-1
-validation_timing: batch_end
-requires_verify_before_next: false
-validation:
-  - npm test => AC-1, AC-2
-risk:
-  - parser boundary
-\`\`\`
-
-#### Boundary
-
-Only parse Markdown metadata; do not execute tasks.
-
-#### Acceptance
-
-- Task metadata is structured.
-- Boundary is retained.
-
-#### Implementation Notes
-
-None yet.
-`;
-  const model = parseSddTasksMarkdown(markdown, { tasksPath: 'specs/master/tasks.md' });
-
-  assert.equal(model.gaps.length, 0);
-  assert.equal(model.tasks.length, 1);
-  assert.equal(model.tasks[0].id, 'T1');
-  assert.equal(model.tasks[0].status, 'pending');
-  assert.equal(model.tasks[0].wave, 1);
-  assert.deepEqual(model.tasks[0].dependsOn, []);
-  assert.deepEqual(model.tasks[0].affectedFiles, ['packages/core/src/index.ts']);
-  assert.equal(model.tasks[0].changeSurface, 'backend_only');
-  assert.equal(model.tasks[0].implementationWave, 'wave-1');
-  assert.equal(model.tasks[0].validationBatch, 'backend-unit-wave-1');
-  assert.equal(model.tasks[0].validationTiming, 'batch_end');
-  assert.equal(model.tasks[0].requiresVerifyBeforeNext, false);
-  assert.deepEqual(model.tasks[0].validation, ['npm test']);
-  assert.deepEqual(model.tasks[0].validationCommands, [{ command: 'npm test', acceptanceRefs: ['AC-1', 'AC-2'], raw: 'npm test => AC-1, AC-2' }]);
-  assert.deepEqual(model.tasks[0].risk, ['parser boundary']);
-  assert.match(model.tasks[0].boundary ?? '', /Only parse Markdown metadata/);
-  assert.deepEqual(model.tasks[0].acceptance, ['Task metadata is structured.', 'Boundary is retained.']);
-});
-
-test('parseSddTasksMarkdown stops companion sections at level-two task headings', () => {
-  const markdown = `# Tasks
-
-## T1: First task
-
-\`\`\`sdd-task
-id: T1
-status: pending
-wave: 1
-depends_on: []
-affected_files:
-  - packages/app/login.ts
-change_surface: backend_only
-validation_timing: task_end
-requires_verify_before_next: true
-validation:
-  - npm test
-risk: []
-\`\`\`
-
-#### Boundary
-
-Only T1.
-
-#### Acceptance
-
-- T1 passes.
-
-#### Implementation Notes
-
-Keep T1 narrow.
-
-## T2: Second task
-
-T2 introduction must not be parsed as T1 notes.
-
-\`\`\`sdd-task
-id: T2
-status: pending
-wave: 1
-depends_on: [T1]
-affected_files:
-  - packages/core/src/index.ts
-change_surface: backend_only
-validation_timing: task_end
-requires_verify_before_next: true
-validation:
-  - npm test
-risk: []
-\`\`\`
-
-#### Boundary
-
-Only T2.
-
-#### Acceptance
-
-- T2 passes.
-`;
-  const model = parseSddTasksMarkdown(markdown, { tasksPath: 'specs/master/tasks.md' });
-
-  assert.equal(model.gaps.length, 0);
-  assert.equal(model.tasks[0].source.heading, 'T1: First task');
-  assert.equal(model.tasks[0].implementationNotes, 'Keep T1 narrow.');
-  assert.equal(model.tasks[0].implementationNotes?.includes('T2 introduction'), false);
-});
-
-test('parseSddTasksMarkdown blocks backend build validation for frontend-only tasks', () => {
-  const markdown = `# Tasks
-
-### T23: JSP rendering copy update
-
-\`\`\`sdd-task
-id: T23
-status: pending
-wave: 1
-depends_on: []
-affected_files:
-  - emp-upms/emp-upms-server/src/main/webapp/WEB-INF/views/demo.jsp
-  - emp-upms/emp-upms-server/src/main/webapp/static/img/banner.svg
-validation:
-  - mvn compile -Ptest -pl emp-upms/emp-upms-server -am
-risk: []
-\`\`\`
-
-#### Boundary
-
-Only JSP copy and static image rendering.
-
-#### Acceptance
-
-- JSP copy renders correctly.
-`;
-  const model = parseSddTasksMarkdown(markdown, { tasksPath: 'specs/master/tasks.md' });
-
-  assert.equal(model.gaps.some((gap) => gap.taskId === 'T23' && gap.field === 'validation' && /frontend-only/.test(gap.message)), true);
-  assert.equal(model.gaps.some((gap) => /Maven\/Gradle/.test(gap.recommendation)), true);
-});
-
-test('parseSddTasksMarkdown validates Phase 8.19 batch contract fields', () => {
-  const markdown = `# Tasks
-
-### T24: Invalid batch contract
-
-\`\`\`sdd-task
-id: T24
-status: pending
-wave: 1
-depends_on: []
-affected_files:
-  - packages/core/src/index.ts
-change_surface: api_only
-validation_timing: batch_end
-requires_verify_before_next: sometimes
-validation:
-  - npm test
-risk: []
-\`\`\`
-
-#### Boundary
-
-Stay in parser scope.
-
-#### Acceptance
-
-- Parser blocks invalid batch contract.
-`;
-  const model = parseSddTasksMarkdown(markdown, { tasksPath: 'specs/master/tasks.md' });
-  const fields = model.gaps.map((gap) => gap.field);
-
-  assert.equal(fields.includes('change_surface'), true);
-  assert.equal(fields.includes('validation_batch'), true);
-  assert.equal(fields.includes('requires_verify_before_next'), true);
-});
-
-test('hashTasksContract includes Phase 8.19 validation grouping fields', () => {
-  const base = `# Tasks
-
-### T1: Hash grouping
-
-\`\`\`sdd-task
-id: T1
-status: pending
-wave: 1
-depends_on: []
-affected_files:
-  - packages/core/src/index.ts
-change_surface: backend_only
-implementation_wave: wave-1
-validation_batch: batch-a
-validation_timing: batch_end
-requires_verify_before_next: false
-validation:
-  - npm test
-risk: []
-\`\`\`
-
-#### Boundary
-
-Stay in parser scope.
-
-#### Acceptance
-
-- Hash includes validation grouping.
-`;
-  const changed = base.replace('validation_batch: batch-a', 'validation_batch: batch-b');
-
-  assert.notEqual(hashTasksContract(base), hashTasksContract(changed));
-});
-
-test('normalizeMarkdownContentV1 applies conservative formatting normalization', () => {
-  const base = `---
-contract: sdd-spec-doc-v1
-based_on_spec_hash: legacy
-created_at: 2026-01-01T00:00:00.000Z
----
-
-# Spec   
-
-<!-- standalone comment -->
-
-- keep list order
-  - keep indentation
-
-\`\`\`md
-<!-- keep fenced comment -->   
-based_on_spec_hash: keep-in-code
-\`\`\`
-`;
-  const equivalent = `---\r\ncontract: sdd-spec-doc-v1\r\nbased_on_spec_hash: changed\r\nupdated_at: 2026-01-02T00:00:00.000Z\r\n---\r\n\r\n# Spec\t\r\n\r\n\r\n<!-- standalone comment -->\r\n\r\n- keep list order\r\n  - keep indentation\r\n\r\n\`\`\`md\r\n<!-- keep fenced comment -->   \r\nbased_on_spec_hash: keep-in-code\r\n\`\`\`\r\n`;
-
-  assert.equal(normalizeMarkdownContentV1(base), normalizeMarkdownContentV1(equivalent));
-  assert.match(normalizeMarkdownContentV1(base), /<!-- keep fenced comment -->   /);
-  assert.doesNotMatch(normalizeMarkdownContentV1(base), /standalone comment/);
-});
-
-test('spec and plan contract hashes ignore runtime metadata and formatting noise only', async () => {
-  const root = await mkdtemp(path.join(tmpdir(), 'sdd-normalized-contract-hash-'));
-  try {
-    const branchDir = path.join(root, 'specs', 'feature');
-    await mkdir(branchDir, { recursive: true });
-    const spec = '# Spec\n\nInitial requirement.\n';
-    const plan = '# Plan\n\nInitial plan.\n';
-    await writeFile(path.join(branchDir, 'spec.md'), spec, 'utf8');
-    await writeFile(path.join(branchDir, 'plan.md'), plan, 'utf8');
-    await writeFile(path.join(branchDir, 'tasks.md'), validTaskMarkdown('T1', []), 'utf8');
-    const base = await parseSddBranch(root, 'feature');
-
-    await writeFile(path.join(branchDir, 'spec.md'), `---\nbased_on_spec_hash: ignored\nupdated_at: now\n---\n\n# Spec  \n\n\nInitial requirement.\n\n<!-- runtime note -->\n`, 'utf8');
-    await writeFile(path.join(branchDir, 'plan.md'), `# Plan\n\nbased_on_plan_hash: ignored\n\nInitial plan.\n`, 'utf8');
-    const normalizedOnly = await parseSddBranch(root, 'feature');
-
-    await writeFile(path.join(branchDir, 'spec.md'), '# Spec\n\nChanged requirement.\n', 'utf8');
-    const changed = await parseSddBranch(root, 'feature');
-
-    assert.notEqual(normalizedOnly.documents.specHash, base.documents.specHash);
-    assert.equal(normalizedOnly.documents.specContractHash, base.documents.specContractHash);
-    assert.equal(normalizedOnly.documents.planContractHash, base.documents.planContractHash);
-    assert.notEqual(changed.documents.specContractHash, base.documents.specContractHash);
-    assert.notEqual(hashNormalizedMarkdownContent('# Spec\n\nChanged requirement.\n'), hashNormalizedMarkdownContent(spec));
-  } finally {
-    await rm(root, { recursive: true, force: true });
-  }
-});
-
-test('parseSddBranch separates raw doc hash from semantic tasks contract hash', async () => {
-  const root = await mkdtemp(path.join(tmpdir(), 'sdd-contract-hash-'));
-  try {
-    const branchDir = path.join(root, 'specs', 'feature');
-    await mkdir(branchDir, { recursive: true });
-    await writeFile(path.join(branchDir, 'spec.md'), '# Spec\n', 'utf8');
-    await writeFile(path.join(branchDir, 'plan.md'), '# Plan\n', 'utf8');
-    const original = validTaskMarkdown('T1', []);
-    await writeFile(path.join(branchDir, 'tasks.md'), original, 'utf8');
-    const base = await parseSddBranch(root, 'feature');
-
-    await writeFile(path.join(branchDir, 'tasks.md'), `${original}\n\n<!-- formatting-only note -->\n`, 'utf8');
-    const formattingOnly = await parseSddBranch(root, 'feature');
-
-    await writeFile(path.join(branchDir, 'tasks.md'), original.replace('risk: []', 'risk:\n  - security'), 'utf8');
-    const semanticChange = await parseSddBranch(root, 'feature');
-
-    assert.notEqual(formattingOnly.documents.tasksHash, base.documents.tasksHash);
-    assert.equal(formattingOnly.documents.tasksContractHash, base.documents.tasksContractHash);
-    assert.notEqual(semanticChange.documents.tasksContractHash, base.documents.tasksContractHash);
-  } finally {
-    await rm(root, { recursive: true, force: true });
-  }
-});
-
-test('parseSddTasksMarkdown does not leak companion sections across indented task headings', () => {
-  const markdown = `# Tasks
-
-  ### ERP-SCRK-1: 固化入库同步状态机边界
-
-\`\`\`sdd-task
-id: ERP-SCRK-1
-status: pending
-wave: 1
-depends_on: []
-affected_files:
-  - src/main/java/com/acme/erp/InboundSyncService.java
-validation:
-  - mvn test -Dtest=InboundSyncServiceTest
-risk:
-  - state-machine
-\`\`\`
-
-#### Boundary
-
-Only state-machine logic.
-
-#### Acceptance
-
-- Terminal states never roll back.
-
-  ### ERP-SCRK-2: 移除 Mapper SQL 拼接
-
-\`\`\`sdd-task
-id: ERP-SCRK-2
-status: pending
-wave: 2
-depends_on:
-  - ERP-SCRK-1
-affected_files:
-  - src/main/java/com/acme/erp/InboundSyncMapper.java
-validation:
-  - mvn test
-risk:
-  - sql
-\`\`\`
-
-#### Boundary
-
-Only mapper SQL parameterization.
-
-#### Acceptance
-
-- Mapper has no SQL string concatenation.
-`;
-  const model = parseSddTasksMarkdown(markdown, { tasksPath: 'specs/master/tasks.md' });
-
-  assert.equal(model.gaps.length, 0);
-  assert.equal(model.tasks.length, 2);
-  assert.deepEqual(model.tasks[0].acceptance, ['Terminal states never roll back.']);
-  assert.deepEqual(model.tasks[1].acceptance, ['Mapper has no SQL string concatenation.']);
-  assert.doesNotMatch(model.tasks[0].boundary ?? '', /Mapper SQL/);
-});
-
-test('parseSddTasksMarkdown reports task metadata and dependency gaps', () => {
-  const markdown = `# Tasks
-
-### T2: Gap case
-
-\`\`\`sdd-task
-id: T2
-status: surprise
-depends_on:
-  - T404
-affected_files: []
-validation: []
-risk: []
-\`\`\`
-`;
-  const model = parseSddTasksMarkdown(markdown);
-  const fields = model.gaps.map((gap) => gap.field);
-
-  assert.equal(model.tasks.length, 1);
-  assert.equal(model.gaps.every((gap) => gap.severity === 'blocking'), true);
-  assert.equal(fields.includes('status'), true);
-  assert.equal(fields.includes('wave'), true);
-  assert.equal(fields.includes('affected_files'), true);
-  assert.equal(fields.includes('validation'), true);
-  assert.equal(fields.includes('Boundary'), true);
-  assert.equal(fields.includes('Acceptance'), true);
-  assert.equal(fields.includes('depends_on'), true);
-  assert.match(renderTaskGapReport({
-    branch: 'master',
-    specPath: 'spec.md',
-    planPath: 'plan.md',
-    tasksPath: 'tasks.md',
-    verifyPath: 'verify.md',
-    documents: { specExists: true, planExists: true, tasksExists: true, verifyExists: true },
-    tasks: model.tasks,
-    gaps: model.gaps
-  }), /BLOCKED/);
-});
-
-test('parseSddBranch reads branch docs and inspectSddTask filters gaps', async () => {
-  const root = await mkdtemp(path.join(tmpdir(), 'sdd-parser-'));
-  try {
-    const branchDir = path.join(root, 'specs', 'feature');
-    await mkdir(branchDir, { recursive: true });
-    await writeFile(path.join(branchDir, 'spec.md'), '# Spec\n', 'utf8');
-    await writeFile(path.join(branchDir, 'plan.md'), '# Plan\n', 'utf8');
-    await writeFile(path.join(branchDir, 'tasks.md'), `# Tasks
-
-### T1: Valid task
-
-\`\`\`sdd-task
-id: T1
-status: pending
-wave: 1
-depends_on: []
-affected_files:
-  - a.ts
-validation:
-  - npm test
-risk: []
-\`\`\`
-
-#### Boundary
-
-Stay in a.ts.
-
-#### Acceptance
-
-- a.ts behavior is covered.
-`, 'utf8');
-
-    const model = await parseSddBranch(root, 'feature');
-    const inspected = inspectSddTask(model, 'T1');
-
-    assert.equal(model.documents.specExists, true);
-    assert.equal(model.documents.planExists, true);
-    assert.equal(model.documents.tasksExists, true);
-    assert.equal(model.gaps.length, 0);
-    assert.equal(inspected.task?.id, 'T1');
-    assert.equal(inspected.gaps.length, 0);
-  } finally {
-    await rm(root, { recursive: true, force: true });
-  }
-});
-
-test('retained phase fallback detects aggregate duplicate task ids and inspect ambiguity', async () => {
-  const root = await mkdtemp(path.join(tmpdir(), 'sdd-duplicate-'));
-  try {
-    const branchDir = path.join(root, 'specs', 'feature');
-    await mkdir(branchDir, { recursive: true });
-    await writeFile(path.join(branchDir, 'spec.md'), '# Spec\n', 'utf8');
-    await writeFile(path.join(branchDir, 'plan.md'), '# Plan\n', 'utf8');
-    await writeFile(path.join(branchDir, 'tasks.md'), '# Tasks\nLegacy index only.\n', 'utf8');
-    await writeFile(path.join(branchDir, 'phase1.0-tasks.md'), validTaskMarkdown('T1', []), 'utf8');
-    await writeFile(path.join(branchDir, 'phase1.1-tasks.md'), validTaskMarkdown('T1', []), 'utf8');
-
-    const model = await parseSddBranch(root, 'feature');
-    const inspected = inspectSddTask(model, 'T1');
-
-    assert.equal(model.tasks.length, 2);
-    assert.equal(model.gaps.some((gap) => gap.field === 'id' && gap.taskId === 'T1' && /Duplicate task id T1 across parsed task files/.test(gap.message)), true);
-    assert.equal(inspected.task, null);
-    assert.equal(inspected.gaps.some((gap) => /ambiguous/.test(gap.message)), true);
-  } finally {
-    await rm(root, { recursive: true, force: true });
-  }
-});
-
-test('retained phase fallback validates cross-file dependencies over aggregate tasks', async () => {
-  const root = await mkdtemp(path.join(tmpdir(), 'sdd-cross-deps-'));
-  try {
-    const branchDir = path.join(root, 'specs', 'feature');
-    await mkdir(branchDir, { recursive: true });
-    await writeFile(path.join(branchDir, 'spec.md'), '# Spec\n', 'utf8');
-    await writeFile(path.join(branchDir, 'plan.md'), '# Plan\n', 'utf8');
-    await writeFile(path.join(branchDir, 'tasks.md'), '# Tasks\nLegacy index only.\n', 'utf8');
-    await writeFile(path.join(branchDir, 'phase1.0-tasks.md'), validTaskMarkdown('P1.0-T1', []), 'utf8');
-    await writeFile(path.join(branchDir, 'phase1.1-tasks.md'), validTaskMarkdown('P1.1-T1', ['P1.0-T1']), 'utf8');
-
-    const model = await parseSddBranch(root, 'feature');
-
-    assert.equal(model.tasks.length, 2);
-    assert.equal(model.gaps.length, 0);
-  } finally {
-    await rm(root, { recursive: true, force: true });
-  }
-});
-
-test('retained phase fallback still reports unknown dependencies', async () => {
-  const root = await mkdtemp(path.join(tmpdir(), 'sdd-unknown-deps-'));
-  try {
-    const branchDir = path.join(root, 'specs', 'feature');
-    await mkdir(branchDir, { recursive: true });
-    await writeFile(path.join(branchDir, 'spec.md'), '# Spec\n', 'utf8');
-    await writeFile(path.join(branchDir, 'plan.md'), '# Plan\n', 'utf8');
-    await writeFile(path.join(branchDir, 'tasks.md'), '# Tasks\nLegacy index only.\n', 'utf8');
-    await writeFile(path.join(branchDir, 'phase1.0-tasks.md'), validTaskMarkdown('P1.0-T1', ['P9.9-T404']), 'utf8');
-
-    const model = await parseSddBranch(root, 'feature');
-
-    assert.equal(model.gaps.some((gap) => gap.type === 'Dependency Gap' && gap.taskId === 'P1.0-T1' && /unknown task P9.9-T404/.test(gap.message)), true);
-  } finally {
-    await rm(root, { recursive: true, force: true });
-  }
-});
+import test from 'node:test';
+import assert from 'node:assert/strict';
+import { mkdir, mkdtemp, rm, writeFile } from 'node:fs/promises';
+import { tmpdir } from 'node:os';
+import path from 'node:path';
+
+import { validTaskMarkdown } from '../test-support/fixtures.js';
+import { hashTasksContract } from './document-hashes.js';
+import { inspectSddTask } from './task-inspection.js';
+import { parseSddBranch, parseSddTasksMarkdown } from './task-parser.js';
+import { renderTaskGapReport } from './task-rendering.js';
+
+test('parseSddTasksMarkdown extracts task metadata and companion sections', () => {
+  const markdown = `# Tasks
+
+### T1: Parser foundation
+
+\`\`\`sdd-task
+id: T1
+status: pending
+wave: 1
+depends_on: []
+affected_files:
+  - packages/core/src/index.ts
+change_surface: backend_only
+implementation_wave: wave-1
+validation_batch: backend-unit-wave-1
+validation_timing: batch_end
+requires_verify_before_next: false
+validation:
+  - npm test => AC-1, AC-2
+risk:
+  - parser boundary
+\`\`\`
+
+#### Boundary
+
+Only parse Markdown metadata; do not execute tasks.
+
+#### Acceptance
+
+- Task metadata is structured.
+- Boundary is retained.
+
+#### Implementation Notes
+
+None yet.
+`;
+  const model = parseSddTasksMarkdown(markdown, { tasksPath: 'specs/master/tasks.md' });
+
+  assert.equal(model.gaps.length, 0);
+  assert.equal(model.tasks.length, 1);
+  assert.equal(model.tasks[0].id, 'T1');
+  assert.equal(model.tasks[0].status, 'pending');
+  assert.equal(model.tasks[0].wave, 1);
+  assert.deepEqual(model.tasks[0].dependsOn, []);
+  assert.deepEqual(model.tasks[0].affectedFiles, ['packages/core/src/index.ts']);
+  assert.equal(model.tasks[0].changeSurface, 'backend_only');
+  assert.equal(model.tasks[0].implementationWave, 'wave-1');
+  assert.equal(model.tasks[0].validationBatch, 'backend-unit-wave-1');
+  assert.equal(model.tasks[0].validationTiming, 'batch_end');
+  assert.equal(model.tasks[0].requiresVerifyBeforeNext, false);
+  assert.deepEqual(model.tasks[0].validation, ['npm test']);
+  assert.deepEqual(model.tasks[0].validationCommands, [{ command: 'npm test', acceptanceRefs: ['AC-1', 'AC-2'], raw: 'npm test => AC-1, AC-2' }]);
+  assert.deepEqual(model.tasks[0].risk, ['parser boundary']);
+  assert.match(model.tasks[0].boundary ?? '', /Only parse Markdown metadata/);
+  assert.deepEqual(model.tasks[0].acceptance, ['Task metadata is structured.', 'Boundary is retained.']);
+});
+
+test('parseSddTasksMarkdown stops companion sections at next major section', () => {
+  const markdown = `# Tasks
+
+## Implementation Tasks
+
+### T2: Update JSP report page
+
+\`\`\`sdd-task
+id: T2
+status: pending
+workType: implementation
+unitType: frontend
+change_surface: frontend_only
+\`\`\`
+
+#### Implementation Notes
+
+Use the renew report JSP only as a style reference.
+
+## Validation Tasks
+
+### T3: Validate JSP report page
+
+\`\`\`sdd-task
+id: T3
+status: pending
+workType: test
+\`\`\`
+`;
+  const model = parseSddTasksMarkdown(markdown, { tasksPath: 'specs/master/tasks.md', validateDependencies: false });
+
+  assert.equal(model.tasks[0].implementationNotes, 'Use the renew report JSP only as a style reference.');
+  assert.doesNotMatch(model.tasks[0].implementationNotes ?? '', /Validation Tasks/);
+});
+
+test('parseSddTasksMarkdown accepts cli-only change surface', () => {
+  const markdown = `# Tasks
+
+### T1: CLI status summary
+
+\`\`\`sdd-task
+id: T1
+status: pending
+wave: 1
+depends_on: []
+affected_files:
+  - packages/cli/src/commands/status.ts
+change_surface: cli_only
+validation:
+  - node --test --import tsx packages/cli/src/commands/cli-regression.test.ts
+\`\`\`
+
+#### Boundary
+
+Only update the CLI status summary.
+
+#### Acceptance
+
+- CLI status summary renders accepted refs.
+`;
+  const model = parseSddTasksMarkdown(markdown, { tasksPath: 'specs/master/tasks.md' });
+
+  assert.equal(model.gaps.some((gap) => gap.field === 'change_surface'), false);
+  assert.equal(model.tasks[0].changeSurface, 'cli_only');
+});
+
+test('parseSddTasksMarkdown blocks backend build validation for frontend-only tasks', () => {
+  const markdown = `# Tasks
+
+### T23: JSP rendering copy update
+
+\`\`\`sdd-task
+id: T23
+status: pending
+wave: 1
+depends_on: []
+affected_files:
+  - emp-upms/emp-upms-server/src/main/webapp/WEB-INF/views/demo.jsp
+  - emp-upms/emp-upms-server/src/main/webapp/static/img/banner.svg
+validation:
+  - mvn compile -Ptest -pl emp-upms/emp-upms-server -am
+risk: []
+\`\`\`
+
+#### Boundary
+
+Only JSP copy and static image rendering.
+
+#### Acceptance
+
+- JSP copy renders correctly.
+`;
+  const model = parseSddTasksMarkdown(markdown, { tasksPath: 'specs/master/tasks.md' });
+
+  assert.equal(model.gaps.some((gap) => gap.taskId === 'T23' && gap.field === 'validation' && /frontend-only/.test(gap.message)), true);
+  assert.equal(model.gaps.some((gap) => /Maven\/Gradle/.test(gap.recommendation)), true);
+});
+
+test('parseSddTasksMarkdown validates Phase 8.19 batch contract fields', () => {
+  const markdown = `# Tasks
+
+### T24: Invalid batch contract
+
+\`\`\`sdd-task
+id: T24
+status: pending
+wave: 1
+depends_on: []
+affected_files:
+  - packages/core/src/index.ts
+change_surface: api_only
+validation_timing: batch_end
+requires_verify_before_next: sometimes
+validation:
+  - npm test
+risk: []
+\`\`\`
+
+#### Boundary
+
+Stay in parser scope.
+
+#### Acceptance
+
+- Parser blocks invalid batch contract.
+`;
+  const model = parseSddTasksMarkdown(markdown, { tasksPath: 'specs/master/tasks.md' });
+  const fields = model.gaps.map((gap) => gap.field);
+
+  assert.equal(fields.includes('change_surface'), true);
+  assert.equal(fields.includes('validation_batch'), true);
+  assert.equal(fields.includes('requires_verify_before_next'), true);
+});
+
+test('hashTasksContract includes Phase 8.19 validation grouping fields', () => {
+  const base = `# Tasks
+
+### T1: Hash grouping
+
+\`\`\`sdd-task
+id: T1
+status: pending
+wave: 1
+depends_on: []
+affected_files:
+  - packages/core/src/index.ts
+change_surface: backend_only
+implementation_wave: wave-1
+validation_batch: batch-a
+validation_timing: batch_end
+requires_verify_before_next: false
+validation:
+  - npm test
+risk: []
+\`\`\`
+
+#### Boundary
+
+Stay in parser scope.
+
+#### Acceptance
+
+- Hash includes validation grouping.
+`;
+  const changed = base.replace('validation_batch: batch-a', 'validation_batch: batch-b');
+
+  assert.notEqual(hashTasksContract(base), hashTasksContract(changed));
+});
+
+test('parseSddBranch separates raw doc hash from semantic tasks contract hash', async () => {
+  const root = await mkdtemp(path.join(tmpdir(), 'sdd-contract-hash-'));
+  try {
+    const branchDir = path.join(root, 'specs', 'feature');
+    await mkdir(branchDir, { recursive: true });
+    await writeFile(path.join(branchDir, 'spec.md'), '# Spec\n', 'utf8');
+    await writeFile(path.join(branchDir, 'plan.md'), '# Plan\n', 'utf8');
+    const original = validTaskMarkdown('T1', []);
+    await writeFile(path.join(branchDir, 'tasks.md'), original, 'utf8');
+    const base = await parseSddBranch(root, 'feature');
+
+    await writeFile(path.join(branchDir, 'tasks.md'), `${original}\n\n<!-- formatting-only note -->\n`, 'utf8');
+    const formattingOnly = await parseSddBranch(root, 'feature');
+
+    await writeFile(path.join(branchDir, 'tasks.md'), original.replace('risk: []', 'risk:\n  - security'), 'utf8');
+    const semanticChange = await parseSddBranch(root, 'feature');
+
+    assert.notEqual(formattingOnly.documents.tasksHash, base.documents.tasksHash);
+    assert.equal(formattingOnly.documents.tasksContractHash, base.documents.tasksContractHash);
+    assert.notEqual(semanticChange.documents.tasksContractHash, base.documents.tasksContractHash);
+  } finally {
+    await rm(root, { recursive: true, force: true });
+  }
+});
+
+test('parseSddTasksMarkdown does not leak companion sections across indented task headings', () => {
+  const markdown = `# Tasks
+
+  ### ERP-SCRK-1: 固化入库同步状态机边界
+
+\`\`\`sdd-task
+id: ERP-SCRK-1
+status: pending
+wave: 1
+depends_on: []
+affected_files:
+  - src/main/java/com/acme/erp/InboundSyncService.java
+validation:
+  - mvn test -Dtest=InboundSyncServiceTest
+risk:
+  - state-machine
+\`\`\`
+
+#### Boundary
+
+Only state-machine logic.
+
+#### Acceptance
+
+- Terminal states never roll back.
+
+  ### ERP-SCRK-2: 移除 Mapper SQL 拼接
+
+\`\`\`sdd-task
+id: ERP-SCRK-2
+status: pending
+wave: 2
+depends_on:
+  - ERP-SCRK-1
+affected_files:
+  - src/main/java/com/acme/erp/InboundSyncMapper.java
+validation:
+  - mvn test
+risk:
+  - sql
+\`\`\`
+
+#### Boundary
+
+Only mapper SQL parameterization.
+
+#### Acceptance
+
+- Mapper has no SQL string concatenation.
+`;
+  const model = parseSddTasksMarkdown(markdown, { tasksPath: 'specs/master/tasks.md' });
+
+  assert.equal(model.gaps.length, 0);
+  assert.equal(model.tasks.length, 2);
+  assert.deepEqual(model.tasks[0].acceptance, ['Terminal states never roll back.']);
+  assert.deepEqual(model.tasks[1].acceptance, ['Mapper has no SQL string concatenation.']);
+  assert.doesNotMatch(model.tasks[0].boundary ?? '', /Mapper SQL/);
+});
+
+test('parseSddTasksMarkdown reports task metadata and dependency gaps', () => {
+  const markdown = `# Tasks
+
+### T2: Gap case
+
+\`\`\`sdd-task
+id: T2
+status: surprise
+depends_on:
+  - T404
+affected_files: []
+validation: []
+risk: []
+\`\`\`
+`;
+  const model = parseSddTasksMarkdown(markdown);
+  const fields = model.gaps.map((gap) => gap.field);
+
+  assert.equal(model.tasks.length, 1);
+  assert.equal(model.gaps.every((gap) => gap.severity === 'blocking'), true);
+  assert.equal(fields.includes('status'), true);
+  assert.equal(fields.includes('wave'), true);
+  assert.equal(fields.includes('affected_files'), true);
+  assert.equal(fields.includes('validation'), true);
+  assert.equal(fields.includes('Boundary'), true);
+  assert.equal(fields.includes('Acceptance'), true);
+  assert.equal(fields.includes('depends_on'), true);
+  assert.match(renderTaskGapReport({
+    branch: 'master',
+    specPath: 'spec.md',
+    planPath: 'plan.md',
+    tasksPath: 'tasks.md',
+    verifyPath: 'verify.md',
+    documents: { specExists: true, planExists: true, tasksExists: true, verifyExists: true },
+    tasks: model.tasks,
+    gaps: model.gaps
+  }), /BLOCKED/);
+});
+
+test('parseSddBranch reads branch docs and inspectSddTask filters gaps', async () => {
+  const root = await mkdtemp(path.join(tmpdir(), 'sdd-parser-'));
+  try {
+    const branchDir = path.join(root, 'specs', 'feature');
+    await mkdir(branchDir, { recursive: true });
+    await writeFile(path.join(branchDir, 'spec.md'), '# Spec\n', 'utf8');
+    await writeFile(path.join(branchDir, 'plan.md'), '# Plan\n', 'utf8');
+    await writeFile(path.join(branchDir, 'tasks.md'), `# Tasks
+
+### T1: Valid task
+
+\`\`\`sdd-task
+id: T1
+status: pending
+wave: 1
+depends_on: []
+affected_files:
+  - a.ts
+validation:
+  - npm test
+risk: []
+\`\`\`
+
+#### Boundary
+
+Stay in a.ts.
+
+#### Acceptance
+
+- a.ts behavior is covered.
+`, 'utf8');
+
+    const model = await parseSddBranch(root, 'feature');
+    const inspected = inspectSddTask(model, 'T1');
+
+    assert.equal(model.documents.specExists, true);
+    assert.equal(model.documents.planExists, true);
+    assert.equal(model.documents.tasksExists, true);
+    assert.equal(model.gaps.length, 0);
+    assert.equal(inspected.task?.id, 'T1');
+    assert.equal(inspected.gaps.length, 0);
+  } finally {
+    await rm(root, { recursive: true, force: true });
+  }
+});
+
+test('retained phase fallback detects aggregate duplicate task ids and inspect ambiguity', async () => {
+  const root = await mkdtemp(path.join(tmpdir(), 'sdd-duplicate-'));
+  try {
+    const branchDir = path.join(root, 'specs', 'feature');
+    await mkdir(branchDir, { recursive: true });
+    await writeFile(path.join(branchDir, 'spec.md'), '# Spec\n', 'utf8');
+    await writeFile(path.join(branchDir, 'plan.md'), '# Plan\n', 'utf8');
+    await writeFile(path.join(branchDir, 'tasks.md'), '# Tasks\nLegacy index only.\n', 'utf8');
+    await writeFile(path.join(branchDir, 'phase1.0-tasks.md'), validTaskMarkdown('T1', []), 'utf8');
+    await writeFile(path.join(branchDir, 'phase1.1-tasks.md'), validTaskMarkdown('T1', []), 'utf8');
+
+    const model = await parseSddBranch(root, 'feature');
+    const inspected = inspectSddTask(model, 'T1');
+
+    assert.equal(model.tasks.length, 2);
+    assert.equal(model.gaps.some((gap) => gap.field === 'id' && gap.taskId === 'T1' && /Duplicate task id T1 across parsed task files/.test(gap.message)), true);
+    assert.equal(inspected.task, null);
+    assert.equal(inspected.gaps.some((gap) => /ambiguous/.test(gap.message)), true);
+  } finally {
+    await rm(root, { recursive: true, force: true });
+  }
+});
+
+test('retained phase fallback validates cross-file dependencies over aggregate tasks', async () => {
+  const root = await mkdtemp(path.join(tmpdir(), 'sdd-cross-deps-'));
+  try {
+    const branchDir = path.join(root, 'specs', 'feature');
+    await mkdir(branchDir, { recursive: true });
+    await writeFile(path.join(branchDir, 'spec.md'), '# Spec\n', 'utf8');
+    await writeFile(path.join(branchDir, 'plan.md'), '# Plan\n', 'utf8');
+    await writeFile(path.join(branchDir, 'tasks.md'), '# Tasks\nLegacy index only.\n', 'utf8');
+    await writeFile(path.join(branchDir, 'phase1.0-tasks.md'), validTaskMarkdown('P1.0-T1', []), 'utf8');
+    await writeFile(path.join(branchDir, 'phase1.1-tasks.md'), validTaskMarkdown('P1.1-T1', ['P1.0-T1']), 'utf8');
+
+    const model = await parseSddBranch(root, 'feature');
+
+    assert.equal(model.tasks.length, 2);
+    assert.equal(model.gaps.length, 0);
+  } finally {
+    await rm(root, { recursive: true, force: true });
+  }
+});
+
+test('retained phase fallback still reports unknown dependencies', async () => {
+  const root = await mkdtemp(path.join(tmpdir(), 'sdd-unknown-deps-'));
+  try {
+    const branchDir = path.join(root, 'specs', 'feature');
+    await mkdir(branchDir, { recursive: true });
+    await writeFile(path.join(branchDir, 'spec.md'), '# Spec\n', 'utf8');
+    await writeFile(path.join(branchDir, 'plan.md'), '# Plan\n', 'utf8');
+    await writeFile(path.join(branchDir, 'tasks.md'), '# Tasks\nLegacy index only.\n', 'utf8');
+    await writeFile(path.join(branchDir, 'phase1.0-tasks.md'), validTaskMarkdown('P1.0-T1', ['P9.9-T404']), 'utf8');
+
+    const model = await parseSddBranch(root, 'feature');
+
+    assert.equal(model.gaps.some((gap) => gap.type === 'Dependency Gap' && gap.taskId === 'P1.0-T1' && /unknown task P9.9-T404/.test(gap.message)), true);
+  } finally {
+    await rm(root, { recursive: true, force: true });
+  }
+});