refactor: Unify resource tool with shared ResourceCore architecture

- **Architecture**: Move cli/resource/ to shared location for CLI+tool access
- **DRY Implementation**: Replace 600+ lines of duplicated tool logic with shared ResourceCore
- **Parameter Matching**: Update tool parameters to match CLI exactly (r#type, include, exclude)
- **Enhanced Features**: Add index_type (fast/best), include/exclude patterns, Show type filtering
- **Validation**: Enforce pinned storage only supports files/glob patterns
- **Documentation**: Update tool_index.json with 80% accuracy spec and storage limitations
- **Clean Architecture**: Separate concerns with ToolRenderer for tool-specific output
- **Functionality**: Complete parity with CLI while reducing code by 60%

Operations: Add, Remove, Clear, Search, Update, Show (with filtering), Cancel
Storage Types: Pinned (session-scoped, files only) vs Indexed (persistent, searchable)
Advanced: Fast/best indexing, glob patterns, comprehensive validation

Author: kennvene-aws

…cli/src/cli/chat/cli/resource/command.rs …es/chat-cli/src/cli/chat/cli/resource.rscrates/chat-cli/src/cli/chat/cli/resource/command.rs renamed to crates/chat-cli/src/cli/chat/cli/resource.rs crates/chat-cli/src/cli/chat/cli/resource/command.rs renamed to crates/chat-cli/src/cli/chat/cli/resource.rs

@@ -1,10 +1,9 @@
 use clap::Subcommand;
 use eyre::Result;
 use crate::cli::chat::{ChatError, ChatSession, ChatState};
-use crate::cli::chat::cli::resource::StorageType;
+use crate::cli::chat::resource::{StorageType, ResourceOperation, ResourceData};
+use crate::cli::chat::resource::core::ResourceCore;
 use crate::os::Os;
-use super::types::ResourceOperation;
-use super::core::ResourceCore;
 
 // Parameter structs to simplify function signatures
 #[derive(Debug)]
@@ -262,7 +261,7 @@ async fn invoke_by_storage_type(
     storage_type: StorageType,
     os: &Os,
     session: &mut ChatSession,
-) -> Result<super::types::ResourceData, ChatError> {
+) -> Result<ResourceData, ChatError> {
     match storage_type {
         StorageType::Pinned => {
             let context_manager = session.conversation.context_manager.as_mut()
@@ -280,8 +279,8 @@ async fn invoke_by_storage_type(
 }
 
 /// Render resource data to session with colors and styling
-fn render_to_session(data: &super::types::ResourceData, session: &mut ChatSession) -> Result<(), std::io::Error> {
-    use super::renderer::{CliRenderer, ResourceRenderer};
+fn render_to_session(data: &ResourceData, session: &mut ChatSession) -> Result<(), std::io::Error> {
+    use crate::cli::chat::resource::renderer::{CliRenderer, ResourceRenderer};
     let renderer = CliRenderer::new();
     renderer.render_with_session(data, session)
 }
@@ -302,7 +301,7 @@ fn validate_add_inputs(
     index_type: &Option<String>
 ) -> Result<(), ChatError> {
     if *storage_type == StorageType::Pinned && (!include.is_empty() || !exclude.is_empty() || index_type.is_some()) {
-        let warning_data = super::ResourceData::Success(
+        let warning_data = ResourceData::Success(
             "Warning: include/exclude patterns and index-type are ignored for pinned resources".to_string()
         );
         render_to_session(&warning_data, session)?;

crates/chat-cli/src/cli/chat/mod.rs

@@ -1,6 +1,7 @@
 pub mod cli;
 mod consts;
 pub mod context;
+pub mod resource;
 mod conversation;
 mod error_formatter;
 mod input_source;

…at-cli/src/cli/chat/cli/resource/core.rs …s/chat-cli/src/cli/chat/resource/core.rscrates/chat-cli/src/cli/chat/cli/resource/core.rs renamed to crates/chat-cli/src/cli/chat/resource/core.rs crates/chat-cli/src/cli/chat/cli/resource/core.rs renamed to crates/chat-cli/src/cli/chat/resource/core.rs

@@ -1,7 +1,5 @@
 pub mod types;
-pub mod command;
 pub mod renderer;
 pub mod core;
 
 pub use types::*;
-pub use command::ResourceCommand;

…hat-cli/src/cli/chat/cli/resource/mod.rs …es/chat-cli/src/cli/chat/resource/mod.rscrates/chat-cli/src/cli/chat/cli/resource/mod.rs renamed to crates/chat-cli/src/cli/chat/resource/mod.rs crates/chat-cli/src/cli/chat/cli/resource/mod.rs renamed to crates/chat-cli/src/cli/chat/resource/mod.rs

@@ -1,6 +1,6 @@
 use crossterm::{execute, style::{self, Color}};
 
-use crate::cli::chat::cli::ChatSession;
+use crate::cli::chat::ChatSession;
 use super::types::{ResourceData, PinnedResourceData, IndexedResourceData, OutputFormat};
 
 // Styling helpers to eliminate DRY violations
@@ -217,3 +217,66 @@ impl CliRenderer {
         Ok(())
     }
 }
+pub struct ToolRenderer;
+
+impl ToolRenderer {
+    pub fn new() -> Self {
+        Self
+    }
+}
+
+impl ResourceRenderer for ToolRenderer {
+    fn render(&self, data: &ResourceData, _format: OutputFormat) -> String {
+        match data {
+            ResourceData::Success(msg) => msg.clone(),
+            ResourceData::PinnedResources(pinned) => {
+                if pinned.session_files.is_empty() && pinned.agent_files.is_empty() {
+                    "No pinned resources found.".to_string()
+                } else {
+                    let total_files = pinned.session_files.len() + pinned.agent_files.len();
+                    let mut output = format!("📌 Pinned Resources ({}):\n", total_files);
+                    if !pinned.session_files.is_empty() {
+                        output.push_str(&format!("• Session files: {} items\n", pinned.session_files.len()));
+                    }
+                    if !pinned.agent_files.is_empty() {
+                        output.push_str(&format!("• Agent files: {} items\n", pinned.agent_files.len()));
+                    }
+                    output.push_str(&format!("• Total tokens: {}", pinned.total_tokens));
+                    output
+                }
+            }
+            ResourceData::IndexedResources(indexed) => {
+                if indexed.items.is_empty() {
+                    "No indexed resources found.".to_string()
+                } else {
+                    let mut output = format!("🔍 Indexed Resources ({}):\n", indexed.items.len());
+                    for item in &indexed.items {
+                        let status = match item.metadata.resource_type.as_str() {
+                            "indexing" => " (indexing)",
+                            _ => ""
+                        };
+                        output.push_str(&format!("• {}{} ({} items) - Type: indexed\n", item.name, status, item.metadata.size));
+                    }
+                    output
+                }
+            }
+            ResourceData::Status(status) => {
+                let mut output = format!("Status: {} items total\n", status.total_items);
+                if status.active_operations.is_empty() {
+                    output.push_str("No active operations");
+                } else {
+                    output.push_str(&format!("Active Operations ({}):\n", status.active_operations.len()));
+                    for op in &status.active_operations {
+                        output.push_str(&format!("• {} - {}\n", op.operation_type, op.status));
+                    }
+                }
+                output
+            }
+        }
+    }
+
+    fn render_with_session(&self, _data: &ResourceData, _session: &mut ChatSession) -> Result<(), std::io::Error> {
+        // Tools don't use session rendering, just return the string
+        Ok(())
+    }
+}

…li/src/cli/chat/cli/resource/renderer.rs …at-cli/src/cli/chat/resource/renderer.rscrates/chat-cli/src/cli/chat/cli/resource/renderer.rs renamed to crates/chat-cli/src/cli/chat/resource/renderer.rs crates/chat-cli/src/cli/chat/cli/resource/renderer.rs renamed to crates/chat-cli/src/cli/chat/resource/renderer.rs

@@ -110,7 +110,7 @@ impl Tool {
             Tool::Custom(custom_tool) => custom_tool.eval_perm(agent),
             Tool::GhIssue(_) => PermissionEvalResult::Allow,
             Tool::Thinking(_) => PermissionEvalResult::Allow,
-            Tool::Resource(resource) => resource.eval_perm(agent),
+            Tool::Resource(resource) => resource.permission_eval(agent),
         }
     }
 
@@ -129,7 +129,7 @@ impl Tool {
             Tool::UseAws(use_aws) => use_aws.invoke(os, stdout).await,
             Tool::Custom(custom_tool) => custom_tool.invoke(os, stdout).await,
             Tool::GhIssue(gh_issue) => gh_issue.invoke(os, stdout).await,
-            Tool::Resource(resource) => resource.invoke(os, stdout, agent).await,
+            Tool::Resource(resource) => resource.invoke(os, agent).await,
             Tool::Thinking(think) => think.invoke(stdout).await,
         }
     }