@polka-codes/cli 0.9.24 → 0.9.25

package/dist/index.js

@@ -58572,7 +58572,7 @@ var {
   Help
 } = import__.default;
 // package.json
-var version = "0.9.24";
+var version = "0.9.25";
 
 // src/commands/chat.ts
 import { readFile as readFile3 } from "node:fs/promises";
@@ -97959,7 +97959,7 @@ var toolInfo15 = {
       return val;
     }, exports_external.boolean().optional().default(false)).describe("Get staged changes instead of unstaged changes."),
     commitRange: exports_external.string().optional().describe('The commit range to get the diff for (e.g., "main...HEAD").'),
-    file: exports_external.string().optional().describe("Get the diff for a specific file."),
+    file: exports_external.string().describe("Get the diff for a specific file."),
     contextLines: exports_external.coerce.number().optional().default(5).describe("Number of context lines to include around changes."),
     includeLineNumbers: exports_external.preprocess((val) => {
       if (typeof val === "string") {
@@ -98056,42 +98056,58 @@ var prompt5 = `
 You are a senior software engineer reviewing code changes.
 
 ## Critical Instructions
-**ONLY review the actual changes shown in the diff.** Do not comment on existing code that wasn't modified.
+- **ONLY review the actual changes shown in the diff.** Do not comment on existing code that wasn't modified.
+- **ONLY run git_diff on files that are reviewable source/config files** per the "File Selection for git_diff" rules below. Do not pass excluded files to git_diff.
+
+## File Selection for git_diff
+Use <file_status> to decide which files to diff. Include only files likely to contain human-authored source or meaningful configuration.
+
+Include (run git_diff):
+- Application/source code
+- UI/templates/assets code
+- Infra/config that affects behavior
+
+Exclude (do NOT run git_diff; do not review):
+- Lockfiles
+- Generated/build artifacts & deps
+- Test artifacts/snapshots
+- Data and fixtures
+- Binary/media/minified/maps
 
 ## Viewing Changes
-- **Use git_diff** to inspect the actual code changes for each relevant file.
-  - **Pull request**: use the provided commit range for the git_diff tool with contextLines: 5 and includeLineNumbers: true
-  - **Local changes**: diff staged or unstaged files using the git_diff tool with contextLines: 5 and includeLineNumbers: true
-- The diff will include line number annotations: [Line N] for additions and [Line N removed] for deletions
-- If a pull request is present you may receive:
+- For each included file, **use git_diff** to inspect the actual code changes:
+  - **Pull request:** use the provided commit range for the git_diff tool with contextLines: 5 and includeLineNumbers: true, but only surface and review the included files.
+  - **Local changes:** diff staged or unstaged included files using git_diff with contextLines: 5 and includeLineNumbers: true.
+- The diff will include line number annotations: [Line N] for additions and [Line N removed] for deletions.
+- You may receive:
   - <pr_title>
   - <pr_description>
   - <commit_messages>
 - A <review_instructions> tag tells you the focus of the review.
-- File status information is provided in <file_status> - use this to understand which files were modified, added, deleted, or renamed.
+- Use <file_status> to understand which files were modified, added, deleted, or renamed and to apply the inclusion/exclusion rules above.
 
 ## Line Number Reporting
-- **IMPORTANT**: Use the line numbers from the annotations in the diff output
-- For additions: Look for [Line N] annotations after the + lines
-- For deletions: Look for [Line N removed] annotations after the - lines
-- For modifications: Report the line number of the new/current code (from [Line N] annotations)
-- Report single lines as "N" and ranges as "N-M"
+- Use the line numbers from the annotations in the diff output.
+- For additions: use the number from the [Line N] annotation after the + line.
+- For deletions: use the number from the [Line N removed] annotation after the - line.
+- For modifications: report the line number of the new/current code (from [Line N]).
+- Report single lines as "N" and ranges as "N-M".
 
 ## Review Guidelines
 Focus exclusively on the changed lines (+ additions, - deletions, modified lines):
-- **Specific issues**: Point to exact problems in the changed code with accurate line references from the annotations
-- **Actionable fixes**: Provide concrete solutions, not vague suggestions
-- **Clear reasoning**: Explain why each issue matters and how to fix it
-- **Avoid generic advice**: No generic suggestions like "add more tests", "improve documentation", or "follow best practices" unless directly related to a specific problem in the diff
+- **Specific issues:** Point to exact problems in the changed code with accurate line references from the annotations.
+- **Actionable fixes:** Provide concrete solutions, not vague suggestions.
+- **Clear reasoning:** Explain why each issue matters and how to fix it.
+- **Avoid generic advice** unless directly tied to a specific problem visible in the diff.
 
 ## What NOT to review
-- Existing unchanged code
-- Overall project structure or architecture (unless directly impacted by changes)
-- Generic best practices unrelated to the specific changes
-- Missing features or functionality not part of this diff
+- Files excluded by the "File Selection for git_diff" rules (do not diff or comment on them).
+- Existing unchanged code.
+- Overall project structure/architecture unless directly impacted by the changes.
+- Missing features or functionality not part of this diff.
 
 ## Output Format
-Do **not** include praise or positive feedback. Ignore generated files such as lock files.
+Do not include praise or positive feedback.
 Only include reviews for actual issues found in the changed code.
 
 Return your review as a JSON object inside a \`\`\`json block, wrapped like:
@@ -98099,7 +98115,7 @@ Return your review as a JSON object inside a \`\`\`json block, wrapped like:
 <tool_parameter_result>
 \`\`\`json
 {
-  "overview": "Summary of specific issues found in the diff changes, or 'No issues found' if the changes look good.",
+  "overview": "Summary of specific issues found in the diff changes, 'No issues found', or 'No reviewable changes' if all modified files were excluded.",
   "specificReviews": [
     {
       "file": "path/filename.ext",
@@ -116696,7 +116712,7 @@ async function runTask(taskArg, _options, command) {
       console.log();
       runner.printUsage();
     }
-    process.exit(0);
+    process.exit(130);
   };
   process.on("SIGINT", sigintHandler);
   const fileContents = [];
@@ -117000,7 +117016,7 @@ async function handleReviewResult(result, isJsonOutput, command) {
         default: false
       });
     } catch {
-      process.exit(0);
+      process.exit(130);
     }
   }
   if (shouldRunTask) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@polka-codes/cli",
-  "version": "0.9.24",
+  "version": "0.9.25",
   "license": "AGPL-3.0",
   "author": "github@polka.codes",
   "type": "module",