From 613acc0c3e5ec63650162b64fa03064a85715cd5 Mon Sep 17 00:00:00 2001
From: Ari Webb <ari@letta.com>
Date: Fri, 27 Feb 2026 11:46:58 -0800
Subject: [PATCH] fix: delete plan subagent (#1195)

---
 src/agent/subagents/builtin/plan.md | 35 -----------------------------
 src/agent/subagents/index.ts        |  3 +--
 src/permissions/checker.ts          |  2 --
 src/tests/hooks/integration.test.ts |  2 +-
 4 files changed, 2 insertions(+), 40 deletions(-)
 delete mode 100644 src/agent/subagents/builtin/plan.md

diff --git a/src/agent/subagents/builtin/plan.md b/src/agent/subagents/builtin/plan.md
deleted file mode 100644
index 5a895d1..0000000
--- a/src/agent/subagents/builtin/plan.md
+++ /dev/null
@@ -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.
diff --git a/src/agent/subagents/index.ts b/src/agent/subagents/index.ts
index fa5922f..88b3725 100644
--- a/src/agent/subagents/index.ts
+++ b/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,
 ];
diff --git a/src/permissions/checker.ts b/src/permissions/checker.ts
index 5a8093d..f13377a 100644
--- a/src/permissions/checker.ts
+++ b/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
diff --git a/src/tests/hooks/integration.test.ts b/src/tests/hooks/integration.test.ts
index b788ca9..b0fd6f2 100644
--- a/src/tests/hooks/integration.test.ts
+++ b/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");