@bonesofspring/ai-rules 0.2.15 → 0.2.16

package/presets/claude/php-laravel/rules/tooling-and-review/preset-twin-sync.md

@@ -24,6 +24,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
 
@@ -48,5 +49,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/svelte/rules/tooling-and-review/preset-layering.md

@@ -23,7 +23,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/claude/svelte/rules/tooling-and-review/preset-no-cross-stack-leakage.md

@@ -14,11 +14,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`).
 
@@ -27,25 +28,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
 
@@ -55,7 +68,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`).
@@ -66,9 +79,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`.
 
@@ -79,4 +92,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/claude/svelte/rules/tooling-and-review/preset-pr-checklist.md

@@ -16,7 +16,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/claude/svelte/rules/tooling-and-review/preset-twin-sync.md

@@ -25,6 +25,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
 
@@ -49,5 +50,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/cursor/android-kotlin/AGENTS.md

@@ -0,0 +1,50 @@
+# Android Kotlin stack — Cursor agent team
+
+Short Cursor entry point. Detailed rules — `.cursor/rules/*.mdc`; scenarios — `.cursor/skills/**`.
+
+**Repo-root agents:** see [`AGENTS.md`](../../AGENTS.md) at the project root (cross-tool setup, commands, security, living conventions pointers).
+
+See `.cursor/rules/README.md` for the full catalog and loading strategy.
+
+## Quick start
+
+| Task | Where to look |
+|------|----------------|
+| Any new work request | `/task` → `commands/task.md` + `agent-team-orchestrator.mdc` (+ `agent-team-intake.mdc` on-demand) |
+| New end-to-end feature | skill `feature-delivery` + `feature-delivery-workflow.mdc` |
+| Stack and layers | `android-app-core.mdc`, `architecture-boundaries.mdc` |
+| Reference modules | `reference-features.mdc` (placeholders `applications/`, `features/<name>/`, `build-logic/`) |
+| Toolchain / Gradle | `gradle-tooling.mdc` (requestable) |
+| After Kotlin/Android edits | `post-change-build.mdc` (**requestable**, required after edits; full gate — `build-verifier`) |
+| Code review MR | skill `code-review` + `code-review-mr.mdc` |
+| UI tests | skill `instrumentation-ui-e2e` + agents `instrumentation-test-*` |
+| Security | `security-android.mdc` |
+| Compose / MVI | `compose-ui.mdc`, `state-and-viewmodels.mdc` |
+| Validation before review | agent `build-verifier` |
+| Technical retro | `/technical-retro` + skill `technical-retro` |
+
+## Dogfood (5 scenarios)
+
+1. `/task` — new feature under `features/<name>/`.
+2. ViewModel bug — debugger + unit test.
+3. Mapper — unit-test-planner/generator.
+4. UI flow — instrumentation-test trio.
+5. Auth/storage — security-reviewer + `security-android.mdc`.
+
+## When to use `/task`
+
+- Implementation, bugfix, migration, tests, MR review, CI-fix — **prefer `/task`**.
+- Simple one-liner with no architectural risk — may skip the pipeline if the user explicitly asks.
+- “How does X work?” questions — agent `codebase-analyzer` (explain-as-is) or answer without `/task`.
+
+## Pipeline routing
+
+Intent detection, profiles (`full` / `standard` / `light`), and default steps — **only** in `agents/task-router.md`. Schema and rename map — `agents/README.md`.
+
+## Install
+
+```bash
+npx @bonesofspring/ai-rules init cursor --preset android-kotlin
+```
+
+Copies rules, commands, agents, hooks, skills, team, `hooks.json`, `BUGBOT.md`, `AGENTS.md` → `.cursor/AGENTS.md`, `MCP.md`, `mcp.json` → `.cursor/mcp.json`, plus `REPO_AGENTS.md` → root `AGENTS.md`. Skill: `write-adr` (on-demand).

package/presets/cursor/android-kotlin/BUGBOT.md

@@ -0,0 +1,16 @@
+# Bugbot — Android Kotlin
+
+Automated review hints for Cursor Bugbot on Android/Kotlin multimodule repos.
+
+## Focus
+
+- Clean Architecture / api-impl boundaries (`architecture-boundaries`)
+- No Hilt-by-default regressions when the repo uses Dagger feature holders
+- Compose screens staying thin; no data-layer leaks into UI
+- Secrets / TLS / logging (`security-android`)
+- Missing unit or instrumentation coverage called out by AC
+
+## Do not
+
+- Require Playwright or XCUITest recipes
+- Demand yarn `lint:js` / `lint:css`

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

@@ -0,0 +1,16 @@
+# MCP (preset `android-kotlin`, Cursor)
+
+Copied to `.cursor/mcp.json` on `ai-rules init cursor --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 Cursor |
+
+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: Settings → MCP → enable servers. `init` overwrites a same-named `mcp.json`.

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

@@ -0,0 +1,35 @@
+# Preset: android-kotlin (Cursor)
+
+Installable Android/Kotlin multimodule preset: Clean Architecture, Jetpack Compose MVI, Dagger feature holders (no Hilt by default), Gradle convention plugins, instrumentation UI-test agents.
+
+```bash
+npx @bonesofspring/ai-rules init cursor --preset android-kotlin
+```
+
+Copies rules, agents, skills, commands, hooks, team, `MCP.md`, `mcp.json` → `.cursor/mcp.json`, plus `REPO_AGENTS.md` → root `AGENTS.md`.
+
+## Layout placeholders
+
+Generic multimodule layout encoded in rules (map your repo in `team/conventions.md` after install):
+
+`applications/`, `core/`, `features/<name>/{api,impl}/`, `build-logic/`, `gradle/libs.versions.toml`
+
+## Agent surface
+
+- UI tests: `instrumentation-test-{planner,generator,healer}` + skill `instrumentation-ui-e2e`
+- Gates: `post-change-build` + `gradle-tooling` + `build-verifier`
+- Claude twin: `presets/claude/android-kotlin/` (same PR policy)
+
+See `rules/README.md` and `agents/README.md`.
+
+## Install troubleshooting
+
+- **Gradle wrapper:** run from the Gradle root where `./gradlew` (or a wrapper script documented in the repo README/CI) lives; agents load `gradle-tooling.mdc` for discovery — do not assume a single hard-coded script name.
+- **Modules:** use `./gradlew projects` (or CI task names) to resolve affected `:module` paths before compile/test gates.
+- **Variants / flavors:** product flavors and build types differ per repo; note the CI variant (e.g. `debug` vs `release`) in `team/conventions.md` and pass `-P` / `--variant` flags only as documented locally.
+- **Multimodule:** scope `./gradlew :feature:impl:compileDebugKotlin` (example pattern) to changed modules; full gate — agent `build-verifier`.
+
+## Preset taxonomy
+
+- Contribution: [`../../docs/PRESET-CONTRIBUTION.md`](../../docs/PRESET-CONTRIBUTION.md)
+- Core→stack mapping: [`rules/README.md`](./rules/README.md)

package/presets/cursor/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/cursor/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/cursor/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. `.cursor/team/tasks/<slug>/brief.md` — acceptance criteria and UI scope.
+2. `.cursor/team/tasks/<slug>/decomposition.md` — UI-related tasks.
+3. Git diff for changed Views / UI under `features/**/Presentation/**` (and design-system if touched).
+4. `compose-ui.mdc` 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 `.cursor/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 `.cursor/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/cursor/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. `.cursor/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 `networking-data.mdc`, `architecture-boundaries.mdc`, `module-public-api.mdc` 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 `.cursor/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 `.cursor/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/cursor/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 `gradle-tooling.mdc` + `post-change-build.mdc`. Run compile / configured lint / unit tests at CI parity even if the developer ran scoped checks.
+
+## Inputs
+
+1. `.cursor/team/tasks/<slug>/brief.md` and `decomposition.md`.
+2. `.cursor/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 `gradle-tooling.mdc` (`./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 `.cursor/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 `.cursor/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/cursor/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. `.cursor/team/tasks/<slug>/pipeline.json` and `brief.md` if present.
+3. Local reproduction: same scheme, configuration, and destination CI uses (`gradle-tooling.mdc`).
+
+## 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 `.cursor/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 `gradle-tooling.mdc`.
+
+## On completion
+
+Write the owned terminal receipt described under **Artifact contract**.
+
+## Artifact contract
+
+- **Authoritative inputs:** read `.cursor/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`.