feat: /help (RAG+MCP) и AI-ревью PR (/review + GitHub Action)

[Sermage]

Вносит в master две фичи поверх уже влитого ollama-провайдера:

• /help <вопрос> — ответ о проекте по документации (RAG) + git-ветка через MCP (PR feat(help): /help <вопрос> — RAG по докам + git-ветка через MCP #12).
• /review + пайплайн AI-ревью PR — diff через GitHub, RAG по докам и коду, разбор баги/архитектура/рекомендации; review_pr.py + .github/workflows/ai-review.yml (self-hosted runner) (PR feat(review): AI-ревью PR — diff через GitHub + RAG (доки+код) #13).

Оба уже прошли ревью в отдельных PR (#12, #13). pytest — 492 passed. Конфликтов с master нет.

🤖 Generated with Claude Code

https://claude.ai/code/session_01WPCiJi7DVBdV2fjN2x3HMd

[github-actions]

🤖 AI-ревью

🐞 Потенциальные баги

1. app/pr_review.py:89-90 — Параметр changed_files аннотирован как необязательный (list[str]), но в строке 90 переопределяется через list(changed_files or []). Если передан None, то операция or [] сработает, но если передан пустой список [] (который falsy), то он так и останется пустым. Лучше использовать changed_files = changed_files or [] — это не баг, но может сбить с толку.

2. app/pr_review.py:99-100 — Проверка if not diff.strip() and not changed_files: использует changed_files, который уже переопределён выше как список. Если changed_files пуст, то второй элемент and будет False, что корректно. Однако если diff пуст, но changed_files содержит файлы, то ревью запустится с пустым diff, а RAG-запрос будет сформирован только из списка файлов. Это может привести к генерации ревью без анализа изменений.

3. app/pr_review.py:113 — Атрибут text в объекте ReviewResult может содержать пустую строку, если client.chat() вернул пустой ответ. В таком случае _EMPTY_REVIEW не будет использован, но результат будет формально пустым.

4. .github/workflows/ai-review.yml:38-39 — Путь RAG_INDEX_PATH: ~/.jarvis/rag/docs_index и CODE_INDEX_PATH: ~/rag-kotlin/index_jarvis_full указываются с тильдой ~. В GitHub Actions через env: тильда может не раскрываться корректно, если действие не запускает оболочку. Лучше использовать абсолютные пути или $HOME/....

🏛 Архитектурные проблемы

1. app/pr_review.py:31-36 — Системный промпт для ревью дублирует требования из задачи, но жёстко зашит в коде use case. Это нарушает принцип единственной ответственности: промпт-инжиниринг должен быть отдельной сущностью или параметром, чтобы его можно было менять без пересборки приложения.

2. app/project_help.py:9-10 — Порт GitContextProvider определён в app/ports.py, но не используется в cli/main.py и не внедряется никуда, кроме answer_project_question. Отсутствует связь с существующей системой команд /help — в diff не видно интеграции.

3. app/pr_review.py и app/project_help.py — Оба use case содержат дублирование: функции _build_context() есть в обоих файлах с почти одинаковой логикой (разница только в git-ветке у project_help.py). Следует вынести в общий модуль app/_context.py или подобный.

4. app/pr_review.py:9-12 — Комментарий упоминает, что use case тестируется на фейках, но в тестовом файле (tests/app/test_pr_review.py) нет проверки взаимодействия с RetrievalEngine через фейки — только базовый проверки. Отсутствует тест на случай, когда engine.is_ready() вернул False.

💡 Рекомендации

1. app/pr_review.py:89-90 — Заменить строку на явную проверку if changed_files is None: changed_files = [] для ясности намерений.

2. app/pr_review.py:100 — Добавить проверку if not diff.strip(): return ReviewResult(text=_EMPTY_REVIEW, files=changed_files, used_context=False) для сценария пустого diff с непустым списком файлов.

3. app/pr_review.py:113 — Добавить пост-проверку: if not text.strip(): return ReviewResult(text=_EMPTY_REVIEW, ...).

4. .github/workflows/ai-review.yml:38-39 — Заменить ~ на $HOME в путях индексов для гарантированной корректности в CI-окружении.

5. app/_context.py (новый файл) — Вынести общие функции _build_context и _added_lines из app/pr_review.py и app/project_help.py в один модуль, чтобы избежать дублирования.

---

Контекст (RAG): cli/main.py · cli/views.py — ── help ──────────────────────────────────────────────────────────────────── · domain/retrieval.py — llm — cross-encoder на LLM: модель оценивает релевантность каждого чанка. · app/retrieval_pipeline.py

Сгенерировано jarvis-cli · review_pr.py