@polka-codes/cli-shared 0.9.15 → 0.9.17

package/dist/index.js

@@ -52239,6 +52239,64 @@ var generateProjectConfig_default = {
   agent: "analyzer"
 };
 
+// ../core/src/AiTool/tools/utils/diffLineNumbers.ts
+function parseHunkHeader(header) {
+  const match = header.match(/^@@ -(\d+)(?:,(\d+))? \+(\d+)(?:,(\d+))? @@/);
+  if (!match)
+    return null;
+  return {
+    oldStart: parseInt(match[1], 10),
+    oldCount: match[2] ? parseInt(match[2], 10) : 1,
+    newStart: parseInt(match[3], 10),
+    newCount: match[4] ? parseInt(match[4], 10) : 1,
+    header
+  };
+}
+function annotateDiffWithLineNumbers(diff) {
+  const lines = diff.split(`
+`);
+  const annotatedLines = [];
+  let currentNewLine = 0;
+  let currentOldLine = 0;
+  let inHunk = false;
+  for (const line of lines) {
+    if (line.startsWith("@@")) {
+      const hunk = parseHunkHeader(line);
+      if (hunk) {
+        currentOldLine = hunk.oldStart;
+        currentNewLine = hunk.newStart;
+        inHunk = true;
+      }
+      annotatedLines.push(line);
+      continue;
+    }
+    if (line.startsWith("diff --git") || line.startsWith("index ") || line.startsWith("---") || line.startsWith("+++")) {
+      annotatedLines.push(line);
+      inHunk = false;
+      continue;
+    }
+    if (!inHunk) {
+      annotatedLines.push(line);
+      continue;
+    }
+    if (line.startsWith("+") && !line.startsWith("+++")) {
+      annotatedLines.push(`${line} [Line ${currentNewLine}]`);
+      currentNewLine++;
+    } else if (line.startsWith("-") && !line.startsWith("---")) {
+      annotatedLines.push(`${line} [Line ${currentOldLine} removed]`);
+      currentOldLine++;
+    } else if (line.startsWith(" ")) {
+      annotatedLines.push(line);
+      currentOldLine++;
+      currentNewLine++;
+    } else {
+      annotatedLines.push(line);
+    }
+  }
+  return annotatedLines.join(`
+`);
+}
+
 // ../core/src/AiTool/tools/gitDiff.ts
 var toolInfo14 = {
   name: "git_diff",
@@ -52255,7 +52313,18 @@ var toolInfo14 = {
       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().optional().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") {
+        const lower = val.toLowerCase();
+        if (lower === "false")
+          return false;
+        if (lower === "true")
+          return true;
+      }
+      return val;
+    }, exports_external.boolean().optional().default(false)).describe("Annotate the diff with line numbers for additions and deletions.")
   }),
   permissionLevel: 1 /* Read */
 };
@@ -52266,8 +52335,8 @@ var handler14 = async (provider2, args) => {
       message: "Not possible to execute command. Abort."
     };
   }
-  const { staged, file: file3, commitRange } = toolInfo14.parameters.parse(args);
-  const commandParts = ["git", "diff", "--no-color", "-U50"];
+  const { staged, file: file3, commitRange, contextLines, includeLineNumbers } = toolInfo14.parameters.parse(args);
+  const commandParts = ["git", "diff", "--no-color", `-U${contextLines}`];
   if (staged) {
     commandParts.push("--staged");
   }
@@ -52287,10 +52356,14 @@ var handler14 = async (provider2, args) => {
           message: "No diff found."
         };
       }
+      let diffOutput = result.stdout;
+      if (includeLineNumbers) {
+        diffOutput = annotateDiffWithLineNumbers(diffOutput);
+      }
       return {
         type: "Reply" /* Reply */,
         message: `<diff file="${file3 ?? "all"}">
-${result.stdout}
+${diffOutput}
 </diff>`
       };
     }
@@ -52326,8 +52399,9 @@ You are a senior software engineer reviewing code changes.
 
 ## 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.
-  - **Local changes**: diff staged or unstaged files using the git_diff tool.
+  - **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:
   - <pr_title>
   - <pr_description>
@@ -52335,9 +52409,16 @@ You are a senior software engineer reviewing code changes.
 - 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.
 
+## 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"
+
 ## Review Guidelines
 Focus exclusively on the changed lines (+ additions, - deletions, modified lines):
-- **Specific issues**: Point to exact problems in the changed code with line references
+- **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
@@ -52400,11 +52481,11 @@ ${fileList}
     }
     let instructions = "";
     if (params.commitRange) {
-      instructions = `Review the pull request. Use the git_diff tool with commit range '${params.commitRange}' to inspect the actual code changes. File status information is already provided above.`;
+      instructions = `Review the pull request. Use the git_diff tool with commit range '${params.commitRange}', contextLines: 5, and includeLineNumbers: true to inspect the actual code changes. The diff will include line number annotations to help you report accurate line numbers. File status information is already provided above.`;
     } else if (params.staged) {
-      instructions = "Review the staged changes. Use the git_diff tool with staged: true to inspect the actual code changes. File status information is already provided above.";
+      instructions = "Review the staged changes. Use the git_diff tool with staged: true, contextLines: 5, and includeLineNumbers: true to inspect the actual code changes. The diff will include line number annotations to help you report accurate line numbers. File status information is already provided above.";
     } else {
-      instructions = "Review the unstaged changes. Use the git_diff tool to inspect the actual code changes. File status information is already provided above.";
+      instructions = "Review the unstaged changes. Use the git_diff tool with contextLines: 5 and includeLineNumbers: true to inspect the actual code changes. The diff will include line number annotations to help you report accurate line numbers. File status information is already provided above.";
     }
     parts.push(`<review_instructions>
 ${instructions}

package/package.json

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