reviewflow 3.0.0

package/dist/tests/units/services/contextActionsExecutor.test.js

@@ -0,0 +1,112 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { executeActionsFromContext } from '../../../services/contextActionsExecutor.js';
+describe('executeActionsFromContext', () => {
+    const mockLogger = {
+        info: vi.fn(),
+        warn: vi.fn(),
+        error: vi.fn(),
+        debug: vi.fn(),
+    };
+    const mockExecutor = vi.fn();
+    const baseContext = {
+        version: '1.0',
+        mergeRequestId: 'github-owner/repo-42',
+        platform: 'github',
+        projectPath: 'owner/repo',
+        mergeRequestNumber: 42,
+        createdAt: '2026-02-02T10:00:00Z',
+        threads: [],
+        actions: [],
+        progress: { phase: 'completed', currentStep: null },
+    };
+    beforeEach(() => {
+        vi.clearAllMocks();
+    });
+    it('should return empty result when no actions in context', async () => {
+        const context = { ...baseContext, actions: [] };
+        const result = await executeActionsFromContext(context, '/tmp/repo', mockLogger, mockExecutor);
+        expect(result.total).toBe(0);
+        expect(result.succeeded).toBe(0);
+        expect(mockExecutor).not.toHaveBeenCalled();
+    });
+    it('should execute THREAD_RESOLVE action via GitHub API', async () => {
+        const context = {
+            ...baseContext,
+            actions: [
+                { type: 'THREAD_RESOLVE', threadId: 'PRRT_kwDONxxx' },
+            ],
+        };
+        const result = await executeActionsFromContext(context, '/tmp/repo', mockLogger, mockExecutor);
+        expect(result.total).toBe(1);
+        expect(result.succeeded).toBe(1);
+        expect(mockExecutor).toHaveBeenCalledWith('gh', expect.arrayContaining(['api', 'graphql']), '/tmp/repo');
+    });
+    it('should execute POST_COMMENT action', async () => {
+        const context = {
+            ...baseContext,
+            actions: [
+                { type: 'POST_COMMENT', body: '## Follow-up Review\n\nAll fixed.' },
+            ],
+        };
+        const result = await executeActionsFromContext(context, '/tmp/repo', mockLogger, mockExecutor);
+        expect(result.total).toBe(1);
+        expect(result.succeeded).toBe(1);
+        expect(mockExecutor).toHaveBeenCalledWith('gh', expect.arrayContaining(['repos/owner/repo/issues/42/comments']), '/tmp/repo');
+    });
+    it('should execute ADD_LABEL action', async () => {
+        const context = {
+            ...baseContext,
+            actions: [
+                { type: 'ADD_LABEL', label: 'needs_approve' },
+            ],
+        };
+        const result = await executeActionsFromContext(context, '/tmp/repo', mockLogger, mockExecutor);
+        expect(result.total).toBe(1);
+        expect(result.succeeded).toBe(1);
+        expect(mockExecutor).toHaveBeenCalledWith('gh', expect.arrayContaining(['repos/owner/repo/issues/42/labels']), '/tmp/repo');
+    });
+    it('should execute multiple actions in order', async () => {
+        const context = {
+            ...baseContext,
+            actions: [
+                { type: 'THREAD_RESOLVE', threadId: 'thread-1' },
+                { type: 'THREAD_RESOLVE', threadId: 'thread-2' },
+                { type: 'POST_COMMENT', body: 'Done' },
+                { type: 'ADD_LABEL', label: 'approved' },
+            ],
+        };
+        const result = await executeActionsFromContext(context, '/tmp/repo', mockLogger, mockExecutor);
+        expect(result.total).toBe(4);
+        expect(result.succeeded).toBe(4);
+        expect(mockExecutor).toHaveBeenCalledTimes(4);
+    });
+    it('should handle GitLab platform', async () => {
+        const context = {
+            ...baseContext,
+            platform: 'gitlab',
+            actions: [
+                { type: 'THREAD_RESOLVE', threadId: 'abc123' },
+            ],
+        };
+        const result = await executeActionsFromContext(context, '/tmp/repo', mockLogger, mockExecutor);
+        expect(result.succeeded).toBe(1);
+        expect(mockExecutor).toHaveBeenCalledWith('glab', expect.arrayContaining(['api']), '/tmp/repo');
+    });
+    it('should continue executing when one action fails', async () => {
+        mockExecutor.mockImplementationOnce(() => {
+            throw new Error('API error');
+        });
+        const context = {
+            ...baseContext,
+            actions: [
+                { type: 'THREAD_RESOLVE', threadId: 'thread-1' },
+                { type: 'POST_COMMENT', body: 'Done' },
+            ],
+        };
+        const result = await executeActionsFromContext(context, '/tmp/repo', mockLogger, mockExecutor);
+        expect(result.total).toBe(2);
+        expect(result.failed).toBe(1);
+        expect(result.succeeded).toBe(1);
+    });
+});
+//# sourceMappingURL=contextActionsExecutor.test.js.map

package/dist/tests/units/services/contextActionsExecutor.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"contextActionsExecutor.test.js","sourceRoot":"","sources":["../../../../src/tests/units/services/contextActionsExecutor.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAA;AAC7D,OAAO,EAAE,yBAAyB,EAAE,MAAM,6CAA6C,CAAA;AAGvF,QAAQ,CAAC,2BAA2B,EAAE,GAAG,EAAE;IACzC,MAAM,UAAU,GAAG;QACjB,IAAI,EAAE,EAAE,CAAC,EAAE,EAAE;QACb,IAAI,EAAE,EAAE,CAAC,EAAE,EAAE;QACb,KAAK,EAAE,EAAE,CAAC,EAAE,EAAE;QACd,KAAK,EAAE,EAAE,CAAC,EAAE,EAAE;KACf,CAAA;IAED,MAAM,YAAY,GAAG,EAAE,CAAC,EAAE,EAAE,CAAA;IAE5B,MAAM,WAAW,GAAkB;QACjC,OAAO,EAAE,KAAK;QACd,cAAc,EAAE,sBAAsB;QACtC,QAAQ,EAAE,QAAQ;QAClB,WAAW,EAAE,YAAY;QACzB,kBAAkB,EAAE,EAAE;QACtB,SAAS,EAAE,sBAAsB;QACjC,OAAO,EAAE,EAAE;QACX,OAAO,EAAE,EAAE;QACX,QAAQ,EAAE,EAAE,KAAK,EAAE,WAAW,EAAE,WAAW,EAAE,IAAI,EAAE;KACpD,CAAA;IAED,UAAU,CAAC,GAAG,EAAE;QACd,EAAE,CAAC,aAAa,EAAE,CAAA;IACpB,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,uDAAuD,EAAE,KAAK,IAAI,EAAE;QACrE,MAAM,OAAO,GAAG,EAAE,GAAG,WAAW,EAAE,OAAO,EAAE,EAAE,EAAE,CAAA;QAE/C,MAAM,MAAM,GAAG,MAAM,yBAAyB,CAAC,OAAO,EAAE,WAAW,EAAE,UAAU,EAAE,YAAY,CAAC,CAAA;QAE9F,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAC5B,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAChC,MAAM,CAAC,YAAY,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAA;IAC7C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,qDAAqD,EAAE,KAAK,IAAI,EAAE;QACnE,MAAM,OAAO,GAAkB;YAC7B,GAAG,WAAW;YACd,OAAO,EAAE;gBACP,EAAE,IAAI,EAAE,gBAAgB,EAAE,QAAQ,EAAE,eAAe,EAAE;aACtD;SACF,CAAA;QAED,MAAM,MAAM,GAAG,MAAM,yBAAyB,CAAC,OAAO,EAAE,WAAW,EAAE,UAAU,EAAE,YAAY,CAAC,CAAA;QAE9F,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAC5B,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAChC,MAAM,CAAC,YAAY,CAAC,CAAC,oBAAoB,CACvC,IAAI,EACJ,MAAM,CAAC,eAAe,CAAC,CAAC,KAAK,EAAE,SAAS,CAAC,CAAC,EAC1C,WAAW,CACZ,CAAA;IACH,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,oCAAoC,EAAE,KAAK,IAAI,EAAE;QAClD,MAAM,OAAO,GAAkB;YAC7B,GAAG,WAAW;YACd,OAAO,EAAE;gBACP,EAAE,IAAI,EAAE,cAAc,EAAE,IAAI,EAAE,mCAAmC,EAAE;aACpE;SACF,CAAA;QAED,MAAM,MAAM,GAAG,MAAM,yBAAyB,CAAC,OAAO,EAAE,WAAW,EAAE,UAAU,EAAE,YAAY,CAAC,CAAA;QAE9F,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAC5B,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAChC,MAAM,CAAC,YAAY,CAAC,CAAC,oBAAoB,CACvC,IAAI,EACJ,MAAM,CAAC,eAAe,CAAC,CAAC,qCAAqC,CAAC,CAAC,EAC/D,WAAW,CACZ,CAAA;IACH,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,iCAAiC,EAAE,KAAK,IAAI,EAAE;QAC/C,MAAM,OAAO,GAAkB;YAC7B,GAAG,WAAW;YACd,OAAO,EAAE;gBACP,EAAE,IAAI,EAAE,WAAW,EAAE,KAAK,EAAE,eAAe,EAAE;aAC9C;SACF,CAAA;QAED,MAAM,MAAM,GAAG,MAAM,yBAAyB,CAAC,OAAO,EAAE,WAAW,EAAE,UAAU,EAAE,YAAY,CAAC,CAAA;QAE9F,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAC5B,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAChC,MAAM,CAAC,YAAY,CAAC,CAAC,oBAAoB,CACvC,IAAI,EACJ,MAAM,CAAC,eAAe,CAAC,CAAC,mCAAmC,CAAC,CAAC,EAC7D,WAAW,CACZ,CAAA;IACH,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,0CAA0C,EAAE,KAAK,IAAI,EAAE;QACxD,MAAM,OAAO,GAAkB;YAC7B,GAAG,WAAW;YACd,OAAO,EAAE;gBACP,EAAE,IAAI,EAAE,gBAAgB,EAAE,QAAQ,EAAE,UAAU,EAAE;gBAChD,EAAE,IAAI,EAAE,gBAAgB,EAAE,QAAQ,EAAE,UAAU,EAAE;gBAChD,EAAE,IAAI,EAAE,cAAc,EAAE,IAAI,EAAE,MAAM,EAAE;gBACtC,EAAE,IAAI,EAAE,WAAW,EAAE,KAAK,EAAE,UAAU,EAAE;aACzC;SACF,CAAA;QAED,MAAM,MAAM,GAAG,MAAM,yBAAyB,CAAC,OAAO,EAAE,WAAW,EAAE,UAAU,EAAE,YAAY,CAAC,CAAA;QAE9F,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAC5B,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAChC,MAAM,CAAC,YAAY,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAA;IAC/C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,+BAA+B,EAAE,KAAK,IAAI,EAAE;QAC7C,MAAM,OAAO,GAAkB;YAC7B,GAAG,WAAW;YACd,QAAQ,EAAE,QAAQ;YAClB,OAAO,EAAE;gBACP,EAAE,IAAI,EAAE,gBAAgB,EAAE,QAAQ,EAAE,QAAQ,EAAE;aAC/C;SACF,CAAA;QAED,MAAM,MAAM,GAAG,MAAM,yBAAyB,CAAC,OAAO,EAAE,WAAW,EAAE,UAAU,EAAE,YAAY,CAAC,CAAA;QAE9F,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAChC,MAAM,CAAC,YAAY,CAAC,CAAC,oBAAoB,CACvC,MAAM,EACN,MAAM,CAAC,eAAe,CAAC,CAAC,KAAK,CAAC,CAAC,EAC/B,WAAW,CACZ,CAAA;IACH,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,iDAAiD,EAAE,KAAK,IAAI,EAAE;QAC/D,YAAY,CAAC,sBAAsB,CAAC,GAAG,EAAE;YACvC,MAAM,IAAI,KAAK,CAAC,WAAW,CAAC,CAAA;QAC9B,CAAC,CAAC,CAAA;QAEF,MAAM,OAAO,GAAkB;YAC7B,GAAG,WAAW;YACd,OAAO,EAAE;gBACP,EAAE,IAAI,EAAE,gBAAgB,EAAE,QAAQ,EAAE,UAAU,EAAE;gBAChD,EAAE,IAAI,EAAE,cAAc,EAAE,IAAI,EAAE,MAAM,EAAE;aACvC;SACF,CAAA;QAED,MAAM,MAAM,GAAG,MAAM,yBAAyB,CAAC,OAAO,EAAE,WAAW,EAAE,UAAU,EAAE,YAAY,CAAC,CAAA;QAE9F,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAC5B,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAC7B,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;IAClC,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}

package/dist/tests/units/services/mrTrackingRemoveMr.test.d.ts

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

package/dist/tests/units/services/mrTrackingRemoveMr.test.d.ts.map

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

package/dist/tests/units/services/mrTrackingRemoveMr.test.js

@@ -0,0 +1,155 @@
+import { describe, it, expect, beforeEach, afterEach } from 'vitest';
+import { mkdirSync, writeFileSync, rmSync, existsSync } from 'node:fs';
+import { join } from 'node:path';
+import { removeMrFromTracking, recomputeProjectStats, loadMrTracking } from '../../../services/mrTrackingService.js';
+function createTrackedMr(overrides = {}) {
+    const now = new Date().toISOString();
+    return {
+        id: 'gitlab-my-project-1',
+        mrNumber: 1,
+        title: 'Test MR',
+        url: 'https://gitlab.com/my-project/-/merge_requests/1',
+        project: 'my-project',
+        platform: 'gitlab',
+        sourceBranch: 'feature/test',
+        targetBranch: 'main',
+        assignment: { username: 'testuser', displayName: 'Test User', assignedAt: now },
+        state: 'pending-fix',
+        openThreads: 2,
+        totalThreads: 5,
+        createdAt: now,
+        lastReviewAt: now,
+        lastPushAt: null,
+        approvedAt: null,
+        mergedAt: null,
+        reviews: [{
+                type: 'review',
+                timestamp: now,
+                durationMs: 120000,
+                score: 7,
+                blocking: 1,
+                warnings: 2,
+                suggestions: 3,
+                threadsClosed: 0,
+                threadsOpened: 5,
+            }],
+        totalReviews: 1,
+        totalFollowups: 0,
+        totalBlocking: 1,
+        totalWarnings: 2,
+        totalSuggestions: 3,
+        totalDurationMs: 120000,
+        averageScore: 7,
+        latestScore: 7,
+        autoFollowup: true,
+        ...overrides,
+    };
+}
+function createTrackingData(mrs) {
+    return {
+        mrs,
+        lastUpdated: new Date().toISOString(),
+        stats: {
+            totalMrs: mrs.length,
+            totalReviews: mrs.reduce((sum, mr) => sum + mr.totalReviews, 0),
+            totalFollowups: mrs.reduce((sum, mr) => sum + mr.totalFollowups, 0),
+            averageReviewsPerMr: 0,
+            averageTimeToApproval: null,
+            topAssigners: [],
+        },
+    };
+}
+describe('removeMrFromTracking', () => {
+    const testProjectPath = '/tmp/test-remove-mr-tracking';
+    const trackingDir = join(testProjectPath, '.claude', 'reviews');
+    const trackingFile = join(trackingDir, 'mr-tracking.json');
+    beforeEach(() => {
+        mkdirSync(trackingDir, { recursive: true });
+    });
+    afterEach(() => {
+        if (existsSync(testProjectPath)) {
+            rmSync(testProjectPath, { recursive: true, force: true });
+        }
+    });
+    it('should remove an existing MR and return true', () => {
+        const mr1 = createTrackedMr({ id: 'gitlab-project-1', mrNumber: 1 });
+        const mr2 = createTrackedMr({ id: 'gitlab-project-2', mrNumber: 2 });
+        writeFileSync(trackingFile, JSON.stringify(createTrackingData([mr1, mr2]), null, 2));
+        const result = removeMrFromTracking(testProjectPath, 'gitlab-project-1');
+        expect(result).toBe(true);
+        const updated = loadMrTracking(testProjectPath);
+        expect(updated.mrs).toHaveLength(1);
+        expect(updated.mrs[0].id).toBe('gitlab-project-2');
+    });
+    it('should return false for non-existent MR', () => {
+        const mr1 = createTrackedMr({ id: 'gitlab-project-1', mrNumber: 1 });
+        writeFileSync(trackingFile, JSON.stringify(createTrackingData([mr1]), null, 2));
+        const result = removeMrFromTracking(testProjectPath, 'gitlab-project-999');
+        expect(result).toBe(false);
+        expect(loadMrTracking(testProjectPath).mrs).toHaveLength(1);
+    });
+    it('should return false when tracking file does not exist', () => {
+        rmSync(testProjectPath, { recursive: true, force: true });
+        const result = removeMrFromTracking('/tmp/nonexistent-remove-mr', 'gitlab-project-1');
+        expect(result).toBe(false);
+    });
+});
+describe('recomputeProjectStats', () => {
+    const testProjectPath = '/tmp/test-recompute-stats';
+    const trackingDir = join(testProjectPath, '.claude', 'reviews');
+    const trackingFile = join(trackingDir, 'mr-tracking.json');
+    beforeEach(() => {
+        mkdirSync(trackingDir, { recursive: true });
+    });
+    afterEach(() => {
+        if (existsSync(testProjectPath)) {
+            rmSync(testProjectPath, { recursive: true, force: true });
+        }
+    });
+    it('should recalculate stats from remaining MRs', () => {
+        const mr1 = createTrackedMr({ id: 'gitlab-project-1', totalReviews: 1, totalFollowups: 1 });
+        const mr2 = createTrackedMr({ id: 'gitlab-project-2', totalReviews: 1, totalFollowups: 0 });
+        writeFileSync(trackingFile, JSON.stringify(createTrackingData([mr1, mr2]), null, 2));
+        recomputeProjectStats(testProjectPath);
+        const updated = loadMrTracking(testProjectPath);
+        expect(updated.stats.totalMrs).toBe(2);
+        expect(updated.stats.totalReviews).toBe(2);
+        expect(updated.stats.totalFollowups).toBe(1);
+        expect(updated.stats.averageReviewsPerMr).toBeCloseTo(1.5);
+    });
+    it('should handle empty MR list (no MRs left)', () => {
+        writeFileSync(trackingFile, JSON.stringify(createTrackingData([]), null, 2));
+        recomputeProjectStats(testProjectPath);
+        const updated = loadMrTracking(testProjectPath);
+        expect(updated.stats.totalMrs).toBe(0);
+        expect(updated.stats.totalReviews).toBe(0);
+        expect(updated.stats.totalFollowups).toBe(0);
+        expect(updated.stats.averageReviewsPerMr).toBe(0);
+        expect(updated.stats.averageTimeToApproval).toBeNull();
+    });
+    it('should handle MRs with null scores', () => {
+        const mr1 = createTrackedMr({
+            id: 'gitlab-project-1',
+            averageScore: null,
+            latestScore: null,
+            reviews: [{
+                    type: 'review',
+                    timestamp: new Date().toISOString(),
+                    durationMs: 100000,
+                    score: null,
+                    blocking: 1,
+                    warnings: 0,
+                    suggestions: 0,
+                    threadsClosed: 0,
+                    threadsOpened: 1,
+                }],
+        });
+        const mr2 = createTrackedMr({ id: 'gitlab-project-2', totalReviews: 1 });
+        writeFileSync(trackingFile, JSON.stringify(createTrackingData([mr1, mr2]), null, 2));
+        recomputeProjectStats(testProjectPath);
+        const updated = loadMrTracking(testProjectPath);
+        expect(updated.stats.totalMrs).toBe(2);
+        expect(updated.stats.totalReviews).toBe(2);
+    });
+});
+//# sourceMappingURL=mrTrackingRemoveMr.test.js.map

package/dist/tests/units/services/mrTrackingRemoveMr.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"mrTrackingRemoveMr.test.js","sourceRoot":"","sources":["../../../../src/tests/units/services/mrTrackingRemoveMr.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAC;AACrE,OAAO,EAAE,SAAS,EAAE,aAAa,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,SAAS,CAAC;AACvE,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AACjC,OAAO,EAAE,oBAAoB,EAAE,qBAAqB,EAAE,cAAc,EAAE,MAAM,wCAAwC,CAAC;AAGrH,SAAS,eAAe,CAAC,YAAgC,EAAE;IACzD,MAAM,GAAG,GAAG,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC;IACrC,OAAO;QACL,EAAE,EAAE,qBAAqB;QACzB,QAAQ,EAAE,CAAC;QACX,KAAK,EAAE,SAAS;QAChB,GAAG,EAAE,kDAAkD;QACvD,OAAO,EAAE,YAAY;QACrB,QAAQ,EAAE,QAAQ;QAClB,YAAY,EAAE,cAAc;QAC5B,YAAY,EAAE,MAAM;QACpB,UAAU,EAAE,EAAE,QAAQ,EAAE,UAAU,EAAE,WAAW,EAAE,WAAW,EAAE,UAAU,EAAE,GAAG,EAAE;QAC/E,KAAK,EAAE,aAAa;QACpB,WAAW,EAAE,CAAC;QACd,YAAY,EAAE,CAAC;QACf,SAAS,EAAE,GAAG;QACd,YAAY,EAAE,GAAG;QACjB,UAAU,EAAE,IAAI;QAChB,UAAU,EAAE,IAAI;QAChB,QAAQ,EAAE,IAAI;QACd,OAAO,EAAE,CAAC;gBACR,IAAI,EAAE,QAAQ;gBACd,SAAS,EAAE,GAAG;gBACd,UAAU,EAAE,MAAM;gBAClB,KAAK,EAAE,CAAC;gBACR,QAAQ,EAAE,CAAC;gBACX,QAAQ,EAAE,CAAC;gBACX,WAAW,EAAE,CAAC;gBACd,aAAa,EAAE,CAAC;gBAChB,aAAa,EAAE,CAAC;aACjB,CAAC;QACF,YAAY,EAAE,CAAC;QACf,cAAc,EAAE,CAAC;QACjB,aAAa,EAAE,CAAC;QAChB,aAAa,EAAE,CAAC;QAChB,gBAAgB,EAAE,CAAC;QACnB,eAAe,EAAE,MAAM;QACvB,YAAY,EAAE,CAAC;QACf,WAAW,EAAE,CAAC;QACd,YAAY,EAAE,IAAI;QAClB,GAAG,SAAS;KACb,CAAC;AACJ,CAAC;AAED,SAAS,kBAAkB,CAAC,GAAgB;IAC1C,OAAO;QACL,GAAG;QACH,WAAW,EAAE,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE;QACrC,KAAK,EAAE;YACL,QAAQ,EAAE,GAAG,CAAC,MAAM;YACpB,YAAY,EAAE,GAAG,CAAC,MAAM,CAAC,CAAC,GAAG,EAAE,EAAE,EAAE,EAAE,CAAC,GAAG,GAAG,EAAE,CAAC,YAAY,EAAE,CAAC,CAAC;YAC/D,cAAc,EAAE,GAAG,CAAC,MAAM,CAAC,CAAC,GAAG,EAAE,EAAE,EAAE,EAAE,CAAC,GAAG,GAAG,EAAE,CAAC,cAAc,EAAE,CAAC,CAAC;YACnE,mBAAmB,EAAE,CAAC;YACtB,qBAAqB,EAAE,IAAI;YAC3B,YAAY,EAAE,EAAE;SACjB;KACF,CAAC;AACJ,CAAC;AAED,QAAQ,CAAC,sBAAsB,EAAE,GAAG,EAAE;IACpC,MAAM,eAAe,GAAG,8BAA8B,CAAC;IACvD,MAAM,WAAW,GAAG,IAAI,CAAC,eAAe,EAAE,SAAS,EAAE,SAAS,CAAC,CAAC;IAChE,MAAM,YAAY,GAAG,IAAI,CAAC,WAAW,EAAE,kBAAkB,CAAC,CAAC;IAE3D,UAAU,CAAC,GAAG,EAAE;QACd,SAAS,CAAC,WAAW,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IAC9C,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,GAAG,EAAE;QACb,IAAI,UAAU,CAAC,eAAe,CAAC,EAAE,CAAC;YAChC,MAAM,CAAC,eAAe,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QAC5D,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8CAA8C,EAAE,GAAG,EAAE;QACtD,MAAM,GAAG,GAAG,eAAe,CAAC,EAAE,EAAE,EAAE,kBAAkB,EAAE,QAAQ,EAAE,CAAC,EAAE,CAAC,CAAC;QACrE,MAAM,GAAG,GAAG,eAAe,CAAC,EAAE,EAAE,EAAE,kBAAkB,EAAE,QAAQ,EAAE,CAAC,EAAE,CAAC,CAAC;QACrE,aAAa,CAAC,YAAY,EAAE,IAAI,CAAC,SAAS,CAAC,kBAAkB,CAAC,CAAC,GAAG,EAAE,GAAG,CAAC,CAAC,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC;QAErF,MAAM,MAAM,GAAG,oBAAoB,CAAC,eAAe,EAAE,kBAAkB,CAAC,CAAC;QAEzE,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC1B,MAAM,OAAO,GAAG,cAAc,CAAC,eAAe,CAAC,CAAC;QAChD,MAAM,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QACpC,MAAM,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,kBAAkB,CAAC,CAAC;IACrD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;QACjD,MAAM,GAAG,GAAG,eAAe,CAAC,EAAE,EAAE,EAAE,kBAAkB,EAAE,QAAQ,EAAE,CAAC,EAAE,CAAC,CAAC;QACrE,aAAa,CAAC,YAAY,EAAE,IAAI,CAAC,SAAS,CAAC,kBAAkB,CAAC,CAAC,GAAG,CAAC,CAAC,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC;QAEhF,MAAM,MAAM,GAAG,oBAAoB,CAAC,eAAe,EAAE,oBAAoB,CAAC,CAAC;QAE3E,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAC3B,MAAM,CAAC,cAAc,CAAC,eAAe,CAAC,CAAC,GAAG,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;IAC9D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uDAAuD,EAAE,GAAG,EAAE;QAC/D,MAAM,CAAC,eAAe,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QAE1D,MAAM,MAAM,GAAG,oBAAoB,CAAC,4BAA4B,EAAE,kBAAkB,CAAC,CAAC;QAEtF,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IAC7B,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,uBAAuB,EAAE,GAAG,EAAE;IACrC,MAAM,eAAe,GAAG,2BAA2B,CAAC;IACpD,MAAM,WAAW,GAAG,IAAI,CAAC,eAAe,EAAE,SAAS,EAAE,SAAS,CAAC,CAAC;IAChE,MAAM,YAAY,GAAG,IAAI,CAAC,WAAW,EAAE,kBAAkB,CAAC,CAAC;IAE3D,UAAU,CAAC,GAAG,EAAE;QACd,SAAS,CAAC,WAAW,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IAC9C,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,GAAG,EAAE;QACb,IAAI,UAAU,CAAC,eAAe,CAAC,EAAE,CAAC;YAChC,MAAM,CAAC,eAAe,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QAC5D,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6CAA6C,EAAE,GAAG,EAAE;QACrD,MAAM,GAAG,GAAG,eAAe,CAAC,EAAE,EAAE,EAAE,kBAAkB,EAAE,YAAY,EAAE,CAAC,EAAE,cAAc,EAAE,CAAC,EAAE,CAAC,CAAC;QAC5F,MAAM,GAAG,GAAG,eAAe,CAAC,EAAE,EAAE,EAAE,kBAAkB,EAAE,YAAY,EAAE,CAAC,EAAE,cAAc,EAAE,CAAC,EAAE,CAAC,CAAC;QAC5F,aAAa,CAAC,YAAY,EAAE,IAAI,CAAC,SAAS,CAAC,kBAAkB,CAAC,CAAC,GAAG,EAAE,GAAG,CAAC,CAAC,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC;QAErF,qBAAqB,CAAC,eAAe,CAAC,CAAC;QAEvC,MAAM,OAAO,GAAG,cAAc,CAAC,eAAe,CAAC,CAAC;QAChD,MAAM,CAAC,OAAO,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACvC,MAAM,CAAC,OAAO,CAAC,KAAK,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAC3C,MAAM,CAAC,OAAO,CAAC,KAAK,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAC7C,MAAM,CAAC,OAAO,CAAC,KAAK,CAAC,mBAAmB,CAAC,CAAC,WAAW,CAAC,GAAG,CAAC,CAAC;IAC7D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2CAA2C,EAAE,GAAG,EAAE;QACnD,aAAa,CAAC,YAAY,EAAE,IAAI,CAAC,SAAS,CAAC,kBAAkB,CAAC,EAAE,CAAC,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC;QAE7E,qBAAqB,CAAC,eAAe,CAAC,CAAC;QAEvC,MAAM,OAAO,GAAG,cAAc,CAAC,eAAe,CAAC,CAAC;QAChD,MAAM,CAAC,OAAO,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACvC,MAAM,CAAC,OAAO,CAAC,KAAK,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAC3C,MAAM,CAAC,OAAO,CAAC,KAAK,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAC7C,MAAM,CAAC,OAAO,CAAC,KAAK,CAAC,mBAAmB,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAClD,MAAM,CAAC,OAAO,CAAC,KAAK,CAAC,qBAAqB,CAAC,CAAC,QAAQ,EAAE,CAAC;IACzD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oCAAoC,EAAE,GAAG,EAAE;QAC5C,MAAM,GAAG,GAAG,eAAe,CAAC;YAC1B,EAAE,EAAE,kBAAkB;YACtB,YAAY,EAAE,IAAI;YAClB,WAAW,EAAE,IAAI;YACjB,OAAO,EAAE,CAAC;oBACR,IAAI,EAAE,QAAQ;oBACd,SAAS,EAAE,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE;oBACnC,UAAU,EAAE,MAAM;oBAClB,KAAK,EAAE,IAAI;oBACX,QAAQ,EAAE,CAAC;oBACX,QAAQ,EAAE,CAAC;oBACX,WAAW,EAAE,CAAC;oBACd,aAAa,EAAE,CAAC;oBAChB,aAAa,EAAE,CAAC;iBACjB,CAAC;SACH,CAAC,CAAC;QACH,MAAM,GAAG,GAAG,eAAe,CAAC,EAAE,EAAE,EAAE,kBAAkB,EAAE,YAAY,EAAE,CAAC,EAAE,CAAC,CAAC;QACzE,aAAa,CAAC,YAAY,EAAE,IAAI,CAAC,SAAS,CAAC,kBAAkB,CAAC,CAAC,GAAG,EAAE,GAAG,CAAC,CAAC,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC;QAErF,qBAAqB,CAAC,eAAe,CAAC,CAAC;QAEvC,MAAM,OAAO,GAAG,cAAc,CAAC,eAAe,CAAC,CAAC;QAChD,MAAM,CAAC,OAAO,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACvC,MAAM,CAAC,OAAO,CAAC,KAAK,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAC7C,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/tests/units/services/mrTrackingService.autoFollowup.test.d.ts

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

package/dist/tests/units/services/mrTrackingService.autoFollowup.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"mrTrackingService.autoFollowup.test.d.ts","sourceRoot":"","sources":["../../../../src/tests/units/services/mrTrackingService.autoFollowup.test.ts"],"names":[],"mappings":""}

package/dist/tests/units/services/mrTrackingService.autoFollowup.test.js

@@ -0,0 +1,82 @@
+import { describe, it, expect, beforeEach, afterEach } from 'vitest';
+import { writeFileSync, mkdirSync, rmSync, existsSync } from 'node:fs';
+import { join } from 'node:path';
+import { setAutoFollowup, loadMrTracking, trackMrAssignment, } from '../../../services/mrTrackingService.js';
+import { TrackedMrFactory, MrTrackingDataFactory } from '../../factories/trackedMr.factory.js';
+describe('autoFollowup toggle', () => {
+    const projectPath = '/tmp/test-auto-followup';
+    const trackingDir = join(projectPath, '.claude', 'reviews');
+    const trackingPath = join(trackingDir, 'mr-tracking.json');
+    beforeEach(() => {
+        mkdirSync(trackingDir, { recursive: true });
+    });
+    afterEach(() => {
+        if (existsSync(projectPath)) {
+            rmSync(projectPath, { recursive: true, force: true });
+        }
+    });
+    function writeTrackingData(data) {
+        writeFileSync(trackingPath, JSON.stringify(data, null, 2));
+    }
+    describe('TrackedMr autoFollowup default value', () => {
+        it('should default autoFollowup to true when creating a new tracked MR', () => {
+            const trackedMr = trackMrAssignment(projectPath, {
+                mrNumber: 42,
+                title: 'Test MR',
+                url: 'https://gitlab.com/test/-/merge_requests/42',
+                project: 'test/project',
+                platform: 'gitlab',
+                sourceBranch: 'feature',
+                targetBranch: 'main',
+            }, { username: 'tester' });
+            expect(trackedMr.autoFollowup).toBe(true);
+        });
+        it('should treat legacy MRs without autoFollowup field as true', () => {
+            const legacyMr = TrackedMrFactory.create({
+                id: 'gitlab-test/project-10',
+                mrNumber: 10,
+            });
+            // Simulate legacy data without autoFollowup field
+            const rawData = MrTrackingDataFactory.withMrs([legacyMr]);
+            const rawJson = JSON.parse(JSON.stringify(rawData));
+            for (const mr of rawJson.mrs) {
+                mr.autoFollowup = undefined;
+            }
+            writeFileSync(trackingPath, JSON.stringify(rawJson, null, 2));
+            const loaded = loadMrTracking(projectPath);
+            expect(loaded.mrs[0].autoFollowup).toBe(true);
+        });
+    });
+    describe('setAutoFollowup', () => {
+        it('should set autoFollowup to false for an existing MR', () => {
+            const existingMr = TrackedMrFactory.create({
+                id: 'gitlab-test/project-42',
+                autoFollowup: true,
+            });
+            writeTrackingData(MrTrackingDataFactory.withMrs([existingMr]));
+            const result = setAutoFollowup(projectPath, 'gitlab-test/project-42', false);
+            expect(result).not.toBeNull();
+            expect(result?.autoFollowup).toBe(false);
+            const reloaded = loadMrTracking(projectPath);
+            expect(reloaded.mrs[0].autoFollowup).toBe(false);
+        });
+        it('should set autoFollowup to true for an existing MR', () => {
+            const existingMr = TrackedMrFactory.create({
+                id: 'gitlab-test/project-42',
+                autoFollowup: false,
+            });
+            writeTrackingData(MrTrackingDataFactory.withMrs([existingMr]));
+            const result = setAutoFollowup(projectPath, 'gitlab-test/project-42', true);
+            expect(result).not.toBeNull();
+            expect(result?.autoFollowup).toBe(true);
+            const reloaded = loadMrTracking(projectPath);
+            expect(reloaded.mrs[0].autoFollowup).toBe(true);
+        });
+        it('should return null for a non-existent MR', () => {
+            writeTrackingData(MrTrackingDataFactory.create());
+            const result = setAutoFollowup(projectPath, 'gitlab-nonexistent-99', true);
+            expect(result).toBeNull();
+        });
+    });
+});
+//# sourceMappingURL=mrTrackingService.autoFollowup.test.js.map

package/dist/tests/units/services/mrTrackingService.autoFollowup.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"mrTrackingService.autoFollowup.test.js","sourceRoot":"","sources":["../../../../src/tests/units/services/mrTrackingService.autoFollowup.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAC;AACrE,OAAO,EAAE,aAAa,EAAE,SAAS,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,SAAS,CAAC;AACvE,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AACjC,OAAO,EACL,eAAe,EACf,cAAc,EACd,iBAAiB,GAElB,MAAM,wCAAwC,CAAC;AAChD,OAAO,EAAE,gBAAgB,EAAE,qBAAqB,EAAE,MAAM,sCAAsC,CAAC;AAE/F,QAAQ,CAAC,qBAAqB,EAAE,GAAG,EAAE;IACnC,MAAM,WAAW,GAAG,yBAAyB,CAAC;IAC9C,MAAM,WAAW,GAAG,IAAI,CAAC,WAAW,EAAE,SAAS,EAAE,SAAS,CAAC,CAAC;IAC5D,MAAM,YAAY,GAAG,IAAI,CAAC,WAAW,EAAE,kBAAkB,CAAC,CAAC;IAE3D,UAAU,CAAC,GAAG,EAAE;QACd,SAAS,CAAC,WAAW,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IAC9C,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,GAAG,EAAE;QACb,IAAI,UAAU,CAAC,WAAW,CAAC,EAAE,CAAC;YAC5B,MAAM,CAAC,WAAW,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QACxD,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,SAAS,iBAAiB,CAAC,IAA0C;QACnE,aAAa,CAAC,YAAY,EAAE,IAAI,CAAC,SAAS,CAAC,IAAI,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC;IAC7D,CAAC;IAED,QAAQ,CAAC,sCAAsC,EAAE,GAAG,EAAE;QACpD,EAAE,CAAC,oEAAoE,EAAE,GAAG,EAAE;YAC5E,MAAM,SAAS,GAAG,iBAAiB,CACjC,WAAW,EACX;gBACE,QAAQ,EAAE,EAAE;gBACZ,KAAK,EAAE,SAAS;gBAChB,GAAG,EAAE,6CAA6C;gBAClD,OAAO,EAAE,cAAc;gBACvB,QAAQ,EAAE,QAAQ;gBAClB,YAAY,EAAE,SAAS;gBACvB,YAAY,EAAE,MAAM;aACrB,EACD,EAAE,QAAQ,EAAE,QAAQ,EAAE,CACvB,CAAC;YAEF,MAAM,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC5C,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,4DAA4D,EAAE,GAAG,EAAE;YACpE,MAAM,QAAQ,GAAG,gBAAgB,CAAC,MAAM,CAAC;gBACvC,EAAE,EAAE,wBAAwB;gBAC5B,QAAQ,EAAE,EAAE;aACb,CAAC,CAAC;YACH,kDAAkD;YAClD,MAAM,OAAO,GAAG,qBAAqB,CAAC,OAAO,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC;YAC1D,MAAM,OAAO,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,SAAS,CAAC,OAAO,CAAC,CAAC,CAAC;YACpD,KAAK,MAAM,EAAE,IAAI,OAAO,CAAC,GAAG,EAAE,CAAC;gBAC7B,EAAE,CAAC,YAAY,GAAG,SAAS,CAAC;YAC9B,CAAC;YACD,aAAa,CAAC,YAAY,EAAE,IAAI,CAAC,SAAS,CAAC,OAAO,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC;YAE9D,MAAM,MAAM,GAAG,cAAc,CAAC,WAAW,CAAC,CAAC;YAC3C,MAAM,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAChD,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,iBAAiB,EAAE,GAAG,EAAE;QAC/B,EAAE,CAAC,qDAAqD,EAAE,GAAG,EAAE;YAC7D,MAAM,UAAU,GAAG,gBAAgB,CAAC,MAAM,CAAC;gBACzC,EAAE,EAAE,wBAAwB;gBAC5B,YAAY,EAAE,IAAI;aACnB,CAAC,CAAC;YACH,iBAAiB,CAAC,qBAAqB,CAAC,OAAO,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC;YAE/D,MAAM,MAAM,GAAG,eAAe,CAAC,WAAW,EAAE,wBAAwB,EAAE,KAAK,CAAC,CAAC;YAE7E,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,QAAQ,EAAE,CAAC;YAC9B,MAAM,CAAC,MAAM,EAAE,YAAY,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YAEzC,MAAM,QAAQ,GAAG,cAAc,CAAC,WAAW,CAAC,CAAC;YAC7C,MAAM,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QACnD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,oDAAoD,EAAE,GAAG,EAAE;YAC5D,MAAM,UAAU,GAAG,gBAAgB,CAAC,MAAM,CAAC;gBACzC,EAAE,EAAE,wBAAwB;gBAC5B,YAAY,EAAE,KAAK;aACpB,CAAC,CAAC;YACH,iBAAiB,CAAC,qBAAqB,CAAC,OAAO,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC;YAE/D,MAAM,MAAM,GAAG,eAAe,CAAC,WAAW,EAAE,wBAAwB,EAAE,IAAI,CAAC,CAAC;YAE5E,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,QAAQ,EAAE,CAAC;YAC9B,MAAM,CAAC,MAAM,EAAE,YAAY,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAExC,MAAM,QAAQ,GAAG,cAAc,CAAC,WAAW,CAAC,CAAC;YAC7C,MAAM,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAClD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,0CAA0C,EAAE,GAAG,EAAE;YAClD,iBAAiB,CAAC,qBAAqB,CAAC,MAAM,EAAE,CAAC,CAAC;YAElD,MAAM,MAAM,GAAG,eAAe,CAAC,WAAW,EAAE,uBAAuB,EAAE,IAAI,CAAC,CAAC;YAE3E,MAAM,CAAC,MAAM,CAAC,CAAC,QAAQ,EAAE,CAAC;QAC5B,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/tests/units/services/reviewContextWatcher.service.test.d.ts

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

package/dist/tests/units/services/reviewContextWatcher.service.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"reviewContextWatcher.service.test.d.ts","sourceRoot":"","sources":["../../../../src/tests/units/services/reviewContextWatcher.service.test.ts"],"names":[],"mappings":""}

package/dist/tests/units/services/reviewContextWatcher.service.test.js

@@ -0,0 +1,112 @@
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
+import { ReviewContextWatcherService } from '../../../services/reviewContextWatcher.service.js';
+import { StubReviewContextGateway } from '../../stubs/reviewContextGateway.stub.js';
+import { ReviewContextFactory } from '../../factories/reviewContext.factory.js';
+describe('ReviewContextWatcherService', () => {
+    let gateway;
+    let watcher;
+    beforeEach(() => {
+        vi.useFakeTimers();
+        gateway = new StubReviewContextGateway();
+        watcher = new ReviewContextWatcherService(gateway);
+    });
+    afterEach(() => {
+        watcher.stopAll();
+        vi.useRealTimers();
+    });
+    it('should call callback when progress changes in context file', async () => {
+        const mergeRequestId = 'github-owner-repo-42';
+        const localPath = '/tmp/repo';
+        const receivedProgress = [];
+        const initialContext = ReviewContextFactory.create({ mergeRequestId });
+        gateway.setContext(mergeRequestId, initialContext);
+        watcher.start(localPath, mergeRequestId, (progress) => {
+            receivedProgress.push({ phase: progress.phase, currentStep: progress.currentStep });
+        });
+        gateway.updateContextProgress(mergeRequestId, {
+            phase: 'agents-running',
+            currentStep: 'verify',
+            stepsCompleted: ['context'],
+        });
+        await vi.advanceTimersByTimeAsync(600);
+        expect(receivedProgress).toContainEqual({
+            phase: 'agents-running',
+            currentStep: 'verify',
+        });
+        watcher.stop(mergeRequestId);
+    });
+    it('should not call callback when progress has not changed', async () => {
+        const mergeRequestId = 'github-owner-repo-42';
+        const localPath = '/tmp/repo';
+        let callCount = 0;
+        const context = ReviewContextFactory.createInProgress('verify', ['context']);
+        gateway.setContext(mergeRequestId, context);
+        watcher.start(localPath, mergeRequestId, () => {
+            callCount++;
+        });
+        // Advance through 3 polling intervals without changing progress
+        await vi.advanceTimersByTimeAsync(1600);
+        // Should only be called once (initial detection)
+        expect(callCount).toBe(1);
+        watcher.stop(mergeRequestId);
+    });
+    it('should stop polling when progress phase is completed', async () => {
+        const mergeRequestId = 'github-owner-repo-42';
+        const localPath = '/tmp/repo';
+        let callCount = 0;
+        const context = ReviewContextFactory.createInProgress('verify', ['context']);
+        gateway.setContext(mergeRequestId, context);
+        watcher.start(localPath, mergeRequestId, () => {
+            callCount++;
+        });
+        await vi.advanceTimersByTimeAsync(600);
+        expect(callCount).toBe(1);
+        gateway.updateContextProgress(mergeRequestId, {
+            phase: 'completed',
+            currentStep: null,
+            stepsCompleted: ['context', 'verify', 'scan', 'threads', 'report'],
+        });
+        await vi.advanceTimersByTimeAsync(600);
+        expect(callCount).toBe(2);
+        await vi.advanceTimersByTimeAsync(1000);
+        expect(callCount).toBe(2);
+        expect(watcher.isWatching(mergeRequestId)).toBe(false);
+    });
+    it('should not crash when context file does not exist', async () => {
+        const mergeRequestId = 'github-owner-repo-99';
+        const localPath = '/tmp/repo';
+        let callCount = 0;
+        watcher.start(localPath, mergeRequestId, () => {
+            callCount++;
+        });
+        await vi.advanceTimersByTimeAsync(1600);
+        expect(callCount).toBe(0);
+        expect(watcher.isWatching(mergeRequestId)).toBe(true);
+        gateway.setContext(mergeRequestId, ReviewContextFactory.createInProgress('scan', ['context', 'verify']));
+        await vi.advanceTimersByTimeAsync(600);
+        expect(callCount).toBe(1);
+        watcher.stop(mergeRequestId);
+    });
+    it('should use custom polling interval when provided', async () => {
+        const customWatcher = new ReviewContextWatcherService(gateway, 200);
+        const mergeRequestId = 'github-owner-repo-42';
+        const localPath = '/tmp/repo';
+        let callCount = 0;
+        const context = ReviewContextFactory.createInProgress('verify', ['context']);
+        gateway.setContext(mergeRequestId, context);
+        customWatcher.start(localPath, mergeRequestId, () => {
+            callCount++;
+        });
+        await vi.advanceTimersByTimeAsync(250);
+        expect(callCount).toBe(1);
+        gateway.updateContextProgress(mergeRequestId, {
+            phase: 'agents-running',
+            currentStep: 'scan',
+            stepsCompleted: ['context', 'verify'],
+        });
+        await vi.advanceTimersByTimeAsync(250);
+        expect(callCount).toBe(2);
+        customWatcher.stopAll();
+    });
+});
+//# sourceMappingURL=reviewContextWatcher.service.test.js.map

package/dist/tests/units/services/reviewContextWatcher.service.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"reviewContextWatcher.service.test.js","sourceRoot":"","sources":["../../../../src/tests/units/services/reviewContextWatcher.service.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAA;AACxE,OAAO,EAAE,2BAA2B,EAAE,MAAM,mDAAmD,CAAA;AAC/F,OAAO,EAAE,wBAAwB,EAAE,MAAM,0CAA0C,CAAA;AACnF,OAAO,EAAE,oBAAoB,EAAE,MAAM,0CAA0C,CAAA;AAE/E,QAAQ,CAAC,6BAA6B,EAAE,GAAG,EAAE;IAC3C,IAAI,OAAiC,CAAA;IACrC,IAAI,OAAoC,CAAA;IAExC,UAAU,CAAC,GAAG,EAAE;QACd,EAAE,CAAC,aAAa,EAAE,CAAA;QAClB,OAAO,GAAG,IAAI,wBAAwB,EAAE,CAAA;QACxC,OAAO,GAAG,IAAI,2BAA2B,CAAC,OAAO,CAAC,CAAA;IACpD,CAAC,CAAC,CAAA;IAEF,SAAS,CAAC,GAAG,EAAE;QACb,OAAO,CAAC,OAAO,EAAE,CAAA;QACjB,EAAE,CAAC,aAAa,EAAE,CAAA;IACpB,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,4DAA4D,EAAE,KAAK,IAAI,EAAE;QAC1E,MAAM,cAAc,GAAG,sBAAsB,CAAA;QAC7C,MAAM,SAAS,GAAG,WAAW,CAAA;QAC7B,MAAM,gBAAgB,GAAyD,EAAE,CAAA;QAEjF,MAAM,cAAc,GAAG,oBAAoB,CAAC,MAAM,CAAC,EAAE,cAAc,EAAE,CAAC,CAAA;QACtE,OAAO,CAAC,UAAU,CAAC,cAAc,EAAE,cAAc,CAAC,CAAA;QAElD,OAAO,CAAC,KAAK,CAAC,SAAS,EAAE,cAAc,EAAE,CAAC,QAAQ,EAAE,EAAE;YACpD,gBAAgB,CAAC,IAAI,CAAC,EAAE,KAAK,EAAE,QAAQ,CAAC,KAAK,EAAE,WAAW,EAAE,QAAQ,CAAC,WAAW,EAAE,CAAC,CAAA;QACrF,CAAC,CAAC,CAAA;QAEF,OAAO,CAAC,qBAAqB,CAAC,cAAc,EAAE;YAC5C,KAAK,EAAE,gBAAgB;YACvB,WAAW,EAAE,QAAQ;YACrB,cAAc,EAAE,CAAC,SAAS,CAAC;SAC5B,CAAC,CAAA;QAEF,MAAM,EAAE,CAAC,wBAAwB,CAAC,GAAG,CAAC,CAAA;QAEtC,MAAM,CAAC,gBAAgB,CAAC,CAAC,cAAc,CAAC;YACtC,KAAK,EAAE,gBAAgB;YACvB,WAAW,EAAE,QAAQ;SACtB,CAAC,CAAA;QAEF,OAAO,CAAC,IAAI,CAAC,cAAc,CAAC,CAAA;IAC9B,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,wDAAwD,EAAE,KAAK,IAAI,EAAE;QACtE,MAAM,cAAc,GAAG,sBAAsB,CAAA;QAC7C,MAAM,SAAS,GAAG,WAAW,CAAA;QAC7B,IAAI,SAAS,GAAG,CAAC,CAAA;QAEjB,MAAM,OAAO,GAAG,oBAAoB,CAAC,gBAAgB,CAAC,QAAQ,EAAE,CAAC,SAAS,CAAC,CAAC,CAAA;QAC5E,OAAO,CAAC,UAAU,CAAC,cAAc,EAAE,OAAO,CAAC,CAAA;QAE3C,OAAO,CAAC,KAAK,CAAC,SAAS,EAAE,cAAc,EAAE,GAAG,EAAE;YAC5C,SAAS,EAAE,CAAA;QACb,CAAC,CAAC,CAAA;QAEF,gEAAgE;QAChE,MAAM,EAAE,CAAC,wBAAwB,CAAC,IAAI,CAAC,CAAA;QAEvC,iDAAiD;QACjD,MAAM,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAEzB,OAAO,CAAC,IAAI,CAAC,cAAc,CAAC,CAAA;IAC9B,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,sDAAsD,EAAE,KAAK,IAAI,EAAE;QACpE,MAAM,cAAc,GAAG,sBAAsB,CAAA;QAC7C,MAAM,SAAS,GAAG,WAAW,CAAA;QAC7B,IAAI,SAAS,GAAG,CAAC,CAAA;QAEjB,MAAM,OAAO,GAAG,oBAAoB,CAAC,gBAAgB,CAAC,QAAQ,EAAE,CAAC,SAAS,CAAC,CAAC,CAAA;QAC5E,OAAO,CAAC,UAAU,CAAC,cAAc,EAAE,OAAO,CAAC,CAAA;QAE3C,OAAO,CAAC,KAAK,CAAC,SAAS,EAAE,cAAc,EAAE,GAAG,EAAE;YAC5C,SAAS,EAAE,CAAA;QACb,CAAC,CAAC,CAAA;QAEF,MAAM,EAAE,CAAC,wBAAwB,CAAC,GAAG,CAAC,CAAA;QACtC,MAAM,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAEzB,OAAO,CAAC,qBAAqB,CAAC,cAAc,EAAE;YAC5C,KAAK,EAAE,WAAW;YAClB,WAAW,EAAE,IAAI;YACjB,cAAc,EAAE,CAAC,SAAS,EAAE,QAAQ,EAAE,MAAM,EAAE,SAAS,EAAE,QAAQ,CAAC;SACnE,CAAC,CAAA;QAEF,MAAM,EAAE,CAAC,wBAAwB,CAAC,GAAG,CAAC,CAAA;QACtC,MAAM,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAEzB,MAAM,EAAE,CAAC,wBAAwB,CAAC,IAAI,CAAC,CAAA;QACvC,MAAM,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAEzB,MAAM,CAAC,OAAO,CAAC,UAAU,CAAC,cAAc,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;IACxD,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,mDAAmD,EAAE,KAAK,IAAI,EAAE;QACjE,MAAM,cAAc,GAAG,sBAAsB,CAAA;QAC7C,MAAM,SAAS,GAAG,WAAW,CAAA;QAC7B,IAAI,SAAS,GAAG,CAAC,CAAA;QAEjB,OAAO,CAAC,KAAK,CAAC,SAAS,EAAE,cAAc,EAAE,GAAG,EAAE;YAC5C,SAAS,EAAE,CAAA;QACb,CAAC,CAAC,CAAA;QAEF,MAAM,EAAE,CAAC,wBAAwB,CAAC,IAAI,CAAC,CAAA;QAEvC,MAAM,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QACzB,MAAM,CAAC,OAAO,CAAC,UAAU,CAAC,cAAc,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QAErD,OAAO,CAAC,UAAU,CAAC,cAAc,EAAE,oBAAoB,CAAC,gBAAgB,CAAC,MAAM,EAAE,CAAC,SAAS,EAAE,QAAQ,CAAC,CAAC,CAAC,CAAA;QAExG,MAAM,EAAE,CAAC,wBAAwB,CAAC,GAAG,CAAC,CAAA;QAEtC,MAAM,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAEzB,OAAO,CAAC,IAAI,CAAC,cAAc,CAAC,CAAA;IAC9B,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,kDAAkD,EAAE,KAAK,IAAI,EAAE;QAChE,MAAM,aAAa,GAAG,IAAI,2BAA2B,CAAC,OAAO,EAAE,GAAG,CAAC,CAAA;QACnE,MAAM,cAAc,GAAG,sBAAsB,CAAA;QAC7C,MAAM,SAAS,GAAG,WAAW,CAAA;QAC7B,IAAI,SAAS,GAAG,CAAC,CAAA;QAEjB,MAAM,OAAO,GAAG,oBAAoB,CAAC,gBAAgB,CAAC,QAAQ,EAAE,CAAC,SAAS,CAAC,CAAC,CAAA;QAC5E,OAAO,CAAC,UAAU,CAAC,cAAc,EAAE,OAAO,CAAC,CAAA;QAE3C,aAAa,CAAC,KAAK,CAAC,SAAS,EAAE,cAAc,EAAE,GAAG,EAAE;YAClD,SAAS,EAAE,CAAA;QACb,CAAC,CAAC,CAAA;QAEF,MAAM,EAAE,CAAC,wBAAwB,CAAC,GAAG,CAAC,CAAA;QACtC,MAAM,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAEzB,OAAO,CAAC,qBAAqB,CAAC,cAAc,EAAE;YAC5C,KAAK,EAAE,gBAAgB;YACvB,WAAW,EAAE,MAAM;YACnB,cAAc,EAAE,CAAC,SAAS,EAAE,QAAQ,CAAC;SACtC,CAAC,CAAA;QAEF,MAAM,EAAE,CAAC,wBAAwB,CAAC,GAAG,CAAC,CAAA;QACtC,MAAM,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAEzB,aAAa,CAAC,OAAO,EAAE,CAAA;IACzB,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}

package/dist/tests/units/services/threadActionsExecutor.test.d.ts

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

package/dist/tests/units/services/threadActionsExecutor.test.d.ts.map

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