@dommaker/harness 1.3.0 → 1.5.0

package/src/cli/commands/__tests__/validate.test.ts

@@ -3,6 +3,7 @@
  */
 
 import { validate, createExampleCheckpoint, createExampleResolutions } from '../validate';
+import { captureIO, type CapturingIO } from '../../command-contract';
 import * as fs from 'fs/promises';
 import { CheckpointValidator } from '../../../core/validators/checkpoint';
 import * as yaml from 'js-yaml';
@@ -42,25 +43,22 @@ const MockCheckpointValidator = CheckpointValidator as jest.Mocked<typeof Checkp
 const mockYaml = yaml as jest.Mocked<typeof yaml>;
 
 describe('validate command', () => {
-  let consoleSpy: jest.SpyInstance;
+  let io: CapturingIO;
 
   beforeEach(() => {
-    jest.clearAllMocks();
-    consoleSpy = jest.spyOn(console, 'log').mockImplementation();
-    process.exitCode = 0;
-  });
 
-  afterEach(() => {
-    consoleSpy.mockRestore();
-    process.exitCode = 0;
+    io = captureIO();
+    jest.clearAllMocks();
   });
 
   describe('validate', () => {
     it('应该跳过无检查点的情况', async () => {
       mockFs.readFile.mockRejectedValue(new Error('file not found'));
       
-      await validate({});
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('没有定义检查点'));
+      const result = await validate({}, io);
+
+      expect(io.outText()).toContain('没有定义检查点');
+      expect(result).toEqual({ kind: 'skip', reason: expect.stringContaining('没有定义检查点') });
     });
 
     it('应该通过所有检查点', async () => {
@@ -76,8 +74,10 @@ describe('validate command', () => {
       };
       (MockCheckpointValidator.getInstance as jest.Mock).mockReturnValue(mockValidator);
 
-      await validate({});
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('所有检查点验证通过'));
+      const result = await validate({}, io);
+
+      expect(io.outText()).toContain('所有检查点验证通过');
+      expect(result).toEqual({ kind: 'ok' });
     });
 
     it('应该显示失败的检查点', async () => {
@@ -97,12 +97,11 @@ describe('validate command', () => {
       (MockCheckpointValidator.getInstance as jest.Mock).mockReturnValue(mockValidator);
 
       // 工单 23：检查点失败一律 exit 1（门控语义）
-      const mockExit = jest.spyOn(process, 'exit').mockImplementation(() => undefined as never);
 
-      await validate({});
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('失败'));
-      expect(mockExit).toHaveBeenCalledWith(1);
-      mockExit.mockRestore();
+      const result = await validate({}, io);
+
+      expect(io.outText()).toContain('失败');
+      expect(result).toEqual({ kind: 'fail', reason: '1 个检查点未通过: test-1' });
     });
 
     it('应该在严格模式下退出', async () => {
@@ -121,11 +120,10 @@ describe('validate command', () => {
       };
       (MockCheckpointValidator.getInstance as jest.Mock).mockReturnValue(mockValidator);
 
-      const mockExit = jest.spyOn(process, 'exit').mockImplementation(() => undefined as never);
+      const result = await validate({ strict: true }, io);
 
-      await validate({ strict: true });
-      expect(mockExit).toHaveBeenCalledWith(1);
-      mockExit.mockRestore();
+      // 工单 23 语义：--strict 与否都是 fail；退出码映射在 bin
+      expect(result).toEqual({ kind: 'fail', reason: '1 个检查点未通过: test-1' });
     });
   });
 
@@ -136,20 +134,20 @@ describe('validate command', () => {
       mockFs.writeFile.mockResolvedValue(undefined);
       mockYaml.dump.mockReturnValue('yaml content');
 
-      await createExampleCheckpoint('/project');
+      await createExampleCheckpoint('/project', io);
       
       expect(mockFs.mkdir).toHaveBeenCalled();
       expect(mockFs.writeFile).toHaveBeenCalled();
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('已创建示例检查点'));
+      expect(io.outText()).toContain('已创建示例检查点');
     });
 
     it('已存在时不覆盖', async () => {
       mockFs.access.mockResolvedValue(undefined);
 
-      await createExampleCheckpoint('/project');
+      await createExampleCheckpoint('/project', io);
 
       expect(mockFs.writeFile).not.toHaveBeenCalled();
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('checkpoints.yml 已存在，跳过'));
+      expect(io.outText()).toContain('checkpoints.yml 已存在，跳过');
     });
   });
 
@@ -159,20 +157,20 @@ describe('validate command', () => {
       mockFs.mkdir.mockResolvedValue(undefined);
       mockFs.writeFile.mockResolvedValue(undefined);
 
-      await createExampleResolutions('/project');
+      await createExampleResolutions('/project', io);
 
       expect(mockFs.mkdir).toHaveBeenCalled();
       expect(mockFs.writeFile).toHaveBeenCalled();
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('已创建 Resolutions 文件'));
+      expect(io.outText()).toContain('已创建 Resolutions 文件');
     });
 
     it('已存在时不覆盖', async () => {
       mockFs.access.mockResolvedValue(undefined);
 
-      await createExampleResolutions('/project');
+      await createExampleResolutions('/project', io);
 
       expect(mockFs.writeFile).not.toHaveBeenCalled();
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('resolutions.json 已存在，跳过'));
+      expect(io.outText()).toContain('resolutions.json 已存在，跳过');
     });
   });
 });

package/src/cli/commands/acceptance.ts

@@ -8,6 +8,8 @@ import chalk from 'chalk';
 import * as fs from 'fs/promises';
 import * as path from 'path';
 import { SpecAcceptanceGate } from '../../gates/acceptance';
+import { log, processIO, type CommandIO, type CommandResult } from '../command-contract';
+
 
 export interface AcceptanceOptions {
   /** 任务 ID */
@@ -25,8 +27,11 @@ export interface AcceptanceOptions {
 /**
  * 执行验收门控
  */
-export async function acceptance(options: AcceptanceOptions): Promise<void> {
-  console.log(chalk.blue('📋 验收标准门控检查...'));
+export async function acceptance(
+  options: AcceptanceOptions,
+  io: CommandIO = processIO,
+): Promise<CommandResult> {
+  log(io, chalk.blue('📋 验收标准门控检查...'));
 
   const projectPath = options.projectPath || process.cwd();
 
@@ -47,36 +52,40 @@ export async function acceptance(options: AcceptanceOptions): Promise<void> {
     const result = await gate.check(context as any);
 
     if (result.passed) {
-      console.log();
-      console.log(chalk.green('✅ 验收标准检查通过'));
+      log(io);
+      log(io, chalk.green('✅ 验收标准检查通过'));
       if (result.details) {
-        console.log(chalk.gray(`   通过项: ${result.details.checkedCriteria ?? 0}`));
-        console.log(chalk.gray(`   总项数: ${result.details.totalCriteria ?? 0}`));
+        log(io, chalk.gray(`   通过项: ${result.details.checkedCriteria ?? 0}`));
+        log(io, chalk.gray(`   总项数: ${result.details.totalCriteria ?? 0}`));
       }
     } else {
-      console.log();
-      console.log(chalk.red('❌ 验收标准检查失败'));
-      console.log(chalk.red(`   ${result.message}`));
+      log(io);
+      log(io, chalk.red('❌ 验收标准检查失败'));
+      log(io, chalk.red(`   ${result.message}`));
       if (result.details?.uncheckedCriteria) {
         (result.details.uncheckedCriteria as string[]).forEach((criteria: string) => {
-          console.log(chalk.red(`   - ${criteria}`));
+          log(io, chalk.red(`   - ${criteria}`));
         });
       }
-      process.exit(1);
+      return { kind: 'fail', reason: `acceptance gate denied: ${result.message}` };
     }
+    return { kind: 'ok' };
   } catch (error: any) {
-    console.log();
-    console.log(chalk.red('❌ 验收标准检查出错'));
-    console.log(chalk.red(`   ${error.message}`));
-    process.exit(1);
+    log(io);
+    log(io, chalk.red('❌ 验收标准检查出错'));
+    log(io, chalk.red(`   ${error.message}`));
+    return { kind: 'fail', reason: `acceptance gate error: ${error.message}` };
   }
 }
 
 /**
  * 列出所有任务及其验收标准
  */
-export async function listAcceptanceCriteria(options: AcceptanceOptions): Promise<void> {
-  console.log(chalk.blue('📋 任务验收标准列表...\n'));
+export async function listAcceptanceCriteria(
+  options: AcceptanceOptions,
+  io: CommandIO = processIO,
+): Promise<CommandResult> {
+  log(io, chalk.blue('📋 任务验收标准列表...\n'));
 
   const projectPath = options.projectPath || process.cwd();
   const tasksPath = options.tasksPath || path.join(projectPath, 'tasks.yml');
@@ -87,25 +96,27 @@ export async function listAcceptanceCriteria(options: AcceptanceOptions): Promis
     const tasks = yaml.load(content) as any;
 
     if (!tasks || typeof tasks !== 'object') {
-      console.log(chalk.yellow('⚠️  未找到任务定义'));
-      return;
+      log(io, chalk.yellow('⚠️  未找到任务定义'));
+      return { kind: 'skip', reason: `未找到任务定义: ${tasksPath}` };
     }
 
     for (const [taskId, task] of Object.entries(tasks)) {
       if (typeof task === 'object' && task !== null) {
-        console.log(chalk.cyan(`${taskId}:`));
+        log(io, chalk.cyan(`${taskId}:`));
         const taskObj = task as any;
         if (taskObj.acceptanceCriteria) {
           taskObj.acceptanceCriteria.forEach((criteria: string, i: number) => {
-            console.log(chalk.gray(`  ${i + 1}. ${criteria}`));
+            log(io, chalk.gray(`  ${i + 1}. ${criteria}`));
           });
         } else {
-          console.log(chalk.gray('  (无验收标准)'));
+          log(io, chalk.gray('  (无验收标准)'));
         }
-        console.log();
+        log(io);
       }
     }
+    return { kind: 'ok' };
   } catch (error: any) {
-    console.log(chalk.red(`❌ 读取 tasks.yml 失败: ${error.message}`));
+    log(io, chalk.red(`❌ 读取 tasks.yml 失败: ${error.message}`));
+    return { kind: 'skip', reason: `读取 tasks.yml 失败: ${error.message}` };
   }
 }

package/src/cli/commands/analyze-sessions.ts

@@ -28,6 +28,7 @@ import {
   hasSemanticContent,
   STOP_WORDS,
 } from '../session-mining';
+import { log, processIO, type CommandIO, type CommandResult } from '../command-contract';
 
 export interface AnalyzeSessionsOptions {
   days?: number;
@@ -52,26 +53,25 @@ interface PatternCandidate {
 
 // ── Main ──
 
-export async function analyzeSessions(options: AnalyzeSessionsOptions): Promise<void> {
+export async function analyzeSessions(options: AnalyzeSessionsOptions, io: CommandIO = processIO): Promise<CommandResult> {
   const transcriptDir = process.env.CLAUDE_TRANSCRIPTS_DIR
     || path.join(os.homedir(), '.claude', 'projects', '-root--claude');
   const memoryDir = path.join(os.homedir(), '.claude', 'projects', '-root-projects', 'memory');
   const days = options.days || 7;
 
   if (!fs.existsSync(transcriptDir)) {
-    console.log(chalk.yellow('No transcripts directory found'));
-    return;
+    log(io, chalk.yellow('No transcripts directory found'));
+    return { kind: 'skip', reason: `会话记录目录不存在: ${transcriptDir}` };
   }
 
-  // 1. Scan transcripts（按修改时间过滤 + 倒序）
+  // 1. Scan transcripts（since 下推 seam（harness#112）：stat 级过滤，窗口外不 parse；倒序）
   const cutoff = Date.now() - days * 24 * 60 * 60 * 1000;
-  const sessions = readTranscriptSessions(transcriptDir)
-    .filter(s => s.mtimeMs >= cutoff)
+  const sessions = readTranscriptSessions(transcriptDir, { since: cutoff })
     .sort((a, b) => b.date.localeCompare(a.date));
 
   if (sessions.length === 0) {
-    console.log(chalk.yellow(`No sessions found in the last ${days} days`));
-    return;
+    log(io, chalk.yellow(`No sessions found in the last ${days} days`));
+    return { kind: 'skip', reason: `最近 ${days} 天没有会话` };
   }
 
   // 2. Extract corrections + N-grams
@@ -86,14 +86,15 @@ export async function analyzeSessions(options: AnalyzeSessionsOptions): Promise<
 
   // 5. Output
   if (options.json) {
-    console.log(JSON.stringify({ sessions: sessions.length, corrections: corrections.length, candidates }, null, 2));
-    return;
+    log(io, JSON.stringify({ sessions: sessions.length, corrections: corrections.length, candidates }, null, 2));
+    return { kind: 'ok' };
   }
 
-  console.log(chalk.blue(`🔍 Analyzing ${sessions.length} sessions (last ${days} days)...\n`));
+  log(io, chalk.blue(`🔍 Analyzing ${sessions.length} sessions (last ${days} days)...\n`));
 
-  printCorrectionSummary(corrections);
-  printCandidates(candidates);
+  printCorrectionSummary(corrections, io);
+  printCandidates(candidates, io);
+  return { kind: 'ok' };
 }
 
 // ── Correction Extraction ──
@@ -301,34 +302,34 @@ function extractConcept(sentences: string[]): string {
 
 // ── Output ──
 
-function printCorrectionSummary(corrections: Correction[]): void {
+function printCorrectionSummary(corrections: Correction[], io: CommandIO): void {
   if (corrections.length === 0) {
-    console.log(chalk.green('No correction patterns found'));
+    log(io, chalk.green('No correction patterns found'));
     return;
   }
 
-  console.log(chalk.yellow(`📢 ${corrections.length} correction signals detected\n`));
+  log(io, chalk.yellow(`📢 ${corrections.length} correction signals detected\n`));
 }
 
-function printCandidates(candidates: PatternCandidate[]): void {
+function printCandidates(candidates: PatternCandidate[], io: CommandIO): void {
   if (candidates.length === 0) {
-    console.log(chalk.green('✅ No new pattern candidates — all recurring concepts already have rules\n'));
+    log(io, chalk.green('✅ No new pattern candidates — all recurring concepts already have rules\n'));
     return;
   }
 
-  console.log(chalk.bold('🔔 Pattern Candidates (suggested rules)\n'));
+  log(io, chalk.bold('🔔 Pattern Candidates (suggested rules)\n'));
 
   const topK = candidates.slice(0, 10);
   for (const c of topK) {
     const icon = c.source === 'correction' ? '🔴' : '🟡';
     const confBar = '█'.repeat(Math.round(c.confidence * 10)) + '░'.repeat(10 - Math.round(c.confidence * 10));
-    console.log(chalk.bold(`${icon} ${c.pattern}`));
-    console.log(chalk.gray(`   Source: ${c.source} | Freq: ${c.frequency} | Sessions: ${c.sessions.length} | Confidence: ${confBar}`));
-    console.log(chalk.cyan(`   → Rule: ${c.suggestedRule}`));
-    console.log();
+    log(io, chalk.bold(`${icon} ${c.pattern}`));
+    log(io, chalk.gray(`   Source: ${c.source} | Freq: ${c.frequency} | Sessions: ${c.sessions.length} | Confidence: ${confBar}`));
+    log(io, chalk.cyan(`   → Rule: ${c.suggestedRule}`));
+    log(io);
   }
 
   if (candidates.length > 10) {
-    console.log(chalk.gray(`... and ${candidates.length - 10} more candidates. Run with --json for full list.`));
+    log(io, chalk.gray(`... and ${candidates.length - 10} more candidates. Run with --json for full list.`));
   }
 }