craftdesk 0.2.0 → 0.4.0

package/tests/unit/git-resolver.test.ts

@@ -1,303 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
-import { GitResolver } from '../../src/services/git-resolver';
-import { createTempDir, cleanupTempDir, writeJsonFile } from '../helpers/test-utils';
-import path from 'path';
-import fs from 'fs-extra';
-import { execSync } from 'child_process';
-
-// Mock execSync for these tests
-vi.mock('child_process', () => ({
-  execSync: vi.fn()
-}));
-
-describe('GitResolver', () => {
-  let gitResolver: GitResolver;
-  let tempDir: string;
-
-  beforeEach(async () => {
-    gitResolver = new GitResolver();
-    tempDir = await createTempDir('git-test-');
-    vi.clearAllMocks();
-  });
-
-  afterEach(async () => {
-    await cleanupTempDir(tempDir);
-  });
-
-  describe('inferCraftType', () => {
-    it('should infer skill type from SKILL.md marker file', async () => {
-      const testDir = await createTempDir('type-test-');
-      await fs.writeFile(path.join(testDir, 'SKILL.md'), '# Skill');
-
-      // Access private method via type assertion
-      const type = await (gitResolver as any).inferCraftType(testDir);
-
-      expect(type).toBe('skill');
-      await cleanupTempDir(testDir);
-    });
-
-    it('should infer agent type from AGENT.md marker file', async () => {
-      const testDir = await createTempDir('type-test-');
-      await fs.writeFile(path.join(testDir, 'AGENT.md'), '# Agent');
-
-      const type = await (gitResolver as any).inferCraftType(testDir);
-
-      expect(type).toBe('agent');
-      await cleanupTempDir(testDir);
-    });
-
-    it('should infer command type from COMMAND.md marker file', async () => {
-      const testDir = await createTempDir('type-test-');
-      await fs.writeFile(path.join(testDir, 'COMMAND.md'), '# Command');
-
-      const type = await (gitResolver as any).inferCraftType(testDir);
-
-      expect(type).toBe('command');
-      await cleanupTempDir(testDir);
-    });
-
-    it('should infer hook type from HOOK.md marker file', async () => {
-      const testDir = await createTempDir('type-test-');
-      await fs.writeFile(path.join(testDir, 'HOOK.md'), '# Hook');
-
-      const type = await (gitResolver as any).inferCraftType(testDir);
-
-      expect(type).toBe('hook');
-      await cleanupTempDir(testDir);
-    });
-
-    it('should infer type from directory name containing "skill"', async () => {
-      const testDir = await createTempDir('my-skill-');
-
-      const type = await (gitResolver as any).inferCraftType(testDir);
-
-      expect(type).toBe('skill');
-      await cleanupTempDir(testDir);
-    });
-
-    it('should infer type from directory name containing "agent"', async () => {
-      const testDir = path.join(tempDir, 'my-agent');
-      await fs.ensureDir(testDir);
-
-      const type = await (gitResolver as any).inferCraftType(testDir);
-
-      expect(type).toBe('agent');
-    });
-
-    it('should default to skill when no indicators found', async () => {
-      const testDir = await createTempDir('random-name-');
-
-      const type = await (gitResolver as any).inferCraftType(testDir);
-
-      expect(type).toBe('skill');
-      await cleanupTempDir(testDir);
-    });
-  });
-
-  describe('resolveGitDependency', () => {
-    it('should resolve git dependency with craftdesk.json', async () => {
-      const mockRepoPath = path.join(tempDir, 'mock-repo');
-      await fs.ensureDir(mockRepoPath);
-
-      const craftDeskJson = {
-        name: 'test-skill',
-        version: '1.0.0',
-        type: 'skill',
-        description: 'Test skill',
-        dependencies: {
-          'helper-lib': '^1.0.0'
-        }
-      };
-
-      await writeJsonFile(path.join(mockRepoPath, 'craftdesk.json'), craftDeskJson);
-
-      // Mock git commands
-      const mockExecSync = vi.mocked(execSync);
-      mockExecSync.mockImplementation((cmd: any, options?: any) => {
-        const cmdStr = cmd.toString();
-        if (cmdStr.includes('git clone')) {
-          // Simulate successful clone by copying our mock repo
-          const targetMatch = cmdStr.match(/git clone.*\s+(.+)$/);
-          if (targetMatch) {
-            fs.copySync(mockRepoPath, targetMatch[1]);
-          }
-          return Buffer.from('');
-        } else if (cmdStr.includes('git rev-parse HEAD')) {
-          const output = 'abc123def456789012345678901234567890abcd\n';
-          return options?.encoding === 'utf8' ? output : Buffer.from(output);
-        }
-        return Buffer.from('');
-      });
-
-      const result = await gitResolver.resolveGitDependency({
-        url: 'https://github.com/test/repo.git',
-        branch: 'main'
-      });
-
-      expect(result.craftDeskJson).toEqual(craftDeskJson);
-      expect(result.resolvedCommit).toBe('abc123def456789012345678901234567890abcd');
-    });
-
-    it('should generate minimal metadata when craftdesk.json is missing', async () => {
-      const mockRepoPath = path.join(tempDir, 'mock-repo-no-json');
-      await fs.ensureDir(mockRepoPath);
-      await fs.writeFile(path.join(mockRepoPath, 'SKILL.md'), '# Skill');
-
-      const mockExecSync = vi.mocked(execSync);
-      mockExecSync.mockImplementation((cmd: any, options?: any) => {
-        const cmdStr = cmd.toString();
-        if (cmdStr.includes('git clone')) {
-          const targetMatch = cmdStr.match(/git clone.*\s+(.+)$/);
-          if (targetMatch) {
-            fs.copySync(mockRepoPath, targetMatch[1]);
-          }
-          return Buffer.from('');
-        } else if (cmdStr.includes('git rev-parse HEAD')) {
-          const output = 'abc123def456789012345678901234567890abcd\n';
-          return options?.encoding === 'utf8' ? output : Buffer.from(output);
-        }
-        return Buffer.from('');
-      });
-
-      const result = await gitResolver.resolveGitDependency({
-        url: 'https://github.com/test/repo.git',
-        branch: 'main'
-      });
-
-      expect(result.craftDeskJson).toBeDefined();
-      expect(result.craftDeskJson!.name).toBe('repo');
-      expect(result.craftDeskJson!.version).toBe('main');
-      expect(result.craftDeskJson!.type).toBe('skill');
-      expect(result.craftDeskJson!.dependencies).toEqual({});
-    });
-
-    it('should handle subdirectory path for monorepos', async () => {
-      const mockRepoPath = path.join(tempDir, 'mock-monorepo');
-      const skillPath = path.join(mockRepoPath, 'packages', 'auth');
-      await fs.ensureDir(skillPath);
-
-      const craftDeskJson = {
-        name: 'auth-skill',
-        version: '2.0.0',
-        type: 'skill',
-        dependencies: {}
-      };
-
-      await writeJsonFile(path.join(skillPath, 'craftdesk.json'), craftDeskJson);
-
-      const mockExecSync = vi.mocked(execSync);
-      mockExecSync.mockImplementation((cmd: any, options?: any) => {
-        const cmdStr = cmd.toString();
-        if (cmdStr.includes('git clone')) {
-          const targetMatch = cmdStr.match(/git clone.*\s+(.+)$/);
-          if (targetMatch) {
-            fs.copySync(mockRepoPath, targetMatch[1]);
-          }
-          return Buffer.from('');
-        } else if (cmdStr.includes('git rev-parse HEAD')) {
-          const output = 'def456789012345678901234567890abcdef123\n';
-          return options?.encoding === 'utf8' ? output : Buffer.from(output);
-        }
-        return Buffer.from('');
-      });
-
-      const result = await gitResolver.resolveGitDependency({
-        url: 'https://github.com/test/monorepo.git',
-        tag: 'v2.0.0',
-        path: 'packages/auth'
-      });
-
-      expect(result.craftDeskJson).toEqual(craftDeskJson);
-      expect(result.path).toBe('packages/auth');
-    });
-  });
-
-  describe('resolveAllDependencies', () => {
-    it('should resolve registry dependencies', async () => {
-      const dependencies = {
-        'ruby-on-rails': '^7.0.0',
-        'postgres-expert': '^1.0.0'
-      };
-
-      const result = await gitResolver.resolveAllDependencies(dependencies);
-
-      expect(result.resolved['ruby-on-rails']).toEqual({
-        needsResolution: true,
-        version: '^7.0.0',
-        registry: undefined
-      });
-
-      expect(result.resolved['postgres-expert']).toEqual({
-        needsResolution: true,
-        version: '^1.0.0',
-        registry: undefined
-      });
-    });
-
-    it('should resolve mixed git and registry dependencies', async () => {
-      const mockRepoPath = path.join(tempDir, 'mock-git-dep');
-      await fs.ensureDir(mockRepoPath);
-
-      const craftDeskJson = {
-        name: 'custom-auth',
-        version: '1.0.0',
-        type: 'agent',
-        dependencies: {
-          'jwt-helper': '^2.0.0'
-        }
-      };
-
-      await writeJsonFile(path.join(mockRepoPath, 'craftdesk.json'), craftDeskJson);
-
-      const mockExecSync = vi.mocked(execSync);
-      mockExecSync.mockImplementation((cmd: any, options?: any) => {
-        const cmdStr = cmd.toString();
-        if (cmdStr.includes('git clone')) {
-          const targetMatch = cmdStr.match(/git clone.*\s+(.+)$/);
-          if (targetMatch) {
-            fs.copySync(mockRepoPath, targetMatch[1]);
-          }
-          return Buffer.from('');
-        } else if (cmdStr.includes('git rev-parse HEAD')) {
-          const output = 'abc123def456789012345678901234567890abcd\n';
-          return options?.encoding === 'utf8' ? output : Buffer.from(output);
-        }
-        return Buffer.from('');
-      });
-
-      const dependencies = {
-        'ruby-on-rails': '^7.0.0',
-        'custom-auth': {
-          git: 'https://github.com/company/auth.git',
-          branch: 'main'
-        }
-      };
-
-      const result = await gitResolver.resolveAllDependencies(dependencies);
-
-      // Check registry dependency
-      expect(result.resolved['ruby-on-rails'].needsResolution).toBe(true);
-
-      // Check git dependency
-      expect(result.resolved['custom-auth'].git).toBe('https://github.com/company/auth.git');
-      expect(result.resolved['custom-auth'].branch).toBe('main');
-      expect(result.resolved['custom-auth'].type).toBe('agent');
-
-      // Check transitive dependency from git
-      expect(result.resolved['jwt-helper']).toBeDefined();
-      expect(result.resolved['jwt-helper'].version).toBe('^2.0.0');
-    });
-
-    it('should avoid circular dependencies', async () => {
-      const dependencies = {
-        'package-a': '^1.0.0'
-      };
-
-      // The algorithm marks visited packages to avoid infinite loops
-      const result = await gitResolver.resolveAllDependencies(dependencies);
-
-      expect(result.resolved['package-a']).toBeDefined();
-      expect(result.lockfile.crafts).toBeDefined();
-    });
-  });
-});

package/tests/unit/installer.test.ts

@@ -1,105 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
-import { Installer } from '../../src/services/installer';
-import { createTempDir, cleanupTempDir } from '../helpers/test-utils';
-import path from 'path';
-
-describe('Installer', () => {
-  let installer: Installer;
-  let tempDir: string;
-  let originalCwd: string;
-
-  beforeEach(async () => {
-    installer = new Installer();
-    tempDir = await createTempDir('installer-test-');
-    originalCwd = process.cwd();
-    process.chdir(tempDir);
-  });
-
-  afterEach(async () => {
-    process.chdir(originalCwd);
-    await cleanupTempDir(tempDir);
-  });
-
-  describe('getTypeDirectory', () => {
-    it('should return correct directory for skill type', () => {
-      const result = (installer as any).getTypeDirectory('skill');
-      expect(result).toBe('skills');
-    });
-
-    it('should return correct directory for agent type', () => {
-      const result = (installer as any).getTypeDirectory('agent');
-      expect(result).toBe('agents');
-    });
-
-    it('should return correct directory for command type', () => {
-      const result = (installer as any).getTypeDirectory('command');
-      expect(result).toBe('commands');
-    });
-
-    it('should return correct directory for hook type', () => {
-      const result = (installer as any).getTypeDirectory('hook');
-      expect(result).toBe('hooks');
-    });
-
-    it('should default to crafts for unknown type', () => {
-      const result = (installer as any).getTypeDirectory('unknown' as any);
-      expect(result).toBe('crafts');
-    });
-  });
-
-  describe('createMetadata', () => {
-    it('should create craftdesk-metadata.json file', async () => {
-      const fs = await import('fs-extra');
-      const craftDir = path.join(tempDir, '.claude', 'skills', 'test-craft');
-      await fs.ensureDir(craftDir);
-
-      const entry = {
-        version: '1.0.0',
-        resolved: 'https://craftdesk.ai/download',
-        integrity: 'sha256-test',
-        type: 'skill' as const,
-        author: 'test-author',
-        dependencies: {}
-      };
-
-      await (installer as any).createMetadata(craftDir, 'test-craft', entry);
-
-      const metadataPath = path.join(craftDir, '.craftdesk-metadata.json');
-      const exists = await fs.pathExists(metadataPath);
-      expect(exists).toBe(true);
-
-      const metadata = await fs.readJson(metadataPath);
-      expect(metadata.name).toBe('test-craft');
-      expect(metadata.version).toBe('1.0.0');
-      expect(metadata.type).toBe('skill');
-      expect(metadata.installedAt).toBeDefined();
-    });
-  });
-
-  describe('installFromLockfile', () => {
-    it('should handle empty lockfile', async () => {
-      const lockfile = {
-        version: '1.0.0',
-        lockfileVersion: 1,
-        crafts: {}
-      };
-
-      await expect(installer.installFromLockfile(lockfile)).resolves.not.toThrow();
-    });
-
-    it('should create install directory structure', async () => {
-      const fs = await import('fs-extra');
-      const lockfile = {
-        version: '1.0.0',
-        lockfileVersion: 1,
-        crafts: {}
-      };
-
-      await installer.installFromLockfile(lockfile);
-
-      const installDir = path.join(tempDir, '.claude');
-      const exists = await fs.pathExists(installDir);
-      expect(exists).toBe(true);
-    });
-  });
-});

package/tsconfig.json

@@ -1,20 +0,0 @@
-{
-  "compilerOptions": {
-    "target": "ES2022",
-    "module": "commonjs",
-    "lib": ["ES2022"],
-    "outDir": "./dist",
-    "rootDir": "./src",
-    "strict": true,
-    "esModuleInterop": true,
-    "skipLibCheck": true,
-    "forceConsistentCasingInFileNames": true,
-    "resolveJsonModule": true,
-    "declaration": true,
-    "declarationMap": true,
-    "sourceMap": true,
-    "moduleResolution": "node"
-  },
-  "include": ["src/**/*"],
-  "exclude": ["node_modules", "dist", "tests"]
-}

package/vitest.config.ts

@@ -1,29 +0,0 @@
-import { defineConfig } from 'vitest/config';
-
-export default defineConfig({
-  test: {
-    globals: true,
-    environment: 'node',
-    pool: 'forks',
-    poolOptions: {
-      forks: {
-        singleFork: true
-      }
-    },
-    coverage: {
-      provider: 'v8',
-      reporter: ['text', 'json', 'html'],
-      exclude: [
-        'node_modules/',
-        'dist/',
-        'bin/',
-        '**/*.d.ts',
-        '**/*.config.*',
-        '**/mockData/',
-        'tests/'
-      ]
-    },
-    include: ['tests/**/*.test.ts'],
-    exclude: ['node_modules', 'dist']
-  }
-});