diff --git a/DOCS/sonnet-full-review.md b/DOCS/sonnet-full-review.md new file mode 100644 index 0000000..9286a8c --- /dev/null +++ b/DOCS/sonnet-full-review.md @@ -0,0 +1,51 @@ +# Полный код-ревью ВСЕГО проекта autotest + +## Контекст + +Flask 3.0 + gunicorn (multi-worker) + vanilla JS + psycopg2 PostgreSQL. +Тестирует Nubes REST API — создаёт/удаляет инстансы облачных сервисов. +Версия: v1.1.20 (сегодня, после 20 правок за день). +За день сделано ~20 правок, накопился технический долг — нужен свежий взгляд. + +## ВСЕ файлы (прочитай КАЖДЫЙ) + +``` +/home/naeel/nubes/autotest/app-autotest/site/app.py — точка входа, регистрация blueprint'ов +/home/naeel/nubes/autotest/app-autotest/site/runner.py — загрузка config.yaml +/home/naeel/nubes/autotest/app-autotest/site/api/auth.py — get_token, get_client, get_client_id, get_stand +/home/naeel/nubes/autotest/app-autotest/site/api/http_client.py — HttpClient, detect_endpoint, stand_name +/home/naeel/nubes/autotest/app-autotest/site/db/pool.py — psycopg2 ThreadedConnectionPool (lazy) +/home/naeel/nubes/autotest/app-autotest/site/db/init_db.py — CREATE TABLE runs + миграции +/home/naeel/nubes/autotest/app-autotest/site/db/save_run.py — INSERT в runs (16 колонок) +/home/naeel/nubes/autotest/app-autotest/site/operations/get_instances.py — GET /instances (пагинация) +/home/naeel/nubes/autotest/app-autotest/site/operations/get_params.py — get_params_with_current_values, _normalize_value_list +/home/naeel/nubes/autotest/app-autotest/site/operations/get_services.py — GET /services, /services/{id} +/home/naeel/nubes/autotest/app-autotest/site/operations/service_list.py — load_service_ids из services_{stand}.txt +/home/naeel/nubes/autotest/app-autotest/site/operations/tracker.py — fcntl.flock file tracker +/home/naeel/nubes/autotest/app-autotest/site/routes/api.py — LEGACY эндпоинты +/home/naeel/nubes/autotest/app-autotest/site/routes/api_test.py — ОСНОВНОЙ: /api/test, /api/params, /api/log, поллинг, _finish_op +/home/naeel/nubes/autotest/app-autotest/site/routes/main.py — / (главная), /api/operations/{svc_id} +/home/naeel/nubes/autotest/app-autotest/site/static/app.js — ВЕСЬ фронтенд (420 строк) +/home/naeel/nubes/autotest/app-autotest/site/templates/index.html — Jinja2 шаблон +``` + +## Что нужно от тебя + +1. **Найти ВСЕ баги** — синтаксические, логические, race conditions, необработанные исключения +2. **Проверить универсальность** — работает ли код для ВСЕХ сервисов (Redis 3 параметра, PostgreSQL 8 map-fixed параметров, Болванка) +3. **Проверить CREATE flow** — сверь с эталонным Terraform flow из /home/naeel/nubes/autotest/DOCS/terraform-operations-full-logic.md +4. **Проверить MODIFY/SUSPEND/DELETE/RESUME** — нет ли там таких же проблем +5. **Проверить фронтенд** — все ли DOM ID совпадают, нет ли необработанных Promise, правильный ли порядок элементов +6. **Проверить на дубликаты** — нет ли мёртвого кода, дублирующихся эндпоинтов +7. **Проверить безопасность** — httponly cookie, токены, CORS + +## Известные недавние баги (уже исправлены, но проверь что фиксы корректны) + +- `_client()` → NameError (не было такой функции) → заменили на get_client() +- `-строка` → TypeError в сортировке → заменили на два sort() +- `_normalize_value_list` не импортирован → добавили импорт +- `\`;` осиротевший HTML в JS → удалили +- `currentSvcName` вместо реального `i.svc` → исправили +- serviceId фильтр (убран, заменён на сортировку) +- dataDescriptor defaultValue не прокидывался → добавили +- required+default params не досылались (шаг 6 Terraform) → добавили