docs(nori): Update noridocs for agent picker work

Author: CSRessel

.claude/settings.json

@@ -1,15 +1,10 @@
 {
   "$schema": "https://json.schemastore.org/claude-code-settings.json",
   "permissions": {
-    "additionalDirectories": [],
     "allow": [
       "Bash(cat:*)",
       "Bash(cd:*)",
-      "Bash(git diff:*)",
-      "Bash(git log:*)",
-      "Bash(git show:*)",
       "Bash(git status:*)",
-      "Bash(ls:*)",
       "Bash(mkdir:*)",
       "Bash(tree:*)",
       "Bash(cargo build:*)",
@@ -26,10 +21,10 @@
       "Bash(cargo test:*)",
       "Bash(cargo t:*)",
       "Bash(cargo search:*)"
+    ],
+    "additionalDirectories": [
+      "~/Documents/source/nori/cli/.claude/profiles",
+      "~/Documents/source/nori/cli/.claude/skills"
     ]
-  },
-  "sandbox": {
-    "enabled": true,
-    "autoAllowBashIfSandboxed": true
   }
 }

codex-rs/acp/docs.md

@@ -31,6 +31,14 @@ The ACP registry in `@/codex-rs/acp/src/registry.rs` is **model-centric** rather
 - `mock-model-alt` uses the same binary as `mock-model` but with provider_slug `mock-acp-alt` for E2E testing agent switching between different configurations
 - Claude ACP is registered for both "claude-4.5" and "claude-acp" model names, using `npx @zed-industries/claude-code-acp` command with no arguments
 
+### Agent Picker Metadata
+
+`list_available_agents()` (also in `acp/src/registry.rs`) returns `Vec<AcpAgentInfo>` so the TUI can render the `/agent` picker:
+- `model_name`, `display_name`, and `description` describe what to present in the selection view.
+- `provider_slug` mirrors the config slug so the UI can explain when different agents reuse the same backend.
+- `codex_tui::nori::agent_picker` consumes these entries to build the selection popup shown by `/agent`, while `/model` uses `acp_model_picker_params()` to surface a disabled message pointing back to `/agent`.
+- Selecting an agent raises `AppEvent::SetPendingAgent`, stores a `PendingAgentSelection`, and defers the actual switch until `AppEvent::SubmitWithAgentSwitch` rebuilds the `ChatWidget` with the new model.
+
 ### Embedded Provider Info
 
 ACP providers embed their configuration directly in `AcpAgentConfig` via `AcpProviderInfo`:

codex-rs/tui/docs.md

@@ -73,6 +73,14 @@ In `spawn_acp_agent()`, the main task must drop its `Arc<AcpBackend>` reference
 - `slash_command.rs`: `/command` parsing and execution
 - `file_search.rs`: Fuzzy file finder
 
+**ACP Agent Switching:**
+
+- `/agent` now opens the Nori-specific agent picker popup in `tui/src/chatwidget.rs`, which drives `nori::agent_picker::agent_picker_params()` and renders the metadata returned by `codex_acp::list_available_agents()` as `SelectionItem`s.
+- Selecting an agent sends `AppEvent::SetPendingAgent`, so both the App and `ChatWidget` store a `pending_agent` (see `PendingAgentSelection` and `PendingAgentInfo`). The UI informs the user that the switch will happen on the next prompt submission.
+- When the next prompt is submitted, `ChatWidget` intercepts the queued `UserMessage`, forwards it as `AppEvent::SubmitWithAgentSwitch`, and lets the App restart the conversation with the new model (clearing the pending flag, updating `Config`, shutting down the old conversation, and creating a `ChatWidget` with `expected_model` to filter out leftover events).
+- `/model` now checks `codex_acp::get_agent_config()`; if the workspace is in ACP mode it shows the disabled `acp_model_picker_params()` view that explicitly tells users to use `/agent` instead of selecting models directly.
+- This workflow avoids disrupting active turns and powers the agent-switching verification in `tui-pty-e2e/tests/agent_switching.rs`, including the message-flow and pending-selection tests added in the last commits.
+
 **Onboarding:**
 
 The `onboarding/` module handles first-run experience:
@@ -122,6 +130,10 @@ Most event types (exec begin/end, MCP calls, elicitation) are queued during acti
 - The `InterruptManager` still contains `ExecApproval` and `ApplyPatchApproval` variants for completeness, but these methods are marked `#[allow(dead_code)]`
 - `on_task_complete()` calls `flush_interrupt_queue()` for any remaining queued items
 
+**ACP File Tracing:**
+
+- The TUI calls `codex_acp::init_file_tracing()` at startup (`tui/src/lib.rs`) to write `.codex-acp.log` in the current directory. Every mock agent logs `ACP agent spawned (pid: ...)` there, which makes the agent-switching tests in `tui-pty-e2e` deterministic and ensures developers can inspect agent subprocess lifecycles during debugging.
+
 **Agent Switch Event Filtering:**
 
 When switching between ACP agents (e.g., via `/agent` command), `ChatWidget` uses an event filtering mechanism to prevent race conditions:

codex-rs/tui/src/nori/docs.md

@@ -48,6 +48,12 @@ The `NoriSessionHeaderCell` struct implements `HistoryCell` and renders:
 
 The banner uses green+bold for alphabetic characters and dark gray for structural characters (pipes, slashes) to create a two-tone visual effect.
 
+**Agent Picker (`agent_picker.rs`):**
+
+- `agent_picker_params()` consumes `codex_acp::list_available_agents()` so `/agent` can display each `AcpAgentInfo` entry (model name, display name, description, provider slug) with a `SelectionAction` that sends `AppEvent::SetPendingAgent`.
+- `acp_model_picker_params()` renders the `/model` fallback page that disables selection when ACP mode is active and points the user back to `/agent`.
+- `PendingAgentSelection` holds the selected model/display name pair so the App and `ChatWidget` can store it until the next prompt triggers `AppEvent::SubmitWithAgentSwitch`, at which point the conversation is rebuilt with the new model and the picker view is dismissed.
+
 ### Things to Know
 
 **Profile Display:**