fix: run only API in compose #33

agent · 2026-08-26T15:12:01Z

agent commented

2026-08-26 15:12:01 +00:00

Из docker-compose.yml удалены отдельные сервисы worker и mcp; в production запускаются api и зависимый postgres.

agent added 1 commit 2026-08-26 15:12:02 +00:00

fix: run only API in compose b028d81e9d

agent commented

2026-08-26 15:13:26 +00:00

Независимое ревью: есть blocking findings.

MCP_DESCRIPTION.md:11,625+ всё ещё описывает отдельный compose-сервис mcp и localhost:8001/mcp; после удаления сервиса это неверная инструкция. Обновите документ на endpoint API (http://localhost:8000/mcp) либо явно отметьте, что MCP в production не разворачивается.
Версия повышена до 0.7.5 в pyproject.toml и app/version.py, но в CHANGELOG.md отсутствует раздел 0.7.5. Добавьте пользовательски значимое изменение текущего PR.
Для существующего production-развёртывания docker compose up -d --build оставит прежние worker/mcp контейнеры как orphan. В инструкции обновления нужен docker compose up -d --build --remove-orphans (либо равнозначный явно описанный шаг), иначе цель «только api и postgres» не будет достигнута после обновления.

Проверены diff и окружение: в новом docker-compose.yml действительно остаются только postgres и api; версии согласованы. git diff --check проходит. compileall уже пройден автором; Docker локально недоступен, поэтому запуск Compose не проверялся.

Независимое ревью: есть blocking findings. 1. `MCP_DESCRIPTION.md:11,625+` всё ещё описывает отдельный compose-сервис `mcp` и `localhost:8001/mcp`; после удаления сервиса это неверная инструкция. Обновите документ на endpoint API (`http://localhost:8000/mcp`) либо явно отметьте, что MCP в production не разворачивается. 2. Версия повышена до `0.7.5` в `pyproject.toml` и `app/version.py`, но в `CHANGELOG.md` отсутствует раздел `0.7.5`. Добавьте пользовательски значимое изменение текущего PR. 3. Для существующего production-развёртывания `docker compose up -d --build` оставит прежние `worker`/`mcp` контейнеры как orphan. В инструкции обновления нужен `docker compose up -d --build --remove-orphans` (либо равнозначный явно описанный шаг), иначе цель «только api и postgres» не будет достигнута после обновления. Проверены diff и окружение: в новом `docker-compose.yml` действительно остаются только `postgres` и `api`; версии согласованы. `git diff --check` проходит. `compileall` уже пройден автором; Docker локально недоступен, поэтому запуск Compose не проверялся.

agent added 1 commit 2026-08-26 15:14:20 +00:00

docs: describe API-only compose deployment 3c2057dcec

agent commented

2026-08-26 15:14:53 +00:00

Повторное независимое ревью после 3c2057d: blocking findings отсутствуют.

Все замечания #815 устранены: MCP_DESCRIPTION.md использует endpoint API http://localhost:8000/mcp; в CHANGELOG.md добавлен раздел 0.7.5; README указывает docker compose up -d --build --remove-orphans, который удалит прежние worker/mcp-контейнеры.

Проверен полный актуальный diff: Compose содержит только сервисы postgres и api, MCP_PORT и ссылки на localhost:8001 отсутствуют, версии pyproject.toml и app/version.py согласованы. git diff --check проходит. Docker в окружении недоступен, поэтому Compose фактически не запускался; compileall был успешно выполнен до этого ревью.

Повторное независимое ревью после `3c2057d`: blocking findings отсутствуют. Все замечания #815 устранены: `MCP_DESCRIPTION.md` использует endpoint API `http://localhost:8000/mcp`; в `CHANGELOG.md` добавлен раздел `0.7.5`; README указывает `docker compose up -d --build --remove-orphans`, который удалит прежние worker/mcp-контейнеры. Проверен полный актуальный diff: Compose содержит только сервисы `postgres` и `api`, `MCP_PORT` и ссылки на `localhost:8001` отсутствуют, версии `pyproject.toml` и `app/version.py` согласованы. `git diff --check` проходит. Docker в окружении недоступен, поэтому Compose фактически не запускался; `compileall` был успешно выполнен до этого ревью.

admin merged commit 3ed1dd8832 into main

2026-08-26 15:16:34 +00:00

admin referenced this issue from a commit

2026-08-26 15:16:36 +00:00

Merge pull request 'fix: run only API in compose' (#33) from fix/api-only-compose 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#33