Feature: Allow add multiple files with /file command (#25)

* feat(command_processor.py): enhance file command to support multiple file inputs and provide detailed processing feedback

This change allows users to process multiple files in a single command, improving usability. It also adds error handling and notifications for successful and failed file processing, ensuring users receive clear feedback on the command's execution.

* refactor(command_processor.py): add type hints for better code clarity and maintainability
fix(command_processor.py): ensure consistent handling of file content types and improve variable declarations

* refactor(command_processor.py): update comments for clarity and remove unused notification code to streamline the file command handling process

* refactor(command_processor.py): remove redundant notification for file processing and simplify success notification format

* refactor(command_processor.py): simplify file command handling by removing unnecessary checks and consolidating logic for better readability and maintainability

* refactor(command_processor.py): update type hint for all_file_contents to specify it contains dictionaries and enhance file content formatting
feat(command_processor.py): wrap file content with XML tags and structure it as a dictionary for better message handling

* refactor(command_processor.py): simplify file content processing by removing unnecessary variable assignments and XML formatting to enhance code clarity and maintainability

Author: lewistransts

AgentCrew/modules/chat/message/command_processor.py

@@ -1,5 +1,5 @@
 from dataclasses import dataclass
-from typing import Optional, Tuple
+from typing import Optional, Tuple, List, Dict, Any, Union
 import os
 
 from AgentCrew.modules.agents.local_agent import LocalAgent
@@ -438,39 +438,69 @@ def _handle_agent_command(self, command: str) -> Tuple[bool, str]:
             )
 
     def _handle_file_command(self, user_input: str) -> CommandResult:
-        """Handle file command."""
-        file_path = user_input[6:].strip()
-        file_path = os.path.expanduser(file_path)
-
-        self.message_handler._notify("file_processing", {"file_path": file_path})
-
-        # Process file with the file handling service
-        if self.message_handler.file_handler is None:
-            self.message_handler.file_handler = FileHandler()
-        file_content = self.message_handler.file_handler.process_file(file_path)
-        # Fallback to llm handle
-        if not file_content:
-            from AgentCrew.modules.agents.base import MessageType
-
-            file_content = self.message_handler.agent.format_message(
-                MessageType.FileContent, {"file_uri": file_path}
-            )
+        """Handle file command with support for multiple files."""
+        # Extract file paths from user input (space-separated)
+        file_paths_str: str = user_input[6:].strip()
+        file_paths: List[str] = [os.path.expanduser(path.strip()) for path in file_paths_str.split() if path.strip()]
+        
+        if not file_paths:
+            self.message_handler._notify("error", "No file paths provided")
+            return CommandResult(handled=True, clear_flag=True)
+
+        processed_files: List[str] = []
+        failed_files: List[str] = []
+        all_file_contents: List[Dict[str, str]] = []
+
+        # Process each file
+        for file_path in file_paths:
+            self.message_handler._notify("file_processing", {"file_path": file_path})
+
+            # Process file with the file handling service
+            if self.message_handler.file_handler is None:
+                self.message_handler.file_handler = FileHandler()
+            file_content = self.message_handler.file_handler.process_file(file_path)
+            
+            # Fallback to llm handle
+            if not file_content:
+                from AgentCrew.modules.agents.base import MessageType
+
+                file_content = self.message_handler.agent.format_message(
+                    MessageType.FileContent, {"file_uri": file_path}
+                )
 
-        if file_content:
+            if file_content:
+                all_file_contents.append(file_content)
+                processed_files.append(file_path)
+                self.message_handler._notify(
+                    "file_processed",
+                    {
+                        "file_path": file_path,
+                        "message": file_content,
+                    },
+                )
+            else:
+                failed_files.append(file_path)
+                self.message_handler._notify(
+                    "error",
+                    f"Failed to process file {file_path} Or Model is not supported",
+                )
+
+        # Add all successfully processed file contents to messages
+        if all_file_contents:
             self.message_handler._messages_append(
-                {"role": "user", "content": [file_content]}
-            )
-            self.message_handler._notify(
-                "file_processed",
-                {
-                    "file_path": file_path,
-                    "message": self.message_handler.agent.history[-1],
-                },
+                {"role": "user", "content": all_file_contents}
             )
-            return CommandResult(handled=True, clear_flag=True)
-        else:
-            self.message_handler._notify(
-                "error",
-                f"Failed to process file {file_path} Or Model is not supported",
-            )
-            return CommandResult(handled=True, clear_flag=True)
+            
+            # Notify about overall processing results
+            if failed_files:
+                self.message_handler._notify(
+                    "system_message",
+                    f"Processed {len(processed_files)} files successfully. Failed to process: {', '.join(failed_files)}"
+                )
+            else:
+                self.message_handler._notify(
+                    "system_message",
+                    f"Successfully processed {len(processed_files)} files: {', '.join(processed_files)}"
+                )
+
+        return CommandResult(handled=True, clear_flag=True)