@polka-codes/cli-shared 0.9.24 → 0.9.26

package/dist/index.js

@@ -36220,8 +36220,7 @@ var toolInfo = {
         }
       }
     ]
-  }),
-  permissionLevel: 0 /* None */
+  })
 };
 var handler = async (provider, args) => {
   if (!provider.askFollowupQuestion) {
@@ -36283,8 +36282,7 @@ var toolInfo2 = {
         }
       }
     ]
-  }),
-  permissionLevel: 0 /* None */
+  })
 };
 var handler2 = async (provider, args) => {
   const parsed = toolInfo2.parameters.safeParse(args);
@@ -36347,8 +36345,7 @@ var toolInfo3 = {
         }
       }
     ]
-  }),
-  permissionLevel: 0 /* None */
+  })
 };
 var handler3 = async (_provider, args) => {
   const parsed = toolInfo3.parameters.safeParse(args);
@@ -36404,8 +36401,7 @@ var toolInfo4 = {
         }
       }
     ]
-  }),
-  permissionLevel: 3 /* Arbitrary */
+  })
 };
 var handler4 = async (provider, args) => {
   if (!provider.executeCommand) {
@@ -36495,8 +36491,7 @@ var toolInfo5 = {
         }
       }
     ]
-  }),
-  permissionLevel: 1 /* Read */
+  })
 };
 var handler5 = async (provider, args) => {
   if (!provider.fetchUrl) {
@@ -36572,8 +36567,7 @@ var toolInfo6 = {
         }
       }
     ]
-  }),
-  permissionLevel: 0 /* None */
+  })
 };
 var handler6 = async (_provider, args) => {
   const parsed = toolInfo6.parameters.safeParse(args);
@@ -36640,8 +36634,7 @@ var toolInfo7 = {
         }
       }
     ]
-  }),
-  permissionLevel: 1 /* Read */
+  })
 };
 var handler7 = async (provider, args) => {
   if (!provider.listFiles) {
@@ -36682,8 +36675,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) {
@@ -36767,8 +36759,7 @@ var toolInfo9 = {
         }
       }
     ]
-  }),
-  permissionLevel: 1 /* Read */
+  })
 };
 var handler9 = async (provider, args) => {
   if (!provider.readFile) {
@@ -36827,8 +36818,7 @@ var toolInfo10 = {
         }
       }
     ]
-  }),
-  permissionLevel: 2 /* Write */
+  })
 };
 var handler10 = async (provider, args) => {
   if (!provider.removeFile) {
@@ -36885,8 +36875,7 @@ var toolInfo11 = {
         }
       }
     ]
-  }),
-  permissionLevel: 2 /* Write */
+  })
 };
 var handler11 = async (provider, args) => {
   if (!provider.renameFile) {
@@ -37099,8 +37088,7 @@ function oldFeature() {
         }
       }
     ]
-  }),
-  permissionLevel: 2 /* Write */
+  })
 };
 var handler12 = async (provider, args) => {
   if (!provider.readFile || !provider.writeFile) {
@@ -37206,8 +37194,7 @@ var toolInfo13 = {
         }
       }
     ]
-  }),
-  permissionLevel: 1 /* Read */
+  })
 };
 var handler13 = async (provider, args) => {
   if (!provider.searchFiles) {
@@ -37292,8 +37279,7 @@ export default App;
         }
       }
     ]
-  }),
-  permissionLevel: 2 /* Write */
+  })
 };
 var handler14 = async (provider, args) => {
   if (!provider.writeFile) {
@@ -37341,7 +37327,6 @@ var getAvailableTools = ({
   provider: provider2,
   allTools: allTools2,
   hasAgent,
-  permissionLevel,
   interactive
 }) => {
   const filteredTools = interactive ? allTools2 : allTools2.filter((tool) => tool.name !== askFollowupQuestion_default.name);
@@ -37354,7 +37339,7 @@ var getAvailableTools = ({
           continue;
       }
     }
-    if (tool.isAvailable(provider2) && tool.permissionLevel <= permissionLevel) {
+    if (tool.isAvailable(provider2)) {
       tools.push(tool);
     }
   }
@@ -61501,14 +61486,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 ? [
@@ -61550,14 +61544,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));
@@ -61647,14 +61650,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_";
@@ -61674,7 +61678,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() {
@@ -61778,7 +61783,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_";
@@ -61798,7 +61802,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;
   }
@@ -61983,7 +61988,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_";
@@ -62003,7 +62007,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) {
@@ -62590,7 +62595,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") {
@@ -62602,8 +62607,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) {
@@ -62687,50 +62691,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",
@@ -62740,8 +62758,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",
@@ -62803,7 +62819,8 @@ ${output}`,
   agent: (options) => {
     return new AnalyzerAgent({
       ...options,
-      additionalTools: [gitDiff_default]
+      additionalTools: [gitDiff_default],
+      requireToolUse: false
     });
   }
 };
@@ -62865,8 +62882,50 @@ var generateGithubPullRequestDetails = makeTool(generateGithubPullRequestDetails
 var reviewDiff = makeAgentTool(reviewDiff_default);
 var generateProjectConfig = makeMultiAgentTool(generateProjectConfig_default);
 var createNewProject = makeMultiAgentTool(createNewProject_default);
-// 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"
+    });
+  }
+}
+// src/config.ts
+var import_lodash3 = __toESM(require_lodash(), 1);
 
 // node_modules/yaml/dist/index.js
 var composer = require_composer();
@@ -65511,9 +65570,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)
   });
@@ -66777,7 +66836,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];
@@ -68161,7 +68220,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;
@@ -68171,10 +68230,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]]);
       }
     }
   });
@@ -68419,10 +68478,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"
         });
@@ -68868,17 +68927,17 @@ Object.setPrototypeOf(createChalk.prototype, Function.prototype);
 for (const [styleName, style] of Object.entries(ansi_styles_default)) {
   styles2[styleName] = {
     get() {
-      const builder = createBuilder(this, createStyler(style.open, style.close, this[STYLER]), this[IS_EMPTY]);
-      Object.defineProperty(this, styleName, { value: builder });
-      return builder;
+      const builder2 = createBuilder(this, createStyler(style.open, style.close, this[STYLER]), this[IS_EMPTY]);
+      Object.defineProperty(this, styleName, { value: builder2 });
+      return builder2;
     }
   };
 }
 styles2.visible = {
   get() {
-    const builder = createBuilder(this, this[STYLER], true);
-    Object.defineProperty(this, "visible", { value: builder });
-    return builder;
+    const builder2 = createBuilder(this, this[STYLER], true);
+    Object.defineProperty(this, "visible", { value: builder2 });
+    return builder2;
   }
 };
 var getModelAnsi = (model, level, type, ...arguments_) => {
@@ -68949,12 +69008,12 @@ var createStyler = (open, close, parent) => {
   };
 };
 var createBuilder = (self2, _styler, _isEmpty) => {
-  const builder = (...arguments_) => applyStyle(builder, arguments_.length === 1 ? "" + arguments_[0] : arguments_.join(" "));
-  Object.setPrototypeOf(builder, proto);
-  builder[GENERATOR] = self2;
-  builder[STYLER] = _styler;
-  builder[IS_EMPTY] = _isEmpty;
-  return builder;
+  const builder2 = (...arguments_) => applyStyle(builder2, arguments_.length === 1 ? "" + arguments_[0] : arguments_.join(" "));
+  Object.setPrototypeOf(builder2, proto);
+  builder2[GENERATOR] = self2;
+  builder2[STYLER] = _styler;
+  builder2[IS_EMPTY] = _isEmpty;
+  return builder2;
 };
 var applyStyle = (self2, string8) => {
   if (self2.level <= 0 || !string8) {

package/package.json

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