sinapse-ai 1.9.0 → 1.9.1

package/packages/installer/tests/unit/ide-sync-integration/ide-sync-integration.test.js

@@ -1,231 +0,0 @@
-'use strict';
-
-/**
- * IDE Sync Integration Tests (Story INS-4.5)
- *
- * Verifies that the installer calls commandSync and commandValidate
- * via the adapter pattern (save cwd, chdir, finally restore).
- */
-
-const path = require('path');
-
-// Mock the ide-sync module before requiring wizard
-const mockCommandSync = jest.fn();
-const mockCommandValidate = jest.fn();
-
-jest.mock('../../../../../.sinapse-ai/infrastructure/scripts/ide-sync/index', () => ({
-  commandSync: mockCommandSync,
-  commandValidate: mockCommandValidate,
-}));
-
-// We need to verify that the wizard source code has the correct integration
-const fs = require('fs');
-const WIZARD_PATH = path.join(
-  __dirname, '..', '..', '..', 'src', 'wizard', 'index.js'
-);
-
-describe('IDE Sync Integration (Story INS-4.5)', () => {
-  let wizardSource;
-
-  beforeAll(() => {
-    wizardSource = fs.readFileSync(WIZARD_PATH, 'utf8');
-  });
-
-  describe('AC1: IDE sync called via adapter pattern', () => {
-    test('wizard imports commandSync and commandValidate from ide-sync', () => {
-      expect(wizardSource).toContain(
-        "const { commandSync, commandValidate } = require('../../../../.sinapse-ai/infrastructure/scripts/ide-sync/index')"
-      );
-    });
-
-    test('wizard uses programmatic API (not child_process.exec)', () => {
-      // Should NOT shell out to ide-sync
-      expect(wizardSource).not.toMatch(/child_process.*ide-sync/);
-      expect(wizardSource).not.toMatch(/exec\(.*ide-sync/);
-      expect(wizardSource).not.toMatch(/spawn\(.*ide-sync/);
-    });
-
-    test('adapter pattern: saves cwd before calling commandSync', () => {
-      expect(wizardSource).toContain('const savedCwd = process.cwd()');
-    });
-
-    test('adapter pattern: uses explicit targetProjectRoot variable (not bare process.cwd())', () => {
-      expect(wizardSource).toContain('const targetProjectRoot = process.cwd()');
-      expect(wizardSource).toContain('process.chdir(targetProjectRoot)');
-    });
-
-    test('adapter pattern: restores cwd in finally block', () => {
-      // The finally block should restore savedCwd
-      expect(wizardSource).toContain('process.chdir(savedCwd)');
-      expect(wizardSource).toMatch(/finally\s*\{[^}]*process\.chdir\(savedCwd\)/s);
-    });
-
-    test('commandSync called with { quiet: true }', () => {
-      expect(wizardSource).toContain("await commandSync({ quiet: true })");
-    });
-
-    test('does NOT pass projectRoot or ides as parameters to commandSync', () => {
-      // commandSync uses process.cwd() internally — no projectRoot param
-      expect(wizardSource).not.toMatch(/commandSync\(\{[^}]*projectRoot/);
-      expect(wizardSource).not.toMatch(/commandSync\(\{[^}]*ides/);
-    });
-  });
-
-  describe('AC3: Graceful failure', () => {
-    test('sync failure is caught and does not propagate', () => {
-      // The try/catch should set ideSyncStatus to failed, not throw
-      expect(wizardSource).toContain("answers.ideSyncStatus = 'failed'");
-    });
-
-    test('failure message suggests sinapse doctor --fix', () => {
-      expect(wizardSource).toContain("sinapse doctor --fix");
-    });
-
-    test('install summary includes sync status on success', () => {
-      expect(wizardSource).toContain("answers.ideSyncStatus = 'synced'");
-    });
-
-    test('install summary includes sync status on failure', () => {
-      expect(wizardSource).toContain("answers.ideSyncStatus = 'failed'");
-    });
-  });
-
-  describe('AC4: Validate sync output', () => {
-    test('commandValidate called after commandSync', () => {
-      // commandValidate should appear after commandSync in the source
-      const syncIndex = wizardSource.indexOf('await commandSync({ quiet: true })');
-      const validateIndex = wizardSource.indexOf('await commandValidate(');
-      expect(syncIndex).toBeGreaterThan(-1);
-      expect(validateIndex).toBeGreaterThan(-1);
-      expect(validateIndex).toBeGreaterThan(syncIndex);
-    });
-
-    test('commandValidate uses same adapter pattern (within same finally block)', () => {
-      // Both commandSync and commandValidate should be within the same
-      // saved cwd / finally block
-      const savedCwdIndex = wizardSource.indexOf('const savedCwd = process.cwd()');
-      const syncIndex = wizardSource.indexOf('await commandSync({ quiet: true })');
-      const validateIndex = wizardSource.indexOf('await commandValidate(');
-      const finallyIndex = wizardSource.indexOf('process.chdir(savedCwd)');
-
-      // All should be in order: savedCwd < sync < validate < finally restore
-      expect(savedCwdIndex).toBeLessThan(syncIndex);
-      expect(syncIndex).toBeLessThan(validateIndex);
-      expect(validateIndex).toBeLessThan(finallyIndex);
-    });
-
-    test('validation drift logged as WARN not ERROR', () => {
-      expect(wizardSource).toContain("answers.ideSyncValidation = 'drift'");
-      // Should use console.warn, not console.error for drift
-      expect(wizardSource).toMatch(/console\.warn\(.*drift/i);
-    });
-
-    test('commandValidate console output suppressed (quiet workaround)', () => {
-      // commandValidate does not support quiet — wizard suppresses console.log
-      expect(wizardSource).toContain('const _origLog = console.log');
-      expect(wizardSource).toContain('console.log = () => {}');
-      // console.log must be restored in a finally block
-      expect(wizardSource).toContain('console.log = _origLog');
-    });
-  });
-
-  describe('AC5: Behavioral tests with mocks', () => {
-    beforeEach(() => {
-      mockCommandSync.mockReset();
-      mockCommandValidate.mockReset();
-    });
-
-    test('commandSync is called with { quiet: true } when invoked', async () => {
-      mockCommandSync.mockResolvedValue(undefined);
-      mockCommandValidate.mockResolvedValue(undefined);
-
-      // Simulate the adapter pattern call sequence
-      const savedCwd = process.cwd();
-      try {
-        await mockCommandSync({ quiet: true });
-        await mockCommandValidate({ quiet: true });
-      } finally {
-        process.chdir(savedCwd);
-      }
-
-      expect(mockCommandSync).toHaveBeenCalledWith({ quiet: true });
-      expect(mockCommandValidate).toHaveBeenCalledWith({ quiet: true });
-    });
-
-    test('commandSync failure does not throw — install continues', async () => {
-      mockCommandSync.mockRejectedValue(new Error('sync failed'));
-
-      let ideSyncStatus = 'unknown';
-      const savedCwd = process.cwd();
-      try {
-        process.chdir(process.cwd());
-        await mockCommandSync({ quiet: true });
-        ideSyncStatus = 'synced';
-      } catch (syncError) {
-        ideSyncStatus = 'failed';
-      } finally {
-        process.chdir(savedCwd);
-      }
-
-      // Install should continue — status is failed, no uncaught throw
-      expect(ideSyncStatus).toBe('failed');
-      expect(process.cwd()).toBe(savedCwd);
-    });
-
-    test('cwd is restored even when commandSync throws', async () => {
-      mockCommandSync.mockRejectedValue(new Error('sync failed'));
-      const originalCwd = process.cwd();
-
-      const savedCwd = process.cwd();
-      try {
-        process.chdir(process.cwd());
-        await mockCommandSync({ quiet: true });
-      } catch {
-        // Expected — sync failed
-      } finally {
-        process.chdir(savedCwd);
-      }
-
-      expect(process.cwd()).toBe(originalCwd);
-    });
-
-    test('commandValidate called after successful sync', async () => {
-      mockCommandSync.mockResolvedValue(undefined);
-      mockCommandValidate.mockResolvedValue(undefined);
-
-      const savedCwd = process.cwd();
-      let validateCalled = false;
-      try {
-        process.chdir(process.cwd());
-        await mockCommandSync({ quiet: true });
-        await mockCommandValidate({ quiet: true });
-        validateCalled = true;
-      } finally {
-        process.chdir(savedCwd);
-      }
-
-      expect(validateCalled).toBe(true);
-      expect(mockCommandValidate).toHaveBeenCalled();
-    });
-
-    test('commandValidate NOT called when commandSync fails', async () => {
-      mockCommandSync.mockRejectedValue(new Error('sync failed'));
-
-      const savedCwd = process.cwd();
-      try {
-        process.chdir(process.cwd());
-        await mockCommandSync({ quiet: true });
-        // This line should NOT be reached
-        await mockCommandValidate({ quiet: true });
-      } catch {
-        // Expected — sync failed, validate not called
-      } finally {
-        process.chdir(savedCwd);
-      }
-
-      expect(mockCommandSync).toHaveBeenCalled();
-      expect(mockCommandValidate).not.toHaveBeenCalled();
-    });
-  });
-});
-

package/packages/installer/tests/unit/merger/env-merger.test.js

@@ -1,191 +0,0 @@
-/**
- * @fileoverview Tests for EnvMerger
- * Story 9.2: .env File Merge Implementation
- */
-
-const { EnvMerger } = require('../../../src/merger/strategies/env-merger.js');
-const { parseEnvFile } = require('../../../src/merger/parsers/env-parser.js');
-
-describe('EnvMerger', () => {
-  let merger;
-
-  beforeEach(() => {
-    merger = new EnvMerger();
-  });
-
-  describe('canMerge', () => {
-    it('should always return true for .env files', () => {
-      expect(merger.canMerge('', '')).toBe(true);
-      expect(merger.canMerge('KEY=value', 'OTHER=value')).toBe(true);
-    });
-  });
-
-  describe('merge', () => {
-    it('should add new variables from new content', async () => {
-      const existing = 'EXISTING_VAR=existing_value';
-      const newContent = 'NEW_VAR=new_value';
-
-      const result = await merger.merge(existing, newContent);
-
-      expect(result.content).toContain('EXISTING_VAR=existing_value');
-      expect(result.content).toContain('NEW_VAR=new_value');
-      expect(result.stats.preserved).toBe(1);
-      expect(result.stats.added).toBe(1);
-    });
-
-    it('should preserve existing values over new values', async () => {
-      const existing = 'API_KEY=my_secret_key';
-      const newContent = 'API_KEY=placeholder_key';
-
-      const result = await merger.merge(existing, newContent);
-
-      // The original value is preserved at the start
-      expect(result.content.startsWith('API_KEY=my_secret_key')).toBe(true);
-      // The suggested value appears in SINAPSE_SUGGESTED comment
-      expect(result.content).toContain('SINAPSE_SUGGESTED: API_KEY=placeholder_key');
-      expect(result.stats.preserved).toBe(1);
-    });
-
-    it('should add SINAPSE_SUGGESTED comment for differing values', async () => {
-      const existing = 'PORT=3000';
-      const newContent = 'PORT=8080';
-
-      const result = await merger.merge(existing, newContent);
-
-      expect(result.content).toContain('PORT=3000');
-      expect(result.content).toContain('# SINAPSE_SUGGESTED: PORT=8080');
-      expect(result.stats.conflicts).toBe(1);
-    });
-
-    it('should preserve comments from existing file', async () => {
-      const existing = `# Database configuration
-DB_HOST=localhost
-DB_PORT=5432`;
-      const newContent = 'NEW_VAR=value';
-
-      const result = await merger.merge(existing, newContent);
-
-      expect(result.content).toContain('# Database configuration');
-      expect(result.content).toContain('DB_HOST=localhost');
-    });
-
-    it('should add new variables in SINAPSE Variables section', async () => {
-      const existing = 'EXISTING=value';
-      const newContent = `SINAPSE_VAR1=value1
-SINAPSE_VAR2=value2`;
-
-      const result = await merger.merge(existing, newContent);
-
-      // Header includes date: "# === SINAPSE Variables (added YYYY-MM-DD) ==="
-      expect(result.content).toMatch(/# === SINAPSE Variables \(added \d{4}-\d{2}-\d{2}\) ===/);
-      expect(result.content).toContain('SINAPSE_VAR1=value1');
-      expect(result.content).toContain('SINAPSE_VAR2=value2');
-    });
-
-    it('should handle empty existing content', async () => {
-      const existing = '';
-      const newContent = 'NEW_VAR=value';
-
-      const result = await merger.merge(existing, newContent);
-
-      expect(result.content).toContain('NEW_VAR=value');
-      expect(result.stats.added).toBe(1);
-    });
-
-    it('should handle empty new content', async () => {
-      const existing = 'EXISTING=value';
-      const newContent = '';
-
-      const result = await merger.merge(existing, newContent);
-
-      expect(result.content).toContain('EXISTING=value');
-      expect(result.stats.preserved).toBe(1);
-    });
-
-    it('should handle multiline values', async () => {
-      const existing = `PRIVATE_KEY="-----BEGIN RSA PRIVATE KEY-----
-MIIEpQIBAAKCAQEA
------END RSA PRIVATE KEY-----"`;
-      const newContent = 'OTHER=value';
-
-      const result = await merger.merge(existing, newContent);
-
-      expect(result.content).toContain('PRIVATE_KEY=');
-      expect(result.content).toContain('OTHER=value');
-    });
-  });
-
-  describe('preview', () => {
-    it('should return same result as merge with preview option', async () => {
-      const existing = 'A=1';
-      const newContent = 'B=2';
-
-      const previewResult = await merger.preview(existing, newContent);
-      const mergeResult = await merger.merge(existing, newContent, { preview: true });
-
-      expect(previewResult.content).toBe(mergeResult.content);
-      expect(previewResult.stats).toEqual(mergeResult.stats);
-    });
-  });
-});
-
-describe('parseEnvFile', () => {
-  it('should parse simple key-value pairs', () => {
-    const content = `KEY1=value1
-KEY2=value2`;
-
-    const result = parseEnvFile(content);
-
-    expect(result.variables.get('KEY1').value).toBe('value1');
-    expect(result.variables.get('KEY2').value).toBe('value2');
-  });
-
-  it('should handle quoted values', () => {
-    const content = `QUOTED="value with spaces"
-SINGLE='single quoted'`;
-
-    const result = parseEnvFile(content);
-
-    expect(result.variables.get('QUOTED').value).toBe('"value with spaces"');
-    expect(result.variables.get('SINGLE').value).toBe("'single quoted'");
-  });
-
-  it('should preserve comments', () => {
-    const content = `# This is a comment
-KEY=value
-# Another comment`;
-
-    const result = parseEnvFile(content);
-
-    expect(result.comments.length).toBeGreaterThan(0);
-    expect(result.variables.get('KEY').value).toBe('value');
-  });
-
-  it('should handle empty lines', () => {
-    const content = `KEY1=value1
-
-KEY2=value2`;
-
-    const result = parseEnvFile(content);
-
-    expect(result.variables.get('KEY1').value).toBe('value1');
-    expect(result.variables.get('KEY2').value).toBe('value2');
-  });
-
-  it('should handle values with equals signs', () => {
-    const content = 'URL=https://example.com?param=value';
-
-    const result = parseEnvFile(content);
-
-    expect(result.variables.get('URL').value).toBe('https://example.com?param=value');
-  });
-
-  it('should handle empty values', () => {
-    const content = 'EMPTY_VAR=';
-
-    const result = parseEnvFile(content);
-
-    expect(result.variables.has('EMPTY_VAR')).toBe(true);
-    expect(result.variables.get('EMPTY_VAR').value).toBe('');
-  });
-});

package/packages/installer/tests/unit/merger/markdown-merger.test.js

@@ -1,262 +0,0 @@
-/**
- * @fileoverview Tests for MarkdownMerger
- * Story 9.3: Markdown Merge Implementation
- */
-
-const { MarkdownMerger } = require('../../../src/merger/strategies/markdown-merger.js');
-const {
-  parseMarkdownSections,
-  slugify,
-  hasSinapseMarkers,
-} = require('../../../src/merger/parsers/markdown-section-parser.js');
-
-describe('MarkdownMerger', () => {
-  let merger;
-
-  beforeEach(() => {
-    merger = new MarkdownMerger();
-  });
-
-  describe('canMerge', () => {
-    it('should return true for content with SINAPSE markers', () => {
-      const content = `# Title
-<!-- SINAPSE-MANAGED-START: section1 -->
-Content
-<!-- SINAPSE-MANAGED-END: section1 -->`;
-
-      expect(merger.canMerge(content, '')).toBe(true);
-    });
-
-    it('should return false for content without SINAPSE markers', () => {
-      const content = `# Title
-Some content without markers`;
-
-      expect(merger.canMerge(content, '')).toBe(false);
-    });
-  });
-
-  describe('merge', () => {
-    it('should update SINAPSE-managed sections', async () => {
-      const existing = `# My Rules
-
-<!-- SINAPSE-MANAGED-START: agent-system -->
-## Old Agent System
-Old content
-<!-- SINAPSE-MANAGED-END: agent-system -->
-
-## My Custom Section
-Custom content`;
-
-      const newContent = `# Template
-
-<!-- SINAPSE-MANAGED-START: agent-system -->
-## Agent System
-New updated content
-<!-- SINAPSE-MANAGED-END: agent-system -->`;
-
-      const result = await merger.merge(existing, newContent);
-
-      expect(result.content).toContain('## Agent System');
-      expect(result.content).toContain('New updated content');
-      expect(result.content).toContain('## My Custom Section');
-      expect(result.content).toContain('Custom content');
-      expect(result.stats.updated).toBe(1);
-    });
-
-    it('should preserve user sections', async () => {
-      const existing = `# Rules
-
-<!-- SINAPSE-MANAGED-START: core -->
-Core content
-<!-- SINAPSE-MANAGED-END: core -->
-
-## My Custom Rules
-1. Rule one
-2. Rule two`;
-
-      const newContent = `# Template
-
-<!-- SINAPSE-MANAGED-START: core -->
-Updated core
-<!-- SINAPSE-MANAGED-END: core -->`;
-
-      const result = await merger.merge(existing, newContent);
-
-      expect(result.content).toContain('## My Custom Rules');
-      expect(result.content).toContain('1. Rule one');
-      expect(result.content).toContain('2. Rule two');
-      expect(result.stats.preserved).toBeGreaterThan(0);
-    });
-
-    it('should add new SINAPSE sections that do not exist', async () => {
-      const existing = `# Rules
-
-<!-- SINAPSE-MANAGED-START: core -->
-Core content
-<!-- SINAPSE-MANAGED-END: core -->`;
-
-      const newContent = `# Template
-
-<!-- SINAPSE-MANAGED-START: core -->
-Updated core
-<!-- SINAPSE-MANAGED-END: core -->
-
-<!-- SINAPSE-MANAGED-START: new-section -->
-New section content
-<!-- SINAPSE-MANAGED-END: new-section -->`;
-
-      const result = await merger.merge(existing, newContent);
-
-      expect(result.content).toContain('<!-- SINAPSE-MANAGED-START: new-section -->');
-      expect(result.content).toContain('New section content');
-      expect(result.stats.added).toBe(1);
-    });
-
-    it('should handle files with no SINAPSE sections', async () => {
-      const existing = `# My Custom Rules
-
-## Section 1
-Content 1
-
-## Section 2
-Content 2`;
-
-      const newContent = `# Template
-
-<!-- SINAPSE-MANAGED-START: core -->
-Core content
-<!-- SINAPSE-MANAGED-END: core -->`;
-
-      const result = await merger.merge(existing, newContent);
-
-      // Should preserve existing content and add SINAPSE section
-      expect(result.content).toContain('## Section 1');
-      expect(result.content).toContain('Content 1');
-    });
-  });
-
-  describe('migrateLegacy', () => {
-    it('should append SINAPSE sections to legacy file', async () => {
-      const existing = `# My Old Rules
-Custom content here`;
-
-      const template = `# Template
-
-<!-- SINAPSE-MANAGED-START: core -->
-Core content
-<!-- SINAPSE-MANAGED-END: core -->`;
-
-      // migrateLegacy expects a parsed template object, so use merge which handles that
-      const result = await merger.merge(existing, template);
-
-      expect(result.content).toContain('# My Old Rules');
-      expect(result.content).toContain('Custom content here');
-      expect(result.content).toContain('<!-- SINAPSE-MANAGED-START: core -->');
-      expect(result.isLegacyMigration).toBe(true);
-    });
-  });
-});
-
-describe('parseMarkdownSections', () => {
-  it('should identify SINAPSE-managed sections', () => {
-    const content = `# Title
-
-<!-- SINAPSE-MANAGED-START: section1 -->
-Managed content
-<!-- SINAPSE-MANAGED-END: section1 -->
-
-## User Section
-User content`;
-
-    const result = parseMarkdownSections(content);
-
-    const managedSection = result.sections.find((s) => s.id === 'section1');
-    expect(managedSection).toBeDefined();
-    expect(managedSection.managed).toBe(true);
-    expect(managedSection.lines.join('\n')).toContain('Managed content');
-  });
-
-  it('should identify user sections', () => {
-    const content = `# Title
-
-<!-- SINAPSE-MANAGED-START: managed -->
-Managed
-<!-- SINAPSE-MANAGED-END: managed -->
-
-## User Section
-User content`;
-
-    const result = parseMarkdownSections(content);
-
-    const userSections = result.sections.filter((s) => !s.managed);
-    expect(userSections.length).toBeGreaterThan(0);
-  });
-
-  it('should handle nested content in managed sections', () => {
-    const content = `<!-- SINAPSE-MANAGED-START: test -->
-## Heading
-- List item 1
-- List item 2
-
-\`\`\`javascript
-const code = true;
-\`\`\`
-<!-- SINAPSE-MANAGED-END: test -->`;
-
-    const result = parseMarkdownSections(content);
-
-    const section = result.sections.find((s) => s.id === 'test');
-    const sectionContent = section.lines.join('\n');
-    expect(sectionContent).toContain('## Heading');
-    expect(sectionContent).toContain('- List item 1');
-    expect(sectionContent).toContain('const code = true');
-  });
-});
-
-describe('slugify', () => {
-  it('should convert text to lowercase slug', () => {
-    expect(slugify('Hello World')).toBe('hello-world');
-  });
-
-  it('should handle special characters', () => {
-    expect(slugify('Test & Example!')).toBe('test-example');
-  });
-
-  it('should collapse multiple dashes', () => {
-    expect(slugify('Multiple   Spaces')).toBe('multiple-spaces');
-  });
-
-  it('should trim leading and trailing dashes', () => {
-    expect(slugify('  Trimmed  ')).toBe('trimmed');
-  });
-
-  it('should handle numbers', () => {
-    // Dots are removed, so numbers become consecutive
-    expect(slugify('Section 1.2.3')).toBe('section-123');
-    // With spaces between numbers
-    expect(slugify('Section 1 2 3')).toBe('section-1-2-3');
-  });
-});
-
-describe('hasSinapseMarkers', () => {
-  it('should return true for content with SINAPSE markers', () => {
-    const content = `<!-- SINAPSE-MANAGED-START: test -->
-Content
-<!-- SINAPSE-MANAGED-END: test -->`;
-
-    expect(hasSinapseMarkers(content)).toBe(true);
-  });
-
-  it('should return false for content without markers', () => {
-    const content = '# Just a heading\nSome content';
-
-    expect(hasSinapseMarkers(content)).toBe(false);
-  });
-
-  it('should return false for incomplete markers', () => {
-    const content = '<!-- SINAPSE-MANAGED-START: test -->';
-
-    expect(hasSinapseMarkers(content)).toBe(false);
-  });
-});
-