@ferris1225/pi-subagents 0.26.0 → 0.26.2

package/README.md

@@ -434,9 +434,8 @@ When the config already exists, re-running `/subagents-setup` opens a menu whose
 model and thinking strength — so changing a single agent no longer walks every enabled
 agent. After one agent's model + strength picks, the wizard returns to the agent picker
 so several agents can be configured in one pass; Esc at any step ends the pass and keeps
-every agent already configured. **Change default thinking strength** sets only the global
-fallback. The rest of the menu toggles injection, scope, concurrency, fix rounds, and
-idle timeout; **Full re-setup** re-runs the whole first-time wizard.
+every agent already configured. The rest of the menu toggles injection, scope, concurrency,
+fix rounds, and idle timeout; **Full re-setup** re-runs the whole first-time wizard.
 
 ```json
 {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ferris1225/pi-subagents",
-  "version": "0.26.0",
+  "version": "0.26.2",
   "description": "Focused sub-agent delegation for pi: explore / worker / reviewer agents in isolated context, with proactive dispatch injection and per-agent model selection.",
   "type": "module",
   "license": "MIT",

package/src/prompt.ts

@@ -16,7 +16,7 @@ import { formatCatalogEntry } from "./agents.ts";
 
 /** Compact role routing hints, emitted only for roles that are enabled. */
 const ROLE_ROUTING: Record<string, string> = {
-	explore: "explore — broad/open-ended code search, multi-file lookups (read-only, cheap); NOT for one-line lookups.",
+	explore: "explore — codebase reconnaissance: broad/open-ended search, multi-file lookups, mapping unfamiliar code, tracing symbols/dependencies (read-only, cheap fast model); NOT for one-line lookups.",
 	worker: "worker — implement/fix/refactor/test a self-contained task worth a separate context (full tools; plans internally).",
 	reviewer: "reviewer — adversarial pre-commit review of a diff (read-only; independent context).",
 };
@@ -49,9 +49,10 @@ Available agents:
 ${catalog}
 
 ${routing ? `Routing:\n${routing}\n` : ""}Dispatch discipline:
-- Handle SIMPLE work INLINE with direct tools: a single lookup, one-line edit, or a quick question is a grep/read/edit in the main context — never a sub-agent. Sub-agents cost startup time, tokens, and a context switch.
-- Delegate only when isolation genuinely pays: broad exploration of an unfamiliar area, a self-contained implementation/fix with its own validation, or a fresh-context review gate.
-- When in doubt, start with a direct tool call in the main context; escalate to a sub-agent only if the work turns out broad.
+- Handle SIMPLE work INLINE with direct tools: a one-line lookup, single edit, or quick question is a grep/read/edit in the main context — never a sub-agent. Sub-agents cost startup time, tokens, and a context switch.
+- Use \`explore\` PROACTIVELY for codebase reconnaissance: mapping an unfamiliar area, multi-file lookups, tracing symbols across modules, or any "where is X / which files reference Y" question that would take several greps or reading multiple files. It runs on a fast cheap model and returns compressed findings, so delegating recon costs little.
+- Delegate only when isolation genuinely pays: a self-contained implementation/fix with its own validation (worker), or a fresh-context review gate (reviewer).
+- When in doubt, start with a direct tool call in the main context; escalate to \`explore\` as soon as the search turns broad or crosses multiple files.
 - For an already-known or trivial target, use a direct search/read tool (e.g. grep/find/read) — do not over-delegate a one-line lookup.
 ${hasMultiple ? "- Run INDEPENDENT tasks in parallel: one subagent call with a `tasks` array, and track them with your todo list. Let the automatically resumed main agent launch dependent work only after its prerequisite result arrives (e.g. explore, then worker, then reviewer).\n" : ""}- Brief each sub-agent as self-contained: goal, exact paths, constraints, expected output. It has NO memory of this conversation.
 - Treat delegated agents as leaf workers: do not ask a sub-agent to dispatch another sub-agent; child processes do not have this tool.

package/src/setup.ts

@@ -391,7 +391,6 @@ async function runMenu(ctx: ExtensionCommandContext, configPath: string, config:
 	const choice = await ctx.ui.select("pi-subagents is already configured. What would you like to change?", [
 		"Enable/disable agents",
 		"Configure an agent (model + thinking)",
-		"Change default thinking strength",
 		"Toggle proactive injection",
 		"Change agent scope",
 		"Change max concurrent sub-agents",
@@ -437,10 +436,6 @@ async function runMenu(ctx: ExtensionCommandContext, configPath: string, config:
 		}
 		if (!configuredAny) return notifyCancelled(ctx);
 		next.agentModels = repairStaleModels(ctx, next.agentModels);
-	} else if (choice.startsWith("Change default thinking")) {
-		const thinkingLevel = await pickThinkingLevel(ctx, config.thinkingLevel);
-		if (thinkingLevel === undefined) return notifyCancelled(ctx);
-		next.thinkingLevel = thinkingLevel;
 	} else if (choice.startsWith("Toggle")) {
 		const injection = await pickInjection(ctx, config.proactiveInjection);
 		if (injection === undefined) return notifyCancelled(ctx);