reviewflow 3.0.0

package/dist/tests/units/entities/reviewScore.valueObject.test.js

@@ -0,0 +1,90 @@
+import { describe, it, expect } from 'vitest';
+import { ReviewScore } from '../../../entities/review/reviewScore.valueObject.js';
+describe('ReviewScore', () => {
+    describe('create', () => {
+        it('should create from valid props', () => {
+            const score = ReviewScore.create({ blocking: 1, warnings: 2, suggestions: 3 });
+            expect(score.blocking).toBe(1);
+            expect(score.warnings).toBe(2);
+            expect(score.suggestions).toBe(3);
+        });
+        it('should throw for negative values', () => {
+            expect(() => ReviewScore.create({ blocking: -1, warnings: 0, suggestions: 0 })).toThrow();
+        });
+    });
+    describe('zero', () => {
+        it('should create score with all zeros', () => {
+            const score = ReviewScore.zero();
+            expect(score.blocking).toBe(0);
+            expect(score.warnings).toBe(0);
+            expect(score.suggestions).toBe(0);
+        });
+    });
+    describe('total', () => {
+        it('should sum all values', () => {
+            const score = ReviewScore.create({ blocking: 1, warnings: 2, suggestions: 3 });
+            expect(score.total).toBe(6);
+        });
+    });
+    describe('severity', () => {
+        it('should return critical when blocking > 0', () => {
+            const score = ReviewScore.create({ blocking: 2, warnings: 1, suggestions: 0 });
+            expect(score.severity).toBe('critical');
+        });
+        it('should return warning when only warnings', () => {
+            const score = ReviewScore.create({ blocking: 0, warnings: 3, suggestions: 0 });
+            expect(score.severity).toBe('warning');
+        });
+        it('should return info when only suggestions', () => {
+            const score = ReviewScore.create({ blocking: 0, warnings: 0, suggestions: 5 });
+            expect(score.severity).toBe('info');
+        });
+        it('should return clean when all zero', () => {
+            const score = ReviewScore.zero();
+            expect(score.severity).toBe('clean');
+        });
+    });
+    describe('isClean', () => {
+        it('should return true when total is 0', () => {
+            const score = ReviewScore.zero();
+            expect(score.isClean).toBe(true);
+        });
+        it('should return false when has issues', () => {
+            const score = ReviewScore.create({ blocking: 0, warnings: 1, suggestions: 0 });
+            expect(score.isClean).toBe(false);
+        });
+    });
+    describe('hasBlockingIssues', () => {
+        it('should return true when blocking > 0', () => {
+            const score = ReviewScore.create({ blocking: 1, warnings: 0, suggestions: 0 });
+            expect(score.hasBlockingIssues).toBe(true);
+        });
+        it('should return false when blocking is 0', () => {
+            const score = ReviewScore.create({ blocking: 0, warnings: 5, suggestions: 10 });
+            expect(score.hasBlockingIssues).toBe(false);
+        });
+    });
+    describe('add', () => {
+        it('should add two scores', () => {
+            const score1 = ReviewScore.create({ blocking: 1, warnings: 2, suggestions: 3 });
+            const score2 = ReviewScore.create({ blocking: 2, warnings: 3, suggestions: 4 });
+            const result = score1.add(score2);
+            expect(result.blocking).toBe(3);
+            expect(result.warnings).toBe(5);
+            expect(result.suggestions).toBe(7);
+        });
+    });
+    describe('toString', () => {
+        it('should format as B/W/S', () => {
+            const score = ReviewScore.create({ blocking: 1, warnings: 2, suggestions: 3 });
+            expect(score.toString()).toBe('1B/2W/3S');
+        });
+    });
+    describe('toJSON', () => {
+        it('should return props object', () => {
+            const score = ReviewScore.create({ blocking: 1, warnings: 2, suggestions: 3 });
+            expect(score.toJSON()).toEqual({ blocking: 1, warnings: 2, suggestions: 3 });
+        });
+    });
+});
+//# sourceMappingURL=reviewScore.valueObject.test.js.map

package/dist/tests/units/entities/reviewScore.valueObject.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"reviewScore.valueObject.test.js","sourceRoot":"","sources":["../../../../src/tests/units/entities/reviewScore.valueObject.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,WAAW,EAAE,MAAM,qDAAqD,CAAC;AAElF,QAAQ,CAAC,aAAa,EAAE,GAAG,EAAE;IAC3B,QAAQ,CAAC,QAAQ,EAAE,GAAG,EAAE;QACtB,EAAE,CAAC,gCAAgC,EAAE,GAAG,EAAE;YACxC,MAAM,KAAK,GAAG,WAAW,CAAC,MAAM,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,WAAW,EAAE,CAAC,EAAE,CAAC,CAAC;YAC/E,MAAM,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAC/B,MAAM,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAC/B,MAAM,CAAC,KAAK,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACpC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,kCAAkC,EAAE,GAAG,EAAE;YAC1C,MAAM,CAAC,GAAG,EAAE,CAAC,WAAW,CAAC,MAAM,CAAC,EAAE,QAAQ,EAAE,CAAC,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,WAAW,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC,OAAO,EAAE,CAAC;QAC5F,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,MAAM,EAAE,GAAG,EAAE;QACpB,EAAE,CAAC,oCAAoC,EAAE,GAAG,EAAE;YAC5C,MAAM,KAAK,GAAG,WAAW,CAAC,IAAI,EAAE,CAAC;YACjC,MAAM,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAC/B,MAAM,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAC/B,MAAM,CAAC,KAAK,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACpC,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,OAAO,EAAE,GAAG,EAAE;QACrB,EAAE,CAAC,uBAAuB,EAAE,GAAG,EAAE;YAC/B,MAAM,KAAK,GAAG,WAAW,CAAC,MAAM,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,WAAW,EAAE,CAAC,EAAE,CAAC,CAAC;YAC/E,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAC9B,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,UAAU,EAAE,GAAG,EAAE;QACxB,EAAE,CAAC,0CAA0C,EAAE,GAAG,EAAE;YAClD,MAAM,KAAK,GAAG,WAAW,CAAC,MAAM,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,WAAW,EAAE,CAAC,EAAE,CAAC,CAAC;YAC/E,MAAM,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QAC1C,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,0CAA0C,EAAE,GAAG,EAAE;YAClD,MAAM,KAAK,GAAG,WAAW,CAAC,MAAM,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,WAAW,EAAE,CAAC,EAAE,CAAC,CAAC;YAC/E,MAAM,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;QACzC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,0CAA0C,EAAE,GAAG,EAAE;YAClD,MAAM,KAAK,GAAG,WAAW,CAAC,MAAM,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,WAAW,EAAE,CAAC,EAAE,CAAC,CAAC;YAC/E,MAAM,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;QACtC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,mCAAmC,EAAE,GAAG,EAAE;YAC3C,MAAM,KAAK,GAAG,WAAW,CAAC,IAAI,EAAE,CAAC;YACjC,MAAM,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;QACvC,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,SAAS,EAAE,GAAG,EAAE;QACvB,EAAE,CAAC,oCAAoC,EAAE,GAAG,EAAE;YAC5C,MAAM,KAAK,GAAG,WAAW,CAAC,IAAI,EAAE,CAAC;YACjC,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACnC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;YAC7C,MAAM,KAAK,GAAG,WAAW,CAAC,MAAM,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,WAAW,EAAE,CAAC,EAAE,CAAC,CAAC;YAC/E,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QACpC,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,mBAAmB,EAAE,GAAG,EAAE;QACjC,EAAE,CAAC,sCAAsC,EAAE,GAAG,EAAE;YAC9C,MAAM,KAAK,GAAG,WAAW,CAAC,MAAM,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,WAAW,EAAE,CAAC,EAAE,CAAC,CAAC;YAC/E,MAAM,CAAC,KAAK,CAAC,iBAAiB,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC7C,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,wCAAwC,EAAE,GAAG,EAAE;YAChD,MAAM,KAAK,GAAG,WAAW,CAAC,MAAM,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,WAAW,EAAE,EAAE,EAAE,CAAC,CAAC;YAChF,MAAM,CAAC,KAAK,CAAC,iBAAiB,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAC9C,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,KAAK,EAAE,GAAG,EAAE;QACnB,EAAE,CAAC,uBAAuB,EAAE,GAAG,EAAE;YAC/B,MAAM,MAAM,GAAG,WAAW,CAAC,MAAM,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,WAAW,EAAE,CAAC,EAAE,CAAC,CAAC;YAChF,MAAM,MAAM,GAAG,WAAW,CAAC,MAAM,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,WAAW,EAAE,CAAC,EAAE,CAAC,CAAC;YAEhF,MAAM,MAAM,GAAG,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC;YAElC,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAChC,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAChC,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACrC,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,UAAU,EAAE,GAAG,EAAE;QACxB,EAAE,CAAC,wBAAwB,EAAE,GAAG,EAAE;YAChC,MAAM,KAAK,GAAG,WAAW,CAAC,MAAM,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,WAAW,EAAE,CAAC,EAAE,CAAC,CAAC;YAC/E,MAAM,CAAC,KAAK,CAAC,QAAQ,EAAE,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QAC5C,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,QAAQ,EAAE,GAAG,EAAE;QACtB,EAAE,CAAC,4BAA4B,EAAE,GAAG,EAAE;YACpC,MAAM,KAAK,GAAG,WAAW,CAAC,MAAM,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,WAAW,EAAE,CAAC,EAAE,CAAC,CAAC;YAC/E,MAAM,CAAC,KAAK,CAAC,MAAM,EAAE,CAAC,CAAC,OAAO,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,WAAW,EAAE,CAAC,EAAE,CAAC,CAAC;QAC/E,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/tests/units/factories/factories.test.d.ts

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

package/dist/tests/units/factories/factories.test.d.ts.map

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

package/dist/tests/units/factories/factories.test.js

@@ -0,0 +1,100 @@
+import { ReviewJobFactory } from '../../factories/reviewJob.factory.js';
+import { GitLabEventFactory } from '../../factories/gitLabEvent.factory.js';
+import { GitHubEventFactory } from '../../factories/gitHubEvent.factory.js';
+describe('ReviewJobFactory', () => {
+    it('should create a valid ReviewJob with defaults', () => {
+        const job = ReviewJobFactory.create();
+        expect(job.id).toBe('gitlab:test-org/test-project:42');
+        expect(job.platform).toBe('gitlab');
+        expect(job.mrNumber).toBe(42);
+        expect(job.jobType).toBe('review');
+    });
+    it('should allow overriding specific fields', () => {
+        const job = ReviewJobFactory.create({ mrNumber: 99, sourceBranch: 'hotfix/urgent' });
+        expect(job.mrNumber).toBe(99);
+        expect(job.sourceBranch).toBe('hotfix/urgent');
+        expect(job.platform).toBe('gitlab');
+    });
+    it('should create a GitHub job', () => {
+        const job = ReviewJobFactory.createGitHub();
+        expect(job.platform).toBe('github');
+        expect(job.mrUrl).toContain('github.com');
+    });
+    it('should create a followup job', () => {
+        const job = ReviewJobFactory.createFollowup();
+        expect(job.jobType).toBe('followup');
+    });
+});
+describe('GitLabEventFactory', () => {
+    it('should create a valid MR event with defaults', () => {
+        const event = GitLabEventFactory.createMergeRequestEvent();
+        expect(event.object_kind).toBe('merge_request');
+        expect(event.object_attributes.state).toBe('opened');
+        expect(event.object_attributes.draft).toBe(false);
+    });
+    it('should allow deep overrides', () => {
+        const event = GitLabEventFactory.createMergeRequestEvent({
+            object_attributes: {
+                iid: 99,
+                title: 'Custom MR',
+            },
+        });
+        expect(event.object_attributes.iid).toBe(99);
+        expect(event.object_attributes.title).toBe('Custom MR');
+        expect(event.object_attributes.state).toBe('opened');
+    });
+    it('should create a draft MR', () => {
+        const event = GitLabEventFactory.createDraftMr();
+        expect(event.object_attributes.draft).toBe(true);
+    });
+    it('should create an MR with reviewer added', () => {
+        const event = GitLabEventFactory.createWithReviewerAdded('claude-reviewer');
+        expect(event.reviewers).toContainEqual(expect.objectContaining({ username: 'claude-reviewer' }));
+        expect(event.changes?.reviewers?.current).toContainEqual({ username: 'claude-reviewer' });
+        expect(event.changes?.reviewers?.previous).toEqual([]);
+    });
+    it('should create a closed MR', () => {
+        const event = GitLabEventFactory.createClosedMr();
+        expect(event.object_attributes.state).toBe('closed');
+        expect(event.object_attributes.action).toBe('close');
+    });
+    it('should create a push event', () => {
+        const event = GitLabEventFactory.createPushEvent();
+        expect(event.object_kind).toBe('push');
+        expect(event.commits.length).toBeGreaterThan(0);
+    });
+});
+describe('GitHubEventFactory', () => {
+    it('should create a valid PR event with defaults', () => {
+        const event = GitHubEventFactory.createPullRequestEvent();
+        expect(event.action).toBe('opened');
+        expect(event.pull_request.state).toBe('open');
+        expect(event.pull_request.draft).toBe(false);
+    });
+    it('should allow deep overrides', () => {
+        const event = GitHubEventFactory.createPullRequestEvent({
+            number: 456,
+            pull_request: {
+                title: 'Custom PR',
+            },
+        });
+        expect(event.number).toBe(456);
+        expect(event.pull_request.title).toBe('Custom PR');
+        expect(event.pull_request.state).toBe('open');
+    });
+    it('should create a draft PR', () => {
+        const event = GitHubEventFactory.createDraftPr();
+        expect(event.pull_request.draft).toBe(true);
+    });
+    it('should create a PR with review requested', () => {
+        const event = GitHubEventFactory.createReviewRequestedPr('claude-reviewer');
+        expect(event.action).toBe('review_requested');
+        expect(event.requested_reviewer?.login).toBe('claude-reviewer');
+    });
+    it('should create a closed PR', () => {
+        const event = GitHubEventFactory.createClosedPr();
+        expect(event.action).toBe('closed');
+        expect(event.pull_request.state).toBe('closed');
+    });
+});
+//# sourceMappingURL=factories.test.js.map

package/dist/tests/units/factories/factories.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"factories.test.js","sourceRoot":"","sources":["../../../../src/tests/units/factories/factories.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,gBAAgB,EAAE,MAAM,sCAAsC,CAAA;AACvE,OAAO,EAAE,kBAAkB,EAAE,MAAM,wCAAwC,CAAA;AAC3E,OAAO,EAAE,kBAAkB,EAAE,MAAM,wCAAwC,CAAA;AAE3E,QAAQ,CAAC,kBAAkB,EAAE,GAAG,EAAE;IAChC,EAAE,CAAC,+CAA+C,EAAE,GAAG,EAAE;QACvD,MAAM,GAAG,GAAG,gBAAgB,CAAC,MAAM,EAAE,CAAA;QAErC,MAAM,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,iCAAiC,CAAC,CAAA;QACtD,MAAM,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;QACnC,MAAM,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;QAC7B,MAAM,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;IACpC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;QACjD,MAAM,GAAG,GAAG,gBAAgB,CAAC,MAAM,CAAC,EAAE,QAAQ,EAAE,EAAE,EAAE,YAAY,EAAE,eAAe,EAAE,CAAC,CAAA;QAEpF,MAAM,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;QAC7B,MAAM,CAAC,GAAG,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,eAAe,CAAC,CAAA;QAC9C,MAAM,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;IACrC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,4BAA4B,EAAE,GAAG,EAAE;QACpC,MAAM,GAAG,GAAG,gBAAgB,CAAC,YAAY,EAAE,CAAA;QAE3C,MAAM,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;QACnC,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC,SAAS,CAAC,YAAY,CAAC,CAAA;IAC3C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,8BAA8B,EAAE,GAAG,EAAE;QACtC,MAAM,GAAG,GAAG,gBAAgB,CAAC,cAAc,EAAE,CAAA;QAE7C,MAAM,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAA;IACtC,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA;AAEF,QAAQ,CAAC,oBAAoB,EAAE,GAAG,EAAE;IAClC,EAAE,CAAC,8CAA8C,EAAE,GAAG,EAAE;QACtD,MAAM,KAAK,GAAG,kBAAkB,CAAC,uBAAuB,EAAE,CAAA;QAE1D,MAAM,CAAC,KAAK,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,eAAe,CAAC,CAAA;QAC/C,MAAM,CAAC,KAAK,CAAC,iBAAiB,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;QACpD,MAAM,CAAC,KAAK,CAAC,iBAAiB,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;IACnD,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,6BAA6B,EAAE,GAAG,EAAE;QACrC,MAAM,KAAK,GAAG,kBAAkB,CAAC,uBAAuB,CAAC;YACvD,iBAAiB,EAAE;gBACjB,GAAG,EAAE,EAAE;gBACP,KAAK,EAAE,WAAW;aACnB;SACF,CAAC,CAAA;QAEF,MAAM,CAAC,KAAK,CAAC,iBAAiB,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;QAC5C,MAAM,CAAC,KAAK,CAAC,iBAAiB,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAA;QACvD,MAAM,CAAC,KAAK,CAAC,iBAAiB,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;IACtD,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,0BAA0B,EAAE,GAAG,EAAE;QAClC,MAAM,KAAK,GAAG,kBAAkB,CAAC,aAAa,EAAE,CAAA;QAEhD,MAAM,CAAC,KAAK,CAAC,iBAAiB,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;IAClD,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;QACjD,MAAM,KAAK,GAAG,kBAAkB,CAAC,uBAAuB,CAAC,iBAAiB,CAAC,CAAA;QAE3E,MAAM,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC,cAAc,CACpC,MAAM,CAAC,gBAAgB,CAAC,EAAE,QAAQ,EAAE,iBAAiB,EAAE,CAAC,CACzD,CAAA;QACD,MAAM,CAAC,KAAK,CAAC,OAAO,EAAE,SAAS,EAAE,OAAO,CAAC,CAAC,cAAc,CAAC,EAAE,QAAQ,EAAE,iBAAiB,EAAE,CAAC,CAAA;QACzF,MAAM,CAAC,KAAK,CAAC,OAAO,EAAE,SAAS,EAAE,QAAQ,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAA;IACxD,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,2BAA2B,EAAE,GAAG,EAAE;QACnC,MAAM,KAAK,GAAG,kBAAkB,CAAC,cAAc,EAAE,CAAA;QAEjD,MAAM,CAAC,KAAK,CAAC,iBAAiB,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;QACpD,MAAM,CAAC,KAAK,CAAC,iBAAiB,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAA;IACtD,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,4BAA4B,EAAE,GAAG,EAAE;QACpC,MAAM,KAAK,GAAG,kBAAkB,CAAC,eAAe,EAAE,CAAA;QAElD,MAAM,CAAC,KAAK,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAA;QACtC,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAA;IACjD,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA;AAEF,QAAQ,CAAC,oBAAoB,EAAE,GAAG,EAAE;IAClC,EAAE,CAAC,8CAA8C,EAAE,GAAG,EAAE;QACtD,MAAM,KAAK,GAAG,kBAAkB,CAAC,sBAAsB,EAAE,CAAA;QAEzD,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;QACnC,MAAM,CAAC,KAAK,CAAC,YAAY,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAA;QAC7C,MAAM,CAAC,KAAK,CAAC,YAAY,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;IAC9C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,6BAA6B,EAAE,GAAG,EAAE;QACrC,MAAM,KAAK,GAAG,kBAAkB,CAAC,sBAAsB,CAAC;YACtD,MAAM,EAAE,GAAG;YACX,YAAY,EAAE;gBACZ,KAAK,EAAE,WAAW;aACnB;SACF,CAAC,CAAA;QAEF,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAA;QAC9B,MAAM,CAAC,KAAK,CAAC,YAAY,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAA;QAClD,MAAM,CAAC,KAAK,CAAC,YAAY,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAA;IAC/C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,0BAA0B,EAAE,GAAG,EAAE;QAClC,MAAM,KAAK,GAAG,kBAAkB,CAAC,aAAa,EAAE,CAAA;QAEhD,MAAM,CAAC,KAAK,CAAC,YAAY,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;IAC7C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,0CAA0C,EAAE,GAAG,EAAE;QAClD,MAAM,KAAK,GAAG,kBAAkB,CAAC,uBAAuB,CAAC,iBAAiB,CAAC,CAAA;QAE3E,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,kBAAkB,CAAC,CAAA;QAC7C,MAAM,CAAC,KAAK,CAAC,kBAAkB,EAAE,KAAK,CAAC,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAA;IACjE,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,2BAA2B,EAAE,GAAG,EAAE;QACnC,MAAM,KAAK,GAAG,kBAAkB,CAAC,cAAc,EAAE,CAAA;QAEjD,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;QACnC,MAAM,CAAC,KAAK,CAAC,YAAY,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;IACjD,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}

package/dist/tests/units/frameworks/claude/ensureProjectMcpConfig.test.d.ts

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

package/dist/tests/units/frameworks/claude/ensureProjectMcpConfig.test.d.ts.map

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

package/dist/tests/units/frameworks/claude/ensureProjectMcpConfig.test.js

@@ -0,0 +1,88 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { ensureProjectMcpConfig } from '../../../../frameworks/claude/claudeInvoker.js';
+import * as fs from 'node:fs';
+vi.mock('node:fs');
+describe('ensureProjectMcpConfig', () => {
+    beforeEach(() => {
+        vi.resetAllMocks();
+        // By default, make resolveMcpServerPath succeed (file exists)
+        // and ensureProjectMcpConfig create new file (no existing .mcp.json)
+        vi.mocked(fs.existsSync).mockImplementation((path) => {
+            const pathStr = String(path);
+            if (pathStr.endsWith('mcpServer.js'))
+                return true;
+            return false;
+        });
+    });
+    it('should create .mcp.json when file does not exist', () => {
+        ensureProjectMcpConfig('/tmp/my-project');
+        expect(fs.writeFileSync).toHaveBeenCalledWith('/tmp/my-project/.mcp.json', expect.any(String));
+        const written = vi.mocked(fs.writeFileSync).mock.calls[0][1];
+        const parsed = JSON.parse(written);
+        expect(parsed.mcpServers).toBeDefined();
+        expect(parsed.mcpServers['review-progress']).toBeDefined();
+        expect(parsed.mcpServers['review-progress'].command).toBe('node');
+        expect(parsed.mcpServers['review-progress'].args).toHaveLength(1);
+        expect(parsed.mcpServers['review-progress'].args[0]).toContain('mcpServer.js');
+    });
+    it('should skip when review-progress config already exists', () => {
+        vi.mocked(fs.existsSync).mockImplementation((path) => {
+            const pathStr = String(path);
+            if (pathStr.endsWith('mcpServer.js'))
+                return true;
+            if (pathStr.endsWith('.mcp.json'))
+                return true;
+            return false;
+        });
+        vi.mocked(fs.readFileSync).mockReturnValue(JSON.stringify({
+            mcpServers: {
+                'review-progress': {
+                    command: 'node',
+                    args: ['/some/existing/path.js'],
+                },
+            },
+        }));
+        ensureProjectMcpConfig('/tmp/my-project');
+        expect(fs.writeFileSync).not.toHaveBeenCalled();
+    });
+    it('should merge with existing config when review-progress is missing', () => {
+        vi.mocked(fs.existsSync).mockImplementation((path) => {
+            const pathStr = String(path);
+            if (pathStr.endsWith('mcpServer.js'))
+                return true;
+            if (pathStr.endsWith('.mcp.json'))
+                return true;
+            return false;
+        });
+        vi.mocked(fs.readFileSync).mockReturnValue(JSON.stringify({
+            mcpServers: {
+                'other-server': { command: 'python', args: ['serve.py'] },
+            },
+        }));
+        ensureProjectMcpConfig('/tmp/my-project');
+        expect(fs.writeFileSync).toHaveBeenCalled();
+        const written = vi.mocked(fs.writeFileSync).mock.calls[0][1];
+        const parsed = JSON.parse(written);
+        expect(parsed.mcpServers['other-server']).toBeDefined();
+        expect(parsed.mcpServers['review-progress']).toBeDefined();
+    });
+    it('should not throw on filesystem error', () => {
+        vi.mocked(fs.existsSync).mockImplementation(() => {
+            throw new Error('permission denied');
+        });
+        expect(() => ensureProjectMcpConfig('/tmp/my-project')).not.toThrow();
+    });
+    it('should not throw on invalid JSON in existing file', () => {
+        vi.mocked(fs.existsSync).mockImplementation((path) => {
+            const pathStr = String(path);
+            if (pathStr.endsWith('mcpServer.js'))
+                return true;
+            if (pathStr.endsWith('.mcp.json'))
+                return true;
+            return false;
+        });
+        vi.mocked(fs.readFileSync).mockReturnValue('not valid json');
+        expect(() => ensureProjectMcpConfig('/tmp/my-project')).not.toThrow();
+    });
+});
+//# sourceMappingURL=ensureProjectMcpConfig.test.js.map

package/dist/tests/units/frameworks/claude/ensureProjectMcpConfig.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"ensureProjectMcpConfig.test.js","sourceRoot":"","sources":["../../../../../src/tests/units/frameworks/claude/ensureProjectMcpConfig.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAC;AAC9D,OAAO,EAAE,sBAAsB,EAAE,MAAM,gDAAgD,CAAC;AACxF,OAAO,KAAK,EAAE,MAAM,SAAS,CAAC;AAE9B,EAAE,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;AAEnB,QAAQ,CAAC,wBAAwB,EAAE,GAAG,EAAE;IACtC,UAAU,CAAC,GAAG,EAAE;QACd,EAAE,CAAC,aAAa,EAAE,CAAC;QACnB,8DAA8D;QAC9D,qEAAqE;QACrE,EAAE,CAAC,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,CAAC,kBAAkB,CAAC,CAAC,IAAI,EAAE,EAAE;YACnD,MAAM,OAAO,GAAG,MAAM,CAAC,IAAI,CAAC,CAAC;YAC7B,IAAI,OAAO,CAAC,QAAQ,CAAC,cAAc,CAAC;gBAAE,OAAO,IAAI,CAAC;YAClD,OAAO,KAAK,CAAC;QACf,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;QAC1D,sBAAsB,CAAC,iBAAiB,CAAC,CAAC;QAE1C,MAAM,CAAC,EAAE,CAAC,aAAa,CAAC,CAAC,oBAAoB,CAC3C,2BAA2B,EAC3B,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,CACnB,CAAC;QAEF,MAAM,OAAO,GAAG,EAAE,CAAC,MAAM,CAAC,EAAE,CAAC,aAAa,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAW,CAAC;QACvE,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;QACnC,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,WAAW,EAAE,CAAC;QACxC,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,iBAAiB,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC;QAC3D,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,iBAAiB,CAAC,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;QAClE,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,iBAAiB,CAAC,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAClE,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,iBAAiB,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,cAAc,CAAC,CAAC;IACjF,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wDAAwD,EAAE,GAAG,EAAE;QAChE,EAAE,CAAC,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,CAAC,kBAAkB,CAAC,CAAC,IAAI,EAAE,EAAE;YACnD,MAAM,OAAO,GAAG,MAAM,CAAC,IAAI,CAAC,CAAC;YAC7B,IAAI,OAAO,CAAC,QAAQ,CAAC,cAAc,CAAC;gBAAE,OAAO,IAAI,CAAC;YAClD,IAAI,OAAO,CAAC,QAAQ,CAAC,WAAW,CAAC;gBAAE,OAAO,IAAI,CAAC;YAC/C,OAAO,KAAK,CAAC;QACf,CAAC,CAAC,CAAC;QACH,EAAE,CAAC,MAAM,CAAC,EAAE,CAAC,YAAY,CAAC,CAAC,eAAe,CAAC,IAAI,CAAC,SAAS,CAAC;YACxD,UAAU,EAAE;gBACV,iBAAiB,EAAE;oBACjB,OAAO,EAAE,MAAM;oBACf,IAAI,EAAE,CAAC,wBAAwB,CAAC;iBACjC;aACF;SACF,CAAC,CAAC,CAAC;QAEJ,sBAAsB,CAAC,iBAAiB,CAAC,CAAC;QAE1C,MAAM,CAAC,EAAE,CAAC,aAAa,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IAClD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mEAAmE,EAAE,GAAG,EAAE;QAC3E,EAAE,CAAC,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,CAAC,kBAAkB,CAAC,CAAC,IAAI,EAAE,EAAE;YACnD,MAAM,OAAO,GAAG,MAAM,CAAC,IAAI,CAAC,CAAC;YAC7B,IAAI,OAAO,CAAC,QAAQ,CAAC,cAAc,CAAC;gBAAE,OAAO,IAAI,CAAC;YAClD,IAAI,OAAO,CAAC,QAAQ,CAAC,WAAW,CAAC;gBAAE,OAAO,IAAI,CAAC;YAC/C,OAAO,KAAK,CAAC;QACf,CAAC,CAAC,CAAC;QACH,EAAE,CAAC,MAAM,CAAC,EAAE,CAAC,YAAY,CAAC,CAAC,eAAe,CAAC,IAAI,CAAC,SAAS,CAAC;YACxD,UAAU,EAAE;gBACV,cAAc,EAAE,EAAE,OAAO,EAAE,QAAQ,EAAE,IAAI,EAAE,CAAC,UAAU,CAAC,EAAE;aAC1D;SACF,CAAC,CAAC,CAAC;QAEJ,sBAAsB,CAAC,iBAAiB,CAAC,CAAC;QAE1C,MAAM,CAAC,EAAE,CAAC,aAAa,CAAC,CAAC,gBAAgB,EAAE,CAAC;QAC5C,MAAM,OAAO,GAAG,EAAE,CAAC,MAAM,CAAC,EAAE,CAAC,aAAa,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAW,CAAC;QACvE,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;QACnC,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,cAAc,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC;QACxD,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,iBAAiB,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC;IAC7D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sCAAsC,EAAE,GAAG,EAAE;QAC9C,EAAE,CAAC,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE;YAC/C,MAAM,IAAI,KAAK,CAAC,mBAAmB,CAAC,CAAC;QACvC,CAAC,CAAC,CAAC;QAEH,MAAM,CAAC,GAAG,EAAE,CAAC,sBAAsB,CAAC,iBAAiB,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,EAAE,CAAC;IACxE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mDAAmD,EAAE,GAAG,EAAE;QAC3D,EAAE,CAAC,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,CAAC,kBAAkB,CAAC,CAAC,IAAI,EAAE,EAAE;YACnD,MAAM,OAAO,GAAG,MAAM,CAAC,IAAI,CAAC,CAAC;YAC7B,IAAI,OAAO,CAAC,QAAQ,CAAC,cAAc,CAAC;gBAAE,OAAO,IAAI,CAAC;YAClD,IAAI,OAAO,CAAC,QAAQ,CAAC,WAAW,CAAC;gBAAE,OAAO,IAAI,CAAC;YAC/C,OAAO,KAAK,CAAC;QACf,CAAC,CAAC,CAAC;QACH,EAAE,CAAC,MAAM,CAAC,EAAE,CAAC,YAAY,CAAC,CAAC,eAAe,CAAC,gBAAgB,CAAC,CAAC;QAE7D,MAAM,CAAC,GAAG,EAAE,CAAC,sBAAsB,CAAC,iBAAiB,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,EAAE,CAAC;IACxE,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/tests/units/frameworks/claude/mcpContext.test.d.ts

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

package/dist/tests/units/frameworks/claude/mcpContext.test.d.ts.map

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

package/dist/tests/units/frameworks/claude/mcpContext.test.js

@@ -0,0 +1,88 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { writeMcpContext, cleanupMcpContext } from '../../../../frameworks/claude/claudeInvoker.js';
+import { ReviewJobFactory } from '../../../factories/reviewJob.factory.js';
+import * as fs from 'node:fs';
+import * as os from 'node:os';
+vi.mock('node:fs');
+vi.mock('node:os');
+describe('MCP Context file management', () => {
+    beforeEach(() => {
+        vi.resetAllMocks();
+        vi.mocked(os.homedir).mockReturnValue('/home/testuser');
+        vi.mocked(fs.existsSync).mockReturnValue(false);
+    });
+    describe('writeMcpContext', () => {
+        it('should write per-job context file with correct content', () => {
+            const job = ReviewJobFactory.create({ id: 'job-123' });
+            writeMcpContext(job);
+            expect(fs.mkdirSync).toHaveBeenCalledWith('/home/testuser/.claude-review/jobs', { recursive: true });
+            expect(fs.writeFileSync).toHaveBeenCalledWith('/home/testuser/.claude-review/jobs/job-123.json', expect.stringContaining('"jobId": "job-123"'));
+        });
+        it('should sanitize job ID with special characters in file path', () => {
+            const job = ReviewJobFactory.create({ id: 'gitlab:org/repo:mr-5' });
+            writeMcpContext(job);
+            expect(fs.writeFileSync).toHaveBeenCalledWith('/home/testuser/.claude-review/jobs/gitlab-org-repo-mr-5.json', expect.any(String));
+        });
+        it('should include mergeRequestId, jobType, and localPath in context', () => {
+            const job = ReviewJobFactory.create({
+                id: 'job-456',
+                platform: 'gitlab',
+                projectPath: 'my-org/my-repo',
+                mrNumber: 10,
+                localPath: '/tmp/repos/my-repo',
+                jobType: 'followup',
+            });
+            writeMcpContext(job);
+            const written = vi.mocked(fs.writeFileSync).mock.calls[0][1];
+            const parsed = JSON.parse(written);
+            expect(parsed.jobId).toBe('job-456');
+            expect(parsed.localPath).toBe('/tmp/repos/my-repo');
+            expect(parsed.mergeRequestId).toBe('gitlab-my-org/my-repo-10');
+            expect(parsed.jobType).toBe('followup');
+            expect(parsed.timestamp).toBeDefined();
+        });
+        it('should write two separate files for concurrent jobs', () => {
+            const jobA = ReviewJobFactory.create({ id: 'job-a' });
+            const jobB = ReviewJobFactory.createFollowup({ id: 'job-b' });
+            writeMcpContext(jobA);
+            writeMcpContext(jobB);
+            const calls = vi.mocked(fs.writeFileSync).mock.calls;
+            const paths = calls.map(([path]) => path);
+            expect(paths).toContain('/home/testuser/.claude-review/jobs/job-a.json');
+            expect(paths).toContain('/home/testuser/.claude-review/jobs/job-b.json');
+        });
+        it('should not throw on write failure', () => {
+            vi.mocked(fs.mkdirSync).mockImplementation(() => {
+                throw new Error('disk full');
+            });
+            const job = ReviewJobFactory.create();
+            expect(() => writeMcpContext(job)).not.toThrow();
+        });
+    });
+    describe('cleanupMcpContext', () => {
+        it('should delete the per-job file', () => {
+            vi.mocked(fs.existsSync).mockReturnValue(true);
+            cleanupMcpContext('job-123');
+            expect(fs.unlinkSync).toHaveBeenCalledWith('/home/testuser/.claude-review/jobs/job-123.json');
+        });
+        it('should not attempt deletion if file does not exist', () => {
+            vi.mocked(fs.existsSync).mockReturnValue(false);
+            cleanupMcpContext('job-123');
+            expect(fs.unlinkSync).not.toHaveBeenCalled();
+        });
+        it('should only remove the specified job file', () => {
+            vi.mocked(fs.existsSync).mockReturnValue(true);
+            cleanupMcpContext('job-a');
+            expect(fs.unlinkSync).toHaveBeenCalledTimes(1);
+            expect(fs.unlinkSync).toHaveBeenCalledWith('/home/testuser/.claude-review/jobs/job-a.json');
+        });
+        it('should not throw on cleanup failure', () => {
+            vi.mocked(fs.existsSync).mockReturnValue(true);
+            vi.mocked(fs.unlinkSync).mockImplementation(() => {
+                throw new Error('permission denied');
+            });
+            expect(() => cleanupMcpContext('job-123')).not.toThrow();
+        });
+    });
+});
+//# sourceMappingURL=mcpContext.test.js.map

package/dist/tests/units/frameworks/claude/mcpContext.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"mcpContext.test.js","sourceRoot":"","sources":["../../../../../src/tests/units/frameworks/claude/mcpContext.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAC;AAC9D,OAAO,EAAE,eAAe,EAAE,iBAAiB,EAAE,MAAM,gDAAgD,CAAC;AACpG,OAAO,EAAE,gBAAgB,EAAE,MAAM,yCAAyC,CAAC;AAC3E,OAAO,KAAK,EAAE,MAAM,SAAS,CAAC;AAC9B,OAAO,KAAK,EAAE,MAAM,SAAS,CAAC;AAE9B,EAAE,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;AACnB,EAAE,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;AAEnB,QAAQ,CAAC,6BAA6B,EAAE,GAAG,EAAE;IAC3C,UAAU,CAAC,GAAG,EAAE;QACd,EAAE,CAAC,aAAa,EAAE,CAAC;QACnB,EAAE,CAAC,MAAM,CAAC,EAAE,CAAC,OAAO,CAAC,CAAC,eAAe,CAAC,gBAAgB,CAAC,CAAC;QACxD,EAAE,CAAC,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,CAAC,eAAe,CAAC,KAAK,CAAC,CAAC;IAClD,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,iBAAiB,EAAE,GAAG,EAAE;QAC/B,EAAE,CAAC,wDAAwD,EAAE,GAAG,EAAE;YAChE,MAAM,GAAG,GAAG,gBAAgB,CAAC,MAAM,CAAC,EAAE,EAAE,EAAE,SAAS,EAAE,CAAC,CAAC;YAEvD,eAAe,CAAC,GAAG,CAAC,CAAC;YAErB,MAAM,CAAC,EAAE,CAAC,SAAS,CAAC,CAAC,oBAAoB,CACvC,oCAAoC,EACpC,EAAE,SAAS,EAAE,IAAI,EAAE,CACpB,CAAC;YACF,MAAM,CAAC,EAAE,CAAC,aAAa,CAAC,CAAC,oBAAoB,CAC3C,iDAAiD,EACjD,MAAM,CAAC,gBAAgB,CAAC,oBAAoB,CAAC,CAC9C,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,6DAA6D,EAAE,GAAG,EAAE;YACrE,MAAM,GAAG,GAAG,gBAAgB,CAAC,MAAM,CAAC,EAAE,EAAE,EAAE,sBAAsB,EAAE,CAAC,CAAC;YAEpE,eAAe,CAAC,GAAG,CAAC,CAAC;YAErB,MAAM,CAAC,EAAE,CAAC,aAAa,CAAC,CAAC,oBAAoB,CAC3C,8DAA8D,EAC9D,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,CACnB,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,kEAAkE,EAAE,GAAG,EAAE;YAC1E,MAAM,GAAG,GAAG,gBAAgB,CAAC,MAAM,CAAC;gBAClC,EAAE,EAAE,SAAS;gBACb,QAAQ,EAAE,QAAQ;gBAClB,WAAW,EAAE,gBAAgB;gBAC7B,QAAQ,EAAE,EAAE;gBACZ,SAAS,EAAE,oBAAoB;gBAC/B,OAAO,EAAE,UAAU;aACpB,CAAC,CAAC;YAEH,eAAe,CAAC,GAAG,CAAC,CAAC;YAErB,MAAM,OAAO,GAAG,EAAE,CAAC,MAAM,CAAC,EAAE,CAAC,aAAa,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAW,CAAC;YACvE,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;YACnC,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;YACrC,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,oBAAoB,CAAC,CAAC;YACpD,MAAM,CAAC,MAAM,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,0BAA0B,CAAC,CAAC;YAC/D,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;YACxC,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,WAAW,EAAE,CAAC;QACzC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,qDAAqD,EAAE,GAAG,EAAE;YAC7D,MAAM,IAAI,GAAG,gBAAgB,CAAC,MAAM,CAAC,EAAE,EAAE,EAAE,OAAO,EAAE,CAAC,CAAC;YACtD,MAAM,IAAI,GAAG,gBAAgB,CAAC,cAAc,CAAC,EAAE,EAAE,EAAE,OAAO,EAAE,CAAC,CAAC;YAE9D,eAAe,CAAC,IAAI,CAAC,CAAC;YACtB,eAAe,CAAC,IAAI,CAAC,CAAC;YAEtB,MAAM,KAAK,GAAG,EAAE,CAAC,MAAM,CAAC,EAAE,CAAC,aAAa,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC;YACrD,MAAM,KAAK,GAAG,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,EAAE,CAAC,IAAI,CAAC,CAAC;YAC1C,MAAM,CAAC,KAAK,CAAC,CAAC,SAAS,CAAC,+CAA+C,CAAC,CAAC;YACzE,MAAM,CAAC,KAAK,CAAC,CAAC,SAAS,CAAC,+CAA+C,CAAC,CAAC;QAC3E,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,mCAAmC,EAAE,GAAG,EAAE;YAC3C,EAAE,CAAC,MAAM,CAAC,EAAE,CAAC,SAAS,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE;gBAC9C,MAAM,IAAI,KAAK,CAAC,WAAW,CAAC,CAAC;YAC/B,CAAC,CAAC,CAAC;YACH,MAAM,GAAG,GAAG,gBAAgB,CAAC,MAAM,EAAE,CAAC;YAEtC,MAAM,CAAC,GAAG,EAAE,CAAC,eAAe,CAAC,GAAG,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,EAAE,CAAC;QACnD,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,mBAAmB,EAAE,GAAG,EAAE;QACjC,EAAE,CAAC,gCAAgC,EAAE,GAAG,EAAE;YACxC,EAAE,CAAC,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,CAAC,eAAe,CAAC,IAAI,CAAC,CAAC;YAE/C,iBAAiB,CAAC,SAAS,CAAC,CAAC;YAE7B,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,CAAC,oBAAoB,CACxC,iDAAiD,CAClD,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,oDAAoD,EAAE,GAAG,EAAE;YAC5D,EAAE,CAAC,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,CAAC,eAAe,CAAC,KAAK,CAAC,CAAC;YAEhD,iBAAiB,CAAC,SAAS,CAAC,CAAC;YAE7B,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;QAC/C,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,2CAA2C,EAAE,GAAG,EAAE;YACnD,EAAE,CAAC,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,CAAC,eAAe,CAAC,IAAI,CAAC,CAAC;YAE/C,iBAAiB,CAAC,OAAO,CAAC,CAAC;YAE3B,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;YAC/C,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,CAAC,oBAAoB,CACxC,+CAA+C,CAChD,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;YAC7C,EAAE,CAAC,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,CAAC,eAAe,CAAC,IAAI,CAAC,CAAC;YAC/C,EAAE,CAAC,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE;gBAC/C,MAAM,IAAI,KAAK,CAAC,mBAAmB,CAAC,CAAC;YACvC,CAAC,CAAC,CAAC;YAEH,MAAM,CAAC,GAAG,EAAE,CAAC,iBAAiB,CAAC,SAAS,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,EAAE,CAAC;QAC3D,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/tests/units/frameworks/claude/resolveMcpServerPath.test.d.ts

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

package/dist/tests/units/frameworks/claude/resolveMcpServerPath.test.d.ts.map

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

package/dist/tests/units/frameworks/claude/resolveMcpServerPath.test.js

@@ -0,0 +1,20 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { resolveMcpServerPath } from '../../../../frameworks/claude/claudeInvoker.js';
+import * as fs from 'node:fs';
+vi.mock('node:fs');
+describe('resolveMcpServerPath', () => {
+    beforeEach(() => {
+        vi.resetAllMocks();
+    });
+    it('should return path when MCP server file exists', () => {
+        vi.mocked(fs.existsSync).mockReturnValue(true);
+        const result = resolveMcpServerPath();
+        expect(result).toContain('mcpServer.js');
+    });
+    it('should throw with helpful message when MCP server not found', () => {
+        vi.mocked(fs.existsSync).mockReturnValue(false);
+        expect(() => resolveMcpServerPath()).toThrow(/MCP server not found/);
+        expect(() => resolveMcpServerPath()).toThrow(/yarn build/);
+    });
+});
+//# sourceMappingURL=resolveMcpServerPath.test.js.map

package/dist/tests/units/frameworks/claude/resolveMcpServerPath.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"resolveMcpServerPath.test.js","sourceRoot":"","sources":["../../../../../src/tests/units/frameworks/claude/resolveMcpServerPath.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAC;AAC9D,OAAO,EAAE,oBAAoB,EAAE,MAAM,gDAAgD,CAAC;AACtF,OAAO,KAAK,EAAE,MAAM,SAAS,CAAC;AAE9B,EAAE,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;AAEnB,QAAQ,CAAC,sBAAsB,EAAE,GAAG,EAAE;IACpC,UAAU,CAAC,GAAG,EAAE;QACd,EAAE,CAAC,aAAa,EAAE,CAAC;IACrB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;QACxD,EAAE,CAAC,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,CAAC,eAAe,CAAC,IAAI,CAAC,CAAC;QAE/C,MAAM,MAAM,GAAG,oBAAoB,EAAE,CAAC;QAEtC,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,cAAc,CAAC,CAAC;IAC3C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6DAA6D,EAAE,GAAG,EAAE;QACrE,EAAE,CAAC,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,CAAC,eAAe,CAAC,KAAK,CAAC,CAAC;QAEhD,MAAM,CAAC,GAAG,EAAE,CAAC,oBAAoB,EAAE,CAAC,CAAC,OAAO,CAAC,sBAAsB,CAAC,CAAC;QACrE,MAAM,CAAC,GAAG,EAAE,CAAC,oBAAoB,EAAE,CAAC,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC;IAC7D,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/tests/units/interface-adapters/adapters/platformAdapter.test.d.ts

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

package/dist/tests/units/interface-adapters/adapters/platformAdapter.test.d.ts.map

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