@diff-review-system/drs 3.3.0 → 4.0.0-rc.4
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/.pi/agents/task/agents-md-updater.md +24 -0
- package/.pi/agents/task/changelog-updater.md +29 -0
- package/.pi/agents/task/review-issue-fixer.md +25 -0
- package/.pi/workflows/github-pr-describe-post.yaml +24 -0
- package/.pi/workflows/github-pr-describe.yaml +23 -0
- package/.pi/workflows/github-pr-post-comment.yaml +19 -0
- package/.pi/workflows/github-pr-review-post.yaml +32 -0
- package/.pi/workflows/github-pr-review.yaml +23 -0
- package/.pi/workflows/github-pr-show-changes.yaml +25 -0
- package/.pi/workflows/gitlab-mr-describe-post.yaml +22 -0
- package/.pi/workflows/gitlab-mr-describe.yaml +21 -0
- package/.pi/workflows/gitlab-mr-post-comment.yaml +17 -0
- package/.pi/workflows/gitlab-mr-review-code-quality.yaml +31 -0
- package/.pi/workflows/gitlab-mr-review-post-code-quality.yaml +40 -0
- package/.pi/workflows/gitlab-mr-review-post.yaml +30 -0
- package/.pi/workflows/gitlab-mr-review.yaml +21 -0
- package/.pi/workflows/gitlab-mr-show-changes.yaml +23 -0
- package/.pi/workflows/local-changelog-update.yaml +23 -0
- package/.pi/workflows/local-fix-review-issues.yaml +42 -0
- package/.pi/workflows/local-review.yaml +17 -0
- package/.pi/workflows/local-staged-review.yaml +17 -0
- package/.pi/workflows/local-update-agents-md.yaml +24 -0
- package/.pi/workflows/tag-changelog-update.yaml +26 -0
- package/README.md +215 -106
- package/dist/ci/runner.d.ts.map +1 -1
- package/dist/ci/runner.js +7 -8
- package/dist/ci/runner.js.map +1 -1
- package/dist/cli/index.js +69 -341
- package/dist/cli/index.js.map +1 -1
- package/dist/cli/init.d.ts.map +1 -1
- package/dist/cli/init.js +25 -23
- package/dist/cli/init.js.map +1 -1
- package/dist/cli/run-agent.d.ts +24 -0
- package/dist/cli/run-agent.d.ts.map +1 -0
- package/dist/cli/run-agent.js +139 -0
- package/dist/cli/run-agent.js.map +1 -0
- package/dist/cli/run-agent.test.d.ts +2 -0
- package/dist/cli/run-agent.test.d.ts.map +1 -0
- package/dist/cli/run-agent.test.js +204 -0
- package/dist/cli/run-agent.test.js.map +1 -0
- package/dist/cli/workflow.d.ts +51 -0
- package/dist/cli/workflow.d.ts.map +1 -0
- package/dist/cli/workflow.js +1229 -0
- package/dist/cli/workflow.js.map +1 -0
- package/dist/cli/workflow.test.d.ts +2 -0
- package/dist/cli/workflow.test.d.ts.map +1 -0
- package/dist/cli/workflow.test.js +1410 -0
- package/dist/cli/workflow.test.js.map +1 -0
- package/dist/lib/agent-id.d.ts +9 -0
- package/dist/lib/agent-id.d.ts.map +1 -0
- package/dist/lib/agent-id.js +32 -0
- package/dist/lib/agent-id.js.map +1 -0
- package/dist/lib/comment-formatter.d.ts +7 -1
- package/dist/lib/comment-formatter.d.ts.map +1 -1
- package/dist/lib/comment-formatter.js +42 -1
- package/dist/lib/comment-formatter.js.map +1 -1
- package/dist/lib/comment-formatter.test.js +33 -0
- package/dist/lib/comment-formatter.test.js.map +1 -1
- package/dist/lib/comment-manager.d.ts +4 -0
- package/dist/lib/comment-manager.d.ts.map +1 -1
- package/dist/lib/comment-manager.js +7 -1
- package/dist/lib/comment-manager.js.map +1 -1
- package/dist/lib/comment-poster.d.ts +2 -2
- package/dist/lib/comment-poster.d.ts.map +1 -1
- package/dist/lib/comment-poster.js +3 -3
- package/dist/lib/comment-poster.js.map +1 -1
- package/dist/lib/comment-poster.test.js +13 -3
- package/dist/lib/comment-poster.test.js.map +1 -1
- package/dist/lib/config-model-overrides.test.d.ts +0 -10
- package/dist/lib/config-model-overrides.test.d.ts.map +1 -1
- package/dist/lib/config-model-overrides.test.js +174 -210
- package/dist/lib/config-model-overrides.test.js.map +1 -1
- package/dist/lib/config.d.ts +111 -34
- package/dist/lib/config.d.ts.map +1 -1
- package/dist/lib/config.js +322 -83
- package/dist/lib/config.js.map +1 -1
- package/dist/lib/config.test.js +282 -2
- package/dist/lib/config.test.js.map +1 -1
- package/dist/lib/context-loader.d.ts +4 -4
- package/dist/lib/context-loader.d.ts.map +1 -1
- package/dist/lib/context-loader.js +10 -7
- package/dist/lib/context-loader.js.map +1 -1
- package/dist/lib/context-loader.test.js +31 -26
- package/dist/lib/context-loader.test.js.map +1 -1
- package/dist/lib/description-executor.js +1 -1
- package/dist/lib/description-executor.js.map +1 -1
- package/dist/lib/description-executor.test.js +10 -3
- package/dist/lib/description-executor.test.js.map +1 -1
- package/dist/lib/diff-lines.d.ts +9 -0
- package/dist/lib/diff-lines.d.ts.map +1 -0
- package/dist/lib/diff-lines.js +32 -0
- package/dist/lib/diff-lines.js.map +1 -0
- package/dist/lib/diff-lines.test.d.ts +2 -0
- package/dist/lib/diff-lines.test.d.ts.map +1 -0
- package/dist/lib/diff-lines.test.js +13 -0
- package/dist/lib/diff-lines.test.js.map +1 -0
- package/dist/lib/logger.d.ts +1 -1
- package/dist/lib/logger.d.ts.map +1 -1
- package/dist/lib/review-core.d.ts.map +1 -1
- package/dist/lib/review-core.js +22 -27
- package/dist/lib/review-core.js.map +1 -1
- package/dist/lib/review-core.test.js +37 -23
- package/dist/lib/review-core.test.js.map +1 -1
- package/dist/lib/review-orchestrator.d.ts.map +1 -1
- package/dist/lib/review-orchestrator.js +11 -8
- package/dist/lib/review-orchestrator.js.map +1 -1
- package/dist/lib/review-orchestrator.test.js +27 -6
- package/dist/lib/review-orchestrator.test.js.map +1 -1
- package/dist/lib/unified-review-executor.d.ts +0 -2
- package/dist/lib/unified-review-executor.d.ts.map +1 -1
- package/dist/lib/unified-review-executor.js +7 -13
- package/dist/lib/unified-review-executor.js.map +1 -1
- package/dist/lib/unified-review-executor.test.js +2 -2
- package/dist/lib/unified-review-executor.test.js.map +1 -1
- package/dist/pi/sdk.d.ts.map +1 -1
- package/dist/pi/sdk.js +37 -12
- package/dist/pi/sdk.js.map +1 -1
- package/dist/pi/sdk.test.js +83 -10
- package/dist/pi/sdk.test.js.map +1 -1
- package/dist/runtime/agent-loader.d.ts +10 -6
- package/dist/runtime/agent-loader.d.ts.map +1 -1
- package/dist/runtime/agent-loader.js +53 -27
- package/dist/runtime/agent-loader.js.map +1 -1
- package/dist/runtime/agent-loader.test.js +116 -119
- package/dist/runtime/agent-loader.test.js.map +1 -1
- package/dist/runtime/built-in-paths.d.ts +1 -0
- package/dist/runtime/built-in-paths.d.ts.map +1 -1
- package/dist/runtime/built-in-paths.js +7 -0
- package/dist/runtime/built-in-paths.js.map +1 -1
- package/dist/runtime/client.d.ts +12 -0
- package/dist/runtime/client.d.ts.map +1 -1
- package/dist/runtime/client.js +83 -58
- package/dist/runtime/client.js.map +1 -1
- package/dist/runtime/client.test.js +264 -15
- package/dist/runtime/client.test.js.map +1 -1
- package/dist/runtime/path-config.d.ts +2 -2
- package/dist/runtime/path-config.d.ts.map +1 -1
- package/dist/runtime/path-config.js +8 -8
- package/dist/runtime/path-config.js.map +1 -1
- package/dist/runtime/path-config.test.js +14 -14
- package/dist/runtime/path-config.test.js.map +1 -1
- package/package.json +3 -4
- package/.pi/agents/review/documentation.md +0 -56
- package/.pi/agents/review/performance.md +0 -53
- package/.pi/agents/review/quality.md +0 -59
- package/.pi/agents/review/security.md +0 -53
- package/.pi/agents/review/style.md +0 -132
- package/dist/cli/describe-mr.d.ts +0 -11
- package/dist/cli/describe-mr.d.ts.map +0 -1
- package/dist/cli/describe-mr.js +0 -134
- package/dist/cli/describe-mr.js.map +0 -1
- package/dist/cli/describe-pr.d.ts +0 -12
- package/dist/cli/describe-pr.d.ts.map +0 -1
- package/dist/cli/describe-pr.js +0 -135
- package/dist/cli/describe-pr.js.map +0 -1
- package/dist/cli/post-comments.d.ts +0 -20
- package/dist/cli/post-comments.d.ts.map +0 -1
- package/dist/cli/post-comments.js +0 -225
- package/dist/cli/post-comments.js.map +0 -1
- package/dist/cli/review-local.d.ts +0 -13
- package/dist/cli/review-local.d.ts.map +0 -1
- package/dist/cli/review-local.integration.test.d.ts +0 -2
- package/dist/cli/review-local.integration.test.d.ts.map +0 -1
- package/dist/cli/review-local.integration.test.js +0 -343
- package/dist/cli/review-local.integration.test.js.map +0 -1
- package/dist/cli/review-local.js +0 -90
- package/dist/cli/review-local.js.map +0 -1
- package/dist/cli/review-local.live.e2e.test.d.ts +0 -2
- package/dist/cli/review-local.live.e2e.test.d.ts.map +0 -1
- package/dist/cli/review-local.live.e2e.test.js +0 -153
- package/dist/cli/review-local.live.e2e.test.js.map +0 -1
- package/dist/cli/review-local.test.d.ts +0 -2
- package/dist/cli/review-local.test.d.ts.map +0 -1
- package/dist/cli/review-local.test.js +0 -164
- package/dist/cli/review-local.test.js.map +0 -1
- package/dist/cli/review-mr.d.ts +0 -22
- package/dist/cli/review-mr.d.ts.map +0 -1
- package/dist/cli/review-mr.js +0 -181
- package/dist/cli/review-mr.js.map +0 -1
- package/dist/cli/review-mr.test.d.ts +0 -2
- package/dist/cli/review-mr.test.d.ts.map +0 -1
- package/dist/cli/review-mr.test.js +0 -142
- package/dist/cli/review-mr.test.js.map +0 -1
- package/dist/cli/review-pr.d.ts +0 -22
- package/dist/cli/review-pr.d.ts.map +0 -1
- package/dist/cli/review-pr.js +0 -181
- package/dist/cli/review-pr.js.map +0 -1
- package/dist/cli/review-pr.test.d.ts +0 -2
- package/dist/cli/review-pr.test.d.ts.map +0 -1
- package/dist/cli/review-pr.test.js +0 -137
- package/dist/cli/review-pr.test.js.map +0 -1
- package/dist/cli/review-url.d.ts +0 -35
- package/dist/cli/review-url.d.ts.map +0 -1
- package/dist/cli/review-url.js +0 -110
- package/dist/cli/review-url.js.map +0 -1
- package/dist/cli/review-url.test.d.ts +0 -2
- package/dist/cli/review-url.test.d.ts.map +0 -1
- package/dist/cli/review-url.test.js +0 -132
- package/dist/cli/review-url.test.js.map +0 -1
- package/dist/cli/show-changes.d.ts +0 -15
- package/dist/cli/show-changes.d.ts.map +0 -1
- package/dist/cli/show-changes.js +0 -184
- package/dist/cli/show-changes.js.map +0 -1
|
@@ -1,164 +0,0 @@
|
|
|
1
|
-
import { beforeEach, describe, expect, it, vi } from 'vitest';
|
|
2
|
-
import { reviewLocal } from './review-local.js';
|
|
3
|
-
const { mockGit, parseDiff, getChangedFiles, getFilesWithDiffs, executeReview, displayReviewSummary, hasBlockingIssues, formatReviewJson, writeReviewJson, printReviewJson, formatTerminalIssue, } = vi.hoisted(() => ({
|
|
4
|
-
mockGit: {
|
|
5
|
-
checkIsRepo: vi.fn(),
|
|
6
|
-
diff: vi.fn(),
|
|
7
|
-
},
|
|
8
|
-
parseDiff: vi.fn(),
|
|
9
|
-
getChangedFiles: vi.fn(),
|
|
10
|
-
getFilesWithDiffs: vi.fn(),
|
|
11
|
-
executeReview: vi.fn(),
|
|
12
|
-
displayReviewSummary: vi.fn(),
|
|
13
|
-
hasBlockingIssues: vi.fn(),
|
|
14
|
-
formatReviewJson: vi.fn(),
|
|
15
|
-
writeReviewJson: vi.fn(),
|
|
16
|
-
printReviewJson: vi.fn(),
|
|
17
|
-
formatTerminalIssue: vi.fn(),
|
|
18
|
-
}));
|
|
19
|
-
vi.mock('simple-git', () => ({
|
|
20
|
-
default: vi.fn(() => mockGit),
|
|
21
|
-
}));
|
|
22
|
-
vi.mock('../lib/diff-parser.js', () => ({
|
|
23
|
-
parseDiff,
|
|
24
|
-
getChangedFiles,
|
|
25
|
-
getFilesWithDiffs,
|
|
26
|
-
}));
|
|
27
|
-
vi.mock('../lib/review-orchestrator.js', () => ({
|
|
28
|
-
executeReview,
|
|
29
|
-
displayReviewSummary,
|
|
30
|
-
hasBlockingIssues,
|
|
31
|
-
}));
|
|
32
|
-
vi.mock('../lib/json-output.js', () => ({
|
|
33
|
-
formatReviewJson,
|
|
34
|
-
writeReviewJson,
|
|
35
|
-
printReviewJson,
|
|
36
|
-
}));
|
|
37
|
-
vi.mock('../lib/comment-formatter.js', () => ({
|
|
38
|
-
formatTerminalIssue,
|
|
39
|
-
}));
|
|
40
|
-
const baseConfig = {
|
|
41
|
-
pi: {},
|
|
42
|
-
gitlab: { url: '', token: '' },
|
|
43
|
-
github: { token: '' },
|
|
44
|
-
review: {
|
|
45
|
-
agents: ['security', 'quality'],
|
|
46
|
-
ignorePatterns: ['*.test.ts'],
|
|
47
|
-
},
|
|
48
|
-
};
|
|
49
|
-
const emptySummary = {
|
|
50
|
-
filesReviewed: 1,
|
|
51
|
-
issuesFound: 0,
|
|
52
|
-
bySeverity: { CRITICAL: 0, HIGH: 0, MEDIUM: 0, LOW: 0 },
|
|
53
|
-
byCategory: {
|
|
54
|
-
SECURITY: 0,
|
|
55
|
-
QUALITY: 0,
|
|
56
|
-
STYLE: 0,
|
|
57
|
-
PERFORMANCE: 0,
|
|
58
|
-
DOCUMENTATION: 0,
|
|
59
|
-
},
|
|
60
|
-
};
|
|
61
|
-
describe('review-local', () => {
|
|
62
|
-
beforeEach(() => {
|
|
63
|
-
vi.clearAllMocks();
|
|
64
|
-
vi.spyOn(console, 'log').mockImplementation(() => { });
|
|
65
|
-
mockGit.checkIsRepo.mockResolvedValue(true);
|
|
66
|
-
mockGit.diff.mockResolvedValue('diff --git a/src/app.ts b/src/app.ts');
|
|
67
|
-
parseDiff.mockReturnValue([{ filename: 'src/app.ts' }]);
|
|
68
|
-
getChangedFiles.mockReturnValue(['src/app.ts']);
|
|
69
|
-
getFilesWithDiffs.mockReturnValue([{ filename: 'src/app.ts', patch: '+const answer = 42;' }]);
|
|
70
|
-
executeReview.mockResolvedValue({
|
|
71
|
-
issues: [],
|
|
72
|
-
summary: emptySummary,
|
|
73
|
-
filesReviewed: 1,
|
|
74
|
-
});
|
|
75
|
-
hasBlockingIssues.mockReturnValue(false);
|
|
76
|
-
formatReviewJson.mockReturnValue({ result: 'ok' });
|
|
77
|
-
writeReviewJson.mockResolvedValue(undefined);
|
|
78
|
-
printReviewJson.mockImplementation(() => { });
|
|
79
|
-
displayReviewSummary.mockImplementation(() => { });
|
|
80
|
-
formatTerminalIssue.mockReturnValue('formatted issue');
|
|
81
|
-
});
|
|
82
|
-
it('runs staged reviews through the shared review executor', async () => {
|
|
83
|
-
await reviewLocal(baseConfig, { staged: true });
|
|
84
|
-
expect(mockGit.diff).toHaveBeenCalledWith(['--cached']);
|
|
85
|
-
expect(executeReview).toHaveBeenCalledWith(expect.objectContaining({
|
|
86
|
-
review: expect.objectContaining({
|
|
87
|
-
agents: ['security', 'quality'],
|
|
88
|
-
}),
|
|
89
|
-
}), expect.objectContaining({
|
|
90
|
-
name: 'Local staged diff',
|
|
91
|
-
files: ['src/app.ts'],
|
|
92
|
-
filesWithDiffs: [{ filename: 'src/app.ts', patch: '+const answer = 42;' }],
|
|
93
|
-
staged: true,
|
|
94
|
-
workingDir: process.cwd(),
|
|
95
|
-
}));
|
|
96
|
-
});
|
|
97
|
-
it('throws when executed outside a git repository', async () => {
|
|
98
|
-
mockGit.checkIsRepo.mockResolvedValue(false);
|
|
99
|
-
await expect(reviewLocal(baseConfig, { staged: false })).rejects.toThrow('Not a git repository');
|
|
100
|
-
});
|
|
101
|
-
it('returns early when there are no local changes', async () => {
|
|
102
|
-
mockGit.diff.mockResolvedValue(' ');
|
|
103
|
-
await reviewLocal(baseConfig, { staged: false });
|
|
104
|
-
expect(parseDiff).not.toHaveBeenCalled();
|
|
105
|
-
expect(executeReview).not.toHaveBeenCalled();
|
|
106
|
-
});
|
|
107
|
-
it('exits with code 1 when blocking issues are found', async () => {
|
|
108
|
-
executeReview.mockResolvedValueOnce({
|
|
109
|
-
issues: [
|
|
110
|
-
{
|
|
111
|
-
category: 'SECURITY',
|
|
112
|
-
severity: 'HIGH',
|
|
113
|
-
title: 'SQL Injection risk',
|
|
114
|
-
file: 'src/app.ts',
|
|
115
|
-
line: 10,
|
|
116
|
-
problem: 'Unsanitized SQL input',
|
|
117
|
-
solution: 'Use parameterized query',
|
|
118
|
-
agent: 'security',
|
|
119
|
-
},
|
|
120
|
-
],
|
|
121
|
-
summary: {
|
|
122
|
-
...emptySummary,
|
|
123
|
-
issuesFound: 1,
|
|
124
|
-
bySeverity: { CRITICAL: 0, HIGH: 1, MEDIUM: 0, LOW: 0 },
|
|
125
|
-
byCategory: { SECURITY: 1, QUALITY: 0, STYLE: 0, PERFORMANCE: 0, DOCUMENTATION: 0 },
|
|
126
|
-
},
|
|
127
|
-
filesReviewed: 1,
|
|
128
|
-
});
|
|
129
|
-
hasBlockingIssues.mockReturnValueOnce(true);
|
|
130
|
-
const exitSpy = vi.spyOn(process, 'exit').mockImplementation((() => undefined));
|
|
131
|
-
await reviewLocal(baseConfig, { staged: false });
|
|
132
|
-
expect(exitSpy).toHaveBeenCalledWith(1);
|
|
133
|
-
exitSpy.mockRestore();
|
|
134
|
-
});
|
|
135
|
-
it('writes and prints JSON output when requested', async () => {
|
|
136
|
-
await reviewLocal(baseConfig, {
|
|
137
|
-
staged: false,
|
|
138
|
-
outputPath: '.drs/review-output.json',
|
|
139
|
-
jsonOutput: true,
|
|
140
|
-
});
|
|
141
|
-
expect(formatReviewJson).toHaveBeenCalledWith(emptySummary, [], {
|
|
142
|
-
source: 'local-unstaged',
|
|
143
|
-
}, undefined);
|
|
144
|
-
expect(writeReviewJson).toHaveBeenCalledWith({ result: 'ok' }, '.drs/review-output.json', process.cwd());
|
|
145
|
-
expect(printReviewJson).toHaveBeenCalledWith({ result: 'ok' });
|
|
146
|
-
});
|
|
147
|
-
it('writes JSON output when only outputPath is provided', async () => {
|
|
148
|
-
await reviewLocal(baseConfig, {
|
|
149
|
-
staged: false,
|
|
150
|
-
outputPath: '.drs/review-output.json',
|
|
151
|
-
jsonOutput: false,
|
|
152
|
-
});
|
|
153
|
-
expect(formatReviewJson).toHaveBeenCalledWith(emptySummary, [], {
|
|
154
|
-
source: 'local-unstaged',
|
|
155
|
-
}, undefined);
|
|
156
|
-
expect(writeReviewJson).toHaveBeenCalledWith({ result: 'ok' }, '.drs/review-output.json', process.cwd());
|
|
157
|
-
expect(printReviewJson).not.toHaveBeenCalled();
|
|
158
|
-
});
|
|
159
|
-
it('surfaces runtime failures from review execution', async () => {
|
|
160
|
-
executeReview.mockRejectedValueOnce(new Error('Pi runtime unavailable'));
|
|
161
|
-
await expect(reviewLocal(baseConfig, { staged: false })).rejects.toThrow('Pi runtime unavailable');
|
|
162
|
-
});
|
|
163
|
-
});
|
|
164
|
-
//# sourceMappingURL=review-local.test.js.map
|
|
@@ -1 +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
DELETED
|
@@ -1,22 +0,0 @@
|
|
|
1
|
-
import type { DRSConfig } from '../lib/config.js';
|
|
2
|
-
export interface ReviewMROptions {
|
|
3
|
-
projectId: string;
|
|
4
|
-
mrIid: number;
|
|
5
|
-
postComments: boolean;
|
|
6
|
-
postErrorComment: boolean;
|
|
7
|
-
fixInCursor?: boolean;
|
|
8
|
-
skipFixInCursor?: boolean;
|
|
9
|
-
describe: boolean;
|
|
10
|
-
postDescription: boolean;
|
|
11
|
-
codeQualityReport?: string;
|
|
12
|
-
outputPath?: string;
|
|
13
|
-
jsonOutput?: boolean;
|
|
14
|
-
baseBranch?: string;
|
|
15
|
-
debug?: boolean;
|
|
16
|
-
thinkingLevel?: string;
|
|
17
|
-
}
|
|
18
|
-
/**
|
|
19
|
-
* Review a GitLab merge request.
|
|
20
|
-
*/
|
|
21
|
-
export declare function reviewMR(config: DRSConfig, options: ReviewMROptions): Promise<void>;
|
|
22
|
-
//# sourceMappingURL=review-mr.d.ts.map
|
|
@@ -1 +0,0 @@
|
|
|
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;AAalD,MAAM,WAAW,eAAe;IAC9B,SAAS,EAAE,MAAM,CAAC;IAClB,KAAK,EAAE,MAAM,CAAC;IACd,YAAY,EAAE,OAAO,CAAC;IACtB,gBAAgB,EAAE,OAAO,CAAC;IAC1B,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,eAAe,CAAC,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;IAChB,aAAa,CAAC,EAAE,MAAM,CAAC;CACxB;AAmKD;;GAEG;AACH,wBAAsB,QAAQ,CAAC,MAAM,EAAE,SAAS,EAAE,OAAO,EAAE,eAAe,GAAG,OAAO,CAAC,IAAI,CAAC,CAmFzF"}
|
package/dist/cli/review-mr.js
DELETED
|
@@ -1,181 +0,0 @@
|
|
|
1
|
-
import { resolveCursorFixLinkOptions } from '../lib/cursor-fix-link.js';
|
|
2
|
-
import { createGitLabClient } from '../gitlab/client.js';
|
|
3
|
-
import { GitLabPlatformAdapter } from '../gitlab/platform-adapter.js';
|
|
4
|
-
import { executeUnifiedReview } from '../lib/unified-review-executor.js';
|
|
5
|
-
/**
|
|
6
|
-
* Parse a GitLab diff to extract valid line numbers for review comments.
|
|
7
|
-
* GitLab only allows comments on lines that are in the diff (added or context).
|
|
8
|
-
*/
|
|
9
|
-
function parseValidLinesFromDiff(diff) {
|
|
10
|
-
const validLines = new Set();
|
|
11
|
-
const lines = diff.split('\n');
|
|
12
|
-
let currentLine = 0;
|
|
13
|
-
for (const line of lines) {
|
|
14
|
-
// Parse hunk header: @@ -old_start,old_count +new_start,new_count @@
|
|
15
|
-
const hunkMatch = line.match(/^@@ -\d+(?:,\d+)? \+(\d+)(?:,\d+)? @@/);
|
|
16
|
-
if (hunkMatch) {
|
|
17
|
-
currentLine = parseInt(hunkMatch[1], 10);
|
|
18
|
-
continue;
|
|
19
|
-
}
|
|
20
|
-
// Skip empty lines or lines without proper diff prefix
|
|
21
|
-
if (!line || line.length === 0)
|
|
22
|
-
continue;
|
|
23
|
-
const prefix = line[0];
|
|
24
|
-
if (prefix === '+') {
|
|
25
|
-
// Added line - can comment on this
|
|
26
|
-
validLines.add(currentLine);
|
|
27
|
-
currentLine++;
|
|
28
|
-
}
|
|
29
|
-
else if (prefix === ' ') {
|
|
30
|
-
// Context line - can comment on this
|
|
31
|
-
validLines.add(currentLine);
|
|
32
|
-
currentLine++;
|
|
33
|
-
}
|
|
34
|
-
else if (prefix === '-') {
|
|
35
|
-
// Removed line - cannot comment on "new" version, skip
|
|
36
|
-
continue;
|
|
37
|
-
}
|
|
38
|
-
}
|
|
39
|
-
return validLines;
|
|
40
|
-
}
|
|
41
|
-
function parseStatusCodeFromMessage(message) {
|
|
42
|
-
const match = message.match(/\b(401|403|404|429)\b/);
|
|
43
|
-
return match ? parseInt(match[1], 10) : undefined;
|
|
44
|
-
}
|
|
45
|
-
function extractStatusCode(error) {
|
|
46
|
-
if (!error || typeof error !== 'object') {
|
|
47
|
-
return undefined;
|
|
48
|
-
}
|
|
49
|
-
const candidate = error;
|
|
50
|
-
if (typeof candidate.statusCode === 'number') {
|
|
51
|
-
return candidate.statusCode;
|
|
52
|
-
}
|
|
53
|
-
if (typeof candidate.status === 'number') {
|
|
54
|
-
return candidate.status;
|
|
55
|
-
}
|
|
56
|
-
if (candidate.response && typeof candidate.response === 'object') {
|
|
57
|
-
if (typeof candidate.response.statusCode === 'number') {
|
|
58
|
-
return candidate.response.statusCode;
|
|
59
|
-
}
|
|
60
|
-
if (typeof candidate.response.status === 'number') {
|
|
61
|
-
return candidate.response.status;
|
|
62
|
-
}
|
|
63
|
-
}
|
|
64
|
-
if (candidate.cause && candidate.cause !== error) {
|
|
65
|
-
return extractStatusCode(candidate.cause);
|
|
66
|
-
}
|
|
67
|
-
return undefined;
|
|
68
|
-
}
|
|
69
|
-
function mapGitLabContextError(error, options) {
|
|
70
|
-
const rawMessage = error instanceof Error ? error.message : String(error);
|
|
71
|
-
const normalized = rawMessage.toLowerCase();
|
|
72
|
-
const statusCode = extractStatusCode(error) ?? parseStatusCodeFromMessage(rawMessage);
|
|
73
|
-
const gitlabUrl = process.env.GITLAB_URL ?? 'https://gitlab.com';
|
|
74
|
-
if (normalized.includes('gitlab_token environment variable is required')) {
|
|
75
|
-
return new Error('GitLab authentication is required. Set GITLAB_TOKEN with an API token that can access the target merge request.');
|
|
76
|
-
}
|
|
77
|
-
if (statusCode === 401 || normalized.includes('unauthorized')) {
|
|
78
|
-
return new Error(`GitLab authentication failed for ${options.projectId}!${options.mrIid}. Verify GITLAB_TOKEN for ${gitlabUrl} and ensure it has API access.`);
|
|
79
|
-
}
|
|
80
|
-
if (statusCode === 403 || normalized.includes('forbidden')) {
|
|
81
|
-
return new Error(`GitLab authorization failed for ${options.projectId}!${options.mrIid}. Ensure the token has permission to read the project and merge request.`);
|
|
82
|
-
}
|
|
83
|
-
if (statusCode === 404 || normalized.includes('not found')) {
|
|
84
|
-
return new Error(`GitLab merge request not found: ${options.projectId}!${options.mrIid}. Verify --project/--mr values and that the token can access the project.`);
|
|
85
|
-
}
|
|
86
|
-
if (statusCode === 429 || normalized.includes('rate limit')) {
|
|
87
|
-
return new Error(`GitLab API rate limit reached while loading ${options.projectId}!${options.mrIid}. Retry after cooldown or use a token with higher limits.`);
|
|
88
|
-
}
|
|
89
|
-
const connectivityError = normalized.includes('fetch failed') ||
|
|
90
|
-
normalized.includes('econnrefused') ||
|
|
91
|
-
normalized.includes('enotfound') ||
|
|
92
|
-
normalized.includes('etimedout');
|
|
93
|
-
if (connectivityError) {
|
|
94
|
-
return new Error(`Unable to reach GitLab at ${gitlabUrl} while loading ${options.projectId}!${options.mrIid}. Check GITLAB_URL and network connectivity.`);
|
|
95
|
-
}
|
|
96
|
-
return new Error(`Failed to load GitLab merge request context for ${options.projectId}!${options.mrIid}: ${rawMessage}`);
|
|
97
|
-
}
|
|
98
|
-
async function loadMergeRequestContext(platformClient, options) {
|
|
99
|
-
try {
|
|
100
|
-
const pullRequest = await platformClient.getPullRequest(options.projectId, options.mrIid);
|
|
101
|
-
const changedFiles = await platformClient.getChangedFiles(options.projectId, options.mrIid);
|
|
102
|
-
return {
|
|
103
|
-
pullRequest,
|
|
104
|
-
changedFiles,
|
|
105
|
-
};
|
|
106
|
-
}
|
|
107
|
-
catch (error) {
|
|
108
|
-
throw mapGitLabContextError(error, options);
|
|
109
|
-
}
|
|
110
|
-
}
|
|
111
|
-
/**
|
|
112
|
-
* Review a GitLab merge request.
|
|
113
|
-
*/
|
|
114
|
-
export async function reviewMR(config, options) {
|
|
115
|
-
let platformClient;
|
|
116
|
-
try {
|
|
117
|
-
const gitlabClient = createGitLabClient();
|
|
118
|
-
platformClient = new GitLabPlatformAdapter(gitlabClient);
|
|
119
|
-
}
|
|
120
|
-
catch (error) {
|
|
121
|
-
throw mapGitLabContextError(error, options);
|
|
122
|
-
}
|
|
123
|
-
const workingDir = process.cwd();
|
|
124
|
-
const { pullRequest, changedFiles } = await loadMergeRequestContext(platformClient, options);
|
|
125
|
-
const cursorFixLinks = resolveCursorFixLinkOptions(config, options.projectId, workingDir, options.fixInCursor, options.skipFixInCursor);
|
|
126
|
-
// Build a map of file -> valid line numbers (lines that are in the diff)
|
|
127
|
-
const validLinesMap = new Map();
|
|
128
|
-
for (const file of changedFiles) {
|
|
129
|
-
if (file.patch && file.status !== 'removed') {
|
|
130
|
-
const validLines = parseValidLinesFromDiff(file.patch);
|
|
131
|
-
validLinesMap.set(file.filename, validLines);
|
|
132
|
-
}
|
|
133
|
-
}
|
|
134
|
-
const platformData = pullRequest.platformData;
|
|
135
|
-
const diffRefs = platformData?.diff_refs;
|
|
136
|
-
// Create line validator - only allow comments on lines that are in the diff
|
|
137
|
-
const lineValidator = {
|
|
138
|
-
isValidLine(file, line) {
|
|
139
|
-
if (!diffRefs?.base_sha || !diffRefs.head_sha || !diffRefs.start_sha) {
|
|
140
|
-
return false;
|
|
141
|
-
}
|
|
142
|
-
const validLines = validLinesMap.get(file);
|
|
143
|
-
return validLines !== undefined && validLines.has(line);
|
|
144
|
-
},
|
|
145
|
-
};
|
|
146
|
-
// Create inline position builder
|
|
147
|
-
const createInlinePosition = (issue, platformDataInput) => {
|
|
148
|
-
const data = platformDataInput;
|
|
149
|
-
const refs = data.diff_refs;
|
|
150
|
-
return {
|
|
151
|
-
path: issue.file,
|
|
152
|
-
line: issue.line,
|
|
153
|
-
baseSha: refs?.base_sha,
|
|
154
|
-
headSha: refs?.head_sha,
|
|
155
|
-
startSha: refs?.start_sha,
|
|
156
|
-
};
|
|
157
|
-
};
|
|
158
|
-
// Execute unified review
|
|
159
|
-
await executeUnifiedReview(config, {
|
|
160
|
-
platformClient,
|
|
161
|
-
projectId: options.projectId,
|
|
162
|
-
prNumber: options.mrIid,
|
|
163
|
-
pullRequest,
|
|
164
|
-
changedFiles,
|
|
165
|
-
postComments: options.postComments,
|
|
166
|
-
postErrorComment: options.postErrorComment,
|
|
167
|
-
codeQualityReport: options.codeQualityReport,
|
|
168
|
-
outputPath: options.outputPath,
|
|
169
|
-
jsonOutput: options.jsonOutput,
|
|
170
|
-
baseBranch: options.baseBranch,
|
|
171
|
-
lineValidator,
|
|
172
|
-
createInlinePosition,
|
|
173
|
-
cursorFixLinks,
|
|
174
|
-
workingDir,
|
|
175
|
-
describe: options.describe,
|
|
176
|
-
postDescription: options.postDescription,
|
|
177
|
-
debug: options.debug,
|
|
178
|
-
thinkingLevel: options.thinkingLevel,
|
|
179
|
-
});
|
|
180
|
-
}
|
|
181
|
-
//# sourceMappingURL=review-mr.js.map
|
|
@@ -1 +0,0 @@
|
|
|
1
|
-
{"version":3,"file":"review-mr.js","sourceRoot":"","sources":["../../src/cli/review-mr.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,2BAA2B,EAAE,MAAM,2BAA2B,CAAC;AACxE,OAAO,EAAE,kBAAkB,EAAE,MAAM,qBAAqB,CAAC;AACzD,OAAO,EAAE,qBAAqB,EAAE,MAAM,+BAA+B,CAAC;AACtE,OAAO,EAAE,oBAAoB,EAAE,MAAM,mCAAmC,CAAC;AA0CzE;;;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,UAAU,GAAG,OAAO,CAAC,GAAG,EAAE,CAAC;IACjC,MAAM,EAAE,WAAW,EAAE,YAAY,EAAE,GAAG,MAAM,uBAAuB,CAAC,cAAc,EAAE,OAAO,CAAC,CAAC;IAC7F,MAAM,cAAc,GAAG,2BAA2B,CAChD,MAAM,EACN,OAAO,CAAC,SAAS,EACjB,UAAU,EACV,OAAO,CAAC,WAAW,EACnB,OAAO,CAAC,eAAe,CACxB,CAAC;IAEF,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,cAAc;QACd,UAAU;QACV,QAAQ,EAAE,OAAO,CAAC,QAAQ;QAC1B,eAAe,EAAE,OAAO,CAAC,eAAe;QACxC,KAAK,EAAE,OAAO,CAAC,KAAK;QACpB,aAAa,EAAE,OAAO,CAAC,aAAa;KACrC,CAAC,CAAC;AACL,CAAC"}
|
|
@@ -1 +0,0 @@
|
|
|
1
|
-
{"version":3,"file":"review-mr.test.d.ts","sourceRoot":"","sources":["../../src/cli/review-mr.test.ts"],"names":[],"mappings":""}
|
|
@@ -1,142 +0,0 @@
|
|
|
1
|
-
import { beforeEach, describe, expect, it, vi } from 'vitest';
|
|
2
|
-
import { reviewMR } from './review-mr.js';
|
|
3
|
-
const { mockGitLabClient, createGitLabClient, executeUnifiedReview } = vi.hoisted(() => ({
|
|
4
|
-
mockGitLabClient: {
|
|
5
|
-
getMergeRequest: vi.fn(),
|
|
6
|
-
getMRChanges: vi.fn(),
|
|
7
|
-
},
|
|
8
|
-
createGitLabClient: vi.fn(),
|
|
9
|
-
executeUnifiedReview: vi.fn(),
|
|
10
|
-
}));
|
|
11
|
-
vi.mock('../gitlab/client.js', () => ({
|
|
12
|
-
createGitLabClient,
|
|
13
|
-
}));
|
|
14
|
-
vi.mock('../lib/unified-review-executor.js', () => ({
|
|
15
|
-
executeUnifiedReview,
|
|
16
|
-
}));
|
|
17
|
-
const baseConfig = {
|
|
18
|
-
pi: {},
|
|
19
|
-
gitlab: { url: 'https://gitlab.com', token: 'token' },
|
|
20
|
-
github: { token: 'token' },
|
|
21
|
-
review: {
|
|
22
|
-
agents: ['security', 'quality'],
|
|
23
|
-
ignorePatterns: [],
|
|
24
|
-
},
|
|
25
|
-
};
|
|
26
|
-
const baseOptions = {
|
|
27
|
-
projectId: 'group/repo',
|
|
28
|
-
mrIid: 42,
|
|
29
|
-
postComments: true,
|
|
30
|
-
postErrorComment: true,
|
|
31
|
-
describe: false,
|
|
32
|
-
postDescription: false,
|
|
33
|
-
};
|
|
34
|
-
describe('review-mr', () => {
|
|
35
|
-
beforeEach(() => {
|
|
36
|
-
vi.clearAllMocks();
|
|
37
|
-
createGitLabClient.mockReturnValue(mockGitLabClient);
|
|
38
|
-
executeUnifiedReview.mockResolvedValue(undefined);
|
|
39
|
-
mockGitLabClient.getMergeRequest.mockResolvedValue({
|
|
40
|
-
iid: 42,
|
|
41
|
-
title: 'Test MR',
|
|
42
|
-
source_branch: 'feature/pi-migration',
|
|
43
|
-
target_branch: 'main',
|
|
44
|
-
author: { name: 'test-user' },
|
|
45
|
-
diff_refs: {
|
|
46
|
-
base_sha: 'base-sha',
|
|
47
|
-
head_sha: 'head-sha',
|
|
48
|
-
start_sha: 'start-sha',
|
|
49
|
-
},
|
|
50
|
-
});
|
|
51
|
-
mockGitLabClient.getMRChanges.mockResolvedValue([
|
|
52
|
-
{
|
|
53
|
-
oldPath: 'src/app.ts',
|
|
54
|
-
newPath: 'src/app.ts',
|
|
55
|
-
newFile: false,
|
|
56
|
-
renamedFile: false,
|
|
57
|
-
deletedFile: false,
|
|
58
|
-
diff: '@@ -8,2 +8,3 @@\n context line\n+new line\n-removed line',
|
|
59
|
-
},
|
|
60
|
-
{
|
|
61
|
-
oldPath: 'src/old.ts',
|
|
62
|
-
newPath: 'src/old.ts',
|
|
63
|
-
newFile: false,
|
|
64
|
-
renamedFile: false,
|
|
65
|
-
deletedFile: true,
|
|
66
|
-
diff: '@@ -1 +0,0 @@\n-old line',
|
|
67
|
-
},
|
|
68
|
-
]);
|
|
69
|
-
});
|
|
70
|
-
it('loads MR context once and forwards diff-aware validators to unified review', async () => {
|
|
71
|
-
await reviewMR(baseConfig, baseOptions);
|
|
72
|
-
expect(createGitLabClient).toHaveBeenCalledTimes(1);
|
|
73
|
-
expect(mockGitLabClient.getMergeRequest).toHaveBeenCalledWith('group/repo', 42);
|
|
74
|
-
expect(mockGitLabClient.getMRChanges).toHaveBeenCalledWith('group/repo', 42);
|
|
75
|
-
const unifiedOptions = vi.mocked(executeUnifiedReview).mock.calls[0][1];
|
|
76
|
-
expect(unifiedOptions.pullRequest).toEqual(expect.objectContaining({
|
|
77
|
-
number: 42,
|
|
78
|
-
title: 'Test MR',
|
|
79
|
-
}));
|
|
80
|
-
expect(unifiedOptions.changedFiles).toEqual(expect.arrayContaining([
|
|
81
|
-
expect.objectContaining({ filename: 'src/app.ts', status: 'modified' }),
|
|
82
|
-
expect.objectContaining({ filename: 'src/old.ts', status: 'removed' }),
|
|
83
|
-
]));
|
|
84
|
-
expect(unifiedOptions.lineValidator?.isValidLine('src/app.ts', 8)).toBe(true);
|
|
85
|
-
expect(unifiedOptions.lineValidator?.isValidLine('src/app.ts', 9)).toBe(true);
|
|
86
|
-
expect(unifiedOptions.lineValidator?.isValidLine('src/app.ts', 10)).toBe(false);
|
|
87
|
-
expect(unifiedOptions.lineValidator?.isValidLine('src/old.ts', 1)).toBe(false);
|
|
88
|
-
const inlinePosition = unifiedOptions.createInlinePosition?.({
|
|
89
|
-
category: 'QUALITY',
|
|
90
|
-
severity: 'HIGH',
|
|
91
|
-
title: 'Example issue',
|
|
92
|
-
file: 'src/app.ts',
|
|
93
|
-
line: 9,
|
|
94
|
-
problem: 'Problem',
|
|
95
|
-
solution: 'Solution',
|
|
96
|
-
agent: 'quality',
|
|
97
|
-
}, {
|
|
98
|
-
diff_refs: {
|
|
99
|
-
base_sha: 'base-sha',
|
|
100
|
-
head_sha: 'head-sha',
|
|
101
|
-
start_sha: 'start-sha',
|
|
102
|
-
},
|
|
103
|
-
});
|
|
104
|
-
expect(inlinePosition).toEqual({
|
|
105
|
-
path: 'src/app.ts',
|
|
106
|
-
line: 9,
|
|
107
|
-
baseSha: 'base-sha',
|
|
108
|
-
headSha: 'head-sha',
|
|
109
|
-
startSha: 'start-sha',
|
|
110
|
-
});
|
|
111
|
-
});
|
|
112
|
-
it('rejects with actionable guidance when GITLAB_TOKEN is missing', async () => {
|
|
113
|
-
createGitLabClient.mockImplementationOnce(() => {
|
|
114
|
-
throw new Error('GITLAB_TOKEN environment variable is required');
|
|
115
|
-
});
|
|
116
|
-
await expect(reviewMR(baseConfig, baseOptions)).rejects.toThrow('Set GITLAB_TOKEN');
|
|
117
|
-
expect(executeUnifiedReview).not.toHaveBeenCalled();
|
|
118
|
-
});
|
|
119
|
-
it('maps GitLab authentication failures to actionable errors', async () => {
|
|
120
|
-
mockGitLabClient.getMergeRequest.mockRejectedValueOnce(new Error('401 Unauthorized'));
|
|
121
|
-
await expect(reviewMR(baseConfig, baseOptions)).rejects.toThrow('GitLab authentication failed for group/repo!42');
|
|
122
|
-
expect(executeUnifiedReview).not.toHaveBeenCalled();
|
|
123
|
-
});
|
|
124
|
-
it('maps GitLab not found failures with project/mr remediation', async () => {
|
|
125
|
-
mockGitLabClient.getMergeRequest.mockRejectedValueOnce(new Error('404 Not Found'));
|
|
126
|
-
await expect(reviewMR(baseConfig, baseOptions)).rejects.toThrow('GitLab merge request not found: group/repo!42');
|
|
127
|
-
expect(executeUnifiedReview).not.toHaveBeenCalled();
|
|
128
|
-
});
|
|
129
|
-
it('disables line validation when diff refs are unavailable', async () => {
|
|
130
|
-
mockGitLabClient.getMergeRequest.mockResolvedValueOnce({
|
|
131
|
-
iid: 42,
|
|
132
|
-
title: 'Missing refs',
|
|
133
|
-
source_branch: 'feature/pi-migration',
|
|
134
|
-
target_branch: 'main',
|
|
135
|
-
author: { name: 'test-user' },
|
|
136
|
-
});
|
|
137
|
-
await reviewMR(baseConfig, baseOptions);
|
|
138
|
-
const unifiedOptions = vi.mocked(executeUnifiedReview).mock.calls[0][1];
|
|
139
|
-
expect(unifiedOptions.lineValidator?.isValidLine('src/app.ts', 8)).toBe(false);
|
|
140
|
-
});
|
|
141
|
-
});
|
|
142
|
-
//# sourceMappingURL=review-mr.test.js.map
|
|
@@ -1 +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
DELETED
|
@@ -1,22 +0,0 @@
|
|
|
1
|
-
import type { DRSConfig } from '../lib/config.js';
|
|
2
|
-
export interface ReviewPROptions {
|
|
3
|
-
owner: string;
|
|
4
|
-
repo: string;
|
|
5
|
-
prNumber: number;
|
|
6
|
-
postComments: boolean;
|
|
7
|
-
postErrorComment: boolean;
|
|
8
|
-
fixInCursor?: boolean;
|
|
9
|
-
skipFixInCursor?: boolean;
|
|
10
|
-
describe: boolean;
|
|
11
|
-
postDescription: boolean;
|
|
12
|
-
outputPath?: string;
|
|
13
|
-
jsonOutput?: boolean;
|
|
14
|
-
baseBranch?: string;
|
|
15
|
-
debug?: boolean;
|
|
16
|
-
thinkingLevel?: string;
|
|
17
|
-
}
|
|
18
|
-
/**
|
|
19
|
-
* Review a GitHub pull request.
|
|
20
|
-
*/
|
|
21
|
-
export declare function reviewPR(config: DRSConfig, options: ReviewPROptions): Promise<void>;
|
|
22
|
-
//# sourceMappingURL=review-pr.d.ts.map
|
|
@@ -1 +0,0 @@
|
|
|
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;AAalD,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,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,eAAe,CAAC,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;IAChB,aAAa,CAAC,EAAE,MAAM,CAAC;CACxB;AA8KD;;GAEG;AACH,wBAAsB,QAAQ,CAAC,MAAM,EAAE,SAAS,EAAE,OAAO,EAAE,eAAe,GAAG,OAAO,CAAC,IAAI,CAAC,CAuEzF"}
|