@dommaker/harness 1.3.0 → 1.4.0

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

@@ -3,6 +3,7 @@
  */
 
 import { init } from '../init';
+import { captureIO, type CapturingIO } from '../../command-contract';
 import * as fs from 'fs/promises';
 
 // Mock fs/promises
@@ -60,19 +61,18 @@ const mockFs = fs as jest.Mocked<typeof fs>;
 // Use explicit projectPath to avoid CI path mismatch
 const PROJECT = '/test/project';
 
+let io: CapturingIO;
+beforeEach(() => {
+  io = captureIO();
+});
+
 describe('init command', () => {
-  let consoleSpy: jest.SpyInstance;
 
   beforeEach(() => {
     jest.clearAllMocks();
     existingFiles.clear();
     mockExistsSync.mockReturnValue(false);
     mockReaddirSync.mockReturnValue([]);
-    consoleSpy = jest.spyOn(console, 'log').mockImplementation();
-  });
-
-  afterEach(() => {
-    consoleSpy.mockRestore();
   });
 
   describe('init', () => {
@@ -80,7 +80,7 @@ describe('init command', () => {
       mockFs.mkdir.mockResolvedValue(undefined);
       mockFs.writeFile.mockResolvedValue(undefined);
 
-      await init({ preset: 'standard', projectPath: PROJECT });
+      await init({ preset: 'standard', projectPath: PROJECT }, io);
       expect(mockFs.mkdir).toHaveBeenCalled();
     });
 
@@ -88,31 +88,31 @@ describe('init command', () => {
       mockFs.mkdir.mockResolvedValue(undefined);
       mockFs.writeFile.mockResolvedValue(undefined);
 
-      await init({ preset: 'strict', projectPath: PROJECT });
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('strict'));
+      await init({ preset: 'strict', projectPath: PROJECT }, io);
+      expect(io.outText()).toContain('strict');
     });
 
     it('应该使用 relaxed 预设', async () => {
       mockFs.mkdir.mockResolvedValue(undefined);
       mockFs.writeFile.mockResolvedValue(undefined);
 
-      await init({ preset: 'relaxed', projectPath: PROJECT });
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('relaxed'));
+      await init({ preset: 'relaxed', projectPath: PROJECT }, io);
+      expect(io.outText()).toContain('relaxed');
     });
 
     it('应该支持自定义项目路径', async () => {
       mockFs.mkdir.mockResolvedValue(undefined);
       mockFs.writeFile.mockResolvedValue(undefined);
 
-      await init({ preset: 'standard', projectPath: '/custom/path' });
+      await init({ preset: 'standard', projectPath: '/custom/path' }, io);
       expect(mockFs.mkdir).toHaveBeenCalled();
     });
 
     it('应该输出代码片段模式', async () => {
-      await init({ preset: 'standard', printSnippets: true });
-      expect(consoleSpy).toHaveBeenCalled();
-      // 打印版与落盘版 plan 匹配模式须一致（#35：模板字面量 \. 运行期退化为 .，防两副本 drift）
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining("grep -E 'plans/.*\\.md$|\\.plan\\.md$'"));
+      await init({ preset: 'standard', printSnippets: true }, io);
+      expect(io.outText()).not.toBe('');
+      // 打印版与落盘版 plan 匹配模式须一致（#35：模板字面量 \. 运行期退化为 .）
+      expect(io.outText()).toContain("grep -E 'plans/.*\\.md$|\\.plan\\.md$'");
     });
   });
 
@@ -121,8 +121,8 @@ describe('init command', () => {
       mockFs.mkdir.mockResolvedValue(undefined);
       mockFs.writeFile.mockResolvedValue(undefined);
 
-      await init({ preset: 'standard', projectPath: PROJECT });
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('未检测到 Git 仓库'));
+      await init({ preset: 'standard', projectPath: PROJECT }, io);
+      expect(io.outText()).toContain('未检测到 Git 仓库');
     });
 
     it('应该提示手动添加当 pre-commit 已存在', async () => {
@@ -132,10 +132,10 @@ describe('init command', () => {
       existingFiles.add(`${PROJECT}/.git/hooks`);
       existingFiles.add(`${PROJECT}/.git/hooks/pre-commit`);
 
-      await init({ preset: 'standard', projectPath: PROJECT });
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('pre-commit 已存在'));
+      await init({ preset: 'standard', projectPath: PROJECT }, io);
+      expect(io.outText()).toContain('pre-commit 已存在');
       // 手动添加提示的片段同样 scoped 化（#36）
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('npx @dommaker/harness check --staged'));
+      expect(io.outText()).toContain('npx @dommaker/harness check --staged');
     });
 
     it('应该创建 pre-commit hook', async () => {
@@ -144,8 +144,8 @@ describe('init command', () => {
       existingFiles.add(`${PROJECT}/.git`);
       existingFiles.add(`${PROJECT}/.git/hooks`);
 
-      await init({ preset: 'standard', projectPath: PROJECT });
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('已创建 .git/hooks/pre-commit'));
+      await init({ preset: 'standard', projectPath: PROJECT }, io);
+      expect(io.outText()).toContain('已创建 .git/hooks/pre-commit');
     });
 
     it('落盘 pre-commit 使用 scoped 包名（#36）', async () => {
@@ -154,16 +154,45 @@ describe('init command', () => {
       existingFiles.add(`${PROJECT}/.git`);
       existingFiles.add(`${PROJECT}/.git/hooks`);
 
-      await init({ preset: 'standard', projectPath: PROJECT });
+      await init({ preset: 'standard', projectPath: PROJECT }, io);
       const writeCalls = mockFs.writeFile.mock.calls;
       const hookCall = writeCalls.find((c: any[]) => String(c[0]).includes('pre-commit'));
       expect(hookCall).toBeDefined();
       expect(hookCall![1]).toContain('npx @dommaker/harness check --staged');
       expect(hookCall![1]).toContain('npx @dommaker/harness posteval-plan');
       expect(hookCall![1]).not.toMatch(/npx harness /);
-      // plan 匹配模式须为转义点（#35：模板字面量 \. 运行期退化为 .，与打印版防 drift）
+      // plan 匹配模式须为转义点（#35：模板字面量 \. 运行期退化为 .）
       expect(hookCall![1]).toContain("grep -E 'plans/.*\\.md$|\\.plan\\.md$'");
     });
+
+    it('pre-commit 片段与落盘 hook 同源（#103：差异行并入共享片段，无第二份文本）', async () => {
+      mockFs.mkdir.mockResolvedValue(undefined);
+      mockFs.writeFile.mockResolvedValue(undefined);
+      existingFiles.add(`${PROJECT}/.git`);
+      existingFiles.add(`${PROJECT}/.git/hooks`);
+
+      await init({ preset: 'standard', projectPath: PROJECT }, io);
+      const hookCall = mockFs.writeFile.mock.calls.find((c: any[]) => String(c[0]).includes('pre-commit'));
+      const hookContent = String(hookCall![1]);
+      expect(hookContent.startsWith('#!/bin/sh')).toBe(true);
+
+      const io2 = captureIO();
+      await init({ preset: 'standard', printSnippets: true }, io2);
+
+      // 落盘 hook = shebang 头 + 共享片段：去头后必须逐字出现在打印片段输出里
+      const body = hookContent.replace(/^#!\/bin\/sh\n# Harness pre-commit hook\n/, '');
+      expect(body.length).toBeGreaterThan(0);
+      expect(io2.outText()).toContain(body);
+
+      // 原 heredoc 独有的差异行已并入共享片段，打印片段同样可见
+      for (const line of [
+        'echo "🔍 Running harness checks..."',
+        'echo "✅ All checks passed"',
+      ]) {
+        expect(hookContent).toContain(line);
+        expect(io2.outText()).toContain(line);
+      }
+    });
   });
 
   describe('GitHub Actions', () => {
@@ -173,15 +202,15 @@ describe('init command', () => {
       mockReaddir.mockResolvedValue(['ci.yml']);
       existingFiles.add(`${PROJECT}/.github/workflows`);
 
-      await init({ preset: 'standard', projectPath: PROJECT });
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('已存在的 CI 配置'));
+      await init({ preset: 'standard', projectPath: PROJECT }, io);
+      expect(io.outText()).toContain('已存在的 CI 配置');
     });
 
     it('落盘 harness-check.yml 使用 scoped 包名（#36）', async () => {
       mockFs.mkdir.mockResolvedValue(undefined);
       mockFs.writeFile.mockResolvedValue(undefined);
 
-      await init({ preset: 'standard', projectPath: PROJECT });
+      await init({ preset: 'standard', projectPath: PROJECT }, io);
       const writeCalls = mockFs.writeFile.mock.calls;
       const workflowCall = writeCalls.find((c: any[]) => String(c[0]).includes('harness-check.yml'));
       expect(workflowCall).toBeDefined();
@@ -197,7 +226,7 @@ describe('init command', () => {
       mockFs.mkdir.mockResolvedValue(undefined);
       mockFs.writeFile.mockResolvedValue(undefined);
 
-      await init({ preset: 'standard', projectPath: PROJECT });
+      await init({ preset: 'standard', projectPath: PROJECT }, io);
       const writes = mockFs.writeFile.mock.calls.map((c: unknown[]) => String(c[0]));
       expect(writes.every((w: string) => !w.includes('CAPABILITIES.md'))).toBe(true);
     });
@@ -209,8 +238,8 @@ describe('init command', () => {
       mockFs.writeFile.mockResolvedValue(undefined);
       existingFiles.add(`${PROJECT}/.harness/custom-constraints.yml`);
 
-      await init({ preset: 'standard', projectPath: PROJECT });
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('custom-constraints.yml 已存在'));
+      await init({ preset: 'standard', projectPath: PROJECT }, io);
+      expect(io.outText()).toContain('custom-constraints.yml 已存在');
     });
   });
 
@@ -219,15 +248,15 @@ describe('init command', () => {
       mockFs.mkdir.mockResolvedValue(undefined);
       mockFs.writeFile.mockResolvedValue(undefined);
 
-      await init({ preset: 'standard', governance: 'standard', projectPath: PROJECT });
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('治理级别: standard'));
+      await init({ preset: 'standard', governance: 'standard', projectPath: PROJECT }, io);
+      expect(io.outText()).toContain('治理级别: standard');
     });
 
     it('应该创建 CHANGELOG.md', async () => {
       mockFs.mkdir.mockResolvedValue(undefined);
       mockFs.writeFile.mockResolvedValue(undefined);
 
-      await init({ preset: 'standard', governance: 'standard', projectPath: PROJECT });
+      await init({ preset: 'standard', governance: 'standard', projectPath: PROJECT }, io);
       const writeCalls = mockFs.writeFile.mock.calls;
       const changelogCall = writeCalls.find((c: any[]) => String(c[0]).includes('CHANGELOG.md'));
       expect(changelogCall).toBeDefined();
@@ -239,8 +268,8 @@ describe('init command', () => {
       mockFs.writeFile.mockResolvedValue(undefined);
       existingFiles.add(`${PROJECT}/CHANGELOG.md`);
 
-      await init({ preset: 'standard', governance: 'standard', projectPath: PROJECT });
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('CHANGELOG.md 已存在'));
+      await init({ preset: 'standard', governance: 'standard', projectPath: PROJECT }, io);
+      expect(io.outText()).toContain('CHANGELOG.md 已存在');
     });
 
     it('应该为 standard 治理创建 src/CONTEXT.md', async () => {
@@ -256,7 +285,7 @@ describe('init command', () => {
         return [];
       });
 
-      await init({ preset: 'standard', governance: 'standard', projectPath: PROJECT });
+      await init({ preset: 'standard', governance: 'standard', projectPath: PROJECT }, io);
       const writeCalls = mockFs.writeFile.mock.calls;
       const contextCall = writeCalls.find((c: any[]) => String(c[0]).includes('CONTEXT.md'));
       expect(contextCall).toBeDefined();
@@ -267,7 +296,7 @@ describe('init command', () => {
       mockFs.mkdir.mockResolvedValue(undefined);
       mockFs.writeFile.mockResolvedValue(undefined);
 
-      await init({ preset: 'standard', governance: 'standard', projectPath: PROJECT });
+      await init({ preset: 'standard', governance: 'standard', projectPath: PROJECT }, io);
       const writeCalls = mockFs.writeFile.mock.calls;
       const contextCall = writeCalls.find((c: any[]) => String(c[0]).includes('CONTEXT.md'));
       expect(contextCall).toBeUndefined();
@@ -277,7 +306,7 @@ describe('init command', () => {
       mockFs.mkdir.mockResolvedValue(undefined);
       mockFs.writeFile.mockResolvedValue(undefined);
 
-      await init({ preset: 'standard', governance: 'minimal', projectPath: PROJECT });
+      await init({ preset: 'standard', governance: 'minimal', projectPath: PROJECT }, io);
       const writeCalls = mockFs.writeFile.mock.calls;
       const contextCall = writeCalls.find((c: any[]) => String(c[0]).includes('CONTEXT.md'));
       expect(contextCall).toBeUndefined();
@@ -287,7 +316,7 @@ describe('init command', () => {
       mockFs.mkdir.mockResolvedValue(undefined);
       mockFs.writeFile.mockResolvedValue(undefined);
 
-      await init({ preset: 'standard', governance: 'standard', projectPath: PROJECT });
+      await init({ preset: 'standard', governance: 'standard', projectPath: PROJECT }, io);
       const writeCalls = mockFs.writeFile.mock.calls;
       const workflowCall = writeCalls.find((c: any[]) => String(c[0]).includes('harness-governance.yml'));
       expect(workflowCall).toBeDefined();
@@ -304,8 +333,8 @@ describe('init command', () => {
       mockFs.writeFile.mockResolvedValue(undefined);
       existingFiles.add(`${PROJECT}/.github/workflows/harness-governance.yml`);
 
-      await init({ preset: 'standard', governance: 'standard', projectPath: PROJECT });
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('harness-governance.yml 已存在'));
+      await init({ preset: 'standard', governance: 'standard', projectPath: PROJECT }, io);
+      expect(io.outText()).toContain('harness-governance.yml 已存在');
     });
 
     it('应该跳过治理 CI workflow 当已有 workflow 覆盖治理命令', async () => {
@@ -320,11 +349,11 @@ describe('init command', () => {
           : Promise.reject(new Error('ENOENT'))
       );
 
-      await init({ preset: 'standard', governance: 'standard', projectPath: PROJECT });
+      await init({ preset: 'standard', governance: 'standard', projectPath: PROJECT }, io);
       const writeCalls = mockFs.writeFile.mock.calls;
       const workflowCall = writeCalls.find((c: any[]) => String(c[0]).includes('harness-governance.yml'));
       expect(workflowCall).toBeUndefined();
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('治理检查已由 ci.yml 覆盖'));
+      expect(io.outText()).toContain('治理检查已由 ci.yml 覆盖');
 
       mockReaddir.mockResolvedValue([]);
       mockFs.readFile.mockRejectedValue(new Error('ENOENT'));
@@ -341,7 +370,7 @@ describe('init command', () => {
           : Promise.reject(new Error('ENOENT'))
       );
 
-      await init({ preset: 'standard', governance: 'standard', projectPath: PROJECT });
+      await init({ preset: 'standard', governance: 'standard', projectPath: PROJECT }, io);
       const writeCalls = mockFs.writeFile.mock.calls;
       const workflowCall = writeCalls.find((c: any[]) => String(c[0]).includes('harness-governance.yml'));
       expect(workflowCall).toBeDefined();
@@ -355,7 +384,7 @@ describe('init command', () => {
       mockFs.mkdir.mockResolvedValue(undefined);
       mockFs.writeFile.mockResolvedValue(undefined);
 
-      await init({ preset: 'standard', governance: 'strict', projectPath: PROJECT });
+      await init({ preset: 'standard', governance: 'strict', projectPath: PROJECT }, io);
       const writeCalls = mockFs.writeFile.mock.calls;
       const workflowCall = writeCalls.find((c: any[]) => String(c[0]).includes('harness-governance.yml'));
       expect(workflowCall![1]).toContain('sync-docs');
@@ -366,7 +395,7 @@ describe('init command', () => {
       mockFs.mkdir.mockResolvedValue(undefined);
       mockFs.writeFile.mockResolvedValue(undefined);
 
-      await init({ preset: 'standard', governance: 'minimal', projectPath: PROJECT });
+      await init({ preset: 'standard', governance: 'minimal', projectPath: PROJECT }, io);
       const writeCalls = mockFs.writeFile.mock.calls;
       const workflowCall = writeCalls.find((c: any[]) => String(c[0]).includes('harness-governance.yml'));
       expect(workflowCall![1]).not.toContain('sync-docs');
@@ -376,7 +405,7 @@ describe('init command', () => {
       mockFs.mkdir.mockResolvedValue(undefined);
       mockFs.writeFile.mockResolvedValue(undefined);
 
-      await init({ preset: 'standard', projectPath: PROJECT });
+      await init({ preset: 'standard', projectPath: PROJECT }, io);
       const writeCalls = mockFs.writeFile.mock.calls;
       const changelogCall = writeCalls.find((c: any[]) => String(c[0]).includes('CHANGELOG.md'));
       const contextCall = writeCalls.find((c: any[]) => String(c[0]).includes('CONTEXT.md'));

package/src/cli/commands/__tests__/knowledge-flywheel-consistency.test.ts

@@ -0,0 +1,108 @@
+/**
+ * 飞轮指标三处一致性（harness#81）
+ *
+ * 同一 fixture 分别走 audit D6 / knowledge stats / knowledge health 三条消费路径，
+ * 断言 refCoverage / avgRefs / consumptionHitRate 逐字段相等——分子只有一份实现
+ * （knowledge/flywheel-metrics），展示层各自做单位与字段名映射。
+ *
+ * fixture 条目全为非 archived / 非 deprecated，三处人口重合：health 的 excludeArchived
+ * 切片与 audit/stats 的 active 过滤在这里是同一集合（差异在人口筛选，不在计算）。
+ * 期望值即口径哨兵：修复前 stats/health 直接数 referencedBy.length，
+ * 同一 fixture 会报 refCoverage 75 / avgRefs 1.5。
+ */
+
+import * as fs from 'fs';
+import { captureIO, lastJsonOutput, type CapturingIO } from '../../command-contract';
+import * as os from 'os';
+import * as path from 'path';
+import { FileKnowledgeStore } from '../../../knowledge/store';
+import { KnowledgeAudit } from '../../../knowledge/audit';
+import { knowledgeHealth, knowledgeStats } from '../knowledge';
+import type { KnowledgeEntry } from '../../../knowledge/types';
+
+function makeEntry(id: string, maturity: KnowledgeEntry['maturity'], referencedBy: string[]): KnowledgeEntry {
+  return {
+    id,
+    type: 'guideline',
+    title: `consistency fixture ${id}`,
+    content: 'content long enough to stay out of the short-content rule and the audit filters',
+    maturity,
+    layer: 'project',
+    created: '2026-09-01T00:00:00.000Z',
+    lastReferenced: '2026-09-01T00:00:00.000Z',
+    contributors: ['human'],
+    projects: ['harness'],
+    tags: [],
+    applicablePhases: [],
+    sourceReferences: [],
+    referencedBy,
+    executionResults: [],
+    consumptionMode: 'reference',
+    origin: 'agent',
+  };
+}
+
+/**
+ * 4 条 active 条目，genuine 引用共 3 条：
+ * - 2 条有真实引用（K-001 / K-004）→ refCoverage 0.5
+ * - K-002 只有自动化引用（prompt-inject / test-agent / monitor），不给条目续命
+ * - dailyEvents 2 / 4 条目 → consumptionHitRate 0.5
+ */
+const FIXTURE: KnowledgeEntry[] = [
+  makeEntry('K-001', 'verified', ['agent-a:2026-09-01', 'search:2026-09-01']),
+  makeEntry('K-002', 'verified', ['prompt-inject:2026-08-30', 'test-agent:2026-08-31', 'monitor:2026-09-01']),
+  makeEntry('K-003', 'draft', []),
+  makeEntry('K-004', 'proven', ['decision-1:2026-07-01', 'studio-web:2026-07-02']),
+];
+
+function setupKnowledgeBase(): string {
+  const projectRoot = fs.mkdtempSync(path.join(os.tmpdir(), 'flywheel-consistency-'));
+  const baseDir = path.join(projectRoot, '.harness', 'knowledge');
+  const store = new FileKnowledgeStore({ baseDir });
+  for (const entry of FIXTURE) store.save(entry);
+  fs.writeFileSync(
+    path.join(baseDir, '.consumption-stats.json'),
+    JSON.stringify({ date: '2026-09-02', dailyEvents: 2, searchHits: 1 }),
+  );
+  return projectRoot;
+}
+
+let io: CapturingIO;
+beforeEach(() => {
+  io = captureIO();
+});
+
+describe('飞轮指标三处同源', () => {
+  it('audit D6 / stats / health 报出同一组数字', async () => {
+    const projectRoot = setupKnowledgeBase();
+    const baseDir = path.join(projectRoot, '.harness', 'knowledge');
+
+    const report = new KnowledgeAudit({ baseDir }).run();
+    const auditDetails = report.dimensions.flywheel.details;
+    await knowledgeStats({ projectPath: projectRoot, json: true }, io);
+    const statsFlywheel = lastJsonOutput<any>(io).flywheel;
+
+    io = captureIO();
+    await knowledgeHealth({ dir: baseDir, json: true }, io);
+    const healthSummary = lastJsonOutput<any>(io).summary;
+
+    // 逐字段一致（avgRefs 在 audit 报告层的呈现名是 avgRefCount）
+    expect(statsFlywheel.refCoverage).toBe(auditDetails.refCoverage);
+    expect(statsFlywheel.avgRefs).toBe(auditDetails.avgRefCount);
+    expect(healthSummary.refCoverage).toBe(auditDetails.refCoverage);
+    expect(healthSummary.avgRefs).toBe(auditDetails.avgRefCount);
+    expect(statsFlywheel.consumptionHitRate).toBe(auditDetails.consumptionHitRate);
+
+    // canonical 值（genuine 口径）：人口 4、有引用 2、genuine 引用 3、dailyEvents 2
+    expect(auditDetails).toMatchObject({
+      activeEntries: 4,
+      entriesWithRefs: 2,
+      refCoverage: 50,
+      avgRefCount: 0.8,
+      dailyConsumptionEvents: 2,
+      consumptionHitRate: 50,
+    });
+    expect(statsFlywheel).toEqual({ refCoverage: 50, avgRefs: 0.8, consumptionHitRate: 50 });
+    expect(healthSummary).toMatchObject({ total: 4, refCoverage: 50, avgRefs: 0.8 });
+  });
+});