fix(agent): TUI agent wiring, scout cleanup, planner paths, subagent default (#556-#559)

- TUI buildPromptInput sends CurrentAgent to server
- Scout prompt drops repo_clone/repo_overview (not implemented)
- Planner path-scoped edit restricted to plans/* and drafts/*
- Bundled .md agent defaults to mode: subagent

Author: bobbyjohnstx

docs/spec/07-agents.md

@@ -8,11 +8,12 @@ Agents define behavioral presets: system prompts, tool permissions, and model pa
 
 ```go
 type Info struct {
-    Name        string             // unique identifier
+    Name        string             // unique identifier (registry key is filename stem; Name may differ via frontmatter)
     Description string             // human-readable description
     Mode        Mode               // "primary", "subagent", "all"
     Native      bool               // true for hardcoded agents
     Hidden      bool               // hidden from user lists
+    Disabled    bool               // soft-disabled; Get/List skip these
     TopP        *float64           // LLM sampling parameter
     Temperature *float64           // LLM sampling parameter
     Color       string             // TUI display color (hex)
@@ -38,9 +39,13 @@ type Info struct {
 
 ### Loading Order
 
-1. **Native agents** — Hardcoded in `registerNativeAgents()` with specific permissions
-2. **Bundled agents** — Loaded from `defaults/*.md` via embedded filesystem (`embed.FS`)
-3. **Config overrides** — Applied from `config.Info.Agents` via `ApplyConfigOverrides()`
+1. **Native agents** — Hardcoded in `registerNativeAgents()` with specific permissions (prompts from `defaults/*.txt`, except `explore` which prefers `explore.md` body)
+2. **Bundled agents** — Loaded from `defaults/*.md` via embedded filesystem (`embed.FS`); skipped when a native agent already owns the name
+3. **User agents** — `LoadUserAgents`: `~/.config/tinycode/agents/*.md` then `~/.config/tinycode/agent/*.md` (compat)
+4. **Project agents** — `.tinycode/agent/*.md` (loaded after user; overwrites non-native)
+5. **Config overrides** — Applied from `config.Info.Agents` via `ApplyConfigOverrides()`
+
+User/project `.md` agents may replace bundled (non-native) agents. Native agents are never overwritten by directory loads.
 
 ### Native Agents
 
@@ -50,49 +55,49 @@ type Info struct {
 | `plan` | Primary | Read-only plan mode | `plan_exit:allow`, `edit:deny` |
 | `general` | Subagent | Multi-step task execution | `todowrite:deny` |
 | `explore` | Subagent | Fast codebase search | Only: `grep`, `glob`, `bash`, `webfetch`, `websearch`, `read` |
-| `scout` | Subagent | External research | Only: `grep`, `glob`, `webfetch`, `websearch`, `read`, `repo_clone`, `repo_overview` |
+| `scout` | Subagent | External research | Only: `grep`, `glob`, `webfetch`, `websearch`, `read` |
 | `compaction` | Primary (hidden) | Context summarization | All denied |
 | `title` | Primary (hidden) | Session title generation | All denied, temp=0.5 |
 | `summary` | Primary (hidden) | Session summary | All denied |
 
 ### Bundled Agents (from `defaults/*.md`)
 
-Loaded from embedded markdown files with YAML frontmatter:
+Loaded from embedded markdown files with YAML frontmatter. Actual set in Go `internal/agent/defaults/`:
 
 | Agent | Description |
 |-------|-------------|
 | `architect` | Code design and architecture review (read-only) |
 | `code-reviewer` | Severity-rated code review with SOLID checks |
-| `code-simplifier` | Refactoring for clarity and maintainability |
+| `code-simplifier` | Refactoring for clarity (archived: disabled by default) |
 | `critic` | Multi-perspective quality review |
 | `debugger` | Root-cause analysis and bug fixing |
-| `designer` | Production-grade UI/UX |
-| `document-specialist` | External library and API reference |
 | `executor` | Focused implementation of scoped tasks |
 | `git-master` | Git history, rebasing, atomic commits |
 | `planner` | Strategic planning and work breakdown |
-| `qa-tester` | Interactive CLI testing |
-| `scientist` | Data analysis and research |
+| `qa-tester` | Interactive CLI testing (archived: disabled by default) |
+| `scientist` | Data analysis and research (archived: disabled by default) |
 | `security-reviewer` | Security vulnerability detection |
 | `test-engineer` | Test strategy and TDD workflows |
-| `tracer` | Evidence-driven causal tracing |
 | `verifier` | Completion verification |
-| `workspace` | Development environment setup |
 | `writer` | Technical documentation |
-| `analyst` | Requirements analysis |
-| `cluster-admin` | Kubernetes/OpenShift operations |
+
+`explore.md` / `explore.compact.md` also live under defaults; the native `explore` agent uses the `explore.md` body as its prompt (permissions stay native). Compact peers of archived agents are disabled alongside the base agent.
 
 ## 7.3 Frontmatter Schema
 
-Agent `.md` files use YAML frontmatter:
+Agent `.md` files use YAML frontmatter. Shared parsing via `applyFrontmatter`:
 
 ```yaml
 ---
+name: optional-display-name
 description: Human-readable description
-mode: primary | subagent | all
+mode: primary | subagent | all   # omitted → subagent (bundled/user .md defaults)
 hidden: true | false
 color: "#ff0000"
 steps: 10
+temperature: 0.7
+top_p: 0.9
+model: ollama/qwen3:8b
 permission:
   edit: deny
   read: allow
@@ -104,6 +109,8 @@ permission:
 System prompt content here...
 ```
 
+The registry map key is always the filename stem. If `name` is set in frontmatter, it updates `Info.Name` only. Omitted `mode` defaults to `ModeSubagent` (not `ModeAll`).
+
 ### Permission Rules in Frontmatter
 
 Permissions support two formats:
@@ -115,7 +122,7 @@ permission:
   read: allow
 ```
 
-**Pattern-specific:**
+**Pattern-specific (nested maps):**
 ```yaml
 permission:
   bash:
@@ -125,23 +132,24 @@ permission:
 
 ## 7.4 Small-Model Variants
 
-When a model's parameter count is ≤8B (detected from model name), the registry returns a **compact variant** if available:
+When a model's parameter count is ≤8B (detected from model name), the registry returns a **compact variant** if available and not disabled:
 
 ```go
 func (r *Registry) Get(name string, modelSizeB *float64) *Info {
     if modelSizeB != nil && *modelSizeB <= 8 {
         compact := r.agents[name+".compact"]
-        if compact != nil {
+        if compact != nil && !compact.Disabled {
             result := *compact
+            result.Name = name       // requested base name, not "*.compact"
             result.Compact = true
             return &result
         }
     }
-    // ... return standard agent
+    // ... return standard agent (nil if missing or Disabled)
 }
 ```
 
-Compact variants are agent files named `<agent>.compact.md` with simplified prompts optimized for smaller models.
+Compact variants are agent files named `<agent>.compact.md` with simplified prompts optimized for smaller models. `Get` never serves disabled agents (base or compact).
 
 ## 7.5 Permission Merging
 
@@ -192,15 +200,15 @@ Users can modify agents via config:
 
 Override fields: `model`, `variant`, `prompt`, `description`, `temperature`, `top_p`, `mode`, `color`, `hidden`, `name`, `steps`, `options`, `permission`, `disable`.
 
-Setting `disable: true` removes the agent from the registry.
+Setting `disable: true` sets `Disabled` on the agent (soft disable). It does **not** remove the agent from the registry. Native agents cannot be disabled. `Get` and `List` skip disabled agents; `ListAll` includes them.
 
 ## 7.7 Default Agent Resolution
 
 `DefaultAgent(configDefault)` resolves the default agent:
 
-1. If `configDefault` is set in config, validate it exists and is a primary, visible agent
-2. Otherwise, use `"build"`
-3. Fallback: first registered non-subagent, non-hidden agent
+1. If `configDefault` is set in config, validate it exists and is a primary, visible, non-disabled agent
+2. Otherwise, use `"build"` if not disabled
+3. Fallback: first registered non-subagent, non-hidden, non-disabled agent
 
 ## 7.8 Agent Model Override
 
@@ -221,12 +229,13 @@ When an agent has a `Model` set, the session manager uses that model instead of
 
 Source: `internal/agent/loader.go`
 
-`LoadUserAgents(configDir, projectDir)` loads user-defined agents from two directories:
+`LoadUserAgents(configDir, projectDir)` loads user-defined agents from:
 
 1. **User config:** `~/.config/tinycode/agents/*.md`
-2. **Project:** `.tinycode/agent/*.md`
+2. **User compat:** `~/.config/tinycode/agent/*.md`
+3. **Project:** `.tinycode/agent/*.md`
 
-Files are parsed identically to bundled agents (frontmatter + body). Agents already in the registry are **not** overwritten — bundled and native agents take precedence.
+Files are parsed identically to bundled agents (frontmatter + body via `applyFrontmatter`). Non-native agents already in the registry **may be overwritten** (project wins over user over bundled). Native agents are never overwritten. Load errors are logged, not silently discarded.
 
 Compact variants (`*.compact.md`) are loaded alongside their full counterparts.