fix: remove dead code src/rag_manager.py (#747) #798

Closed

sleepy wants to merge 0 commits from fix/remove-rag-manager-747 into main

sleepy commented

2026-06-03 15:32:13 +02:00

Owner

Removes src/rag_manager.py, a 59-line pure delegation class (RAGManager) where every method just called self.vector_rag.same_method() with no added logic, caching, or transformation.

Changes:

Deleted src/rag_manager.py
Updated services/docs/init.py - removed RAGManager re-export
Updated services/docs/service.py - use VectorRAG directly
Updated scripts/index_documents.py - use VectorRAG directly

src/rag_singleton.py:get_rag_manager() already returned a VectorRAG instance, so no callers through that path needed changes.

Closes #747

Removes src/rag_manager.py, a 59-line pure delegation class (RAGManager) where every method just called self.vector_rag.same_method() with no added logic, caching, or transformation. Changes: - Deleted src/rag_manager.py - Updated services/docs/__init__.py - removed RAGManager re-export - Updated services/docs/service.py - use VectorRAG directly - Updated scripts/index_documents.py - use VectorRAG directly src/rag_singleton.py:get_rag_manager() already returned a VectorRAG instance, so no callers through that path needed changes. Closes #747

sleepy added 1 commit

2026-06-03 15:32:13 +02:00

fix: remove dead code src/rag_manager.py (#747 ) 971b51b29f

RAGManager was a zero-value 59-line wrapper where every method just
delegates to self.vector_rag.<same_method>() with no added logic,
caching, or transformation.

- Delete src/rag_manager.py
- Replace all RAGManager imports with direct VectorRAG imports
- Update services/docs/{__init__,service}.py and scripts/index_documents.py
- rag_singleton already returned VectorRAG, so no change needed there

sleepy closed this pull request

2026-06-18 18:31:10 +02:00