revert(session_plan): remove the per-session plan toolset

agent-schema.json

@@ -2152,7 +2152,6 @@
             "background_jobs",
             "tasks",
             "plan",
-            "session_plan",
             "session_context",
             "todo",
             "fetch",
@@ -2494,7 +2493,6 @@
                 "background_jobs",
                 "tasks",
                 "plan",
-                "session_plan",
                 "session_context",
                 "todo",
                 "fetch",

docs/concepts/tools/index.md

@@ -53,7 +53,6 @@ Docker Agent ships with several built-in tools that require no external dependen
 | [Scheduler](../../tools/scheduler/index.md) | Schedule instructions to run at a time or on a recurring interval |
 | [Webhook](../../tools/webhook/index.md) | Outbound notifications to Slack, Discord, Telegram, IFTTT, and more |
 | [Plan](../../tools/plan/index.md) | Shared persistent scratchpad for multi-agent collaboration |
-| [Session Plan](../../tools/session_plan/index.md) | Per-session plan tracker for the draft/review/execute workflow |
 | [Session Context](../../tools/session_context/index.md) | Reference a previous session as context |
 
 ## MCP Tools

docs/configuration/tools/index.md

@@ -26,7 +26,6 @@ Built-in tools are included with Docker Agent and require no external dependenci
 | `environment` | Report the OS and resolved shell (read-only, no arguments, auto-approved) | [Environment](../../tools/environment/index.md) |
 | `think` | Reasoning scratchpad | [Think](../../tools/think/index.md) |
 | `plan` | Shared persistent scratchpad for multi-agent collaboration | [Plan](../../tools/plan/index.md) |
-| `session_plan` | Per-session markdown plan for the draft-review-execute workflow | [Session Plan](../../tools/session_plan/index.md) |
 | `session_context` | Reference a previous session as context (read-only) | [Session Context](../../tools/session_context/index.md) |
 | `todo` | Task list management | [Todo](../../tools/todo/index.md) |
 | `memory` | Persistent key-value storage (SQLite) | [Memory](../../tools/memory/index.md) |

docs/data/nav.yml

@@ -103,8 +103,6 @@
           url: /tools/think/
         - title: Plan
           url: /tools/plan/
-        - title: Session Plan
-          url: /tools/session_plan/
         - title: Todo
           url: /tools/todo/
         - title: Tasks

docs/features/cli/index.md

@@ -723,7 +723,7 @@ $ docker agent plans get --session <session-id>       # a session's plan
 $ docker agent plans export --session <session-id> --output ./session-plan.md
 ```
 
-Plans live under the data directory (`~/.cagent/plans/` and `~/.cagent/session_plans/` by default), so `--data-dir` selects which store the commands operate on.
+Plans live under the data directory (`~/.cagent/plans/` by default), so `--data-dir` selects which store the commands operate on.
 
 ### `docker agent debug`
 

docs/features/tui/index.md

@@ -81,7 +81,7 @@ Type `/` during a session to see available commands, or press <kbd>Ctrl</kbd>+<k
 | `/snapshots`       | List captured snapshots (only when snapshots are enabled)                            |
 | `/export`          | Export the session as HTML                                                           |
 | `/sessions`        | Browse and load past sessions                                                        |
-| `/plans`           | Browse and manage plans: every [shared plan](../../tools/plan/index.md) plus the current session's [session plan](../../tools/session_plan/index.md). Filter, open a detail view, refresh, export to a file, and — for shared plans — set status, edit/create in `$VISUAL`/`$EDITOR`, and delete, all guarded against concurrent edits. Not available in the lean TUI |
+| `/plans`           | Browse and manage [shared plans](../../tools/plan/index.md). Filter, open a detail view, refresh, export to a file, set status, edit/create in `$VISUAL`/`$EDITOR`, and delete, all guarded against concurrent edits. Not available in the lean TUI |
 | `/model`           | Change the model for the current agent                                               |
 | `/effort`          | Set the current model's reasoning-effort level (`/effort <none\|minimal\|low\|medium\|high\|xhigh\|max>`, or `/effort` alone to pick from the supported levels; reasoning models only). Press <kbd>Tab</kbd> after `/effort` and a space to complete a level the current model supports |
 | `/settings`        | Manage appearance, behavior, and notification preferences                           |

docs/tools/plan/index.md

@@ -142,7 +142,7 @@ $ docker agent plans update release --file ./plan.md --expected-version 1
 
 ### The `/plans` browser in the TUI
 
-Inside the full-screen TUI, the `/plans` slash command (also in the <kbd>Ctrl</kbd>+<kbd>K</kbd> command palette) opens a plan browser over the same store the agents use, so changes made by agents mid-session appear immediately. The list shows every shared plan plus the current session's [session plan](../session_plan/index.md), with each plan's scope, identity (name, or session ID for the session plan), status, version (`-` for the unversioned session plan), last update time, and title.
+Inside the full-screen TUI, the `/plans` slash command (also in the <kbd>Ctrl</kbd>+<kbd>K</kbd> command palette) opens a plan browser over the same store the agents use, so changes made by agents mid-session appear immediately. The list shows every shared plan with its identity (name), status, version, last update time, and title.
 
 Keybindings:
 
@@ -151,7 +151,7 @@ Keybindings:
 | <kbd>↑</kbd>/<kbd>↓</kbd>, mouse | Navigate; <kbd>Enter</kbd> or double-click opens a detail view with the full metadata and scrollable markdown content |
 | <kbd>/</kbd> | Filter by name, title, status, or scope (<kbd>Esc</kbd> leaves filter mode) |
 | <kbd>r</kbd> | Refresh from storage |
-| <kbd>x</kbd> | Export the selected plan to `<name>.md` (shared) or `session-plan-<short-id>.md` (session) in the session's working directory. An existing file is never overwritten — the export fails with a notification instead |
+| <kbd>x</kbd> | Export the selected plan to `<name>.md` in the session's working directory. An existing file is never overwritten — the export fails with a notification instead |
 | <kbd>s</kbd> | Set a shared plan's free-form status via a small input dialog |
 | <kbd>e</kbd> | Edit a shared plan's content in `$VISUAL`/`$EDITOR` |
 | <kbd>n</kbd> | Create a new shared plan: pick a name, then draft the content in `$VISUAL`/`$EDITOR` (an empty draft aborts) |

pkg/runtime/client.go

@@ -88,7 +88,6 @@ func NewClient(baseURL string, opts ...ClientOption) (*Client, error) {
 			"stream_started":         func() Event { return &StreamStartedEvent{} },
 			"shell":                  func() Event { return &ShellOutputEvent{} },
 			"session_title":          func() Event { return &SessionTitleEvent{} },
-			"session_plan_updated":   func() Event { return &SessionPlanUpdatedEvent{} },
 			"plan_changed":           func() Event { return &PlanChangedEvent{} },
 			"session_summary":        func() Event { return &SessionSummaryEvent{} },
 			"session_compaction":     func() Event { return &SessionCompactionEvent{} },

pkg/runtime/event.go

@@ -445,29 +445,6 @@ func SessionTitle(sessionID, title string) Event {
 
 func (e *SessionTitleEvent) GetSessionID() string { return e.SessionID }
 
-// SessionPlanUpdatedEvent fires when the session_plan toolset writes a plan.
-// Content and Path let a UI render the plan inline without re-reading the file.
-type SessionPlanUpdatedEvent struct {
-	AgentContext
-
-	Type      string `json:"type"`
-	SessionID string `json:"session_id"`
-	Content   string `json:"content,omitempty"`
-	Path      string `json:"path,omitempty"`
-}
-
-func SessionPlanUpdated(sessionID, content, path, agentName string) Event {
-	return &SessionPlanUpdatedEvent{
-		Type:         "session_plan_updated",
-		SessionID:    sessionID,
-		Content:      content,
-		Path:         path,
-		AgentContext: newAgentContext(agentName),
-	}
-}
-
-func (e *SessionPlanUpdatedEvent) GetSessionID() string { return e.SessionID }
-
 // PlanChangedEvent fires after an agent successfully mutates a shared plan
 // (write, status change, or delete) through the plan toolset. It carries
 // identity and version only — no content — so a UI refreshes through its own

pkg/runtime/loop.go

@@ -33,7 +33,6 @@ import (
 	"github.com/docker/docker-agent/pkg/tools/builtin/modelpicker"
 	"github.com/docker/docker-agent/pkg/tools/builtin/plan"
 	"github.com/docker/docker-agent/pkg/tools/builtin/sessioncontext"
-	"github.com/docker/docker-agent/pkg/tools/builtin/sessionplan"
 	"github.com/docker/docker-agent/pkg/tools/builtin/skills"
 	"github.com/docker/docker-agent/pkg/tools/builtin/transfertask"
 	"github.com/docker/docker-agent/pkg/userconfig"
@@ -47,9 +46,6 @@ func (r *LocalRuntime) registerDefaultTools() {
 	r.toolMap[modelpicker.ToolNameChangeModel] = r.handleChangeModel
 	r.toolMap[modelpicker.ToolNameRevertModel] = r.handleRevertModel
 	r.toolMap[skills.ToolNameRunSkill] = r.handleRunSkill
-	r.toolMap[sessionplan.ToolNameWriteSessionPlan] = r.handleWriteSessionPlan
-	r.toolMap[sessionplan.ToolNameReadSessionPlan] = r.handleReadSessionPlan
-	r.toolMap[sessionplan.ToolNameExitPlanMode] = r.handleExitPlanMode
 	r.toolMap[sessioncontext.ToolNameListSessions] = r.handleListSessions
 	r.toolMap[sessioncontext.ToolNameReadSession] = r.handleReadSession