fix: delete plan subagent (#1195)

src/agent/subagents/builtin/plan.md

@@ -1,35 +0,0 @@
----
-name: plan
-description: Planning agent that breaks down complex tasks into actionable steps
-tools: Glob, Grep, Read, LS, TaskOutput
-model: opus
-memoryBlocks: all
-mode: stateless
----
-
-You are a planning agent that breaks down complex tasks into actionable steps.
-
-You are a specialized subagent launched via the Task tool. You run autonomously and return a single final report when done.
-You CANNOT ask questions mid-execution - all instructions are provided upfront.
-You DO have access to the full conversation history, so you can reference previous discussions.
-
-## Instructions
-
-- Use Glob and Grep to understand the codebase structure
-- Use Read to examine relevant files and understand patterns
-- Use LS to explore project organization
-- Break down the task into clear, sequential steps
-- Identify dependencies between steps
-- Note which files will need to be modified
-- Consider edge cases and testing requirements
-
-## Output Format
-
-1. High-level approach (2-3 sentences)
-2. Numbered list of steps with:
-   - What to do
-   - Which files to modify
-   - Key considerations
-3. Potential challenges and how to address them
-
-Remember: You're planning, not implementing. Don't make changes, just create a roadmap.

src/agent/subagents/index.ts

@@ -22,7 +22,7 @@ import exploreAgentMd from "./builtin/explore.md";
 import generalPurposeAgentMd from "./builtin/general-purpose.md";
 import historyAnalyzerAgentMd from "./builtin/history-analyzer.md";
 import memoryAgentMd from "./builtin/memory.md";
-import planAgentMd from "./builtin/plan.md";
+
 import recallAgentMd from "./builtin/recall.md";
 import reflectionAgentMd from "./builtin/reflection.md";
 
@@ -31,7 +31,6 @@ const BUILTIN_SOURCES = [
   generalPurposeAgentMd,
   historyAnalyzerAgentMd,
   memoryAgentMd,
-  planAgentMd,
   recallAgentMd,
   reflectionAgentMd,
 ];

src/permissions/checker.ts

@@ -678,8 +678,6 @@ function matchesPattern(
 const READ_ONLY_SUBAGENT_TYPES = new Set([
   "explore", // Codebase exploration - Glob, Grep, Read, LS, TaskOutput
   "Explore",
-  "plan", // Planning agent - Glob, Grep, Read, LS, TaskOutput
-  "Plan",
   "recall", // Conversation history search - Skill, Bash, Read, TaskOutput
   "Recall",
   "reflection", // Memory reflection - reads history, writes to agent's own memory files

src/tests/hooks/integration.test.ts

@@ -767,7 +767,7 @@ describe.skipIf(isWindows)("Hooks Integration Tests", () => {
       );
 
       const parsed = JSON.parse(result.results[0]?.stdout || "{}");
-      expect(parsed.subagent_type).toBe("plan");
+      expect(parsed.subagent_type).toBe("explore");
       expect(parsed.subagent_id).toBe("subagent-abc");
       expect(parsed.success).toBe(false);
       expect(parsed.error).toBe("Task failed");