@bonesofspring/ai-rules 0.2.15 → 0.2.16

package/CHANGELOG.md

@@ -2,6 +2,14 @@
 
 ## Unreleased
 
+### Android Kotlin preset (`android-kotlin`)
+
+- New installable preset **`android-kotlin`** (Cursor + Claude twins): Clean Architecture multimodule Android/Kotlin — Compose Material 3, MVI/UDF ViewModels, Dagger feature holders (no Hilt by default), `api`/`impl` modules, Gradle convention plugins / version catalog discovery, instrumentation UI-test agents.
+- Agent surface: `instrumentation-test-{planner,generator,healer}` + skill `instrumentation-ui-e2e`; gates `post-change-build` + `gradle-tooling` + `build-verifier`.
+- Structure scripts include `android-kotlin` (`STACKS` + `ANDROID_FORBIDDEN_*`); other stacks forbid `instrumentation-test-*` stems; shared-core `preset-*` matrices updated and embeds re-synced.
+- Install: `npx @bonesofspring/ai-rules init cursor|claude --preset android-kotlin`.
+- Dogfood default remains **`next`** (no switch to android-kotlin).
+
 ### Preset EN translation polish (patch)
 
 - Hardened `check-preset-i18n.sh`: ±3-line keep-marker window (matches style guide), scans `.md`/`.mdc` plus hook/script suffixes; wired as hard gate via `check:preset-i18n` and `check-preset-structure` / CI.

package/README.md

@@ -53,7 +53,7 @@ Where things go on `init`:
 
 Skills of note: `write-adr` (all stacks); `react-performance` (**next only**); `vue-performance` (**nuxt only**); `svelte-performance` (**svelte only**). Living gotchas: `team/conventions.md`. Opt-in rule samples eval: `yarn check:rule-eval` (workflow `rule-eval.yml`; not part of `check:preset-structure`).
 
-The package ships presets including **`next`** (Next.js stack), **`nuxt`** (Nuxt 3 / Vue 3 / Pinia), **`svelte`** (SvelteKit / Svelte 5 / TypeScript / `store-svelte`), **`ios-swift`** (iOS apps on Swift / SwiftUI), **`go`** (Go backends with hexagonal architecture), **`java`** (Java / Spring Boot hexagonal backends), **`mcp-ts`** (TypeScript MCP servers with `@modelcontextprotocol/sdk`), **`php-hexagonal`** (PHP / Composer hexagonal backends), and **`php-laravel`** (Laravel backend/API with Eloquent, Form Requests, Policies, Actions/Services). Other preset names: see `presets/` in the [repo](https://github.com/bonesofspring/ai-rules). Name is a single path segment, no `/` or `.`.
+The package ships presets including **`next`** (Next.js stack), **`nuxt`** (Nuxt 3 / Vue 3 / Pinia), **`svelte`** (SvelteKit / Svelte 5 / TypeScript / `store-svelte`), **`ios-swift`** (iOS apps on Swift / SwiftUI), **`android-kotlin`** (Android multimodule Kotlin / Compose / Gradle), **`go`** (Go backends with hexagonal architecture), **`java`** (Java / Spring Boot hexagonal backends), **`mcp-ts`** (TypeScript MCP servers with `@modelcontextprotocol/sdk`), **`php-hexagonal`** (PHP / Composer hexagonal backends), and **`php-laravel`** (Laravel backend/API with Eloquent, Form Requests, Policies, Actions/Services). Other preset names: see `presets/` in the [repo](https://github.com/bonesofspring/ai-rules). Name is a single path segment, no `/` or `.`.
 
 ```bash
 npx @bonesofspring/ai-rules init cursor --preset nuxt
@@ -62,6 +62,8 @@ npx @bonesofspring/ai-rules init cursor --preset svelte
 npx @bonesofspring/ai-rules init claude --preset svelte
 npx @bonesofspring/ai-rules init cursor --preset ios-swift
 npx @bonesofspring/ai-rules init claude --preset ios-swift
+npx @bonesofspring/ai-rules init cursor --preset android-kotlin
+npx @bonesofspring/ai-rules init claude --preset android-kotlin
 npx @bonesofspring/ai-rules init cursor --preset go
 npx @bonesofspring/ai-rules init claude --preset go
 npx @bonesofspring/ai-rules init cursor --preset java

package/bin/cli.js

@@ -73,6 +73,7 @@ Examples:
   npx @bonesofspring/ai-rules init cursor --preset nuxt
   npx @bonesofspring/ai-rules init cursor --preset svelte
   npx @bonesofspring/ai-rules init cursor --preset ios-swift
+  npx @bonesofspring/ai-rules init cursor --preset android-kotlin
   npx @bonesofspring/ai-rules init cursor --preset go
   npx @bonesofspring/ai-rules init cursor --preset java
   npx @bonesofspring/ai-rules init cursor --preset mcp-ts
@@ -82,6 +83,7 @@ Examples:
   npx @bonesofspring/ai-rules init claude --preset nuxt
   npx @bonesofspring/ai-rules init claude --preset svelte
   npx @bonesofspring/ai-rules init claude --preset ios-swift
+  npx @bonesofspring/ai-rules init claude --preset android-kotlin
   npx @bonesofspring/ai-rules init claude --preset go
   npx @bonesofspring/ai-rules init claude --preset java
   npx @bonesofspring/ai-rules init claude --preset mcp-ts

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bonesofspring/ai-rules",
-  "version": "0.2.15",
+  "version": "0.2.16",
   "description": "Presets of Cursor and Claude rules/commands for Revy Ross personal use",
   "license": "MIT",
   "author": "Revy Ross",

package/presets/_shared/core/meta/preset-layering.md

@@ -13,7 +13,7 @@ Rules for **preset authors** (`packages/ai-rules/presets/**`). Not always-on in
 ## Where to put a new rule
 
 1. Shared principles for all stacks → **core** + adapters with `<!-- shared-core: … -->`.
-2. Only one of the installable stacks (`next` / `nuxt` / `svelte` / `ios-swift` / `go` / `java` / `mcp-ts` / `php-hexagonal` / `php-laravel`) → **stack-specific** (not in `_shared`).
+2. Only one of the installable stacks (`next` / `nuxt` / `svelte` / `ios-swift` / `android-kotlin` / `go` / `java` / `mcp-ts` / `php-hexagonal` / `php-laravel`) → **stack-specific** (not in `_shared`).
 3. Only Cursor or only Claude packaging → platform artifact / frontmatter; do not duplicate semantics.
 
 ## SoT

package/presets/_shared/core/meta/preset-no-cross-stack-leakage.md

@@ -4,11 +4,12 @@ Ban leakage of stack-specific names, paths, and handoff stems across presets.
 
 ## Principles
 
-- `next` must not require iOS-only agents/tooling (`xcuitest-*`, `xcodebuild` as required handoff, Keychain-only rules).
-- `nuxt` must not require iOS-only agents/tooling (`xcuitest-*`, `xcodebuild` as required handoff, Keychain-only rules).
-- `svelte` must not require iOS-only agents/tooling (`xcuitest-*`, `xcodebuild` as required handoff, Keychain-only rules).
-- `ios-swift` must not ship Playwright handoffs or Next lint recipes as required commands.
-- `go` / `java` / `php-hexagonal` / `php-laravel` / `mcp-ts` must not ship Playwright, XCUITest, Next lint recipes, or `xcodebuild`/`SwiftLint` as required handoffs.
+- `next` must not require iOS-only or Android-only agents/tooling (`xcuitest-*`, `instrumentation-test-*`, `xcodebuild` as required handoff, Keychain-only rules).
+- `nuxt` must not require iOS-only or Android-only agents/tooling (`xcuitest-*`, `instrumentation-test-*`, `xcodebuild` as required handoff, Keychain-only rules).
+- `svelte` must not require iOS-only or Android-only agents/tooling (`xcuitest-*`, `instrumentation-test-*`, `xcodebuild` as required handoff, Keychain-only rules).
+- `ios-swift` must not ship Playwright handoffs, Android `instrumentation-test-*` handoffs, or Next lint recipes as required commands.
+- `android-kotlin` must not ship Playwright, XCUITest, `xcodebuild`, or Next lint recipes as required handoffs.
+- `go` / `java` / `php-hexagonal` / `php-laravel` / `mcp-ts` must not ship Playwright, XCUITest, Android instrumentation agents, Next lint recipes, or `xcodebuild`/`SwiftLint` as required handoffs.
 - `mcp-ts` must not ship Go tool commands (`go test`, `go vet`, `golangci-lint`) or `integration-test-*` agents as required delivery gates.
 - Shared core must not embed stack paths (`app/src/**`, `Features/**`, `playwright`, `xcodebuild`).
 
@@ -17,25 +18,37 @@ Ban leakage of stack-specific names, paths, and handoff stems across presets.
 In ios-swift `chain-team-phases.sh` / agent handoffs (Cursor + Claude):
 
 - `playwright-test-`, `playwright-agents`, `playwright-e2e`, `user-playwright`
+- `instrumentation-test-` as required Android handoffs
 - `app/__tests__/e2e` as a required path
 - Positive yarn recipes `lint:js` / `lint:css` as commands to run (negation prose alone is weak — prefer zero occurrences)
 
 Required instead: `xcuitest-test-*` stems; build-verifier mentions `xcodebuild` or `post-change-build` / `xcode-tooling`.
 
+## Forbidden in android-kotlin hooks/agents (FAIL)
+
+In android-kotlin `chain-team-phases.sh` / agent handoffs (Cursor + Claude):
+
+- `playwright-test-`, `playwright-agents`, `playwright-e2e`, `user-playwright`
+- `app/__tests__/e2e` as a required path
+- `xcuitest-test-`, positive `xcodebuild` / `SwiftLint` as required handoffs
+- Positive yarn recipes `lint:js` / `lint:css` as commands to run
+
+Required instead: `instrumentation-test-*` stems; build-verifier mentions `gradle-tooling` and/or `post-change-build` (Gradle wrapper discovery — not `xcodebuild` as the default gate).
+
 ## Forbidden in next hooks/agents (FAIL when present as required)
 
-- `xcuitest-test-` as handoff stems
+- `xcuitest-test-` / `instrumentation-test-` as handoff stems
 - Requiring `xcodebuild` / SwiftLint as the default post-change gate for Next app work
 
 ## Forbidden in nuxt hooks/agents (FAIL when present as required)
 
-- `xcuitest-test-` as handoff stems
+- `xcuitest-test-` / `instrumentation-test-` as handoff stems
 - Requiring `xcodebuild` / SwiftLint as the default post-change gate for Nuxt app work
 - Frontend profile mirrors `next`: Playwright / `lint:js` / `lint:css` allowed
 
 ## Forbidden in svelte hooks/agents (FAIL when present as required)
 
-- `xcuitest-test-` as handoff stems
+- `xcuitest-test-` / `instrumentation-test-` as handoff stems
 - Requiring `xcodebuild` / SwiftLint as the default post-change gate for SvelteKit app work
 - Frontend profile mirrors `next` / `nuxt`: Playwright / `lint:js` / `lint:css` allowed
 
@@ -45,7 +58,7 @@ Shared backend-stack bans (`go`, `java`, `php-hexagonal`, and `php-laravel`):
 
 - `playwright-test-`, `playwright-e2e`, `user-playwright`, `app/__tests__/e2e`
 - Positive yarn recipes `lint:js` / `lint:css`
-- `xcuitest-test-`, `xcodebuild`, `SwiftLint` as required handoffs
+- `xcuitest-test-`, `instrumentation-test-`, `xcodebuild`, `SwiftLint` as required handoffs
 - Agent file `accessibility-reviewer` (no UI surface)
 
 Required instead: `integration-test-*`; build-verifier / `post-change-test` plus stack tooling — `go-tooling` (`go test`, `go vet`), `java-tooling` (`mvn test` / `./gradlew test`), or `php-tooling` (`php artisan test` / `composer test` / `vendor/bin/phpunit`).
@@ -56,9 +69,9 @@ Same UI/mobile bans as backend stacks, plus:
 
 - `playwright-test-`, `playwright-agents`, `playwright-e2e`, `user-playwright`, `app/__tests__/e2e`
 - Positive yarn recipes `lint:js` / `lint:css` as required gates
-- `xcuitest-test-`, `xcodebuild`, `SwiftLint` as required handoffs
+- `xcuitest-test-`, `instrumentation-test-`, `xcodebuild`, `SwiftLint` as required handoffs
 - Go tool commands as required MCP delivery gates: `go test`, `go vet`, `golangci-lint`
-- Agent files: `playwright-test-*`, `xcuitest-test-*`, `integration-test-*`, `accessibility-reviewer`
+- Agent files: `playwright-test-*`, `xcuitest-test-*`, `instrumentation-test-*`, `integration-test-*`, `accessibility-reviewer`
 
 Required instead: package-manager–discovered typecheck/test (`mcp-ts-tooling`, `post-change-test`); Phase 1 gates `build-verifier` + `security-reviewer`.
 
@@ -69,4 +82,4 @@ Required instead: package-manager–discovered typecheck/test (`mcp-ts-tooling`,
 
 ## Chain fork
 
-`hooks/chain-team-phases.sh` must differ **byte-wise** pairwise among `next`, `nuxt`, `svelte`, `ios-swift`, `go`, `java`, `mcp-ts`, `php-hexagonal`, and `php-laravel` (per platform).
+`hooks/chain-team-phases.sh` must differ **byte-wise** pairwise among `next`, `nuxt`, `svelte`, `ios-swift`, `android-kotlin`, `go`, `java`, `mcp-ts`, `php-hexagonal`, and `php-laravel` (per platform).

package/presets/_shared/core/meta/preset-pr-checklist.md

@@ -6,7 +6,7 @@ Checklist for PRs that touch `packages/ai-rules/presets/**` (and dogfood `.curso
 
 - [ ] **SoT:** edits in `packages/…` (not only root `.cursor/`)
 - [ ] **Core:** if `_shared/core/**` changed — all `<!-- shared-core: … -->` consumers updated (or documented intentional fork)
-- [ ] **Twin:** Cursor change → Claude twin **same PR** (**FAIL** if missing on next, nuxt, svelte, ios-swift, go, java, mcp-ts, php-hexagonal, and php-laravel — see `preset-twin-sync`)
+- [ ] **Twin:** Cursor change → Claude twin **same PR** (**FAIL** if missing on next, nuxt, svelte, ios-swift, android-kotlin, go, java, mcp-ts, php-hexagonal, and php-laravel — see `preset-twin-sync`)
 - [ ] **Mapping:** `claude/<stack>/rules/README.md` mapping table is current
 - [ ] **Token budget:** alwaysApply / session-start ≤ 3; meta rules not always-on
 - [ ] **Leakage:** no foreign stack agent stems / handoff strings (`preset-no-cross-stack-leakage`)

package/presets/_shared/core/meta/preset-twin-sync.md

@@ -15,6 +15,7 @@ Cursor `.mdc` — **SoT per stack**. Claude topic `.md` — derived twin.
 | `mcp-ts` | `presets/cursor/mcp-ts/rules/*.mdc` | `presets/claude/mcp-ts/rules/<topic>/` | Keep in `claude/mcp-ts/rules/README.md` |
 | `php-hexagonal` | `presets/cursor/php-hexagonal/rules/*.mdc` | `presets/claude/php-hexagonal/rules/<topic>/` | Keep in `claude/php-hexagonal/rules/README.md` |
 | `php-laravel` | `presets/cursor/php-laravel/rules/*.mdc` | `presets/claude/php-laravel/rules/<topic>/` | Keep in `claude/php-laravel/rules/README.md` |
+| `android-kotlin` | `presets/cursor/android-kotlin/rules/*.mdc` | `presets/claude/android-kotlin/rules/<topic>/` | Keep in `claude/android-kotlin/rules/README.md` |
 
 ## Same-PR checklist
 
@@ -39,5 +40,6 @@ Cursor `.mdc` — **SoT per stack**. Claude topic `.md` — derived twin.
 - **mcp-ts:** missing Claude twin for a changed Cursor rule → **FAIL** (same as next / ios-swift / go / java; mapping table required).
 - **php-hexagonal:** missing Claude twin for a changed Cursor rule → **FAIL** (same as next / ios-swift / go / java / mcp-ts; mapping table required).
 - **php-laravel:** missing Claude twin for a changed Cursor rule → **FAIL** (same as next / ios-swift / go / java / mcp-ts / php-hexagonal; mapping table required).
+- **android-kotlin:** missing Claude twin for a changed Cursor rule → **FAIL** (same as next / ios-swift / go / java / mcp-ts / php-*; mapping table required).
 
 Dogfood: sync packages → root `.cursor/` after Cursor SoT changes (not the reverse). Use `packages/ai-rules/scripts/sync-dogfood-cursor.sh`.

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`.

package/presets/claude/android-kotlin/agents/ci-investigator.md

@@ -0,0 +1,74 @@
+---
+name: ci-investigator
+description: CI failure investigator. Diagnoses failing PR checks (./gradlew, ktlint/detekt, unit, Android instrumentation) and applies minimal fixes. Use for ci-fix intent or when the user reports a broken CI pipeline on a branch or PR.
+readonly: false
+model: inherit
+---
+
+You are a CI failure investigator for an **Android Kotlin** project (Android Studio / AGP Cloud, GitHub Actions Android, or similar).
+
+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: same scheme, configuration, and destination CI uses (`tooling-and-review/gradle-tooling.md`).
+
+## Process
+
+1. Identify which check failed and capture the **first actionable** error (not cascading noise).
+2. Classify: `./gradlew` compile, ktlint/detekt/ktlint format, unit JUnit, Android instrumentation, signing/provisioning, Gradle modules resolve, flaky UI test.
+3. Reproduce locally when feasible (`./gradlew build` / `./gradlew test` / lint scripts as in repo CI).
+4. Inspect changed files in the git diff related to the failure.
+5. Apply **minimal** fix — do not refactor unrelated code.
+6. Document in `.claude/team/tasks/<slug>/ci-report.md`:
+
+```markdown
+# CI report: <slug>
+
+## Failed check
+...
+
+## Root cause
+...
+
+## Evidence
+Log excerpt / local reproduction...
+
+## Fix applied
+...
+
+resolved: true | false
+
+## Validation
+Commands run and results.
+```
+
+## When to hand off
+
+- Feature work beyond CI scope → document in `ci-report.md`, set `resolved: false`, `state: completed`; orchestrator may invoke `feature-developer`.
+- Flaky Android instrumentation after root cause identified → prefer `instrumentation-test-healer`.
+- Investigation-only request → stop after report without code changes.
+
+## Stack constraints
+
+- Do **not** run `yarn`, `lint:js`, Playwright, or Next.js scripts.
+- Prefer Gradle modules, build variants, and product flavors documented in `tooling-and-review/gradle-tooling.md`.
+
+## 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:** `ci-report.md` plus any minimal CI fix paths; upsert only the manifest entry assigned by handoff (`artifactOutputId`; fallback `ci-report`) and preserve all foreign entries.
+- **Terminal receipt:** cover applicable AC/task IDs, list authoritative evidence paths, and emit `completed` or `blocked`. 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.
+
+Provide summary: failed check, root cause, files changed, validation commands run, `resolved`.
+
+## Anti-sycophancy discipline
+
+- When writing or changing code, load / follow rule `anti-sycophancy-discipline`.