@diff-review-system/drs 1.1.2 → 2.1.0
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/.opencode/agent/describe/pr-describer.md +221 -0
- package/.opencode/agent/review/documentation.md +56 -0
- package/.opencode/agent/review/performance.md +32 -139
- package/.opencode/agent/review/quality.md +36 -113
- package/.opencode/agent/review/security.md +32 -97
- package/.opencode/agent/review/style.md +4 -0
- package/.opencode/agent/review/unified-reviewer.md +74 -0
- package/.opencode/opencode.jsonc +4 -29
- package/.opencode/tool/write_json_output.ts +24 -0
- package/README.md +48 -21
- package/dist/ci/runner.d.ts.map +1 -1
- package/dist/ci/runner.js +2 -0
- package/dist/ci/runner.js.map +1 -1
- package/dist/cli/describe-mr.d.ts +11 -0
- package/dist/cli/describe-mr.d.ts.map +1 -0
- package/dist/cli/describe-mr.js +104 -0
- package/dist/cli/describe-mr.js.map +1 -0
- package/dist/cli/describe-pr.d.ts +12 -0
- package/dist/cli/describe-pr.d.ts.map +1 -0
- package/dist/cli/describe-pr.js +105 -0
- package/dist/cli/describe-pr.js.map +1 -0
- package/dist/cli/index.js +222 -16
- package/dist/cli/index.js.map +1 -1
- package/dist/cli/init.d.ts +1 -1
- package/dist/cli/init.d.ts.map +1 -1
- package/dist/cli/init.js +273 -145
- package/dist/cli/init.js.map +1 -1
- package/dist/cli/post-comments.d.ts +16 -0
- package/dist/cli/post-comments.d.ts.map +1 -0
- package/dist/cli/post-comments.js +222 -0
- package/dist/cli/post-comments.js.map +1 -0
- package/dist/cli/review-local.d.ts +3 -0
- package/dist/cli/review-local.d.ts.map +1 -1
- package/dist/cli/review-local.js +44 -18
- package/dist/cli/review-local.js.map +1 -1
- package/dist/cli/review-mr.d.ts +6 -0
- package/dist/cli/review-mr.d.ts.map +1 -1
- package/dist/cli/review-mr.js +63 -7
- package/dist/cli/review-mr.js.map +1 -1
- package/dist/cli/review-pr.d.ts +6 -0
- package/dist/cli/review-pr.d.ts.map +1 -1
- package/dist/cli/review-pr.js +8 -1
- package/dist/cli/review-pr.js.map +1 -1
- package/dist/cli/show-changes.d.ts +15 -0
- package/dist/cli/show-changes.d.ts.map +1 -0
- package/dist/cli/show-changes.js +184 -0
- package/dist/cli/show-changes.js.map +1 -0
- package/dist/github/platform-adapter.d.ts.map +1 -1
- package/dist/github/platform-adapter.js +4 -2
- package/dist/github/platform-adapter.js.map +1 -1
- package/dist/gitlab/client.d.ts.map +1 -1
- package/dist/gitlab/client.js +1 -1
- package/dist/gitlab/client.js.map +1 -1
- package/dist/gitlab/platform-adapter.d.ts.map +1 -1
- package/dist/gitlab/platform-adapter.js +6 -5
- package/dist/gitlab/platform-adapter.js.map +1 -1
- package/dist/lib/change-summary.d.ts +8 -0
- package/dist/lib/change-summary.d.ts.map +1 -0
- package/dist/lib/change-summary.js +2 -0
- package/dist/lib/change-summary.js.map +1 -0
- package/dist/lib/comment-formatter.d.ts +3 -2
- package/dist/lib/comment-formatter.d.ts.map +1 -1
- package/dist/lib/comment-formatter.js +16 -2
- package/dist/lib/comment-formatter.js.map +1 -1
- package/dist/lib/comment-formatter.test.d.ts +2 -0
- package/dist/lib/comment-formatter.test.d.ts.map +1 -0
- package/dist/lib/comment-formatter.test.js +607 -0
- package/dist/lib/comment-formatter.test.js.map +1 -0
- package/dist/lib/comment-manager.test.d.ts +2 -0
- package/dist/lib/comment-manager.test.d.ts.map +1 -0
- package/dist/lib/comment-manager.test.js +618 -0
- package/dist/lib/comment-manager.test.js.map +1 -0
- package/dist/lib/comment-poster.d.ts +21 -0
- package/dist/lib/comment-poster.d.ts.map +1 -0
- package/dist/lib/comment-poster.js +96 -0
- package/dist/lib/comment-poster.js.map +1 -0
- package/dist/lib/comment-poster.test.d.ts +5 -0
- package/dist/lib/comment-poster.test.d.ts.map +1 -0
- package/dist/lib/comment-poster.test.js +215 -0
- package/dist/lib/comment-poster.test.js.map +1 -0
- package/dist/lib/config-model-overrides.test.js +71 -41
- package/dist/lib/config-model-overrides.test.js.map +1 -1
- package/dist/lib/config.d.ts +65 -7
- package/dist/lib/config.d.ts.map +1 -1
- package/dist/lib/config.js +122 -22
- package/dist/lib/config.js.map +1 -1
- package/dist/lib/config.test.js +31 -2
- package/dist/lib/config.test.js.map +1 -1
- package/dist/lib/context-compression.d.ts +19 -0
- package/dist/lib/context-compression.d.ts.map +1 -0
- package/dist/lib/context-compression.js +170 -0
- package/dist/lib/context-compression.js.map +1 -0
- package/dist/lib/context-compression.test.d.ts +2 -0
- package/dist/lib/context-compression.test.d.ts.map +1 -0
- package/dist/lib/context-compression.test.js +33 -0
- package/dist/lib/context-compression.test.js.map +1 -0
- package/dist/lib/context-loader.d.ts.map +1 -1
- package/dist/lib/context-loader.js +8 -1
- package/dist/lib/context-loader.js.map +1 -1
- package/dist/lib/context-loader.test.d.ts +2 -0
- package/dist/lib/context-loader.test.d.ts.map +1 -0
- package/dist/lib/context-loader.test.js +207 -0
- package/dist/lib/context-loader.test.js.map +1 -0
- package/dist/lib/describe-core.d.ts +9 -0
- package/dist/lib/describe-core.d.ts.map +1 -0
- package/dist/lib/describe-core.js +71 -0
- package/dist/lib/describe-core.js.map +1 -0
- package/dist/lib/describe-core.test.d.ts +2 -0
- package/dist/lib/describe-core.test.d.ts.map +1 -0
- package/dist/lib/describe-core.test.js +208 -0
- package/dist/lib/describe-core.test.js.map +1 -0
- package/dist/lib/describe-output-path.test.d.ts +2 -0
- package/dist/lib/describe-output-path.test.d.ts.map +1 -0
- package/dist/lib/describe-output-path.test.js +51 -0
- package/dist/lib/describe-output-path.test.js.map +1 -0
- package/dist/lib/describe-parser.d.ts +3 -0
- package/dist/lib/describe-parser.d.ts.map +1 -0
- package/dist/lib/describe-parser.js +163 -0
- package/dist/lib/describe-parser.js.map +1 -0
- package/dist/lib/describe-parser.test.d.ts +2 -0
- package/dist/lib/describe-parser.test.d.ts.map +1 -0
- package/dist/lib/describe-parser.test.js +282 -0
- package/dist/lib/describe-parser.test.js.map +1 -0
- package/dist/lib/description-executor.d.ts +22 -0
- package/dist/lib/description-executor.d.ts.map +1 -0
- package/dist/lib/description-executor.js +72 -0
- package/dist/lib/description-executor.js.map +1 -0
- package/dist/lib/description-formatter.d.ts +37 -0
- package/dist/lib/description-formatter.d.ts.map +1 -0
- package/dist/lib/description-formatter.js +219 -0
- package/dist/lib/description-formatter.js.map +1 -0
- package/dist/lib/diff-parser.d.ts +11 -0
- package/dist/lib/diff-parser.d.ts.map +1 -1
- package/dist/lib/diff-parser.js +37 -0
- package/dist/lib/diff-parser.js.map +1 -1
- package/dist/lib/issue-parser.d.ts +1 -1
- package/dist/lib/issue-parser.d.ts.map +1 -1
- package/dist/lib/issue-parser.js +16 -14
- package/dist/lib/issue-parser.js.map +1 -1
- package/dist/lib/json-output-schema.d.ts +207 -0
- package/dist/lib/json-output-schema.d.ts.map +1 -0
- package/dist/lib/json-output-schema.js +124 -0
- package/dist/lib/json-output-schema.js.map +1 -0
- package/dist/lib/json-output-schema.test.d.ts +2 -0
- package/dist/lib/json-output-schema.test.d.ts.map +1 -0
- package/dist/lib/json-output-schema.test.js +92 -0
- package/dist/lib/json-output-schema.test.js.map +1 -0
- package/dist/lib/json-output.d.ts +43 -0
- package/dist/lib/json-output.d.ts.map +1 -0
- package/dist/lib/json-output.js +34 -0
- package/dist/lib/json-output.js.map +1 -0
- package/dist/lib/output-paths.d.ts +6 -0
- package/dist/lib/output-paths.d.ts.map +1 -0
- package/dist/lib/output-paths.js +5 -0
- package/dist/lib/output-paths.js.map +1 -0
- package/dist/lib/platform-client.d.ts +1 -1
- package/dist/lib/platform-client.d.ts.map +1 -1
- package/dist/lib/repository-validator.d.ts +57 -0
- package/dist/lib/repository-validator.d.ts.map +1 -0
- package/dist/lib/repository-validator.js +225 -0
- package/dist/lib/repository-validator.js.map +1 -0
- package/dist/lib/repository-validator.test.d.ts +5 -0
- package/dist/lib/repository-validator.test.d.ts.map +1 -0
- package/dist/lib/repository-validator.test.js +341 -0
- package/dist/lib/repository-validator.test.js.map +1 -0
- package/dist/lib/review-core.d.ts +66 -0
- package/dist/lib/review-core.d.ts.map +1 -0
- package/dist/lib/review-core.js +449 -0
- package/dist/lib/review-core.js.map +1 -0
- package/dist/lib/review-core.test.d.ts +2 -0
- package/dist/lib/review-core.test.d.ts.map +1 -0
- package/dist/lib/review-core.test.js +552 -0
- package/dist/lib/review-core.test.js.map +1 -0
- package/dist/lib/review-orchestrator.d.ts +27 -10
- package/dist/lib/review-orchestrator.d.ts.map +1 -1
- package/dist/lib/review-orchestrator.js +51 -110
- package/dist/lib/review-orchestrator.js.map +1 -1
- package/dist/lib/review-orchestrator.test.d.ts +2 -0
- package/dist/lib/review-orchestrator.test.d.ts.map +1 -0
- package/dist/lib/review-orchestrator.test.js +413 -0
- package/dist/lib/review-orchestrator.test.js.map +1 -0
- package/dist/lib/review-output-path.test.d.ts +2 -0
- package/dist/lib/review-output-path.test.d.ts.map +1 -0
- package/dist/lib/review-output-path.test.js +83 -0
- package/dist/lib/review-output-path.test.js.map +1 -0
- package/dist/lib/review-parser.d.ts +2 -0
- package/dist/lib/review-parser.d.ts.map +1 -0
- package/dist/lib/review-parser.js +100 -0
- package/dist/lib/review-parser.js.map +1 -0
- package/dist/lib/unified-review-executor.d.ts +21 -4
- package/dist/lib/unified-review-executor.d.ts.map +1 -1
- package/dist/lib/unified-review-executor.js +84 -151
- package/dist/lib/unified-review-executor.js.map +1 -1
- package/dist/lib/unified-review-executor.test.d.ts +5 -0
- package/dist/lib/unified-review-executor.test.d.ts.map +1 -0
- package/dist/lib/unified-review-executor.test.js +350 -0
- package/dist/lib/unified-review-executor.test.js.map +1 -0
- package/dist/lib/write-json-output.d.ts +13 -0
- package/dist/lib/write-json-output.d.ts.map +1 -0
- package/dist/lib/write-json-output.js +37 -0
- package/dist/lib/write-json-output.js.map +1 -0
- package/dist/opencode/agent-loader.d.ts +3 -4
- package/dist/opencode/agent-loader.d.ts.map +1 -1
- package/dist/opencode/agent-loader.js +48 -34
- package/dist/opencode/agent-loader.js.map +1 -1
- package/dist/opencode/agent-skill-overlay.d.ts +11 -0
- package/dist/opencode/agent-skill-overlay.d.ts.map +1 -0
- package/dist/opencode/agent-skill-overlay.js +164 -0
- package/dist/opencode/agent-skill-overlay.js.map +1 -0
- package/dist/opencode/client.d.ts +11 -3
- package/dist/opencode/client.d.ts.map +1 -1
- package/dist/opencode/client.js +237 -71
- package/dist/opencode/client.js.map +1 -1
- package/dist/opencode/client.test.d.ts +2 -0
- package/dist/opencode/client.test.d.ts.map +1 -0
- package/dist/opencode/client.test.js +317 -0
- package/dist/opencode/client.test.js.map +1 -0
- package/dist/opencode/opencode-paths.d.ts +2 -0
- package/dist/opencode/opencode-paths.d.ts.map +1 -0
- package/dist/opencode/opencode-paths.js +7 -0
- package/dist/opencode/opencode-paths.js.map +1 -0
- package/dist/opencode/skill-loader.d.ts +6 -0
- package/dist/opencode/skill-loader.d.ts.map +1 -0
- package/dist/opencode/skill-loader.js +36 -0
- package/dist/opencode/skill-loader.js.map +1 -0
- package/package.json +7 -7
- package/.opencode/agent/github-reviewer.md +0 -77
- package/.opencode/agent/gitlab-reviewer.md +0 -77
- package/.opencode/agent/local-reviewer.md +0 -63
|
@@ -0,0 +1,222 @@
|
|
|
1
|
+
import chalk from 'chalk';
|
|
2
|
+
import { readFile } from 'fs/promises';
|
|
3
|
+
import { resolve } from 'path';
|
|
4
|
+
import { createGitHubClient } from '../github/client.js';
|
|
5
|
+
import { GitHubPlatformAdapter } from '../github/platform-adapter.js';
|
|
6
|
+
import { createGitLabClient } from '../gitlab/client.js';
|
|
7
|
+
import { GitLabPlatformAdapter } from '../gitlab/platform-adapter.js';
|
|
8
|
+
import { enforceRepoBranchMatch, postReviewComments } from '../lib/unified-review-executor.js';
|
|
9
|
+
function parseReviewJson(raw, inputPath) {
|
|
10
|
+
let parsed;
|
|
11
|
+
try {
|
|
12
|
+
parsed = JSON.parse(raw);
|
|
13
|
+
}
|
|
14
|
+
catch (error) {
|
|
15
|
+
throw new Error(`Failed to parse JSON from ${inputPath}: ${error instanceof Error ? error.message : String(error)}`);
|
|
16
|
+
}
|
|
17
|
+
if (!parsed || typeof parsed !== 'object') {
|
|
18
|
+
throw new Error(`Invalid review JSON in ${inputPath}: expected an object`);
|
|
19
|
+
}
|
|
20
|
+
const obj = parsed;
|
|
21
|
+
if (!obj.summary || typeof obj.summary !== 'object') {
|
|
22
|
+
throw new Error(`Invalid review JSON in ${inputPath}: missing summary`);
|
|
23
|
+
}
|
|
24
|
+
if (!Array.isArray(obj.issues)) {
|
|
25
|
+
throw new Error(`Invalid review JSON in ${inputPath}: issues must be an array`);
|
|
26
|
+
}
|
|
27
|
+
return obj;
|
|
28
|
+
}
|
|
29
|
+
function parseNumber(value, label) {
|
|
30
|
+
if (typeof value === 'number' && Number.isFinite(value))
|
|
31
|
+
return value;
|
|
32
|
+
if (typeof value === 'string') {
|
|
33
|
+
const parsed = parseInt(value, 10);
|
|
34
|
+
if (Number.isFinite(parsed))
|
|
35
|
+
return parsed;
|
|
36
|
+
}
|
|
37
|
+
throw new Error(`${label} is required`);
|
|
38
|
+
}
|
|
39
|
+
function resolveGitHubTarget(options, reviewJson) {
|
|
40
|
+
let owner = options.owner;
|
|
41
|
+
let repo = options.repo;
|
|
42
|
+
const projectFromMetadata = reviewJson.metadata?.project;
|
|
43
|
+
if ((!owner || !repo) && projectFromMetadata && projectFromMetadata.includes('/')) {
|
|
44
|
+
const [metaOwner, metaRepo] = projectFromMetadata.split('/');
|
|
45
|
+
if (!owner)
|
|
46
|
+
owner = metaOwner;
|
|
47
|
+
if (!repo)
|
|
48
|
+
repo = metaRepo;
|
|
49
|
+
}
|
|
50
|
+
if (!owner || !repo) {
|
|
51
|
+
throw new Error('GitHub requires --owner and --repo (or a review JSON with metadata.project)');
|
|
52
|
+
}
|
|
53
|
+
const prNumber = parseNumber(options.prNumber ?? reviewJson.metadata?.source, 'PR number (--pr)');
|
|
54
|
+
const projectId = `${owner}/${repo}`;
|
|
55
|
+
if (projectFromMetadata && projectFromMetadata !== projectId) {
|
|
56
|
+
throw new Error(`Review JSON project (${projectFromMetadata}) does not match provided project (${projectId})`);
|
|
57
|
+
}
|
|
58
|
+
return { owner, repo, prNumber, projectId };
|
|
59
|
+
}
|
|
60
|
+
function resolveGitLabTarget(options, reviewJson) {
|
|
61
|
+
const projectId = options.projectId ?? reviewJson.metadata?.project;
|
|
62
|
+
if (!projectId) {
|
|
63
|
+
throw new Error('GitLab requires --project (or a review JSON with metadata.project)');
|
|
64
|
+
}
|
|
65
|
+
const mrIid = parseNumber(options.mrIid ?? reviewJson.metadata?.source, 'MR IID (--mr)');
|
|
66
|
+
return { projectId, mrIid };
|
|
67
|
+
}
|
|
68
|
+
/**
|
|
69
|
+
* Post comments from a saved review JSON to GitHub or GitLab.
|
|
70
|
+
*/
|
|
71
|
+
export async function postCommentsFromJson(options) {
|
|
72
|
+
const workingDir = options.workingDir || process.cwd();
|
|
73
|
+
const inputPath = resolve(workingDir, options.inputPath);
|
|
74
|
+
const raw = await readFile(inputPath, 'utf-8');
|
|
75
|
+
const reviewJson = parseReviewJson(raw, inputPath);
|
|
76
|
+
const isGitHub = Boolean(options.owner || options.repo || options.prNumber);
|
|
77
|
+
const isGitLab = Boolean(options.projectId || options.mrIid);
|
|
78
|
+
if (isGitHub && isGitLab) {
|
|
79
|
+
throw new Error('Specify either GitHub options (--owner/--repo/--pr) or GitLab options (--project/--mr), not both.');
|
|
80
|
+
}
|
|
81
|
+
if (!isGitHub && !isGitLab) {
|
|
82
|
+
throw new Error('Specify GitHub options (--owner/--repo/--pr) or GitLab options (--project/--mr) to post comments.');
|
|
83
|
+
}
|
|
84
|
+
if (reviewJson.issues.length === 0) {
|
|
85
|
+
console.log(chalk.yellow('✓ No issues in review JSON - only summary will be posted\n'));
|
|
86
|
+
}
|
|
87
|
+
if (isGitHub) {
|
|
88
|
+
const { owner, repo, prNumber, projectId } = resolveGitHubTarget(options, reviewJson);
|
|
89
|
+
const githubClient = createGitHubClient();
|
|
90
|
+
const platformClient = new GitHubPlatformAdapter(githubClient);
|
|
91
|
+
const pr = await platformClient.getPullRequest(projectId, prNumber);
|
|
92
|
+
if (!options.skipRepoCheck || !options.skipBranchCheck) {
|
|
93
|
+
await enforceRepoBranchMatch(workingDir, projectId, pr, {
|
|
94
|
+
skipRepoCheck: options.skipRepoCheck,
|
|
95
|
+
skipBranchCheck: options.skipBranchCheck,
|
|
96
|
+
});
|
|
97
|
+
}
|
|
98
|
+
const files = await githubClient.getPRFiles(owner, repo, prNumber);
|
|
99
|
+
const validLinesMap = new Map();
|
|
100
|
+
for (const file of files) {
|
|
101
|
+
if (file.patch && file.status !== 'removed') {
|
|
102
|
+
const validLines = parseValidLinesFromPatch(file.patch);
|
|
103
|
+
validLinesMap.set(file.filename, validLines);
|
|
104
|
+
}
|
|
105
|
+
}
|
|
106
|
+
const lineValidator = {
|
|
107
|
+
isValidLine(file, line) {
|
|
108
|
+
const validLines = validLinesMap.get(file);
|
|
109
|
+
return validLines !== undefined && validLines.has(line);
|
|
110
|
+
},
|
|
111
|
+
};
|
|
112
|
+
const createInlinePosition = (issue, platformData) => {
|
|
113
|
+
const data = platformData;
|
|
114
|
+
return {
|
|
115
|
+
path: issue.file,
|
|
116
|
+
line: issue.line,
|
|
117
|
+
commitSha: data.head.sha,
|
|
118
|
+
};
|
|
119
|
+
};
|
|
120
|
+
await postReviewComments(platformClient, projectId, prNumber, reviewJson.summary, reviewJson.issues, undefined, pr.platformData, lineValidator, createInlinePosition);
|
|
121
|
+
return;
|
|
122
|
+
}
|
|
123
|
+
const { projectId, mrIid } = resolveGitLabTarget(options, reviewJson);
|
|
124
|
+
const gitlabClient = createGitLabClient();
|
|
125
|
+
const platformClient = new GitLabPlatformAdapter(gitlabClient);
|
|
126
|
+
const pr = await platformClient.getPullRequest(projectId, mrIid);
|
|
127
|
+
if (!options.skipRepoCheck || !options.skipBranchCheck) {
|
|
128
|
+
await enforceRepoBranchMatch(workingDir, projectId, pr, {
|
|
129
|
+
skipRepoCheck: options.skipRepoCheck,
|
|
130
|
+
skipBranchCheck: options.skipBranchCheck,
|
|
131
|
+
});
|
|
132
|
+
}
|
|
133
|
+
const changes = await gitlabClient.getMRChanges(projectId, mrIid);
|
|
134
|
+
const validLinesMap = new Map();
|
|
135
|
+
for (const change of changes) {
|
|
136
|
+
if (change.diff && !change.deletedFile) {
|
|
137
|
+
const validLines = parseValidLinesFromDiff(change.diff);
|
|
138
|
+
validLinesMap.set(change.newPath, validLines);
|
|
139
|
+
}
|
|
140
|
+
}
|
|
141
|
+
const lineValidator = {
|
|
142
|
+
isValidLine(file, line) {
|
|
143
|
+
const validLines = validLinesMap.get(file);
|
|
144
|
+
return validLines !== undefined && validLines.has(line);
|
|
145
|
+
},
|
|
146
|
+
};
|
|
147
|
+
const createInlinePosition = (issue, platformData) => {
|
|
148
|
+
const data = platformData;
|
|
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
|
+
await postReviewComments(platformClient, projectId, mrIid, reviewJson.summary, reviewJson.issues, undefined, pr.platformData, lineValidator, createInlinePosition);
|
|
159
|
+
}
|
|
160
|
+
/**
|
|
161
|
+
* Parse a GitHub diff patch to extract valid line numbers for review comments
|
|
162
|
+
* GitHub only allows comments on lines that are in the diff (added, removed, or context)
|
|
163
|
+
*/
|
|
164
|
+
function parseValidLinesFromPatch(patch) {
|
|
165
|
+
const validLines = new Set();
|
|
166
|
+
const lines = patch.split('\n');
|
|
167
|
+
let currentLine = 0;
|
|
168
|
+
for (const line of lines) {
|
|
169
|
+
const hunkMatch = line.match(/^@@ -\d+(?:,\d+)? \+(\d+)(?:,\d+)? @@/);
|
|
170
|
+
if (hunkMatch) {
|
|
171
|
+
currentLine = parseInt(hunkMatch[1], 10);
|
|
172
|
+
continue;
|
|
173
|
+
}
|
|
174
|
+
if (!line || line.length === 0)
|
|
175
|
+
continue;
|
|
176
|
+
const prefix = line[0];
|
|
177
|
+
if (prefix === '+') {
|
|
178
|
+
validLines.add(currentLine);
|
|
179
|
+
currentLine++;
|
|
180
|
+
}
|
|
181
|
+
else if (prefix === ' ') {
|
|
182
|
+
validLines.add(currentLine);
|
|
183
|
+
currentLine++;
|
|
184
|
+
}
|
|
185
|
+
else if (prefix === '-') {
|
|
186
|
+
continue;
|
|
187
|
+
}
|
|
188
|
+
}
|
|
189
|
+
return validLines;
|
|
190
|
+
}
|
|
191
|
+
/**
|
|
192
|
+
* Parse a GitLab diff to extract valid line numbers for review comments
|
|
193
|
+
* GitLab only allows comments on lines that are in the diff (added or context)
|
|
194
|
+
*/
|
|
195
|
+
function parseValidLinesFromDiff(diff) {
|
|
196
|
+
const validLines = new Set();
|
|
197
|
+
const lines = diff.split('\n');
|
|
198
|
+
let currentLine = 0;
|
|
199
|
+
for (const line of lines) {
|
|
200
|
+
const hunkMatch = line.match(/^@@ -\d+(?:,\d+)? \+(\d+)(?:,\d+)? @@/);
|
|
201
|
+
if (hunkMatch) {
|
|
202
|
+
currentLine = parseInt(hunkMatch[1], 10);
|
|
203
|
+
continue;
|
|
204
|
+
}
|
|
205
|
+
if (!line || line.length === 0)
|
|
206
|
+
continue;
|
|
207
|
+
const prefix = line[0];
|
|
208
|
+
if (prefix === '+') {
|
|
209
|
+
validLines.add(currentLine);
|
|
210
|
+
currentLine++;
|
|
211
|
+
}
|
|
212
|
+
else if (prefix === ' ') {
|
|
213
|
+
validLines.add(currentLine);
|
|
214
|
+
currentLine++;
|
|
215
|
+
}
|
|
216
|
+
else if (prefix === '-') {
|
|
217
|
+
continue;
|
|
218
|
+
}
|
|
219
|
+
}
|
|
220
|
+
return validLines;
|
|
221
|
+
}
|
|
222
|
+
//# sourceMappingURL=post-comments.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"post-comments.js","sourceRoot":"","sources":["../../src/cli/post-comments.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,MAAM,OAAO,CAAC;AAC1B,OAAO,EAAE,QAAQ,EAAE,MAAM,aAAa,CAAC;AACvC,OAAO,EAAE,OAAO,EAAE,MAAM,MAAM,CAAC;AAI/B,OAAO,EAAE,kBAAkB,EAAE,MAAM,qBAAqB,CAAC;AACzD,OAAO,EAAE,qBAAqB,EAAE,MAAM,+BAA+B,CAAC;AACtE,OAAO,EAAE,kBAAkB,EAAE,MAAM,qBAAqB,CAAC;AACzD,OAAO,EAAE,qBAAqB,EAAE,MAAM,+BAA+B,CAAC;AACtE,OAAO,EAAE,sBAAsB,EAAE,kBAAkB,EAAE,MAAM,mCAAmC,CAAC;AAc/F,SAAS,eAAe,CAAC,GAAW,EAAE,SAAiB;IACrD,IAAI,MAAe,CAAC;IACpB,IAAI,CAAC;QACH,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;IAC3B,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,MAAM,IAAI,KAAK,CACb,6BAA6B,SAAS,KAAK,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE,CACpG,CAAC;IACJ,CAAC;IAED,IAAI,CAAC,MAAM,IAAI,OAAO,MAAM,KAAK,QAAQ,EAAE,CAAC;QAC1C,MAAM,IAAI,KAAK,CAAC,0BAA0B,SAAS,sBAAsB,CAAC,CAAC;IAC7E,CAAC;IAED,MAAM,GAAG,GAAG,MAAiC,CAAC;IAE9C,IAAI,CAAC,GAAG,CAAC,OAAO,IAAI,OAAO,GAAG,CAAC,OAAO,KAAK,QAAQ,EAAE,CAAC;QACpD,MAAM,IAAI,KAAK,CAAC,0BAA0B,SAAS,mBAAmB,CAAC,CAAC;IAC1E,CAAC;IAED,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC;QAC/B,MAAM,IAAI,KAAK,CAAC,0BAA0B,SAAS,2BAA2B,CAAC,CAAC;IAClF,CAAC;IAED,OAAO,GAAkC,CAAC;AAC5C,CAAC;AAED,SAAS,WAAW,CAAC,KAAkC,EAAE,KAAa;IACpE,IAAI,OAAO,KAAK,KAAK,QAAQ,IAAI,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC;QAAE,OAAO,KAAK,CAAC;IACtE,IAAI,OAAO,KAAK,KAAK,QAAQ,EAAE,CAAC;QAC9B,MAAM,MAAM,GAAG,QAAQ,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC;QACnC,IAAI,MAAM,CAAC,QAAQ,CAAC,MAAM,CAAC;YAAE,OAAO,MAAM,CAAC;IAC7C,CAAC;IACD,MAAM,IAAI,KAAK,CAAC,GAAG,KAAK,cAAc,CAAC,CAAC;AAC1C,CAAC;AAED,SAAS,mBAAmB,CAC1B,OAA4B,EAC5B,UAA4B;IAO5B,IAAI,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC;IAC1B,IAAI,IAAI,GAAG,OAAO,CAAC,IAAI,CAAC;IAExB,MAAM,mBAAmB,GAAG,UAAU,CAAC,QAAQ,EAAE,OAAO,CAAC;IACzD,IAAI,CAAC,CAAC,KAAK,IAAI,CAAC,IAAI,CAAC,IAAI,mBAAmB,IAAI,mBAAmB,CAAC,QAAQ,CAAC,GAAG,CAAC,EAAE,CAAC;QAClF,MAAM,CAAC,SAAS,EAAE,QAAQ,CAAC,GAAG,mBAAmB,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;QAC7D,IAAI,CAAC,KAAK;YAAE,KAAK,GAAG,SAAS,CAAC;QAC9B,IAAI,CAAC,IAAI;YAAE,IAAI,GAAG,QAAQ,CAAC;IAC7B,CAAC;IAED,IAAI,CAAC,KAAK,IAAI,CAAC,IAAI,EAAE,CAAC;QACpB,MAAM,IAAI,KAAK,CAAC,6EAA6E,CAAC,CAAC;IACjG,CAAC;IAED,MAAM,QAAQ,GAAG,WAAW,CAAC,OAAO,CAAC,QAAQ,IAAI,UAAU,CAAC,QAAQ,EAAE,MAAM,EAAE,kBAAkB,CAAC,CAAC;IAClG,MAAM,SAAS,GAAG,GAAG,KAAK,IAAI,IAAI,EAAE,CAAC;IAErC,IAAI,mBAAmB,IAAI,mBAAmB,KAAK,SAAS,EAAE,CAAC;QAC7D,MAAM,IAAI,KAAK,CACb,wBAAwB,mBAAmB,sCAAsC,SAAS,GAAG,CAC9F,CAAC;IACJ,CAAC;IAED,OAAO,EAAE,KAAK,EAAE,IAAI,EAAE,QAAQ,EAAE,SAAS,EAAE,CAAC;AAC9C,CAAC;AAED,SAAS,mBAAmB,CAC1B,OAA4B,EAC5B,UAA4B;IAK5B,MAAM,SAAS,GAAG,OAAO,CAAC,SAAS,IAAI,UAAU,CAAC,QAAQ,EAAE,OAAO,CAAC;IACpE,IAAI,CAAC,SAAS,EAAE,CAAC;QACf,MAAM,IAAI,KAAK,CAAC,oEAAoE,CAAC,CAAC;IACxF,CAAC;IAED,MAAM,KAAK,GAAG,WAAW,CAAC,OAAO,CAAC,KAAK,IAAI,UAAU,CAAC,QAAQ,EAAE,MAAM,EAAE,eAAe,CAAC,CAAC;IAEzF,OAAO,EAAE,SAAS,EAAE,KAAK,EAAE,CAAC;AAC9B,CAAC;AAED;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,oBAAoB,CAAC,OAA4B;IACrE,MAAM,UAAU,GAAG,OAAO,CAAC,UAAU,IAAI,OAAO,CAAC,GAAG,EAAE,CAAC;IACvD,MAAM,SAAS,GAAG,OAAO,CAAC,UAAU,EAAE,OAAO,CAAC,SAAS,CAAC,CAAC;IACzD,MAAM,GAAG,GAAG,MAAM,QAAQ,CAAC,SAAS,EAAE,OAAO,CAAC,CAAC;IAC/C,MAAM,UAAU,GAAG,eAAe,CAAC,GAAG,EAAE,SAAS,CAAC,CAAC;IAEnD,MAAM,QAAQ,GAAG,OAAO,CAAC,OAAO,CAAC,KAAK,IAAI,OAAO,CAAC,IAAI,IAAI,OAAO,CAAC,QAAQ,CAAC,CAAC;IAC5E,MAAM,QAAQ,GAAG,OAAO,CAAC,OAAO,CAAC,SAAS,IAAI,OAAO,CAAC,KAAK,CAAC,CAAC;IAE7D,IAAI,QAAQ,IAAI,QAAQ,EAAE,CAAC;QACzB,MAAM,IAAI,KAAK,CACb,mGAAmG,CACpG,CAAC;IACJ,CAAC;IAED,IAAI,CAAC,QAAQ,IAAI,CAAC,QAAQ,EAAE,CAAC;QAC3B,MAAM,IAAI,KAAK,CACb,mGAAmG,CACpG,CAAC;IACJ,CAAC;IAED,IAAI,UAAU,CAAC,MAAM,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACnC,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,4DAA4D,CAAC,CAAC,CAAC;IAC1F,CAAC;IAED,IAAI,QAAQ,EAAE,CAAC;QACb,MAAM,EAAE,KAAK,EAAE,IAAI,EAAE,QAAQ,EAAE,SAAS,EAAE,GAAG,mBAAmB,CAAC,OAAO,EAAE,UAAU,CAAC,CAAC;QACtF,MAAM,YAAY,GAAG,kBAAkB,EAAE,CAAC;QAC1C,MAAM,cAAc,GAAG,IAAI,qBAAqB,CAAC,YAAY,CAAC,CAAC;QAE/D,MAAM,EAAE,GAAG,MAAM,cAAc,CAAC,cAAc,CAAC,SAAS,EAAE,QAAQ,CAAC,CAAC;QACpE,IAAI,CAAC,OAAO,CAAC,aAAa,IAAI,CAAC,OAAO,CAAC,eAAe,EAAE,CAAC;YACvD,MAAM,sBAAsB,CAAC,UAAU,EAAE,SAAS,EAAE,EAAE,EAAE;gBACtD,aAAa,EAAE,OAAO,CAAC,aAAa;gBACpC,eAAe,EAAE,OAAO,CAAC,eAAe;aACzC,CAAC,CAAC;QACL,CAAC;QAED,MAAM,KAAK,GAAG,MAAM,YAAY,CAAC,UAAU,CAAC,KAAK,EAAE,IAAI,EAAE,QAAQ,CAAC,CAAC;QACnE,MAAM,aAAa,GAAG,IAAI,GAAG,EAAuB,CAAC;QACrD,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;YACzB,IAAI,IAAI,CAAC,KAAK,IAAI,IAAI,CAAC,MAAM,KAAK,SAAS,EAAE,CAAC;gBAC5C,MAAM,UAAU,GAAG,wBAAwB,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;gBACxD,aAAa,CAAC,GAAG,CAAC,IAAI,CAAC,QAAQ,EAAE,UAAU,CAAC,CAAC;YAC/C,CAAC;QACH,CAAC;QAED,MAAM,aAAa,GAAkB;YACnC,WAAW,CAAC,IAAY,EAAE,IAAY;gBACpC,MAAM,UAAU,GAAG,aAAa,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;gBAC3C,OAAO,UAAU,KAAK,SAAS,IAAI,UAAU,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;YAC1D,CAAC;SACF,CAAC;QAEF,MAAM,oBAAoB,GAAG,CAC3B,KAAkB,EAClB,YAAqB,EACE,EAAE;YACzB,MAAM,IAAI,GAAG,YAAyC,CAAC;YACvD,OAAO;gBACL,IAAI,EAAE,KAAK,CAAC,IAAI;gBAChB,IAAI,EAAE,KAAK,CAAC,IAAK;gBACjB,SAAS,EAAE,IAAI,CAAC,IAAI,CAAC,GAAG;aACzB,CAAC;QACJ,CAAC,CAAC;QAEF,MAAM,kBAAkB,CACtB,cAAc,EACd,SAAS,EACT,QAAQ,EACR,UAAU,CAAC,OAAO,EAClB,UAAU,CAAC,MAAM,EACjB,SAAS,EACT,EAAE,CAAC,YAAY,EACf,aAAa,EACb,oBAAoB,CACrB,CAAC;QACF,OAAO;IACT,CAAC;IAED,MAAM,EAAE,SAAS,EAAE,KAAK,EAAE,GAAG,mBAAmB,CAAC,OAAO,EAAE,UAAU,CAAC,CAAC;IACtE,MAAM,YAAY,GAAG,kBAAkB,EAAE,CAAC;IAC1C,MAAM,cAAc,GAAG,IAAI,qBAAqB,CAAC,YAAY,CAAC,CAAC;IAE/D,MAAM,EAAE,GAAG,MAAM,cAAc,CAAC,cAAc,CAAC,SAAS,EAAE,KAAK,CAAC,CAAC;IACjE,IAAI,CAAC,OAAO,CAAC,aAAa,IAAI,CAAC,OAAO,CAAC,eAAe,EAAE,CAAC;QACvD,MAAM,sBAAsB,CAAC,UAAU,EAAE,SAAS,EAAE,EAAE,EAAE;YACtD,aAAa,EAAE,OAAO,CAAC,aAAa;YACpC,eAAe,EAAE,OAAO,CAAC,eAAe;SACzC,CAAC,CAAC;IACL,CAAC;IAED,MAAM,OAAO,GAAG,MAAM,YAAY,CAAC,YAAY,CAAC,SAAS,EAAE,KAAK,CAAC,CAAC;IAClE,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,MAAM,aAAa,GAAkB;QACnC,WAAW,CAAC,IAAY,EAAE,IAAY;YACpC,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,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,CAAC,QAAQ;YACtB,OAAO,EAAE,IAAI,CAAC,QAAQ;YACtB,QAAQ,EAAE,IAAI,CAAC,SAAS;SACzB,CAAC;IACJ,CAAC,CAAC;IAEF,MAAM,kBAAkB,CACtB,cAAc,EACd,SAAS,EACT,KAAK,EACL,UAAU,CAAC,OAAO,EAClB,UAAU,CAAC,MAAM,EACjB,SAAS,EACT,EAAE,CAAC,YAAY,EACf,aAAa,EACb,oBAAoB,CACrB,CAAC;AACJ,CAAC;AAED;;;GAGG;AACH,SAAS,wBAAwB,CAAC,KAAa;IAC7C,MAAM,UAAU,GAAG,IAAI,GAAG,EAAU,CAAC;IACrC,MAAM,KAAK,GAAG,KAAK,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAChC,IAAI,WAAW,GAAG,CAAC,CAAC;IAEpB,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,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,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,UAAU,CAAC,GAAG,CAAC,WAAW,CAAC,CAAC;YAC5B,WAAW,EAAE,CAAC;QAChB,CAAC;aAAM,IAAI,MAAM,KAAK,GAAG,EAAE,CAAC;YAC1B,UAAU,CAAC,GAAG,CAAC,WAAW,CAAC,CAAC;YAC5B,WAAW,EAAE,CAAC;QAChB,CAAC;aAAM,IAAI,MAAM,KAAK,GAAG,EAAE,CAAC;YAC1B,SAAS;QACX,CAAC;IACH,CAAC;IAED,OAAO,UAAU,CAAC;AACpB,CAAC;AAED;;;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,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,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,UAAU,CAAC,GAAG,CAAC,WAAW,CAAC,CAAC;YAC5B,WAAW,EAAE,CAAC;QAChB,CAAC;aAAM,IAAI,MAAM,KAAK,GAAG,EAAE,CAAC;YAC1B,UAAU,CAAC,GAAG,CAAC,WAAW,CAAC,CAAC;YAC5B,WAAW,EAAE,CAAC;QAChB,CAAC;aAAM,IAAI,MAAM,KAAK,GAAG,EAAE,CAAC;YAC1B,SAAS;QACX,CAAC;IACH,CAAC;IAED,OAAO,UAAU,CAAC;AACpB,CAAC"}
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"review-local.d.ts","sourceRoot":"","sources":["../../src/cli/review-local.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;
|
|
1
|
+
{"version":3,"file":"review-local.d.ts","sourceRoot":"","sources":["../../src/cli/review-local.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AAWlD,MAAM,WAAW,kBAAkB;IACjC,MAAM,EAAE,OAAO,CAAC;IAChB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,KAAK,CAAC,EAAE,OAAO,CAAC;CACjB;AAED;;GAEG;AACH,wBAAsB,WAAW,CAAC,MAAM,EAAE,SAAS,EAAE,OAAO,EAAE,kBAAkB,GAAG,OAAO,CAAC,IAAI,CAAC,CA4F/F"}
|
package/dist/cli/review-local.js
CHANGED
|
@@ -1,8 +1,9 @@
|
|
|
1
1
|
import simpleGit from 'simple-git';
|
|
2
2
|
import chalk from 'chalk';
|
|
3
|
-
import { parseDiff, getChangedFiles } from '../lib/diff-parser.js';
|
|
3
|
+
import { parseDiff, getChangedFiles, getFilesWithDiffs } from '../lib/diff-parser.js';
|
|
4
4
|
import { formatTerminalIssue } from '../lib/comment-formatter.js';
|
|
5
5
|
import { executeReview, displayReviewSummary, hasBlockingIssues, } from '../lib/review-orchestrator.js';
|
|
6
|
+
import { formatReviewJson, writeReviewJson, printReviewJson } from '../lib/json-output.js';
|
|
6
7
|
/**
|
|
7
8
|
* Review local git diff before pushing
|
|
8
9
|
*/
|
|
@@ -22,41 +23,66 @@ export async function reviewLocal(config, options) {
|
|
|
22
23
|
console.log(chalk.yellow('✓ No changes to review\n'));
|
|
23
24
|
return;
|
|
24
25
|
}
|
|
25
|
-
// Parse diff to get changed files
|
|
26
|
+
// Parse diff to get changed files with their diffs
|
|
26
27
|
const diffs = parseDiff(diffText);
|
|
27
28
|
const changedFiles = getChangedFiles(diffs);
|
|
29
|
+
const filesWithDiffs = getFilesWithDiffs(diffs);
|
|
28
30
|
if (changedFiles.length === 0) {
|
|
29
31
|
console.log(chalk.yellow('✓ No files to review\n'));
|
|
30
32
|
return;
|
|
31
33
|
}
|
|
32
|
-
// Execute review using common orchestrator
|
|
34
|
+
// Execute review using common orchestrator - pass diff content directly
|
|
33
35
|
const source = {
|
|
34
36
|
name: `Local ${options.staged ? 'staged' : 'unstaged'} diff`,
|
|
35
37
|
files: changedFiles,
|
|
36
|
-
|
|
37
|
-
|
|
38
|
-
},
|
|
38
|
+
filesWithDiffs, // Pass actual diff content so agents don't need to run git
|
|
39
|
+
context: {},
|
|
39
40
|
workingDir: cwd,
|
|
41
|
+
debug: options.debug,
|
|
42
|
+
staged: options.staged,
|
|
40
43
|
};
|
|
41
44
|
const result = await executeReview(config, source);
|
|
42
|
-
//
|
|
43
|
-
|
|
44
|
-
|
|
45
|
-
|
|
46
|
-
|
|
47
|
-
|
|
45
|
+
// Handle JSON output
|
|
46
|
+
const wantsJsonOutput = options.jsonOutput || options.outputPath;
|
|
47
|
+
if (wantsJsonOutput) {
|
|
48
|
+
const jsonOutput = formatReviewJson(result.summary, result.issues, {
|
|
49
|
+
source: `local-${options.staged ? 'staged' : 'unstaged'}`,
|
|
50
|
+
});
|
|
51
|
+
if (options.outputPath) {
|
|
52
|
+
await writeReviewJson(jsonOutput, options.outputPath, cwd);
|
|
53
|
+
console.log(chalk.green(`\n✓ Review results written to ${options.outputPath}\n`));
|
|
48
54
|
}
|
|
49
|
-
|
|
50
|
-
|
|
51
|
-
|
|
52
|
-
|
|
55
|
+
if (options.jsonOutput) {
|
|
56
|
+
printReviewJson(jsonOutput);
|
|
57
|
+
}
|
|
58
|
+
}
|
|
59
|
+
// Display results (platform-specific: terminal output)
|
|
60
|
+
// Only show terminal output if not doing JSON-only output
|
|
61
|
+
if (!options.jsonOutput) {
|
|
62
|
+
if (result.issues.length > 0) {
|
|
63
|
+
displayReviewSummary(result);
|
|
64
|
+
// Display issues in terminal
|
|
65
|
+
for (const issue of result.issues) {
|
|
66
|
+
console.log(formatTerminalIssue(issue));
|
|
67
|
+
}
|
|
68
|
+
// Recommendation
|
|
69
|
+
if (hasBlockingIssues(result)) {
|
|
70
|
+
console.log(chalk.red.bold('\n⚠️ Recommendation: Fix critical/high issues before pushing\n'));
|
|
71
|
+
process.exit(1);
|
|
72
|
+
}
|
|
73
|
+
else {
|
|
74
|
+
console.log(chalk.green('\n✓ No critical issues found. Safe to push.\n'));
|
|
75
|
+
}
|
|
53
76
|
}
|
|
54
77
|
else {
|
|
55
|
-
console.log(chalk.green('\n✓ No
|
|
78
|
+
console.log(chalk.green('\n✓ No issues found! Code looks good.\n'));
|
|
56
79
|
}
|
|
57
80
|
}
|
|
58
81
|
else {
|
|
59
|
-
|
|
82
|
+
// Still exit with error code for blocking issues even in JSON mode
|
|
83
|
+
if (hasBlockingIssues(result)) {
|
|
84
|
+
process.exit(1);
|
|
85
|
+
}
|
|
60
86
|
}
|
|
61
87
|
}
|
|
62
88
|
//# sourceMappingURL=review-local.js.map
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"review-local.js","sourceRoot":"","sources":["../../src/cli/review-local.ts"],"names":[],"mappings":"AAAA,OAAO,SAAS,MAAM,YAAY,CAAC;AACnC,OAAO,KAAK,MAAM,OAAO,CAAC;AAE1B,OAAO,EAAE,SAAS,EAAE,eAAe,EAAE,MAAM,uBAAuB,CAAC;
|
|
1
|
+
{"version":3,"file":"review-local.js","sourceRoot":"","sources":["../../src/cli/review-local.ts"],"names":[],"mappings":"AAAA,OAAO,SAAS,MAAM,YAAY,CAAC;AACnC,OAAO,KAAK,MAAM,OAAO,CAAC;AAE1B,OAAO,EAAE,SAAS,EAAE,eAAe,EAAE,iBAAiB,EAAE,MAAM,uBAAuB,CAAC;AACtF,OAAO,EAAE,mBAAmB,EAAE,MAAM,6BAA6B,CAAC;AAClE,OAAO,EACL,aAAa,EACb,oBAAoB,EACpB,iBAAiB,GAElB,MAAM,+BAA+B,CAAC;AACvC,OAAO,EAAE,gBAAgB,EAAE,eAAe,EAAE,eAAe,EAAE,MAAM,uBAAuB,CAAC;AAS3F;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,WAAW,CAAC,MAAiB,EAAE,OAA2B;IAC9E,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,kCAAkC,CAAC,CAAC,CAAC;IAEjE,MAAM,GAAG,GAAG,SAAS,EAAE,CAAC;IACxB,MAAM,GAAG,GAAG,OAAO,CAAC,GAAG,EAAE,CAAC;IAE1B,qCAAqC;IACrC,MAAM,MAAM,GAAG,MAAM,GAAG,CAAC,WAAW,EAAE,CAAC;IACvC,IAAI,CAAC,MAAM,EAAE,CAAC;QACZ,MAAM,IAAI,KAAK,CAAC,sEAAsE,CAAC,CAAC;IAC1F,CAAC;IAED,0CAA0C;IAC1C,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,WAAW,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,UAAU,eAAe,CAAC,CAAC,CAAC;IAE1F,MAAM,QAAQ,GAAG,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM,GAAG,CAAC,IAAI,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,CAAC,MAAM,GAAG,CAAC,IAAI,EAAE,CAAC;IAElF,IAAI,CAAC,QAAQ,IAAI,QAAQ,CAAC,IAAI,EAAE,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC9C,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,0BAA0B,CAAC,CAAC,CAAC;QACtD,OAAO;IACT,CAAC;IAED,mDAAmD;IACnD,MAAM,KAAK,GAAG,SAAS,CAAC,QAAQ,CAAC,CAAC;IAClC,MAAM,YAAY,GAAG,eAAe,CAAC,KAAK,CAAC,CAAC;IAC5C,MAAM,cAAc,GAAG,iBAAiB,CAAC,KAAK,CAAC,CAAC;IAEhD,IAAI,YAAY,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC9B,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,wBAAwB,CAAC,CAAC,CAAC;QACpD,OAAO;IACT,CAAC;IAED,wEAAwE;IACxE,MAAM,MAAM,GAAiB;QAC3B,IAAI,EAAE,SAAS,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,UAAU,OAAO;QAC5D,KAAK,EAAE,YAAY;QACnB,cAAc,EAAE,2DAA2D;QAC3E,OAAO,EAAE,EAAE;QACX,UAAU,EAAE,GAAG;QACf,KAAK,EAAE,OAAO,CAAC,KAAK;QACpB,MAAM,EAAE,OAAO,CAAC,MAAM;KACvB,CAAC;IAEF,MAAM,MAAM,GAAG,MAAM,aAAa,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAEnD,qBAAqB;IACrB,MAAM,eAAe,GAAG,OAAO,CAAC,UAAU,IAAI,OAAO,CAAC,UAAU,CAAC;IAEjE,IAAI,eAAe,EAAE,CAAC;QACpB,MAAM,UAAU,GAAG,gBAAgB,CAAC,MAAM,CAAC,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE;YACjE,MAAM,EAAE,SAAS,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,UAAU,EAAE;SAC1D,CAAC,CAAC;QAEH,IAAI,OAAO,CAAC,UAAU,EAAE,CAAC;YACvB,MAAM,eAAe,CAAC,UAAU,EAAE,OAAO,CAAC,UAAU,EAAE,GAAG,CAAC,CAAC;YAC3D,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,KAAK,CAAC,iCAAiC,OAAO,CAAC,UAAU,IAAI,CAAC,CAAC,CAAC;QACpF,CAAC;QAED,IAAI,OAAO,CAAC,UAAU,EAAE,CAAC;YACvB,eAAe,CAAC,UAAU,CAAC,CAAC;QAC9B,CAAC;IACH,CAAC;IAED,uDAAuD;IACvD,0DAA0D;IAC1D,IAAI,CAAC,OAAO,CAAC,UAAU,EAAE,CAAC;QACxB,IAAI,MAAM,CAAC,MAAM,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC7B,oBAAoB,CAAC,MAAM,CAAC,CAAC;YAE7B,6BAA6B;YAC7B,KAAK,MAAM,KAAK,IAAI,MAAM,CAAC,MAAM,EAAE,CAAC;gBAClC,OAAO,CAAC,GAAG,CAAC,mBAAmB,CAAC,KAAK,CAAC,CAAC,CAAC;YAC1C,CAAC;YAED,iBAAiB;YACjB,IAAI,iBAAiB,CAAC,MAAM,CAAC,EAAE,CAAC;gBAC9B,OAAO,CAAC,GAAG,CACT,KAAK,CAAC,GAAG,CAAC,IAAI,CAAC,iEAAiE,CAAC,CAClF,CAAC;gBACF,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAClB,CAAC;iBAAM,CAAC;gBACN,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,KAAK,CAAC,+CAA+C,CAAC,CAAC,CAAC;YAC5E,CAAC;QACH,CAAC;aAAM,CAAC;YACN,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,KAAK,CAAC,yCAAyC,CAAC,CAAC,CAAC;QACtE,CAAC;IACH,CAAC;SAAM,CAAC;QACN,mEAAmE;QACnE,IAAI,iBAAiB,CAAC,MAAM,CAAC,EAAE,CAAC;YAC9B,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAClB,CAAC;IACH,CAAC;AACH,CAAC"}
|
package/dist/cli/review-mr.d.ts
CHANGED
|
@@ -3,7 +3,13 @@ export interface ReviewMROptions {
|
|
|
3
3
|
projectId: string;
|
|
4
4
|
mrIid: number;
|
|
5
5
|
postComments: boolean;
|
|
6
|
+
describe: boolean;
|
|
7
|
+
postDescription: boolean;
|
|
6
8
|
codeQualityReport?: string;
|
|
9
|
+
outputPath?: string;
|
|
10
|
+
jsonOutput?: boolean;
|
|
11
|
+
baseBranch?: string;
|
|
12
|
+
debug?: boolean;
|
|
7
13
|
}
|
|
8
14
|
/**
|
|
9
15
|
* Review a GitLab merge request
|
|
@@ -1 +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,iBAAiB,CAAC,EAAE,MAAM,CAAC;
|
|
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,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,CAwEzF"}
|
package/dist/cli/review-mr.js
CHANGED
|
@@ -1,6 +1,42 @@
|
|
|
1
1
|
import { createGitLabClient } from '../gitlab/client.js';
|
|
2
2
|
import { GitLabPlatformAdapter } from '../gitlab/platform-adapter.js';
|
|
3
3
|
import { executeUnifiedReview } from '../lib/unified-review-executor.js';
|
|
4
|
+
/**
|
|
5
|
+
* Parse a GitLab diff to extract valid line numbers for review comments
|
|
6
|
+
* GitLab only allows comments on lines that are in the diff (added or context)
|
|
7
|
+
*/
|
|
8
|
+
function parseValidLinesFromDiff(diff) {
|
|
9
|
+
const validLines = new Set();
|
|
10
|
+
const lines = diff.split('\n');
|
|
11
|
+
let currentLine = 0;
|
|
12
|
+
for (const line of lines) {
|
|
13
|
+
// Parse hunk header: @@ -old_start,old_count +new_start,new_count @@
|
|
14
|
+
const hunkMatch = line.match(/^@@ -\d+(?:,\d+)? \+(\d+)(?:,\d+)? @@/);
|
|
15
|
+
if (hunkMatch) {
|
|
16
|
+
currentLine = parseInt(hunkMatch[1], 10);
|
|
17
|
+
continue;
|
|
18
|
+
}
|
|
19
|
+
// Skip empty lines or lines without proper diff prefix
|
|
20
|
+
if (!line || line.length === 0)
|
|
21
|
+
continue;
|
|
22
|
+
const prefix = line[0];
|
|
23
|
+
if (prefix === '+') {
|
|
24
|
+
// Added line - can comment on this
|
|
25
|
+
validLines.add(currentLine);
|
|
26
|
+
currentLine++;
|
|
27
|
+
}
|
|
28
|
+
else if (prefix === ' ') {
|
|
29
|
+
// Context line - can comment on this
|
|
30
|
+
validLines.add(currentLine);
|
|
31
|
+
currentLine++;
|
|
32
|
+
}
|
|
33
|
+
else if (prefix === '-') {
|
|
34
|
+
// Removed line - cannot comment on "new" version, skip
|
|
35
|
+
continue;
|
|
36
|
+
}
|
|
37
|
+
}
|
|
38
|
+
return validLines;
|
|
39
|
+
}
|
|
4
40
|
/**
|
|
5
41
|
* Review a GitLab merge request
|
|
6
42
|
*/
|
|
@@ -10,23 +46,37 @@ export async function reviewMR(config, options) {
|
|
|
10
46
|
const platformClient = new GitLabPlatformAdapter(gitlabClient);
|
|
11
47
|
// Fetch MR details to get diff refs
|
|
12
48
|
const mr = await gitlabClient.getMergeRequest(options.projectId, options.mrIid);
|
|
13
|
-
//
|
|
14
|
-
|
|
49
|
+
// Fetch MR changes to build valid lines map
|
|
50
|
+
const changes = await gitlabClient.getMRChanges(options.projectId, options.mrIid);
|
|
51
|
+
// Build a map of file -> valid line numbers (lines that are in the diff)
|
|
52
|
+
const validLinesMap = new Map();
|
|
53
|
+
for (const change of changes) {
|
|
54
|
+
if (change.diff && !change.deletedFile) {
|
|
55
|
+
const validLines = parseValidLinesFromDiff(change.diff);
|
|
56
|
+
validLinesMap.set(change.newPath, validLines);
|
|
57
|
+
}
|
|
58
|
+
}
|
|
59
|
+
// Create line validator - only allow comments on lines that are in the diff
|
|
15
60
|
const diffRefs = mr.diff_refs;
|
|
16
61
|
const lineValidator = {
|
|
17
62
|
isValidLine(file, line) {
|
|
18
|
-
|
|
63
|
+
if (!diffRefs?.base_sha || !diffRefs.head_sha || !diffRefs.start_sha) {
|
|
64
|
+
return false;
|
|
65
|
+
}
|
|
66
|
+
const validLines = validLinesMap.get(file);
|
|
67
|
+
return validLines !== undefined && validLines.has(line);
|
|
19
68
|
},
|
|
20
69
|
};
|
|
21
70
|
// Create inline position builder
|
|
22
71
|
const createInlinePosition = (issue, platformData) => {
|
|
23
|
-
const
|
|
72
|
+
const data = platformData;
|
|
73
|
+
const refs = data.diff_refs;
|
|
24
74
|
return {
|
|
25
75
|
path: issue.file,
|
|
26
76
|
line: issue.line,
|
|
27
|
-
baseSha: refs
|
|
28
|
-
headSha: refs
|
|
29
|
-
startSha: refs
|
|
77
|
+
baseSha: refs?.base_sha,
|
|
78
|
+
headSha: refs?.head_sha,
|
|
79
|
+
startSha: refs?.start_sha,
|
|
30
80
|
};
|
|
31
81
|
};
|
|
32
82
|
// Execute unified review
|
|
@@ -36,8 +86,14 @@ export async function reviewMR(config, options) {
|
|
|
36
86
|
prNumber: options.mrIid,
|
|
37
87
|
postComments: options.postComments,
|
|
38
88
|
codeQualityReport: options.codeQualityReport,
|
|
89
|
+
outputPath: options.outputPath,
|
|
90
|
+
jsonOutput: options.jsonOutput,
|
|
91
|
+
baseBranch: options.baseBranch,
|
|
39
92
|
lineValidator,
|
|
40
93
|
createInlinePosition,
|
|
94
|
+
describe: options.describe,
|
|
95
|
+
postDescription: options.postDescription,
|
|
96
|
+
debug: options.debug,
|
|
41
97
|
});
|
|
42
98
|
}
|
|
43
99
|
//# sourceMappingURL=review-mr.js.map
|
|
@@ -1 +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;
|
|
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;AAiBzE;;;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,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-pr.d.ts
CHANGED
|
@@ -4,6 +4,12 @@ export interface ReviewPROptions {
|
|
|
4
4
|
repo: string;
|
|
5
5
|
prNumber: number;
|
|
6
6
|
postComments: boolean;
|
|
7
|
+
describe: boolean;
|
|
8
|
+
postDescription: boolean;
|
|
9
|
+
outputPath?: string;
|
|
10
|
+
jsonOutput?: boolean;
|
|
11
|
+
baseBranch?: string;
|
|
12
|
+
debug?: boolean;
|
|
7
13
|
}
|
|
8
14
|
/**
|
|
9
15
|
* Review a GitHub pull request
|
|
@@ -1 +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;
|
|
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,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,CAyDzF"}
|
package/dist/cli/review-pr.js
CHANGED
|
@@ -65,10 +65,11 @@ export async function reviewPR(config, options) {
|
|
|
65
65
|
};
|
|
66
66
|
// Create inline position builder
|
|
67
67
|
const createInlinePosition = (issue, platformData) => {
|
|
68
|
+
const data = platformData;
|
|
68
69
|
return {
|
|
69
70
|
path: issue.file,
|
|
70
71
|
line: issue.line,
|
|
71
|
-
commitSha:
|
|
72
|
+
commitSha: data.head.sha,
|
|
72
73
|
};
|
|
73
74
|
};
|
|
74
75
|
// Execute unified review
|
|
@@ -77,9 +78,15 @@ export async function reviewPR(config, options) {
|
|
|
77
78
|
projectId,
|
|
78
79
|
prNumber: options.prNumber,
|
|
79
80
|
postComments: options.postComments,
|
|
81
|
+
outputPath: options.outputPath,
|
|
82
|
+
jsonOutput: options.jsonOutput,
|
|
83
|
+
baseBranch: options.baseBranch,
|
|
80
84
|
lineValidator,
|
|
81
85
|
createInlinePosition,
|
|
82
86
|
workingDir: process.cwd(),
|
|
87
|
+
describe: options.describe,
|
|
88
|
+
postDescription: options.postDescription,
|
|
89
|
+
debug: options.debug,
|
|
83
90
|
});
|
|
84
91
|
}
|
|
85
92
|
//# sourceMappingURL=review-pr.js.map
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"review-pr.js","sourceRoot":"","sources":["../../src/cli/review-pr.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;
|
|
1
|
+
{"version":3,"file":"review-pr.js","sourceRoot":"","sources":["../../src/cli/review-pr.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;AAiBzE;;;GAGG;AACH,SAAS,wBAAwB,CAAC,KAAa;IAC7C,MAAM,UAAU,GAAG,IAAI,GAAG,EAAU,CAAC;IACrC,MAAM,KAAK,GAAG,KAAK,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAChC,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,SAAS,GAAG,GAAG,OAAO,CAAC,KAAK,IAAI,OAAO,CAAC,IAAI,EAAE,CAAC;IAErD,sCAAsC;IACtC,MAAM,KAAK,GAAG,MAAM,YAAY,CAAC,UAAU,CAAC,OAAO,CAAC,KAAK,EAAE,OAAO,CAAC,IAAI,EAAE,OAAO,CAAC,QAAQ,CAAC,CAAC;IAE3F,yEAAyE;IACzE,MAAM,aAAa,GAAG,IAAI,GAAG,EAAuB,CAAC;IACrD,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,IAAI,IAAI,CAAC,KAAK,IAAI,IAAI,CAAC,MAAM,KAAK,SAAS,EAAE,CAAC;YAC5C,MAAM,UAAU,GAAG,wBAAwB,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YACxD,aAAa,CAAC,GAAG,CAAC,IAAI,CAAC,QAAQ,EAAE,UAAU,CAAC,CAAC;QAC/C,CAAC;IACH,CAAC;IAED,wBAAwB;IACxB,MAAM,aAAa,GAAkB;QACnC,WAAW,CAAC,IAAY,EAAE,IAAY;YACpC,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,YAAyC,CAAC;QACvD,OAAO;YACL,IAAI,EAAE,KAAK,CAAC,IAAI;YAChB,IAAI,EAAE,KAAK,CAAC,IAAK;YACjB,SAAS,EAAE,IAAI,CAAC,IAAI,CAAC,GAAG;SACzB,CAAC;IACJ,CAAC,CAAC;IAEF,yBAAyB;IACzB,MAAM,oBAAoB,CAAC,MAAM,EAAE;QACjC,cAAc;QACd,SAAS;QACT,QAAQ,EAAE,OAAO,CAAC,QAAQ;QAC1B,YAAY,EAAE,OAAO,CAAC,YAAY;QAClC,UAAU,EAAE,OAAO,CAAC,UAAU;QAC9B,UAAU,EAAE,OAAO,CAAC,UAAU;QAC9B,UAAU,EAAE,OAAO,CAAC,UAAU;QAC9B,aAAa;QACb,oBAAoB;QACpB,UAAU,EAAE,OAAO,CAAC,GAAG,EAAE;QACzB,QAAQ,EAAE,OAAO,CAAC,QAAQ;QAC1B,eAAe,EAAE,OAAO,CAAC,eAAe;QACxC,KAAK,EAAE,OAAO,CAAC,KAAK;KACrB,CAAC,CAAC;AACL,CAAC"}
|
|
@@ -0,0 +1,15 @@
|
|
|
1
|
+
import type { DRSConfig } from '../lib/config.js';
|
|
2
|
+
export interface ShowChangesOptions {
|
|
3
|
+
owner?: string;
|
|
4
|
+
repo?: string;
|
|
5
|
+
prNumber?: number;
|
|
6
|
+
projectId?: string;
|
|
7
|
+
mrIid?: number;
|
|
8
|
+
baseBranch?: string;
|
|
9
|
+
file?: string;
|
|
10
|
+
outputPath?: string;
|
|
11
|
+
jsonOutput?: boolean;
|
|
12
|
+
workingDir?: string;
|
|
13
|
+
}
|
|
14
|
+
export declare function showChanges(config: DRSConfig, options: ShowChangesOptions): Promise<void>;
|
|
15
|
+
//# sourceMappingURL=show-changes.d.ts.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"show-changes.d.ts","sourceRoot":"","sources":["../../src/cli/show-changes.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AAQlD,MAAM,WAAW,kBAAkB;IACjC,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAyGD,wBAAsB,WAAW,CAAC,MAAM,EAAE,SAAS,EAAE,OAAO,EAAE,kBAAkB,GAAG,OAAO,CAAC,IAAI,CAAC,CAqH/F"}
|