Summary

Closes #2287

The isUnstableAgent check in category-resolver.ts was checking both the resolved model (actualModel) and the category's default model (resolved.config.model). This meant that even when a user overrides visual-engineering or writing to use a stable model like anthropic/claude-sonnet-4-6, the default model name (google/gemini-3.1-pro or kimi-for-coding/k2p5) still triggered isUnstableAgent = true.

This routed tasks through executeUnstableAgentTask() instead of the stable execution path, causing a race condition where manager.getTask(task.id) returns undefined after the task completes before the polling loop catches it, resulting in "Task not found" errors.

Changes

• src/tools/delegate-task/category-resolver.ts — Only check the resolved/actual model for unstable agent detection, not the category's hardcoded default model

Before

const unstableModel = actualModel?.toLowerCase()
const categoryConfigModel = resolved.config.model?.toLowerCase()
const isUnstableAgent = resolved.config.is_unstable_agent === true 
  || [unstableModel, categoryConfigModel].some(m => ...)

After

const resolvedModel = actualModel?.toLowerCase()
const isUnstableAgent = resolved.config.is_unstable_agent === true 
  || (resolvedModel ? resolvedModel.includes("gemini") || ... : false)

Impact

Summary by cubic

Fixes unstable-agent detection to consider only the resolved model, so stable overrides use the stable execution path. Resolves #2287 by removing misroutes that caused intermittent "Task not found" errors.

• Bug Fixes
  • Updated src/tools/delegate-task/category-resolver.ts to check only the resolved model in isUnstableAgent, not the category default.
  • Behavior is unchanged when no override is provided and the category’s default model is unstable.

^{Written for commit eb79d29. Summary will update on new commits.}