@polka-codes/runner 0.9.24 → 0.9.26

package/dist/index.js

@@ -30458,7 +30458,7 @@ var {
   Help
 } = import__.default;
 // package.json
-var version = "0.9.24";
+var version = "0.9.26";
 
 // src/runner.ts
 import { execSync } from "node:child_process";
@@ -41582,8 +41582,7 @@ var toolInfo = {
         }
       }
     ]
-  }),
-  permissionLevel: 0 /* None */
+  })
 };
 var handler = async (provider, args) => {
   if (!provider.askFollowupQuestion) {
@@ -41645,8 +41644,7 @@ var toolInfo2 = {
         }
       }
     ]
-  }),
-  permissionLevel: 0 /* None */
+  })
 };
 var handler2 = async (provider, args) => {
   const parsed = toolInfo2.parameters.safeParse(args);
@@ -41709,8 +41707,7 @@ var toolInfo3 = {
         }
       }
     ]
-  }),
-  permissionLevel: 0 /* None */
+  })
 };
 var handler3 = async (_provider, args) => {
   const parsed = toolInfo3.parameters.safeParse(args);
@@ -41766,8 +41763,7 @@ var toolInfo4 = {
         }
       }
     ]
-  }),
-  permissionLevel: 3 /* Arbitrary */
+  })
 };
 var handler4 = async (provider, args) => {
   if (!provider.executeCommand) {
@@ -41857,8 +41853,7 @@ var toolInfo5 = {
         }
       }
     ]
-  }),
-  permissionLevel: 1 /* Read */
+  })
 };
 var handler5 = async (provider, args) => {
   if (!provider.fetchUrl) {
@@ -41934,8 +41929,7 @@ var toolInfo6 = {
         }
       }
     ]
-  }),
-  permissionLevel: 0 /* None */
+  })
 };
 var handler6 = async (_provider, args) => {
   const parsed = toolInfo6.parameters.safeParse(args);
@@ -42002,8 +41996,7 @@ var toolInfo7 = {
         }
       }
     ]
-  }),
-  permissionLevel: 1 /* Read */
+  })
 };
 var handler7 = async (provider, args) => {
   if (!provider.listFiles) {
@@ -42044,8 +42037,7 @@ var toolInfo8 = {
   description: "Read a binary file from a URL or local path. Use file:// prefix to access local files. This can be used to access non-text files such as PDFs or images.",
   parameters: exports_external.object({
     url: exports_external.string().describe("The URL or local path of the file to read.")
-  }),
-  permissionLevel: 1 /* Read */
+  })
 };
 var handler8 = async (provider, args) => {
   if (!provider.readBinaryFile) {
@@ -42129,8 +42121,7 @@ var toolInfo9 = {
         }
       }
     ]
-  }),
-  permissionLevel: 1 /* Read */
+  })
 };
 var handler9 = async (provider, args) => {
   if (!provider.readFile) {
@@ -42189,8 +42180,7 @@ var toolInfo10 = {
         }
       }
     ]
-  }),
-  permissionLevel: 2 /* Write */
+  })
 };
 var handler10 = async (provider, args) => {
   if (!provider.removeFile) {
@@ -42247,8 +42237,7 @@ var toolInfo11 = {
         }
       }
     ]
-  }),
-  permissionLevel: 2 /* Write */
+  })
 };
 var handler11 = async (provider, args) => {
   if (!provider.renameFile) {
@@ -42461,8 +42450,7 @@ function oldFeature() {
         }
       }
     ]
-  }),
-  permissionLevel: 2 /* Write */
+  })
 };
 var handler12 = async (provider, args) => {
   if (!provider.readFile || !provider.writeFile) {
@@ -42568,8 +42556,7 @@ var toolInfo13 = {
         }
       }
     ]
-  }),
-  permissionLevel: 1 /* Read */
+  })
 };
 var handler13 = async (provider, args) => {
   if (!provider.searchFiles) {
@@ -42654,8 +42641,7 @@ export default App;
         }
       }
     ]
-  }),
-  permissionLevel: 2 /* Write */
+  })
 };
 var handler14 = async (provider, args) => {
   if (!provider.writeFile) {
@@ -42703,7 +42689,6 @@ var getAvailableTools = ({
   provider: provider2,
   allTools: allTools2,
   hasAgent,
-  permissionLevel,
   interactive
 }) => {
   const filteredTools = interactive ? allTools2 : allTools2.filter((tool) => tool.name !== askFollowupQuestion_default.name);
@@ -42716,7 +42701,7 @@ var getAvailableTools = ({
           continue;
       }
     }
-    if (tool.isAvailable(provider2) && tool.permissionLevel <= permissionLevel) {
+    if (tool.isAvailable(provider2)) {
       tools.push(tool);
     }
   }
@@ -66863,14 +66848,23 @@ Retrying request ${i + 2} of ${retryCount}`);
       return { type: "exit", reason: { type: "Pause", responses: toolResponses } };
     }
     if (toolResponses.length === 0) {
+      if (this.config.requireToolUse) {
+        return {
+          type: "reply",
+          message: [
+            {
+              role: "user",
+              content: responsePrompts.requireUseToolNative
+            }
+          ]
+        };
+      }
       return {
-        type: "reply",
-        message: [
-          {
-            role: "user",
-            content: responsePrompts.requireUseToolNative
-          }
-        ]
+        type: "exit",
+        reason: {
+          type: "Exit" /* Exit */,
+          message: response.filter((c) => c.type === "text").map((c) => c.content).join("")
+        }
       };
     }
     const mediaUserMessage = medias.length > 0 ? [
@@ -66912,14 +66906,23 @@ Retrying request ${i + 2} of ${retryCount}`);
       };
     }
     if (toolResponses.length === 0) {
+      if (this.config.requireToolUse) {
+        return {
+          type: "reply",
+          message: [
+            {
+              role: "user",
+              content: responsePrompts.requireUseTool
+            }
+          ]
+        };
+      }
       return {
-        type: "reply",
-        message: [
-          {
-            role: "user",
-            content: responsePrompts.requireUseTool
-          }
-        ]
+        type: "exit",
+        reason: {
+          type: "Exit" /* Exit */,
+          message: response.filter((c) => c.type === "text").map((c) => c.content).join("")
+        }
       };
     }
     const finalResp = toolResponses.filter((resp) => resp.type === "response").flatMap(({ tool: tool2, response: response2 }) => responsePrompts.toolResults(tool2, response2));
@@ -67009,14 +67012,15 @@ ${customScripts(scripts)}
 `;
 
 // ../core/src/Agent/AnalyzerAgent/index.ts
+var agentTools = [askFollowupQuestion_default, attemptCompletion_default, delegate_default, fetchUrl_default, handOver_default, listFiles_default, readBinaryFile_default, readFile_default, searchFiles_default];
+
 class AnalyzerAgent extends AgentBase {
   constructor(options) {
-    const combinedTools = [...options.additionalTools ?? [], ...Object.values(exports_allTools)];
+    const combinedTools = [...options.additionalTools ?? [], ...agentTools];
     const tools = getAvailableTools({
       provider: options.provider,
       allTools: combinedTools,
       hasAgent: (options.agents?.length ?? 0) > 0,
-      permissionLevel: 1 /* Read */,
       interactive: true
     });
     const toolNamePrefix = options.toolFormat === "native" ? "" : "tool_";
@@ -67036,7 +67040,8 @@ class AnalyzerAgent extends AgentBase {
       toolFormat: options.toolFormat,
       parameters: options.parameters ?? {},
       usageMeter: options.usageMeter ?? new UsageMeter,
-      requestTimeoutSeconds: options.requestTimeoutSeconds
+      requestTimeoutSeconds: options.requestTimeoutSeconds,
+      requireToolUse: options.requireToolUse ?? true
     });
   }
   onBeforeInvokeTool() {
@@ -67140,7 +67145,6 @@ class CodeFixerAgent extends AgentBase {
       provider: options.provider,
       allTools: combinedTools,
       hasAgent: (options.agents?.length ?? 0) > 0,
-      permissionLevel: 3 /* Arbitrary */,
       interactive: true
     });
     const toolNamePrefix = options.toolFormat === "native" ? "" : "tool_";
@@ -67160,7 +67164,8 @@ class CodeFixerAgent extends AgentBase {
       toolFormat: options.toolFormat,
       parameters: options.parameters ?? {},
       usageMeter: options.usageMeter ?? new UsageMeter,
-      requestTimeoutSeconds: options.requestTimeoutSeconds
+      requestTimeoutSeconds: options.requestTimeoutSeconds,
+      requireToolUse: options.requireToolUse ?? true
     });
     this.#maxRetries = options.maxRetries ?? 5;
   }
@@ -67345,7 +67350,6 @@ class CoderAgent extends AgentBase {
       provider: options.provider,
       allTools: combinedTools,
       hasAgent: (options.agents?.length ?? 0) > 0,
-      permissionLevel: 3 /* Arbitrary */,
       interactive: true
     });
     const toolNamePrefix = options.toolFormat === "native" ? "" : "tool_";
@@ -67365,7 +67369,8 @@ class CoderAgent extends AgentBase {
       toolFormat: options.toolFormat,
       parameters: options.parameters ?? {},
       usageMeter: options.usageMeter ?? new UsageMeter,
-      requestTimeoutSeconds: options.requestTimeoutSeconds
+      requestTimeoutSeconds: options.requestTimeoutSeconds,
+      requireToolUse: options.requireToolUse ?? true
     });
   }
   async#runScript(scriptName, shouldReplyWithError) {
@@ -67952,7 +67957,7 @@ var toolInfo15 = {
       return val;
     }, exports_external.boolean().optional().default(false)).describe("Get staged changes instead of unstaged changes."),
     commitRange: exports_external.string().optional().describe('The commit range to get the diff for (e.g., "main...HEAD").'),
-    file: exports_external.string().optional().describe("Get the diff for a specific file."),
+    file: exports_external.string().describe("Get the diff for a specific file."),
     contextLines: exports_external.coerce.number().optional().default(5).describe("Number of context lines to include around changes."),
     includeLineNumbers: exports_external.preprocess((val) => {
       if (typeof val === "string") {
@@ -67964,8 +67969,7 @@ var toolInfo15 = {
       }
       return val;
     }, exports_external.boolean().optional().default(false)).describe("Annotate the diff with line numbers for additions and deletions.")
-  }),
-  permissionLevel: 1 /* Read */
+  })
 };
 var handler15 = async (provider2, args) => {
   if (!provider2.executeCommand) {
@@ -68049,50 +68053,64 @@ var prompt5 = `
 You are a senior software engineer reviewing code changes.
 
 ## Critical Instructions
-**ONLY review the actual changes shown in the diff.** Do not comment on existing code that wasn't modified.
+- **ONLY review the actual changes shown in the diff.** Do not comment on existing code that wasn't modified.
+- **ONLY run git_diff on files that are reviewable source/config files** per the "File Selection for git_diff" rules below. Do not pass excluded files to git_diff.
+
+## File Selection for git_diff
+Use <file_status> to decide which files to diff. Include only files likely to contain human-authored source or meaningful configuration.
+
+Include (run git_diff):
+- Application/source code
+- UI/templates/assets code
+- Infra/config that affects behavior
+
+Exclude (do NOT run git_diff; do not review):
+- Lockfiles
+- Generated/build artifacts & deps
+- Test artifacts/snapshots
+- Data and fixtures
+- Binary/media/minified/maps
 
 ## Viewing Changes
-- **Use git_diff** to inspect the actual code changes for each relevant file.
-  - **Pull request**: use the provided commit range for the git_diff tool with contextLines: 5 and includeLineNumbers: true
-  - **Local changes**: diff staged or unstaged files using the git_diff tool with contextLines: 5 and includeLineNumbers: true
-- The diff will include line number annotations: [Line N] for additions and [Line N removed] for deletions
-- If a pull request is present you may receive:
+- For each included file, **use git_diff** to inspect the actual code changes:
+  - **Pull request:** use the provided commit range for the git_diff tool with contextLines: 5 and includeLineNumbers: true, but only surface and review the included files.
+  - **Local changes:** diff staged or unstaged included files using git_diff with contextLines: 5 and includeLineNumbers: true.
+- The diff will include line number annotations: [Line N] for additions and [Line N removed] for deletions.
+- You may receive:
   - <pr_title>
   - <pr_description>
   - <commit_messages>
 - A <review_instructions> tag tells you the focus of the review.
-- File status information is provided in <file_status> - use this to understand which files were modified, added, deleted, or renamed.
+- Use <file_status> to understand which files were modified, added, deleted, or renamed and to apply the inclusion/exclusion rules above.
 
 ## Line Number Reporting
-- **IMPORTANT**: Use the line numbers from the annotations in the diff output
-- For additions: Look for [Line N] annotations after the + lines
-- For deletions: Look for [Line N removed] annotations after the - lines
-- For modifications: Report the line number of the new/current code (from [Line N] annotations)
-- Report single lines as "N" and ranges as "N-M"
+- Use the line numbers from the annotations in the diff output.
+- For additions: use the number from the [Line N] annotation after the + line.
+- For deletions: use the number from the [Line N removed] annotation after the - line.
+- For modifications: report the line number of the new/current code (from [Line N]).
+- Report single lines as "N" and ranges as "N-M".
 
 ## Review Guidelines
 Focus exclusively on the changed lines (+ additions, - deletions, modified lines):
-- **Specific issues**: Point to exact problems in the changed code with accurate line references from the annotations
-- **Actionable fixes**: Provide concrete solutions, not vague suggestions
-- **Clear reasoning**: Explain why each issue matters and how to fix it
-- **Avoid generic advice**: No generic suggestions like "add more tests", "improve documentation", or "follow best practices" unless directly related to a specific problem in the diff
+- **Specific issues:** Point to exact problems in the changed code with accurate line references from the annotations.
+- **Actionable fixes:** Provide concrete solutions, not vague suggestions.
+- **Clear reasoning:** Explain why each issue matters and how to fix it.
+- **Avoid generic advice** unless directly tied to a specific problem visible in the diff.
 
 ## What NOT to review
-- Existing unchanged code
-- Overall project structure or architecture (unless directly impacted by changes)
-- Generic best practices unrelated to the specific changes
-- Missing features or functionality not part of this diff
+- Files excluded by the "File Selection for git_diff" rules (do not diff or comment on them).
+- Existing unchanged code.
+- Overall project structure/architecture unless directly impacted by the changes.
+- Missing features or functionality not part of this diff.
 
 ## Output Format
-Do **not** include praise or positive feedback. Ignore generated files such as lock files.
+Do not include praise or positive feedback.
 Only include reviews for actual issues found in the changed code.
 
 Return your review as a JSON object inside a \`\`\`json block, wrapped like:
-<tool_attempt_completion>
-<tool_parameter_result>
 \`\`\`json
 {
-  "overview": "Summary of specific issues found in the diff changes, or 'No issues found' if the changes look good.",
+  "overview": "Summary of specific issues found in the diff changes, 'No issues found', or 'No reviewable changes' if all modified files were excluded.",
   "specificReviews": [
     {
       "file": "path/filename.ext",
@@ -68102,8 +68120,6 @@ Return your review as a JSON object inside a \`\`\`json block, wrapped like:
   ]
 }
 \`\`\`
-</tool_parameter_result>
-</tool_attempt_completion>
 `;
 var reviewDiff_default = {
   name: "reviewDiff",
@@ -68165,7 +68181,8 @@ ${output}`,
   agent: (options) => {
     return new AnalyzerAgent({
       ...options,
-      additionalTools: [gitDiff_default]
+      additionalTools: [gitDiff_default],
+      requireToolUse: false
     });
   }
 };
@@ -68227,8 +68244,50 @@ var generateGithubPullRequestDetails = makeTool(generateGithubPullRequestDetails
 var reviewDiff = makeAgentTool(reviewDiff_default);
 var generateProjectConfig = makeMultiAgentTool(generateProjectConfig_default);
 var createNewProject = makeMultiAgentTool(createNewProject_default);
-// ../cli-shared/src/config.ts
+// ../core/src/workflow/utils.ts
 var import_lodash2 = __toESM(require_lodash(), 1);
+// ../core/src/workflow/builder.ts
+class StepsBuilder {
+  #steps = [];
+  build() {
+    if (this.#steps.length === 0) {
+      throw new Error("A workflow must have at least one step.");
+    }
+    const rootStep = this.#steps.length > 1 ? {
+      id: "root",
+      type: "sequential",
+      steps: this.#steps
+    } : this.#steps[0];
+    return rootStep;
+  }
+  step(step) {
+    this.#steps.push(step);
+    return this;
+  }
+  parallel(id, step) {
+    return this.step({
+      id,
+      type: "parallel",
+      step
+    });
+  }
+  custom(id, run) {
+    return this.step({
+      id,
+      type: "custom",
+      run
+    });
+  }
+  agent(id, spec2) {
+    return this.step({
+      ...spec2,
+      id,
+      type: "agent"
+    });
+  }
+}
+// ../cli-shared/src/config.ts
+var import_lodash3 = __toESM(require_lodash(), 1);
 
 // ../cli-shared/node_modules/yaml/dist/index.js
 var composer = require_composer();
@@ -70873,9 +70932,9 @@ class ZodUnion4 extends ZodType4 {
     return this._def.options;
   }
 }
-ZodUnion4.create = (types, params) => {
+ZodUnion4.create = (types2, params) => {
   return new ZodUnion4({
-    options: types,
+    options: types2,
     typeName: ZodFirstPartyTypeKind4.ZodUnion,
     ...processCreateParams2(params)
   });
@@ -72132,7 +72191,7 @@ function mergeConfigs(configs) {
     return {};
   }
   const mergedConfig = configs.reduce((acc, config4) => {
-    const merged = import_lodash2.merge({}, acc, config4);
+    const merged = import_lodash3.merge({}, acc, config4);
     let accRules = acc.rules ?? [];
     if (typeof accRules === "string") {
       accRules = [accRules];
@@ -73509,7 +73568,7 @@ function lookup(path) {
   }
   return $types[extension2] || false;
 }
-function populateMaps(extensions, types) {
+function populateMaps(extensions, types2) {
   Object.keys(db).forEach(function forEachMimeType(type) {
     var mime = db[type];
     var exts = mime.extensions;
@@ -73519,10 +73578,10 @@ function populateMaps(extensions, types) {
     extensions[type] = exts;
     for (var i = 0;i < exts.length; i++) {
       var extension2 = exts[i];
-      types[extension2] = _preferredType(extension2, types[extension2], type);
-      const legacyType = _preferredTypeLegacy(extension2, types[extension2], type);
-      if (legacyType !== types[extension2]) {
-        $_extensionConflicts.push([extension2, legacyType, types[extension2]]);
+      types2[extension2] = _preferredType(extension2, types2[extension2], type);
+      const legacyType = _preferredTypeLegacy(extension2, types2[extension2], type);
+      if (legacyType !== types2[extension2]) {
+        $_extensionConflicts.push([extension2, legacyType, types2[extension2]]);
       }
     }
   });
@@ -73767,10 +73826,10 @@ var getProvider = (_agentName, _config, options = {}) => {
         mediaType
       };
     },
-    executeCommand: (command, _needApprove) => {
+    executeCommand: (command2, _needApprove) => {
       return new Promise((resolve4, reject) => {
-        options.command?.onStarted(command);
-        const child = spawn2(command, [], {
+        options.command?.onStarted(command2);
+        const child = spawn2(command2, [], {
           shell: true,
           stdio: "pipe"
         });
@@ -76461,9 +76520,9 @@ class ZodUnion5 extends ZodType5 {
     return this._def.options;
   }
 }
-ZodUnion5.create = (types, params) => {
+ZodUnion5.create = (types2, params) => {
   return new ZodUnion5({
-    options: types,
+    options: types2,
     typeName: ZodFirstPartyTypeKind5.ZodUnion,
     ...processCreateParams3(params)
   });
@@ -78072,8 +78131,8 @@ class Runner {
     const config5 = loadConfig() ?? {};
     this.provider = getProvider("coder", config5 || {}, {
       command: {
-        onStarted(command) {
-          console.log(`$ >>>> $ ${command}`);
+        onStarted(command2) {
+          console.log(`$ >>>> $ ${command2}`);
         },
         onStdout(data) {
           process.stdout.write(data);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@polka-codes/runner",
-  "version": "0.9.24",
+  "version": "0.9.26",
   "license": "AGPL-3.0",
   "author": "github@polka.codes",
   "type": "module",