@polka-codes/cli 0.9.91 → 0.9.93

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,37 @@ 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;
+      if (typeof metadata !== "object" || metadata === null) {
+        continue;
+      }
+      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 +25689,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;
@@ -75767,16 +75813,64 @@ var require_sql_wasm = __commonJS((exports, module) => {
 import { AsyncLocalStorage as AsyncLocalStorage2 } from "node:async_hooks";
 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 { mkdir as mkdir2, readdir as readdir2, readFile as readFile4, rename as rename2, unlink as unlink2, writeFile as writeFile2 } from "node:fs/promises";
+import { basename as basename2, dirname as dirname3, resolve as resolve5 } from "node:path";
 import { fileURLToPath } from "node:url";
 
 class FileLock {
   lockfilePath;
   static LOCK_TIMEOUT = 30000;
+  static CLEANUP_AGE = 600000;
+  static lastCleanupTime = 0;
+  static CLEANUP_THROTTLE = 60000;
   constructor(dbPath) {
     this.lockfilePath = `${dbPath}.lock`;
   }
+  static resetCleanupThrottle() {
+    FileLock.lastCleanupTime = 0;
+  }
+  static async cleanupOldLockFiles(dbPath, maxAge = FileLock.CLEANUP_AGE, force = false) {
+    const now2 = Date.now();
+    if (!force && now2 - FileLock.lastCleanupTime < FileLock.CLEANUP_THROTTLE) {
+      return;
+    }
+    FileLock.lastCleanupTime = now2;
+    try {
+      const lockDir = dirname3(dbPath);
+      const dbBaseName = basename2(dbPath);
+      const files = await readdir2(lockDir);
+      const now3 = Date.now();
+      let cleanedCount = 0;
+      for (const file2 of files) {
+        if (!file2.startsWith(`${dbBaseName}.lock.`)) {
+          continue;
+        }
+        const match = file2.match(/\.lock\.(released|stale|invalid|corrupt)\.(\d+)$/);
+        if (!match) {
+          continue;
+        }
+        const filePath = resolve5(lockDir, file2);
+        const timestamp = Number.parseInt(match[2], 10);
+        const age = now3 - timestamp;
+        if (age > maxAge) {
+          try {
+            await unlink2(filePath);
+            cleanedCount++;
+          } catch (error48) {
+            const errorCode = error48?.code;
+            if (errorCode !== "ENOENT") {
+              console.warn(`[FileLock] Failed to delete old lock file ${file2}: ${error48 instanceof Error ? error48.message : String(error48)}`);
+            }
+          }
+        }
+      }
+      if (cleanedCount > 0) {
+        console.log(`[FileLock] Cleaned up ${cleanedCount} old lock file(s) (older than ${maxAge}ms)`);
+      }
+    } catch (error48) {
+      console.debug(`[FileLock] Cleanup encountered an error: ${error48 instanceof Error ? error48.message : String(error48)}`);
+    }
+  }
   async acquire(retries = 10, delay2 = 100) {
     for (let i2 = 0;i2 < retries; i2++) {
       try {
@@ -75827,6 +75921,8 @@ class FileLock {
   async release() {
     try {
       await rename2(this.lockfilePath, `${this.lockfilePath}.released.${Date.now()}`);
+      const dbPath = this.lockfilePath.slice(0, -5);
+      FileLock.cleanupOldLockFiles(dbPath).catch(() => {});
     } catch (error48) {
       const errorCode = error48.code;
       if (errorCode !== "ENOENT") {
@@ -75910,6 +76006,9 @@ var init_sqlite_memory_store = __esm(() => {
     inTransaction = false;
     transactionMutex = new ReentrantMutex;
     fileLock;
+    static resetCleanupThrottle() {
+      FileLock.resetCleanupThrottle();
+    }
     getDbPath() {
       return this.config.path || DEFAULT_MEMORY_CONFIG.path;
     }
@@ -75946,6 +76045,7 @@ var init_sqlite_memory_store = __esm(() => {
           if (!existsSync4(dir)) {
             await mkdir2(dir, { recursive: true, mode: 448 });
           }
+          FileLock.cleanupOldLockFiles(dbPath).catch(() => {});
           let dbData;
           if (existsSync4(dbPath)) {
             const lock = this.getFileLock();
@@ -94025,9 +94125,11 @@ function createGitReadBinaryFile(commit2) {
         }
       };
     }
-    const result = await provider3.executeCommand(`git show ${quotedCommit}:${quotedUrl}`, false);
+    const isWindows2 = process.platform === "win32";
+    const command = isWindows2 ? `cmd /c "git show ${quotedCommit}:${quotedUrl} | base64 -w 0 2>&1"` : `sh -c "git show ${quotedCommit}:${quotedUrl} | base64 2>&1"`;
+    const result = await provider3.executeCommand(command, false);
     if (result.exitCode === 0) {
-      const base64Data = Buffer.from(result.stdout, "binary").toString("base64");
+      const base64Data = result.stdout.replace(/\n/g, "");
       return {
         success: true,
         message: {
@@ -94043,11 +94145,13 @@ function createGitReadBinaryFile(commit2) {
         }
       };
     } else {
+      const isBase64Error = result.stderr.includes("not recognized") || result.stderr.includes("command not found") || result.stderr.includes("base64");
+      const errorMessage = isBase64Error ? `Failed to read binary file: base64 command not found. On Windows, ensure Git Bash or Unix tools are installed and in PATH.` : `Failed to read binary file: ${result.stderr}`;
       return {
         success: false,
         message: {
           type: "error-text",
-          value: `Failed to read binary file: ${result.stderr}`
+          value: errorMessage
         }
       };
     }
@@ -94441,7 +94545,7 @@ var reviewWorkflow = async (input, context) => {
     return { overview: "No changes to review.", specificReviews: [] };
   }
   const targetCommit = extractTargetCommit(range, pr2);
-  const isRange = range && range.includes("..");
+  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];
@@ -112236,7 +112340,7 @@ var {
   Help
 } = import__.default;
 // package.json
-var version = "0.9.91";
+var version = "0.9.93";
 
 // src/commands/agent.ts
 init_src();
@@ -120933,8 +121037,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));
@@ -121353,6 +121457,7 @@ var reviewCommand = new Command("review").description("Review a GitHub pull requ
   const workflowOpts = getBaseWorkflowOptions(command);
   if (json2) {
     workflowOpts.interactive = false;
+    workflowOpts.silent = true;
   }
   const { verbose } = workflowOpts;
   const logger = createLogger({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@polka-codes/cli",
-  "version": "0.9.91",
+  "version": "0.9.93",
   "license": "AGPL-3.0",
   "author": "github@polka.codes",
   "type": "module",
@@ -29,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",