@ferris1225/pi-subagents 2.3.1 → 3.0.0

package/README.md

@@ -61,11 +61,11 @@ index.
   into the main system prompt, so the main model sends broad searches to `explore`,
   self-contained implementations to `worker`, explicit cleanup intent to `cleaner`,
   and pre-commit reviews to `reviewer`. You just use pi; delegation happens by itself.
-- **Vision-capable image tasks** — flag screenshot/mockup/design work with
-  `vision: true`. The configured vision model hands directly to the current main
-  model on model/provider failure. Setup lists only in-scope, image-capable models
-  from providers with configured authentication, and failures never silently
-  rewrite your configuration.
+- **Multimodal work is a model choice, not a mode** — an agent that should see
+  screenshots, mockups, or its own rendered pages simply gets a multimodal model
+  through `/subagents-setup` (the picker labels each model `vision` or
+  `text-only`). The agent reads images with its `read` tool on whatever model it
+  runs; no per-task flag, no separate vision override.
 - **Results come back on their own** — completions are delivered as messages that
   wake the main agent automatically, even mid-turn. No polling, no `sleep`, no
   "go check" step. `subagent_wait` is a **non-blocking** in-turn lookup by default
@@ -201,15 +201,16 @@ subagent({
 
 ### Cleanup routing and lifecycle
 
-The injected guidance routes `cleaner` by explicit semantic intent: **code cleanup**,
-**dead code**, **代码清理**, **精简**, **清理冗余**, **简化**, or **去除过度设计**.
-Requested periodic maintenance passes also qualify; PR counts do not, and cleaner is
-never run automatically as the pre-commit gate.
+The injected guidance routes `cleaner` by explicit semantic intent in any
+language the conversation uses: **code cleanup**, **dead code**, redundancy,
+simplification, or over-engineering. Requested periodic maintenance passes also
+qualify; PR counts do not, and cleaner is never run automatically as the
+pre-commit gate.
 
-- **Audit mode:** audit/find/report/review or 审计/查找/检查/报告 wording produces
-  read-only ranked evidence.
-- **Apply mode:** explicit remove/clean/simplify/refactor or 清理/删除/移除/精简/简化/重构
-  wording permits the smallest proven edits plus narrow-then-broad verification.
+- **Audit mode:** audit/find/report/review wording produces read-only ranked
+  evidence.
+- **Apply mode:** explicit remove/clean/simplify/refactor wording permits the
+  smallest proven edits plus narrow-then-broad verification.
 
 ```text
 explicit cleanup intent → cleaner (audit or apply)
@@ -220,33 +221,23 @@ cleaner apply → reviewer gate → worker auto-fix (on REVIEW_FAIL) → reviewe
 portion runs only when enabled by `maxFixRounds`; cleaner itself is not a pre-commit
 hook or a PR-count scheduler.
 
-### Vision tasks (screenshots / mockups / designs)
+### Image work (screenshots / mockups / designs)
 
-When a task may require viewing images — frontend work, UI review, design
-comparisons — set `vision: true` and give the sub-agent the exact image paths:
+There is no vision flag or separate vision model. Give the agent a multimodal
+model in `/subagents-setup` and name the exact image paths in the task:
 
 ```ts
 subagent({
   agent: "reviewer",
   task: "Compare the UI in screenshots/settings.png against the mockup design.png; list every visual mismatch.",
-  vision: true,
 });
 ```
 
-The sub-agent reads images with its `read` tool. Runtime order for a
-vision-flagged run is `visionModel` → current main-window model. A stale or
-currently unavailable vision selection is skipped immediately without rewriting
-it. A vision-flagged auto-fix chain keeps the flag for worker/re-review rounds
-because they may need to inspect the same images.
-
-As a deterministic fallback, a brief that names an image file (an extension such
-as `.png`, `.jpg`, `.webp`) or describes frontend/UI work — Vue/React/Svelte
-pages, mockups, screenshots, page styling — is dispatched as
-vision even without the flag: the model cannot be swapped mid-run, so a worker
-that screenshots and visually verifies what it builds must start on a
-vision-capable model. How do you verify which model ran? The live widget line,
-dispatch result row, and `subagent_status` all show each run's effective model
-id, and a selected→main handoff is labeled with its origin.
+The sub-agent reads images with its `read` tool on its configured model; the
+setup picker labels each model `vision` or `text-only` so the choice is visible.
+The live widget line, dispatch result row, and `subagent_status` all show each
+run's effective model id, and a selected→main handoff is labeled with its
+origin.
 
 ### Controlling and stopping
 
@@ -302,8 +293,8 @@ sessions show the recovery paths again until the artifacts are removed.
 ## Configuration
 
 Stored at `~/.pi/agent/pi-subagents.json` (follows `PI_CODING_AGENT_DIR` when
-set). `/subagents-setup` has five top-level choices: enable agents, configure one
-agent's model/thinking, choose a vision model, runtime settings, or full setup.
+set). `/subagents-setup` has four top-level choices: enable agents, configure one
+agent's model/thinking, runtime settings, or full setup.
 After one agent's model + thinking 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. There is no backup pool or global thinking menu. Model pickers show only in-scope
@@ -321,7 +312,6 @@ direct-file settings.
   "agentThinkingLevels": {
     "reviewer": "high"
   },
-  "visionModel": "anthropic/claude-sonnet-4-5",
   "notifyOnReviewPass": false,
   "maxResultLines": 80,
   "proactiveInjection": true,
@@ -337,7 +327,6 @@ direct-file settings.
 | `enabledAgents` | Agent names exposed to discovery and prompt injection. An empty array disables all agents. |
 | `agentModels` | Optional selected `provider/model-id` per agent. Missing = current main model. Model-level failure hands directly to current main. |
 | `agentThinkingLevels` | Optional manual preference per agent. Missing = Auto (agent frontmatter preference, or `high`, clamped to the effective model's supported levels). |
-| `visionModel` | Optional vision-capable model for `vision: true` tasks (screenshots/mockups/designs). Unset = falls back to the main session's current model. |
 | `notifyOnReviewPass` | When `true`, a passing reviewer result is delivered without waking the main agent (default `false`). |
 | `maxResultLines` | Max lines of a sub-agent result carried in the completion message (default `80`). Longer results are truncated; full text is written to an extension-named temporary `.md`. At session start and on each write, only recognized result files older than 7 days are removed; each canonical project path has its own newest-50 bucket. |
 | `proactiveInjection` | Whether to add the delegation directive to the main system prompt. |
@@ -349,7 +338,7 @@ direct-file settings.
 ### Model routing and thinking
 
 ```text
-selected agent/vision model → current main-window model
+selected agent model → current main-window model
 ```
 
 Without a selected model, current main runs immediately; agent frontmatter model

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ferris1225/pi-subagents",
-  "version": "2.3.1",
+  "version": "3.0.0",
   "description": "Controllable background sub-agent threads for pi: specialized roles, capability-aware thinking, direct main-model fallback, auto-fix chains, and Git worktree isolation.",
   "type": "module",
   "license": "MIT",

package/src/announcements.ts

@@ -13,12 +13,6 @@ const ANNOUNCEMENTS: Array<{
 	condition: (config: Awaited<ReturnType<typeof loadConfig>>) => boolean;
 	message: string;
 }> = [
-	{
-		key: "visionModel",
-		condition: (config) => config.visionModel === undefined,
-		message:
-			"pi-subagents: new — a vision-capable model can now handle image tasks (screenshots, mockups, designs). Run /subagents-setup to configure it; until set, vision tasks use the main session's current model.",
-	},
 	{
 		key: "cleanerAgent",
 		condition: (config) => !config.enabledAgents.includes("cleaner"),

package/src/config.ts

@@ -99,15 +99,8 @@ export interface SubagentsConfig {
 	 */
 	idleTimeoutSec: number;
 	/**
-	 * Vision-capable model for tasks flagged `vision: true` (viewing screenshots,
-	 * mockups, designs). Unset means such tasks fall back to the main session's
-	 * current model.
-	 */
-	visionModel?: string;
-	/**
-	 * One-time feature announcements already shown to the user (e.g. "vision
-	 * model" after an update that introduced it). Persisted so the notice never
-	 * nags again.
+	 * One-time feature announcements already shown to the user. Persisted so
+	 * the notice never nags again.
 	 */
 	announcedFeatures: string[];
 }
@@ -212,10 +205,6 @@ export function normalizeConfig(raw: unknown): SubagentsConfig {
 		config.idleTimeoutSec = Math.max(0, Math.min(IDLE_TIMEOUT_SEC_LIMIT, Math.round(raw.idleTimeoutSec)));
 	}
 
-	if (isModelReference(raw.visionModel)) {
-		config.visionModel = (raw.visionModel as string).trim();
-	}
-
 	if (Array.isArray(raw.announcedFeatures)) {
 		config.announcedFeatures = raw.announcedFeatures.filter(
 			(feature): feature is string => typeof feature === "string" && feature.trim().length > 0,

package/src/dispatch.ts

@@ -3,9 +3,6 @@
  * child processes, single or parallel. Owns the public dispatch contract,
  * per-run status tracking, the auto-fix chain (REVIEW_FAIL → worker → re-review),
  * and completion delivery. Stable thread generations live in thread-lifecycle.ts.
- *
- * Vision: a task flagged `vision: true` uses the configured vision model, then
- * hands directly to the current main-window model on model/provider failure.
  */
 
 import { StringEnum } from "@earendil-works/pi-ai";
@@ -58,9 +55,6 @@ export { FORK_CONTINUATION_PROMPT, isWorktreeCapableAgent } from "./thread-lifec
 
 const NON_BLANK_TASK_OPTIONS = { minLength: 1, pattern: "\\S" } as const;
 
-const VISION_DESCRIPTION =
-	"Set true when the task may require viewing images (screenshots, mockups, designs) — the configured vision model is used first, then model-level failures hand directly to the current main-window model. A task that names an image file or describes frontend/UI work (Vue/React, mockups, screenshots, page styling) is dispatched as vision automatically";
-
 const ISOLATION_DESCRIPTION =
 	"Filesystem isolation: shared uses the caller's working tree; worktree creates a detached temporary Git worktree (write-capable agents, including worker and cleaner, only)";
 
@@ -75,7 +69,6 @@ const TaskItem = Type.Object({
 		description: "Self-contained task to delegate (the agent has no memory of this conversation)",
 	}),
 	cwd: Type.Optional(Type.String({ description: "Working directory for the agent process" })),
-	vision: Type.Optional(Type.Boolean({ description: VISION_DESCRIPTION })),
 	isolation: IsolationSchema,
 });
 
@@ -86,7 +79,6 @@ const SubagentParams = Type.Object({
 	),
 	tasks: Type.Optional(Type.Array(TaskItem, { description: "Array of {agent, task} for parallel execution" })),
 	cwd: Type.Optional(Type.String({ description: "Working directory for the agent process (single mode)" })),
-	vision: Type.Optional(Type.Boolean({ description: VISION_DESCRIPTION })),
 	isolation: IsolationSchema,
 });
 
@@ -95,27 +87,6 @@ export function defaultIsolationMode(mode: "single" | "parallel", agentName: str
 	return mode === "parallel" && agentName === "worker" ? "worktree" : "shared";
 }
 
-const IMAGE_FILE_PATTERN = /\.(?:png|jpe?g|webp|gif|bmp|tiff?|avif)\b/i;
-/** Frontend/UI work: the agent may screenshot and visually verify what it
- * builds, and the model cannot be swapped mid-run — vision must be on at
- * spawn. Only language-neutral signals: code terms and file names stay in
- * English even in non-English briefs; purely semantic wording is the LLM's
- * job via the vision flag, not a per-language keyword list. Word boundaries
- * keep ordinary words (build, guide, reactor) from matching. */
-const FRONTEND_PATTERN = /\b(?:vue|react|svelte|angular|nuxt|next\.?js|vite|tailwind|screenshots?|mockups?|wireframes?|UI)\b/i;
-const FRONTEND_FILE_PATTERN = /\.(?:vue|html?|css|svg)\b/i;
-
-/** Deterministic vision fallback: a brief that names an image file or describes
- * frontend/UI work is vision work even when the dispatcher forgot the flag —
- * prompting alone cannot guarantee `vision: true`, and a non-vision model can
- * neither see reference images nor visually verify what it builds. */
-export function taskImpliesVision(task: string): boolean {
-	return IMAGE_FILE_PATTERN.test(task) || FRONTEND_FILE_PATTERN.test(task) || FRONTEND_PATTERN.test(task);
-}
-
-const wantsVision = (flag: boolean | undefined, task: string): boolean =>
-	flag === true || taskImpliesVision(task);
-
 const autoFixRootTails = new Map<string, Promise<void>>();
 
 async function canonicalAutoFixRoot(cwd: string): Promise<string> {
@@ -172,7 +143,6 @@ export function registerSubagentTool(pi: ExtensionAPI, runtime: SubagentRuntime)
 			"It starts agents in the background and immediately returns control to the main window; completion messages automatically wake the main agent to continue.",
 			"Each agent has no memory of this conversation — brief it fully (goal, exact paths, constraints, expected output).",
 			"Results arrive as wake-up messages automatically — you do NOT need to wait. If you must get a result in-turn, subagent_wait is a non-blocking lookup by default (pass timeoutMs to block).",
-			"Vision: set vision: true when the task may require viewing images (screenshots, mockups, design files — e.g. frontend work) — the configured vision model is used first, then model-level failures hand directly to the current main-window model.",
 		].join(" "),
 		promptSnippet:
 			"Start background subagents: explore (read-only search), worker (implement), cleaner (explicit evidence-first cleanup), reviewer (pre-commit review); completion automatically resumes the main agent. Simple tasks: use direct tools, not subagents.",
@@ -188,7 +158,6 @@ export function registerSubagentTool(pi: ExtensionAPI, runtime: SubagentRuntime)
 			"Use isolation: worktree only for worker, cleaner, or another write-capable agent and only inside a Git repository with a committed HEAD; parallel worker tasks default to worktree, while cleaner must opt in. Setup or integration failures never silently fall back to shared.",
 			"NEVER sleep or poll, and do NOT call subagent_wait to hold the turn — subagent ends the turn immediately and the result arrives as a message that wakes you automatically (even mid-turn). Ending your turn is the default and the only correct way to wait.",
 			"If you must keep the turn for a result, call subagent_wait with an explicit timeoutMs (non-blocking by default) — never bash sleep/timeout to wait for a sub-agent.",
-			"When a delegated task may require viewing images (frontend screenshots, mockups, design comparisons), pass vision: true and give the sub-agent the exact image paths — it reads them with its read tool. The configured vision model is used first; model-level failures hand directly to the current main-window model.",
 			"When a sub-agent result arrives it is already shown to the user — do NOT restate, paraphrase, or summarize it; reply with only your own conclusion or next action (often just one line), since duplicating the result wastes tokens for nothing.",
 		],
 		parameters: SubagentParams,
@@ -320,13 +289,10 @@ export function registerSubagentTool(pi: ExtensionAPI, runtime: SubagentRuntime)
 				executionCwd: string,
 				signal: AbortSignal,
 				meta: RunChainMeta,
-				vision = false,
 			): Promise<{ runId?: number; result: SingleResult }> => {
 				const agent = agents.find((candidate) => candidate.name === agentName);
 				if (!agent) return { result: failedStartResult(agentName, task, `Unknown agent: "${agentName}".`) };
-				// Vision chains use the vision override first; every model-level failure
-				// hands directly to the current main model with re-clamped thinking.
-				const route = resolveDispatchModelRoute(agent, config, ctx, vision);
+				const route = resolveDispatchModelRoute(agent, config, ctx);
 				const thinkingLevel = route.thinkingLevel;
 				const runId = monitor.addRun(agent.name, task, route.agent.model, thinkingLevel, meta);
 				const onLive = makeLiveHandler(runId);
@@ -397,7 +363,6 @@ export function registerSubagentTool(pi: ExtensionAPI, runtime: SubagentRuntime)
 				parentGroupId: string,
 				parentRunId: number,
 				executionCwd: string,
-				vision = false,
 			): void => {
 				const parentThreadAtStart = runtime.threads.get(parentRunId);
 				if (!parentThreadAtStart) return;
@@ -446,7 +411,7 @@ export function registerSubagentTool(pi: ExtensionAPI, runtime: SubagentRuntime)
 								groupId: parentGroupId,
 								relationLabel: `fix round ${round}`,
 								parentRunId,
-							}, vision);
+							});
 							// Preserve the newest sub-step before checking chain ownership. A
 							// destructive stop invalidates ownsParent() while this child is
 							// aborting, and its partial output must become the parent's stopped
@@ -470,7 +435,7 @@ export function registerSubagentTool(pi: ExtensionAPI, runtime: SubagentRuntime)
 								groupId: parentGroupId,
 								relationLabel: `re-review round ${round}`,
 								parentRunId,
-							}, vision);
+							});
 							if (
 								runtime.threads.get(parentRunId) === parentThreadAtStart &&
 								parentThreadAtStart.generation === parentGeneration
@@ -649,7 +614,6 @@ export function registerSubagentTool(pi: ExtensionAPI, runtime: SubagentRuntime)
 						item.agent,
 						item.task,
 						item.cwd,
-						wantsVision(item.vision, item.task),
 						defaultIsolationMode("parallel", item.agent, item.isolation as IsolationMode | undefined),
 					));
 				}
@@ -687,7 +651,6 @@ export function registerSubagentTool(pi: ExtensionAPI, runtime: SubagentRuntime)
 				params.agent as string,
 				params.task as string,
 				params.cwd,
-				wantsVision(params.vision, params.task as string),
 				defaultIsolationMode("single", params.agent as string, params.isolation as IsolationMode | undefined),
 			);
 			if (result.exitCode !== -1) {

package/src/index.ts

@@ -69,7 +69,7 @@ export default function (pi: ExtensionAPI): void {
 	registerLookupTools(pi, runtime);
 
 	pi.registerCommand("subagents-setup", {
-		description: "Configure pi-subagents: agents, selected models, capability-aware thinking, vision, and runtime settings",
+		description: "Configure pi-subagents: agents, selected models, capability-aware thinking, and runtime settings",
 		handler: async (_args, ctx) => {
 			await runSetup(ctx, configPath);
 		},

package/src/models.ts

@@ -1,7 +1,7 @@
 /*
  * Model routing, capability-aware thinking, and setup-picker helpers.
  *
- * Runtime has one explicit fallback only: a configured agent/vision model hands
+ * Runtime has one explicit fallback only: a configured agent model hands
  * off directly to the current main-window model. Setup lists only currently
  * available models and derives thinking choices from Pi's model metadata.
  */
@@ -20,8 +20,6 @@ export type ModelContext = Pick<ExtensionContext, "model" | "modelRegistry"> &
 
 export const CURRENT_MAIN_MODEL = "__current_main_model__";
 
-export type ModelPickerSlot = "agent" | "vision";
-
 export interface ModelPickerItem {
 	value: string;
 	label: string;
@@ -136,11 +134,10 @@ function modelCapabilities(model: ModelListEntry): string {
 	return `${input} · thinking: ${thinking}`;
 }
 
-/** Build one searchable list for agent or vision selection. Only models Pi
+/** Build one searchable list for agent model selection. Only models Pi
  * currently reports as available are supplied by setup. */
 export function buildModelPickerItems(options: {
 	models: readonly ModelListEntry[];
-	slot: ModelPickerSlot;
 	configuredRef?: string;
 	mainRef?: string;
 }): ModelPickerItem[] {
@@ -153,7 +150,6 @@ export function buildModelPickerItems(options: {
 	}
 
 	const refs = [...byRef.keys()]
-		.filter((ref) => options.slot !== "vision" || byRef.get(ref)?.input.includes("image") === true)
 		.sort((left, right) => {
 			const leftRank = left === configuredRef ? 0 : left === mainRef ? 1 : 2;
 			const rightRank = right === configuredRef ? 0 : right === mainRef ? 1 : 2;
@@ -163,9 +159,7 @@ export function buildModelPickerItems(options: {
 	const dynamic: ModelPickerItem = {
 		value: CURRENT_MAIN_MODEL,
 		label: "Current main model (dynamic)",
-		description: options.slot === "vision"
-			? "Clear vision override; use the current main model for image tasks"
-			: "Clear agent override; use the current main model dynamically",
+		description: "Clear agent override; use the current main model dynamically",
 	};
 	const items: ModelPickerItem[] = [dynamic];
 	for (const ref of refs) {

package/src/prompt.ts

@@ -66,10 +66,6 @@ ${hasMultiple ? `- Run INDEPENDENT tasks in parallel: one subagent call with a \
 - Treat delegated agents as leaf workers: do not ask a sub-agent to dispatch another sub-agent; child processes do not have this tool. Use \`subagent_control fork\` on a parked/settled retained thread when you need an independent continuation with preserved context and a new run id.
 - Trust but verify: a sub-agent's summary describes intent, not outcome. Check the actual changes/results before reporting work done.
 ${hasExplore ? "- Treat `explore` findings as a retrieval index, never as sole proof for edits, deletion, security, compatibility, persistence, or dynamic reachability. Re-read load-bearing files before acting. An underpowered model can be false economy on complex dynamic, concurrent, migration, or security-sensitive code; use a stronger model or specialist there.\n" : ""}
-Vision tasks:
-- Judge whether a delegated task may require viewing images (frontend screenshots, mockups, design files, visual regression comparisons). If it might, pass \`vision: true\` in the subagent call and give the sub-agent the exact image paths — it reads them with its read tool. Naming an image file or describing frontend/UI work (Vue/React page, mockup, page styling) dispatches as vision automatically, but keep passing the flag for image work described without those signals.
-- \`vision: true\` runs the sub-agent on the vision-capable model configured in /subagents-setup; when none is configured it falls back to the main session's current model. Do not skip the flag because the agent's default model looks fast — a non-vision model cannot see the images.
-
 Result handoff (do not re-state):
 - A sub-agent's result arrives as a message that is already shown to the user. Do NOT restate, paraphrase, or re-summarize its findings in your reply — that just burns tokens duplicating what is already visible. The user can read the result above.
 - Reply only with what you ADD: your own conclusion, the next action you are taking, or a one-line acknowledgement. When the result already answers the user, a single sentence is enough — then end your turn or proceed.

package/src/runtime.ts

@@ -52,7 +52,6 @@ export interface SubagentThread {
 	cwd: string;
 	/** Actual child cwd (the equivalent path inside an isolated worktree). */
 	executionCwd: string;
-	vision: boolean;
 	thinkingLevel?: ThinkingLevel;
 	isolation: IsolationMode;
 	worktree?: WorktreeIsolation;

package/src/setup.ts

@@ -37,7 +37,6 @@ import {
 	modelRef,
 	resolveThinkingLevel,
 	supportedThinkingLevels,
-	type ModelPickerSlot,
 } from "./models.ts";
 import { promptSelectMany, promptSelectOne } from "./ui.ts";
 import { discoverAgents } from "./agents.ts";
@@ -96,14 +95,12 @@ async function pickEnabledAgents(
 async function pickConfiguredModel(
 	ctx: ExtensionCommandContext,
 	title: string,
-	slot: ModelPickerSlot,
 	configuredRef: string | undefined,
 	escNote: string,
 ): Promise<string | undefined> {
 	const models = availableModelsInScope(ctx);
 	const items = buildModelPickerItems({
 		models,
-		slot,
 		configuredRef,
 		mainRef: currentModelRef(ctx),
 	});
@@ -122,20 +119,7 @@ async function pickAgentModel(
 	currentRef: string | undefined,
 	escNote = "cancels setup",
 ): Promise<string | undefined> {
-	return pickConfiguredModel(ctx, `Model for "${agentName}"?`, "agent", currentRef, escNote);
-}
-
-async function pickVisionModel(
-	ctx: ExtensionCommandContext,
-	currentRef: string | undefined,
-): Promise<string | undefined> {
-	return pickConfiguredModel(
-		ctx,
-		"Vision model for image tasks? (failure → current main model)",
-		"vision",
-		currentRef,
-		"cancels setup",
-	);
+	return pickConfiguredModel(ctx, `Model for "${agentName}"?`, currentRef, escNote);
 }
 
 const THINKING_LEVEL_HINTS: Record<ThinkingLevel, string> = {
@@ -304,8 +288,6 @@ async function runFullSetup(ctx: ExtensionCommandContext, configPath: string, ba
 		agentModels = applyAgentModelChoice(agentModels, agentName, choice);
 	}
 
-	const visionModel = await pickVisionModel(ctx, base.visionModel);
-	if (visionModel === undefined) return notifyCancelled(ctx);
 	const injection = await pickInjection(ctx, base.proactiveInjection);
 	if (injection === undefined) return notifyCancelled(ctx);
 	const scope = await pickScope(ctx, base.agentScope);
@@ -349,7 +331,6 @@ async function runFullSetup(ctx: ExtensionCommandContext, configPath: string, ba
 		idleTimeoutSec,
 		announcedFeatures: base.announcedFeatures,
 	};
-	if (visionModel !== CURRENT_MAIN_MODEL) next.visionModel = visionModel;
 	await saveConfig(next, configPath);
 	ctx.ui.notify(`pi-subagents configured with Auto thinking. Saved to ${configPath}`, "info");
 }
@@ -395,7 +376,6 @@ async function runMenu(ctx: ExtensionCommandContext, configPath: string, config:
 	const choice = await ctx.ui.select("pi-subagents settings", [
 		"Enable/disable agents",
 		"Configure an agent (model + thinking)",
-		"Change vision model",
 		"Runtime settings",
 		"Full re-setup",
 	]);
@@ -427,11 +407,6 @@ async function runMenu(ctx: ExtensionCommandContext, configPath: string, config:
 			else next.agentThinkingLevels[picked.name] = picked.strength;
 		}
 		if (!configuredAny) return notifyCancelled(ctx);
-	} else if (choice.startsWith("Change vision")) {
-		const visionModel = await pickVisionModel(ctx, config.visionModel);
-		if (visionModel === undefined) return notifyCancelled(ctx);
-		if (visionModel === CURRENT_MAIN_MODEL) delete next.visionModel;
-		else next.visionModel = visionModel;
 	} else {
 		const updated = await updateRuntimeSetting(ctx, next);
 		if (updated === undefined) return notifyCancelled(ctx);

package/src/thread-lifecycle.ts

@@ -92,12 +92,11 @@ export function resolveDispatchModelRoute(
 	agent: AgentConfig,
 	config: SubagentsConfig,
 	ctx: ExtensionContext,
-	vision: boolean,
 ): DispatchModelRoute {
 	const availableModels = availableModelsInScope(ctx);
 	const mainRef = currentModelRef(ctx);
 	const route = resolveAgentModelRoute({
-		selectedRef: vision ? config.visionModel : config.agentModels[agent.name],
+		selectedRef: config.agentModels[agent.name],
 		mainRef,
 		declaredDefaultRef: agent.model,
 		availableRefs: availableModels.map(modelRef),
@@ -137,7 +136,6 @@ interface BackgroundDispatcherOptions extends DispatchEnvironment {
 		parentGroupId: string,
 		parentRunId: number,
 		executionCwd: string,
-		vision?: boolean,
 	) => void;
 }
 
@@ -145,7 +143,6 @@ type BackgroundStarter = (
 	agentName: string,
 	task: string,
 	cwd: string | undefined,
-	vision?: boolean,
 	isolation?: IsolationMode,
 ) => Promise<SingleResult>;
 
@@ -204,7 +201,6 @@ export function createBackgroundDispatcher(options: BackgroundDispatcherOptions)
 		agentName: string,
 		task: string,
 		cwd: string | undefined,
-		vision = false,
 		isolation: IsolationMode = "shared",
 		existingThread?: SubagentThread,
 		newObjectiveOnResume = false,
@@ -253,7 +249,7 @@ export function createBackgroundDispatcher(options: BackgroundDispatcherOptions)
 			}
 		}
 		const executionCwd = worktree?.cwd ?? originalCwd;
-		const resolvedRoute = resolveDispatchModelRoute(agent, runConfig, runCtx, vision);
+		const resolvedRoute = resolveDispatchModelRoute(agent, runConfig, runCtx);
 		// Isolation is a persistent system-level invariant, not a one-shot task
 		// prefix: queued retargets, live retargets, resumes, and main-model
 		// handoffs all keep the same worktree boundary.
@@ -319,7 +315,6 @@ export function createBackgroundDispatcher(options: BackgroundDispatcherOptions)
 			thread.task = task;
 			thread.cwd = originalCwd;
 			thread.executionCwd = executionCwd;
-			thread.vision = vision;
 			thread.thinkingLevel = thinkingLevel;
 			thread.isolation = isolation;
 			thread.worktree = worktree;
@@ -343,7 +338,6 @@ export function createBackgroundDispatcher(options: BackgroundDispatcherOptions)
 				task,
 				cwd: originalCwd,
 				executionCwd,
-				vision,
 				thinkingLevel,
 				isolation,
 				worktree,
@@ -622,7 +616,6 @@ export function createBackgroundDispatcher(options: BackgroundDispatcherOptions)
 					thread.agentName,
 					nextTask,
 					thread.cwd,
-					thread.vision,
 					thread.isolation,
 					thread,
 					objective !== undefined,
@@ -784,7 +777,6 @@ export function createBackgroundDispatcher(options: BackgroundDispatcherOptions)
 					thread.agentName,
 					childTask,
 					thread.cwd,
-					thread.vision,
 					thread.isolation,
 					undefined,
 					false,
@@ -926,7 +918,7 @@ export function createBackgroundDispatcher(options: BackgroundDispatcherOptions)
 					// the same row now represents the chain until it resolves.
 					monitor.setStatus(runId, "running");
 					monitor.setActivity(runId, "auto-fix chain running");
-					startFixLoop(result, `fix-${runId}`, runId, thread.executionCwd, vision);
+					startFixLoop(result, `fix-${runId}`, runId, thread.executionCwd);
 					return;
 				}
 				// Claim terminal settlement synchronously before the first slow await.