@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.
- package/assets/checks/check-file-size.mjs +19 -4
- package/assets/checks/check-state-next.mjs +202 -0
- package/assets/checks/check-states.mjs +142 -0
- package/assets/checks/check-turn-map.mjs +146 -0
- package/assets/checks/rt-kit-checks.config.mjs +16 -2
- package/assets/defaults/project.sh +67 -7
- package/assets/defaults/turn-map.md +48 -0
- package/assets/hooks/browser-device-id.sh +2 -0
- package/assets/hooks/browser-guard-device-id.sh +5 -1
- package/assets/hooks/browser-guard-no-asking.sh +5 -1
- package/assets/hooks/browser-guard-no-listing.sh +2 -0
- package/assets/hooks/browser-guard-no-other-drivers.sh +7 -3
- package/assets/hooks/browser-guard-require-select.sh +6 -2
- package/assets/hooks/claim-guard.sh +5 -1
- package/assets/hooks/commit-msg.sh +2 -0
- package/assets/hooks/conscience-guard.sh +5 -1
- package/assets/hooks/constitution-index.sh +2 -0
- package/assets/hooks/dev-server-guard.sh +7 -3
- package/assets/hooks/dispatch.sh +69 -0
- package/assets/hooks/docs-guard.sh +8 -4
- package/assets/hooks/exam-guard.sh +7 -3
- package/assets/hooks/git-guard-delivery-signature.sh +2 -0
- package/assets/hooks/git-guard-delivery.sh +39 -5
- package/assets/hooks/git-guard-main.sh +8 -4
- package/assets/hooks/git-guard-push-tests.sh +8 -4
- package/assets/hooks/glossary-load.sh +2 -0
- package/assets/hooks/grill-gate.sh +6 -2
- package/assets/hooks/handoff-entry-guard.sh +6 -2
- package/assets/hooks/handoff-write.sh +124 -0
- package/assets/hooks/hook-input.sh +54 -0
- package/assets/hooks/lint-after-edit.sh +7 -3
- package/assets/hooks/observe.sh +2 -0
- package/assets/hooks/postmortem-guard.sh +5 -1
- package/assets/hooks/proposal-guard.sh +5 -1
- package/assets/hooks/prose-style-guard.sh +7 -3
- package/assets/hooks/qa-dataid-guard.sh +6 -2
- package/assets/hooks/rerun-guard.sh +7 -3
- package/assets/hooks/reuse-first-guard.sh +7 -3
- package/assets/hooks/roles.sh +2 -0
- package/assets/hooks/rule-article.sh +99 -0
- package/assets/hooks/skill-gate-layers.sh +2 -0
- package/assets/hooks/skill-gate-rearm.sh +5 -1
- package/assets/hooks/skill-gate.sh +25 -2
- package/assets/hooks/skill-loaded.sh +5 -1
- package/assets/hooks/sql-guard-parse.sh +2 -0
- package/assets/hooks/sql-guard-request.sh +4 -1
- package/assets/hooks/sql-guard-target.sh +2 -0
- package/assets/hooks/sql-guard-write.sh +2 -0
- package/assets/hooks/sql-guard.sh +6 -2
- package/assets/hooks/task-context-load.sh +2 -0
- package/assets/hooks/task-flow-guard.sh +8 -4
- package/assets/hooks/turn-entry-load.sh +62 -0
- package/assets/hooks/turn-exit-guard.sh +44 -17
- package/assets/hooks/utf8.sh +35 -0
- package/assets/hooks/waiting-turn-guard.sh +5 -1
- package/assets/hooks/window-fill-guard.sh +37 -5
- package/assets/laws/work-conduct.md +34 -9
- package/assets/patterns/dependencies-upgrade.md +1 -1
- package/assets/patterns/doc-style-write.md +3 -3
- package/assets/patterns/git-workflow-commit.azure.md +2 -202
- package/assets/patterns/git-workflow-commit.github.md +2 -258
- package/assets/patterns/git-workflow-commit.gitlab.md +1 -217
- package/assets/patterns/git-workflow-docker.md +3 -3
- package/assets/patterns/git-workflow-merge.md +3 -2
- package/assets/patterns/git-workflow-migration.md +3 -3
- package/assets/patterns/git-workflow-pr.azure.md +224 -0
- package/assets/patterns/git-workflow-pr.github.md +280 -0
- package/assets/patterns/git-workflow-pr.gitlab.md +240 -0
- package/assets/patterns/git-workflow-restart.md +3 -3
- package/assets/patterns/git-workflow-secrets.md +3 -3
- package/assets/patterns/task-flow-archive.md +193 -0
- package/assets/patterns/task-flow-close.md +11 -160
- package/assets/patterns/task-flow-handoff.md +22 -4
- package/assets/patterns/task-flow-resume.md +6 -0
- package/assets/patterns/task-flow-start.md +41 -0
- package/assets/patterns/turn-entry-map.md +81 -0
- package/assets/pitfalls/doc-style.md +80 -0
- package/assets/pitfalls/git-workflow.azure.md +50 -0
- package/assets/pitfalls/git-workflow.github.md +78 -0
- package/assets/pitfalls/git-workflow.gitlab.md +49 -0
- package/assets/pitfalls/spec-driven.md +36 -0
- package/assets/pitfalls/styling-bem.md +45 -0
- package/assets/pitfalls/task-flow.md +62 -0
- package/assets/pitfalls/testing.md +70 -0
- package/assets/rules/deploy-flow.azure.md +106 -0
- package/assets/rules/deploy-flow.github.md +113 -0
- package/assets/rules/deploy-flow.gitlab.md +108 -0
- package/assets/rules/doc-style.md +25 -76
- package/assets/rules/git-workflow.azure.md +6 -92
- package/assets/rules/git-workflow.github.md +14 -127
- package/assets/rules/git-workflow.gitlab.md +6 -93
- package/assets/rules/spec-driven.md +39 -30
- package/assets/rules/styling-bem.md +20 -39
- package/assets/rules/task-flow.md +17 -186
- package/assets/rules/testing.md +3 -64
- package/assets/rules/turn-conduct.md +206 -0
- package/assets/rules/turn-entry.md +93 -0
- package/assets/rules/typescript-conventions.md +15 -0
- package/assets/skills/agent-kit.md +35 -12
- package/assets/templates/pitfalls.md +10 -0
- package/assets/templates/rule.md +5 -3
- package/lib/assets.d.ts.map +1 -1
- package/lib/assets.js +6 -1
- package/lib/assets.js.map +1 -1
- package/lib/cargo.d.ts +42 -0
- package/lib/cargo.d.ts.map +1 -1
- package/lib/cargo.js +2 -0
- package/lib/cargo.js.map +1 -1
- package/lib/cascade.d.ts.map +1 -1
- package/lib/cascade.js +19 -1
- package/lib/cascade.js.map +1 -1
- package/lib/commands.d.ts.map +1 -1
- package/lib/commands.js +65 -4
- package/lib/commands.js.map +1 -1
- package/lib/config.d.ts +16 -1
- package/lib/config.d.ts.map +1 -1
- package/lib/config.js +8 -0
- package/lib/config.js.map +1 -1
- package/lib/hooks-map.d.ts +13 -0
- package/lib/hooks-map.d.ts.map +1 -1
- package/lib/hooks-map.js +33 -1
- package/lib/hooks-map.js.map +1 -1
- package/lib/observations.d.ts +35 -1
- package/lib/observations.d.ts.map +1 -1
- package/lib/observations.js +14 -2
- package/lib/observations.js.map +1 -1
- package/lib/ship.d.ts +2 -0
- package/lib/ship.d.ts.map +1 -1
- package/lib/ship.js +2 -0
- package/lib/ship.js.map +1 -1
- package/lib/thresholds.d.ts +49 -0
- package/lib/thresholds.d.ts.map +1 -0
- package/lib/thresholds.js +151 -0
- package/lib/thresholds.js.map +1 -0
- package/package.json +1 -1
- package/rt-tools-agent-kit-0.12.0.tgz +0 -0
- package/assets/commands/agent-kit-digest.md +0 -88
- package/assets/commands/rules-review.md +0 -98
- package/rt-tools-agent-kit-0.10.0.tgz +0 -0
|
@@ -11,6 +11,9 @@ description: Правило под «Закон о документации пр
|
|
|
11
11
|
быть верно про тексты; здесь — из каких слоёв они сложены в этом дереве и что сверяет машина.
|
|
12
12
|
Формулировки — правило `doc-style` под тем же законом.
|
|
13
13
|
|
|
14
|
+
**Холодная часть:** `pitfalls.md` рядом — ловушки, грабли, на которые уже наступали.
|
|
15
|
+
Грузится по требованию, а не вместе с правилом.
|
|
16
|
+
|
|
14
17
|
## Как это называется здесь
|
|
15
18
|
|
|
16
19
|
```
|
|
@@ -21,6 +24,7 @@ description: Правило под «Закон о документации пр
|
|
|
21
24
|
├─ ПРАВИЛО .claude/skills/<правило>/SKILL.md (kind: rule, law: <закон>)
|
|
22
25
|
│ привязывает закон к этому проекту; несколько правил на закон
|
|
23
26
|
│ .claude/skills/<правило>/implementation.md — привязка к коду
|
|
27
|
+
│ .claude/skills/<правило>/pitfalls.md — холодная часть: грузится по требованию
|
|
24
28
|
│
|
|
25
29
|
│ └─ ПАТТЕРН .claude/skills/<правило>-<что>/SKILL.md (kind: pattern, rule: <правило>)
|
|
26
30
|
│ готовый код и конкретные приёмы; минимум один на правило
|
|
@@ -125,6 +129,13 @@ flowchart TD
|
|
|
125
129
|
- **Слоёв законов два, а имя закона одно на оба.** Общий лежит в корне конституции, закон
|
|
126
130
|
приложения — в `application/`; ни `law:`, ни `**Законы:**` слоя не называют, поэтому имена
|
|
127
131
|
законов уникальны по всему дереву конституции.
|
|
132
|
+
- **У правила бывает третий файл, и в него уходит то, что при решении не читают.** Ловушки и
|
|
133
|
+
поведение по разборам происшествий нужны не тому, кто принимает обычное решение, а тому, кто
|
|
134
|
+
разбирает промах или спорит с гардом, — а грузятся они вместе с правилом каждый раз и растут
|
|
135
|
+
быстрее статей. Такой текст уезжает в `pitfalls.md` рядом, правило называет его строкой в
|
|
136
|
+
шапке, и грузится он по требованию. Отказ гейта о холодной части молчит: он зовёт правило, а о
|
|
137
|
+
третьем файле говорит само правило.
|
|
138
|
+
|
|
128
139
|
- **Спек объявляет законы, которые применяет, и связь сверяется в обе стороны.** Закон,
|
|
129
140
|
названный в тексте спека, обязан стоять в шапке: иначе по закону не узнать, какие домены
|
|
130
141
|
на нём стоят.
|
|
@@ -134,6 +145,28 @@ flowchart TD
|
|
|
134
145
|
про дерево, где того гарда не разложили; поправить это дерево не может ничем, если у ресурса
|
|
135
146
|
нет надстройки. Требование ресурса к ресурсу при этом объявляется строкой в шапке, а не
|
|
136
147
|
выводится из такой фразы.
|
|
148
|
+
- **Статья правила говорит о своей применимости сама, строкой признака при себе.** Отказ гейта
|
|
149
|
+
зовёт правило целиком, а под конкретную правку подпадает одна его статья: платить за решение
|
|
150
|
+
полной ценой правила — значит учить не читать лишнего, то есть работать хуже разведанным.
|
|
151
|
+
Признак стоит при статье, а не в карте гейта: карта знает путь и правило, но не знает, какая
|
|
152
|
+
из двух десятков статей про этот путь.
|
|
153
|
+
|
|
154
|
+
```markdown
|
|
155
|
+
- **Заголовок статьи.** Текст статьи, как обычно.
|
|
156
|
+
<!-- rt-when: *.scss *.css -->
|
|
157
|
+
```
|
|
158
|
+
|
|
159
|
+
Образцы разделены пробелом и сверяются с путём правки как образцы оболочки, а не поиском по
|
|
160
|
+
словам: поиск называет не ту статью и молчит об этом. Образец без каталога сверяется и с
|
|
161
|
+
именем файла. Комментарий не виден в собранной разметке, поэтому статья читается человеком
|
|
162
|
+
как прежде.
|
|
163
|
+
|
|
164
|
+
- **Статья без признака законна, и правило без единого признака — тоже.** Признак ставится тем
|
|
165
|
+
статьям, чьё правило отбивает на правке файла; остальные размечаются по мере того, как их
|
|
166
|
+
отбития попадут в сводку. Отсутствие признака означает «эту статью по пути правки не
|
|
167
|
+
выбирают», а не промах: требовать его у всех значило бы размечать наугад те правила, которые
|
|
168
|
+
ни разу никого не отбили.
|
|
169
|
+
|
|
137
170
|
- **Паттерн находится по полю `rule:`, а не по приставке имени.** Приставку имени несут не все
|
|
138
171
|
паттерны, и поиск по имени правила таких не видит: сверка ищет их полем, человек — разделом
|
|
139
172
|
«Паттерны» самого правила. Счёт паттернов, собранный приставками, выходит меньше настоящего, а
|
|
@@ -161,6 +194,12 @@ flowchart TD
|
|
|
161
194
|
доменов раньше, чем код научился в него приходить, и всё это время читалось описанием
|
|
162
195
|
работающего.
|
|
163
196
|
|
|
197
|
+
Полноту разметки статей признаком применимости не считает ничто. Правило, у которого признака
|
|
198
|
+
нет ни у одной статьи, отбивает прежним текстом — и от размеченного отличается только тем, что
|
|
199
|
+
исполнитель читает его целиком; ни одна проверка об этом не говорит. Судит это сводка
|
|
200
|
+
наблюдений: правило, которое чаще прочего отбивает на правке файла, и есть первое, что
|
|
201
|
+
размечают.
|
|
202
|
+
|
|
164
203
|
Полноту разделов у самого закона, правила и паттерна здесь не проверяет ничто. Статьи о том,
|
|
165
204
|
что текст судится не слабее своей копии, что невыбранная редакция судится наравне с выбранной и
|
|
166
205
|
что набор разделов объявлен отдельно от образца, исполняются там, где эти тексты пишут, — в
|
|
@@ -172,33 +211,3 @@ flowchart TD
|
|
|
172
211
|
|
|
173
212
|
- `spec-driven-domain` — заведение и правка спека домена, сценарии, привязка.
|
|
174
213
|
- `spec-driven-rule` — заведение закона, правила и паттерна.
|
|
175
|
-
|
|
176
|
-
## Ловушки
|
|
177
|
-
|
|
178
|
-
- **`tasks.md` в спеке не заводить.** Шаги — артефакт сессии, им место в ветке или в описании
|
|
179
|
-
PR. Как только в директории появляются «шаги», спек снова становится планом и умирает после
|
|
180
|
-
мержа.
|
|
181
|
-
- **Спек описывает установившееся, а не предстоящее.** Единственное место, где он говорит о
|
|
182
|
-
будущем, — `proposed/<фича>/`. После выкатки его текст вливается в спек домена, директория
|
|
183
|
-
удаляется, идентификаторы сценариев не меняются.
|
|
184
|
-
- **Семантику полей не сверяет ничто.** Проверка знает имена процедур, коды отказа и связь
|
|
185
|
-
сценариев с тестами; что означает пустое поле — не знает. Правка `.proto` поэтому тянет
|
|
186
|
-
спеки всех доменов, чьи процедуры она задела, в той же ветке.
|
|
187
|
-
- **Живость символа считается совпадением имени по всему дереву, а не вызовом.** Символу
|
|
188
|
-
хватает второго упоминания где угодно — в чужом поле с тем же именем, в атрибуте разметки.
|
|
189
|
-
Место, где правило исполняется на самом деле, подтверждается только чтением кода.
|
|
190
|
-
- **Якорь в `tools/*.mjs` сверяется почти ничем:** живость считается только для `.ts`, а
|
|
191
|
-
исходники обходятся по `apps`, `libs` и `prisma`. Правило, привязанное к проверке, поэтому
|
|
192
|
-
читается вместе с её телом.
|
|
193
|
-
- **Зелёная проверка не значит, что структура верна.** Спутники с привязкой сначала лежали
|
|
194
|
-
рядом с законами, и проверка была зелёной именно потому, что структура совпадала с тем,
|
|
195
|
-
чего проверка сама и ждала.
|
|
196
|
-
- **Конфликт мержа в спеке разрешается сохранением обеих сторон, а не выбором одной.** Две
|
|
197
|
-
ветки дописывают в конец одних и тех же списков — сценариев, правил, строк привязки, — и
|
|
198
|
-
обе стороны верны: конфликт здесь не спор, а две дописи в одно место. Номера сценариев при
|
|
199
|
-
разрешении не пересчитываются: идентификатор — ключ связи с тестами, и сдвиг номеров рвёт
|
|
200
|
-
сверку у соседей, которых правка не касалась. Порядок сохранённых сторон держится
|
|
201
|
-
одинаковым в `spec.md`, `scenarios.md` и `implementation.md`: иначе правило, его сценарий и
|
|
202
|
-
его привязка перестают находиться друг по другу. После разрешения гоняется
|
|
203
|
-
`npm run check:specs` — конфликт в спеке кода не задевает, и ни сборка, ни линтеры его не
|
|
204
|
-
увидят.
|
|
@@ -12,6 +12,9 @@ description: Правило под «Закон о фронтовом прило
|
|
|
12
12
|
`component-structure`, состояние — `angular-patterns`, окружение браузера —
|
|
13
13
|
`platform-access`, слой обращения к серверу — `api-layer`. Все пять под одним законом.
|
|
14
14
|
|
|
15
|
+
**Холодная часть:** `pitfalls.md` рядом — ловушки, грабли, на которые уже наступали.
|
|
16
|
+
Грузится по требованию, а не вместе с правилом.
|
|
17
|
+
|
|
15
18
|
## Как это называется здесь
|
|
16
19
|
|
|
17
20
|
| В законе | Здесь |
|
|
@@ -51,28 +54,45 @@ flowchart TD
|
|
|
51
54
|
- **Оформление берётся токеном `--rt-*`, а не пишется значением на месте.** Составные
|
|
52
55
|
значения — `box-shadow`, `text-shadow` — берутся готовым токеном целиком, а не собираются
|
|
53
56
|
из частей.
|
|
57
|
+
<!-- rt-when: *.scss *.css -->
|
|
58
|
+
|
|
54
59
|
- **У каждого класса элемента есть своё правило стилей.** Класс без правила выглядит рабочим
|
|
55
60
|
и молча ничего не делает.
|
|
61
|
+
<!-- rt-when: *.scss *.css -->
|
|
62
|
+
|
|
56
63
|
- **Класс ставится директивой, а не строкой в атрибуте.** Имя блока `rtElem` получает
|
|
57
64
|
инъекцией от ближайшего предка с `rtBlock`, и повторить этот разбор по тексту шаблона нечем.
|
|
65
|
+
<!-- rt-when: *.html *.scss -->
|
|
66
|
+
|
|
58
67
|
- **Раскладка объявлена в общем слое приложения, а не в стилях экрана.** У компонента экрана
|
|
59
68
|
вне кита файл стилей по умолчанию пустой.
|
|
69
|
+
<!-- rt-when: *.scss *.css -->
|
|
70
|
+
|
|
60
71
|
- **Шторку и окно открывает служба кита, а не номер слоя.** Числа шкалы сравниваются только
|
|
61
72
|
между соседями по разметке; служба выносит разметку наружу, к `<body>`, и сравнивать её
|
|
62
73
|
становится не с чем.
|
|
74
|
+
<!-- rt-when: *.ts *.scss -->
|
|
75
|
+
|
|
63
76
|
- **Номер слоя берётся из шкалы, а не пишется числом в файле компонента.** Шкала — единственное
|
|
64
77
|
место, где слои видно рядом: написанное на месте число в неё не попадает, и следующий узел
|
|
65
78
|
занимает тот же номер, ничего об этом не узнав.
|
|
79
|
+
<!-- rt-when: *.scss *.css -->
|
|
80
|
+
|
|
66
81
|
- **Размер элемента управления выбирается по признаку указателя, а не по ширине экрана.**
|
|
67
82
|
Планшет в ландшафте шире порога узкого вьюпорта, а нажимают по нему пальцем: `pointer: coarse`
|
|
68
83
|
отвечает про способ нажатия, ширина — про место под раскладку. Ступени берутся у кита, а не
|
|
69
84
|
назначаются пикселями.
|
|
85
|
+
<!-- rt-when: *.scss *.css -->
|
|
86
|
+
|
|
70
87
|
- **Файл стилей не длиннее 500 строк.** Предел общий с кодом и текстами, но stylelint длину не
|
|
71
88
|
судит вовсе — держит его проверка дерева. Выросший файл экрана делится по блокам, а общая
|
|
72
89
|
раскладка уходит в свой слой.
|
|
90
|
+
<!-- rt-when: *.scss *.css -->
|
|
91
|
+
|
|
73
92
|
- **Предупреждение stylelint роняет прогон наравне с ошибкой.** `!important` объявлен
|
|
74
93
|
предупреждением, а прогон идёт с `--max-warnings 0`: иначе запрет читается как пожелание —
|
|
75
94
|
два таких предупреждения лежали в дереве, а `npm run stylelint` возвращал ноль и гейтом не был.
|
|
95
|
+
<!-- rt-when: *.scss *.css -->
|
|
76
96
|
|
|
77
97
|
## Чего из закона здесь нет
|
|
78
98
|
|
|
@@ -85,42 +105,3 @@ flowchart TD
|
|
|
85
105
|
- `styling-bem-layout` — экран на общем слое раскладки, блоки приложения.
|
|
86
106
|
- `styling-bem-component` — стили компонента кита, `:host`, модификаторы, язык оформления сайта.
|
|
87
107
|
- `styling-bem-sheet` — шторка и окно поверх страницы: чем открываются, подложка, замер.
|
|
88
|
-
|
|
89
|
-
## Ловушки
|
|
90
|
-
|
|
91
|
-
- **`rtElem` без предка с `rtBlock` роняет отрисовку в рантайме** — сборка и линт молчат.
|
|
92
|
-
- **Спроецированный узел блока-предка не имеет.** `rtElem` берёт имя блока инъекцией от
|
|
93
|
-
ближайшего предка **по месту объявления шаблона**, а не по месту вставки: элемент, который
|
|
94
|
-
экран объявляет у себя и отдаёт в проекцию чужого компонента, ищет `rtBlock` в своём шаблоне
|
|
95
|
-
и не находит. Отрисовка падает в рантайме, сборка и линт зелёные. Класс на такой узел
|
|
96
|
-
вешается правилом по селектору кита в общем слое раскладки, а не директивой.
|
|
97
|
-
- **Элемент с `backdrop-filter` или своим `z-index` замыкает потомков в свой слой.** Липкая
|
|
98
|
-
шапка с размытием — самый частый случай: номер слоя у того, что лежит внутри неё,
|
|
99
|
-
сравнивается не с соседями по странице, а только с соседями внутри шапки, и нижняя панель
|
|
100
|
-
накрывает открытую шторку вместе с её кнопкой. Проверяется это `elementFromPoint` в центре
|
|
101
|
-
кнопки: сборка, линт и скриншот показывают тут целую страницу.
|
|
102
|
-
- **До узла, вынесенного к `<body>`, стили компонента не достают.** Превью и заглушку переноса
|
|
103
|
-
кладёт туда библиотека, а правила компонента заскоуплены атрибутом: файл выглядит рабочим и
|
|
104
|
-
не красит ничего. Такие правила объявляются в общем слое приложения. Ни сборка, ни линт, ни
|
|
105
|
-
проверка «класс без правила» этого не видят: класса такого в шаблоне нет вовсе, и пролежать
|
|
106
|
-
это может несколько задач подряд.
|
|
107
|
-
- **Имя токена не сверяется ничем.** Ссылка на несуществующий токен собирается, проходит
|
|
108
|
-
stylelint и проверку класса без правила, а свойство молча берёт наследованное значение:
|
|
109
|
-
правило выглядит написанным и не красит ничего. Ловится это только замером в браузере, а
|
|
110
|
-
имена берутся из объявлений кита, а не по догадке о том, как токен должен был бы называться.
|
|
111
|
-
- **`rtBlock` на `<ng-container>` класса не ставит вовсе:** узел это комментарий, и имя блока
|
|
112
|
-
он только объявляет потомкам. Класс блока экрана вешает хост через `host: { class: … }`.
|
|
113
|
-
- **`justify-content: center` во flex-контейнере с `overflow-x` уводит первые элементы за
|
|
114
|
-
нулевой скролл** — доскроллить до них невозможно. В прокручиваемых лентах —
|
|
115
|
-
`justify-content: safe center`.
|
|
116
|
-
- **`scrollbar-gutter: stable` на корне не заводить:** резерв под полосу прокрутки сужает
|
|
117
|
-
содержащий блок для `position: fixed`, и попап, выровненный по правому краю, встаёт на
|
|
118
|
-
ширину резерва левее своей кнопки.
|
|
119
|
-
- **`& + :host` невалиден:** изнутри компонента до соседнего хоста не дотянуться. Разделитель
|
|
120
|
-
между повторяющимися хостами — `:host(:not(:first-of-type))`.
|
|
121
|
-
- **`[attr.aria-disabled]` визуального состояния не даёт:** браузер стилизует `:disabled`, но
|
|
122
|
-
атрибуты `aria-*` — нет. К каждому `aria-disabled` заводится правило `[aria-disabled='true']`.
|
|
123
|
-
- **Гарнитуру с `body` элементы формы не наследуют:** браузер задаёт `button`, `input`,
|
|
124
|
-
`select` и `textarea` свой шрифт. Наследование включено глобально — сбрасывать его нельзя.
|
|
125
|
-
- **Комментарии-выключатели stylelint не ставятся.** Селекторы объединяются вложенностью.
|
|
126
|
-
- **При переносе стилей новых объявлений не появляется** — только перемещение существующих.
|
|
@@ -2,7 +2,7 @@
|
|
|
2
2
|
name: task-flow
|
|
3
3
|
kind: rule
|
|
4
4
|
law: work-conduct
|
|
5
|
-
description: Правило под «Закон о ведении
|
|
5
|
+
description: Правило под «Закон о ведении работы» — та его часть, что про ход работы от просьбы владельца до слияния. Брать в начале любой работы от владельца, при правке папок задач и договорённостей о продукте, при возвращении к незаконченной задаче. Называет разбор просьбы до первой правки, папку задачи по имени ветки, состояния работы и их обязательные действия, договорённость о продукте до кода, разбор папки при закрытии. Готовый порядок — в паттернах task-flow-start, task-flow-resume, task-flow-close и task-flow-archive. Чем кончается ход и что стерегут стражи — правило turn-conduct.
|
|
6
6
|
---
|
|
7
7
|
|
|
8
8
|
# Ведение работы — как это устроено здесь
|
|
@@ -11,6 +11,10 @@ description: Правило под «Закон о ведении работы»
|
|
|
11
11
|
про ход работы; здесь — чем это названо в этом дереве, где лежит и что из закона у нас не
|
|
12
12
|
проверяется.
|
|
13
13
|
|
|
14
|
+
**Холодная часть:** `pitfalls.md` рядом — ловушки и поведение по разборам происшествий.
|
|
15
|
+
Грузится по требованию, а не вместе с правилом: при обычном решении она не нужна — она нужна
|
|
16
|
+
тому, кто разбирает промах или спорит с гардом.
|
|
17
|
+
|
|
14
18
|
**Требует:** `hooks/task-flow-guard.sh`, `hooks/task-context-load.sh`, `hooks/grill-gate.sh`, `hooks/window-fill-guard.sh`, `hooks/turn-exit-guard.sh`
|
|
15
19
|
|
|
16
20
|
## Как это называется здесь
|
|
@@ -40,12 +44,8 @@ description: Правило под «Закон о ведении работы»
|
|
|
40
44
|
пока действие не сделано, работа стоит в том же состоянии. Список шагов этого не давал: шаг
|
|
41
45
|
кончался, а что делать дальше, выводилось из соседних строк.
|
|
42
46
|
|
|
43
|
-
Состояние объявляется
|
|
44
|
-
перезаписывается вместе с
|
|
45
|
-
|
|
46
|
-
```markdown
|
|
47
|
-
- **Состояние:** `этап-идёт`
|
|
48
|
-
```
|
|
47
|
+
Состояние объявляется в разделе «Где стоим» хода работы машиночитаемой строкой
|
|
48
|
+
``- **Состояние:** `этап-идёт` `` и перезаписывается вместе с ним.
|
|
49
49
|
|
|
50
50
|
| Состояние | Вход в него | Обязательное действие | Ведёт паттерн |
|
|
51
51
|
| ------------------------- | ----------------------------------------------- | ------------------------------------------------------ | ------------------ |
|
|
@@ -58,41 +58,20 @@ description: Правило под «Закон о ведении работы»
|
|
|
58
58
|
| `этапы-кончились` | все этапы отмечены | прогнать набор и открыть PR черновиком | `task-flow-close` |
|
|
59
59
|
| `работа-отдана` | PR открыт черновиком | взять следующую задачу | `task-flow-resume` |
|
|
60
60
|
| `разбор-кончился` | прогон зелёный, замечаний нет | влить договорённость, привести тексты, разобрать папку | `task-flow-close` |
|
|
61
|
-
| `папка-разобрана` | папки в ветке нет, запись в архиве есть | снять черновик и попросить влить | `task-flow-
|
|
62
|
-
| `влито` | PR слит человеком | разбор работы правилами и сверка очереди | `task-flow-
|
|
61
|
+
| `папка-разобрана` | папки в ветке нет, запись в архиве есть | снять черновик и попросить влить | `task-flow-archive` |
|
|
62
|
+
| `влито` | PR слит человеком | разбор работы правилами и сверка очереди | `task-flow-archive` |
|
|
63
63
|
|
|
64
64
|
Ни у одного состояния обязательное действие не звучит как «ждать»: ожидание чужого шага
|
|
65
|
-
состоянием работы не
|
|
66
|
-
взгляда быстрее не
|
|
67
|
-
следующую задачу, а не на открытый PR.
|
|
68
|
-
|
|
69
|
-
|
|
70
|
-
|
|
71
|
-
Ведёт закрытие захода паттерн `task-flow-handoff`.
|
|
65
|
+
состоянием работы не является — прогон, разбор владельцем и слияние идут без исполнителя и от
|
|
66
|
+
взгляда быстрее не становятся, поэтому в `работа-отдана` обязательное действие смотрит на
|
|
67
|
+
следующую задачу, а не на открытый PR. Заполнение окна захода состоянием тоже не бывает: заход
|
|
68
|
+
кончается передачей, работа остаётся в том состоянии, в каком стояла, а следующий заход читает
|
|
69
|
+
строку и продолжает с неё. Чем ход кончается и что его концом не бывает — правило
|
|
70
|
+
`turn-conduct` под тем же законом.
|
|
72
71
|
|
|
73
72
|
Перечень показывается владельцу в начале работы, и на нём же отмечается, где стоим: иначе после
|
|
74
73
|
шести вопросов не видно ни того, что будет дальше, ни сколько всего впереди.
|
|
75
74
|
|
|
76
|
-
## Выходы хода
|
|
77
|
-
|
|
78
|
-
Ход кончается четырьмя способами, и других нет:
|
|
79
|
-
|
|
80
|
-
| Выход | Чем подтверждается |
|
|
81
|
-
| -------------------------------------------------- | --------------------------------------------------------------- |
|
|
82
|
-
| вопрос владельцу, ответа на который в правилах нет | вопрос задан, и за тот же ход правила читались |
|
|
83
|
-
| отказ гарда | отказ назван владельцу, обход не искался |
|
|
84
|
-
| заполненное окно захода | ход работы дописан, передача написана |
|
|
85
|
-
| работа отдана, и следующая начата | PR открыт, и по следующей задаче сделано действие, а не сказано |
|
|
86
|
-
|
|
87
|
-
Всё остальное — продолжение хода, а не его конец. Веха ходом не кончается: ни коммит, ни
|
|
88
|
-
прочитанная договорённость, ни граница «прочитал — сейчас правлю», ни зелёная проверка. Слова
|
|
89
|
-
«иду дальше» и «работаю дальше» владелец читает как совершающееся действие, и писать их вместо
|
|
90
|
-
результата нельзя.
|
|
91
|
-
|
|
92
|
-
Три способа кончить ход выглядят работой и ею не являются: сводка о чужом шаге, меню при
|
|
93
|
-
назначенном порядке и объявление намерения. Что при этом должно быть верно — ниже, в статьях
|
|
94
|
-
о применении закона.
|
|
95
|
-
|
|
96
75
|
## Ход
|
|
97
76
|
|
|
98
77
|
Ход работы от просьбы владельца до закрытия: где стоит разбор, что требует гард и куда девается
|
|
@@ -175,104 +154,8 @@ flowchart TD
|
|
|
175
154
|
правку кода приложения, и работа, которая туда не доходит, проходит мимо него — но папку
|
|
176
155
|
заводит всё равно: статья правила говорит «под любую работу, без исключений». Прочитанный
|
|
177
156
|
как признак, гард становится разрешением работать без замысла везде, куда он не смотрит.
|
|
178
|
-
- **Сводка о чужом шаге.** Прогон, разбор владельцем и слияние идут без исполнителя и от взгляда
|
|
179
|
-
быстрее не становятся. Ход, кончившийся такой сводкой, владелец читает как работу: она полна,
|
|
180
|
-
в ней названы номера и состояния, и пустоты за ней не видно. О чужом шаге говорят вместе с
|
|
181
|
-
начатым своим, а не вместо него. За один заход это было нарушено четырежды, и готовая работа
|
|
182
|
-
простояла в невлитом PR почти три часа.
|
|
183
|
-
- **Меню при назначенном порядке.** Выбор, предложенный владельцу, пока эпик не кончился, — это
|
|
184
|
-
просьба назначить порядок заново. Работы в эпике не осталось — так и говорится: эпик
|
|
185
|
-
кончился, — а не «чем займёмся».
|
|
186
|
-
- **Объявление намерения.** «Беру следующую задачу» — не то же самое, что взять её: фраза живёт
|
|
187
|
-
до конца хода, а работа не двигается. Названо может быть только сделанное: номер заведённой
|
|
188
|
-
задачи, имя заведённой ветки, переведённая колонка.
|
|
189
|
-
|
|
190
|
-
- **Прерывание работы владельцем называется вслух.** Пришло задание, останавливающее начатое, —
|
|
191
|
-
исполнитель говорит, что стоит, на чём остановлено и что будет с прежней работой, и только
|
|
192
|
-
потом берётся за новое. Молчание об этом владелец читает как «прежнее кончилось».
|
|
193
|
-
- **Остановка называется отдельной репликой.** Не строкой в конце отчёта: там она тонет —
|
|
194
|
-
владелец читает отчёт как рассказ о сделанном. Называются три вещи: что стоит, чего оно ждёт
|
|
195
|
-
и что владелец может решить.
|
|
196
|
-
- **Выходы хода стережёт страж, а не память исполнителя.** Он читает объявленное состояние
|
|
197
|
-
работы и то, что за ход по ней сделано: правку файла или команду, меняющую дерево. Ход, в
|
|
198
|
-
котором не было ни того ни другого, возвращается исполнителю вместе со следующим шагом из
|
|
199
|
-
хода работы. Отданную и влитую работу страж не судит: она уже дождалась чужого шага.
|
|
200
|
-
- **Этап замысла объявляется закрытым только после того, как его команда проверки прошла.**
|
|
201
|
-
Строка «Чем проверяется» несёт команду обратными кавычками и то, что в её выводе означает
|
|
202
|
-
«сошлось». Страж читает прежний номер этапа из истории ветки и не выпускает ход, в котором
|
|
203
|
-
номер вырос, а команда не запускалась: через заход отмеченное по памяти неотличимо от
|
|
204
|
-
проверенного.
|
|
205
|
-
- **Слово об остановке страж читает у владельца, а не у исполнителя.** Иначе остановку
|
|
206
|
-
объявляет тот, кому она в эту минуту удобна, и запрет держится ровно до первого неудобства.
|
|
207
|
-
- **Утверждение о состоянии дерева стережёт гард утверждения, а не память исполнителя.** Всё,
|
|
208
|
-
что ответ владельцу говорит о дереве, несёт команду и её вывод: сказанное без команды
|
|
209
|
-
утверждением не считается — ни «проверено», ни «снято», ни «готово». Гард читает текст,
|
|
210
|
-
сказанный владельцу за ход, и ищет команду того же хода; у каждого слова назван свой род
|
|
211
|
-
команды, потому что общий признак «команда была» подтверждал бы одно другим. Прошлый ход не
|
|
212
|
-
годится: состояние дерева меняется, и вчерашний вывод о нынешнем молчит. Восемь разборов
|
|
213
|
-
подряд пришлись на этот промах, и каждый раз в правило дописывалась ещё одна статья —
|
|
214
|
-
держит его теперь машина.
|
|
215
|
-
- **Слово-утверждение гард ловит, неверный вывод — нет.** Об образце, судимом по одному его
|
|
216
|
-
файлу, и о пути, которым человек не пойдёт, судить нечем: там нет ни слова, ни команды, с
|
|
217
|
-
которой сверять. Это известная граница гарда, и держат её статьи ниже, а не он.
|
|
218
|
-
- **Ход о чужом шаге стережёт гард ожидания, а не память исполнителя.** Он отбивает завершение
|
|
219
|
-
хода, в котором о чужом шаге сказано, а по следующей задаче не сделано ни одного действия —
|
|
220
|
-
ни заведения задачи, ни ветки, ни папки, ни перевода колонки. Чужой шаг он узнаёт по двум
|
|
221
|
-
признакам: в ходе открыт PR либо в ходе прочитан красный прогон. Оба берутся из самого хода:
|
|
222
|
-
спросить хостинг было бы точнее, но сетевой вызов на завершении хода падает вместе со связью
|
|
223
|
-
и отбивал бы работу вместо промаха, а вывод команды о прогоне в записи хода уже лежит. Слова
|
|
224
|
-
«беру следующую задачу» гард действием не считает — ровно потому, что их и произносят вместо
|
|
225
|
-
неё.
|
|
226
|
-
- **Конец прогона узнаётся возвратом фоновой команды, а не взглядом на страницу.** Ожидание,
|
|
227
|
-
запущенное в фоне отдельным ходом, возвращает исполнителя к PR само; до тех пор ход занят
|
|
228
|
-
следующей задачей. Взгляд на страницу этого не даёт: он либо повторяется вхолостую, либо не
|
|
229
|
-
повторяется вовсе, и оба исхода со стороны выглядят одинаково — работа не двигается.
|
|
230
|
-
- **Отказ гарда кончает ход.** Другого пути к отбитой правке не ищут: ни командой оболочки, ни
|
|
231
|
-
соседним инструментом, ни правкой самого гарда. Отбитая правка либо делается после того, как
|
|
232
|
-
условие отказа выполнено, либо не делается вовсе — и тогда владельцу называется отказ, а не
|
|
233
|
-
результат. Обход стоит дороже отказа: гард отбивает один файл, а обойдённый гард снимает
|
|
234
|
-
требование со всего дерева и молчит об этом. Держится это не только памятью — гарды судят и
|
|
235
|
-
команду оболочки, которая пишет файл.
|
|
236
|
-
- **Ход, в котором владельцу задан вопрос, не заканчивается, пока за этот же ход не читались
|
|
237
|
-
законы и правила.** Чтением считается любой из трёх путей: загрузка правила, чтение файла
|
|
238
|
-
законов или правил, поиск по ним. Отбивает гард разговора — на завершении хода, а не на
|
|
239
|
-
инструменте вопроса: спрашивают чаще прозой, чем меню. Найденное ложится в раздел «Что уже
|
|
240
|
-
сказано в правилах» разбора.
|
|
241
|
-
- **Действия, которые исполнитель не делает без слова владельца, перечислены в компаньоне
|
|
242
|
-
правила.** Список у каждого дерева свой — пакет знает только требование, чтобы список был
|
|
243
|
-
назван. Не названный, он выводится из общих слов, и «делай, что нужно по плану» становится
|
|
244
|
-
разрешением на пуш и правку общих документов заодно с коммитом. Оценка «это безопасно» списка
|
|
245
|
-
не заменяет: её назначает тот, кому она в эту минуту удобна, и она плывёт. За один заход одна
|
|
246
|
-
и та же команда была сначала слишком опасной, чтобы её позвать, а через два хода —
|
|
247
|
-
достаточно безопасной, чтобы позвать без спроса.
|
|
248
|
-
- **У отказа от необратимого действия есть безопасная часть, и она делается.** Требование
|
|
249
|
-
спросить владельца относится к действию, а не к ходу: работа, у которой отделима часть без
|
|
250
|
-
последствий, делится, а не откладывается целиком. Список вариантов, поданный вместо работы,
|
|
251
|
-
читается как работа — тем полнее, чем аккуратнее он составлен: он пронумерован, в нём названы
|
|
252
|
-
цифры, и именно поэтому пустота хода за ним не видна. Владельцу называется, что уже сделано и
|
|
253
|
-
что осталось за его словом, — а не выбор из вариантов вместо и того и другого.
|
|
254
|
-
- **Ход, в котором исполнитель признал промах, не заканчивается, пока записи о происшествии
|
|
255
|
-
нет.** Отбивает гард происшествия — на завершении хода: к моменту признания промах уже
|
|
256
|
-
случился, и ловить раньше нечего. Признание ловится набором образцов, а не пониманием смысла;
|
|
257
|
-
промах, признанный словами вне набора, гард пропускает, и это его известная граница, а не
|
|
258
|
-
обещание.
|
|
259
|
-
- **Заход, начатый с передачи, входит в работу тем же правилом, что и всякий другой.** Передача
|
|
260
|
-
лежит вне дерева, её не читает ни одна проверка, и написана она вчера: всё, что в ней стоит,
|
|
261
|
-
проверяется деревом. Порядок входа — четыре шага в паттерне возвращения; стережёт его гард, а
|
|
262
|
-
не память: порядок, записанный только словами, исполняется, пока о нём помнят.
|
|
263
|
-
- **Состояние незаконченной работы приходит в контекст на запуске сессии.** Замысел и ход
|
|
264
|
-
работы отдаются целиком, разбор просьбы — путём. Ветка вида `<КЛЮЧ>-*` без папки даёт
|
|
265
|
-
предупреждение с готовой командой, но сессию не рвёт.
|
|
266
157
|
- **Сделанное отмечается только в ходе работы.** «Где стоим» перезаписывается каждым заходом,
|
|
267
158
|
а не дописывается: это первое, что читает следующий заход.
|
|
268
|
-
- **Заполнение окна захода стережёт гард, а не память исполнителя.** На первом пороге он
|
|
269
|
-
напоминает выбирать точку остановки, на втором отбивает всё, кроме записи хода работы,
|
|
270
|
-
передачи и команд поставки. Размер окна и оба порога дерево задаёт само; не задавшее размера
|
|
271
|
-
стража не получает.
|
|
272
|
-
- **Заход закрывается передачей, которая лежит вне дерева.** Состояние работы коммитится ходом
|
|
273
|
-
работы, а передача его пересказывает для вставки в новый заход: рабочее дерево, ветка,
|
|
274
|
-
сделанное, следующий шаг и особенности захода. В историю она не едет — иначе рядом с ходом
|
|
275
|
-
работы заводится вторая запись об одном и том же.
|
|
276
159
|
- **Слово для нового понятия ищется в словаре дерева.** Общую часть словаря везёт пакет,
|
|
277
160
|
предметную дописывает дерево надстройкой; словарь уезжает в контекст на запуске сессии
|
|
278
161
|
целиком, поэтому «не читал» основанием не бывает.
|
|
@@ -373,57 +256,5 @@ flowchart TD
|
|
|
373
256
|
|
|
374
257
|
- `task-flow-start` — разведка, разбор, договорённость, замысел, задача и ветка.
|
|
375
258
|
- `task-flow-resume` — возвращение к незаконченной работе новым заходом.
|
|
376
|
-
- `task-flow-close` — вливание договорённости,
|
|
377
|
-
- `task-flow-
|
|
378
|
-
|
|
379
|
-
## Ловушки
|
|
380
|
-
|
|
381
|
-
- **Папка называется именем ветки, один в один.** Хук запуска ищет её по
|
|
382
|
-
`git branch --show-current`, и папка, названная иначе, не находится ничем: работа идёт с
|
|
383
|
-
пустым контекстом, а владельца просят пересказать то, что уже записано.
|
|
384
|
-
- **Разбор просьбы задним числом не переписывается.** Пересказ незаметно подгоняется под уже
|
|
385
|
-
сделанное, и сверять результат становится не с чем. Решение, изменённое по ходу, дописывается
|
|
386
|
-
в ход работы, а не правится в разборе.
|
|
387
|
-
- **Договорённость о продукте не кладётся в папку задачи.** Папка умирает с мержем, а
|
|
388
|
-
договорённость обязана его пережить: её сценарии получают номера в общей нумерации домена,
|
|
389
|
-
и на них ссылаются заголовки тестов. Обратное тоже верно — ход работы не кладётся в
|
|
390
|
-
`proposed/`: спек, в котором завелись шаги, снова становится планом и умирает после мержа.
|
|
391
|
-
- **У меню нет строки «вопрос не тот».** Меню годится для выбора значения из закрытого набора;
|
|
392
|
-
пока постановка вопроса не подтверждена, отвергнуть её владельцу нечем — он выбирает из
|
|
393
|
-
вариантов, выведенных из неверной посылки. Настройки владельца, требующие меню, требование
|
|
394
|
-
не снимают: тогда к каждому вопросу добавляется свободный вариант, и он же — единственное
|
|
395
|
-
место, где вопрос отвергается целиком. Три вопроса ушли одним меню, у одного постановка была
|
|
396
|
-
ложной, и сказать «вопрос не тот» было нечем. Выбор слова, имени и термина узким вопросом не
|
|
397
|
-
является никогда.
|
|
398
|
-
- **Субагент вопросов владельцу не задаёт.** Ни роли, ни конвейер до него не достучатся —
|
|
399
|
-
они возвращают текст главному агенту. Поэтому разбор ведёт главный агент, а роли стоят по
|
|
400
|
-
обе стороны от него.
|
|
401
|
-
- **Если дефект чинится правкой одного общего числа, спроси владельца, тем ли способом ты его
|
|
402
|
-
чинишь.** Замер показывает, что дефект ушёл, — но не то, что причину вылечили. В одной
|
|
403
|
-
задаче так ушли две правки подряд: сначала подняли общее число у соседнего узла, потом
|
|
404
|
-
перенесли узел в другое место разметки. Обе владелец отверг, а нужный способ назвал сам.
|
|
405
|
-
Спрашивают до правки, а не показывают замер после.
|
|
406
|
-
- **Путь, предложенный человеку, судится числом его шагов и тем, чем ему для этого надо
|
|
407
|
-
владеть.** Со стороны кода вариант выглядит дешёвым — «меньше путей», «строку запуска не
|
|
408
|
-
трогаем», — а человеку он стоит захода на сервер. Однажды рекомендуемым вариантом так стояла
|
|
409
|
-
выдача токена, за которой владельцу надо было идти по ssh в работающий контейнер; отбивал
|
|
410
|
-
этот вариант он сам. Цена называется со стороны того, кто пойдёт: сколько шагов и что ему для
|
|
411
|
-
них нужно. Пересказ действующего порядка без такой оценки владелец читает как одобрение.
|
|
412
|
-
- **Эпик по теме читается до того, как решается раскладка.** Замысел эпика держит решения,
|
|
413
|
-
которые пережили десяток задач, и разведка по коду их не находит: снятое решение следа в
|
|
414
|
-
дереве не оставляет. Домен, заведённый генератором и снесённый через полчаса, стоял в замысле
|
|
415
|
-
прямым запретом — но замысел открыли уже после того, как он был заведён во второй раз.
|
|
416
|
-
- **Работа, которая разбирает чужую папку задачи, разбирает и свою — одним коммитом.** Свою
|
|
417
|
-
папку она заводит наравне со всеми: исключения из этого требования нет. Круг, которым
|
|
418
|
-
исключение оправдывали, закрывается не отказом от папки, а порядком разбора: последний
|
|
419
|
-
коммит снимает обе, и после работы неубранного не остаётся. Однажды такой разбор оставил
|
|
420
|
-
свою папку, и на неё пришлось заводить третью задачу — лечится это порядком, а не правом
|
|
421
|
-
работать без замысла на диске. Как разобрать две папки — паттерн `task-flow-close`.
|
|
422
|
-
- **Слово для нового понятия берётся из `docs/GLOSSARY.md` или заводится там же.** Третий файл
|
|
423
|
-
папки задачи называется `progress.md`, а не `journal.md`, ровно поэтому: журнал в этом
|
|
424
|
-
дереве один, и он другой.
|
|
425
|
-
- **Черновик папки задачи называется тем же коротким именем, что и будущая ветка.** Команда
|
|
426
|
-
заведения ищет черновик по нему и, не найдя, молча собирает папку с образца: работа при этом
|
|
427
|
-
идёт дальше, а разбор просьбы остаётся лежать в брошенном каталоге, и следующий заход
|
|
428
|
-
расспрашивает владельца заново. Имя черновику дают словами просьбы, а ветке — терминологией
|
|
429
|
-
договорённости; те же слова, да не те же.
|
|
259
|
+
- `task-flow-close` — снятие черновика, вливание договорённости, приведение текстов.
|
|
260
|
+
- `task-flow-archive` — разбор папки задачи, переезд в описание прошлого, разбор правилами.
|
package/assets/rules/testing.md
CHANGED
|
@@ -12,6 +12,9 @@ description: Правило под «Закон о проверяемости».
|
|
|
12
12
|
применяется. Проверка работающего приложения глазами и замером — правило
|
|
13
13
|
`browser-verification` под тем же законом.
|
|
14
14
|
|
|
15
|
+
**Холодная часть:** `pitfalls.md` рядом — ловушки, грабли, на которые уже наступали.
|
|
16
|
+
Грузится по требованию, а не вместе с правилом.
|
|
17
|
+
|
|
15
18
|
## Как это называется здесь
|
|
16
19
|
|
|
17
20
|
| В законе | Здесь |
|
|
@@ -167,67 +170,3 @@ flowchart TD
|
|
|
167
170
|
- `testing-unit` — тест на чистую функцию, на процедуру Connect и разовый тест-доказательство,
|
|
168
171
|
который не коммитится.
|
|
169
172
|
- `testing-e2e` — прогон сквозных тестов, стенд под настоящим nginx, выключатели.
|
|
170
|
-
|
|
171
|
-
## Ловушки
|
|
172
|
-
|
|
173
|
-
- **Зелёный `nx test <проект>` не значит, что хоть один файл исполнялся.** Либа без своего
|
|
174
|
-
`vitest.config.mts` не запускает ничего — так тесты домена броней не запускались ни разу.
|
|
175
|
-
Либа с конфигом, но без единого `*.spec.ts`, проходит зелёной из-за `passWithNoTests: true`,
|
|
176
|
-
который обычно стоит в каждом конфиге дерева, и на глаз эти два случая неотличимы: в обоих
|
|
177
|
-
прогон успешен. Доля либ без единого теста меряется пересчётом ниже — в дереве, где его
|
|
178
|
-
завели впервые, она вышла почти в две трети. Перед правкой в
|
|
179
|
-
незнакомой либе проверяется, есть ли в ней хоть один `*.spec.ts`; если нет — первый
|
|
180
|
-
заводится этой же правкой, а не откладывается: откладывать здесь не с чего, долг уже
|
|
181
|
-
накоплен. Пересчёт: `for d in $(find libs -name vitest.config.mts -exec dirname {} \;); do
|
|
182
|
-
[ -z "$(find "$d" -name '*.spec.ts')" ] && echo "$d"; done | wc -l`.
|
|
183
|
-
- **Зелёная сводка покрытия не значит, что тесты проходят.** Сверка читает заголовки тестов и
|
|
184
|
-
сопоставляет их со сценариями спека; исполняется ли тест и чем он кончается — она не знает
|
|
185
|
-
вовсе, и падающий тест значится в ней покрытием. Три сценария одной панели падали и до правки
|
|
186
|
-
экрана, а нашлось это только прогоном. Перед правкой экрана его сквозные тесты гоняются один
|
|
187
|
-
раз до первой строки кода: иначе чужое падение читается как своя регрессия, а своё — как
|
|
188
|
-
чужое.
|
|
189
|
-
- **«Executable doesn't exist» — состояние машины, а не дефект правки.** Установлен только
|
|
190
|
-
chromium, `firefox` и `webkit` падают всегда: гонять `--project=chromium`, узкий экран —
|
|
191
|
-
`--project=mobile-chrome`. Та же ошибка приходит после смены версии Playwright: браузер
|
|
192
|
-
ставится под конкретную версию, и после подъёма нужен повторный
|
|
193
|
-
`npx playwright install chromium`. Девять тестов так и упали, и это выглядело регрессией
|
|
194
|
-
обновления.
|
|
195
|
-
- **Первому прогону сразу после установки браузера верить нельзя.** Два падения сквозного
|
|
196
|
-
набора не повторились ни при отдельном прогоне тех же тестов, ни при втором полном. Такой
|
|
197
|
-
прогон повторяют, а выводы делают по второму.
|
|
198
|
-
- Сквозная спека, которой нужен вход, без учётных данных в окружении пропускается молча — в
|
|
199
|
-
отчёте она значится `skipped`, и прогон выглядит успешным. Имена переменных — при дереве.
|
|
200
|
-
- **Справочник флоу вторых сценариев не заводит.** В `docs/E2E_<ДОМЕН>_FLOWS.md` кладут то,
|
|
201
|
-
чего в спеке домена нет и быть не должно: `qa-dataid` элементов, состояния разметки, ловушки
|
|
202
|
-
стенда. Обещанное поведение остаётся сценарием в `scenarios.md`: если списать его во второе
|
|
203
|
-
место, копии разойдутся молча — `npm run check:specs` этого не увидит.
|
|
204
|
-
- `npx nx serve` проверкой не считается: это шаг из правила `browser-verification`, а не тест.
|
|
205
|
-
- **Кадр, зависящий от загрузки машины, проверяет машину, а не вёрстку.** Ожидание отсчётом
|
|
206
|
-
времени этим и кончается: на свободной машине набор зелен целиком, на занятой падает, и какой
|
|
207
|
-
именно кадр не успел — дело случая. Лечится ожиданием события, а не удлинением отсчёта: шрифты
|
|
208
|
-
подняты, картинки нарисованы, движение остановлено, положение узла не менялось два кадра
|
|
209
|
-
подряд. Пока ожидание идёт по времени, «проверено снимками» означает «машина была свободна», и
|
|
210
|
-
перезапуск, давший зелёное, этого не отменяет, а прячет. Восемь кадров расходились с эталоном
|
|
211
|
-
на 0,15–0,74 % в задании конвейера и проходили на той же машине вне его.
|
|
212
|
-
- **Стенд, поднятый предыдущим шагом, останавливается перед съёмкой.** Оставленный работать, он
|
|
213
|
-
соревнуется за машину с тем, что снимают, и делает исход прогона зависящим от того, чем занят
|
|
214
|
-
сосед. Нагрузка, которую задание создаёт себе само — соседняя витрина, только что законченная
|
|
215
|
-
сборка, — ничем не отличается от чужой.
|
|
216
|
-
- **Свой стенд снимается перед тем, как звать набор.** Прогон переиспользует поднятое на его
|
|
217
|
-
портах, и стенд, оставленный для замера, отдаёт ему чужую сборку с чужими данными. Красное при
|
|
218
|
-
этом приходит не строкой про занятый порт, а десятком спек про экраны — то есть выглядит
|
|
219
|
-
дефектом правки: за один заход так покраснели сначала шесть новых тестов, потом гейт пуша, и
|
|
220
|
-
оба раза причиной был свой же стенд. Разобранный занятый порт эту сторону не закрывает: он про
|
|
221
|
-
чужой стенд, а этот — про свой.
|
|
222
|
-
- **Разбор упавшего кадра начинается с чисел, а не с картинки расхождения.** Доля площади
|
|
223
|
-
говорит, сколько разошлось, и молчит о том, что именно: сдвиг всего кадра на пиксель,
|
|
224
|
-
переставленные строки и рябь на сглаженных уголках выглядят на картинке одинаково — «стало
|
|
225
|
-
другим». Читаются координаты разошедшихся точек и величина расхождения по каналу: сдвинутые
|
|
226
|
-
границы блоков — это раскладка, разошедшийся текст при неподвижных границах — это данные,
|
|
227
|
-
единица-две по каналу на кривых краях — это цвет. Три расхождения одного набора разобрались
|
|
228
|
-
ровно так, и ни одно из трёх не оказалось дефектом экрана.
|
|
229
|
-
- **Ожидаемое значение теста не берётся из кода, который тест проверяет.** Вывезенное из
|
|
230
|
-
проверяемой либы, оно делает тест зелёным при любом значении: «колесо показывает пять строк»
|
|
231
|
-
сходится и тогда, когда строк стало три. Ожидаемое пишется числом в самой спеке рядом с
|
|
232
|
-
проверкой, а общий модуль сквозных спек держит приёмы — открыть, дождаться, снять со
|
|
233
|
-
страницы, — но не то, что от страницы ожидается.
|