@dommaker/harness 1.3.0 → 1.5.0

package/src/cli/commands/__tests__/status-extra.test.ts

@@ -3,6 +3,7 @@
  */
 
 import { status } from '../status';
+import { captureIO, type CapturingIO } from '../../command-contract';
 import * as fs from 'fs';
 import { TraceAnalyzer } from '../../../monitoring/trace-analyzer';
 
@@ -42,22 +43,21 @@ jest.mock('chalk', () => ({
 const mockFs = fs as jest.Mocked<typeof fs>;
 const MockTraceAnalyzer = TraceAnalyzer as jest.MockedClass<typeof TraceAnalyzer>;
 
+let io: CapturingIO;
+beforeEach(() => {
+  io = captureIO();
+});
+
 describe('status command - 补充覆盖', () => {
-  let consoleSpy: jest.SpyInstance;
 
   beforeEach(() => {
     jest.clearAllMocks();
-    consoleSpy = jest.spyOn(console, 'log').mockImplementation();
     mockFs.existsSync.mockReturnValue(true);
     mockFs.readFileSync.mockReturnValue('');
     mockFs.writeFileSync.mockImplementation();
     mockFs.mkdirSync.mockImplementation();
   });
 
-  afterEach(() => {
-    consoleSpy.mockRestore();
-  });
-
   describe('异常检测完整流程', () => {
     it('有异常时应该显示详细信息和下一步建议', async () => {
       const mockTraces = [
@@ -87,9 +87,9 @@ describe('status command - 补充覆盖', () => {
       };
       (MockTraceAnalyzer as any).mockImplementation(() => mockAnalyze);
 
-      await status({ anomalies: true });
+      await status({ anomalies: true }, io);
       
-      const output = consoleSpy.mock.calls.map(c => c[0]).join('\n');
+      const output = io.outText();
       expect(output).toContain('异常');
       expect(mockAnalyze.detectAnomalies).toHaveBeenCalled();
     });
@@ -103,9 +103,9 @@ describe('status command - 补充覆盖', () => {
       };
       (MockTraceAnalyzer as any).mockImplementation(() => mockAnalyze);
 
-      await status({ anomalies: true });
+      await status({ anomalies: true }, io);
       
-      const output = consoleSpy.mock.calls.map(c => c[0]).join('\n');
+      const output = io.outText();
       expect(output).toContain('未发现异常');
     });
   });
@@ -128,9 +128,9 @@ describe('status command - 补充覆盖', () => {
       };
       (MockTraceAnalyzer as any).mockImplementation(() => mockAnalyze);
 
-      await status({});
+      await status({}, io);
       
-      const output = consoleSpy.mock.calls.map(c => c[0]).join('\n');
+      const output = io.outText();
       expect(output).toContain('约束统计');
     });
 
@@ -149,9 +149,9 @@ describe('status command - 补充覆盖', () => {
       };
       (MockTraceAnalyzer as any).mockImplementation(() => mockAnalyze);
 
-      await status({});
+      await status({}, io);
       
-      expect(consoleSpy).toHaveBeenCalled();
+      expect(io.outText()).not.toBe('');
     });
   });
 
@@ -173,9 +173,9 @@ describe('status command - 补充覆盖', () => {
       };
       (MockTraceAnalyzer as any).mockImplementation(() => mockAnalyze);
 
-      await status({ detail: true });
+      await status({ detail: true }, io);
       
-      const output = consoleSpy.mock.calls.map(c => c[0]).join('\n');
+      const output = io.outText();
       expect(output).toContain('约束统计');
     });
   });
@@ -190,10 +190,10 @@ describe('status command - 补充覆盖', () => {
       };
       (MockTraceAnalyzer as any).mockImplementation(() => mockAnalyze);
 
-      await status({});
+      await status({}, io);
       
       // 应该成功处理，不会抛出异常
-      expect(consoleSpy).toHaveBeenCalled();
+      expect(io.outText()).not.toBe('');
     });
   });
 });

package/src/cli/commands/__tests__/status-skipped-lines.test.ts

@@ -0,0 +1,72 @@
+/**
+ * harness status 的坏行提示（harness#100）
+ *
+ * 冻结两件事：
+ * 1. `记录数`（traceCount）保持**原始非空行数口径**（合法 + 坏行），逐字节不变——
+ *    改口径属行为变更，#100 明确不做。
+ * 2. 坏行数是一条**新增的独立提示**，走 stderr（CLI 侧契约：诊断信息进 stderr，
+ *    与 #96 `failure list` 同一去向），stdout 报告体不受污染。
+ *
+ * 用真实临时工程根（不 mock fs），才能对账真实文件行数。
+ */
+
+import * as path from 'path';
+import { createProjectFixture } from '../../../test-setup/project-fixture';
+import { captureIO, type CapturingIO } from '../../command-contract';
+import { status } from '../status';
+
+const TRACE_REL = path.join('.harness', 'logs', 'traces.log');
+
+const VALID_A = JSON.stringify({ constraintId: 'alpha', level: 'iron_law', timestamp: 1, result: 'pass' });
+const VALID_B = JSON.stringify({ constraintId: 'beta', level: 'guideline', timestamp: 2, result: 'pass' });
+const BAD = '{"constraintId":"broken"';
+
+function projectWith(lines: string[]): string {
+  return createProjectFixture({ name: 'status-skiplines', files: { [TRACE_REL]: lines.join('\n') + '\n' } });
+}
+
+let io: CapturingIO;
+beforeEach(() => {
+  io = captureIO();
+});
+
+describe('harness status 坏行提示（harness#100）', () => {
+  it('traceCount 口径不变：3 行合法 vs 2 合法 + 1 坏行，stdout 逐字节一致', async () => {
+    const clean = projectWith([VALID_A, VALID_B, VALID_A]);
+    const dirty = projectWith([VALID_A, BAD, VALID_B]);
+
+    const cleanIo = captureIO();
+    await status({ projectPath: clean }, cleanIo);
+    const dirtyIo = captureIO();
+    await status({ projectPath: dirty }, dirtyIo);
+
+    // 原始行数口径：两者都显示 3 条（改前即 records.length + skippedLines）
+    expect(cleanIo.outText()).toContain('记录数: 3 条');
+    expect(dirtyIo.outText()).toContain('记录数: 3 条');
+    expect(dirtyIo.outText()).toBe(cleanIo.outText());
+  });
+
+  it('坏行数落到 stderr 的独立一行，stdout 不含该提示', async () => {
+    const root = projectWith([VALID_A, BAD, BAD, VALID_B]);
+
+    await status({ projectPath: root }, io);
+
+    expect(io.outText()).not.toContain('损坏');
+    expect(io.errText()).toContain('2 行损坏');
+    // 独立一行：不与既有 stdout 行合并，也不止一行输出
+    expect(io.errLines()).toHaveLength(1);
+  });
+
+  it('无损坏时零噪声：stderr 为空', async () => {
+    await status({ projectPath: projectWith([VALID_A, VALID_B]) }, io);
+
+    expect(io.errText()).toBe('');
+  });
+
+  it('--anomalies 模式同样告知（提示在读文件后立即发出，不受展示分支影响）', async () => {
+    await status({ projectPath: projectWith([VALID_A, BAD]), anomalies: true }, io);
+
+    expect(io.errText()).toContain('1 行损坏');
+    expect(io.outText()).toContain('记录数: 2 条');
+  });
+});

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

@@ -3,6 +3,7 @@
  */
 
 import { status } from '../status';
+import { captureIO, type CapturingIO } from '../../command-contract';
 import * as fs from 'fs';
 import { TraceAnalyzer } from '../../../monitoring/trace-analyzer';
 
@@ -42,27 +43,26 @@ jest.mock('chalk', () => ({
 const mockFs = fs as jest.Mocked<typeof fs>;
 const MockTraceAnalyzer = TraceAnalyzer as jest.MockedClass<typeof TraceAnalyzer>;
 
+let io: CapturingIO;
+beforeEach(() => {
+  io = captureIO();
+});
+
 describe('status command', () => {
-  let consoleSpy: jest.SpyInstance;
 
   beforeEach(() => {
     jest.clearAllMocks();
-    consoleSpy = jest.spyOn(console, 'log').mockImplementation();
     mockFs.existsSync.mockReturnValue(true);
     mockFs.readFileSync.mockReturnValue('');
     mockFs.writeFileSync.mockImplementation();
     mockFs.mkdirSync.mockImplementation();
   });
 
-  afterEach(() => {
-    consoleSpy.mockRestore();
-  });
-
   describe('未初始化情况', () => {
     it('应该显示未初始化提示', async () => {
       mockFs.existsSync.mockReturnValue(false);
-      await status({});
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('未初始化'));
+      await status({}, io);
+      expect(io.outText()).toContain('未初始化');
     });
   });
 
@@ -73,8 +73,8 @@ describe('status command', () => {
         .mockReturnValueOnce(true) // .harness dir
         .mockReturnValueOnce(false); // traces file
 
-      await status({});
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('暂无 Trace'));
+      await status({}, io);
+      expect(io.outText()).toContain('暂无 Trace');
     });
   });
 
@@ -95,8 +95,8 @@ describe('status command', () => {
       };
       (MockTraceAnalyzer as any).mockImplementation(() => mockAnalyze);
 
-      await status({});
-      expect(consoleSpy).toHaveBeenCalled();
+      await status({}, io);
+      expect(io.outText()).not.toBe('');
     });
 
     it('应该显示 Iron Laws 统计', async () => {
@@ -114,8 +114,8 @@ describe('status command', () => {
       };
       (MockTraceAnalyzer as any).mockImplementation(() => mockAnalyze);
 
-      await status({ detail: true });
-      expect(consoleSpy).toHaveBeenCalled();
+      await status({ detail: true }, io);
+      expect(io.outText()).not.toBe('');
     });
   });
 
@@ -137,8 +137,8 @@ describe('status command', () => {
       };
       (MockTraceAnalyzer as any).mockImplementation(() => mockAnalyze);
 
-      await status({ anomalies: true });
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('异常'));
+      await status({ anomalies: true }, io);
+      expect(io.outText()).toContain('异常');
     });
   });
 
@@ -158,8 +158,8 @@ describe('status command', () => {
       };
       (MockTraceAnalyzer as any).mockImplementation(() => mockAnalyze);
 
-      await status({ detail: true });
-      expect(consoleSpy).toHaveBeenCalled();
+      await status({ detail: true }, io);
+      expect(io.outText()).not.toBe('');
     });
   });
 
@@ -173,7 +173,7 @@ describe('status command', () => {
       };
       (MockTraceAnalyzer as any).mockImplementation(() => mockAnalyze);
 
-      await status({});
+      await status({}, io);
       expect(mockFs.writeFileSync).toHaveBeenCalled();
     });
   });
@@ -190,8 +190,8 @@ describe('status command', () => {
       };
       (MockTraceAnalyzer as any).mockImplementation(() => mockAnalyze);
 
-      await status({});
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('Guidelines'));
+      await status({}, io);
+      expect(io.outText()).toContain('Guidelines');
     });
   });
 
@@ -205,8 +205,8 @@ describe('status command', () => {
       };
       (MockTraceAnalyzer as any).mockImplementation(() => mockAnalyze);
 
-      await status({ anomalies: true });
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('未发现异常'));
+      await status({ anomalies: true }, io);
+      expect(io.outText()).toContain('未发现异常');
     });
 
     it('应该显示异常详情', async () => {
@@ -220,9 +220,9 @@ describe('status command', () => {
       };
       (MockTraceAnalyzer as any).mockImplementation(() => mockAnalyze);
 
-      await status({ anomalies: true });
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('异常'));
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('下一步建议'));
+      await status({ anomalies: true }, io);
+      expect(io.outText()).toContain('异常');
+      expect(io.outText()).toContain('下一步建议');
     });
   });
 
@@ -237,8 +237,8 @@ describe('status command', () => {
       };
       (MockTraceAnalyzer as any).mockImplementation(() => mockAnalyze);
 
-      await status({});
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('状态良好'));
+      await status({}, io);
+      expect(io.outText()).toContain('状态良好');
     });
 
     it('应该显示积累数据建议当 trace < 100', async () => {
@@ -250,8 +250,8 @@ describe('status command', () => {
       };
       (MockTraceAnalyzer as any).mockImplementation(() => mockAnalyze);
 
-      await status({});
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('继续积累数据'));
+      await status({}, io);
+      expect(io.outText()).toContain('继续积累数据');
     });
 
     it('应该显示诊断建议当有异常', async () => {
@@ -265,8 +265,8 @@ describe('status command', () => {
       };
       (MockTraceAnalyzer as any).mockImplementation(() => mockAnalyze);
 
-      await status({});
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('harness status --detail'));
+      await status({}, io);
+      expect(io.outText()).toContain('harness status --detail');
     });
   });
 
@@ -282,8 +282,8 @@ describe('status command', () => {
       };
       (MockTraceAnalyzer as any).mockImplementation(() => mockAnalyze);
 
-      await status({ detail: true });
-      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('检查:'));
+      await status({ detail: true }, io);
+      expect(io.outText()).toContain('检查:');
     });
   });
 });