@bonesofspring/ai-rules 0.2.15 → 0.2.17

package/presets/claude/svelte/rules/architecture/product-specs-authoring.md

@@ -0,0 +1,121 @@
+---
+paths:
+  - docs/specs/**/*
+---
+
+<!-- shared-core: architecture/product-specs-authoring.md -->
+
+# Product specs authoring
+
+Load when **Generate** or **Bootstrap** is requested (`/spec-start`, router `spec-gen`, brief AC, or analyst author mode). Discovery-only work stays on **`product-specs`**.
+
+## Modes
+
+1. **Consume** — default; never invent INDEX/specs (see `product-specs`).
+2. **Generate** — create/update page/feature specs + INDEX rows from brief AC, user intent, Figma, or code archaeology. Inventing *product* behavior is forbidden without a stated source (AC / user / UI copy / code-observed behavior labeled `status: draft` or `inferred`).
+3. **Bootstrap** — create `INDEX.md` + `_templates/` copies + first domain spec under greenfield policy.
+
+## Greenfield policy
+
+Agents **may** create INDEX + first specs **only when** one of:
+
+1. User runs `/spec-start` or explicitly asks to bootstrap `docs/specs`.
+2. Brief/AC includes generation or bootstrap language.
+3. Router detects `spec-gen` intent.
+
+Otherwise: missing INDEX → optional tree; do **not** invent INDEX merely because discovery loaded.
+
+## Conflict precedence
+
+1. **Human gate** decision (approved brief/spec edits).
+2. **Current task `brief.md` AC** — if AC contradicts an existing spec, analyst must reconcile (update spec as `draft` or revise AC) before developer; no silent contradiction.
+3. **Approved `docs/specs/**`** — behavior SoT for feature-developer once gate passed.
+4. **Code / `.ai/context`** — discovery evidence only; never override approved spec without gate.
+
+Label inferred-from-code lines with `status: draft` until human promotes.
+
+## INDEX.md contract (v1)
+
+| Column | Required | Notes |
+|--------|----------|-------|
+| `id` | yes | Stable kebab/id matching frontmatter `id` |
+| `type` | yes | `page` \| `feature` |
+| `domain` | yes | Domain folder under `docs/specs/domains/<domain>/` |
+| `title` | yes | Human label |
+| `path` | yes | Repo-relative path to markdown file |
+| `routes` | no | Summary; may duplicate frontmatter |
+| `aliases` | no | Comma-separated PageName / deep-link aliases |
+| `status` | no | `draft` \| `active` \| `deprecated` |
+
+Layout: `docs/specs/domains/<domain>/<id>.md` (page) and optional `docs/specs/domains/<domain>/feature.md`. Adding/renaming/removing a linked file **must** update INDEX in the **same change set**.
+
+## Frontmatter v1
+
+| Field | Type | Required | Notes |
+|-------|------|----------|-------|
+| `id` | string | yes | Unique; matches INDEX |
+| `type` | enum `page` \| `feature` | yes | Backend/MCP: prefer `feature` |
+| `domain` | string | yes | |
+| `routes` | string[] | yes (may be `[]`) | Web: URL paths; mobile: deep links/screens; backend: HTTP paths; mcp-ts: tool names |
+| `ui` | string[] | yes (may be `[]`) | Component/screen refs; empty on backend/mcp |
+| `store` | string[] | yes (may be `[]`) | Slice/module refs; empty when N/A |
+| `api` | string[] | yes (may be `[]`) | Services/ports/tools |
+| `aliases` | string[] | yes (may be `[]`) | |
+| `e2e` | string[] | yes (may be `[]`) | Stack-appropriate test paths |
+| `status` | enum | no | default `draft` on agent generate |
+| `owners` | string[] | no | |
+| `related` | string[] | no | Spec ids or paths |
+
+Arrays are **always present** (empty allowed). Do not omit keys for N/A stacks.
+
+## Body headings (minimum)
+
+**`page` required:** Overview; Entry / actors; Primary flows; States (loading / empty / error / success as applicable — mark N/A explicitly); Acceptance criteria (links or bullets to brief AC / product source).
+
+**`page` optional:** Analytics, Accessibility, Edge cases, Related specs.
+
+**`feature` required:** Overview; Shared behavior / constraints; Related pages (links).
+
+**`feature` optional:** Cross-cutting states, Analytics.
+
+Compliant = valid frontmatter + all required headings present (non-empty prose or explicit `N/A`) + INDEX row with resolvable `path`.
+
+## Templates & assets
+
+**Package SoT:** `packages/ai-rules/presets/_shared/assets/docs-specs/` (`INDEX.md` stub, `_templates/page.md`, `_templates/feature.md`; optional `schemas/frontmatter-v1.schema.json`).
+
+**CLI `init`:** when consumer `docs/specs/` is **absent**, copy INDEX stub + `_templates/*` (idempotent; **never overwrite** existing INDEX). When the tree already exists, skip seed entirely.
+
+**Skill fallback:** if consumer `_templates/` missing during author mode, copy from package/skill assets before authoring.
+
+## Author roles
+
+| Role | Responsibility |
+|------|----------------|
+| **task-analyst** | Primary author (Generate + Bootstrap); INDEX bootstrap; brief **Related specs** + **Spec gaps / generation requested**; loads this stem + skill `spec-authoring` |
+| **tech-writer** | Post-delivery sync where present; never invent product behavior without AC/code source |
+| **feature-developer** | Consumes approved specs as SoT; mechanical INDEX/frontmatter fixes only on renames |
+| **mcp-ts** | No tech-writer — author + post-sync via **task-analyst** only |
+
+## Self-check before handoff (soft block)
+
+Block handoff via `open_questions` until fixed:
+
+- [ ] Required frontmatter keys present; arrays always present (may be `[]`)
+- [ ] Required body headings present (or explicit `N/A`)
+- [ ] INDEX row added/updated with resolvable `path` in the **same change set**
+- [ ] Sources cited; invented product behavior absent (or labeled `draft` / `inferred`)
+- [ ] Conflict with brief AC reconciled or flagged
+
+## Related
+
+- `product-specs` — discovery / consume
+- Skill `spec-authoring` — procedural checklist
+- Optional consumer script `scripts/check-product-specs.sh` (WARN default; `--strict` FAIL)
+
+
+## Stack notes
+
+- Author via skill `spec-authoring`; templates from package assets / consumer `docs/specs/_templates/`.
+- Field semantics match discovery stem (`routes` = URL paths; `e2e` = Playwright).
+- Requestable — not always-on / session-start.

package/presets/claude/svelte/rules/architecture/product-specs.md

@@ -0,0 +1,62 @@
+---
+paths:
+  - docs/specs/**/*
+---
+
+<!-- shared-core: architecture/product-specs.md -->
+
+# Product specs (docs/specs)
+
+Product page/feature **behavior** specs live under the consumer repo `docs/specs/**`. This stem covers **discovery and presence** — not layer implementation order (see `feature-delivery-workflow`) and not code-structure mirroring (see `reference-features`).
+
+**Authoring** (Generate / Bootstrap modes, INDEX/frontmatter v1, body checklist, greenfield, conflict precedence) → load **`product-specs-authoring`** when generation is requested (`/spec-start`, brief AC, or router `spec-gen`).
+
+## Modes (summary)
+
+| Mode | When | Behavior |
+|------|------|----------|
+| **Consume** (default) | INDEX exists or globs hit `docs/specs/**` without generate intent | Discover via INDEX; soft-fail on miss; **never invent** INDEX/specs |
+| **Generate** | Explicit generate intent / AC / `/spec-start` with existing tree | Create/update page/feature specs + INDEX from stated sources |
+| **Bootstrap** | Explicit bootstrap / `/spec-start` when tree absent | Create INDEX + templates + first domain spec |
+
+Greenfield: agents may create INDEX/specs **only** on explicit intent, AC, or `/spec-start`. Missing INDEX alone does **not** authorize inventing the tree.
+
+## Presence policy
+
+- If `docs/specs/INDEX.md` is **missing** → the tree is **optional** (greenfield OK for *consume*). Continue with code/brief discovery; do not invent an INDEX solely because this stem loaded.
+- If INDEX **exists** → it is the **required discovery path** for work that touches a listed page, feature, route, or alias.
+
+## Discovery algorithm
+
+1. Always start at `docs/specs/INDEX.md` when present.
+2. Resolve the entry by: route, stable `id`, PageName / alias, or domain folder.
+3. Read the linked page spec (and optional domain `feature.md` when the INDEX or page Related links it).
+
+## Miss policy (INDEX present, row or file missing)
+
+- Record the gap in `open_questions` / handoff gaps.
+- Fall back to code discovery and consumer `.ai/context/**` when present.
+- **Do not invent** product behavior, states, interactions, or contracts.
+- Hard-fail / block only when the task’s acceptance criteria explicitly require a named spec that is absent (human/gate decision).
+
+## Authoring pointer
+
+Same-changeset INDEX updates, frontmatter v1, required body headings, templates, and conflict precedence live in **`product-specs-authoring`**. Do not paste the full checklist here.
+
+## Non-goals
+
+- Not a replacement for `.ai/context/**` (external AI Code Review glossary) or `reference-features` (file-layout mirror).
+- Not Tessl / an external host as primary SoT.
+- No stack filesystem path recipes in this core body — path examples belong only in adapter **Stack notes**.
+
+## Related
+
+- `product-specs-authoring` — generation modes, contracts, self-check.
+- `reference-features` — code structure mirror (how to layout files), not product behavior.
+- Consumer `.ai/context/**` when present — review/domain glossary; cross-link, do not replace.
+
+
+## Stack notes
+
+- `routes`: SvelteKit URL paths. `ui` / `store`: routes/components + store modules. `e2e`: Playwright paths under repo `tests/e2e/**` (or equivalent).
+- Requestable / `docs/specs/**` globs — not always-on / session-start.

package/presets/claude/svelte/rules/tooling-and-review/agent-team-orchestrator.md

@@ -54,6 +54,8 @@ Work request without `/task` — see **`agent-team-intake.md`** (when to suggest
 
 Full prompts: `.claude/agents/*.md`. Artifact conventions: `.claude/team/README.md`; schema SoT: **agent-artifact-contracts**.
 
+**Product specs:** When `/task`, `/feature-start`, or `/spec-start` touches pages/features/`docs/specs`: if INDEX exists and no generate intent → instruct next agent to load requestable **`product-specs`** (consume; soft-fail; never invent). If generate/bootstrap/`spec-gen` → load **`product-specs-authoring`** + skill **`spec-authoring`** inside **task-analyst** (not a separate pipeline step); pass created spec paths + INDEX diff in handoff; human gate must approve before feature-developer. No-op if INDEX absent and no author intent.
+
 When editing `packages/ai-rules/presets/**`, follow **`preset-pr-checklist.mdc`** (+ twin-sync / layering / token-budget / leakage meta-rules).
 
 ## Dynamic pipeline

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/claude/svelte/skills/feature-delivery/SKILL.md

@@ -6,9 +6,10 @@ description: Delivers SvelteKit frontend features end-to-end across domain types
 # Feature Delivery
 
 1. Read task brief, acceptance criteria, and decomposition if present.
-2. Read **`feature-delivery-workflow`** and **`reference-features`** — follow layer order and mirror closest reference feature.
-3. Add or update focused tests for changed behavior.
-4. Validation from SvelteKit root: **invoke** **`post-change-lint`** after code edits (requestable, not always-on — but **must** when code changed). Pipeline: scoped lint OK if next step is `build-verifier`.
+2. If `docs/specs/INDEX.md` exists **or** brief/handoff lists approved generated specs, resolve via **`product-specs`** and treat approved specs as behavior SoT before layer work (soft-fail on miss — do not invent behavior). If generation was requested but not yet approved, stop for human gate — do not implement against draft-only gaps.
+3. Read **`feature-delivery-workflow`** and **`reference-features`** — follow layer order and mirror closest reference feature.
+4. Add or update focused tests for changed behavior.
+5. Validation from SvelteKit root: **invoke** **`post-change-lint`** after code edits (requestable, not always-on — but **must** when code changed). Pipeline: scoped lint OK if next step is `build-verifier`.
 
 ## Scope discipline
 

package/presets/claude/svelte/skills/spec-authoring/SKILL.md

@@ -0,0 +1,26 @@
+---
+name: spec-authoring
+description: Author or bootstrap consumer docs/specs (INDEX, page/feature templates, frontmatter v1). Use with task-analyst author mode, /spec-start, or when brief AC requests Generate/Bootstrap.
+---
+
+# Spec authoring
+
+Requestable — not always-on. Load **`product-specs`** + **`product-specs-authoring`** first.
+
+## Steps
+
+1. **Resolve mode** — Generate (tree exists) vs Bootstrap (create INDEX + templates + first spec). Refuse inventing INDEX without explicit intent/AC/`/spec-start`.
+2. **Ensure templates** — if consumer `docs/specs/_templates/` missing, copy from package SoT `presets/_shared/assets/docs-specs/_templates/` (or seeded init assets).
+3. **Fill frontmatter v1** — all array keys present (may be `[]`); prefer `type: feature` on backend/MCP; default `status: draft`.
+4. **Fill required body headings** — page: Overview, Entry/actors, Primary flows, States, Acceptance criteria; feature: Overview, Shared behavior/constraints, Related pages. Use `N/A` explicitly when needed.
+5. **Update INDEX** in the **same change set** (columns: id, type, domain, title, path, optional routes/aliases/status).
+6. **Self-check** against `product-specs-authoring` checklist; put blockers in `open_questions`.
+7. **Handoff** — list created/updated paths + INDEX diff for human gate (`after:task-analyst`). Approve before feature-developer.
+
+## Sources
+
+Cite AC / user / Figma / labeled code inference. Do not invent product behavior.
+
+## mcp-ts
+
+Analyst-only path (no tech-writer). `routes` = tool names; `ui`/`store` = `[]`.

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.