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

Improve PR, add pwd

Browse Source
This commit is contained in:
Colin McDonnell
2025-12-13 12:28:59 -08:00
parent 6371584c80
commit 6b79fd4e29
3 changed files with 1176 additions and 1166 deletions
Download Patch File Download Diff File Expand all files Collapse all files
agents/instructions.ts
+4
View File
@@ -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()}
`; `;
}; };
entry
+1165 -1164
View File
File diff suppressed because one or more lines are too long
modes.ts
+7 -2
View File
@@ -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")