craftdesk 0.2.0 → 0.4.0

package/tests/fixtures/craftdesk-git.json

@@ -1,17 +0,0 @@
-{
-  "name": "git-test-project",
-  "version": "1.0.0",
-  "type": "skill",
-  "description": "Test project with git dependencies",
-  "dependencies": {
-    "custom-auth": {
-      "git": "https://github.com/company/auth.git",
-      "branch": "main"
-    },
-    "monorepo-skill": {
-      "git": "https://github.com/company/monorepo.git",
-      "tag": "v3.0.0",
-      "path": "skills/auth"
-    }
-  }
-}

package/tests/fixtures/craftdesk.json

@@ -1,15 +0,0 @@
-{
-  "name": "test-project",
-  "version": "1.0.0",
-  "type": "skill",
-  "description": "Test project for unit tests",
-  "dependencies": {
-    "ruby-on-rails": "^7.0.0",
-    "postgres-expert": "^1.0.0"
-  },
-  "registries": {
-    "default": {
-      "url": "https://craftdesk.ai"
-    }
-  }
-}

package/tests/helpers/test-utils.ts

@@ -1,60 +0,0 @@
-import fs from 'fs-extra';
-import path from 'path';
-
-/**
- * Creates a temporary test directory
- */
-export async function createTempDir(prefix: string = 'test-'): Promise<string> {
-  const tmpDir = path.join(process.cwd(), '.test-tmp', `${prefix}${Date.now()}`);
-  await fs.ensureDir(tmpDir);
-  return tmpDir;
-}
-
-/**
- * Cleans up a temporary directory
- */
-export async function cleanupTempDir(dirPath: string): Promise<void> {
-  if (dirPath.includes('.test-tmp')) {
-    await fs.remove(dirPath);
-  }
-}
-
-/**
- * Copies a fixture file to a target directory
- */
-export async function copyFixture(fixtureName: string, targetDir: string): Promise<void> {
-  const fixturePath = path.join(__dirname, '..', 'fixtures', fixtureName);
-  const targetPath = path.join(targetDir, path.basename(fixtureName));
-  await fs.copy(fixturePath, targetPath);
-}
-
-/**
- * Reads a JSON file and returns parsed content
- */
-export async function readJsonFile<T = any>(filePath: string): Promise<T> {
-  const content = await fs.readFile(filePath, 'utf-8');
-  return JSON.parse(content);
-}
-
-/**
- * Writes a JSON object to a file
- */
-export async function writeJsonFile(filePath: string, data: any): Promise<void> {
-  await fs.writeFile(filePath, JSON.stringify(data, null, 2), 'utf-8');
-}
-
-/**
- * Mock registry response for testing
- */
-export function createMockCraftInfo(name: string, version: string = '1.0.0') {
-  return {
-    name,
-    version,
-    type: 'skill',
-    author: 'test-author',
-    description: 'Test craft',
-    download_url: `https://craftdesk.ai/api/v1/crafts/test-author/${name}/versions/${version}/download`,
-    integrity: 'sha256-test123',
-    dependencies: {}
-  };
-}

package/tests/integration/init-command.test.ts

@@ -1,70 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import { execSync } from 'child_process';
-import { createTempDir, cleanupTempDir, readJsonFile } from '../helpers/test-utils';
-import path from 'path';
-import fs from 'fs-extra';
-
-describe('craftdesk init command', () => {
-  let tempDir: string;
-  let originalCwd: string;
-  const cliPath = path.join(__dirname, '../../dist/index.js');
-
-  beforeEach(async () => {
-    tempDir = await createTempDir('init-test-');
-    originalCwd = process.cwd();
-    process.chdir(tempDir);
-  });
-
-  afterEach(async () => {
-    process.chdir(originalCwd);
-    await cleanupTempDir(tempDir);
-  });
-
-  it('should create a new craftdesk.json file', async () => {
-    // Run init command
-    execSync(`node ${cliPath} init --name test-project --project-version 1.0.0 --type skill`, {
-      stdio: 'pipe'
-    });
-
-    const craftdeskPath = path.join(tempDir, 'craftdesk.json');
-    const exists = await fs.pathExists(craftdeskPath);
-    expect(exists).toBe(true);
-
-    const config = await readJsonFile(craftdeskPath);
-    expect(config.name).toBe('test-project');
-    expect(config.version).toBe('1.0.0');
-    expect(config.type).toBe('skill');
-  });
-
-  it('should include default registry configuration', async () => {
-    execSync(`node ${cliPath} init --name test --project-version 1.0.0 --type agent`, {
-      stdio: 'pipe'
-    });
-
-    const config = await readJsonFile(path.join(tempDir, 'craftdesk.json'));
-    expect(config.registries).toBeDefined();
-    expect(config.registries.default).toBeDefined();
-    expect(config.registries.default.url).toBe('https://craftdesk.ai');
-  });
-
-  it('should not overwrite existing craftdesk.json', async () => {
-    // Create existing file
-    await fs.writeJson(path.join(tempDir, 'craftdesk.json'), {
-      name: 'existing',
-      version: '2.0.0'
-    });
-
-    // Try to init again
-    try {
-      execSync(`node ${cliPath} init --name test --project-version 1.0.0 --type skill`, {
-        stdio: 'pipe'
-      });
-    } catch (error) {
-      // Expected to fail
-    }
-
-    const config = await readJsonFile(path.join(tempDir, 'craftdesk.json'));
-    expect(config.name).toBe('existing');
-    expect(config.version).toBe('2.0.0');
-  });
-});

package/tests/integration/list-command.test.ts

@@ -1,77 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import { execSync } from 'child_process';
-import { createTempDir, cleanupTempDir, writeJsonFile } from '../helpers/test-utils';
-import path from 'path';
-import fs from 'fs-extra';
-
-describe('craftdesk list command', () => {
-  let tempDir: string;
-  let originalCwd: string;
-  const cliPath = path.join(__dirname, '../../dist/index.js');
-
-  beforeEach(async () => {
-    tempDir = await createTempDir('list-test-');
-    originalCwd = process.cwd();
-    process.chdir(tempDir);
-  });
-
-  afterEach(async () => {
-    process.chdir(originalCwd);
-    await cleanupTempDir(tempDir);
-  });
-
-  it('should list installed crafts', async () => {
-    // Create craftdesk.json
-    await writeJsonFile(path.join(tempDir, 'craftdesk.json'), {
-      name: 'test-project',
-      version: '1.0.0',
-      dependencies: {
-        'ruby-on-rails': '^7.0.0',
-        'postgres-expert': '^1.0.0'
-      }
-    });
-
-    // Create mock installed crafts
-    const skillDir = path.join(tempDir, '.claude', 'skills', 'ruby-on-rails');
-    await fs.ensureDir(skillDir);
-    await writeJsonFile(path.join(skillDir, '.craftdesk-metadata.json'), {
-      name: 'ruby-on-rails',
-      version: '7.1.0',
-      type: 'skill',
-      installedAt: new Date().toISOString()
-    });
-
-    const agentDir = path.join(tempDir, '.claude', 'agents', 'postgres-expert');
-    await fs.ensureDir(agentDir);
-    await writeJsonFile(path.join(agentDir, '.craftdesk-metadata.json'), {
-      name: 'postgres-expert',
-      version: '1.2.0',
-      type: 'agent',
-      installedAt: new Date().toISOString()
-    });
-
-    const output = execSync(`node ${cliPath} list`, {
-      encoding: 'utf-8'
-    });
-
-    expect(output).toContain('ruby-on-rails');
-    expect(output).toContain('7.1.0');
-    expect(output).toContain('postgres-expert');
-    expect(output).toContain('1.2.0');
-  });
-
-  it('should handle no installed crafts', async () => {
-    // Create craftdesk.json first
-    await writeJsonFile(path.join(tempDir, 'craftdesk.json'), {
-      name: 'empty-project',
-      version: '1.0.0',
-      dependencies: {}
-    });
-
-    const output = execSync(`node ${cliPath} list`, {
-      encoding: 'utf-8'
-    });
-
-    expect(output).toContain('No crafts installed');
-  });
-});

package/tests/unit/config-manager.test.ts

@@ -1,228 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
-import { ConfigManager } from '../../src/services/config-manager';
-import { createTempDir, cleanupTempDir, writeJsonFile } from '../helpers/test-utils';
-import path from 'path';
-
-describe('ConfigManager', () => {
-  let configManager: ConfigManager;
-  let tempDir: string;
-  let originalCwd: string;
-
-  beforeEach(async () => {
-    configManager = new ConfigManager();
-    tempDir = await createTempDir('config-test-');
-    originalCwd = process.cwd();
-    process.chdir(tempDir);
-  });
-
-  afterEach(async () => {
-    process.chdir(originalCwd);
-    await cleanupTempDir(tempDir);
-  });
-
-  describe('getRegistryForCraft', () => {
-    it('should return default registry when no craftdesk.json exists', async () => {
-      const registry = await configManager.getRegistryForCraft('test-craft');
-      expect(registry).toBe('https://craftdesk.ai');
-    });
-
-    it('should return default registry from craftdesk.json', async () => {
-      await writeJsonFile(path.join(tempDir, 'craftdesk.json'), {
-        name: 'test',
-        version: '1.0.0',
-        registries: {
-          default: {
-            url: 'https://custom-registry.com'
-          }
-        }
-      });
-
-      const registry = await configManager.getRegistryForCraft('test-craft');
-      expect(registry).toBe('https://custom-registry.com');
-    });
-
-    it('should return scoped registry for scoped craft', async () => {
-      await writeJsonFile(path.join(tempDir, 'craftdesk.json'), {
-        name: 'test',
-        version: '1.0.0',
-        registries: {
-          default: {
-            url: 'https://craftdesk.ai'
-          },
-          'company-private': {
-            url: 'https://company.internal',
-            scope: '@company'
-          }
-        }
-      });
-
-      const registry = await configManager.getRegistryForCraft('@company/auth');
-      expect(registry).toBe('https://company.internal');
-    });
-
-    it('should fallback to default registry for unmatched scoped craft', async () => {
-      await writeJsonFile(path.join(tempDir, 'craftdesk.json'), {
-        name: 'test',
-        version: '1.0.0',
-        registries: {
-          default: {
-            url: 'https://craftdesk.ai'
-          },
-          'company-private': {
-            url: 'https://company.internal',
-            scope: '@company'
-          }
-        }
-      });
-
-      const registry = await configManager.getRegistryForCraft('@other/craft');
-      expect(registry).toBe('https://craftdesk.ai');
-    });
-
-    it('should use final fallback when no registries configured', async () => {
-      await writeJsonFile(path.join(tempDir, 'craftdesk.json'), {
-        name: 'test',
-        version: '1.0.0'
-      });
-
-      const registry = await configManager.getRegistryForCraft('test-craft');
-      expect(registry).toBe('https://craftdesk.ai');
-    });
-  });
-
-  describe('getAuthToken', () => {
-    it('should return null when no craftdesk.json exists', async () => {
-      const token = await configManager.getAuthToken('default');
-      expect(token).toBeNull();
-    });
-
-    it('should return null when registry not configured', async () => {
-      await writeJsonFile(path.join(tempDir, 'craftdesk.json'), {
-        name: 'test',
-        version: '1.0.0',
-        registries: {
-          default: {
-            url: 'https://craftdesk.ai'
-          }
-        }
-      });
-
-      const token = await configManager.getAuthToken('nonexistent');
-      expect(token).toBeNull();
-    });
-
-    it('should return token from environment variable', async () => {
-      await writeJsonFile(path.join(tempDir, 'craftdesk.json'), {
-        name: 'test',
-        version: '1.0.0',
-        registries: {
-          'company-private': {
-            url: 'https://company.internal',
-            scope: '@company'
-          }
-        }
-      });
-
-      process.env.CRAFTDESK_AUTH_COMPANY_PRIVATE = 'test-token-123';
-
-      const token = await configManager.getAuthToken('company-private');
-      expect(token).toBe('test-token-123');
-
-      delete process.env.CRAFTDESK_AUTH_COMPANY_PRIVATE;
-    });
-
-    it('should handle registry names with hyphens', async () => {
-      await writeJsonFile(path.join(tempDir, 'craftdesk.json'), {
-        name: 'test',
-        version: '1.0.0',
-        registries: {
-          'my-custom-registry': {
-            url: 'https://custom.com'
-          }
-        }
-      });
-
-      process.env.CRAFTDESK_AUTH_MY_CUSTOM_REGISTRY = 'custom-token';
-
-      const token = await configManager.getAuthToken('my-custom-registry');
-      expect(token).toBe('custom-token');
-
-      delete process.env.CRAFTDESK_AUTH_MY_CUSTOM_REGISTRY;
-    });
-
-    it('should return null when environment variable not set', async () => {
-      await writeJsonFile(path.join(tempDir, 'craftdesk.json'), {
-        name: 'test',
-        version: '1.0.0',
-        registries: {
-          private: {
-            url: 'https://private.com'
-          }
-        }
-      });
-
-      const token = await configManager.getAuthToken('private');
-      expect(token).toBeNull();
-    });
-  });
-
-  describe('getInstallPath', () => {
-    it('should return .claude directory', () => {
-      const installPath = configManager.getInstallPath();
-      expect(installPath).toBe('.claude');
-    });
-  });
-
-  describe('getCraftDeskJson', () => {
-    it('should return null when craftdesk.json does not exist', async () => {
-      const result = await configManager.getCraftDeskJson();
-      expect(result).toBeNull();
-    });
-
-    it('should return parsed craftdesk.json', async () => {
-      const craftDeskData = {
-        name: 'test-project',
-        version: '1.0.0',
-        type: 'skill',
-        description: 'Test project'
-      };
-
-      await writeJsonFile(path.join(tempDir, 'craftdesk.json'), craftDeskData);
-
-      const result = await configManager.getCraftDeskJson();
-      expect(result).toEqual(craftDeskData);
-    });
-
-    it('should cache craftdesk.json after first read', async () => {
-      const craftDeskData = {
-        name: 'test-project',
-        version: '1.0.0'
-      };
-
-      await writeJsonFile(path.join(tempDir, 'craftdesk.json'), craftDeskData);
-
-      // First call
-      const result1 = await configManager.getCraftDeskJson();
-
-      // Modify file on disk
-      await writeJsonFile(path.join(tempDir, 'craftdesk.json'), {
-        name: 'modified',
-        version: '2.0.0'
-      });
-
-      // Second call should return cached value
-      const result2 = await configManager.getCraftDeskJson();
-
-      expect(result1).toEqual(craftDeskData);
-      expect(result2).toEqual(craftDeskData);
-    });
-
-    it('should return null for malformed JSON', async () => {
-      const fs = await import('fs-extra');
-      await fs.writeFile(path.join(tempDir, 'craftdesk.json'), 'not valid json', 'utf-8');
-
-      const result = await configManager.getCraftDeskJson();
-      expect(result).toBeNull();
-    });
-  });
-});