@polka-codes/core 0.9.4 → 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>;
 }
@@ -628,6 +629,7 @@ declare const _default_7: {
             path: z.ZodString;
             maxCount: z.ZodDefault<z.ZodOptional<z.ZodCoercedNumber<unknown>>>;
             recursive: z.ZodPipe<z.ZodTransform<unknown, unknown>, z.ZodDefault<z.ZodOptional<z.ZodBoolean>>>;
+            includeIgnored: z.ZodPipe<z.ZodTransform<unknown, unknown>, z.ZodDefault<z.ZodOptional<z.ZodBoolean>>>;
         }, z.core.$strip>;
         readonly permissionLevel: PermissionLevel.Read;
     }, FilesystemProvider>;
@@ -638,6 +640,7 @@ declare const _default_7: {
         path: z.ZodString;
         maxCount: z.ZodDefault<z.ZodOptional<z.ZodCoercedNumber<unknown>>>;
         recursive: z.ZodPipe<z.ZodTransform<unknown, unknown>, z.ZodDefault<z.ZodOptional<z.ZodBoolean>>>;
+        includeIgnored: z.ZodPipe<z.ZodTransform<unknown, unknown>, z.ZodDefault<z.ZodOptional<z.ZodBoolean>>>;
     }, z.core.$strip>;
     permissionLevel: PermissionLevel.Read;
 };
@@ -652,6 +655,7 @@ declare const _default_8: {
         readonly description: "Request to read the contents of one or multiple files at the specified paths. Use comma separated paths to read multiple files. Use this when you need to examine the contents of an existing file you do not know the contents of, for example to analyze code, review text files, or extract information from configuration files. May not be suitable for other types of binary files, as it returns the raw content as a string. Try to list all the potential files are relevent to the task, and then use this tool to read all the relevant files.";
         readonly parameters: z.ZodObject<{
             path: z.ZodPipe<z.ZodTransform<string[], unknown>, z.ZodArray<z.ZodString>>;
+            includeIgnored: z.ZodPipe<z.ZodTransform<unknown, unknown>, z.ZodDefault<z.ZodOptional<z.ZodBoolean>>>;
         }, z.core.$strip>;
         readonly permissionLevel: PermissionLevel.Read;
     }, FilesystemProvider>;
@@ -660,6 +664,7 @@ declare const _default_8: {
     description: "Request to read the contents of one or multiple files at the specified paths. Use comma separated paths to read multiple files. Use this when you need to examine the contents of an existing file you do not know the contents of, for example to analyze code, review text files, or extract information from configuration files. May not be suitable for other types of binary files, as it returns the raw content as a string. Try to list all the potential files are relevent to the task, and then use this tool to read all the relevant files.";
     parameters: z.ZodObject<{
         path: z.ZodPipe<z.ZodTransform<string[], unknown>, z.ZodArray<z.ZodString>>;
+        includeIgnored: z.ZodPipe<z.ZodTransform<unknown, unknown>, z.ZodDefault<z.ZodOptional<z.ZodBoolean>>>;
     }, z.core.$strip>;
     permissionLevel: PermissionLevel.Read;
 };
@@ -717,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;
 };
@@ -807,11 +812,11 @@ export { ExitReason as ExitReason_alias_1 }
 export { ExitReason as ExitReason_alias_2 }
 
 declare type FilesystemProvider = {
-    readFile?: (path: string) => Promise<string | undefined>;
+    readFile?: (path: string, includeIgnored: boolean) => Promise<string | undefined>;
     writeFile?: (path: string, content: string) => Promise<void>;
     removeFile?: (path: string) => Promise<void>;
     renameFile?: (sourcePath: string, targetPath: string) => Promise<void>;
-    listFiles?: (path: string, recursive: boolean, maxCount: number) => Promise<[string[], boolean]>;
+    listFiles?: (path: string, recursive: boolean, maxCount: number, includeIgnored: boolean) => Promise<[string[], boolean]>;
     searchFiles?: (path: string, regex: string, filePattern: string) => Promise<string[]>;
 };
 export { FilesystemProvider }
@@ -1023,11 +1028,11 @@ export { makeTool }
 export { makeTool as makeTool_alias_1 }
 
 declare class MockProvider implements ToolProvider {
-    readFile(_path: string): Promise<string>;
+    readFile(_path: string, _includeIgnored?: boolean): Promise<string>;
     writeFile(_path: string, _content: string): Promise<void>;
     removeFile(_path: string): Promise<void>;
     renameFile(_sourcePath: string, _targetPath: string): Promise<void>;
-    listFiles(_path: string, _recursive: boolean, _maxCount: number): Promise<[string[], boolean]>;
+    listFiles(_path: string, _recursive: boolean, _maxCount: number, _includeIgnored?: boolean): Promise<[string[], boolean]>;
     searchFiles(_path: string, _regex: string, _filePattern: string): Promise<string[]>;
     executeCommand(_command: string, _needApprove: boolean): Promise<{
         stdout: string;
@@ -1571,6 +1576,7 @@ export declare const toolInfo_alias_7: {
         path: z.ZodString;
         maxCount: z.ZodDefault<z.ZodOptional<z.ZodCoercedNumber<unknown>>>;
         recursive: z.ZodPipe<z.ZodTransform<unknown, unknown>, z.ZodDefault<z.ZodOptional<z.ZodBoolean>>>;
+        includeIgnored: z.ZodPipe<z.ZodTransform<unknown, unknown>, z.ZodDefault<z.ZodOptional<z.ZodBoolean>>>;
     }, z.core.$strip>;
     readonly permissionLevel: PermissionLevel.Read;
 };
@@ -1580,6 +1586,7 @@ export declare const toolInfo_alias_8: {
     readonly description: "Request to read the contents of one or multiple files at the specified paths. Use comma separated paths to read multiple files. Use this when you need to examine the contents of an existing file you do not know the contents of, for example to analyze code, review text files, or extract information from configuration files. May not be suitable for other types of binary files, as it returns the raw content as a string. Try to list all the potential files are relevent to the task, and then use this tool to read all the relevant files.";
     readonly parameters: z.ZodObject<{
         path: z.ZodPipe<z.ZodTransform<string[], unknown>, z.ZodArray<z.ZodString>>;
+        includeIgnored: z.ZodPipe<z.ZodTransform<unknown, unknown>, z.ZodDefault<z.ZodOptional<z.ZodBoolean>>>;
     }, z.core.$strip>;
     readonly permissionLevel: PermissionLevel.Read;
 };

package/dist/index.js

@@ -458,7 +458,15 @@ var toolInfo7 = {
         if (lower === "true") return true;
       }
       return val;
-    }, z7.boolean().optional().default(true)).describe("Whether to list files recursively. Use true for recursive listing, false or omit for top-level only.").meta({ usageValue: "true or false (optional)" })
+    }, z7.boolean().optional().default(true)).describe("Whether to list files recursively. Use true for recursive listing, false or omit for top-level only.").meta({ usageValue: "true or false (optional)" }),
+    includeIgnored: z7.preprocess((val) => {
+      if (typeof val === "string") {
+        const lower = val.toLowerCase();
+        if (lower === "false") return false;
+        if (lower === "true") return true;
+      }
+      return val;
+    }, z7.boolean().optional().default(false)).describe("Whether to include ignored files. Use true to include files ignored by .gitignore.").meta({ usageValue: "true or false (optional)" })
   }).meta({
     examples: [
       {
@@ -479,8 +487,8 @@ var handler7 = async (provider, args) => {
       message: "Not possible to list files. Abort."
     };
   }
-  const { path, maxCount, recursive } = toolInfo7.parameters.parse(args);
-  const [files, limitReached] = await provider.listFiles(path, recursive, maxCount);
+  const { path, maxCount, recursive, includeIgnored } = toolInfo7.parameters.parse(args);
+  const [files, limitReached] = await provider.listFiles(path, recursive, maxCount, includeIgnored);
   return {
     type: "Reply" /* Reply */,
     message: `<list_files_path>${path}</list_files_path>
@@ -509,7 +517,15 @@ var toolInfo8 = {
       if (!val) return [];
       const values = Array.isArray(val) ? val : [val];
       return values.flatMap((i) => typeof i === "string" ? i.split(",") : []).filter((s) => s.length > 0);
-    }, z8.array(z8.string())).describe("The path of the file to read").meta({ usageValue: "Comma separated paths here" })
+    }, z8.array(z8.string())).describe("The path of the file to read").meta({ usageValue: "Comma separated paths here" }),
+    includeIgnored: z8.preprocess((val) => {
+      if (typeof val === "string") {
+        const lower = val.toLowerCase();
+        if (lower === "false") return false;
+        if (lower === "true") return true;
+      }
+      return val;
+    }, z8.boolean().optional().default(false)).describe("Whether to include ignored files. Use true to include files ignored by .gitignore.").meta({ usageValue: "true or false (optional)" })
   }).meta({
     examples: [
       {
@@ -535,10 +551,10 @@ var handler8 = async (provider, args) => {
       message: "Not possible to read file. Abort."
     };
   }
-  const { path: paths } = toolInfo8.parameters.parse(args);
+  const { path: paths, includeIgnored } = toolInfo8.parameters.parse(args);
   const resp = [];
   for (const path of paths) {
-    const fileContent = await provider.readFile(path);
+    const fileContent = await provider.readFile(path, includeIgnored);
     if (!fileContent) {
       resp.push(`<read_file_file_content path="${path}" file_not_found="true" />`);
     } else {
@@ -546,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>`);
       }
     }
   }
@@ -855,9 +871,16 @@ 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);
+    const fileContent = await provider.readFile(path, false);
     if (fileContent == null) {
       return {
         type: "Error" /* Error */,
@@ -940,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 */,
@@ -955,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}`
     };
   }
 };
@@ -1037,7 +1067,7 @@ var writeToFile_default = {
 
 // src/tools/provider.ts
 var MockProvider = class {
-  async readFile(_path) {
+  async readFile(_path, _includeIgnored) {
     return "mock content";
   }
   async writeFile(_path, _content) {
@@ -1049,7 +1079,7 @@ var MockProvider = class {
   async renameFile(_sourcePath, _targetPath) {
     return;
   }
-  async listFiles(_path, _recursive, _maxCount) {
+  async listFiles(_path, _recursive, _maxCount, _includeIgnored) {
     return [["mock-file.txt"], false];
   }
   async searchFiles(_path, _regex, _filePattern) {
@@ -1836,6 +1866,7 @@ ${instance.prompt}`;
         resetTimeout();
         const streamTextOptions = {
           model: this.ai,
+          temperature: 0,
           messages,
           providerOptions: this.config.parameters?.providerOptions,
           onChunk: async ({ chunk }) => {
@@ -1904,6 +1935,13 @@ ${instance.prompt}`;
     }
     this.#messages.push(...respMessages);
     if (this.config.toolFormat === "native") {
+      const assistantText = respMessages.map((msg) => {
+        if (typeof msg.content === "string") {
+          return msg.content;
+        }
+        return msg.content.map((part) => part.type === "text" || part.type === "reasoning" ? part.text : "").join("");
+      }).join("\n");
+      await this.#callback({ kind: "EndRequest" /* EndRequest */, agent: this, message: assistantText });
       return respMessages.flatMap((msg) => {
         if (msg.role === "assistant") {
           const content = msg.content;
@@ -2598,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;
     }
   }
 };
@@ -3193,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
+
+**Context**:
+- Implementing changes for issue #123 - Focus on clean code and maintainability
 
-This PR removes unnecessary debug print statements and updates order validation
-to use the new validate_and_process method for improved 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>
 
@@ -3494,6 +3527,7 @@ ${output}`,
 var executeTool = async (definition, ai, params, usageMeter) => {
   const resp = await generateText({
     model: ai,
+    temperature: 0,
     system: definition.prompt,
     messages: [
       {

package/package.json

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