@rt-tools/agent-kit 0.11.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 (115) hide show
  1. package/assets/checks/check-file-size.mjs +19 -4
  2. package/assets/checks/check-state-next.mjs +10 -2
  3. package/assets/checks/rt-kit-checks.config.mjs +16 -2
  4. package/assets/defaults/project.sh +9 -1
  5. package/assets/defaults/turn-map.md +8 -6
  6. package/assets/hooks/browser-guard-device-id.sh +3 -1
  7. package/assets/hooks/browser-guard-no-asking.sh +3 -1
  8. package/assets/hooks/browser-guard-no-other-drivers.sh +5 -3
  9. package/assets/hooks/browser-guard-require-select.sh +4 -2
  10. package/assets/hooks/claim-guard.sh +3 -1
  11. package/assets/hooks/conscience-guard.sh +3 -1
  12. package/assets/hooks/dev-server-guard.sh +5 -3
  13. package/assets/hooks/dispatch.sh +69 -0
  14. package/assets/hooks/docs-guard.sh +6 -4
  15. package/assets/hooks/exam-guard.sh +5 -3
  16. package/assets/hooks/git-guard-delivery.sh +37 -5
  17. package/assets/hooks/git-guard-main.sh +6 -4
  18. package/assets/hooks/git-guard-push-tests.sh +6 -4
  19. package/assets/hooks/grill-gate.sh +4 -2
  20. package/assets/hooks/handoff-entry-guard.sh +4 -2
  21. package/assets/hooks/handoff-write.sh +27 -6
  22. package/assets/hooks/hook-input.sh +54 -0
  23. package/assets/hooks/lint-after-edit.sh +5 -3
  24. package/assets/hooks/postmortem-guard.sh +3 -1
  25. package/assets/hooks/proposal-guard.sh +3 -1
  26. package/assets/hooks/prose-style-guard.sh +5 -3
  27. package/assets/hooks/qa-dataid-guard.sh +4 -2
  28. package/assets/hooks/rerun-guard.sh +5 -3
  29. package/assets/hooks/reuse-first-guard.sh +5 -3
  30. package/assets/hooks/rule-article.sh +99 -0
  31. package/assets/hooks/skill-gate-rearm.sh +3 -1
  32. package/assets/hooks/skill-gate.sh +23 -2
  33. package/assets/hooks/skill-loaded.sh +3 -1
  34. package/assets/hooks/sql-guard-request.sh +2 -1
  35. package/assets/hooks/sql-guard.sh +4 -2
  36. package/assets/hooks/task-flow-guard.sh +6 -4
  37. package/assets/hooks/turn-exit-guard.sh +42 -17
  38. package/assets/hooks/waiting-turn-guard.sh +3 -1
  39. package/assets/hooks/window-fill-guard.sh +6 -4
  40. package/assets/laws/work-conduct.md +5 -9
  41. package/assets/patterns/dependencies-upgrade.md +1 -1
  42. package/assets/patterns/doc-style-write.md +3 -3
  43. package/assets/patterns/git-workflow-commit.azure.md +2 -202
  44. package/assets/patterns/git-workflow-commit.github.md +2 -258
  45. package/assets/patterns/git-workflow-commit.gitlab.md +1 -217
  46. package/assets/patterns/git-workflow-docker.md +3 -3
  47. package/assets/patterns/git-workflow-merge.md +3 -2
  48. package/assets/patterns/git-workflow-migration.md +3 -3
  49. package/assets/patterns/git-workflow-pr.azure.md +224 -0
  50. package/assets/patterns/git-workflow-pr.github.md +280 -0
  51. package/assets/patterns/git-workflow-pr.gitlab.md +240 -0
  52. package/assets/patterns/git-workflow-restart.md +3 -3
  53. package/assets/patterns/git-workflow-secrets.md +3 -3
  54. package/assets/patterns/task-flow-archive.md +193 -0
  55. package/assets/patterns/task-flow-close.md +3 -173
  56. package/assets/patterns/task-flow-handoff.md +4 -4
  57. package/assets/pitfalls/doc-style.md +80 -0
  58. package/assets/pitfalls/git-workflow.azure.md +50 -0
  59. package/assets/pitfalls/git-workflow.github.md +78 -0
  60. package/assets/pitfalls/git-workflow.gitlab.md +49 -0
  61. package/assets/pitfalls/spec-driven.md +36 -0
  62. package/assets/pitfalls/styling-bem.md +45 -0
  63. package/assets/pitfalls/task-flow.md +62 -0
  64. package/assets/pitfalls/testing.md +70 -0
  65. package/assets/rules/deploy-flow.azure.md +106 -0
  66. package/assets/rules/deploy-flow.github.md +113 -0
  67. package/assets/rules/deploy-flow.gitlab.md +108 -0
  68. package/assets/rules/doc-style.md +25 -76
  69. package/assets/rules/git-workflow.azure.md +6 -92
  70. package/assets/rules/git-workflow.github.md +14 -127
  71. package/assets/rules/git-workflow.gitlab.md +6 -93
  72. package/assets/rules/spec-driven.md +39 -30
  73. package/assets/rules/styling-bem.md +20 -39
  74. package/assets/rules/task-flow.md +17 -199
  75. package/assets/rules/testing.md +3 -64
  76. package/assets/rules/turn-conduct.md +206 -0
  77. package/assets/rules/typescript-conventions.md +15 -0
  78. package/assets/skills/agent-kit.md +35 -12
  79. package/assets/templates/pitfalls.md +10 -0
  80. package/assets/templates/rule.md +5 -3
  81. package/bin/agent-kit.d.ts.map +1 -1
  82. package/bin/agent-kit.js +1 -42
  83. package/bin/agent-kit.js.map +1 -1
  84. package/lib/assets.d.ts.map +1 -1
  85. package/lib/assets.js +6 -1
  86. package/lib/assets.js.map +1 -1
  87. package/lib/cascade.d.ts.map +1 -1
  88. package/lib/cascade.js +19 -1
  89. package/lib/cascade.js.map +1 -1
  90. package/lib/commands.d.ts.map +1 -1
  91. package/lib/commands.js +1 -0
  92. package/lib/commands.js.map +1 -1
  93. package/lib/config.d.ts +16 -1
  94. package/lib/config.d.ts.map +1 -1
  95. package/lib/config.js +8 -0
  96. package/lib/config.js.map +1 -1
  97. package/lib/hooks-map.d.ts +13 -0
  98. package/lib/hooks-map.d.ts.map +1 -1
  99. package/lib/hooks-map.js +33 -1
  100. package/lib/hooks-map.js.map +1 -1
  101. package/lib/ship.d.ts +1 -2
  102. package/lib/ship.d.ts.map +1 -1
  103. package/lib/ship.js +0 -54
  104. package/lib/ship.js.map +1 -1
  105. package/package.json +1 -1
  106. package/rt-tools-agent-kit-0.12.0.tgz +0 -0
  107. package/assets/commands/agent-kit-digest.md +0 -89
  108. package/assets/commands/rules-review.md +0 -98
  109. package/assets/patterns/cargo-triage-mark.md +0 -119
  110. package/assets/rules/cargo-triage.md +0 -126
  111. package/lib/cargo-state.d.ts +0 -62
  112. package/lib/cargo-state.d.ts.map +0 -1
  113. package/lib/cargo-state.js +0 -118
  114. package/lib/cargo-state.js.map +0 -1
  115. package/rt-tools-agent-kit-0.11.0.tgz +0 -0
@@ -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
+ он умеет, и слияние происходит без всего остального.
@@ -0,0 +1,280 @@
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 тела не видно, а в списке задач номер иначе приходится искать глазами по
25
+ колонке слева. Тот же номер несёт и имя ветки — `<КЛЮЧ>-<номер>-<короткий-slug>`, — поэтому
26
+ задача, ветка и PR читаются как одно.
27
+
28
+ Задача говорит, что не так; PR тем же номером отчитывается, что сделано:
29
+
30
+ ```
31
+ задача [<КЛЮЧ>-86] Пустой MAIL_OWNER — письма владельцу не уходят молча
32
+ PR [<КЛЮЧ>-86] Письмо владельцу с незаполненным адресом попадает в логи
33
+
34
+ задача [<КЛЮЧ>-101] Вернуть оверлей загрузки таблицы и включить stylelint гейтом
35
+ PR [<КЛЮЧ>-101] Stylelint включён гейтом
36
+
37
+ задача [<КЛЮЧ>-212] Сайт не собирается: компонентам кита проставлен префикс приложения вместо своего
38
+ PR [<КЛЮЧ>-212] Виджет переписки зовёт кит его собственными именами
39
+ ```
40
+
41
+ Инфинитив из задачи в заголовок PR не переносится: «исправить» становится «исправлено»,
42
+ «вернуть» — «возвращено», «добавить» — «добавлено».
43
+
44
+ Номер в заголовке обязан совпасть с номером ветки: гард поставки сверяет их до отправки
45
+ команды, а сверка очереди — у каждого открытого PR.
46
+
47
+ Тип и область — `fix(site):`, `docs(common):` — в заголовок PR не идут: это формат заголовка
48
+ коммита, и там его сверяет `commitlint`. В списке PR он занимает место, ничего не добавляя:
49
+ род правки и область уже видны метками.
50
+
51
+ ## Не готовое к слиянию открывается черновиком
52
+
53
+ Правка кода отдаётся человеку открытым PR: запушенная ветка ему не показывается нигде. Открытый
54
+ PR при этом читается как приглашение влить, поэтому у незаконченной работы он открывается
55
+ черновиком — кнопку слияния у черновика хостинг блокирует сам:
56
+
57
+ ```bash
58
+ GH_TOKEN="$TOKEN" gh pr create --draft --title '[<КЛЮЧ>-86] …' --body-file тело.md
59
+ ```
60
+
61
+ Черновиком идёт всё, что ждёт прогона конвейера, доработки или ответа на вопрос. Вопрос
62
+ задаётся в самом PR, а не остаётся в голове исполнителя: человек читает PR, а не переписку
63
+ захода.
64
+
65
+ Снимается черновик отдельным вызовом, и это тот самый ход, которым исполнитель говорит, что
66
+ решение готово:
67
+
68
+ ```bash
69
+ GH_TOKEN="$TOKEN" gh pr ready 86
70
+ ```
71
+
72
+ До снятия молчание исполнителя значит «ещё не готово», после — «можно вливать». Снятие
73
+ черновика и просьба влить идут одним ходом: снятый черновик, о котором человеку не сказали,
74
+ ждёт разбора ровно так же, как не снятый.
75
+
76
+ ## PR прикрепляется к задаче
77
+
78
+ Тело начинается со строки связи — по ней на борде заполняется поле «Linked pull requests».
79
+ Ревьювер, исполнитель и метки задаются той же командой, и PR без них не открывается:
80
+
81
+ ```bash
82
+ GH_TOKEN="$TOKEN" gh pr create --title '[<КЛЮЧ>-86] Письмо владельцу с незаполненным адресом попадает в логи' \
83
+ --reviewer <владелец> --assignee <бот> --label bug --label area:api \
84
+ --body 'Closes #86
85
+
86
+ …'
87
+ ```
88
+
89
+ Ревьювер — всегда владелец: без запроса разбора PR не показывается ему в очереди. Исполнитель —
90
+ та же учётная запись, от которой идёт машинная работа. Метки берутся у задачи целиком — и род
91
+ правки, и все её области; читаются они у задачи, а не выбираются по памяти:
92
+
93
+ ```bash
94
+ /opt/homebrew/bin/gh issue view 86 --json labels --jq '.labels | map(.name) | join(",")'
95
+ ```
96
+
97
+ Строка `Closes #<номер>` обязательна: без неё PR не прикрепляется к задаче, и сверка очереди
98
+ это находит. Она же и означает, что задача закрывается целиком — половину задачи одним PR не
99
+ выкатывают: у задачи одна ветка, и работа, которая в неё не влезает, делится на задачи до
100
+ того, как ветка заводится.
101
+
102
+ У уже открытого PR то же ставится тремя вызовами REST. `gh pr edit` здесь не годится: он
103
+ запрашивает карточки Projects (classic), получает отказ о снятом API и до правки не доходит.
104
+
105
+ ```bash
106
+ GH=/opt/homebrew/bin/gh
107
+ REPO=<владелец>/<репозиторий>
108
+
109
+ $GH api -X POST "repos/$REPO/issues/205/labels" -f 'labels[]=bug' -f 'labels[]=area:api'
110
+ $GH api -X POST "repos/$REPO/issues/205/assignees" -f 'assignees[]=<бот>'
111
+ $GH api -X POST "repos/$REPO/pulls/205/requested_reviewers" -f 'reviewers[]=<владелец>'
112
+ ```
113
+
114
+ Тем же вызовом правится и само тело: `-f body=` переписывает его целиком, поэтому строка
115
+ `Closes #<номер>` пишется заново вместе с остальным текстом.
116
+
117
+ ```bash
118
+ $GH api -X PATCH "repos/$REPO/pulls/205" -f body="$(cat тело.md)"
119
+ ```
120
+
121
+ Тело перечитывается всякий раз, когда в ветку что-то влилось после публикации: PR
122
+ утверждает про дерево, а дерево с тех пор изменилось.
123
+
124
+ ## Образец тела PR
125
+
126
+ Четыре раздела, и порядок между ними один: строка связи, что сделано, чем подтверждено,
127
+ оставшийся шаг. Раздел, которому нечего сказать, пишется словами — пустой заголовок и снятый
128
+ заголовок читаются одинаково, а значат разное.
129
+
130
+ ```markdown
131
+ Closes #86
132
+
133
+ ## Что сделано
134
+
135
+ - <правка, названная тем, что она меняет для читателя, а не тем, какие файлы задела>
136
+
137
+ ## Чем подтверждено
138
+
139
+ - <проверка>: <её вывод одной строкой>
140
+ - Не гонялось: <что в набор не вошло и почему>
141
+
142
+ ## Оставшийся шаг
143
+
144
+ После одобрения ветка получает ещё один коммит — разбор папки задачи, — и только потом
145
+ вливается. До этого коммита вливать рано: папка уедет в главную ветку неразобранной.
146
+ ```
147
+
148
+ Раздел «Оставшийся шаг» стоит последним и переписывается тем же вызовом, что и остальное тело,
149
+ — в тот ход, которым папка разбирается и снимается черновик:
150
+
151
+ ```markdown
152
+ ## Оставшийся шаг
153
+
154
+ Не осталось: папка задачи разобрана коммитом `<sha>`, черновик снят. Можно вливать.
155
+ ```
156
+
157
+ Стоит он там потому, что решение о слиянии принимается на этой странице, а не в переписке:
158
+ сказанное владельцу вслух живёт до следующей реплики, а тело лежит у самой кнопки. Одно другого
159
+ не отменяет — порядок обоих сообщений владельцу описывает паттерн закрытия работы.
160
+
161
+ Проверить тело машиной нечем: ни одна сверка его не читает, а хостинг спрашивает только про
162
+ заголовок. Держится образец тем, кто пишет тело, — как и слова вслух.
163
+
164
+ ## Состояние PR читается, а не додумывается
165
+
166
+ Вызовы, которыми ставятся ревьювер, метки и исполнитель, отвечают нулевым кодом и тогда, когда
167
+ ничего не сделали: запрос разбора на автора PR GitHub молча выбрасывает. Поэтому после них PR
168
+ перечитывают:
169
+
170
+ ```bash
171
+ # Ключи латиницей: кириллический ключ без кавычек `jq` не разбирает и падает на нём
172
+ $GH api "repos/$REPO/pulls/321" \
173
+ --jq '{author: .user.login, reviewers: [.requested_reviewers[].login], labels: [.labels[].name]}'
174
+ ```
175
+
176
+ Автор здесь — `<бот>`. Если им оказался владелец, ревьювера у PR не будет вовсе:
177
+ назначить автора ревьювером нельзя, а отказа на такой запрос не приходит. Владельцу называют
178
+ то, что прочитали, а не то, что заказывали.
179
+
180
+ Учётная запись, из-под которой пришлось пушить, в этот вызов не переносится: пуш и авторство
181
+ PR выбираются отдельно, и `GH_TOKEN` для публикации — всегда токен бота.
182
+
183
+ Перечитывают его и по времени, а не только после вызовов, которые молча ничего не сделали:
184
+ состояние PR читается перед тем, как что-либо о нём сказать. Между «прогон зелёный» и следующей
185
+ фразой владелец успевает влить PR, и всё сказанное о нём после этого — про вчерашний день. Так
186
+ владельцу и было предложено влить то, что он влил часом раньше.
187
+
188
+ Открытый PR означает, что задача ждёт разбора, — колонка переставляется тем же движением:
189
+
190
+ ```bash
191
+ npm run task:move -- 86 in-review
192
+ ```
193
+
194
+ Голым GraphQL по идентификаторам проекта, элемента и варианта поля это не пишется: команда
195
+ знает их сама, а собранный по памяти запрос молча ставит не ту колонку — отказа у борды на
196
+ это нет.
197
+
198
+ ## Что проверяется до публикации PR
199
+
200
+ Проверок на самом PR нет: выкатка запускается пушем в главную ветку, и до мержа никто не
201
+ гоняет ничего. Линтеры, юниты и сценарии хуков снимает гейт пуша — ниже то, чего он не знает.
202
+
203
+ 1. **Главная ветка влита в эту ветку** — `git fetch origin && git merge origin/main`.
204
+ Свежесть основания между ветками одного захода не наследуется: вторая и третья ветка
205
+ отводятся после своего `fetch`, а не от ссылки, подтянутой под первую. Пока идёт работа,
206
+ главная уходит вперёд — чаще всего собственным PR того же исполнителя, влитым час
207
+ назад.
208
+ Всё, что проверяется ниже, проверяется от этого основания: PR с разошедшейся ветки
209
+ показывает ревьюверу правку вперемешку с чужой. Порядок и разбор конфликта — паттерн
210
+ `git-workflow-merge`.
211
+ 2. **В ветке только та правка, за которой её заводили** — `git diff main...HEAD --stat`. Чужой
212
+ домен в списке файлов означает, что правка расползлась, и её надо вернуть в свои границы.
213
+ 3. **Ни мока, ни подменённого ответа, ни отладочной строки** — `git diff main...HEAD` читается
214
+ целиком, а не по именам файлов. На прод они уезжают молча и портят настоящие данные.
215
+ 4. **Документ едет тем же коммитом.** Пару называет `docs-guard`, но спек домена и правку его
216
+ поведения он не знает — это остаётся за автором.
217
+ 5. **Проверки текстов и раскладки зелёные** — те, что дерево завело в `tools/`: сверка
218
+ сценариев с тестами, путей в документах, раскладки либ, повторов и классов без правила.
219
+ Какие именно есть здесь — `implementation.md` правила.
220
+ 6. **Все приложения дерева собираются** — `nx build` по каждому. Гейт пуша сборку не гоняет:
221
+ она длиннее всего, что он успевает сделать между командой и пушем.
222
+ 7. **Видимый текст заведён во всех локалях перевода** — тестом полноты словарей, если дерево
223
+ переводится.
224
+ 8. **Правка вёрстки подтверждена замером**, а не взглядом, и снята при узком экране — паттерн
225
+ `browser-verification-measure`.
226
+ 9. **Правка разметки публичного сайта проверена на прод-сборке по всем локалям перевода** — паттерн
227
+ `seo-verify`.
228
+ 10. **Тело PR собрано по образцу** — начинается строкой `Closes #<номер>`, несёт разделы «Что
229
+ сделано», «Чем подтверждено» и «Оставшийся шаг», а метки, ревьювер и исполнитель стоят.
230
+ Раздел оставшегося шага к этому моменту говорит, что шагов не осталось: черновик снимается
231
+ после разбора папки, а не до него.
232
+ 11. **Заголовок PR несёт номер задачи и называет её сделанной:** `[<КЛЮЧ>-<номер>] <Что сделано>`,
233
+ тем же номером, что стоит у задачи и в имени ветки. Инфинитив из задачи в него не
234
+ переносится, тип и область коммита — тоже.
235
+ 12. **Очередь работ сходится** — `npm run check:board`. Задача на борде, с исполнителем и
236
+ номером в заголовке; PR один на задачу, и закрывает он её целиком.
237
+ 13. **Состояние PR прочитано, а не выведено из кодов возврата** — автор `<бот>`,
238
+ ревьювер — владелец, метки те же, что у задачи. Владельцу называют прочитанное.
239
+ 14. **Набор взят из файла конвейера, а не собран по памяти.** Гейт пуша заведомо уже: он стоит
240
+ между командой и пушем, и всё, что дольше секунд, из него вынесено. Что гоняет конвейер,
241
+ написано в его файле — этот список и повторяется локально; зелёный гейт полнотой набора не
242
+ является.
243
+ 15. **Набор пересмотрен после вливания главной ветки.** Он выбирается по тому, что ветка везёт
244
+ теперь, а не по тому, что правил автор. Ветка, не тронувшая ни строки показа, прогоняет
245
+ снимки витрин: с момента вливания их гоняет конвейер на её коде, и красное придёт на её
246
+ PR.
247
+
248
+ Список этот — про снятие черновика, а не про его открытие. Черновиком PR открывается раньше:
249
+ пока работа идёт, человеку показывают то, что уже есть, вместе с тем, чего ещё нет. Пункты 1–15
250
+ проходятся перед `gh pr ready`, и невыполненный пункт означает, что черновик не снимается, — а
251
+ не то, что PR не открывается.
252
+
253
+ Сразу после публикации задача переставляется в разбор — `npm run task:move -- <номер>
254
+ in-review`, — и `npm run check:board` прогоняется ещё раз: до открытия PR колонку он не судит,
255
+ а после открытия расхождение видит.
256
+
257
+ Сделанное рассуждением и сделанное замером в теле PR разводятся прямо: непроверенное,
258
+ названное проверенным, ревьювер принимает за проверенное.
259
+
260
+ **Раздел «Чем подтверждено» называет и то, что не гонялось.** Список одного прогнанного
261
+ неотличим от полного набора, и ревьювер по нему решает, что можно не перепроверять. Цена ошибки
262
+ здесь не красный конвейер, а доверие к разделу: однажды прочитанный как полнота, дальше он
263
+ перепроверяется весь.
264
+
265
+ ## Частые промахи
266
+
267
+ Промахи про заведение задачи, ветку и коммит — паттерн
268
+ `git-workflow-commit`.
269
+
270
+ - Второй строкой `Closes` в одном PR задача больше не закрывается: две задачи в одной ветке
271
+ откатываются только вместе. Либо это одна задача — и вторая поглощается, — либо две ветки.
272
+ - Половина задачи, уехавшая своим PR, тоже промах: тело такого PR начинается со слов «Часть
273
+ #<номер>» вместо `Closes`, задача остаётся открытой, и после отката видно её целой. Работа,
274
+ которая в одну ветку не влезает, делится на задачи до того, как ветка заводится.
275
+ - PR открыт без ревьювера: он не попадает во входящие владельца вовсе, и очередь стоит,
276
+ выглядя работающей. Так шестнадцать PR ждали разбора, которого никто не запрашивал.
277
+ - Метки поставлены по названию PR, а не прочитаны у задачи: область теряется, и по борде не
278
+ видно, что правка задела ещё и сайт.
279
+ - Задача закрыта не полностью, а метки перенесены целиком: тикет остаётся открытым, и это
280
+ говорится в теле PR, а не подразумевается строкой `Closes`.