diff --git a/DOCS/sonnet-review-round2.md b/DOCS/sonnet-review-round2.md index 078bfa5..030a04d 100644 --- a/DOCS/sonnet-review-round2.md +++ b/DOCS/sonnet-review-round2.md @@ -7,7 +7,7 @@ ### 🔴 Баг 1: Инстансы всех сервисов показываются под любым выбранным -**Файл:** `site/routes/main.py`, строка 193-194, функция `api_operations(svc_id)` +**Файл:** `/home/naeel/nubes/autotest/app-autotest/site/routes/main.py`, строка 193-194, функция `api_operations(svc_id)` ```python for i in instances: @@ -30,9 +30,9 @@ for i in instances: **Данные:** API `/services/{id}` возвращает `svcShort=redis` **План:** -1. `main.py` `/api/operations/{svc_id}` — добавить `svcShort` в JSON-ответ (из `get_service_detail`) -2. `app.js` `selectService()` — сохранить `currentSvcShort = d.svcShort` -3. `app.js` `makeCreateDisplayName()` — `autotest-${currentSvcShort}-${3rand}` +1. `/home/naeel/nubes/autotest/app-autotest/site/routes/main.py` `/api/operations/{svc_id}` — добавить `svcShort` в JSON-ответ +2. `/home/naeel/nubes/autotest/app-autotest/site/static/app.js` `selectService()` — сохранить `currentSvcShort = d.svcShort` +3. `/home/naeel/nubes/autotest/app-autotest/site/static/app.js` `makeCreateDisplayName()` — `autotest-${currentSvcShort}-${3rand}` 4. Коллизии обрабатываются в `_unique_display_name()` на бэкенде ### 🟡 Задача 3: description (descr) при создании @@ -43,14 +43,14 @@ POST /instances {"serviceId":1, "displayName":"dummy-11255555", "descr":""} ``` -В autotest `api_test.py` строка 212 уже шлёт `"descr": ""` — пустое. Нужно заполнить: +В autotest `/home/naeel/nubes/autotest/app-autotest/site/routes/api_test.py` строка 212 уже шлёт `"descr": ""` — пустое. Нужно заполнить: ``` "descr": "created by autotest v1.1.13" ``` **План:** -1. `app.js` `executeOp()` — передать `descr` в тело запроса -2. `api_test.py` `/api/test` — принять `descr` из JSON, подставить в payload +1. `/home/naeel/nubes/autotest/app-autotest/site/static/app.js` `executeOp()` — передать `descr` в тело запроса +2. `/home/naeel/nubes/autotest/app-autotest/site/routes/api_test.py` `/api/test` — принять `descr` из JSON, подставить в payload ## Вопросы к Sonnet @@ -58,9 +58,9 @@ POST /instances 2. Для задачи 2: как лучше генерировать 3 случайных символа? `Math.random().toString(36).slice(2,5)`? Не будет ли коллизий? 3. Для задачи 3: `descr` уже есть в payload. Достаточно ли просто подставить строку, или нужно что-то ещё? 4. Есть ли ещё какие-то баги, которые мы упустили? -5. `api_test.py` содержит ДУБЛИКАТ эндпоинта `/api/operations/{svc_id}` (строка 108-148), который никогда не вызывается из-за порядка регистрации blueprint'ов. Удалить ли его, или он нужен для чего-то? +5. `/home/naeel/nubes/autotest/app-autotest/site/routes/api_test.py` содержит ДУБЛИКАТ эндпоинта `/api/operations/{svc_id}` (строка 108-148), который никогда не вызывается из-за порядка регистрации blueprint'ов. Удалить ли его, или он нужен для чего-то? **Файлы для проверки (только эти):** -- `site/routes/main.py` (строки 155-230) -- `site/routes/api_test.py` (строки 108-148, 195-280) -- `site/static/app.js` (строки 85-130, 240-270) +- `/home/naeel/nubes/autotest/app-autotest/site/routes/main.py` (строки 155-230) +- `/home/naeel/nubes/autotest/app-autotest/site/routes/api_test.py` (строки 108-148, 195-280) +- `/home/naeel/nubes/autotest/app-autotest/site/static/app.js` (строки 85-130, 240-270)