smart-commit-copilot-cli 0.1.12 → 0.1.14
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/CHANGELOG.md +35 -0
- package/README.md +90 -10
- package/docs/configuration.md +93 -17
- package/docs/contracts.md +106 -2
- package/docs/getting-started.md +18 -1
- package/docs/integrations.md +2 -0
- package/docs/publish.md +1 -1
- package/docs/release-checklist.md +1 -1
- package/docs/releases/0.1.13-draft.md +63 -0
- package/docs/releases/0.1.14-draft.md +94 -0
- package/docs/releases/0.1.8-draft.md +3 -3
- package/docs/verification.md +7 -0
- package/examples/config/smart-commit.json +19 -5
- package/out/cli.js +7 -1
- package/out/cli.js.map +1 -1
- package/out/cliApp.js +102 -8
- package/out/cliApp.js.map +1 -1
- package/out/commands/bridge.js +3 -3
- package/out/commands/bridge.js.map +1 -1
- package/out/commands/commitMessage.js +239 -0
- package/out/commands/commitMessage.js.map +1 -0
- package/out/commands/pullRequest.js +3 -3
- package/out/commands/pullRequest.js.map +1 -1
- package/out/commands/pullRequestReview.js +220 -0
- package/out/commands/pullRequestReview.js.map +1 -0
- package/out/commands/report.js +96 -18
- package/out/commands/report.js.map +1 -1
- package/out/config/cliArgs.js +96 -21
- package/out/config/cliArgs.js.map +1 -1
- package/out/config/env.js +68 -28
- package/out/config/env.js.map +1 -1
- package/out/config/file.js +456 -11
- package/out/config/file.js.map +1 -1
- package/out/config/index.js +5 -1
- package/out/config/index.js.map +1 -1
- package/out/config/legacySmartCommit.js +120 -14
- package/out/config/legacySmartCommit.js.map +1 -1
- package/out/config/merge.js +27 -11
- package/out/config/merge.js.map +1 -1
- package/out/config/schema.js +57 -7
- package/out/config/schema.js.map +1 -1
- package/out/contracts.js +244 -11
- package/out/contracts.js.map +1 -1
- package/out/git.js +101 -0
- package/out/git.js.map +1 -1
- package/out/localization.js +106 -0
- package/out/localization.js.map +1 -0
- package/out/passHistory/index.js +1 -1
- package/out/passHistory/index.js.map +1 -1
- package/out/pullRequest/api.js +39 -26
- package/out/pullRequest/api.js.map +1 -1
- package/out/pullRequest/config.js +36 -2
- package/out/pullRequest/config.js.map +1 -1
- package/out/pullRequest/index.js +6 -0
- package/out/pullRequest/index.js.map +1 -1
- package/out/pullRequest/mergeGate.js +34 -0
- package/out/pullRequest/mergeGate.js.map +1 -0
- package/out/pullRequest/platform.js +13 -0
- package/out/pullRequest/platform.js.map +1 -1
- package/out/pullRequest/reviewApi.js +424 -0
- package/out/pullRequest/reviewApi.js.map +1 -0
- package/out/pullRequest/reviewComment.js +176 -0
- package/out/pullRequest/reviewComment.js.map +1 -0
- package/out/pullRequest/reviewGate.js +43 -0
- package/out/pullRequest/reviewGate.js.map +1 -0
- package/out/pullRequest/reviewWorkflow.js +313 -0
- package/out/pullRequest/reviewWorkflow.js.map +1 -0
- package/out/pullRequest/urlParser.js +135 -0
- package/out/pullRequest/urlParser.js.map +1 -0
- package/out/pullRequest/workflow.js +17 -4
- package/out/pullRequest/workflow.js.map +1 -1
- package/out/renderOutput.js +59 -0
- package/out/renderOutput.js.map +1 -1
- package/out/reporting/aggregate.js +1 -1
- package/out/reporting/aggregate.js.map +1 -1
- package/out/reporting/index.js +2 -2
- package/out/reporting/index.js.map +1 -1
- package/out/reporting/render.js +5 -1
- package/out/reporting/render.js.map +1 -1
- package/out/reporting/timeWindow.js +52 -2
- package/out/reporting/timeWindow.js.map +1 -1
- package/out/reporting/types.js.map +1 -1
- package/out/review/chunkAggregate.js +53 -0
- package/out/review/chunkAggregate.js.map +1 -0
- package/out/review/detailLocator.js +19 -12
- package/out/review/detailLocator.js.map +1 -1
- package/out/review/diffReviewRunner.js +191 -0
- package/out/review/diffReviewRunner.js.map +1 -0
- package/out/review/index.js +12 -45
- package/out/review/index.js.map +1 -1
- package/out/review/mockProvider.js +55 -44
- package/out/review/mockProvider.js.map +1 -1
- package/out/review/openaiProvider.js +31 -20
- package/out/review/openaiProvider.js.map +1 -1
- package/out/review/parser.js +39 -2
- package/out/review/parser.js.map +1 -1
- package/out/review/prompt.js +180 -8
- package/out/review/prompt.js.map +1 -1
- package/package.json +1 -1
|
@@ -0,0 +1,191 @@
|
|
|
1
|
+
"use strict";
|
|
2
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
+
exports.runDiffReview = runDiffReview;
|
|
4
|
+
const git_1 = require("../git");
|
|
5
|
+
const responseCorrection_1 = require("../llm/responseCorrection");
|
|
6
|
+
const chunkAggregate_1 = require("./chunkAggregate");
|
|
7
|
+
const detailLocator_1 = require("./detailLocator");
|
|
8
|
+
const parser_1 = require("./parser");
|
|
9
|
+
async function runDiffReview(runnerInput) {
|
|
10
|
+
const { input } = runnerInput;
|
|
11
|
+
if (!input.diff.trim()) {
|
|
12
|
+
throw new Error("Cannot review an empty diff.");
|
|
13
|
+
}
|
|
14
|
+
const maxDiffChars = input.maxDiffChars ?? 0;
|
|
15
|
+
const useChunked = maxDiffChars > 0 && input.diff.length > maxDiffChars;
|
|
16
|
+
const execution = useChunked
|
|
17
|
+
? await runChunkedReview(runnerInput, maxDiffChars)
|
|
18
|
+
: await runSingleStageReview(runnerInput);
|
|
19
|
+
return {
|
|
20
|
+
...execution,
|
|
21
|
+
blocked: (0, parser_1.shouldBlockReviewResult)(execution.result, input.threshold)
|
|
22
|
+
};
|
|
23
|
+
}
|
|
24
|
+
async function runSingleStageReview(runnerInput) {
|
|
25
|
+
const { provider, input } = runnerInput;
|
|
26
|
+
const rawReview = await provider.generateReview(input);
|
|
27
|
+
const result = await validateReviewWithRepair(runnerInput, rawReview, input.diff, input);
|
|
28
|
+
return {
|
|
29
|
+
result: { ...result, reviewMode: "single", summarySource: "direct" },
|
|
30
|
+
reviewMode: "single",
|
|
31
|
+
summarySource: "direct"
|
|
32
|
+
};
|
|
33
|
+
}
|
|
34
|
+
async function runChunkedReview(runnerInput, maxDiffChars) {
|
|
35
|
+
const { provider, input, onLog } = runnerInput;
|
|
36
|
+
const chunks = (0, git_1.buildDiffChunks)(input.diff, maxDiffChars);
|
|
37
|
+
const summaryDiff = (0, git_1.limitDiff)(input.diff, maxDiffChars);
|
|
38
|
+
if (chunks.length === 0) {
|
|
39
|
+
throw new Error("Failed to build review chunks from the diff.");
|
|
40
|
+
}
|
|
41
|
+
onLog?.(`[REVIEW][INFO] Large diff detected (${input.diff.length} chars > ${maxDiffChars}). Switching to chunked review with ${chunks.length} chunks.`);
|
|
42
|
+
const chunkResults = [];
|
|
43
|
+
for (const chunk of chunks) {
|
|
44
|
+
onLog?.(`[REVIEW][CHUNK][START] ${chunk.chunkIndex}/${chunk.chunkCount} files=${chunk.changedFiles.length || 0} truncated=${chunk.truncated ? "yes" : "no"}`);
|
|
45
|
+
const rawChunkReview = await provider.reviewChunk(input, chunk);
|
|
46
|
+
const chunkInput = buildChunkInput(input, chunk);
|
|
47
|
+
const parsedChunkReview = await validateReviewWithRepair(runnerInput, rawChunkReview, chunk.diff, chunkInput);
|
|
48
|
+
chunkResults.push(parsedChunkReview);
|
|
49
|
+
logChunkReviewResult(onLog, chunk, parsedChunkReview);
|
|
50
|
+
}
|
|
51
|
+
const localAggregate = (0, chunkAggregate_1.aggregateChunkReviews)(chunkResults, input.threshold);
|
|
52
|
+
let summarizedReview;
|
|
53
|
+
let summarySource = "summarized";
|
|
54
|
+
try {
|
|
55
|
+
const rawSummaryReview = await provider.reviewSummary(input, chunkResults);
|
|
56
|
+
summarizedReview = await validateReviewWithRepair(runnerInput, rawSummaryReview, summaryDiff.diff, {
|
|
57
|
+
...input,
|
|
58
|
+
diff: summaryDiff.diff
|
|
59
|
+
});
|
|
60
|
+
if (summarizedReview.decision !== localAggregate.decision || summarizedReview.score !== localAggregate.score) {
|
|
61
|
+
onLog?.(`[REVIEW][SUMMARY][WARN] Summary review score=${formatReviewScore(summarizedReview.score)}, decision=${summarizedReview.decision}; falling back to local aggregate score=${formatReviewScore(localAggregate.score)}, decision=${localAggregate.decision}.`);
|
|
62
|
+
summarizedReview = undefined;
|
|
63
|
+
summarySource = "local-fallback";
|
|
64
|
+
}
|
|
65
|
+
else if (summarizedReview.details.length === 0 && localAggregate.details.length > 0) {
|
|
66
|
+
onLog?.("[REVIEW][SUMMARY][WARN] Summary review returned no actionable details; falling back to local aggregate.");
|
|
67
|
+
summarizedReview = undefined;
|
|
68
|
+
summarySource = "local-fallback";
|
|
69
|
+
}
|
|
70
|
+
}
|
|
71
|
+
catch (error) {
|
|
72
|
+
const message = error instanceof Error ? error.message : String(error);
|
|
73
|
+
summarySource = "local-fallback";
|
|
74
|
+
onLog?.(`[REVIEW][SUMMARY][WARN] Summary review failed, falling back to local aggregate: ${message}`);
|
|
75
|
+
}
|
|
76
|
+
const merged = mergeChunkedReview(provider.name, localAggregate, summarizedReview);
|
|
77
|
+
return {
|
|
78
|
+
result: { ...merged, reviewMode: "chunked", summarySource, chunkCount: chunks.length },
|
|
79
|
+
reviewMode: "chunked",
|
|
80
|
+
chunkCount: chunks.length,
|
|
81
|
+
summarySource
|
|
82
|
+
};
|
|
83
|
+
}
|
|
84
|
+
function buildChunkInput(input, chunk) {
|
|
85
|
+
return {
|
|
86
|
+
...input,
|
|
87
|
+
diff: chunk.diff,
|
|
88
|
+
changedFiles: chunk.changedFiles
|
|
89
|
+
};
|
|
90
|
+
}
|
|
91
|
+
async function validateReviewWithRepair(runnerInput, rawResponse, diff, repairInput) {
|
|
92
|
+
const { provider, input, onLog } = runnerInput;
|
|
93
|
+
return (0, responseCorrection_1.runWithResponseCorrectionRetries)({
|
|
94
|
+
initialResponse: rawResponse,
|
|
95
|
+
maxCorrectionRetries: runnerInput.llmResponseCorrectionRetryCount,
|
|
96
|
+
validateResponse: async (response) => {
|
|
97
|
+
const parsed = tryParseAndValidate(response, provider.name, input.reviewLanguage, input.threshold, diff);
|
|
98
|
+
if (!parsed.ok) {
|
|
99
|
+
throw new ReviewValidationError(parsed.issues);
|
|
100
|
+
}
|
|
101
|
+
return parsed.result;
|
|
102
|
+
},
|
|
103
|
+
isRetryableError: (error) => error instanceof ReviewValidationError,
|
|
104
|
+
onRetryableError: ({ correctionAttempt, maxCorrectionRetries }) => {
|
|
105
|
+
onLog?.(`[REVIEW][WARN] Review response validation failed; repair pass ${correctionAttempt}/${Math.max(maxCorrectionRetries, 1)}.`);
|
|
106
|
+
},
|
|
107
|
+
requestCorrection: ({ error, invalidResponse }) => provider.repairReview(repairInput, invalidResponse, error.issues)
|
|
108
|
+
});
|
|
109
|
+
}
|
|
110
|
+
function tryParseAndValidate(raw, providerName, reviewLanguage, threshold, diff) {
|
|
111
|
+
try {
|
|
112
|
+
const result = (0, detailLocator_1.alignReviewResultWithDiff)((0, parser_1.parseReviewResponse)(raw, providerName), diff);
|
|
113
|
+
const issues = [
|
|
114
|
+
...(0, parser_1.validateReviewResult)(result, threshold),
|
|
115
|
+
...(0, parser_1.validateReviewContentLanguage)(result, reviewLanguage)
|
|
116
|
+
];
|
|
117
|
+
if (issues.length > 0) {
|
|
118
|
+
return { ok: false, issues };
|
|
119
|
+
}
|
|
120
|
+
return { ok: true, result };
|
|
121
|
+
}
|
|
122
|
+
catch (error) {
|
|
123
|
+
return {
|
|
124
|
+
ok: false,
|
|
125
|
+
issues: [error instanceof Error ? error.message : String(error)]
|
|
126
|
+
};
|
|
127
|
+
}
|
|
128
|
+
}
|
|
129
|
+
function mergeChunkedReview(providerName, localAggregate, summarizedReview) {
|
|
130
|
+
const mergedDetails = mergeReviewDetails(localAggregate.details, summarizedReview?.details ?? []);
|
|
131
|
+
return {
|
|
132
|
+
decision: localAggregate.decision,
|
|
133
|
+
score: localAggregate.score,
|
|
134
|
+
summary: summarizedReview?.summary ?? localAggregate.summary,
|
|
135
|
+
provider: providerName,
|
|
136
|
+
details: mergedDetails,
|
|
137
|
+
raw: localAggregate.raw
|
|
138
|
+
};
|
|
139
|
+
}
|
|
140
|
+
function mergeReviewDetails(localDetails, summarizedDetails) {
|
|
141
|
+
if (summarizedDetails.length === 0) {
|
|
142
|
+
return (0, chunkAggregate_1.dedupeReviewDetails)(localDetails);
|
|
143
|
+
}
|
|
144
|
+
const mergedByKey = new Map();
|
|
145
|
+
for (const detail of localDetails) {
|
|
146
|
+
mergedByKey.set(buildReviewDetailKey(detail), detail);
|
|
147
|
+
}
|
|
148
|
+
for (const summarizedDetail of summarizedDetails) {
|
|
149
|
+
const key = buildReviewDetailKey(summarizedDetail);
|
|
150
|
+
const existing = mergedByKey.get(key);
|
|
151
|
+
if (!existing) {
|
|
152
|
+
continue;
|
|
153
|
+
}
|
|
154
|
+
mergedByKey.set(key, mergeReviewDetail(existing, summarizedDetail));
|
|
155
|
+
}
|
|
156
|
+
return [...mergedByKey.values()];
|
|
157
|
+
}
|
|
158
|
+
function mergeReviewDetail(base, candidate) {
|
|
159
|
+
return {
|
|
160
|
+
...base,
|
|
161
|
+
severity: candidate.severity ?? base.severity,
|
|
162
|
+
filePath: base.filePath ?? candidate.filePath,
|
|
163
|
+
lineNumber: base.lineNumber ?? candidate.lineNumber
|
|
164
|
+
};
|
|
165
|
+
}
|
|
166
|
+
function buildReviewDetailKey(detail) {
|
|
167
|
+
return [
|
|
168
|
+
detail.severity,
|
|
169
|
+
detail.filePath ?? "",
|
|
170
|
+
detail.lineNumber ?? "",
|
|
171
|
+
normalizeReviewDetailMessage(detail.message)
|
|
172
|
+
].join("|");
|
|
173
|
+
}
|
|
174
|
+
function normalizeReviewDetailMessage(message) {
|
|
175
|
+
return message.replace(/\s+/g, " ").trim().toLowerCase();
|
|
176
|
+
}
|
|
177
|
+
function logChunkReviewResult(onLog, chunk, result) {
|
|
178
|
+
onLog?.(`[REVIEW][CHUNK][END] ${chunk.chunkIndex}/${chunk.chunkCount} score=${formatReviewScore(result.score)}, decision=${result.decision}, files=${chunk.changedFiles.join(", ") || "none"}`);
|
|
179
|
+
}
|
|
180
|
+
function formatReviewScore(score) {
|
|
181
|
+
return score === null ? "N/A" : String(score);
|
|
182
|
+
}
|
|
183
|
+
class ReviewValidationError extends Error {
|
|
184
|
+
issues;
|
|
185
|
+
constructor(issues) {
|
|
186
|
+
super(issues.join("; "));
|
|
187
|
+
this.issues = issues;
|
|
188
|
+
this.name = "ReviewValidationError";
|
|
189
|
+
}
|
|
190
|
+
}
|
|
191
|
+
//# sourceMappingURL=diffReviewRunner.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"diffReviewRunner.js","sourceRoot":"","sources":["../../src/review/diffReviewRunner.ts"],"names":[],"mappings":";;AA4BA,sCAgBC;AA5CD,gCAA+D;AAC/D,kEAA6E;AAC7E,qDAAmG;AACnG,mDAA4D;AAC5D,qCAKkB;AAmBX,KAAK,UAAU,aAAa,CAAC,WAAkC;IACpE,MAAM,EAAE,KAAK,EAAE,GAAG,WAAW,CAAC;IAC9B,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,EAAE,EAAE,CAAC;QACvB,MAAM,IAAI,KAAK,CAAC,8BAA8B,CAAC,CAAC;IAClD,CAAC;IAED,MAAM,YAAY,GAAG,KAAK,CAAC,YAAY,IAAI,CAAC,CAAC;IAC7C,MAAM,UAAU,GAAG,YAAY,GAAG,CAAC,IAAI,KAAK,CAAC,IAAI,CAAC,MAAM,GAAG,YAAY,CAAC;IACxE,MAAM,SAAS,GAAG,UAAU;QAC1B,CAAC,CAAC,MAAM,gBAAgB,CAAC,WAAW,EAAE,YAAY,CAAC;QACnD,CAAC,CAAC,MAAM,oBAAoB,CAAC,WAAW,CAAC,CAAC;IAE5C,OAAO;QACL,GAAG,SAAS;QACZ,OAAO,EAAE,IAAA,gCAAuB,EAAC,SAAS,CAAC,MAAM,EAAE,KAAK,CAAC,SAAS,CAAC;KACpE,CAAC;AACJ,CAAC;AAED,KAAK,UAAU,oBAAoB,CACjC,WAAkC;IAElC,MAAM,EAAE,QAAQ,EAAE,KAAK,EAAE,GAAG,WAAW,CAAC;IACxC,MAAM,SAAS,GAAG,MAAM,QAAQ,CAAC,cAAc,CAAC,KAAK,CAAC,CAAC;IACvD,MAAM,MAAM,GAAG,MAAM,wBAAwB,CAAC,WAAW,EAAE,SAAS,EAAE,KAAK,CAAC,IAAI,EAAE,KAAK,CAAC,CAAC;IAEzF,OAAO;QACL,MAAM,EAAE,EAAE,GAAG,MAAM,EAAE,UAAU,EAAE,QAAQ,EAAE,aAAa,EAAE,QAAQ,EAAE;QACpE,UAAU,EAAE,QAAQ;QACpB,aAAa,EAAE,QAAQ;KACxB,CAAC;AACJ,CAAC;AAED,KAAK,UAAU,gBAAgB,CAC7B,WAAkC,EAClC,YAAoB;IAEpB,MAAM,EAAE,QAAQ,EAAE,KAAK,EAAE,KAAK,EAAE,GAAG,WAAW,CAAC;IAC/C,MAAM,MAAM,GAAG,IAAA,qBAAe,EAAC,KAAK,CAAC,IAAI,EAAE,YAAY,CAAC,CAAC;IACzD,MAAM,WAAW,GAAG,IAAA,eAAS,EAAC,KAAK,CAAC,IAAI,EAAE,YAAY,CAAC,CAAC;IACxD,IAAI,MAAM,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACxB,MAAM,IAAI,KAAK,CAAC,8CAA8C,CAAC,CAAC;IAClE,CAAC;IAED,KAAK,EAAE,CACL,uCAAuC,KAAK,CAAC,IAAI,CAAC,MAAM,YAAY,YAAY,uCAAuC,MAAM,CAAC,MAAM,UAAU,CAC/I,CAAC;IAEF,MAAM,YAAY,GAA4B,EAAE,CAAC;IACjD,KAAK,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;QAC3B,KAAK,EAAE,CACL,0BAA0B,KAAK,CAAC,UAAU,IAAI,KAAK,CAAC,UAAU,UAAU,KAAK,CAAC,YAAY,CAAC,MAAM,IAAI,CAAC,cAAc,KAAK,CAAC,SAAS,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,IAAI,EAAE,CACrJ,CAAC;QAEF,MAAM,cAAc,GAAG,MAAM,QAAQ,CAAC,WAAW,CAAC,KAAK,EAAE,KAAK,CAAC,CAAC;QAChE,MAAM,UAAU,GAAG,eAAe,CAAC,KAAK,EAAE,KAAK,CAAC,CAAC;QACjD,MAAM,iBAAiB,GAAG,MAAM,wBAAwB,CAAC,WAAW,EAAE,cAAc,EAAE,KAAK,CAAC,IAAI,EAAE,UAAU,CAAC,CAAC;QAC9G,YAAY,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAC;QACrC,oBAAoB,CAAC,KAAK,EAAE,KAAK,EAAE,iBAAiB,CAAC,CAAC;IACxD,CAAC;IAED,MAAM,cAAc,GAAG,IAAA,sCAAqB,EAAC,YAAY,EAAE,KAAK,CAAC,SAAS,CAAC,CAAC;IAC5E,IAAI,gBAAmD,CAAC;IACxD,IAAI,aAAa,GAA4C,YAAY,CAAC;IAE1E,IAAI,CAAC;QACH,MAAM,gBAAgB,GAAG,MAAM,QAAQ,CAAC,aAAa,CAAC,KAAK,EAAE,YAAY,CAAC,CAAC;QAC3E,gBAAgB,GAAG,MAAM,wBAAwB,CAAC,WAAW,EAAE,gBAAgB,EAAE,WAAW,CAAC,IAAI,EAAE;YACjG,GAAG,KAAK;YACR,IAAI,EAAE,WAAW,CAAC,IAAI;SACvB,CAAC,CAAC;QACH,IAAI,gBAAgB,CAAC,QAAQ,KAAK,cAAc,CAAC,QAAQ,IAAI,gBAAgB,CAAC,KAAK,KAAK,cAAc,CAAC,KAAK,EAAE,CAAC;YAC7G,KAAK,EAAE,CACL,gDAAgD,iBAAiB,CAAC,gBAAgB,CAAC,KAAK,CAAC,cAAc,gBAAgB,CAAC,QAAQ,2CAA2C,iBAAiB,CAAC,cAAc,CAAC,KAAK,CAAC,cAAc,cAAc,CAAC,QAAQ,GAAG,CAC3P,CAAC;YACF,gBAAgB,GAAG,SAAS,CAAC;YAC7B,aAAa,GAAG,gBAAgB,CAAC;QACnC,CAAC;aAAM,IAAI,gBAAgB,CAAC,OAAO,CAAC,MAAM,KAAK,CAAC,IAAI,cAAc,CAAC,OAAO,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACtF,KAAK,EAAE,CAAC,yGAAyG,CAAC,CAAC;YACnH,gBAAgB,GAAG,SAAS,CAAC;YAC7B,aAAa,GAAG,gBAAgB,CAAC;QACnC,CAAC;IACH,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,MAAM,OAAO,GAAG,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;QACvE,aAAa,GAAG,gBAAgB,CAAC;QACjC,KAAK,EAAE,CAAC,mFAAmF,OAAO,EAAE,CAAC,CAAC;IACxG,CAAC;IAED,MAAM,MAAM,GAAG,kBAAkB,CAAC,QAAQ,CAAC,IAAI,EAAE,cAAc,EAAE,gBAAgB,CAAC,CAAC;IACnF,OAAO;QACL,MAAM,EAAE,EAAE,GAAG,MAAM,EAAE,UAAU,EAAE,SAAS,EAAE,aAAa,EAAE,UAAU,EAAE,MAAM,CAAC,MAAM,EAAE;QACtF,UAAU,EAAE,SAAS;QACrB,UAAU,EAAE,MAAM,CAAC,MAAM;QACzB,aAAa;KACd,CAAC;AACJ,CAAC;AAED,SAAS,eAAe,CAAC,KAA2B,EAAE,KAAgB;IACpE,OAAO;QACL,GAAG,KAAK;QACR,IAAI,EAAE,KAAK,CAAC,IAAI;QAChB,YAAY,EAAE,KAAK,CAAC,YAAY;KACjC,CAAC;AACJ,CAAC;AAED,KAAK,UAAU,wBAAwB,CACrC,WAAkC,EAClC,WAAmB,EACnB,IAAY,EACZ,WAAiC;IAEjC,MAAM,EAAE,QAAQ,EAAE,KAAK,EAAE,KAAK,EAAE,GAAG,WAAW,CAAC;IAC/C,OAAO,IAAA,qDAAgC,EAA+C;QACpF,eAAe,EAAE,WAAW;QAC5B,oBAAoB,EAAE,WAAW,CAAC,+BAA+B;QACjE,gBAAgB,EAAE,KAAK,EAAE,QAAQ,EAAE,EAAE;YACnC,MAAM,MAAM,GAAG,mBAAmB,CAAC,QAAQ,EAAE,QAAQ,CAAC,IAAI,EAAE,KAAK,CAAC,cAAc,EAAE,KAAK,CAAC,SAAS,EAAE,IAAI,CAAC,CAAC;YACzG,IAAI,CAAC,MAAM,CAAC,EAAE,EAAE,CAAC;gBACf,MAAM,IAAI,qBAAqB,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC;YACjD,CAAC;YAED,OAAO,MAAM,CAAC,MAAM,CAAC;QACvB,CAAC;QACD,gBAAgB,EAAE,CAAC,KAAK,EAAkC,EAAE,CAAC,KAAK,YAAY,qBAAqB;QACnG,gBAAgB,EAAE,CAAC,EAAE,iBAAiB,EAAE,oBAAoB,EAAE,EAAE,EAAE;YAChE,KAAK,EAAE,CACL,iEAAiE,iBAAiB,IAAI,IAAI,CAAC,GAAG,CAAC,oBAAoB,EAAE,CAAC,CAAC,GAAG,CAC3H,CAAC;QACJ,CAAC;QACD,iBAAiB,EAAE,CAAC,EAAE,KAAK,EAAE,eAAe,EAAE,EAAE,EAAE,CAChD,QAAQ,CAAC,YAAY,CAAC,WAAW,EAAE,eAAe,EAAE,KAAK,CAAC,MAAM,CAAC;KACpE,CAAC,CAAC;AACL,CAAC;AAED,SAAS,mBAAmB,CAC1B,GAAW,EACX,YAAoB,EACpB,cAAsD,EACtD,SAAiB,EACjB,IAAY;IAEZ,IAAI,CAAC;QACH,MAAM,MAAM,GAAG,IAAA,yCAAyB,EAAC,IAAA,4BAAmB,EAAC,GAAG,EAAE,YAAY,CAAC,EAAE,IAAI,CAAC,CAAC;QACvF,MAAM,MAAM,GAAG;YACb,GAAG,IAAA,6BAAoB,EAAC,MAAM,EAAE,SAAS,CAAC;YAC1C,GAAG,IAAA,sCAA6B,EAAC,MAAM,EAAE,cAAc,CAAC;SACzD,CAAC;QACF,IAAI,MAAM,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACtB,OAAO,EAAE,EAAE,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC;QAC/B,CAAC;QAED,OAAO,EAAE,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE,CAAC;IAC9B,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,OAAO;YACL,EAAE,EAAE,KAAK;YACT,MAAM,EAAE,CAAC,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;SACjE,CAAC;IACJ,CAAC;AACH,CAAC;AAED,SAAS,kBAAkB,CACzB,YAAoB,EACpB,cAAmC,EACnC,gBAAwC;IAExC,MAAM,aAAa,GAAG,kBAAkB,CAAC,cAAc,CAAC,OAAO,EAAE,gBAAgB,EAAE,OAAO,IAAI,EAAE,CAAC,CAAC;IAClG,OAAO;QACL,QAAQ,EAAE,cAAc,CAAC,QAAQ;QACjC,KAAK,EAAE,cAAc,CAAC,KAAK;QAC3B,OAAO,EAAE,gBAAgB,EAAE,OAAO,IAAI,cAAc,CAAC,OAAO;QAC5D,QAAQ,EAAE,YAAY;QACtB,OAAO,EAAE,aAAa;QACtB,GAAG,EAAE,cAAc,CAAC,GAAG;KACxB,CAAC;AACJ,CAAC;AAED,SAAS,kBAAkB,CAAC,YAA4B,EAAE,iBAAiC;IACzF,IAAI,iBAAiB,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACnC,OAAO,IAAA,oCAAmB,EAAC,YAAY,CAAC,CAAC;IAC3C,CAAC;IAED,MAAM,WAAW,GAAG,IAAI,GAAG,EAAwB,CAAC;IACpD,KAAK,MAAM,MAAM,IAAI,YAAY,EAAE,CAAC;QAClC,WAAW,CAAC,GAAG,CAAC,oBAAoB,CAAC,MAAM,CAAC,EAAE,MAAM,CAAC,CAAC;IACxD,CAAC;IAED,KAAK,MAAM,gBAAgB,IAAI,iBAAiB,EAAE,CAAC;QACjD,MAAM,GAAG,GAAG,oBAAoB,CAAC,gBAAgB,CAAC,CAAC;QACnD,MAAM,QAAQ,GAAG,WAAW,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC;QACtC,IAAI,CAAC,QAAQ,EAAE,CAAC;YACd,SAAS;QACX,CAAC;QAED,WAAW,CAAC,GAAG,CAAC,GAAG,EAAE,iBAAiB,CAAC,QAAQ,EAAE,gBAAgB,CAAC,CAAC,CAAC;IACtE,CAAC;IAED,OAAO,CAAC,GAAG,WAAW,CAAC,MAAM,EAAE,CAAC,CAAC;AACnC,CAAC;AAED,SAAS,iBAAiB,CAAC,IAAkB,EAAE,SAAuB;IACpE,OAAO;QACL,GAAG,IAAI;QACP,QAAQ,EAAE,SAAS,CAAC,QAAQ,IAAI,IAAI,CAAC,QAAQ;QAC7C,QAAQ,EAAE,IAAI,CAAC,QAAQ,IAAI,SAAS,CAAC,QAAQ;QAC7C,UAAU,EAAE,IAAI,CAAC,UAAU,IAAI,SAAS,CAAC,UAAU;KACpD,CAAC;AACJ,CAAC;AAED,SAAS,oBAAoB,CAAC,MAAoB;IAChD,OAAO;QACL,MAAM,CAAC,QAAQ;QACf,MAAM,CAAC,QAAQ,IAAI,EAAE;QACrB,MAAM,CAAC,UAAU,IAAI,EAAE;QACvB,4BAA4B,CAAC,MAAM,CAAC,OAAO,CAAC;KAC7C,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;AACd,CAAC;AAED,SAAS,4BAA4B,CAAC,OAAe;IACnD,OAAO,OAAO,CAAC,OAAO,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC;AAC3D,CAAC;AAED,SAAS,oBAAoB,CAC3B,KAA8C,EAC9C,KAAgB,EAChB,MAA6B;IAE7B,KAAK,EAAE,CACL,wBAAwB,KAAK,CAAC,UAAU,IAAI,KAAK,CAAC,UAAU,UAAU,iBAAiB,CAAC,MAAM,CAAC,KAAK,CAAC,cAAc,MAAM,CAAC,QAAQ,WAAW,KAAK,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,MAAM,EAAE,CACvL,CAAC;AACJ,CAAC;AAED,SAAS,iBAAiB,CAAC,KAAqC;IAC9D,OAAO,KAAK,KAAK,IAAI,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;AAChD,CAAC;AAED,MAAM,qBAAsB,SAAQ,KAAK;IACX;IAA5B,YAA4B,MAAgB;QAC1C,KAAK,CAAC,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;QADC,WAAM,GAAN,MAAM,CAAU;QAE1C,IAAI,CAAC,IAAI,GAAG,uBAAuB,CAAC;IACtC,CAAC;CACF"}
|
package/out/review/index.js
CHANGED
|
@@ -15,32 +15,26 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
|
|
|
15
15
|
};
|
|
16
16
|
Object.defineProperty(exports, "__esModule", { value: true });
|
|
17
17
|
exports.runReviewExecution = runReviewExecution;
|
|
18
|
-
const
|
|
19
|
-
const openaiProvider_1 = require("./openaiProvider");
|
|
20
|
-
const detailLocator_1 = require("./detailLocator");
|
|
18
|
+
const diffReviewRunner_1 = require("./diffReviewRunner");
|
|
21
19
|
const parser_1 = require("./parser");
|
|
20
|
+
const openaiProvider_1 = require("./openaiProvider");
|
|
22
21
|
const mockProvider_1 = require("./mockProvider");
|
|
23
22
|
__exportStar(require("./types"), exports);
|
|
24
23
|
__exportStar(require("./provider"), exports);
|
|
25
24
|
async function runReviewExecution(input, options) {
|
|
26
25
|
const provider = options.provider ?? createProvider(options.config, options.providerMode ?? resolveProviderMode(), options.mockMode);
|
|
27
|
-
const
|
|
28
|
-
|
|
29
|
-
|
|
30
|
-
|
|
31
|
-
|
|
32
|
-
const parsed = tryParseAndValidate(response, provider.name, input.reviewLanguage, input.threshold, input.diff);
|
|
33
|
-
if (!parsed.ok) {
|
|
34
|
-
throw new ReviewResponseValidationError(parsed.issues);
|
|
35
|
-
}
|
|
36
|
-
return parsed.result;
|
|
37
|
-
},
|
|
38
|
-
isRetryableError: (error) => error instanceof ReviewResponseValidationError,
|
|
39
|
-
requestCorrection: ({ error, invalidResponse }) => provider.repairReview(input, invalidResponse, error.issues)
|
|
26
|
+
const runner = await (0, diffReviewRunner_1.runDiffReview)({
|
|
27
|
+
provider,
|
|
28
|
+
input,
|
|
29
|
+
llmResponseCorrectionRetryCount: options.config.connection.llmResponseCorrectionRetryCount,
|
|
30
|
+
onLog: options.onLog
|
|
40
31
|
});
|
|
41
32
|
return {
|
|
42
|
-
...result,
|
|
43
|
-
decision: (0, parser_1.shouldBlockReviewResult)(result, input.threshold) ? "block" : "pass"
|
|
33
|
+
...runner.result,
|
|
34
|
+
decision: (0, parser_1.shouldBlockReviewResult)(runner.result, input.threshold) ? "block" : "pass",
|
|
35
|
+
reviewMode: runner.reviewMode,
|
|
36
|
+
summarySource: runner.summarySource,
|
|
37
|
+
chunkCount: runner.chunkCount
|
|
44
38
|
};
|
|
45
39
|
}
|
|
46
40
|
function resolveProviderMode() {
|
|
@@ -51,31 +45,4 @@ function resolveProviderMode() {
|
|
|
51
45
|
function createProvider(config, mode, mockMode) {
|
|
52
46
|
return mode === "mock" ? (0, mockProvider_1.createMockReviewProvider)({ mode: mockMode }) : (0, openaiProvider_1.createOpenAICompatibleReviewProvider)(config);
|
|
53
47
|
}
|
|
54
|
-
function tryParseAndValidate(raw, providerName, reviewLanguage, threshold, diff) {
|
|
55
|
-
try {
|
|
56
|
-
const result = (0, detailLocator_1.alignReviewResultWithDiff)((0, parser_1.parseReviewResponse)(raw, providerName), diff);
|
|
57
|
-
const issues = [
|
|
58
|
-
...(0, parser_1.validateReviewResult)(result, threshold),
|
|
59
|
-
...(0, parser_1.validateReviewContentLanguage)(result, reviewLanguage)
|
|
60
|
-
];
|
|
61
|
-
if (issues.length > 0) {
|
|
62
|
-
return { ok: false, issues };
|
|
63
|
-
}
|
|
64
|
-
return { ok: true, result };
|
|
65
|
-
}
|
|
66
|
-
catch (error) {
|
|
67
|
-
return {
|
|
68
|
-
ok: false,
|
|
69
|
-
issues: [error instanceof Error ? error.message : String(error)]
|
|
70
|
-
};
|
|
71
|
-
}
|
|
72
|
-
}
|
|
73
|
-
class ReviewResponseValidationError extends Error {
|
|
74
|
-
issues;
|
|
75
|
-
constructor(issues) {
|
|
76
|
-
super(issues.join("; "));
|
|
77
|
-
this.issues = issues;
|
|
78
|
-
this.name = "ReviewResponseValidationError";
|
|
79
|
-
}
|
|
80
|
-
}
|
|
81
48
|
//# sourceMappingURL=index.js.map
|
package/out/review/index.js.map
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"index.js","sourceRoot":"","sources":["../../src/review/index.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;
|
|
1
|
+
{"version":3,"file":"index.js","sourceRoot":"","sources":["../../src/review/index.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;AAWA,gDAyBC;AAnCD,yDAAmD;AACnD,qCAAmD;AACnD,qDAAwE;AACxE,iDAA0D;AAI1D,0CAAwB;AACxB,6CAA2B;AAEpB,KAAK,UAAU,kBAAkB,CACtC,KAA2B,EAC3B,OAMC;IAED,MAAM,QAAQ,GAAG,OAAO,CAAC,QAAQ,IAAI,cAAc,CAAC,OAAO,CAAC,MAAM,EAAE,OAAO,CAAC,YAAY,IAAI,mBAAmB,EAAE,EAAE,OAAO,CAAC,QAAQ,CAAC,CAAC;IACrI,MAAM,MAAM,GAAG,MAAM,IAAA,gCAAa,EAAC;QACjC,QAAQ;QACR,KAAK;QACL,+BAA+B,EAAE,OAAO,CAAC,MAAM,CAAC,UAAU,CAAC,+BAA+B;QAC1F,KAAK,EAAE,OAAO,CAAC,KAAK;KACrB,CAAC,CAAC;IAEH,OAAO;QACL,GAAG,MAAM,CAAC,MAAM;QAChB,QAAQ,EAAE,IAAA,gCAAuB,EAAC,MAAM,CAAC,MAAM,EAAE,KAAK,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM;QACpF,UAAU,EAAE,MAAM,CAAC,UAAU;QAC7B,aAAa,EAAE,MAAM,CAAC,aAAa;QACnC,UAAU,EAAE,MAAM,CAAC,UAAU;KAC9B,CAAC;AACJ,CAAC;AAED,SAAS,mBAAmB;IAC1B,OAAO,OAAO,CAAC,GAAG,CAAC,4BAA4B,EAAE,IAAI,EAAE,CAAC,WAAW,EAAE,KAAK,MAAM;QAC9E,CAAC,CAAC,MAAM;QACR,CAAC,CAAC,mBAAmB,CAAC;AAC1B,CAAC;AAED,SAAS,cAAc,CACrB,MAAiB,EACjB,IAAkC,EAClC,QAAiB;IAEjB,OAAO,IAAI,KAAK,MAAM,CAAC,CAAC,CAAC,IAAA,uCAAwB,EAAC,EAAE,IAAI,EAAE,QAAQ,EAAE,CAAC,CAAC,CAAC,CAAC,IAAA,qDAAoC,EAAC,MAAM,CAAC,CAAC;AACvH,CAAC"}
|
|
@@ -2,54 +2,58 @@
|
|
|
2
2
|
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
3
|
exports.createMockReviewProvider = createMockReviewProvider;
|
|
4
4
|
function createMockReviewProvider(config = {}) {
|
|
5
|
-
|
|
6
|
-
|
|
7
|
-
|
|
8
|
-
|
|
9
|
-
|
|
10
|
-
|
|
11
|
-
|
|
12
|
-
|
|
13
|
-
if (mode === "block") {
|
|
14
|
-
return JSON.stringify({
|
|
15
|
-
score: Math.min(input.threshold, 4),
|
|
16
|
-
decision: "block",
|
|
17
|
-
summary: localizeReviewText(input.reviewLanguage, "block-summary", summaryHint),
|
|
18
|
-
details: [
|
|
19
|
-
{
|
|
20
|
-
severity: "P1",
|
|
21
|
-
message: localizeReviewText(input.reviewLanguage, "block-detail")
|
|
22
|
-
}
|
|
23
|
-
]
|
|
24
|
-
});
|
|
25
|
-
}
|
|
26
|
-
if (mode === "block-high-score") {
|
|
27
|
-
return JSON.stringify({
|
|
28
|
-
score: Math.max(input.threshold + 1, 9),
|
|
29
|
-
decision: "block",
|
|
30
|
-
summary: localizeReviewText(input.reviewLanguage, "block-high-score-summary", summaryHint),
|
|
31
|
-
details: [
|
|
32
|
-
{
|
|
33
|
-
severity: "P2",
|
|
34
|
-
message: localizeReviewText(input.reviewLanguage, "block-high-score-detail")
|
|
35
|
-
}
|
|
36
|
-
]
|
|
37
|
-
});
|
|
38
|
-
}
|
|
39
|
-
if (mode === "na") {
|
|
40
|
-
return JSON.stringify({
|
|
41
|
-
score: null,
|
|
42
|
-
decision: "pass",
|
|
43
|
-
summary: localizeReviewText(input.reviewLanguage, "na-summary", summaryHint),
|
|
44
|
-
details: []
|
|
45
|
-
});
|
|
46
|
-
}
|
|
5
|
+
const resolveMode = () => config.mode?.trim().toLowerCase() ?? process.env.SMART_COMMIT_REVIEW_MODE?.trim().toLowerCase();
|
|
6
|
+
const buildReview = (input) => {
|
|
7
|
+
const mode = resolveMode();
|
|
8
|
+
const summaryHint = input.changedFiles.length > 0 ? `${input.changedFiles.length} changed file(s)` : "no changed files";
|
|
9
|
+
if (mode === "invalid") {
|
|
10
|
+
return "not-json";
|
|
11
|
+
}
|
|
12
|
+
if (mode === "block") {
|
|
47
13
|
return JSON.stringify({
|
|
48
|
-
score: Math.
|
|
14
|
+
score: Math.min(input.threshold, 4),
|
|
15
|
+
decision: "block",
|
|
16
|
+
summary: localizeReviewText(input.reviewLanguage, "block-summary", summaryHint),
|
|
17
|
+
details: [
|
|
18
|
+
{
|
|
19
|
+
severity: "P1",
|
|
20
|
+
message: localizeReviewText(input.reviewLanguage, "block-detail")
|
|
21
|
+
}
|
|
22
|
+
]
|
|
23
|
+
});
|
|
24
|
+
}
|
|
25
|
+
if (mode === "block-high-score") {
|
|
26
|
+
return JSON.stringify({
|
|
27
|
+
score: Math.max(input.threshold + 1, 9),
|
|
28
|
+
decision: "block",
|
|
29
|
+
summary: localizeReviewText(input.reviewLanguage, "block-high-score-summary", summaryHint),
|
|
30
|
+
details: [
|
|
31
|
+
{
|
|
32
|
+
severity: "P2",
|
|
33
|
+
message: localizeReviewText(input.reviewLanguage, "block-high-score-detail")
|
|
34
|
+
}
|
|
35
|
+
]
|
|
36
|
+
});
|
|
37
|
+
}
|
|
38
|
+
if (mode === "na") {
|
|
39
|
+
return JSON.stringify({
|
|
40
|
+
score: null,
|
|
49
41
|
decision: "pass",
|
|
50
|
-
summary: localizeReviewText(input.reviewLanguage, "
|
|
42
|
+
summary: localizeReviewText(input.reviewLanguage, "na-summary", summaryHint),
|
|
51
43
|
details: []
|
|
52
44
|
});
|
|
45
|
+
}
|
|
46
|
+
return JSON.stringify({
|
|
47
|
+
score: Math.max(input.threshold + 1, 8),
|
|
48
|
+
decision: "pass",
|
|
49
|
+
summary: localizeReviewText(input.reviewLanguage, "pass-summary", summaryHint),
|
|
50
|
+
details: []
|
|
51
|
+
});
|
|
52
|
+
};
|
|
53
|
+
return {
|
|
54
|
+
name: "mock",
|
|
55
|
+
async generateReview(input) {
|
|
56
|
+
return buildReview(input);
|
|
53
57
|
},
|
|
54
58
|
async repairReview(input, invalidResponse, issues) {
|
|
55
59
|
const summaryHint = input.changedFiles.length > 0 ? `${input.changedFiles.length} changed file(s)` : "no changed files";
|
|
@@ -65,6 +69,13 @@ function createMockReviewProvider(config = {}) {
|
|
|
65
69
|
}
|
|
66
70
|
]
|
|
67
71
|
});
|
|
72
|
+
},
|
|
73
|
+
async reviewChunk(input, chunk) {
|
|
74
|
+
return buildReview({ ...input, diff: chunk.diff, changedFiles: chunk.changedFiles });
|
|
75
|
+
},
|
|
76
|
+
async reviewSummary(input, chunkReviews) {
|
|
77
|
+
void chunkReviews;
|
|
78
|
+
return buildReview(input);
|
|
68
79
|
}
|
|
69
80
|
};
|
|
70
81
|
}
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"mockProvider.js","sourceRoot":"","sources":["../../src/review/mockProvider.ts"],"names":[],"mappings":";;
|
|
1
|
+
{"version":3,"file":"mockProvider.js","sourceRoot":"","sources":["../../src/review/mockProvider.ts"],"names":[],"mappings":";;AAIA,4DAqFC;AArFD,SAAgB,wBAAwB,CAAC,SAA4B,EAAE;IACrE,MAAM,WAAW,GAAG,GAAuB,EAAE,CAC3C,MAAM,CAAC,IAAI,EAAE,IAAI,EAAE,CAAC,WAAW,EAAE,IAAI,OAAO,CAAC,GAAG,CAAC,wBAAwB,EAAE,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC;IAElG,MAAM,WAAW,GAAG,CAAC,KAA2B,EAAU,EAAE;QAC1D,MAAM,IAAI,GAAG,WAAW,EAAE,CAAC;QAC3B,MAAM,WAAW,GAAG,KAAK,CAAC,YAAY,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,KAAK,CAAC,YAAY,CAAC,MAAM,kBAAkB,CAAC,CAAC,CAAC,kBAAkB,CAAC;QAExH,IAAI,IAAI,KAAK,SAAS,EAAE,CAAC;YACvB,OAAO,UAAU,CAAC;QACpB,CAAC;QAED,IAAI,IAAI,KAAK,OAAO,EAAE,CAAC;YACnB,OAAO,IAAI,CAAC,SAAS,CAAC;gBACpB,KAAK,EAAE,IAAI,CAAC,GAAG,CAAC,KAAK,CAAC,SAAS,EAAE,CAAC,CAAC;gBACnC,QAAQ,EAAE,OAAO;gBACjB,OAAO,EAAE,kBAAkB,CAAC,KAAK,CAAC,cAAc,EAAE,eAAe,EAAE,WAAW,CAAC;gBAC/E,OAAO,EAAE;oBACP;wBACE,QAAQ,EAAE,IAAI;wBACd,OAAO,EAAE,kBAAkB,CAAC,KAAK,CAAC,cAAc,EAAE,cAAc,CAAC;qBAClE;iBACF;aACF,CAAC,CAAC;QACL,CAAC;QAED,IAAI,IAAI,KAAK,kBAAkB,EAAE,CAAC;YAChC,OAAO,IAAI,CAAC,SAAS,CAAC;gBACpB,KAAK,EAAE,IAAI,CAAC,GAAG,CAAC,KAAK,CAAC,SAAS,GAAG,CAAC,EAAE,CAAC,CAAC;gBACvC,QAAQ,EAAE,OAAO;gBACjB,OAAO,EAAE,kBAAkB,CAAC,KAAK,CAAC,cAAc,EAAE,0BAA0B,EAAE,WAAW,CAAC;gBAC1F,OAAO,EAAE;oBACP;wBACE,QAAQ,EAAE,IAAI;wBACd,OAAO,EAAE,kBAAkB,CAAC,KAAK,CAAC,cAAc,EAAE,yBAAyB,CAAC;qBAC7E;iBACF;aACF,CAAC,CAAC;QACL,CAAC;QAED,IAAI,IAAI,KAAK,IAAI,EAAE,CAAC;YAClB,OAAO,IAAI,CAAC,SAAS,CAAC;gBACpB,KAAK,EAAE,IAAI;gBACX,QAAQ,EAAE,MAAM;gBAChB,OAAO,EAAE,kBAAkB,CAAC,KAAK,CAAC,cAAc,EAAE,YAAY,EAAE,WAAW,CAAC;gBAC5E,OAAO,EAAE,EAAE;aACZ,CAAC,CAAC;QACL,CAAC;QAEH,OAAO,IAAI,CAAC,SAAS,CAAC;YACpB,KAAK,EAAE,IAAI,CAAC,GAAG,CAAC,KAAK,CAAC,SAAS,GAAG,CAAC,EAAE,CAAC,CAAC;YACvC,QAAQ,EAAE,MAAM;YAChB,OAAO,EAAE,kBAAkB,CAAC,KAAK,CAAC,cAAc,EAAE,cAAc,EAAE,WAAW,CAAC;YAC9E,OAAO,EAAE,EAAE;SACZ,CAAC,CAAC;IACL,CAAC,CAAC;IAEF,OAAO;QACL,IAAI,EAAE,MAAM;QACZ,KAAK,CAAC,cAAc,CAAC,KAA2B;YAC9C,OAAO,WAAW,CAAC,KAAK,CAAC,CAAC;QAC5B,CAAC;QACD,KAAK,CAAC,YAAY,CAAC,KAA2B,EAAE,eAAuB,EAAE,MAAgB;YACvF,MAAM,WAAW,GAAG,KAAK,CAAC,YAAY,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,KAAK,CAAC,YAAY,CAAC,MAAM,kBAAkB,CAAC,CAAC,CAAC,kBAAkB,CAAC;YACxH,KAAK,eAAe,CAAC;YACrB,OAAO,IAAI,CAAC,SAAS,CAAC;gBACpB,KAAK,EAAE,IAAI,CAAC,GAAG,CAAC,KAAK,CAAC,SAAS,GAAG,CAAC,EAAE,CAAC,CAAC;gBACvC,QAAQ,EAAE,MAAM;gBAChB,OAAO,EAAE,kBAAkB,CAAC,KAAK,CAAC,cAAc,EAAE,gBAAgB,EAAE,WAAW,CAAC;gBAChF,OAAO,EAAE;oBACP;wBACE,QAAQ,EAAE,OAAO;wBACjB,OAAO,EAAE,kBAAkB,CAAC,KAAK,CAAC,cAAc,EAAE,eAAe,EAAE,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;qBACvF;iBACF;aACF,CAAC,CAAC;QACL,CAAC;QACD,KAAK,CAAC,WAAW,CAAC,KAA2B,EAAE,KAAsB;YACnE,OAAO,WAAW,CAAC,EAAE,GAAG,KAAK,EAAE,IAAI,EAAE,KAAK,CAAC,IAAI,EAAE,YAAY,EAAE,KAAK,CAAC,YAAY,EAAE,CAAC,CAAC;QACvF,CAAC;QACD,KAAK,CAAC,aAAa,CAAC,KAA2B,EAAE,YAAqC;YACpF,KAAK,YAAY,CAAC;YAClB,OAAO,WAAW,CAAC,KAAK,CAAC,CAAC;QAC5B,CAAC;KACF,CAAC;AACJ,CAAC;AAED,SAAS,kBAAkB,CACzB,QAAgD,EAChD,GAQmB,EACnB,KAAK,GAAG,EAAE;IAEV,QAAQ,QAAQ,EAAE,CAAC;QACjB,KAAK,OAAO;YACV,OAAO,mBAAmB,CAAC,GAAG,EAAE,KAAK,CAAC,CAAC;QACzC,KAAK,OAAO;YACV,OAAO,8BAA8B,CAAC,GAAG,EAAE,KAAK,CAAC,CAAC;QACpD,KAAK,IAAI;YACP,OAAO,iBAAiB,CAAC,GAAG,EAAE,KAAK,CAAC,CAAC;QACvC,KAAK,IAAI;YACP,OAAO,mBAAmB,CAAC,GAAG,EAAE,KAAK,CAAC,CAAC;QACzC,KAAK,IAAI;YACP,OAAO,kBAAkB,CAAC,GAAG,EAAE,KAAK,CAAC,CAAC;QACxC,KAAK,IAAI;YACP,OAAO,oBAAoB,CAAC,GAAG,EAAE,KAAK,CAAC,CAAC;QAC1C;YACE,OAAO,mBAAmB,CAAC,GAAG,EAAE,KAAK,CAAC,CAAC;IAC3C,CAAC;AACH,CAAC;AAED,SAAS,mBAAmB,CAAC,GAAW,EAAE,KAAa;IACrD,QAAQ,GAAG,EAAE,CAAC;QACZ,KAAK,cAAc;YACjB,OAAO,sCAAsC,KAAK,IAAI,CAAC;QACzD,KAAK,eAAe;YAClB,OAAO,uCAAuC,KAAK,IAAI,CAAC;QAC1D,KAAK,cAAc;YACjB,OAAO,sBAAsB,CAAC;QAChC,KAAK,0BAA0B;YAC7B,OAAO,gEAAgE,KAAK,IAAI,CAAC;QACnF,KAAK,yBAAyB;YAC5B,OAAO,0BAA0B,CAAC;QACpC,KAAK,YAAY;YACf,OAAO,wEAAwE,KAAK,IAAI,CAAC;QAC3F,KAAK,gBAAgB;YACnB,OAAO,wCAAwC,KAAK,IAAI,CAAC;QAC3D,KAAK,eAAe;YAClB,OAAO,0BAA0B,KAAK,EAAE,CAAC;QAC3C;YACE,OAAO,KAAK,CAAC;IACjB,CAAC;AACH,CAAC;AAED,SAAS,mBAAmB,CAAC,GAAW,EAAE,KAAa;IACrD,QAAQ,GAAG,EAAE,CAAC;QACZ,KAAK,cAAc;YACjB,OAAO,eAAe,KAAK,IAAI,CAAC;QAClC,KAAK,eAAe;YAClB,OAAO,eAAe,KAAK,IAAI,CAAC;QAClC,KAAK,cAAc;YACjB,OAAO,SAAS,CAAC;QACnB,KAAK,0BAA0B;YAC7B,OAAO,qBAAqB,KAAK,IAAI,CAAC;QACxC,KAAK,yBAAyB;YAC5B,OAAO,UAAU,CAAC;QACpB,KAAK,YAAY;YACf,OAAO,oBAAoB,KAAK,IAAI,CAAC;QACvC,KAAK,gBAAgB;YACnB,OAAO,iBAAiB,KAAK,IAAI,CAAC;QACpC,KAAK,eAAe;YAClB,OAAO,WAAW,KAAK,EAAE,CAAC;QAC5B;YACE,OAAO,KAAK,CAAC;IACjB,CAAC;AACH,CAAC;AAED,SAAS,8BAA8B,CAAC,GAAW,EAAE,KAAa;IAChE,QAAQ,GAAG,EAAE,CAAC;QACZ,KAAK,cAAc;YACjB,OAAO,eAAe,KAAK,IAAI,CAAC;QAClC,KAAK,eAAe;YAClB,OAAO,eAAe,KAAK,IAAI,CAAC;QAClC,KAAK,cAAc;YACjB,OAAO,SAAS,CAAC;QACnB,KAAK,0BAA0B;YAC7B,OAAO,qBAAqB,KAAK,IAAI,CAAC;QACxC,KAAK,yBAAyB;YAC5B,OAAO,UAAU,CAAC;QACpB,KAAK,YAAY;YACf,OAAO,oBAAoB,KAAK,IAAI,CAAC;QACvC,KAAK,gBAAgB;YACnB,OAAO,iBAAiB,KAAK,IAAI,CAAC;QACpC,KAAK,eAAe;YAClB,OAAO,WAAW,KAAK,EAAE,CAAC;QAC5B;YACE,OAAO,KAAK,CAAC;IACjB,CAAC;AACH,CAAC;AAED,SAAS,iBAAiB,CAAC,GAAW,EAAE,KAAa;IACnD,QAAQ,GAAG,EAAE,CAAC;QACZ,KAAK,cAAc;YACjB,OAAO,wBAAwB,KAAK,IAAI,CAAC;QAC3C,KAAK,eAAe;YAClB,OAAO,8BAA8B,KAAK,IAAI,CAAC;QACjD,KAAK,cAAc;YACjB,OAAO,oBAAoB,CAAC;QAC9B,KAAK,0BAA0B;YAC7B,OAAO,qDAAqD,KAAK,IAAI,CAAC;QACxE,KAAK,yBAAyB;YAC5B,OAAO,wBAAwB,CAAC;QAClC,KAAK,YAAY;YACf,OAAO,0DAA0D,KAAK,IAAI,CAAC;QAC7E,KAAK,gBAAgB;YACnB,OAAO,kCAAkC,KAAK,IAAI,CAAC;QACrD,KAAK,eAAe;YAClB,OAAO,8BAA8B,KAAK,EAAE,CAAC;QAC/C;YACE,OAAO,KAAK,CAAC;IACjB,CAAC;AACH,CAAC;AAED,SAAS,mBAAmB,CAAC,GAAW,EAAE,KAAa;IACrD,QAAQ,GAAG,EAAE,CAAC;QACZ,KAAK,cAAc;YACjB,OAAO,+BAA+B,KAAK,IAAI,CAAC;QAClD,KAAK,eAAe;YAClB,OAAO,8CAA8C,KAAK,IAAI,CAAC;QACjE,KAAK,cAAc;YACjB,OAAO,gCAAgC,CAAC;QAC1C,KAAK,0BAA0B;YAC7B,OAAO,+DAA+D,KAAK,IAAI,CAAC;QAClF,KAAK,yBAAyB;YAC5B,OAAO,kCAAkC,CAAC;QAC5C,KAAK,YAAY;YACf,OAAO,iDAAiD,KAAK,IAAI,CAAC;QACpE,KAAK,gBAAgB;YACnB,OAAO,mEAAmE,KAAK,IAAI,CAAC;QACtF,KAAK,eAAe;YAClB,OAAO,0BAA0B,KAAK,EAAE,CAAC;QAC3C;YACE,OAAO,KAAK,CAAC;IACjB,CAAC;AACH,CAAC;AAED,SAAS,kBAAkB,CAAC,GAAW,EAAE,KAAa;IACpD,QAAQ,GAAG,EAAE,CAAC;QACZ,KAAK,cAAc;YACjB,OAAO,uBAAuB,KAAK,IAAI,CAAC;QAC1C,KAAK,eAAe;YAClB,OAAO,sBAAsB,KAAK,IAAI,CAAC;QACzC,KAAK,cAAc;YACjB,OAAO,cAAc,CAAC;QACxB,KAAK,0BAA0B;YAC7B,OAAO,kCAAkC,KAAK,IAAI,CAAC;QACrD,KAAK,yBAAyB;YAC5B,OAAO,eAAe,CAAC;QACzB,KAAK,YAAY;YACf,OAAO,kCAAkC,KAAK,IAAI,CAAC;QACrD,KAAK,gBAAgB;YACnB,OAAO,6BAA6B,KAAK,IAAI,CAAC;QAChD,KAAK,eAAe;YAClB,OAAO,kBAAkB,KAAK,EAAE,CAAC;QACnC;YACE,OAAO,KAAK,CAAC;IACjB,CAAC;AACH,CAAC;AAED,SAAS,oBAAoB,CAAC,GAAW,EAAE,KAAa;IACtD,QAAQ,GAAG,EAAE,CAAC;QACZ,KAAK,cAAc;YACjB,OAAO,wBAAwB,KAAK,IAAI,CAAC;QAC3C,KAAK,eAAe;YAClB,OAAO,uBAAuB,KAAK,IAAI,CAAC;QAC1C,KAAK,cAAc;YACjB,OAAO,iBAAiB,CAAC;QAC3B,KAAK,0BAA0B;YAC7B,OAAO,+BAA+B,KAAK,IAAI,CAAC;QAClD,KAAK,yBAAyB;YAC5B,OAAO,kBAAkB,CAAC;QAC5B,KAAK,YAAY;YACf,OAAO,0BAA0B,KAAK,IAAI,CAAC;QAC7C,KAAK,gBAAgB;YACnB,OAAO,0BAA0B,KAAK,IAAI,CAAC;QAC7C,KAAK,eAAe;YAClB,OAAO,mBAAmB,KAAK,EAAE,CAAC;QACpC;YACE,OAAO,KAAK,CAAC;IACjB,CAAC;AACH,CAAC"}
|
|
@@ -5,21 +5,25 @@ const openaiCompatible_1 = require("../openaiCompatible");
|
|
|
5
5
|
const promptContext_1 = require("../promptContext");
|
|
6
6
|
const prompt_1 = require("./prompt");
|
|
7
7
|
function createOpenAICompatibleReviewProvider(config) {
|
|
8
|
+
const buildOptions = (input) => ({
|
|
9
|
+
promptAugmentation: (0, promptContext_1.buildPromptAugmentation)({
|
|
10
|
+
repositoryPath: input.repositoryPath,
|
|
11
|
+
skill: {
|
|
12
|
+
...config.review.skill,
|
|
13
|
+
promptTuning: input.skillPromptTuning ?? config.review.skill.promptTuning
|
|
14
|
+
},
|
|
15
|
+
categoryLabel: "review skill",
|
|
16
|
+
builtinCategory: "code-review"
|
|
17
|
+
}),
|
|
18
|
+
selectedSkillId: config.review.skill.id,
|
|
19
|
+
selectedSkillPath: config.review.skill.path
|
|
20
|
+
});
|
|
8
21
|
return {
|
|
9
22
|
name: "openai-compatible",
|
|
10
23
|
async generateReview(input) {
|
|
11
24
|
return (0, openaiCompatible_1.requestOpenAICompatibleText)({
|
|
12
25
|
config,
|
|
13
|
-
messages: (0, prompt_1.buildReviewMessages)(input,
|
|
14
|
-
promptAugmentation: (0, promptContext_1.buildPromptAugmentation)({
|
|
15
|
-
repositoryPath: input.repositoryPath,
|
|
16
|
-
skill: config.review.skill,
|
|
17
|
-
categoryLabel: "review skill",
|
|
18
|
-
builtinCategory: "code-review"
|
|
19
|
-
}),
|
|
20
|
-
selectedSkillId: config.review.skill.id,
|
|
21
|
-
selectedSkillPath: config.review.skill.path
|
|
22
|
-
}),
|
|
26
|
+
messages: (0, prompt_1.buildReviewMessages)(input, buildOptions(input)),
|
|
23
27
|
temperature: 0.1,
|
|
24
28
|
taskLabel: "review"
|
|
25
29
|
});
|
|
@@ -27,16 +31,23 @@ function createOpenAICompatibleReviewProvider(config) {
|
|
|
27
31
|
async repairReview(input, invalidResponse, issues) {
|
|
28
32
|
return (0, openaiCompatible_1.requestOpenAICompatibleText)({
|
|
29
33
|
config,
|
|
30
|
-
messages: (0, prompt_1.buildReviewRepairMessages)(input, invalidResponse, issues,
|
|
31
|
-
|
|
32
|
-
|
|
33
|
-
|
|
34
|
-
|
|
35
|
-
|
|
36
|
-
|
|
37
|
-
|
|
38
|
-
|
|
39
|
-
|
|
34
|
+
messages: (0, prompt_1.buildReviewRepairMessages)(input, invalidResponse, issues, buildOptions(input)),
|
|
35
|
+
temperature: 0.1,
|
|
36
|
+
taskLabel: "review"
|
|
37
|
+
});
|
|
38
|
+
},
|
|
39
|
+
async reviewChunk(input, chunk) {
|
|
40
|
+
return (0, openaiCompatible_1.requestOpenAICompatibleText)({
|
|
41
|
+
config,
|
|
42
|
+
messages: (0, prompt_1.buildChunkReviewMessages)(input, chunk, buildOptions(input)),
|
|
43
|
+
temperature: 0.1,
|
|
44
|
+
taskLabel: "review"
|
|
45
|
+
});
|
|
46
|
+
},
|
|
47
|
+
async reviewSummary(input, chunkReviews) {
|
|
48
|
+
return (0, openaiCompatible_1.requestOpenAICompatibleText)({
|
|
49
|
+
config,
|
|
50
|
+
messages: (0, prompt_1.buildSummaryReviewMessages)(input, chunkReviews, buildOptions(input)),
|
|
40
51
|
temperature: 0.1,
|
|
41
52
|
taskLabel: "review"
|
|
42
53
|
});
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"openaiProvider.js","sourceRoot":"","sources":["../../src/review/openaiProvider.ts"],"names":[],"mappings":";;
|
|
1
|
+
{"version":3,"file":"openaiProvider.js","sourceRoot":"","sources":["../../src/review/openaiProvider.ts"],"names":[],"mappings":";;AAcA,oFAkDC;AA/DD,0DAAkE;AAClE,oDAA2D;AAE3D,qCAOkB;AAGlB,SAAgB,oCAAoC,CAAC,MAAiB;IACpE,MAAM,YAAY,GAAG,CAAC,KAA2B,EAAuB,EAAE,CAAC,CAAC;QAC1E,kBAAkB,EAAE,IAAA,uCAAuB,EAAC;YAC1C,cAAc,EAAE,KAAK,CAAC,cAAc;YACpC,KAAK,EAAE;gBACL,GAAG,MAAM,CAAC,MAAM,CAAC,KAAK;gBACtB,YAAY,EAAE,KAAK,CAAC,iBAAiB,IAAI,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,YAAY;aAC1E;YACD,aAAa,EAAE,cAAc;YAC7B,eAAe,EAAE,aAAa;SAC/B,CAAC;QACF,eAAe,EAAE,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE;QACvC,iBAAiB,EAAE,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI;KAC5C,CAAC,CAAC;IAEH,OAAO;QACL,IAAI,EAAE,mBAAmB;QACzB,KAAK,CAAC,cAAc,CAAC,KAA2B;YAC9C,OAAO,IAAA,8CAA2B,EAAC;gBACjC,MAAM;gBACN,QAAQ,EAAE,IAAA,4BAAmB,EAAC,KAAK,EAAE,YAAY,CAAC,KAAK,CAAC,CAAC;gBACzD,WAAW,EAAE,GAAG;gBAChB,SAAS,EAAE,QAAQ;aACpB,CAAC,CAAC;QACL,CAAC;QACD,KAAK,CAAC,YAAY,CAAC,KAA2B,EAAE,eAAuB,EAAE,MAAgB;YACvF,OAAO,IAAA,8CAA2B,EAAC;gBACjC,MAAM;gBACN,QAAQ,EAAE,IAAA,kCAAyB,EAAC,KAAK,EAAE,eAAe,EAAE,MAAM,EAAE,YAAY,CAAC,KAAK,CAAC,CAAC;gBACxF,WAAW,EAAE,GAAG;gBAChB,SAAS,EAAE,QAAQ;aACpB,CAAC,CAAC;QACL,CAAC;QACD,KAAK,CAAC,WAAW,CAAC,KAA2B,EAAE,KAAsB;YACnE,OAAO,IAAA,8CAA2B,EAAC;gBACjC,MAAM;gBACN,QAAQ,EAAE,IAAA,iCAAwB,EAAC,KAAK,EAAE,KAAK,EAAE,YAAY,CAAC,KAAK,CAAC,CAAC;gBACrE,WAAW,EAAE,GAAG;gBAChB,SAAS,EAAE,QAAQ;aACpB,CAAC,CAAC;QACL,CAAC;QACD,KAAK,CAAC,aAAa,CAAC,KAA2B,EAAE,YAAqC;YACpF,OAAO,IAAA,8CAA2B,EAAC;gBACjC,MAAM;gBACN,QAAQ,EAAE,IAAA,mCAA0B,EAAC,KAAK,EAAE,YAAY,EAAE,YAAY,CAAC,KAAK,CAAC,CAAC;gBAC9E,WAAW,EAAE,GAAG;gBAChB,SAAS,EAAE,QAAQ;aACpB,CAAC,CAAC;QACL,CAAC;KACF,CAAC;AACJ,CAAC"}
|
package/out/review/parser.js
CHANGED
|
@@ -1,11 +1,44 @@
|
|
|
1
1
|
"use strict";
|
|
2
2
|
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
+
exports.stripReviewChunkPrefix = stripReviewChunkPrefix;
|
|
4
|
+
exports.countReviewDetailsBySeverity = countReviewDetailsBySeverity;
|
|
5
|
+
exports.buildConsistentReviewSummary = buildConsistentReviewSummary;
|
|
3
6
|
exports.parseReviewResponse = parseReviewResponse;
|
|
4
7
|
exports.validateReviewResult = validateReviewResult;
|
|
5
8
|
exports.validateReviewContentLanguage = validateReviewContentLanguage;
|
|
6
9
|
exports.shouldBlockReviewResult = shouldBlockReviewResult;
|
|
7
10
|
const protocol_1 = require("../commitMessage/protocol");
|
|
8
11
|
const validSeverities = new Set(["P0", "P1", "P2", "P3", "OTHER"]);
|
|
12
|
+
const reviewSeverities = ["P0", "P1", "P2", "P3", "OTHER"];
|
|
13
|
+
const chunkPrefixPattern = /^\s*\[\s*chunk\s*\d+\s*\]\s*/i;
|
|
14
|
+
function stripReviewChunkPrefix(value) {
|
|
15
|
+
return value.replace(chunkPrefixPattern, "").trim();
|
|
16
|
+
}
|
|
17
|
+
function countReviewDetailsBySeverity(details) {
|
|
18
|
+
const counts = {
|
|
19
|
+
P0: 0,
|
|
20
|
+
P1: 0,
|
|
21
|
+
P2: 0,
|
|
22
|
+
P3: 0,
|
|
23
|
+
OTHER: 0
|
|
24
|
+
};
|
|
25
|
+
for (const detail of details) {
|
|
26
|
+
counts[detail.severity] += 1;
|
|
27
|
+
}
|
|
28
|
+
return counts;
|
|
29
|
+
}
|
|
30
|
+
function buildConsistentReviewSummary(input) {
|
|
31
|
+
const counts = countReviewDetailsBySeverity(input.details);
|
|
32
|
+
const severityParts = reviewSeverities
|
|
33
|
+
.filter((severity) => counts[severity] > 0)
|
|
34
|
+
.map((severity) => `${severity}: ${counts[severity]}`);
|
|
35
|
+
if (severityParts.length === 0) {
|
|
36
|
+
return input.decision === "block"
|
|
37
|
+
? "Review blocked, but no actionable issues were returned."
|
|
38
|
+
: "No review issues found.";
|
|
39
|
+
}
|
|
40
|
+
return `${input.decision === "block" ? "Blocking review issues found" : "Review issues found"} (${severityParts.join(", ")}).`;
|
|
41
|
+
}
|
|
9
42
|
function parseReviewResponse(raw, provider) {
|
|
10
43
|
const trimmed = raw.trim();
|
|
11
44
|
const jsonText = extractJsonPayload(trimmed);
|
|
@@ -52,7 +85,7 @@ function validateReviewContentLanguage(result, language) {
|
|
|
52
85
|
allowChineseInBackticks: true
|
|
53
86
|
});
|
|
54
87
|
for (const [index, detail] of result.details.entries()) {
|
|
55
|
-
issues.push(...(0, protocol_1.validateLanguageText)(detail.message, language, `details[${index}].message`, {
|
|
88
|
+
issues.push(...(0, protocol_1.validateLanguageText)(stripReviewChunkPrefix(detail.message), language, `details[${index}].message`, {
|
|
56
89
|
allowChineseInBackticks: true
|
|
57
90
|
}));
|
|
58
91
|
}
|
|
@@ -87,6 +120,10 @@ function parseReviewDetails(value) {
|
|
|
87
120
|
if (typeof messageRaw !== "string" || !messageRaw.trim()) {
|
|
88
121
|
continue;
|
|
89
122
|
}
|
|
123
|
+
const message = stripReviewChunkPrefix(messageRaw);
|
|
124
|
+
if (!message) {
|
|
125
|
+
continue;
|
|
126
|
+
}
|
|
90
127
|
const severity = typeof severityRaw === "string" && validSeverities.has(severityRaw)
|
|
91
128
|
? severityRaw
|
|
92
129
|
: "OTHER";
|
|
@@ -96,7 +133,7 @@ function parseReviewDetails(value) {
|
|
|
96
133
|
: undefined;
|
|
97
134
|
details.push({
|
|
98
135
|
severity,
|
|
99
|
-
message
|
|
136
|
+
message,
|
|
100
137
|
...(filePath ? { filePath } : {}),
|
|
101
138
|
...(lineNumber !== undefined ? { lineNumber } : {})
|
|
102
139
|
});
|
package/out/review/parser.js.map
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"parser.js","sourceRoot":"","sources":["../../src/review/parser.ts"],"names":[],"mappings":";;
|
|
1
|
+
{"version":3,"file":"parser.js","sourceRoot":"","sources":["../../src/review/parser.ts"],"names":[],"mappings":";;AAeA,wDAEC;AAED,oEAcC;AAED,oEAkBC;AAED,kDA8BC;AAED,oDAgBC;AAED,sEAiBC;AAED,0DASC;AApID,wDAAiE;AAUjE,MAAM,eAAe,GAAG,IAAI,GAAG,CAAiB,CAAC,IAAI,EAAE,IAAI,EAAE,IAAI,EAAE,IAAI,EAAE,OAAO,CAAC,CAAC,CAAC;AACnF,MAAM,gBAAgB,GAAG,CAAC,IAAI,EAAE,IAAI,EAAE,IAAI,EAAE,IAAI,EAAE,OAAO,CAAU,CAAC;AACpE,MAAM,kBAAkB,GAAG,+BAA+B,CAAC;AAE3D,SAAgB,sBAAsB,CAAC,KAAa;IAClD,OAAO,KAAK,CAAC,OAAO,CAAC,kBAAkB,EAAE,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC;AACtD,CAAC;AAED,SAAgB,4BAA4B,CAAC,OAAuB;IAClE,MAAM,MAAM,GAAmC;QAC7C,EAAE,EAAE,CAAC;QACL,EAAE,EAAE,CAAC;QACL,EAAE,EAAE,CAAC;QACL,EAAE,EAAE,CAAC;QACL,KAAK,EAAE,CAAC;KACT,CAAC;IAEF,KAAK,MAAM,MAAM,IAAI,OAAO,EAAE,CAAC;QAC7B,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;IAC/B,CAAC;IAED,OAAO,MAAM,CAAC;AAChB,CAAC;AAED,SAAgB,4BAA4B,CAAC,KAG5C;IACC,MAAM,MAAM,GAAG,4BAA4B,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;IAC3D,MAAM,aAAa,GAAG,gBAAgB;SACnC,MAAM,CAAC,CAAC,QAAQ,EAAE,EAAE,CAAC,MAAM,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC;SAC1C,GAAG,CAAC,CAAC,QAAQ,EAAE,EAAE,CAAC,GAAG,QAAQ,KAAK,MAAM,CAAC,QAAQ,CAAC,EAAE,CAAC,CAAC;IAEzD,IAAI,aAAa,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC/B,OAAO,KAAK,CAAC,QAAQ,KAAK,OAAO;YAC/B,CAAC,CAAC,yDAAyD;YAC3D,CAAC,CAAC,yBAAyB,CAAC;IAChC,CAAC;IAED,OAAO,GACL,KAAK,CAAC,QAAQ,KAAK,OAAO,CAAC,CAAC,CAAC,8BAA8B,CAAC,CAAC,CAAC,qBAChE,KAAK,aAAa,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC;AACpC,CAAC;AAED,SAAgB,mBAAmB,CAAC,GAAW,EAAE,QAAgB;IAC/D,MAAM,OAAO,GAAG,GAAG,CAAC,IAAI,EAAE,CAAC;IAC3B,MAAM,QAAQ,GAAG,kBAAkB,CAAC,OAAO,CAAC,CAAC;IAC7C,IAAI,OAA4B,CAAC;IAEjC,IAAI,CAAC;QACH,OAAO,GAAG,IAAI,CAAC,KAAK,CAAC,QAAQ,CAAwB,CAAC;IACxD,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,MAAM,OAAO,GAAG,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;QACvE,MAAM,IAAI,KAAK,CAAC,gCAAgC,OAAO,EAAE,CAAC,CAAC;IAC7D,CAAC;IAED,MAAM,KAAK,GAAG,gBAAgB,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC;IAC9C,MAAM,QAAQ,GAAG,OAAO,CAAC,QAAQ,KAAK,OAAO,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC;IACjE,MAAM,OAAO,GACX,OAAO,OAAO,CAAC,OAAO,KAAK,QAAQ,IAAI,OAAO,CAAC,OAAO,CAAC,IAAI,EAAE;QAC3D,CAAC,CAAC,OAAO,CAAC,OAAO,CAAC,IAAI,EAAE;QACxB,CAAC,CAAC,QAAQ,KAAK,OAAO;YACpB,CAAC,CAAC,iCAAiC;YACnC,CAAC,CAAC,gCAAgC,CAAC;IACzC,MAAM,OAAO,GAAG,kBAAkB,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC;IAEpD,OAAO;QACL,QAAQ;QACR,KAAK;QACL,OAAO;QACP,QAAQ;QACR,OAAO;QACP,GAAG,EAAE,OAAO;KACb,CAAC;AACJ,CAAC;AAED,SAAgB,oBAAoB,CAAC,MAA6B,EAAE,SAAiB;IACnF,MAAM,MAAM,GAAa,EAAE,CAAC;IAE5B,IAAI,CAAC,MAAM,CAAC,OAAO,CAAC,IAAI,EAAE,EAAE,CAAC;QAC3B,MAAM,CAAC,IAAI,CAAC,2BAA2B,CAAC,CAAC;IAC3C,CAAC;IAED,IAAI,MAAM,CAAC,QAAQ,KAAK,OAAO,IAAI,MAAM,CAAC,OAAO,CAAC,MAAM,KAAK,CAAC,IAAI,MAAM,CAAC,KAAK,KAAK,IAAI,EAAE,CAAC;QACxF,MAAM,CAAC,IAAI,CAAC,iEAAiE,CAAC,CAAC;IACjF,CAAC;IAED,IAAI,MAAM,CAAC,KAAK,KAAK,IAAI,IAAI,CAAC,MAAM,CAAC,KAAK,GAAG,CAAC,IAAI,MAAM,CAAC,KAAK,GAAG,EAAE,CAAC,EAAE,CAAC;QACrE,MAAM,CAAC,IAAI,CAAC,gCAAgC,CAAC,CAAC;IAChD,CAAC;IAED,OAAO,MAAM,CAAC;AAChB,CAAC;AAED,SAAgB,6BAA6B,CAC3C,MAA0D,EAC1D,QAAwB;IAExB,MAAM,MAAM,GAAG,IAAA,+BAAoB,EAAC,MAAM,CAAC,OAAO,EAAE,QAAQ,EAAE,SAAS,EAAE;QACvE,uBAAuB,EAAE,IAAI;KAC9B,CAAC,CAAC;IAEH,KAAK,MAAM,CAAC,KAAK,EAAE,MAAM,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,OAAO,EAAE,EAAE,CAAC;QACvD,MAAM,CAAC,IAAI,CACT,GAAG,IAAA,+BAAoB,EAAC,sBAAsB,CAAC,MAAM,CAAC,OAAO,CAAC,EAAE,QAAQ,EAAE,WAAW,KAAK,WAAW,EAAE;YACrG,uBAAuB,EAAE,IAAI;SAC9B,CAAC,CACH,CAAC;IACJ,CAAC;IAED,OAAO,MAAM,CAAC;AAChB,CAAC;AAED,SAAgB,uBAAuB,CACrC,MAAyD,EACzD,SAAiB;IAEjB,IAAI,MAAM,CAAC,KAAK,KAAK,IAAI,EAAE,CAAC;QAC1B,OAAO,MAAM,CAAC,QAAQ,KAAK,OAAO,CAAC;IACrC,CAAC;IAED,OAAO,MAAM,CAAC,KAAK,IAAI,SAAS,CAAC;AACnC,CAAC;AAED,SAAS,gBAAgB,CAAC,KAAc;IACtC,IAAI,KAAK,KAAK,IAAI,IAAI,KAAK,KAAK,KAAK,EAAE,CAAC;QACtC,OAAO,IAAI,CAAC;IACd,CAAC;IAED,IAAI,OAAO,KAAK,KAAK,QAAQ,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,EAAE,CAAC;QACzD,MAAM,IAAI,KAAK,CAAC,4DAA4D,CAAC,CAAC;IAChF,CAAC;IAED,OAAO,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,IAAI,CAAC,GAAG,CAAC,EAAE,EAAE,KAAK,CAAC,CAAC,CAAC;AAC1C,CAAC;AAED,SAAS,kBAAkB,CAAC,KAAc;IACxC,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,EAAE,CAAC;QAC1B,OAAO,EAAE,CAAC;IACZ,CAAC;IAED,MAAM,OAAO,GAAmB,EAAE,CAAC;IACnC,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,IAAI,CAAC,IAAI,IAAI,OAAO,IAAI,KAAK,QAAQ,IAAI,KAAK,CAAC,OAAO,CAAC,IAAI,CAAC,EAAE,CAAC;YAC7D,SAAS;QACX,CAAC;QAED,MAAM,WAAW,GAAG,UAAU,IAAI,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC,CAAC,SAAS,CAAC;QACnE,MAAM,UAAU,GAAG,SAAS,IAAI,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,CAAC,SAAS,CAAC;QAChE,IAAI,OAAO,UAAU,KAAK,QAAQ,IAAI,CAAC,UAAU,CAAC,IAAI,EAAE,EAAE,CAAC;YACzD,SAAS;QACX,CAAC;QAED,MAAM,OAAO,GAAG,sBAAsB,CAAC,UAAU,CAAC,CAAC;QACnD,IAAI,CAAC,OAAO,EAAE,CAAC;YACb,SAAS;QACX,CAAC;QAED,MAAM,QAAQ,GACZ,OAAO,WAAW,KAAK,QAAQ,IAAI,eAAe,CAAC,GAAG,CAAC,WAA6B,CAAC;YACnF,CAAC,CAAE,WAA8B;YACjC,CAAC,CAAC,OAAO,CAAC;QACd,MAAM,QAAQ,GAAG,UAAU,IAAI,IAAI,IAAI,OAAO,IAAI,CAAC,QAAQ,KAAK,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,IAAI,EAAE,IAAI,SAAS,CAAC,CAAC,CAAC,SAAS,CAAC;QACzH,MAAM,UAAU,GACd,YAAY,IAAI,IAAI,IAAI,OAAO,IAAI,CAAC,UAAU,KAAK,QAAQ,IAAI,MAAM,CAAC,SAAS,CAAC,IAAI,CAAC,UAAU,CAAC;YAC9F,CAAC,CAAC,IAAI,CAAC,UAAU;YACjB,CAAC,CAAC,SAAS,CAAC;QAEhB,OAAO,CAAC,IAAI,CAAC;YACX,QAAQ;YACR,OAAO;YACP,GAAG,CAAC,QAAQ,CAAC,CAAC,CAAC,EAAE,QAAQ,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;YACjC,GAAG,CAAC,UAAU,KAAK,SAAS,CAAC,CAAC,CAAC,EAAE,UAAU,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;SACpD,CAAC,CAAC;IACL,CAAC;IAED,OAAO,OAAO,CAAC;AACjB,CAAC;AAED,SAAS,kBAAkB,CAAC,GAAW;IACrC,MAAM,WAAW,GAAG,GAAG,CAAC,KAAK,CAAC,+BAA+B,CAAC,CAAC;IAC/D,IAAI,WAAW,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QACrB,OAAO,WAAW,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;IAC/B,CAAC;IAED,MAAM,WAAW,GAAG,GAAG,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC;IACrC,MAAM,SAAS,GAAG,GAAG,CAAC,WAAW,CAAC,GAAG,CAAC,CAAC;IAEvC,IAAI,WAAW,IAAI,CAAC,IAAI,SAAS,GAAG,WAAW,EAAE,CAAC;QAChD,OAAO,GAAG,CAAC,KAAK,CAAC,WAAW,EAAE,SAAS,GAAG,CAAC,CAAC,CAAC;IAC/C,CAAC;IAED,OAAO,GAAG,CAAC;AACb,CAAC"}
|