docs(opus): раунд 3 — фокус переведён на сам провайдер (устойчивость/корректность)
Жёсткий бюджет ради токенов: ≤10 файлов, ≤5 находок по ≤3 строки, плюс одна строка «что проверяемо только замером». M6 снят, модификаторы — фон. Границы: ядро, resources_core, шаблоны/хелперы генератора. В журнал добавлены крит-мнение по раунду 2 и текст раунда 3.
This commit is contained in:
Repinoid
@@ -701,15 +701,109 @@ Read , lines 174 to 193
|
|||||||
|
|
||||||
---
|
---
|
||||||
|
|
||||||
|
## Ход 11 — Запрос пользователя (дословно)
|
||||||
|
|
||||||
|
```
|
||||||
|
так дополни задание ему, пусть ещё анализирует НО не нало много - токены могути закончиться
|
||||||
|
```
|
||||||
|
|
||||||
|
### Критическое мнение агента о раунде 2 (дословно, отправлено пользователю)
|
||||||
|
|
||||||
|
```
|
||||||
|
**Что стало лучше**
|
||||||
|
- **M1** — Opus честно перепроверил и исправился: `provider.go:182/186/187`. Совпадает с моим чтением → доверие к остальным номерам выросло.
|
||||||
|
- **M2** — догадка устранена: приведена конкретная строка скрипта, область (только `TOOLS/`) и паттерн (`.ServiceID==N`/`.ID==N`). Это уже факт, и факт полезный: дерево `provider/` стражем не покрыто **вообще**.
|
||||||
|
- **M3** — `S4` снят с текстуальной опорой (`ARCHITECTURE.md:134` в разделе про generated-код), `S2` переклассифицирован в пробел спеки с опорой (`:12`, `:110` vs `:13`, `:213`). Честно и корректно.
|
||||||
|
- **U2** — новая ценная находка, которой у меня не было: **третий** словарь жизненного цикла — `delete_strategy` (compile-time в YAML) против runtime-флагов. Настоящий архитектурный дефект.
|
||||||
|
- **M4** — шкала задана, `R5` понижен. Замечание принято по существу.
|
||||||
|
|
||||||
|
**Что остаётся слабым**
|
||||||
|
- **M5 — это чтение кода, а не замер.** Вся защита от вечного diff держится на утверждении «в `Read` смысловое сравнение, и при совпадении state не перезаписывается». Это правдоподобно и проверяемо, но **не проверено прогоном** `terraform apply` ×2 / `plan`. По нашему же правилу «рекомендация без замера — догадка». Opus не предложил верификацию.
|
||||||
|
- **Риск переоценён/недооценён.** Самое опасное (R1: `Update` инстанс-ресурса затирает `ipSpaceName`) осталось **нерешённым** — за M6. При этом Opus сам пишет, что для `vc_nsxt` modify содержит 5 параметров, и запись `params[372]` присутствует **всегда**. Если так — это не «война дрейфов», а **молчаливое затирание SNAT** при любой правке эджа. Это надо было ставить вопросом №1 прошлого раунда, а не откладывать.
|
||||||
|
- **Ничего про сам провайдер.** Раунд целиком про модификаторы — ровно то, что вы просили не переоценивать. Ядро, генератор, устойчивость, корректность — не тронуто.
|
||||||
|
|
||||||
|
**Моё мнение**
|
||||||
|
Раунд 2 — честный и в плюс (правки по номерам, снятие `S4`, находка `U2`). Но по вашему приоритету он **мимо цели**: модификаторы, а не провайдер; и при этом главный вопрос (затирание `ipSpaceName` при `Update` эджа + отсутствие замера) остался открытым. Логичный следующий ход — развернуть раунд на «провайдер целиком», а `M6` **отложить**, а не разворачивать.
|
||||||
|
```
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Ход 12 — Задание агента (раунд 3) — текст, отправляемый Opus
|
||||||
|
|
||||||
|
Сохранён отдельным файлом: `NOTES/20_prompts/prompt_for_opus_provider_core_round3.md`.
|
||||||
|
Содержимое (дословно):
|
||||||
|
|
||||||
|
```markdown
|
||||||
|
# Промпт для Opus 4.8 — РАУНД 3: сам провайдер (устойчивость и корректность)
|
||||||
|
|
||||||
|
> Продолжение диалога. Раунды 1–2 — `HISTORY/OPUS/2026-09-30_provider_architecture_modifiers_dialog.md`.
|
||||||
|
> Режим и формат ответа — как раньше: сжато, тезисы, `файл:строка`, без догадок.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Вводная (смена приоритета)
|
||||||
|
|
||||||
|
Модификаторы (`nubes_vc_org_ip_allocation`, `nubes_vc_nsxt_snat`) — новая, возможно сложная, но
|
||||||
|
**НЕБОЛЬШАЯ** часть сервиса. Чрезмерный упор на них не нужен.
|
||||||
|
|
||||||
|
**Главное — САМ провайдер: его устойчивость и корректность.**
|
||||||
|
|
||||||
|
- `M6` (доступ к двум файлам для доразбора R1) — **снять**; углубление по модификаторам больше не требуется.
|
||||||
|
- Прошлые находки (`S1`, `S2`, `R2`–`R6`, `U1`–`U3`) **не повторять**.
|
||||||
|
|
||||||
|
## Бюджет (жёстко — экономим токены)
|
||||||
|
|
||||||
|
- Прочитать **не более 10 файлов** суммарно. Ранее прочитанные повторно не открывать.
|
||||||
|
- Ответ — **не более 5 находок**, каждая **не более 3 строк**.
|
||||||
|
- Никаких вступлений, повторения прошлых отчётов, «а ещё могу».
|
||||||
|
|
||||||
|
## Что анализировать (провайдер целиком)
|
||||||
|
|
||||||
|
1. **Жизненный цикл инстанс-ресурса:** `create` / adopt / `suspend` / resume / `modify` / redeploy /
|
||||||
|
`delete` и повторный `apply` — где теряется корректность состояния.
|
||||||
|
2. **Досылка и нормализация параметров** (`WithDefaults`, zero-value fallback, дефолты `map-fixed`,
|
||||||
|
регистр UUID): где риск затереть значение или получить ложный diff.
|
||||||
|
3. **`Read` / refresh:** устойчив ли state у генерируемых ресурсов; где возможен вечный diff.
|
||||||
|
4. **Устойчивость ядра:** ретраи и ошибки, `LockInstance`, идемпотентность, поведение при дубликатах
|
||||||
|
и параллельных `apply`.
|
||||||
|
5. **Генератор → код:** какие классы дефектов порождает шаблон.
|
||||||
|
|
||||||
|
## Границы доступа
|
||||||
|
|
||||||
|
Разрешено читать (только это):
|
||||||
|
|
||||||
|
- `provider/internal/core/**`
|
||||||
|
- `provider/internal/resources_core/**`
|
||||||
|
- `provider/internal/provider/provider.go`
|
||||||
|
- `TOOLS/resource-generator/internal/templates/**`
|
||||||
|
- `TOOLS/resource-generator/internal/params/params.go`
|
||||||
|
- `TOOLS/resource-generator/internal/helpers/helpers.go`
|
||||||
|
- `TOOLS/resource-generator/internal/loader/loader.go`
|
||||||
|
- `TOOLS/resource-generator/internal/writers/writers.go`
|
||||||
|
|
||||||
|
Запрещено: `HISTORY/**`, `NOTES/**`, `docs/**`, `HAR/**`, `TMP/**`, `DEV_STAND/**`, `TEST_STAND/**`,
|
||||||
|
`PROD_STAND/**`, git-история. Нужен файл вне списка → вопрос мне, не чтение.
|
||||||
|
|
||||||
|
## Формат ответа
|
||||||
|
|
||||||
|
- **ТОП-5 находок** по убыванию критичности: суть → место (`файл:строка`) → последствие (≤ 3 строк).
|
||||||
|
- Затем **одна строка**: что из этого проверяемо только замером (прогон `apply`/`plan`), а не чтением.
|
||||||
|
- Находок меньше пяти — столько, сколько есть. Водой не добивать.
|
||||||
|
|
||||||
|
## Стоп-правило
|
||||||
|
|
||||||
|
Не хватает файла или данных — один короткий вопрос. Не догадываться.
|
||||||
|
```
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
## Статус
|
## Статус
|
||||||
|
|
||||||
- Раунд 1: отчёт получен. Раунд 2: отправлен и **отвечен** (M1–M6, U1–U3).
|
- Раунды 1–2: завершены (отчёты получены, замечания отработаны).
|
||||||
- **Открытый вопрос Opus (M6)**: просит разрешить чтение двух файлов —
|
- **Раунд 3 отправлен**: фокус переведён с модификаторов на **сам провайдер** (устойчивость, корректность);
|
||||||
реализация `RunInstanceOperationUniversalWithDefaults` (ядро) и хелпер `ParamFormat` (`funcs.go`).
|
жёсткий бюджет (≤ 10 файлов, ≤ 5 находок ≤ 3 строк). `M6` снят. Ответ ещё НЕ получен.
|
||||||
Решение по доступу НЕ принято.
|
- Артефакты: `752244f` — промпт раунда 1;
|
||||||
- **Новая вводная пользователя**: не делать чрезмерный упор на модификаторах; главное — сам провайдер
|
`NOTES/20_prompts/prompt_for_opus_provider_architecture_modifiers_round2.md` — замечания раунда 2;
|
||||||
(устойчивость, корректность). Учитывать при формировании следующих раундов.
|
`NOTES/20_prompts/prompt_for_opus_provider_core_round3.md` — задание раунда 3.
|
||||||
- Артефакты: коммит `752244f` — промпт раунда 1;
|
|
||||||
`NOTES/20_prompts/prompt_for_opus_provider_architecture_modifiers_round2.md` — замечания раунда 2.
|
|
||||||
- Настоящий документ ведётся по прямому указанию пользователя «документируй ВЕСЬ диалог,
|
- Настоящий документ ведётся по прямому указанию пользователя «документируй ВЕСЬ диалог,
|
||||||
без сокращений».
|
без сокращений».
|
||||||
|
|||||||
@@ -0,0 +1,59 @@
|
|||||||
|
# Промпт для Opus 4.8 — РАУНД 3: сам провайдер (устойчивость и корректность)
|
||||||
|
|
||||||
|
> Продолжение диалога. Раунды 1–2 — `HISTORY/OPUS/2026-09-30_provider_architecture_modifiers_dialog.md`.
|
||||||
|
> Режим и формат ответа — как раньше: сжато, тезисы, `файл:строка`, без догадок.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Вводная (смена приоритета)
|
||||||
|
|
||||||
|
Модификаторы (`nubes_vc_org_ip_allocation`, `nubes_vc_nsxt_snat`) — новая, возможно сложная, но
|
||||||
|
**НЕБОЛЬШАЯ** часть сервиса. Чрезмерный упор на них не нужен.
|
||||||
|
|
||||||
|
**Главное — САМ провайдер: его устойчивость и корректность.**
|
||||||
|
|
||||||
|
- `M6` (доступ к двум файлам для доразбора R1) — **снять**; углубление по модификаторам больше не требуется.
|
||||||
|
- Прошлые находки (`S1`, `S2`, `R2`–`R6`, `U1`–`U3`) **не повторять**.
|
||||||
|
|
||||||
|
## Бюджет (жёстко — экономим токены)
|
||||||
|
|
||||||
|
- Прочитать **не более 10 файлов** суммарно. Ранее прочитанные повторно не открывать.
|
||||||
|
- Ответ — **не более 5 находок**, каждая **не более 3 строк**.
|
||||||
|
- Никаких вступлений, повторения прошлых отчётов, «а ещё могу».
|
||||||
|
|
||||||
|
## Что анализировать (провайдер целиком)
|
||||||
|
|
||||||
|
1. **Жизненный цикл инстанс-ресурса:** `create` / adopt / `suspend` / resume / `modify` / redeploy /
|
||||||
|
`delete` и повторный `apply` — где теряется корректность состояния.
|
||||||
|
2. **Досылка и нормализация параметров** (`WithDefaults`, zero-value fallback, дефолты `map-fixed`,
|
||||||
|
регистр UUID): где риск затереть значение или получить ложный diff.
|
||||||
|
3. **`Read` / refresh:** устойчив ли state у генерируемых ресурсов; где возможен вечный diff.
|
||||||
|
4. **Устойчивость ядра:** ретраи и ошибки, `LockInstance`, идемпотентность, поведение при дубликатах
|
||||||
|
и параллельных `apply`.
|
||||||
|
5. **Генератор → код:** какие классы дефектов порождает шаблон.
|
||||||
|
|
||||||
|
## Границы доступа
|
||||||
|
|
||||||
|
Разрешено читать (только это):
|
||||||
|
|
||||||
|
- `provider/internal/core/**`
|
||||||
|
- `provider/internal/resources_core/**`
|
||||||
|
- `provider/internal/provider/provider.go`
|
||||||
|
- `TOOLS/resource-generator/internal/templates/**`
|
||||||
|
- `TOOLS/resource-generator/internal/params/params.go`
|
||||||
|
- `TOOLS/resource-generator/internal/helpers/helpers.go`
|
||||||
|
- `TOOLS/resource-generator/internal/loader/loader.go`
|
||||||
|
- `TOOLS/resource-generator/internal/writers/writers.go`
|
||||||
|
|
||||||
|
Запрещено: `HISTORY/**`, `NOTES/**`, `docs/**`, `HAR/**`, `TMP/**`, `DEV_STAND/**`, `TEST_STAND/**`,
|
||||||
|
`PROD_STAND/**`, git-история. Нужен файл вне списка → вопрос мне, не чтение.
|
||||||
|
|
||||||
|
## Формат ответа
|
||||||
|
|
||||||
|
- **ТОП-5 находок** по убыванию критичности: суть → место (`файл:строка`) → последствие (≤ 3 строк).
|
||||||
|
- Затем **одна строка**: что из этого проверяемо только замером (прогон `apply`/`plan`), а не чтением.
|
||||||
|
- Находок меньше пяти — столько, сколько есть. Водой не добивать.
|
||||||
|
|
||||||
|
## Стоп-правило
|
||||||
|
|
||||||
|
Не хватает файла или данных — один короткий вопрос. Не догадываться.
|
||||||
Reference in New Issue
Block a user