secondpair 0.1.2 ā 0.1.4
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +26 -3
- package/dist/bitbucket/auth.d.ts +13 -0
- package/dist/bitbucket/auth.js +37 -0
- package/dist/bitbucket/auth.js.map +1 -0
- package/dist/bitbucket/comments.d.ts +4 -14
- package/dist/bitbucket/comments.js +5 -36
- package/dist/bitbucket/comments.js.map +1 -1
- package/dist/cli.js +17 -29
- package/dist/cli.js.map +1 -1
- package/dist/config.d.ts +10 -0
- package/dist/config.js +66 -4
- package/dist/config.js.map +1 -1
- package/dist/finding-id.d.ts +1 -1
- package/dist/finding-id.js +4 -4
- package/dist/finding-id.js.map +1 -1
- package/dist/github/auth.d.ts +4 -0
- package/dist/github/auth.js +16 -0
- package/dist/github/auth.js.map +1 -0
- package/dist/github/comments.d.ts +4 -2
- package/dist/github/comments.js +8 -1
- package/dist/github/comments.js.map +1 -1
- package/dist/gitlab/auth.d.ts +11 -0
- package/dist/gitlab/auth.js +22 -0
- package/dist/gitlab/auth.js.map +1 -0
- package/dist/gitlab/comments.d.ts +1 -9
- package/dist/gitlab/comments.js +1 -20
- package/dist/gitlab/comments.js.map +1 -1
- package/dist/host.d.ts +3 -0
- package/dist/host.js +15 -0
- package/dist/host.js.map +1 -0
- package/dist/llm/prompt.d.ts +6 -0
- package/dist/llm/prompt.js +42 -1
- package/dist/llm/prompt.js.map +1 -1
- package/dist/report/cli.d.ts +8 -3
- package/dist/report/cli.js +11 -1
- package/dist/report/cli.js.map +1 -1
- package/dist/report/json.d.ts +5 -0
- package/dist/report/json.js +2 -0
- package/dist/report/json.js.map +1 -1
- package/dist/retry.d.ts +1 -0
- package/dist/retry.js +16 -0
- package/dist/retry.js.map +1 -0
- package/dist/review.d.ts +16 -0
- package/dist/review.js +80 -19
- package/dist/review.js.map +1 -1
- package/dist/specialized-review.d.ts +29 -0
- package/dist/specialized-review.js +50 -0
- package/dist/specialized-review.js.map +1 -0
- package/dist/types.d.ts +16 -1
- package/package.json +3 -3
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"auth.js","sourceRoot":"","sources":["../../src/github/auth.ts"],"names":[],"mappings":"AAAA,mDAAmD;AACnD,MAAM,UAAU,cAAc;IAC5B,MAAM,KAAK,GAAG,OAAO,CAAC,GAAG,CAAC,YAAY,CAAC;IACvC,IAAI,CAAC,KAAK,EAAE,CAAC;QACX,MAAM,IAAI,KAAK,CAAC,oDAAoD,CAAC,CAAC;IACxE,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED,wFAAwF;AACxF,MAAM,UAAU,iBAAiB,CAAC,QAA4B;IAC5D,MAAM,IAAI,GAAG,QAAQ,IAAI,OAAO,CAAC,GAAG,CAAC,iBAAiB,CAAC;IACvD,IAAI,CAAC,IAAI;QAAE,MAAM,IAAI,KAAK,CAAC,kEAAkE,CAAC,CAAC;IAC/F,OAAO,IAAI,CAAC;AACd,CAAC"}
|
|
@@ -1,10 +1,12 @@
|
|
|
1
1
|
import type { Octokit } from "@octokit/rest";
|
|
2
2
|
import type { PrRef } from "../diff/github.js";
|
|
3
3
|
import type { Finding, ReviewResult, Severity } from "../types.js";
|
|
4
|
-
export declare const AGENT_MARKER = "
|
|
4
|
+
export declare const AGENT_MARKER = "[secondpair]: # (agent marker)";
|
|
5
5
|
export declare const SEVERITY_EMOJI: Record<Severity, string>;
|
|
6
6
|
export declare function formatCommentBody(f: Finding): string;
|
|
7
|
-
export declare function formatReviewBody(result: ReviewResult
|
|
7
|
+
export declare function formatReviewBody(result: ReviewResult & {
|
|
8
|
+
highLevelReview?: boolean;
|
|
9
|
+
}, failed: boolean): string;
|
|
8
10
|
export interface PostReviewOptions {
|
|
9
11
|
octokit: Octokit;
|
|
10
12
|
pr: PrRef;
|
package/dist/github/comments.js
CHANGED
|
@@ -1,6 +1,9 @@
|
|
|
1
1
|
import { collectIdsFromBodies, embedFindingId, parseFindingId } from "../finding-id.js";
|
|
2
2
|
import { collectWontFixIds } from "../suppress-signals.js";
|
|
3
|
-
|
|
3
|
+
// Reference-link definition, not an HTML comment: some renderers (Bitbucket)
|
|
4
|
+
// HTML-escape raw comments instead of hiding them, leaving `<!-- -->` visible
|
|
5
|
+
// in the rendered comment. `[label]: # (...)` is invisible everywhere.
|
|
6
|
+
export const AGENT_MARKER = "[secondpair]: # (agent marker)";
|
|
4
7
|
export const SEVERITY_EMOJI = {
|
|
5
8
|
critical: "š„",
|
|
6
9
|
high: "š“",
|
|
@@ -32,6 +35,9 @@ export function formatReviewBody(result, failed) {
|
|
|
32
35
|
: Object.entries(counts)
|
|
33
36
|
.map(([sev, n]) => `${SEVERITY_EMOJI[sev]} ${n} ${sev}`)
|
|
34
37
|
.join(" Ā· ");
|
|
38
|
+
const highLevelLine = result.highLevelReview
|
|
39
|
+
? "\nā ļø **Large diff** ā high-level review only (critical/high severity). Consider splitting this PR."
|
|
40
|
+
: "";
|
|
35
41
|
const recon = result.reconciliation;
|
|
36
42
|
const reconLine = recon
|
|
37
43
|
? `\n_Lifecycle:_ ${recon.new.length} new Ā· ${recon.persistent.length} persistent Ā· ${recon.resolved.length} resolved Ā· ${recon.suppressed.length} suppressed`
|
|
@@ -42,6 +48,7 @@ export function formatReviewBody(result, failed) {
|
|
|
42
48
|
result.summary.trim(),
|
|
43
49
|
"",
|
|
44
50
|
countLine,
|
|
51
|
+
highLevelLine,
|
|
45
52
|
reconLine,
|
|
46
53
|
failed ? "\nā **Check failed**: findings at or above the configured severity threshold." : "",
|
|
47
54
|
AGENT_MARKER,
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"comments.js","sourceRoot":"","sources":["../../src/github/comments.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,oBAAoB,EAAE,cAAc,EAAE,cAAc,EAAE,MAAM,kBAAkB,CAAC;AACxF,OAAO,EAAE,iBAAiB,EAAE,MAAM,wBAAwB,CAAC;AAG3D,MAAM,CAAC,MAAM,YAAY,GAAG,
|
|
1
|
+
{"version":3,"file":"comments.js","sourceRoot":"","sources":["../../src/github/comments.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,oBAAoB,EAAE,cAAc,EAAE,cAAc,EAAE,MAAM,kBAAkB,CAAC;AACxF,OAAO,EAAE,iBAAiB,EAAE,MAAM,wBAAwB,CAAC;AAG3D,6EAA6E;AAC7E,8EAA8E;AAC9E,uEAAuE;AACvE,MAAM,CAAC,MAAM,YAAY,GAAG,gCAAgC,CAAC;AAE7D,MAAM,CAAC,MAAM,cAAc,GAA6B;IACtD,QAAQ,EAAE,IAAI;IACd,IAAI,EAAE,IAAI;IACV,MAAM,EAAE,IAAI;IACZ,GAAG,EAAE,IAAI;IACT,IAAI,EAAE,GAAG;CACV,CAAC;AAEF,MAAM,UAAU,iBAAiB,CAAC,CAAU;IAC1C,MAAM,KAAK,GAAG;QACZ,GAAG,cAAc,CAAC,CAAC,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,QAAQ,CAAC,WAAW,EAAE,UAAU,CAAC,CAAC,QAAQ,mBAAmB,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,UAAU,GAAG,GAAG,CAAC,GAAG;QACnI,EAAE;QACF,KAAK,CAAC,CAAC,KAAK,IAAI;QAChB,EAAE;QACF,CAAC,CAAC,IAAI,CAAC,IAAI,EAAE;KACd,CAAC;IACF,IAAI,CAAC,CAAC,UAAU,EAAE,CAAC;QACjB,KAAK,CAAC,IAAI,CAAC,EAAE,EAAE,eAAe,EAAE,CAAC,CAAC,UAAU,CAAC,OAAO,CAAC,KAAK,EAAE,EAAE,CAAC,EAAE,KAAK,CAAC,CAAC;IAC1E,CAAC;IACD,KAAK,CAAC,IAAI,CAAC,EAAE,EAAE,YAAY,CAAC,CAAC;IAC7B,IAAI,IAAI,GAAG,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC5B,IAAI,CAAC,CAAC,EAAE;QAAE,IAAI,GAAG,cAAc,CAAC,IAAI,EAAE,CAAC,CAAC,EAAE,CAAC,CAAC;IAC5C,OAAO,IAAI,CAAC;AACd,CAAC;AAED,MAAM,UAAU,gBAAgB,CAC9B,MAAoD,EACpD,MAAe;IAEf,MAAM,MAAM,GAAG,eAAe,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC;IAChD,MAAM,SAAS,GACb,MAAM,CAAC,QAAQ,CAAC,MAAM,KAAK,CAAC;QAC1B,CAAC,CAAC,cAAc;QAChB,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC;aACnB,GAAG,CAAC,CAAC,CAAC,GAAG,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,GAAG,cAAc,CAAC,GAAe,CAAC,IAAI,CAAC,IAAI,GAAG,EAAE,CAAC;aACnE,IAAI,CAAC,KAAK,CAAC,CAAC;IACrB,MAAM,aAAa,GAAG,MAAM,CAAC,eAAe;QAC1C,CAAC,CAAC,oGAAoG;QACtG,CAAC,CAAC,EAAE,CAAC;IACP,MAAM,KAAK,GAAG,MAAM,CAAC,cAAc,CAAC;IACpC,MAAM,SAAS,GAAG,KAAK;QACrB,CAAC,CAAC,kBAAkB,KAAK,CAAC,GAAG,CAAC,MAAM,UAAU,KAAK,CAAC,UAAU,CAAC,MAAM,iBAAiB,KAAK,CAAC,QAAQ,CAAC,MAAM,eAAe,KAAK,CAAC,UAAU,CAAC,MAAM,aAAa;QAC9J,CAAC,CAAC,EAAE,CAAC;IACP,OAAO;QACL,oBAAoB;QACpB,EAAE;QACF,MAAM,CAAC,OAAO,CAAC,IAAI,EAAE;QACrB,EAAE;QACF,SAAS;QACT,aAAa;QACb,SAAS;QACT,MAAM,CAAC,CAAC,CAAC,+EAA+E,CAAC,CAAC,CAAC,EAAE;QAC7F,YAAY;KACb,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AACf,CAAC;AAED,SAAS,eAAe,CAAC,QAAmB;IAC1C,MAAM,MAAM,GAAsC,EAAE,CAAC;IACrD,KAAK,MAAM,CAAC,IAAI,QAAQ;QAAE,MAAM,CAAC,CAAC,CAAC,QAAQ,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,CAAC;IAC7E,OAAO,MAAM,CAAC;AAChB,CAAC;AAWD,kEAAkE;AAClE,MAAM,CAAC,KAAK,UAAU,oBAAoB,CAAC,OAAgB,EAAE,EAAS;IACpE,MAAM,QAAQ,GAAG,MAAM,OAAO,CAAC,QAAQ,CAAC,OAAO,CAAC,KAAK,CAAC,kBAAkB,EAAE;QACxE,GAAG,EAAE;QACL,QAAQ,EAAE,GAAG;KACd,CAAC,CAAC;IACH,OAAO,oBAAoB,CACzB,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,YAAY,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CACzE,CAAC;AACJ,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,qBAAqB,CACzC,OAAgB,EAChB,EAAS;IAET,MAAM,QAAQ,GAAG,MAAM,OAAO,CAAC,QAAQ,CAAC,OAAO,CAAC,KAAK,CAAC,kBAAkB,EAAE;QACxE,GAAG,EAAE;QACL,QAAQ,EAAE,GAAG;KACd,CAAC,CAAC;IACH,MAAM,IAAI,GAAG,IAAI,GAAG,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC,OAAO,CAAC,EAAE,EAAE,OAAO,CAAC,CAAC,CAAC,CAAC;IACvE,OAAO,iBAAiB,CACtB,QAAQ,CAAC,GAAG,CAAC,CAAC,OAAO,EAAE,EAAE;QACvB,MAAM,MAAM,GAAG,OAAO,CAAC,cAAc;YACnC,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,cAAc,CAAC;YAClC,CAAC,CAAC,SAAS,CAAC;QACd,OAAO;YACL,IAAI,EAAE,OAAO,CAAC,IAAI;YAClB,OAAO,EAAE,OAAO,CAAC,IAAI,CAAC,QAAQ,CAAC,YAAY,CAAC;YAC5C,aAAa,EACX,MAAM,EAAE,IAAI,CAAC,QAAQ,CAAC,YAAY,CAAC,KAAK,IAAI;gBAC1C,CAAC,CAAC,cAAc,CAAC,MAAM,CAAC,IAAI,CAAC;gBAC7B,CAAC,CAAC,IAAI;SACX,CAAC;IACJ,CAAC,CAAC,CACH,CAAC;AACJ,CAAC;AAQD,uFAAuF;AACvF,MAAM,CAAC,KAAK,UAAU,oBAAoB,CACxC,OAAgB,EAChB,EAAS,EACT,GAAqB,EACrB,MAA6B,GAAG,EAAE,GAAE,CAAC;IAErC,MAAM,MAAM,GAAG,IAAI,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,GAAG,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,EAAE,CAAC,WAAW,EAAE,CAAC,CAAC,CAAC;IAClE,IAAI,MAAM,CAAC,IAAI,KAAK,CAAC;QAAE,OAAO,CAAC,CAAC;IAEhC,IAAI,OAAuB,CAAC;IAC5B,IAAI,CAAC;QACH,OAAO,GAAG,EAAE,CAAC;QACb,IAAI,MAAM,GAAkB,IAAI,CAAC;QACjC,SAAS,CAAC;YACR,MAAM,IAAI,GAAG,CAAC,MAAM,OAAO,CAAC,OAAO,CACjC;;;;;;;;;UASE,EACF,EAAE,KAAK,EAAE,EAAE,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE,CAAC,IAAI,EAAE,MAAM,EAAE,EAAE,CAAC,WAAW,EAAE,MAAM,EAAE,CACnE,CASA,CAAC;YACF,MAAM,IAAI,GAAG,IAAI,CAAC,UAAU,EAAE,WAAW,EAAE,aAAa,CAAC;YACzD,OAAO,CAAC,IAAI,CAAC,GAAG,CAAC,IAAI,EAAE,KAAK,IAAI,EAAE,CAAC,CAAC,CAAC;YACrC,IAAI,CAAC,IAAI,EAAE,QAAQ,EAAE,WAAW,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,SAAS;gBAAE,MAAM;YACpE,MAAM,GAAG,IAAI,CAAC,QAAQ,CAAC,SAAS,CAAC;QACnC,CAAC;IACH,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,GAAG,CAAC,+CAAgD,GAAa,CAAC,OAAO,EAAE,CAAC,CAAC;QAC7E,OAAO,CAAC,CAAC;IACX,CAAC;IAED,IAAI,QAAQ,GAAG,CAAC,CAAC;IACjB,KAAK,MAAM,MAAM,IAAI,OAAO,EAAE,CAAC;QAC7B,IACE,MAAM,CAAC,UAAU;YACjB,CAAC,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,OAAO,EAAE,EAAE;gBACtC,MAAM,EAAE,GAAG,cAAc,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;gBACxC,OAAO,EAAE,IAAI,IAAI,IAAI,MAAM,CAAC,GAAG,CAAC,EAAE,CAAC,WAAW,EAAE,CAAC,CAAC;YACpD,CAAC,CAAC,EACF,CAAC;YACD,SAAS;QACX,CAAC;QACD,IAAI,CAAC;YACH,MAAM,OAAO,CAAC,OAAO,CACnB;;UAEE,EACF,EAAE,EAAE,EAAE,MAAM,CAAC,EAAE,EAAE,CAClB,CAAC;YACF,QAAQ,EAAE,CAAC;QACb,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,GAAG,CAAC,gDAAgD,MAAM,CAAC,EAAE,KAAM,GAAa,CAAC,OAAO,EAAE,CAAC,CAAC;QAC9F,CAAC;IACH,CAAC;IACD,OAAO,QAAQ,CAAC;AAClB,CAAC;AAED;;;GAGG;AACH,MAAM,CAAC,KAAK,UAAU,UAAU,CAAC,IAAuB;IACtD,MAAM,EAAE,OAAO,EAAE,EAAE,EAAE,MAAM,EAAE,GAAG,GAAG,GAAG,EAAE,GAAE,CAAC,EAAE,GAAG,IAAI,CAAC;IAErD,MAAM,MAAM,GAAG,MAAM,CAAC,cAAc,IAAI,MAAM,CAAC,QAAQ,CAAC;IACxD,MAAM,WAAW,GAAG,MAAM,oBAAoB,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC;IAE5D,MAAM,QAAQ,GAAG,EAAE,CAAC;IACpB,IAAI,OAAO,GAAG,CAAC,CAAC;IAChB,KAAK,MAAM,CAAC,IAAI,MAAM,EAAE,CAAC;QACvB,IAAI,CAAC,CAAC,EAAE,IAAI,WAAW,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC;YAClC,OAAO,EAAE,CAAC;YACV,SAAS;QACX,CAAC;QACD,QAAQ,CAAC,IAAI,CAAC;YACZ,IAAI,EAAE,CAAC,CAAC,IAAI;YACZ,IAAI,EAAE,CAAC,CAAC,QAAQ;YAChB,GAAG,CAAC,CAAC,CAAC,QAAQ,GAAG,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,EAAE,UAAU,EAAE,CAAC,CAAC,UAAU,EAAE,UAAU,EAAE,OAAgB,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;YAChG,IAAI,EAAE,OAAgB;YACtB,IAAI,EAAE,iBAAiB,CAAC,CAAC,CAAC;SAC3B,CAAC,CAAC;IACL,CAAC;IACD,IAAI,OAAO;QAAE,GAAG,CAAC,YAAY,OAAO,+CAA+C,CAAC,CAAC;IAErF,MAAM,IAAI,GAAG,gBAAgB,CAAC,MAAM,EAAE,IAAI,CAAC,MAAM,CAAC,CAAC;IAEnD,IAAI,CAAC;QACH,MAAM,OAAO,CAAC,KAAK,CAAC,YAAY,CAAC;YAC/B,GAAG,EAAE;YACL,SAAS,EAAE,IAAI,CAAC,OAAO;YACvB,KAAK,EAAE,SAAS;YAChB,IAAI;YACJ,QAAQ;SACT,CAAC,CAAC;QACH,GAAG,CAAC,sBAAsB,QAAQ,CAAC,MAAM,qBAAqB,CAAC,CAAC;IAClE,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,GAAG,CAAC,yBAA0B,GAAa,CAAC,OAAO,iCAAiC,CAAC,CAAC;QACtF,MAAM,YAAY,GAAG;YACnB,IAAI;YACJ,EAAE;YACF,wCAAwC;YACxC,GAAG,MAAM,CAAC,GAAG,CACX,CAAC,CAAC,EAAE,EAAE,CACJ,OAAO,CAAC,CAAC,QAAQ,QAAQ,CAAC,CAAC,IAAI,IAAI,CAAC,CAAC,UAAU,IAAI,CAAC,CAAC,QAAQ,QAAQ,CAAC,CAAC,KAAK,GAAG,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,EAAE,KAAK,CAAC,CAAC,CAAC,EAAE,SAAS,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,EAAE,CACxJ;SACF,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACb,MAAM,OAAO,CAAC,KAAK,CAAC,YAAY,CAAC;YAC/B,GAAG,EAAE;YACL,SAAS,EAAE,IAAI,CAAC,OAAO;YACvB,KAAK,EAAE,SAAS;YAChB,IAAI,EAAE,YAAY;SACnB,CAAC,CAAC;IACL,CAAC;IAED,MAAM,WAAW,GAAG,MAAM,CAAC,cAAc,EAAE,QAAQ,IAAI,EAAE,CAAC;IAC1D,IAAI,WAAW,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC3B,MAAM,QAAQ,GAAG,MAAM,oBAAoB,CAAC,OAAO,EAAE,EAAE,EAAE,WAAW,EAAE,GAAG,CAAC,CAAC;QAC3E,IAAI,QAAQ;YAAE,GAAG,CAAC,YAAY,QAAQ,gCAAgC,CAAC,CAAC;IAC1E,CAAC;AACH,CAAC"}
|
|
@@ -0,0 +1,11 @@
|
|
|
1
|
+
export interface GlRef {
|
|
2
|
+
/** GitLab server base URL, e.g. https://gitlab.com */
|
|
3
|
+
serverUrl: string;
|
|
4
|
+
/** URL-encoded project path ("group%2Fproject") or numeric project id. */
|
|
5
|
+
projectId: string;
|
|
6
|
+
mrIid: number;
|
|
7
|
+
}
|
|
8
|
+
/** Resolve a GitLab token from the environment. CI_JOB_TOKEN cannot post MR notes. */
|
|
9
|
+
export declare function resolveGlToken(): string;
|
|
10
|
+
/** Resolve project/MR from flags or GitLab CI env vars. */
|
|
11
|
+
export declare function resolveGlRef(repoSlug: string | undefined, pr: number | undefined): GlRef;
|
|
@@ -0,0 +1,22 @@
|
|
|
1
|
+
/** Resolve a GitLab token from the environment. CI_JOB_TOKEN cannot post MR notes. */
|
|
2
|
+
export function resolveGlToken() {
|
|
3
|
+
const token = process.env.GITLAB_TOKEN || process.env.GL_TOKEN;
|
|
4
|
+
if (!token) {
|
|
5
|
+
throw new Error("GitLab auth required: set GITLAB_TOKEN (personal or project access token with api scope). " +
|
|
6
|
+
"CI_JOB_TOKEN cannot post merge request notes.");
|
|
7
|
+
}
|
|
8
|
+
return token;
|
|
9
|
+
}
|
|
10
|
+
/** Resolve project/MR from flags or GitLab CI env vars. */
|
|
11
|
+
export function resolveGlRef(repoSlug, pr) {
|
|
12
|
+
const env = process.env;
|
|
13
|
+
const serverUrl = (env.CI_SERVER_URL || "https://gitlab.com").replace(/\/$/, "");
|
|
14
|
+
const projectId = repoSlug ? encodeURIComponent(repoSlug) : env.CI_PROJECT_ID;
|
|
15
|
+
const mrIid = pr ?? (env.CI_MERGE_REQUEST_IID ? parseInt(env.CI_MERGE_REQUEST_IID, 10) : undefined);
|
|
16
|
+
if (!projectId || mrIid == null || Number.isNaN(mrIid)) {
|
|
17
|
+
throw new Error("GitLab MR not identified. Pass --repo group/project and --pr <iid>, " +
|
|
18
|
+
"or run in a merge_request_event pipeline (CI_PROJECT_ID / CI_MERGE_REQUEST_IID).");
|
|
19
|
+
}
|
|
20
|
+
return { serverUrl, projectId, mrIid };
|
|
21
|
+
}
|
|
22
|
+
//# sourceMappingURL=auth.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"auth.js","sourceRoot":"","sources":["../../src/gitlab/auth.ts"],"names":[],"mappings":"AAQA,sFAAsF;AACtF,MAAM,UAAU,cAAc;IAC5B,MAAM,KAAK,GAAG,OAAO,CAAC,GAAG,CAAC,YAAY,IAAI,OAAO,CAAC,GAAG,CAAC,QAAQ,CAAC;IAC/D,IAAI,CAAC,KAAK,EAAE,CAAC;QACX,MAAM,IAAI,KAAK,CACb,4FAA4F;YAC1F,+CAA+C,CAClD,CAAC;IACJ,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED,2DAA2D;AAC3D,MAAM,UAAU,YAAY,CAAC,QAA4B,EAAE,EAAsB;IAC/E,MAAM,GAAG,GAAG,OAAO,CAAC,GAAG,CAAC;IACxB,MAAM,SAAS,GAAG,CAAC,GAAG,CAAC,aAAa,IAAI,oBAAoB,CAAC,CAAC,OAAO,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC;IACjF,MAAM,SAAS,GAAG,QAAQ,CAAC,CAAC,CAAC,kBAAkB,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,aAAa,CAAC;IAC9E,MAAM,KAAK,GAAG,EAAE,IAAI,CAAC,GAAG,CAAC,oBAAoB,CAAC,CAAC,CAAC,QAAQ,CAAC,GAAG,CAAC,oBAAoB,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC;IACpG,IAAI,CAAC,SAAS,IAAI,KAAK,IAAI,IAAI,IAAI,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,EAAE,CAAC;QACvD,MAAM,IAAI,KAAK,CACb,sEAAsE;YACpE,kFAAkF,CACrF,CAAC;IACJ,CAAC;IACD,OAAO,EAAE,SAAS,EAAE,SAAS,EAAE,KAAK,EAAE,CAAC;AACzC,CAAC"}
|
|
@@ -1,18 +1,10 @@
|
|
|
1
1
|
import type { ReviewResult } from "../types.js";
|
|
2
|
-
|
|
3
|
-
/** GitLab server base URL, e.g. https://gitlab.com */
|
|
4
|
-
serverUrl: string;
|
|
5
|
-
/** URL-encoded project path ("group%2Fproject") or numeric project id. */
|
|
6
|
-
projectId: string;
|
|
7
|
-
mrIid: number;
|
|
8
|
-
}
|
|
2
|
+
import { type GlRef } from "./auth.js";
|
|
9
3
|
export interface GlDiffRefs {
|
|
10
4
|
base_sha: string;
|
|
11
5
|
head_sha: string;
|
|
12
6
|
start_sha: string;
|
|
13
7
|
}
|
|
14
|
-
/** Resolve project/MR from flags or GitLab CI env vars. */
|
|
15
|
-
export declare function resolveGlRef(repoSlug: string | undefined, pr: number | undefined): GlRef;
|
|
16
8
|
/** Fetch MR changes and reconstruct a unified diff the parser understands. */
|
|
17
9
|
export declare function getGlMrDiff(ref: GlRef): Promise<{
|
|
18
10
|
diffText: string;
|
package/dist/gitlab/comments.js
CHANGED
|
@@ -2,26 +2,7 @@ import { formatBbCommentBody, formatBbSummaryBody } from "../bitbucket/comments.
|
|
|
2
2
|
import { collectIdsFromBodies, parseFindingId } from "../finding-id.js";
|
|
3
3
|
import { AGENT_MARKER, SEVERITY_EMOJI } from "../github/comments.js";
|
|
4
4
|
import { collectWontFixIds } from "../suppress-signals.js";
|
|
5
|
-
|
|
6
|
-
const token = process.env.GITLAB_TOKEN || process.env.GL_TOKEN;
|
|
7
|
-
if (!token) {
|
|
8
|
-
throw new Error("GitLab auth required: set GITLAB_TOKEN (personal or project access token with api scope). " +
|
|
9
|
-
"CI_JOB_TOKEN cannot post merge request notes.");
|
|
10
|
-
}
|
|
11
|
-
return token;
|
|
12
|
-
}
|
|
13
|
-
/** Resolve project/MR from flags or GitLab CI env vars. */
|
|
14
|
-
export function resolveGlRef(repoSlug, pr) {
|
|
15
|
-
const env = process.env;
|
|
16
|
-
const serverUrl = (env.CI_SERVER_URL || "https://gitlab.com").replace(/\/$/, "");
|
|
17
|
-
const projectId = repoSlug ? encodeURIComponent(repoSlug) : env.CI_PROJECT_ID;
|
|
18
|
-
const mrIid = pr ?? (env.CI_MERGE_REQUEST_IID ? parseInt(env.CI_MERGE_REQUEST_IID, 10) : undefined);
|
|
19
|
-
if (!projectId || mrIid == null || Number.isNaN(mrIid)) {
|
|
20
|
-
throw new Error("GitLab MR not identified. Pass --repo group/project and --pr <iid>, " +
|
|
21
|
-
"or run in a merge_request_event pipeline (CI_PROJECT_ID / CI_MERGE_REQUEST_IID).");
|
|
22
|
-
}
|
|
23
|
-
return { serverUrl, projectId, mrIid };
|
|
24
|
-
}
|
|
5
|
+
import { resolveGlToken } from "./auth.js";
|
|
25
6
|
function glApi(ref) {
|
|
26
7
|
return `${ref.serverUrl}/api/v4/projects/${ref.projectId}/merge_requests/${ref.mrIid}`;
|
|
27
8
|
}
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"comments.js","sourceRoot":"","sources":["../../src/gitlab/comments.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,mBAAmB,EAAE,mBAAmB,EAAE,MAAM,0BAA0B,CAAC;AACpF,OAAO,EAAE,oBAAoB,EAAE,cAAc,EAAE,MAAM,kBAAkB,CAAC;AACxE,OAAO,EAAE,YAAY,EAAE,cAAc,EAAE,MAAM,uBAAuB,CAAC;AACrE,OAAO,EAAE,iBAAiB,EAAE,MAAM,wBAAwB,CAAC;
|
|
1
|
+
{"version":3,"file":"comments.js","sourceRoot":"","sources":["../../src/gitlab/comments.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,mBAAmB,EAAE,mBAAmB,EAAE,MAAM,0BAA0B,CAAC;AACpF,OAAO,EAAE,oBAAoB,EAAE,cAAc,EAAE,MAAM,kBAAkB,CAAC;AACxE,OAAO,EAAE,YAAY,EAAE,cAAc,EAAE,MAAM,uBAAuB,CAAC;AACrE,OAAO,EAAE,iBAAiB,EAAE,MAAM,wBAAwB,CAAC;AAE3D,OAAO,EAAE,cAAc,EAAc,MAAM,WAAW,CAAC;AAQvD,SAAS,KAAK,CAAC,GAAU;IACvB,OAAO,GAAG,GAAG,CAAC,SAAS,oBAAoB,GAAG,CAAC,SAAS,mBAAmB,GAAG,CAAC,KAAK,EAAE,CAAC;AACzF,CAAC;AAED,KAAK,UAAU,OAAO,CAAC,GAAW,EAAE,OAAoB,EAAE;IACxD,MAAM,GAAG,GAAG,MAAM,KAAK,CAAC,GAAG,EAAE;QAC3B,GAAG,IAAI;QACP,OAAO,EAAE;YACP,eAAe,EAAE,cAAc,EAAE;YACjC,MAAM,EAAE,kBAAkB;YAC1B,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,cAAc,EAAE,kBAAkB,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;YAC5D,GAAG,CAAC,IAAI,CAAC,OAAO,IAAI,EAAE,CAAC;SACxB;KACF,CAAC,CAAC;IACH,IAAI,CAAC,GAAG,CAAC,EAAE,EAAE,CAAC;QACZ,MAAM,IAAI,GAAG,MAAM,GAAG,CAAC,IAAI,EAAE,CAAC;QAC9B,MAAM,IAAI,KAAK,CACb,cAAc,IAAI,CAAC,MAAM,IAAI,KAAK,IAAI,GAAG,YAAY,GAAG,CAAC,MAAM,MAAM,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,EAAE,CAC1F,CAAC;IACJ,CAAC;IACD,OAAO,GAAG,CAAC;AACb,CAAC;AAED,8EAA8E;AAC9E,MAAM,CAAC,KAAK,UAAU,WAAW,CAC/B,GAAU;IAEV,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,GAAG,KAAK,CAAC,GAAG,CAAC,UAAU,CAAC,CAAC;IACnD,MAAM,IAAI,GAAG,CAAC,MAAM,GAAG,CAAC,IAAI,EAAE,CAS7B,CAAC;IACF,IAAI,CAAC,IAAI,CAAC,SAAS;QAAE,MAAM,IAAI,KAAK,CAAC,wDAAwD,CAAC,CAAC;IAC/F,MAAM,KAAK,GAAG,CAAC,IAAI,CAAC,OAAO,IAAI,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE;QAC3C,MAAM,OAAO,GAAG,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,QAAQ,EAAE,CAAC;QAC7D,MAAM,OAAO,GAAG,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,QAAQ,EAAE,CAAC;QACjE,OAAO,gBAAgB,CAAC,CAAC,QAAQ,MAAM,CAAC,CAAC,QAAQ,SAAS,OAAO,SAAS,OAAO,KAAK,CAAC,CAAC,IAAI,EAAE,CAAC;IACjG,CAAC,CAAC,CAAC;IACH,OAAO,EAAE,QAAQ,EAAE,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,EAAE,QAAQ,EAAE,IAAI,CAAC,SAAS,EAAE,CAAC;AAChE,CAAC;AAWD,MAAM,CAAC,KAAK,UAAU,WAAW,CAAC,GAAU;IAC1C,MAAM,KAAK,GAAa,EAAE,CAAC;IAC3B,IAAI,IAAI,GAAG,CAAC,CAAC;IACb,SAAS,CAAC;QACR,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,GAAG,KAAK,CAAC,GAAG,CAAC,4BAA4B,IAAI,EAAE,CAAC,CAAC;QAC3E,MAAM,KAAK,GAAG,CAAC,MAAM,GAAG,CAAC,IAAI,EAAE,CAAa,CAAC;QAC7C,KAAK,CAAC,IAAI,CAAC,GAAG,KAAK,CAAC,CAAC;QACrB,MAAM,IAAI,GAAG,GAAG,CAAC,OAAO,CAAC,GAAG,CAAC,aAAa,CAAC,CAAC;QAC5C,IAAI,CAAC,IAAI;YAAE,MAAM;QACjB,IAAI,GAAG,QAAQ,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC;IAC5B,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,gBAAgB,CAAC,GAAU;IAC/C,MAAM,KAAK,GAAG,MAAM,WAAW,CAAC,GAAG,CAAC,CAAC;IACrC,OAAO,oBAAoB,CACzB,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,QAAQ,CAAC,YAAY,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,IAAI,EAAE,CAAC,CAC7E,CAAC;AACJ,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,uBAAuB,CAAC,GAAU;IACtD,MAAM,KAAK,GAAG,MAAM,WAAW,CAAC,GAAG,CAAC,CAAC;IACrC,MAAM,IAAI,GAAG,IAAI,GAAG,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC,IAAI,CAAC,EAAE,IAAI,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,EAAE,IAAI,CAAU,CAAC,CAAC,CAAC,CAAC,CAAC;IACnG,MAAM,kBAAkB,GAAG,IAAI,GAAG,EAAkB,CAAC;IACrD,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,MAAM,IAAI,GAAG,IAAI,CAAC,IAAI,IAAI,EAAE,CAAC;QAC7B,MAAM,EAAE,GAAG,IAAI,CAAC,QAAQ,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,cAAc,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;QACrE,IAAI,IAAI,CAAC,aAAa,IAAI,EAAE;YAAE,kBAAkB,CAAC,GAAG,CAAC,IAAI,CAAC,aAAa,EAAE,EAAE,CAAC,CAAC;IAC/E,CAAC;IACD,OAAO,iBAAiB,CACtB,KAAK,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE;QACjB,IAAI,MAAM,GAAG,IAAI,CAAC,GAAG,CAAC,IAAI,CAAC,SAAS,IAAI,IAAI,CAAC,QAAQ,IAAI,CAAC,CAAC,CAAC,CAAC;QAC7D,OAAO,MAAM,EAAE,CAAC;YACd,MAAM,IAAI,GAAG,MAAM,CAAC,IAAI,IAAI,EAAE,CAAC;YAC/B,IAAI,IAAI,CAAC,QAAQ,CAAC,YAAY,CAAC,EAAE,CAAC;gBAChC,OAAO,EAAE,IAAI,EAAE,IAAI,CAAC,IAAI,IAAI,EAAE,EAAE,aAAa,EAAE,cAAc,CAAC,IAAI,CAAC,EAAE,CAAC;YACxE,CAAC;YACD,MAAM,GAAG,IAAI,CAAC,GAAG,CAAC,MAAM,CAAC,SAAS,IAAI,MAAM,CAAC,QAAQ,IAAI,CAAC,CAAC,CAAC,CAAC;QAC/D,CAAC;QACD,MAAM,IAAI,GAAG,IAAI,CAAC,IAAI,IAAI,EAAE,CAAC;QAC7B,OAAO;YACL,IAAI;YACJ,OAAO,EAAE,IAAI,CAAC,QAAQ,CAAC,YAAY,CAAC;YACpC,aAAa,EAAE,IAAI,CAAC,aAAa;gBAC/B,CAAC,CAAC,kBAAkB,CAAC,GAAG,CAAC,IAAI,CAAC,aAAa,CAAC,IAAI,IAAI;gBACpD,CAAC,CAAC,IAAI;SACT,CAAC;IACJ,CAAC,CAAC,CACH,CAAC;AACJ,CAAC;AAWD,oFAAoF;AACpF,MAAM,CAAC,KAAK,UAAU,0BAA0B,CAC9C,GAAU,EACV,GAAqB,EACrB,MAA6B,GAAG,EAAE,GAAE,CAAC;IAErC,MAAM,MAAM,GAAG,IAAI,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,GAAG,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,EAAE,CAAC,WAAW,EAAE,CAAC,CAAC,CAAC;IAClE,IAAI,MAAM,CAAC,IAAI,KAAK,CAAC;QAAE,OAAO,CAAC,CAAC;IAEhC,MAAM,WAAW,GAAmB,EAAE,CAAC;IACvC,IAAI,CAAC;QACH,IAAI,IAAI,GAAG,CAAC,CAAC;QACb,SAAS,CAAC;YACR,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,GAAG,KAAK,CAAC,GAAG,CAAC,kCAAkC,IAAI,EAAE,CAAC,CAAC;YACjF,WAAW,CAAC,IAAI,CAAC,GAAI,CAAC,MAAM,GAAG,CAAC,IAAI,EAAE,CAAoB,CAAC,CAAC;YAC5D,MAAM,IAAI,GAAG,GAAG,CAAC,OAAO,CAAC,GAAG,CAAC,aAAa,CAAC,CAAC;YAC5C,IAAI,CAAC,IAAI;gBAAE,MAAM;YACjB,IAAI,GAAG,QAAQ,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC;QAC5B,CAAC;IACH,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,GAAG,CAAC,4CAA6C,GAAa,CAAC,OAAO,EAAE,CAAC,CAAC;QAC1E,OAAO,CAAC,CAAC;IACX,CAAC;IAED,IAAI,QAAQ,GAAG,CAAC,CAAC;IACjB,KAAK,MAAM,UAAU,IAAI,WAAW,EAAE,CAAC;QACrC,MAAM,OAAO,GAAG,UAAU,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE;YAC7C,MAAM,EAAE,GAAG,cAAc,CAAC,IAAI,CAAC,IAAI,IAAI,EAAE,CAAC,CAAC;YAC3C,OAAO,EAAE,IAAI,IAAI,IAAI,MAAM,CAAC,GAAG,CAAC,EAAE,CAAC,WAAW,EAAE,CAAC,CAAC;QACpD,CAAC,CAAC,CAAC;QACH,MAAM,UAAU,GAAG,UAAU,CAAC,KAAK,CAAC,IAAI,CACtC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,UAAU,KAAK,KAAK,IAAI,IAAI,CAAC,QAAQ,KAAK,IAAI,CAC9D,CAAC;QACF,IAAI,CAAC,OAAO,IAAI,CAAC,UAAU;YAAE,SAAS;QAEtC,IAAI,CAAC;YACH,MAAM,OAAO,CAAC,GAAG,KAAK,CAAC,GAAG,CAAC,gBAAgB,UAAU,CAAC,EAAE,EAAE,EAAE;gBAC1D,MAAM,EAAE,KAAK;gBACb,IAAI,EAAE,IAAI,CAAC,SAAS,CAAC,EAAE,QAAQ,EAAE,IAAI,EAAE,CAAC;aACzC,CAAC,CAAC;YACH,QAAQ,EAAE,CAAC;QACb,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,GAAG,CAAC,6CAA6C,UAAU,CAAC,EAAE,KAAM,GAAa,CAAC,OAAO,EAAE,CAAC,CAAC;QAC/F,CAAC;IACH,CAAC;IACD,OAAO,QAAQ,CAAC;AAClB,CAAC;AAUD;;;GAGG;AACH,MAAM,CAAC,KAAK,UAAU,YAAY,CAAC,IAAyB;IAC1D,MAAM,EAAE,GAAG,EAAE,QAAQ,EAAE,MAAM,EAAE,GAAG,GAAG,GAAG,EAAE,GAAE,CAAC,EAAE,GAAG,IAAI,CAAC;IAEvD,MAAM,KAAK,GAAG,MAAM,WAAW,CAAC,GAAG,CAAC,CAAC;IACrC,MAAM,IAAI,GAAG,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,QAAQ,CAAC,YAAY,CAAC,CAAC,CAAC;IACjE,MAAM,WAAW,GAAG,oBAAoB,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,IAAI,EAAE,CAAC,CAAC,CAAC;IACxE,MAAM,iBAAiB,GAAG,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,IAAI,IAAI,CAAC,CAAC;IAE3D,IAAI,CAAC,iBAAiB,IAAI,MAAM,CAAC,QAAQ,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACvD,MAAM,OAAO,CAAC,GAAG,KAAK,CAAC,GAAG,CAAC,QAAQ,EAAE;YACnC,MAAM,EAAE,MAAM;YACd,IAAI,EAAE,IAAI,CAAC,SAAS,CAAC,EAAE,IAAI,EAAE,mBAAmB,CAAC,MAAM,EAAE,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC;SACzE,CAAC,CAAC;IACL,CAAC;IAED,MAAM,MAAM,GAAG,MAAM,CAAC,cAAc,IAAI,MAAM,CAAC,QAAQ,CAAC;IACxD,IAAI,MAAM,GAAG,CAAC,CAAC;IACf,IAAI,OAAO,GAAG,CAAC,CAAC;IAChB,MAAM,YAAY,GAAc,EAAE,CAAC;IACnC,KAAK,MAAM,CAAC,IAAI,MAAM,EAAE,CAAC;QACvB,IAAI,CAAC,CAAC,EAAE,IAAI,WAAW,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC;YAClC,OAAO,EAAE,CAAC;YACV,SAAS;QACX,CAAC;QACD,IAAI,CAAC;YACH,MAAM,OAAO,CAAC,GAAG,KAAK,CAAC,GAAG,CAAC,cAAc,EAAE;gBACzC,MAAM,EAAE,MAAM;gBACd,IAAI,EAAE,IAAI,CAAC,SAAS,CAAC;oBACnB,IAAI,EAAE,mBAAmB,CAAC,CAAC,CAAC;oBAC5B,QAAQ,EAAE;wBACR,aAAa,EAAE,MAAM;wBACrB,QAAQ,EAAE,QAAQ,CAAC,QAAQ;wBAC3B,SAAS,EAAE,QAAQ,CAAC,SAAS;wBAC7B,QAAQ,EAAE,QAAQ,CAAC,QAAQ;wBAC3B,QAAQ,EAAE,CAAC,CAAC,IAAI;wBAChB,QAAQ,EAAE,CAAC,CAAC,QAAQ;qBACrB;iBACF,CAAC;aACH,CAAC,CAAC;YACH,MAAM,EAAE,CAAC;QACX,CAAC;QAAC,MAAM,CAAC;YACP,YAAY,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACvB,CAAC;IACH,CAAC;IAED,IAAI,YAAY,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC5B,MAAM,IAAI,GAAG;YACX,8CAA8C;YAC9C,GAAG,YAAY,CAAC,GAAG,CACjB,CAAC,CAAC,EAAE,EAAE,CACJ,KAAK,cAAc,CAAC,CAAC,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,QAAQ,QAAQ,CAAC,CAAC,IAAI,IAAI,CAAC,CAAC,UAAU,IAAI,CAAC,CAAC,QAAQ,QAAQ,CAAC,CAAC,KAAK,SAAS,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,EAAE,CACvJ;YACD,EAAE;YACF,YAAY;SACb,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACb,MAAM,OAAO,CAAC,GAAG,KAAK,CAAC,GAAG,CAAC,QAAQ,EAAE,EAAE,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,IAAI,CAAC,SAAS,CAAC,EAAE,IAAI,EAAE,CAAC,EAAE,CAAC,CAAC;IAC3F,CAAC;IAED,GAAG,CACD,kBAAkB,MAAM,uBAAuB;QAC7C,CAAC,OAAO,CAAC,CAAC,CAAC,aAAa,OAAO,eAAe,CAAC,CAAC,CAAC,EAAE,CAAC;QACpD,CAAC,YAAY,CAAC,MAAM,CAAC,CAAC,CAAC,KAAK,YAAY,CAAC,MAAM,0BAA0B,CAAC,CAAC,CAAC,EAAE,CAAC;QAC/E,GAAG,CACN,CAAC;IAEF,MAAM,WAAW,GAAG,MAAM,CAAC,cAAc,EAAE,QAAQ,IAAI,EAAE,CAAC;IAC1D,IAAI,WAAW,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC3B,MAAM,QAAQ,GAAG,MAAM,0BAA0B,CAAC,GAAG,EAAE,WAAW,EAAE,GAAG,CAAC,CAAC;QACzE,IAAI,QAAQ;YAAE,GAAG,CAAC,oBAAoB,QAAQ,oCAAoC,CAAC,CAAC;IACtF,CAAC;AACH,CAAC"}
|
package/dist/host.d.ts
ADDED
package/dist/host.js
ADDED
|
@@ -0,0 +1,15 @@
|
|
|
1
|
+
/** Resolve which platform to talk to: explicit --host flag, else detected from CI env vars. */
|
|
2
|
+
export function detectHost(explicit) {
|
|
3
|
+
const env = process.env;
|
|
4
|
+
const host = explicit ??
|
|
5
|
+
(env.BITBUCKET_WORKSPACE || env.BITBUCKET_PR_ID
|
|
6
|
+
? "bitbucket"
|
|
7
|
+
: env.GITLAB_CI
|
|
8
|
+
? "gitlab"
|
|
9
|
+
: "github");
|
|
10
|
+
if (host !== "github" && host !== "bitbucket" && host !== "gitlab") {
|
|
11
|
+
throw new Error(`--host must be github, bitbucket, or gitlab, got "${explicit}".`);
|
|
12
|
+
}
|
|
13
|
+
return host;
|
|
14
|
+
}
|
|
15
|
+
//# sourceMappingURL=host.js.map
|
package/dist/host.js.map
ADDED
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"host.js","sourceRoot":"","sources":["../src/host.ts"],"names":[],"mappings":"AAEA,+FAA+F;AAC/F,MAAM,UAAU,UAAU,CAAC,QAA4B;IACrD,MAAM,GAAG,GAAG,OAAO,CAAC,GAAG,CAAC;IACxB,MAAM,IAAI,GACR,QAAQ;QACR,CAAC,GAAG,CAAC,mBAAmB,IAAI,GAAG,CAAC,eAAe;YAC7C,CAAC,CAAC,WAAW;YACb,CAAC,CAAC,GAAG,CAAC,SAAS;gBACb,CAAC,CAAC,QAAQ;gBACV,CAAC,CAAC,QAAQ,CAAC,CAAC;IAClB,IAAI,IAAI,KAAK,QAAQ,IAAI,IAAI,KAAK,WAAW,IAAI,IAAI,KAAK,QAAQ,EAAE,CAAC;QACnE,MAAM,IAAI,KAAK,CAAC,qDAAqD,QAAQ,IAAI,CAAC,CAAC;IACrF,CAAC;IACD,OAAO,IAAI,CAAC;AACd,CAAC"}
|
package/dist/llm/prompt.d.ts
CHANGED
|
@@ -1,5 +1,9 @@
|
|
|
1
1
|
import type { FileDiff, ReviewConfig } from "../types.js";
|
|
2
2
|
export declare const REVIEW_SYSTEM_PROMPT = "You are a senior software engineer reviewing a pull request. You are rigorous but pragmatic: you flag real problems, not style preferences the team hasn't asked for.\n\nReview the diff for these categories:\n- bug: logic errors, off-by-one mistakes, incorrect conditions, broken error handling, race conditions, wrong API usage\n- security: injection, unsafe deserialization, secrets in code, missing authorization or validation at trust boundaries, path traversal\n- missing-tests: changed or new behavior with no corresponding test change, especially branches and error paths\n- naming: identifiers inconsistent with the surrounding codebase's conventions, or misleading about what they do\n- complexity: functions that grew too long or deeply nested to reason about, duplicated logic that already exists elsewhere in the repo\n- custom: anything the repository's custom instructions ask you to flag\n\nRules:\n1. Comment ONLY on lines that appear as added (+) lines in the diff. Use the new-file line numbers printed at the start of each line.\n2. Use the REPOSITORY CONTEXT section (summaries and symbols of related files) to judge consistency, spot callers the change might break, and avoid flagging things the codebase already handles elsewhere. Do not report findings about context files themselves \u2014 only about the diff.\n3. Severity rubric:\n - critical: will corrupt data, break production, or is an exploitable vulnerability\n - high: incorrect behavior on realistic inputs, or a security weakness\n - medium: likely bug under edge cases, missing tests for risky logic, significant maintainability problem\n - low: minor issues worth fixing but not blocking\n - info: observations and suggestions, no action strictly required\n4. Report every real issue you find, including ones you are uncertain about \u2014 set confidence accordingly; a downstream filter handles thresholds. Do not pad the review with nitpicks to seem thorough.\n5. When you provide a suggestion, it must be a drop-in replacement for exactly the lines in start_line..end_line, complete and correctly indented.\n6. Keep each finding's body focused: what is wrong, why it matters, how to fix it.\n7. When the REPOSITORY CONTEXT contradicts a suspicion (the pattern is established elsewhere, a caller already handles the case, a convention explains the choice), do not report it as high or critical unless you can cite the specific added lines that break it \u2014 cite that evidence in the body, or lower the severity and confidence accordingly. Never report a critical/high finding that rests on guesses about code you cannot see.";
|
|
3
|
+
export declare const HIGH_LEVEL_SYSTEM_PROMPT = "You are a senior software engineer giving a high-level-only review of a large pull request. It is too large for a thorough line-by-line pass, so this review only surfaces the issues serious enough to matter regardless.\n\nReview categories: bug, security, missing-tests, naming, complexity, custom (same meanings as a normal review).\n\nRules:\n1. Comment ONLY on lines that appear as added (+) lines in the diff. Use the new-file line numbers printed at the start of each line.\n2. Report ONLY critical and high severity findings \u2014 architectural risks, likely bugs, security issues. Do not report medium, low, or info findings in this pass.\n3. In your summary, explicitly recommend splitting this PR into smaller, independently reviewable pieces, and say why \u2014 do not soften or omit this recommendation.\n4. Never report a critical/high finding that rests on guesses about code you cannot see \u2014 cite the specific added lines that justify it.";
|
|
4
|
+
export declare const SECURITY_LENS_SYSTEM_PROMPT = "You are a senior application security engineer reviewing a pull request. You ONLY report the \"security\" category: injection, unsafe deserialization, secrets in code, missing authorization or validation at trust boundaries, path traversal, and similar exploitable weaknesses. Do not report bugs, missing tests, naming, or complexity issues \u2014 other reviewers own those; report only what a specialized security pass would catch.\n\nRules:\n1. Comment ONLY on lines that appear as added (+) lines in the diff. Use the new-file line numbers printed at the start of each line.\n2. Use the REPOSITORY CONTEXT section (when present) to judge consistency and spot callers the change might break. Do not report findings about context files themselves \u2014 only about the diff.\n3. Severity rubric:\n - critical: will corrupt data, break production, or is an exploitable vulnerability\n - high: incorrect behavior on realistic inputs, or a security weakness\n - medium: likely bug under edge cases, missing tests for risky logic, significant maintainability problem\n - low: minor issues worth fixing but not blocking\n - info: observations and suggestions, no action strictly required\n4. Report every real issue you find, including ones you are uncertain about \u2014 set confidence accordingly; a downstream filter handles thresholds. Do not pad the review with nitpicks to seem thorough.\n5. When you provide a suggestion, it must be a drop-in replacement for exactly the lines in start_line..end_line, complete and correctly indented.\n6. Keep each finding's body focused: what is wrong, why it matters, how to fix it.\n7. Never report a critical/high finding that rests on guesses about code you cannot see \u2014 cite the specific added lines that justify it.";
|
|
5
|
+
export declare const CORRECTNESS_LENS_SYSTEM_PROMPT = "You are a senior software engineer reviewing a pull request for correctness. You ONLY report these categories:\n- bug: logic errors, off-by-one mistakes, incorrect conditions, broken error handling, race conditions, wrong API usage\n- missing-tests: changed or new behavior with no corresponding test change, especially branches and error paths\nDo not report security, naming, or complexity issues \u2014 other reviewers own those.\n\nRules:\n1. Comment ONLY on lines that appear as added (+) lines in the diff. Use the new-file line numbers printed at the start of each line.\n2. Use the REPOSITORY CONTEXT section (when present) to judge consistency and spot callers the change might break. Do not report findings about context files themselves \u2014 only about the diff.\n3. Severity rubric:\n - critical: will corrupt data, break production, or is an exploitable vulnerability\n - high: incorrect behavior on realistic inputs, or a security weakness\n - medium: likely bug under edge cases, missing tests for risky logic, significant maintainability problem\n - low: minor issues worth fixing but not blocking\n - info: observations and suggestions, no action strictly required\n4. Report every real issue you find, including ones you are uncertain about \u2014 set confidence accordingly; a downstream filter handles thresholds. Do not pad the review with nitpicks to seem thorough.\n5. When you provide a suggestion, it must be a drop-in replacement for exactly the lines in start_line..end_line, complete and correctly indented.\n6. Keep each finding's body focused: what is wrong, why it matters, how to fix it.\n7. Never report a critical/high finding that rests on guesses about code you cannot see \u2014 cite the specific added lines that justify it.";
|
|
6
|
+
export declare const QUALITY_LENS_SYSTEM_PROMPT = "You are a senior software engineer reviewing a pull request for code quality and maintainability. You ONLY report these categories:\n- naming: identifiers inconsistent with the surrounding codebase's conventions, or misleading about what they do\n- complexity: functions that grew too long or deeply nested to reason about, duplicated logic that already exists elsewhere in the repo\n- custom: anything the repository's custom instructions ask you to flag\nDo not report bugs, missing tests, or security issues \u2014 other reviewers own those.\n\nRules:\n1. Comment ONLY on lines that appear as added (+) lines in the diff. Use the new-file line numbers printed at the start of each line.\n2. Use the REPOSITORY CONTEXT section (when present) to judge consistency and spot callers the change might break. Do not report findings about context files themselves \u2014 only about the diff.\n3. Severity rubric:\n - critical: will corrupt data, break production, or is an exploitable vulnerability\n - high: incorrect behavior on realistic inputs, or a security weakness\n - medium: likely bug under edge cases, missing tests for risky logic, significant maintainability problem\n - low: minor issues worth fixing but not blocking\n - info: observations and suggestions, no action strictly required\n4. Report every real issue you find, including ones you are uncertain about \u2014 set confidence accordingly; a downstream filter handles thresholds. Do not pad the review with nitpicks to seem thorough.\n5. When you provide a suggestion, it must be a drop-in replacement for exactly the lines in start_line..end_line, complete and correctly indented.\n6. Keep each finding's body focused: what is wrong, why it matters, how to fix it.\n7. Never report a critical/high finding that rests on guesses about code you cannot see \u2014 cite the specific added lines that justify it.";
|
|
3
7
|
export declare const CRITIQUE_SYSTEM_PROMPT = "You are re-reading your own PR review before it is posted. Your ONLY job is to remove findings you would walk back if a developer pushed back \u2014 speculative issues, style opinions dressed as bugs, problems the surrounding code or repository context already handles, or duplicates.\n\nRules:\n1. You may ONLY drop findings. Never edit, merge, reword, or add findings.\n2. Keep every finding you would defend in a face-to-face review \u2014 when in doubt, keep it. Dropping a real bug is far worse than keeping a borderline nitpick.\n3. Return the ids of the findings to KEEP.";
|
|
4
8
|
export declare function buildCritiqueUserPrompt(input: {
|
|
5
9
|
findings: {
|
|
@@ -19,5 +23,7 @@ export interface ReviewPromptInput {
|
|
|
19
23
|
config: ReviewConfig;
|
|
20
24
|
/** e.g. "PR #42: Add session refresh" or "local diff vs origin/main" */
|
|
21
25
|
changeDescription: string;
|
|
26
|
+
/** Rendered `detectSignals` output; empty when signal_detector is off or nothing fired. */
|
|
27
|
+
signals?: string;
|
|
22
28
|
}
|
|
23
29
|
export declare function buildReviewUserPrompt(input: ReviewPromptInput): string;
|
package/dist/llm/prompt.js
CHANGED
|
@@ -22,6 +22,44 @@ Rules:
|
|
|
22
22
|
5. When you provide a suggestion, it must be a drop-in replacement for exactly the lines in start_line..end_line, complete and correctly indented.
|
|
23
23
|
6. Keep each finding's body focused: what is wrong, why it matters, how to fix it.
|
|
24
24
|
7. When the REPOSITORY CONTEXT contradicts a suspicion (the pattern is established elsewhere, a caller already handles the case, a convention explains the choice), do not report it as high or critical unless you can cite the specific added lines that break it ā cite that evidence in the body, or lower the severity and confidence accordingly. Never report a critical/high finding that rests on guesses about code you cannot see.`;
|
|
25
|
+
export const HIGH_LEVEL_SYSTEM_PROMPT = `You are a senior software engineer giving a high-level-only review of a large pull request. It is too large for a thorough line-by-line pass, so this review only surfaces the issues serious enough to matter regardless.
|
|
26
|
+
|
|
27
|
+
Review categories: bug, security, missing-tests, naming, complexity, custom (same meanings as a normal review).
|
|
28
|
+
|
|
29
|
+
Rules:
|
|
30
|
+
1. Comment ONLY on lines that appear as added (+) lines in the diff. Use the new-file line numbers printed at the start of each line.
|
|
31
|
+
2. Report ONLY critical and high severity findings ā architectural risks, likely bugs, security issues. Do not report medium, low, or info findings in this pass.
|
|
32
|
+
3. In your summary, explicitly recommend splitting this PR into smaller, independently reviewable pieces, and say why ā do not soften or omit this recommendation.
|
|
33
|
+
4. Never report a critical/high finding that rests on guesses about code you cannot see ā cite the specific added lines that justify it.`;
|
|
34
|
+
const REVIEW_RULES = `Rules:
|
|
35
|
+
1. Comment ONLY on lines that appear as added (+) lines in the diff. Use the new-file line numbers printed at the start of each line.
|
|
36
|
+
2. Use the REPOSITORY CONTEXT section (when present) to judge consistency and spot callers the change might break. Do not report findings about context files themselves ā only about the diff.
|
|
37
|
+
3. Severity rubric:
|
|
38
|
+
- critical: will corrupt data, break production, or is an exploitable vulnerability
|
|
39
|
+
- high: incorrect behavior on realistic inputs, or a security weakness
|
|
40
|
+
- medium: likely bug under edge cases, missing tests for risky logic, significant maintainability problem
|
|
41
|
+
- low: minor issues worth fixing but not blocking
|
|
42
|
+
- info: observations and suggestions, no action strictly required
|
|
43
|
+
4. Report every real issue you find, including ones you are uncertain about ā set confidence accordingly; a downstream filter handles thresholds. Do not pad the review with nitpicks to seem thorough.
|
|
44
|
+
5. When you provide a suggestion, it must be a drop-in replacement for exactly the lines in start_line..end_line, complete and correctly indented.
|
|
45
|
+
6. Keep each finding's body focused: what is wrong, why it matters, how to fix it.
|
|
46
|
+
7. Never report a critical/high finding that rests on guesses about code you cannot see ā cite the specific added lines that justify it.`;
|
|
47
|
+
export const SECURITY_LENS_SYSTEM_PROMPT = `You are a senior application security engineer reviewing a pull request. You ONLY report the "security" category: injection, unsafe deserialization, secrets in code, missing authorization or validation at trust boundaries, path traversal, and similar exploitable weaknesses. Do not report bugs, missing tests, naming, or complexity issues ā other reviewers own those; report only what a specialized security pass would catch.
|
|
48
|
+
|
|
49
|
+
${REVIEW_RULES}`;
|
|
50
|
+
export const CORRECTNESS_LENS_SYSTEM_PROMPT = `You are a senior software engineer reviewing a pull request for correctness. You ONLY report these categories:
|
|
51
|
+
- bug: logic errors, off-by-one mistakes, incorrect conditions, broken error handling, race conditions, wrong API usage
|
|
52
|
+
- missing-tests: changed or new behavior with no corresponding test change, especially branches and error paths
|
|
53
|
+
Do not report security, naming, or complexity issues ā other reviewers own those.
|
|
54
|
+
|
|
55
|
+
${REVIEW_RULES}`;
|
|
56
|
+
export const QUALITY_LENS_SYSTEM_PROMPT = `You are a senior software engineer reviewing a pull request for code quality and maintainability. You ONLY report these categories:
|
|
57
|
+
- naming: identifiers inconsistent with the surrounding codebase's conventions, or misleading about what they do
|
|
58
|
+
- complexity: functions that grew too long or deeply nested to reason about, duplicated logic that already exists elsewhere in the repo
|
|
59
|
+
- custom: anything the repository's custom instructions ask you to flag
|
|
60
|
+
Do not report bugs, missing tests, or security issues ā other reviewers own those.
|
|
61
|
+
|
|
62
|
+
${REVIEW_RULES}`;
|
|
25
63
|
export const CRITIQUE_SYSTEM_PROMPT = `You are re-reading your own PR review before it is posted. Your ONLY job is to remove findings you would walk back if a developer pushed back ā speculative issues, style opinions dressed as bugs, problems the surrounding code or repository context already handles, or duplicates.
|
|
26
64
|
|
|
27
65
|
Rules:
|
|
@@ -38,7 +76,7 @@ export function buildReviewUserPrompt(input) {
|
|
|
38
76
|
const parts = [];
|
|
39
77
|
parts.push(`Reviewing: ${input.changeDescription}`);
|
|
40
78
|
if (input.config.custom_instructions.trim()) {
|
|
41
|
-
parts.push(`# CUSTOM REVIEW INSTRUCTIONS (
|
|
79
|
+
parts.push(`# CUSTOM REVIEW INSTRUCTIONS (repository-provided)\n${input.config.custom_instructions.trim()}`);
|
|
42
80
|
}
|
|
43
81
|
if (input.context.trim()) {
|
|
44
82
|
parts.push(`# REPOSITORY CONTEXT\nSummaries and exported symbols of files related to this change (importers listed first ā they depend on the changed code):\n\n${input.context}`);
|
|
@@ -46,6 +84,9 @@ export function buildReviewUserPrompt(input) {
|
|
|
46
84
|
else {
|
|
47
85
|
parts.push(`# REPOSITORY CONTEXT\n(none available ā no codemap index; review the diff on its own)`);
|
|
48
86
|
}
|
|
87
|
+
if (input.signals?.trim()) {
|
|
88
|
+
parts.push(`# SIGNALS\nDeterministic pointers into added lines that touch error handling, control flow, or framework hooks. Not findings by themselves ā use them to decide where to look closer.\n\n${input.signals.trim()}`);
|
|
89
|
+
}
|
|
49
90
|
const diffs = input.files.map((f) => renderDiffForPrompt(f)).join("\n\n");
|
|
50
91
|
parts.push(`# DIFF\nEach line is prefixed with its new-file line number. Only '+' lines are commentable.\n\n${diffs}`);
|
|
51
92
|
return parts.join("\n\n");
|
package/dist/llm/prompt.js.map
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"prompt.js","sourceRoot":"","sources":["../../src/llm/prompt.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,mBAAmB,EAAE,MAAM,kBAAkB,CAAC;AAEvD,MAAM,CAAC,MAAM,oBAAoB,GAAG;;;;;;;;;;;;;;;;;;;;;;8aAsB0Y,CAAC;AAE/a,MAAM,CAAC,MAAM,sBAAsB,GAAG;;;;;2CAKK,CAAC;AAE5C,MAAM,UAAU,uBAAuB,CAAC,KAGvC;IACC,MAAM,IAAI,GAAG,KAAK,CAAC,QAAQ;SACxB,GAAG,CACF,CAAC,CAAC,EAAE,EAAE,CACJ,SAAS,CAAC,CAAC,EAAE,iBAAiB,CAAC,CAAC,IAAI,IAAI,CAAC,CAAC,UAAU,KAAK,CAAC,CAAC,QAAQ,eAAe,CAAC,CAAC,KAAK,aAAa,CAAC,CAAC,IAAI,EAAE,CACjH;SACA,IAAI,CAAC,IAAI,CAAC,CAAC;IACd,OAAO,wBAAwB,KAAK,CAAC,iBAAiB,mBAAmB,IAAI,6BAA6B,CAAC;AAC7G,CAAC;
|
|
1
|
+
{"version":3,"file":"prompt.js","sourceRoot":"","sources":["../../src/llm/prompt.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,mBAAmB,EAAE,MAAM,kBAAkB,CAAC;AAEvD,MAAM,CAAC,MAAM,oBAAoB,GAAG;;;;;;;;;;;;;;;;;;;;;;8aAsB0Y,CAAC;AAE/a,MAAM,CAAC,MAAM,wBAAwB,GAAG;;;;;;;;yIAQiG,CAAC;AAE1I,MAAM,YAAY,GAAG;;;;;;;;;;;;yIAYoH,CAAC;AAE1I,MAAM,CAAC,MAAM,2BAA2B,GAAG;;EAEzC,YAAY,EAAE,CAAC;AAEjB,MAAM,CAAC,MAAM,8BAA8B,GAAG;;;;;EAK5C,YAAY,EAAE,CAAC;AAEjB,MAAM,CAAC,MAAM,0BAA0B,GAAG;;;;;;EAMxC,YAAY,EAAE,CAAC;AAEjB,MAAM,CAAC,MAAM,sBAAsB,GAAG;;;;;2CAKK,CAAC;AAE5C,MAAM,UAAU,uBAAuB,CAAC,KAGvC;IACC,MAAM,IAAI,GAAG,KAAK,CAAC,QAAQ;SACxB,GAAG,CACF,CAAC,CAAC,EAAE,EAAE,CACJ,SAAS,CAAC,CAAC,EAAE,iBAAiB,CAAC,CAAC,IAAI,IAAI,CAAC,CAAC,UAAU,KAAK,CAAC,CAAC,QAAQ,eAAe,CAAC,CAAC,KAAK,aAAa,CAAC,CAAC,IAAI,EAAE,CACjH;SACA,IAAI,CAAC,IAAI,CAAC,CAAC;IACd,OAAO,wBAAwB,KAAK,CAAC,iBAAiB,mBAAmB,IAAI,6BAA6B,CAAC;AAC7G,CAAC;AAaD,MAAM,UAAU,qBAAqB,CAAC,KAAwB;IAC5D,MAAM,KAAK,GAAa,EAAE,CAAC;IAE3B,KAAK,CAAC,IAAI,CAAC,cAAc,KAAK,CAAC,iBAAiB,EAAE,CAAC,CAAC;IAEpD,IAAI,KAAK,CAAC,MAAM,CAAC,mBAAmB,CAAC,IAAI,EAAE,EAAE,CAAC;QAC5C,KAAK,CAAC,IAAI,CAAC,uDAAuD,KAAK,CAAC,MAAM,CAAC,mBAAmB,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC;IAC/G,CAAC;IAED,IAAI,KAAK,CAAC,OAAO,CAAC,IAAI,EAAE,EAAE,CAAC;QACzB,KAAK,CAAC,IAAI,CAAC,uJAAuJ,KAAK,CAAC,OAAO,EAAE,CAAC,CAAC;IACrL,CAAC;SAAM,CAAC;QACN,KAAK,CAAC,IAAI,CAAC,uFAAuF,CAAC,CAAC;IACtG,CAAC;IAED,IAAI,KAAK,CAAC,OAAO,EAAE,IAAI,EAAE,EAAE,CAAC;QAC1B,KAAK,CAAC,IAAI,CACR,4LAA4L,KAAK,CAAC,OAAO,CAAC,IAAI,EAAE,EAAE,CACnN,CAAC;IACJ,CAAC;IAED,MAAM,KAAK,GAAG,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,mBAAmB,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;IAC1E,KAAK,CAAC,IAAI,CAAC,mGAAmG,KAAK,EAAE,CAAC,CAAC;IAEvH,OAAO,KAAK,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;AAC5B,CAAC"}
|
package/dist/report/cli.d.ts
CHANGED
|
@@ -1,5 +1,10 @@
|
|
|
1
1
|
import { type Finding, type ReviewResult, type Severity } from "../types.js";
|
|
2
|
-
export declare function formatReport(result: ReviewResult
|
|
2
|
+
export declare function formatReport(result: ReviewResult & {
|
|
3
|
+
highLevelReview?: boolean;
|
|
4
|
+
reviewBrief?: {
|
|
5
|
+
blastRadius: string[];
|
|
6
|
+
};
|
|
7
|
+
}): string;
|
|
3
8
|
export declare function summaryCounts(findings: Finding[]): string;
|
|
4
|
-
/** True when any finding meets or exceeds the fail_on severity threshold. */
|
|
5
|
-
export declare function shouldFail(findings: Finding[], failOn: Severity): boolean;
|
|
9
|
+
/** True when any finding meets or exceeds the fail_on severity threshold. "off" never fails. */
|
|
10
|
+
export declare function shouldFail(findings: Finding[], failOn: Severity | "off"): boolean;
|
package/dist/report/cli.js
CHANGED
|
@@ -11,8 +11,16 @@ export function formatReport(result) {
|
|
|
11
11
|
const lines = [];
|
|
12
12
|
lines.push(pc.bold("PR Review"));
|
|
13
13
|
lines.push("");
|
|
14
|
+
if (result.highLevelReview) {
|
|
15
|
+
lines.push(pc.yellow("ā Large diff ā high-level review only (critical/high). Consider splitting this PR."));
|
|
16
|
+
lines.push("");
|
|
17
|
+
}
|
|
14
18
|
lines.push(result.summary.trim());
|
|
15
19
|
lines.push("");
|
|
20
|
+
if (result.reviewBrief?.blastRadius.length) {
|
|
21
|
+
lines.push(pc.dim(`blast radius: ${result.reviewBrief.blastRadius.length} downstream file(s)`));
|
|
22
|
+
lines.push("");
|
|
23
|
+
}
|
|
16
24
|
if (result.findings.length === 0) {
|
|
17
25
|
lines.push(pc.green("ā No findings."));
|
|
18
26
|
return lines.join("\n");
|
|
@@ -55,8 +63,10 @@ export function summaryCounts(findings) {
|
|
|
55
63
|
.map(([sev, n]) => `${n} ${sev}`);
|
|
56
64
|
return pc.bold(`${findings.length} finding(s): ${parts.join(", ")}`);
|
|
57
65
|
}
|
|
58
|
-
/** True when any finding meets or exceeds the fail_on severity threshold. */
|
|
66
|
+
/** True when any finding meets or exceeds the fail_on severity threshold. "off" never fails. */
|
|
59
67
|
export function shouldFail(findings, failOn) {
|
|
68
|
+
if (failOn === "off")
|
|
69
|
+
return false;
|
|
60
70
|
const threshold = severityRank(failOn);
|
|
61
71
|
return findings.some((f) => severityRank(f.severity) <= threshold);
|
|
62
72
|
}
|
package/dist/report/cli.js.map
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"cli.js","sourceRoot":"","sources":["../../src/report/cli.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,MAAM,YAAY,CAAC;AAC5B,OAAO,EAAE,YAAY,EAAkD,MAAM,aAAa,CAAC;AAE3F,MAAM,cAAc,GAA4C;IAC9D,QAAQ,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,CAAC,EAAE,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,WAAW,EAAE,GAAG,CAAC,CAAC;IAC3D,IAAI,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC;IACpC,MAAM,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,EAAE,CAAC,MAAM,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC;IACzC,GAAG,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,EAAE,CAAC,IAAI,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC;IACpC,IAAI,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC;CACrC,CAAC;AAEF,MAAM,UAAU,YAAY,
|
|
1
|
+
{"version":3,"file":"cli.js","sourceRoot":"","sources":["../../src/report/cli.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,MAAM,YAAY,CAAC;AAC5B,OAAO,EAAE,YAAY,EAAkD,MAAM,aAAa,CAAC;AAE3F,MAAM,cAAc,GAA4C;IAC9D,QAAQ,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,CAAC,EAAE,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,WAAW,EAAE,GAAG,CAAC,CAAC;IAC3D,IAAI,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC;IACpC,MAAM,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,EAAE,CAAC,MAAM,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC;IACzC,GAAG,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,EAAE,CAAC,IAAI,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC;IACpC,IAAI,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC;CACrC,CAAC;AAEF,MAAM,UAAU,YAAY,CAC1B,MAA6F;IAE7F,MAAM,KAAK,GAAa,EAAE,CAAC;IAE3B,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC,CAAC;IACjC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACf,IAAI,MAAM,CAAC,eAAe,EAAE,CAAC;QAC3B,KAAK,CAAC,IAAI,CACR,EAAE,CAAC,MAAM,CAAC,oFAAoF,CAAC,CAChG,CAAC;QACF,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACjB,CAAC;IACD,KAAK,CAAC,IAAI,CAAC,MAAM,CAAC,OAAO,CAAC,IAAI,EAAE,CAAC,CAAC;IAClC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACf,IAAI,MAAM,CAAC,WAAW,EAAE,WAAW,CAAC,MAAM,EAAE,CAAC;QAC3C,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,GAAG,CAAC,iBAAiB,MAAM,CAAC,WAAW,CAAC,WAAW,CAAC,MAAM,qBAAqB,CAAC,CAAC,CAAC;QAChG,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACjB,CAAC;IAED,IAAI,MAAM,CAAC,QAAQ,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACjC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,KAAK,CAAC,gBAAgB,CAAC,CAAC,CAAC;QACvC,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC1B,CAAC;IAED,MAAM,MAAM,GAAG,IAAI,GAAG,EAAqB,CAAC;IAC5C,KAAK,MAAM,CAAC,IAAI,MAAM,CAAC,QAAQ,EAAE,CAAC;QAChC,IAAI,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC;YAAE,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC;QAChD,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAE,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAC9B,CAAC;IAED,KAAK,MAAM,CAAC,IAAI,EAAE,QAAQ,CAAC,IAAI,MAAM,EAAE,CAAC;QACtC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,IAAI,CAAC,EAAE,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACxC,KAAK,MAAM,CAAC,IAAI,QAAQ,EAAE,CAAC;YACzB,MAAM,KAAK,GAAG,CAAC,CAAC,UAAU,KAAK,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,UAAU,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,UAAU,IAAI,CAAC,CAAC,QAAQ,EAAE,CAAC;YAClG,KAAK,CAAC,IAAI,CACR,KAAK,cAAc,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,IAAI,EAAE,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC,QAAQ,GAAG,CAAC,IAAI,EAAE,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC,GAAG,CAAC,IAAI,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,UAAU,GAAG,GAAG,CAAC,IAAI,CAAC,EAAE,CACzJ,CAAC;YACF,KAAK,MAAM,QAAQ,IAAI,CAAC,CAAC,IAAI,CAAC,IAAI,EAAE,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC;gBACjD,KAAK,CAAC,IAAI,CAAC,SAAS,QAAQ,EAAE,CAAC,CAAC;YAClC,CAAC;YACD,IAAI,CAAC,CAAC,UAAU,EAAE,CAAC;gBACjB,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,GAAG,CAAC,mBAAmB,CAAC,CAAC,CAAC;gBACxC,KAAK,MAAM,CAAC,IAAI,CAAC,CAAC,UAAU,CAAC,KAAK,CAAC,IAAI,CAAC;oBAAE,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,KAAK,CAAC,WAAW,CAAC,EAAE,CAAC,CAAC,CAAC;YACjF,CAAC;YACD,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACjB,CAAC;IACH,CAAC;IAED,KAAK,CAAC,IAAI,CAAC,aAAa,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC;IAC3C,IAAI,MAAM,CAAC,cAAc,EAAE,CAAC;QAC1B,MAAM,CAAC,GAAG,MAAM,CAAC,cAAc,CAAC;QAChC,KAAK,CAAC,IAAI,CACR,EAAE,CAAC,GAAG,CACJ,cAAc,CAAC,CAAC,GAAG,CAAC,MAAM,UAAU,CAAC,CAAC,UAAU,CAAC,MAAM,iBAAiB,CAAC,CAAC,QAAQ,CAAC,MAAM,eAAe,CAAC,CAAC,UAAU,CAAC,MAAM,aAAa,CACzI,CACF,CAAC;IACJ,CAAC;IACD,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC1B,CAAC;AAED,MAAM,UAAU,aAAa,CAAC,QAAmB;IAC/C,MAAM,MAAM,GAAG,IAAI,GAAG,EAAoB,CAAC;IAC3C,KAAK,MAAM,CAAC,IAAI,QAAQ;QAAE,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC;IACpF,MAAM,KAAK,GAAG,CAAC,GAAG,MAAM,CAAC,OAAO,EAAE,CAAC;SAChC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,GAAG,YAAY,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;SACvD,GAAG,CAAC,CAAC,CAAC,GAAG,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,GAAG,CAAC,IAAI,GAAG,EAAE,CAAC,CAAC;IACpC,OAAO,EAAE,CAAC,IAAI,CAAC,GAAG,QAAQ,CAAC,MAAM,gBAAgB,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;AACvE,CAAC;AAED,gGAAgG;AAChG,MAAM,UAAU,UAAU,CAAC,QAAmB,EAAE,MAAwB;IACtE,IAAI,MAAM,KAAK,KAAK;QAAE,OAAO,KAAK,CAAC;IACnC,MAAM,SAAS,GAAG,YAAY,CAAC,MAAM,CAAC,CAAC;IACvC,OAAO,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,YAAY,CAAC,CAAC,CAAC,QAAQ,CAAC,IAAI,SAAS,CAAC,CAAC;AACrE,CAAC"}
|
package/dist/report/json.d.ts
CHANGED
|
@@ -1,3 +1,4 @@
|
|
|
1
|
+
import type { ReviewBrief } from "../review.js";
|
|
1
2
|
import type { Finding, Reconciliation, ReviewResult, RunStats } from "../types.js";
|
|
2
3
|
export interface JsonReport {
|
|
3
4
|
meta: {
|
|
@@ -10,9 +11,13 @@ export interface JsonReport {
|
|
|
10
11
|
findings: ReviewResult["findings"];
|
|
11
12
|
reconciliation?: Reconciliation;
|
|
12
13
|
stats?: RunStats;
|
|
14
|
+
highLevelReview?: boolean;
|
|
15
|
+
reviewBrief?: ReviewBrief;
|
|
13
16
|
}
|
|
14
17
|
export declare function buildJsonReport(result: ReviewResult & {
|
|
15
18
|
stats?: RunStats;
|
|
19
|
+
highLevelReview?: boolean;
|
|
20
|
+
reviewBrief?: ReviewBrief;
|
|
16
21
|
}, meta: {
|
|
17
22
|
model: string;
|
|
18
23
|
changeDescription: string;
|
package/dist/report/json.js
CHANGED
|
@@ -7,6 +7,8 @@ export function buildJsonReport(result, meta) {
|
|
|
7
7
|
findings: result.findings,
|
|
8
8
|
reconciliation: result.reconciliation,
|
|
9
9
|
stats: result.stats,
|
|
10
|
+
highLevelReview: result.highLevelReview,
|
|
11
|
+
reviewBrief: result.reviewBrief,
|
|
10
12
|
};
|
|
11
13
|
}
|
|
12
14
|
/** One structured log line for CI parsing: counts, tokens, model. */
|
package/dist/report/json.js.map
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"json.js","sourceRoot":"","sources":["../../src/report/json.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,SAAS,CAAC;AACrC,OAAO,EAAE,QAAQ,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;
|
|
1
|
+
{"version":3,"file":"json.js","sourceRoot":"","sources":["../../src/report/json.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,SAAS,CAAC;AACrC,OAAO,EAAE,QAAQ,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AAmBvD,MAAM,UAAU,eAAe,CAC7B,MAAiG,EACjG,IAAwE;IAExE,OAAO;QACL,IAAI,EAAE,EAAE,WAAW,EAAE,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE,EAAE,GAAG,IAAI,EAAE;QACxD,OAAO,EAAE,MAAM,CAAC,OAAO;QACvB,QAAQ,EAAE,MAAM,CAAC,QAAQ;QACzB,cAAc,EAAE,MAAM,CAAC,cAAc;QACrC,KAAK,EAAE,MAAM,CAAC,KAAK;QACnB,eAAe,EAAE,MAAM,CAAC,eAAe;QACvC,WAAW,EAAE,MAAM,CAAC,WAAW;KAChC,CAAC;AACJ,CAAC;AAED,qEAAqE;AACrE,MAAM,UAAU,oBAAoB,CAAC,KAAe;IAClD,OAAO,qBAAqB,IAAI,CAAC,SAAS,CAAC,KAAK,CAAC,EAAE,CAAC;AACtD,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,eAAe,CAAC,IAAY,EAAE,MAAkB;IACpE,MAAM,SAAS,CAAC,IAAI,EAAE,IAAI,CAAC,SAAS,CAAC,MAAM,EAAE,IAAI,EAAE,CAAC,CAAC,GAAG,IAAI,EAAE,MAAM,CAAC,CAAC;AACxE,CAAC;AAED,yEAAyE;AACzE,MAAM,CAAC,KAAK,UAAU,eAAe,CAAC,UAAkB;IACtD,MAAM,QAAQ,GAAG,MAAM,oBAAoB,CAAC,UAAU,CAAC,CAAC;IACxD,MAAM,GAAG,GAAG,IAAI,GAAG,EAAU,CAAC;IAC9B,KAAK,MAAM,CAAC,IAAI,QAAQ,EAAE,CAAC;QACzB,IAAI,CAAC,CAAC,EAAE;YAAE,GAAG,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,WAAW,EAAE,CAAC,CAAC;IACxC,CAAC;IACD,IAAI,CAAC,UAAU,CAAC,UAAU,CAAC;QAAE,OAAO,GAAG,CAAC;IACxC,IAAI,CAAC;QACH,MAAM,GAAG,GAAG,IAAI,CAAC,KAAK,CAAC,MAAM,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC,CAExD,CAAC;QACF,KAAK,MAAM,EAAE,IAAI,GAAG,CAAC,cAAc,EAAE,GAAG,IAAI,EAAE;YAAE,GAAG,CAAC,GAAG,CAAC,EAAE,CAAC,WAAW,EAAE,CAAC,CAAC;QAC1E,KAAK,MAAM,EAAE,IAAI,GAAG,CAAC,cAAc,EAAE,UAAU,IAAI,EAAE;YAAE,GAAG,CAAC,GAAG,CAAC,EAAE,CAAC,WAAW,EAAE,CAAC,CAAC;IACnF,CAAC;IAAC,MAAM,CAAC;QACP,YAAY;IACd,CAAC;IACD,OAAO,GAAG,CAAC;AACb,CAAC;AAED,iFAAiF;AACjF,MAAM,CAAC,KAAK,UAAU,oBAAoB,CAAC,UAAkB;IAC3D,IAAI,CAAC,UAAU,CAAC,UAAU,CAAC;QAAE,OAAO,EAAE,CAAC;IACvC,IAAI,CAAC;QACH,MAAM,GAAG,GAAG,IAAI,CAAC,KAAK,CAAC,MAAM,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC,CAExD,CAAC;QACF,OAAO,CAAC,GAAG,CAAC,QAAQ,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,EAAE,IAAI,CAAC,CAAC,IAAI,IAAI,CAAC,CAAC,QAAQ,IAAI,CAAC,CAAC,KAAK,CAAC,CAAC;IAC1F,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,EAAE,CAAC;IACZ,CAAC;AACH,CAAC"}
|
package/dist/retry.d.ts
ADDED
|
@@ -0,0 +1 @@
|
|
|
1
|
+
export declare function withRetry<T>(fn: () => Promise<T>, attempts?: number): Promise<T>;
|
package/dist/retry.js
ADDED
|
@@ -0,0 +1,16 @@
|
|
|
1
|
+
export async function withRetry(fn, attempts = 3) {
|
|
2
|
+
let last;
|
|
3
|
+
for (let i = 0; i < attempts; i++) {
|
|
4
|
+
try {
|
|
5
|
+
return await fn();
|
|
6
|
+
}
|
|
7
|
+
catch (err) {
|
|
8
|
+
last = err;
|
|
9
|
+
if (i < attempts - 1) {
|
|
10
|
+
await new Promise((r) => setTimeout(r, 500 * 2 ** i));
|
|
11
|
+
}
|
|
12
|
+
}
|
|
13
|
+
}
|
|
14
|
+
throw last;
|
|
15
|
+
}
|
|
16
|
+
//# sourceMappingURL=retry.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"retry.js","sourceRoot":"","sources":["../src/retry.ts"],"names":[],"mappings":"AAAA,MAAM,CAAC,KAAK,UAAU,SAAS,CAAI,EAAoB,EAAE,QAAQ,GAAG,CAAC;IACnE,IAAI,IAAa,CAAC;IAClB,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,QAAQ,EAAE,CAAC,EAAE,EAAE,CAAC;QAClC,IAAI,CAAC;YACH,OAAO,MAAM,EAAE,EAAE,CAAC;QACpB,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,IAAI,GAAG,GAAG,CAAC;YACX,IAAI,CAAC,GAAG,QAAQ,GAAG,CAAC,EAAE,CAAC;gBACrB,MAAM,IAAI,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,UAAU,CAAC,CAAC,EAAE,GAAG,GAAG,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YACxD,CAAC;QACH,CAAC;IACH,CAAC;IACD,MAAM,IAAI,CAAC;AACb,CAAC"}
|
package/dist/review.d.ts
CHANGED
|
@@ -1,3 +1,4 @@
|
|
|
1
|
+
import { type ContextEntry, type Signal } from "repocairn";
|
|
1
2
|
import { type FileDiff, type Finding, type ReviewConfig, type ReviewResult, type RunStats } from "./types.js";
|
|
2
3
|
export interface RunReviewOptions {
|
|
3
4
|
cwd: string;
|
|
@@ -18,7 +19,22 @@ export interface RunReviewOutput extends ReviewResult {
|
|
|
18
19
|
files: FileDiff[];
|
|
19
20
|
usedContext: boolean;
|
|
20
21
|
stats: RunStats;
|
|
22
|
+
/** Files that import the changed files, ranked by import count ā the "blast radius" of this PR. */
|
|
23
|
+
blastRadius: ContextEntry[];
|
|
24
|
+
/** True if the diff exceeded `huge_pr_token_threshold` and only a critical/high, split-recommending pass ran. */
|
|
25
|
+
highLevelReview: boolean;
|
|
26
|
+
/** Deterministic summary of the change, assembled from Phase A/B output before any LLM call. */
|
|
27
|
+
reviewBrief: ReviewBrief;
|
|
21
28
|
}
|
|
29
|
+
export interface ReviewBrief {
|
|
30
|
+
files: string[];
|
|
31
|
+
/** Paths of files that import the changed files (blastRadius, path-only). */
|
|
32
|
+
blastRadius: string[];
|
|
33
|
+
signals: Signal[];
|
|
34
|
+
totalTokens: number;
|
|
35
|
+
}
|
|
36
|
+
/** Sum of the same per-file token cost `chunkFiles` uses, across the whole diff. */
|
|
37
|
+
export declare function totalDiffTokens(files: FileDiff[]): number;
|
|
22
38
|
export declare function runReview(opts: RunReviewOptions): Promise<RunReviewOutput>;
|
|
23
39
|
/**
|
|
24
40
|
* Collapse findings that share a fingerprint id ā the same underlying issue
|