MCP Email Service

# AccountManager 实例化问题修复总结 ## 修复日期 2025-10-24 ## 问题描述 多个优化/并行操作模块在内部重新创建 `AccountManager()` 实例，导致当账户配置仅存在于内存中（未持久化到磁盘）时，这些模块无法访问到账户信息，从而抛出 "No email accounts configured" 错误。 ## 修复策略 所有受影响的类和函数现在都接受一个可选的 `account_manager` 参数。如果提供了该参数，则重用外部传入的实例；如果未提供，则回退到创建新实例（保持向后兼容）。 模式： ```python def function(param1, param2, account_manager=None): if account_manager is None: account_manager = AccountManager() # 使用 account_manager... ``` 或对于类： ```python class SomeClass: def __init__(self, param1, account_manager=None): self.account_manager = account_manager or AccountManager() ``` ## 已修复的文件 ### 🔴 高优先级（核心功能） #### 1. ✅ `src/operations/optimized_search.py` **修复内容**： - 函数 `search_single_account` 和 `search_all_accounts_parallel` 现在接受可选的 `account_manager` 参数 - `src/services/email_service.py` (Line 538) 已更新，在调用时传递 `self.account_manager` **变更**： - Line 47: `def search_single_account(..., account_manager=None)` - Line 64: `manager = account_manager or AccountManager()` - Line 133: `def search_all_accounts_parallel(..., account_manager=None)` - Line 169: `manager = account_manager or AccountManager()` **调用方修复**： - `src/services/email_service.py` Line 538: 添加 `account_manager=self.account_manager` --- #### 2. ✅ `src/operations/optimized_fetch.py` **修复内容**： - 函数 `fetch_all_providers_optimized` 现在接受可选的 `account_manager` 参数 - `src/services/email_service.py` (Line 113) 已更新，在调用时传递 `self.account_manager` **变更**： - Line 136: 函数签名添加 `account_manager=None` 参数 - Line 144: 文档字符串添加参数说明 - Line 157-159: 添加条件判断，重用或创建实例 **调用方修复**： - `src/services/email_service.py` Line 110-113: 添加 `account_manager=self.account_manager` --- #### 3. ✅ `src/operations/parallel_operations.py` **修复内容**： - 类 `ParallelOperations` 的 `__init__` 现在接受可选的 `account_manager` 参数 **变更**： - Line 18: `__init__(self, max_workers: int = 5, account_manager=None)` - Line 24: 文档字符串添加参数说明 - Line 28: `self.account_manager = account_manager or AccountManager()` **注意**： - 全局实例 `parallel_ops` (Line 438) 仍使用默认参数 - 如果需要重用 account_manager，应在服务层创建新实例 --- #### 4. ✅ `src/operations/email_sync.py` **修复内容**： - 类 `EmailSyncManager` 的 `__init__` 现在接受可选的 `account_manager` 参数 **变更**： - Line 23: `__init__(self, db_path=None, config=None, account_manager=None)` - Line 24-30: 添加完整的文档字符串 - Line 32: `self.account_manager = account_manager or AccountManager()` --- ### 🟡 中优先级（辅助功能） #### 5. ✅ `src/operations/fast_fetch.py` **修复内容**： - 函数 `fast_fetch_all_accounts` 现在接受可选的 `account_manager` 参数 **变更**： - Line 79: 函数签名添加 `account_manager=None` - Line 80-90: 添加完整的文档字符串 - Line 98-99: 添加条件判断 --- #### 6. ✅ `src/operations/multi_folder_fetch.py` **修复内容**： - 类 `MultiFolderFetcher` 的 `__init__` 现在接受可选的 `account_manager` 参数 **变更**： - Line 33: `__init__(self, max_workers=5, account_manager=None)` - Line 34-40: 添加完整的文档字符串 - Line 43: `self.account_manager = account_manager or AccountManager()` **注意**： - 全局实例 `multi_folder_fetcher` (Line 294) 仍使用默认参数 --- #### 7. ✅ `src/operations/parallel_search.py` **修复内容**： - 类 `ParallelSearchOperations` 的 `__init__` 现在接受可选的 `account_manager` 参数 **变更**： - Line 19: `__init__(self, max_workers=5, account_manager=None)` - Line 20-26: 添加完整的文档字符串 - Line 29: `self.account_manager = account_manager or AccountManager()` **注意**： - 全局实例 `parallel_search` (Line 254) 仍使用默认参数 - 该模块可能已被 `optimized_search` 替代 --- ## 未修复的文件（合理用法） 以下文件中的 `AccountManager()` 创建是合理的，无需修复： ### 脚本和工具 - `src/mcp_tools.py` - MCP 服务器主入口 - `src/legacy_operations.py` - 模块级全局实例 - `clients/mailbox_client/client.py` - 独立客户端 - `scripts/*.py` - 独立脚本 - `tests/*.py` - 测试代码 ### 可能需要审查的文件 - `src/database_integration.py` - 如果被其他模块依赖，可能需要类似修复 - `src/operations/folder_scan.py` - 独立工具，低优先级 --- ## 编译验证 所有修复的文件已通过 Python 编译检查： ```bash ✅ src/operations/optimized_search.py ✅ src/operations/optimized_fetch.py ✅ src/services/email_service.py ✅ src/operations/parallel_operations.py ✅ src/operations/email_sync.py ✅ src/operations/fast_fetch.py ✅ src/operations/multi_folder_fetch.py ✅ src/operations/parallel_search.py ``` 命令： ```bash python3 -m compileall <files> ``` --- ## 向后兼容性 ✅ **完全向后兼容** 所有修改都使用可选参数（默认为 `None`），因此： - 现有代码无需修改即可继续工作 - 新代码可以选择传递 `account_manager` 以获得更好的一致性 - 如果不传递参数，行为与修复前完全相同 --- ## 测试建议 ### 场景 1：动态账户（内存配置） ```python from src.account_manager import AccountManager from src.services.email_service import EmailService # 创建临时账户（不持久化） account_manager = AccountManager() account_manager.add_account({ "id": "temp-001", "email": "test@example.com", "password": "xxx", "imap_server": "imap.example.com", "provider": "custom" }, persist=False) # 使用服务层 email_service = EmailService(account_manager) # 测试优化路径 result = email_service.list_emails(unread_only=True) # ✅ 应该成功，不会报 "No accounts" 错误 result = email_service.search_emails(query="test") # ✅ 应该成功，不会报 "No accounts" 错误 ``` ### 场景 2：持久化账户（文件配置） ```python # 标准用法，无需修改 email_service = EmailService(AccountManager()) result = email_service.list_emails() # ✅ 继续正常工作 ``` ### 场景 3：并行操作 ```python from src.operations.parallel_operations import ParallelOperations # 使用共享的 account_manager account_manager = AccountManager() parallel_ops = ParallelOperations(account_manager=account_manager) # 执行批量操作 result = parallel_ops.execute_batch_operation(...) # ✅ 使用相同的账户配置 ``` --- ## 相关文档 - **问题分析报告**: `ACCOUNT_MANAGER_ISSUES_REPORT.md` - **架构文档**: `docs/ARCHITECTURE.md` - **原子操作升级**: `docs/ATOMIC_OPERATIONS_UPGRADE.md` --- ## 贡献者 - 问题发现：用户反馈 - 问题分析：Cursor AI + 人工审查 - 修复实施：自动化修复 + 人工验证 - 测试：编译检查通过 --- ## 下一步建议 ### 短期（可选） 1. 在单元测试中添加场景 1 的测试用例 2. 在文档中说明推荐的使用模式 ### 中期（可选） 3. 审查 `database_integration.py` 和 `folder_scan.py` 4. 考虑在全局实例创建时也支持传递参数 ### 长期（架构改进） 5. 引入依赖注入容器统一管理实例 6. 考虑使用单例模式管理 AccountManager（需谨慎评估） --- **修复完成！** 🎉 所有核心和辅助功能模块现在都支持重用 `AccountManager` 实例，解决了内存配置无法被优化路径识别的问题。