docs: Update tool count to 59, add depth limit to discover_files

- Update MCP_IMPROVEMENTS.md to reflect 59 tools (was 55)
- Mark implemented tools in potential tools table
- Add MAX_DISCOVERY_DEPTH (20) to prevent excessive recursion
- Track depth in discover_files stack for safety

Addresses CodeRabbit review comments

Author: rishitank

docs/MCP_IMPROVEMENTS.md

@@ -5,7 +5,7 @@ This document outlines potential improvements to make the Context Engine MCP Ser
 ## Current Implementation Status
 
 ### ✅ Fully Implemented
-- **Tools** - All 55 tools for retrieval, indexing, memory, planning, review, navigation, and workspace analysis
+- **Tools** - All 59 tools for retrieval, indexing, memory, planning, review, navigation, and workspace analysis
 - **JSON-RPC 2.0** - Full request/response/notification handling
 - **Stdio Transport** - Standard input/output for MCP clients
 - **HTTP Transport** - Axum-based HTTP server with SSE
@@ -156,26 +156,29 @@ Allow clients to cancel in-progress operations.
 
 ## Enhanced Tool Capabilities
 
-### Current Tools (49)
+### Current Tools (59)
 - **Retrieval (6):** semantic_search, grep_search, file_search, etc.
 - **Index (5):** index_status, index_directory, clear_index, etc.
 - **Memory (4):** memory_store, memory_retrieve, memory_list, memory_delete
 - **Planning (20):** create_review, analyze_changes, etc.
 - **Review (14):** review_code, suggest_fixes, etc.
+- **Navigation (3):** find_references, go_to_definition, diff_files
+- **Workspace (7):** workspace_stats, git_status, extract_symbols, git_blame, git_log, dependency_graph, file_outline
 
 ### Potential New Tools
 
-| Tool | Description | Priority |
-|------|-------------|----------|
-| `diff_files` | Compare two files | High |
-| `find_references` | Find all references to a symbol | High |
-| `go_to_definition` | Find definition of a symbol | High |
-| `call_hierarchy` | Show call graph for a function | Medium |
-| `type_hierarchy` | Show class/type inheritance | Medium |
-| `ast_query` | Query AST with tree-sitter | Medium |
-| `git_blame` | Show git blame for a file | Low |
-| `git_history` | Show commit history | Low |
-| `dependency_graph` | Show module dependencies | Low |
+| Tool | Description | Priority | Status |
+|------|-------------|----------|--------|
+| `diff_files` | Compare two files | High | ✅ Implemented |
+| `find_references` | Find all references to a symbol | High | ✅ Implemented |
+| `go_to_definition` | Find definition of a symbol | High | ✅ Implemented |
+| `call_hierarchy` | Show call graph for a function | Medium | 🔲 Planned |
+| `type_hierarchy` | Show class/type inheritance | Medium | 🔲 Planned |
+| `ast_query` | Query AST with tree-sitter | Medium | 🔲 Planned |
+| `git_blame` | Show git blame for a file | Low | ✅ Implemented |
+| `git_history` | Show commit history | Low | ✅ Implemented (git_log) |
+| `dependency_graph` | Show module dependencies | Low | ✅ Implemented |
+| `file_outline` | Get structured outline of symbols | Low | ✅ Implemented |
 
 ---
 

src/mcp/resources.rs

@@ -262,7 +262,10 @@ impl ResourceRegistry {
         Ok(())
     }
 
-    /// Discover files in directory (with pagination).
+    /// Maximum recursion depth for file discovery to prevent excessive traversal.
+    const MAX_DISCOVERY_DEPTH: usize = 20;
+
+    /// Discover files in directory (with pagination and depth limit).
     async fn discover_files(
         &self,
         dir: &std::path::Path,
@@ -272,14 +275,20 @@ impl ResourceRegistry {
         use tokio::fs::read_dir;
 
         let mut files = Vec::new();
-        let mut stack = vec![dir.to_path_buf()];
+        // Stack contains (path, depth) tuples
+        let mut stack = vec![(dir.to_path_buf(), 0usize)];
         let mut past_cursor = after.is_none();
 
-        while let Some(current) = stack.pop() {
+        while let Some((current, depth)) = stack.pop() {
             if files.len() >= limit {
                 break;
             }
 
+            // Skip if we've exceeded the maximum depth
+            if depth > Self::MAX_DISCOVERY_DEPTH {
+                continue;
+            }
+
             let mut entries = match read_dir(&current).await {
                 Ok(e) => e,
                 Err(_) => continue,
@@ -299,7 +308,7 @@ impl ResourceRegistry {
                 }
 
                 if path.is_dir() {
-                    stack.push(path);
+                    stack.push((path, depth + 1));
                 } else if path.is_file() {
                     let relative = path
                         .strip_prefix(dir)