Repository navigation
[TODO] Category 5: Cleanup/Refactoring (3 items) #205
Description
Parent Issue: #75
Description
3 TODO comments related to cleanup and refactoring tasks.
Items
1. ConversationFileManager Injection (Quick Win)
File: backend/api/conversation_files.py:159
TODO: This will be injected when ConversationFileManager is availableFix: ConversationFileManager now exists - inject it properly
2. Remove Placeholder Code (Quick Win)
File: backend/api/conversation_files.py:340
# TODO: Remove this when ConversationFileManager is integratedFix: Remove placeholder code, use actual ConversationFileManager
3. Redis Service Singleton Pattern
File: backend/api/redis_service.py:39
# TODO: Optimize to singleton pattern for production after development completeFix: Convert to singleton pattern following HTTPClient pattern
Acceptance Criteria
- ConversationFileManager properly injected
- Placeholder code removed
- Redis service uses singleton pattern
- TODO comments removed
Priority
Low - Code cleanup
Quick Wins
Items 1 and 2 can likely be fixed together in < 30 minutes
Activity
mrveiss commentedon Nov 26, 2025
Starting work on Issue #205 - Cleanup/Refactoring
mrveiss commentedon Nov 26, 2025
All 3 items completed:
get_service_manager()to singleton with:_service_managerglobal instance_service_manager_startedflag for tracking initializationCommitted: 65d23b0
refactor(redis-service): Convert to singleton pattern (#205)
refactor(redis-service): Convert to singleton pattern (#205)