@polka-codes/cli 0.9.90 → 0.9.92

package/dist/index.js

@@ -16534,12 +16534,12 @@ var init_memory = __esm(() => {
   memoryConfigSchema = exports_external.object({
     enabled: exports_external.boolean().optional().default(true),
     type: exports_external.enum(["sqlite", "memory"]).optional().default("sqlite"),
-    path: exports_external.string().optional().default("~/.config/polka-codes/memory.sqlite")
+    path: exports_external.string().optional().default("~/.config/polkacodes/memory/memory.sqlite")
   }).strict().optional();
   DEFAULT_MEMORY_CONFIG = {
     enabled: true,
     type: "sqlite",
-    path: "~/.config/polka-codes/memory.sqlite"
+    path: "~/.config/polkacodes/memory/memory.sqlite"
   };
 });
 
@@ -25517,6 +25517,7 @@ var init_tools2 = __esm(() => {
 // ../core/src/UsageMeter.ts
 class UsageMeter {
   #totals = { input: 0, output: 0, cachedRead: 0, cost: 0, messageCount: 0 };
+  #providerMetadataEntries = [];
   #modelInfos;
   #maxMessages;
   #maxCost;
@@ -25598,8 +25599,17 @@ class UsageMeter {
     this.#totals.cachedRead += result.cachedRead || 0;
     this.#totals.cost += result.cost || 0;
     this.#totals.messageCount += 1;
+    if (resp.providerMetadata && Object.keys(resp.providerMetadata).length > 0) {
+      const providerKey = Object.keys(resp.providerMetadata)[0];
+      this.#providerMetadataEntries.push({
+        provider: providerKey || llm.provider,
+        model: llm.modelId,
+        metadata: resp.providerMetadata[providerKey] || resp.providerMetadata,
+        timestamp: Date.now()
+      });
+    }
   }
-  setUsage(newUsage) {
+  setUsage(newUsage, options = {}) {
     if (newUsage.input != null)
       this.#totals.input = newUsage.input;
     if (newUsage.output != null)
@@ -25610,12 +25620,16 @@ class UsageMeter {
       this.#totals.cost = newUsage.cost;
     if (newUsage.messageCount != null)
       this.#totals.messageCount = newUsage.messageCount;
+    if (options.clearMetadata) {
+      this.#providerMetadataEntries = [];
+    }
   }
   incrementMessageCount(n = 1) {
     this.#totals.messageCount += n;
   }
   resetUsage() {
     this.#totals = { input: 0, output: 0, cachedRead: 0, cost: 0, messageCount: 0 };
+    this.#providerMetadataEntries = [];
   }
   isLimitExceeded() {
     const messageCount = this.#maxMessages > 0 && this.#totals.messageCount >= this.#maxMessages;
@@ -25637,6 +25651,34 @@ class UsageMeter {
   get usage() {
     return { ...this.#totals };
   }
+  get providerMetadata() {
+    return [...this.#providerMetadataEntries];
+  }
+  get cacheStats() {
+    const entries = this.#providerMetadataEntries;
+    const totalRequests = entries.length;
+    let totalCachedTokens = 0;
+    let requestsWithCache = 0;
+    for (const entry of entries) {
+      const metadata = entry.metadata;
+      const cachedTokens = metadata.cachedPromptTokens ?? metadata.cacheReadTokens ?? metadata.prompt_cache_hit_tokens ?? 0;
+      if (cachedTokens > 0) {
+        totalCachedTokens += cachedTokens;
+        requestsWithCache++;
+      }
+    }
+    const cacheHitRate = totalRequests > 0 ? requestsWithCache / totalRequests : 0;
+    return {
+      totalCachedTokens,
+      totalRequests,
+      requestsWithCache,
+      cacheHitRate,
+      entries: [...this.#providerMetadataEntries]
+    };
+  }
+  clearProviderMetadata() {
+    this.#providerMetadataEntries = [];
+  }
   merge(other) {
     const otherUsage = other.usage;
     this.#totals.input += otherUsage.input;
@@ -25644,6 +25686,7 @@ class UsageMeter {
     this.#totals.cachedRead += otherUsage.cachedRead;
     this.#totals.cost += otherUsage.cost;
     this.#totals.messageCount += otherUsage.messageCount;
+    this.#providerMetadataEntries.push(...other.providerMetadata);
   }
   getUsageText() {
     const u = this.usage;
@@ -73481,7 +73524,7 @@ var init_provider = __esm(() => {
 
 // ../../node_modules/sql.js/dist/sql-wasm.js
 var require_sql_wasm = __commonJS((exports, module) => {
-  var __dirname = "/Users/xiliangchen/projects/polka-codes/node_modules/sql.js/dist";
+  var __dirname = "/home/ubuntu/polka-codes/node_modules/sql.js/dist";
   var initSqlJsPromise = undefined;
   var initSqlJs = function(moduleConfig) {
     if (initSqlJsPromise) {
@@ -75769,6 +75812,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 +75919,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 +77791,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 +77838,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 +94068,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 +94107,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 +94484,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?.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 +112279,7 @@ var {
   Help
 } = import__.default;
 // package.json
-var version = "0.9.90";
+var version = "0.9.92";
 
 // src/commands/agent.ts
 init_src();
@@ -115346,7 +115494,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,
@@ -120828,8 +120976,8 @@ async function memoryStatus() {
     const stats = await store.getStats();
     const globalConfigPath = getGlobalConfigPath();
     const config4 = await loadConfigAtPath(globalConfigPath);
-    const memoryConfig = config4?.memory || { path: "~/.config/polka-codes/memory.sqlite" };
-    const dbPath = resolveHomePath(memoryConfig.path || "~/.config/polka-codes/memory.sqlite");
+    const memoryConfig = config4?.memory || { path: "~/.config/polkacodes/memory/memory.sqlite" };
+    const dbPath = resolveHomePath(memoryConfig.path || "~/.config/polkacodes/memory/memory.sqlite");
     console.log(`
 Memory Store Status:`);
     console.log("─".repeat(80));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@polka-codes/cli",
-  "version": "0.9.90",
+  "version": "0.9.92",
   "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",
@@ -28,8 +29,8 @@
     "@inquirer/prompts": "^8.1.0",
     "@modelcontextprotocol/sdk": "^1.25.1",
     "@openrouter/ai-sdk-provider": "^1.5.4",
-    "@polka-codes/cli-shared": "0.9.89",
-    "@polka-codes/core": "0.9.89",
+    "@polka-codes/cli-shared": "0.9.91",
+    "@polka-codes/core": "0.9.91",
     "ai": "^5.0.117",
     "chalk": "^5.6.2",
     "commander": "^14.0.2",