@polka-codes/cli 0.9.5 → 0.9.7

package/dist/index.js

@@ -39731,7 +39731,7 @@ var {
   Help
 } = import__.default;
 // package.json
-var version = "0.9.5";
+var version = "0.9.7";
 
 // ../../node_modules/@inquirer/core/dist/esm/lib/key.js
 var isUpKey = (key) => key.name === "up" || key.name === "k" || key.ctrl && key.name === "p";
@@ -52675,7 +52675,7 @@ var handler8 = async (provider, args) => {
       if (isEmpty) {
         resp.push(`<read_file_file_content path="${path}" is_empty="true" />`);
       } else {
-        resp.push(`<read_file_file_conten path="${path}">${fileContent}</read_file_file_content>`);
+        resp.push(`<read_file_file_content path="${path}">${fileContent}</read_file_file_content>`);
       }
     }
   }
@@ -52975,8 +52975,15 @@ var handler11 = async (provider, args) => {
       message: "Not possible to replace in file. Abort."
     };
   }
+  const parsed = toolInfo11.parameters.safeParse(args);
+  if (!parsed.success) {
+    return {
+      type: "Invalid" /* Invalid */,
+      message: `Invalid arguments for replace_in_file: ${parsed.error.message}`
+    };
+  }
+  const { path, diff } = parsed.data;
   try {
-    const { path, diff } = toolInfo11.parameters.parse(args);
     const fileContent = await provider.readFile(path, false);
     if (fileContent == null) {
       return {
@@ -53054,8 +53061,15 @@ var handler12 = async (provider, args) => {
       message: "Not possible to search files. Abort."
     };
   }
+  const parsed = toolInfo12.parameters.safeParse(args);
+  if (!parsed.success) {
+    return {
+      type: "Invalid" /* Invalid */,
+      message: `Invalid arguments for search_files: ${parsed.error.message}`
+    };
+  }
+  const { path, regex, filePattern } = parsed.data;
   try {
-    const { path, regex, filePattern } = toolInfo12.parameters.parse(args);
     const files = await provider.searchFiles(path, regex, filePattern ?? "*");
     return {
       type: "Reply" /* Reply */,
@@ -53070,8 +53084,8 @@ ${files.join(`
     };
   } catch (error40) {
     return {
-      type: "Invalid" /* Invalid */,
-      message: `Invalid arguments for search_files: ${error40}`
+      type: "Error" /* Error */,
+      message: `Error searching files: ${error40}`
     };
   }
 };
@@ -77457,52 +77471,39 @@ class CoderAgent extends AgentBase {
       usageMeter: options.usageMeter ?? new UsageMeter
     });
   }
-  async onBeforeInvokeTool(name17, _args) {
-    if (name17 !== attemptCompletion_default.name) {
-      return;
-    }
+  async#runScript(scriptName, shouldReplyWithError) {
     const executeCommand = this.config.provider.executeCommand;
     if (!executeCommand) {
       return;
     }
-    const format = this.config.scripts?.format;
-    const formatCommand = typeof format === "string" ? format : format?.command;
-    if (formatCommand) {
-      try {
-        await executeCommand(formatCommand, false);
-      } catch (error81) {
-        console.warn(`Failed to format code using command: ${formatCommand}`, error81);
-      }
-    }
-    const check3 = this.config.scripts?.check;
-    const checkCommand = typeof check3 === "string" ? check3 : check3?.command;
-    if (checkCommand) {
+    const script = this.config.scripts?.[scriptName];
+    const command = typeof script === "string" ? script : script?.command;
+    if (command) {
       try {
-        const { exitCode, stdout, stderr } = await executeCommand(checkCommand, false);
-        if (exitCode !== 0) {
+        const { exitCode, stdout, stderr } = await executeCommand(command, false);
+        if (exitCode !== 0 && shouldReplyWithError) {
           return {
             type: "Reply" /* Reply */,
-            message: responsePrompts.commandResult(checkCommand, exitCode, stdout, stderr)
+            message: responsePrompts.commandResult(command, exitCode, stdout, stderr)
           };
         }
       } catch (error81) {
-        console.warn(`Failed to check code using command: ${checkCommand}`, error81);
+        console.warn(`Failed to run ${scriptName} using command: ${command}`, error81);
       }
     }
-    const test = this.config.scripts?.test;
-    const testCommand = typeof test === "string" ? test : test?.command;
-    if (testCommand) {
-      try {
-        const { exitCode, stdout, stderr } = await executeCommand(testCommand, false);
-        if (exitCode !== 0) {
-          return {
-            type: "Reply" /* Reply */,
-            message: responsePrompts.commandResult(testCommand, exitCode, stdout, stderr)
-          };
-        }
-      } catch (error81) {
-        console.warn(`Failed to test code using command: ${testCommand}`, error81);
-      }
+  }
+  async onBeforeInvokeTool(name17, _args) {
+    if (name17 !== attemptCompletion_default.name) {
+      return;
+    }
+    await this.#runScript("format", false);
+    const checkResult = await this.#runScript("check", true);
+    if (checkResult) {
+      return checkResult;
+    }
+    const testResult = await this.#runScript("test", true);
+    if (testResult) {
+      return testResult;
     }
   }
 }
@@ -78010,10 +78011,18 @@ Example Output:
 <tool_output>
 <tool_output_pr_title>Refactor Order Validation and Remove Debug Logs</tool_output_pr_title>
 <tool_output_pr_description>
-closes #123
+Closes #123
+
+**Context**:
+- Implementing changes for issue #123 - Focus on clean code and maintainability
+
+**Summary of Changes**:
+- Refactored order validation logic to use a new \`validate_and_process\` method.
+- Removed debug print statements from \`user_service.py\`.
 
-This PR removes unnecessary debug print statements and updates order validation
-to use the new validate_and_process method for improved maintainability.
+**Highlights of Changed Code**:
+- \`order_service.py\`: Replaced direct call to \`process_order\` with \`validate_and_process\`.
+- \`user_service.py\`: Removed \`print("Debug info")\`.
 </tool_output_pr_description>
 </tool_output>
 
@@ -110740,15 +110749,9 @@ async function reviewPR(prIdentifier, spinner, sharedAiOptions, isJsonOutput) {
   try {
     spinner.text = `Checking out PR #${prNumber}...`;
     execSync3(`gh pr checkout ${prNumber}`, { stdio: "pipe" });
-  } catch (_error) {
+  } catch (error120) {
     spinner.fail(`Error checking out PR #${prNumber}. Make sure the PR number is correct and you have access to the repository.`);
-    process.exit(1);
-  }
-  const gitRoot = execSync3("git rev-parse --show-toplevel", { encoding: "utf-8" }).trim();
-  const remoteUrl = execSync3("git remote get-url origin", { cwd: gitRoot, encoding: "utf-8" }).trim();
-  const remoteMatch = remoteUrl.match(/github\.com[/:](?<owner>[^/]+)\/(?<repo>[^/]+)\.git$/);
-  if (!remoteMatch?.groups) {
-    spinner.fail("Could not determine GitHub repository owner and repo from remote URL.");
+    console.error(error120);
     process.exit(1);
   }
   spinner.text = "Fetching pull request details...";
@@ -110767,6 +110770,7 @@ async function reviewPR(prIdentifier, spinner, sharedAiOptions, isJsonOutput) {
   } catch (_error) {
     console.warn("Warning: Could not retrieve file changes list");
   }
+  printChangedFiles(changedFiles, spinner, isJsonOutput);
   spinner.text = "Generating review...";
   const result = await reviewDiff(sharedAiOptions, {
     commitRange: `${defaultBranch}...HEAD`,
@@ -110792,6 +110796,7 @@ async function reviewLocal(spinner, sharedAiOptions, isJsonOutput) {
   if (hasStagedChanges) {
     spinner.text = "Generating review for staged changes...";
     const stagedFiles = changedFiles.filter((file4) => file4.status.includes("staged"));
+    printChangedFiles(stagedFiles, spinner, isJsonOutput);
     const result2 = await reviewDiff(sharedAiOptions, {
       staged: true,
       changedFiles: stagedFiles
@@ -110807,6 +110812,7 @@ async function reviewLocal(spinner, sharedAiOptions, isJsonOutput) {
   if (hasUnstagedChanges) {
     spinner.text = "Generating review for unstaged changes...";
     const unstagedFiles = changedFiles.filter((file4) => file4.status.includes("unstaged") || file4.status.includes("Untracked"));
+    printChangedFiles(unstagedFiles, spinner, isJsonOutput);
     const result2 = await reviewDiff(sharedAiOptions, {
       staged: false,
       changedFiles: unstagedFiles
@@ -110843,6 +110849,7 @@ async function reviewLocal(spinner, sharedAiOptions, isJsonOutput) {
   } catch (_error) {
     console.warn("Warning: Could not retrieve file changes list");
   }
+  printChangedFiles(branchChangedFiles, spinner, isJsonOutput);
   spinner.text = `Generating review for changes between '${defaultBranch}' and '${currentBranch}'...`;
   const result = await reviewDiff(sharedAiOptions, {
     commitRange: `${defaultBranch}...${currentBranch}`,
@@ -110874,6 +110881,20 @@ ${item.review}
   }
   return formatted;
 }
+function printChangedFiles(changedFiles, spinner, isJsonOutput) {
+  if (changedFiles.length === 0) {
+    return;
+  }
+  spinner.stop();
+  const stream = isJsonOutput ? process.stderr : process.stdout;
+  const log = (message) => stream.write(`${message}
+`);
+  log("Changed files:");
+  for (const file4 of changedFiles) {
+    log(`- ${file4.status}: ${file4.path}`);
+  }
+  spinner.start();
+}
 
 // src/commands/task.ts
 var readStdin = async (timeoutMs = 30000) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@polka-codes/cli",
-  "version": "0.9.5",
+  "version": "0.9.7",
   "license": "AGPL-3.0",
   "author": "github@polka.codes",
   "type": "module",
@@ -23,10 +23,10 @@
     "@ai-sdk/openai": "2.0.0-beta.11",
     "@ai-sdk/provider": "2.0.0-beta.1",
     "@ai-sdk/provider-utils": "3.0.0-beta.5",
-    "@inquirer/prompts": "^7.2.3",
+    "@inquirer/prompts": "^7.8.0",
     "@openrouter/ai-sdk-provider": "^1.0.0-beta.6",
-    "@polka-codes/cli-shared": "0.9.4",
-    "@polka-codes/core": "0.9.4",
+    "@polka-codes/cli-shared": "0.9.6",
+    "@polka-codes/core": "0.9.6",
     "ai": "5.0.0-beta.24",
     "commander": "^13.0.0",
     "dotenv": "^16.4.7",