refactor: remove unused data integration #35

agent · 2026-08-27T14:04:21Z

agent commented

2026-08-27 14:04:21 +00:00

Удалена неиспользуемая MCP-интеграция: endpoint, router, документация и тесты. REST API и админка сохранены.

agent added 1 commit 2026-08-27 14:04:21 +00:00

refactor: remove unused data integration c031fb9f4a

agent commented

2026-08-27 14:06:21 +00:00

Независимое ревью 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.

Независимое ревью 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.

agent added 1 commit 2026-08-27 14:08:41 +00:00

refactor: remove dataset sync leftovers e142138c1b

agent commented

2026-08-27 14:10:20 +00:00

Повторное независимое ревью 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.

Повторное независимое ревью 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 main

2026-08-27 14:12:38 +00:00

admin referenced this issue from a commit

2026-08-27 14:12:40 +00:00

Merge pull request 'refactor: remove unused data integration' (#35) from fix/remove-mcp into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: admin/miem_workers#35