3.9 KiB
Архитектурный аудит polygon v0.2.5 — ответ Claude Fable 5
Дата: 2026-07-31
Итоговая оценка
Архитектура соразмерна задаче и в целом здоровая. Blueprint'ы, data-driven конфиги, единый config/loader.py, синхронный run — правильный выбор. Код читаемый, докстринги честные, критичные контракты задокументированы.
Главный структурный риск: сервис задеплоен как публичный shared-эндпоинт, но спроектирован как эксклюзивный однопользовательский стенд.
Находки
🔴 1. Основной API полностью открыт на публичном домене
MOCK_AUTH_TOKEN защищает только /_mock/*. POST /instances, /instanceOperations, run — без проверки.
🔴 2. sleep(DELAY) блокирует /health → риск рестарта контейнера
1 sync-воркер, sleep в run замораживает весь сервер включая /health. Комбинация delay=30 + несколько run → healthcheck не отвечает → Nubes рестартит под → состояние потеряно.
🔴 3. Гарантия «1 воркер» через setdefault — не работает
os.environ.setdefault("WEB_CONCURRENCY", "1") выполняется при импорте app, а gunicorn читает WEB_CONCURRENCY при старте мастера — до импорта. Если платформа выставит WEB_CONCURRENCY=4, будет 4 воркера и 4 несвязанных MockState.
🟡 4. 500-ки на невалидном вводе
set_param: int(param_id) без try → ValueError → 500. /_mock/delay: float("garbage") → 500. Отрицательные значения принимаются молча.
🟡 5. _extract_subresource_name — хрупкий
Берёт первое непустое строковое значение. Порядок = порядок set_param. Если executor отправит пароль раньше имени — subresource назовётся паролем.
🟡 6. Мок слишком «добрый»
- validate-cfs всегда 200 (required не проверяются)
- modify принимает isModifiable: false
- операция на creating-инстансе (modify до завершения create)
- svcOperationId не сверяется с serviceId инстанса
- isSuccessful всегда True — негативные сценарии нельзя тестировать
🟢 7. Мелочи
- operations/op_params не чистятся (кроме reset)
- stages: [] всегда пуст
- total в пагинации — проверить с реальным API
Ответы на вопросы
Q1. In-memory vs Redis. Redis не нужен. Потеря состояния при рестарте — фича.
Q2. Параллельные тесты. (A) polygon per-session (рекомендовано) или (B) namespace-изоляция.
Q3. Генерация YAML при старте. Нет. Оффлайн + drift-check тест.
Q4. Мониторинг. Prometheus избыточен. Достаточно логов + /_mock/state.
Q5. Готовность к CI. К последовательному — готов. Блокеры: auth API (1), cap DELAY (2), симуляция ошибок (6).
Приоритеты
- Токен на весь API + cap DELAY — дёшево, закрывает оба 🔴
- /_mock/fail-next — открывает негативные тесты
- Runtime-проверка воркеров + фикс int(param_id)
- Модель владения (per-session) — отложить