@bonesofspring/ai-rules 0.2.14 → 0.2.16

package/presets/cursor/ios-swift/rules/README.md

@@ -1,34 +1,34 @@
-### Разница .mdc и .md в .cursor/rules
+### Difference between .mdc and .md in .cursor/rules
 
 #### .md
-Обычный markdown‑файл. Cursor видит его как документацию/контент, но не как правило, которое нужно автоматически применять.
+Ordinary markdown. Cursor treats it as documentation/content, not as a rule to apply automatically.
 
 #### .mdc
-Специальный формат Cursor Rule:
-- сверху обязателен YAML‑frontmatter с полями вроде description, globs, alwaysApply;
-- файл лежит в .cursor/rules/;
-- Cursor интерпретирует его как конфигурацию поведения ассистента для этого репо (автоматически подмешивает содержимое в контекст, когда правило подходит).
+Cursor Rule format:
+- YAML frontmatter at the top with fields such as description, globs, alwaysApply;
+- lives under `.cursor/rules/`;
+- Cursor treats it as assistant behavior configuration for this repo (injects content when the rule matches).
 
-- То есть .mdc = “markdown + config для Cursor”, .md = просто текст без управляющего смысла для ассистента.
+- So `.mdc` = “markdown + Cursor config”; `.md` = plain text with no control meaning for the assistant.
 
-### Пресет ios-swift — карта правил
+### Preset ios-swift — rule map
 
-**alwaysApply budget:** **≤3** always-on правила и **≤140** строк тела (после YAML) суммарно (stretch ≤120). Новые alwaysApply только вместе с сокращением существующего бюджета.
+**alwaysApply budget:** **≤3** always-on rules and **≤140** body lines (after YAML) total (stretch ≤120). New alwaysApply only together with shrinking the existing budget.
 
-| Файл | Скоуп | Тема |
+| File | Scope | Topic |
 |------|--------|------|
-| `ios-app-core.mdc` | **always** | Стек, структура, tips (+ pointers на tooling/build) |
-| `architecture-boundaries.mdc` | **always** | Границы Presentation / Domain / Data, DI |
-| `code-quality-and-refactoring.mdc` | **always** | Качество и лёгкий refactor |
+| `ios-app-core.mdc` | **always** | Stack, structure, tips (+ pointers to tooling/build) |
+| `architecture-boundaries.mdc` | **always** | Presentation / Domain / Data boundaries, DI |
+| `code-quality-and-refactoring.mdc` | **always** | Quality and light refactor |
 | `anti-sycophancy-discipline.mdc` | requestable | Full VERIFY/anti-sycophancy discipline (~17 rules) |
 | `design-guidance.mdc` | requestable | Smells/GoF (Fowler≤12, GoF≤8) |
-| `xcode-tooling.mdc` | requestable | Схемы, xcodebuild, SwiftLint/Format |
-| `post-change-build.mdc` | requestable | Сборка/lint после Swift edits (**обязателен** после code edits) |
-| `agent-team-intake.mdc` | requestable | Подсказка `/task` |
+| `xcode-tooling.mdc` | requestable | Schemes, xcodebuild, SwiftLint/Format |
+| `post-change-build.mdc` | requestable | Build/lint after Swift edits (**required** after code edits) |
+| `agent-team-intake.mdc` | requestable | `/task` hint |
 | `agent-team-orchestrator.mdc` | requestable | Pipeline, gates, build-verifier |
-| `feature-delivery-workflow.mdc` | requestable | Чеклист доставки фичи |
-| `code-review-mr.mdc` | requestable | Ревью MR/diff |
-| `reference-features.mdc` | globs Features/** | Эталоны Feature + dogfood |
+| `feature-delivery-workflow.mdc` | requestable | Feature delivery checklist |
+| `code-review-mr.mdc` | requestable | MR/diff review |
+| `reference-features.mdc` | globs Features/** | Reference Features + dogfood |
 | `technical-retro.mdc` | requestable | Alias → `/technical-retro` |
 | `preset-layering.mdc` (+ `preset-*`) | globs presets/** | Author meta (never alwaysApply) |
 | `swiftui-ui.mdc` | Presentation, DesignSystem | SwiftUI, a11y |
@@ -42,9 +42,9 @@
 | `tests-unit.mdc` | `*Tests` | Unit |
 | `tests-ui.mdc` | `*UITests` | XCUITest (+ pointer to xcuitest agents/skill) |
 
-**Всего topic `.mdc`:** 22 domain + 5 preset-meta (без README).
+**Topic `.mdc` total:** 22 domain + 5 preset-meta (excluding README).
 
-**Twin depth (Cursor↔Claude):** domain rules ≥15 body lines. Thin requestable **aliases** (`agent-team-intake`, `technical-retro`) могут быть короче — как в next (soft AC-19).
+**Twin depth (Cursor↔Claude):** domain rules ≥15 body lines. Thin requestable **aliases** (`agent-team-intake`, `technical-retro`) may be shorter — as in next (soft AC-19).
 
 ### Shared core → stack mapping
 
@@ -67,7 +67,7 @@ Claude map: `presets/claude/ios-swift/rules/README.md`. Taxonomy: `presets/_shar
 
 ### Presentation edit bundle
 
-При правке `**/Presentation/**` или `**/DesignSystem/**`: грузить `swiftui-ui` + `state-and-viewmodels` (+ `navigation-coordinators` при правке навигации). Не подмешивать orchestrator.
+When editing `**/Presentation/**` or `**/DesignSystem/**`: load `swiftui-ui` + `state-and-viewmodels` (+ `navigation-coordinators` when editing navigation). Do not pull in the orchestrator.
 
 ### Loading strategy
 
@@ -85,7 +85,7 @@ globs
      security, modules, conventions, tests-*
 ```
 
-Установка: `npx @bonesofspring/ai-rules init cursor --preset ios-swift`
+Install: `npx @bonesofspring/ai-rules init cursor --preset ios-swift`
 
 ## Metrics policy mapping
 

package/presets/cursor/ios-swift/rules/agent-team-intake.mdc

@@ -11,6 +11,6 @@ When the user message looks like a **work request** (implement, add, fix, refact
 
 1. Prefer **`/task <their request>`** or invoke **task-router** first.
 2. Do not jump straight to coding without router + pipeline when scope is non-trivial.
-3. Pure questions («как работает X», «объясни») — prefer agent **`codebase-analyzer`** (explain-as-is); trivial one-liners may be answered inline. Do not open `/task`.
+3. Pure questions (“how does X work”, “explain”) — prefer agent **`codebase-analyzer`** (explain-as-is); trivial one-liners may be answered inline. Do not open `/task`.
 
-Exceptions: user explicitly says «без pipeline», «просто сделай», or continues an active slug.
+Exceptions: user explicitly says “no pipeline”, “just do it”, or continues an active slug.

package/presets/cursor/ios-swift/rules/agent-team-orchestrator.mdc

@@ -9,7 +9,7 @@ alwaysApply: false
 
 Parent agent = **manager**. Router plans; specialists execute. Artifacts: `.cursor/team/tasks/<slug>/`.
 
-Work request без `/task` — см. **`agent-team-intake.mdc`**.
+Work request without `/task` — see **`agent-team-intake.mdc`**.
 
 ## Entry points
 
@@ -119,6 +119,6 @@ Honor optional `steps[i].model` (`cheap` \| `standard` \| `strong`); else agent
 
 ## Skip pipeline when
 
-- Pure questions («как работает X») — prefer **`codebase-analyzer`**; typo / one-file trivial fix; user «без pipeline»; single-line docs-only.
+- Pure questions (“how does X work?”) — prefer **`codebase-analyzer`**; typo / one-file trivial fix; user “no pipeline”; single-line docs-only.
 
 Borderline — **`agent-team-intake.mdc`**.

package/presets/cursor/ios-swift/rules/architecture-boundaries.mdc

@@ -1,34 +1,34 @@
 ---
-description: Архитектурные границы и правила зависимостей iOS приложения
+description: Architectural boundaries and dependency rules for an iOS app
 alwaysApply: true
 ---
 
-# Границы между слоями
+# Layer boundaries
 
 - **Presentation** (Views, ViewModels, DesignSystem):
-  - Может: Domain (entities, use case protocols), DesignSystem, Core (без UI).
-  - Не должен: импортировать Data (репозитории, DTO, API-клиенты); знать HTTP/Codable/эндпоинты; маппить DTO → domain.
+  - May: Domain (entities, use case protocols), DesignSystem, Core (non-UI).
+  - Must not: import Data (repositories, DTOs, API clients); know HTTP/Codable/endpoints; map DTO → domain.
 - **Domain**:
-  - Может: Core (базовые типы, протоколы).
-  - Не должен: импортировать SwiftUI, UIKit, сеть; знать конкретные реализации репозиториев.
+  - May: Core (base types, protocols).
+  - Must not: import SwiftUI, UIKit, networking; know concrete repository implementations.
 - **Data**:
-  - Может: Domain (протоколы, entities), Core.
-  - Не должен: импортировать SwiftUI/UIKit; зависеть от View/ViewModel.
+  - May: Domain (protocols, entities), Core.
+  - Must not: import SwiftUI/UIKit; depend on View/ViewModel.
 
-# Зависимости и модули
+# Dependencies and modules
 
-- Направление: **Presentation → Domain ← Data**.
-- Переиспользуемое — в Core / DesignSystem; к другой фиче — только **public API** (протокол, facade, coordinator), не deep-import.
-- SPM: Domain не линкуется с Data; реализации Data подключаются через DI в `App/`.
-- Раскладка фичи — как в `ios-app-core.mdc` / соседних `Features/*` репо (Presentation / Domain / Data + тесты).
+- Direction: **Presentation → Domain ← Data**.
+- Shared code — in Core / DesignSystem; to another feature — only **public API** (protocol, facade, coordinator), not deep-import.
+- SPM: Domain does not link Data; Data implementations are wired via DI in `App/`.
+- Feature layout — as in `ios-app-core.mdc` / neighboring `Features/*` in the repo (Presentation / Domain / Data + tests).
 
-# DI и composition root
+# DI and composition root
 
-- Сборка графа — в `App/` (или `DI/` / CompositionRoot), не во View.
-- ViewModel получает **протоколы** (use case / repository), не конкретные Data-типы.
-- Preview и тесты — mock/stub реализаций.
+- Graph assembly — in `App/` (or `DI/` / CompositionRoot), not in the View.
+- ViewModel receives **protocols** (use case / repository), not concrete Data types.
+- Previews and tests — mock/stub implementations.
 
-# Требование к агенту
+# Agent requirement
 
-- Файлы — в соответствующих слоях; протоколы репозиториев в Domain, реализации в Data.
-- Не «коротить» слои (API из View «ради скорости»).
+- Place files in the matching layers; repository protocols in Domain, implementations in Data.
+- Do not “short-circuit” layers (API from a View “for speed”).

package/presets/cursor/ios-swift/rules/code-quality-and-refactoring.mdc

@@ -1,52 +1,53 @@
 ---
-description: Поддержка и улучшение качества кода и архитектуры iOS
+description: Maintain and improve iOS code quality and architecture
 alwaysApply: true
 ---
 
 <!-- shared-core: quality/code-quality-and-refactoring.md -->
 
-# Поддержка существующего стиля
+# Match existing style
 
-- Следовать существующим паттернам (имена, структура, типизация / DI / error handling); минимизировать стилистический шум (лишний format/rename).
-- Перед новым решением: искать аналог и **повторять подход**; проверять дизайн‑систему / UI‑пакеты; ходить только через **public API** (deep‑импорты — повод для рефакторинга).
+- Follow existing patterns (names, structure, typing / DI / error handling); avoid drive-by formatting or renames.
+- Before a new solution: find an analog and **repeat the approach**; check the design system / UI packages; go only through the **public API** (deep imports are a reason to refactor).
 
-# Принципы
+# Principles
 
-- **KISS:** самое простое изменение в паттернах репозитория.
-- **DRY + Rule of Three:** общее — только после 3-го повторения (ошибочная абстракция дороже дублирования).
-- **YAGNI:** без API, флагов, слоёв «на будущее» без текущего требования.
-- **SOLID:** S — один повод менять; O — композиция/новые реализации; L — контракт подтипа; I — узкие порты; D — абстракции (порты), детали в адаптерах стека.
-- **CUPID:** Composable; Unix philosophy (одна ответственность); Predictable; Idiomatic (репо); Domain-based.
-- **Composition over inheritance;** Law of Demeter — без цепочек `a.b.c.d`; через public API / facade.
+- **KISS:** the simplest change in the repository’s patterns.
+- **DRY + Rule of Three:** share only after the 3rd repetition (wrong abstraction costs more than duplication).
+- **YAGNI:** no APIs, flags, or “for later” layers without a current requirement.
+- **SOLID:** S — one reason to change; O — composition/new implementations; L — subtype contract; I — narrow ports; D — abstractions (ports), details in stack adapters.
+- **CUPID:** Composable; Unix philosophy (one responsibility); Predictable; Idiomatic (repo); Domain-based.
+- **Composition over inheritance;** Law of Demeter — no `a.b.c.d` chains; via public API / facade.
 
-# Рефакторинг при изменениях
+# Refactoring during changes
 
-- Лёгкий refactor OK, если меньше дублирования, выше читаемость, без ломки публичных контрактов.
-- Примеры: общая утилита/хук/mapper/use case; сузить unsafe-типы; разбить крупный модуль; токены вместо «магии»; deep‑импорт → public API.
-- Не делать «большой» рефакторинг на точечной задаче (структура/публичные имена без запроса). Крупное — минимальными шагами; код остаётся собираемым.
+- Light refactor is OK if it reduces duplication, improves readability, and does not break public contracts.
+- Examples: shared utility/hook/mapper/use case; narrow unsafe types; split a large module; tokens instead of “magic”; deep import → public API.
+- Do not do a “big” refactor on a point task (structure/public names without a request). Break large work into minimal steps; keep the repo buildable after each step.
 
-# Требование к агенту
+# Agent requirement
 
-- **Boy scout:** оставлять модуль немного лучше; не жертвовать архитектурой и слоями ради краткости.
+- **Boy scout:** leave the module a bit better; do not sacrifice architecture and layers for brevity.
 
 ## Agent discipline (anti-sycophancy)
 
 Essentials below. Full procedural set (~17 rules) → load **`anti-sycophancy-discipline`**.
 
 - Unverifiable third-party API → mark `// VERIFY: lib.symbol @ version` (or stack equivalent); never invent signatures.
-- Before «is this correct?» — ≥3 failure modes (empty, boundary, concurrency/state) or name what you could not check.
+- Before "is this correct?" — ≥3 failure modes (empty, boundary, concurrency/state) or name what you could not check.
 - Refactor without tests → propose characterization test; if declined, label `UNTESTED`.
-- Comments = WHY only when non-obvious; ban self-referential «added for issue Y».
+- Comments = WHY only when non-obvious; ban self-referential "added for issue Y".
 - Urgency/authority: name the trade-off once, then comply — no apology loops.
 - Match verification to risk; report by what was **verified**, not attempted.
 
 Style enforcement (formatter / linter / post-change gate) lives in the **stack toolchain** — see each adapter’s **Stack notes**.
 
-> Stack-specific lint gates, примеры и toolchain — в секции **Stack notes** адаптера пресета.
+> Stack-specific lint gates, examples, and toolchain — in the preset adapter **Stack notes** section.
+
 ## Stack notes
 
-- Перед новым UI‑контролом проверять DesignSystem и существующие компоненты.
-- Public API: протоколы и facades между фичами.
-- Допустимые примеры: общий mapper/validation; заменить force unwrap; разбить перегруженный ViewModel на use cases; design tokens вместо magic numbers/colors; убрать прямую зависимость View от Data в пользу протокола.
-- Post-change gate: **`post-change-build.mdc`** + `xcode-tooling.mdc` (не yarn lint).
+- Before adding a new UI control, check DesignSystem and existing components.
+- Public API: protocols and facades between features.
+- Acceptable examples: shared mapper/validation; replace force unwrap; split an overloaded ViewModel into use cases; design tokens instead of magic numbers/colors; remove a direct View→Data dependency in favor of a protocol.
+- Post-change gate: **`post-change-build.mdc`** + `xcode-tooling.mdc` (not yarn lint).
 - Style: **SwiftLint** (+ SwiftFormat when configured) — authoritative style gate; do not duplicate linter rules in prose.

package/presets/cursor/ios-swift/rules/code-review-mr.mdc

@@ -1,5 +1,5 @@
 ---
-description: Требования к code review для iOS merge requests. Use when reviewing MR/diff/PR, evaluating a branch, or when the user asks for code review.
+description: Code review requirements for iOS merge requests. Use when reviewing MR/diff/PR, evaluating a branch, or when the user asks for code review.
 alwaysApply: false
 ---
 
@@ -22,9 +22,9 @@ On-demand rule for review flows. **Procedure and output format:** skill `code-re
 - UI conventions of the stack
 - Unit and e2e/UI test conventions
 - Post-change lint/build gate before final report
-- Design smells **in MR diff scope** — load `design-guidance`; не разворачивать полный каталог Fowler вне диффа
-- Pattern name (GoF) — только если репозиторий уже использует эту форму или match минимален
-- Principles (KISS / DRY+Ro3 / YAGNI / SOLID / CUPID) — через always-on `code-quality-and-refactoring`, не дублировать эссе в review
+- Design smells **in MR diff scope** — load `design-guidance`; do not expand the full Fowler catalog outside the diff
+- Pattern name (GoF) — only if the repository already uses that shape or the match is minimal
+- Principles (KISS / DRY+Ro3 / YAGNI / SOLID / CUPID) — via always-on `code-quality-and-refactoring`, do not duplicate the essay in review
 
 Concrete rule stems — in **Stack notes**.
 
@@ -33,7 +33,7 @@ Concrete rule stems — in **Stack notes**.
 - Focus on MR diff, not the whole repo.
 - Use local `git diff` — do not invent hosting metadata.
 - Boy scout rule: suggest fixes feasible within the MR scope.
-- No «большие рефакторинги» without an explicit request (`code-quality-and-refactoring`).
+- No “big refactors” without an explicit request (`code-quality-and-refactoring`).
 
 ## Output contract
 
@@ -46,39 +46,39 @@ Procedure details live in skill `code-review`. Required shape:
 
 ## Stack notes
 
-On-demand правило для review-потоков (не always-on).
-
-- **Когда применять**
-  - Запросы: «проведи ревью», «оцени MR/ветку/дифф», «посмотри изменения».
-  - Опираться на локальный репозиторий: ветка, `git diff`, открытые файлы.
-
-- **Что обязан проверить агент**
-  - **Архитектура и слои** (`architecture-boundaries.mdc`, `ios-app-core.mdc`):
-    - View не вызывает API и не знает DTO.
-    - ViewModel не зависит от Data-реализаций напрямую.
-    - Domain не импортирует SwiftUI/UIKit.
-    - Data не тянет Presentation.
-  - **Зависимости и модули**:
-    - DI через протоколы; composition root в App.
-    - Нет deep-import между фичами.
-    - Файлы в правильных слоях (`Presentation`, `Domain`, `Data`, `DesignSystem`).
-  - **Swift и concurrency** (`swift-conventions.mdc`):
-    - `@MainActor` на UI; нет необоснованных force unwrap.
-    - Typed errors; корректная отмена Task где нужно.
+On-demand rule for review flows (not always-on).
+
+- **When to apply**
+  - Requests: “review this”, “evaluate the MR/branch/diff”, “look at the changes”.
+  - Rely on the local repository: branch, `git diff`, open files.
+
+- **What the agent must check**
+  - **Architecture and layers** (`architecture-boundaries.mdc`, `ios-app-core.mdc`):
+    - View does not call APIs and does not know DTOs.
+    - ViewModel does not depend on Data implementations directly.
+    - Domain does not import SwiftUI/UIKit.
+    - Data does not pull Presentation.
+  - **Dependencies and modules**:
+    - DI via protocols; composition root in App.
+    - No deep-import between features.
+    - Files in the correct layers (`Presentation`, `Domain`, `Data`, `DesignSystem`).
+  - **Swift and concurrency** (`swift-conventions.mdc`):
+    - `@MainActor` on UI; no unjustified force unwraps.
+    - Typed errors; correct Task cancellation where needed.
   - **UI** (`swiftui-ui.mdc`):
-    - Тонкие Views; Design System tokens; accessibility identifiers для тестируемых элементов.
-  - **Сеть и data** (`networking-services.mdc`):
-    - DTO отделены от domain; mappers покрывают edge cases.
-  - **Тесты**:
-    - Нетривиальная логика — unit (`tests-unit.mdc`); пользовательские потоки — UI (`tests-ui.mdc`).
-    - Указать, какие тесты добавить или обновить.
-
-- **Глубина и формат**
-  - Фокус на **диффе MR**, не на всём проекте.
-  - Сначала обзор (цель MR, риски), затем конкретные комментарии с файлом и предложением fix по паттернам репо.
-  - Без «больших рефакторингов» без запроса (`code-quality-and-refactoring.mdc`).
-
-- **Ограничения**
-  - Не выдумывать метаданные CI/MR с хостинга без локальных данных.
-  - Boy scout rule в рамках MR.
-  - BUGBOT automated review — см. корневой `BUGBOT.md` (Cursor).
+    - Thin Views; Design System tokens; accessibility identifiers for testable elements.
+  - **Networking and data** (`networking-services.mdc`):
+    - DTOs separated from domain; mappers cover edge cases.
+  - **Tests**:
+    - Non-trivial logic — unit (`tests-unit.mdc`); user flows — UI (`tests-ui.mdc`).
+    - State which tests to add or update.
+
+- **Depth and format**
+  - Focus on the **MR diff**, not the whole project.
+  - Overview first (MR goal, risks), then concrete comments with file and a fix suggestion matching repo patterns.
+  - No “big refactors” without a request (`code-quality-and-refactoring.mdc`).
+
+- **Constraints**
+  - Do not invent CI/MR hosting metadata without local data.
+  - Boy scout rule within the MR.
+  - BUGBOT automated review — see root `BUGBOT.md` (Cursor).

package/presets/cursor/ios-swift/rules/design-guidance.mdc

@@ -9,98 +9,98 @@ alwaysApply: false
 
 # Design guidance (smells & patterns)
 
-Словарь **code smells** (Fowler) и **design patterns** (GoF) для агентов при implement / review / debug.
+Glossary of **code smells** (Fowler) and **design patterns** (GoF) for agents during implement / review / debug.
 
-Применять только в scope задачи или MR. Лёгкий refactor — да; «большой» rewrite без явного запроса — нет.
+Apply only within the task or MR scope. Light refactor — yes; a “big” rewrite without an explicit request — no.
 
-Называть паттерн **только если** репозиторий уже так делает или это минимальный match. Иначе — extract/split/move без ярлыка GoF.
+Name a pattern **only if** the repository already does it that way or the match is minimal. Otherwise — extract/split/move without a GoF label.
 
-Не заменяет always-on Principles в `code-quality-and-refactoring` (KISS, DRY + Rule of Three, YAGNI, SOLID, CUPID, composition, Demeter).
+Does not replace always-on Principles in `code-quality-and-refactoring` (KISS, DRY + Rule of Three, YAGNI, SOLID, CUPID, composition, Demeter).
 
 ## When to load
 
-- **Feature delivery / implement** — оценка структуры модулей, границ и дублирования.
-- **Code review** — smells и pattern fit **в диффе MR**, не по всему репо.
-- **Debug** — root-cause похож на wrong abstraction, Feature Envy, Message Chains, Shotgun Surgery.
-- **Light refactor only** — улучшения в рамках boy-scout / MR scope; не расширять задачу ради каталога.
+- **Feature delivery / implement** — assess module structure, boundaries, and duplication.
+- **Code review** — smells and pattern fit **in the MR diff**, not across the whole repo.
+- **Debug** — root cause looks like wrong abstraction, Feature Envy, Message Chains, Shotgun Surgery.
+- **Light refactor only** — improvements within boy-scout / MR scope; do not expand the task for the catalog.
 
-Агенты и skills подгружают stem по one-liner wiring — не копировать этот каталог в промпт агента.
+Agents and skills load this stem via one-liner wiring — do not copy this catalog into the agent prompt.
 
 ## How to use (agent rules of thumb)
 
-1. Сначала сверить с Principles (always-on): проще ли решение, нет ли ранней абстракции, соблюдён ли public API.
-2. Затем отметить smells **в изменённых файлах** задачи/MR; предложить fix только если он укладывается в light refactor.
-3. Паттерн GoF — опциональный ярлык после того, как структура уже ясна; не начинать дизайн с выбора паттерна.
-4. Если smell и паттерн конфликтуют с границами слоёв стека — побеждают stack boundaries и existing repo patterns.
-5. Не требовать переименования «под GoF», если поведение уже корректно и читаемо.
+1. First check against Principles (always-on): is the solution simpler, is there no premature abstraction, is the public API respected.
+2. Then mark smells **in the changed files** of the task/MR; propose a fix only if it fits a light refactor.
+3. A GoF pattern is an optional label after the structure is already clear; do not start design by picking a pattern.
+4. If a smell or pattern conflicts with stack layer boundaries — stack boundaries and existing repo patterns win.
+5. Do not require renaming “for GoF” if behavior is already correct and readable.
 
 ## Code smells (Fowler, curated ≤12)
 
-Таблица: smell → detect hint → fix hint (одна линия каждый). Без механических меню рефакторинга.
+Table: smell → detect hint → fix hint (one line each). No mechanical refactoring menus.
 
 | # | Smell | Detect | Fix hint |
 |---|-------|--------|----------|
-| 1 | Duplicate Code | одинаковая логика в ≥2 местах | dedupe после Rule of Three |
-| 2 | Long Method / Long Module | единица делает слишком много | extract в рамках scope; без big rewrite |
-| 3 | Feature Envy | метод чаще трогает чужие данные, чем свои | сдвинуть логику к данным / public API |
-| 4 | Data Clumps | одни и те же группы аргументов кочуют вместе | сгруппировать в type / value object |
-| 5 | Primitive Obsession | сырые string/int вместо доменных понятий | typed ids / value objects при нужде домена |
-| 6 | Speculative Generality | неиспользуемые хуки, флаги, абстракции «на будущее» | удалить (YAGNI) |
-| 7 | Message Chains | цепочки `a.b.c.d` | Demeter; facade / public API модуля |
-| 8 | Middle Man | класс почти только прокидывает вызовы | убрать pass-through **или** оставить как boundary |
-| 9 | Divergent Change | один модуль меняется по разным причинам | split по поводам изменения |
-| 10 | Shotgun Surgery | одно смысловое изменение разъезжается по многим файлам | consolidate ответственность |
-| 11 | Inappropriate Intimacy | доступ к чужим internals / private деталям | только public API / ports / facades |
-| 12 | Refused Bequest / Parallel Inheritance | подтип игнорирует базу; зеркальные иерархии классов | composition; не плодить параллельные деревья |
-
-**Skip Fowler (не в core):** Lazy Class, Incomplete Library Class, Comments-as-smell dogma, полный каталог 70+, пошаговые recipe-меню.
-
-Приоритет при конфликте smells: Inappropriate Intimacy и Message Chains (границы модулей) > Duplicate Code (после Rule of Three) > Long Method внутри уже правильного модуля.
+| 1 | Duplicate Code | same logic in ≥2 places | dedupe after Rule of Three |
+| 2 | Long Method / Long Module | unit does too much | extract within scope; no big rewrite |
+| 3 | Feature Envy | method touches foreign data more than its own | move logic toward the data / public API |
+| 4 | Data Clumps | the same argument groups travel together | group into a type / value object |
+| 5 | Primitive Obsession | raw string/int instead of domain concepts | typed ids / value objects when the domain needs them |
+| 6 | Speculative Generality | unused hooks, flags, “for later” abstractions | remove (YAGNI) |
+| 7 | Message Chains | `a.b.c.d` chains | Demeter; facade / module public API |
+| 8 | Middle Man | class almost only forwards calls | remove pass-through **or** keep as a boundary |
+| 9 | Divergent Change | one module changes for different reasons | split by reasons to change |
+| 10 | Shotgun Surgery | one semantic change spreads across many files | consolidate responsibility |
+| 11 | Inappropriate Intimacy | access to foreign internals / private details | public API / ports / facades only |
+| 12 | Refused Bequest / Parallel Inheritance | subtype ignores base; mirrored class hierarchies | composition; do not grow parallel trees |
+
+**Skip Fowler (not in core):** Lazy Class, Incomplete Library Class, Comments-as-smell dogma, full 70+ catalog, step-by-step recipe menus.
+
+Smell conflict priority: Inappropriate Intimacy and Message Chains (module boundaries) > Duplicate Code (after Rule of Three) > Long Method inside an already correct module.
 
 ## Design patterns (GoF, curated ≤8)
 
-Suggest only when repo already uses the shape or the match is minimal. Prefer extract-function over naming Strategy ради ярлыка.
+Suggest only when repo already uses the shape or the match is minimal. Prefer extract-function over naming Strategy for the label alone.
 
-Паттерны ниже — **разрешённый минимум** для агентов; остальной GoF по умолчанию в Skip.
+Patterns below are the **allowed minimum** for agents; the rest of GoF is Skip by default.
 
 | Pattern | When to suggest | Anti-trigger |
 |---------|-----------------|--------------|
-| Strategy | взаимозаменяемые алгоритмы уже (или явно скоро) выбираются политикой | один алгоритм без ветвления политики |
-| Adapter | обернуть внешний / чужой API под локальный port | лишняя обёртка вокруг своего же API |
-| Facade | упростить multi-step subsystem за public API | facade превращается в god-object |
-| Factory Method | полиморфное создание уже принято в стиле репо | factory на каждый тривиальный `new` |
-| Decorator | аддитивное поведение без взрыва subclass-иерархий | decorator «для слоёв» без реальной нужды |
-| Observer | events / subscriptions уже есть в архитектуре | самодельный pub/sub без прецедента в репо |
-| Template Method | общий algorithm skeleton + вариативные hooks | наследование ради пары отличающихся строк |
-| Composite | дерево однородных узлов (UI/domain), как уже в репо | Composite на плоский список без иерархии |
+| Strategy | interchangeable algorithms already (or clearly soon) chosen by policy | one algorithm with no policy branching |
+| Adapter | wrap external / foreign API under a local port | extra wrapper around your own API |
+| Facade | simplify a multi-step subsystem behind a public API | facade turns into a god-object |
+| Factory Method | polymorphic creation already accepted in repo style | factory for every trivial `new` |
+| Decorator | additive behavior without exploding subclass hierarchies | decorator “for layers” with no real need |
+| Observer | events / subscriptions already exist in the architecture | homemade pub/sub with no repo precedent |
+| Template Method | shared algorithm skeleton + variable hooks | inheritance for a couple of differing lines |
+| Composite | tree of homogeneous nodes (UI/domain), as already in the repo | Composite on a flat list with no hierarchy |
 
-Если сомневаешься между двумя паттернами — выбери более простой extract/compose без имени; ярлык можно добавить на review, если репо уже так говорит.
+If unsure between two patterns — choose the simpler extract/compose without a name; the label may be added on review if the repo already speaks that way.
 
 ## Skip / avoid by default
 
 | Catalog | Skip |
 |---------|------|
 | GoF | Singleton, Abstract Factory, Flyweight, Memento, Interpreter, Visitor, Bridge, Prototype |
-| GoF (conditional) | Command, Chain of Responsibility, State — упоминать **только если** репо уже использует |
-| Fowler | всё сверх 12 smells выше |
-| DDD | Aggregates, bounded-context maps, event sourcing, CQRS — **не в core** |
-| DDD (pointer) | ubiquitous language anti-pattern — расширять в `feature-delivery-workflow`, не здесь |
-| GRASP | список Larman как отдельные правила — skip (перекрывается SOLID/CUPID) |
-| Clean Code dogma | жёсткие caps на строки функции / «ровно один assert» — skip |
+| GoF (conditional) | Command, Chain of Responsibility, State — mention **only if** the repo already uses them |
+| Fowler | everything beyond the 12 smells above |
+| DDD | Aggregates, bounded-context maps, event sourcing, CQRS — **not in core** |
+| DDD (pointer) | ubiquitous language anti-pattern — expand in `feature-delivery-workflow`, not here |
+| GRASP | Larman list as separate rules — skip (covered by SOLID/CUPID) |
+| Clean Code dogma | hard caps on function lines / “exactly one assert” — skip |
 
-Не вводить второй stem (`refactoring-smells` / `design-patterns-for-agents`) — один каталог: `design-guidance`.
+Do not introduce a second stem (`refactoring-smells` / `design-patterns-for-agents`) — one catalog: `design-guidance`.
 
 ## Out of scope for this stem
 
-- Полный GoF (23) и полный Fowler catalog.
-- Stack-specific рецепты (пути модулей, lint-команды) — только в **Stack notes** адаптера.
-- Security, a11y, perf — отдельные review-правила; здесь только структурные smells/patterns.
-- Четвёртое always-on правило — запрещено; этот stem только requestable / agent-load.
+- Full GoF (23) and full Fowler catalog.
+- Stack-specific recipes (module paths, lint commands) — only in the adapter **Stack notes**.
+- Security, a11y, perf — separate review rules; here only structural smells/patterns.
+- A fourth always-on rule — forbidden; this stem is requestable / agent-load only.
 
 ## Cross-refs
 
 - Always-on Principles: `code-quality-and-refactoring`.
-- DIP / ports vs adapters: stack `architecture-boundaries` (детали стека — в Stack notes адаптеров).
+- DIP / ports vs adapters: stack `architecture-boundaries` (stack details — in adapter Stack notes).
 - Ubiquitous language: `feature-delivery-workflow` (lineage).
 - MR checklist (smells in scope; pattern name only if repo uses it): `code-review-mr`.
 - Token / twin policy for authors: `preset-token-budget`, `preset-twin-sync` (meta, not consumer always-on).