cc-devflow 4.5.10 → 4.5.12

package/lib/skill-runtime/__tests__/autopilot.test.js

@@ -1,253 +0,0 @@
-const fs = require('fs');
-const os = require('os');
-const path = require('path');
-
-const { runAutopilot } = require('../operations/autopilot');
-const { runApprove } = require('../operations/approve');
-const {
-  getTasksMarkdownPath,
-  getTaskManifestPath,
-  getReportCardPath,
-  getReleaseNotePath,
-  getRuntimeStatePath
-} = require('../store');
-const {
-  getIntentResumeIndexPath,
-  getIntentPrBriefPath
-} = require('../artifacts');
-const { getChangePaths } = require('../paths');
-
-jest.setTimeout(20000);
-
-function writeJson(filePath, value) {
-  fs.mkdirSync(path.dirname(filePath), { recursive: true });
-  fs.writeFileSync(filePath, `${JSON.stringify(value, null, 2)}\n`);
-}
-
-function markManifestReviewsPassed(repoRoot, changeId) {
-  const manifestPath = getTaskManifestPath(repoRoot, changeId);
-  const manifest = JSON.parse(fs.readFileSync(manifestPath, 'utf8'));
-  manifest.tasks = manifest.tasks.map((task) => ({
-    ...task,
-    reviews: {
-      spec: 'pass',
-      code: 'pass'
-    }
-  }));
-  manifest.spec = manifest.spec || {
-    primaryCapability: 'autopilot-runtime',
-    specFiles: ['devflow/specs/autopilot-runtime.md']
-  };
-  fs.writeFileSync(manifestPath, `${JSON.stringify(manifest, null, 2)}\n`);
-}
-
-function writeCleanReviewLedger(repoRoot, changeId) {
-  const change = getChangePaths(repoRoot, changeId);
-  const ledgerPath = path.join(change.reviewDir, 'review-ledger.jsonl');
-  fs.mkdirSync(path.dirname(ledgerPath), { recursive: true });
-  fs.writeFileSync(ledgerPath, [
-    JSON.stringify({
-      schema: 'review-ledger.v2',
-      change: change.changeKey,
-      reviewId: 'RVW-20260512-001',
-      createdAt: '2026-05-12T00:00:00.000Z',
-      createdBy: 'cc-devflow-cli',
-      event: 'review-started',
-      mode: 'implementation',
-      scope: 'current-diff',
-      baseSha: 'abc123',
-      headSha: 'def456',
-      selectedNodes: [],
-      skippedNodes: [],
-      riskLanes: []
-    }),
-    JSON.stringify({
-      schema: 'review-ledger.v2',
-      change: change.changeKey,
-      reviewId: 'RVW-20260512-001',
-      createdAt: '2026-05-12T00:01:00.000Z',
-      createdBy: 'cc-devflow-cli',
-      event: 'review-closed',
-      status: 'clean',
-      blockingCount: 0,
-      warningCount: 0,
-      next: 'cc-check'
-    })
-  ].join('\n'));
-}
-
-describe('runAutopilot', () => {
-  test('stops at the approval gate after planning without writing approval-phase handoff markdown', async () => {
-    const repoRoot = fs.mkdtempSync(path.join(os.tmpdir(), 'cc-devflow-autopilot-'));
-
-    writeJson(path.join(repoRoot, 'package.json'), {
-      name: 'autopilot-smoke',
-      version: '1.0.0',
-      scripts: {
-        lint: 'node -e "process.exit(0)"',
-        typecheck: 'node -e "process.exit(0)"',
-        test: 'node -e "process.exit(0)"'
-      }
-    });
-
-    const goal = 'Ship thin autopilot loop';
-    fs.mkdirSync(path.dirname(getTasksMarkdownPath(repoRoot, 'REQ-123', { goal })), { recursive: true });
-    fs.writeFileSync(
-      getTasksMarkdownPath(repoRoot, 'REQ-123', { goal }),
-      [
-        '- [ ] T001 启动 autopilot 闭环 (src/a.ts)',
-        '- [ ] T002 多文件委派任务 [P] (src/a.ts,src/b.ts)'
-      ].join('\n')
-    );
-
-    const result = await runAutopilot({
-      repoRoot,
-      changeId: 'REQ-123',
-      goal
-    });
-
-    const runtimeState = JSON.parse(fs.readFileSync(getRuntimeStatePath(repoRoot, 'REQ-123'), 'utf8'));
-
-    expect(result.executed).toEqual(['init', 'snapshot', 'plan']);
-    expect(result.currentStage).toBe('approve');
-    expect(runtimeState.changeKey).toBe('REQ-123-ship-thin-autopilot-loop');
-    expect(fs.existsSync(getReportCardPath(repoRoot, 'REQ-123'))).toBe(false);
-    expect(fs.existsSync(getIntentPrBriefPath(repoRoot, 'REQ-123'))).toBe(false);
-    expect(fs.existsSync(getIntentResumeIndexPath(repoRoot, 'REQ-123'))).toBe(false);
-  });
-
-  test('resumes after approval, executes delegated work, and prepares a PR from task state', async () => {
-    const repoRoot = fs.mkdtempSync(path.join(os.tmpdir(), 'cc-devflow-autopilot-workers-'));
-
-    writeJson(path.join(repoRoot, 'package.json'), {
-      name: 'autopilot-worker-smoke',
-      version: '1.0.0',
-      scripts: {
-        lint: 'node -e "process.exit(0)"',
-        typecheck: 'node -e "process.exit(0)"',
-        test: 'node -e "process.exit(0)"'
-      }
-    });
-
-    const goal = 'Ship thin autopilot loop with delegated workers';
-    fs.mkdirSync(path.dirname(getTasksMarkdownPath(repoRoot, 'REQ-123', { goal })), { recursive: true });
-    fs.writeFileSync(
-      getTasksMarkdownPath(repoRoot, 'REQ-123', { goal }),
-      [
-        '- [ ] T001 直接执行任务 (src/a.ts)',
-        '- [ ] T002 委派实现任务 [P] (src/a.ts,src/b.ts)'
-      ].join('\n')
-    );
-
-    await runAutopilot({
-      repoRoot,
-      changeId: 'REQ-123',
-      goal
-    });
-
-    await runApprove({
-      repoRoot,
-      changeId: 'REQ-123',
-      executionMode: 'delegate'
-    });
-
-    const firstRun = await runAutopilot({
-      repoRoot,
-      changeId: 'REQ-123',
-      workerCommand: 'printf "delegate-ok"'
-    });
-
-    const manifest = JSON.parse(fs.readFileSync(getTaskManifestPath(repoRoot, 'REQ-123'), 'utf8'));
-    const report = JSON.parse(fs.readFileSync(getReportCardPath(repoRoot, 'REQ-123'), 'utf8'));
-
-    expect(firstRun.executed).toEqual(expect.arrayContaining(['delegate', 'worker-run', 'dispatch', 'verify']));
-    expect(firstRun.currentStage).toBe('verify');
-    expect(manifest.tasks.find((task) => task.id === 'T002').status).toBe('passed');
-    expect(report.review.status).toBe('blocked');
-    expect(fs.existsSync(getIntentPrBriefPath(repoRoot, 'REQ-123'))).toBe(false);
-
-    markManifestReviewsPassed(repoRoot, 'REQ-123');
-    writeCleanReviewLedger(repoRoot, 'REQ-123');
-
-    const secondRun = await runAutopilot({
-      repoRoot,
-      changeId: 'REQ-123',
-      from: 'verify',
-      skipReview: true
-    });
-
-    const prBrief = fs.readFileSync(getIntentPrBriefPath(repoRoot, 'REQ-123'), 'utf8');
-
-    expect(secondRun.executed).toEqual(expect.arrayContaining(['verify', 'prepare-pr']));
-    expect(secondRun.currentStage).toBe('prepare-pr');
-    expect(prBrief).toContain('planning/task-manifest.json');
-    expect(prBrief).not.toContain('checkpoint.json');
-  });
-
-  test('runs release after prepare-pr when requested for an approved plan', async () => {
-    const repoRoot = fs.mkdtempSync(path.join(os.tmpdir(), 'cc-devflow-autopilot-release-'));
-
-    writeJson(path.join(repoRoot, 'package.json'), {
-      name: 'autopilot-release-smoke',
-      version: '1.0.0',
-      scripts: {
-        lint: 'node -e "process.exit(0)"',
-        typecheck: 'node -e "process.exit(0)"',
-        test: 'node -e "process.exit(0)"'
-      }
-    });
-
-    const goal = 'Ship thin autopilot loop and release';
-    fs.mkdirSync(path.dirname(getTasksMarkdownPath(repoRoot, 'REQ-123', { goal })), { recursive: true });
-    fs.writeFileSync(
-      getTasksMarkdownPath(repoRoot, 'REQ-123', { goal }),
-      '- [ ] T001 发布前收尾 (src/a.ts)'
-    );
-
-    await runAutopilot({
-      repoRoot,
-      changeId: 'REQ-123',
-      goal
-    });
-
-    await runApprove({
-      repoRoot,
-      changeId: 'REQ-123',
-      executionMode: 'direct'
-    });
-
-    const firstRun = await runAutopilot({
-      repoRoot,
-      changeId: 'REQ-123',
-      release: true
-    });
-
-    const manifest = JSON.parse(fs.readFileSync(getTaskManifestPath(repoRoot, 'REQ-123'), 'utf8'));
-    const report = JSON.parse(fs.readFileSync(getReportCardPath(repoRoot, 'REQ-123'), 'utf8'));
-
-    expect(firstRun.currentStage).toBe('verify');
-    expect(report.review.status).toBe('blocked');
-    expect(fs.existsSync(getReleaseNotePath(repoRoot, 'REQ-123'))).toBe(false);
-
-    markManifestReviewsPassed(repoRoot, 'REQ-123');
-    writeCleanReviewLedger(repoRoot, 'REQ-123');
-
-    const result = await runAutopilot({
-      repoRoot,
-      changeId: 'REQ-123',
-      from: 'verify',
-      release: true,
-      skipReview: true
-    });
-
-    const runtimeState = JSON.parse(fs.readFileSync(getRuntimeStatePath(repoRoot, 'REQ-123'), 'utf8'));
-    const releaseNote = fs.readFileSync(getReleaseNotePath(repoRoot, 'REQ-123'), 'utf8');
-
-    expect(result.executed).toEqual(expect.arrayContaining(['verify', 'prepare-pr', 'release']));
-    expect(result.lifecycleStatus).toBe('released');
-    expect(runtimeState.status).toBe('released');
-    expect(releaseNote).toContain('Release Note - REQ-123');
-    expect(fs.existsSync(getIntentPrBriefPath(repoRoot, 'REQ-123'))).toBe(false);
-    expect(fs.existsSync(getIntentResumeIndexPath(repoRoot, 'REQ-123'))).toBe(false);
-  });
-});

package/lib/skill-runtime/__tests__/benchmark-artifacts.test.js

@@ -1,165 +0,0 @@
-/**
- * [INPUT]: 依赖 scripts/benchmark-artifacts.js 导出的 runBenchmarkArtifacts 和临时 artifact fixture。
- * [OUTPUT]: 验证 benchmark:artifacts 使用 ceil(len/4) 估算并报告 profile 阈值 savings。
- * [POS]: REQ-003-minimize-workflow-artifacts T017 的 Red/Green 证据。
- * [PROTOCOL]: 变更时更新此头部，然后检查 CLAUDE.md
- */
-
-const fs = require('fs');
-const os = require('os');
-const path = require('path');
-const { spawnSync } = require('child_process');
-
-const { runBenchmarkArtifacts } = require('../../../scripts/benchmark-artifacts');
-
-const REPO_ROOT = path.resolve(__dirname, '../../..');
-const BENCHMARK_SCRIPT = path.join(REPO_ROOT, 'scripts', 'benchmark-artifacts.js');
-
-function writeText(filePath, text) {
-  fs.mkdirSync(path.dirname(filePath), { recursive: true });
-  fs.writeFileSync(filePath, text);
-}
-
-function writeJson(filePath, value) {
-  writeText(filePath, `${JSON.stringify(value, null, 2)}\n`);
-}
-
-function contractTasks({ changeKey, profile = 'standard', filler = '' }) {
-  return [
-    '# Tasks',
-    '',
-    '## Contract Summary',
-    '',
-    `Change: ${changeKey}`,
-    'Mode: plan',
-    `Profile: ${profile}`,
-    'Approval: approved',
-    '',
-    'Goal:',
-    '- Minimize workflow artifacts.',
-    '',
-    'Do Not Do:',
-    '- Do not change token estimator math.',
-    '',
-    'Approved Direction:',
-    '- Use tasks.md plus generated JSON records.',
-    '',
-    'Acceptance:',
-    '- Benchmark savings stay above threshold.',
-    '',
-    'Verification:',
-    '',
-    '```bash',
-    'npm run benchmark:artifacts',
-    '```',
-    '',
-    'Risk / Escalate If:',
-    '- Savings fall below profile threshold.',
-    '',
-    filler,
-    '## Phase 1',
-    '',
-    '- [ ] T001 benchmark minimized artifact surface',
-    '  Vertical slice: Slice 1',
-    ''
-  ].join('\n');
-}
-
-function seedLegacyBaseline(repoRoot, changeKey, size = 6000) {
-  const changeDir = path.join(repoRoot, 'devflow', 'changes', changeKey);
-  writeText(path.join(changeDir, 'planning', 'design.md'), `# Design\n\n${'d'.repeat(size)}\n`);
-  writeText(path.join(changeDir, 'planning', 'analysis.md'), `# Analysis\n\n${'a'.repeat(size / 2)}\n`);
-  writeText(path.join(changeDir, 'planning', 'tasks.md'), `# Tasks\n\n${'t'.repeat(size / 2)}\n`);
-  writeJson(path.join(changeDir, 'planning', 'task-manifest.json'), { changeId: changeKey, tasks: [] });
-  writeJson(path.join(changeDir, 'change-meta.json'), { changeId: changeKey, goal: ['legacy'] });
-  writeJson(path.join(changeDir, 'review', 'report-card.json'), { overall: 'pass' });
-}
-
-function seedMinimizedChange(repoRoot, changeKey, options = {}) {
-  const changeDir = path.join(repoRoot, 'devflow', 'changes', changeKey);
-  writeText(path.join(changeDir, 'planning', 'tasks.md'), contractTasks({ changeKey, ...options }));
-  writeJson(path.join(changeDir, 'planning', 'task-manifest.json'), {
-    changeId: changeKey,
-    metadata: { source: 'tasks.md', generatedBy: 'cc-devflow task-contract', planVersion: 1 },
-    tasks: []
-  });
-  writeJson(path.join(changeDir, 'change-meta.json'), {
-    changeId: changeKey,
-    _meta: { generatedBy: 'cc-devflow task-contract' }
-  });
-  writeJson(path.join(changeDir, 'review', 'review-ledger.jsonl'), { note: 'counted as text by benchmark' });
-  writeJson(path.join(changeDir, 'review', 'review-findings.json'), { findings: [] });
-  writeJson(path.join(changeDir, 'review', 'report-card.json'), { overall: 'pass' });
-}
-
-describe('benchmark:artifacts', () => {
-  let repoRoot;
-
-  beforeEach(() => {
-    repoRoot = fs.mkdtempSync(path.join(os.tmpdir(), 'cc-devflow-benchmark-artifacts-'));
-    seedLegacyBaseline(repoRoot, 'REQ-001-legacy-baseline');
-    seedLegacyBaseline(repoRoot, 'REQ-002-legacy-baseline');
-  });
-
-  afterEach(() => {
-    fs.rmSync(repoRoot, { recursive: true, force: true });
-  });
-
-  test('reports standard savings >= 30% for REQ-003-example', () => {
-    seedMinimizedChange(repoRoot, 'REQ-003-example', { profile: 'standard' });
-
-    const result = runBenchmarkArtifacts(repoRoot);
-    const row = result.rows.find((item) => item.changeKey === 'REQ-003-example');
-
-    expect(result.code).toBe(0);
-    expect(row).toMatchObject({
-      profile: 'standard',
-      threshold_pct: 30,
-      correctness_pass: true
-    });
-    expect(row.savings_vs_baseline_pct).toBeGreaterThanOrEqual(30);
-  });
-
-  test('reports tiny savings >= 60% for tiny fixture', () => {
-    seedMinimizedChange(repoRoot, 'REQ-004-tiny-example', { profile: 'tiny' });
-
-    const result = runBenchmarkArtifacts(repoRoot);
-    const row = result.rows.find((item) => item.changeKey === 'REQ-004-tiny-example');
-
-    expect(result.code).toBe(0);
-    expect(row).toMatchObject({
-      profile: 'tiny',
-      threshold_pct: 60,
-      correctness_pass: true
-    });
-    expect(row.savings_vs_baseline_pct).toBeGreaterThanOrEqual(60);
-  });
-
-  test('exits 1 when savings are below the profile threshold', () => {
-    seedMinimizedChange(repoRoot, 'REQ-005-bloated-example', {
-      profile: 'standard',
-      filler: 'x'.repeat(20000)
-    });
-
-    const result = runBenchmarkArtifacts(repoRoot);
-
-    expect(result.code).toBe(1);
-    expect(result.rows[0]).toMatchObject({ correctness_pass: false });
-  });
-
-  test('CLI prints stdout JSON array', () => {
-    seedMinimizedChange(repoRoot, 'REQ-003-example', { profile: 'standard' });
-
-    const result = spawnSync(process.execPath, [BENCHMARK_SCRIPT, repoRoot], { encoding: 'utf8' });
-    const rows = JSON.parse(result.stdout);
-
-    expect(result.status).toBe(0);
-    expect(Array.isArray(rows)).toBe(true);
-    expect(rows[0]).toHaveProperty('savings_vs_baseline_pct');
-  });
-
-  test('package.json exposes npm run benchmark:artifacts', () => {
-    const pkg = JSON.parse(fs.readFileSync(path.join(REPO_ROOT, 'package.json'), 'utf8'));
-    expect(pkg.scripts['benchmark:artifacts']).toBe('node scripts/benchmark-artifacts.js');
-  });
-});

package/lib/skill-runtime/__tests__/delegation.test.js

@@ -1,97 +0,0 @@
-const fs = require('fs');
-const os = require('os');
-const path = require('path');
-
-const {
-  syncDelegationRuntime,
-  ensureWorkerWorkspace,
-  getWorkerStatePath,
-  getWorkerJournalPath,
-  getWorkerAssignmentPath,
-  getMessageBusPath,
-  getTeamMemoryPath,
-  getWorkerWorkspacePath
-} = require('../delegation');
-const { getTeamStatePath } = require('../team-state');
-
-describe('syncDelegationRuntime', () => {
-  test('writes workspace scratch files and team-state truth without polluting devflow/changes', async () => {
-    const repoRoot = fs.mkdtempSync(path.join(os.tmpdir(), 'cc-devflow-delegation-'));
-    fs.writeFileSync(path.join(repoRoot, 'package.json'), JSON.stringify({ name: 'delegation-smoke', version: '1.0.0' }, null, 2));
-
-    const manifest = {
-      changeId: 'REQ-123',
-      goal: 'Delegate selected tasks',
-      createdAt: '2026-03-25T01:00:00.000Z',
-      updatedAt: '2026-03-25T01:01:00.000Z',
-      tasks: [
-        {
-          id: 'T001',
-          title: 'Single file direct task',
-          type: 'IMPL',
-          dependsOn: [],
-          touches: ['src/a.ts'],
-          run: ['echo direct'],
-          checks: [],
-          status: 'pending',
-          attempts: 0,
-          maxRetries: 1
-        },
-        {
-          id: 'T002',
-          title: 'Multi file delegated task',
-          type: 'IMPL',
-          dependsOn: [],
-          touches: ['src/a.ts', 'src/b.ts'],
-          run: ['echo delegated'],
-          checks: [],
-          status: 'pending',
-          attempts: 0,
-          maxRetries: 1
-        }
-      ],
-      metadata: {
-        source: 'default',
-        generatedBy: 'test',
-        planVersion: 3
-      }
-    };
-
-    const result = await syncDelegationRuntime(repoRoot, 'REQ-123', manifest);
-
-    const teamState = JSON.parse(fs.readFileSync(getTeamStatePath(repoRoot, 'REQ-123'), 'utf8'));
-    const workerId = teamState.team.taskAssignments.T002;
-    const assignment = fs.readFileSync(getWorkerAssignmentPath(repoRoot, 'REQ-123', workerId), 'utf8');
-    const messageBus = fs.readFileSync(getMessageBusPath(repoRoot, 'REQ-123'), 'utf8');
-    const teamMemory = fs.readFileSync(getTeamMemoryPath(repoRoot, 'REQ-123'), 'utf8');
-    const workerState = fs.readFileSync(getWorkerStatePath(repoRoot, 'REQ-123', workerId), 'utf8');
-    const workerJournal = fs.readFileSync(getWorkerJournalPath(repoRoot, 'REQ-123', workerId), 'utf8');
-    const workerWorkspace = getWorkerWorkspacePath(repoRoot, 'REQ-123', workerId);
-
-    expect(result.delegatedCount).toBe(1);
-    expect(assignment).toContain(`Assignment: ${workerId}`);
-    expect(assignment).toContain('`T002` Multi file delegated task');
-    expect(messageBus).toContain('queued for T002');
-    expect(teamMemory).toContain('Worker Count: 1');
-    expect(teamState.planVersion).toBe(3);
-    expect(workerId).toMatch(/^delegate-/);
-    expect(workerState).toContain('Status: `idle`');
-    expect(workerJournal).toContain('Assigned Tasks: T002');
-    expect(workerWorkspace).toContain('devflow/workspaces');
-    expect(fs.existsSync(path.join(repoRoot, 'devflow/changes/REQ-123-delegate-selected-tasks/execution/workers'))).toBe(false);
-  });
-
-  test('falls back to controller workspace when git worktree is unavailable', async () => {
-    const repoRoot = fs.mkdtempSync(path.join(os.tmpdir(), 'cc-devflow-no-git-'));
-    const worker = await ensureWorkerWorkspace(repoRoot, 'REQ-123', {
-      taskId: 'T001',
-      workerId: 'delegate-01',
-      role: 'dev-implementer',
-      route: 'delegate'
-    });
-
-    expect(worker.mode).toBe('fallback');
-    expect(worker.cwd).toBe(repoRoot);
-    expect(worker.reason).toBe('git-unavailable');
-  });
-});