@polka-codes/core 0.9.5 → 0.9.6

package/dist/_tsup-dts-rollup.d.ts

@@ -246,6 +246,7 @@ export declare const codeFixingStrategies = "\n====\n\nCODE FIXING STRATEGIES\n\
  * Using Scripts: format, check, test
  */
 declare class CoderAgent extends AgentBase {
+    #private;
     constructor(options: CoderAgentOptions);
     protected onBeforeInvokeTool(name: string, _args: Record<string, string>): Promise<ToolResponse | undefined>;
 }
@@ -721,7 +722,7 @@ export declare const default_alias_1: {
 export declare const default_alias_2: {
     readonly name: "generateGithubPullRequestDetails";
     readonly description: "Generates a GitHub pull request title and description from git commits";
-    readonly prompt: "\n# Generate Github Pull Request Details\n\nYou are given:\n- A branch name in <tool_input_branch_name>.\n- An optional context message in <tool_input_context> (which may or may not be present).\n- All commit messages combined in <tool_input_commit_messages>.\n- All diffs combined in <tool_input_commit_diff>.\n\nYour task:\n1. Consider the optional context (if provided).\n   - If an issue number is found, add \"Closes #xxx\" at the beginning of the PR description\n   - IMPORTANT: Use ONLY the exact format \"Closes #xxx\" at the beginning of the description\n   - DO NOT use variations like \"Closes issue #xxx\" or other formats\n2. Analyze the combined commit messages and diffs.\n3. Produce a single GitHub Pull Request title.\n4. Produce a Pull Request description that explains the changes.\n\nUse the following template for the Pull Request description:\n\n---\n**Context (if provided)**:\n- Acknowledge any guiding concerns or instructions.\n\n**Summary of Changes**:\n- Provide a concise list or overview of what changed.\n\n**Highlights of Changed Code**:\n- Mention only the specific sections or functionalities updated, without showing full surrounding context.\n\n**Additional Information (if needed)**:\n- Testing steps (if applicable).\n- Any notes or caveats.\n\n---\n\nOutput format:\n<tool_output>\n  <tool_output_pr_title>YOUR PR TITLE HERE</tool_output_pr_title>\n  <tool_output_pr_description>\n    YOUR PR DESCRIPTION HERE\n  </tool_output_pr_description>\n</tool_output>\n\nBelow is an **example** of the input and output:\n\nExample Input:\n<tool_input>\n<tool_input_branch_name>feature/refactor-logging</tool_input_branch_name>\n<tool_input_context>Implementing changes for issue #123 - Focus on clean code and maintainability</tool_input_context>\n<tool_input_commit_messages>\nRemove debug logs\nRefactor order validation logic\n</tool_input_commit_messages>\n<tool_input_commit_diff>\ndiff --git a/user_service.py b/user_service.py\n- print(\"Debug info\")\n+ # Removed debug print statements\n\ndiff --git a/order_service.py b/order_service.py\n- if is_valid_order(order):\n-     process_order(order)\n+ validate_and_process(order)\n</tool_input_commit_diff>\n</tool_input>\n\nExample Output:\n<tool_output>\n<tool_output_pr_title>Refactor Order Validation and Remove Debug Logs</tool_output_pr_title>\n<tool_output_pr_description>\ncloses #123\n\nThis PR removes unnecessary debug print statements and updates order validation\nto use the new validate_and_process method for improved maintainability.\n</tool_output_pr_description>\n</tool_output>\n\n---\n\nUse the above format whenever you receive <tool_input> that may include a branch name, an optional context, aggregated commit messages in a single tag, and a combined diff in a single tag. Provide your final output strictly in <tool_output> with <tool_output_pr_title> and <tool_output_pr_description>. Only highlight the changed code and avoid including the context around the changes in the description.\n";
+    readonly prompt: "\n# Generate Github Pull Request Details\n\nYou are given:\n- A branch name in <tool_input_branch_name>.\n- An optional context message in <tool_input_context> (which may or may not be present).\n- All commit messages combined in <tool_input_commit_messages>.\n- All diffs combined in <tool_input_commit_diff>.\n\nYour task:\n1. Consider the optional context (if provided).\n   - If an issue number is found, add \"Closes #xxx\" at the beginning of the PR description\n   - IMPORTANT: Use ONLY the exact format \"Closes #xxx\" at the beginning of the description\n   - DO NOT use variations like \"Closes issue #xxx\" or other formats\n2. Analyze the combined commit messages and diffs.\n3. Produce a single GitHub Pull Request title.\n4. Produce a Pull Request description that explains the changes.\n\nUse the following template for the Pull Request description:\n\n---\n**Context (if provided)**:\n- Acknowledge any guiding concerns or instructions.\n\n**Summary of Changes**:\n- Provide a concise list or overview of what changed.\n\n**Highlights of Changed Code**:\n- Mention only the specific sections or functionalities updated, without showing full surrounding context.\n\n**Additional Information (if needed)**:\n- Testing steps (if applicable).\n- Any notes or caveats.\n\n---\n\nOutput format:\n<tool_output>\n  <tool_output_pr_title>YOUR PR TITLE HERE</tool_output_pr_title>\n  <tool_output_pr_description>\n    YOUR PR DESCRIPTION HERE\n  </tool_output_pr_description>\n</tool_output>\n\nBelow is an **example** of the input and output:\n\nExample Input:\n<tool_input>\n<tool_input_branch_name>feature/refactor-logging</tool_input_branch_name>\n<tool_input_context>Implementing changes for issue #123 - Focus on clean code and maintainability</tool_input_context>\n<tool_input_commit_messages>\nRemove debug logs\nRefactor order validation logic\n</tool_input_commit_messages>\n<tool_input_commit_diff>\ndiff --git a/user_service.py b/user_service.py\n- print(\"Debug info\")\n+ # Removed debug print statements\n\ndiff --git a/order_service.py b/order_service.py\n- if is_valid_order(order):\n-     process_order(order)\n+ validate_and_process(order)\n</tool_input_commit_diff>\n</tool_input>\n\nExample Output:\n<tool_output>\n<tool_output_pr_title>Refactor Order Validation and Remove Debug Logs</tool_output_pr_title>\n<tool_output_pr_description>\nCloses #123\n\n**Context**:\n- Implementing changes for issue #123 - Focus on clean code and maintainability\n\n**Summary of Changes**:\n- Refactored order validation logic to use a new `validate_and_process` method.\n- Removed debug print statements from `user_service.py`.\n\n**Highlights of Changed Code**:\n- `order_service.py`: Replaced direct call to `process_order` with `validate_and_process`.\n- `user_service.py`: Removed `print(\"Debug info\")`.\n</tool_output_pr_description>\n</tool_output>\n\n---\n\nUse the above format whenever you receive <tool_input> that may include a branch name, an optional context, aggregated commit messages in a single tag, and a combined diff in a single tag. Provide your final output strictly in <tool_output> with <tool_output_pr_title> and <tool_output_pr_description>. Only highlight the changed code and avoid including the context around the changes in the description.\n";
     readonly formatInput: (params: Input) => string;
     readonly parseOutput: (output: string) => Output;
 };

package/dist/index.js

@@ -562,7 +562,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>`);
       }
     }
   }
@@ -871,8 +871,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 {
@@ -956,8 +963,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 */,
@@ -971,8 +985,8 @@ ${files.join("\n")}
     };
   } catch (error) {
     return {
-      type: "Invalid" /* Invalid */,
-      message: `Invalid arguments for search_files: ${error}`
+      type: "Error" /* Error */,
+      message: `Error searching files: ${error}`
     };
   }
 };
@@ -2622,52 +2636,39 @@ var CoderAgent = class extends AgentBase {
       usageMeter: options.usageMeter ?? new UsageMeter()
     });
   }
-  async onBeforeInvokeTool(name, _args) {
-    if (name !== 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) {
+    const script = this.config.scripts?.[scriptName];
+    const command = typeof script === "string" ? script : script?.command;
+    if (command) {
       try {
-        await executeCommand(formatCommand, false);
-      } catch (error) {
-        console.warn(`Failed to format code using command: ${formatCommand}`, error);
-      }
-    }
-    const check = this.config.scripts?.check;
-    const checkCommand = typeof check === "string" ? check : check?.command;
-    if (checkCommand) {
-      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 (error) {
-        console.warn(`Failed to check code using command: ${checkCommand}`, error);
+        console.warn(`Failed to run ${scriptName} using command: ${command}`, error);
       }
     }
-    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 (error) {
-        console.warn(`Failed to test code using command: ${testCommand}`, error);
-      }
+  }
+  async onBeforeInvokeTool(name, _args) {
+    if (name !== 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;
     }
   }
 };
@@ -3217,10 +3218,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
 
-This PR removes unnecessary debug print statements and updates order validation
-to use the new validate_and_process method for improved maintainability.
+**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\`.
+
+**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>
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@polka-codes/core",
-  "version": "0.9.5",
+  "version": "0.9.6",
   "license": "AGPL-3.0",
   "author": "github@polka.codes",
   "type": "module",