docs: add implementation plan and spec

GAP_ANALYSIS_SUMMARY.md

@@ -0,0 +1,138 @@
+# Architecture Gap Analysis - Executive Summary
+
+**Date**: 2026-02-28  
+**Author**: Subagent (document-arch-gap-analysis)  
+**Status**: ✅ COMPLETE - Ready for implementation
+
+---
+
+## TL;DR
+
+The new `deepchatAgentPresenter` architecture has **excellent streaming and message persistence** infrastructure, but is **missing the entire permission flow**. This is a **P0 MVP blocker** that requires immediate attention.
+
+---
+
+## What's Working ✅
+
+1. **Session Management** - `newAgentPresenter` creates/activates/deletes sessions correctly
+2. **Message Streaming** - `processStream()` handles LLM streaming with tool calls
+3. **Message Persistence** - `deepchat_messages` table stores messages correctly
+4. **Event System** - `SESSION_EVENTS` and `STREAM_EVENTS` work end-to-end
+5. **Frontend Stores** - `sessionStore` and `messageStore` handle state correctly
+
+---
+
+## What's Broken 🔴
+
+### Critical: Permission Flow Missing
+
+**Problem**: `executeTools()` in `dispatch.ts` calls tools **without any permission checks**.
+
+```typescript
+// Current code (BROKEN):
+for (const tc of state.completedToolCalls) {
+  const { rawData } = await toolPresenter.callTool(toolCall)  // ← NO PERMISSION CHECK!
+  // ...
+}
+```
+
+**Required Fix**:
+1. Create `PermissionChecker` class
+2. Check permissions BEFORE calling tools
+3. Create permission request blocks
+4. Pause stream and wait for user approval
+5. Resume after approval
+
+**Files to Modify**:
+- `src/main/presenter/deepchatAgentPresenter/dispatch.ts` - add permission check
+- `src/main/presenter/deepchatAgentPresenter/permissionChecker.ts` - CREATE NEW
+- `src/main/presenter/newAgentPresenter/index.ts` - add `handlePermissionResponse()`
+- `src/renderer/src/components/chat/ChatStatusBar.vue` - convert to dropdown
+
+**Database Changes**:
+- Add `permission_mode TEXT DEFAULT 'default'` to `new_sessions` table
+- Create `permission_whitelists` table
+
+---
+
+## Other Missing Features 🟡
+
+### Message Operations (P1 - High Priority)
+
+1. **Edit User Message** - Not implemented
+2. **Retry/Regenerate** - Not implemented (no variants, just append)
+3. **Fork Session** - Not implemented
+
+### Session Configuration 🟢 (P2 - Medium)
+
+- Currently only stores: `providerId`, `modelId`
+- Missing: `temperature`, `contextLength`, `maxTokens`, `systemPrompt`
+- Can use defaults for MVP
+
+---
+
+## Implementation Priority
+
+### P0: Critical (MVP Blockers) - 1-2 weeks
+
+1. ✅ Add `permission_mode` to `new_sessions` table
+2. 🔴 Create `PermissionChecker` class
+3. 🔴 Integrate permission check in `executeTools()`
+4. 🔴 Add `handlePermissionResponse()` IPC method
+5. 🔴 Update `ChatStatusBar` with permission dropdown
+6. 🔴 Implement whitelist storage and matching
+7. 🔴 Enforce `projectDir` boundary in full access mode
+
+### P1: High (Core Functionality) - 1 week
+
+1. Implement `editUserMessage()`
+2. Implement `retryMessage()` (no variants)
+3. Implement `forkSessionFromMessage()`
+4. Frontend UI for message actions
+
+### P2: Medium (Nice to Have) - 3-5 days
+
+1. Extend session configuration (temperature, etc.)
+2. Add 'paused' status for permission wait state
+3. Error recovery improvements
+
+---
+
+## Detailed Documentation
+
+See `docs/specs/agentpresenter-mvp-replacement/gap-analysis.md` for:
+- Complete functional comparison (old vs new)
+- Architecture diagrams
+- Implementation details
+- File reference map
+
+---
+
+## Next Steps
+
+1. **Review gap-analysis.md** with Claude
+2. **Start with P0 permission flow** implementation
+3. **Create database migration** for `permission_mode` and whitelists
+4. **Implement PermissionChecker** class
+5. **Test end-to-end permission flow**
+
+---
+
+## Files Created/Modified
+
+**Created**:
+- `docs/specs/agentpresenter-mvp-replacement/gap-analysis.md` (29KB)
+
+**Modified**:
+- `docs/specs/agentpresenter-mvp-replacement/spec.md` - Added implementation notes
+- `docs/specs/agentpresenter-mvp-replacement/plan.md` - Updated phases with status
+- `docs/specs/agentpresenter-mvp-replacement/tasks.md` - Added detailed tasks
+
+**Committed**: ✅ Yes (commit fc17e245)  
+**Pushed**: ❌ No (authentication required)
+
+---
+
+## Contact
+
+Questions? Review the full gap analysis or reach out to the development team.