feat: Split MCP into Query and File servers, add tests, configure Tesseract French (BRIC-10)

[Kaiohz]

Summary

• Split single mcp_tools.py into two separate MCP servers:
  • RAGAnythingQuery at /rag/mcp — query_knowledge_base, query_knowledge_base_multimodal
  • RAGAnythingFiles at /files/mcp — list_files, read_file
• Remove stdio transport support (streamable HTTP only), remove MCP_TRANSPORT config field
• Add tesseract-ocr-fra to Dockerfile for French OCR support
• Replace print() with logger.debug() in kreuzberg_adapter
• Add ~85 new tests covering MCP tools, MinioAdapter, LightRAG multimodal, config properties, indexing request validators

Test Results

• 216 tests passing, 0 failures
• Coverage: 82% → 90%
• QA: 28/28 integration tests, 8/8 acceptance criteria verified
• SonarQube: 0 new issues
• Trivy: 0 critical/high findings
• Linter: ruff check + format clean

DO NOT MERGE

Per user request, this PR should not be merged yet.

Jira

BRIC-10

[Kaiohz]

📊 Code Confidence Score: 8/10

✅ Points positifs

• Architecture hexagonale respectée : nouveau port StoragePort.list_folders() + PostgresHealthPort, use cases séparés (ListFoldersUseCase, LivenessCheckUseCase), adapters MinIO et asyncpg
• Séparation MCP propre : mcp_query_tools.py (Query) / mcp_file_tools.py (File) — chaque serveur MCP a sa responsabilité
• ~1500 lignes de tests (9 fichiers) : mcp_query (379l), minio (321l), lightrag multimodal (252l), health liveness (205l), mcp_file (177l), config (106l), list_folders (34l), file_routes (76l), requests indexing (65l)
• Liveness probe GET /health/live checke Postgres + MinIO
• MCP_TRANSPORT supprimé — streamable HTTP uniquement, simplifie la config
• tesseract-ocr-fra ajouté au Dockerfile

⚠️ Points d'attention

• kreuzberg_adapter.py : ExtractionConfig en dur au niveau module (_KREUZBERG_CONFIG) — pas injectable, difficile à tester avec d'autres configs
• Pas de validation sur prefix dans list_folders — énumération de buckets possible si utilisateur malveillant
• .DS_Store dans le diff — à retirer du commit

🔴 Risques identifiés

• Aucun test d'intégration pour les endpoints MCP montés sur FastAPI (les tests unitaires mockent les use cases, mais personne ne teste que les routes MCP sont bien montées et répondent)

Recommandation

Merger après avoir retiré le .DS_Store et idéalement ajouté une validation sur le paramètre prefix.

---

Review automatique par SoluBot 🤖

[Kaiohz]

🟢 Code Review — BRIC-10: Split MCP into Query & File servers

✅ Points positifs

• Architecture hexagonale respectée : mcp_query_tools.py + mcp_file_tools.py bien séparés
• ~85 nouveaux tests, couverture 82% → 90%
• Liveness probe /health/live vérifie Postgres + MinIO
• tesseract-ocr-fra dans Dockerfile pour OCR français
• Validation path traversal sur read_file
• Error handling propre : 404/422
• Suppression du transport stdio (streamable HTTP only) = simplification
• ListFoldersUseCase ajouté pour les prefixes de buckets

⚠️ Points d'attention

1. kreuzberg_adapter.py : ExtractionConfig hardcodé au niveau module (_KREUZBERG_CONFIG) — pas injectable, difficile à tester avec d'autres configs
2. Pas de validation prefix dans list_folders — risque d'énumération si un utilisateur malveillant envoie des prefixes wildcards

🔴 Risques

• Pas de tests d'intégration pour les nouveaux endpoints MCP montés sur FastAPI

Score de confiance : 8/10