@gaunt-sloth/review 0.1.6 → 0.1.7
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/dist/modules/reviewModule.js +2 -1
- package/dist/modules/reviewModule.js.map +1 -1
- package/dist/sources/ghIssueSource.d.ts +1 -1
- package/dist/sources/ghIssueSource.js +14 -5
- package/dist/sources/ghIssueSource.js.map +1 -1
- package/dist/sources/ghPrDiffSource.d.ts +1 -1
- package/dist/sources/ghPrDiffSource.js +15 -8
- package/dist/sources/ghPrDiffSource.js.map +1 -1
- package/dist/sources/ghPrViewSource.d.ts +8 -0
- package/dist/sources/ghPrViewSource.js +63 -0
- package/dist/sources/ghPrViewSource.js.map +1 -0
- package/dist/sources/jiraIssueLegacySource.d.ts +1 -1
- package/dist/sources/jiraIssueLegacySource.js.map +1 -1
- package/package.json +3 -3
|
@@ -40,7 +40,8 @@ export async function review(source, preamble, diff, config, command = 'review',
|
|
|
40
40
|
}
|
|
41
41
|
catch (error) {
|
|
42
42
|
displayDebug(error instanceof Error ? error : String(error));
|
|
43
|
-
|
|
43
|
+
const reason = error instanceof Error ? error.message : String(error);
|
|
44
|
+
displayError(reason ? `Failed to run review with agent.\n\n${reason}` : 'Failed to run review with agent.');
|
|
44
45
|
}
|
|
45
46
|
finally {
|
|
46
47
|
await runner.cleanup();
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"reviewModule.js","sourceRoot":"","sources":["../../src/modules/reviewModule.ts"],"names":[],"mappings":"AACA,OAAO,EACL,qBAAqB,EACrB,YAAY,EACZ,YAAY,EACZ,WAAW,EACX,cAAc,EACd,cAAc,EACd,eAAe,EACf,kBAAkB,EAClB,kBAAkB,GACnB,MAAM,yCAAyC,CAAC;AACjD,OAAO,EAAE,wBAAwB,EAAE,MAAM,yBAAyB,CAAC;AACnE,OAAO,EAAE,YAAY,EAAE,aAAa,EAAE,MAAM,0BAA0B,CAAC;AACvE,OAAO,EAAE,cAAc,EAAE,MAAM,0CAA0C,CAAC;AAC1E,OAAO,EAAE,WAAW,EAAE,MAAM,sBAAsB,CAAC;AACnD,OAAO,EAAE,iBAAiB,EAAE,MAAM,8CAA8C,CAAC;AACjF,OAAO,EACL,0BAA0B,EAC1B,wBAAwB,GAEzB,MAAM,yCAAyC,CAAC;AACjD,OAAO,EAAE,cAAc,EAAE,WAAW,EAAE,MAAM,0CAA0C,CAAC;AACvF,OAAO,EAAE,WAAW,EAAE,MAAM,wCAAwC,CAAC;AAGrE,MAAM,CAAC,KAAK,UAAU,MAAM,CAC1B,MAAc,EACd,QAAgB,EAChB,IAAY,EACZ,MAAiB,EACjB,UAA2B,QAAQ,EACnC,SAA0B;IAE1B,MAAM,iBAAiB,GAAG,MAAM,CAAC,YAAY,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,iBAAiB,CAAC,YAAY,CAAC,CAAC;IAChG,MAAM,QAAQ,GAAG,CAAC,IAAI,aAAa,CAAC,QAAQ,CAAC,EAAE,IAAI,YAAY,CAAC,IAAI,CAAC,CAAC,CAAC;IAEvE,8CAA8C;IAC9C,MAAM,QAAQ,GAAG,wBAAwB,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAC1D,IAAI,QAAQ,EAAE,CAAC;QACb,kBAAkB,CAAC,QAAQ,EAAE,MAAM,CAAC,yBAAyB,CAAC,CAAC;IACjE,CAAC;IAED,MAAM,UAAU,GAAG,MAAM,CAAC,QAAQ,EAAE,CAAC,OAAO,CAAC,EAAE,MAAM,CAAC;IACtD,IAAI,UAAU,IAAI,UAAU,CAAC,OAAO,KAAK,KAAK,EAAE,CAAC;QAC/C,MAAM,cAAc,GAAG,MAAM,CAAC,UAAU,IAAI,EAAE,CAAC;QAC/C,MAAM,2BAA2B,GAAG,cAAc,CAAC,MAAM,CAAC,CAAC,EAAE,EAAE,EAAE;YAC/D,OAAO,CAAC,CACN,OAAO,EAAE,KAAK,QAAQ;gBACtB,EAAE,KAAK,IAAI;gBACX,MAAM,IAAI,EAAE;gBACX,EAAwB,CAAC,IAAI,KAAK,aAAa,CACjD,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,iFAAiF;QACjF,MAAM,oBAAoB,GAAG,MAAM,0BAA0B,CAAC,UAAU,EAAE,MAAM,CAAC,CAAC;QAClF,MAAM,CAAC,UAAU,GAAG,CAAC,GAAG,2BAA2B,EAAE,oBAAoB,CAAC,CAAC;IAC7E,CAAC;IAED,mFAAmF;IACnF,wFAAwF;IACxF,MAAM,kBAAkB,GAAmB,SAAS,IAAI;QACtD,iBAAiB,EAAE,KAAK,EAAE,UAAU,EAAE,EAAE,CAAC,UAAU,IAAI,EAAE;KAC1D,CAAC;IACF,MAAM,MAAM,GAAG,IAAI,cAAc,CAAC,qBAAqB,EAAE,kBAAkB,CAAC,CAAC;IAC7E,IAAI,CAAC;QACH,MAAM,MAAM,CAAC,IAAI,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,WAAW,EAAE,CAAC,CAAC;QACtD,MAAM,MAAM,CAAC,eAAe,CAAC,QAAQ,CAAC,CAAC;IACzC,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,YAAY,CAAC,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC;QAC7D,YAAY,CAAC,
|
|
1
|
+
{"version":3,"file":"reviewModule.js","sourceRoot":"","sources":["../../src/modules/reviewModule.ts"],"names":[],"mappings":"AACA,OAAO,EACL,qBAAqB,EACrB,YAAY,EACZ,YAAY,EACZ,WAAW,EACX,cAAc,EACd,cAAc,EACd,eAAe,EACf,kBAAkB,EAClB,kBAAkB,GACnB,MAAM,yCAAyC,CAAC;AACjD,OAAO,EAAE,wBAAwB,EAAE,MAAM,yBAAyB,CAAC;AACnE,OAAO,EAAE,YAAY,EAAE,aAAa,EAAE,MAAM,0BAA0B,CAAC;AACvE,OAAO,EAAE,cAAc,EAAE,MAAM,0CAA0C,CAAC;AAC1E,OAAO,EAAE,WAAW,EAAE,MAAM,sBAAsB,CAAC;AACnD,OAAO,EAAE,iBAAiB,EAAE,MAAM,8CAA8C,CAAC;AACjF,OAAO,EACL,0BAA0B,EAC1B,wBAAwB,GAEzB,MAAM,yCAAyC,CAAC;AACjD,OAAO,EAAE,cAAc,EAAE,WAAW,EAAE,MAAM,0CAA0C,CAAC;AACvF,OAAO,EAAE,WAAW,EAAE,MAAM,wCAAwC,CAAC;AAGrE,MAAM,CAAC,KAAK,UAAU,MAAM,CAC1B,MAAc,EACd,QAAgB,EAChB,IAAY,EACZ,MAAiB,EACjB,UAA2B,QAAQ,EACnC,SAA0B;IAE1B,MAAM,iBAAiB,GAAG,MAAM,CAAC,YAAY,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,iBAAiB,CAAC,YAAY,CAAC,CAAC;IAChG,MAAM,QAAQ,GAAG,CAAC,IAAI,aAAa,CAAC,QAAQ,CAAC,EAAE,IAAI,YAAY,CAAC,IAAI,CAAC,CAAC,CAAC;IAEvE,8CAA8C;IAC9C,MAAM,QAAQ,GAAG,wBAAwB,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAC1D,IAAI,QAAQ,EAAE,CAAC;QACb,kBAAkB,CAAC,QAAQ,EAAE,MAAM,CAAC,yBAAyB,CAAC,CAAC;IACjE,CAAC;IAED,MAAM,UAAU,GAAG,MAAM,CAAC,QAAQ,EAAE,CAAC,OAAO,CAAC,EAAE,MAAM,CAAC;IACtD,IAAI,UAAU,IAAI,UAAU,CAAC,OAAO,KAAK,KAAK,EAAE,CAAC;QAC/C,MAAM,cAAc,GAAG,MAAM,CAAC,UAAU,IAAI,EAAE,CAAC;QAC/C,MAAM,2BAA2B,GAAG,cAAc,CAAC,MAAM,CAAC,CAAC,EAAE,EAAE,EAAE;YAC/D,OAAO,CAAC,CACN,OAAO,EAAE,KAAK,QAAQ;gBACtB,EAAE,KAAK,IAAI;gBACX,MAAM,IAAI,EAAE;gBACX,EAAwB,CAAC,IAAI,KAAK,aAAa,CACjD,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,iFAAiF;QACjF,MAAM,oBAAoB,GAAG,MAAM,0BAA0B,CAAC,UAAU,EAAE,MAAM,CAAC,CAAC;QAClF,MAAM,CAAC,UAAU,GAAG,CAAC,GAAG,2BAA2B,EAAE,oBAAoB,CAAC,CAAC;IAC7E,CAAC;IAED,mFAAmF;IACnF,wFAAwF;IACxF,MAAM,kBAAkB,GAAmB,SAAS,IAAI;QACtD,iBAAiB,EAAE,KAAK,EAAE,UAAU,EAAE,EAAE,CAAC,UAAU,IAAI,EAAE;KAC1D,CAAC;IACF,MAAM,MAAM,GAAG,IAAI,cAAc,CAAC,qBAAqB,EAAE,kBAAkB,CAAC,CAAC;IAC7E,IAAI,CAAC;QACH,MAAM,MAAM,CAAC,IAAI,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,WAAW,EAAE,CAAC,CAAC;QACtD,MAAM,MAAM,CAAC,eAAe,CAAC,QAAQ,CAAC,CAAC;IACzC,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,YAAY,CAAC,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC;QAC7D,MAAM,MAAM,GAAG,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;QACtE,YAAY,CACV,MAAM,CAAC,CAAC,CAAC,uCAAuC,MAAM,EAAE,CAAC,CAAC,CAAC,kCAAkC,CAC9F,CAAC;IACJ,CAAC;YAAS,CAAC;QACT,MAAM,MAAM,CAAC,OAAO,EAAE,CAAC;IACzB,CAAC;IAED,iBAAiB,EAAE,IAAI,EAAE,CAAC;IAE1B,kBAAkB,CAAC,UAAU,EAAE,OAAO,CAAC,CAAC;IAExC,yCAAyC;IACzC,IAAI,QAAQ,EAAE,CAAC;QACb,IAAI,CAAC;YACH,eAAe,EAAE,CAAC;YAClB,kBAAkB,EAAE,CAAC;YACrB,cAAc,CAAC,mCAAmC,QAAQ,EAAE,CAAC,CAAC;QAChE,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,YAAY,CAAC,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC;YAC7D,YAAY,CAAC,mCAAmC,QAAQ,EAAE,CAAC,CAAC;QAC9D,CAAC;IACH,CAAC;IAED,cAAc,CAAC,wBAAwB,CAAC,CAAC;AAC3C,CAAC;AAED,SAAS,kBAAkB,CAAC,UAAoC,EAAE,OAAwB;IACxF,IAAI,CAAC,UAAU,IAAI,UAAU,CAAC,OAAO,KAAK,KAAK,EAAE,CAAC;QAChD,mDAAmD;QACnD,OAAO;IACT,CAAC;IAED,MAAM,MAAM,GAAG,WAAW,CAAuB,wBAAwB,CAAC,CAAC;IAC3E,IAAI,CAAC,MAAM,EAAE,CAAC;QACZ,cAAc,CAAC,gDAAgD,OAAO,WAAW,CAAC,CAAC;QACnF,WAAW,CAAC,CAAC,CAAC,CAAC,CAAC,4EAA4E;QAC5F,OAAO;IACT,CAAC;IAED,MAAM,SAAS,GAAG,UAAU,CAAC,aAAa,IAAI,MAAM,CAAC,aAAa,CAAC;IACnE,MAAM,SAAS,GAAG,UAAU,CAAC,SAAS,IAAI,MAAM,CAAC,SAAS,CAAC;IAC3D,MAAM,WAAW,GAAG,GAAG,MAAM,CAAC,IAAI,IAAI,SAAS,gBAAgB,SAAS,GAAG,CAAC;IAC5E,WAAW,CAAC,iBAAiB,CAAC,CAAC;IAE/B,IAAI,MAAM,CAAC,IAAI,IAAI,SAAS,EAAE,CAAC;QAC7B,cAAc,CAAC,QAAQ,WAAW,EAAE,CAAC,CAAC;IACxC,CAAC;SAAM,CAAC;QACN,YAAY,CAAC,QAAQ,WAAW,EAAE,CAAC,CAAC;QACpC,IAAI,UAAU,CAAC,iBAAiB,IAAI,IAAI,EAAE,CAAC;YACzC,WAAW,CAAC,CAAC,CAAC,CAAC;QACjB,CAAC;IACH,CAAC;IAED,IAAI,MAAM,CAAC,OAAO,EAAE,CAAC;QACnB,WAAW,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC;IAC9B,CAAC;AACH,CAAC"}
|
|
@@ -2,7 +2,7 @@ import type { ProviderConfig } from './types.js';
|
|
|
2
2
|
/**
|
|
3
3
|
* Gets GitHub issue using GitHub CLI
|
|
4
4
|
* @param _ config (unused in this source)
|
|
5
|
-
* @param issueId GitHub issue number
|
|
5
|
+
* @param issueId GitHub issue number or full issue URL (for issues in other repositories)
|
|
6
6
|
* @returns GitHub issue content or null if not found
|
|
7
7
|
*/
|
|
8
8
|
export declare function get(_: ProviderConfig | null, issueId: string | undefined): Promise<string | null>;
|
|
@@ -1,10 +1,14 @@
|
|
|
1
1
|
import { displayWarning } from '@gaunt-sloth/core/utils/consoleUtils.js';
|
|
2
2
|
import { execAsync } from '@gaunt-sloth/core/utils/systemUtils.js';
|
|
3
3
|
import { ProgressIndicator } from '@gaunt-sloth/core/utils/ProgressIndicator.js';
|
|
4
|
+
// Either a bare issue number or a full issue URL. The strict shape matters because the value
|
|
5
|
+
// is interpolated into a shell command; URLs with a loose charset could smuggle shell
|
|
6
|
+
// metacharacters (issue references may originate from untrusted PR descriptions).
|
|
7
|
+
const ISSUE_REF_PATTERN = /^(?:\d+|https:\/\/github\.com\/[\w.-]+\/[\w.-]+\/issues\/\d+)$/;
|
|
4
8
|
/**
|
|
5
9
|
* Gets GitHub issue using GitHub CLI
|
|
6
10
|
* @param _ config (unused in this source)
|
|
7
|
-
* @param issueId GitHub issue number
|
|
11
|
+
* @param issueId GitHub issue number or full issue URL (for issues in other repositories)
|
|
8
12
|
* @returns GitHub issue content or null if not found
|
|
9
13
|
*/
|
|
10
14
|
export async function get(_, issueId) {
|
|
@@ -12,20 +16,25 @@ export async function get(_, issueId) {
|
|
|
12
16
|
displayWarning('No GitHub issue number provided');
|
|
13
17
|
return null;
|
|
14
18
|
}
|
|
19
|
+
if (!ISSUE_REF_PATTERN.test(issueId)) {
|
|
20
|
+
displayWarning(`Invalid GitHub issue reference "${issueId}"; expected an issue number or a full https://github.com/<owner>/<repo>/issues/<number> URL.`);
|
|
21
|
+
return null;
|
|
22
|
+
}
|
|
23
|
+
const issueLabel = /^\d+$/.test(issueId) ? `#${issueId}` : issueId;
|
|
15
24
|
try {
|
|
16
25
|
// Use the GitHub CLI to fetch issue details
|
|
17
|
-
const progress = new ProgressIndicator(`Fetching GitHub issue
|
|
26
|
+
const progress = new ProgressIndicator(`Fetching GitHub issue ${issueLabel}`);
|
|
18
27
|
const issueContent = await execAsync(`gh issue view ${issueId}`);
|
|
19
28
|
progress.stop();
|
|
20
29
|
if (!issueContent) {
|
|
21
|
-
displayWarning(`No content found for GitHub issue
|
|
30
|
+
displayWarning(`No content found for GitHub issue ${issueLabel}`);
|
|
22
31
|
return null;
|
|
23
32
|
}
|
|
24
|
-
return `GitHub Issue:
|
|
33
|
+
return `GitHub Issue: ${issueLabel}\n\n${issueContent}`;
|
|
25
34
|
}
|
|
26
35
|
catch (error) {
|
|
27
36
|
displayWarning(`
|
|
28
|
-
Failed to get GitHub issue
|
|
37
|
+
Failed to get GitHub issue ${issueLabel}: ${error instanceof Error ? error.message : String(error)}
|
|
29
38
|
Consider checking if gh cli (https://cli.github.com/) is installed and authenticated.
|
|
30
39
|
`);
|
|
31
40
|
return null;
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"ghIssueSource.js","sourceRoot":"","sources":["../../src/sources/ghIssueSource.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,cAAc,EAAE,MAAM,yCAAyC,CAAC;AAEzE,OAAO,EAAE,SAAS,EAAE,MAAM,wCAAwC,CAAC;AACnE,OAAO,EAAE,iBAAiB,EAAE,MAAM,8CAA8C,CAAC;AAEjF;;;;;GAKG;AACH,MAAM,CAAC,KAAK,UAAU,GAAG,CACvB,CAAwB,EACxB,OAA2B;IAE3B,IAAI,CAAC,OAAO,EAAE,CAAC;QACb,cAAc,CAAC,iCAAiC,CAAC,CAAC;QAClD,OAAO,IAAI,CAAC;IACd,CAAC;IAED,IAAI,CAAC;QACH,4CAA4C;QAC5C,MAAM,QAAQ,GAAG,IAAI,iBAAiB,CAAC,
|
|
1
|
+
{"version":3,"file":"ghIssueSource.js","sourceRoot":"","sources":["../../src/sources/ghIssueSource.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,cAAc,EAAE,MAAM,yCAAyC,CAAC;AAEzE,OAAO,EAAE,SAAS,EAAE,MAAM,wCAAwC,CAAC;AACnE,OAAO,EAAE,iBAAiB,EAAE,MAAM,8CAA8C,CAAC;AAEjF,6FAA6F;AAC7F,sFAAsF;AACtF,kFAAkF;AAClF,MAAM,iBAAiB,GAAG,gEAAgE,CAAC;AAE3F;;;;;GAKG;AACH,MAAM,CAAC,KAAK,UAAU,GAAG,CACvB,CAAwB,EACxB,OAA2B;IAE3B,IAAI,CAAC,OAAO,EAAE,CAAC;QACb,cAAc,CAAC,iCAAiC,CAAC,CAAC;QAClD,OAAO,IAAI,CAAC;IACd,CAAC;IAED,IAAI,CAAC,iBAAiB,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC;QACrC,cAAc,CACZ,mCAAmC,OAAO,8FAA8F,CACzI,CAAC;QACF,OAAO,IAAI,CAAC;IACd,CAAC;IAED,MAAM,UAAU,GAAG,OAAO,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,IAAI,OAAO,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC;IAEnE,IAAI,CAAC;QACH,4CAA4C;QAC5C,MAAM,QAAQ,GAAG,IAAI,iBAAiB,CAAC,yBAAyB,UAAU,EAAE,CAAC,CAAC;QAC9E,MAAM,YAAY,GAAG,MAAM,SAAS,CAAC,iBAAiB,OAAO,EAAE,CAAC,CAAC;QACjE,QAAQ,CAAC,IAAI,EAAE,CAAC;QAEhB,IAAI,CAAC,YAAY,EAAE,CAAC;YAClB,cAAc,CAAC,qCAAqC,UAAU,EAAE,CAAC,CAAC;YAClE,OAAO,IAAI,CAAC;QACd,CAAC;QAED,OAAO,iBAAiB,UAAU,OAAO,YAAY,EAAE,CAAC;IAC1D,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,cAAc,CAAC;6BACU,UAAU,KAAK,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC;;KAE7F,CAAC,CAAC;QACH,OAAO,IAAI,CAAC;IACd,CAAC;AACH,CAAC"}
|
|
@@ -2,7 +2,7 @@ import type { ProviderConfig } from './types.js';
|
|
|
2
2
|
/**
|
|
3
3
|
* Gets PR diff using GitHub CLI
|
|
4
4
|
* @param _ config (unused in this source)
|
|
5
|
-
* @param prId GitHub PR number
|
|
5
|
+
* @param prId GitHub PR number. When omitted, GitHub CLI resolves the PR for the current branch.
|
|
6
6
|
* @returns GitHub PR diff content or null if not found
|
|
7
7
|
*/
|
|
8
8
|
export declare function get(_: ProviderConfig | null, prId: string | undefined): Promise<string | null>;
|
|
@@ -1,28 +1,35 @@
|
|
|
1
1
|
import { execAsync } from '@gaunt-sloth/core/utils/systemUtils.js';
|
|
2
2
|
import { ProgressIndicator } from '@gaunt-sloth/core/utils/ProgressIndicator.js';
|
|
3
|
+
import { displayWarning } from '@gaunt-sloth/core/utils/consoleUtils.js';
|
|
4
|
+
// PR IDs may be supplied by LLM-driven tools and are interpolated into a shell command.
|
|
5
|
+
// Keep the accepted shape strict to prevent shell metacharacters from reaching execAsync.
|
|
6
|
+
const PR_ID_PATTERN = /^\d+$/;
|
|
3
7
|
/**
|
|
4
8
|
* Gets PR diff using GitHub CLI
|
|
5
9
|
* @param _ config (unused in this source)
|
|
6
|
-
* @param prId GitHub PR number
|
|
10
|
+
* @param prId GitHub PR number. When omitted, GitHub CLI resolves the PR for the current branch.
|
|
7
11
|
* @returns GitHub PR diff content or null if not found
|
|
8
12
|
*/
|
|
9
13
|
export async function get(_, prId) {
|
|
10
|
-
if (!prId) {
|
|
11
|
-
|
|
14
|
+
if (prId && !PR_ID_PATTERN.test(prId)) {
|
|
15
|
+
displayWarning(`Invalid GitHub PR number "${prId}"; expected a numeric string.`);
|
|
16
|
+
return null;
|
|
12
17
|
}
|
|
18
|
+
const prLabel = prId ? `#${prId}` : 'for current branch';
|
|
19
|
+
const ghCommand = prId ? `gh pr diff ${prId}` : 'gh pr diff';
|
|
13
20
|
// Use the GitHub CLI to fetch PR diff
|
|
14
|
-
const progress = new ProgressIndicator(`Fetching GitHub PR
|
|
21
|
+
const progress = new ProgressIndicator(`Fetching GitHub PR ${prLabel} diff`);
|
|
15
22
|
try {
|
|
16
|
-
const prDiffContent = await execAsync(
|
|
23
|
+
const prDiffContent = await execAsync(ghCommand);
|
|
17
24
|
progress.stop();
|
|
18
25
|
if (!prDiffContent) {
|
|
19
|
-
throw new Error(`No diff content found for GitHub PR
|
|
26
|
+
throw new Error(`No diff content found for GitHub PR ${prLabel}`);
|
|
20
27
|
}
|
|
21
|
-
return `GitHub PR Diff:
|
|
28
|
+
return `GitHub PR Diff: ${prLabel}\n\n${prDiffContent}`;
|
|
22
29
|
}
|
|
23
30
|
catch (error) {
|
|
24
31
|
progress.stop();
|
|
25
|
-
throw new Error(`Failed to get GitHub PR diff
|
|
32
|
+
throw new Error(`Failed to get GitHub PR diff ${prLabel}: ${error instanceof Error ? error.message : String(error)}
|
|
26
33
|
Consider checking if gh cli (https://cli.github.com/) is installed and authenticated.`);
|
|
27
34
|
}
|
|
28
35
|
}
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"ghPrDiffSource.js","sourceRoot":"","sources":["../../src/sources/ghPrDiffSource.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,SAAS,EAAE,MAAM,wCAAwC,CAAC;AACnE,OAAO,EAAE,iBAAiB,EAAE,MAAM,8CAA8C,CAAC;
|
|
1
|
+
{"version":3,"file":"ghPrDiffSource.js","sourceRoot":"","sources":["../../src/sources/ghPrDiffSource.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,SAAS,EAAE,MAAM,wCAAwC,CAAC;AACnE,OAAO,EAAE,iBAAiB,EAAE,MAAM,8CAA8C,CAAC;AACjF,OAAO,EAAE,cAAc,EAAE,MAAM,yCAAyC,CAAC;AAEzE,wFAAwF;AACxF,0FAA0F;AAC1F,MAAM,aAAa,GAAG,OAAO,CAAC;AAE9B;;;;;GAKG;AACH,MAAM,CAAC,KAAK,UAAU,GAAG,CACvB,CAAwB,EACxB,IAAwB;IAExB,IAAI,IAAI,IAAI,CAAC,aAAa,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;QACtC,cAAc,CAAC,6BAA6B,IAAI,+BAA+B,CAAC,CAAC;QACjF,OAAO,IAAI,CAAC;IACd,CAAC;IAED,MAAM,OAAO,GAAG,IAAI,CAAC,CAAC,CAAC,IAAI,IAAI,EAAE,CAAC,CAAC,CAAC,oBAAoB,CAAC;IACzD,MAAM,SAAS,GAAG,IAAI,CAAC,CAAC,CAAC,cAAc,IAAI,EAAE,CAAC,CAAC,CAAC,YAAY,CAAC;IAE7D,sCAAsC;IACtC,MAAM,QAAQ,GAAG,IAAI,iBAAiB,CAAC,sBAAsB,OAAO,OAAO,CAAC,CAAC;IAC7E,IAAI,CAAC;QACH,MAAM,aAAa,GAAG,MAAM,SAAS,CAAC,SAAS,CAAC,CAAC;QACjD,QAAQ,CAAC,IAAI,EAAE,CAAC;QAEhB,IAAI,CAAC,aAAa,EAAE,CAAC;YACnB,MAAM,IAAI,KAAK,CAAC,uCAAuC,OAAO,EAAE,CAAC,CAAC;QACpE,CAAC;QAED,OAAO,mBAAmB,OAAO,OAAO,aAAa,EAAE,CAAC;IAC1D,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,QAAQ,CAAC,IAAI,EAAE,CAAC;QAChB,MAAM,IAAI,KAAK,CAAC,gCAAgC,OAAO,KAAK,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC;sFAChC,CAAC,CAAC;IACtF,CAAC;AACH,CAAC"}
|
|
@@ -0,0 +1,8 @@
|
|
|
1
|
+
import type { ProviderConfig } from './types.js';
|
|
2
|
+
/**
|
|
3
|
+
* Gets GitHub PR metadata using GitHub CLI.
|
|
4
|
+
* @param _ config (unused in this source)
|
|
5
|
+
* @param prId GitHub PR number. When omitted, GitHub CLI resolves the PR for the current branch.
|
|
6
|
+
* @returns GitHub PR metadata or null if not found
|
|
7
|
+
*/
|
|
8
|
+
export declare function get(_: ProviderConfig | null, prId: string | undefined): Promise<string | null>;
|
|
@@ -0,0 +1,63 @@
|
|
|
1
|
+
import { execAsync } from '@gaunt-sloth/core/utils/systemUtils.js';
|
|
2
|
+
import { ProgressIndicator } from '@gaunt-sloth/core/utils/ProgressIndicator.js';
|
|
3
|
+
import { debugLog } from '@gaunt-sloth/core/utils/debugUtils.js';
|
|
4
|
+
import { displayWarning } from '@gaunt-sloth/core/utils/consoleUtils.js';
|
|
5
|
+
// PR IDs may be supplied by LLM-driven tools and are interpolated into a shell command.
|
|
6
|
+
// Keep the accepted shape strict to prevent shell metacharacters from reaching execAsync.
|
|
7
|
+
const PR_ID_PATTERN = /^\d+$/;
|
|
8
|
+
/**
|
|
9
|
+
* Gets GitHub PR metadata using GitHub CLI.
|
|
10
|
+
* @param _ config (unused in this source)
|
|
11
|
+
* @param prId GitHub PR number. When omitted, GitHub CLI resolves the PR for the current branch.
|
|
12
|
+
* @returns GitHub PR metadata or null if not found
|
|
13
|
+
*/
|
|
14
|
+
export async function get(_, prId) {
|
|
15
|
+
if (prId && !PR_ID_PATTERN.test(prId)) {
|
|
16
|
+
displayWarning(`Invalid GitHub PR number "${prId}"; expected a numeric string.`);
|
|
17
|
+
return null;
|
|
18
|
+
}
|
|
19
|
+
const prLabel = prId ? `#${prId}` : 'for current branch';
|
|
20
|
+
const ghCommand = prId
|
|
21
|
+
? `gh pr view ${prId} --json number,title,body,url,headRefName,baseRefName`
|
|
22
|
+
: 'gh pr view --json number,title,body,url,headRefName,baseRefName';
|
|
23
|
+
const progress = new ProgressIndicator(`Fetching GitHub PR ${prLabel} metadata`);
|
|
24
|
+
try {
|
|
25
|
+
const prViewJson = await execAsync(ghCommand);
|
|
26
|
+
progress.stop();
|
|
27
|
+
if (!prViewJson) {
|
|
28
|
+
throw new Error(`No metadata found for GitHub PR ${prLabel}`);
|
|
29
|
+
}
|
|
30
|
+
let prView;
|
|
31
|
+
try {
|
|
32
|
+
prView = JSON.parse(prViewJson);
|
|
33
|
+
}
|
|
34
|
+
catch (parseError) {
|
|
35
|
+
// gh can prepend warning lines to otherwise valid output; surface the raw output for
|
|
36
|
+
// debugging since the rethrown message only carries the SyntaxError text.
|
|
37
|
+
debugLog(`Failed to parse gh pr view output as JSON:\n${prViewJson}`);
|
|
38
|
+
throw parseError;
|
|
39
|
+
}
|
|
40
|
+
return formatPrView(prView, prLabel);
|
|
41
|
+
}
|
|
42
|
+
catch (error) {
|
|
43
|
+
progress.stop();
|
|
44
|
+
throw new Error(`Failed to get GitHub PR ${prLabel} metadata: ${error instanceof Error ? error.message : String(error)}
|
|
45
|
+
Consider checking if gh cli (https://cli.github.com/) is installed and authenticated.`);
|
|
46
|
+
}
|
|
47
|
+
}
|
|
48
|
+
function formatPrView(prView, fallbackLabel) {
|
|
49
|
+
const label = prView.number ? `#${prView.number}` : fallbackLabel;
|
|
50
|
+
return [
|
|
51
|
+
`GitHub PR: ${label}`,
|
|
52
|
+
prView.title ? `Title: ${prView.title}` : undefined,
|
|
53
|
+
prView.url ? `URL: ${prView.url}` : undefined,
|
|
54
|
+
prView.headRefName ? `Head branch: ${prView.headRefName}` : undefined,
|
|
55
|
+
prView.baseRefName ? `Base branch: ${prView.baseRefName}` : undefined,
|
|
56
|
+
'',
|
|
57
|
+
'Description:',
|
|
58
|
+
prView.body || '',
|
|
59
|
+
]
|
|
60
|
+
.filter((line) => line !== undefined)
|
|
61
|
+
.join('\n');
|
|
62
|
+
}
|
|
63
|
+
//# sourceMappingURL=ghPrViewSource.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"ghPrViewSource.js","sourceRoot":"","sources":["../../src/sources/ghPrViewSource.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,SAAS,EAAE,MAAM,wCAAwC,CAAC;AACnE,OAAO,EAAE,iBAAiB,EAAE,MAAM,8CAA8C,CAAC;AACjF,OAAO,EAAE,QAAQ,EAAE,MAAM,uCAAuC,CAAC;AACjE,OAAO,EAAE,cAAc,EAAE,MAAM,yCAAyC,CAAC;AAEzE,wFAAwF;AACxF,0FAA0F;AAC1F,MAAM,aAAa,GAAG,OAAO,CAAC;AAW9B;;;;;GAKG;AACH,MAAM,CAAC,KAAK,UAAU,GAAG,CACvB,CAAwB,EACxB,IAAwB;IAExB,IAAI,IAAI,IAAI,CAAC,aAAa,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;QACtC,cAAc,CAAC,6BAA6B,IAAI,+BAA+B,CAAC,CAAC;QACjF,OAAO,IAAI,CAAC;IACd,CAAC;IAED,MAAM,OAAO,GAAG,IAAI,CAAC,CAAC,CAAC,IAAI,IAAI,EAAE,CAAC,CAAC,CAAC,oBAAoB,CAAC;IACzD,MAAM,SAAS,GAAG,IAAI;QACpB,CAAC,CAAC,cAAc,IAAI,uDAAuD;QAC3E,CAAC,CAAC,iEAAiE,CAAC;IAEtE,MAAM,QAAQ,GAAG,IAAI,iBAAiB,CAAC,sBAAsB,OAAO,WAAW,CAAC,CAAC;IACjF,IAAI,CAAC;QACH,MAAM,UAAU,GAAG,MAAM,SAAS,CAAC,SAAS,CAAC,CAAC;QAC9C,QAAQ,CAAC,IAAI,EAAE,CAAC;QAEhB,IAAI,CAAC,UAAU,EAAE,CAAC;YAChB,MAAM,IAAI,KAAK,CAAC,mCAAmC,OAAO,EAAE,CAAC,CAAC;QAChE,CAAC;QAED,IAAI,MAAwB,CAAC;QAC7B,IAAI,CAAC;YACH,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,UAAU,CAAqB,CAAC;QACtD,CAAC;QAAC,OAAO,UAAU,EAAE,CAAC;YACpB,qFAAqF;YACrF,0EAA0E;YAC1E,QAAQ,CAAC,+CAA+C,UAAU,EAAE,CAAC,CAAC;YACtE,MAAM,UAAU,CAAC;QACnB,CAAC;QACD,OAAO,YAAY,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IACvC,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,QAAQ,CAAC,IAAI,EAAE,CAAC;QAChB,MAAM,IAAI,KAAK,CAAC,2BAA2B,OAAO,cAAc,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC;sFACpC,CAAC,CAAC;IACtF,CAAC;AACH,CAAC;AAED,SAAS,YAAY,CAAC,MAAwB,EAAE,aAAqB;IACnE,MAAM,KAAK,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,IAAI,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC,aAAa,CAAC;IAClE,OAAO;QACL,cAAc,KAAK,EAAE;QACrB,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,UAAU,MAAM,CAAC,KAAK,EAAE,CAAC,CAAC,CAAC,SAAS;QACnD,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,QAAQ,MAAM,CAAC,GAAG,EAAE,CAAC,CAAC,CAAC,SAAS;QAC7C,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,gBAAgB,MAAM,CAAC,WAAW,EAAE,CAAC,CAAC,CAAC,SAAS;QACrE,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,gBAAgB,MAAM,CAAC,WAAW,EAAE,CAAC,CAAC,CAAC,SAAS;QACrE,EAAE;QACF,cAAc;QACd,MAAM,CAAC,IAAI,IAAI,EAAE;KAClB;SACE,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,KAAK,SAAS,CAAC;SACpC,IAAI,CAAC,IAAI,CAAC,CAAC;AAChB,CAAC"}
|
|
@@ -5,4 +5,4 @@ import type { JiraLegacyConfig } from '#src/sources/types.js';
|
|
|
5
5
|
* @param issueId Jira issue ID
|
|
6
6
|
* @returns Jira issue content
|
|
7
7
|
*/
|
|
8
|
-
export declare function get(config: Partial<JiraLegacyConfig
|
|
8
|
+
export declare function get(config: Partial<JiraLegacyConfig> | null, issueId: string | undefined): Promise<string | null>;
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"jiraIssueLegacySource.js","sourceRoot":"","sources":["../../src/sources/jiraIssueLegacySource.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,OAAO,EAAE,YAAY,EAAE,cAAc,EAAE,MAAM,yCAAyC,CAAC;AAChG,OAAO,EAAE,GAAG,EAAE,MAAM,wCAAwC,CAAC;AAY7D;;;;;GAKG;AACH,MAAM,CAAC,KAAK,UAAU,GAAG,CACvB,
|
|
1
|
+
{"version":3,"file":"jiraIssueLegacySource.js","sourceRoot":"","sources":["../../src/sources/jiraIssueLegacySource.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,OAAO,EAAE,YAAY,EAAE,cAAc,EAAE,MAAM,yCAAyC,CAAC;AAChG,OAAO,EAAE,GAAG,EAAE,MAAM,wCAAwC,CAAC;AAY7D;;;;;GAKG;AACH,MAAM,CAAC,KAAK,UAAU,GAAG,CACvB,MAAwC,EACxC,OAA2B;IAE3B,IAAI,CAAC,MAAM,EAAE,CAAC;QACZ,cAAc,CAAC,yBAAyB,CAAC,CAAC;QAC1C,OAAO,IAAI,CAAC;IACd,CAAC;IACD,IAAI,CAAC,OAAO,EAAE,CAAC;QACb,cAAc,CAAC,sBAAsB,CAAC,CAAC;QACvC,OAAO,IAAI,CAAC;IACd,CAAC;IACD,IAAI,CAAC,MAAM,CAAC,OAAO,EAAE,CAAC;QACpB,cAAc,CAAC,2BAA2B,CAAC,CAAC;QAC5C,OAAO,IAAI,CAAC;IACd,CAAC;IAED,mDAAmD;IACnD,MAAM,QAAQ,GAAG,GAAG,CAAC,aAAa,IAAI,MAAM,CAAC,QAAQ,CAAC;IACtD,IAAI,CAAC,QAAQ,EAAE,CAAC;QACd,MAAM,IAAI,KAAK,CACb,sHAAsH,CACvH,CAAC;IACJ,CAAC;IAED,gDAAgD;IAChD,MAAM,KAAK,GAAG,GAAG,CAAC,qBAAqB,IAAI,MAAM,CAAC,KAAK,CAAC;IACxD,IAAI,CAAC,KAAK,EAAE,CAAC;QACX,MAAM,IAAI,KAAK,CACb,uIAAuI,CACxI,CAAC;IACJ,CAAC;IAED,IAAI,CAAC;QACH,MAAM,KAAK,GAAG,MAAM,YAAY,CAC9B;YACE,GAAI,MAA2B;YAC/B,QAAQ;YACR,KAAK;SACN,EACD,OAAO,CACR,CAAC;QACF,IAAI,CAAC,KAAK,EAAE,CAAC;YACX,OAAO,IAAI,CAAC;QACd,CAAC;QAED,MAAM,OAAO,GAAG,KAAK,CAAC,MAAM,CAAC,OAAO,CAAC;QACrC,MAAM,WAAW,GAAG,KAAK,CAAC,MAAM,CAAC,WAAW,CAAC;QAE7C,OAAO,eAAe,OAAO,cAAc,OAAO,qBAAqB,WAAW,EAAE,CAAC;IACvF,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,YAAY,CACV,6BAA6B,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE,CACtF,CAAC;QACF,OAAO,IAAI,CAAC;IACd,CAAC;AACH,CAAC;AAED;;;;;GAKG;AACH,KAAK,UAAU,YAAY,CAAC,MAAwB,EAAE,OAAe;IACnE,MAAM,IAAI,GAAG,MAAM,CAAC,IAAI,CAAC,GAAG,MAAM,CAAC,QAAQ,IAAI,MAAM,CAAC,KAAK,EAAE,CAAC,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC;IAClF,MAAM,GAAG,GAAG,GAAG,MAAM,CAAC,OAAO,GAAG,OAAO,EAAE,CAAC;IAC1C,IAAI,MAAM,CAAC,UAAU,EAAE,CAAC;QACtB,OAAO,CAAC,sBAAsB,MAAM,CAAC,UAAU,GAAG,OAAO,EAAE,CAAC,CAAC;IAC/D,CAAC;IACD,OAAO,CAAC,4BAA4B,GAAG,CAAC,OAAO,CAAC,cAAc,EAAE,EAAE,CAAC,EAAE,CAAC,CAAC;IACvE,MAAM,QAAQ,GAAG,MAAM,KAAK,CAAC,GAAG,EAAE;QAChC,OAAO,EAAE;YACP,aAAa,EAAE,SAAS,IAAI,EAAE;YAC9B,MAAM,EAAE,kBAAkB;SAC3B;KACF,CAAC,CAAC;IAEH,IAAI,CAAC,QAAQ,CAAC,EAAE,EAAE,CAAC;QACjB,MAAM,IAAI,KAAK,CAAC,mCAAmC,GAAG,iBAAiB,QAAQ,CAAC,UAAU,EAAE,CAAC,CAAC;IAChG,CAAC;IAED,OAAO,QAAQ,CAAC,IAAI,EAAE,CAAC;AACzB,CAAC"}
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@gaunt-sloth/review",
|
|
3
|
-
"version": "0.1.
|
|
3
|
+
"version": "0.1.7",
|
|
4
4
|
"description": "Review functionality for Gaunt Sloth",
|
|
5
5
|
"license": "MIT",
|
|
6
6
|
"author": "Andrew Kondratev",
|
|
@@ -34,10 +34,10 @@
|
|
|
34
34
|
"#src/*.js": "./dist/*.js"
|
|
35
35
|
},
|
|
36
36
|
"dependencies": {
|
|
37
|
-
"@gaunt-sloth/core": "0.1.
|
|
37
|
+
"@gaunt-sloth/core": "0.1.7"
|
|
38
38
|
},
|
|
39
39
|
"peerDependencies": {
|
|
40
|
-
"@gaunt-sloth/tools": "0.1.
|
|
40
|
+
"@gaunt-sloth/tools": "0.1.7"
|
|
41
41
|
},
|
|
42
42
|
"peerDependenciesMeta": {
|
|
43
43
|
"@gaunt-sloth/tools": {
|