app/services/crawler.py:29, 105, 149, 197 и связанные app/services/dataset_versions.py, app/models.py, migrations/004_dataset_versions.sql оставляют MCP-специфичный контур: crawler продолжает строить и сохранять dataset_versions при каждом crawl, хотя после удаления app/mcp.py единственных потребителей service_info/sync больше нет. Для заявленного полного удаления MCP нужно убрать этот runtime-путь и связанные модели/миграцию/тесты (либо обосновать их независимое REST-потребление).
Non-blocking:
README.md:119 всё ещё сообщает версию 0.7.6, тогда как PR поднимает её до 0.7.7; поправить документацию.
tests/test_api.py покрывает health, несколько REST-ответов и refresh админки, но не проверяет, что /mcp отсутствует; добавьте небольшой regression assertion (404) для явно заявленного удаления endpoint.
Проверки: git diff --check и compileall проходят; pytest не удалось запустить в окружении ревьюера — отсутствуют pytest и зависимости FastAPI. REST/admin router подключены без изменений по diff.
Независимое ревью PR #35 (diff origin/main...origin/fix/remove-mcp).
Blocking:
- app/services/crawler.py:29, 105, 149, 197 и связанные app/services/dataset_versions.py, app/models.py, migrations/004_dataset_versions.sql оставляют MCP-специфичный контур: crawler продолжает строить и сохранять dataset_versions при каждом crawl, хотя после удаления app/mcp.py единственных потребителей service_info/sync больше нет. Для заявленного полного удаления MCP нужно убрать этот runtime-путь и связанные модели/миграцию/тесты (либо обосновать их независимое REST-потребление).
Non-blocking:
- README.md:119 всё ещё сообщает версию 0.7.6, тогда как PR поднимает её до 0.7.7; поправить документацию.
- tests/test_api.py покрывает health, несколько REST-ответов и refresh админки, но не проверяет, что /mcp отсутствует; добавьте небольшой regression assertion (404) для явно заявленного удаления endpoint.
Проверки: git diff --check и compileall проходят; pytest не удалось запустить в окружении ревьюера — отсутствуют pytest и зависимости FastAPI. REST/admin router подключены без изменений по diff.
предыдущий finding устранён: runtime-остатки dataset_versions удалены из моделей, crawler и сервисов; migrations/004_dataset_versions.sql остаётся только исторической миграцией и приложением не импортируется/не выполняется;
README содержит версию 0.7.7, pyproject.toml и app/version.py согласованы;
добавлена проверка отсутствия /mcp (404);
/mcp больше не регистрируется, REST API и админские маршруты в diff не затронуты;
Non-blocking: полный pytest не запущен — в окружении отсутствует команда pytest. По diff дополнительных корректностных, регрессионных, security или архитектурных проблем не найдено.
Review complete without blocking findings.
Повторное независимое ревью PR #35 после `e142138`.
Blocking findings: нет.
Проверено:
- предыдущий finding устранён: runtime-остатки `dataset_versions` удалены из моделей, crawler и сервисов; `migrations/004_dataset_versions.sql` остаётся только исторической миграцией и приложением не импортируется/не выполняется;
- README содержит версию `0.7.7`, `pyproject.toml` и `app/version.py` согласованы;
- добавлена проверка отсутствия `/mcp` (`404`);
- `/mcp` больше не регистрируется, REST API и админские маршруты в diff не затронуты;
- `git diff --check` и `python3 -m compileall -q app tests` проходят.
Non-blocking: полный pytest не запущен — в окружении отсутствует команда `pytest`. По diff дополнительных корректностных, регрессионных, security или архитектурных проблем не найдено.
Review complete without blocking findings.
admin
merged commit cf810f9cad into main2026-08-27 14:12:38 +00:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Удалена неиспользуемая MCP-интеграция: endpoint, router, документация и тесты. REST API и админка сохранены.
Независимое ревью PR #35 (diff origin/main...origin/fix/remove-mcp).
Blocking:
Non-blocking:
Проверки: git diff --check и compileall проходят; pytest не удалось запустить в окружении ревьюера — отсутствуют pytest и зависимости FastAPI. REST/admin router подключены без изменений по diff.
Повторное независимое ревью PR #35 после
e142138.Blocking findings: нет.
Проверено:
dataset_versionsудалены из моделей, crawler и сервисов;migrations/004_dataset_versions.sqlостаётся только исторической миграцией и приложением не импортируется/не выполняется;0.7.7,pyproject.tomlиapp/version.pyсогласованы;/mcp(404);/mcpбольше не регистрируется, REST API и админские маршруты в diff не затронуты;git diff --checkиpython3 -m compileall -q app testsпроходят.Non-blocking: полный pytest не запущен — в окружении отсутствует команда
pytest. По diff дополнительных корректностных, регрессионных, security или архитектурных проблем не найдено.Review complete without blocking findings.