обслуживание: найденная дельта останавливает работу предложением, а не отказом
Стоп по найденной дельта-спеке говорил только «поведение меняется, дальше идёт решение». Классификация при этом падала на человека в момент, когда весь материал для неё у исполнителя, а задача выглядела сломанной, хотя она просто оказалась шире своего типа. Порядок теперь из трёх шагов: назвать тип, которым задача оказалась (fix — расходится с заявленным, feature — снаружи появляется то, чего не было), объяснить простым языком, что нашлось, и дать два решения — переформулировать запись и решать процессом того типа следующим прогоном либо прекратить работу. Третьего решения, «доделать как обслуживание», нет. Тип исполнитель предлагает, меняет его av-dev-tasks:tasks и только после ответа: переклеенный на ходу тип назначает себе другой процесс и другую глубину проверки. Сделанное при любом решении остаётся в рабочем дереве незакоммиченным.
This commit is contained in:
@@ -49,17 +49,67 @@
|
||||
либо отправил бы в полный цикл ради пустого change, либо принял бы как
|
||||
исключение, а исключения не исполняются.
|
||||
|
||||
## Дельта нашлась по ходу — это стоп
|
||||
## Дельта нашлась по ходу — стоп, и у него свой порядок
|
||||
|
||||
Признак тот же, что на шаге 7 сценария решения: **меняется ли то, что записано в
|
||||
`openspec/specs/`**. Обнаружилось, что меняется, — работа перестала быть
|
||||
обслуживанием в ту же секунду, и продолжать её нельзя: коммит обслуживания
|
||||
заявляет «поведение не менялось», а оно меняется.
|
||||
|
||||
**Сделанное не выбрасывается.** Оно остаётся в рабочем дереве и уезжает в change
|
||||
следующим прогоном, который идёт сценарием решения. Коммитить его сообщением про
|
||||
обслуживание нельзя: так поведение меняется молча, и ни ревью дизайна, ни
|
||||
чекпоинт по нему уже не пройдут никогда.
|
||||
**Задача при этом не сломалась — она оказалась шире своего типа.** Поэтому стоп
|
||||
здесь не «бросить и доложить», а три шага по порядку.
|
||||
|
||||
**1. Назови тип, которым задача оказалась.** Разрез тот же, по которому типы и
|
||||
разведены:
|
||||
|
||||
- **`fix`** — поведение расходится с **заявленным**: спека уже описывает верное,
|
||||
и правка возвращает систему к записанному;
|
||||
- **`feature`** — снаружи появляется то, чего не было: спеке нужно новое
|
||||
требование.
|
||||
|
||||
Тип называется прямо и с причиной. «Нужно менять спеки» без имени типа
|
||||
перекладывает классификацию на человека в тот момент, когда весь материал для неё
|
||||
у тебя.
|
||||
|
||||
**2. Объясни человеку простым языком.** Экран текста, не больше:
|
||||
|
||||
- **что просили сделать** — одной фразой из записи;
|
||||
- **что нашлось** — какое поведение меняется, словами домена, а не именами
|
||||
файлов и функций;
|
||||
- **почему это перестало быть обслуживанием** — одной фразой: у обслуживания
|
||||
поведение не меняется по определению;
|
||||
- **чем задача становится** — `fix` или `feature`, с причиной из разреза выше;
|
||||
- **что уже сделано** и что из этого лежит в рабочем дереве.
|
||||
|
||||
Проверка на простой язык та же, что у чекпоинтов двух других сценариев: **в
|
||||
тексте нет `SHALL`, нет имён файлов и функций там, где вещь называется
|
||||
по-русски, и нет слов, которых нет в паспорте проекта.**
|
||||
|
||||
**3. Дай два решения и жди ответа.** Их ровно два, и оба законны:
|
||||
|
||||
- **переформулировать запись** — тип меняется на названный, и дальше задача идёт
|
||||
**процессом своего типа**: сценарием решения, следующим прогоном. Формат записи
|
||||
правит `av-dev-tasks:tasks`, а не ты: у нового типа своя схема разделов, и
|
||||
готовность её проверит `ready` — той же машиной, что и на входе. Прогон
|
||||
обслуживания на этом кончается, исход — «меняется спека»;
|
||||
- **прекратить работу** — человек не готов расширять задачу сейчас. Исход тот
|
||||
же, запись остаётся как была, вопрос записывается там, где проект держит
|
||||
вопросы.
|
||||
|
||||
**Третьего решения — «доделать как обслуживание» — нет.** Оно и есть то самое
|
||||
молчаливое изменение поведения, против которого стоит весь разрез: под коммитом,
|
||||
заявляющим «поменяли оснастку», уехала бы правка, не прошедшая ни ревью дизайна,
|
||||
ни чекпоинта, и не оставившая следа в спеках.
|
||||
|
||||
**Сделанное не выбрасывается ни при каком из двух решений.** Оно остаётся в
|
||||
рабочем дереве незакоммиченным: при переформулировке уезжает в change следующим
|
||||
прогоном, при отказе — человек решает сам, откатить или оставить. Коммитить его
|
||||
сообщением про обслуживание нельзя.
|
||||
|
||||
**Это единственное место сценария, где ждут ответа**, и плановым стопом оно не
|
||||
становится: плановый стоп проходят все прогоны, а этот — только те, где задача
|
||||
оказалась не тем, чем объявлена. Прогон, дошедший до него, стоит дороже обычного
|
||||
— и это довод за проверку признака на шаге 1, а не после написанного кода.
|
||||
|
||||
## OpenSpec здесь не предпосылка
|
||||
|
||||
@@ -79,6 +129,10 @@
|
||||
решать, вырождается в обряд одобрения и обесценивает те стопы, где решать есть
|
||||
что.
|
||||
|
||||
**Место, где ответа всё же ждут, одно, и плановым оно не является** — стоп по
|
||||
найденной дельте (раздел «Дельта нашлась по ходу»). Через него проходят не все
|
||||
прогоны, а только те, где задача оказалась не тем, чем объявлена.
|
||||
|
||||
**Правило необратимого при этом действует полностью** (SKILL.md, «Когда
|
||||
спрашивать вне чекпоинта»), и здесь оно опаснее, чем кажется. Единственный
|
||||
сценарий без планового стопа — ровно тот, чья работа чаще прочих лезет в
|
||||
@@ -114,9 +168,11 @@ flowchart TD
|
||||
- **сделана** — определение сделанного выполнено целиком;
|
||||
- **не доведена** — с причиной и с записанным вопросом; названо, что именно
|
||||
сделано и до какой границы;
|
||||
- **меняется спека** — работа оказалась изменением поведения. Стоп; сделанное
|
||||
остаётся в рабочем дереве и незакоммиченным, дальше идёт сценарий решения
|
||||
следующим прогоном;
|
||||
- **меняется спека** — работа оказалась шире своего типа. Стоп с объяснением и
|
||||
двумя решениями человека: переформулировать запись в `fix` или `feature` и
|
||||
решать её процессом того типа следующим прогоном — либо прекратить. Сделанное
|
||||
остаётся в рабочем дереве незакоммиченным. Доклад называет **оба**: какой тип
|
||||
предложен и что человек выбрал;
|
||||
- **нужна разведка** — форма правки неизвестна (мажорное обновление, смена
|
||||
сборщика, переезд гейта на другой инструмент) **или сработал триггер ADR**:
|
||||
дорогой откат, намеренный отказ от очевидного подхода, пересмотр прежнего
|
||||
@@ -284,8 +340,13 @@ Change ты не передаёшь — его нет.
|
||||
## Границы: чего обслуживание не делает
|
||||
|
||||
- **Не меняет поведения.** Обнаружилось, что меняет, — стоп с исходом «меняется
|
||||
спека». Это единственная граница сценария, у которой есть проверяемый признак,
|
||||
и она же единственная, которую выгодно нарушить молча.
|
||||
спека»: назвать тип, объяснить, дать два решения. Это единственная граница
|
||||
сценария, у которой есть проверяемый признак, и она же единственная, которую
|
||||
выгодно нарушить молча.
|
||||
- **Не переписывает запись задачи сам.** Тип ты **предлагаешь** с причиной,
|
||||
меняет его `av-dev-tasks:tasks` и только после ответа человека: исполнитель,
|
||||
переклеивший тип на ходу, назначает себе другой процесс и другую глубину
|
||||
проверки.
|
||||
- **Не решает, нужна ли работа.** Как и оба соседних сценария: «надо ли» —
|
||||
вопрос человека.
|
||||
- **Не нарезает пачку на задачи.** «Обновить зависимости и переписать сборку» —
|
||||
@@ -301,6 +362,8 @@ Change ты не передаёшь — его нет.
|
||||
Общее ядро доклада — в SKILL.md; сверх него сценарий обязан назвать:
|
||||
|
||||
- **что подтвердило признак** — тип записи и то, что дельта-спек не нашлось;
|
||||
дельта нашлась — **какой тип предложен, с причиной, и что человек выбрал**:
|
||||
переформулировать или прекратить;
|
||||
- **что стало иначе для разработчика** — одной фразой, адресуясь ему, а не
|
||||
выдуманному пользователю;
|
||||
- **состав гейта до и после**, если правка его трогала; не сверялся — почему;
|
||||
|
||||
Reference in New Issue
Block a user