Prompt for Opus — Code Review v1.2.0

Проект: Nubes Autotest (Flask + vanilla JS + PostgreSQL) План: /home/naeel/nubes/autotest/DOCS/opus-plan-2026-07-31.md Вопросы/ответы: /home/naeel/nubes/autotest/DOCS/opus-questions-2026-07-31.md

Что сделано

Реализованы Фазы 1-3 + начало Фазы 4. Модальный редактор сценариев НЕ сделан.

Изменённые/новые файлы (для ревью)

Новые файлы:

1. /home/naeel/nubes/autotest/app-autotest/site/api/utils.py

find_uid(resp) — поиск UUID по верхнему уровню + вложенным dict
uid_from_location(loc) — UUID из Location-заголовка
Заменяет 2 разные реализации из api_test.py + scenario.py

2. /home/naeel/nubes/autotest/app-autotest/site/operations/executor.py

execute_operation(client, service_id, operation, instance_uid, params, svc_op_id=None, display_name=None)
Делает всё до /run: POST /instances → POST /instanceOperations → send_params_terraform → /run
Ветвится по operation=="create" (свой payload, без svcOperationId)
Возвращает dict {ok, error, failed_step, instance_uid, op_uid, display_name}
НЕ поллит

3. /home/naeel/nubes/autotest/app-autotest/site/operations/poll.py

poll_until_done(client, op_uid, timeout=1800) → {status, is_successful, error_log, stages, duration, svc}
Общий цикл для async (_finish_op) и sync (run_scenario)

4. /home/naeel/nubes/autotest/app-autotest/site/static/js/params-render.js

renderParamRow(p, allInst) — рендер параметра
renderMapFixedRow(p, dfl) — рендер map-fixed с вложенными полями
collectParams(containerSelector) — сбор параметров из DOM
Вынесено из operations.js, используется обоими (ручной + сценарный)

Изменённые файлы:

5. /home/naeel/nubes/autotest/app-autotest/site/operations/scenario.py

Удалены _find_uid, _uid_from_location
Добавлен import из executor, poll
_resolve_instance_uid(step, bindings, instance_map) — приоритет: uid > ref > service_id
_update_bindings(scenario_run_id, bindings) — персист в scenario_runs.instance_bindings
run_scenario — create/non-create через executor, поллинг через poll_until_done, output→bindings

6. /home/naeel/nubes/autotest/app-autotest/site/routes/api_test.py

Удалены _find_uid, _uid_from_location (импорт из api.utils)
CREATE и non-CREATE флоу заменены на execute_operation
_finish_op заменён на poll_until_done
CMDB delete оставлен как предпроверка (не через executor)

7. /home/naeel/nubes/autotest/app-autotest/site/routes/api_scenario_defs.py

_validate_steps расширен: output (уникальность, только create), instance_ref (ссылка на output), instance_uid
Старый формат (без новых ключей) проходит валидацию (обратная совместимость)

8. /home/naeel/nubes/autotest/app-autotest/site/db/init_db.py

Startup cleanup: UPDATE scenario_runs SET status='TIMEOUT' WHERE status='RUNNING' AND created_at < NOW() - INTERVAL '1 hour'

9. /home/naeel/nubes/autotest/app-autotest/site/static/js/operations.js

Удалены renderParamRow, renderMapFixedRow, collectParams (теперь в params-render.js)

10. /home/naeel/nubes/autotest/app-autotest/site/templates/index.html

Добавлен <script src="js/params-render.js"> перед instances.js

11. /home/naeel/nubes/autotest/app-autotest/site/app.py

VERSION = "1.2.0"

Вопросы для ревью

executor.py: корректно ли обрабатываются все ошибки? Не потерялся ли tracker_add (в старом коде был ДО params/run, в executor его нет — он вызывается в api_test.py после executor)?
scenario.py: правильно ли резолвится instance_uid? Не сломается ли старый dummy_test без output/instance_ref?
api_test.py: CMDB delete остался как предпроверка — ок ли это?
init_db.py: startup cleanup — не затрёт ли легитимные долгоиграющие сценарии?
params-render.js: collectParams теперь с параметром containerSelector — не сломает ли это существующие вызовы без аргумента?
Общая архитектура: все ли зависимости корректны? Нет ли циклических импортов?

4.9 KiB Raw Blame History