@bonesofspring/ai-rules 0.2.5 → 0.2.7

package/presets/cursor/next/rules/agent-team-intake.mdc

@@ -3,12 +3,14 @@ description: Use when user posts a work request without /task — suggest or use
 alwaysApply: false
 ---
 
+<!-- shared-core: agent-team/agent-team-intake.md -->
+
 # Agent team intake
 
 When the user message looks like a **work request** (implement, add, fix, refactor, review MR, write tests, spike) — not a question about how code works:
 
 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», «объясни») — answer normally, no `/task`.
+3. Pure questions («как работает X», «объясни») — 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.

package/presets/cursor/next/rules/agent-team-orchestrator.mdc

@@ -3,6 +3,8 @@ description: Orchestrates the agent team via task-router and pipeline.json. Use
 alwaysApply: false
 ---
 
+<!-- shared-core: agent-team/agent-team-orchestrator.md -->
+
 # Agent team orchestrator
 
 Parent agent = **manager**. Router plans; specialists execute. Artifacts: `.cursor/team/tasks/<slug>/`.
@@ -19,7 +21,9 @@ Work request без `/task` — см. **`agent-team-intake.mdc`** (когда п
 | `/feature-continue <slug>` | Alias of task-continue |
 | `/technical-retro [slug]` | Retro with agent team block |
 
-## Roles
+## Stack notes
+
+### Roles
 
 | Agent | App / prod writes | Typical intent |
 |-------|-------------------|----------------|
@@ -31,7 +35,7 @@ Work request без `/task` — см. **`agent-team-intake.mdc`** (когда п
 | `debugger` | minimal fix | bugfix |
 | `ci-investigator` | minimal CI fix | ci-fix — minimal CI repair |
 | `feature-developer` | yes | feature, bugfix, refactor, migration, a11y |
-| `build-verifier` | no (team only) | lint/type-check/unit gate after developer |
+| `build-verifier` | no (team only) | lint/type-check/unit gate after developer; preset-structure when editing presets |
 | `accessibility-reviewer` | no (team only) | a11y, UI-heavy feature |
 | `performance-auditor` | no (team only) | perf-audit, perf-sensitive feature |
 | `code-reviewer` | no (team only) | most pipelines, review-only |
@@ -49,6 +53,8 @@ Work request без `/task` — см. **`agent-team-intake.mdc`** (когда п
 
 Full prompts: `.cursor/agents/*.md`. Artifact conventions: `.cursor/team/README.md`.
 
+When editing `packages/ai-rules/presets/**`, follow **`preset-pr-checklist.mdc`** (+ twin-sync / layering / token-budget / leakage meta-rules).
+
 ## Dynamic pipeline
 
 ```mermaid
@@ -145,7 +151,7 @@ If `pipeline.json` is missing (old `/feature-start` tasks), fall back to fixed p
 
 Do **not** run `/task` + router for:
 
-- Pure questions («как работает X», «объясни»).
+- Pure questions («как работает X», «объясни») — prefer agent **`codebase-analyzer`** for deep traces; inline OK for trivial one-liners.
 - Typo / one-file fix / trivial config with no architecture risk.
 - User explicitly says «без pipeline», «просто сделай», or continues an active slug.
 - Single-line `docs-only` with no code impact.

package/presets/cursor/next/rules/api-services.mdc

@@ -2,6 +2,7 @@
 description: Конвенции API сервисов и мапперов
 globs: app/src/api/services/**/*.ts
 alwaysApply: false
+impact: HIGH
 ---
 
 # Роль API слоя
@@ -55,4 +56,24 @@ alwaysApply: false
   - компоненты не должны зависеть от DTO;
   - store не должен сам собирать URL/коды эндпоинтов и не обходить сервисы; **транспортный тип ответа** и **контракт ошибки** из `@/types` (как у прикладного клиента) допустимы во thunk при разборе `catch`/payload, если так выстроен сервис (см. `store-rtk.mdc`, `http-client.mdc`).
 - При использовании API‑сервисов в UI, store и утилитах **вне** `app/src/api/**` импортировать **только** из `@/api` (корневой barrel), см. **`public-imports.mdc`**; внутри слоя API — по относительным путям или `@/api/services/**` / `@/api/clients/**`, не дублируя публичный контракт мимо корневого barrel для внешних потребителей.
+## Incorrect
 
+```ts
+// Returning raw DTO to store/UI without mapping
+export const getCard = async (id: string) => {
+  const { data } = await cardsClient.get(`/cards/${id}`)
+  return data // backend DTO leaked upward
+}
+```
+
+## Correct
+
+```ts
+import type { TCard } from '@/types'
+import { mapCardResponse } from './cardResponseMappers'
+
+export const getCard = async (id: string): Promise<TCard> => {
+  const { data } = await cardsClient.get(`/cards/${id}`)
+  return mapCardResponse(data)
+}
+```

package/presets/cursor/next/rules/architecture-boundaries.mdc

@@ -2,6 +2,7 @@
 description: Архитектурные границы и правила импортов
 globs: app/src/api/**/*,app/src/store/**/*,app/src/types/**/*,app/src/lib/**/*
 alwaysApply: false
+impact: CRITICAL
 ---
 
 # Границы между слоями
@@ -63,4 +64,30 @@ alwaysApply: false
 - Разместить файлы в **соответствующих слоях**.
 - Проверить существующие фичи с аналогичной структурой и **повторить их организацию**.
 - Не "коротить" слои (например, не вызывать API прямо из компонента только ради упрощения).
+## Incorrect
 
+```ts
+// UI calling HTTP client / deep service path directly
+import { httpClient } from '@/lib/clients/HttpClient'
+import { fetchOrders } from '@/api/services/OrdersApi/orders.service'
+
+export const OrdersPage = () => {
+  // bypasses store and @/api barrel
+  void httpClient.get('/orders')
+  return null
+}
+```
+
+## Correct
+
+```ts
+import { useAppDispatch } from '@/store'
+import { loadOrders } from '@/store/slices/Orders'
+
+export const OrdersPage = () => {
+  const dispatch = useAppDispatch()
+  // UI → store → services from @/api (inside thunk)
+  void dispatch(loadOrders())
+  return null
+}
+```

package/presets/cursor/next/rules/arrow-functions.mdc

@@ -2,45 +2,30 @@
 description: Предпочитать стрелочные функции при написании кода
 globs: app/src/**/*.{ts,tsx,js,jsx}
 alwaysApply: false
+impact: LOW
 ---
-
 # Стрелочные функции
 
-При разработке **использовать стрелочный синтаксис** для функций, где это допустимо в TypeScript/JavaScript.
-
-## Что делать
+Предпочитать стрелочный синтаксис для нового кода в `app/src/**`, где это допустимо в TypeScript/JavaScript.
 
-- Объявлять функции как **`const имя = (...) => { ... }`** вместо **`function имя(...) { ... }`**, если не нужны особенности объявления `function`.
-- Колбэки и обработчики — стрелочные функции: `.map((x) => ...)`, `onClick={() => ...}`.
-- React-компоненты и хуки — как стрелочные функции с явной типизацией пропсов/возврата по принятому в проекте стилю.
+**Style enforcement:** детальные pref — в ESLint (`app/eslint.config.mjs`) и Stylelint; после правок — **`post-change-lint`**. Это правило — краткий intent, не дублировать линтер.
 
-## Исключения (допустимо не стрелка)
+## Исключения
 
-- **Генераторы** (`function*`) — стрелкой не выразить.
-- **Методы класса** — если в коде используются классы, допустимы обычные методы (`method() {}`), а не стрелки в теле класса (из‑за `this`).
-- Когда осознанно нужны **подъём (hoisting)** или **имя функции в стеке** только у `function` — редкие случаи; иначе предпочитать стрелку.
+- Генераторы (`function*`)
+- Методы класса, где нужен `this` прототипа
+- Редкие случаи hoisting / именованного `function` в стеке
 
-## Примеры
+## Incorrect
 
-```typescript
-// ❌ Избегать для нового кода
+```ts
 function formatLabel(id: string): string {
   return id.toUpperCase()
 }
-
-// ✅ Предпочтительно
-const formatLabel = (id: string): string => {
-  return id.toUpperCase()
-}
 ```
 
-```tsx
-// ✅ Предпочтительно
-const UserCard = ({ name }: { name: string }) => {
-  return <span>{name}</span>
-}
-```
+## Correct
 
-## Требование к агенту
-
-При генерации и правке кода **по умолчанию выбирать стрелочные функции**; отступать к `function` только в случаях из раздела исключений.
+```ts
+const formatLabel = (id: string): string => id.toUpperCase()
+```

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

@@ -3,15 +3,37 @@ description: Поддержка и улучшение качества кода
 alwaysApply: true
 ---
 
+<!-- shared-core: quality/code-quality-and-refactoring.md -->
+
 # Поддержка существующего стиля
 
 - Новые изменения должны:
-  - следовать существующим паттернам (имена, структура, типизация);
-  - минимизировать "стилистический шум" (лишние правки форматирования, rename без нужды).
+  - следовать существующим паттернам (имена, структура, типизация / DI / error handling стека);
+  - минимизировать «стилистический шум» (лишние правки форматирования, rename без нужды).
 - Перед добавлением нового решения:
-  - искать аналогичное в коде и **повторять подход**, а не изобретать новый.
-  - проверять, нет ли уже подходящего компонента или паттерна в существующем UI‑коде и пакетах проекта, прежде чем добавлять новый кастомный контрол.
-  - использовать при обращении к чужим модулям только их **public API** (index/barrel‑файлы и явно экспортируемые сущности), а deep‑импорты внутренних файлов рассматривать как повод для рефакторинга.
+  - искать аналогичное в коде и **повторять подход**, а не изобретать новый;
+  - проверять, нет ли уже подходящего компонента или паттерна в дизайн‑системе / UI‑пакетах проекта, прежде чем добавлять новый кастомный контрол;
+  - использовать при обращении к чужим модулям только их **public API** (barrel / протоколы / facades), а deep‑импорты внутренних файлов рассматривать как повод для рефакторинга.
+
+# Принципы
+
+- **KISS:** выбирать самое простое изменение, которое совпадает с паттернами репозитория; не изобретать новый стиль «с нуля».
+- **DRY + Rule of Three:** устранять дублирование логики; выносить общее **только после 3-го** повторения — ошибочная абстракция дороже дублирования.
+- **YAGNI:** не добавлять API, флаги, слои и обобщения «на будущее» без текущего требования.
+- **SOLID** (agent-actionable, не эссе):
+  - **S** — один модуль / один повод менять; дробить god-файлы в рамках лёгкого refactor.
+  - **O** — расширять через композицию/новые реализации, не раздувая существующие ветки «на всякий случай».
+  - **L** — подтип не ломает контракт базового типа/протокола.
+  - **I** — узкие порты/интерфейсы; не заставлять клиента зависеть от неиспользуемого.
+  - **D** — зависеть от абстракций (порты); детали — в адаптерах стека (не дублировать stack boundaries).
+- **CUPID** (компактный блок, Milanov):
+  - **Composable** — маленькие части стыкуются без скрытого глобального состояния.
+  - **Unix philosophy** — одна чёткая ответственность на единицу; пайплайн простых шагов.
+  - **Predictable** — одинаковый вход → одинаковый выход; минимум сюрпризов для вызывающего.
+  - **Idiomatic** — следовать идиомам языка и **существующим** конвенциям репо.
+  - **Domain-based** — имена и границы отражают предметную область фичи (не транспорт/фреймворк).
+- **Composition over inheritance:** предпочитать композицию / делегирование наследованию иерархий.
+- **Law of Demeter (hint):** не строить цепочки `a.b.c.d`; ходить через public API / facade модуля.
 
 # Рефакторинг при изменениях
 
@@ -19,27 +41,42 @@ alwaysApply: true
   - уменьшает дублирование;
   - повышает читаемость;
   - не ломает публичные контракты модулей.
-- Примеры допустимых улучшений:
-  - вынести дублирующуюся логику в общий хук/утилиту;
-  - типизировать `any` и `unknown`, если это безболезненно;
-  - разделить слишком крупный компонент на несколько более простых;
-  - заменить локальные «магические» CSS‑значения (цвета, отступы, размеры) на токены/примитивы дизайна проекта;
+- Примеры допустимых улучшений (абстрактно):
+  - вынести дублирующуюся логику в общую утилиту / хук / mapper / use case;
+  - сузить небезопасные типы и unsafe-операции стека;
+  - разделить слишком крупный модуль на несколько более простых;
+  - заменить локальные «магические» значения на токены/примитивы дизайн‑системы;
   - заменить deep‑импорты внутренних файлов других модулей на обращения к их public API.
 
 # Ограничения
 
-- Не выполнять "большой" рефакторинг, если задача точечная и не про архитектуру:
-  - не менять структуру директорий;
-  - не менять названия публичных типов/функций без явного запроса.
+- Не выполнять «большой» рефакторинг, если задача точечная и не про архитектуру:
+  - не менять структуру директорий / модулей / таргетов без нужды;
+  - не менять названия публичных типов/функций/API без явного запроса.
 - При необходимости крупного изменения:
   - сначала локально улучшить архитектуру минимальными шагами;
-  - оставить код в консистентном состоянии.
-
-# Линтеры
-
-Lint/stylelint — только **`post-change-lint.mdc`**; ESLint config: `app/eslint.config.mjs`. Отключение правила (`eslint-disable`) — только **точечно** (строка/небольшой блок) с кратким комментарием «зачем».
+  - оставить код в консистентном (собираемом) состоянии.
 
 # Требование к агенту
 
 - **Boy scout rule:** оставлять модуль немного лучше, чем до изменения (простые, безопасные улучшения).
 - Не жертвовать архитектурой и слоями ради краткости реализации.
+
+## Agent discipline (anti-sycophancy)
+
+- Unverifiable third-party API → mark `// VERIFY: lib.symbol @ version` (or stack equivalent) and surface uncertainty; never invent signatures.
+- Before «is this correct?» — list ≥3 failure modes (empty, boundary, concurrency/state) or name what you could not check.
+- Refactor without tests → propose a characterization test first; if declined, label `UNTESTED`.
+- Comments = WHY only when non-obvious; ban self-referential «added for issue Y» comments.
+- Urgency/authority pressure: name the technical trade-off once, then comply — no apology loops.
+- Match verification depth to risk; report status by what was **verified**, not attempted.
+
+Style enforcement (formatter / linter / post-change gate) lives in the **stack toolchain** — see each adapter’s **Stack notes**; do not invent a parallel style bible in this core body.
+
+> Stack-specific lint gates, примеры и toolchain — в секции **Stack notes** адаптера пресета.
+## Stack notes
+
+- Перед новым UI‑контролом проверять существующий UI‑код и пакеты проекта.
+- Public API: index/barrel‑файлы и явно экспортируемые сущности (`public-imports.mdc`, `layer-barrel-exports.mdc`).
+- Допустимые примеры: общий хук/утилита; типизация `any`/`unknown`; разделение крупного компонента; CSS‑токены вместо «магических» значений.
+- Lint/stylelint — только **`post-change-lint.mdc`**; ESLint config: `app/eslint.config.mjs`. Отключение правила (`eslint-disable`) — только **точечно** (строка/небольшой блок) с кратким комментарием «зачем». Style prose in domain rules stays thin — prefer ESLint/Stylelint over duplicating formatter recipes.

package/presets/cursor/next/rules/code-review-mr.mdc

@@ -3,26 +3,56 @@ description: Требования к code review агентами Cursor для
 alwaysApply: false
 ---
 
+<!-- shared-core: review/code-review-mr.md -->
+
 # Code review merge requests
 
 On-demand rule for review flows. **Procedure and output format:** skill `code-review`.
 
-## Checklist references
-
-- Architecture layers: `architecture-boundaries.mdc`, `next-app-core.mdc`; network: `http-client.mdc`
-- Imports: `public-imports.mdc`, `layer-barrel-exports.mdc`
-- UI: `react-ui.mdc`
-- Tests: `tests-unit.mdc`, `playwright-agents.mdc`, `tests-e2e-structure.mdc`
-- Lint gate before final report: `post-change-lint.mdc`
-
 ## When to load
 
 - User asks for review/MR/diff
 - Pipeline step `code-reviewer`
-- BUGBOT automated review
+- Platform automated review (e.g. BUGBOT) when present
+
+## Checklist references (abstract)
+
+- Architecture layers and stack core rules
+- Public imports / module boundaries
+- 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
+
+Concrete rule stems — in **Stack notes**.
 
 ## Constraints
 
 - 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`).
+
+## Output contract
+
+Procedure details live in skill `code-review`. Required shape:
+
+- Each finding cites **`path:line`**, the problem, impact, and a concrete fix direction.
+- Rank by severity: **`blocker` | `important` | `nit`** (list blocker → nit).
+- Vague («looks risky») is not a finding — cite evidence or ask for surrounding file context.
+- End with a verdict on its own line: **`Safe to merge | needs changes | reject`**.
+
+## Stack notes
+
+On-demand rule for review flows. **Procedure and output format:** skill `code-review`.
+
+### Checklist references
+
+- Architecture layers: `architecture-boundaries.mdc`, `next-app-core.mdc`; network: `http-client.mdc`
+- Imports: `public-imports.mdc`, `layer-barrel-exports.mdc`
+- UI: `react-ui.mdc`
+- Security: `security-next.mdc`
+- Tests: `tests-unit.mdc`, `playwright-agents.mdc`, `tests-e2e-structure.mdc`
+- Lint gate before final report: `post-change-lint.mdc`

package/presets/cursor/next/rules/design-guidance.mdc

@@ -0,0 +1,111 @@
+---
+description: Design smells (Fowler) and curated GoF patterns for agents. Load when assessing structure, smells, or pattern fit.
+globs:
+  - app/src/**/*.{ts,tsx}
+alwaysApply: false
+---
+
+<!-- shared-core: quality/design-guidance.md -->
+
+# Design guidance (smells & patterns)
+
+Словарь **code smells** (Fowler) и **design patterns** (GoF) для агентов при implement / review / debug.
+
+Применять только в scope задачи или MR. Лёгкий refactor — да; «большой» rewrite без явного запроса — нет.
+
+Называть паттерн **только если** репозиторий уже так делает или это минимальный match. Иначе — extract/split/move без ярлыка GoF.
+
+Не заменяет always-on Principles в `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; не расширять задачу ради каталога.
+
+Агенты и skills подгружают stem по one-liner wiring — не копировать этот каталог в промпт агента.
+
+## 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», если поведение уже корректно и читаемо.
+
+## Code smells (Fowler, curated ≤12)
+
+Таблица: smell → detect hint → fix hint (одна линия каждый). Без механических меню рефакторинга.
+
+| # | 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 внутри уже правильного модуля.
+
+## Design patterns (GoF, curated ≤8)
+
+Suggest only when repo already uses the shape or the match is minimal. Prefer extract-function over naming Strategy ради ярлыка.
+
+Паттерны ниже — **разрешённый минимум** для агентов; остальной GoF по умолчанию в Skip.
+
+| 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 на плоский список без иерархии |
+
+Если сомневаешься между двумя паттернами — выбери более простой extract/compose без имени; ярлык можно добавить на review, если репо уже так говорит.
+
+## 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 |
+
+Не вводить второй stem (`refactoring-smells` / `design-patterns-for-agents`) — один каталог: `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.
+
+## Cross-refs
+
+- Always-on Principles: `code-quality-and-refactoring`.
+- DIP / ports vs adapters: stack `architecture-boundaries` (детали стека — в 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).
+
+## Stack notes
+
+- Prefer hooks/mappers over HOC or new pattern names when the repo already extracts that way.
+- Suggest Strategy/Adapter only when ports or policy branching already exist in `app/src/**`.

package/presets/cursor/next/rules/feature-delivery-workflow.mdc

@@ -3,8 +3,12 @@ description: Сквозной чеклист новой фичи и матриц
 alwaysApply: false
 ---
 
+<!-- shared-core: architecture/feature-delivery-workflow.md -->
+
 # Доставка фичи (сквозной порядок)
 
+## Stack notes
+
 Типичная фича с данными с backend и общим состоянием. Детали слоёв — `architecture-boundaries.mdc`, `next-app-core.mdc`.
 
 ## Чеклист (порядок работ)
@@ -61,6 +65,8 @@ flowchart LR
 - HTTP-клиент из компонента/thunk в обход сервиса.
 - Deep-import `@/api/services/**` из UI/store.
 - `{...props}` в компоненты — `no-props-spread.mdc`.
+- **Ubiquitous language:** имена Domain/типов отражают предметную область фичи — не смешивать транспорт/фреймворк-термины в домен без нужды.
+- Load `design-guidance` (rule stem) when assessing structure, smells, or pattern fit.
 
 ## Матрица: зона → правила
 

package/presets/cursor/next/rules/http-client.mdc

@@ -4,6 +4,7 @@ globs:
   - app/src/lib/clients/HttpClient/**/*.ts
   - app/src/api/clients/**/*.ts
 alwaysApply: false
+impact: HIGH
 ---
 
 # HTTP-транспорт
@@ -40,3 +41,23 @@ alwaysApply: false
 
 - Меняя реализацию общего клиента — **обновить или добавить behavior‑тесты** рядом с модулем клиента (`tests-unit.mdc`).
 - Не вводить **второй полноценный HTTP‑стек** без явной задачи и согласования с архитектурой репозитория.
+## Incorrect
+
+```ts
+// Inside api/services — ad-hoc fetch bypasses shared client
+export const getSettings = async () => {
+  const res = await fetch('/api/settings')
+  return res.json()
+}
+```
+
+## Correct
+
+```ts
+import { appHttpClient } from '@/api/clients/appHttpClient'
+
+export const getSettings = async () => {
+  const { data } = await appHttpClient.get('/settings')
+  return mapSettingsResponse(data)
+}
+```

package/presets/cursor/next/rules/layer-barrel-exports.mdc

@@ -2,6 +2,7 @@
 description: Двухуровневые barrel-экспорты для слоёв с регламентированным public API
 globs: app/src/**/index.ts
 alwaysApply: false
+impact: MEDIUM
 ---
 
 # Barrel-экспорты слоёв с public API
@@ -57,3 +58,17 @@ alwaysApply: false
 | Core | `app/src/core/index.ts` | ESLint: `@/core/index` |
 
 Иллюстрация двух уровней (API): локальный `services/.../<Feature>/index.ts` → фасад `*ApiService.ts` → `app/src/api/index.ts`.
+## Incorrect
+
+```ts
+// app/src/api/index.ts — leaking internals
+export { mapCardResponse } from './services/CardsApi/cardResponseMappers'
+export { rawCardsClient } from './clients/rawCardsClient'
+```
+
+## Correct
+
+```ts
+// app/src/api/index.ts — public facade only
+export { cardsApi } from './services/CardsApi'
+```

package/presets/cursor/next/rules/next-app-router.mdc

@@ -22,7 +22,7 @@ alwaysApply: false
 
 ## Маршруты и UX
 
-- Для async routes использовать **`loading.tsx`**, **`error.tsx`**, **`not-found.tsx`** по образцу соседних routes.
+- Для async data routes **обязательны** sibling **`loading.tsx`** и **`error.tsx`** (и `not-found.tsx` где уместно) — по образцу соседних routes.
 - **Suspense** — для медленных client/server секций; fallback согласован с дизайн-системой.
 - Навигация — `navigation-router-stack.mdc` + router/link паттерны проекта.
 
@@ -31,6 +31,18 @@ alwaysApply: false
 - RSC/route handlers **не импортируют UI-компоненты с client-only зависимостями** напрямую в server tree без `'use client'` границы.
 - Доменные типы — `@/types`; вызовы backend — через `@/api` / services, не raw `fetch` из page.tsx без слоя API.
 
+## Anti-hallucination (App Router)
+
+Меньше угаданного кода лучше, чем больше. Сверяй API с версией Next в `package.json` и соседними routes.
+
+| ❌ Avoid | ✅ Prefer |
+|---------|----------|
+| Sync `params` / `searchParams` / `cookies()` / `headers()` when the installed Next treats them as async | `await params` / `await cookies()` / patterns from existing pages |
+| Invented `getSession()`-only server auth without project verify | Server session verify used by the repo (`security-next.mdc`) |
+| Auth enforcement only in Edge middleware | Layout/page/Server Action checks + middleware refresh if present |
+| New App Router APIs not in the installed Next major | Copy from a sibling route; mark `VERIFY` if unsure |
+| Client `fetch` in `page.tsx` bypassing `@/api` | Services via `@/api` / established data loaders |
+
 ## Эталон
 
 Смотри существующий route той же сложности в `app/src/app/**` и повтори структуру (`reference-features.mdc`).

package/presets/cursor/next/rules/preset-layering.mdc

@@ -0,0 +1,36 @@
+---
+description: Preset authoring: Core → Stack → Platform layering. Use when editing packages/ai-rules/presets/**.
+globs:
+  - packages/ai-rules/presets/**/*
+  - .cursor/rules/**/*
+  - .cursor/team/tasks/**/*
+alwaysApply: false
+---
+
+<!-- shared-core: meta/preset-layering.md -->
+
+# Preset layering
+
+Правила для **авторов пресетов** (`packages/ai-rules/presets/**`). Не always-on в consumer apps.
+
+## Слои
+
+| Слой | Где | Что класть |
+|------|-----|------------|
+| **Core** | `presets/_shared/core/**` | Stack-agnostic prose + checklists; без frontmatter; без путей `app/**`, `Features/**`, Playwright, xcodebuild |
+| **Stack** | `presets/{cursor\|claude}/<stack>/` | Domain/tooling, always-on trio стека, stack examples в **Stack notes** |
+| **Platform** | Cursor `.mdc` / Claude `.md` | Frontmatter (`alwaysApply`/`globs` vs `paths:`), platform-only (`BUGBOT.md`, `CLAUDE.md`) |
+
+## Куда класть новое правило
+
+1. Общие принципы для всех стеков → **core** + адаптеры с `<!-- shared-core: … -->`.
+2. Только один из installable стеков (`next` / `ios-swift` / `go`) → **stack-specific** (не в `_shared`).
+3. Только Cursor или только Claude packaging → platform artifact / frontmatter, не дублировать семантику.
+
+## SoT
+
+- Packages path — SoT; root `.cursor/` — dogfood sync.
+- Per stack: **Cursor** `presets/cursor/<stack>/` → Claude twin same PR.
+- `_shared` **не** `--preset` и не копируется CLI `init`.
+
+См. также: `preset-twin-sync`, `preset-token-budget`, `preset-pr-checklist`, `preset-no-cross-stack-leakage`.