docs: add comprehensive code review report

[nedeadinside]

This PR adds a comprehensive code review report in REVIEW.md based on a deep analysis of the RAG service codebase.
The review identifies several critical and high-priority issues, such as deprecated LangChain API usage, incorrect UUIDv5 generation, and missing failure rollback mechanisms for vector store ingestion.
The report is formatted according to the requested guidelines and grouped by severity, providing specific file paths, problem descriptions, impact assessments, and actionable recommendations.
No source code or existing files were modified in this PR.

---

PR created automatically by Jules for task 11874559675838300408 started by @nedeadinside

[google-labs-jules]

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.

---

For security, I will only act on instructions from the user who triggered this task.

[Copilot]

Pull request overview

This PR adds a standalone REVIEW.md document containing a severity-ranked code review report for the RAG service, intended to catalog high-impact issues and recommended fixes without changing source code.

Changes:

• Added REVIEW.md with a structured code review report (summary, prioritized issues, and detailed findings).
• Included file/line references and suggested remediation steps for each finding.

Comments suppressed due to low confidence (7)

REVIEW.md:45

• Диапазон строк для проблемы с отсутствием rollback в Pipeline.ingest указан некорректно. Вызов self._qdrant.upsert(...) находится ближе к концу метода (строки 59–86), поэтому текущая ссылка :54-71 не покрывает фактическое место записи в Qdrant.

- Файл:строка: `rag/src/pipeline.py:54-71`

REVIEW.md:53

• Неверно указан номер строки в rag/src/models/config.py для yaml_file="/app/config.yaml". В текущем файле это находится в AppConfig.model_config на 124-й строке, а не на 75-й.

- Файл:строка: `rag/src/models/config.py:75`

REVIEW.md:59

• В ссылке на DoclingClient указан неверный номер строки для обработки httpx.HTTPStatusError: блок except httpx.HTTPStatusError as e: находится на 52-й строке rag/src/clients/docling.py, поэтому :48 не соответствует текущему коду.

- Файл:строка: `rag/src/clients/embedder.py:44`, `rag/src/clients/docling.py:48`

REVIEW.md:65

• Неверно указан номер строки в rag/src/worker/tasks.py для участка, где в webhook отправляется error: str(e). Сейчас это происходит в блоке except на 90-й строке, а не на 78-й.

- Файл:строка: `rag/src/worker/tasks.py:78`

REVIEW.md:71

• Неверно указан номер строки в rag/src/clients/docling.py для чтения файла целиком: content = await asyncio.to_thread(path.read_bytes) находится на 41-й строке, а не на 38-й (38-я — про заголовок API key).

- Файл:строка: `rag/src/clients/docling.py:38`

REVIEW.md:79

• Неверно указан номер строки в rag/src/clients/embedder.py для выражения с конкатенацией эмбеддингов: list comprehension с flatten находится на 74-й строке, а не на 72-й.

- Файл:строка: `rag/src/clients/embedder.py:72`

REVIEW.md:85

• Указанный путь rag/config.yaml не существует в репозитории — конфигурация лежит в корневом config.yaml. Кроме того, redis_url с redis://rag-redis:6379/0 сейчас находится на 5-й строке этого файла.

- Файл:строка: `rag/config.yaml:7` (или дефолтное значение)

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.