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

Open

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

This branch is already included in the target branch. There is nothing to merge.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin fix/remove-rag-manager-747:fix/remove-rag-manager-747

git switch fix/remove-rag-manager-747

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main

git merge --no-ff fix/remove-rag-manager-747

git switch fix/remove-rag-manager-747

git rebase main

git switch main

git merge --ff-only fix/remove-rag-manager-747

git switch fix/remove-rag-manager-747

git rebase main

git switch main

git merge --no-ff fix/remove-rag-manager-747

git switch main

git merge --squash fix/remove-rag-manager-747