@bonesofspring/ai-rules 0.2.5 → 0.2.6

package/presets/cursor/go/skills/integration-testing/SKILL.md

@@ -0,0 +1,15 @@
+---
+name: integration-testing
+description: Plans, generates, and heals Go integration and adapter contract tests (httptest, testcontainers). Use for integration-test-* agents or adapter/e2e coverage.
+---
+
+# Integration testing (Go)
+
+1. Read `tests-integration` / `tests-e2e` rules and task brief.
+2. Prefer testing driven adapters behind ports; HTTP adapters with `httptest` + fakes.
+3. Use build tags / CI job split if the repo already does.
+4. Do not introduce Playwright or XCUITest.
+
+## Agents
+
+`integration-test-planner` → `integration-test-generator` → `integration-test-healer` as needed.

package/presets/cursor/go/skills/technical-retro/SKILL.md

@@ -0,0 +1,10 @@
+---
+name: technical-retro
+description: Facilitates technical retrospectives for tasks, sprints, incidents, releases, and agent-team pipelines. Use when the user asks for retro, postmortem, process review, or action items after delivery.
+---
+
+# Technical Retro
+
+Same structure as next technical-retro skill. For `/task` pipelines include router, analyst, developer, **build-verifier** (go test), reviewer, QA/integration tests, hooks, rules.
+
+Preset feedback: propose updates to `.cursor/rules|agents|skills` — do not apply without approval.

package/presets/cursor/go/skills/unit-testing/SKILL.md

@@ -0,0 +1,14 @@
+---
+name: unit-testing
+description: Plans, generates, and heals table-driven Go unit tests for domain, use cases, and mappers using *_test.go and tests-unit rules. Use for unit-test planning, generation, or healing.
+---
+
+# Unit testing (Go)
+
+1. Prefer table-driven `testing` packages beside code under test.
+2. Fake outbound ports for use cases; no real DB/network.
+3. Follow `tests-unit.mdc`; heal with `unit-test-healer` when red.
+
+## Agents
+
+`unit-test-planner` → `unit-test-generator` → `unit-test-healer`.

package/presets/cursor/go/team/README.md

@@ -0,0 +1,31 @@
+# Agent team artifacts (go)
+
+Каталог артефактов пайплайна «команда агентов» для Go backend (hexagonal / ports & adapters).
+
+## Layout
+
+```
+.cursor/team/   # Claude: .claude/team/
+  active-task.json
+  fixtures/
+  tasks/<slug>/
+    pipeline.json
+    status.json
+    brief.md
+    decomposition.md
+    …
+```
+
+## Fixtures
+
+| Fixture | Notes |
+|---------|-------|
+| `feature-full.json` | Full profile; QA scope `e2e-only` = **adapter/integration tests** (not UI e2e); Playwright/XCUITest stems in `skipped` |
+| `feature-light.json` | developer → build-verifier (`go test` / `go vet` / golangci-lint) |
+| `bugfix-standard.json` | debugger → developer (skipIf) → verifier → reviewer |
+
+**Never** schedule Playwright/XCUITest-only steps for this preset. Use `integration-test-*` (or QA with `e2e-only` = integration) for adapter-level coverage.
+
+## Creating task dirs
+
+Bootstrap `tasks/<slug>/` by **writing** artifacts (Write tool), not Shell `mkdir`. Cursor sandbox → `Operation not permitted` otherwise (worse on Desktop/Documents).

package/presets/cursor/go/team/fixtures/bugfix-standard.json

@@ -0,0 +1,37 @@
+{
+  "slug": "example-bugfix-standard",
+  "intent": "bugfix",
+  "profile": "standard",
+  "summary": "Regression fix with review",
+  "steps": [
+    {
+      "agent": "debugger",
+      "label": "Root cause"
+    },
+    {
+      "agent": "feature-developer",
+      "label": "Fix if needed",
+      "skipIf": "debugger.fixed"
+    },
+    {
+      "agent": "build-verifier",
+      "label": "go test / go vet / golangci-lint"
+    },
+    {
+      "agent": "code-reviewer",
+      "label": "Code review"
+    }
+  ],
+  "humanGates": [],
+  "autoChain": true,
+  "skipped": [
+    {
+      "agent": "playwright-test-planner",
+      "reason": "Go backend uses integration-test-* — not in this bugfix path"
+    },
+    {
+      "agent": "xcuitest-test-planner",
+      "reason": "Go backend uses integration-test-* — not in this bugfix path"
+    }
+  ]
+}

package/presets/cursor/go/team/fixtures/feature-full.json

@@ -0,0 +1,43 @@
+{
+  "slug": "example-feature-full",
+  "intent": "feature",
+  "profile": "full",
+  "summary": "Multi-layer Go use case with integration-test coverage",
+  "steps": [
+    { "agent": "task-analyst", "label": "Clarify and decompose" },
+    { "agent": "solution-architect", "label": "Design boundaries", "model": "strong" },
+    { "agent": "feature-developer", "label": "Implement + unit tests", "scope": "unit-in-dev" },
+    { "agent": "build-verifier", "label": "go test / go vet / golangci-lint", "model": "cheap" },
+    { "agent": "security-reviewer", "label": "Security review", "model": "strong" },
+    { "agent": "code-reviewer", "label": "Code review", "model": "cheap" },
+    { "agent": "qa-tester", "label": "Integration tests", "scope": "e2e-only" }
+  ],
+  "humanGates": ["after:task-analyst"],
+  "autoChain": true,
+  "skipped": [
+    {
+      "agent": "playwright-test-planner",
+      "reason": "Go backend uses integration-test-* — not Playwright"
+    },
+    {
+      "agent": "playwright-test-generator",
+      "reason": "Go backend uses integration-test-*"
+    },
+    {
+      "agent": "playwright-test-healer",
+      "reason": "Go backend uses integration-test-*"
+    },
+    {
+      "agent": "xcuitest-test-planner",
+      "reason": "Go backend uses integration-test-* — not XCUITest"
+    },
+    {
+      "agent": "xcuitest-test-generator",
+      "reason": "Go backend uses integration-test-*"
+    },
+    {
+      "agent": "xcuitest-test-healer",
+      "reason": "Go backend uses integration-test-*"
+    }
+  ]
+}

package/presets/cursor/go/team/fixtures/feature-light.json

@@ -0,0 +1,17 @@
+{
+  "slug": "example-feature-light",
+  "intent": "feature",
+  "profile": "light",
+  "summary": "Trivial single-package change with explicit AC",
+  "steps": [
+    { "agent": "feature-developer", "label": "Implement", "model": "cheap" },
+    { "agent": "build-verifier", "label": "go test / go vet / golangci-lint", "model": "cheap" }
+  ],
+  "humanGates": [],
+  "autoChain": true,
+  "skipped": [
+    { "agent": "task-analyst", "reason": "Explicit AC and single-package scope" },
+    { "agent": "code-reviewer", "reason": "Light profile — non-trivial module/service changes should use standard" },
+    { "agent": "qa-tester", "reason": "No integration-test AC" }
+  ]
+}

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

@@ -28,6 +28,13 @@ npx @bonesofspring/ai-rules init cursor --preset ios-swift
 
 Requestable (не always-on): `post-change-build`, `xcode-tooling`, `agent-team-*`, `code-review-mr`, `feature-delivery-workflow`, `reference-features`.
 
+## Preset taxonomy & contributing
+
+- **Layers:** Core → Stack → Platform — [`../../_shared/README.md`](../../_shared/README.md)
+- **Contribution guide:** [`../../docs/PRESET-CONTRIBUTION.md`](../../docs/PRESET-CONTRIBUTION.md) (SoT, twins, token budget, PR checklist)
+- **Author meta-rules:** `preset-*.mdc` (globs `packages/ai-rules/presets/**`, never alwaysApply)
+- **Core→stack mapping:** [`rules/README.md`](./rules/README.md)
+
 ## Sync with Claude
 
 SoT = этот Cursor-пресет. После правок `.mdc` / agents / skills — обновить twin в `presets/claude/ios-swift/` **в том же PR**.

package/presets/cursor/ios-swift/agents/build-verifier.md

@@ -26,17 +26,26 @@ Never require yarn/`lint:js`/Playwright for iOS app work.
 
 ## Preset-structure validation (when scope is preset / packages/ai-rules)
 
-When validating **ios-swift preset** changes (or `scope: preset-structure-validation`), **also** run these checks and **FAIL** on any mismatch:
-
-1. **Chain fork check:** `hooks/chain-team-phases.sh` must **not** be byte-identical to `presets/cursor/next/hooks/chain-team-phases.sh` (Claude: vs `presets/claude/next/hooks/…`).
-2. **Forbidden handoff strings** in ios-swift `chain-team-phases.sh` (Cursor + Claude): `playwright-test-`, `playwright-agents`, `playwright-e2e`, `user-playwright`, `app/__tests__/e2e`, and positive yarn lint recipes (`lint:js` / `lint:css` as commands to run — negation prose alone is not enough; prefer zero occurrences).
-3. **Required handoff stems:** `AGENT_HANDOFF` must include `xcuitest-test-planner` (or generator/healer) and `build-verifier` text must mention `xcodebuild` or `post-change-build` / `xcode-tooling`.
-4. **Roster:** 21 agent stems; no `playwright-test-*` files under ios-swift `agents/`.
-5. **Token budget:** Cursor `alwaysApply: true` count ≤ 3; Claude rules without `paths:` === 3.
+When validating **ios-swift preset** changes (or `scope: preset-structure-validation` / `preset-artifacts-only`), **also** run these checks. Skip xcodebuild if there are **no** Swift app changes.
+
+### FAIL (all stacks + ios-specific)
+
+1. **Token budget:** Cursor `alwaysApply: true` count ≤ **3**; Claude rules without `paths:` === **3** (**exclude `**/README.md`** — Option A). Meta `preset-*` rules must **not** be `alwaysApply: true`.
+2. **Chain fork check:** `hooks/chain-team-phases.sh` must **not** be byte-identical to `presets/cursor/next/hooks/chain-team-phases.sh` (Claude: vs `presets/claude/next/hooks/…`).
+3. **Forbidden handoff strings** in ios-swift `chain-team-phases.sh` (Cursor + Claude): `playwright-test-`, `playwright-agents`, `playwright-e2e`, `user-playwright`, `app/__tests__/e2e`, and positive yarn lint recipes (`lint:js` / `lint:css` as commands to run — negation prose alone is not enough; prefer zero occurrences). See `preset-no-cross-stack-leakage.mdc`.
+4. **Required handoff stems:** `AGENT_HANDOFF` must include `xcuitest-test-planner` (or generator/healer) and `build-verifier` text must mention `xcodebuild` or `post-change-build` / `xcode-tooling`.
+5. **Roster:** 21 agent stems; no `playwright-test-*` files under ios-swift `agents/`.
+6. **Twin presence (hard):** Cursor rule change → Claude twin present **same PR** — **FAIL** if missing (next, ios-swift, and go).
+7. **Mapping / README:** `claude/ios-swift/rules/README.md` mapping table present; core→stack mapping documented when `_shared` consumers exist.
+8. **shared-core consumers:** if `_shared/core/**` changed, every matching `<!-- shared-core: … -->` consumer in this PR must be updated (or intentional fork documented).
+9. **Session-start body budget:** sum of the **3** intended always-on / session-start rule bodies ≤ **120** lines (stretch ≤100); do **not** include `**/README.md` — record in report.
+10. **Embed drift (soft by default):** `packages/ai-rules/scripts/check-shared-core-drift.sh`; `AI_RULES_DRIFT_FAIL=1` to enforce.
+11. **Dogfood (monorepo soft):** `packages/ai-rules/scripts/sync-dogfood-cursor.sh`; mass deletion without migration doc → risk / FAIL for maintainers.
 
 ### Soft rules (do not FAIL alone)
 
-- **AC-19 thin aliases:** `agent-team-intake` and similar thin requestable aliases may be &lt;15 body lines if they mirror next’s thin-alias pattern. Enforce ≥15 strictly on **domain** rules (navigation, persistence, security, modules, networking, UI).
+- **Thin aliases:** `agent-team-intake`, `technical-retro` may be &lt;15 body lines. Enforce ≥15 on **domain** rules (navigation, persistence, security, modules, networking, UI).
+- Embed vs lineage modes: `docs/PRESET-CONTRIBUTION.md`, `_shared/README.md`.
 
 ## Output
 

package/presets/cursor/ios-swift/agents/code-reviewer.md

@@ -73,3 +73,7 @@ Update `status.json`:
 If **REQUEST_CHANGES**, set `"state": "changes_requested"` (hook will re-invoke feature-developer).
 
 Do not mix this output with retrospective facilitation — keep review and retro separate.
+
+## Design guidance
+
+- Load `design-guidance` (rule stem) when assessing structure, smells, or pattern fit.

package/presets/cursor/ios-swift/agents/debugger.md

@@ -70,3 +70,7 @@ Update `status.json`:
 Handoff summary: root cause, recommended fix, files touched (if any), `fixApplied`.
 
 Do not perform formal code review or write XCUITest plans — those are separate agents.
+
+## Design guidance
+
+- Load `design-guidance` (rule stem) when assessing structure, smells, or pattern fit.

package/presets/cursor/ios-swift/agents/feature-developer.md

@@ -33,3 +33,7 @@ When copying from `next` into `ios-swift` (or another stack):
 ## On completion
 
 `status.json`: `currentAgent: feature-developer`, `state: completed`. Handoff by layer + gaps for verifier/reviewer. No formal code review.
+
+## Design guidance
+
+- Load `design-guidance` (rule stem) when assessing structure, smells, or pattern fit.

package/presets/cursor/ios-swift/agents/migration-specialist.md

@@ -71,3 +71,7 @@ Update `status.json`:
 ```
 
 Hand off: orchestrator runs human gate (if configured), then `feature-developer` executes phases from the plan.
+
+## Design guidance
+
+- Load `design-guidance` (rule stem) when assessing structure, smells, or pattern fit.

package/presets/cursor/ios-swift/agents/performance-auditor.md

@@ -70,3 +70,7 @@ Update `status.json`:
 For standalone `perf-audit` intent, suggest `/technical-retro` or a follow-up `/task` to implement fixes.
 
 Do not modify production code.
+
+## Design guidance
+
+- Load `design-guidance` (rule stem) when assessing structure, smells, or pattern fit.

package/presets/cursor/ios-swift/agents/qa-tester.md

@@ -63,3 +63,7 @@ Provide summary:
 - Test run results
 - Gaps vs acceptance criteria
 - Suggest `/technical-retro` with the slug when done
+
+## Design guidance
+
+- Load `design-guidance` (rule stem) when assessing structure, smells, or pattern fit.

package/presets/cursor/ios-swift/agents/solution-architect.md

@@ -15,3 +15,7 @@ You are a solution architect for **iOS Feature modules** (Presentation / Domain
 - Alternatives table + chosen design + risks; keep token-light.
 
 Handoff: architecture approved or awaiting gate → status update.
+
+## Design guidance
+
+- Load `design-guidance` (rule stem) when assessing structure, smells, or pattern fit.

package/presets/cursor/ios-swift/agents/task-analyst.md

@@ -23,3 +23,7 @@ Ask only blocking questions (AC gaps, Feature scope, platforms, offline/auth, AP
 3. Update `status.json` → `awaiting_approval` if human gate after analyst.
 
 Do not invent architecture stacks (TCA, SwiftData, …) — frame «как в репо».
+
+## Design guidance (optional)
+
+- Optional: Load `design-guidance` (rule stem) when assessing structure, smells, or pattern fit — only when decomposing refactor/architecture tasks (not default).

package/presets/cursor/ios-swift/agents/unit-test-generator.md

@@ -52,3 +52,7 @@ Update `status.json`:
   "updatedAt": "<ISO8601>"
 }
 ```
+
+## Design guidance
+
+- Load `design-guidance` (rule stem) when assessing structure, smells, or pattern fit.

package/presets/cursor/ios-swift/agents/unit-test-healer.md

@@ -51,3 +51,7 @@ Update `status.json`:
   "updatedAt": "<ISO8601>"
 }
 ```
+
+## Design guidance
+
+- Load `design-guidance` (rule stem) when assessing structure, smells, or pattern fit.

package/presets/cursor/ios-swift/agents/unit-test-planner.md

@@ -62,3 +62,7 @@ Update `status.json`:
 ```
 
 Do not write test source files — that is `unit-test-generator`'s job.
+
+## Design guidance
+
+- Load `design-guidance` (rule stem) when assessing structure, smells, or pattern fit.

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

@@ -20,6 +20,7 @@
 | `ios-app-core.mdc` | **always** | Стек, структура, tips (+ pointers на tooling/build) |
 | `architecture-boundaries.mdc` | **always** | Границы Presentation / Domain / Data, DI |
 | `code-quality-and-refactoring.mdc` | **always** | Качество и лёгкий refactor |
+| `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` |
@@ -28,6 +29,7 @@
 | `code-review-mr.mdc` | requestable | Ревью MR/diff |
 | `reference-features.mdc` | globs Features/** | Эталоны Feature + 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 |
 | `navigation-coordinators.mdc` | Navigation / App | NavigationStack, deep links |
 | `networking-services.mdc` | Data, Network | API, DTO, mappers (HTTP only) |
@@ -39,10 +41,28 @@
 | `tests-unit.mdc` | `*Tests` | Unit |
 | `tests-ui.mdc` | `*UITests` | XCUITest (+ pointer to xcuitest agents/skill) |
 
-**Всего topic `.mdc`:** 21 (без README).
+**Всего topic `.mdc`:** 21 domain + 5 preset-meta (без README).
 
 **Twin depth (Cursor↔Claude):** domain rules ≥15 body lines. Thin requestable **aliases** (`agent-team-intake`, `technical-retro`) могут быть короче — как в next (soft AC-19).
 
+### Shared core → stack mapping
+
+SoT prose: `packages/ai-rules/presets/_shared/core/**`. Adapters: `<!-- shared-core: … -->`.
+
+| Shared core | This stack adapter | Delta |
+|-------------|--------------------|-------|
+| `quality/code-quality-and-refactoring.md` | `code-quality-and-refactoring.mdc` | Stack notes: force unwrap, DesignSystem |
+| `quality/design-guidance.md` | `design-guidance.mdc` | requestable smells/GoF |
+| `agent-team/agent-team-orchestrator.md` | `agent-team-orchestrator.mdc` | Roles (XCUITest); preset PR pointer |
+| `agent-team/agent-team-intake.md` | `agent-team-intake.mdc` | thin + iOS scope note |
+| `architecture/feature-delivery-workflow.md` | `feature-delivery-workflow.mdc` | Feature modules checklist |
+| `architecture/reference-features.template.md` | `reference-features.mdc` | `Features/**` table + dogfood |
+| `review/code-review-mr.md` | `code-review-mr.mdc` | iOS checklist |
+| `review/technical-retro.md` | `technical-retro.mdc` | thin alias |
+| `meta/preset-*.md` | `preset-*.mdc` | author-only globs |
+
+Claude map: `presets/claude/ios-swift/rules/README.md`. Taxonomy: `presets/_shared/README.md`.
+
 ### Presentation edit bundle
 
 При правке `**/Presentation/**` или `**/DesignSystem/**`: грузить `swiftui-ui` + `state-and-viewmodels` (+ `navigation-coordinators` при правке навигации). Не подмешивать orchestrator.
@@ -55,7 +75,8 @@ alwaysApply (3)
 
 requestable
   └─ orchestrator, intake, feature-delivery, code-review-mr,
-     post-change-build, xcode-tooling, reference-features, technical-retro
+     post-change-build, xcode-tooling, reference-features, technical-retro,
+     preset-* (author meta on packages/ai-rules/presets/**)
 
 globs
   └─ swiftui, navigation, viewmodels, networking, persistence,

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

@@ -3,6 +3,8 @@ 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:
@@ -12,5 +14,3 @@ When the user message looks like a **work request** (implement, add, fix, refact
 3. Pure questions («как работает X», «объясни») — answer normally, no `/task`.
 
 Exceptions: user explicitly says «без pipeline», «просто сделай», or continues an active slug.
-
-Scope: iOS/Swift Feature modules (`Features/`, Presentation/Domain/Data) — not `app/src` / Playwright.

package/presets/cursor/ios-swift/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 (iOS)
+## Stack notes
+
+### Roles (iOS)
 
 | Agent | Prod writes | Typical intent |
 |-------|-------------|----------------|
@@ -31,7 +35,7 @@ Work request без `/task` — см. **`agent-team-intake.mdc`**.
 | `debugger` | minimal | bugfix |
 | `ci-investigator` | minimal CI | ci-fix |
 | `feature-developer` | yes | feature, bugfix, refactor, migration, a11y |
-| `build-verifier` | no | xcodebuild + SwiftLint/Format + unit smoke |
+| `build-verifier` | no | xcodebuild + SwiftLint/Format + unit smoke; preset-structure for preset PRs |
 | `accessibility-reviewer` | no | VoiceOver / Dynamic Type |
 | `performance-auditor` | no | launch / Instruments |
 | `code-reviewer` | no | most pipelines, review-only |
@@ -47,6 +51,8 @@ Work request без `/task` — см. **`agent-team-intake.mdc`**.
 
 Full prompts: `.cursor/agents/*.md`. Artifacts: `.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
 
 **Source of truth for order:** `pipeline.json` → `steps[]`. Never hardcode analyst → dev → review → QA when `pipeline.json` exists.

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

@@ -3,37 +3,70 @@ description: Поддержка и улучшение качества кода
 alwaysApply: true
 ---
 
+<!-- shared-core: quality/code-quality-and-refactoring.md -->
+
 # Поддержка существующего стиля
 
 - Новые изменения должны:
-  - следовать существующим паттернам (имена, структура, DI, error handling);
-  - минимизировать «стилистический шум» (лишний reformat, rename без нужды).
-- Перед новым решением:
-  - искать аналог в коде и **повторять подход**;
-  - проверять DesignSystem и существующие компоненты перед новым UI-контролом;
-  - обращаться к другим фичам только через public API (протоколы, facades).
+  - следовать существующим паттернам (имена, структура, типизация / DI / error handling стека);
+  - минимизировать «стилистический шум» (лишние правки форматирования, rename без нужды).
+- Перед добавлением нового решения:
+  - искать аналогичное в коде и **повторять подход**, а не изобретать новый;
+  - проверять, нет ли уже подходящего компонента или паттерна в дизайн‑системе / 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 модуля.
 
 # Рефакторинг при изменениях
 
-- Допустим лёгкий refactor, если он:
+- Разрешён лёгкий refactor, если он:
   - уменьшает дублирование;
   - повышает читаемость;
-  - не ломает публичные контракты.
-- Примеры:
-  - вынести дублирующийся mapper или validation;
-  - заменить force unwrap на безопасные конструкции;
-  - разбить перегруженный ViewModel на use cases;
-  - заменить magic numbers/colors на design tokens;
-  - убрать прямую зависимость View от Data в пользу протокола.
+  - не ломает публичные контракты модулей.
+- Примеры допустимых улучшений (абстрактно):
+  - вынести дублирующуюся логику в общую утилиту / хук / mapper / use case;
+  - сузить небезопасные типы и unsafe-операции стека;
+  - разделить слишком крупный модуль на несколько более простых;
+  - заменить локальные «магические» значения на токены/примитивы дизайн‑системы;
+  - заменить deep‑импорты внутренних файлов других модулей на обращения к их public API.
 
 # Ограничения
 
-- Не делать «большой» рефакторинг при точечной задаче:
-  - не менять структуру модулей/таргетов;
-  - не переименовывать public API без запроса.
-- Крупные изменения — минимальными шагами, код остаётся компилируемым.
+- Не выполнять «большой» рефакторинг, если задача точечная и не про архитектуру:
+  - не менять структуру директорий / модулей / таргетов без нужды;
+  - не менять названия публичных типов/функций/API без явного запроса.
+- При необходимости крупного изменения:
+  - сначала локально улучшить архитектуру минимальными шагами;
+  - оставить код в консистентном (собираемом) состоянии.
 
 # Требование к агенту
 
-- **Boy scout rule:** оставлять модуль чуть лучше (безопасность optional, typed errors, tests).
-- Не жертвовать слоями ради краткости.
+- **Boy scout rule:** оставлять модуль немного лучше, чем до изменения (простые, безопасные улучшения).
+- Не жертвовать архитектурой и слоями ради краткости реализации.
+
+> Stack-specific lint gates, примеры и toolchain — в секции **Stack notes** адаптера пресета.
+
+## 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).

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

@@ -3,8 +3,40 @@ description: Требования к code review для iOS merge requests. Use
 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`.
+
+## When to load
+
+- User asks for review/MR/diff
+- Pipeline step `code-reviewer`
+- 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`).
+
+## Stack notes
+
 On-demand правило для review-потоков (не always-on).
 
 - **Когда применять**