ShockVPN/shockbot
0
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Dont include quick links if review has no comments

Browse Source
This commit is contained in:
Colin McDonnell
2026-01-16 21:44:15 +00:00
committed by pullfrog[bot]
parent cb925556e8
commit 93d74a9bea
2 changed files with 17 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
entry
+8 -4
View File
@@ -119176,9 +119176,13 @@ function CreatePullRequestReviewTool(ctx) {
throw new Error(`createReview returned invalid data: ${JSON.stringify(result.data)}`);
}
const reviewId = result.data.id;
const apiUrl = process.env.API_URL || "https://pullfrog.com";
const fixAllUrl = `${apiUrl}/trigger/${ctx.repo.owner}/${ctx.repo.name}/${pull_number}?action=fix&review_id=${reviewId}`;
const fixApprovedUrl = `${apiUrl}/trigger/${ctx.repo.owner}/${ctx.repo.name}/${pull_number}?action=fix-approved&review_id=${reviewId}`;
const customParts = [];
if (comments.length > 0) {
const apiUrl = process.env.API_URL || "https://pullfrog.com";
const fixAllUrl = `${apiUrl}/trigger/${ctx.repo.owner}/${ctx.repo.name}/${pull_number}?action=fix&review_id=${reviewId}`;
const fixApprovedUrl = `${apiUrl}/trigger/${ctx.repo.owner}/${ctx.repo.name}/${pull_number}?action=fix-approved&review_id=${reviewId}`;
customParts.push(`[Fix all \u2794](${fixAllUrl})`, `[Fix \u{1F44D}s \u2794](${fixApprovedUrl})`);
}
const footer = buildPullfrogFooter({
workflowRun: {
owner: ctx.repo.owner,
@@ -119186,7 +119190,7 @@ function CreatePullRequestReviewTool(ctx) {
runId: ctx.runId,
jobId: ctx.jobId
},
customParts: [`[Fix all \u2794](${fixAllUrl})`, `[Fix \u{1F44D}s \u2794](${fixApprovedUrl})`]
customParts
});
const updatedBody = (body || "") + footer;
await ctx.octokit.rest.pulls.updateReview({
mcp/review.ts
+9 -4
View File
@@ -113,9 +113,14 @@ export function CreatePullRequestReviewTool(ctx: ToolContext) {
const reviewId = result.data.id;
// build quick links footer and update the review body
const apiUrl = process.env.API_URL || "https://pullfrog.com";
const fixAllUrl = `${apiUrl}/trigger/${ctx.repo.owner}/${ctx.repo.name}/${pull_number}?action=fix&review_id=${reviewId}`;
const fixApprovedUrl = `${apiUrl}/trigger/${ctx.repo.owner}/${ctx.repo.name}/${pull_number}?action=fix-approved&review_id=${reviewId}`;
// only include "Fix all" and "Fix 👍s" links if there are actual review comments
const customParts: string[] = [];
if (comments.length > 0) {
const apiUrl = process.env.API_URL || "https://pullfrog.com";
const fixAllUrl = `${apiUrl}/trigger/${ctx.repo.owner}/${ctx.repo.name}/${pull_number}?action=fix&review_id=${reviewId}`;
const fixApprovedUrl = `${apiUrl}/trigger/${ctx.repo.owner}/${ctx.repo.name}/${pull_number}?action=fix-approved&review_id=${reviewId}`;
customParts.push(`[Fix all ➔](${fixAllUrl})`, `[Fix 👍s ➔](${fixApprovedUrl})`);
}
const footer = buildPullfrogFooter({
workflowRun: {
@@ -124,7 +129,7 @@ export function CreatePullRequestReviewTool(ctx: ToolContext) {
runId: ctx.runId,
jobId: ctx.jobId,
},
customParts: [`[Fix all ➔](${fixAllUrl})`, `[Fix 👍s ➔](${fixApprovedUrl})`],
customParts,
});
const updatedBody = (body || "") + footer;