reviewflow 3.7.0 → 3.8.0

package/dist/tests/units/usecases/insights/buildAiInsightsPrompt.test.js

@@ -0,0 +1,172 @@
+import { describe, it, expect } from 'vitest';
+import { buildAiInsightsPrompt } from '../../../../usecases/insights/buildAiInsightsPrompt.js';
+import { ReviewStatsFactory } from '../../../../tests/factories/projectStats.factory.js';
+import { TrackedMrFactory } from '../../../../tests/factories/trackedMr.factory.js';
+function createReviewContent(mrNumber, score) {
+    return `# Code Review - MR !${mrNumber} (feature: something)
+
+## Executive Summary
+
+| Audit | Score |
+|-------|-------|
+| Clean Architecture | ${score}/10 |
+
+**Score Global : ${score}/10** -- Good review.
+
+## Corrections Bloquantes
+
+### 1. Missing error handling
+
+Some blocking issue description.
+
+## Constats Positifs
+
+| Aspect | Note |
+|--------|------|
+| Testing | 9/10 |
+`;
+}
+describe('buildAiInsightsPrompt', () => {
+    it('should include developer name in the prompt', () => {
+        const reviews = [
+            ReviewStatsFactory.create({ assignedBy: 'alice', mrNumber: 1, score: 8 }),
+        ];
+        const prompt = buildAiInsightsPrompt({
+            reviews,
+            reviewContents: new Map(),
+            trackedMrs: [],
+            language: 'fr',
+        });
+        expect(prompt).toContain('alice');
+    });
+    it('should include average score for each developer', () => {
+        const reviews = [
+            ReviewStatsFactory.create({ id: 'r1', assignedBy: 'alice', mrNumber: 1, score: 8 }),
+            ReviewStatsFactory.create({ id: 'r2', assignedBy: 'alice', mrNumber: 2, score: 6 }),
+        ];
+        const prompt = buildAiInsightsPrompt({
+            reviews,
+            reviewContents: new Map(),
+            trackedMrs: [],
+            language: 'fr',
+        });
+        expect(prompt).toContain('7');
+    });
+    it('should include review content for recent reviews', () => {
+        const reviews = [
+            ReviewStatsFactory.create({ id: 'r1', assignedBy: 'alice', mrNumber: 42, score: 8 }),
+        ];
+        const reviewContents = new Map();
+        reviewContents.set('42', createReviewContent(42, 8));
+        const prompt = buildAiInsightsPrompt({
+            reviews,
+            reviewContents,
+            trackedMrs: [],
+            language: 'fr',
+        });
+        expect(prompt).toContain('Missing error handling');
+    });
+    it('should include tracked MR lifecycle data', () => {
+        const reviews = [
+            ReviewStatsFactory.create({ assignedBy: 'alice', mrNumber: 1, score: 8 }),
+        ];
+        const trackedMrs = [
+            TrackedMrFactory.create({
+                assignment: { username: 'alice', assignedAt: '2024-01-15T10:00:00Z' },
+                totalReviews: 3,
+                totalFollowups: 1,
+            }),
+        ];
+        const prompt = buildAiInsightsPrompt({
+            reviews,
+            reviewContents: new Map(),
+            trackedMrs,
+            language: 'fr',
+        });
+        expect(prompt).toContain('alice');
+        expect(prompt).toMatch(/total.*reviews|reviews.*total/i);
+    });
+    it('should specify the output language', () => {
+        const reviews = [
+            ReviewStatsFactory.create({ assignedBy: 'alice', mrNumber: 1, score: 8 }),
+        ];
+        const promptFr = buildAiInsightsPrompt({
+            reviews,
+            reviewContents: new Map(),
+            trackedMrs: [],
+            language: 'fr',
+        });
+        expect(promptFr).toContain('French');
+        const promptEn = buildAiInsightsPrompt({
+            reviews,
+            reviewContents: new Map(),
+            trackedMrs: [],
+            language: 'en',
+        });
+        expect(promptEn).toContain('English');
+    });
+    it('should request JSON output format', () => {
+        const reviews = [
+            ReviewStatsFactory.create({ assignedBy: 'alice', mrNumber: 1, score: 8 }),
+        ];
+        const prompt = buildAiInsightsPrompt({
+            reviews,
+            reviewContents: new Map(),
+            trackedMrs: [],
+            language: 'fr',
+        });
+        expect(prompt).toContain('JSON');
+        expect(prompt).toContain('developerName');
+        expect(prompt).toContain('titleExplanation');
+    });
+    it('should handle multiple developers', () => {
+        const reviews = [
+            ReviewStatsFactory.create({ id: 'r1', assignedBy: 'alice', mrNumber: 1, score: 8 }),
+            ReviewStatsFactory.create({ id: 'r2', assignedBy: 'bob', mrNumber: 2, score: 6 }),
+        ];
+        const prompt = buildAiInsightsPrompt({
+            reviews,
+            reviewContents: new Map(),
+            trackedMrs: [],
+            language: 'fr',
+        });
+        expect(prompt).toContain('alice');
+        expect(prompt).toContain('bob');
+    });
+    it('should skip reviews without assignedBy', () => {
+        const reviews = [
+            ReviewStatsFactory.create({ id: 'r1', assignedBy: undefined, mrNumber: 1, score: 8 }),
+            ReviewStatsFactory.create({ id: 'r2', assignedBy: 'alice', mrNumber: 2, score: 7 }),
+        ];
+        const prompt = buildAiInsightsPrompt({
+            reviews,
+            reviewContents: new Map(),
+            trackedMrs: [],
+            language: 'fr',
+        });
+        expect(prompt).toContain('alice');
+        expect(prompt).not.toContain('Developer: unknown');
+    });
+    it('should truncate review content for older reviews', () => {
+        const reviews = Array.from({ length: 10 }, (_, index) => ReviewStatsFactory.create({
+            id: `r${index}`,
+            assignedBy: 'alice',
+            mrNumber: index + 1,
+            score: 7,
+            timestamp: new Date(2026, 0, index + 1).toISOString(),
+        }));
+        const reviewContents = new Map();
+        for (let index = 1; index <= 10; index++) {
+            reviewContents.set(String(index), createReviewContent(index, 7));
+        }
+        const prompt = buildAiInsightsPrompt({
+            reviews,
+            reviewContents,
+            trackedMrs: [],
+            language: 'fr',
+        });
+        expect(prompt.length).toBeGreaterThan(0);
+        expect(prompt.length).toBeLessThan(200000);
+    });
+});
+//# sourceMappingURL=buildAiInsightsPrompt.test.js.map

package/dist/tests/units/usecases/insights/buildAiInsightsPrompt.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"buildAiInsightsPrompt.test.js","sourceRoot":"","sources":["../../../../../src/tests/units/usecases/insights/buildAiInsightsPrompt.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,qBAAqB,EAAE,MAAM,8CAA8C,CAAC;AACrF,OAAO,EAAE,kBAAkB,EAAE,MAAM,2CAA2C,CAAC;AAG/E,OAAO,EAAE,gBAAgB,EAAE,MAAM,wCAAwC,CAAC;AAE1E,SAAS,mBAAmB,CAAC,QAAgB,EAAE,KAAa;IAC1D,OAAO,uBAAuB,QAAQ;;;;;;yBAMf,KAAK;;mBAEX,KAAK;;;;;;;;;;;;;CAavB,CAAC;AACF,CAAC;AAED,QAAQ,CAAC,uBAAuB,EAAE,GAAG,EAAE;IACrC,EAAE,CAAC,6CAA6C,EAAE,GAAG,EAAE;QACrD,MAAM,OAAO,GAAkB;YAC7B,kBAAkB,CAAC,MAAM,CAAC,EAAE,UAAU,EAAE,OAAO,EAAE,QAAQ,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC;SAC1E,CAAC;QAEF,MAAM,MAAM,GAAG,qBAAqB,CAAC;YACnC,OAAO;YACP,cAAc,EAAE,IAAI,GAAG,EAAE;YACzB,UAAU,EAAE,EAAE;YACd,QAAQ,EAAE,IAAI;SACf,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,OAAO,CAAC,CAAC;IACpC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iDAAiD,EAAE,GAAG,EAAE;QACzD,MAAM,OAAO,GAAkB;YAC7B,kBAAkB,CAAC,MAAM,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE,OAAO,EAAE,QAAQ,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC;YACnF,kBAAkB,CAAC,MAAM,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE,OAAO,EAAE,QAAQ,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC;SACpF,CAAC;QAEF,MAAM,MAAM,GAAG,qBAAqB,CAAC;YACnC,OAAO;YACP,cAAc,EAAE,IAAI,GAAG,EAAE;YACzB,UAAU,EAAE,EAAE;YACd,QAAQ,EAAE,IAAI;SACf,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,GAAG,CAAC,CAAC;IAChC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;QAC1D,MAAM,OAAO,GAAkB;YAC7B,kBAAkB,CAAC,MAAM,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC;SACrF,CAAC;QAEF,MAAM,cAAc,GAAG,IAAI,GAAG,EAAkB,CAAC;QACjD,cAAc,CAAC,GAAG,CAAC,IAAI,EAAE,mBAAmB,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC;QAErD,MAAM,MAAM,GAAG,qBAAqB,CAAC;YACnC,OAAO;YACP,cAAc;YACd,UAAU,EAAE,EAAE;YACd,QAAQ,EAAE,IAAI;SACf,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,wBAAwB,CAAC,CAAC;IACrD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0CAA0C,EAAE,GAAG,EAAE;QAClD,MAAM,OAAO,GAAkB;YAC7B,kBAAkB,CAAC,MAAM,CAAC,EAAE,UAAU,EAAE,OAAO,EAAE,QAAQ,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC;SAC1E,CAAC;QAEF,MAAM,UAAU,GAAgB;YAC9B,gBAAgB,CAAC,MAAM,CAAC;gBACtB,UAAU,EAAE,EAAE,QAAQ,EAAE,OAAO,EAAE,UAAU,EAAE,sBAAsB,EAAE;gBACrE,YAAY,EAAE,CAAC;gBACf,cAAc,EAAE,CAAC;aAClB,CAAC;SACH,CAAC;QAEF,MAAM,MAAM,GAAG,qBAAqB,CAAC;YACnC,OAAO;YACP,cAAc,EAAE,IAAI,GAAG,EAAE;YACzB,UAAU;YACV,QAAQ,EAAE,IAAI;SACf,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,OAAO,CAAC,CAAC;QAClC,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,gCAAgC,CAAC,CAAC;IAC3D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oCAAoC,EAAE,GAAG,EAAE;QAC5C,MAAM,OAAO,GAAkB;YAC7B,kBAAkB,CAAC,MAAM,CAAC,EAAE,UAAU,EAAE,OAAO,EAAE,QAAQ,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC;SAC1E,CAAC;QAEF,MAAM,QAAQ,GAAG,qBAAqB,CAAC;YACrC,OAAO;YACP,cAAc,EAAE,IAAI,GAAG,EAAE;YACzB,UAAU,EAAE,EAAE;YACd,QAAQ,EAAE,IAAI;SACf,CAAC,CAAC;QAEH,MAAM,CAAC,QAAQ,CAAC,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAC;QAErC,MAAM,QAAQ,GAAG,qBAAqB,CAAC;YACrC,OAAO;YACP,cAAc,EAAE,IAAI,GAAG,EAAE;YACzB,UAAU,EAAE,EAAE;YACd,QAAQ,EAAE,IAAI;SACf,CAAC,CAAC;QAEH,MAAM,CAAC,QAAQ,CAAC,CAAC,SAAS,CAAC,SAAS,CAAC,CAAC;IACxC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mCAAmC,EAAE,GAAG,EAAE;QAC3C,MAAM,OAAO,GAAkB;YAC7B,kBAAkB,CAAC,MAAM,CAAC,EAAE,UAAU,EAAE,OAAO,EAAE,QAAQ,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC;SAC1E,CAAC;QAEF,MAAM,MAAM,GAAG,qBAAqB,CAAC;YACnC,OAAO;YACP,cAAc,EAAE,IAAI,GAAG,EAAE;YACzB,UAAU,EAAE,EAAE;YACd,QAAQ,EAAE,IAAI;SACf,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC;QACjC,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,eAAe,CAAC,CAAC;QAC1C,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,kBAAkB,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mCAAmC,EAAE,GAAG,EAAE;QAC3C,MAAM,OAAO,GAAkB;YAC7B,kBAAkB,CAAC,MAAM,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE,OAAO,EAAE,QAAQ,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC;YACnF,kBAAkB,CAAC,MAAM,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE,KAAK,EAAE,QAAQ,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC;SAClF,CAAC;QAEF,MAAM,MAAM,GAAG,qBAAqB,CAAC;YACnC,OAAO;YACP,cAAc,EAAE,IAAI,GAAG,EAAE;YACzB,UAAU,EAAE,EAAE;YACd,QAAQ,EAAE,IAAI;SACf,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,OAAO,CAAC,CAAC;QAClC,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,KAAK,CAAC,CAAC;IAClC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wCAAwC,EAAE,GAAG,EAAE;QAChD,MAAM,OAAO,GAAkB;YAC7B,kBAAkB,CAAC,MAAM,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE,SAAS,EAAE,QAAQ,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC;YACrF,kBAAkB,CAAC,MAAM,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE,OAAO,EAAE,QAAQ,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC;SACpF,CAAC;QAEF,MAAM,MAAM,GAAG,qBAAqB,CAAC;YACnC,OAAO;YACP,cAAc,EAAE,IAAI,GAAG,EAAE;YACzB,UAAU,EAAE,EAAE;YACd,QAAQ,EAAE,IAAI;SACf,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,OAAO,CAAC,CAAC;QAClC,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,oBAAoB,CAAC,CAAC;IACrD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;QAC1D,MAAM,OAAO,GAAkB,KAAK,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,CAAC,CAAC,EAAE,KAAK,EAAE,EAAE,CACrE,kBAAkB,CAAC,MAAM,CAAC;YACxB,EAAE,EAAE,IAAI,KAAK,EAAE;YACf,UAAU,EAAE,OAAO;YACnB,QAAQ,EAAE,KAAK,GAAG,CAAC;YACnB,KAAK,EAAE,CAAC;YACR,SAAS,EAAE,IAAI,IAAI,CAAC,IAAI,EAAE,CAAC,EAAE,KAAK,GAAG,CAAC,CAAC,CAAC,WAAW,EAAE;SACtD,CAAC,CACH,CAAC;QAEF,MAAM,cAAc,GAAG,IAAI,GAAG,EAAkB,CAAC;QACjD,KAAK,IAAI,KAAK,GAAG,CAAC,EAAE,KAAK,IAAI,EAAE,EAAE,KAAK,EAAE,EAAE,CAAC;YACzC,cAAc,CAAC,GAAG,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE,mBAAmB,CAAC,KAAK,EAAE,CAAC,CAAC,CAAC,CAAC;QACnE,CAAC;QAED,MAAM,MAAM,GAAG,qBAAqB,CAAC;YACnC,OAAO;YACP,cAAc;YACd,UAAU,EAAE,EAAE;YACd,QAAQ,EAAE,IAAI;SACf,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC;QACzC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,MAAM,CAAC,CAAC;IAC7C,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/tests/units/usecases/insights/computeDeveloperInsights.usecase.test.d.ts

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

package/dist/tests/units/usecases/insights/computeDeveloperInsights.usecase.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"computeDeveloperInsights.usecase.test.d.ts","sourceRoot":"","sources":["../../../../../src/tests/units/usecases/insights/computeDeveloperInsights.usecase.test.ts"],"names":[],"mappings":""}

package/dist/tests/units/usecases/insights/computeDeveloperInsights.usecase.test.js

@@ -0,0 +1,449 @@
+import { describe, it, expect } from 'vitest';
+import { computeDeveloperInsights } from '../../../../usecases/insights/computeDeveloperInsights.usecase.js';
+import { ReviewStatsFactory } from '../../../../tests/factories/projectStats.factory.js';
+function createReviewsForDeveloper(developerName, count, overrides = {}) {
+    return Array.from({ length: count }, (_, index) => ReviewStatsFactory.create({
+        id: `${developerName}-${index}`,
+        assignedBy: developerName,
+        mrNumber: index + 1,
+        ...overrides,
+    }));
+}
+describe('computeDeveloperInsights', () => {
+    describe('grouping by developer', () => {
+        it('should return empty array when no reviews are provided', () => {
+            const result = computeDeveloperInsights([]);
+            expect(result).toEqual([]);
+        });
+        it('should skip reviews without assignedBy', () => {
+            const reviews = [
+                ReviewStatsFactory.create({ assignedBy: undefined }),
+                ReviewStatsFactory.create({ assignedBy: undefined }),
+            ];
+            const result = computeDeveloperInsights(reviews);
+            expect(result).toEqual([]);
+        });
+        it('should skip developers with fewer than 5 reviews', () => {
+            const reviews = createReviewsForDeveloper('alice', 4);
+            const result = computeDeveloperInsights(reviews);
+            expect(result).toEqual([]);
+        });
+        it('should include developers with exactly 5 reviews', () => {
+            const reviews = createReviewsForDeveloper('alice', 5, { score: 7 });
+            const result = computeDeveloperInsights(reviews);
+            expect(result).toHaveLength(1);
+            expect(result[0].developerName).toBe('alice');
+            expect(result[0].reviewCount).toBe(5);
+        });
+        it('should group reviews by developer name', () => {
+            const aliceReviews = createReviewsForDeveloper('alice', 6, { score: 7 });
+            const bobReviews = createReviewsForDeveloper('bob', 5, { score: 8 });
+            const reviews = [...aliceReviews, ...bobReviews];
+            const result = computeDeveloperInsights(reviews);
+            expect(result).toHaveLength(2);
+            const names = result.map((insight) => insight.developerName);
+            expect(names).toContain('alice');
+            expect(names).toContain('bob');
+        });
+    });
+    describe('category level computation', () => {
+        it('should compute quality level based on score relative to team average', () => {
+            const aliceReviews = createReviewsForDeveloper('alice', 5, {
+                score: 9,
+                blocking: 0,
+                warnings: 0,
+            });
+            const bobReviews = createReviewsForDeveloper('bob', 5, {
+                score: 5,
+                blocking: 3,
+                warnings: 5,
+            });
+            const reviews = [...aliceReviews, ...bobReviews];
+            const result = computeDeveloperInsights(reviews);
+            const alice = result.find((insight) => insight.developerName === 'alice');
+            const bob = result.find((insight) => insight.developerName === 'bob');
+            expect(alice).toBeDefined();
+            expect(bob).toBeDefined();
+            expect(alice?.categoryLevels.quality.level).toBeGreaterThan(bob?.categoryLevels.quality.level ?? 10);
+        });
+        it('should compute responsiveness level based on review duration', () => {
+            const fastReviews = createReviewsForDeveloper('alice', 5, {
+                duration: 30000,
+                score: 7,
+            });
+            const slowReviews = createReviewsForDeveloper('bob', 5, {
+                duration: 300000,
+                score: 7,
+            });
+            const reviews = [...fastReviews, ...slowReviews];
+            const result = computeDeveloperInsights(reviews);
+            const alice = result.find((insight) => insight.developerName === 'alice');
+            const bob = result.find((insight) => insight.developerName === 'bob');
+            expect(alice?.categoryLevels.responsiveness.level).toBeGreaterThan(bob?.categoryLevels.responsiveness.level ?? 10);
+        });
+        it('should compute code volume level using diff stats', () => {
+            const largeVolume = createReviewsForDeveloper('alice', 5, {
+                score: 8,
+                diffStats: { commitsCount: 5, additions: 500, deletions: 100 },
+            });
+            const smallVolume = createReviewsForDeveloper('bob', 5, {
+                score: 8,
+                diffStats: { commitsCount: 1, additions: 10, deletions: 5 },
+            });
+            const reviews = [...largeVolume, ...smallVolume];
+            const result = computeDeveloperInsights(reviews);
+            const alice = result.find((insight) => insight.developerName === 'alice');
+            const bob = result.find((insight) => insight.developerName === 'bob');
+            expect(alice).toBeDefined();
+            expect(bob).toBeDefined();
+            expect(alice?.categoryLevels.codeVolume.level).toBeGreaterThan(bob?.categoryLevels.codeVolume.level ?? 10);
+        });
+        it('should clamp levels between 1 and 10', () => {
+            const reviews = createReviewsForDeveloper('alice', 5, {
+                score: 10,
+                blocking: 0,
+                warnings: 0,
+                duration: 1000,
+            });
+            const result = computeDeveloperInsights(reviews);
+            const alice = result[0];
+            for (const category of Object.values(alice.categoryLevels)) {
+                expect(category.level).toBeGreaterThanOrEqual(1);
+                expect(category.level).toBeLessThanOrEqual(10);
+            }
+        });
+        it('should produce integer levels', () => {
+            const reviews = createReviewsForDeveloper('alice', 7, { score: 7.3 });
+            const result = computeDeveloperInsights(reviews);
+            const alice = result[0];
+            for (const category of Object.values(alice.categoryLevels)) {
+                expect(Number.isInteger(category.level)).toBe(true);
+            }
+        });
+    });
+    describe('trend computation', () => {
+        it('should detect improving trend when recent reviews are better', () => {
+            const olderReviews = Array.from({ length: 10 }, (_, index) => ReviewStatsFactory.create({
+                id: `alice-old-${index}`,
+                assignedBy: 'alice',
+                mrNumber: index + 1,
+                score: 5,
+                timestamp: new Date(2024, 0, index + 1).toISOString(),
+            }));
+            const recentReviews = Array.from({ length: 10 }, (_, index) => ReviewStatsFactory.create({
+                id: `alice-new-${index}`,
+                assignedBy: 'alice',
+                mrNumber: index + 11,
+                score: 9,
+                timestamp: new Date(2024, 6, index + 1).toISOString(),
+            }));
+            const reviews = [...olderReviews, ...recentReviews];
+            const result = computeDeveloperInsights(reviews);
+            const alice = result[0];
+            expect(alice.categoryLevels.quality.trend).toBe('improving');
+        });
+        it('should detect declining trend when recent reviews are worse', () => {
+            const olderReviews = Array.from({ length: 10 }, (_, index) => ReviewStatsFactory.create({
+                id: `alice-old-${index}`,
+                assignedBy: 'alice',
+                mrNumber: index + 1,
+                score: 9,
+                blocking: 0,
+                timestamp: new Date(2024, 0, index + 1).toISOString(),
+            }));
+            const recentReviews = Array.from({ length: 10 }, (_, index) => ReviewStatsFactory.create({
+                id: `alice-new-${index}`,
+                assignedBy: 'alice',
+                mrNumber: index + 11,
+                score: 4,
+                blocking: 5,
+                timestamp: new Date(2024, 6, index + 1).toISOString(),
+            }));
+            const reviews = [...olderReviews, ...recentReviews];
+            const result = computeDeveloperInsights(reviews);
+            const alice = result[0];
+            expect(alice.categoryLevels.quality.trend).toBe('declining');
+        });
+        it('should detect stable trend when scores are consistent', () => {
+            const reviews = Array.from({ length: 20 }, (_, index) => ReviewStatsFactory.create({
+                id: `alice-${index}`,
+                assignedBy: 'alice',
+                mrNumber: index + 1,
+                score: 7,
+                blocking: 1,
+                warnings: 2,
+                duration: 60000,
+                timestamp: new Date(2024, 0, index + 1).toISOString(),
+            }));
+            const result = computeDeveloperInsights(reviews);
+            const alice = result[0];
+            expect(alice.categoryLevels.quality.trend).toBe('stable');
+        });
+    });
+    describe('strengths and weaknesses', () => {
+        it('should identify strengths when level >= 7', () => {
+            const reviews = createReviewsForDeveloper('alice', 5, {
+                score: 10,
+                blocking: 0,
+                warnings: 0,
+                duration: 60000,
+            });
+            const result = computeDeveloperInsights(reviews);
+            const alice = result[0];
+            expect(alice.strengths.length).toBeGreaterThan(0);
+        });
+        it('should identify weaknesses when level <= 4', () => {
+            const aliceReviews = createReviewsForDeveloper('alice', 5, {
+                score: 2,
+                blocking: 10,
+                warnings: 15,
+                duration: 600000,
+            });
+            const bobReviews = createReviewsForDeveloper('bob', 5, {
+                score: 9,
+                blocking: 0,
+                warnings: 0,
+                duration: 10000,
+            });
+            const result = computeDeveloperInsights([...aliceReviews, ...bobReviews]);
+            const alice = result.find((insight) => insight.developerName === 'alice');
+            expect(alice?.weaknesses.length).toBeGreaterThan(0);
+        });
+    });
+    describe('top priority', () => {
+        it('should set top priority to lowest level category', () => {
+            const aliceReviews = createReviewsForDeveloper('alice', 5, {
+                score: 3,
+                blocking: 8,
+                warnings: 10,
+                duration: 10000,
+            });
+            const bobReviews = createReviewsForDeveloper('bob', 5, {
+                score: 9,
+                blocking: 0,
+                warnings: 0,
+                duration: 100000,
+            });
+            const result = computeDeveloperInsights([...aliceReviews, ...bobReviews]);
+            const alice = result.find((insight) => insight.developerName === 'alice');
+            expect(alice?.topPriority).not.toBeNull();
+        });
+        it('should set top priority to null when all levels are high', () => {
+            const reviews = createReviewsForDeveloper('alice', 5, {
+                score: 10,
+                blocking: 0,
+                warnings: 0,
+                duration: 10000,
+                diffStats: { commitsCount: 5, additions: 200, deletions: 50 },
+            });
+            const result = computeDeveloperInsights(reviews);
+            const alice = result[0];
+            if (alice.topPriority !== null) {
+                const priorityLevel = alice.categoryLevels[alice.topPriority].level;
+                expect(priorityLevel).toBeLessThanOrEqual(6);
+            }
+        });
+    });
+    describe('developer titles', () => {
+        it('should assign architect title when quality is dominant', () => {
+            const aliceReviews = createReviewsForDeveloper('alice', 10, {
+                score: 10,
+                blocking: 0,
+                warnings: 0,
+                duration: 300000,
+            });
+            const bobReviews = createReviewsForDeveloper('bob', 10, {
+                score: 3,
+                blocking: 5,
+                warnings: 8,
+                duration: 30000,
+            });
+            const result = computeDeveloperInsights([...aliceReviews, ...bobReviews]);
+            const alice = result.find((insight) => insight.developerName === 'alice');
+            expect(alice?.categoryLevels.quality.level).toBeGreaterThan(alice?.categoryLevels.responsiveness.level ?? 10);
+            expect(alice?.title).toBe('architect');
+        });
+        it('should assign firefighter title when responsiveness is dominant', () => {
+            const aliceReviews = createReviewsForDeveloper('alice', 10, {
+                score: 7,
+                blocking: 1,
+                warnings: 2,
+                duration: 5000,
+            });
+            const bobReviews = createReviewsForDeveloper('bob', 10, {
+                score: 7,
+                blocking: 1,
+                warnings: 2,
+                duration: 500000,
+            });
+            const result = computeDeveloperInsights([...aliceReviews, ...bobReviews]);
+            const alice = result.find((insight) => insight.developerName === 'alice');
+            expect(alice?.title).toBe('firefighter');
+        });
+        it('should assign polyvalent title when all categories are balanced', () => {
+            const reviews = Array.from({ length: 20 }, (_, index) => ReviewStatsFactory.create({
+                id: `alice-${index}`,
+                assignedBy: 'alice',
+                mrNumber: index + 1,
+                score: 7,
+                blocking: 1,
+                warnings: 1,
+                duration: 60000,
+                diffStats: { commitsCount: 3, additions: 100, deletions: 50 },
+            }));
+            const result = computeDeveloperInsights(reviews);
+            const alice = result[0];
+            const levels = Object.values(alice.categoryLevels).map((category) => category.level);
+            const maxLevel = Math.max(...levels);
+            const minLevel = Math.min(...levels);
+            if (maxLevel - minLevel <= 2) {
+                expect(alice.title).toBe('polyvalent');
+            }
+        });
+    });
+    describe('overall level', () => {
+        it('should compute overall level as weighted average of category levels', () => {
+            const reviews = createReviewsForDeveloper('alice', 5, { score: 7 });
+            const result = computeDeveloperInsights(reviews);
+            const alice = result[0];
+            expect(alice.overallLevel).toBeGreaterThanOrEqual(1);
+            expect(alice.overallLevel).toBeLessThanOrEqual(10);
+            expect(Number.isInteger(alice.overallLevel)).toBe(true);
+        });
+    });
+    describe('single developer team', () => {
+        it('should use absolute benchmarks instead of team average for single developer', () => {
+            const reviews = createReviewsForDeveloper('alice', 10, {
+                score: 8,
+                blocking: 0,
+                warnings: 1,
+                duration: 60000,
+            });
+            const result = computeDeveloperInsights(reviews);
+            expect(result).toHaveLength(1);
+            const alice = result[0];
+            expect(alice.categoryLevels.quality.level).toBeGreaterThanOrEqual(1);
+            expect(alice.categoryLevels.quality.level).toBeLessThanOrEqual(10);
+        });
+    });
+    describe('metrics population', () => {
+        it('should populate metrics with raw developer numbers', () => {
+            const reviews = createReviewsForDeveloper('alice', 6, {
+                score: 8,
+                blocking: 1,
+                warnings: 2,
+                duration: 60000,
+                diffStats: { commitsCount: 3, additions: 200, deletions: 50 },
+            });
+            const result = computeDeveloperInsights(reviews);
+            const alice = result[0];
+            expect(alice.metrics).toBeDefined();
+            expect(alice.metrics.averageScore).toBe(8);
+            expect(alice.metrics.averageBlocking).toBe(1);
+            expect(alice.metrics.averageWarnings).toBe(2);
+            expect(alice.metrics.averageDuration).toBe(60000);
+            expect(alice.metrics.averageAdditions).toBe(200);
+            expect(alice.metrics.averageDeletions).toBe(50);
+        });
+        it('should compute first review quality rate based on reviews with score >= 7', () => {
+            const goodReviews = Array.from({ length: 3 }, (_, index) => ReviewStatsFactory.create({
+                id: `alice-good-${index}`,
+                assignedBy: 'alice',
+                mrNumber: index + 1,
+                score: 8,
+                blocking: 0,
+            }));
+            const badReviews = Array.from({ length: 2 }, (_, index) => ReviewStatsFactory.create({
+                id: `alice-bad-${index}`,
+                assignedBy: 'alice',
+                mrNumber: index + 4,
+                score: 5,
+                blocking: 2,
+            }));
+            const result = computeDeveloperInsights([...goodReviews, ...badReviews]);
+            const alice = result[0];
+            expect(alice.metrics.firstReviewQualityRate).toBe(0.6);
+        });
+    });
+    describe('insight descriptions', () => {
+        it('should generate insight descriptions for strengths and weaknesses', () => {
+            const aliceReviews = createReviewsForDeveloper('alice', 10, {
+                score: 9,
+                blocking: 0,
+                warnings: 0,
+                duration: 30000,
+            });
+            const bobReviews = createReviewsForDeveloper('bob', 10, {
+                score: 4,
+                blocking: 3,
+                warnings: 5,
+                duration: 300000,
+            });
+            const result = computeDeveloperInsights([...aliceReviews, ...bobReviews]);
+            const alice = result.find((insight) => insight.developerName === 'alice');
+            expect(alice).toBeDefined();
+            expect(alice?.insightDescriptions.length).toBeGreaterThan(0);
+        });
+        it('should include category and type in each description', () => {
+            const aliceReviews = createReviewsForDeveloper('alice', 10, {
+                score: 9,
+                blocking: 0,
+                warnings: 0,
+                duration: 30000,
+            });
+            const bobReviews = createReviewsForDeveloper('bob', 10, {
+                score: 4,
+                blocking: 3,
+                warnings: 5,
+                duration: 300000,
+            });
+            const result = computeDeveloperInsights([...aliceReviews, ...bobReviews]);
+            const alice = result.find((insight) => insight.developerName === 'alice');
+            if (alice && alice.insightDescriptions.length > 0) {
+                for (const description of alice.insightDescriptions) {
+                    expect(['quality', 'responsiveness', 'codeVolume', 'iteration']).toContain(description.category);
+                    expect(['strength', 'weakness']).toContain(description.type);
+                    expect(description.descriptionKey).toBeTruthy();
+                }
+            }
+        });
+    });
+    describe('edge cases', () => {
+        it('should handle reviews with null scores', () => {
+            const reviews = createReviewsForDeveloper('alice', 5, { score: null });
+            const result = computeDeveloperInsights(reviews);
+            expect(result).toHaveLength(1);
+            const alice = result[0];
+            expect(alice.categoryLevels.quality.level).toBeGreaterThanOrEqual(1);
+        });
+        it('should handle reviews without diff stats', () => {
+            const reviews = createReviewsForDeveloper('alice', 5, {
+                score: 7,
+                diffStats: undefined,
+            });
+            const result = computeDeveloperInsights(reviews);
+            expect(result).toHaveLength(1);
+            const alice = result[0];
+            expect(alice.categoryLevels.codeVolume.level).toBeGreaterThanOrEqual(1);
+        });
+        it('should produce valid DeveloperInsight objects', () => {
+            const reviews = [
+                ...createReviewsForDeveloper('alice', 8, { score: 7, duration: 60000 }),
+                ...createReviewsForDeveloper('bob', 6, { score: 5, duration: 120000 }),
+            ];
+            const result = computeDeveloperInsights(reviews);
+            for (const insight of result) {
+                expect(insight.developerName).toBeTruthy();
+                expect(insight.overallLevel).toBeGreaterThanOrEqual(1);
+                expect(insight.overallLevel).toBeLessThanOrEqual(10);
+                expect(insight.reviewCount).toBeGreaterThanOrEqual(5);
+                for (const category of Object.values(insight.categoryLevels)) {
+                    expect(category.level).toBeGreaterThanOrEqual(1);
+                    expect(category.level).toBeLessThanOrEqual(10);
+                    expect(['improving', 'declining', 'stable']).toContain(category.trend);
+                }
+            }
+        });
+    });
+});
+//# sourceMappingURL=computeDeveloperInsights.usecase.test.js.map