Compare commits
2 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
 Colin McDonnell
|
3724572346 | ||
|
Colin McDonnell
|
6b79fd4e29 |
@@ -223,5 +223,9 @@ The following is structured data about the GitHub event that triggered this run
|
|||||||
${encodedEvent}`
|
${encodedEvent}`
|
||||||
: ""
|
: ""
|
||||||
}
|
}
|
||||||
|
|
||||||
|
************* RUNTIME CONTEXT *************
|
||||||
|
|
||||||
|
working_directory: ${process.cwd()}
|
||||||
`;
|
`;
|
||||||
};
|
};
|
||||||
|
|||||||
@@ -97443,7 +97443,7 @@ function query({
|
|||||||
// package.json
|
// package.json
|
||||||
var package_default = {
|
var package_default = {
|
||||||
name: "@pullfrog/action",
|
name: "@pullfrog/action",
|
||||||
version: "0.0.133",
|
version: "0.0.134",
|
||||||
type: "module",
|
type: "module",
|
||||||
files: [
|
files: [
|
||||||
"index.js",
|
"index.js",
|
||||||
@@ -97972,12 +97972,17 @@ ${disableProgressComment ? "" : `
|
|||||||
prompt: `Follow these steps:
|
prompt: `Follow these steps:
|
||||||
1. Get PR info with ${ghPullfrogMcpName}/get_pull_request (this automatically prepares the repository by fetching and checking out the PR branch)
|
1. Get PR info with ${ghPullfrogMcpName}/get_pull_request (this automatically prepares the repository by fetching and checking out the PR branch)
|
||||||
|
|
||||||
2. View diff: git diff origin/<base>...origin/<head> (use line numbers from this for inline comments, replace <base> and <head> with 'base' and 'head' from PR info)
|
2. View diff: \`git diff origin/<base>...origin/<head>\` (replace <base> and <head> with 'base' and 'head' from PR info)
|
||||||
|
|
||||||
3. Read files from the checked-out PR branch to understand the implementation
|
3. Read files from the checked-out PR branch to understand the implementation. Always use **relative paths** from repo root (e.g., \`src/index.ts\`), never absolute paths.
|
||||||
|
|
||||||
4. Submit review using ${ghPullfrogMcpName}/submit_pull_request_review
|
4. Submit review using ${ghPullfrogMcpName}/submit_pull_request_review
|
||||||
|
|
||||||
|
**CRITICAL: File paths and line numbers for inline comments**
|
||||||
|
- Use **relative paths** from repo root (e.g., \`packages/core/src/utils.ts\`)
|
||||||
|
- For line numbers, use the NEW file line number from the diff (shown after \`+\` in hunk headers like \`@@ -10,5 +12,8 @@\` means new file starts at line 12)
|
||||||
|
- Only comment on lines that appear in the diff - GitHub will reject comments on unchanged lines
|
||||||
|
|
||||||
**CRITICAL: Prioritize per-line feedback over summary text.**
|
**CRITICAL: Prioritize per-line feedback over summary text.**
|
||||||
- ALL specific feedback MUST go in the 'comments' array with file paths and line numbers from the diff
|
- ALL specific feedback MUST go in the 'comments' array with file paths and line numbers from the diff
|
||||||
- for issues appearing in multiple places, comment on the FIRST occurrence and reference others (e.g., "also at lines X, Y" or "similar issue in otherFile.ts:42")
|
- for issues appearing in multiple places, comment on the FIRST occurrence and reference others (e.g., "also at lines X, Y" or "similar issue in otherFile.ts:42")
|
||||||
@@ -98215,6 +98220,10 @@ ${encodedEvent ? `************* EVENT DATA *************
|
|||||||
The following is structured data about the GitHub event that triggered this run (e.g., issue body, PR details, comment content). Use this context to understand the full situation.
|
The following is structured data about the GitHub event that triggered this run (e.g., issue body, PR details, comment content). Use this context to understand the full situation.
|
||||||
|
|
||||||
${encodedEvent}` : ""}
|
${encodedEvent}` : ""}
|
||||||
|
|
||||||
|
************* RUNTIME CONTEXT *************
|
||||||
|
|
||||||
|
working_directory: ${process.cwd()}
|
||||||
`;
|
`;
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|||||||
@@ -85,12 +85,17 @@ ${
|
|||||||
prompt: `Follow these steps:
|
prompt: `Follow these steps:
|
||||||
1. Get PR info with ${ghPullfrogMcpName}/get_pull_request (this automatically prepares the repository by fetching and checking out the PR branch)
|
1. Get PR info with ${ghPullfrogMcpName}/get_pull_request (this automatically prepares the repository by fetching and checking out the PR branch)
|
||||||
|
|
||||||
2. View diff: git diff origin/<base>...origin/<head> (use line numbers from this for inline comments, replace <base> and <head> with 'base' and 'head' from PR info)
|
2. View diff: \`git diff origin/<base>...origin/<head>\` (replace <base> and <head> with 'base' and 'head' from PR info)
|
||||||
|
|
||||||
3. Read files from the checked-out PR branch to understand the implementation
|
3. Read files from the checked-out PR branch to understand the implementation. Always use **relative paths** from repo root (e.g., \`src/index.ts\`), never absolute paths.
|
||||||
|
|
||||||
4. Submit review using ${ghPullfrogMcpName}/submit_pull_request_review
|
4. Submit review using ${ghPullfrogMcpName}/submit_pull_request_review
|
||||||
|
|
||||||
|
**CRITICAL: File paths and line numbers for inline comments**
|
||||||
|
- Use **relative paths** from repo root (e.g., \`packages/core/src/utils.ts\`)
|
||||||
|
- For line numbers, use the NEW file line number from the diff (shown after \`+\` in hunk headers like \`@@ -10,5 +12,8 @@\` means new file starts at line 12)
|
||||||
|
- Only comment on lines that appear in the diff - GitHub will reject comments on unchanged lines
|
||||||
|
|
||||||
**CRITICAL: Prioritize per-line feedback over summary text.**
|
**CRITICAL: Prioritize per-line feedback over summary text.**
|
||||||
- ALL specific feedback MUST go in the 'comments' array with file paths and line numbers from the diff
|
- ALL specific feedback MUST go in the 'comments' array with file paths and line numbers from the diff
|
||||||
- for issues appearing in multiple places, comment on the FIRST occurrence and reference others (e.g., "also at lines X, Y" or "similar issue in otherFile.ts:42")
|
- for issues appearing in multiple places, comment on the FIRST occurrence and reference others (e.g., "also at lines X, Y" or "similar issue in otherFile.ts:42")
|
||||||
|
|||||||
+1
-1
@@ -1,6 +1,6 @@
|
|||||||
{
|
{
|
||||||
"name": "@pullfrog/action",
|
"name": "@pullfrog/action",
|
||||||
"version": "0.0.133",
|
"version": "0.0.134",
|
||||||
"type": "module",
|
"type": "module",
|
||||||
"files": [
|
"files": [
|
||||||
"index.js",
|
"index.js",
|
||||||
|
|||||||
Reference in New Issue
Block a user