sillyspec 3.20.3 → 3.20.4

package/test/contract-artifacts.test.mjs

@@ -0,0 +1,323 @@
+/**
+ * endpoint-extractor 和 contract-matrix 测试
+ */
+
+import { describe, it } from 'node:test'
+import assert from 'node:assert/strict'
+import { writeFileSync, mkdirSync, rmSync } from 'fs'
+import { join } from 'path'
+import { tmpdir } from 'os'
+import {
+  extractFastApiEndpoints,
+  extractFrontendApiCalls,
+  normalizePath,
+  diffApiParity,
+} from '../src/endpoint-extractor.js'
+import {
+  classifyTask,
+} from '../src/contract-matrix.js'
+
+// ─── 路径归一化 ─────────────────────────────────────────────────────────
+
+describe('normalizePath', () => {
+  it('模板字符串归一化', () => {
+    assert.equal(normalizePath('/api/ppm/project-plan/${id}/plan-nodes'), '/api/ppm/project-plan/{param}/plan-nodes')
+  })
+
+  it('Express 风格参数归一化', () => {
+    assert.equal(normalizePath('/api/users/:userId/posts'), '/api/users/{param}/posts')
+  })
+
+  it('无参数不改变', () => {
+    assert.equal(normalizePath('/api/ppm/plan-node'), '/api/ppm/plan-node')
+  })
+})
+
+// ─── FastAPI 端点提取 ──────────────────────────────────────────────────
+
+describe('extractFastApiEndpoints', () => {
+  const tmpDir = join(tmpdir(), 'sillyspec-test-fastapi')
+  const routerFile = join(tmpDir, 'router.py')
+
+  it('提取单行装饰器端点', () => {
+    mkdirSync(tmpDir, { recursive: true })
+    writeFileSync(routerFile, [
+      'from fastapi import APIRouter',
+      'router = APIRouter(prefix="/api/ppm")',
+      '',
+      '@router.get("/plan-node")',
+      'async def list_plan_nodes():',
+      '    pass',
+      '',
+      '@router.post("/plan-node")',
+      'async def create_plan_node():',
+      '    pass',
+      '',
+      '@router.get("/project-plan/{plan_id}/plan-nodes")',
+      'async def list_ps_plan_nodes(plan_id: str):',
+      '    pass',
+    ].join('\n'), 'utf8')
+
+    const endpoints = extractFastApiEndpoints(routerFile)
+    assert.equal(endpoints.length, 3)
+    assert.equal(endpoints[0].method, 'GET')
+    assert.equal(endpoints[0].path, '/api/ppm/plan-node')
+    assert.equal(endpoints[1].method, 'POST')
+    assert.equal(endpoints[1].path, '/api/ppm/plan-node')
+    assert.equal(endpoints[2].method, 'GET')
+    assert.equal(endpoints[2].path, '/api/ppm/project-plan/{plan_id}/plan-nodes')
+  })
+
+  it('prefix + 路径正确合并', () => {
+    mkdirSync(tmpDir, { recursive: true })
+    writeFileSync(routerFile, [
+      'router = APIRouter(prefix="/api/v2")',
+      '@router.get("/users")',
+      'async def list_users():',
+      '    pass',
+    ].join('\n'), 'utf8')
+
+    const endpoints = extractFastApiEndpoints(routerFile)
+    assert.equal(endpoints.length, 1)
+    assert.equal(endpoints[0].path, '/api/v2/users')
+  })
+
+  it('空文件返回空', () => {
+    mkdirSync(tmpDir, { recursive: true })
+    writeFileSync(routerFile, '', 'utf8')
+    const endpoints = extractFastApiEndpoints(routerFile)
+    assert.equal(endpoints.length, 0)
+  })
+
+  // cleanup
+  it('cleanup', () => {
+    try { rmSync(tmpDir, { recursive: true, force: true }) } catch {}
+  })
+})
+
+// ─── 前端 API 调用提取 ─────────────────────────────────────────────────
+
+describe('extractFrontendApiCalls', () => {
+  const tmpDir = join(tmpdir(), 'sillyspec-test-frontend')
+  const apiFile = join(tmpDir, 'plan.ts')
+
+  it('提取 apiFetch 调用', () => {
+    mkdirSync(tmpDir, { recursive: true })
+    writeFileSync(apiFile, [
+      'export async function listPlanNodes(params: PageReq): Promise<PlanNode[]> {',
+      '  return apiFetch<PlanNode[]>("/api/ppm/plan-node", { query: params });',
+      '}',
+      '',
+      'export async function getProjectPlan(planId: string): Promise<ProjectPlan> {',
+      '  return apiFetch<ProjectPlan>(`/api/ppm/project-plan/${planId}`);',
+      '}',
+      '',
+      'export async function listPlanNodesByPlan(planId: string): Promise<PsPlanNode[]> {',
+      '  return apiFetch<PsPlanNode[]>(`/api/ppm/project-plan/${planId}/plan-nodes`);',
+      '}',
+      '',
+      'export async function createPlanNode(body: CreateReq): Promise<PlanNode> {',
+      '  return apiFetch<PlanNode>("/api/ppm/plan-node", {',
+      '    method: "POST",',
+      '    json: body,',
+      '  });',
+      '}',
+      '',
+      'export async function deletePlan(id: string): Promise<void> {',
+      '  await apiFetch(`/api/ppm/plan-node/${id}`, { method: "DELETE" });',
+      '}',
+    ].join('\n'), 'utf8')
+
+    const calls = extractFrontendApiCalls(apiFile)
+    assert.ok(calls.length >= 5)
+
+    // GET /api/ppm/plan-node
+    const listCall = calls.find(c => c.raw === '/api/ppm/plan-node' && c.method === 'GET')
+    assert.ok(listCall, 'should find GET /api/ppm/plan-node')
+
+    // GET with template string → 归一化
+    const detailCall = calls.find(c => c.path === '/api/ppm/project-plan/{param}')
+    assert.ok(detailCall, 'should find GET /api/ppm/project-plan/{param}')
+
+    // POST
+    const createCall = calls.find(c => c.method === 'POST')
+    assert.ok(createCall, 'should find POST call')
+
+    // DELETE
+    const deleteCall = calls.find(c => c.method === 'DELETE')
+    assert.ok(deleteCall, 'should find DELETE call')
+  })
+
+  it('模板字符串归一化为 {param}', () => {
+    mkdirSync(tmpDir, { recursive: true })
+    writeFileSync(apiFile, [
+      'const id = "123";',
+      'apiFetch(`/api/users/${id}/profile`);',
+    ].join('\n'), 'utf8')
+
+    const calls = extractFrontendApiCalls(apiFile)
+    assert.equal(calls.length, 1)
+    assert.equal(calls[0].path, '/api/users/{param}/profile')
+  })
+
+  // cleanup
+  it('cleanup', () => {
+    try { rmSync(tmpDir, { recursive: true, force: true }) } catch {}
+  })
+})
+
+// ─── Parity Check ──────────────────────────────────────────────────────
+
+describe('diffApiParity', () => {
+  it('前端调用后端不存在路径时 missingBackend', () => {
+    const frontendCalls = [
+      { method: 'GET', path: '/api/ppm/plan-node', source: 'plan.ts', line: 10 },
+      { method: 'GET', path: '/api/ppm/project-plan/{param}/plan-nodes', source: 'plan.ts', line: 20 },
+    ]
+    const backendEndpoints = [
+      { method: 'GET', path: '/api/ppm/plan-node', source: 'router.py' },
+      // 缺少 /project-plan/{id}/plan-nodes
+    ]
+
+    const result = diffApiParity(frontendCalls, backendEndpoints)
+    assert.equal(result.missingBackend.length, 1)
+    assert.equal(result.missingBackend[0].path, '/api/ppm/project-plan/{param}/plan-nodes')
+    assert.equal(result.unusedBackend.length, 0)
+    assert.equal(result.ok, false)
+  })
+
+  it('全部匹配时 ok', () => {
+    const frontendCalls = [
+      { method: 'GET', path: '/api/ppm/plan-node', source: 'plan.ts', line: 10 },
+    ]
+    const backendEndpoints = [
+      { method: 'GET', path: '/api/ppm/plan-node', source: 'router.py' },
+    ]
+
+    const result = diffApiParity(frontendCalls, backendEndpoints)
+    assert.equal(result.ok, true)
+    assert.equal(result.missingBackend.length, 0)
+  })
+
+  it('后端有但前端未调用的路径在 unusedBackend', () => {
+    const frontendCalls = []
+    const backendEndpoints = [
+      { method: 'GET', path: '/api/ppm/internal/health', source: 'router.py' },
+    ]
+
+    const result = diffApiParity(frontendCalls, backendEndpoints)
+    assert.equal(result.unusedBackend.length, 1)
+    assert.equal(result.unusedBackend[0].path, '/api/ppm/internal/health')
+  })
+})
+
+// ─── Task 分类 ──────────────────────────────────────────────────────────
+
+describe('classifyTask', () => {
+  it('后端 router task 识别为 provider', () => {
+    const content = '## 目标\n实现 plan 子域后端 router，包含 APIRouter 路由注册。'
+    const result = classifyTask(content)
+    assert.ok(result.isProvider)
+  })
+
+  it('前端 API client task 识别为 consumer', () => {
+    const content = '## 目标\n为前端提供统一 API client，使用 apiFetch 封装。'
+    const result = classifyTask(content)
+    assert.ok(result.isConsumer)
+  })
+
+  it('纯文档 task 两者都不是', () => {
+    const content = '## 目标\n更新 README 文档。'
+    const result = classifyTask(content)
+    assert.ok(!result.isProvider || result.confidence < 0.5)
+    assert.ok(!result.isConsumer || result.confidence < 0.5)
+  })
+})
+
+// ─── 集成测试：复现 PPM 真实场景 ───────────────────────────────────────
+
+describe('PPM 真实场景：跨 task 端点漏实现', () => {
+  const tmpDir = join(tmpdir(), 'sillyspec-test-ppm')
+  const routerFile = join(tmpDir, 'router.py')
+  const apiClientFile = join(tmpDir, 'plan.ts')
+
+  it('verify 应该 FAIL：前端调用 /project-plan/{id}/plan-nodes 但后端未实现', () => {
+    mkdirSync(tmpDir, { recursive: true })
+
+    // 模拟 task-04 后端实现：只注册了基础 CRUD，遗漏嵌套端点
+    writeFileSync(routerFile, [
+      'router = APIRouter(prefix="/api/ppm")',
+      '',
+      '@router.get("/plan-node")',
+      'async def list_plan_nodes():',
+      '    pass',
+      '',
+      '@router.post("/plan-node")',
+      'async def create_plan_node():',
+      '    pass',
+      '',
+      '@router.get("/project-plan")',
+      'async def list_project_plans():',
+      '    pass',
+      '',
+      '@router.post("/project-plan")',
+      'async def create_project_plan():',
+      '    pass',
+    ].join('\n'), 'utf8')
+
+    // 模拟 task-09 前端 API client：调用了后端不存在的嵌套端点
+    writeFileSync(apiClientFile, [
+      'export async function listProjectPlans(params: PageReq) {',
+      '  return apiFetch<ProjectPlan[]>("/api/ppm/project-plan", pageQuery(params));',
+      '}',
+      '',
+      'export async function listPlanNodes(planId: string) {',
+      '  // ← 后端没实现这个嵌套端点',
+      '  return apiFetch<PsPlanNode[]>(`/api/ppm/project-plan/${planId}/plan-nodes`);',
+      '}',
+      '',
+      'export async function listPlanNodeDetails(nodeId: string) {',
+      '  // ← 后端也没实现这个',
+      '  return apiFetch<PlanNodeDetail[]>(`/api/ppm/plan-node/${nodeId}/details`);',
+      '}',
+    ].join('\n'), 'utf8')
+
+    // 提取后端端点
+    const backendEndpoints = extractFastApiEndpoints(routerFile)
+    const backendPaths = new Set(backendEndpoints.map(e => normalizePath(e.path)))
+
+    // 提取前端调用
+    const frontendCalls = extractFrontendApiCalls(apiClientFile)
+    const { missingBackend, ok } = diffApiParity(frontendCalls, backendEndpoints)
+
+    // 断言：后端只实现了基础 CRUD
+    assert.ok(backendPaths.has('/api/ppm/plan-node'), '后端应有 GET /plan-node')
+    assert.ok(backendPaths.has('/api/ppm/project-plan'), '后端应有 GET /project-plan')
+
+    // 断言：parity check 失败
+    assert.equal(ok, false, 'verify 应该 FAIL')
+    assert.equal(missingBackend.length, 2, '应发现 2 个缺失端点')
+
+    // 断言：缺失的端点正是 PPM 真实 bug 的那两个
+    const missingPaths = missingBackend.map(m => m.path)
+    assert.ok(
+      missingPaths.includes('/api/ppm/project-plan/{param}/plan-nodes'),
+      '应捕获缺失的 /project-plan/{id}/plan-nodes'
+    )
+    assert.ok(
+      missingPaths.includes('/api/ppm/plan-node/{param}/details'),
+      '应捕获缺失的 /plan-node/{id}/details'
+    )
+
+    // 断言：diff 输出带 source 文件路径
+    const planNodesGap = missingBackend.find(
+      m => m.path === '/api/ppm/project-plan/{param}/plan-nodes'
+    )
+    assert.ok(planNodesGap.consumerFile.includes('plan.ts'), '应带前端文件路径')
+    assert.equal(planNodesGap.consumerLine, 7, '应带行号')
+  })
+
+  it('cleanup', () => {
+    try { rmSync(tmpDir, { recursive: true, force: true }) } catch {}
+  })
+})

package/test/decision-supersede.test.mjs

@@ -0,0 +1,277 @@
+/**
+ * Stage Contract: decisions.md supersede 关系测试
+ *
+ * 验证被 supersede 的旧版本不再被要求引用
+ */
+import { readFileSync } from 'fs'
+import { join } from 'path'
+
+// 直接测试 extractCurrentDecisionIds 的行为
+// 由于它不是 exported，我们通过 validateBrainstormOutputs 间接测试
+
+let failed = 0
+const failures = []
+
+function assert(condition, msg) {
+  if (!condition) {
+    failed++
+    failures.push(msg)
+    console.log(`  ❌ FAIL: ${msg}`)
+  } else {
+    console.log(`  ✅ PASS: ${msg}`)
+  }
+}
+
+console.log('=== decisions.md supersede 关系测试 ===\n')
+
+// 动态导入 stage-contract.js 的内部函数
+// 由于 extractCurrentDecisionIds 不是 exported，我们用 validateBrainstormOutputs 来间接验证
+const contractPath = join(import.meta.dirname, '..', 'src', 'stage-contract.js')
+const contractSource = readFileSync(contractPath, 'utf8')
+
+// 提取并 eval 需要的函数
+// 通过创建临时模块来测试
+const { existsSync, mkdirSync, writeFileSync, mkdtempSync, rmSync } = await import('fs')
+const { join: pJoin } = await import('path')
+const { tmpdir } = await import('os')
+
+// 我们通过 validateBrainstormOutputs 来端到端测试
+const mod = await import(pJoin(import.meta.dirname, '..', 'src', 'stage-contract.js'))
+
+// ─────────────────────────────────────────
+// Test 1: 被 supersede 的旧版本不警告
+// ─────────────────────────────────────────
+console.log('--- Test 1: 被 supersede 的旧版本不警告 ---')
+{
+  const tmpDir = mkdtempSync(join(tmpdir(), 'sillyspec-supersede-'))
+  // 模拟 .sillyspec/changes 结构
+  const changeDir = pJoin(tmpDir, '.sillyspec', 'changes', '2026-06-25-test')
+  mkdirSync(changeDir, { recursive: true })
+
+  // decisions.md: D-004@v2 supersedes D-004@v1
+  writeFileSync(pJoin(changeDir, 'decisions.md'), `# Decisions
+
+## D-004@v1
+- status: accepted
+- priority: P1
+
+## D-004@v2
+- status: accepted
+- priority: P1
+- supersedes: D-004@v1
+`)
+
+  // design.md 只引用 D-004@v2（不引用 v1）
+  writeFileSync(pJoin(changeDir, 'design.md'), `# Design
+
+## 目标
+修复 bug。
+
+## 方案
+用方案 B。
+
+## 决策
+D-004@v2: 选择方案 B
+`)
+
+  writeFileSync(pJoin(changeDir, 'requirements.md'), `# Requirements
+
+## FR-01
+需求 A。
+
+## 决策覆盖
+- D-004@v2 → FR-01
+`)
+
+  writeFileSync(pJoin(changeDir, 'tasks.md'), `# Tasks
+- task-01: 实现
+`)
+
+  // 用 validateBrainstormOutputs 验证
+  const result = mod.runValidators("brainstorm", tmpDir, '2026-06-25-test')
+
+  // 不应有关于 D-004@v1 的未引用警告
+  const v1Warnings = result.warnings.filter(w => w.includes('D-004@V1'))
+  assert(v1Warnings.length === 0, `不应有 D-004@V1 未引用警告，实际 warnings: ${JSON.stringify(v1Warnings)}`)
+
+  rmSync(tmpDir, { recursive: true, force: true })
+}
+
+// ─────────────────────────────────────────
+// Test 2: 没有 supersede 关系时，旧版本仍被校验
+// ─────────────────────────────────────────
+console.log('\n--- Test 2: 无 supersede 关系时旧版本仍校验 ---')
+{
+  const tmpDir = mkdtempSync(join(tmpdir(), 'sillyspec-nosupersede-'))
+  const changeDir = pJoin(tmpDir, '.sillyspec', 'changes', '2026-06-25-test')
+  mkdirSync(changeDir, { recursive: true })
+
+  // decisions.md: 两个独立决策，无 supersede 关系
+  writeFileSync(pJoin(changeDir, 'decisions.md'), `# Decisions
+
+## D-004@v1
+- status: accepted
+- priority: P1
+
+## D-005@v1
+- status: accepted
+- priority: P1
+`)
+
+  // design.md 只引用 D-004@v1，不引用 D-005@v1
+  writeFileSync(pJoin(changeDir, 'design.md'), `# Design
+
+## 目标
+修复 bug。
+
+## 方案
+用方案 B。
+
+## 决策
+D-004@v1: 选择方案 B
+## 文件变更清单
+| 操作 | 文件 | 说明 |
+## 风险登记
+- 风险 A
+## 自审
+- 已检查
+`)
+
+  writeFileSync(pJoin(changeDir, 'requirements.md'), `# Requirements
+## FR-01
+需求 A。
+`)
+
+  writeFileSync(pJoin(changeDir, 'proposal.md'), `# Proposal
+## 不在范围内
+- 无
+`)
+  writeFileSync(pJoin(changeDir, 'tasks.md'), `# Tasks
+- task-01: 实现
+`)
+
+  const result = mod.runValidators("brainstorm", tmpDir, '2026-06-25-test')
+
+  // D-005@v1 应该有未引用警告
+  const v5Warnings = result.warnings.filter(w => w.includes('D-005@V1'))
+  assert(v5Warnings.length > 0, `应有 D-005@V1 未引用警告`)
+
+  rmSync(tmpDir, { recursive: true, force: true })
+}
+
+// ─────────────────────────────────────────
+// Test 3: status=superseded 的旧版本不警告（原有行为，回归测试）
+// ─────────────────────────────────────────
+console.log('\n--- Test 3: status=superseded 的旧版本不警告（回归） ---')
+{
+  const tmpDir = mkdtempSync(join(tmpdir(), 'sillyspec-status-'))
+  const changeDir = pJoin(tmpDir, '.sillyspec', 'changes', '2026-06-25-test')
+  mkdirSync(changeDir, { recursive: true })
+
+  writeFileSync(pJoin(changeDir, 'decisions.md'), `# Decisions
+
+## D-004@v1
+- status: superseded
+- priority: P1
+
+## D-004@v2
+- status: accepted
+- priority: P1
+`)
+
+  writeFileSync(pJoin(changeDir, 'design.md'), `# Design
+
+## 目标
+修复 bug。
+
+## 方案
+用方案 B。
+
+## 决策
+D-004@v2: 选择方案 B
+`)
+
+  writeFileSync(pJoin(changeDir, 'requirements.md'), `# Requirements
+## FR-01
+需求 A。
+`)
+
+  writeFileSync(pJoin(changeDir, 'tasks.md'), `# Tasks
+- task-01: 实现
+`)
+
+  const result = mod.runValidators("brainstorm", tmpDir, '2026-06-25-test')
+
+  const v1Warnings = result.warnings.filter(w => w.includes('D-004@V1'))
+  assert(v1Warnings.length === 0, `status=superseded 的 D-004@V1 不应警告`)
+
+  rmSync(tmpDir, { recursive: true, force: true })
+}
+
+// ─────────────────────────────────────────
+// Test 4: 多级 supersede 链（v1 ← v2 ← v3）
+// ─────────────────────────────────────────
+console.log('\n--- Test 4: 多级 supersede 链 ---')
+{
+  const tmpDir = mkdtempSync(join(tmpdir(), 'sillyspec-chain-'))
+  const changeDir = pJoin(tmpDir, '.sillyspec', 'changes', '2026-06-25-test')
+  mkdirSync(changeDir, { recursive: true })
+
+  writeFileSync(pJoin(changeDir, 'decisions.md'), `# Decisions
+
+## D-004@v1
+- status: accepted
+- priority: P1
+
+## D-004@v2
+- status: accepted
+- priority: P1
+- supersedes: D-004@v1
+
+## D-004@v3
+- status: accepted
+- priority: P1
+- supersedes: D-004@v2
+`)
+
+  // design.md 只引用 D-004@v3
+  writeFileSync(pJoin(changeDir, 'design.md'), `# Design
+
+## 目标
+修复 bug。
+
+## 方案
+用方案 C。
+
+## 决策
+D-004@v3: 选择方案 C
+`)
+
+  writeFileSync(pJoin(changeDir, 'requirements.md'), `# Requirements
+## FR-01
+需求 A。
+- D-004@v3 → FR-01
+`)
+
+  writeFileSync(pJoin(changeDir, 'tasks.md'), `# Tasks
+- task-01: 实现
+`)
+
+  const result = mod.runValidators("brainstorm", tmpDir, '2026-06-25-test')
+
+  const oldWarnings = result.warnings.filter(w => w.includes('D-004@V1') || w.includes('D-004@V2'))
+  assert(oldWarnings.length === 0, `多级 supersede 链中 v1/v2 不应警告，实际: ${JSON.stringify(oldWarnings)}`)
+
+  rmSync(tmpDir, { recursive: true, force: true })
+}
+
+// ── 结果 ──
+console.log(`\n${'='.repeat(50)}`)
+console.log(`✅ 通过: ${4 - failed}  ❌ 失败: ${failed}`)
+if (failures.length > 0) {
+  console.log(`失败项:`)
+  failures.forEach(f => console.log(`  - ${f}`))
+}
+console.log(`${'='.repeat(50)}`)
+
+if (failed > 0) process.exit(1)