@rt-tools/agent-kit 0.10.0 → 0.12.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (139) hide show
  1. package/assets/checks/check-file-size.mjs +19 -4
  2. package/assets/checks/check-state-next.mjs +202 -0
  3. package/assets/checks/check-states.mjs +142 -0
  4. package/assets/checks/check-turn-map.mjs +146 -0
  5. package/assets/checks/rt-kit-checks.config.mjs +16 -2
  6. package/assets/defaults/project.sh +67 -7
  7. package/assets/defaults/turn-map.md +48 -0
  8. package/assets/hooks/browser-device-id.sh +2 -0
  9. package/assets/hooks/browser-guard-device-id.sh +5 -1
  10. package/assets/hooks/browser-guard-no-asking.sh +5 -1
  11. package/assets/hooks/browser-guard-no-listing.sh +2 -0
  12. package/assets/hooks/browser-guard-no-other-drivers.sh +7 -3
  13. package/assets/hooks/browser-guard-require-select.sh +6 -2
  14. package/assets/hooks/claim-guard.sh +5 -1
  15. package/assets/hooks/commit-msg.sh +2 -0
  16. package/assets/hooks/conscience-guard.sh +5 -1
  17. package/assets/hooks/constitution-index.sh +2 -0
  18. package/assets/hooks/dev-server-guard.sh +7 -3
  19. package/assets/hooks/dispatch.sh +69 -0
  20. package/assets/hooks/docs-guard.sh +8 -4
  21. package/assets/hooks/exam-guard.sh +7 -3
  22. package/assets/hooks/git-guard-delivery-signature.sh +2 -0
  23. package/assets/hooks/git-guard-delivery.sh +39 -5
  24. package/assets/hooks/git-guard-main.sh +8 -4
  25. package/assets/hooks/git-guard-push-tests.sh +8 -4
  26. package/assets/hooks/glossary-load.sh +2 -0
  27. package/assets/hooks/grill-gate.sh +6 -2
  28. package/assets/hooks/handoff-entry-guard.sh +6 -2
  29. package/assets/hooks/handoff-write.sh +124 -0
  30. package/assets/hooks/hook-input.sh +54 -0
  31. package/assets/hooks/lint-after-edit.sh +7 -3
  32. package/assets/hooks/observe.sh +2 -0
  33. package/assets/hooks/postmortem-guard.sh +5 -1
  34. package/assets/hooks/proposal-guard.sh +5 -1
  35. package/assets/hooks/prose-style-guard.sh +7 -3
  36. package/assets/hooks/qa-dataid-guard.sh +6 -2
  37. package/assets/hooks/rerun-guard.sh +7 -3
  38. package/assets/hooks/reuse-first-guard.sh +7 -3
  39. package/assets/hooks/roles.sh +2 -0
  40. package/assets/hooks/rule-article.sh +99 -0
  41. package/assets/hooks/skill-gate-layers.sh +2 -0
  42. package/assets/hooks/skill-gate-rearm.sh +5 -1
  43. package/assets/hooks/skill-gate.sh +25 -2
  44. package/assets/hooks/skill-loaded.sh +5 -1
  45. package/assets/hooks/sql-guard-parse.sh +2 -0
  46. package/assets/hooks/sql-guard-request.sh +4 -1
  47. package/assets/hooks/sql-guard-target.sh +2 -0
  48. package/assets/hooks/sql-guard-write.sh +2 -0
  49. package/assets/hooks/sql-guard.sh +6 -2
  50. package/assets/hooks/task-context-load.sh +2 -0
  51. package/assets/hooks/task-flow-guard.sh +8 -4
  52. package/assets/hooks/turn-entry-load.sh +62 -0
  53. package/assets/hooks/turn-exit-guard.sh +44 -17
  54. package/assets/hooks/utf8.sh +35 -0
  55. package/assets/hooks/waiting-turn-guard.sh +5 -1
  56. package/assets/hooks/window-fill-guard.sh +37 -5
  57. package/assets/laws/work-conduct.md +34 -9
  58. package/assets/patterns/dependencies-upgrade.md +1 -1
  59. package/assets/patterns/doc-style-write.md +3 -3
  60. package/assets/patterns/git-workflow-commit.azure.md +2 -202
  61. package/assets/patterns/git-workflow-commit.github.md +2 -258
  62. package/assets/patterns/git-workflow-commit.gitlab.md +1 -217
  63. package/assets/patterns/git-workflow-docker.md +3 -3
  64. package/assets/patterns/git-workflow-merge.md +3 -2
  65. package/assets/patterns/git-workflow-migration.md +3 -3
  66. package/assets/patterns/git-workflow-pr.azure.md +224 -0
  67. package/assets/patterns/git-workflow-pr.github.md +280 -0
  68. package/assets/patterns/git-workflow-pr.gitlab.md +240 -0
  69. package/assets/patterns/git-workflow-restart.md +3 -3
  70. package/assets/patterns/git-workflow-secrets.md +3 -3
  71. package/assets/patterns/task-flow-archive.md +193 -0
  72. package/assets/patterns/task-flow-close.md +11 -160
  73. package/assets/patterns/task-flow-handoff.md +22 -4
  74. package/assets/patterns/task-flow-resume.md +6 -0
  75. package/assets/patterns/task-flow-start.md +41 -0
  76. package/assets/patterns/turn-entry-map.md +81 -0
  77. package/assets/pitfalls/doc-style.md +80 -0
  78. package/assets/pitfalls/git-workflow.azure.md +50 -0
  79. package/assets/pitfalls/git-workflow.github.md +78 -0
  80. package/assets/pitfalls/git-workflow.gitlab.md +49 -0
  81. package/assets/pitfalls/spec-driven.md +36 -0
  82. package/assets/pitfalls/styling-bem.md +45 -0
  83. package/assets/pitfalls/task-flow.md +62 -0
  84. package/assets/pitfalls/testing.md +70 -0
  85. package/assets/rules/deploy-flow.azure.md +106 -0
  86. package/assets/rules/deploy-flow.github.md +113 -0
  87. package/assets/rules/deploy-flow.gitlab.md +108 -0
  88. package/assets/rules/doc-style.md +25 -76
  89. package/assets/rules/git-workflow.azure.md +6 -92
  90. package/assets/rules/git-workflow.github.md +14 -127
  91. package/assets/rules/git-workflow.gitlab.md +6 -93
  92. package/assets/rules/spec-driven.md +39 -30
  93. package/assets/rules/styling-bem.md +20 -39
  94. package/assets/rules/task-flow.md +17 -186
  95. package/assets/rules/testing.md +3 -64
  96. package/assets/rules/turn-conduct.md +206 -0
  97. package/assets/rules/turn-entry.md +93 -0
  98. package/assets/rules/typescript-conventions.md +15 -0
  99. package/assets/skills/agent-kit.md +35 -12
  100. package/assets/templates/pitfalls.md +10 -0
  101. package/assets/templates/rule.md +5 -3
  102. package/lib/assets.d.ts.map +1 -1
  103. package/lib/assets.js +6 -1
  104. package/lib/assets.js.map +1 -1
  105. package/lib/cargo.d.ts +42 -0
  106. package/lib/cargo.d.ts.map +1 -1
  107. package/lib/cargo.js +2 -0
  108. package/lib/cargo.js.map +1 -1
  109. package/lib/cascade.d.ts.map +1 -1
  110. package/lib/cascade.js +19 -1
  111. package/lib/cascade.js.map +1 -1
  112. package/lib/commands.d.ts.map +1 -1
  113. package/lib/commands.js +65 -4
  114. package/lib/commands.js.map +1 -1
  115. package/lib/config.d.ts +16 -1
  116. package/lib/config.d.ts.map +1 -1
  117. package/lib/config.js +8 -0
  118. package/lib/config.js.map +1 -1
  119. package/lib/hooks-map.d.ts +13 -0
  120. package/lib/hooks-map.d.ts.map +1 -1
  121. package/lib/hooks-map.js +33 -1
  122. package/lib/hooks-map.js.map +1 -1
  123. package/lib/observations.d.ts +35 -1
  124. package/lib/observations.d.ts.map +1 -1
  125. package/lib/observations.js +14 -2
  126. package/lib/observations.js.map +1 -1
  127. package/lib/ship.d.ts +2 -0
  128. package/lib/ship.d.ts.map +1 -1
  129. package/lib/ship.js +2 -0
  130. package/lib/ship.js.map +1 -1
  131. package/lib/thresholds.d.ts +49 -0
  132. package/lib/thresholds.d.ts.map +1 -0
  133. package/lib/thresholds.js +151 -0
  134. package/lib/thresholds.js.map +1 -0
  135. package/package.json +1 -1
  136. package/rt-tools-agent-kit-0.12.0.tgz +0 -0
  137. package/assets/commands/agent-kit-digest.md +0 -88
  138. package/assets/commands/rules-review.md +0 -98
  139. package/rt-tools-agent-kit-0.10.0.tgz +0 -0
@@ -2,7 +2,7 @@
2
2
  name: git-workflow-commit
3
3
  kind: pattern
4
4
  rule: git-workflow
5
- description: Паттерн правила git-workflow для дерева на GitLab. Брать на заведение задачи, ветки, коммит, пуш и создание MR — заведение задачи со всеми шагами, перевод по спискам доски, слияние двух задач в одну, сверка очереди работ, работа от учётной записи машинной работы, формат заголовка, строка связи с задачей, ревьювер, исполнитель и метки MR, образец описания MR с разделом об оставшемся шаге, чеклист проверок до публикации, обход требования документа. Не брать для миграций и перезапуска прода — это паттерны git-workflow-migration и git-workflow-restart.
5
+ description: Паттерн правила git-workflow. Брать на заведение задачи, ветки, коммит и пуш — заведение задачи с меткой списка, перевод по спискам доски, слияние двух задач в одну, работа от учётной записи машинной работы, обход требования документа. Не брать на открытие MR это паттерн git-workflow-pr; не брать для миграций и перезапуска прода — это паттерны git-workflow-migration и git-workflow-restart.
6
6
  ---
7
7
 
8
8
  # Ветка, коммит и MR
@@ -15,7 +15,6 @@ description: Паттерн правила git-workflow для дерева на
15
15
  - Заводится задача, с которой начинается правка.
16
16
  - Заводится ветка под задачу.
17
17
  - Готовится коммит или пуш.
18
- - Открывается MR.
19
18
  - Работа перешла на следующий шаг, и задача переставляется в другой список доски.
20
19
 
21
20
  ## Сначала задача на доске, потом ветка
@@ -155,211 +154,6 @@ chore(deploy): docker-compose for vps
155
154
  Docs-skip: правка только в тестах хука, зеркала у него нет
156
155
  ```
157
156
 
158
- ## Номер задачи стоит в её заголовке и в заголовке MR
159
-
160
- Форма одна на оба — `[<КЛЮЧ>-<номер>] <текст>`. Номер стоит в самом заголовке, а не только в
161
- теле: в списке MR тела не видно, а в списке задач номер иначе приходится искать глазами. Тот же
162
- номер несёт и имя ветки, поэтому задача, ветка и MR читаются как одно.
163
-
164
- Задача говорит, что не так; MR тем же номером отчитывается, что сделано:
165
-
166
- ```
167
- задача [<КЛЮЧ>-86] Пустой адрес владельца — письма не уходят молча
168
- MR [<КЛЮЧ>-86] Письмо владельцу с незаполненным адресом попадает в логи
169
- ```
170
-
171
- Инфинитив из задачи в заголовок MR не переносится: «исправить» становится «исправлено»,
172
- «вернуть» — «возвращено», «добавить» — «добавлено».
173
-
174
- Тип и область — `fix(site):`, `docs(common):` — в заголовок MR не идут: это формат заголовка
175
- коммита, и там его сверяет `commitlint`. В списке MR он занимает место, ничего не добавляя:
176
- род правки и область уже видны метками.
177
-
178
- ## Не готовое к слиянию открывается черновиком
179
-
180
- Правка кода отдаётся человеку открытым MR: запушенная ветка ему не показывается нигде. Открытый
181
- MR при этом читается как приглашение влить, поэтому у незаконченной работы он открывается
182
- черновиком — кнопку слияния у черновика хостинг блокирует сам:
183
-
184
- ```bash
185
- GITLAB_TOKEN="$TOKEN" glab mr create --draft --title '[<КЛЮЧ>-86] …' --description "$(cat тело.md)"
186
- ```
187
-
188
- Признак черновика здесь — приставка `Draft:` в заголовке MR, и правится он вместе с ним:
189
- переписав заголовок вручную, черновик снимают, не заметив этого.
190
-
191
- Черновиком идёт всё, что ждёт прогона конвейера, доработки или ответа на вопрос. Вопрос
192
- задаётся в самом MR, а не остаётся в голове исполнителя: человек читает MR, а не переписку
193
- захода.
194
-
195
- Снимается черновик отдельным вызовом, и это тот самый ход, которым исполнитель говорит, что
196
- решение готово:
197
-
198
- ```bash
199
- GITLAB_TOKEN="$TOKEN" glab mr update 86 --ready
200
- ```
201
-
202
- До снятия молчание исполнителя значит «ещё не готово», после — «можно вливать». Снятие
203
- черновика и просьба влить идут одним ходом: снятый черновик, о котором человеку не сказали,
204
- ждёт разбора ровно так же, как не снятый.
205
-
206
- ## MR прикрепляется к задаче
207
-
208
- Описание начинается со строки связи. Ревьювер, исполнитель и метки задаются той же командой, и
209
- MR без них не открывается:
210
-
211
- ```bash
212
- GITLAB_TOKEN="$TOKEN" glab mr create \
213
- --title '[<КЛЮЧ>-86] Письмо владельцу с незаполненным адресом попадает в логи' \
214
- --assignee <бот> --reviewer <владелец> --label bug --label area:api \
215
- --target-branch main --remove-source-branch \
216
- --description 'Closes #86
217
-
218
- …'
219
- ```
220
-
221
- Ревьювер — всегда владелец: без запроса разбора MR не показывается ему в очереди. Исполнитель —
222
- та же учётная запись, от которой идёт машинная работа. Метки читаются у задачи, а не выбираются
223
- по памяти:
224
-
225
- ```bash
226
- glab issue view 86 --output json | jq -r '[.labels[]] | join(",")'
227
- ```
228
-
229
- Строка `Closes #<номер>` обязательна: без неё MR не прикрепляется к задаче, и сверка очереди
230
- это находит. Она же означает, что задача закрывается целиком — половину задачи одним MR не
231
- выкатывают: у задачи одна ветка, и работа, которая в неё не влезает, делится на задачи до того,
232
- как ветка заводится.
233
-
234
- У уже открытого MR то же ставится правкой:
235
-
236
- ```bash
237
- glab mr update 205 --label bug --label area:api --assignee <бот> --reviewer <владелец>
238
- glab mr update 205 --description "$(cat тело.md)"
239
- ```
240
-
241
- Правка описания переписывает его целиком, поэтому строка `Closes #<номер>` пишется заново
242
- вместе с остальным текстом. Описание перечитывается всякий раз, когда в ветку что-то влилось
243
- после публикации: MR утверждает про дерево, а дерево с тех пор изменилось.
244
-
245
- ## Образец описания MR
246
-
247
- Четыре раздела, и порядок между ними один: строка связи, что сделано, чем подтверждено,
248
- оставшийся шаг. Раздел, которому нечего сказать, пишется словами — пустой заголовок и снятый
249
- заголовок читаются одинаково, а значат разное.
250
-
251
- ```markdown
252
- Closes #86
253
-
254
- ## Что сделано
255
-
256
- - <правка, названная тем, что она меняет для читателя, а не тем, какие файлы задела>
257
-
258
- ## Чем подтверждено
259
-
260
- - <проверка>: <её вывод одной строкой>
261
- - Не гонялось: <что в набор не вошло и почему>
262
-
263
- ## Оставшийся шаг
264
-
265
- После одобрения ветка получает ещё один коммит — разбор папки задачи, — и только потом
266
- вливается. До этого коммита вливать рано: папка уедет в главную ветку неразобранной.
267
- ```
268
-
269
- Раздел «Оставшийся шаг» стоит последним и переписывается тем же вызовом, что и остальное
270
- описание, — в тот ход, которым папка разбирается и снимается черновик:
271
-
272
- ```markdown
273
- ## Оставшийся шаг
274
-
275
- Не осталось: папка задачи разобрана коммитом `<sha>`, черновик снят. Можно вливать.
276
- ```
277
-
278
- Стоит он там потому, что решение о слиянии принимается на этой странице, а не в переписке:
279
- сказанное владельцу вслух живёт до следующей реплики, а описание лежит у самой кнопки. Одно
280
- другого не отменяет — порядок обоих сообщений владельцу описывает паттерн закрытия работы.
281
-
282
- Проверить описание машиной нечем: ни одна сверка его не читает, а хостинг спрашивает только про
283
- заголовок. Держится образец тем, кто пишет описание, — как и слова вслух.
284
-
285
- ## Состояние MR читается, а не додумывается
286
-
287
- Команды правки отвечают нулевым кодом и тогда, когда ничего не сделали: токен без права на
288
- проект молча не ставит ни метку, ни ревьювера. Поэтому после них MR перечитывают:
289
-
290
- ```bash
291
- glab mr view 205 --output json \
292
- | jq '{author: .author.username, reviewers: [.reviewers[].username], labels: .labels}'
293
- ```
294
-
295
- Владельцу называют то, что прочитали, а не то, что заказывали.
296
-
297
- Учётная запись, из-под которой пришлось пушить, в этот вызов не переносится: пуш и авторство
298
- MR выбираются отдельно, и токен для публикации — всегда токен машинной работы.
299
-
300
- Перечитывают его и по времени, а не только после вызовов, которые молча ничего не сделали:
301
- состояние MR читается перед тем, как что-либо о нём сказать. Между «прогон зелёный» и следующей
302
- фразой владелец успевает влить MR, и всё сказанное о нём после этого — про вчерашний день. Так
303
- владельцу и было предложено влить то, что он влил часом раньше.
304
-
305
- Открытый MR означает, что задача ждёт разбора, — список переставляется тем же движением:
306
-
307
- ```bash
308
- npm run task:move -- 86 in-review
309
- ```
310
-
311
- ## Что проверяется до публикации MR
312
-
313
- Проверок на самом MR нет ровно до тех пор, пока конвейер не запущен, а запускается он пушем.
314
- Линтеры, юниты и сценарии хуков снимает гейт пуша — ниже то, чего он не знает.
315
-
316
- 1. **Главная ветка влита в эту ветку** — `git fetch origin && git merge origin/main`.
317
- Всё, что проверяется ниже, проверяется от этого основания: MR с разошедшейся ветки
318
- показывает ревьюверу правку вперемешку с чужой. Порядок и разбор конфликта — паттерн
319
- `git-workflow-merge`.
320
- 2. **В ветке только та правка, за которой её заводили** — `git diff main...HEAD --stat`. Чужой
321
- домен в списке файлов означает, что правка расползлась, и её надо вернуть в свои границы.
322
- 3. **Ни мока, ни подменённого ответа, ни отладочной строки** — `git diff main...HEAD` читается
323
- целиком, а не по именам файлов. На прод они уезжают молча и портят настоящие данные.
324
- 4. **Документ едет тем же коммитом.** Пару называет `docs-guard`, но спек домена и правку его
325
- поведения он не знает — это остаётся за автором.
326
- 5. **Проверки текстов и раскладки зелёные** — те, что дерево завело в `tools/`. Какие именно
327
- есть здесь — `implementation.md` правила.
328
- 6. **Все приложения дерева собираются** — `nx build` по каждому. Гейт пуша сборку не гоняет.
329
- 7. **Видимый текст заведён во всех локалях перевода** — тестом полноты словарей, если дерево
330
- переводится.
331
- 8. **Правка вёрстки подтверждена замером**, а не взглядом, и снята при узком экране — паттерн
332
- `browser-verification-measure`.
333
- 9. **Правка разметки публичного сайта проверена на прод-сборке по всем локалям перевода** —
334
- паттерн `seo-verify`.
335
- 10. **Описание MR собрано по образцу** — начинается строкой `Closes #<номер>`, несёт разделы
336
- «Что сделано», «Чем подтверждено» и «Оставшийся шаг», а метки, ревьювер и исполнитель
337
- стоят. Раздел оставшегося шага к этому моменту говорит, что шагов не осталось: черновик
338
- снимается после разбора папки, а не до него.
339
- 11. **Заголовок MR несёт номер задачи и называет её сделанной:** `[<КЛЮЧ>-<номер>] <Что
340
- сделано>`, тем же номером, что стоит у задачи и в имени ветки.
341
- 12. **Очередь работ сходится** — `npm run check:board`.
342
- 13. **Состояние MR прочитано, а не выведено из кодов возврата.**
343
- 14. **Набор взят из файла конвейера, а не собран по памяти.** Гейт пуша заведомо уже: он стоит
344
- между командой и пушем, и всё, что дольше секунд, из него вынесено. Что гоняет конвейер,
345
- написано в его файле — этот список и повторяется локально; зелёный гейт полнотой набора не
346
- является.
347
- 15. **Набор пересмотрен после вливания главной ветки.** Он выбирается по тому, что ветка везёт
348
- теперь, а не по тому, что правил автор. Ветка, не тронувшая ни строки показа, прогоняет
349
- снимки витрин: с момента вливания их гоняет конвейер на её коде, и красное придёт на её
350
- PR.
351
-
352
- Сразу после публикации задача переставляется в разбор, и сверка очереди прогоняется ещё раз: до
353
- открытия MR список она не судит, а после открытия расхождение видит.
354
-
355
- Сделанное рассуждением и сделанное замером в теле MR разводятся прямо: непроверенное,
356
- названное проверенным, ревьювер принимает за проверенное.
357
-
358
- **Раздел «Чем подтверждено» называет и то, что не гонялось.** Список одного прогнанного
359
- неотличим от полного набора, и ревьювер по нему решает, что можно не перепроверять. Цена ошибки
360
- здесь не красный конвейер, а доверие к разделу: однажды прочитанный как полнота, дальше он
361
- перепроверяется весь.
362
-
363
157
  ## Частые промахи
364
158
 
365
159
  - Метка списка не поставлена при заведении: задача есть, а на доске её нет. Доска показывает
@@ -371,14 +165,4 @@ npm run task:move -- 86 in-review
371
165
  команда обрывается на первом промахе целиком. Следующий `git commit --amend` при этом уносит
372
166
  в коммит всё, что осталось в индексе. Состав коммита читается `git show --stat` сразу после
373
167
  него, а не на разборе MR.
374
- - MR открыт без ревьювера: он не попадает во входящие владельца, и очередь стоит, выглядя
375
- работающей.
376
- - Метки поставлены по названию MR, а не прочитаны у задачи: область теряется, и по доске не
377
- видно, что правка задела ещё и соседний домен.
378
- - Вторая строка `Closes` в одном MR: две задачи в одной ветке откатываются только вместе. Либо
379
- это одна задача — и вторая поглощается, — либо две ветки.
380
- - Половина задачи, уехавшая своим MR: описание такого MR начинается со слов «Часть #<номер>»
381
- вместо `Closes`, задача остаётся открытой, и после отката видно её целой.
382
- - `--remove-source-branch` забыт: ветки задач копятся в репозитории, и по списку веток больше
383
- не видно, какая работа идёт сейчас.
384
168
  - Правка владельца ни токена, ни переменных не берёт — они только для машинной работы.
@@ -1,13 +1,13 @@
1
1
  ---
2
2
  name: git-workflow-docker
3
3
  kind: pattern
4
- rule: git-workflow
5
- description: Паттерн правила git-workflow. Брать при работе с образами на своей машине — подъём и перезапуск демона, диагностика «висящей» команды, сборка под платформу прод-сервера, одноразовый контейнер рядом с чужими, вход в реестр из службы. Не брать для команд прод-сервера и выбора образа — это паттерн git-workflow-restart, и не для наката миграций — это git-workflow-migration.
4
+ rule: deploy-flow
5
+ description: Паттерн правила deploy-flow. Брать при работе с образами на своей машине — подъём и перезапуск демона, диагностика «висящей» команды, сборка под платформу прод-сервера, одноразовый контейнер рядом с чужими, вход в реестр из службы. Не брать для команд прод-сервера и выбора образа — это паттерн git-workflow-restart, и не для наката миграций — это git-workflow-migration.
6
6
  ---
7
7
 
8
8
  # Образы на своей машине
9
9
 
10
- Паттерн правила `git-workflow`. Что при этом должно быть верно — закон
10
+ Паттерн правила `deploy-flow`. Что при этом должно быть верно — закон
11
11
  `docs/constitution/delivery.md`.
12
12
 
13
13
  Демон здесь чужой: на машине владельца в нём живут его хранилище разработки, его стенд и
@@ -2,7 +2,7 @@
2
2
  name: git-workflow-merge
3
3
  kind: pattern
4
4
  rule: git-workflow
5
- description: Паттерн правила git-workflow. Брать, когда главная ветка вливается в ветку задачи и разрешается конфликт — порядок мержа, разбор конфликта по роду файла, сверка дописанного веткой с очередью работ, проверки после разрешения, перечитывание тела уже открытого PR. Не брать для заведения ветки, коммита и PR — это паттерн git-workflow-commit.
5
+ description: Паттерн правила git-workflow. Брать, когда главная ветка вливается в ветку задачи и разрешается конфликт — порядок мержа, разбор конфликта по роду файла, сверка дописанного веткой с очередью работ, проверки после разрешения, перечитывание тела уже открытого PR. Не брать для заведения ветки и коммита — это паттерн git-workflow-commit, для PR — git-workflow-pr.
6
6
  ---
7
7
 
8
8
  # Мерж главной ветки в ветку задачи
@@ -73,7 +73,8 @@ bash .claude/hooks/tests/run.sh # если конфликт задел х
73
73
  ```
74
74
 
75
75
  Коммит мержа подписывается ботом тем же способом, что и любой другой, — паттерн
76
- `git-workflow-commit`. После пуша состояние читается у самого PR, а не по своему дереву:
76
+ `git-workflow-commit`. После пуша состояние читается у самого PR, а не по своему дереву, —
77
+ подробнее об этом паттерн `git-workflow-pr`:
77
78
 
78
79
  ```bash
79
80
  /opt/homebrew/bin/gh pr view <номер> --json mergeable,mergeStateStatus
@@ -1,13 +1,13 @@
1
1
  ---
2
2
  name: git-workflow-migration
3
3
  kind: pattern
4
- rule: git-workflow
5
- description: Паттерн правила git-workflow. Брать при правке prisma/schema.prisma и prisma/migrations/** — готовые команды одноразового контейнера, написание файла миграции через migrate diff, накат локальной базы. Не брать для коммита и PR — это паттерн git-workflow-commit.
4
+ rule: deploy-flow
5
+ description: Паттерн правила deploy-flow. Брать при правке prisma/schema.prisma и prisma/migrations/** — готовые команды одноразового контейнера, написание файла миграции через migrate diff, накат локальной базы. Не брать для коммита — это паттерн git-workflow-commit, для PR — git-workflow-pr.
6
6
  ---
7
7
 
8
8
  # Миграция и прогон цепочки
9
9
 
10
- Паттерн правила `git-workflow`. Что при этом должно быть верно — закон
10
+ Паттерн правила `deploy-flow`. Что при этом должно быть верно — закон
11
11
  `docs/constitution/delivery.md`.
12
12
 
13
13
  ## Когда брать
@@ -0,0 +1,224 @@
1
+ ---
2
+ name: git-workflow-pr
3
+ kind: pattern
4
+ rule: git-workflow
5
+ description: Паттерн правила git-workflow. Брать на открытие PR и всё, что с ним связано, — формат заголовка, черновик и его снятие, привязка к рабочему элементу, ревьювер и исполнитель, образец тела с разделом об оставшемся шаге, чтение состояния PR, перевод элемента в состояние разбора, чеклист проверок до снятия черновика. Не брать на заведение элемента, ветки и коммит — это паттерн git-workflow-commit.
6
+ ---
7
+
8
+ # Заявка на слияние
9
+
10
+ Паттерн правила `git-workflow`. Что при этом должно быть верно — закон
11
+ `docs/constitution/delivery.md`. Заведение задачи, ветки и коммит — паттерн
12
+ `git-workflow-commit`.
13
+
14
+ ## Когда брать
15
+
16
+ - Открывается PR по готовой ветке.
17
+ - Правится заголовок, тело или метки уже открытого PR.
18
+ - Снимается черновик, и работа отдаётся на разбор.
19
+ - Читается состояние PR перед тем, как что-либо о нём сказать владельцу.
20
+
21
+ ## Номер элемента стоит в его заголовке и в заголовке PR
22
+
23
+ Форма одна на оба — `[<номер>] <текст>`. Номер стоит в самом заголовке, а не только в теле: в
24
+ списке PR тела не видно. Тот же номер несёт и имя ветки, поэтому элемент, ветка и PR читаются
25
+ как одно.
26
+
27
+ Элемент говорит, что не так; PR тем же номером отчитывается, что сделано:
28
+
29
+ ```
30
+ элемент [86] Пустой адрес владельца — письма не уходят молча
31
+ PR [86] Письмо владельцу с незаполненным адресом попадает в логи
32
+ ```
33
+
34
+ Инфинитив в заголовок PR не переносится: «исправить» становится «исправлено», «вернуть» —
35
+ «возвращено», «добавить» — «добавлено».
36
+
37
+ Тип и область — `fix(site):`, `docs(common):` — в заголовок PR не идут: это формат заголовка
38
+ коммита, и там его сверяет `commitlint`.
39
+
40
+ ## Не готовое к слиянию открывается черновиком
41
+
42
+ Правка кода отдаётся человеку открытым PR: запушенная ветка ему не показывается нигде. Открытый
43
+ PR при этом читается как приглашение влить, поэтому у незаконченной работы он открывается
44
+ черновиком — завершение у черновика хостинг блокирует сам:
45
+
46
+ ```bash
47
+ AZURE_DEVOPS_EXT_PAT="$TOKEN" az repos pr create --draft true --title '[<КЛЮЧ>-86] …' \
48
+ --description "$(cat тело.md)"
49
+ ```
50
+
51
+ Конвейер проверок у черновика по умолчанию не запускается: молчание прогона за зелёный прогон
52
+ не принимается, и набор гоняется на своей машине либо запуском вручную.
53
+
54
+ Черновиком идёт всё, что ждёт прогона конвейера, доработки или ответа на вопрос. Вопрос
55
+ задаётся в самом PR, а не остаётся в голове исполнителя: человек читает PR, а не переписку
56
+ захода.
57
+
58
+ Снимается черновик отдельным вызовом, и это тот самый ход, которым исполнитель говорит, что
59
+ решение готово:
60
+
61
+ ```bash
62
+ AZURE_DEVOPS_EXT_PAT="$TOKEN" az repos pr update --id 86 --draft false
63
+ ```
64
+
65
+ До снятия молчание исполнителя значит «ещё не готово», после — «можно вливать». Снятие
66
+ черновика и просьба влить идут одним ходом: снятый черновик, о котором человеку не сказали,
67
+ ждёт разбора ровно так же, как не снятый.
68
+
69
+ ## PR привязывается к элементу при создании
70
+
71
+ Привязка задаётся флагом, а не правкой после: у токена может не быть права править чужой
72
+ элемент, и вторая команда обойдётся молча, оставив PR ни с чем не связанным.
73
+
74
+ ```bash
75
+ AZURE_DEVOPS_EXT_PAT="$TOKEN" az repos pr create \
76
+ --title '[86] Письмо владельцу с незаполненным адресом попадает в логи' \
77
+ --source-branch 86-mail-owner-silence --target-branch main \
78
+ --work-items 86 --reviewers <владелец> --delete-source-branch true \
79
+ --description 'Закрывает рабочий элемент 86.'
80
+ ```
81
+
82
+ Ревьювер — всегда владелец: без запроса разбора PR не показывается ему в очереди. Один PR
83
+ закрывает элемент целиком — половину задачи одним PR не выкатывают: у задачи одна ветка, и
84
+ работа, которая в неё не влезает, делится на задачи до того, как ветка заводится.
85
+
86
+ У уже открытого PR то же ставится правкой:
87
+
88
+ ```bash
89
+ az repos pr work-item add --id 205 --work-items 86
90
+ az repos pr reviewer add --id 205 --reviewers <владелец>
91
+ az repos pr update --id 205 --description "$(cat тело.md)"
92
+ ```
93
+
94
+ Правка описания переписывает его целиком. Тело перечитывается всякий раз, когда в ветку что-то
95
+ влилось после публикации: PR утверждает про дерево, а дерево с тех пор изменилось.
96
+
97
+ ## Образец тела PR
98
+
99
+ Четыре раздела, и порядок между ними один: строка связи, что сделано, чем подтверждено,
100
+ оставшийся шаг. Раздел, которому нечего сказать, пишется словами — пустой заголовок и снятый
101
+ заголовок читаются одинаково, а значат разное.
102
+
103
+ ```markdown
104
+ Закрывает рабочий элемент 86.
105
+
106
+ ## Что сделано
107
+
108
+ - <правка, названная тем, что она меняет для читателя, а не тем, какие файлы задела>
109
+
110
+ ## Чем подтверждено
111
+
112
+ - <проверка>: <её вывод одной строкой>
113
+ - Не гонялось: <что в набор не вошло и почему>
114
+
115
+ ## Оставшийся шаг
116
+
117
+ После одобрения ветка получает ещё один коммит — разбор папки задачи, — и только потом
118
+ вливается. До этого коммита вливать рано: папка уедет в главную ветку неразобранной.
119
+ ```
120
+
121
+ Раздел «Оставшийся шаг» стоит последним и переписывается тем же вызовом, что и остальное тело,
122
+ — в тот ход, которым папка разбирается и снимается черновик:
123
+
124
+ ```markdown
125
+ ## Оставшийся шаг
126
+
127
+ Не осталось: папка задачи разобрана коммитом `<sha>`, черновик снят. Можно вливать.
128
+ ```
129
+
130
+ Стоит он там потому, что решение о слиянии принимается на этой странице, а не в переписке:
131
+ сказанное владельцу вслух живёт до следующей реплики, а тело лежит у самой кнопки. Одно другого
132
+ не отменяет — порядок обоих сообщений владельцу описывает паттерн закрытия работы.
133
+
134
+ Проверить тело машиной нечем: ни одна сверка его не читает, а хостинг спрашивает только про
135
+ заголовок. Держится образец тем, кто пишет тело, — как и слова вслух.
136
+
137
+ ## Состояние PR читается, а не додумывается
138
+
139
+ Команды правки отвечают нулевым кодом и тогда, когда ничего не сделали. Поэтому после них PR
140
+ перечитывают:
141
+
142
+ ```bash
143
+ az repos pr show --id 205 \
144
+ --query '{author: createdBy.uniqueName, reviewers: reviewers[].uniqueName, work: workItemRefs[].id}'
145
+ ```
146
+
147
+ Владельцу называют то, что прочитали, а не то, что заказывали.
148
+
149
+ Перечитывают его и по времени, а не только после вызовов, которые молча ничего не сделали:
150
+ состояние PR читается перед тем, как что-либо о нём сказать. Между «прогон зелёный» и следующей
151
+ фразой владелец успевает влить PR, и всё сказанное о нём после этого — про вчерашний день. Так
152
+ владельцу и было предложено влить то, что он влил часом раньше.
153
+
154
+ Открытый PR означает, что элемент ждёт разбора, — состояние переставляется тем же движением:
155
+
156
+ ```bash
157
+ npm run task:move -- 86 in-review
158
+ ```
159
+
160
+ ## Что проверяется до публикации PR
161
+
162
+ Конвейер видит только отправленное, а отправляется оно пушем. Линтеры, юниты и сценарии хуков
163
+ снимает гейт пуша — ниже то, чего он не знает.
164
+
165
+ 1. **Главная ветка влита в эту ветку** — `git fetch origin && git merge origin/main`.
166
+ Всё, что проверяется ниже, проверяется от этого основания: PR с разошедшейся ветки
167
+ показывает ревьюверу правку вперемешку с чужой. Порядок и разбор конфликта — паттерн
168
+ `git-workflow-merge`.
169
+ 2. **В ветке только та правка, за которой её заводили** — `git diff main...HEAD --stat`. Чужой
170
+ домен в списке файлов означает, что правка расползлась, и её надо вернуть в свои границы.
171
+ 3. **Ни мока, ни подменённого ответа, ни отладочной строки** — `git diff main...HEAD` читается
172
+ целиком, а не по именам файлов. На прод они уезжают молча и портят настоящие данные.
173
+ 4. **Документ едет тем же коммитом.** Пару называет `docs-guard`, но спек домена и правку его
174
+ поведения он не знает — это остаётся за автором.
175
+ 5. **Проверки текстов и раскладки зелёные** — те, что дерево завело в `tools/`. Какие именно
176
+ есть здесь — `implementation.md` правила.
177
+ 6. **Все приложения дерева собираются** — `nx build` по каждому. Гейт пуша сборку не гоняет.
178
+ 7. **Видимый текст заведён во всех локалях перевода** — тестом полноты словарей, если дерево
179
+ переводится.
180
+ 8. **Правка вёрстки подтверждена замером**, а не взглядом, и снята при узком экране — паттерн
181
+ `browser-verification-measure`.
182
+ 9. **Правка разметки публичного сайта проверена на прод-сборке по всем локалям перевода** —
183
+ паттерн `seo-verify`.
184
+ 10. **PR привязан к рабочему элементу**, ревьювер и исполнитель стоят, а тело собрано по
185
+ образцу — разделы «Что сделано», «Чем подтверждено» и «Оставшийся шаг». Раздел оставшегося
186
+ шага к этому моменту говорит, что шагов не осталось: черновик снимается после разбора
187
+ папки, а не до него.
188
+ 11. **Заголовок PR несёт номер элемента и называет работу сделанной:** `[<номер>] <Что
189
+ сделано>`, тем же номером, что стоит у элемента и в имени ветки.
190
+ 12. **Очередь работ сходится** — `npm run check:board`.
191
+ 13. **Состояние PR прочитано, а не выведено из кодов возврата.**
192
+ 14. **Набор взят из файла конвейера, а не собран по памяти.** Гейт пуша заведомо уже: он стоит
193
+ между командой и пушем, и всё, что дольше секунд, из него вынесено. Что гоняет конвейер,
194
+ написано в его файле — этот список и повторяется локально; зелёный гейт полнотой набора не
195
+ является.
196
+ 15. **Набор пересмотрен после вливания главной ветки.** Он выбирается по тому, что ветка везёт
197
+ теперь, а не по тому, что правил автор. Ветка, не тронувшая ни строки показа, прогоняет
198
+ снимки витрин: с момента вливания их гоняет конвейер на её коде, и красное придёт на её
199
+ PR.
200
+
201
+ Сразу после публикации элемент переводится в разбор, и сверка очереди прогоняется ещё раз: до
202
+ открытия PR состояние она не судит, а после открытия расхождение видит.
203
+
204
+ Сделанное рассуждением и сделанное замером в теле PR разводятся прямо: непроверенное,
205
+ названное проверенным, ревьювер принимает за проверенное.
206
+
207
+ **Раздел «Чем подтверждено» называет и то, что не гонялось.** Список одного прогнанного
208
+ неотличим от полного набора, и ревьювер по нему решает, что можно не перепроверять. Цена ошибки
209
+ здесь не красный конвейер, а доверие к разделу: однажды прочитанный как полнота, дальше он
210
+ перепроверяется весь.
211
+
212
+ ## Частые промахи
213
+
214
+ Промахи про заведение задачи, ветку и коммит — паттерн
215
+ `git-workflow-commit`.
216
+
217
+ - PR открыт без `--work-items`: связи нет, и по очереди работ не видно, за чем эта правка.
218
+ - `AB#<номер>` в коммите принят за привязку PR: он связывает коммит, а очередь читает связь PR.
219
+ - PR открыт без ревьювера: он не попадает во входящие владельца, и очередь стоит, выглядя
220
+ работающей.
221
+ - `--delete-source-branch` забыт: ветки задач копятся в репозитории, и по списку веток больше
222
+ не видно, какая работа идёт сейчас.
223
+ - Автозавершение включено до того, как прогнаны проверки до пуша: конвейер зелёный на том, что
224
+ он умеет, и слияние происходит без всего остального.