@dommaker/harness 0.18.0 → 1.0.0

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

@@ -0,0 +1,162 @@
+/**
+ * release 命令测试（O6）
+ *
+ * Seam：release(options) 公开入口。
+ * 隔离面：
+ *   - child_process.execSync → 按命令分发的 fixture（git/npm/tsc 全部 mock，绝不真执行）
+ *   - fs.existsSync / readFileSync → fixture 包（package.json + dist 关键文件）
+ *   - process.exit → 抛异常断言化（exit code 可断言，且阻断后续真实副作用）
+ *
+ * 测试止于 dry-run 与「发布前闸门」（分支/同步/干净树/tag 冲突），
+ * 不覆盖真实 npm version/publish 路径——禁止真发版。
+ */
+
+import * as fs from 'fs';
+import { execSync } from 'child_process';
+import { release } from '../release';
+
+jest.mock('child_process', () => ({
+  ...jest.requireActual('child_process'),
+  execSync: jest.fn(),
+}));
+
+jest.mock('fs', () => ({
+  ...jest.requireActual('fs'),
+  existsSync: jest.fn(),
+  readFileSync: jest.fn(),
+}));
+
+jest.mock('chalk', () => {
+  const id = (s: string) => s;
+  const chalkFn = Object.assign(id, {
+    gray: id,
+    blue: id,
+    green: id,
+    yellow: id,
+    cyan: id,
+    bold: id,
+    red: id,
+  });
+  return {
+    __esModule: true,
+    default: chalkFn,
+    gray: id,
+    blue: id,
+    green: id,
+    yellow: id,
+    cyan: id,
+    bold: id,
+    red: id,
+  };
+});
+
+const mockExecSync = execSync as jest.MockedFunction<typeof execSync>;
+const mockFs = fs as jest.Mocked<typeof fs>;
+
+const PKG_JSON = JSON.stringify({ name: '@dommaker/harness', version: '0.18.0' });
+
+describe('release command', () => {
+  let consoleSpy: jest.SpyInstance;
+  let errorSpy: jest.SpyInstance;
+  let exitSpy: jest.SpyInstance;
+  let pkgExists: boolean;
+  let tagListResult: string;
+
+  beforeEach(() => {
+    jest.clearAllMocks();
+    pkgExists = true;
+    tagListResult = '';
+    consoleSpy = jest.spyOn(console, 'log').mockImplementation();
+    errorSpy = jest.spyOn(console, 'error').mockImplementation();
+    exitSpy = jest.spyOn(process, 'exit').mockImplementation((code?: string | number | null) => {
+      throw new Error(`__exit_${code}__`);
+    });
+
+    mockFs.existsSync.mockImplementation((p) => {
+      if (String(p).endsWith('package.json')) return pkgExists;
+      return true; // dist 关键文件视为存在
+    });
+    mockFs.readFileSync.mockReturnValue(PKG_JSON);
+
+    mockExecSync.mockImplementation((cmd: string) => {
+      const c = String(cmd);
+      if (c.startsWith('git rev-parse --abbrev-ref')) return 'master';
+      if (c.startsWith('git rev-list --count')) return '0';
+      if (c.startsWith('git status --porcelain')) return '';
+      if (c.startsWith('git tag -l')) return tagListResult;
+      if (c.startsWith('npx tsc')) return 'build ok';
+      return '';
+    });
+  });
+
+  afterEach(() => {
+    consoleSpy.mockRestore();
+    errorSpy.mockRestore();
+    exitSpy.mockRestore();
+  });
+
+  test('非包目录：exit 1 并提示 Not a package', async () => {
+    pkgExists = false;
+
+    await expect(release({})).rejects.toThrow('__exit_1__');
+    expect(errorSpy).toHaveBeenCalledWith(expect.stringContaining('Not a package'));
+  });
+
+  test('非 master/main 分支：exit 1 并提示', async () => {
+    mockExecSync.mockImplementation((cmd: string) =>
+      String(cmd).startsWith('git rev-parse --abbrev-ref') ? 'feat/x' : '');
+
+    await expect(release({})).rejects.toThrow('__exit_1__');
+    expect(errorSpy).toHaveBeenCalledWith(expect.stringContaining('Must be on master or main branch'));
+  });
+
+  test('落后于远程：exit 1 并提示 behind', async () => {
+    mockExecSync.mockImplementation((cmd: string) => {
+      const c = String(cmd);
+      if (c.startsWith('git rev-parse --abbrev-ref')) return 'master';
+      if (c.startsWith('git rev-list --count')) return '3';
+      return '';
+    });
+
+    await expect(release({})).rejects.toThrow('__exit_1__');
+    expect(errorSpy).toHaveBeenCalledWith(expect.stringContaining('behind origin'));
+  });
+
+  test('工作树不干净：exit 1 并提示 Uncommitted changes', async () => {
+    mockExecSync.mockImplementation((cmd: string) => {
+      const c = String(cmd);
+      if (c.startsWith('git rev-parse --abbrev-ref')) return 'master';
+      if (c.startsWith('git rev-list --count')) return '0';
+      if (c.startsWith('git status --porcelain')) return ' M package.json';
+      return '';
+    });
+
+    await expect(release({})).rejects.toThrow('__exit_1__');
+    expect(errorSpy).toHaveBeenCalledWith(expect.stringContaining('Uncommitted changes'));
+  });
+
+  test('dry-run patch：计算目标版本 0.18.1，exit 0', async () => {
+    await expect(release({ bumpType: 'patch', dryRun: 'true' })).rejects.toThrow('__exit_0__');
+    expect(consoleSpy).toHaveBeenCalledWith(
+      expect.stringContaining('Dry-run complete. Would publish: @dommaker/harness@0.18.1'),
+    );
+  });
+
+  test('dry-run minor：计算目标版本 0.19.0', async () => {
+    await expect(release({ bumpType: 'minor', dryRun: 'true' })).rejects.toThrow('__exit_0__');
+    expect(consoleSpy).toHaveBeenCalledWith(
+      expect.stringContaining('Dry-run complete. Would publish: @dommaker/harness@0.19.0'),
+    );
+  });
+
+  test('目标 tag 已存在：发布前闸门 exit 1（不触 npm version）', async () => {
+    tagListResult = 'v0.18.1';
+
+    await expect(release({ bumpType: 'patch' })).rejects.toThrow('__exit_1__');
+    expect(errorSpy).toHaveBeenCalledWith(expect.stringContaining('already exists'));
+    // 闸门阻断：npm version / npm publish 从未执行
+    const commands = mockExecSync.mock.calls.map(c => String(c[0]));
+    expect(commands.some(c => c.startsWith('npm version'))).toBe(false);
+    expect(commands.some(c => c.startsWith('npm publish'))).toBe(false);
+  });
+});

package/src/cli/commands/__tests__/update-user-model.test.ts

@@ -0,0 +1,205 @@
+/**
+ * update-user-model 命令测试（O1：--days flag + --json/--dry-run 兼容）
+ *
+ * Seam：updateUserModel(options) 公开入口。
+ * 隔离面：
+ *   - os.homedir → 测试临时目录（state/profile 落 TEST_HOME，不触碰真实 ~/.claude）
+ *   - readTranscriptSessions → fixture 会话（extractCorrectionMatches 等纯函数保持真实）
+ */
+
+import * as fs from 'fs';
+import * as path from 'path';
+import { updateUserModel } from '../update-user-model';
+import { readTranscriptSessions, type MinedSession } from '../../session-mining';
+
+const TEST_HOME = '/tmp/harness-uum-test-home';
+
+jest.mock('os', () => ({
+  ...jest.requireActual('os'),
+  homedir: () => '/tmp/harness-uum-test-home',
+}));
+
+jest.mock('../../session-mining', () => ({
+  ...jest.requireActual('../../session-mining'),
+  readTranscriptSessions: jest.fn(),
+}));
+
+jest.mock('chalk', () => {
+  const id = (s: string) => s;
+  const chalkFn = Object.assign(id, {
+    gray: id,
+    blue: id,
+    green: id,
+    yellow: id,
+    cyan: id,
+    bold: id,
+    red: id,
+  });
+  return {
+    __esModule: true,
+    default: chalkFn,
+    gray: id,
+    blue: id,
+    green: id,
+    yellow: id,
+    cyan: id,
+    bold: id,
+    red: id,
+  };
+});
+
+const mockReadTranscriptSessions = readTranscriptSessions as jest.MockedFunction<
+  typeof readTranscriptSessions
+>;
+
+const STATE_FILE = path.join(TEST_HOME, '.claude', 'user-model-state.json');
+
+function todayStr(offsetDays = 0): string {
+  return new Date(Date.now() - offsetDays * 86_400_000).toISOString().slice(0, 10);
+}
+
+function mkSession(partial: Partial<MinedSession> = {}): MinedSession {
+  return {
+    id: 'session-1',
+    date: todayStr(),
+    mtimeMs: Date.now(),
+    turns: [
+      { role: 'user', content: '数据库迁移方案需要执行' },
+      { role: 'assistant', content: '好的' },
+    ],
+    toolCalls: ['Read'],
+    ...partial,
+  };
+}
+
+function lastJsonOutput(consoleSpy: jest.SpyInstance): Record<string, unknown> {
+  const jsonLine = consoleSpy.mock.calls.map(c => c[0]).join('\n');
+  return JSON.parse(jsonLine);
+}
+
+describe('update-user-model command', () => {
+  let consoleSpy: jest.SpyInstance;
+
+  beforeEach(() => {
+    jest.clearAllMocks();
+    fs.rmSync(TEST_HOME, { recursive: true, force: true });
+    fs.mkdirSync(path.join(TEST_HOME, '.claude', 'projects', '-root-projects', 'memory'), { recursive: true });
+    process.env.CLAUDE_TRANSCRIPTS_DIR = path.join(TEST_HOME, 'transcripts');
+    consoleSpy = jest.spyOn(console, 'log').mockImplementation();
+  });
+
+  afterEach(() => {
+    consoleSpy.mockRestore();
+    delete process.env.CLAUDE_TRANSCRIPTS_DIR;
+  });
+
+  test('无新会话：提示 No new sessions to process', async () => {
+    mockReadTranscriptSessions.mockReturnValue([]);
+
+    await updateUserModel({});
+
+    expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('No new sessions to process'));
+    expect(fs.existsSync(STATE_FILE)).toBe(false);
+  });
+
+  test('--json：输出 newSessions 与 new_pattern 变化', async () => {
+    mockReadTranscriptSessions.mockReturnValue([
+      mkSession({ id: 'session-a' }),
+      mkSession({ id: 'session-b' }),
+    ]);
+
+    await updateUserModel({ json: true, dryRun: true });
+
+    const output = lastJsonOutput(consoleSpy);
+    expect(output.newSessions).toBe(2);
+    expect(output.changes).toEqual(
+      expect.arrayContaining([
+        expect.objectContaining({
+          type: 'new_pattern',
+          key: '数据库迁移方案需要执行',
+        }),
+      ]),
+    );
+  });
+
+  test('--dry-run：只展示变化，不写 state 文件', async () => {
+    mockReadTranscriptSessions.mockReturnValue([mkSession({ id: 'session-a' })]);
+
+    await updateUserModel({ dryRun: true });
+
+    expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('Processed 1 new sessions'));
+    expect(fs.existsSync(STATE_FILE)).toBe(false);
+  });
+
+  test('默认（非 dry-run）：落盘 state 并记录已处理会话', async () => {
+    mockReadTranscriptSessions.mockReturnValue([mkSession({ id: 'session-a' })]);
+
+    await updateUserModel({});
+
+    expect(fs.existsSync(STATE_FILE)).toBe(true);
+    const state = JSON.parse(fs.readFileSync(STATE_FILE, 'utf-8'));
+    expect(state.sessionsProcessed).toContain('session-a');
+  });
+
+  test('--days 1：只处理最近 1 天的会话', async () => {
+    mockReadTranscriptSessions.mockReturnValue([
+      mkSession({ id: 'today' }),
+      mkSession({
+        id: 'three-days-ago',
+        date: todayStr(3),
+        turns: [
+          { role: 'user', content: '旧会话概念内容测试' },
+          { role: 'assistant', content: '' },
+        ],
+      }),
+    ]);
+
+    await updateUserModel({ json: true, dryRun: true, days: 1 });
+
+    const output = lastJsonOutput(consoleSpy);
+    expect(output.newSessions).toBe(1);
+  });
+
+  test('缺省 --days：处理全部未处理会话（向后兼容）', async () => {
+    mockReadTranscriptSessions.mockReturnValue([
+      mkSession({ id: 'today' }),
+      mkSession({
+        id: 'three-days-ago',
+        date: todayStr(3),
+        turns: [
+          { role: 'user', content: '旧会话概念内容测试' },
+          { role: 'assistant', content: '' },
+        ],
+      }),
+    ]);
+
+    await updateUserModel({ json: true, dryRun: true });
+
+    const output = lastJsonOutput(consoleSpy);
+    expect(output.newSessions).toBe(2);
+  });
+
+  test('sessionsProcessed 去重：已处理会话不重复计入（与 --days 正交）', async () => {
+    fs.writeFileSync(
+      STATE_FILE,
+      JSON.stringify({
+        lastUpdated: '',
+        sessionsProcessed: ['session-a'],
+        patterns: {},
+        lensWeights: {},
+        principleWeights: {},
+        evolutionLog: [],
+      }),
+      'utf-8',
+    );
+    mockReadTranscriptSessions.mockReturnValue([
+      mkSession({ id: 'session-a' }),
+      mkSession({ id: 'session-b' }),
+    ]);
+
+    await updateUserModel({ json: true, dryRun: true, days: 7 });
+
+    const output = lastJsonOutput(consoleSpy);
+    expect(output.newSessions).toBe(1);
+  });
+});

package/src/cli/commands/check.ts

@@ -49,7 +49,6 @@ export async function check(options: CheckOptions): Promise<void> {
     const merged = configLoader.hasCustomConfig()
       ? configLoader.mergeConstraints()
       : configLoader.mergeConstraints({ preset: options.preset || 'standard' });
-    constraintChecker.setCustomConfig(merged);
     if (merged.custom.length > 0) {
       console.log(chalk.gray(`自定义约束: ${merged.custom.length} 条`));
     }
@@ -73,8 +72,8 @@ export async function check(options: CheckOptions): Promise<void> {
     }
     console.log(chalk.gray(`触发条件: ${[context.operation, ...(context.extraTriggers ?? [])].join(', ')}`));
 
-    // 执行三层检查
-    const result = await constraintChecker.checkConstraints(context);
+    // 执行三层检查（per-request 传 customConfig，避免单例状态污染）
+    const result = await constraintChecker.checkConstraints(context, merged);
 
     // 输出结果
     console.log();
@@ -83,7 +82,6 @@ export async function check(options: CheckOptions): Promise<void> {
     const skippedResults = [
       ...result.ironLaws,
       ...result.guidelines,
-      ...result.tips,
     ].filter(r => r.skipped);
 
     // Iron Laws
@@ -118,16 +116,6 @@ export async function check(options: CheckOptions): Promise<void> {
       console.log(chalk.green(`✅ 指导原则: ${passedGuidelines}/${evaluatedGuidelines.length} 通过`));
     }
 
-    // Tips
-    if (result.tipCount > 0) {
-      console.log(chalk.blue(`💡 提示: ${result.tipCount} 条`));
-      result.tips.filter(r => !r.satisfied).forEach(r => {
-        if (r.constraint) {
-          console.log(chalk.blue(`   - ${r.constraint.id}: ${r.constraint.message}`));
-        }
-      });
-    }
-
     // Skipped：约定未采用 / 证据未接线，未评估（不计通过/失败）
     if (skippedResults.length > 0) {
       console.log(chalk.gray(`⏭️  跳过评估: ${skippedResults.length} 条（约定未采用或证据未接线，不计通过/失败）`));