principles-disciple 1.8.0 → 1.8.2

package/tests/core/migration.test.ts

@@ -0,0 +1,84 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { migrateDirectoryStructure } from '../../src/core/migration.js';
+import * as fs from 'fs';
+import * as path from 'path';
+
+vi.mock('fs');
+
+describe('Directory Structure Migration', () => {
+    const workspaceDir = '/mock/workspace';
+    const mockLogger = {
+        info: vi.fn(),
+        warn: vi.fn(),
+        error: vi.fn(),
+    };
+    const mockApi = { logger: mockLogger } as any;
+
+    beforeEach(() => {
+        vi.clearAllMocks();
+    });
+
+    it('should move non-security files from docs/ to new locations', () => {
+        const legacyPlan = path.join(workspaceDir, 'docs', 'PLAN.md');
+        const newPlan = '/mock/workspace/PLAN.md';
+
+        vi.mocked(fs.existsSync).mockImplementation((p) => {
+            const pathStr = p.toString();
+            if (pathStr === path.join(workspaceDir, 'docs')) return true;
+            if (pathStr === legacyPlan) return true;
+            // Destination directories don't exist yet
+            if (pathStr === path.join(workspaceDir, '.principles')) return false;
+            // Destination files don't exist yet
+            if (pathStr === newPlan) return false;
+            return false;
+        });
+
+        migrateDirectoryStructure(mockApi, workspaceDir);
+
+        // Verify it moved PLAN.md to root
+        expect(fs.renameSync).toHaveBeenCalledWith(legacyPlan, newPlan);
+
+        expect(mockLogger.info).toHaveBeenCalledWith(expect.stringContaining('Successfully migrated'));
+    });
+
+    it('should not migrate security-sensitive docs files', () => {
+        const legacyProfile = path.join(workspaceDir, 'docs', 'PROFILE.json');
+        const legacyScorecard = path.join(workspaceDir, 'docs', 'AGENT_SCORECARD.json');
+        const newProfile = '/mock/workspace/.principles/PROFILE.json';
+        const newScorecard = '/mock/workspace/.principles/.state/AGENT_SCORECARD.json';
+
+        vi.mocked(fs.existsSync).mockImplementation((p) => {
+            const pathStr = p.toString();
+            if (pathStr === path.join(workspaceDir, 'docs')) return true;
+            if (pathStr === legacyProfile) return true;
+            if (pathStr === legacyScorecard) return true;
+            if (pathStr === newProfile) return false;
+            if (pathStr === newScorecard) return false;
+            return false;
+        });
+
+        migrateDirectoryStructure(mockApi, workspaceDir);
+
+        expect(fs.renameSync).not.toHaveBeenCalledWith(legacyProfile, newProfile);
+        expect(fs.renameSync).not.toHaveBeenCalledWith(legacyScorecard, newScorecard);
+    });
+
+    it('should migrate legacy evolution_directive.json to EVOLUTION_DIRECTIVE instead of queue', () => {
+        const legacyDirective = path.join(workspaceDir, 'memory', '.state', 'evolution_directive.json');
+        const newDirective = '/mock/workspace/.state/evolution_directive.json';
+        const wrongQueuePath = '/mock/workspace/.state/evolution_queue.json';
+
+        vi.mocked(fs.existsSync).mockImplementation((p) => {
+            const pathStr = p.toString();
+            if (pathStr === legacyDirective) return true;
+            if (pathStr === newDirective) return false;
+            if (pathStr === wrongQueuePath) return false;
+            return false;
+        });
+
+        migrateDirectoryStructure(mockApi, workspaceDir);
+
+        expect(fs.renameSync).toHaveBeenCalledWith(legacyDirective, newDirective);
+        expect(fs.renameSync).not.toHaveBeenCalledWith(legacyDirective, wrongQueuePath);
+    });
+});