sumulige-claude 1.5.1 → 1.5.2

package/tests/marketplace.test.js

@@ -1,420 +0,0 @@
-/**
- * Marketplace 模块单元测试 - 扩展版
- * 测试技能市场管理和 YAML 解析功能
- */
-
-const { parseSimpleYaml } = require('../lib/marketplace');
-
-describe('Marketplace Module', () => {
-
-  // ============================================================================
-  // parseSimpleYaml 单元测试
-  // ============================================================================
-
-  describe('parseSimpleYaml', () => {
-    it('should parse version number', () => {
-      const yaml = `version: 1
-skills:
-  - name: test-skill`;
-      expect(parseSimpleYaml(yaml).version).toBe(1);
-    });
-
-    it('should parse version as integer', () => {
-      const yaml = `version: 2
-skills:`;
-      expect(typeof parseSimpleYaml(yaml).version).toBe('number');
-    });
-
-    it('should handle empty YAML', () => {
-      expect(parseSimpleYaml('')).toEqual({ skills: [] });
-    });
-
-    it('should handle skills section with no skills', () => {
-      const yaml = `version: 1
-skills:`;
-      expect(parseSimpleYaml(yaml).skills).toEqual([]);
-    });
-
-    it('should parse single skill name only', () => {
-      const yaml = `skills:
-  - name: test-skill`;
-      const result = parseSimpleYaml(yaml);
-      expect(result.skills).toHaveLength(1);
-      expect(result.skills[0].name).toBe('test-skill');
-    });
-
-    it('should parse multiple skill names', () => {
-      const yaml = `skills:
-  - name: skill-one
-  - name: skill-two
-  - name: skill-three`;
-      const result = parseSimpleYaml(yaml);
-      expect(result.skills).toHaveLength(3);
-      expect(result.skills[0].name).toBe('skill-one');
-      expect(result.skills[1].name).toBe('skill-two');
-      expect(result.skills[2].name).toBe('skill-three');
-    });
-
-    it('should parse nested source object', () => {
-      const yaml = `skills:
-  - name: test
-  source:
-    repo: owner/repo
-    ref: main`;
-      const result = parseSimpleYaml(yaml);
-      expect(result.skills[0].source).toEqual({ repo: 'owner/repo', ref: 'main' });
-    });
-
-    it('should parse nested target object', () => {
-      const yaml = `skills:
-  - name: test
-  target:
-    category: tools
-    path: /path/to/skill`;
-      const result = parseSimpleYaml(yaml);
-      expect(result.skills[0].target).toEqual({ category: 'tools', path: '/path/to/skill' });
-    });
-
-    it('should parse nested author object', () => {
-      const yaml = `skills:
-  - name: test
-  author:
-    name: Test Author
-    github: testauthor`;
-      const result = parseSimpleYaml(yaml);
-      expect(result.skills[0].author).toEqual({ name: 'Test Author', github: 'testauthor' });
-    });
-
-    it('should parse nested sync object', () => {
-      const yaml = `skills:
-  - name: test
-  sync:
-    include: *.md
-    exclude: node_modules`;
-      const result = parseSimpleYaml(yaml);
-      expect(result.skills[0].sync).toEqual({ include: '*.md', exclude: 'node_modules' });
-    });
-
-    it('should parse boolean true in nested properties', () => {
-      const yaml = `skills:
-  - name: test
-  sync:
-    enabled: true`;
-      const result = parseSimpleYaml(yaml);
-      expect(result.skills[0].sync.enabled).toBe(true);
-    });
-
-    it('should parse boolean false in nested properties', () => {
-      const yaml = `skills:
-  - name: test
-  sync:
-    enabled: false`;
-      const result = parseSimpleYaml(yaml);
-      expect(result.skills[0].sync.enabled).toBe(false);
-    });
-
-    it('should parse empty array in nested properties', () => {
-      const yaml = `skills:
-  - name: test
-  sync:
-    tags: []`;
-      const result = parseSimpleYaml(yaml);
-      expect(result.skills[0].sync.tags).toEqual([]);
-    });
-
-    it('should handle properties with hyphens', () => {
-      const yaml = `skills:
-  - name: my-skill
-  author:
-    github-url: https://github.com/test`;
-      const result = parseSimpleYaml(yaml);
-      expect(result.skills[0].author['github-url']).toBe('https://github.com/test');
-    });
-
-    it('should handle properties with underscores', () => {
-      const yaml = `skills:
-  - name: test
-  author:
-    github_user: test_user`;
-      const result = parseSimpleYaml(yaml);
-      expect(result.skills[0].author.github_user).toBe('test_user');
-    });
-
-    it('should skip comments at start', () => {
-      const yaml = `# Comment at start
-skills:
-  - name: test`;
-      const result = parseSimpleYaml(yaml);
-      expect(result.skills[0].name).toBe('test');
-    });
-
-    it('should skip comments in skills section', () => {
-      const yaml = `skills:
-  # Inline comment
-  - name: test`;
-      const result = parseSimpleYaml(yaml);
-      expect(result.skills[0].name).toBe('test');
-    });
-
-    it('should skip empty lines', () => {
-      const yaml = `skills:
-
-  - name: test
-
-  - name: test2`;
-      const result = parseSimpleYaml(yaml);
-      expect(result.skills).toHaveLength(2);
-    });
-
-    it('should handle complex real-world YAML', () => {
-      const yaml = `version: 1
-skills:
-  - name: dev-browser
-  description: Browser automation
-  native: true
-  target:
-    category: tools
-    path: template/.claude/skills/tools/dev-browser
-  - name: gastown
-  source:
-    repo: SawyerHood/gastown
-    path: skills/gastown
-    ref: main
-  target:
-    category: tools
-    path: template/.claude/skills/tools/gastown
-  author:
-    name: SawyerHood
-    github: SawyerHood
-  license: MIT
-  verified: false`;
-      const result = parseSimpleYaml(yaml);
-      expect(result.version).toBe(1);
-      expect(result.skills).toHaveLength(2);
-      expect(result.skills[0].name).toBe('dev-browser');
-      expect(result.skills[0].native).toBe(true);
-      expect(result.skills[0].target.category).toBe('tools');
-      expect(result.skills[1].name).toBe('gastown');
-      expect(result.skills[1].source.repo).toBe('SawyerHood/gastown');
-      expect(result.skills[1].author.github).toBe('SawyerHood');
-      expect(result.skills[1].verified).toBe(false);
-    });
-
-    it('should handle multiple nested objects in one skill', () => {
-      const yaml = `skills:
-  - name: complex-skill
-  source:
-    repo: owner/repo
-    ref: main
-  target:
-    category: workflow
-    path: /path
-  author:
-    name: Author
-    github: author
-  sync:
-    include: *.md
-    exclude: node_modules`;
-      const result = parseSimpleYaml(yaml);
-      expect(result.skills[0].source.repo).toBe('owner/repo');
-      expect(result.skills[0].target.category).toBe('workflow');
-      expect(result.skills[0].author.name).toBe('Author');
-      expect(result.skills[0].sync.include).toBe('*.md');
-    });
-
-    it('should parse license property', () => {
-      const yaml = `skills:
-  - name: test
-  license: MIT`;
-      const result = parseSimpleYaml(yaml);
-      expect(result.skills[0].license).toBe('MIT');
-    });
-
-    it('should parse homepage property', () => {
-      const yaml = `skills:
-  - name: test
-  homepage: https://example.com`;
-      const result = parseSimpleYaml(yaml);
-      expect(result.skills[0].homepage).toBe('https://example.com');
-    });
-
-    it('should parse verified property', () => {
-      const yaml = `skills:
-  - name: test
-  verified: false`;
-      const result = parseSimpleYaml(yaml);
-      expect(result.skills[0].verified).toBe(false);
-    });
-
-    it('should handle quoted strings', () => {
-      const yaml = `skills:
-  - name: test
-  description: "A quoted description"`;
-      const result = parseSimpleYaml(yaml);
-      expect(result.skills[0].description).toBe('A quoted description');
-    });
-
-    it('should handle single-quoted strings', () => {
-      const yaml = `skills:
-  - name: test
-  description: 'A single-quoted description'`;
-      const result = parseSimpleYaml(yaml);
-      expect(result.skills[0].description).toBe('A single-quoted description');
-    });
-
-    it('should handle URLs with special characters', () => {
-      const yaml = `skills:
-  - name: test
-  homepage: https://github.com/owner/repo/tree/main/skills`;
-      const result = parseSimpleYaml(yaml);
-      expect(result.skills[0].homepage).toBe('https://github.com/owner/repo/tree/main/skills');
-    });
-  });
-
-  // ============================================================================
-  // 市场常量定义测试
-  // ============================================================================
-
-  describe('Marketplace constants', () => {
-    it('should define SOURCES_FILE path', () => {
-      const path = require('path');
-      const sourcesPath = path.join(__dirname, '..', 'sources.yaml');
-      expect(sourcesPath).toContain('sources.yaml');
-    });
-
-    it('should define MARKETPLACE_FILE path', () => {
-      const path = require('path');
-      const marketplacePath = path.join(__dirname, '..', '.claude-plugin', 'marketplace.json');
-      expect(marketplacePath).toContain('marketplace.json');
-    });
-
-    it('should define CATEGORIES_FILE path', () => {
-      const path = require('path');
-      const categoriesPath = path.join(__dirname, '..', 'config', 'skill-categories.json');
-      expect(categoriesPath).toContain('skill-categories.json');
-    });
-  });
-
-  // ============================================================================
-  // YAML 格式验证测试
-  // ============================================================================
-
-  describe('YAML format validation', () => {
-    it('should recognize skills section header', () => {
-      const yaml = `skills:`;
-      const lines = yaml.split('\n');
-      const hasSkillsSection = lines.some(l => l.trim() === 'skills:');
-      expect(hasSkillsSection).toBe(true);
-    });
-
-    it('should recognize version field', () => {
-      const yaml = `version: 1`;
-      const lines = yaml.split('\n');
-      const hasVersion = lines.some(l => l.trim().startsWith('version:'));
-      expect(hasVersion).toBe(true);
-    });
-
-    it('should recognize skill entry marker', () => {
-      const yaml = `  - name: test-skill`;
-      const isSkillEntry = yaml.trim().startsWith('- name:');
-      expect(isSkillEntry).toBe(true);
-    });
-
-    it('should validate indent pattern for skill entries', () => {
-      const line = `  - name: test`;
-      const indent = line.search(/\S/);
-      expect(indent).toBe(2);
-    });
-
-    it('should validate indent pattern for nested properties', () => {
-      const line = `    repo: owner/repo`;
-      const indent = line.search(/\S/);
-      expect(indent).toBe(4);
-    });
-  });
-
-  // ============================================================================
-  // 模块导出测试
-  // ============================================================================
-
-  describe('exports', () => {
-    it('should export marketplaceCommands object', () => {
-      const marketplace = require('../lib/marketplace');
-      expect(marketplace.marketplaceCommands).toBeDefined();
-      expect(typeof marketplace.marketplaceCommands).toBe('object');
-    });
-
-    it('should export all marketplace commands', () => {
-      const marketplace = require('../lib/marketplace');
-      const commands = marketplace.marketplaceCommands;
-
-      expect(commands['marketplace:list']).toBeDefined();
-      expect(commands['marketplace:install']).toBeDefined();
-      expect(commands['marketplace:sync']).toBeDefined();
-      expect(commands['marketplace:add']).toBeDefined();
-      expect(commands['marketplace:remove']).toBeDefined();
-      expect(commands['marketplace:status']).toBeDefined();
-    });
-
-    it('should have exactly 6 commands', () => {
-      const marketplace = require('../lib/marketplace');
-      const commands = marketplace.marketplaceCommands;
-
-      expect(Object.keys(commands).length).toBe(6);
-    });
-
-    it('should export functions with correct types', () => {
-      const marketplace = require('../lib/marketplace');
-      const commands = marketplace.marketplaceCommands;
-
-      Object.values(commands).forEach(cmd => {
-        expect(typeof cmd).toBe('function');
-      });
-    });
-  });
-
-  // ============================================================================
-  // 命令功能测试
-  // ============================================================================
-
-  describe('command features', () => {
-    it('should parse repo format correctly', () => {
-      const repo = 'owner/repo-name';
-      const match = repo.match(/^([^/]+)\/(.+)$/);
-      expect(match).toBeTruthy();
-      expect(match[1]).toBe('owner');
-      expect(match[2]).toBe('repo-name');
-    });
-
-    it('should reject invalid repo format', () => {
-      const repo = 'invalid-format';
-      const match = repo.match(/^([^/]+)\/(.+)$/);
-      expect(match).toBeNull();
-    });
-
-    it('should normalize skill name to kebab-case', () => {
-      const name = 'Owner/Repo_Name';
-      const skillName = name.toLowerCase().replace(/[^a-z0-9-]/g, '-');
-      expect(skillName).toBe('owner-repo-name');
-    });
-
-    it('should detect duplicate skills in sources list', () => {
-      const sources = [
-        { name: 'skill-one' },
-        { name: 'skill-two' },
-        { name: 'skill-three' }
-      ];
-      const exists = sources.some(s => s.name === 'skill-two');
-      expect(exists).toBe(true);
-    });
-
-    it('should not find non-existent skill', () => {
-      const sources = [
-        { name: 'skill-one' },
-        { name: 'skill-two' }
-      ];
-      const exists = sources.some(s => s.name === 'skill-three');
-      expect(exists).toBe(false);
-    });
-  });
-});

package/tests/migrations.test.js

@@ -1,187 +0,0 @@
-/**
- * Migrations 模块单元测试
- */
-
-const fs = require('fs');
-const path = require('path');
-const mockFs = require('mock-fs');
-
-// 在 mock 之前加载模块
-const migrations = require('../lib/migrations');
-
-describe('Migrations Module', () => {
-  let originalFs = { ...fs };
-
-  beforeEach(() => {
-    // Mock 文件系统，保留一些必要的路径
-    mockFs({
-      '/test-project': {
-        '.claude': {}
-      }
-    }, {
-      // 不 mock 项目的 lib 目录
-      [path.join(__dirname, '../lib')]: true
-    });
-  });
-
-  afterEach(() => {
-    mockFs.restore();
-  });
-
-  describe('getProjectVersion', () => {
-    it('should return 1.0.0 when version file does not exist', () => {
-      const version = migrations.getProjectVersion('/test-project');
-      expect(version).toBe('1.0.0');
-    });
-
-    it('should read version from file', () => {
-      const versionFile = path.join('/test-project', '.claude', '.version');
-      fs.writeFileSync(versionFile, '1.0.5\n');
-
-      const version = migrations.getProjectVersion('/test-project');
-      expect(version).toBe('1.0.5');
-    });
-
-    it('should trim whitespace from version', () => {
-      const versionFile = path.join('/test-project', '.claude', '.version');
-      fs.writeFileSync(versionFile, '  1.0.8  \n');
-
-      const version = migrations.getProjectVersion('/test-project');
-      expect(version).toBe('1.0.8');
-    });
-
-    it('should return 1.0.0 on read error', () => {
-      const version = migrations.getProjectVersion('/nonexistent');
-      expect(version).toBe('1.0.0');
-    });
-  });
-
-  describe('setProjectVersion', () => {
-    it('should write version to file', () => {
-      migrations.setProjectVersion('/test-project', '1.0.7');
-
-      const versionFile = path.join('/test-project', '.claude', '.version');
-      expect(fs.existsSync(versionFile)).toBe(true);
-
-      const content = fs.readFileSync(versionFile, 'utf-8');
-      expect(content.trim()).toBe('1.0.7');
-    });
-
-    it('should add newline after version', () => {
-      migrations.setProjectVersion('/test-project', '1.0.9');
-
-      const versionFile = path.join('/test-project', '.claude', '.version');
-      const content = fs.readFileSync(versionFile, 'utf-8');
-      expect(content).toBe('1.0.9\n');
-    });
-
-    it('should create .claude directory if not exists', () => {
-      // 注意：setProjectVersion 不会自动创建目录
-      // 需要先创建目录
-      fs.mkdirSync(path.join('/new-project', '.claude'), { recursive: true });
-      migrations.setProjectVersion('/new-project', '1.0.1');
-
-      const versionFile = path.join('/new-project', '.claude', '.version');
-      expect(fs.existsSync(versionFile)).toBe(true);
-    });
-  });
-
-  describe('compareVersions', () => {
-    it('should return -1 when v1 < v2', () => {
-      expect(migrations.compareVersions('1.0.0', '1.0.1')).toBe(-1);
-      expect(migrations.compareVersions('1.0.0', '1.1.0')).toBe(-1);
-      expect(migrations.compareVersions('1.0.0', '2.0.0')).toBe(-1);
-    });
-
-    it('should return 1 when v1 > v2', () => {
-      expect(migrations.compareVersions('1.0.1', '1.0.0')).toBe(1);
-      expect(migrations.compareVersions('1.1.0', '1.0.0')).toBe(1);
-      expect(migrations.compareVersions('2.0.0', '1.0.0')).toBe(1);
-    });
-
-    it('should return 0 when v1 === v2', () => {
-      expect(migrations.compareVersions('1.0.0', '1.0.0')).toBe(0);
-      expect(migrations.compareVersions('1.5.10', '1.5.10')).toBe(0);
-    });
-
-    it('should handle version comparison correctly', () => {
-      // Patch version comparison
-      expect(migrations.compareVersions('1.0.5', '1.0.10')).toBe(-1);
-      expect(migrations.compareVersions('1.0.10', '1.0.5')).toBe(1);
-
-      // Minor version comparison
-      expect(migrations.compareVersions('1.5.0', '1.10.0')).toBe(-1);
-
-      // Major version comparison
-      expect(migrations.compareVersions('2.0.0', '10.0.0')).toBe(-1);
-    });
-
-    it('should handle version strings with different formats', () => {
-      expect(migrations.compareVersions('1', '1.0.0')).toBe(0);
-      expect(migrations.compareVersions('1.0', '1.0.0')).toBe(0);
-    });
-  });
-
-  describe('runMigrations', () => {
-    it('should return success when already at latest version', () => {
-      // 设置当前版本为最新
-      migrations.setProjectVersion('/test-project', migrations.TEMPLATE_VERSION);
-
-      const result = migrations.runMigrations('/test-project', true);
-
-      expect(result.success).toBe(true);
-      expect(result.migrations).toEqual([]);
-      expect(result.currentVersion).toBe(migrations.TEMPLATE_VERSION);
-    });
-
-    it('should return success when version is newer than template', () => {
-      // 设置一个未来版本
-      migrations.setProjectVersion('/test-project', '2.0.0');
-
-      const result = migrations.runMigrations('/test-project', true);
-
-      expect(result.success).toBe(true);
-      expect(result.migrations).toEqual([]);
-    });
-
-    it('should return correct structure when pending migrations exist', () => {
-      // 设置旧版本
-      migrations.setProjectVersion('/test-project', '1.0.0');
-
-      // 创建需要迁移的 settings.json
-      const settingsFile = path.join('/test-project', '.claude', 'settings.json');
-      fs.writeFileSync(
-        settingsFile,
-        JSON.stringify({ matcher: {}, hooks: [] }, null, 2)
-      );
-
-      const result = migrations.runMigrations('/test-project', true);
-
-      expect(result).toBeDefined();
-      expect(typeof result.success).toBe('boolean');
-      expect(result.currentVersion).toBeDefined();
-    });
-
-    it('should handle missing settings.json gracefully', () => {
-      migrations.setProjectVersion('/test-project', '1.0.0');
-
-      const result = migrations.runMigrations('/test-project', true);
-
-      expect(result.success).toBe(true);
-    });
-  });
-
-  describe('exports', () => {
-    it('should export TEMPLATE_VERSION constant', () => {
-      expect(migrations.TEMPLATE_VERSION).toBeDefined();
-      expect(typeof migrations.TEMPLATE_VERSION).toBe('string');
-    });
-
-    it('should export all functions', () => {
-      expect(typeof migrations.getProjectVersion).toBe('function');
-      expect(typeof migrations.setProjectVersion).toBe('function');
-      expect(typeof migrations.compareVersions).toBe('function');
-      expect(typeof migrations.runMigrations).toBe('function');
-    });
-  });
-});