sauron-cli 1.3.0 → 1.4.0

package/OpenSpec-main/test/core/list.test.ts

@@ -1,165 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import { promises as fs } from 'fs';
-import path from 'path';
-import os from 'os';
-import { ListCommand } from '../../src/core/list.js';
-
-describe('ListCommand', () => {
-  let tempDir: string;
-  let originalLog: typeof console.log;
-  let logOutput: string[] = [];
-
-  beforeEach(async () => {
-    // Create temp directory
-    tempDir = path.join(os.tmpdir(), `openspec-list-test-${Date.now()}`);
-    await fs.mkdir(tempDir, { recursive: true });
-
-    // Mock console.log to capture output
-    originalLog = console.log;
-    console.log = (...args: any[]) => {
-      logOutput.push(args.join(' '));
-    };
-    logOutput = [];
-  });
-
-  afterEach(async () => {
-    // Restore console.log
-    console.log = originalLog;
-
-    // Clean up temp directory
-    await fs.rm(tempDir, { recursive: true, force: true });
-  });
-
-  describe('execute', () => {
-    it('should handle missing openspec/changes directory', async () => {
-      const listCommand = new ListCommand();
-      
-      await expect(listCommand.execute(tempDir, 'changes')).rejects.toThrow(
-        "No OpenSpec changes directory found. Run 'openspec init' first."
-      );
-    });
-
-    it('should handle empty changes directory', async () => {
-      const changesDir = path.join(tempDir, 'openspec', 'changes');
-      await fs.mkdir(changesDir, { recursive: true });
-
-      const listCommand = new ListCommand();
-      await listCommand.execute(tempDir, 'changes');
-
-      expect(logOutput).toEqual(['No active changes found.']);
-    });
-
-    it('should exclude archive directory', async () => {
-      const changesDir = path.join(tempDir, 'openspec', 'changes');
-      await fs.mkdir(path.join(changesDir, 'archive'), { recursive: true });
-      await fs.mkdir(path.join(changesDir, 'my-change'), { recursive: true });
-      
-      // Create tasks.md with some tasks
-      await fs.writeFile(
-        path.join(changesDir, 'my-change', 'tasks.md'),
-        '- [x] Task 1\n- [ ] Task 2\n'
-      );
-
-      const listCommand = new ListCommand();
-      await listCommand.execute(tempDir, 'changes');
-
-      expect(logOutput).toContain('Changes:');
-      expect(logOutput.some(line => line.includes('my-change'))).toBe(true);
-      expect(logOutput.some(line => line.includes('archive'))).toBe(false);
-    });
-
-    it('should count tasks correctly', async () => {
-      const changesDir = path.join(tempDir, 'openspec', 'changes');
-      await fs.mkdir(path.join(changesDir, 'test-change'), { recursive: true });
-      
-      await fs.writeFile(
-        path.join(changesDir, 'test-change', 'tasks.md'),
-        `# Tasks
-- [x] Completed task 1
-- [x] Completed task 2
-- [ ] Incomplete task 1
-- [ ] Incomplete task 2
-- [ ] Incomplete task 3
-Regular text that should be ignored
-`
-      );
-
-      const listCommand = new ListCommand();
-      await listCommand.execute(tempDir, 'changes');
-
-      expect(logOutput.some(line => line.includes('2/5 tasks'))).toBe(true);
-    });
-
-    it('should show complete status for fully completed changes', async () => {
-      const changesDir = path.join(tempDir, 'openspec', 'changes');
-      await fs.mkdir(path.join(changesDir, 'completed-change'), { recursive: true });
-      
-      await fs.writeFile(
-        path.join(changesDir, 'completed-change', 'tasks.md'),
-        '- [x] Task 1\n- [x] Task 2\n- [x] Task 3\n'
-      );
-
-      const listCommand = new ListCommand();
-      await listCommand.execute(tempDir, 'changes');
-
-      expect(logOutput.some(line => line.includes('✓ Complete'))).toBe(true);
-    });
-
-    it('should handle changes without tasks.md', async () => {
-      const changesDir = path.join(tempDir, 'openspec', 'changes');
-      await fs.mkdir(path.join(changesDir, 'no-tasks'), { recursive: true });
-
-      const listCommand = new ListCommand();
-      await listCommand.execute(tempDir, 'changes');
-
-      expect(logOutput.some(line => line.includes('no-tasks') && line.includes('No tasks'))).toBe(true);
-    });
-
-    it('should sort changes alphabetically when sort=name', async () => {
-      const changesDir = path.join(tempDir, 'openspec', 'changes');
-      await fs.mkdir(path.join(changesDir, 'zebra'), { recursive: true });
-      await fs.mkdir(path.join(changesDir, 'alpha'), { recursive: true });
-      await fs.mkdir(path.join(changesDir, 'middle'), { recursive: true });
-
-      const listCommand = new ListCommand();
-      await listCommand.execute(tempDir, 'changes', { sort: 'name' });
-
-      const changeLines = logOutput.filter(line =>
-        line.includes('alpha') || line.includes('middle') || line.includes('zebra')
-      );
-
-      expect(changeLines[0]).toContain('alpha');
-      expect(changeLines[1]).toContain('middle');
-      expect(changeLines[2]).toContain('zebra');
-    });
-
-    it('should handle multiple changes with various states', async () => {
-      const changesDir = path.join(tempDir, 'openspec', 'changes');
-      
-      // Complete change
-      await fs.mkdir(path.join(changesDir, 'completed'), { recursive: true });
-      await fs.writeFile(
-        path.join(changesDir, 'completed', 'tasks.md'),
-        '- [x] Task 1\n- [x] Task 2\n'
-      );
-
-      // Partial change
-      await fs.mkdir(path.join(changesDir, 'partial'), { recursive: true });
-      await fs.writeFile(
-        path.join(changesDir, 'partial', 'tasks.md'),
-        '- [x] Done\n- [ ] Not done\n- [ ] Also not done\n'
-      );
-
-      // No tasks
-      await fs.mkdir(path.join(changesDir, 'no-tasks'), { recursive: true });
-
-      const listCommand = new ListCommand();
-      await listCommand.execute(tempDir);
-
-      expect(logOutput).toContain('Changes:');
-      expect(logOutput.some(line => line.includes('completed') && line.includes('✓ Complete'))).toBe(true);
-      expect(logOutput.some(line => line.includes('partial') && line.includes('1/3 tasks'))).toBe(true);
-      expect(logOutput.some(line => line.includes('no-tasks') && line.includes('No tasks'))).toBe(true);
-    });
-  });
-});

package/OpenSpec-main/test/core/migration.test.ts

@@ -1,150 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import path from 'path';
-import os from 'os';
-import { randomUUID } from 'crypto';
-import fs from 'node:fs';
-import { promises as fsp } from 'node:fs';
-import { AI_TOOLS, type AIToolOption } from '../../src/core/config.js';
-import { CommandAdapterRegistry } from '../../src/core/command-generation/index.js';
-import { saveGlobalConfig, getGlobalConfigPath } from '../../src/core/global-config.js';
-import { migrateIfNeeded, scanInstalledWorkflows } from '../../src/core/migration.js';
-
-const CLAUDE_TOOL = AI_TOOLS.find((tool) => tool.value === 'claude') as AIToolOption | undefined;
-
-function ensureClaudeTool(): AIToolOption {
-  if (!CLAUDE_TOOL) {
-    throw new Error('Claude tool definition not found');
-  }
-  return CLAUDE_TOOL;
-}
-
-async function writeSkill(projectPath: string, dirName: string): Promise<void> {
-  const skillFile = path.join(projectPath, '.claude', 'skills', dirName, 'SKILL.md');
-  await fsp.mkdir(path.dirname(skillFile), { recursive: true });
-  await fsp.writeFile(skillFile, 'name: test\n', 'utf-8');
-}
-
-async function writeManagedCommand(projectPath: string, workflowId: string): Promise<void> {
-  const adapter = CommandAdapterRegistry.get('claude');
-  if (!adapter) {
-    throw new Error('Claude adapter not found');
-  }
-  const commandPath = adapter.getFilePath(workflowId);
-  const fullPath = path.isAbsolute(commandPath)
-    ? commandPath
-    : path.join(projectPath, commandPath);
-  await fsp.mkdir(path.dirname(fullPath), { recursive: true });
-  await fsp.writeFile(fullPath, '# command\n', 'utf-8');
-}
-
-function readRawConfig(): Record<string, unknown> {
-  return JSON.parse(fs.readFileSync(getGlobalConfigPath(), 'utf-8')) as Record<string, unknown>;
-}
-
-describe('migration', () => {
-  let projectDir: string;
-  let configHome: string;
-  let originalEnv: NodeJS.ProcessEnv;
-
-  beforeEach(async () => {
-    projectDir = path.join(os.tmpdir(), `openspec-migration-project-${randomUUID()}`);
-    configHome = path.join(os.tmpdir(), `openspec-migration-config-${randomUUID()}`);
-    await fsp.mkdir(projectDir, { recursive: true });
-    await fsp.mkdir(configHome, { recursive: true });
-    originalEnv = { ...process.env };
-    process.env.XDG_CONFIG_HOME = configHome;
-  });
-
-  afterEach(async () => {
-    process.env = originalEnv;
-    await fsp.rm(projectDir, { recursive: true, force: true });
-    await fsp.rm(configHome, { recursive: true, force: true });
-  });
-
-  it('migrates to custom skills delivery when only managed skills are detected', async () => {
-    await writeSkill(projectDir, 'openspec-explore');
-    await writeSkill(projectDir, 'openspec-apply-change');
-
-    migrateIfNeeded(projectDir, [ensureClaudeTool()]);
-
-    const config = readRawConfig();
-    expect(config.profile).toBe('custom');
-    expect(config.delivery).toBe('skills');
-    expect(config.workflows).toEqual(['explore', 'apply']);
-  });
-
-  it('migrates to custom commands delivery when only managed commands are detected', async () => {
-    await writeManagedCommand(projectDir, 'explore');
-    await writeManagedCommand(projectDir, 'archive');
-
-    migrateIfNeeded(projectDir, [ensureClaudeTool()]);
-
-    const config = readRawConfig();
-    expect(config.profile).toBe('custom');
-    expect(config.delivery).toBe('commands');
-    expect(config.workflows).toEqual(['explore', 'archive']);
-  });
-
-  it('migrates to custom both delivery when managed skills and commands are detected', async () => {
-    await writeSkill(projectDir, 'openspec-explore');
-    await writeManagedCommand(projectDir, 'apply');
-
-    migrateIfNeeded(projectDir, [ensureClaudeTool()]);
-
-    const config = readRawConfig();
-    expect(config.profile).toBe('custom');
-    expect(config.delivery).toBe('both');
-    expect(config.workflows).toEqual(['explore', 'apply']);
-  });
-
-  it('does not migrate when profile is already explicitly configured', async () => {
-    saveGlobalConfig({
-      featureFlags: {},
-      profile: 'core',
-      delivery: 'both',
-    });
-    await writeSkill(projectDir, 'openspec-explore');
-
-    migrateIfNeeded(projectDir, [ensureClaudeTool()]);
-
-    const config = readRawConfig();
-    expect(config.profile).toBe('core');
-    expect(config.delivery).toBe('both');
-    expect(config.workflows).toBeUndefined();
-  });
-
-  it('preserves explicit delivery value during migration', async () => {
-    // Raw config has explicit delivery but no profile yet.
-    saveGlobalConfig({
-      featureFlags: {},
-      delivery: 'both',
-    });
-    await writeSkill(projectDir, 'openspec-explore');
-
-    migrateIfNeeded(projectDir, [ensureClaudeTool()]);
-
-    const config = readRawConfig();
-    expect(config.profile).toBe('custom');
-    expect(config.delivery).toBe('both');
-    expect(config.workflows).toEqual(['explore']);
-  });
-
-  it('does not migrate when no managed workflow artifacts are detected', async () => {
-    migrateIfNeeded(projectDir, [ensureClaudeTool()]);
-
-    expect(fs.existsSync(getGlobalConfigPath())).toBe(false);
-  });
-
-  it('ignores unknown custom skill and command files when scanning workflows', async () => {
-    await writeSkill(projectDir, 'my-custom-skill');
-    const customCommandPath = path.join(projectDir, '.claude', 'commands', 'opsx', 'my-custom.md');
-    await fsp.mkdir(path.dirname(customCommandPath), { recursive: true });
-    await fsp.writeFile(customCommandPath, '# custom\n', 'utf-8');
-
-    const workflows = scanInstalledWorkflows(projectDir, [ensureClaudeTool()]);
-    expect(workflows).toEqual([]);
-
-    migrateIfNeeded(projectDir, [ensureClaudeTool()]);
-    expect(fs.existsSync(getGlobalConfigPath())).toBe(false);
-  });
-});

package/OpenSpec-main/test/core/parsers/change-parser.test.ts

@@ -1,52 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import path from 'path';
-import { promises as fs } from 'fs';
-import os from 'os';
-import { ChangeParser } from '../../../src/core/parsers/change-parser.js';
-
-async function withTempDir(run: (dir: string) => Promise<void>) {
-  const dir = await fs.mkdtemp(path.join(os.tmpdir(), 'openspec-change-parser-'));
-  try {
-    await run(dir);
-  } finally {
-    // Best-effort cleanup
-    try { await fs.rm(dir, { recursive: true, force: true }); } catch {}
-  }
-}
-
-describe('ChangeParser', () => {
-  it('parses simple What Changes bullet list', async () => {
-    const content = `# Test Change\n\n## Why\nWe need it because reasons that are sufficiently long.\n\n## What Changes\n- **spec-a:** Add a new requirement to A\n- **spec-b:** Rename requirement X to Y\n- **spec-c:** Remove obsolete requirement`;
-
-    const parser = new ChangeParser(content, process.cwd());
-    const change = await parser.parseChangeWithDeltas('test-change');
-
-    expect(change.name).toBe('test-change');
-    expect(change.deltas.length).toBe(3);
-    expect(change.deltas[0].spec).toBe('spec-a');
-    expect(['ADDED', 'MODIFIED', 'REMOVED', 'RENAMED']).toContain(change.deltas[1].operation);
-  });
-
-  it('prefers delta-format specs over simple bullets when both exist', async () => {
-    await withTempDir(async (dir) => {
-      const changeDir = dir;
-      const specsDir = path.join(changeDir, 'specs', 'foo');
-      await fs.mkdir(specsDir, { recursive: true });
-
-      const content = `# Test Change\n\n## Why\nWe need it because reasons that are sufficiently long.\n\n## What Changes\n- **foo:** Add something via bullets (should be overridden)`;
-      const deltaSpec = `# Delta for Foo\n\n## ADDED Requirements\n\n### Requirement: New thing\n\n#### Scenario: basic\nGiven X\nWhen Y\nThen Z`;
-
-      await fs.writeFile(path.join(specsDir, 'spec.md'), deltaSpec, 'utf8');
-
-      const parser = new ChangeParser(content, changeDir);
-      const change = await parser.parseChangeWithDeltas('test-change');
-
-      expect(change.deltas.length).toBeGreaterThan(0);
-      // Since delta spec exists, the description should reflect delta-derived entries
-      expect(change.deltas[0].spec).toBe('foo');
-      expect(change.deltas[0].description).toContain('Add requirement:');
-      expect(change.deltas[0].operation).toBe('ADDED');
-      expect(change.deltas[0].requirement).toBeDefined();
-    });
-  });
-});