Cut the justification and the example out of the review rule
This commit is contained in:
parent
85365c8e92
commit
fbe6cf3786
3 changed files with 10 additions and 14 deletions
|
|
@ -71,8 +71,8 @@ Go-бэкенд издательского художественного пер
|
|||
- **При компакции/сжатии контекста ВСЕГДА сохранять:** список изменённых файлов · незакрытые находки ревью и их диспозиции · обязательства и открытые вопросы сессии · команды тестов (D39.121).
|
||||
- **Тесты и гейты не подгонять под зелень:** править или удалять тест/голден/гейт, чтобы он прошёл, — НЕДОПУСТИМО; несогласие с тестом — вопрос оркестратору пингом, не правка (D39.121).
|
||||
- **Мандат самопроверки в промтах сессий (обязателен, решение владельца):** промты полигон-сессий — и вообще любых пишущих код / запросы к моделям — ДОЛЖНЫ явно требовать ревью ИСПОЛНЕНИЕМ: своего кода + сформированных запросов к моделям + полученных результатов. Сессии регулярно ошибаются и багуют, и это искажает результат; **самоотчёт «проверено» без исполнения регулярно оказывается ложным и стоит денег**. Бэкенд-промты — явный бэкенд-ревью после кода (обычно отрабатывает по опыту, но требовать явно). Пост-хок адверсариальная верификация оркестратора при лендинге — второй рубеж, НЕ замена самопроверки.
|
||||
⚠ **Исполнения мало: промт заказывает адверсариальный проход сессии по СВОЕЙ готовой работе — но ГЛУБИНУ и веер сессия выбирает сама под предмет.** Промт задаёт НАПРАВЛЕНИЕ (что именно в этом паке уязвимо), не регламент: где-то это один точечный опровергатель, где-то шесть линз. Основание — замер 28.08: где проход был, он дал шесть дефектов, ЧЕТЫРЕ из них внёс сам пак; где не был — пак сдался с неработающей починкой внутри и был бы залендан.
|
||||
⚠ Тяжёлая модель — не больше ОДНОГО агента (или одной подсессии воркфлоу) и только на самую тяжёлую линзу; модель задавать агенту ЯВНО и запрещать ему порождать своих.
|
||||
⚠ **Исполнения мало: промт заказывает адверсариальный проход сессии по СВОЕЙ готовой работе; глубину и веер сессия выбирает под предмет, промт даёт направление — что в этом паке уязвимо.**
|
||||
⚠ Тяжёлая модель — один агент максимум. Модель задавай агенту ЯВНО и знай, сколько их у тебя работает.
|
||||
- **Git-координация мультисессий:**
|
||||
- **Коммитит только оркестратор.** Сессии зон — бэкенд, полигон, фронт, платформа — не коммитят: своё дерево готовят и передают на лендинг (исключение: пре-рег фризы полигона). Чужую зону не трогает НИКТО.
|
||||
- **Форма коммита — ТОЛЬКО с явным списком путей: `git commit -- <путь> <путь>`.** Причина механическая: голый `git commit` уносит **ВЕСЬ индекс**, включая чужие застейдженные файлы, и чужая работа уезжает под твоим сообщением. Pathspec-форма индекс не трогает.
|
||||
|
|
|
|||
|
|
@ -192,12 +192,10 @@ content-hash: бесплатная смета назовёт одно число
|
|||
При этом леджер не двинулся ни на цент, а в стор не ушло ничего сверх first-touch.
|
||||
4. **Свои посадки мутаций:** минимум по одной на пункт; вердикт — по ДЕЛЬТЕ против чистой базовой
|
||||
линии и по ТОПИЧНОСТИ упавшего теста, не по цвету батареи.
|
||||
5. **Адверсариальный проход по своей готовой работе — заказан; ГЛУБИНУ и веер выбираешь сама под
|
||||
предмет.** Направление ниже — моё, регламент — нет.
|
||||
⚠ **Куда смотреть в ЭТОМ паке:** деньги (может ли потолок дать перерасход или отказать законной работе) · тождество бесплатной сметы и платного прогона · шов (форма ответа, argv).
|
||||
Дефекты, которые внёс сам пак, называй первыми — они самые ценные и приёмке не видны.
|
||||
⚠ Тяжёлая модель — один агент максимум, на самую тяжёлую линзу; модель задавай ЯВНО, порождать
|
||||
своих агентов запрещай.
|
||||
5. **Адверсариальный проход по своей готовой работе — заказан; глубину и веер выбираешь сама.**
|
||||
⚠ Куда смотреть в ЭТОМ паке: деньги (перерасход либо отказ законной работе) · тождество бесплатной сметы и платного прогона · шов (форма ответа, argv).
|
||||
Дефекты, которые внёс сам пак, называй первыми — приёмке они не видны.
|
||||
⚠ Тяжёлая модель — один агент максимум. Модель задавай ЯВНО и знай, сколько их работает.
|
||||
|
||||
## §5. Оси ревью — 1–3, вправе заменить с аргументом
|
||||
|
||||
|
|
|
|||
|
|
@ -147,12 +147,10 @@ D39.166 и фикс-раунды). Если строка описывает со
|
|||
спавна → `abandon`), как его и нашли, а не подложенной строкой в базе.
|
||||
4. **Свои посадки мутаций:** минимум по одной на пункт; вердикт — по ДЕЛЬТЕ и ТОПИЧНОСТИ упавшего
|
||||
теста, не по цвету батареи.
|
||||
5. **Адверсариальный проход по своей готовой работе — заказан; ГЛУБИНУ и веер выбираешь сама под
|
||||
предмет.** Направление ниже — моё, регламент — нет.
|
||||
⚠ **Куда смотреть в ЭТОМ паке:** что видит тот, у кого отозвали доступ · может ли починка вернуть холд дважды или чужой · поток под нагрузкой.
|
||||
Дефекты, которые внёс сам пак, называй первыми — они самые ценные и приёмке не видны.
|
||||
⚠ Тяжёлая модель — один агент максимум, на самую тяжёлую линзу; модель задавай ЯВНО, порождать
|
||||
своих агентов запрещай.
|
||||
5. **Адверсариальный проход по своей готовой работе — заказан; глубину и веер выбираешь сама.**
|
||||
⚠ Куда смотреть в ЭТОМ паке: что видит тот, у кого отозвали доступ · может ли починка вернуть холд дважды или чужой · поток под нагрузкой.
|
||||
Дефекты, которые внёс сам пак, называй первыми — приёмке они не видны.
|
||||
⚠ Тяжёлая модель — один агент максимум. Модель задавай ЯВНО и знай, сколько их работает.
|
||||
|
||||
## §5. Оси ревью — 1–3, вправе заменить с аргументом
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue