@polka-codes/cli 0.9.96 → 0.9.98

package/dist/index.js

@@ -75842,7 +75842,6 @@ class FileLock {
       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;
@@ -75857,7 +75856,6 @@ class FileLock {
         if (age > maxAge) {
           try {
             await unlink2(filePath);
-            cleanedCount++;
           } catch (error48) {
             const errorCode = error48?.code;
             if (errorCode !== "ENOENT") {
@@ -75866,9 +75864,6 @@ class FileLock {
           }
         }
       }
-      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)}`);
     }
@@ -92231,6 +92226,7 @@ var init_env = () => {};
 
 // src/getModel.ts
 import { appendFileSync } from "node:fs";
+import { inspect } from "node:util";
 function headersToObject(headers) {
   if (!headers) {
     return;
@@ -92263,10 +92259,10 @@ var AiProvider, getModel = (config4, debugLogging = false) => {
   const fetchOverride = debugLogging || TRACING_FILE ? async (url2, options) => {
     const requestBody = options?.body ? JSON.parse(options.body) : undefined;
     if (debugLogging) {
-      console.log("-> Request URL:", url2);
-      console.log("-> Request Headers:", options?.headers);
-      console.log("-> Request Body:");
-      console.dir(requestBody, { depth: null });
+      console.error("-> Request URL:", url2);
+      console.error("-> Request Headers:", options?.headers);
+      console.error("-> Request Body:");
+      console.error(inspect(requestBody, { depth: null, colors: process.stderr.isTTY }));
     }
     if (TRACING_FILE) {
       appendFileSync(TRACING_FILE, `${JSON.stringify({
@@ -92280,7 +92276,7 @@ var AiProvider, getModel = (config4, debugLogging = false) => {
     }
     const res = await fetch(url2, options);
     if (debugLogging) {
-      console.log("<- Response Status:", res.status);
+      console.error("<- Response Status:", res.status);
     }
     const contentType = res.headers.get("content-type") || "";
     if (contentType.includes("text/event-stream") && res.body) {
@@ -92296,7 +92292,7 @@ var AiProvider, getModel = (config4, debugLogging = false) => {
             if (value) {
               const text2 = decoder.decode(value);
               if (debugLogging) {
-                console.log("<- Stream chunk:", text2.replace(/\n/g, "\\n"));
+                console.error("<- Stream chunk:", text2.replace(/\n/g, "\\n"));
               }
               if (TRACING_FILE) {
                 for (const line of text2.split(`
@@ -92347,8 +92343,8 @@ var AiProvider, getModel = (config4, debugLogging = false) => {
       responseBody = full;
     }
     if (debugLogging) {
-      console.log("<- Response Body:");
-      console.dir(responseBody, { depth: null });
+      console.error("<- Response Body:");
+      console.error(inspect(responseBody, { depth: null, colors: process.stderr.isTTY }));
     }
     if (TRACING_FILE) {
       appendFileSync(TRACING_FILE, `${JSON.stringify({
@@ -112342,7 +112338,7 @@ var {
   Help
 } = import__.default;
 // package.json
-var version = "0.9.96";
+var version = "0.9.98";
 
 // src/commands/agent.ts
 init_src();
@@ -116318,15 +116314,18 @@ class AutonomousAgent {
 }
 
 // src/logger.ts
+import { formatWithOptions } from "node:util";
 function createStreamConsole(stdout, stderr) {
+  const isStdoutTTY = "isTTY" in stdout && stdout.isTTY;
+  const isStderrTTY = "isTTY" in stderr && stderr.isTTY;
   return {
-    log: (...args) => stdout.write(`${args.map(String).join(" ")}
+    log: (...args) => stdout.write(`${formatWithOptions({ colors: isStdoutTTY }, ...args)}
 `),
-    info: (...args) => stdout.write(`${args.map(String).join(" ")}
+    info: (...args) => stdout.write(`${formatWithOptions({ colors: isStdoutTTY }, ...args)}
 `),
-    warn: (...args) => stderr.write(`${args.map(String).join(" ")}
+    warn: (...args) => stderr.write(`${formatWithOptions({ colors: isStderrTTY }, ...args)}
 `),
-    error: (...args) => stderr.write(`${args.map(String).join(" ")}
+    error: (...args) => stderr.write(`${formatWithOptions({ colors: isStderrTTY }, ...args)}
 `)
   };
 }
@@ -116808,7 +116807,7 @@ function addSharedOptions(command) {
 async function parseOptions(options, { cwdArg, commandName } = {}, home = os3.homedir(), env2 = getEnv()) {
   const cwd = options.baseDir ?? cwdArg ?? process.cwd();
   if (options.baseDir) {
-    console.log("Using base directory:", cwd);
+    console.error("Using base directory:", cwd);
     process.chdir(cwd);
   }
   const config4 = await loadConfig2(options.config, cwd, home) ?? {};
@@ -116990,22 +116989,22 @@ async function createCommit(input, _context) {
   }
   return { message: input.message };
 }
-async function printChangeFile(_input, _context) {
+async function printChangeFile(_input, context) {
   const { stagedFiles, unstagedFiles } = getLocalChanges();
   if (stagedFiles.length === 0 && unstagedFiles.length === 0) {
-    console.log("No changes to commit.");
+    context.workflowContext.logger.info("No changes to commit.");
   } else {
     if (stagedFiles.length > 0) {
-      console.log("Staged files:");
+      context.workflowContext.logger.info("Staged files:");
       for (const file2 of stagedFiles) {
-        console.log(`- ${file2.status}: ${file2.path}`);
+        context.workflowContext.logger.info(`- ${file2.status}: ${file2.path}`);
       }
     }
     if (unstagedFiles.length > 0) {
-      console.log(`
+      context.workflowContext.logger.info(`
 Unstaged files:`);
       for (const file2 of unstagedFiles) {
-        console.log(`- ${file2.status}: ${file2.path}`);
+        context.workflowContext.logger.info(`- ${file2.status}: ${file2.path}`);
       }
     }
   }
@@ -117043,43 +117042,6 @@ async function select(input2, context) {
 }
 async function executeCommand(input2) {
   return new Promise((resolve7, reject) => {
-    const ALLOWED_COMMANDS = [
-      "git",
-      "npm",
-      "npx",
-      "node",
-      "bun",
-      "python",
-      "python3",
-      "ls",
-      "cat",
-      "grep",
-      "find",
-      "head",
-      "tail",
-      "wc",
-      "sort",
-      "uniq",
-      "awk",
-      "sed",
-      "curl",
-      "wget",
-      "rsync",
-      "cp",
-      "mv",
-      "rm",
-      "mkdir",
-      "chmod",
-      "chown",
-      "file",
-      "which",
-      "type",
-      "echo"
-    ];
-    if (!ALLOWED_COMMANDS.includes(input2.command)) {
-      reject(new Error(`Command not allowed: ${input2.command}`));
-      return;
-    }
     const child = input2.shell === true ? input2.args && input2.args.length > 0 ? spawn4(input2.command, input2.args, { shell: true, stdio: "pipe" }) : spawn4(input2.command, { shell: true, stdio: "pipe" }) : spawn4(input2.command, input2.args, {
       shell: false,
       stdio: "pipe"
@@ -118162,7 +118124,7 @@ var fetchOllamaModels = async () => {
     const data = await resp.json();
     return data.models.map((model) => model.name);
   } catch (_error) {
-    console.log("Unable to fetch Ollama models");
+    console.error("Unable to fetch Ollama models");
     return [];
   }
 };
@@ -120178,7 +120140,7 @@ async function getMemoryStore(logger) {
     const cwd = process.cwd();
     const scope = detectProjectScope(cwd);
     const dbPath = memoryConfig.path || DEFAULT_MEMORY_CONFIG.path;
-    const resolvedDbPath = path11.resolve(cwd, dbPath);
+    const resolvedDbPath = path11.resolve(resolveHomePath(dbPath));
     const sqliteStore = new SQLiteMemoryStore({ enabled: true, type: "sqlite", path: resolvedDbPath }, scope);
     const memoryManager = new MemoryManager(sqliteStore);
     memoryStoreCache = {
@@ -121620,27 +121582,24 @@ var reviewCommand = new Command("review").description("Review a GitHub pull requ
   const { json: json2, pr: pr3, range, context: contextOption, loop: maxIterations, yes: yesOption } = options;
   const yes = maxIterations > 1 || yesOption;
   let changesAppliedInThisIteration = false;
-  let context = contextOption;
-  let filesToReview = files;
-  if (files.length === 1 && !existsSync7(files[0]) && files[0].includes(" ") && !context) {
-    if (!json2) {
-      console.warn("Warning: The argument looks like context but was passed as a file. Treating it as context. Please use --context for review instructions in the future.");
-    }
-    context = files[0];
-    filesToReview = [];
-  }
   const workflowOpts = getBaseWorkflowOptions(command);
   if (json2) {
     workflowOpts.interactive = false;
-    workflowOpts.silent = true;
   }
   const { verbose } = workflowOpts;
   const logger = createLogger({
-    verbose: json2 ? -1 : verbose
+    verbose
   });
+  let context = contextOption;
+  let filesToReview = files;
+  if (files.length === 1 && !existsSync7(files[0]) && files[0].includes(" ") && !context) {
+    logger.warn("Warning: The argument looks like context but was passed as a file. Treating it as context. Please use --context for review instructions in the future.");
+    context = files[0];
+    filesToReview = [];
+  }
   for (let i2 = 0;i2 < maxIterations; i2++) {
     changesAppliedInThisIteration = false;
-    if (i2 > 0 && !json2) {
+    if (i2 > 0) {
       logger.debug(`Re-running review (iteration ${i2 + 1} of ${maxIterations})...`);
     }
     const reviewResult = await reviewCode({
@@ -121713,9 +121672,7 @@ ${formattedReview}`;
       }
     }
     if (maxIterations > 1 && !changesAppliedInThisIteration) {
-      if (!json2) {
-        logger.debug("No more review feedback to apply. Exiting loop.");
-      }
+      logger.debug("No more review feedback to apply. Exiting loop.");
       break;
     }
   }

package/package.json

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