sauron-cli 1.3.0 → 1.4.1

package/OpenSpec-main/test/core/shared/skill-generation.test.ts

@@ -1,301 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import {
-  getSkillTemplates,
-  getCommandTemplates,
-  getCommandContents,
-  generateSkillContent,
-} from '../../../src/core/shared/skill-generation.js';
-
-describe('skill-generation', () => {
-  describe('getSkillTemplates', () => {
-    it('should return all 11 skill templates', () => {
-      const templates = getSkillTemplates();
-      expect(templates).toHaveLength(11);
-    });
-
-    it('should have unique directory names', () => {
-      const templates = getSkillTemplates();
-      const dirNames = templates.map(t => t.dirName);
-      const uniqueDirNames = new Set(dirNames);
-      expect(uniqueDirNames.size).toBe(templates.length);
-    });
-
-    it('should include all expected skills', () => {
-      const templates = getSkillTemplates();
-      const dirNames = templates.map(t => t.dirName);
-
-      expect(dirNames).toContain('openspec-explore');
-      expect(dirNames).toContain('openspec-new-change');
-      expect(dirNames).toContain('openspec-continue-change');
-      expect(dirNames).toContain('openspec-apply-change');
-      expect(dirNames).toContain('openspec-ff-change');
-      expect(dirNames).toContain('openspec-sync-specs');
-      expect(dirNames).toContain('openspec-archive-change');
-      expect(dirNames).toContain('openspec-bulk-archive-change');
-      expect(dirNames).toContain('openspec-verify-change');
-      expect(dirNames).toContain('openspec-onboard');
-      expect(dirNames).toContain('openspec-propose');
-    });
-
-    it('should have valid template structure', () => {
-      const templates = getSkillTemplates();
-
-      for (const { template, dirName, workflowId } of templates) {
-        expect(template.name).toBeTruthy();
-        expect(template.description).toBeTruthy();
-        expect(template.instructions).toBeTruthy();
-        expect(dirName).toBeTruthy();
-        expect(workflowId).toBeTruthy();
-      }
-    });
-
-    it('should have unique workflow IDs', () => {
-      const templates = getSkillTemplates();
-      const ids = templates.map(t => t.workflowId);
-      const uniqueIds = new Set(ids);
-      expect(uniqueIds.size).toBe(templates.length);
-    });
-
-    it('should filter by workflow IDs when provided', () => {
-      const filtered = getSkillTemplates(['propose', 'explore', 'apply', 'archive']);
-      expect(filtered).toHaveLength(4);
-      const ids = filtered.map(t => t.workflowId);
-      expect(ids).toContain('propose');
-      expect(ids).toContain('explore');
-      expect(ids).toContain('apply');
-      expect(ids).toContain('archive');
-      expect(ids).not.toContain('new');
-      expect(ids).not.toContain('ff');
-    });
-
-    it('should return all templates when filter is undefined', () => {
-      const all = getSkillTemplates();
-      const noFilter = getSkillTemplates(undefined);
-      expect(noFilter).toHaveLength(all.length);
-    });
-
-    it('should return empty array when filter matches nothing', () => {
-      const filtered = getSkillTemplates(['nonexistent']);
-      expect(filtered).toHaveLength(0);
-    });
-
-    it('should return single template when filter has one workflow', () => {
-      const filtered = getSkillTemplates(['propose']);
-      expect(filtered).toHaveLength(1);
-      expect(filtered[0].workflowId).toBe('propose');
-      expect(filtered[0].dirName).toBe('openspec-propose');
-    });
-  });
-
-  describe('getCommandTemplates', () => {
-    it('should return all 11 command templates', () => {
-      const templates = getCommandTemplates();
-      expect(templates).toHaveLength(11);
-    });
-
-    it('should have unique IDs', () => {
-      const templates = getCommandTemplates();
-      const ids = templates.map(t => t.id);
-      const uniqueIds = new Set(ids);
-      expect(uniqueIds.size).toBe(templates.length);
-    });
-
-    it('should include all expected commands', () => {
-      const templates = getCommandTemplates();
-      const ids = templates.map(t => t.id);
-
-      expect(ids).toContain('explore');
-      expect(ids).toContain('new');
-      expect(ids).toContain('continue');
-      expect(ids).toContain('apply');
-      expect(ids).toContain('ff');
-      expect(ids).toContain('sync');
-      expect(ids).toContain('archive');
-      expect(ids).toContain('bulk-archive');
-      expect(ids).toContain('verify');
-      expect(ids).toContain('onboard');
-      expect(ids).toContain('propose');
-    });
-
-    it('should filter by workflow IDs when provided', () => {
-      const filtered = getCommandTemplates(['propose', 'explore', 'apply', 'archive']);
-      expect(filtered).toHaveLength(4);
-      const ids = filtered.map(t => t.id);
-      expect(ids).toContain('propose');
-      expect(ids).toContain('explore');
-      expect(ids).toContain('apply');
-      expect(ids).toContain('archive');
-      expect(ids).not.toContain('new');
-      expect(ids).not.toContain('ff');
-    });
-
-    it('should return all templates when filter is undefined', () => {
-      const all = getCommandTemplates();
-      const noFilter = getCommandTemplates(undefined);
-      expect(noFilter).toHaveLength(all.length);
-    });
-
-    it('should return empty array when filter matches nothing', () => {
-      const filtered = getCommandTemplates(['nonexistent']);
-      expect(filtered).toHaveLength(0);
-    });
-  });
-
-  describe('getCommandContents', () => {
-    it('should return all 11 command contents', () => {
-      const contents = getCommandContents();
-      expect(contents).toHaveLength(11);
-    });
-
-    it('should have valid content structure', () => {
-      const contents = getCommandContents();
-
-      for (const content of contents) {
-        expect(content.id).toBeTruthy();
-        expect(content.name).toBeTruthy();
-        expect(content.description).toBeTruthy();
-        expect(content.body).toBeTruthy();
-      }
-    });
-
-    it('should have matching IDs with command templates', () => {
-      const templates = getCommandTemplates();
-      const contents = getCommandContents();
-
-      const templateIds = templates.map(t => t.id).sort();
-      const contentIds = contents.map(c => c.id).sort();
-
-      expect(contentIds).toEqual(templateIds);
-    });
-
-    it('should filter by workflow IDs when provided', () => {
-      const filtered = getCommandContents(['propose', 'explore']);
-      expect(filtered).toHaveLength(2);
-      const ids = filtered.map(c => c.id);
-      expect(ids).toContain('propose');
-      expect(ids).toContain('explore');
-      expect(ids).not.toContain('new');
-    });
-
-    it('should return all contents when filter is undefined', () => {
-      const all = getCommandContents();
-      const noFilter = getCommandContents(undefined);
-      expect(noFilter).toHaveLength(all.length);
-    });
-  });
-
-  describe('generateSkillContent', () => {
-    it('should generate valid YAML frontmatter', () => {
-      const template = {
-        name: 'test-skill',
-        description: 'Test description',
-        instructions: 'Test instructions',
-        license: 'MIT',
-        compatibility: 'Test compatibility',
-        metadata: {
-          author: 'test-author',
-          version: '2.0',
-        },
-      };
-
-      const content = generateSkillContent(template, '0.23.0');
-
-      expect(content).toMatch(/^---\n/);
-      expect(content).toContain('name: test-skill');
-      expect(content).toContain('description: Test description');
-      expect(content).toContain('license: MIT');
-      expect(content).toContain('compatibility: Test compatibility');
-      expect(content).toContain('author: test-author');
-      expect(content).toContain('version: "2.0"');
-      expect(content).toContain('generatedBy: "0.23.0"');
-      expect(content).toContain('Test instructions');
-    });
-
-    it('should use default values for optional fields', () => {
-      const template = {
-        name: 'minimal-skill',
-        description: 'Minimal description',
-        instructions: 'Minimal instructions',
-      };
-
-      const content = generateSkillContent(template, '0.24.0');
-
-      expect(content).toContain('license: MIT');
-      expect(content).toContain('compatibility: Requires openspec CLI.');
-      expect(content).toContain('author: openspec');
-      expect(content).toContain('version: "1.0"');
-      expect(content).toContain('generatedBy: "0.24.0"');
-    });
-
-    it('should embed the provided version in generatedBy field', () => {
-      const template = {
-        name: 'version-test',
-        description: 'Test version embedding',
-        instructions: 'Instructions',
-      };
-
-      const content1 = generateSkillContent(template, '0.23.0');
-      expect(content1).toContain('generatedBy: "0.23.0"');
-
-      const content2 = generateSkillContent(template, '1.0.0');
-      expect(content2).toContain('generatedBy: "1.0.0"');
-
-      const content3 = generateSkillContent(template, '0.24.0-beta.1');
-      expect(content3).toContain('generatedBy: "0.24.0-beta.1"');
-    });
-
-    it('should end frontmatter with separator and blank line', () => {
-      const template = {
-        name: 'test',
-        description: 'Test',
-        instructions: 'Body content',
-      };
-
-      const content = generateSkillContent(template, '0.23.0');
-
-      expect(content).toMatch(/---\n\nBody content\n$/);
-    });
-
-    it('should apply transformInstructions callback when provided', () => {
-      const template = {
-        name: 'transform-test',
-        description: 'Test transform callback',
-        instructions: 'Use /opsx:new to start and /opsx:apply to implement.',
-      };
-
-      const transformer = (text: string) => text.replace(/\/opsx:/g, '/opsx-');
-      const content = generateSkillContent(template, '0.23.0', transformer);
-
-      expect(content).toContain('/opsx-new');
-      expect(content).toContain('/opsx-apply');
-      expect(content).not.toContain('/opsx:new');
-      expect(content).not.toContain('/opsx:apply');
-    });
-
-    it('should not transform instructions when callback is undefined', () => {
-      const template = {
-        name: 'no-transform-test',
-        description: 'Test without transform',
-        instructions: 'Use /opsx:new to start.',
-      };
-
-      const content = generateSkillContent(template, '0.23.0', undefined);
-
-      expect(content).toContain('/opsx:new');
-    });
-
-    it('should support custom transformInstructions logic', () => {
-      const template = {
-        name: 'custom-transform',
-        description: 'Test custom transform',
-        instructions: 'Some PLACEHOLDER text here.',
-      };
-
-      const customTransformer = (text: string) => text.replace('PLACEHOLDER', 'REPLACED');
-      const content = generateSkillContent(template, '0.23.0', customTransformer);
-
-      expect(content).toContain('Some REPLACED text here.');
-      expect(content).not.toContain('PLACEHOLDER');
-    });
-  });
-});

package/OpenSpec-main/test/core/shared/tool-detection.test.ts

@@ -1,333 +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 { randomUUID } from 'crypto';
-import {
-  SKILL_NAMES,
-  getToolsWithSkillsDir,
-  getToolSkillStatus,
-  getToolStates,
-  extractGeneratedByVersion,
-  getToolVersionStatus,
-  getConfiguredTools,
-  getAllToolVersionStatus,
-} from '../../../src/core/shared/tool-detection.js';
-
-describe('tool-detection', () => {
-  let testDir: string;
-
-  beforeEach(async () => {
-    testDir = path.join(os.tmpdir(), `openspec-test-${randomUUID()}`);
-    await fs.mkdir(testDir, { recursive: true });
-  });
-
-  afterEach(async () => {
-    await fs.rm(testDir, { recursive: true, force: true });
-  });
-
-  describe('SKILL_NAMES', () => {
-    it('should contain all skill names matching COMMAND_IDS', () => {
-      expect(SKILL_NAMES).toHaveLength(11);
-      expect(SKILL_NAMES).toContain('openspec-explore');
-      expect(SKILL_NAMES).toContain('openspec-new-change');
-      expect(SKILL_NAMES).toContain('openspec-continue-change');
-      expect(SKILL_NAMES).toContain('openspec-apply-change');
-      expect(SKILL_NAMES).toContain('openspec-ff-change');
-      expect(SKILL_NAMES).toContain('openspec-sync-specs');
-      expect(SKILL_NAMES).toContain('openspec-archive-change');
-      expect(SKILL_NAMES).toContain('openspec-bulk-archive-change');
-      expect(SKILL_NAMES).toContain('openspec-verify-change');
-      expect(SKILL_NAMES).toContain('openspec-onboard');
-      expect(SKILL_NAMES).toContain('openspec-propose');
-    });
-  });
-
-  describe('getToolsWithSkillsDir', () => {
-    it('should return tools that have skillsDir configured', () => {
-      const tools = getToolsWithSkillsDir();
-      expect(tools).toContain('claude');
-      expect(tools).toContain('cursor');
-      expect(tools).toContain('windsurf');
-      expect(tools.length).toBeGreaterThan(0);
-    });
-  });
-
-  describe('getToolSkillStatus', () => {
-    it('should return not configured for unknown tool', () => {
-      const status = getToolSkillStatus(testDir, 'unknown-tool');
-      expect(status.configured).toBe(false);
-      expect(status.fullyConfigured).toBe(false);
-      expect(status.skillCount).toBe(0);
-    });
-
-    it('should return not configured when no skills exist', () => {
-      const status = getToolSkillStatus(testDir, 'claude');
-      expect(status.configured).toBe(false);
-      expect(status.fullyConfigured).toBe(false);
-      expect(status.skillCount).toBe(0);
-    });
-
-    it('should detect when one skill exists', async () => {
-      const skillDir = path.join(testDir, '.claude', 'skills', 'openspec-explore');
-      await fs.mkdir(skillDir, { recursive: true });
-      await fs.writeFile(path.join(skillDir, 'SKILL.md'), 'test content');
-
-      const status = getToolSkillStatus(testDir, 'claude');
-      expect(status.configured).toBe(true);
-      expect(status.fullyConfigured).toBe(false);
-      expect(status.skillCount).toBe(1);
-    });
-
-    it('should detect when all skills exist', async () => {
-      for (const skillName of SKILL_NAMES) {
-        const skillDir = path.join(testDir, '.claude', 'skills', skillName);
-        await fs.mkdir(skillDir, { recursive: true });
-        await fs.writeFile(path.join(skillDir, 'SKILL.md'), 'test content');
-      }
-
-      const status = getToolSkillStatus(testDir, 'claude');
-      expect(status.configured).toBe(true);
-      expect(status.fullyConfigured).toBe(true);
-      expect(status.skillCount).toBe(SKILL_NAMES.length);
-    });
-  });
-
-  describe('getToolStates', () => {
-    it('should return status for all tools with skillsDir', () => {
-      const states = getToolStates(testDir);
-      expect(states.has('claude')).toBe(true);
-      expect(states.has('cursor')).toBe(true);
-
-      const claudeStatus = states.get('claude');
-      expect(claudeStatus?.configured).toBe(false);
-    });
-
-    it('should detect configured tools', async () => {
-      const skillDir = path.join(testDir, '.claude', 'skills', 'openspec-explore');
-      await fs.mkdir(skillDir, { recursive: true });
-      await fs.writeFile(path.join(skillDir, 'SKILL.md'), 'test content');
-
-      const states = getToolStates(testDir);
-      expect(states.get('claude')?.configured).toBe(true);
-      expect(states.get('cursor')?.configured).toBe(false);
-    });
-  });
-
-  describe('extractGeneratedByVersion', () => {
-    it('should return null for non-existent file', () => {
-      const version = extractGeneratedByVersion(path.join(testDir, 'missing.md'));
-      expect(version).toBeNull();
-    });
-
-    it('should return null when generatedBy is not present', async () => {
-      const filePath = path.join(testDir, 'skill.md');
-      await fs.writeFile(filePath, `---
-name: openspec-explore
-metadata:
-  author: openspec
-  version: "1.0"
----
-
-Content here
-`);
-
-      const version = extractGeneratedByVersion(filePath);
-      expect(version).toBeNull();
-    });
-
-    it('should extract generatedBy version with double quotes', async () => {
-      const filePath = path.join(testDir, 'skill.md');
-      await fs.writeFile(filePath, `---
-name: openspec-explore
-metadata:
-  author: openspec
-  version: "1.0"
-  generatedBy: "0.23.0"
----
-
-Content here
-`);
-
-      const version = extractGeneratedByVersion(filePath);
-      expect(version).toBe('0.23.0');
-    });
-
-    it('should extract generatedBy version with single quotes', async () => {
-      const filePath = path.join(testDir, 'skill.md');
-      await fs.writeFile(filePath, `---
-name: openspec-explore
-metadata:
-  generatedBy: '0.24.0'
----
-
-Content here
-`);
-
-      const version = extractGeneratedByVersion(filePath);
-      expect(version).toBe('0.24.0');
-    });
-
-    it('should extract generatedBy version without quotes', async () => {
-      const filePath = path.join(testDir, 'skill.md');
-      await fs.writeFile(filePath, `---
-name: openspec-explore
-metadata:
-  generatedBy: 0.25.0
----
-
-Content here
-`);
-
-      const version = extractGeneratedByVersion(filePath);
-      expect(version).toBe('0.25.0');
-    });
-  });
-
-  describe('getToolVersionStatus', () => {
-    it('should return not configured for unknown tool', () => {
-      const status = getToolVersionStatus(testDir, 'unknown-tool', '0.23.0');
-      expect(status.configured).toBe(false);
-      expect(status.generatedByVersion).toBeNull();
-      expect(status.needsUpdate).toBe(false);
-    });
-
-    it('should return not configured when no skills exist', () => {
-      const status = getToolVersionStatus(testDir, 'claude', '0.23.0');
-      expect(status.configured).toBe(false);
-      expect(status.generatedByVersion).toBeNull();
-      expect(status.needsUpdate).toBe(false);
-    });
-
-    it('should detect needsUpdate when generatedBy is missing', async () => {
-      const skillDir = path.join(testDir, '.claude', 'skills', 'openspec-explore');
-      await fs.mkdir(skillDir, { recursive: true });
-      await fs.writeFile(path.join(skillDir, 'SKILL.md'), `---
-name: openspec-explore
-metadata:
-  author: openspec
-  version: "1.0"
----
-
-Content here
-`);
-
-      const status = getToolVersionStatus(testDir, 'claude', '0.23.0');
-      expect(status.configured).toBe(true);
-      expect(status.generatedByVersion).toBeNull();
-      expect(status.needsUpdate).toBe(true);
-    });
-
-    it('should detect needsUpdate when version differs', async () => {
-      const skillDir = path.join(testDir, '.claude', 'skills', 'openspec-explore');
-      await fs.mkdir(skillDir, { recursive: true });
-      await fs.writeFile(path.join(skillDir, 'SKILL.md'), `---
-name: openspec-explore
-metadata:
-  author: openspec
-  version: "1.0"
-  generatedBy: "0.22.0"
----
-
-Content here
-`);
-
-      const status = getToolVersionStatus(testDir, 'claude', '0.23.0');
-      expect(status.configured).toBe(true);
-      expect(status.generatedByVersion).toBe('0.22.0');
-      expect(status.needsUpdate).toBe(true);
-    });
-
-    it('should not need update when version matches', async () => {
-      const skillDir = path.join(testDir, '.claude', 'skills', 'openspec-explore');
-      await fs.mkdir(skillDir, { recursive: true });
-      await fs.writeFile(path.join(skillDir, 'SKILL.md'), `---
-name: openspec-explore
-metadata:
-  author: openspec
-  version: "1.0"
-  generatedBy: "0.23.0"
----
-
-Content here
-`);
-
-      const status = getToolVersionStatus(testDir, 'claude', '0.23.0');
-      expect(status.configured).toBe(true);
-      expect(status.generatedByVersion).toBe('0.23.0');
-      expect(status.needsUpdate).toBe(false);
-    });
-
-    it('should include tool name in status', async () => {
-      const skillDir = path.join(testDir, '.claude', 'skills', 'openspec-explore');
-      await fs.mkdir(skillDir, { recursive: true });
-      await fs.writeFile(path.join(skillDir, 'SKILL.md'), 'content');
-
-      const status = getToolVersionStatus(testDir, 'claude', '0.23.0');
-      expect(status.toolId).toBe('claude');
-      expect(status.toolName).toBe('Claude Code');
-    });
-  });
-
-  describe('getConfiguredTools', () => {
-    it('should return empty array when no tools are configured', () => {
-      const tools = getConfiguredTools(testDir);
-      expect(tools).toEqual([]);
-    });
-
-    it('should return configured tools', async () => {
-      // Setup Claude
-      const claudeSkillDir = path.join(testDir, '.claude', 'skills', 'openspec-explore');
-      await fs.mkdir(claudeSkillDir, { recursive: true });
-      await fs.writeFile(path.join(claudeSkillDir, 'SKILL.md'), 'content');
-
-      // Setup Cursor
-      const cursorSkillDir = path.join(testDir, '.cursor', 'skills', 'openspec-explore');
-      await fs.mkdir(cursorSkillDir, { recursive: true });
-      await fs.writeFile(path.join(cursorSkillDir, 'SKILL.md'), 'content');
-
-      const tools = getConfiguredTools(testDir);
-      expect(tools).toContain('claude');
-      expect(tools).toContain('cursor');
-      expect(tools).toHaveLength(2);
-    });
-  });
-
-  describe('getAllToolVersionStatus', () => {
-    it('should return empty array when no tools are configured', () => {
-      const statuses = getAllToolVersionStatus(testDir, '0.23.0');
-      expect(statuses).toEqual([]);
-    });
-
-    it('should return version status for all configured tools', async () => {
-      // Setup Claude with old version
-      const claudeSkillDir = path.join(testDir, '.claude', 'skills', 'openspec-explore');
-      await fs.mkdir(claudeSkillDir, { recursive: true });
-      await fs.writeFile(path.join(claudeSkillDir, 'SKILL.md'), `---
-metadata:
-  generatedBy: "0.22.0"
----
-`);
-
-      // Setup Cursor with current version
-      const cursorSkillDir = path.join(testDir, '.cursor', 'skills', 'openspec-explore');
-      await fs.mkdir(cursorSkillDir, { recursive: true });
-      await fs.writeFile(path.join(cursorSkillDir, 'SKILL.md'), `---
-metadata:
-  generatedBy: "0.23.0"
----
-`);
-
-      const statuses = getAllToolVersionStatus(testDir, '0.23.0');
-      expect(statuses).toHaveLength(2);
-
-      const claudeStatus = statuses.find(s => s.toolId === 'claude');
-      expect(claudeStatus?.generatedByVersion).toBe('0.22.0');
-      expect(claudeStatus?.needsUpdate).toBe(true);
-
-      const cursorStatus = statuses.find(s => s.toolId === 'cursor');
-      expect(cursorStatus?.generatedByVersion).toBe('0.23.0');
-      expect(cursorStatus?.needsUpdate).toBe(false);
-    });
-  });
-});