docs(skills): добавлены TDD по выбору и линза запахов Фаулера
- Зачем: - Исполнитель и ревьюер получают полезные приемы по месту применения. - Что: - Добавлена условная ссылка на /tdd по имени скилла в поручении исполнителю. - Из code-review Мэтта Покока перенесены названия запахов как подсказки для суждения. - Подключена линза сопровождаемости с учетом правил проекта и соразмерности изменений. - Проверка: - Сверены двенадцать названий запахов с исходным скиллом. - Проверены 61 относительная ссылка, 11 якорей и git diff --check.
This commit is contained in:
1 parent
79617075d9
commit
af8632274b
2 files changed
+20
No files matched your search
@@ -76,6 +76,9 @@ your available context, raise that while there is still room to explain the
|
|||||||
remaining work.
|
remaining work.
|
||||||
```
|
```
|
||||||
|
|
||||||
|
Если для задачи уместен TDD, укажи в поручении скилл `/tdd` по имени,
|
||||||
|
ожидаемое поведение и уже согласованные границы проверки.
|
||||||
|
|
||||||
## Независимое ревью
|
## Независимое ревью
|
||||||
|
|
||||||
Параметры обзора должны назвать:
|
Параметры обзора должны назвать:
|
||||||
@@ -108,6 +111,20 @@ shape with it, identify decisions left open by the statement, and judge each
|
|||||||
decision against repository values, risk, and task proportion.
|
decision against repository values, risk, and task proportion.
|
||||||
```
|
```
|
||||||
|
|
||||||
|
Если линия оценивает структуру и сопровождаемость кода, добавь этот фокус.
|
||||||
|
Названия запахов взяты из `/code-review` Мэтта Покока (Фаулер, _Refactoring_,
|
||||||
|
гл. 3):
|
||||||
|
|
||||||
|
```text
|
||||||
|
Use Fowler's code smells as prompts for judgment: Mysterious Name,
|
||||||
|
Duplicated Code, Feature Envy, Data Clumps, Primitive Obsession,
|
||||||
|
Repeated Switches, Shotgun Surgery, Divergent Change, Speculative Generality,
|
||||||
|
Message Chains, Middle Man, Refused Bequest.
|
||||||
|
Treat them as heuristics; documented repository conventions take precedence.
|
||||||
|
For a finding, explain the maintenance consequences in this project and
|
||||||
|
judge the proposed remedy against those costs and the task's scope.
|
||||||
|
```
|
||||||
|
|
||||||
## Разбор и перепроверка
|
## Разбор и перепроверка
|
||||||
|
|
||||||
Исполнитель разбирает реализацию по
|
Исполнитель разбирает реализацию по
|
||||||
|
|||||||
@@ -21,6 +21,9 @@
|
|||||||
достаточно, если он закрывает существенный риск. Две сфокусированные линии
|
достаточно, если он закрывает существенный риск. Две сфокусированные линии
|
||||||
оправданы, когда их независимый вклад стоит цены.
|
оправданы, когда их независимый вклад стоит цены.
|
||||||
|
|
||||||
|
Для ревью формы и сопровождаемости бери подходящую линзу из
|
||||||
|
[шаблонов поручений](prompt-templates.md#независимое-ревью).
|
||||||
|
|
||||||
Для двух линий заранее раздели фокус, например:
|
Для двух линий заранее раздели фокус, например:
|
||||||
|
|
||||||
- соответствие задаче, границам и наблюдаемому результату;
|
- соответствие задаче, границам и наблюдаемому результату;
|
||||||
|
|||||||
Reference in new issue
Block a user