clawt 2.13.0 → 2.14.0

package/tests/unit/constants/config.test.ts

@@ -1,5 +1,6 @@
 import { describe, it, expect } from 'vitest';
 import { CONFIG_DEFINITIONS, DEFAULT_CONFIG, CONFIG_DESCRIPTIONS } from '../../../src/constants/config.js';
+import { VALID_TERMINAL_APPS } from '../../../src/constants/terminal.js';
 
 describe('CONFIG_DEFINITIONS', () => {
   it('所有配置项都有 defaultValue 和 description', () => {
@@ -21,7 +22,7 @@ describe('DEFAULT_CONFIG', () => {
 
   it('每个 key 的值等于对应 CONFIG_DEFINITIONS 的 defaultValue', () => {
     for (const [key, def] of Object.entries(CONFIG_DEFINITIONS)) {
-      expect((DEFAULT_CONFIG as Record<string, unknown>)[key]).toBe(def.defaultValue);
+      expect((DEFAULT_CONFIG as unknown as Record<string, unknown>)[key]).toBe(def.defaultValue);
     }
   });
 
@@ -55,3 +56,25 @@ describe('CONFIG_DESCRIPTIONS', () => {
     }
   });
 });
+
+describe('CONFIG_DEFINITIONS — allowedValues', () => {
+  it('terminalApp 的 allowedValues 与 VALID_TERMINAL_APPS 一致', () => {
+    expect(CONFIG_DEFINITIONS.terminalApp.allowedValues).toEqual(VALID_TERMINAL_APPS);
+  });
+
+  it('有 allowedValues 的配置项，defaultValue 必须在其中', () => {
+    for (const [, def] of Object.entries(CONFIG_DEFINITIONS)) {
+      if (def.allowedValues) {
+        expect(def.allowedValues).toContain(def.defaultValue);
+      }
+    }
+  });
+
+  it('非 string 类型的配置项不应有 allowedValues', () => {
+    for (const [, def] of Object.entries(CONFIG_DEFINITIONS)) {
+      if (typeof def.defaultValue !== 'string') {
+        expect(def.allowedValues).toBeUndefined();
+      }
+    }
+  });
+});

package/tests/unit/utils/config-strategy.test.ts

@@ -0,0 +1,217 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+
+// mock enquirer（必须在所有 import 之前）
+const { mockSelectRun, mockInputRun } = vi.hoisted(() => {
+  const mockSelectRun = vi.fn();
+  const mockInputRun = vi.fn();
+  return { mockSelectRun, mockInputRun };
+});
+
+vi.mock('enquirer', () => ({
+  default: {
+    Select: function MockSelect() { return { run: mockSelectRun }; },
+    Input: function MockInput() { return { run: mockInputRun }; },
+  },
+}));
+
+vi.mock('../../../src/logger/index.js', () => ({
+  logger: { debug: vi.fn(), info: vi.fn(), warn: vi.fn(), error: vi.fn() },
+}));
+
+vi.mock('../../../src/constants/index.js', async (importOriginal) => {
+  const original = await importOriginal<typeof import('../../../src/constants/index.js')>();
+  return {
+    ...original,
+    DEFAULT_CONFIG: {
+      autoDeleteBranch: false,
+      claudeCodeCommand: 'claude',
+      autoPullPush: false,
+      confirmDestructiveOps: true,
+      maxConcurrency: 0,
+      terminalApp: 'auto',
+    },
+    CONFIG_DEFINITIONS: {
+      autoDeleteBranch: { defaultValue: false, description: '自动删除分支' },
+      claudeCodeCommand: { defaultValue: 'claude', description: 'Claude Code CLI 命令' },
+      autoPullPush: { defaultValue: false, description: '自动 pull/push' },
+      confirmDestructiveOps: { defaultValue: true, description: '破坏性操作确认' },
+      maxConcurrency: { defaultValue: 0, description: '最大并发数' },
+      terminalApp: { defaultValue: 'auto', description: '终端应用', allowedValues: ['auto', 'iterm2', 'terminal'] },
+    },
+    MESSAGES: {
+      CONFIG_INVALID_BOOLEAN: (key: string) =>
+        `配置项 ${key} 为布尔类型，仅接受 true 或 false`,
+      CONFIG_INVALID_NUMBER: (key: string) =>
+        `配置项 ${key} 为数字类型，请输入有效的数字`,
+      CONFIG_INVALID_ENUM: (key: string, validValues: readonly string[]) =>
+        `配置项 ${key} 仅接受以下值: ${validValues.join(', ')}`,
+      CONFIG_INPUT_PROMPT: (key: string) => `输入 ${key} 的新值`,
+    },
+  };
+});
+
+import {
+  isValidConfigKey,
+  getValidConfigKeys,
+  parseConfigValue,
+  promptConfigValue,
+  formatConfigValue,
+} from '../../../src/utils/config-strategy.js';
+
+beforeEach(() => {
+  mockSelectRun.mockReset();
+  mockInputRun.mockReset();
+});
+
+describe('isValidConfigKey', () => {
+  it('有效 key 返回 true', () => {
+    expect(isValidConfigKey('autoDeleteBranch')).toBe(true);
+    expect(isValidConfigKey('maxConcurrency')).toBe(true);
+    expect(isValidConfigKey('terminalApp')).toBe(true);
+  });
+
+  it('无效 key 返回 false', () => {
+    expect(isValidConfigKey('foobar')).toBe(false);
+    expect(isValidConfigKey('')).toBe(false);
+    expect(isValidConfigKey('AUTODELETE')).toBe(false);
+  });
+});
+
+describe('getValidConfigKeys', () => {
+  it('返回所有配置项名称', () => {
+    const keys = getValidConfigKeys();
+    expect(keys).toContain('autoDeleteBranch');
+    expect(keys).toContain('claudeCodeCommand');
+    expect(keys).toContain('autoPullPush');
+    expect(keys).toContain('confirmDestructiveOps');
+    expect(keys).toContain('maxConcurrency');
+    expect(keys).toContain('terminalApp');
+    expect(keys).toHaveLength(6);
+  });
+});
+
+describe('parseConfigValue', () => {
+  describe('布尔类型策略', () => {
+    it('解析 "true" 为 true', () => {
+      const result = parseConfigValue('autoDeleteBranch', 'true');
+      expect(result).toEqual({ success: true, value: true });
+    });
+
+    it('解析 "false" 为 false', () => {
+      const result = parseConfigValue('autoDeleteBranch', 'false');
+      expect(result).toEqual({ success: true, value: false });
+    });
+
+    it('无效布尔值返回错误', () => {
+      const result = parseConfigValue('autoDeleteBranch', 'abc');
+      expect(result.success).toBe(false);
+      if (!result.success) {
+        expect(result.error).toContain('布尔类型');
+      }
+    });
+  });
+
+  describe('数字类型策略', () => {
+    it('解析有效数字', () => {
+      const result = parseConfigValue('maxConcurrency', '4');
+      expect(result).toEqual({ success: true, value: 4 });
+    });
+
+    it('解析 0', () => {
+      const result = parseConfigValue('maxConcurrency', '0');
+      expect(result).toEqual({ success: true, value: 0 });
+    });
+
+    it('无效数字返回错误', () => {
+      const result = parseConfigValue('maxConcurrency', 'abc');
+      expect(result.success).toBe(false);
+      if (!result.success) {
+        expect(result.error).toContain('数字类型');
+      }
+    });
+  });
+
+  describe('字符串 + 有 allowedValues 策略（枚举）', () => {
+    it('有效枚举值通过校验', () => {
+      const result = parseConfigValue('terminalApp', 'iterm2');
+      expect(result).toEqual({ success: true, value: 'iterm2' });
+    });
+
+    it('所有枚举值均可接受', () => {
+      expect(parseConfigValue('terminalApp', 'auto')).toEqual({ success: true, value: 'auto' });
+      expect(parseConfigValue('terminalApp', 'terminal')).toEqual({ success: true, value: 'terminal' });
+    });
+
+    it('无效枚举值返回错误', () => {
+      const result = parseConfigValue('terminalApp', 'invalid');
+      expect(result.success).toBe(false);
+      if (!result.success) {
+        expect(result.error).toContain('仅接受以下值');
+      }
+    });
+  });
+
+  describe('字符串 + 无 allowedValues 策略（自由输入）', () => {
+    it('任意字符串值通过校验', () => {
+      const result = parseConfigValue('claudeCodeCommand', 'cc');
+      expect(result).toEqual({ success: true, value: 'cc' });
+    });
+
+    it('空字符串也通过校验', () => {
+      const result = parseConfigValue('claudeCodeCommand', '');
+      expect(result).toEqual({ success: true, value: '' });
+    });
+  });
+});
+
+describe('promptConfigValue', () => {
+  it('布尔类型使用 Select 提示', async () => {
+    mockSelectRun.mockResolvedValueOnce('true');
+    const result = await promptConfigValue('autoDeleteBranch', false);
+    expect(result).toBe(true);
+    expect(mockSelectRun).toHaveBeenCalledTimes(1);
+  });
+
+  it('数字类型使用 Input 提示', async () => {
+    mockInputRun.mockResolvedValueOnce('8');
+    const result = await promptConfigValue('maxConcurrency', 0);
+    expect(result).toBe(8);
+    expect(mockInputRun).toHaveBeenCalledTimes(1);
+  });
+
+  it('字符串 + 有 allowedValues 使用 Select 提示', async () => {
+    mockSelectRun.mockResolvedValueOnce('iterm2');
+    const result = await promptConfigValue('terminalApp', 'auto');
+    expect(result).toBe('iterm2');
+    expect(mockSelectRun).toHaveBeenCalledTimes(1);
+  });
+
+  it('字符串 + 无 allowedValues 使用 Input 提示', async () => {
+    mockInputRun.mockResolvedValueOnce('cc');
+    const result = await promptConfigValue('claudeCodeCommand', 'claude');
+    expect(result).toBe('cc');
+    expect(mockInputRun).toHaveBeenCalledTimes(1);
+  });
+});
+
+describe('formatConfigValue', () => {
+  it('true 显示为绿色', () => {
+    const result = formatConfigValue(true);
+    expect(result).toContain('true');
+  });
+
+  it('false 显示为黄色', () => {
+    const result = formatConfigValue(false);
+    expect(result).toContain('false');
+  });
+
+  it('数字显示为 cyan', () => {
+    const result = formatConfigValue(42);
+    expect(result).toContain('42');
+  });
+
+  it('字符串显示为 cyan', () => {
+    const result = formatConfigValue('hello');
+    expect(result).toContain('hello');
+  });
+});

package/tests/unit/utils/config.test.ts

@@ -18,7 +18,7 @@ vi.mock('../../../src/utils/fs.js', () => ({
 }));
 
 import { existsSync, readFileSync, writeFileSync } from 'node:fs';
-import { loadConfig, getConfigValue, writeDefaultConfig, writeConfig, ensureClawtDirs } from '../../../src/utils/config.js';
+import { loadConfig, getConfigValue, writeDefaultConfig, writeConfig, saveConfig, ensureClawtDirs } from '../../../src/utils/config.js';
 import { DEFAULT_CONFIG } from '../../../src/constants/index.js';
 import { ensureDir } from '../../../src/utils/fs.js';
 
@@ -95,3 +95,15 @@ describe('ensureClawtDirs', () => {
     expect(mockedEnsureDir).toHaveBeenCalledTimes(3);
   });
 });
+
+describe('saveConfig', () => {
+  it('将配置对象写入配置文件', () => {
+    const customConfig = { ...DEFAULT_CONFIG, autoDeleteBranch: true };
+    saveConfig(customConfig);
+    expect(mockedWriteFileSync).toHaveBeenCalledWith(
+      expect.any(String),
+      JSON.stringify(customConfig, null, 2),
+      'utf-8',
+    );
+  });
+});