@bonesofspring/ai-rules 0.2.1 → 0.2.2

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

@@ -1,6 +1,6 @@
 ---
 description: Паттерны React/Next UI и стилей (preset)
-globs: app/src/ui/**/*.tsx,app/src/ui/**/*.ts
+globs: src/ui/**/*.tsx
 alwaysApply: false
 ---
 
@@ -22,43 +22,14 @@ alwaysApply: false
 - `styles.ts` или соседний модуль стилей — по конвенции репозитория (CSS Modules, CSS-in-JS, и т.д.).
 - `index.ts` — реэкспорт (если нужно наружу).
 
-## Дочерние компоненты (`./components`)
-
-- Локальные подкомпоненты, используемые только этим блоком, выносятся в **`./components/<ComponentName>/`** относительно папки родительского компонента (не оставлять крупные куски JSX и «внутренние» компоненты в том же файле, что и родитель).
-- У каждого такого подкомпонента — **своя** папка с тем же именем, что и публичное имя компонента:
-  - `ComponentName.tsx` — разметка и композиция;
-  - `styles.ts` — только стили этого подкомпонента (не тянуть `styles` соседних компонентов — см. § Стили ниже);
-  - при необходимости — `ComponentName.data.ts`, `ComponentName.utils.ts`, `ComponentName.hooks.ts` по тем же правилам префикса, что и у родителя;
-  - опционально `index.ts` с реэкспортом.
-- Родитель импортирует подкомпонент из `./components/...`, а не держит его реализацию inline.
-
 Для переиспользуемого компонента:
 - Папка с именем компонента:
   - `ComponentName.tsx`
   - файлы стилей по принятой схеме;
-  - опционально: `types.ts`, `ComponentName.hooks.ts` (см. ниже про префикс).
-
-## Вспомогательные модули рядом с компонентом (именование)
-
-**Префикс имени файла = публичное имя компонента** (как у `ComponentName.tsx` в этой папке), в **PascalCase**. Не вводить отдельные «говорящие» имена файлов по смыслу содержимого (`analysisPreparationIconMap.ts`, `buildAnalysisIdToNameMap.ts` и т.п.) — так теряется связь с компонентом и плодятся одноразовые названия.
-
-**Суффикс по роли:**
-
-| Суффикс | Назначение | Примеры содержимого |
-|--------|------------|---------------------|
-| `ComponentName.data.ts` | Статические данные и конфигурация для UI | мапы `id → иконка/лейбл`, константы списков, таблицы соответствий для отображения |
-| `ComponentName.utils.ts` | Чистые функции без React | форматирование, предобработка пропсов/данных для рендера, `build…`/`map…`‑хелперы |
-| `ComponentName.hooks.ts` | Хуки, используемые только этим блоком | локальные `use…` (если не вынесены в `src/ui/hooks/**`) |
-
-- Несколько констант/мапов или несколько функций — **по-прежнему один** `.data.ts` и один `.utils.ts`, не дробить по «темам» отдельными файлами без веской причины (размер, разные зоны ответственности на уровне подкомпонентов).
-- Если логика принадлежит **подкомпоненту** в подпапке (`components/Child/Child.tsx`), те же правила применяются к **`Child.data.ts`**, **`Child.utils.ts`** относительно этого подкомпонента.
-- Тесты для утилит и данных — рядом в `__tests__/` или с суффиксом `.test.ts`, согласно `tests-unit.mdc`, с тем же префиксом (`RequestForAnalysisRecommendations.utils.test.ts` и т.д.).
+  - опционально: `types.ts`, `hooks.ts`.
 
 # Стили и дизайн‑токены
 
-- **Корневой** 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‑стилей, кроме простых случаев;
@@ -68,11 +39,10 @@ alwaysApply: false
 
 # Пропсы и типизация
 
-- **Не передавать пропы в компоненты через spread** (`<Foo {...x} />`). Только явные атрибуты; подробности и исключения — `no-props-spread.mdc` (в `app/src/ui` это дополнительно ловит ESLint).
 - Описывать пропсы через `type Props = { ... }` или `interface Props { ... }`.
 - Не использовать `any`; при необходимости:
   - обобщения (`<T>`), `unknown`, тип‑предикаты и user‑defined type guards.
-- Для доменных сущностей использовать типы из `@/types` (и enum из `@/types/enums`), а не описывать их заново (`public-imports.mdc`).
+- Для доменных сущностей использовать типы из `@/types/**`, а не описывать их заново.
 
 # Логика и side effects
 

package/presets/cursor/next/rules/store-rtk.mdc

@@ -1,6 +1,6 @@
 ---
 description: Redux Toolkit и состояние приложения
-globs: app/src/store/**/*.ts
+globs: src/store/**/*.ts
 alwaysApply: false
 ---
 
@@ -16,7 +16,7 @@ alwaysApply: false
 
 # Структура слайсов
 
-- Каждый доменный модуль — свой слайс в `app/src/store/slices/**`.
+- Каждый доменный модуль — свой слайс в `src/store/slices/**`.
 - Слайс экспортирует:
   - `reducer` по умолчанию;
   - `actions` именованным экспортом;
@@ -30,23 +30,17 @@ alwaysApply: false
 - Асинхронные запросы:
   - через `createAsyncThunk` или RTK Query.
   - внутри thunk:
-    - вызывать API через сервисы, импортированные из `@/api` (**`public-imports.mdc`**);
-    - не вызывать HTTP‑клиент напрямую — только через эти сервисы.
+    - вызывать API через сервисы из `@/api/services/**`;
+    - не вызывать HTTP‑клиент напрямую — только через сервисы `@/api/services/**`.
 - Сайд‑эффекты (логирование, аналитика, работа с файлами):
-  - выносить в middleware (`app/src/store/middleware/**`) или специализированные слайсы.
+  - выносить в middleware (`src/store/middleware/**`) или специализированные слайсы.
 
 # Типизация
 
 - Использовать `RootState`, `AppDispatch` и типизированные хуки `useAppDispatch`, `useAppSelector` (если есть).
-- **HTTP и ошибки API**:
-  - не импортировать типы **сторонних HTTP‑библиотек**, которых нет в актуальных зависимостях и существующих слайсах (ориентир — **`package.json`** и соседние файлы);
-  - когда сервис возвращает **обёртку ответа** прикладного клиента — использовать **соответствующий тип из `@/types`** (как в barrel и в аналогичных thunk’ах);
-  - при ошибках после вызова сервиса — опираться на **тот же класс/контракт ошибки транспорта**, что использует общий клиент (из `@/types`), и разбирать тело/статус **по полям текущей реализации**, а не по воображаемому API;
-  - в **`catch`** предпочитать **`instanceof`** на класс ошибки транспорта из `@/types` (если он есть в коде) вместо голого `as`, когда это выразимо без шума (см. `no-type-assertion-as-import-export.mdc`).
 - Для сущностей:
-  - доменные типы (включая вычисляемые поля) определять в `app/src/types/**`, экспортировать через barrel и импортировать в слайсы из `@/types` как **источник правды** для структуры данных (`public-imports.mdc`);
+  - доменные типы (включая вычисляемые поля) определять в `@/types/**` и импортировать в слайсы как **источник правды** для структуры данных;
   - избегать дублирования описаний сущностей в нескольких местах.
-- **Граница домена**: в **state** хранить доменные модели; тип **обёртки ответа клиента** допустим как тип **возвращаемого значения thunk** или промежуточно до маппинга — без дублирования DTO в полях state без нужды (согласовано с `api-services.mdc` и `http-client.mdc`).
 
 # Тестирование слайсов
 
@@ -62,5 +56,4 @@ alwaysApply: false
 - Не класть логику API внутрь редюсеров/компонентов.
 - Строго типизировать state и actions.
 - Использовать единый стиль именования actions и селекторов, как в существующих слайсах.
-- При работе с ошибками и ответами HTTP опираться на **`http-client.mdc`** и контракты из `@/types`, а не на типы внешних HTTP‑библиотек вне зависимостей проекта.
 

package/presets/cursor/next/rules/tests-unit.mdc

@@ -1,31 +1,17 @@
 ---
 description: Unit и интеграционные тесты (Testing Library и runner проекта)
-globs:
-  - app/src/**/*.spec.{ts,tsx}
-  - app/src/**/*.test.{ts,tsx}
+globs: src/**/*.test.{ts,tsx}
 alwaysApply: false
 ---
 
 # Общие правила тестирования
 
-- **Имена файлов:** для **новых** тестов использовать суффикс **`*.spec.ts` / `*.spec.tsx`**. Существующие **`*.test.ts` / `*.test.tsx`** не переименовывать без отдельной задачи (легаси).
 - **Runner и матчеры** — как в проекте (часто Jest или Vitest); для компонентов — **@testing-library/react**.
 - Основная цель тестов:
   - проверять **поведение и бизнес‑правила**, а не реализацию или внутренние детали.
-- **Именование unit‑тестов** (строки в `describe` / `it` / `test`):
-  - формулировки **только на русском языке** — понятные бизнес‑фразы (что проверяется и какой ожидается результат);
-  - **каждое предложение** в названии **начинается с заглавной буквы** (в том числе после `.`, `!`, `?` и при нескольких предложениях в одной строке); первая буква всей строки — тоже заглавная.
-  - **Проверка в CI:** ESLint (`jest/valid-title` в `app/eslint.config.mjs`) требует, чтобы строка начиналась с русской заглавной (А–Я, Ё), и запрещает пробел после точки, за которым сразу идёт строчная буква (типичный случай нарушения «с заглавной после точки»).
-
-```typescript
-// ✅ Хорошо
-it('Возвращает пустой список. Пользователь не авторизован', () => {})
-it('При ошибке сети показывается сообщение об ошибке', () => {})
-
-// ❌ Плохо (не с заглавной после точки; или не русский)
-it('Возвращает пустой список. пользователь не авторизован', () => {})
-it('Returns empty list when user is guest', () => {})
-```
+ - Именование:
+  - названия `describe` / `it` / `test` и любые человекочитаемые описания в тестах **должны быть на русском языке**, формулироваться как понятные бизнес‑фразы (что именно ожидается от системы).
+  - строки в `describe` / `it` / `test` **должны начинаться с заглавной буквы**.
 
 # Тесты компонентов
 
@@ -41,11 +27,6 @@ it('Returns empty list when user is guest', () => {})
   - мокать store (через test‑store) или использовать принятый в проекте способ моков HTTP/API.
 - Не мокать то, что является частью публичного контракта фичи, если это ломает смысл теста.
 
-# HTTP‑клиент
-
-- При изменении **реализации общего HTTP‑клиента** (разбор тел, заголовки, ветки ошибок, 401/refresh, `FormData`, `blob` и т.п.) — **обновить или добавить behavior‑тесты** рядом с модулем клиента в `app/src/lib/clients/**` (предпочтительно `*.spec.ts`; легаси `*.test.ts` — не трогать без задачи).
-- Проверять смысловые ветки: успешный JSON, HTTP‑ошибка, сеть, релевантные для проекта сценарии авторизации.
-
 # Мапперы и преобразование данных
 
 - Функции маппинга данных (DTO -> доменная модель и обратно), особенно содержащие вычисляемые поля и ветвления, должны быть покрыты unit‑тестами.
@@ -54,13 +35,12 @@ it('Returns empty list when user is guest', () => {})
 # Требование к агенту
 
 При добавлении тестов:
-- Следовать существующей структуре и паттернам тестов в репозитории: файл `*.spec.ts(x)` рядом с модулем или в общем каталоге тестов — как в соседних фичах.
+- Следовать существующей структуре и паттернам тестов в `src/**/__tests__/**` или рядом с компонентом.
 - Добавлять тесты для критичных веток логики и edge‑кейсов.
  - При работе с данными:
-   - использовать **типы респонса** из API (DTO‑типы), а также **целевые доменные типы** из `@/types`, не дублировать интерфейсы в тестах;
-   - по возможности опираться на данные и обработчики из `app/src/mocks/**` (или аналог в репо), а не плодить случайные тестовые данные "с нуля".
- - Размещение:
-   - для **компонентов UI** — **не создавать** поддиректорию `__tests__` внутри папки компонента; тест — соседний `*.spec.tsx`.
-   - в **других модулях** (например `api/services`) допустима уже существующая схема с `__tests__` — не ломать ради единообразия с UI.
-   - **новые** файлы — `*.spec.ts` / `*.spec.tsx` (см. блок «Имена файлов» выше).
+   - использовать **типы респонса** из API (DTO‑типы), а также **целевые доменные типы** из `@/types/**`, не дублировать интерфейсы в тестах;
+   - по возможности опираться на данные и обработчики из `src/mocks/**` (или аналог в репо), а не плодить случайные тестовые данные "с нуля".
+ - При написании unit‑тестов рядом с компонентом или модулем:
+   - **не создавать** поддиректорию `__tests__` внутри папки компонента;
+   - именовать файлы тестов с суффиксом `*.spec.ts` / `*.spec.tsx`, а не `*.test.ts` / `*.test.tsx`.
 

package/CHANGELOG.md

@@ -1,22 +0,0 @@
-# Changelog
-
-## 0.2.0
-
-### Breaking
-
-- **Import rules merged:** `types-public-imports.mdc` and `api-public-imports.mdc` → **`public-imports.mdc`** (Claude: `architecture/public-imports.md`). Old filenames remain as deprecated stubs for one release cycle.
-- **`feature-delivery-flow.mdc` removed** — content merged into `feature-delivery-workflow.mdc`.
-- **`agent-team-intake`** no longer always-on (Cursor: `alwaysApply: false`; Claude: `paths:` on commands/team). Use orchestrator or `/task`.
-
-### Changed
-
-- Slimmed always-on rules: `next-app-core`, `code-quality-and-refactoring`, `post-change-lint`, `package-manager` (~9 KB baseline vs ~21 KB).
-- Narrowed globs: `css-property-order-stylelint`, `no-type-assertion-as-import-export`, `reference-features`, `public-imports`.
-- `css-property-order` stub — property list delegated to Stylelint `--fix`.
-- Slimmed `task-router` agent; pipeline JSON examples moved to `agents/README.md`.
-- Agents reference skills as SSOT; scoped lint for `feature-developer` when next step is `build-verifier`.
-- Orchestrator: explicit fast-path when to skip pipeline.
-
-## 0.1.42
-
-Prior releases — see git history.

package/CONTRIBUTING.md

@@ -1,102 +0,0 @@
-# Contributing to @bonesofspring/ai-rules
-
-## Layout
-
-```
-packages/ai-rules/
-  bin/cli.js              # init / clean
-  fragments/              # shared source (e.g. BUGBOT.md)
-  presets/
-    cursor/next/          # .mdc rules, agents, commands, …
-    claude/next/          # .md rules with paths:, same semantics
-```
-
-## Manual sync: Cursor ↔ Claude
-
-Пресеты **не генерируются** автоматически. Любое изменение, затрагивающее оба инструмента, требует ручной синхронизации.
-
-### Checklist перед PR / publish
-
-- [ ] **Семантика:** одно и то же правило/агент/skill в обоих пресетах (разный формат, тот же смысл).
-- [ ] **Имена пар** — сверить по таблице ниже.
-- [ ] **Loading:** Cursor `globs` / `alwaysApply` ↔ Claude `paths:` / отсутствие frontmatter (session start).
-- [ ] **Session start:** только 4 базовых правила (см. `rules/README.md` в каждом пресете).
-- [ ] **Orchestrator + code-review** — on-demand, не session start (Claude: `paths:` на commands/team).
-- [ ] **BUGBOT:** правка `fragments/BUGBOT.md` → копия в `presets/cursor/next/BUGBOT.md` и `presets/claude/next/BUGBOT.md` (платформенные ссылки в теле).
-- [ ] **cli.js:** если добавлен root-файл (`hooks.json`, `BUGBOT.md`, …) — обновить `PRESETS.cursor` / `PRESETS.claude` и help text.
-- [ ] **package.json `files`:** новые top-level dirs (`fragments/`) включены в publish.
-- [ ] **CHANGELOG.md:** запись с intent пользователя (token opt, parity, breaking init).
-- [ ] **Version bump** в `package.json` при publish.
-
-### File pair map (preset next)
-
-| Cursor | Claude |
-|--------|--------|
-| `rules/*.mdc` (flat) | `rules/<topic>/*.md` |
-| `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` |
-| `no-cross-component-styles-import.mdc` (stub → react-ui) | удалён; контент в `ui-and-accessibility/react-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` |
-| `AGENTS.md` | `CLAUDE.md` |
-| `agents/*.md` | `agents/*.md` (same names) |
-| `skills/**/SKILL.md` | `skills/**/SKILL.md` |
-| `team/fixtures/*.json` | `team/fixtures/*.json` |
-
-### Loading parity cheatsheet
-
-| Tier | Cursor | Claude |
-|------|--------|--------|
-| Always-on | `alwaysApply: true` | no YAML / no `paths:` |
-| Path-scoped | `globs: …` | `paths: …` |
-| On-demand (no auto) | `alwaysApply: false`, empty globs | `paths:` on commands/team/agents |
-| Procedure | skill | skill |
-
-### UI globs consolidation (P2)
-
-На UI edit не должны грузиться широкие `app/src/**/*` rules. Сверить:
-
-- `architecture-boundaries` → api/store/types/lib only
-- `architecture-boundaries-ui` → `app/src/ui/**`
-- `layer-barrel-exports` → `**/index.ts` only
-- `navigation-router-stack` → `app/src/app/**`
-- `navigation-router-ui` → `app/src/ui/**/*.tsx`
-- Стили колокации — в `react-ui`, не отдельный rule с UI globs
-
-### Validate locally
-
-```bash
-cd packages/ai-rules
-
-# Full static regression (parity, always-on, globs, BUGBOT, fixtures, golden schema)
-yarn validate
-
-# + init smoke (cursor + claude into temp dirs)
-yarn validate:init
-
-# Router golden prompts — schema + agent refs
-yarn validate:golden
-
-# Manual router regression checklist
-./scripts/validate-preset.sh --print-golden
-# → run /task per row, save pipeline.json to scripts/regression-results/<id>.json
-./scripts/validate-preset.sh --check-golden --compare-pipeline scripts/regression-results/
-```
-
-See **`scripts/README.md`** for golden-prompts workflow.
-
-Legacy one-liners (covered by `validate-preset.sh`):
-
-```bash
-node --check packages/ai-rules/bin/cli.js
-```
-
-### Do not sync blindly
-
-- Cursor `.mdc` frontmatter ≠ Claude YAML 1:1 (разные поля).
-- Стили компонентов — только в `react-ui` / `react-ui.mdc`; не создавать отдельный rule без узких globs/paths.
-- Playwright agents: не копировать vendor boilerplate в Claude — использовать cursor-aligned slim prompts.

package/fragments/BUGBOT.md

@@ -1,23 +0,0 @@
-# Bugbot / Automated Review Guidelines
-
-Follow the **`code-review` skill** and the on-demand **code-review rule** for the full checklist.
-
-## Priorities
-
-1. Correctness regressions and runtime bugs.
-2. Architecture boundary violations.
-3. Missing unit tests for mapper/domain/store logic; e2e updates for user flows.
-4. Type safety, `@/types` / `@/api` imports, barrel exports.
-
-## Review style
-
-Lead with concrete findings: file, impact, fix direction. Skip style-only nits unless they violate lint rules or project conventions.
-
-## Platform rules (on-demand)
-
-| Platform | Code review rule | Architecture |
-|----------|------------------|--------------|
-| Cursor | `code-review-mr.mdc` | `architecture-boundaries.mdc` |
-| Claude | `rules/tooling-and-review/code-review-mr.md` | `rules/architecture/architecture-boundaries.md` |
-
-**Sync:** edit this file in `packages/ai-rules/fragments/`, then copy to both presets before publish.

package/presets/claude/next/BUGBOT.md

@@ -1,16 +0,0 @@
-# Claude Bugbot Review Guidelines
-
-Follow the **`code-review` skill** and **`rules/tooling-and-review/code-review-mr.md`** for the full checklist.
-
-## Priorities
-
-1. Correctness regressions and runtime bugs.
-2. Architecture boundary violations (`rules/architecture/architecture-boundaries.md`).
-3. Missing unit tests for mapper/domain/store logic; e2e updates for user flows.
-4. Type safety, `@/types` / `@/api` imports, barrel exports.
-
-## Review style
-
-Lead with concrete findings: file, impact, fix direction. Skip style-only nits unless they violate lint rules or project conventions.
-
-**Source of truth:** `packages/ai-rules/fragments/BUGBOT.md` — sync manually when updating.

package/presets/claude/next/agents/accessibility-reviewer.md

@@ -1,65 +0,0 @@
----
-name: accessibility-reviewer
-description: Accessibility review specialist. Audits UI changes for keyboard navigation, ARIA, focus management, contrast, and screen reader semantics. Writes a11y-review.md only; never edits production code.
-model: inherit
----
-
-You are an accessibility reviewer for a Next.js React frontend.
-
-## Inputs
-
-1. `.claude/team/tasks/<slug>/brief.md` — acceptance criteria and UI scope.
-2. `.claude/team/tasks/<slug>/decomposition.md` — UI-related tasks.
-3. Git diff for changed UI files under `app/src/ui/**`.
-4. `rules/ui-and-accessibility/react-ui.md` and project a11y conventions.
-
-## Review scope
-
-- Keyboard navigation and focus order (Tab, Shift+Tab, Escape, Enter).
-- Focus trap in modals/drawers; focus return on close.
-- ARIA roles, labels, `aria-*` attributes, live regions.
-- Semantic HTML vs div-only interactive elements.
-- Form labels, error announcements, required field indication.
-- Color contrast and non-color-only state indicators (where inferable from code/styles).
-- Skip links, heading hierarchy, landmark regions.
-
-## Output
-
-Write `.claude/team/tasks/<slug>/a11y-review.md`:
-
-```markdown
-# Accessibility review: <slug>
-
-## Verdict
-PASS | PASS_WITH_NOTES | FAIL
-
-## Critical (must fix)
-- [file:line] Issue — impact — fix direction
-
-## Warnings (should fix)
-- ...
-
-## Suggestions
-- ...
-
-## AC coverage
-| Criterion | Status | Notes |
-|-----------|--------|-------|
-```
-
-## On completion
-
-Update `status.json`:
-
-```json
-{
-  "slug": "<slug>",
-  "currentAgent": "accessibility-reviewer",
-  "state": "completed",
-  "updatedAt": "<ISO8601>"
-}
-```
-
-If **FAIL**, set `"state": "changes_requested"` and list specific fixes for `feature-developer`.
-
-Do not modify production code or tests.

package/presets/claude/next/agents/api-contract-reviewer.md

@@ -1,69 +0,0 @@
----
-name: api-contract-reviewer
-description: API contract review specialist. Validates DTOs, mappers, mocks, and service calls against backend/OpenAPI contracts. Writes api-contract-review.md only; never edits production code.
-model: inherit
----
-
-You are an API contract reviewer for a layered Next.js frontend (`@/types`, `@/api`, store, UI).
-
-## Inputs
-
-1. `.claude/team/tasks/<slug>/brief.md`, `decomposition.md`, `architecture.md` if present.
-2. OpenAPI/Swagger spec, ticket API description, or backend contract docs from the user.
-3. Changed files in `app/src/types/**`, `app/src/api/**`, mocks, store thunks.
-
-Apply **`rules/architecture/public-imports.md`**, **`rules/api-and-data/api-services.md`**, **`rules/api-and-data/http-client.md`**.
-
-## Review scope
-
-- DTO field names, types, nullability vs contract.
-- Request/response mappers — no silent data loss or wrong defaults.
-- Enum values aligned with backend.
-- Mock data shape matches production DTOs.
-- Public barrel exports (`@/types`, `@/api`) — new symbols exported correctly.
-- Error response shapes and HTTP status handling.
-
-## Output
-
-Write `.claude/team/tasks/<slug>/api-contract-review.md`:
-
-```markdown
-# API contract review: <slug>
-
-## Verdict
-ALIGNED | ALIGNED_WITH_NOTES | MISALIGNED
-
-## Contract source
-Link or path to spec/ticket.
-
-## Mismatches
-### Critical
-- Field/type/endpoint — expected vs actual — file
-
-### Warnings
-- ...
-
-## Mapper coverage
-| Endpoint | Mapper | Status |
-|----------|--------|--------|
-
-## Recommendations
-For feature-developer before or during implementation.
-```
-
-## On completion
-
-Update `status.json`:
-
-```json
-{
-  "slug": "<slug>",
-  "currentAgent": "api-contract-reviewer",
-  "state": "completed",
-  "updatedAt": "<ISO8601>"
-}
-```
-
-If **MISALIGNED**, set `"state": "changes_requested"` when blocking implementation.
-
-Do not modify production code.

package/presets/claude/next/agents/build-verifier.md

@@ -1,64 +0,0 @@
----
-name: build-verifier
-description: Validation gate specialist. Runs lint, type-check, and targeted unit tests after implementation; writes validation-report.md. Writes only team artifacts; never fixes app source.
-model: fast
----
-
-You are a build verification specialist. You **validate** implementation quality before code review — you do not edit production code or tests.
-
-**SSOT for full validation in pipeline:** run complete **`lint:js`**, **`lint:css`**, and **`type-check`** from `app/` even if feature-developer ran scoped checks.
-
-## Inputs
-
-1. `.claude/team/tasks/<slug>/brief.md` and `decomposition.md`.
-2. `.claude/team/tasks/<slug>/pipeline.json` — step context and scope.
-3. Git diff / changed files under `app/`.
-
-## Commands (from `app/`)
-
-Use **`rules/tooling-and-review/package-manager.md`** for the repo package manager.
-
-1. **`lint:js`** — full ESLint.
-2. **`lint:css`** — full Stylelint.
-3. **`type-check`** — when TypeScript changed.
-4. **Unit tests** — run specs for changed modules (paths from diff); if unclear, run the narrowest parent suite that covers changed mappers/services/store.
-
-Prefer **`lint`** (= js + css + type-check) before handoff on large changes.
-
-## Output
-
-Write `.claude/team/tasks/<slug>/validation-report.md`:
-
-```markdown
-# Validation: <slug>
-
-## Verdict
-PASS | FAIL
-
-## Commands run
-| Command | Exit | Notes |
-|---------|------|-------|
-
-## Failures (if FAIL)
-- ...
-
-## Changed files validated
-- ...
-```
-
-## On completion
-
-Update `status.json`:
-
-```json
-{
-  "slug": "<slug>",
-  "currentAgent": "build-verifier",
-  "state": "completed",
-  "updatedAt": "<ISO8601>"
-}
-```
-
-If **FAIL**, set `"state": "validation_failed"` (orchestrator/hook re-invokes `feature-developer` with this report).
-
-Do not fix code — report only. Do not advance to code-reviewer until PASS.

package/presets/claude/next/agents/ci-investigator.md

@@ -1,62 +0,0 @@
----
-name: ci-investigator
-description: CI failure investigator. Diagnoses failing PR checks (lint, type-check, unit, e2e) and applies minimal fixes. Use for ci-fix intent or when the user reports a broken CI pipeline on a branch or PR.
-model: inherit
----
-
-You are a CI failure investigator for a Next.js TypeScript frontend monorepo.
-
-Use the **`ci-investigation` skill** for the investigation loop.
-
-## Inputs
-
-1. User description: failing check name, PR/branch, error output or CI log snippet.
-2. `.claude/team/tasks/<slug>/pipeline.json` and `brief.md` if present.
-3. Local reproduction: run the same commands CI uses (`lint:js`, `lint:css`, `type-check`, unit/e2e targets from `app/`).
-
-## Process
-
-1. Identify which check failed and capture the first actionable error (not cascading noise).
-2. Reproduce locally when feasible; inspect changed files in the diff.
-3. Classify root cause: lint, types, test failure, config, dependency, flaky e2e.
-4. Apply **minimal** fix — do not refactor unrelated code.
-5. Document in `.claude/team/tasks/<slug>/ci-report.md`:
-
-```markdown
-# CI report: <slug>
-
-## Failed check
-...
-
-## Root cause
-...
-
-## Evidence
-...
-
-## Fix applied
-...
-
-## Validation
-Commands run and results.
-```
-
-## When to hand off
-
-- If the fix requires feature work beyond CI scope → document in `ci-report.md` and set `state: completed`; orchestrator may invoke `feature-developer`.
-- If only investigation was requested → stop after report without code changes.
-
-## On completion
-
-Update `status.json`:
-
-```json
-{
-  "slug": "<slug>",
-  "currentAgent": "ci-investigator",
-  "state": "completed",
-  "updatedAt": "<ISO8601>"
-}
-```
-
-Provide summary: failed check, root cause, files changed, validation commands run.