@polka-codes/cli 0.9.90 → 0.9.91

package/dist/index.js

@@ -75769,6 +75769,7 @@ import { randomUUID } from "node:crypto";
 import { existsSync as existsSync4 } from "node:fs";
 import { mkdir as mkdir2, readFile as readFile4, rename as rename2, writeFile as writeFile2 } from "node:fs/promises";
 import { dirname as dirname3, resolve as resolve5 } from "node:path";
+import { fileURLToPath } from "node:url";
 
 class FileLock {
   lockfilePath;
@@ -75875,7 +75876,24 @@ async function getSqlJs() {
   if (SqlJsInitPromise) {
     return SqlJsInitPromise;
   }
-  SqlJsInitPromise = import_sql.default({});
+  const moduleDir = dirname3(fileURLToPath(import.meta.url));
+  const candidates = [
+    resolve5(moduleDir, "sql-wasm.wasm"),
+    resolve5(moduleDir, "..", "dist", "sql-wasm.wasm"),
+    resolve5(moduleDir, "..", "node_modules", "sql.js", "dist", "sql-wasm.wasm"),
+    resolve5(moduleDir, "..", "..", "..", "node_modules", "sql.js", "dist", "sql-wasm.wasm"),
+    resolve5(process.cwd(), "node_modules", "sql.js", "dist", "sql-wasm.wasm")
+  ];
+  SqlJsInitPromise = import_sql.default({
+    locateFile: (file2) => {
+      for (const candidate of candidates) {
+        if (existsSync4(candidate)) {
+          return candidate;
+        }
+      }
+      return file2;
+    }
+  });
   SqlJs = await SqlJsInitPromise;
   return SqlJs;
 }
@@ -77730,15 +77748,15 @@ ${value}
 }
 function getReviewInstructions(params) {
   if (params.targetCommit) {
-    return `Review the changes in commit '${params.targetCommit}'. The gitDiff tool is NOT available for past commits. Use readFile to inspect the file contents and identify issues in the modified files listed in <file_status>.`;
+    return `Review the changes in commit '${params.targetCommit}'. Use the gitDiff tool with the file parameter to inspect what changed in each file. Focus your review on the actual changes shown in the diff.`;
   }
   if (params.commitRange) {
-    return `Review the pull request. Use the gitDiff tool with commit range '${params.commitRange}' to inspect the actual code changes.`;
+    return `Review the pull request or commit range '${params.commitRange}'. Use the gitDiff tool with the file parameter to inspect the actual code changes.`;
   }
   if (params.staged) {
-    return "Review the staged changes. Use the gitDiff tool with staged: true to inspect the actual code changes.";
+    return "Review the staged changes. Use the gitDiff tool with the file parameter and staged: true to inspect the actual code changes.";
   }
-  return "Review the unstaged changes. Use the gitDiff tool to inspect the actual code changes.";
+  return "Review the unstaged changes. Use the gitDiff tool with the file parameter to inspect the actual code changes.";
 }
 function formatReviewToolInput(params) {
   const fileList = params.changedFiles && params.changedFiles.length > 0 ? params.changedFiles.map((file2) => {
@@ -77777,18 +77795,20 @@ ${TOOL_USAGE_INSTRUCTION}
     -   **Include**: Source code, config files, and template files.
     -   **Exclude**: Lockfiles, build artifacts, test snapshots, binary/media files, data and fixtures and other generated files.
 3.  **Inspect Changes**:
-    -   When the \`gitDiff\` tool is available: Use it on one file at a time to see the exact changes. When reviewing pull requests, use the \`commitRange\` parameter provided in the review instructions.
-    -   When the \`gitDiff\` tool is NOT available (reviewing past commits): Use \`readFile\` to inspect the file contents. You will not have access to the exact diff, so focus on reviewing the code for issues in the modified files listed in \`<file_status>\`.
+    -   The \`gitDiff\` tool is ALWAYS available for reviewing changes.
+    -   When reviewing pull requests or commit ranges: Use \`gitDiff\` with the file parameter to see exact changes.
+    -   When reviewing local changes: Use \`gitDiff\` with staged: true for staged changes, or without parameters for unstaged changes.
+    -   When reviewing a specific commit: Use \`gitDiff\` with the file parameter to see what changed in that commit.
 4.  **Analyze and Review**: Analyze the code for issues. When using \`gitDiff\`, focus only on the modified lines (additions/deletions). Provide specific, actionable feedback with accurate line numbers.
 
 ## Critical Rules
 
--   **Focus on Changes**: When using \`gitDiff\`, ONLY review the actual changes shown in the diff. Do not comment on existing, unmodified code. When \`gitDiff\` is not available, focus on the modified files listed in \`<file_status>\`.
+-   **Focus on Changes**: When using \`gitDiff\`, ONLY review the actual changes shown in the diff. Do not comment on existing, unmodified code.
 -   **Focus Scope**: Do not comment on overall project structure or architecture unless directly impacted by the changes in the diff.
 -   **No Feature Requests**: Do not comment on missing features or functionality that are not part of this diff.
--   **One File at a Time**: Review files individually using \`gitDiff\` (when available) with the specific file path, or \`readFile\` when reviewing past commits.
--   **No Empty Diffs**: MUST NOT call \`gitDiff\` with an empty or omitted file parameter when the tool is available.
--   **Accurate Line Numbers**: When using \`gitDiff\`, use the line numbers from the diff annotations (\`[Line N]\` for additions, \`[Line N removed]\` for deletions). When \`gitDiff\` is not available, use line numbers from \`readFile\` output.
+-   **One File at a Time**: Review files individually using \`gitDiff\` with the specific file path.
+-   **No Empty Diffs**: MUST NOT call \`gitDiff\` with an empty or omitted file parameter. Always specify a file path.
+-   **Accurate Line Numbers**: When using \`gitDiff\`, use the line numbers from the diff annotations (\`[Line N]\` for additions, \`[Line N removed]\` for deletions).
 -   **No Praise**: Provide only reviews for actual issues found. Do not include praise or positive feedback.
 -   **Clear Reasoning**: For each issue, provide clear reasoning explaining why it's a problem and what the impact could be.
 -   **Specific Advice**: Avoid generic advice. Provide concrete, actionable suggestions specific to the code being reviewed.
@@ -94005,9 +94025,9 @@ function createGitReadBinaryFile(commit2) {
         }
       };
     }
-    const result = await provider3.executeCommand(`git show ${quotedCommit}:${quotedUrl} | base64`, false);
+    const result = await provider3.executeCommand(`git show ${quotedCommit}:${quotedUrl}`, false);
     if (result.exitCode === 0) {
-      const base64Data = result.stdout.replace(/\n/g, "");
+      const base64Data = Buffer.from(result.stdout, "binary").toString("base64");
       return {
         success: true,
         message: {
@@ -94044,6 +94064,91 @@ function createGitAwareTools(commit2) {
     readBinaryFile: createGitReadBinaryFile(commit2)
   };
 }
+function createGitAwareDiff(commit2) {
+  const toolInfo20 = {
+    name: "git_diff",
+    description: `Get the git diff for commit ${commit2}. Shows the exact changes introduced by this specific commit. Use this to inspect what changed in each file. Always specify a file path.`,
+    parameters: exports_external.object({
+      file: exports_external.string().describe("Get the diff for a specific file within the commit. This parameter is required.").meta({ usageValue: "File path here (required)" }),
+      contextLines: exports_external.coerce.number().int().min(0).optional().default(5).describe("Number of context lines to include around changes.").meta({ usageValue: "Context lines count (optional)" }),
+      includeLineNumbers: exports_external.preprocess((val) => {
+        if (typeof val === "string") {
+          const lower2 = val.toLowerCase();
+          if (lower2 === "false")
+            return false;
+          if (lower2 === "true")
+            return true;
+        }
+        return val;
+      }, exports_external.boolean().optional().default(true)).describe("Annotate the diff with line numbers for additions and deletions.").meta({ usageValue: "true or false (optional)" })
+    })
+  };
+  const handler20 = async (provider3, args) => {
+    if (!provider3.executeCommand) {
+      return {
+        success: false,
+        message: {
+          type: "error-text",
+          value: "Not possible to execute command."
+        }
+      };
+    }
+    const { file: file2, contextLines, includeLineNumbers } = toolInfo20.parameters.parse(args);
+    const quotedCommit = quoteForShell(commit2);
+    let command = `git show --no-color --format= -U${contextLines} ${quotedCommit}`;
+    if (file2) {
+      const quotedFile = quoteForShell(file2);
+      command = `git show --no-color --format= -U${contextLines} ${quotedCommit} -- ${quotedFile}`;
+    }
+    try {
+      const result = await provider3.executeCommand(command, false);
+      if (result.exitCode === 0) {
+        if (!result.stdout.trim()) {
+          return {
+            success: true,
+            message: {
+              type: "text",
+              value: "No diff found."
+            }
+          };
+        }
+        let diffOutput = result.stdout;
+        if (includeLineNumbers) {
+          diffOutput = annotateDiffWithLineNumbers(diffOutput);
+        }
+        return {
+          success: true,
+          message: {
+            type: "text",
+            value: `<diff file="${file2 ?? "all"}">
+${diffOutput}
+</diff>`
+          }
+        };
+      }
+      return {
+        success: false,
+        message: {
+          type: "error-text",
+          value: `\`${command}\` exited with code ${result.exitCode}:
+${result.stderr}`
+        }
+      };
+    } catch (error48) {
+      return {
+        success: false,
+        message: {
+          type: "error-text",
+          value: error48 instanceof Error ? error48.message : String(error48)
+        }
+      };
+    }
+  };
+  return {
+    ...toolInfo20,
+    handler: handler20
+  };
+}
 var init_git_file_tools = __esm(() => {
   init_zod();
 });
@@ -94336,10 +94441,10 @@ var reviewWorkflow = async (input, context) => {
     return { overview: "No changes to review.", specificReviews: [] };
   }
   const targetCommit = extractTargetCommit(range, pr2);
-  const isReviewingCommit = targetCommit !== null;
-  const finalChangeInfo = targetCommit ? { ...changeInfo, targetCommit } : changeInfo;
-  const fileTools = isReviewingCommit && targetCommit ? createGitAwareTools(targetCommit) : { readFile: readFile_default, listFiles: listFiles_default, readBinaryFile: readBinaryFile_default };
-  const reviewTools = isReviewingCommit ? [fileTools.readFile, fileTools.readBinaryFile, fileTools.listFiles] : [readFile_default, readBinaryFile_default, searchFiles_default, listFiles_default, gitDiff_default];
+  const isRange = range && range.includes("..");
+  const finalChangeInfo = targetCommit && !isRange ? { ...changeInfo, targetCommit } : changeInfo;
+  const fileTools = targetCommit && !isRange ? createGitAwareTools(targetCommit) : { readFile: readFile_default, listFiles: listFiles_default, readBinaryFile: readBinaryFile_default };
+  const reviewTools = targetCommit && !isRange ? [fileTools.readFile, fileTools.readBinaryFile, fileTools.listFiles, createGitAwareDiff(targetCommit)] : [readFile_default, readBinaryFile_default, searchFiles_default, listFiles_default, gitDiff_default];
   const result = await step("review", async () => {
     const defaultContext = await getDefaultContext("review");
     const memoryContext = await tools3.getMemoryContext();
@@ -112131,7 +112236,7 @@ var {
   Help
 } = import__.default;
 // package.json
-var version = "0.9.90";
+var version = "0.9.91";
 
 // src/commands/agent.ts
 init_src();
@@ -115346,7 +115451,7 @@ class TaskHistory {
         return r2.actualTime === 0 ? 0 : 100;
       }
       return Math.abs((r2.estimatedTime - r2.actualTime) / r2.estimatedTime) * 100;
-    }).filter((e2) => !isNaN(e2));
+    }).filter((e2) => !Number.isNaN(e2));
     const avgErrorPercentage = errorPercentages.reduce((sum, e2) => sum + e2, 0) / errorPercentages.length;
     return {
       averageError: avgError,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@polka-codes/cli",
-  "version": "0.9.90",
+  "version": "0.9.91",
   "license": "AGPL-3.0",
   "author": "github@polka.codes",
   "type": "module",
@@ -14,7 +14,8 @@
     }
   },
   "scripts": {
-    "build": "bun build src/index.ts --outdir dist --target node"
+    "build": "bun build src/index.ts --outdir dist --target node && bun run build:copy-wasm",
+    "build:copy-wasm": "mkdir -p dist && cp ../../node_modules/sql.js/dist/sql-wasm.wasm dist/"
   },
   "dependencies": {
     "@ai-sdk/anthropic": "^2.0.57",