@diff-review-system/drs 2.2.1 → 3.0.0

package/dist/cli/review-local.test.js

@@ -0,0 +1,164 @@
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+import { reviewLocal } from './review-local.js';
+const { mockGit, parseDiff, getChangedFiles, getFilesWithDiffs, executeReview, displayReviewSummary, hasBlockingIssues, formatReviewJson, writeReviewJson, printReviewJson, formatTerminalIssue, } = vi.hoisted(() => ({
+    mockGit: {
+        checkIsRepo: vi.fn(),
+        diff: vi.fn(),
+    },
+    parseDiff: vi.fn(),
+    getChangedFiles: vi.fn(),
+    getFilesWithDiffs: vi.fn(),
+    executeReview: vi.fn(),
+    displayReviewSummary: vi.fn(),
+    hasBlockingIssues: vi.fn(),
+    formatReviewJson: vi.fn(),
+    writeReviewJson: vi.fn(),
+    printReviewJson: vi.fn(),
+    formatTerminalIssue: vi.fn(),
+}));
+vi.mock('simple-git', () => ({
+    default: vi.fn(() => mockGit),
+}));
+vi.mock('../lib/diff-parser.js', () => ({
+    parseDiff,
+    getChangedFiles,
+    getFilesWithDiffs,
+}));
+vi.mock('../lib/review-orchestrator.js', () => ({
+    executeReview,
+    displayReviewSummary,
+    hasBlockingIssues,
+}));
+vi.mock('../lib/json-output.js', () => ({
+    formatReviewJson,
+    writeReviewJson,
+    printReviewJson,
+}));
+vi.mock('../lib/comment-formatter.js', () => ({
+    formatTerminalIssue,
+}));
+const baseConfig = {
+    pi: {},
+    gitlab: { url: '', token: '' },
+    github: { token: '' },
+    review: {
+        agents: ['security', 'quality'],
+        ignorePatterns: ['*.test.ts'],
+    },
+};
+const emptySummary = {
+    filesReviewed: 1,
+    issuesFound: 0,
+    bySeverity: { CRITICAL: 0, HIGH: 0, MEDIUM: 0, LOW: 0 },
+    byCategory: {
+        SECURITY: 0,
+        QUALITY: 0,
+        STYLE: 0,
+        PERFORMANCE: 0,
+        DOCUMENTATION: 0,
+    },
+};
+describe('review-local', () => {
+    beforeEach(() => {
+        vi.clearAllMocks();
+        vi.spyOn(console, 'log').mockImplementation(() => { });
+        mockGit.checkIsRepo.mockResolvedValue(true);
+        mockGit.diff.mockResolvedValue('diff --git a/src/app.ts b/src/app.ts');
+        parseDiff.mockReturnValue([{ filename: 'src/app.ts' }]);
+        getChangedFiles.mockReturnValue(['src/app.ts']);
+        getFilesWithDiffs.mockReturnValue([{ filename: 'src/app.ts', patch: '+const answer = 42;' }]);
+        executeReview.mockResolvedValue({
+            issues: [],
+            summary: emptySummary,
+            filesReviewed: 1,
+        });
+        hasBlockingIssues.mockReturnValue(false);
+        formatReviewJson.mockReturnValue({ result: 'ok' });
+        writeReviewJson.mockResolvedValue(undefined);
+        printReviewJson.mockImplementation(() => { });
+        displayReviewSummary.mockImplementation(() => { });
+        formatTerminalIssue.mockReturnValue('formatted issue');
+    });
+    it('runs staged reviews through the shared review executor', async () => {
+        await reviewLocal(baseConfig, { staged: true });
+        expect(mockGit.diff).toHaveBeenCalledWith(['--cached']);
+        expect(executeReview).toHaveBeenCalledWith(expect.objectContaining({
+            review: expect.objectContaining({
+                agents: ['security', 'quality'],
+            }),
+        }), expect.objectContaining({
+            name: 'Local staged diff',
+            files: ['src/app.ts'],
+            filesWithDiffs: [{ filename: 'src/app.ts', patch: '+const answer = 42;' }],
+            staged: true,
+            workingDir: process.cwd(),
+        }));
+    });
+    it('throws when executed outside a git repository', async () => {
+        mockGit.checkIsRepo.mockResolvedValue(false);
+        await expect(reviewLocal(baseConfig, { staged: false })).rejects.toThrow('Not a git repository');
+    });
+    it('returns early when there are no local changes', async () => {
+        mockGit.diff.mockResolvedValue('   ');
+        await reviewLocal(baseConfig, { staged: false });
+        expect(parseDiff).not.toHaveBeenCalled();
+        expect(executeReview).not.toHaveBeenCalled();
+    });
+    it('exits with code 1 when blocking issues are found', async () => {
+        executeReview.mockResolvedValueOnce({
+            issues: [
+                {
+                    category: 'SECURITY',
+                    severity: 'HIGH',
+                    title: 'SQL Injection risk',
+                    file: 'src/app.ts',
+                    line: 10,
+                    problem: 'Unsanitized SQL input',
+                    solution: 'Use parameterized query',
+                    agent: 'security',
+                },
+            ],
+            summary: {
+                ...emptySummary,
+                issuesFound: 1,
+                bySeverity: { CRITICAL: 0, HIGH: 1, MEDIUM: 0, LOW: 0 },
+                byCategory: { SECURITY: 1, QUALITY: 0, STYLE: 0, PERFORMANCE: 0, DOCUMENTATION: 0 },
+            },
+            filesReviewed: 1,
+        });
+        hasBlockingIssues.mockReturnValueOnce(true);
+        const exitSpy = vi.spyOn(process, 'exit').mockImplementation((() => undefined));
+        await reviewLocal(baseConfig, { staged: false });
+        expect(exitSpy).toHaveBeenCalledWith(1);
+        exitSpy.mockRestore();
+    });
+    it('writes and prints JSON output when requested', async () => {
+        await reviewLocal(baseConfig, {
+            staged: false,
+            outputPath: '.drs/review-output.json',
+            jsonOutput: true,
+        });
+        expect(formatReviewJson).toHaveBeenCalledWith(emptySummary, [], {
+            source: 'local-unstaged',
+        }, undefined);
+        expect(writeReviewJson).toHaveBeenCalledWith({ result: 'ok' }, '.drs/review-output.json', process.cwd());
+        expect(printReviewJson).toHaveBeenCalledWith({ result: 'ok' });
+    });
+    it('writes JSON output when only outputPath is provided', async () => {
+        await reviewLocal(baseConfig, {
+            staged: false,
+            outputPath: '.drs/review-output.json',
+            jsonOutput: false,
+        });
+        expect(formatReviewJson).toHaveBeenCalledWith(emptySummary, [], {
+            source: 'local-unstaged',
+        }, undefined);
+        expect(writeReviewJson).toHaveBeenCalledWith({ result: 'ok' }, '.drs/review-output.json', process.cwd());
+        expect(printReviewJson).not.toHaveBeenCalled();
+    });
+    it('surfaces runtime failures from review execution', async () => {
+        executeReview.mockRejectedValueOnce(new Error('Pi runtime unavailable'));
+        await expect(reviewLocal(baseConfig, { staged: false })).rejects.toThrow('Pi runtime unavailable');
+    });
+});
+//# sourceMappingURL=review-local.test.js.map

package/dist/cli/review-local.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-local.test.js","sourceRoot":"","sources":["../../src/cli/review-local.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAE9D,OAAO,EAAE,WAAW,EAAE,MAAM,mBAAmB,CAAC;AAEhD,MAAM,EACJ,OAAO,EACP,SAAS,EACT,eAAe,EACf,iBAAiB,EACjB,aAAa,EACb,oBAAoB,EACpB,iBAAiB,EACjB,gBAAgB,EAChB,eAAe,EACf,eAAe,EACf,mBAAmB,GACpB,GAAG,EAAE,CAAC,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC;IACpB,OAAO,EAAE;QACP,WAAW,EAAE,EAAE,CAAC,EAAE,EAAE;QACpB,IAAI,EAAE,EAAE,CAAC,EAAE,EAAE;KACd;IACD,SAAS,EAAE,EAAE,CAAC,EAAE,EAAE;IAClB,eAAe,EAAE,EAAE,CAAC,EAAE,EAAE;IACxB,iBAAiB,EAAE,EAAE,CAAC,EAAE,EAAE;IAC1B,aAAa,EAAE,EAAE,CAAC,EAAE,EAAE;IACtB,oBAAoB,EAAE,EAAE,CAAC,EAAE,EAAE;IAC7B,iBAAiB,EAAE,EAAE,CAAC,EAAE,EAAE;IAC1B,gBAAgB,EAAE,EAAE,CAAC,EAAE,EAAE;IACzB,eAAe,EAAE,EAAE,CAAC,EAAE,EAAE;IACxB,eAAe,EAAE,EAAE,CAAC,EAAE,EAAE;IACxB,mBAAmB,EAAE,EAAE,CAAC,EAAE,EAAE;CAC7B,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,YAAY,EAAE,GAAG,EAAE,CAAC,CAAC;IAC3B,OAAO,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,OAAO,CAAC;CAC9B,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,uBAAuB,EAAE,GAAG,EAAE,CAAC,CAAC;IACtC,SAAS;IACT,eAAe;IACf,iBAAiB;CAClB,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,+BAA+B,EAAE,GAAG,EAAE,CAAC,CAAC;IAC9C,aAAa;IACb,oBAAoB;IACpB,iBAAiB;CAClB,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,uBAAuB,EAAE,GAAG,EAAE,CAAC,CAAC;IACtC,gBAAgB;IAChB,eAAe;IACf,eAAe;CAChB,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,6BAA6B,EAAE,GAAG,EAAE,CAAC,CAAC;IAC5C,mBAAmB;CACpB,CAAC,CAAC,CAAC;AAEJ,MAAM,UAAU,GAAG;IACjB,EAAE,EAAE,EAAE;IACN,MAAM,EAAE,EAAE,GAAG,EAAE,EAAE,EAAE,KAAK,EAAE,EAAE,EAAE;IAC9B,MAAM,EAAE,EAAE,KAAK,EAAE,EAAE,EAAE;IACrB,MAAM,EAAE;QACN,MAAM,EAAE,CAAC,UAAU,EAAE,SAAS,CAAC;QAC/B,cAAc,EAAE,CAAC,WAAW,CAAC;KAC9B;CACsB,CAAC;AAE1B,MAAM,YAAY,GAAG;IACnB,aAAa,EAAE,CAAC;IAChB,WAAW,EAAE,CAAC;IACd,UAAU,EAAE,EAAE,QAAQ,EAAE,CAAC,EAAE,IAAI,EAAE,CAAC,EAAE,MAAM,EAAE,CAAC,EAAE,GAAG,EAAE,CAAC,EAAE;IACvD,UAAU,EAAE;QACV,QAAQ,EAAE,CAAC;QACX,OAAO,EAAE,CAAC;QACV,KAAK,EAAE,CAAC;QACR,WAAW,EAAE,CAAC;QACd,aAAa,EAAE,CAAC;KACjB;CACF,CAAC;AAEF,QAAQ,CAAC,cAAc,EAAE,GAAG,EAAE;IAC5B,UAAU,CAAC,GAAG,EAAE;QACd,EAAE,CAAC,aAAa,EAAE,CAAC;QACnB,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;QAEtD,OAAO,CAAC,WAAW,CAAC,iBAAiB,CAAC,IAAI,CAAC,CAAC;QAC5C,OAAO,CAAC,IAAI,CAAC,iBAAiB,CAAC,sCAAsC,CAAC,CAAC;QAEvE,SAAS,CAAC,eAAe,CAAC,CAAC,EAAE,QAAQ,EAAE,YAAY,EAAE,CAAC,CAAC,CAAC;QACxD,eAAe,CAAC,eAAe,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC;QAChD,iBAAiB,CAAC,eAAe,CAAC,CAAC,EAAE,QAAQ,EAAE,YAAY,EAAE,KAAK,EAAE,qBAAqB,EAAE,CAAC,CAAC,CAAC;QAE9F,aAAa,CAAC,iBAAiB,CAAC;YAC9B,MAAM,EAAE,EAAE;YACV,OAAO,EAAE,YAAY;YACrB,aAAa,EAAE,CAAC;SACjB,CAAC,CAAC;QAEH,iBAAiB,CAAC,eAAe,CAAC,KAAK,CAAC,CAAC;QACzC,gBAAgB,CAAC,eAAe,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC,CAAC;QACnD,eAAe,CAAC,iBAAiB,CAAC,SAAS,CAAC,CAAC;QAC7C,eAAe,CAAC,kBAAkB,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;QAC7C,oBAAoB,CAAC,kBAAkB,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;QAClD,mBAAmB,CAAC,eAAe,CAAC,iBAAiB,CAAC,CAAC;IACzD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wDAAwD,EAAE,KAAK,IAAI,EAAE;QACtE,MAAM,WAAW,CAAC,UAAU,EAAE,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC,CAAC;QAEhD,MAAM,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,oBAAoB,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC;QACxD,MAAM,CAAC,aAAa,CAAC,CAAC,oBAAoB,CACxC,MAAM,CAAC,gBAAgB,CAAC;YACtB,MAAM,EAAE,MAAM,CAAC,gBAAgB,CAAC;gBAC9B,MAAM,EAAE,CAAC,UAAU,EAAE,SAAS,CAAC;aAChC,CAAC;SACH,CAAC,EACF,MAAM,CAAC,gBAAgB,CAAC;YACtB,IAAI,EAAE,mBAAmB;YACzB,KAAK,EAAE,CAAC,YAAY,CAAC;YACrB,cAAc,EAAE,CAAC,EAAE,QAAQ,EAAE,YAAY,EAAE,KAAK,EAAE,qBAAqB,EAAE,CAAC;YAC1E,MAAM,EAAE,IAAI;YACZ,UAAU,EAAE,OAAO,CAAC,GAAG,EAAE;SAC1B,CAAC,CACH,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+CAA+C,EAAE,KAAK,IAAI,EAAE;QAC7D,OAAO,CAAC,WAAW,CAAC,iBAAiB,CAAC,KAAK,CAAC,CAAC;QAE7C,MAAM,MAAM,CAAC,WAAW,CAAC,UAAU,EAAE,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CACtE,sBAAsB,CACvB,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+CAA+C,EAAE,KAAK,IAAI,EAAE;QAC7D,OAAO,CAAC,IAAI,CAAC,iBAAiB,CAAC,KAAK,CAAC,CAAC;QAEtC,MAAM,WAAW,CAAC,UAAU,EAAE,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,CAAC;QAEjD,MAAM,CAAC,SAAS,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;QACzC,MAAM,CAAC,aAAa,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kDAAkD,EAAE,KAAK,IAAI,EAAE;QAChE,aAAa,CAAC,qBAAqB,CAAC;YAClC,MAAM,EAAE;gBACN;oBACE,QAAQ,EAAE,UAAU;oBACpB,QAAQ,EAAE,MAAM;oBAChB,KAAK,EAAE,oBAAoB;oBAC3B,IAAI,EAAE,YAAY;oBAClB,IAAI,EAAE,EAAE;oBACR,OAAO,EAAE,uBAAuB;oBAChC,QAAQ,EAAE,yBAAyB;oBACnC,KAAK,EAAE,UAAU;iBAClB;aACF;YACD,OAAO,EAAE;gBACP,GAAG,YAAY;gBACf,WAAW,EAAE,CAAC;gBACd,UAAU,EAAE,EAAE,QAAQ,EAAE,CAAC,EAAE,IAAI,EAAE,CAAC,EAAE,MAAM,EAAE,CAAC,EAAE,GAAG,EAAE,CAAC,EAAE;gBACvD,UAAU,EAAE,EAAE,QAAQ,EAAE,CAAC,EAAE,OAAO,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,WAAW,EAAE,CAAC,EAAE,aAAa,EAAE,CAAC,EAAE;aACpF;YACD,aAAa,EAAE,CAAC;SACjB,CAAC,CAAC;QACH,iBAAiB,CAAC,mBAAmB,CAAC,IAAI,CAAC,CAAC;QAE5C,MAAM,OAAO,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC,kBAAkB,CAAC,CAAC,GAAG,EAAE,CAAC,SAAS,CAAQ,CAAC,CAAC;QAEvF,MAAM,WAAW,CAAC,UAAU,EAAE,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,CAAC;QAEjD,MAAM,CAAC,OAAO,CAAC,CAAC,oBAAoB,CAAC,CAAC,CAAC,CAAC;QACxC,OAAO,CAAC,WAAW,EAAE,CAAC;IACxB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8CAA8C,EAAE,KAAK,IAAI,EAAE;QAC5D,MAAM,WAAW,CAAC,UAAU,EAAE;YAC5B,MAAM,EAAE,KAAK;YACb,UAAU,EAAE,yBAAyB;YACrC,UAAU,EAAE,IAAI;SACjB,CAAC,CAAC;QAEH,MAAM,CAAC,gBAAgB,CAAC,CAAC,oBAAoB,CAC3C,YAAY,EACZ,EAAE,EACF;YACE,MAAM,EAAE,gBAAgB;SACzB,EACD,SAAS,CACV,CAAC;QACF,MAAM,CAAC,eAAe,CAAC,CAAC,oBAAoB,CAC1C,EAAE,MAAM,EAAE,IAAI,EAAE,EAChB,yBAAyB,EACzB,OAAO,CAAC,GAAG,EAAE,CACd,CAAC;QACF,MAAM,CAAC,eAAe,CAAC,CAAC,oBAAoB,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC,CAAC;IACjE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qDAAqD,EAAE,KAAK,IAAI,EAAE;QACnE,MAAM,WAAW,CAAC,UAAU,EAAE;YAC5B,MAAM,EAAE,KAAK;YACb,UAAU,EAAE,yBAAyB;YACrC,UAAU,EAAE,KAAK;SAClB,CAAC,CAAC;QAEH,MAAM,CAAC,gBAAgB,CAAC,CAAC,oBAAoB,CAC3C,YAAY,EACZ,EAAE,EACF;YACE,MAAM,EAAE,gBAAgB;SACzB,EACD,SAAS,CACV,CAAC;QACF,MAAM,CAAC,eAAe,CAAC,CAAC,oBAAoB,CAC1C,EAAE,MAAM,EAAE,IAAI,EAAE,EAChB,yBAAyB,EACzB,OAAO,CAAC,GAAG,EAAE,CACd,CAAC;QACF,MAAM,CAAC,eAAe,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IACjD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iDAAiD,EAAE,KAAK,IAAI,EAAE;QAC/D,aAAa,CAAC,qBAAqB,CAAC,IAAI,KAAK,CAAC,wBAAwB,CAAC,CAAC,CAAC;QAEzE,MAAM,MAAM,CAAC,WAAW,CAAC,UAAU,EAAE,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CACtE,wBAAwB,CACzB,CAAC;IACJ,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/cli/review-mr.d.ts

@@ -13,7 +13,7 @@ export interface ReviewMROptions {
     debug?: boolean;
 }
 /**
- * Review a GitLab merge request
+ * Review a GitLab merge request.
  */
 export declare function reviewMR(config: DRSConfig, options: ReviewMROptions): Promise<void>;
 //# sourceMappingURL=review-mr.d.ts.map

package/dist/cli/review-mr.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"review-mr.d.ts","sourceRoot":"","sources":["../../src/cli/review-mr.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AAOlD,MAAM,WAAW,eAAe;IAC9B,SAAS,EAAE,MAAM,CAAC;IAClB,KAAK,EAAE,MAAM,CAAC;IACd,YAAY,EAAE,OAAO,CAAC;IACtB,gBAAgB,EAAE,OAAO,CAAC;IAC1B,QAAQ,EAAE,OAAO,CAAC;IAClB,eAAe,EAAE,OAAO,CAAC;IACzB,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,KAAK,CAAC,EAAE,OAAO,CAAC;CACjB;AAwCD;;GAEG;AACH,wBAAsB,QAAQ,CAAC,MAAM,EAAE,SAAS,EAAE,OAAO,EAAE,eAAe,GAAG,OAAO,CAAC,IAAI,CAAC,CAyEzF"}
+{"version":3,"file":"review-mr.d.ts","sourceRoot":"","sources":["../../src/cli/review-mr.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AAYlD,MAAM,WAAW,eAAe;IAC9B,SAAS,EAAE,MAAM,CAAC;IAClB,KAAK,EAAE,MAAM,CAAC;IACd,YAAY,EAAE,OAAO,CAAC;IACtB,gBAAgB,EAAE,OAAO,CAAC;IAC1B,QAAQ,EAAE,OAAO,CAAC;IAClB,eAAe,EAAE,OAAO,CAAC;IACzB,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,KAAK,CAAC,EAAE,OAAO,CAAC;CACjB;AAmKD;;GAEG;AACH,wBAAsB,QAAQ,CAAC,MAAM,EAAE,SAAS,EAAE,OAAO,EAAE,eAAe,GAAG,OAAO,CAAC,IAAI,CAAC,CAwEzF"}

package/dist/cli/review-mr.js

@@ -2,8 +2,8 @@ import { createGitLabClient } from '../gitlab/client.js';
 import { GitLabPlatformAdapter } from '../gitlab/platform-adapter.js';
 import { executeUnifiedReview } from '../lib/unified-review-executor.js';
 /**
- * Parse a GitLab diff to extract valid line numbers for review comments
- * GitLab only allows comments on lines that are in the diff (added or context)
+ * Parse a GitLab diff to extract valid line numbers for review comments.
+ * GitLab only allows comments on lines that are in the diff (added or context).
  */
 function parseValidLinesFromDiff(diff) {
     const validLines = new Set();
@@ -37,27 +37,100 @@ function parseValidLinesFromDiff(diff) {
     }
     return validLines;
 }
+function parseStatusCodeFromMessage(message) {
+    const match = message.match(/\b(401|403|404|429)\b/);
+    return match ? parseInt(match[1], 10) : undefined;
+}
+function extractStatusCode(error) {
+    if (!error || typeof error !== 'object') {
+        return undefined;
+    }
+    const candidate = error;
+    if (typeof candidate.statusCode === 'number') {
+        return candidate.statusCode;
+    }
+    if (typeof candidate.status === 'number') {
+        return candidate.status;
+    }
+    if (candidate.response && typeof candidate.response === 'object') {
+        if (typeof candidate.response.statusCode === 'number') {
+            return candidate.response.statusCode;
+        }
+        if (typeof candidate.response.status === 'number') {
+            return candidate.response.status;
+        }
+    }
+    if (candidate.cause && candidate.cause !== error) {
+        return extractStatusCode(candidate.cause);
+    }
+    return undefined;
+}
+function mapGitLabContextError(error, options) {
+    const rawMessage = error instanceof Error ? error.message : String(error);
+    const normalized = rawMessage.toLowerCase();
+    const statusCode = extractStatusCode(error) ?? parseStatusCodeFromMessage(rawMessage);
+    const gitlabUrl = process.env.GITLAB_URL ?? 'https://gitlab.com';
+    if (normalized.includes('gitlab_token environment variable is required')) {
+        return new Error('GitLab authentication is required. Set GITLAB_TOKEN with an API token that can access the target merge request.');
+    }
+    if (statusCode === 401 || normalized.includes('unauthorized')) {
+        return new Error(`GitLab authentication failed for ${options.projectId}!${options.mrIid}. Verify GITLAB_TOKEN for ${gitlabUrl} and ensure it has API access.`);
+    }
+    if (statusCode === 403 || normalized.includes('forbidden')) {
+        return new Error(`GitLab authorization failed for ${options.projectId}!${options.mrIid}. Ensure the token has permission to read the project and merge request.`);
+    }
+    if (statusCode === 404 || normalized.includes('not found')) {
+        return new Error(`GitLab merge request not found: ${options.projectId}!${options.mrIid}. Verify --project/--mr values and that the token can access the project.`);
+    }
+    if (statusCode === 429 || normalized.includes('rate limit')) {
+        return new Error(`GitLab API rate limit reached while loading ${options.projectId}!${options.mrIid}. Retry after cooldown or use a token with higher limits.`);
+    }
+    const connectivityError = normalized.includes('fetch failed') ||
+        normalized.includes('econnrefused') ||
+        normalized.includes('enotfound') ||
+        normalized.includes('etimedout');
+    if (connectivityError) {
+        return new Error(`Unable to reach GitLab at ${gitlabUrl} while loading ${options.projectId}!${options.mrIid}. Check GITLAB_URL and network connectivity.`);
+    }
+    return new Error(`Failed to load GitLab merge request context for ${options.projectId}!${options.mrIid}: ${rawMessage}`);
+}
+async function loadMergeRequestContext(platformClient, options) {
+    try {
+        const pullRequest = await platformClient.getPullRequest(options.projectId, options.mrIid);
+        const changedFiles = await platformClient.getChangedFiles(options.projectId, options.mrIid);
+        return {
+            pullRequest,
+            changedFiles,
+        };
+    }
+    catch (error) {
+        throw mapGitLabContextError(error, options);
+    }
+}
 /**
- * Review a GitLab merge request
+ * Review a GitLab merge request.
  */
 export async function reviewMR(config, options) {
-    // Create GitLab client and adapter
-    const gitlabClient = createGitLabClient();
-    const platformClient = new GitLabPlatformAdapter(gitlabClient);
-    // Fetch MR details to get diff refs
-    const mr = await gitlabClient.getMergeRequest(options.projectId, options.mrIid);
-    // Fetch MR changes to build valid lines map
-    const changes = await gitlabClient.getMRChanges(options.projectId, options.mrIid);
+    let platformClient;
+    try {
+        const gitlabClient = createGitLabClient();
+        platformClient = new GitLabPlatformAdapter(gitlabClient);
+    }
+    catch (error) {
+        throw mapGitLabContextError(error, options);
+    }
+    const { pullRequest, changedFiles } = await loadMergeRequestContext(platformClient, options);
     // Build a map of file -> valid line numbers (lines that are in the diff)
     const validLinesMap = new Map();
-    for (const change of changes) {
-        if (change.diff && !change.deletedFile) {
-            const validLines = parseValidLinesFromDiff(change.diff);
-            validLinesMap.set(change.newPath, validLines);
+    for (const file of changedFiles) {
+        if (file.patch && file.status !== 'removed') {
+            const validLines = parseValidLinesFromDiff(file.patch);
+            validLinesMap.set(file.filename, validLines);
         }
     }
+    const platformData = pullRequest.platformData;
+    const diffRefs = platformData?.diff_refs;
     // Create line validator - only allow comments on lines that are in the diff
-    const diffRefs = mr.diff_refs;
     const lineValidator = {
         isValidLine(file, line) {
             if (!diffRefs?.base_sha || !diffRefs.head_sha || !diffRefs.start_sha) {
@@ -68,8 +141,8 @@ export async function reviewMR(config, options) {
         },
     };
     // Create inline position builder
-    const createInlinePosition = (issue, platformData) => {
-        const data = platformData;
+    const createInlinePosition = (issue, platformDataInput) => {
+        const data = platformDataInput;
         const refs = data.diff_refs;
         return {
             path: issue.file,
@@ -84,6 +157,8 @@ export async function reviewMR(config, options) {
         platformClient,
         projectId: options.projectId,
         prNumber: options.mrIid,
+        pullRequest,
+        changedFiles,
         postComments: options.postComments,
         postErrorComment: options.postErrorComment,
         codeQualityReport: options.codeQualityReport,

package/dist/cli/review-mr.js.map

@@ -1 +1 @@
-{"version":3,"file":"review-mr.js","sourceRoot":"","sources":["../../src/cli/review-mr.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,kBAAkB,EAAE,MAAM,qBAAqB,CAAC;AACzD,OAAO,EAAE,qBAAqB,EAAE,MAAM,+BAA+B,CAAC;AACtE,OAAO,EAAE,oBAAoB,EAAE,MAAM,mCAAmC,CAAC;AAkBzE;;;GAGG;AACH,SAAS,uBAAuB,CAAC,IAAY;IAC3C,MAAM,UAAU,GAAG,IAAI,GAAG,EAAU,CAAC;IACrC,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAC/B,IAAI,WAAW,GAAG,CAAC,CAAC;IAEpB,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,qEAAqE;QACrE,MAAM,SAAS,GAAG,IAAI,CAAC,KAAK,CAAC,uCAAuC,CAAC,CAAC;QACtE,IAAI,SAAS,EAAE,CAAC;YACd,WAAW,GAAG,QAAQ,CAAC,SAAS,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;YACzC,SAAS;QACX,CAAC;QAED,uDAAuD;QACvD,IAAI,CAAC,IAAI,IAAI,IAAI,CAAC,MAAM,KAAK,CAAC;YAAE,SAAS;QAEzC,MAAM,MAAM,GAAG,IAAI,CAAC,CAAC,CAAC,CAAC;QACvB,IAAI,MAAM,KAAK,GAAG,EAAE,CAAC;YACnB,mCAAmC;YACnC,UAAU,CAAC,GAAG,CAAC,WAAW,CAAC,CAAC;YAC5B,WAAW,EAAE,CAAC;QAChB,CAAC;aAAM,IAAI,MAAM,KAAK,GAAG,EAAE,CAAC;YAC1B,qCAAqC;YACrC,UAAU,CAAC,GAAG,CAAC,WAAW,CAAC,CAAC;YAC5B,WAAW,EAAE,CAAC;QAChB,CAAC;aAAM,IAAI,MAAM,KAAK,GAAG,EAAE,CAAC;YAC1B,uDAAuD;YACvD,SAAS;QACX,CAAC;IACH,CAAC;IAED,OAAO,UAAU,CAAC;AACpB,CAAC;AAED;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,QAAQ,CAAC,MAAiB,EAAE,OAAwB;IACxE,mCAAmC;IACnC,MAAM,YAAY,GAAG,kBAAkB,EAAE,CAAC;IAC1C,MAAM,cAAc,GAAG,IAAI,qBAAqB,CAAC,YAAY,CAAC,CAAC;IAE/D,oCAAoC;IACpC,MAAM,EAAE,GAAG,MAAM,YAAY,CAAC,eAAe,CAAC,OAAO,CAAC,SAAS,EAAE,OAAO,CAAC,KAAK,CAAC,CAAC;IAEhF,4CAA4C;IAC5C,MAAM,OAAO,GAAG,MAAM,YAAY,CAAC,YAAY,CAAC,OAAO,CAAC,SAAS,EAAE,OAAO,CAAC,KAAK,CAAC,CAAC;IAElF,yEAAyE;IACzE,MAAM,aAAa,GAAG,IAAI,GAAG,EAAuB,CAAC;IACrD,KAAK,MAAM,MAAM,IAAI,OAAO,EAAE,CAAC;QAC7B,IAAI,MAAM,CAAC,IAAI,IAAI,CAAC,MAAM,CAAC,WAAW,EAAE,CAAC;YACvC,MAAM,UAAU,GAAG,uBAAuB,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC;YACxD,aAAa,CAAC,GAAG,CAAC,MAAM,CAAC,OAAO,EAAE,UAAU,CAAC,CAAC;QAChD,CAAC;IACH,CAAC;IAED,4EAA4E;IAC5E,MAAM,QAAQ,GAAG,EAAE,CAAC,SAMP,CAAC;IACd,MAAM,aAAa,GAAkB;QACnC,WAAW,CAAC,IAAY,EAAE,IAAY;YACpC,IAAI,CAAC,QAAQ,EAAE,QAAQ,IAAI,CAAC,QAAQ,CAAC,QAAQ,IAAI,CAAC,QAAQ,CAAC,SAAS,EAAE,CAAC;gBACrE,OAAO,KAAK,CAAC;YACf,CAAC;YACD,MAAM,UAAU,GAAG,aAAa,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;YAC3C,OAAO,UAAU,KAAK,SAAS,IAAI,UAAU,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;QAC1D,CAAC;KACF,CAAC;IAEF,iCAAiC;IACjC,MAAM,oBAAoB,GAAG,CAC3B,KAAkB,EAClB,YAAqB,EACE,EAAE;QACzB,MAAM,IAAI,GAAG,YAEZ,CAAC;QACF,MAAM,IAAI,GAAG,IAAI,CAAC,SAAS,CAAC;QAC5B,OAAO;YACL,IAAI,EAAE,KAAK,CAAC,IAAI;YAChB,IAAI,EAAE,KAAK,CAAC,IAAK;YACjB,OAAO,EAAE,IAAI,EAAE,QAAQ;YACvB,OAAO,EAAE,IAAI,EAAE,QAAQ;YACvB,QAAQ,EAAE,IAAI,EAAE,SAAS;SAC1B,CAAC;IACJ,CAAC,CAAC;IAEF,yBAAyB;IACzB,MAAM,oBAAoB,CAAC,MAAM,EAAE;QACjC,cAAc;QACd,SAAS,EAAE,OAAO,CAAC,SAAS;QAC5B,QAAQ,EAAE,OAAO,CAAC,KAAK;QACvB,YAAY,EAAE,OAAO,CAAC,YAAY;QAClC,gBAAgB,EAAE,OAAO,CAAC,gBAAgB;QAC1C,iBAAiB,EAAE,OAAO,CAAC,iBAAiB;QAC5C,UAAU,EAAE,OAAO,CAAC,UAAU;QAC9B,UAAU,EAAE,OAAO,CAAC,UAAU;QAC9B,UAAU,EAAE,OAAO,CAAC,UAAU;QAC9B,aAAa;QACb,oBAAoB;QACpB,QAAQ,EAAE,OAAO,CAAC,QAAQ;QAC1B,eAAe,EAAE,OAAO,CAAC,eAAe;QACxC,KAAK,EAAE,OAAO,CAAC,KAAK;KACrB,CAAC,CAAC;AACL,CAAC"}
+{"version":3,"file":"review-mr.js","sourceRoot":"","sources":["../../src/cli/review-mr.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,kBAAkB,EAAE,MAAM,qBAAqB,CAAC;AACzD,OAAO,EAAE,qBAAqB,EAAE,MAAM,+BAA+B,CAAC;AACtE,OAAO,EAAE,oBAAoB,EAAE,MAAM,mCAAmC,CAAC;AAuCzE;;;GAGG;AACH,SAAS,uBAAuB,CAAC,IAAY;IAC3C,MAAM,UAAU,GAAG,IAAI,GAAG,EAAU,CAAC;IACrC,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAC/B,IAAI,WAAW,GAAG,CAAC,CAAC;IAEpB,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,qEAAqE;QACrE,MAAM,SAAS,GAAG,IAAI,CAAC,KAAK,CAAC,uCAAuC,CAAC,CAAC;QACtE,IAAI,SAAS,EAAE,CAAC;YACd,WAAW,GAAG,QAAQ,CAAC,SAAS,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;YACzC,SAAS;QACX,CAAC;QAED,uDAAuD;QACvD,IAAI,CAAC,IAAI,IAAI,IAAI,CAAC,MAAM,KAAK,CAAC;YAAE,SAAS;QAEzC,MAAM,MAAM,GAAG,IAAI,CAAC,CAAC,CAAC,CAAC;QACvB,IAAI,MAAM,KAAK,GAAG,EAAE,CAAC;YACnB,mCAAmC;YACnC,UAAU,CAAC,GAAG,CAAC,WAAW,CAAC,CAAC;YAC5B,WAAW,EAAE,CAAC;QAChB,CAAC;aAAM,IAAI,MAAM,KAAK,GAAG,EAAE,CAAC;YAC1B,qCAAqC;YACrC,UAAU,CAAC,GAAG,CAAC,WAAW,CAAC,CAAC;YAC5B,WAAW,EAAE,CAAC;QAChB,CAAC;aAAM,IAAI,MAAM,KAAK,GAAG,EAAE,CAAC;YAC1B,uDAAuD;YACvD,SAAS;QACX,CAAC;IACH,CAAC;IAED,OAAO,UAAU,CAAC;AACpB,CAAC;AAED,SAAS,0BAA0B,CAAC,OAAe;IACjD,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,uBAAuB,CAAC,CAAC;IACrD,OAAO,KAAK,CAAC,CAAC,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC;AACpD,CAAC;AAED,SAAS,iBAAiB,CAAC,KAAc;IACvC,IAAI,CAAC,KAAK,IAAI,OAAO,KAAK,KAAK,QAAQ,EAAE,CAAC;QACxC,OAAO,SAAS,CAAC;IACnB,CAAC;IAED,MAAM,SAAS,GAAG,KAAwB,CAAC;IAE3C,IAAI,OAAO,SAAS,CAAC,UAAU,KAAK,QAAQ,EAAE,CAAC;QAC7C,OAAO,SAAS,CAAC,UAAU,CAAC;IAC9B,CAAC;IAED,IAAI,OAAO,SAAS,CAAC,MAAM,KAAK,QAAQ,EAAE,CAAC;QACzC,OAAO,SAAS,CAAC,MAAM,CAAC;IAC1B,CAAC;IAED,IAAI,SAAS,CAAC,QAAQ,IAAI,OAAO,SAAS,CAAC,QAAQ,KAAK,QAAQ,EAAE,CAAC;QACjE,IAAI,OAAO,SAAS,CAAC,QAAQ,CAAC,UAAU,KAAK,QAAQ,EAAE,CAAC;YACtD,OAAO,SAAS,CAAC,QAAQ,CAAC,UAAU,CAAC;QACvC,CAAC;QAED,IAAI,OAAO,SAAS,CAAC,QAAQ,CAAC,MAAM,KAAK,QAAQ,EAAE,CAAC;YAClD,OAAO,SAAS,CAAC,QAAQ,CAAC,MAAM,CAAC;QACnC,CAAC;IACH,CAAC;IAED,IAAI,SAAS,CAAC,KAAK,IAAI,SAAS,CAAC,KAAK,KAAK,KAAK,EAAE,CAAC;QACjD,OAAO,iBAAiB,CAAC,SAAS,CAAC,KAAK,CAAC,CAAC;IAC5C,CAAC;IAED,OAAO,SAAS,CAAC;AACnB,CAAC;AAED,SAAS,qBAAqB,CAAC,KAAc,EAAE,OAAwB;IACrE,MAAM,UAAU,GAAG,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;IAC1E,MAAM,UAAU,GAAG,UAAU,CAAC,WAAW,EAAE,CAAC;IAC5C,MAAM,UAAU,GAAG,iBAAiB,CAAC,KAAK,CAAC,IAAI,0BAA0B,CAAC,UAAU,CAAC,CAAC;IACtF,MAAM,SAAS,GAAG,OAAO,CAAC,GAAG,CAAC,UAAU,IAAI,oBAAoB,CAAC;IAEjE,IAAI,UAAU,CAAC,QAAQ,CAAC,+CAA+C,CAAC,EAAE,CAAC;QACzE,OAAO,IAAI,KAAK,CACd,iHAAiH,CAClH,CAAC;IACJ,CAAC;IAED,IAAI,UAAU,KAAK,GAAG,IAAI,UAAU,CAAC,QAAQ,CAAC,cAAc,CAAC,EAAE,CAAC;QAC9D,OAAO,IAAI,KAAK,CACd,oCAAoC,OAAO,CAAC,SAAS,IAAI,OAAO,CAAC,KAAK,6BAA6B,SAAS,gCAAgC,CAC7I,CAAC;IACJ,CAAC;IAED,IAAI,UAAU,KAAK,GAAG,IAAI,UAAU,CAAC,QAAQ,CAAC,WAAW,CAAC,EAAE,CAAC;QAC3D,OAAO,IAAI,KAAK,CACd,mCAAmC,OAAO,CAAC,SAAS,IAAI,OAAO,CAAC,KAAK,0EAA0E,CAChJ,CAAC;IACJ,CAAC;IAED,IAAI,UAAU,KAAK,GAAG,IAAI,UAAU,CAAC,QAAQ,CAAC,WAAW,CAAC,EAAE,CAAC;QAC3D,OAAO,IAAI,KAAK,CACd,mCAAmC,OAAO,CAAC,SAAS,IAAI,OAAO,CAAC,KAAK,2EAA2E,CACjJ,CAAC;IACJ,CAAC;IAED,IAAI,UAAU,KAAK,GAAG,IAAI,UAAU,CAAC,QAAQ,CAAC,YAAY,CAAC,EAAE,CAAC;QAC5D,OAAO,IAAI,KAAK,CACd,+CAA+C,OAAO,CAAC,SAAS,IAAI,OAAO,CAAC,KAAK,2DAA2D,CAC7I,CAAC;IACJ,CAAC;IAED,MAAM,iBAAiB,GACrB,UAAU,CAAC,QAAQ,CAAC,cAAc,CAAC;QACnC,UAAU,CAAC,QAAQ,CAAC,cAAc,CAAC;QACnC,UAAU,CAAC,QAAQ,CAAC,WAAW,CAAC;QAChC,UAAU,CAAC,QAAQ,CAAC,WAAW,CAAC,CAAC;IAEnC,IAAI,iBAAiB,EAAE,CAAC;QACtB,OAAO,IAAI,KAAK,CACd,6BAA6B,SAAS,kBAAkB,OAAO,CAAC,SAAS,IAAI,OAAO,CAAC,KAAK,8CAA8C,CACzI,CAAC;IACJ,CAAC;IAED,OAAO,IAAI,KAAK,CACd,mDAAmD,OAAO,CAAC,SAAS,IAAI,OAAO,CAAC,KAAK,KAAK,UAAU,EAAE,CACvG,CAAC;AACJ,CAAC;AAED,KAAK,UAAU,uBAAuB,CACpC,cAAqC,EACrC,OAAwB;IAExB,IAAI,CAAC;QACH,MAAM,WAAW,GAAG,MAAM,cAAc,CAAC,cAAc,CAAC,OAAO,CAAC,SAAS,EAAE,OAAO,CAAC,KAAK,CAAC,CAAC;QAC1F,MAAM,YAAY,GAAG,MAAM,cAAc,CAAC,eAAe,CAAC,OAAO,CAAC,SAAS,EAAE,OAAO,CAAC,KAAK,CAAC,CAAC;QAE5F,OAAO;YACL,WAAW;YACX,YAAY;SACb,CAAC;IACJ,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,MAAM,qBAAqB,CAAC,KAAK,EAAE,OAAO,CAAC,CAAC;IAC9C,CAAC;AACH,CAAC;AAED;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,QAAQ,CAAC,MAAiB,EAAE,OAAwB;IACxE,IAAI,cAAqC,CAAC;IAE1C,IAAI,CAAC;QACH,MAAM,YAAY,GAAG,kBAAkB,EAAE,CAAC;QAC1C,cAAc,GAAG,IAAI,qBAAqB,CAAC,YAAY,CAAC,CAAC;IAC3D,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,MAAM,qBAAqB,CAAC,KAAK,EAAE,OAAO,CAAC,CAAC;IAC9C,CAAC;IAED,MAAM,EAAE,WAAW,EAAE,YAAY,EAAE,GAAG,MAAM,uBAAuB,CAAC,cAAc,EAAE,OAAO,CAAC,CAAC;IAE7F,yEAAyE;IACzE,MAAM,aAAa,GAAG,IAAI,GAAG,EAAuB,CAAC;IACrD,KAAK,MAAM,IAAI,IAAI,YAAY,EAAE,CAAC;QAChC,IAAI,IAAI,CAAC,KAAK,IAAI,IAAI,CAAC,MAAM,KAAK,SAAS,EAAE,CAAC;YAC5C,MAAM,UAAU,GAAG,uBAAuB,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YACvD,aAAa,CAAC,GAAG,CAAC,IAAI,CAAC,QAAQ,EAAE,UAAU,CAAC,CAAC;QAC/C,CAAC;IACH,CAAC;IAED,MAAM,YAAY,GAAG,WAAW,CAAC,YAA0D,CAAC;IAC5F,MAAM,QAAQ,GAAG,YAAY,EAAE,SAAS,CAAC;IAEzC,4EAA4E;IAC5E,MAAM,aAAa,GAAkB;QACnC,WAAW,CAAC,IAAY,EAAE,IAAY;YACpC,IAAI,CAAC,QAAQ,EAAE,QAAQ,IAAI,CAAC,QAAQ,CAAC,QAAQ,IAAI,CAAC,QAAQ,CAAC,SAAS,EAAE,CAAC;gBACrE,OAAO,KAAK,CAAC;YACf,CAAC;YACD,MAAM,UAAU,GAAG,aAAa,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;YAC3C,OAAO,UAAU,KAAK,SAAS,IAAI,UAAU,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;QAC1D,CAAC;KACF,CAAC;IAEF,iCAAiC;IACjC,MAAM,oBAAoB,GAAG,CAC3B,KAAkB,EAClB,iBAA0B,EACH,EAAE;QACzB,MAAM,IAAI,GAAG,iBAEZ,CAAC;QACF,MAAM,IAAI,GAAG,IAAI,CAAC,SAAS,CAAC;QAC5B,OAAO;YACL,IAAI,EAAE,KAAK,CAAC,IAAI;YAChB,IAAI,EAAE,KAAK,CAAC,IAAK;YACjB,OAAO,EAAE,IAAI,EAAE,QAAQ;YACvB,OAAO,EAAE,IAAI,EAAE,QAAQ;YACvB,QAAQ,EAAE,IAAI,EAAE,SAAS;SAC1B,CAAC;IACJ,CAAC,CAAC;IAEF,yBAAyB;IACzB,MAAM,oBAAoB,CAAC,MAAM,EAAE;QACjC,cAAc;QACd,SAAS,EAAE,OAAO,CAAC,SAAS;QAC5B,QAAQ,EAAE,OAAO,CAAC,KAAK;QACvB,WAAW;QACX,YAAY;QACZ,YAAY,EAAE,OAAO,CAAC,YAAY;QAClC,gBAAgB,EAAE,OAAO,CAAC,gBAAgB;QAC1C,iBAAiB,EAAE,OAAO,CAAC,iBAAiB;QAC5C,UAAU,EAAE,OAAO,CAAC,UAAU;QAC9B,UAAU,EAAE,OAAO,CAAC,UAAU;QAC9B,UAAU,EAAE,OAAO,CAAC,UAAU;QAC9B,aAAa;QACb,oBAAoB;QACpB,QAAQ,EAAE,OAAO,CAAC,QAAQ;QAC1B,eAAe,EAAE,OAAO,CAAC,eAAe;QACxC,KAAK,EAAE,OAAO,CAAC,KAAK;KACrB,CAAC,CAAC;AACL,CAAC"}

package/dist/cli/review-mr.test.d.ts

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

package/dist/cli/review-mr.test.d.ts.map

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

package/dist/cli/review-mr.test.js

@@ -0,0 +1,142 @@
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+import { reviewMR } from './review-mr.js';
+const { mockGitLabClient, createGitLabClient, executeUnifiedReview } = vi.hoisted(() => ({
+    mockGitLabClient: {
+        getMergeRequest: vi.fn(),
+        getMRChanges: vi.fn(),
+    },
+    createGitLabClient: vi.fn(),
+    executeUnifiedReview: vi.fn(),
+}));
+vi.mock('../gitlab/client.js', () => ({
+    createGitLabClient,
+}));
+vi.mock('../lib/unified-review-executor.js', () => ({
+    executeUnifiedReview,
+}));
+const baseConfig = {
+    pi: {},
+    gitlab: { url: 'https://gitlab.com', token: 'token' },
+    github: { token: 'token' },
+    review: {
+        agents: ['security', 'quality'],
+        ignorePatterns: [],
+    },
+};
+const baseOptions = {
+    projectId: 'group/repo',
+    mrIid: 42,
+    postComments: true,
+    postErrorComment: true,
+    describe: false,
+    postDescription: false,
+};
+describe('review-mr', () => {
+    beforeEach(() => {
+        vi.clearAllMocks();
+        createGitLabClient.mockReturnValue(mockGitLabClient);
+        executeUnifiedReview.mockResolvedValue(undefined);
+        mockGitLabClient.getMergeRequest.mockResolvedValue({
+            iid: 42,
+            title: 'Test MR',
+            source_branch: 'feature/pi-migration',
+            target_branch: 'main',
+            author: { name: 'test-user' },
+            diff_refs: {
+                base_sha: 'base-sha',
+                head_sha: 'head-sha',
+                start_sha: 'start-sha',
+            },
+        });
+        mockGitLabClient.getMRChanges.mockResolvedValue([
+            {
+                oldPath: 'src/app.ts',
+                newPath: 'src/app.ts',
+                newFile: false,
+                renamedFile: false,
+                deletedFile: false,
+                diff: '@@ -8,2 +8,3 @@\n context line\n+new line\n-removed line',
+            },
+            {
+                oldPath: 'src/old.ts',
+                newPath: 'src/old.ts',
+                newFile: false,
+                renamedFile: false,
+                deletedFile: true,
+                diff: '@@ -1 +0,0 @@\n-old line',
+            },
+        ]);
+    });
+    it('loads MR context once and forwards diff-aware validators to unified review', async () => {
+        await reviewMR(baseConfig, baseOptions);
+        expect(createGitLabClient).toHaveBeenCalledTimes(1);
+        expect(mockGitLabClient.getMergeRequest).toHaveBeenCalledWith('group/repo', 42);
+        expect(mockGitLabClient.getMRChanges).toHaveBeenCalledWith('group/repo', 42);
+        const unifiedOptions = vi.mocked(executeUnifiedReview).mock.calls[0][1];
+        expect(unifiedOptions.pullRequest).toEqual(expect.objectContaining({
+            number: 42,
+            title: 'Test MR',
+        }));
+        expect(unifiedOptions.changedFiles).toEqual(expect.arrayContaining([
+            expect.objectContaining({ filename: 'src/app.ts', status: 'modified' }),
+            expect.objectContaining({ filename: 'src/old.ts', status: 'removed' }),
+        ]));
+        expect(unifiedOptions.lineValidator?.isValidLine('src/app.ts', 8)).toBe(true);
+        expect(unifiedOptions.lineValidator?.isValidLine('src/app.ts', 9)).toBe(true);
+        expect(unifiedOptions.lineValidator?.isValidLine('src/app.ts', 10)).toBe(false);
+        expect(unifiedOptions.lineValidator?.isValidLine('src/old.ts', 1)).toBe(false);
+        const inlinePosition = unifiedOptions.createInlinePosition?.({
+            category: 'QUALITY',
+            severity: 'HIGH',
+            title: 'Example issue',
+            file: 'src/app.ts',
+            line: 9,
+            problem: 'Problem',
+            solution: 'Solution',
+            agent: 'quality',
+        }, {
+            diff_refs: {
+                base_sha: 'base-sha',
+                head_sha: 'head-sha',
+                start_sha: 'start-sha',
+            },
+        });
+        expect(inlinePosition).toEqual({
+            path: 'src/app.ts',
+            line: 9,
+            baseSha: 'base-sha',
+            headSha: 'head-sha',
+            startSha: 'start-sha',
+        });
+    });
+    it('rejects with actionable guidance when GITLAB_TOKEN is missing', async () => {
+        createGitLabClient.mockImplementationOnce(() => {
+            throw new Error('GITLAB_TOKEN environment variable is required');
+        });
+        await expect(reviewMR(baseConfig, baseOptions)).rejects.toThrow('Set GITLAB_TOKEN');
+        expect(executeUnifiedReview).not.toHaveBeenCalled();
+    });
+    it('maps GitLab authentication failures to actionable errors', async () => {
+        mockGitLabClient.getMergeRequest.mockRejectedValueOnce(new Error('401 Unauthorized'));
+        await expect(reviewMR(baseConfig, baseOptions)).rejects.toThrow('GitLab authentication failed for group/repo!42');
+        expect(executeUnifiedReview).not.toHaveBeenCalled();
+    });
+    it('maps GitLab not found failures with project/mr remediation', async () => {
+        mockGitLabClient.getMergeRequest.mockRejectedValueOnce(new Error('404 Not Found'));
+        await expect(reviewMR(baseConfig, baseOptions)).rejects.toThrow('GitLab merge request not found: group/repo!42');
+        expect(executeUnifiedReview).not.toHaveBeenCalled();
+    });
+    it('disables line validation when diff refs are unavailable', async () => {
+        mockGitLabClient.getMergeRequest.mockResolvedValueOnce({
+            iid: 42,
+            title: 'Missing refs',
+            source_branch: 'feature/pi-migration',
+            target_branch: 'main',
+            author: { name: 'test-user' },
+        });
+        await reviewMR(baseConfig, baseOptions);
+        const unifiedOptions = vi.mocked(executeUnifiedReview).mock.calls[0][1];
+        expect(unifiedOptions.lineValidator?.isValidLine('src/app.ts', 8)).toBe(false);
+    });
+});
+//# sourceMappingURL=review-mr.test.js.map

package/dist/cli/review-mr.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-mr.test.js","sourceRoot":"","sources":["../../src/cli/review-mr.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAI9D,OAAO,EAAE,QAAQ,EAAE,MAAM,gBAAgB,CAAC;AAE1C,MAAM,EAAE,gBAAgB,EAAE,kBAAkB,EAAE,oBAAoB,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC;IACvF,gBAAgB,EAAE;QAChB,eAAe,EAAE,EAAE,CAAC,EAAE,EAAE;QACxB,YAAY,EAAE,EAAE,CAAC,EAAE,EAAE;KACtB;IACD,kBAAkB,EAAE,EAAE,CAAC,EAAE,EAAE;IAC3B,oBAAoB,EAAE,EAAE,CAAC,EAAE,EAAE;CAC9B,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,qBAAqB,EAAE,GAAG,EAAE,CAAC,CAAC;IACpC,kBAAkB;CACnB,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,mCAAmC,EAAE,GAAG,EAAE,CAAC,CAAC;IAClD,oBAAoB;CACrB,CAAC,CAAC,CAAC;AAEJ,MAAM,UAAU,GAAG;IACjB,EAAE,EAAE,EAAE;IACN,MAAM,EAAE,EAAE,GAAG,EAAE,oBAAoB,EAAE,KAAK,EAAE,OAAO,EAAE;IACrD,MAAM,EAAE,EAAE,KAAK,EAAE,OAAO,EAAE;IAC1B,MAAM,EAAE;QACN,MAAM,EAAE,CAAC,UAAU,EAAE,SAAS,CAAC;QAC/B,cAAc,EAAE,EAAE;KACnB;CACsB,CAAC;AAE1B,MAAM,WAAW,GAAG;IAClB,SAAS,EAAE,YAAY;IACvB,KAAK,EAAE,EAAE;IACT,YAAY,EAAE,IAAI;IAClB,gBAAgB,EAAE,IAAI;IACtB,QAAQ,EAAE,KAAK;IACf,eAAe,EAAE,KAAK;CACvB,CAAC;AAEF,QAAQ,CAAC,WAAW,EAAE,GAAG,EAAE;IACzB,UAAU,CAAC,GAAG,EAAE;QACd,EAAE,CAAC,aAAa,EAAE,CAAC;QAEnB,kBAAkB,CAAC,eAAe,CAAC,gBAAgB,CAAC,CAAC;QACrD,oBAAoB,CAAC,iBAAiB,CAAC,SAAS,CAAC,CAAC;QAElD,gBAAgB,CAAC,eAAe,CAAC,iBAAiB,CAAC;YACjD,GAAG,EAAE,EAAE;YACP,KAAK,EAAE,SAAS;YAChB,aAAa,EAAE,sBAAsB;YACrC,aAAa,EAAE,MAAM;YACrB,MAAM,EAAE,EAAE,IAAI,EAAE,WAAW,EAAE;YAC7B,SAAS,EAAE;gBACT,QAAQ,EAAE,UAAU;gBACpB,QAAQ,EAAE,UAAU;gBACpB,SAAS,EAAE,WAAW;aACvB;SACF,CAAC,CAAC;QAEH,gBAAgB,CAAC,YAAY,CAAC,iBAAiB,CAAC;YAC9C;gBACE,OAAO,EAAE,YAAY;gBACrB,OAAO,EAAE,YAAY;gBACrB,OAAO,EAAE,KAAK;gBACd,WAAW,EAAE,KAAK;gBAClB,WAAW,EAAE,KAAK;gBAClB,IAAI,EAAE,0DAA0D;aACjE;YACD;gBACE,OAAO,EAAE,YAAY;gBACrB,OAAO,EAAE,YAAY;gBACrB,OAAO,EAAE,KAAK;gBACd,WAAW,EAAE,KAAK;gBAClB,WAAW,EAAE,IAAI;gBACjB,IAAI,EAAE,0BAA0B;aACjC;SACF,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4EAA4E,EAAE,KAAK,IAAI,EAAE;QAC1F,MAAM,QAAQ,CAAC,UAAU,EAAE,WAAW,CAAC,CAAC;QAExC,MAAM,CAAC,kBAAkB,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;QACpD,MAAM,CAAC,gBAAgB,CAAC,eAAe,CAAC,CAAC,oBAAoB,CAAC,YAAY,EAAE,EAAE,CAAC,CAAC;QAChF,MAAM,CAAC,gBAAgB,CAAC,YAAY,CAAC,CAAC,oBAAoB,CAAC,YAAY,EAAE,EAAE,CAAC,CAAC;QAE7E,MAAM,cAAc,GAAG,EAAE,CAAC,MAAM,CAAC,oBAAoB,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAyB,CAAC;QAEhG,MAAM,CAAC,cAAc,CAAC,WAAW,CAAC,CAAC,OAAO,CACxC,MAAM,CAAC,gBAAgB,CAAC;YACtB,MAAM,EAAE,EAAE;YACV,KAAK,EAAE,SAAS;SACjB,CAAC,CACH,CAAC;QACF,MAAM,CAAC,cAAc,CAAC,YAAY,CAAC,CAAC,OAAO,CACzC,MAAM,CAAC,eAAe,CAAC;YACrB,MAAM,CAAC,gBAAgB,CAAC,EAAE,QAAQ,EAAE,YAAY,EAAE,MAAM,EAAE,UAAU,EAAE,CAAC;YACvE,MAAM,CAAC,gBAAgB,CAAC,EAAE,QAAQ,EAAE,YAAY,EAAE,MAAM,EAAE,SAAS,EAAE,CAAC;SACvE,CAAC,CACH,CAAC;QAEF,MAAM,CAAC,cAAc,CAAC,aAAa,EAAE,WAAW,CAAC,YAAY,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC9E,MAAM,CAAC,cAAc,CAAC,aAAa,EAAE,WAAW,CAAC,YAAY,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC9E,MAAM,CAAC,cAAc,CAAC,aAAa,EAAE,WAAW,CAAC,YAAY,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAChF,MAAM,CAAC,cAAc,CAAC,aAAa,EAAE,WAAW,CAAC,YAAY,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAE/E,MAAM,cAAc,GAAG,cAAc,CAAC,oBAAoB,EAAE,CAC1D;YACE,QAAQ,EAAE,SAAS;YACnB,QAAQ,EAAE,MAAM;YAChB,KAAK,EAAE,eAAe;YACtB,IAAI,EAAE,YAAY;YAClB,IAAI,EAAE,CAAC;YACP,OAAO,EAAE,SAAS;YAClB,QAAQ,EAAE,UAAU;YACpB,KAAK,EAAE,SAAS;SACF,EAChB;YACE,SAAS,EAAE;gBACT,QAAQ,EAAE,UAAU;gBACpB,QAAQ,EAAE,UAAU;gBACpB,SAAS,EAAE,WAAW;aACvB;SACF,CACF,CAAC;QAEF,MAAM,CAAC,cAAc,CAAC,CAAC,OAAO,CAAC;YAC7B,IAAI,EAAE,YAAY;YAClB,IAAI,EAAE,CAAC;YACP,OAAO,EAAE,UAAU;YACnB,OAAO,EAAE,UAAU;YACnB,QAAQ,EAAE,WAAW;SACtB,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+DAA+D,EAAE,KAAK,IAAI,EAAE;QAC7E,kBAAkB,CAAC,sBAAsB,CAAC,GAAG,EAAE;YAC7C,MAAM,IAAI,KAAK,CAAC,+CAA+C,CAAC,CAAC;QACnE,CAAC,CAAC,CAAC;QAEH,MAAM,MAAM,CAAC,QAAQ,CAAC,UAAU,EAAE,WAAW,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAAC,kBAAkB,CAAC,CAAC;QACpF,MAAM,CAAC,oBAAoB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IACtD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0DAA0D,EAAE,KAAK,IAAI,EAAE;QACxE,gBAAgB,CAAC,eAAe,CAAC,qBAAqB,CAAC,IAAI,KAAK,CAAC,kBAAkB,CAAC,CAAC,CAAC;QAEtF,MAAM,MAAM,CAAC,QAAQ,CAAC,UAAU,EAAE,WAAW,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAC7D,gDAAgD,CACjD,CAAC;QACF,MAAM,CAAC,oBAAoB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IACtD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4DAA4D,EAAE,KAAK,IAAI,EAAE;QAC1E,gBAAgB,CAAC,eAAe,CAAC,qBAAqB,CAAC,IAAI,KAAK,CAAC,eAAe,CAAC,CAAC,CAAC;QAEnF,MAAM,MAAM,CAAC,QAAQ,CAAC,UAAU,EAAE,WAAW,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAC7D,+CAA+C,CAChD,CAAC;QACF,MAAM,CAAC,oBAAoB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IACtD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yDAAyD,EAAE,KAAK,IAAI,EAAE;QACvE,gBAAgB,CAAC,eAAe,CAAC,qBAAqB,CAAC;YACrD,GAAG,EAAE,EAAE;YACP,KAAK,EAAE,cAAc;YACrB,aAAa,EAAE,sBAAsB;YACrC,aAAa,EAAE,MAAM;YACrB,MAAM,EAAE,EAAE,IAAI,EAAE,WAAW,EAAE;SAC9B,CAAC,CAAC;QAEH,MAAM,QAAQ,CAAC,UAAU,EAAE,WAAW,CAAC,CAAC;QAExC,MAAM,cAAc,GAAG,EAAE,CAAC,MAAM,CAAC,oBAAoB,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAyB,CAAC;QAChG,MAAM,CAAC,cAAc,CAAC,aAAa,EAAE,WAAW,CAAC,YAAY,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IACjF,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/cli/review-pr.d.ts

@@ -13,7 +13,7 @@ export interface ReviewPROptions {
     debug?: boolean;
 }
 /**
- * Review a GitHub pull request
+ * Review a GitHub pull request.
  */
 export declare function reviewPR(config: DRSConfig, options: ReviewPROptions): Promise<void>;
 //# sourceMappingURL=review-pr.d.ts.map

package/dist/cli/review-pr.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"review-pr.d.ts","sourceRoot":"","sources":["../../src/cli/review-pr.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AAOlD,MAAM,WAAW,eAAe;IAC9B,KAAK,EAAE,MAAM,CAAC;IACd,IAAI,EAAE,MAAM,CAAC;IACb,QAAQ,EAAE,MAAM,CAAC;IACjB,YAAY,EAAE,OAAO,CAAC;IACtB,gBAAgB,EAAE,OAAO,CAAC;IAC1B,QAAQ,EAAE,OAAO,CAAC;IAClB,eAAe,EAAE,OAAO,CAAC;IACzB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,KAAK,CAAC,EAAE,OAAO,CAAC;CACjB;AAwCD;;GAEG;AACH,wBAAsB,QAAQ,CAAC,MAAM,EAAE,SAAS,EAAE,OAAO,EAAE,eAAe,GAAG,OAAO,CAAC,IAAI,CAAC,CA0DzF"}
+{"version":3,"file":"review-pr.d.ts","sourceRoot":"","sources":["../../src/cli/review-pr.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AAYlD,MAAM,WAAW,eAAe;IAC9B,KAAK,EAAE,MAAM,CAAC;IACd,IAAI,EAAE,MAAM,CAAC;IACb,QAAQ,EAAE,MAAM,CAAC;IACjB,YAAY,EAAE,OAAO,CAAC;IACtB,gBAAgB,EAAE,OAAO,CAAC;IAC1B,QAAQ,EAAE,OAAO,CAAC;IAClB,eAAe,EAAE,OAAO,CAAC;IACzB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,KAAK,CAAC,EAAE,OAAO,CAAC;CACjB;AA8KD;;GAEG;AACH,wBAAsB,QAAQ,CAAC,MAAM,EAAE,SAAS,EAAE,OAAO,EAAE,eAAe,GAAG,OAAO,CAAC,IAAI,CAAC,CA6DzF"}