@bonesofspring/ai-rules 0.2.14 → 0.2.16

package/presets/_shared/core/quality/design-guidance.md

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

package/presets/_shared/core/review/code-review-mr.md

@@ -15,9 +15,9 @@ On-demand rule for review flows. **Procedure and output format:** skill `code-re
 - UI conventions of the stack
 - Unit and e2e/UI test conventions
 - Post-change lint/build gate before final report
-- Design smells **in MR diff scope** — load `design-guidance`; не разворачивать полный каталог Fowler вне диффа
-- Pattern name (GoF) — только если репозиторий уже использует эту форму или match минимален
-- Principles (KISS / DRY+Ro3 / YAGNI / SOLID / CUPID) — через always-on `code-quality-and-refactoring`, не дублировать эссе в review
+- Design smells **in MR diff scope** — load `design-guidance`; do not expand the full Fowler catalog outside the diff
+- Pattern name (GoF) — only if the repository already uses that shape or the match is minimal
+- Principles (KISS / DRY+Ro3 / YAGNI / SOLID / CUPID) — via always-on `code-quality-and-refactoring`, do not duplicate the essay in review
 
 Concrete rule stems — in **Stack notes**.
 
@@ -26,7 +26,7 @@ Concrete rule stems — in **Stack notes**.
 - Focus on MR diff, not the whole repo.
 - Use local `git diff` — do not invent hosting metadata.
 - Boy scout rule: suggest fixes feasible within the MR scope.
-- No «большие рефакторинги» without an explicit request (`code-quality-and-refactoring`).
+- No “big refactors” without an explicit request (`code-quality-and-refactoring`).
 
 ## Output contract
 

package/presets/_shared/core/review/technical-retro.md

@@ -1,9 +1,9 @@
-# Техническое ретро — rule alias
+# Technical retro — rule alias
 
-Полный сценарий (включая блок **«Работа агентов»** для slug из team tasks) — в slash-команде:
+Full scenario (including the **“Agent work”** block for a slug from team tasks) lives in the slash command:
 
 **`commands/technical-retro.md`** → `/technical-retro`
 
 When a task has `metrics.json`, use the local aggregation script and state the cohort, sample size, incomplete artifacts, attribution confidence, active attempt duration, and human-gate wait separately. Do not score malformed artifacts or gate wait as agent work.
 
-Используй rule только если command недоступен. Не дублируй содержимое command в ответе — следуй command-файлу и skill `technical-retro`.
+Use this rule only if the command is unavailable. Do not duplicate the command contents in the response — follow the command file and skill `technical-retro`.

package/presets/claude/android-kotlin/CLAUDE.md

@@ -0,0 +1,38 @@
+# Android Kotlin stack — project instructions
+
+Short Claude Code entry point. Detailed rules — modular topic `.md` files under `rules/` (derived from Cursor SoT).
+
+**Repo-root agents:** see [`AGENTS.md`](../../AGENTS.md) at the project root (cross-tool setup, commands, security, living conventions pointers).
+
+See @rules/README.md for mapping and loading strategy.
+
+## Conventions
+
+- One topic per file; path scope — frontmatter `paths:`.
+- **Session-start rules** (no `paths:`) — exactly **three**: `android-app-core`, `boundaries`, `code-quality`.
+- Requestable / paths: orchestrator, intake, feature-delivery, code-review, post-change-build, gradle-tooling, reference-features, security-android, technical-retro, domain globs.
+- Slash commands — `commands/`; skills — `skills/`; subagents — `agents/`.
+
+## Quick start
+
+| Task | Where to look |
+|------|----------------|
+| Work request | `commands/task.md` + `tooling-and-review/agent-team-orchestrator.md` |
+| End-to-end feature | skill `feature-delivery` + `architecture/feature-delivery.md` |
+| Reference modules | `architecture/reference-features.md` (placeholders `applications/`, `features/<name>/`) |
+| Stack | `stack/android-app-core.md` |
+| Toolchain | `tooling-and-review/gradle-tooling.md` |
+| After Kotlin edits | `tooling-and-review/post-change-build.md` (requestable, required) |
+| UI tests | skill `instrumentation-ui-e2e` |
+| Security | `tooling-and-review/security-android.md` |
+| Validation | agent `build-verifier` |
+| How does X work? | agent `codebase-analyzer` (explain-as-is) |
+| Retro | `commands/technical-retro.md` |
+
+## Install
+
+```bash
+npx @bonesofspring/ai-rules init claude --preset android-kotlin
+```
+
+Copies `CLAUDE.md`, `rules/**`, `commands/**`, `agents/**`, `skills/**`, `hooks/**`, `team/**`, `MCP.md` into `.claude/`, plus `mcp.json` → **`.mcp.json`** and `REPO_AGENTS.md` → root **`AGENTS.md`**. Skill: `write-adr` (on-demand).

package/presets/claude/android-kotlin/MCP.md

@@ -0,0 +1,16 @@
+# MCP (preset `android-kotlin`, Claude)
+
+Copied to `.claude/mcp.json` on `ai-rules init claude --preset android-kotlin`.
+
+## Servers
+
+| Name | Package | Purpose |
+|------|---------|---------|
+| `context7` | `@upstash/context7-mcp` | Up-to-date Kotlin / Jetpack Compose / Android SDK docs |
+| `figma` | remote `https://mcp.figma.com/mcp` | Design context → Jetpack Compose; OAuth in the Claude Code / MCP client |
+
+Playwright / Chrome DevTools are **not** part of the **android-kotlin preset** (browser e2e is forbidden for the agent team).
+
+Do not put secrets in the file. Optional: `CONTEXT7_API_KEY` in the environment. Figma — OAuth (do not commit the token).
+
+After init: enable MCP servers in the client. `init` overwrites a same-named `.mcp.json`.

package/presets/claude/android-kotlin/README.md

@@ -0,0 +1,32 @@
+# Preset: android-kotlin (Claude)
+
+Claude twin of the Cursor **android-kotlin** preset. Topic rules under `rules/` map 1:1 from Cursor `.mdc` SoT — see `rules/README.md`.
+
+```bash
+npx @bonesofspring/ai-rules init claude --preset android-kotlin
+```
+
+| In the preset | Where `init claude --preset android-kotlin` copies |
+|---------------|---------------------------------------------------|
+| `CLAUDE.md` | `.claude/CLAUDE.md` |
+| `rules/**` | `.claude/rules/**` |
+| `agents/**`, `skills/**`, `commands/**`, `hooks/**`, `team/**` | `.claude/…` |
+| `mcp.json` | **`.mcp.json`** at repo root |
+| `MCP.md` | `.claude/MCP.md` |
+| `REPO_AGENTS.md` | root `AGENTS.md` |
+
+Session-start trio (no `paths:`): `stack/android-app-core.md`, `architecture/boundaries.md`, `tooling-and-review/code-quality.md`.
+
+## Layout placeholders
+
+Same vendor-neutral layout as Cursor SoT: `applications/`, `core/`, `features/<name>/{api,impl}/`, `build-logic/`, `gradle/libs.versions.toml`. Map local type names and module paths in **`team/conventions.md`** after install.
+
+## Install troubleshooting
+
+- **Gradle wrapper:** discover `./gradlew` or the wrapper script from repo docs/CI; see `tooling-and-review/gradle-tooling.md`.
+- **Modules / variants / flavors:** resolve Gradle module paths and the CI build variant before reproducing failures; document local names in `team/conventions.md`.
+- **Multimodule:** narrow compile/test tasks to changed modules; full validation — agent `build-verifier`.
+
+## Parity with Cursor
+
+SoT = `presets/cursor/android-kotlin/`. Update Claude twin in the **same PR** as Cursor edits.

package/presets/claude/android-kotlin/REPO_AGENTS.md

@@ -0,0 +1,48 @@
+# Android Kotlin — agent instructions
+
+Cross-tool repo-root guide for the **android-kotlin** multimodule preset. Tool-specific maps stay under `.cursor/AGENTS.md` and `.claude/CLAUDE.md`.
+
+## Cross-tool setup
+
+- Treat this file as the **shared** entry for any agent (Cursor, Claude Code, or other).
+- Tool maps live in `.cursor/AGENTS.md` / `.claude/CLAUDE.md` when that tool is installed.
+- Install:
+
+```bash
+npx @bonesofspring/ai-rules init cursor --preset android-kotlin
+npx @bonesofspring/ai-rules init claude --preset android-kotlin
+```
+
+`init` overwrites this file from the preset (`REPO_AGENTS.md` → `AGENTS.md`). Living gotchas and local type names belong in `team/conventions.md`, not here.
+
+## Commands (test / lint / build)
+
+Discover the Gradle wrapper (`./gradlew` or a documented wrapper script from README/CI). Modules/variants — from Gradle + `gradle/libs.versions.toml` / `build-logic/`.
+
+| Gate | Typical command |
+|------|-----------------|
+| Compile | `./gradlew :module:compile<Variant>Kotlin` (or project alias) |
+| Lint | Android Lint / ktlint / detekt **only if** configured in repo/CI |
+| Unit tests | `./gradlew :module:test<Variant>UnitTest` (narrowest suite) |
+| Instrumentation | Connected/managed device tasks as configured in CI |
+
+After code edits: invoke **`post-change-build`** (requestable) + **`gradle-tooling`**. Full gate: agent `build-verifier`.
+
+## Security basics
+
+- Do not commit secrets (API keys, keystore passwords, tokens, `.env`).
+- Review destructive git before running; respect shell hooks.
+- Do not weaken TLS / cleartext policy unless the user explicitly asks.
+
+## Tool entry points
+
+| Tool | Map |
+|------|-----|
+| Cursor | `.cursor/AGENTS.md` (optional if Cursor not installed) |
+| Claude Code | `.claude/CLAUDE.md` (optional if Claude not installed) |
+
+Rules: `.cursor/rules/` or `.claude/rules/`. Skills: `.cursor/skills/` or `.claude/skills/`. UI tests: `instrumentation-test-*` (never Playwright/XCUITest as required handoffs).
+
+## Living conventions
+
+Local type names (ViewModel base, DI holders, design-system package), flavors, and wrapper quirks: **`.cursor/team/conventions.md`** and/or **`.claude/team/conventions.md`** (fill after init).

package/presets/claude/android-kotlin/agents/README.md

@@ -0,0 +1,44 @@
+# Agents (android-kotlin)
+
+Full agent-team roster (~22). Production-write policy matches ios-swift mobile depth with Gradle + instrumentation.
+
+## Rename map (from next / ios-swift / java)
+
+| Source | android-kotlin |
+|--------|----------------|
+| `playwright-test-planner` | **`instrumentation-test-planner`** |
+| `playwright-test-generator` | **`instrumentation-test-generator`** |
+| `playwright-test-healer` | **`instrumentation-test-healer`** |
+| skill `playwright-e2e` / `xcuitest-e2e` | **`instrumentation-ui-e2e`** |
+| `xcuitest-test-*` | **skipped** (iOS-only) |
+| `integration-test-*` | **skipped** (JVM backend) |
+| `post-change-lint` / `post-change-test` | **`post-change-build`** |
+| `xcode-tooling` / `java-tooling` | **`gradle-tooling`** |
+| `react-performance` | **absent** (leakage FAIL) |
+
+When copying next pipeline templates: put Playwright / XCUITest / integration-test stems in `skipped[]` and schedule `instrumentation-test-*` when UI AC requires it.
+
+## Roster
+
+| Agent | Production writes |
+|-------|-------------------|
+| task-router, task-analyst, solution-architect, migration-specialist, api-contract-reviewer | team only |
+| codebase-analyzer | no (explain-as-is) |
+| feature-developer | yes (Kotlin/Android sources) |
+| build-verifier | no (report only) |
+| debugger, ci-investigator | minimal |
+| code-reviewer, accessibility-reviewer, security-reviewer, performance-auditor | team only |
+| qa-tester, unit-test-*, instrumentation-test-* | tests only |
+| tech-writer | docs only |
+
+## Model tiers (mirror ios-swift)
+
+| Tier | Model hint | Typical agents |
+|------|------------|----------------|
+| **cheap** | `fast` / cheap | build-verifier, some generators/healers |
+| **standard** | `inherit` | feature-developer, debugger, ci-investigator, a11y, qa, unit-*, instrumentation-* |
+| **strong** | `strong` | solution-architect, security-reviewer (when scheduled) |
+
+## Pipeline schema
+
+See `team/fixtures/*.json` and `hooks/chain-team-phases.sh` handoffs (`instrumentation-test-*`, `gradle-tooling`, `post-change-build`).

package/presets/claude/android-kotlin/agents/accessibility-reviewer.md

@@ -0,0 +1,60 @@
+---
+name: accessibility-reviewer
+description: Accessibility review specialist. Audits TalkBack labels/semantics, font scale, reduce motion, Compose testTag, contrast, and touch targets for Android UI. Readonly — writes a11y-review.md only; never edits production code.
+readonly: false
+model: inherit
+---
+
+You are an accessibility reviewer for an **Android Kotlin / Jetpack Compose** (and View system where present) 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 Views / UI under `features/**/Presentation/**` (and design-system if touched).
+4. `ui-and-accessibility/compose.md` and project a11y conventions.
+
+## Review scope
+
+- TalkBack / screen reader: labels, `contentDescription`, semantics; meaningful accessible elements (not unlabeled buttons/icons).
+- Font scale / large text: layouts do not clip or overlap at larger sizes (infer from modifiers / typography).
+- Reduce motion for custom animations when present.
+- Compose `testTag` (or project semantics matchers) on interactive / assertable elements for instrumentation (`{screen}__{element}`) — not iOS `accessibilityIdentifier`.
+- Contrast and non-color-only state (where inferable from Design System / asset usage).
+- Touch targets roughly ≥48dp when obvious from layout; focus order for custom controls.
+- Sheet/modal dismissal discoverability for TalkBack users.
+
+## 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 |
+|-----------|--------|-------|
+```
+
+## Artifact contract
+
+- **Authoritative inputs:** read `.claude/team/tasks/<slug>/artifact-manifest.json` first; consume handoff `artifactInputIds` when supplied, otherwise consume the manifest entries for brief/decomposition, implementation or UI diff scope, and prior review artifacts when present.
+- **Owned outputs:** `a11y-review.md`; upsert only the manifest entry assigned by handoff (`artifactOutputId`; fallback `a11y-review`) and preserve all foreign entries.
+- **Terminal receipt:** cover applicable AC/task IDs, list authoritative evidence paths, and emit `completed` or `changes_requested`. Bind it to the supplied `attemptId` when present.
+- **Shared state:** never create or mutate `status.json` or `metrics.json`; the orchestration runtime/hook owns lifecycle, gates, retries, attempts, and timestamps.
+
+If **FAIL**, emit receipt outcome `changes_requested` and list specific fixes for `feature-developer`.
+
+Do not modify production code or tests.

package/presets/claude/android-kotlin/agents/api-contract-reviewer.md

@@ -0,0 +1,70 @@
+---
+name: api-contract-reviewer
+description: API contract review specialist. Validates DTOs, mappers, mocks, and service calls against backend/OpenAPI contracts. Readonly — writes api-contract-review.md only; never edits production code.
+readonly: false
+model: inherit
+---
+
+You are an API contract reviewer for a layered **Android Kotlin** Feature module (`features/<name>/{api,impl}` — Domain / Data / Presentation).
+
+## 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: DTO models, mappers, API clients/services, mocks/fixtures under Data (and Domain entities).
+
+Apply `api-and-data/networking.md`, `architecture/boundaries.md`, `architecture/module-public-api.md` as relevant.
+
+## Review scope
+
+- DTO field names, types, nullability / optionality vs contract.
+- Request/response mappers — no silent data loss or wrong defaults.
+- Enum / status values aligned with backend.
+- Mock/fixture JSON shape matches production DTOs.
+- Public module exports for new types when using Gradle modules Feature modules.
+- Error response shapes and HTTP status handling in clients.
+- Domain models must not leak transport DTOs into Presentation.
+
+## 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
+
+Write the owned terminal receipt described under **Artifact contract**.
+
+## Artifact contract
+
+- **Authoritative inputs:** read `.claude/team/tasks/<slug>/artifact-manifest.json` first; consume handoff `artifactInputIds` when supplied, otherwise consume the manifest entries for `brief.md`, `decomposition.md`, and other authoritative prior artifacts for this step.
+- **Owned outputs:** `api-contract-review.md`; upsert only the manifest entry assigned by handoff (`artifactOutputId`; fallback `api-contract-review`) and preserve all foreign entries.
+- **Terminal receipt:** cover applicable AC/task IDs, list authoritative evidence paths, and emit `completed` or `changes_requested`. Bind it to the supplied `attemptId` when present.
+- **Shared state:** never create or mutate `status.json` or `metrics.json`; the orchestration runtime/hook owns lifecycle, gates, retries, attempts, and timestamps.
+
+If **MISALIGNED**, emit receipt outcome `changes_requested` when blocking implementation.
+
+Do not modify production code.
+

package/presets/claude/android-kotlin/agents/build-verifier.md

@@ -0,0 +1,67 @@
+---
+name: build-verifier
+description: Validation gate specialist. Discovers Gradle compile/lint/unit after implementation; writes validation-report.md. Never fixes app source. For preset packaging tasks, validates android-kotlin structure.
+readonly: false
+model: fast
+---
+
+You **validate** Android/Kotlin implementation quality — do not edit production code or tests.
+
+**SSOT for full validation:** use `tooling-and-review/gradle-tooling.md` + `tooling-and-review/post-change-build.md`. Run compile / configured lint / unit tests at CI parity even if the 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 (Kotlin/Android modules or preset paths).
+
+## App / feature validation (default)
+
+1. Discover wrapper via `tooling-and-review/gradle-tooling.md` (`./gradlew` or documented wrapper).
+2. Resolve modules/variants from Gradle + `gradle/libs.versions.toml` / `build-logic/`.
+3. Compile affected modules for the relevant variant.
+4. Run Android Lint / ktlint / detekt **only if** repo/CI already configures them.
+5. Unit-test smoke for changed modules (narrowest `*Test` task).
+
+Never require Playwright, XCUITest, `xcodebuild`, or yarn `lint:js`/`lint:css` for Android app work.
+
+## Preset-structure validation (when scope is preset / packages/ai-rules)
+
+When validating **android-kotlin preset** changes (or `scope: preset-structure-validation`), **also** run these checks. Skip Gradle app builds if there are **no** Android app changes.
+
+### FAIL (android-kotlin packaging)
+
+1. **Token budget:** Cursor `alwaysApply: true` count ≤ **3**; Claude rules without `paths:` === **3** (**exclude `**/README.md`**). Meta `preset-*` rules must **not** be always-on.
+2. **Chain fork:** `hooks/chain-team-phases.sh` byte-≠ every other stack (Cursor + Claude).
+3. **Forbidden handoff strings** in android-kotlin `chain-team-phases.sh` (Cursor + Claude): `playwright-test-`, `playwright-agents`, `playwright-e2e`, `user-playwright`, `app/__tests__/e2e`, `xcuitest-test-`, positive `xcodebuild` / `SwiftLint` as required handoffs, positive yarn `lint:js` / `lint:css`.
+4. **Required handoff stems:** `AGENT_HANDOFF` includes `instrumentation-test-planner` (or generator/healer); `build-verifier` prose mentions `gradle-tooling` and/or `post-change-build`.
+5. **Roster:** ~22 agent stems; no `playwright-test-*`, `xcuitest-test-*`, `integration-test-*` files; no `react-performance` skill.
+6. **Twin presence (hard):** Cursor rule → Claude twin + mapping row — **FAIL** if missing.
+7. **Mapping / README:** `claude/android-kotlin/rules/README.md` mapping table present.
+8. **shared-core consumers:** if `_shared/core/**` changed, matching embeds updated (or intentional fork documented).
+9. **Session-start body budget:** sum of the 3 always-on / session-start bodies ≤ **140** lines.
+10. **Embed drift:** `packages/ai-rules/scripts/check-shared-core-drift.sh` with `AI_RULES_DRIFT_FAIL=1`.
+11. **Root AGENTS template:** ship `REPO_AGENTS.md` → **FAIL** if missing.
+12. **Hooks cookbook:** `hooks.json` must **not** reference `hooks/examples/`.
+13. **AC-11 vendor neutrality:** no private consumer repo names, brand/product vendor strings, or org-specific type names in `presets/{cursor,claude}/android-kotlin/**`.
+14. Prefer `yarn check:preset-structure` (twin-map + token-budget + leakage + i18n).
+
+### Soft rules
+
+- Thin aliases (`agent-team-intake`, `technical-retro`) may be &lt;15 body lines; domain rules ≥15.
+- Peer installable presets include `android-kotlin` alongside `next`, `nuxt`, `svelte`, `ios-swift`, `go`, `java`, `mcp-ts`, `php-hexagonal`, `php-laravel`.
+
+## Output
+
+Write `.claude/team/tasks/<slug>/validation-report.md` with verdict, commands, failures, changed files.
+
+## On completion
+
+Write the owned validation receipt (`completed` or `validation_failed`). Do not fix code — report only.
+
+## Artifact contract
+
+- **Authoritative inputs:** read `.claude/team/tasks/<slug>/artifact-manifest.json` first; consume handoff `artifactInputIds` when supplied.
+- **Owned outputs:** `validation-report.md`; upsert only the assigned manifest entry; preserve foreign entries.
+- **Terminal receipt:** cover applicable AC/task IDs; emit `completed` or `validation_failed`.
+- **Shared state:** never mutate `status.json` or `metrics.json`.