@bonesofspring/ai-rules 0.2.0 → 0.2.1

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

@@ -17,7 +17,7 @@ alwaysApply: true
 
 | Слой | Каталог | Детали |
 |------|---------|--------|
-| UI | `app/src/ui/**` (pages, components) | `react-ui.mdc`, `architecture-boundaries.mdc` |
+| UI | `app/src/ui/**` (pages, components) | `react-ui.mdc`, `architecture-boundaries-ui.mdc` |
 | Store | `app/src/store/**` (slices, middleware) | `store-rtk.mdc` |
 | API | `app/src/api/**` (services, clients, barrel) | `api-services.mdc`, `http-client.mdc` |
 | HTTP | `app/src/lib/clients/**`, `app/src/api/clients/**` | `http-client.mdc` |

package/presets/cursor/next/rules/no-cross-component-styles-import.mdc

@@ -1,58 +1,8 @@
 ---
-description: Стили компонентов — колокация, импорты, корневой styled как Root
-globs:
-  - app/src/ui/**/*.ts
-  - app/src/ui/**/*.tsx
+description: Стили компонентов — consolidated into react-ui.mdc
 alwaysApply: false
 ---
 
-# Стили только рядом с компонентом
+# Moved to react-ui.mdc
 
-## Имя корневого styled-элемента: `Root`
-
-- В `styles.ts` (или аналоге) **корневой** styled-элемент — тот, что оборачивает весь JSX компонента — экспортируется как **`Root`**.
-- В разметке: `<s.Root>...</s.Root>` при `import * as s from './styles'`.
-- Вложенные и соседние примитивы именуются по смыслу: `Title`, `List`, `Item`, `Header` и т.д.
-- Если нет одной styled-обёртки на весь компонент (только фрагмент или один нативный тег без своего styled), экспорта `Root` может не быть. Когда вводится **одна** внешняя styled-обёртка всего JSX — она называется **`Root`**, а не `Container`, `Wrapper`, `Block` и т.п.
-
-```tsx
-// ✅ Хорошо
-export const Root = styled.div` ... `
-// в компоненте: <s.Root>...</s.Root>
-
-// ❌ Плохо для единственной обёртки всего компонента
-export const Service = styled.div` ... `
-```
-
-## Правило
-
-**Запрещено** импортировать модули стилей, которые лежат в каталоге **другого** UI-компонента или блока (не в том же каталоге, что и текущий файл).
-
-Под «модулями стилей» имеются в виду в первую очередь:
-
-- `styles.ts` / `styles.tsx` рядом с компонентом;
-- `*.module.css`, `*.module.scss` и аналоги, принадлежащие конкретному компоненту;
-- любые файлы, которые **экспортируют только styled-примитивы** для одного компонента.
-
-## Разрешено
-
-- `import * as s from './styles'` — стили **в той же папке**, что и компонент.
-- Импорты **общих** примитивов дизайн-системы, токенов, общих UI из `@/ui/components/...` или пакетов, если это **не** приватный `styles` чужой фичи.
-- Повторное использование визуала через **сам компонент** (композиция), а не через его `styles`.
-
-## Примеры
-
-```tsx
-// ✅ Хорошо — локальный styles
-import * as s from './styles'
-
-// ❌ Плохо — стили родителя/соседа
-import * as s from '../../styles'
-import * as s from '../RequestForAnalysisServices/styles'
-```
-
-## Почему
-
-- Единое имя **`Root`** ускоряет чтение: сразу видно входную точку разметки компонента.
-- Стили и разметка компонента должны меняться вместе, без скрытой связи через чужие файлы.
-- Упрощается рефакторинг и поиск владельца стилей.
+Правила колокации стилей, имени **`Root`** и запрета импорта чужих `styles` — в **`react-ui.mdc`** § «Стили и дизайн‑токены». Правило подгружается автоматически при правках `app/src/ui/**`.

package/presets/cursor/next/rules/react-ui.mdc

@@ -27,7 +27,7 @@ alwaysApply: false
 - Локальные подкомпоненты, используемые только этим блоком, выносятся в **`./components/<ComponentName>/`** относительно папки родительского компонента (не оставлять крупные куски JSX и «внутренние» компоненты в том же файле, что и родитель).
 - У каждого такого подкомпонента — **своя** папка с тем же именем, что и публичное имя компонента:
   - `ComponentName.tsx` — разметка и композиция;
-  - `styles.ts` — только стили этого подкомпонента (см. `no-cross-component-styles-import.mdc`: не тянуть `styles` соседних компонентов);
+  - `styles.ts` — только стили этого подкомпонента (не тянуть `styles` соседних компонентов — см. § Стили ниже);
   - при необходимости — `ComponentName.data.ts`, `ComponentName.utils.ts`, `ComponentName.hooks.ts` по тем же правилам префикса, что и у родителя;
   - опционально `index.ts` с реэкспортом.
 - Родитель импортирует подкомпонент из `./components/...`, а не держит его реализацию inline.
@@ -56,8 +56,9 @@ alwaysApply: false
 
 # Стили и дизайн‑токены
 
-- **Корневой** styled-элемент компонента в соседнем `styles` — **`Root`** (`<s.Root>`). Подробнее: `no-cross-component-styles-import.mdc`.
-- Порядок объявлений в CSS / `styled` — как требует Stylelint: см. `css-property-order-stylelint.mdc`.
+- **Корневой** styled в соседнем `styles` — экспорт **`Root`**; в JSX: `<s.Root>` при `import * as s from './styles'`. Вложенные — `Title`, `List` и т.д. Не `Container`/`Wrapper` для единственной обёртки.
+- **Запрещено** импортировать `styles` другого компонента (`../Other/styles`, `../../styles`). Разрешено: `./styles`, общие примитивы из `@/ui/components/...` или пакетов дизайн-системы.
+- Порядок CSS / `styled` — Stylelint idiomatic-order: `css-property-order-stylelint.mdc` (грузится на `styles.ts` / `.css`).
 - Для визуала использовать **тот стек стилей и токенов, который уже в проекте** (переменные, тема, общие классы, дизайн‑пакет).
 - Избегать:
   - inline‑стилей, кроме простых случаев;

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

@@ -7,6 +7,7 @@
 ```
 .cursor/team/
   active-task.json
+  fixtures/                  # pipeline profile examples (feature-full, feature-light, bugfix-standard)
   tasks/
     <slug>/
       pipeline.json

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

@@ -0,0 +1,15 @@
+{
+  "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": "Validation gate" },
+    { "agent": "code-reviewer", "label": "Code review" }
+  ],
+  "humanGates": [],
+  "autoChain": true,
+  "skipped": []
+}

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

@@ -0,0 +1,16 @@
+{
+  "slug": "example-feature-full",
+  "intent": "feature",
+  "profile": "full",
+  "summary": "Multi-layer feature with e2e coverage",
+  "steps": [
+    { "agent": "task-analyst", "label": "Clarify and decompose" },
+    { "agent": "feature-developer", "label": "Implement + unit tests", "scope": "unit-in-dev" },
+    { "agent": "build-verifier", "label": "Validation gate" },
+    { "agent": "code-reviewer", "label": "Code review" },
+    { "agent": "qa-tester", "label": "E2E", "scope": "e2e-only" }
+  ],
+  "humanGates": ["after:task-analyst"],
+  "autoChain": true,
+  "skipped": []
+}

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

@@ -0,0 +1,17 @@
+{
+  "slug": "example-feature-light",
+  "intent": "feature",
+  "profile": "light",
+  "summary": "Trivial single-file change with explicit AC",
+  "steps": [
+    { "agent": "feature-developer", "label": "Implement" },
+    { "agent": "build-verifier", "label": "Validation gate" }
+  ],
+  "humanGates": [],
+  "autoChain": true,
+  "skipped": [
+    { "agent": "task-analyst", "reason": "Explicit AC and single-file scope" },
+    { "agent": "code-reviewer", "reason": "Light profile — non-trivial app changes should use standard" },
+    { "agent": "qa-tester", "reason": "No e2e AC" }
+  ]
+}

package/scripts/README.md

@@ -0,0 +1,84 @@
+# Preset validation scripts
+
+Static regression checks for `presets/cursor/next` and `presets/claude/next`.
+
+## Quick start
+
+```bash
+cd packages/ai-rules
+
+# Full static validation (no LLM)
+./scripts/validate-preset.sh
+
+# + init smoke into temp dirs
+./scripts/validate-preset.sh --init-smoke
+
+# Validate golden-prompts.json (agent names, schema)
+./scripts/validate-preset.sh --check-golden
+```
+
+Or via npm:
+
+```bash
+yarn validate
+yarn validate:init
+yarn validate:golden
+```
+
+## Router regression (manual + semi-auto)
+
+Golden prompts live in `golden-prompts.json` — one case per `task-router` intent (+ light/full variants).
+
+### 1. Print checklist
+
+```bash
+./scripts/validate-preset.sh --print-golden
+```
+
+### 2. Run in Cursor (fresh chat each)
+
+For each row, run `/task <prompt>`. After **task-router** finishes, copy `pipeline.json` from `.cursor/team/tasks/<slug>/` to:
+
+```
+scripts/regression-results/<id>.json
+```
+
+Use the `id` from golden-prompts (e.g. `feature-standard.json`).
+
+### 3. Compare outputs
+
+```bash
+mkdir -p scripts/regression-results
+./scripts/validate-preset.sh --check-golden --compare-pipeline scripts/regression-results/
+```
+
+The compare step checks:
+
+- `intent`, `firstAgent`
+- `requiredSequence` (ordered subsequence — extra steps allowed)
+- `mustNotInclude`
+- `stepConstraints` (`skipIf`)
+- `humanGates`
+
+`profile` mismatches are warnings (router may omit explicit profile when default applies).
+
+### 4. Commit baseline (optional)
+
+When router behavior intentionally changes, update `golden-prompts.json` and refresh committed `regression-results/` snapshots.
+
+## Files
+
+| File | Purpose |
+|------|---------|
+| `validate-preset.sh` | Main validator |
+| `preset-manifest.json` | Always-on lists, rule pairs, UI glob rules, BUGBOT sync |
+| `golden-prompts.json` | Expected router outcomes per intent |
+| `regression-results/.gitkeep` | Placeholder for manual run outputs |
+
+## What is NOT automated
+
+- LLM routing quality (flaky by model/temperature)
+- Behavioral golden tasks on a real Next.js repo
+- Token cost of path-scoped rules on real file edits
+
+Those stay manual / scheduled regression on a dogfood project.

package/scripts/golden-prompts.json

@@ -0,0 +1,276 @@
+{
+  "version": 1,
+  "description": "Golden prompts for task-router regression. Run manually via /task or semi-auto with validate-preset.sh --check-golden.",
+  "prompts": [
+    {
+      "id": "feature-standard",
+      "prompt": "Добавь фильтр по дате в Order History",
+      "intent": "feature",
+      "profile": "standard",
+      "firstAgent": "task-analyst",
+      "requiredSequence": [
+        "task-analyst",
+        "feature-developer",
+        "build-verifier",
+        "code-reviewer"
+      ],
+      "mustInclude": ["build-verifier"],
+      "mustNotInclude": [],
+      "humanGates": ["after:task-analyst"],
+      "notes": "Default feature path; qa-tester optional unless e2e AC explicit."
+    },
+    {
+      "id": "feature-full",
+      "prompt": "Реализуй оплату картой: UI, API сервис, store и e2e сценарии checkout",
+      "intent": "feature",
+      "profile": "full",
+      "firstAgent": "task-analyst",
+      "requiredSequence": [
+        "task-analyst",
+        "feature-developer",
+        "build-verifier",
+        "code-reviewer",
+        "qa-tester"
+      ],
+      "mustInclude": ["build-verifier", "qa-tester"],
+      "mustNotInclude": [],
+      "humanGates": ["after:task-analyst"],
+      "notes": "Cross-layer + e2e AC → full profile with qa-tester."
+    },
+    {
+      "id": "feature-light",
+      "prompt": "Исправь typo в тексте кнопки Submit в OrderHistoryButton — один файл app/src/ui/components/OrderHistoryButton/OrderHistoryButton.tsx, AC: label «Отправить»",
+      "intent": "feature",
+      "profile": "light",
+      "firstAgent": "feature-developer",
+      "requiredSequence": ["feature-developer", "build-verifier"],
+      "mustInclude": ["build-verifier"],
+      "mustNotInclude": ["task-analyst"],
+      "humanGates": [],
+      "notes": "Explicit single-file AC → light; skip analyst."
+    },
+    {
+      "id": "bugfix-standard",
+      "prompt": "Order History падает при пустом списке — пофикси regression",
+      "intent": "bugfix",
+      "profile": "standard",
+      "firstAgent": "debugger",
+      "requiredSequence": [
+        "debugger",
+        "feature-developer",
+        "build-verifier",
+        "code-reviewer"
+      ],
+      "mustInclude": ["debugger", "build-verifier"],
+      "mustNotInclude": [],
+      "humanGates": [],
+      "stepConstraints": [
+        {
+          "agent": "feature-developer",
+          "skipIf": "debugger.fixed"
+        }
+      ],
+      "notes": "Bugfix standard; qa-tester(regression) optional per router."
+    },
+    {
+      "id": "bugfix-light",
+      "prompt": "Просто поправь null-check в mapper — один файл, явный AC, без архитектурного риска",
+      "intent": "bugfix",
+      "profile": "light",
+      "firstAgent": "debugger",
+      "requiredSequence": ["debugger", "build-verifier"],
+      "mustInclude": ["build-verifier"],
+      "mustNotInclude": ["task-analyst", "code-reviewer"],
+      "humanGates": [],
+      "notes": "Light bugfix may skip developer when debugger.fixed."
+    },
+    {
+      "id": "ci-fix",
+      "prompt": "CI упал на type-check в PR — красный пайплайн",
+      "intent": "ci-fix",
+      "profile": "standard",
+      "firstAgent": "ci-investigator",
+      "requiredSequence": [
+        "ci-investigator",
+        "feature-developer",
+        "build-verifier",
+        "code-reviewer"
+      ],
+      "mustInclude": ["ci-investigator", "build-verifier"],
+      "mustNotInclude": ["task-analyst"],
+      "humanGates": [],
+      "stepConstraints": [
+        {
+          "agent": "feature-developer",
+          "skipIf": "ci-investigator.resolved"
+        }
+      ]
+    },
+    {
+      "id": "migration",
+      "prompt": "Миграция HTTP-клиента на fetch — upgrade major version",
+      "intent": "migration",
+      "profile": "standard",
+      "firstAgent": "task-analyst",
+      "requiredSequence": [
+        "task-analyst",
+        "migration-specialist",
+        "feature-developer",
+        "build-verifier",
+        "code-reviewer"
+      ],
+      "mustInclude": ["migration-specialist", "build-verifier"],
+      "mustNotInclude": [],
+      "humanGates": ["after:migration-specialist"]
+    },
+    {
+      "id": "perf-audit",
+      "prompt": "Страница Order History медленно грузится — perf audit bundle и LCP",
+      "intent": "perf-audit",
+      "profile": "light",
+      "firstAgent": "performance-auditor",
+      "requiredSequence": ["performance-auditor"],
+      "mustInclude": ["performance-auditor"],
+      "mustNotInclude": ["feature-developer", "build-verifier"],
+      "humanGates": []
+    },
+    {
+      "id": "a11y",
+      "prompt": "Проверь a11y формы логина — WCAG и keyboard navigation",
+      "intent": "a11y",
+      "profile": "standard",
+      "firstAgent": "task-analyst",
+      "requiredSequence": [
+        "task-analyst",
+        "feature-developer",
+        "build-verifier",
+        "accessibility-reviewer",
+        "code-reviewer"
+      ],
+      "mustInclude": ["accessibility-reviewer", "build-verifier"],
+      "mustNotInclude": [],
+      "humanGates": ["after:task-analyst"]
+    },
+    {
+      "id": "review-only",
+      "prompt": "Сделай ревью моих изменений в store/orders — review MR diff",
+      "intent": "review-only",
+      "profile": "light",
+      "firstAgent": "code-reviewer",
+      "requiredSequence": ["code-reviewer"],
+      "mustInclude": ["code-reviewer"],
+      "mustNotInclude": ["feature-developer", "build-verifier", "task-analyst"],
+      "humanGates": []
+    },
+    {
+      "id": "test-only",
+      "prompt": "Покрой тестами модуль payments — unit и e2e, без изменения прод-кода",
+      "intent": "test-only",
+      "profile": "standard",
+      "firstAgent": "task-analyst",
+      "requiredSequence": ["task-analyst", "qa-tester"],
+      "mustInclude": ["qa-tester"],
+      "mustNotInclude": ["feature-developer"],
+      "humanGates": ["after:task-analyst"]
+    },
+    {
+      "id": "unit-only-plan",
+      "prompt": "Покрой unit-тестами mapper payments в api/services",
+      "intent": "unit-only",
+      "profile": "standard",
+      "firstAgent": "unit-test-planner",
+      "requiredSequence": ["unit-test-planner", "unit-test-generator"],
+      "mustInclude": ["unit-test-planner"],
+      "mustNotInclude": ["feature-developer", "qa-tester"],
+      "humanGates": ["after:unit-test-planner"],
+      "notes": "New unit coverage → planner then generator."
+    },
+    {
+      "id": "unit-only-heal",
+      "prompt": "Почини unit тест OrderHistoryMapper.spec.ts — падает после рефактора",
+      "intent": "unit-only",
+      "profile": "standard",
+      "firstAgent": "unit-test-healer",
+      "requiredSequence": ["unit-test-healer"],
+      "mustInclude": ["unit-test-healer"],
+      "mustNotInclude": ["unit-test-planner", "feature-developer"],
+      "humanGates": [],
+      "notes": "Broken existing unit → healer path, no planner gate."
+    },
+    {
+      "id": "e2e-only-plan",
+      "prompt": "Напиши e2e для сценария оплаты — playwright browser test",
+      "intent": "e2e-only",
+      "profile": "standard",
+      "firstAgent": "playwright-test-planner",
+      "requiredSequence": [
+        "playwright-test-planner",
+        "playwright-test-generator"
+      ],
+      "mustInclude": ["playwright-test-planner"],
+      "mustNotInclude": ["feature-developer", "qa-tester"],
+      "humanGates": ["after:playwright-test-planner"]
+    },
+    {
+      "id": "e2e-only-heal",
+      "prompt": "Почини e2e checkout.spec.ts — flaky playwright test",
+      "intent": "e2e-only",
+      "profile": "standard",
+      "firstAgent": "playwright-test-healer",
+      "requiredSequence": ["playwright-test-healer"],
+      "mustInclude": ["playwright-test-healer"],
+      "mustNotInclude": ["playwright-test-planner", "feature-developer"],
+      "humanGates": []
+    },
+    {
+      "id": "docs-only",
+      "prompt": "Напиши changelog для релиза фильтров и migration guide",
+      "intent": "docs-only",
+      "profile": "standard",
+      "firstAgent": "task-analyst",
+      "requiredSequence": ["task-analyst", "tech-writer"],
+      "mustInclude": ["tech-writer"],
+      "mustNotInclude": ["feature-developer", "build-verifier"],
+      "humanGates": ["after:task-analyst"]
+    },
+    {
+      "id": "refactor",
+      "prompt": "Рефакторинг store/orders без изменения поведения — почисти дубли",
+      "intent": "refactor",
+      "profile": "standard",
+      "firstAgent": "task-analyst",
+      "requiredSequence": [
+        "task-analyst",
+        "feature-developer",
+        "build-verifier",
+        "code-reviewer"
+      ],
+      "mustInclude": ["build-verifier"],
+      "mustNotInclude": [],
+      "humanGates": ["after:task-analyst"]
+    },
+    {
+      "id": "spike",
+      "prompt": "Исследуй spike: можно ли перейти на React Server Components для Order History?",
+      "intent": "spike",
+      "profile": "standard",
+      "firstAgent": "task-analyst",
+      "requiredSequence": ["task-analyst", "solution-architect"],
+      "mustInclude": ["solution-architect"],
+      "mustNotInclude": ["feature-developer", "build-verifier"],
+      "humanGates": ["after:solution-architect"]
+    },
+    {
+      "id": "retro",
+      "prompt": "Сделай ретро по задаче order-history-date-filter — postmortem",
+      "intent": "retro",
+      "noPipeline": true,
+      "firstAgent": null,
+      "requiredSequence": [],
+      "mustInclude": [],
+      "mustNotInclude": ["feature-developer", "task-analyst"],
+      "humanGates": [],
+      "notes": "Orchestrator runs /technical-retro; no pipeline.json steps."
+    }
+  ]
+}

package/scripts/preset-manifest.json

@@ -0,0 +1,72 @@
+{
+  "version": 1,
+  "alwaysOnCursor": [
+    "next-app-core.mdc",
+    "code-quality-and-refactoring.mdc",
+    "post-change-lint.mdc",
+    "package-manager.mdc"
+  ],
+  "alwaysOnClaude": [
+    "stack/next-app-core.md",
+    "tooling-and-review/code-quality.md",
+    "tooling-and-review/post-change-lint.md",
+    "tooling-and-review/package-manager.md"
+  ],
+  "alwaysOnMaxBytes": 12288,
+  "rulePairs": [
+    ["architecture-boundaries.mdc", "architecture/architecture-boundaries.md"],
+    ["architecture-boundaries-ui.mdc", "architecture/architecture-boundaries-ui.md"],
+    ["layer-barrel-exports.mdc", "architecture/layer-barrel-exports.md"],
+    ["navigation-router-stack.mdc", "stack/navigation-router.md"],
+    ["navigation-router-ui.mdc", "stack/navigation-router-ui.md"],
+    ["react-ui.mdc", "ui-and-accessibility/react-ui.md"],
+    ["code-review-mr.mdc", "tooling-and-review/code-review-mr.md"],
+    ["agent-team-orchestrator.mdc", "tooling-and-review/agent-team-orchestrator.md"],
+    ["public-imports.mdc", "architecture/public-imports.md"],
+    ["feature-delivery-workflow.mdc", "architecture/feature-delivery-workflow.md"],
+    ["next-app-core.mdc", "stack/next-app-core.md"],
+    ["post-change-lint.mdc", "tooling-and-review/post-change-lint.md"],
+    ["package-manager.mdc", "tooling-and-review/package-manager.md"],
+    ["code-quality-and-refactoring.mdc", "tooling-and-review/code-quality.md"]
+  ],
+  "syncedRootFiles": [
+    {
+      "source": "fragments/BUGBOT.md",
+      "targets": ["presets/cursor/next/BUGBOT.md", "presets/claude/next/BUGBOT.md"]
+    }
+  ],
+  "parityDirs": ["agents", "skills", "team/fixtures"],
+  "uiGlobChecks": [
+    {
+      "id": "architecture-boundaries-no-ui",
+      "file": "architecture-boundaries.mdc",
+      "mustNotMatch": "app/src/ui"
+    },
+    {
+      "id": "architecture-boundaries-ui-scoped",
+      "file": "architecture-boundaries-ui.mdc",
+      "mustMatch": "app/src/ui"
+    },
+    {
+      "id": "layer-barrel-index-only",
+      "file": "layer-barrel-exports.mdc",
+      "mustMatch": "index.ts",
+      "mustNotMatch": "app/src/ui/\\*\\*"
+    },
+    {
+      "id": "navigation-router-stack-app-only",
+      "file": "navigation-router-stack.mdc",
+      "mustMatch": "app/src/app",
+      "mustNotMatch": "app/src/ui"
+    },
+    {
+      "id": "navigation-router-ui-tsx",
+      "file": "navigation-router-ui.mdc",
+      "mustMatch": "app/src/ui/\\*\\*/\\*\\.tsx"
+    }
+  ],
+  "wideGlobDenylist": [
+    "architecture-boundaries.mdc",
+    "navigation-router-stack.mdc"
+  ]
+}