reasonix 0.27.3 → 0.28.0

package/dist/cli/{prompt-YUL7CYKY.js → prompt-VF7B6BWR.js}

@@ -2,9 +2,9 @@
 import {
   CODE_SYSTEM_PROMPT,
   codeSystemPrompt
-} from "./chunk-R2L5YEEF.js";
+} from "./chunk-COFBA5FV.js";
 export {
   CODE_SYSTEM_PROMPT,
   codeSystemPrompt
 };
-//# sourceMappingURL=prompt-YUL7CYKY.js.map
+//# sourceMappingURL=prompt-VF7B6BWR.js.map

package/dist/index.js

@@ -3636,6 +3636,11 @@ function parseFrontmatter(raw) {
 function isValidSkillName(name) {
   return VALID_SKILL_NAME.test(name);
 }
+function parseAllowedTools(raw) {
+  if (raw === void 0) return void 0;
+  const names = raw.split(",").map((s) => s.trim()).filter(Boolean);
+  return names.length > 0 ? Object.freeze(names) : void 0;
+}
 var SkillStore = class {
   homeDir;
   projectRoot;
@@ -3735,7 +3740,7 @@ var SkillStore = class {
       body: body.trim(),
       scope,
       path: path2,
-      allowedTools: data["allowed-tools"],
+      allowedTools: parseAllowedTools(data["allowed-tools"]),
       runAs: parseRunAs(data.runAs),
       model: data.model?.startsWith("deepseek-") ? data.model : void 0
     };
@@ -5408,6 +5413,50 @@ function registerPlanTool(registry, opts = {}) {
   return registry;
 }
 
+// src/tools/subagent-types.ts
+var EXPLORE_SYSTEM = `You are an exploration subagent. Wide-net read-only investigation; return one distilled answer.
+
+How to operate:
+- Read-only tools only (read_file, search_files, search_content, directory_tree, list_directory, get_file_info).
+- For "find all places that call / reference / use X" \u2014 use search_content (content grep), NOT search_files (which only matches names).
+- Cast a wide net first to map the territory, then read the 3-10 most relevant files in full. Stop as soon as you can answer.
+- The parent does not see your tool calls \u2014 over-exploration is pure waste.
+
+Final answer:
+- One paragraph or short bullets; lead with the conclusion.
+- Cite file:line ranges when they back the claim.
+- No follow-up offers, no "let me know if you need more" \u2014 the parent will ask again.
+
+${NEGATIVE_CLAIM_RULE}
+
+${TUI_FORMATTING_RULES}`;
+var VERIFY_SYSTEM = `You are a verify subagent. Narrow check \u2014 return YES / NO / INCONCLUSIVE with evidence. Do not expand scope.
+
+How to operate:
+- Read only what's needed to verify the specific claim. No exploration past the claim.
+- Use search_content / read_file to confirm the exact behavior, type, or call site in question.
+- Cap at 6-8 tool calls. If you can't verify in that, return INCONCLUSIVE plus what's missing.
+
+Final answer:
+- Lead with VERIFIED / NOT VERIFIED / INCONCLUSIVE.
+- Cite file:line for the evidence.
+- One paragraph or a few bullets. No follow-up offers.
+
+${NEGATIVE_CLAIM_RULE}
+
+${TUI_FORMATTING_RULES}`;
+var TYPES = {
+  explore: { system: EXPLORE_SYSTEM, maxToolIters: 20 },
+  verify: { system: VERIFY_SYSTEM, maxToolIters: 8 }
+};
+var SUBAGENT_TYPE_NAMES = Object.freeze(
+  Object.keys(TYPES)
+);
+function getSubagentType(name) {
+  if (typeof name !== "string") return void 0;
+  return TYPES[name];
+}
+
 // src/tools/subagent.ts
 var DEFAULT_SUBAGENT_SYSTEM = `You are a Reasonix subagent. The parent agent spawned you to handle one focused subtask, then return.
 
@@ -5424,6 +5473,8 @@ ${ESCALATION_CONTRACT}
 ${TUI_FORMATTING_RULES}`;
 var DEFAULT_MAX_RESULT_CHARS2 = 8e3;
 var DEFAULT_MAX_ITERS = 16;
+var MIN_MAX_ITERS = 1;
+var MAX_MAX_ITERS = 32;
 var DEFAULT_SUBAGENT_MODEL = "deepseek-v4-flash";
 var DEFAULT_SUBAGENT_EFFORT = "high";
 var SUBAGENT_TOOL_NAME = "spawn_subagent";
@@ -5444,7 +5495,41 @@ async function spawnSubagent(opts) {
     iter: 0,
     elapsedMs: 0
   });
-  const childTools = forkRegistryExcluding(opts.parentRegistry, NEVER_INHERITED_TOOLS);
+  if (opts.allowedTools) {
+    const missing = opts.allowedTools.filter((n) => !opts.parentRegistry.has(n));
+    if (missing.length > 0) {
+      const errorMessage2 = `subagent allow-list names tool(s) not registered in the parent: ${missing.join(", ")}. Fix the skill's \`allowed-tools\` frontmatter or check spelling.`;
+      sink?.current?.({
+        kind: "end",
+        task: taskPreview,
+        skillName,
+        model,
+        iter: 0,
+        elapsedMs: Date.now() - startedAt,
+        error: errorMessage2,
+        turns: 0,
+        costUsd: 0,
+        usage: new Usage()
+      });
+      return {
+        success: false,
+        output: "",
+        error: errorMessage2,
+        turns: 0,
+        toolIters: 0,
+        elapsedMs: Date.now() - startedAt,
+        costUsd: 0,
+        model,
+        skillName,
+        usage: new Usage()
+      };
+    }
+  }
+  const childTools = opts.allowedTools ? forkRegistryWithAllowList(
+    opts.parentRegistry,
+    new Set(opts.allowedTools),
+    NEVER_INHERITED_TOOLS
+  ) : forkRegistryExcluding(opts.parentRegistry, NEVER_INHERITED_TOOLS);
   const childPrefix = new ImmutablePrefix({
     system: opts.system,
     toolSpecs: childTools.specs()
@@ -5609,6 +5694,17 @@ function registerSubagentTool(parentRegistry, opts) {
           type: "string",
           enum: ["deepseek-v4-flash", "deepseek-v4-pro"],
           description: "Which DeepSeek model the subagent runs on. Default is 'deepseek-v4-flash' \u2014 cheap and fast, fine for explore/research-style subtasks. Override to 'deepseek-v4-pro' (~12\xD7 more expensive) when the subtask genuinely needs the stronger model: cross-file architecture, subtle bug hunts, anything where flash has empirically underperformed."
+        },
+        max_iters: {
+          type: "integer",
+          minimum: MIN_MAX_ITERS,
+          maximum: MAX_MAX_ITERS,
+          description: `Cap on the subagent's tool-call iterations. Default 16 (or the type's default when 'type' is set). Hard range: ${MIN_MAX_ITERS}-${MAX_MAX_ITERS}; out-of-range values are clamped to the nearest end.`
+        },
+        type: {
+          type: "string",
+          enum: [...SUBAGENT_TYPE_NAMES],
+          description: "Optional persona shaping the system prompt and default iter budget. 'explore' = wide-net read-only investigation (20-iter budget, returns a distilled answer). 'verify' = narrow yes/no check with evidence (8-iter budget). Omit when supplying your own 'system' prompt or when the default generic persona fits. Caller-supplied 'system' / 'max_iters' override the type's defaults."
         }
       },
       required: ["task"]
@@ -5620,15 +5716,17 @@ function registerSubagentTool(parentRegistry, opts) {
           error: "spawn_subagent requires a non-empty 'task' argument."
         });
       }
-      const system = typeof args.system === "string" && args.system.trim().length > 0 ? args.system.trim() : defaultSystem;
+      const typeSpec = getSubagentType(args.type);
+      const system = typeof args.system === "string" && args.system.trim().length > 0 ? args.system.trim() : typeSpec?.system ?? defaultSystem;
       const model = typeof args.model === "string" && args.model.startsWith("deepseek-") ? args.model : defaultModel;
+      const callerIters = clampMaxIters(args.max_iters);
       const result = await spawnSubagent({
         client: opts.client,
         parentRegistry,
         system,
         task,
         model,
-        maxToolIters,
+        maxToolIters: callerIters ?? typeSpec?.maxToolIters ?? maxToolIters,
         maxResultChars,
         sink,
         parentSignal: ctx?.signal
@@ -5638,6 +5736,13 @@ function registerSubagentTool(parentRegistry, opts) {
   });
   return parentRegistry;
 }
+function clampMaxIters(raw) {
+  if (typeof raw !== "number" || !Number.isFinite(raw)) return void 0;
+  const n = Math.floor(raw);
+  if (n < MIN_MAX_ITERS) return MIN_MAX_ITERS;
+  if (n > MAX_MAX_ITERS) return MAX_MAX_ITERS;
+  return n;
+}
 function forkRegistryExcluding(parent, exclude) {
   const child = new ToolRegistry();
   for (const spec of parent.specs()) {
@@ -5650,6 +5755,19 @@ function forkRegistryExcluding(parent, exclude) {
   if (parent.planMode) child.setPlanMode(true);
   return child;
 }
+function forkRegistryWithAllowList(parent, allow, alsoExclude) {
+  const child = new ToolRegistry();
+  for (const spec of parent.specs()) {
+    const name = spec.function.name;
+    if (!allow.has(name)) continue;
+    if (alsoExclude.has(name)) continue;
+    const def = parent.get(name);
+    if (!def) continue;
+    child.register(def);
+  }
+  if (parent.planMode) child.setPlanMode(true);
+  return child;
+}
 
 // src/tools/shell.ts
 import * as pathMod7 from "path";