sauron-cli 1.3.0 → 1.4.0

package/OpenSpec-main/test/commands/spec.test.ts

@@ -1,324 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import { promises as fs } from 'fs';
-import path from 'path';
-import { execSync } from 'child_process';
-
-describe('spec command', () => {
-  const projectRoot = process.cwd();
-  const testDir = path.join(projectRoot, 'test-spec-command-tmp');
-  const specsDir = path.join(testDir, 'openspec', 'specs');
-  const openspecBin = path.join(projectRoot, 'bin', 'openspec.js');
-  
-  
-  beforeEach(async () => {
-    await fs.mkdir(specsDir, { recursive: true });
-    
-    // Create test spec files
-    const testSpec = `## Purpose
-This is a test specification for the authentication system.
-
-## Requirements
-
-### Requirement: User Authentication
-The system SHALL provide secure user authentication
-
-#### Scenario: Successful login
-- **GIVEN** a user with valid credentials
-- **WHEN** they submit the login form  
-- **THEN** they are authenticated
-
-### Requirement: Password Reset
-The system SHALL allow users to reset their password
-
-#### Scenario: Reset via email
-- **GIVEN** a user with a registered email
-- **WHEN** they request a password reset
-- **THEN** they receive a reset link`;
-
-    await fs.mkdir(path.join(specsDir, 'auth'), { recursive: true });
-    await fs.writeFile(path.join(specsDir, 'auth', 'spec.md'), testSpec);
-    
-    const testSpec2 = `## Purpose
-This specification defines the payment processing system.
-
-## Requirements
-
-### Requirement: Process Payments
-The system SHALL process credit card payments securely`;
-
-    await fs.mkdir(path.join(specsDir, 'payment'), { recursive: true });
-    await fs.writeFile(path.join(specsDir, 'payment', 'spec.md'), testSpec2);
-  });
-
-  afterEach(async () => {
-    await fs.rm(testDir, { recursive: true, force: true });
-  });
-
-  describe('spec show', () => {
-    it('should display spec in text format', async () => {
-      const originalCwd = process.cwd();
-      try {
-        process.chdir(testDir);
-        const output = execSync(`node ${openspecBin} spec show auth`, {
-          encoding: 'utf-8'
-        });
-        
-        // Raw passthrough should match spec.md content
-        const raw = await fs.readFile(path.join(specsDir, 'auth', 'spec.md'), 'utf-8');
-        expect(output.trim()).toBe(raw.trim());
-      } finally {
-        process.chdir(originalCwd);
-      }
-    });
-
-    it('should output spec as JSON with --json flag', () => {
-      const originalCwd = process.cwd();
-      try {
-        process.chdir(testDir);
-        const output = execSync(`node ${openspecBin} spec show auth --json`, {
-          encoding: 'utf-8'
-        });
-        
-        const json = JSON.parse(output);
-        expect(json.id).toBe('auth');
-        expect(json.title).toBe('auth');
-        expect(json.overview).toContain('test specification');
-        expect(json.requirements).toHaveLength(2);
-        expect(json.metadata.format).toBe('openspec');
-      } finally {
-        process.chdir(originalCwd);
-      }
-    });
-
-    it('should filter to show only requirements with --requirements flag (JSON only)', () => {
-      const originalCwd = process.cwd();
-      try {
-        process.chdir(testDir);
-        const output = execSync(`node ${openspecBin} spec show auth --json --requirements`, {
-          encoding: 'utf-8'
-        });
-        
-        const json = JSON.parse(output);
-        expect(json.requirements).toHaveLength(2);
-        // Scenarios should be excluded when --requirements is used
-        expect(json.requirements.every((r: any) => Array.isArray(r.scenarios) && r.scenarios.length === 0)).toBe(true);
-      } finally {
-        process.chdir(originalCwd);
-      }
-    });
-
-    it('should exclude scenarios with --no-scenarios flag (JSON only)', () => {
-      const originalCwd = process.cwd();
-      try {
-        process.chdir(testDir);
-        const output = execSync(`node ${openspecBin} spec show auth --json --no-scenarios`, {
-          encoding: 'utf-8'
-        });
-        
-        const json = JSON.parse(output);
-        expect(json.requirements).toHaveLength(2);
-        expect(json.requirements.every((r: any) => Array.isArray(r.scenarios) && r.scenarios.length === 0)).toBe(true);
-      } finally {
-        process.chdir(originalCwd);
-      }
-    });
-
-    it('should show specific requirement with -r flag (JSON only)', () => {
-      const originalCwd = process.cwd();
-      try {
-        process.chdir(testDir);
-        const output = execSync(`node ${openspecBin} spec show auth --json -r 1`, {
-          encoding: 'utf-8'
-        });
-        
-        const json = JSON.parse(output);
-        expect(json.requirements).toHaveLength(1);
-        expect(json.requirements[0].text).toContain('The system SHALL provide secure user authentication');
-      } finally {
-        process.chdir(originalCwd);
-      }
-    });
-
-    it('should return JSON with filtered requirements', () => {
-      const originalCwd = process.cwd();
-      try {
-        process.chdir(testDir);
-        const output = execSync(`node ${openspecBin} spec show auth --json --no-scenarios`, {
-          encoding: 'utf-8'
-        });
-        
-        const json = JSON.parse(output);
-        expect(json.requirements).toHaveLength(2);
-        expect(json.requirements[0].scenarios).toHaveLength(0);
-      } finally {
-        process.chdir(originalCwd);
-      }
-    });
-  });
-
-  describe('spec list', () => {
-    it('should list all available specs (IDs only by default)', () => {
-      const originalCwd = process.cwd();
-      try {
-        process.chdir(testDir);
-        const output = execSync(`node ${openspecBin} spec list`, {
-          encoding: 'utf-8'
-        });
-        
-        expect(output).toContain('auth');
-        expect(output).toContain('payment');
-        // Default should not include counts or teasers
-        expect(output).not.toMatch(/Requirements:\s*\d+/);
-      } finally {
-        process.chdir(originalCwd);
-      }
-    });
-
-    it('should output spec list as JSON with --json flag', () => {
-      const originalCwd = process.cwd();
-      try {
-        process.chdir(testDir);
-        const output = execSync(`node ${openspecBin} spec list --json`, {
-          encoding: 'utf-8'
-        });
-        
-        const json = JSON.parse(output);
-        expect(json).toHaveLength(2);
-        expect(json.find((s: any) => s.id === 'auth')).toBeDefined();
-        expect(json.find((s: any) => s.id === 'payment')).toBeDefined();
-        expect(json[0].requirementCount).toBeDefined();
-      } finally {
-        process.chdir(originalCwd);
-      }
-    });
-  });
-
-  describe('spec validate', () => {
-    it('should validate a valid spec', () => {
-      const originalCwd = process.cwd();
-      try {
-        process.chdir(testDir);
-        const output = execSync(`node ${openspecBin} spec validate auth`, {
-          encoding: 'utf-8'
-        });
-        
-        expect(output).toContain("Specification 'auth' is valid");
-      } finally {
-        process.chdir(originalCwd);
-      }
-    });
-
-    it('should output validation report as JSON with --json flag', () => {
-      const originalCwd = process.cwd();
-      try {
-        process.chdir(testDir);
-        const output = execSync(`node ${openspecBin} spec validate auth --json`, {
-          encoding: 'utf-8'
-        });
-        
-        const json = JSON.parse(output);
-        expect(json.valid).toBeDefined();
-        expect(json.issues).toBeDefined();
-        expect(json.summary).toBeDefined();
-        expect(json.summary.errors).toBeDefined();
-        expect(json.summary.warnings).toBeDefined();
-      } finally {
-        process.chdir(originalCwd);
-      }
-    });
-
-    it('should validate with strict mode', () => {
-      const originalCwd = process.cwd();
-      try {
-        process.chdir(testDir);
-        const output = execSync(`node ${openspecBin} spec validate auth --strict --json`, {
-          encoding: 'utf-8'
-        });
-        
-        const json = JSON.parse(output);
-        expect(json.valid).toBeDefined();
-        // In strict mode, warnings also affect validity
-      } finally {
-        process.chdir(originalCwd);
-      }
-    });
-
-    it('should detect invalid spec structure', async () => {
-      const invalidSpec = `## Purpose
-
-## Requirements
-This section has no actual requirements`;
-
-      await fs.mkdir(path.join(specsDir, 'invalid'), { recursive: true });
-      await fs.writeFile(path.join(specsDir, 'invalid', 'spec.md'), invalidSpec);
-
-      const originalCwd = process.cwd();
-      try {
-        process.chdir(testDir);
-        
-        // This should exit with non-zero code
-        let exitCode = 0;
-        try {
-          execSync(`node ${openspecBin} spec validate invalid`, {
-            encoding: 'utf-8'
-          });
-        } catch (error: any) {
-          exitCode = error.status;
-        }
-        
-        expect(exitCode).not.toBe(0);
-      } finally {
-        process.chdir(originalCwd);
-      }
-    });
-  });
-
-  describe('error handling', () => {
-    it('should handle non-existent spec gracefully', () => {
-      const originalCwd = process.cwd();
-      try {
-        process.chdir(testDir);
-        
-        let error: any;
-        try {
-          execSync(`node ${openspecBin} spec show nonexistent`, {
-            encoding: 'utf-8'
-          });
-        } catch (e) {
-          error = e;
-        }
-        
-        expect(error).toBeDefined();
-        expect(error.status).not.toBe(0);
-        expect(error.stderr.toString()).toContain('not found');
-      } finally {
-        process.chdir(originalCwd);
-      }
-    });
-
-    it('should handle missing specs directory gracefully', async () => {
-      await fs.rm(specsDir, { recursive: true, force: true });
-      const originalCwd = process.cwd();
-      try {
-        process.chdir(testDir);
-        const output = execSync(`node ${openspecBin} spec list`, { encoding: 'utf-8' });
-        expect(output.trim()).toBe('No items found');
-      } finally {
-        process.chdir(originalCwd);
-      }
-    });
-
-    it('should honor --no-color (no ANSI escapes)', () => {
-      const originalCwd = process.cwd();
-      try {
-        process.chdir(testDir);
-        const output = execSync(`node ${openspecBin} --no-color spec list --long`, { encoding: 'utf-8' });
-        // Basic ANSI escape pattern
-        const hasAnsi = /\u001b\[[0-9;]*m/.test(output);
-        expect(hasAnsi).toBe(false);
-      } finally {
-        process.chdir(originalCwd);
-      }
-    });
-  });
-});

package/OpenSpec-main/test/commands/validate.enriched-output.test.ts

@@ -1,49 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import { promises as fs } from 'fs';
-import path from 'path';
-import { execSync } from 'child_process';
-
-describe('validate command enriched human output', () => {
-  const projectRoot = process.cwd();
-  const testDir = path.join(projectRoot, 'test-validate-enriched-tmp');
-  const changesDir = path.join(testDir, 'openspec', 'changes');
-  const bin = path.join(projectRoot, 'bin', 'openspec.js');
-
-
-  beforeEach(async () => {
-    await fs.mkdir(changesDir, { recursive: true });
-  });
-
-  afterEach(async () => {
-    await fs.rm(testDir, { recursive: true, force: true });
-  });
-
-  it('prints Next steps footer and guidance on invalid change', async () => {
-    const changeContent = `# Test Change\n\n## Why\nThis is a sufficiently long explanation to pass the why length requirement for validation purposes.\n\n## What Changes\nThere are changes proposed, but no delta specs provided yet.`;
-    const changeId = 'c-next-steps';
-    const changePath = path.join(changesDir, changeId);
-    await fs.mkdir(changePath, { recursive: true });
-    await fs.writeFile(path.join(changePath, 'proposal.md'), changeContent);
-
-    const originalCwd = process.cwd();
-    try {
-      process.chdir(testDir);
-      let code = 0;
-      let stderr = '';
-      try {
-        execSync(`node ${bin} change validate ${changeId}`, { encoding: 'utf-8', stdio: 'pipe' });
-      } catch (e: any) {
-        code = e?.status ?? 1;
-        stderr = e?.stderr?.toString?.() ?? '';
-      }
-      expect(code).not.toBe(0);
-      expect(stderr).toContain('has issues');
-      expect(stderr).toContain('Next steps:');
-      expect(stderr).toContain('openspec change show');
-    } finally {
-      process.chdir(originalCwd);
-    }
-  });
-});
-
-

package/OpenSpec-main/test/commands/validate.test.ts

@@ -1,147 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import { promises as fs } from 'fs';
-import path from 'path';
-import { runCLI } from '../helpers/run-cli.js';
-
-describe('top-level validate command', () => {
-  const projectRoot = process.cwd();
-  const testDir = path.join(projectRoot, 'test-validate-command-tmp');
-  const changesDir = path.join(testDir, 'openspec', 'changes');
-  const specsDir = path.join(testDir, 'openspec', 'specs');
-
-  beforeEach(async () => {
-    await fs.mkdir(changesDir, { recursive: true });
-    await fs.mkdir(specsDir, { recursive: true });
-
-    // Create a valid spec
-    const specContent = [
-      '## Purpose',
-      'This spec ensures the validation harness exercises a deterministic alpha module for automated tests.',
-      '',
-      '## Requirements',
-      '',
-      '### Requirement: Alpha module SHALL produce deterministic output',
-      'The alpha module SHALL produce a deterministic response for validation.',
-      '',
-      '#### Scenario: Deterministic alpha run',
-      '- **GIVEN** a configured alpha module',
-      '- **WHEN** the module runs the default flow',
-      '- **THEN** the output matches the expected fixture result',
-    ].join('\n');
-    await fs.mkdir(path.join(specsDir, 'alpha'), { recursive: true });
-    await fs.writeFile(path.join(specsDir, 'alpha', 'spec.md'), specContent, 'utf-8');
-
-    // Create a simple change with bullets (parser supports this)
-    const changeContent = `# Test Change\n\n## Why\nBecause reasons that are sufficiently long for validation.\n\n## What Changes\n- **alpha:** Add something`;
-    await fs.mkdir(path.join(changesDir, 'c1'), { recursive: true });
-    await fs.writeFile(path.join(changesDir, 'c1', 'proposal.md'), changeContent, 'utf-8');
-    const deltaContent = [
-      '## ADDED Requirements',
-      '### Requirement: Validator SHALL support alpha change deltas',
-      'The validator SHALL accept deltas provided by the test harness.',
-      '',
-      '#### Scenario: Apply alpha delta',
-      '- **GIVEN** the test change delta',
-      '- **WHEN** openspec validate runs',
-      '- **THEN** the validator reports the change as valid',
-    ].join('\n');
-    const c1DeltaDir = path.join(changesDir, 'c1', 'specs', 'alpha');
-    await fs.mkdir(c1DeltaDir, { recursive: true });
-    await fs.writeFile(path.join(c1DeltaDir, 'spec.md'), deltaContent, 'utf-8');
-
-    // Duplicate name for ambiguity test
-    await fs.mkdir(path.join(changesDir, 'dup'), { recursive: true });
-    await fs.writeFile(path.join(changesDir, 'dup', 'proposal.md'), changeContent, 'utf-8');
-    const dupDeltaDir = path.join(changesDir, 'dup', 'specs', 'dup');
-    await fs.mkdir(dupDeltaDir, { recursive: true });
-    await fs.writeFile(path.join(dupDeltaDir, 'spec.md'), deltaContent, 'utf-8');
-    await fs.mkdir(path.join(specsDir, 'dup'), { recursive: true });
-    await fs.writeFile(path.join(specsDir, 'dup', 'spec.md'), specContent, 'utf-8');
-  });
-
-  afterEach(async () => {
-    await fs.rm(testDir, { recursive: true, force: true });
-  });
-
-  it('prints a helpful hint when no args in non-interactive mode', async () => {
-    const result = await runCLI(['validate'], { cwd: testDir });
-    expect(result.exitCode).toBe(1);
-    expect(result.stderr).toContain('Nothing to validate. Try one of:');
-  });
-
-  it('validates all with --all and outputs JSON summary', async () => {
-    const result = await runCLI(['validate', '--all', '--json'], { cwd: testDir });
-    expect(result.exitCode).toBe(0);
-    const output = result.stdout.trim();
-    expect(output).not.toBe('');
-    const json = JSON.parse(output);
-    expect(Array.isArray(json.items)).toBe(true);
-    expect(json.summary?.totals?.items).toBeDefined();
-    expect(json.version).toBe('1.0');
-  });
-
-  it('validates only specs with --specs and respects --concurrency', async () => {
-    const result = await runCLI(['validate', '--specs', '--json', '--concurrency', '1'], { cwd: testDir });
-    expect(result.exitCode).toBe(0);
-    const output = result.stdout.trim();
-    expect(output).not.toBe('');
-    const json = JSON.parse(output);
-    expect(json.items.every((i: any) => i.type === 'spec')).toBe(true);
-  });
-
-  it('errors on ambiguous item names and suggests type override', async () => {
-    const result = await runCLI(['validate', 'dup'], { cwd: testDir });
-    expect(result.exitCode).toBe(1);
-    expect(result.stderr).toContain('Ambiguous item');
-  });
-
-  it('accepts change proposals saved with CRLF line endings', async () => {
-    const changeId = 'crlf-change';
-    const toCrlf = (segments: string[]) => segments.join('\n').replace(/\n/g, '\r\n');
-
-    const crlfContent = toCrlf([
-      '# CRLF Proposal',
-      '',
-      '## Why',
-      'This change verifies validation works with Windows line endings.',
-      '',
-      '## What Changes',
-      '- **alpha:** Ensure validation passes on CRLF files',
-    ]);
-
-    await fs.mkdir(path.join(changesDir, changeId), { recursive: true });
-    await fs.writeFile(path.join(changesDir, changeId, 'proposal.md'), crlfContent, 'utf-8');
-
-    const deltaContent = toCrlf([
-      '## ADDED Requirements',
-      '### Requirement: Parser SHALL accept CRLF change proposals',
-      'The parser SHALL accept CRLF change proposals without manual edits.',
-      '',
-      '#### Scenario: Validate CRLF change',
-      '- **GIVEN** a change proposal saved with CRLF line endings',
-      '- **WHEN** a developer runs openspec validate on the proposal',
-      '- **THEN** validation succeeds without section errors',
-    ]);
-
-    const deltaDir = path.join(changesDir, changeId, 'specs', 'alpha');
-    await fs.mkdir(deltaDir, { recursive: true });
-    await fs.writeFile(path.join(deltaDir, 'spec.md'), deltaContent, 'utf-8');
-
-    const result = await runCLI(['validate', changeId], { cwd: testDir });
-    expect(result.exitCode).toBe(0);
-  });
-
-  it('respects --no-interactive flag passed via CLI', async () => {
-    // This test ensures Commander.js --no-interactive flag is correctly parsed
-    // and passed to the validate command. The flag sets options.interactive = false
-    // (not options.noInteractive = true) due to Commander.js convention.
-    const result = await runCLI(['validate', '--specs', '--no-interactive'], {
-      cwd: testDir,
-      // Don't set OPEN_SPEC_INTERACTIVE to ensure we're testing the flag itself
-      env: { ...process.env, OPEN_SPEC_INTERACTIVE: undefined },
-    });
-    expect(result.exitCode).toBe(0);
-    // Should complete without hanging and without prompts
-    expect(result.stderr).not.toContain('What would you like to validate?');
-  });
-});