clawt 3.1.2 → 3.2.0

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

@@ -48,6 +48,7 @@ vi.mock('../../../src/constants/index.js', async (importOriginal) => {
       CONFIG_INVALID_ENUM: (key: string, validValues: readonly string[]) =>
         `配置项 ${key} 仅接受以下值: ${validValues.join(', ')}`,
       CONFIG_INPUT_PROMPT: (key: string) => `输入 ${key} 的新值`,
+      CONFIG_SELECT_PROMPT: '选择要修改的配置项',
     },
   };
 });
@@ -58,6 +59,7 @@ import {
   parseConfigValue,
   promptConfigValue,
   formatConfigValue,
+  interactiveConfigEditor,
 } from '../../../src/utils/config-strategy.js';
 
 beforeEach(() => {
@@ -184,7 +186,7 @@ describe('promptConfigValue', () => {
 
   it('字符串 + 有 allowedValues 使用 Select 提示', async () => {
     mockSelectRun.mockResolvedValueOnce('iterm2');
-    const result = await promptConfigValue('terminalApp', 'auto');
+    const result = await promptConfigValue('terminalApp', 'auto', ['auto', 'iterm2', 'terminal']);
     expect(result).toBe('iterm2');
     expect(mockSelectRun).toHaveBeenCalledTimes(1);
   });
@@ -217,4 +219,78 @@ describe('formatConfigValue', () => {
     const result = formatConfigValue('hello');
     expect(result).toContain('hello');
   });
+
+  it('undefined 显示为 (未设置)', () => {
+    const result = formatConfigValue(undefined);
+    expect(result).toContain('未设置');
+  });
+
+  it('null 显示为 (未设置)', () => {
+    const result = formatConfigValue(null);
+    expect(result).toContain('未设置');
+  });
+});
+
+describe('interactiveConfigEditor', () => {
+  it('选择配置项并返回新值（布尔类型）', async () => {
+    // 第一次 Select 选择配置项，第二次 Select 选择布尔值
+    mockSelectRun.mockResolvedValueOnce('enabled');
+    mockSelectRun.mockResolvedValueOnce('true');
+
+    const config = { enabled: false, name: 'test' };
+    const definitions = {
+      enabled: { description: '是否启用' },
+      name: { description: '名称' },
+    };
+
+    const result = await interactiveConfigEditor(config, definitions);
+    expect(result.key).toBe('enabled');
+    expect(result.newValue).toBe(true);
+  });
+
+  it('选择配置项并返回新值（字符串类型）', async () => {
+    mockSelectRun.mockResolvedValueOnce('name');
+    mockInputRun.mockResolvedValueOnce('new-name');
+
+    const config = { enabled: false, name: 'test' };
+    const definitions = {
+      enabled: { description: '是否启用' },
+      name: { description: '名称' },
+    };
+
+    const result = await interactiveConfigEditor(config, definitions);
+    expect(result.key).toBe('name');
+    expect(result.newValue).toBe('new-name');
+  });
+
+  it('支持自定义 selectPrompt', async () => {
+    mockSelectRun.mockResolvedValueOnce('name');
+    mockInputRun.mockResolvedValueOnce('value');
+
+    const config = { name: 'test' };
+    const definitions = { name: { description: '名称' } };
+
+    await interactiveConfigEditor(config, definitions, {
+      selectPrompt: '自定义提示',
+    });
+
+    // 验证 Select 被调用（无需检查具体参数，因为 mock 不保留构造参数）
+    expect(mockSelectRun).toHaveBeenCalledTimes(1);
+  });
+
+  it('有 allowedValues 的字符串配置项使用 Select 提示', async () => {
+    mockSelectRun.mockResolvedValueOnce('mode');
+    mockSelectRun.mockResolvedValueOnce('fast');
+
+    const config = { mode: 'normal' };
+    const definitions = {
+      mode: { description: '模式', allowedValues: ['normal', 'fast', 'slow'] as const },
+    };
+
+    const result = await interactiveConfigEditor(config, definitions);
+    expect(result.key).toBe('mode');
+    expect(result.newValue).toBe('fast');
+    // 两次 Select：一次选择配置项，一次选择枚举值
+    expect(mockSelectRun).toHaveBeenCalledTimes(2);
+  });
 });

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

@@ -49,6 +49,7 @@ import {
   saveProjectConfig,
   requireProjectConfig,
   getMainWorkBranch,
+  getValidateRunCommand,
 } from '../../../src/utils/project-config.js';
 
 const mockedExistsSync = vi.mocked(existsSync);
@@ -134,3 +135,34 @@ describe('getMainWorkBranch', () => {
     expect(getMainWorkBranch()).toBe('develop');
   });
 });
+
+describe('getValidateRunCommand', () => {
+  it('配置中有 validateRunCommand 时返回对应值', () => {
+    mockedExistsSync.mockReturnValue(true);
+    mockedReadFileSync.mockReturnValue(JSON.stringify({
+      clawtMainWorkBranch: 'main',
+      validateRunCommand: 'npm test',
+    }));
+    expect(getValidateRunCommand()).toBe('npm test');
+  });
+
+  it('配置中无 validateRunCommand 时返回 undefined', () => {
+    mockedExistsSync.mockReturnValue(true);
+    mockedReadFileSync.mockReturnValue(JSON.stringify({ clawtMainWorkBranch: 'main' }));
+    expect(getValidateRunCommand()).toBeUndefined();
+  });
+
+  it('配置文件不存在时返回 undefined', () => {
+    mockedExistsSync.mockReturnValue(false);
+    expect(getValidateRunCommand()).toBeUndefined();
+  });
+
+  it('validateRunCommand 为空字符串时返回 undefined', () => {
+    mockedExistsSync.mockReturnValue(true);
+    mockedReadFileSync.mockReturnValue(JSON.stringify({
+      clawtMainWorkBranch: 'main',
+      validateRunCommand: '',
+    }));
+    expect(getValidateRunCommand()).toBeUndefined();
+  });
+});

package/tests/unit/utils/validate-snapshot.test.ts

@@ -106,10 +106,10 @@ describe('readSnapshotTreeHash', () => {
 });
 
 describe('writeSnapshot', () => {
-  it('正确写入两个文件', () => {
+  it('正确写入三个文件', () => {
     writeSnapshot('proj', 'branch', 'tree123', 'head456');
     expect(mockedEnsureDir).toHaveBeenCalledWith('/tmp/test-snapshots/proj');
-    expect(mockedWriteFileSync).toHaveBeenCalledTimes(2);
+    expect(mockedWriteFileSync).toHaveBeenCalledTimes(3);
     expect(mockedWriteFileSync).toHaveBeenCalledWith(
       '/tmp/test-snapshots/proj/branch.tree',
       'tree123',
@@ -120,6 +120,11 @@ describe('writeSnapshot', () => {
       'head456',
       'utf-8',
     );
+    expect(mockedWriteFileSync).toHaveBeenCalledWith(
+      '/tmp/test-snapshots/proj/branch.staged',
+      '',
+      'utf-8',
+    );
   });
 });
 
@@ -127,7 +132,7 @@ describe('removeSnapshot', () => {
   it('删除存在的文件', () => {
     mockedExistsSync.mockReturnValue(true);
     removeSnapshot('proj', 'branch');
-    expect(mockedUnlinkSync).toHaveBeenCalledTimes(2);
+    expect(mockedUnlinkSync).toHaveBeenCalledTimes(3);
   });
 
   it('文件不存在时不抛错', () => {