@planu/cli 0.88.1 → 0.90.1

package/dist/tools/generate-tests.test.js

@@ -0,0 +1,222 @@
+// Tests for handleGenerateTests
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+vi.mock('../i18n/index.js', () => ({
+    t: (key) => key,
+    ti: (key, _vars) => key,
+}));
+vi.mock('../storage/index.js', () => ({
+    knowledgeStore: { getKnowledge: vi.fn() },
+    specStore: { getSpec: vi.fn() },
+}));
+import { handleGenerateTests } from './generate-tests.js';
+import { knowledgeStore, specStore } from '../storage/index.js';
+const mockGetKnowledge = vi.mocked(knowledgeStore.getKnowledge);
+const mockGetSpec = vi.mocked(specStore.getSpec);
+const baseKnowledge = {
+    language: 'typescript',
+    framework: 'express',
+    projectPath: '/test',
+    stack: ['vitest'],
+    conventions: {},
+    architecture: { primary: 'monolith', secondary: [], layers: [] },
+    database: 'postgresql',
+    testCommand: 'npm test',
+    buildCommand: 'npm run build',
+    linting: { detectedLinters: [] },
+    apiContracts: [],
+    layers: [],
+};
+const baseSpec = {
+    id: 'SPEC-001',
+    slug: 'user-auth',
+    title: 'User Authentication',
+    type: 'feature',
+    scope: 'feature',
+    target: 'backend',
+    tags: ['auth'],
+    status: 'approved',
+    risk: 'medium',
+    difficulty: 3,
+    estimation: { devHours: 8, reviewHours: 2, totalCostUsd: 100 },
+    dependencies: [],
+    createdAt: '2025-01-01',
+    updatedAt: '2025-01-01',
+};
+beforeEach(() => {
+    vi.clearAllMocks();
+});
+describe('handleGenerateTests', () => {
+    it('should return error when project not found', async () => {
+        mockGetKnowledge.mockResolvedValue(null);
+        const result = await handleGenerateTests({
+            specId: 'SPEC-001',
+            projectId: 'missing',
+        });
+        expect(result.isError).toBe(true);
+        expect(result.content[0].text).toContain('errors.projectNotFound');
+    });
+    it('should return error when spec not found', async () => {
+        mockGetKnowledge.mockResolvedValue(baseKnowledge);
+        mockGetSpec.mockResolvedValue(null);
+        const result = await handleGenerateTests({
+            specId: 'SPEC-999',
+            projectId: 'proj-1',
+        });
+        expect(result.isError).toBe(true);
+        expect(result.content[0].text).toContain('errors.specNotFound');
+    });
+    it('should generate unit test for backend spec', async () => {
+        mockGetKnowledge.mockResolvedValue(baseKnowledge);
+        mockGetSpec.mockResolvedValue(baseSpec);
+        const result = await handleGenerateTests({
+            specId: 'SPEC-001',
+            projectId: 'proj-1',
+        });
+        expect(result.isError).toBeUndefined();
+        const text = result.content[0].text;
+        expect(text).toContain('Unit Tests');
+        expect(text).toContain('User Authentication');
+        expect(text).toContain('vitest');
+        expect(text).toContain('Coverage target');
+        expect(text).toContain('80%');
+        expect(text).toContain('Integration Tests'); // backend => integration
+        expect(text).not.toContain('E2E Tests'); // backend only => no e2e
+        expect(text).toContain('Manual Tests');
+        expect(text).toContain('Generated Test Files');
+        expect(text).toContain('Commands');
+    });
+    it('should generate e2e test for fullstack spec', async () => {
+        mockGetKnowledge.mockResolvedValue(baseKnowledge);
+        mockGetSpec.mockResolvedValue({ ...baseSpec, target: 'fullstack' });
+        const result = await handleGenerateTests({
+            specId: 'SPEC-001',
+            projectId: 'proj-1',
+        });
+        const text = result.content[0].text;
+        expect(text).toContain('E2E Tests');
+        expect(text).toContain('Integration Tests');
+    });
+    it('should generate e2e test for frontend spec without integration', async () => {
+        mockGetKnowledge.mockResolvedValue(baseKnowledge);
+        mockGetSpec.mockResolvedValue({ ...baseSpec, target: 'frontend' });
+        const result = await handleGenerateTests({
+            specId: 'SPEC-001',
+            projectId: 'proj-1',
+        });
+        const text = result.content[0].text;
+        expect(text).toContain('E2E Tests');
+        // frontend-only: no integration tests generated
+    });
+    it('should use pytest for python language', async () => {
+        mockGetKnowledge.mockResolvedValue({
+            ...baseKnowledge,
+            language: 'python',
+            stack: ['pytest'],
+        });
+        mockGetSpec.mockResolvedValue(baseSpec);
+        const result = await handleGenerateTests({
+            specId: 'SPEC-001',
+            projectId: 'proj-1',
+        });
+        const text = result.content[0].text;
+        expect(text).toContain('pytest');
+        expect(text).toContain('.py');
+    });
+    it('should use cypress when in stack', async () => {
+        mockGetKnowledge.mockResolvedValue({
+            ...baseKnowledge,
+            stack: ['vitest', 'cypress'],
+        });
+        mockGetSpec.mockResolvedValue({ ...baseSpec, target: 'fullstack' });
+        const result = await handleGenerateTests({
+            specId: 'SPEC-001',
+            projectId: 'proj-1',
+        });
+        const text = result.content[0].text;
+        expect(text).toContain('cy.');
+    });
+    it('should include existing test command in commands', async () => {
+        mockGetKnowledge.mockResolvedValue(baseKnowledge);
+        mockGetSpec.mockResolvedValue(baseSpec);
+        const result = await handleGenerateTests({
+            specId: 'SPEC-001',
+            projectId: 'proj-1',
+        });
+        const text = result.content[0].text;
+        expect(text).toContain('npm test');
+    });
+    it('should default to vitest for unknown language', async () => {
+        mockGetKnowledge.mockResolvedValue({
+            ...baseKnowledge,
+            language: 'unknown-lang',
+            stack: [],
+        });
+        mockGetSpec.mockResolvedValue(baseSpec);
+        const result = await handleGenerateTests({
+            specId: 'SPEC-001',
+            projectId: 'proj-1',
+        });
+        const text = result.content[0].text;
+        expect(text).toContain('vitest');
+    });
+    it('should omit existing test command when testCommand is null', async () => {
+        mockGetKnowledge.mockResolvedValue({
+            ...baseKnowledge,
+            testCommand: null,
+        });
+        mockGetSpec.mockResolvedValue(baseSpec);
+        const result = await handleGenerateTests({
+            specId: 'SPEC-001',
+            projectId: 'proj-1',
+        });
+        const text = result.content[0].text;
+        expect(text).not.toContain('# Run all tests');
+        expect(text).toContain('vitest'); // still shows framework commands
+    });
+    it('should omit framework commands for unknown framework', async () => {
+        mockGetKnowledge.mockResolvedValue({
+            ...baseKnowledge,
+            language: 'dart',
+            stack: [],
+            testCommand: null,
+        });
+        mockGetSpec.mockResolvedValue(baseSpec);
+        const result = await handleGenerateTests({
+            specId: 'SPEC-001',
+            projectId: 'proj-1',
+        });
+        const text = result.content[0].text;
+        // dart defaults to 'flutter-test' which has no entry in frameworkCmds
+        expect(text).not.toContain('Commands');
+    });
+    it('should use jest when jest is in stack', async () => {
+        mockGetKnowledge.mockResolvedValue({
+            ...baseKnowledge,
+            stack: ['jest'],
+        });
+        mockGetSpec.mockResolvedValue(baseSpec);
+        const result = await handleGenerateTests({
+            specId: 'SPEC-001',
+            projectId: 'proj-1',
+        });
+        const text = result.content[0].text;
+        expect(text).toContain('jest');
+        // jest commands should be present
+        expect(text).toContain('npx jest');
+    });
+    it('should use pytest when pytest is in stack (non-python language)', async () => {
+        mockGetKnowledge.mockResolvedValue({
+            ...baseKnowledge,
+            language: 'typescript',
+            stack: ['pytest'],
+        });
+        mockGetSpec.mockResolvedValue(baseSpec);
+        const result = await handleGenerateTests({
+            specId: 'SPEC-001',
+            projectId: 'proj-1',
+        });
+        const text = result.content[0].text;
+        expect(text).toContain('pytest');
+    });
+});
+//# sourceMappingURL=generate-tests.test.js.map

package/dist/tools/generate-tests.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"generate-tests.test.js","sourceRoot":"","sources":["../../src/tools/generate-tests.test.ts"],"names":[],"mappings":"AAAA,gCAAgC;AAChC,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAC;AAE9D,EAAE,CAAC,IAAI,CAAC,kBAAkB,EAAE,GAAG,EAAE,CAAC,CAAC;IACjC,CAAC,EAAE,CAAC,GAAW,EAAE,EAAE,CAAC,GAAG;IACvB,EAAE,EAAE,CAAC,GAAW,EAAE,KAA6B,EAAE,EAAE,CAAC,GAAG;CACxD,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,qBAAqB,EAAE,GAAG,EAAE,CAAC,CAAC;IACpC,cAAc,EAAE,EAAE,YAAY,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE;IACzC,SAAS,EAAE,EAAE,OAAO,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE;CAChC,CAAC,CAAC,CAAC;AAEJ,OAAO,EAAE,mBAAmB,EAAE,MAAM,qBAAqB,CAAC;AAC1D,OAAO,EAAE,cAAc,EAAE,SAAS,EAAE,MAAM,qBAAqB,CAAC;AAEhE,MAAM,gBAAgB,GAAG,EAAE,CAAC,MAAM,CAAC,cAAc,CAAC,YAAY,CAAC,CAAC;AAChE,MAAM,WAAW,GAAG,EAAE,CAAC,MAAM,CAAC,SAAS,CAAC,OAAO,CAAC,CAAC;AAEjD,MAAM,aAAa,GAAG;IACpB,QAAQ,EAAE,YAAY;IACtB,SAAS,EAAE,SAAS;IACpB,WAAW,EAAE,OAAO;IACpB,KAAK,EAAE,CAAC,QAAQ,CAAC;IACjB,WAAW,EAAE,EAAE;IACf,YAAY,EAAE,EAAE,OAAO,EAAE,UAAU,EAAE,SAAS,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE;IAChE,QAAQ,EAAE,YAAY;IACtB,WAAW,EAAE,UAAU;IACvB,YAAY,EAAE,eAAe;IAC7B,OAAO,EAAE,EAAE,eAAe,EAAE,EAAE,EAAE;IAChC,YAAY,EAAE,EAAE;IAChB,MAAM,EAAE,EAAE;CACX,CAAC;AAEF,MAAM,QAAQ,GAAG;IACf,EAAE,EAAE,UAAU;IACd,IAAI,EAAE,WAAW;IACjB,KAAK,EAAE,qBAAqB;IAC5B,IAAI,EAAE,SAAS;IACf,KAAK,EAAE,SAAS;IAChB,MAAM,EAAE,SAAS;IACjB,IAAI,EAAE,CAAC,MAAM,CAAC;IACd,MAAM,EAAE,UAAU;IAClB,IAAI,EAAE,QAAQ;IACd,UAAU,EAAE,CAAC;IACb,UAAU,EAAE,EAAE,QAAQ,EAAE,CAAC,EAAE,WAAW,EAAE,CAAC,EAAE,YAAY,EAAE,GAAG,EAAE;IAC9D,YAAY,EAAE,EAAE;IAChB,SAAS,EAAE,YAAY;IACvB,SAAS,EAAE,YAAY;CACxB,CAAC;AAEF,UAAU,CAAC,GAAG,EAAE;IACd,EAAE,CAAC,aAAa,EAAE,CAAC;AACrB,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,qBAAqB,EAAE,GAAG,EAAE;IACnC,EAAE,CAAC,4CAA4C,EAAE,KAAK,IAAI,EAAE;QAC1D,gBAAgB,CAAC,iBAAiB,CAAC,IAAI,CAAC,CAAC;QAEzC,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC;YACvC,MAAM,EAAE,UAAU;YAClB,SAAS,EAAE,SAAS;SACrB,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAClC,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAE,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,wBAAwB,CAAC,CAAC;IACtE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yCAAyC,EAAE,KAAK,IAAI,EAAE;QACvD,gBAAgB,CAAC,iBAAiB,CAAC,aAAsB,CAAC,CAAC;QAC3D,WAAW,CAAC,iBAAiB,CAAC,IAAI,CAAC,CAAC;QAEpC,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC;YACvC,MAAM,EAAE,UAAU;YAClB,SAAS,EAAE,QAAQ;SACpB,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAClC,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAE,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,qBAAqB,CAAC,CAAC;IACnE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4CAA4C,EAAE,KAAK,IAAI,EAAE;QAC1D,gBAAgB,CAAC,iBAAiB,CAAC,aAAsB,CAAC,CAAC;QAC3D,WAAW,CAAC,iBAAiB,CAAC,QAAiB,CAAC,CAAC;QAEjD,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC;YACvC,MAAM,EAAE,UAAU;YAClB,SAAS,EAAE,QAAQ;SACpB,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,aAAa,EAAE,CAAC;QACvC,MAAM,IAAI,GAAG,MAAM,CAAC,OAAO,CAAC,CAAC,CAAE,CAAC,IAAI,CAAC;QACrC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC;QACrC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,qBAAqB,CAAC,CAAC;QAC9C,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAC;QACjC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,iBAAiB,CAAC,CAAC;QAC1C,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,KAAK,CAAC,CAAC;QAC9B,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,mBAAmB,CAAC,CAAC,CAAC,yBAAyB;QACtE,MAAM,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,WAAW,CAAC,CAAC,CAAC,yBAAyB;QAClE,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,cAAc,CAAC,CAAC;QACvC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,sBAAsB,CAAC,CAAC;QAC/C,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,UAAU,CAAC,CAAC;IACrC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6CAA6C,EAAE,KAAK,IAAI,EAAE;QAC3D,gBAAgB,CAAC,iBAAiB,CAAC,aAAsB,CAAC,CAAC;QAC3D,WAAW,CAAC,iBAAiB,CAAC,EAAE,GAAG,QAAQ,EAAE,MAAM,EAAE,WAAW,EAAW,CAAC,CAAC;QAE7E,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC;YACvC,MAAM,EAAE,UAAU;YAClB,SAAS,EAAE,QAAQ;SACpB,CAAC,CAAC;QAEH,MAAM,IAAI,GAAG,MAAM,CAAC,OAAO,CAAC,CAAC,CAAE,CAAC,IAAI,CAAC;QACrC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,WAAW,CAAC,CAAC;QACpC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,mBAAmB,CAAC,CAAC;IAC9C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gEAAgE,EAAE,KAAK,IAAI,EAAE;QAC9E,gBAAgB,CAAC,iBAAiB,CAAC,aAAsB,CAAC,CAAC;QAC3D,WAAW,CAAC,iBAAiB,CAAC,EAAE,GAAG,QAAQ,EAAE,MAAM,EAAE,UAAU,EAAW,CAAC,CAAC;QAE5E,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC;YACvC,MAAM,EAAE,UAAU;YAClB,SAAS,EAAE,QAAQ;SACpB,CAAC,CAAC;QAEH,MAAM,IAAI,GAAG,MAAM,CAAC,OAAO,CAAC,CAAC,CAAE,CAAC,IAAI,CAAC;QACrC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,WAAW,CAAC,CAAC;QACpC,gDAAgD;IAClD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uCAAuC,EAAE,KAAK,IAAI,EAAE;QACrD,gBAAgB,CAAC,iBAAiB,CAAC;YACjC,GAAG,aAAa;YAChB,QAAQ,EAAE,QAAQ;YAClB,KAAK,EAAE,CAAC,QAAQ,CAAC;SACT,CAAC,CAAC;QACZ,WAAW,CAAC,iBAAiB,CAAC,QAAiB,CAAC,CAAC;QAEjD,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC;YACvC,MAAM,EAAE,UAAU;YAClB,SAAS,EAAE,QAAQ;SACpB,CAAC,CAAC;QAEH,MAAM,IAAI,GAAG,MAAM,CAAC,OAAO,CAAC,CAAC,CAAE,CAAC,IAAI,CAAC;QACrC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAC;QACjC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,KAAK,CAAC,CAAC;IAChC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kCAAkC,EAAE,KAAK,IAAI,EAAE;QAChD,gBAAgB,CAAC,iBAAiB,CAAC;YACjC,GAAG,aAAa;YAChB,KAAK,EAAE,CAAC,QAAQ,EAAE,SAAS,CAAC;SACpB,CAAC,CAAC;QACZ,WAAW,CAAC,iBAAiB,CAAC,EAAE,GAAG,QAAQ,EAAE,MAAM,EAAE,WAAW,EAAW,CAAC,CAAC;QAE7E,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC;YACvC,MAAM,EAAE,UAAU;YAClB,SAAS,EAAE,QAAQ;SACpB,CAAC,CAAC;QAEH,MAAM,IAAI,GAAG,MAAM,CAAC,OAAO,CAAC,CAAC,CAAE,CAAC,IAAI,CAAC;QACrC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,KAAK,CAAC,CAAC;IAChC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kDAAkD,EAAE,KAAK,IAAI,EAAE;QAChE,gBAAgB,CAAC,iBAAiB,CAAC,aAAsB,CAAC,CAAC;QAC3D,WAAW,CAAC,iBAAiB,CAAC,QAAiB,CAAC,CAAC;QAEjD,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC;YACvC,MAAM,EAAE,UAAU;YAClB,SAAS,EAAE,QAAQ;SACpB,CAAC,CAAC;QAEH,MAAM,IAAI,GAAG,MAAM,CAAC,OAAO,CAAC,CAAC,CAAE,CAAC,IAAI,CAAC;QACrC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,UAAU,CAAC,CAAC;IACrC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+CAA+C,EAAE,KAAK,IAAI,EAAE;QAC7D,gBAAgB,CAAC,iBAAiB,CAAC;YACjC,GAAG,aAAa;YAChB,QAAQ,EAAE,cAAc;YACxB,KAAK,EAAE,EAAE;SACD,CAAC,CAAC;QACZ,WAAW,CAAC,iBAAiB,CAAC,QAAiB,CAAC,CAAC;QAEjD,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC;YACvC,MAAM,EAAE,UAAU;YAClB,SAAS,EAAE,QAAQ;SACpB,CAAC,CAAC;QAEH,MAAM,IAAI,GAAG,MAAM,CAAC,OAAO,CAAC,CAAC,CAAE,CAAC,IAAI,CAAC;QACrC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAC;IACnC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4DAA4D,EAAE,KAAK,IAAI,EAAE;QAC1E,gBAAgB,CAAC,iBAAiB,CAAC;YACjC,GAAG,aAAa;YAChB,WAAW,EAAE,IAAI;SACT,CAAC,CAAC;QACZ,WAAW,CAAC,iBAAiB,CAAC,QAAiB,CAAC,CAAC;QAEjD,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC;YACvC,MAAM,EAAE,UAAU;YAClB,SAAS,EAAE,QAAQ;SACpB,CAAC,CAAC;QAEH,MAAM,IAAI,GAAG,MAAM,CAAC,OAAO,CAAC,CAAC,CAAE,CAAC,IAAI,CAAC;QACrC,MAAM,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,iBAAiB,CAAC,CAAC;QAC9C,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAC,CAAC,iCAAiC;IACrE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sDAAsD,EAAE,KAAK,IAAI,EAAE;QACpE,gBAAgB,CAAC,iBAAiB,CAAC;YACjC,GAAG,aAAa;YAChB,QAAQ,EAAE,MAAM;YAChB,KAAK,EAAE,EAAE;YACT,WAAW,EAAE,IAAI;SACT,CAAC,CAAC;QACZ,WAAW,CAAC,iBAAiB,CAAC,QAAiB,CAAC,CAAC;QAEjD,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC;YACvC,MAAM,EAAE,UAAU;YAClB,SAAS,EAAE,QAAQ;SACpB,CAAC,CAAC;QAEH,MAAM,IAAI,GAAG,MAAM,CAAC,OAAO,CAAC,CAAC,CAAE,CAAC,IAAI,CAAC;QACrC,sEAAsE;QACtE,MAAM,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,UAAU,CAAC,CAAC;IACzC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uCAAuC,EAAE,KAAK,IAAI,EAAE;QACrD,gBAAgB,CAAC,iBAAiB,CAAC;YACjC,GAAG,aAAa;YAChB,KAAK,EAAE,CAAC,MAAM,CAAC;SACP,CAAC,CAAC;QACZ,WAAW,CAAC,iBAAiB,CAAC,QAAiB,CAAC,CAAC;QAEjD,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC;YACvC,MAAM,EAAE,UAAU;YAClB,SAAS,EAAE,QAAQ;SACpB,CAAC,CAAC;QAEH,MAAM,IAAI,GAAG,MAAM,CAAC,OAAO,CAAC,CAAC,CAAE,CAAC,IAAI,CAAC;QACrC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC;QAC/B,kCAAkC;QAClC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,UAAU,CAAC,CAAC;IACrC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iEAAiE,EAAE,KAAK,IAAI,EAAE;QAC/E,gBAAgB,CAAC,iBAAiB,CAAC;YACjC,GAAG,aAAa;YAChB,QAAQ,EAAE,YAAY;YACtB,KAAK,EAAE,CAAC,QAAQ,CAAC;SACT,CAAC,CAAC;QACZ,WAAW,CAAC,iBAAiB,CAAC,QAAiB,CAAC,CAAC;QAEjD,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC;YACvC,MAAM,EAAE,UAAU;YAClB,SAAS,EAAE,QAAQ;SACpB,CAAC,CAAC;QAEH,MAAM,IAAI,GAAG,MAAM,CAAC,OAAO,CAAC,CAAC,CAAE,CAAC,IAAI,CAAC;QACrC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAC;IACnC,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/tools/init-constitution.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=init-constitution.test.d.ts.map

package/dist/tools/init-constitution.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"init-constitution.test.d.ts","sourceRoot":"","sources":["../../src/tools/init-constitution.test.ts"],"names":[],"mappings":""}

package/dist/tools/init-constitution.test.js

@@ -0,0 +1,398 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+vi.mock('../i18n/index.js', () => ({
+    ti: vi.fn((_key, params) => `translated:${JSON.stringify(params)}`),
+}));
+vi.mock('../storage/index.js', () => ({
+    knowledgeStore: {
+        getKnowledge: vi.fn(),
+        getConstitution: vi.fn(),
+        saveConstitution: vi.fn(),
+    },
+}));
+vi.mock('node:fs/promises', () => ({
+    readFile: vi.fn(),
+}));
+import { handleInitConstitution } from './init-constitution.js';
+import { knowledgeStore } from '../storage/index.js';
+import { readFile } from 'node:fs/promises';
+const makeKnowledge = (overrides = {}) => ({
+    projectPath: '/test/project',
+    language: 'TypeScript',
+    framework: 'Express',
+    packageManager: 'npm',
+    database: 'postgresql',
+    architecture: {
+        primary: 'layered',
+        secondary: [],
+        layers: [
+            { name: 'domain', neverDependsOn: ['infrastructure'], responsibility: 'Business logic' },
+        ],
+        boundaries: [],
+        communicationPatterns: [],
+        deploymentUnits: [],
+    },
+    linting: {
+        maturityScore: 80,
+        detectedLinters: [{ tool: 'eslint', status: 'configured' }],
+        rulesConflicts: [],
+    },
+    ...overrides,
+});
+beforeEach(() => {
+    vi.clearAllMocks();
+    vi.mocked(readFile).mockRejectedValue(new Error('not found'));
+});
+describe('handleInitConstitution', () => {
+    it('should return error when project not found', async () => {
+        vi.mocked(knowledgeStore.getKnowledge).mockResolvedValue(null);
+        const result = await handleInitConstitution({ projectId: 'proj-123' });
+        expect(result.isError).toBe(true);
+        expect(result.content).toHaveLength(1);
+    });
+    it('should create constitution with auto-detected principles', async () => {
+        vi.mocked(knowledgeStore.getKnowledge).mockResolvedValue(makeKnowledge());
+        vi.mocked(knowledgeStore.getConstitution).mockResolvedValue(null);
+        vi.mocked(knowledgeStore.saveConstitution).mockResolvedValue(undefined);
+        const result = await handleInitConstitution({ projectId: 'proj-123' });
+        expect(result.isError).toBeUndefined();
+        const parsed = JSON.parse(result.content[0].text);
+        expect(parsed.projectId).toBe('proj-123');
+        expect(parsed.version).toBe(1);
+        expect(parsed.isUpdate).toBe(false);
+        expect(parsed.principlesCount).toBeGreaterThan(0);
+        expect(parsed.bySource['auto-detected']).toBeGreaterThan(0);
+        expect(knowledgeStore.saveConstitution).toHaveBeenCalled();
+    });
+    it('should increment version on existing constitution', async () => {
+        vi.mocked(knowledgeStore.getKnowledge).mockResolvedValue(makeKnowledge());
+        vi.mocked(knowledgeStore.getConstitution).mockResolvedValue({
+            projectId: 'proj-123',
+            principles: [
+                {
+                    id: 'auto-1',
+                    category: 'security',
+                    principle: 'Immutable principle',
+                    rationale: 'test',
+                    enforceLevel: 'strict',
+                    autoEnforceable: true,
+                    source: 'auto-detected',
+                    immutable: true,
+                },
+            ],
+            createdAt: '2024-01-01',
+            updatedAt: '2024-01-01',
+            version: 2,
+        });
+        vi.mocked(knowledgeStore.saveConstitution).mockResolvedValue(undefined);
+        const result = await handleInitConstitution({ projectId: 'proj-123' });
+        const parsed = JSON.parse(result.content[0].text);
+        expect(parsed.version).toBe(3);
+        expect(parsed.isUpdate).toBe(true);
+    });
+    it('should add manual principles', async () => {
+        vi.mocked(knowledgeStore.getKnowledge).mockResolvedValue(makeKnowledge());
+        vi.mocked(knowledgeStore.getConstitution).mockResolvedValue(null);
+        vi.mocked(knowledgeStore.saveConstitution).mockResolvedValue(undefined);
+        const manualPrinciple = {
+            id: 'custom-1',
+            category: 'workflow',
+            principle: 'Always write tests first',
+            rationale: 'TDD practice',
+            enforceLevel: 'strict',
+            autoEnforceable: false,
+            source: 'user-defined',
+            immutable: false,
+        };
+        const result = await handleInitConstitution({
+            projectId: 'proj-123',
+            principles: [manualPrinciple],
+        });
+        const parsed = JSON.parse(result.content[0].text);
+        expect(parsed.principles.some((p) => p.id === 'custom-1')).toBe(true);
+    });
+    it('should skip auto-detect when disabled', async () => {
+        vi.mocked(knowledgeStore.getKnowledge).mockResolvedValue(makeKnowledge());
+        vi.mocked(knowledgeStore.getConstitution).mockResolvedValue(null);
+        vi.mocked(knowledgeStore.saveConstitution).mockResolvedValue(undefined);
+        const result = await handleInitConstitution({
+            projectId: 'proj-123',
+            autoDetect: false,
+            principles: [
+                {
+                    id: 'manual-1',
+                    category: 'workflow',
+                    principle: 'Manual only',
+                    rationale: 'test',
+                    enforceLevel: 'recommended',
+                    autoEnforceable: false,
+                    source: 'user-defined',
+                    immutable: false,
+                },
+            ],
+        });
+        const parsed = JSON.parse(result.content[0].text);
+        expect(parsed.bySource['auto-detected']).toBe(0);
+        expect(parsed.principlesCount).toBe(1);
+    });
+    it('should extract principles from CLAUDE.md', async () => {
+        vi.mocked(knowledgeStore.getKnowledge).mockResolvedValue(makeKnowledge());
+        vi.mocked(knowledgeStore.getConstitution).mockResolvedValue(null);
+        vi.mocked(knowledgeStore.saveConstitution).mockResolvedValue(undefined);
+        vi.mocked(readFile).mockImplementation(((path) => {
+            if (path.includes('CLAUDE.md')) {
+                return Promise.resolve('# Rules\n- Always use TypeScript strict mode for all projects\n- Run security checks before deployment\n');
+            }
+            return Promise.reject(new Error('not found'));
+        }));
+        const result = await handleInitConstitution({ projectId: 'proj-123' });
+        const parsed = JSON.parse(result.content[0].text);
+        expect(parsed.bySource['claude-md']).toBeGreaterThan(0);
+    });
+    it('should handle errors gracefully', async () => {
+        vi.mocked(knowledgeStore.getKnowledge).mockRejectedValue(new Error('DB error'));
+        const result = await handleInitConstitution({ projectId: 'proj-123' });
+        expect(result.isError).toBe(true);
+    });
+    it('should handle knowledge without database', async () => {
+        vi.mocked(knowledgeStore.getKnowledge).mockResolvedValue(makeKnowledge({ database: 'unknown', packageManager: null, framework: null }));
+        vi.mocked(knowledgeStore.getConstitution).mockResolvedValue(null);
+        vi.mocked(knowledgeStore.saveConstitution).mockResolvedValue(undefined);
+        const result = await handleInitConstitution({ projectId: 'proj-123' });
+        expect(result.isError).toBeUndefined();
+        const parsed = JSON.parse(result.content[0].text);
+        expect(parsed.principlesCount).toBeGreaterThan(0);
+    });
+    // --- Coverage for linting principles with no configured linters (line 117) ---
+    it('should not add linting principle when no linters are configured', async () => {
+        vi.mocked(knowledgeStore.getKnowledge).mockResolvedValue(makeKnowledge({
+            linting: {
+                maturityScore: 0,
+                detectedLinters: [{ tool: 'eslint', status: 'not-configured' }],
+                rulesConflicts: [],
+            },
+        }));
+        vi.mocked(knowledgeStore.getConstitution).mockResolvedValue(null);
+        vi.mocked(knowledgeStore.saveConstitution).mockResolvedValue(undefined);
+        const result = await handleInitConstitution({ projectId: 'proj-123' });
+        const parsed = JSON.parse(result.content[0].text);
+        const lintPrinciple = parsed.principles.find((p) => p.principle.includes('linting'));
+        expect(lintPrinciple).toBeUndefined();
+    });
+    // --- Coverage for classifyPrinciple: tech-stack category (line 225) ---
+    it('should classify stack/framework/language/database principles as tech-stack', async () => {
+        vi.mocked(knowledgeStore.getKnowledge).mockResolvedValue(makeKnowledge());
+        vi.mocked(knowledgeStore.getConstitution).mockResolvedValue(null);
+        vi.mocked(knowledgeStore.saveConstitution).mockResolvedValue(undefined);
+        vi.mocked(readFile).mockImplementation(((path) => {
+            if (path.includes('CLAUDE.md')) {
+                return Promise.resolve('# Rules\n- Use the approved tech stack for all features\n- The database must be PostgreSQL\n');
+            }
+            return Promise.reject(new Error('not found'));
+        }));
+        const result = await handleInitConstitution({ projectId: 'proj-123' });
+        const parsed = JSON.parse(result.content[0].text);
+        const techStackPrinciples = parsed.principles.filter((p) => p.category === 'tech-stack' && p.source === 'claude-md');
+        expect(techStackPrinciples.length).toBeGreaterThan(0);
+    });
+    // --- Coverage for classifyPrinciple: conventions category ---
+    it('should classify naming convention principles as conventions', async () => {
+        vi.mocked(knowledgeStore.getKnowledge).mockResolvedValue(makeKnowledge());
+        vi.mocked(knowledgeStore.getConstitution).mockResolvedValue(null);
+        vi.mocked(knowledgeStore.saveConstitution).mockResolvedValue(undefined);
+        vi.mocked(readFile).mockImplementation(((path) => {
+            if (path.includes('CLAUDE.md')) {
+                return Promise.resolve('# Rules\n- Use camelCase naming convention for all variables\n');
+            }
+            return Promise.reject(new Error('not found'));
+        }));
+        const result = await handleInitConstitution({ projectId: 'proj-123' });
+        const parsed = JSON.parse(result.content[0].text);
+        const conventionPrinciples = parsed.principles.filter((p) => p.category === 'conventions' && p.source === 'claude-md');
+        expect(conventionPrinciples.length).toBeGreaterThan(0);
+    });
+    // --- Coverage for classifyPrinciple: workflow category (default, line 227) ---
+    it('should classify unrecognized principles as workflow', async () => {
+        vi.mocked(knowledgeStore.getKnowledge).mockResolvedValue(makeKnowledge());
+        vi.mocked(knowledgeStore.getConstitution).mockResolvedValue(null);
+        vi.mocked(knowledgeStore.saveConstitution).mockResolvedValue(undefined);
+        vi.mocked(readFile).mockImplementation(((path) => {
+            if (path.includes('CLAUDE.md')) {
+                return Promise.resolve('# Rules\n- Always review pull requests before merging them\n');
+            }
+            return Promise.reject(new Error('not found'));
+        }));
+        const result = await handleInitConstitution({ projectId: 'proj-123' });
+        const parsed = JSON.parse(result.content[0].text);
+        const workflowPrinciples = parsed.principles.filter((p) => p.category === 'workflow' && p.source === 'claude-md');
+        expect(workflowPrinciples.length).toBeGreaterThan(0);
+    });
+    // --- Coverage for manual principle overriding auto-detected (line 263) ---
+    it('should override auto-detected principle with same ID as manual principle', async () => {
+        vi.mocked(knowledgeStore.getKnowledge).mockResolvedValue(makeKnowledge());
+        vi.mocked(knowledgeStore.getConstitution).mockResolvedValue(null);
+        vi.mocked(knowledgeStore.saveConstitution).mockResolvedValue(undefined);
+        const result = await handleInitConstitution({
+            projectId: 'proj-123',
+            principles: [
+                {
+                    id: 'auto-1', // same ID as first auto-detected principle
+                    category: 'architecture',
+                    principle: 'Custom architecture rule',
+                    rationale: 'Override',
+                    enforceLevel: 'recommended',
+                    autoEnforceable: false,
+                    source: 'user-defined',
+                    immutable: false,
+                },
+            ],
+        });
+        const parsed = JSON.parse(result.content[0].text);
+        const overriddenPrinciple = parsed.principles.find((p) => p.id === 'auto-1');
+        expect(overriddenPrinciple.principle).toBe('Custom architecture rule');
+        expect(overriddenPrinciple.source).toBe('user-defined');
+    });
+    // --- Coverage for immutable principle preservation: existing immutable not in new (line 279) ---
+    it('should preserve immutable principles from existing constitution even when not auto-detected', async () => {
+        vi.mocked(knowledgeStore.getKnowledge).mockResolvedValue(makeKnowledge());
+        vi.mocked(knowledgeStore.getConstitution).mockResolvedValue({
+            projectId: 'proj-123',
+            principles: [
+                {
+                    id: 'custom-immutable',
+                    category: 'security',
+                    principle: 'Never bypass auth checks',
+                    rationale: 'Critical security',
+                    enforceLevel: 'strict',
+                    autoEnforceable: false,
+                    source: 'user-defined',
+                    immutable: true,
+                },
+            ],
+            createdAt: '2024-01-01',
+            updatedAt: '2024-01-01',
+            version: 1,
+        });
+        vi.mocked(knowledgeStore.saveConstitution).mockResolvedValue(undefined);
+        const result = await handleInitConstitution({ projectId: 'proj-123' });
+        const parsed = JSON.parse(result.content[0].text);
+        const immutablePrinciple = parsed.principles.find((p) => p.id === 'custom-immutable');
+        // Should be added back even though it was not in the auto-detected set
+        expect(immutablePrinciple).toBeDefined();
+        expect(immutablePrinciple.principle).toBe('Never bypass auth checks');
+    });
+    // --- Coverage for layers without neverDependsOn ---
+    it('should not add layer dependency principle when neverDependsOn is empty', async () => {
+        vi.mocked(knowledgeStore.getKnowledge).mockResolvedValue(makeKnowledge({
+            architecture: {
+                primary: 'layered',
+                secondary: [],
+                layers: [
+                    { name: 'domain', neverDependsOn: [], responsibility: 'Business logic' },
+                ],
+            },
+        }));
+        vi.mocked(knowledgeStore.getConstitution).mockResolvedValue(null);
+        vi.mocked(knowledgeStore.saveConstitution).mockResolvedValue(undefined);
+        const result = await handleInitConstitution({ projectId: 'proj-123' });
+        const parsed = JSON.parse(result.content[0].text);
+        const layerPrinciples = parsed.principles.filter((p) => p.principle.includes('must never depend on'));
+        expect(layerPrinciples).toHaveLength(0);
+    });
+    // --- Coverage for extractPrinciplesFromClaudeMd: .cursorrules file ---
+    it('should extract principles from .cursorrules file', async () => {
+        vi.mocked(knowledgeStore.getKnowledge).mockResolvedValue(makeKnowledge());
+        vi.mocked(knowledgeStore.getConstitution).mockResolvedValue(null);
+        vi.mocked(knowledgeStore.saveConstitution).mockResolvedValue(undefined);
+        vi.mocked(readFile).mockImplementation(((path) => {
+            if (path.includes('.cursorrules')) {
+                return Promise.resolve('# Cursor Rules\n- Always test before committing code changes\n');
+            }
+            return Promise.reject(new Error('not found'));
+        }));
+        const result = await handleInitConstitution({ projectId: 'proj-123' });
+        const parsed = JSON.parse(result.content[0].text);
+        expect(parsed.bySource['claude-md']).toBeGreaterThan(0);
+    });
+    // --- Coverage for classifyPrinciple: security ---
+    it('should classify security-related principles from CLAUDE.md', async () => {
+        vi.mocked(knowledgeStore.getKnowledge).mockResolvedValue(makeKnowledge());
+        vi.mocked(knowledgeStore.getConstitution).mockResolvedValue(null);
+        vi.mocked(knowledgeStore.saveConstitution).mockResolvedValue(undefined);
+        vi.mocked(readFile).mockImplementation(((path) => {
+            if (path.includes('CLAUDE.md')) {
+                return Promise.resolve('# Security\n- Always encrypt sensitive user data at rest\n');
+            }
+            return Promise.reject(new Error('not found'));
+        }));
+        const result = await handleInitConstitution({ projectId: 'proj-123' });
+        const parsed = JSON.parse(result.content[0].text);
+        const securityPrinciples = parsed.principles.filter((p) => p.category === 'security' && p.source === 'claude-md');
+        expect(securityPrinciples.length).toBeGreaterThan(0);
+    });
+    // --- Coverage for classifyPrinciple: architecture ---
+    it('should classify architecture-related principles from CLAUDE.md', async () => {
+        vi.mocked(knowledgeStore.getKnowledge).mockResolvedValue(makeKnowledge());
+        vi.mocked(knowledgeStore.getConstitution).mockResolvedValue(null);
+        vi.mocked(knowledgeStore.saveConstitution).mockResolvedValue(undefined);
+        vi.mocked(readFile).mockImplementation(((path) => {
+            if (path.includes('CLAUDE.md')) {
+                return Promise.resolve('# Arch\n- Follow the layered architecture pattern strictly\n');
+            }
+            return Promise.reject(new Error('not found'));
+        }));
+        const result = await handleInitConstitution({ projectId: 'proj-123' });
+        const parsed = JSON.parse(result.content[0].text);
+        const archPrinciples = parsed.principles.filter((p) => p.category === 'architecture' && p.source === 'claude-md');
+        expect(archPrinciples.length).toBeGreaterThan(0);
+    });
+    // --- Coverage for classifyPrinciple: quality ---
+    it('should classify quality/test-related principles from CLAUDE.md', async () => {
+        vi.mocked(knowledgeStore.getKnowledge).mockResolvedValue(makeKnowledge());
+        vi.mocked(knowledgeStore.getConstitution).mockResolvedValue(null);
+        vi.mocked(knowledgeStore.saveConstitution).mockResolvedValue(undefined);
+        vi.mocked(readFile).mockImplementation(((path) => {
+            if (path.includes('CLAUDE.md')) {
+                return Promise.resolve('# Quality\n- All code must have comprehensive test coverage\n');
+            }
+            return Promise.reject(new Error('not found'));
+        }));
+        const result = await handleInitConstitution({ projectId: 'proj-123' });
+        const parsed = JSON.parse(result.content[0].text);
+        const qualityPrinciples = parsed.principles.filter((p) => p.category === 'quality' && p.source === 'claude-md');
+        expect(qualityPrinciples.length).toBeGreaterThan(0);
+    });
+    // --- Coverage for non-Error thrown in catch block ---
+    it('should handle non-Error thrown values in catch block', async () => {
+        vi.mocked(knowledgeStore.getKnowledge).mockRejectedValue('string error');
+        const result = await handleInitConstitution({ projectId: 'proj-123' });
+        expect(result.isError).toBe(true);
+        expect(result.content[0].text).toContain('string error');
+    });
+    // --- Coverage for empty layers array ---
+    it('should handle empty layers array in architecture', async () => {
+        vi.mocked(knowledgeStore.getKnowledge).mockResolvedValue(makeKnowledge({
+            architecture: {
+                primary: 'monolith',
+                secondary: [],
+                layers: [],
+            },
+        }));
+        vi.mocked(knowledgeStore.getConstitution).mockResolvedValue(null);
+        vi.mocked(knowledgeStore.saveConstitution).mockResolvedValue(undefined);
+        const result = await handleInitConstitution({ projectId: 'proj-123' });
+        expect(result.isError).toBeUndefined();
+        const parsed = JSON.parse(result.content[0].text);
+        expect(parsed.principlesCount).toBeGreaterThan(0);
+    });
+    // --- Coverage for no language set ---
+    it('should skip tech-stack language principle when language is empty', async () => {
+        vi.mocked(knowledgeStore.getKnowledge).mockResolvedValue(makeKnowledge({ language: '' }));
+        vi.mocked(knowledgeStore.getConstitution).mockResolvedValue(null);
+        vi.mocked(knowledgeStore.saveConstitution).mockResolvedValue(undefined);
+        const result = await handleInitConstitution({ projectId: 'proj-123' });
+        const parsed = JSON.parse(result.content[0].text);
+        const langPrinciple = parsed.principles.find((p) => p.source === 'auto-detected' && p.principle.startsWith('Use ') && p.principle.includes('for all new code'));
+        expect(langPrinciple).toBeUndefined();
+    });
+});
+//# sourceMappingURL=init-constitution.test.js.map