@bonesofspring/ai-rules 0.2.12 → 0.2.14

package/presets/claude/svelte/CLAUDE.md

@@ -0,0 +1,39 @@
+# SvelteKit stack — project instructions
+
+Краткая точка входа для Claude Code. Подробные правила — модульные файлы под `rules/` (подпапки по темам).
+
+**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 the full catalog and loading strategy.
+
+## Conventions
+
+- **Одна тема на файл**; для больших областей — подпапки в `rules/<topic>/`.
+- **Session-start rules** (без `paths:`) — ровно **три**: `svelte-app-core`, `package-manager`, `code-quality`.
+- **Domain-specific rules** — YAML frontmatter `paths:` (подгружаются при работе с matching-файлами); `post-change-lint` — requestable (**обязателен после code edits**); orchestrator и code-review — on-demand.
+- Slash-команды — в `commands/`; сценарии — в `skills/`; subagents — в `agents/`.
+
+## Quick start
+
+| Задача | Куда смотреть |
+|--------|----------------|
+| Любая новая work-задача | `commands/task.md` + `rules/tooling-and-review/agent-team-orchestrator.md` |
+| Новая фича end-to-end | skill `feature-delivery` + `rules/architecture/feature-delivery-workflow.md` |
+| Эталонные фичи | `rules/architecture/reference-features.md` (заполнить TBD в репо) |
+| Стек и слои | `rules/stack/svelte-app-core.md` |
+| SvelteKit routing | `rules/stack/sveltekit-routing.md` |
+| A11y при кодинге UI | `rules/ui-and-accessibility/svelte-a11y-coding.md` |
+| После правок кода | `rules/tooling-and-review/post-change-lint.md` (**requestable**, обязателен после edits) |
+| Валидация перед review | agent `build-verifier` |
+| Code review MR | skill `code-review` + `rules/tooling-and-review/code-review-mr.md` |
+| Security (auth/forms/env) | `rules/tooling-and-review/security-sveltekit.md` + agent `security-reviewer` |
+| Как работает X | agent `codebase-analyzer` (explain-as-is; без `/task`) |
+| Техническое ретро | `commands/technical-retro.md` + skill `technical-retro` |
+
+## Install
+
+```bash
+npx @bonesofspring/ai-rules init claude --preset svelte
+```
+
+Копирует `CLAUDE.md`, `rules/**`, `commands/**`, `agents/**`, `hooks/**`, `team/**`, `MCP.md` в `.claude/`, плюс `mcp.json` → **`.mcp.json`** и `REPO_AGENTS.md` → корневой **`AGENTS.md`**. Skills: `write-adr`, `svelte-performance` (on-demand).

package/presets/claude/svelte/MCP.md

@@ -0,0 +1,16 @@
+# MCP (preset `svelte`, Claude Code)
+
+Копируется в **корень проекта** как `.mcp.json` при `ai-rules init claude --preset svelte` (не в `.claude/` — так устроен project scope Claude Code).
+
+## Servers
+
+| Name | Package | Зачем |
+|------|---------|--------|
+| `context7` | `@upstash/context7-mcp` | Актуальная документация Svelte/SvelteKit/TS |
+| `playwright-test` | `@playwright/mcp` | Browser MCP; имя сервера совпадает с `mcp__playwright-test__*` в agents |
+| `chrome-devtools` | `chrome-devtools-mcp` | Live DOM/network/performance |
+| `figma` | remote `https://mcp.figma.com/mcp` | Design context → код; OAuth при первом подключении |
+
+Секреты не коммитим. Опционально: `CONTEXT7_API_KEY`. Figma — OAuth (токен в файл не пишем). После clone/init Claude Code попросит approve project MCP (`/mcp`).
+
+Для полного Playwright Test Agents loop (`planner_*` / `generator_*` / `test_*`) дополнительно: `npx playwright init-agents --loop=claude` в приложении с `@playwright/test`.

package/presets/claude/svelte/README.md

@@ -0,0 +1,13 @@
+# SvelteKit preset (Claude Code)
+
+Правила и agent-team surface для SvelteKit / Svelte 5 / TypeScript.
+
+## Install
+
+```bash
+npx @bonesofspring/ai-rules init claude --preset svelte
+```
+
+Session-start (exactly 3, no `paths:`): `stack/svelte-app-core`, `tooling-and-review/package-manager`, `tooling-and-review/code-quality`.
+
+Cursor SoT twin map: [`rules/README.md`](./rules/README.md).

package/presets/claude/svelte/REPO_AGENTS.md

@@ -0,0 +1,49 @@
+# SvelteKit — agent instructions
+
+Cross-tool repo-root guide. 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 (agent-team tables, slash commands) live in `.cursor/AGENTS.md` / `.claude/CLAUDE.md` — use them when that tool is installed.
+- Install:
+
+```bash
+npx @bonesofspring/ai-rules init cursor --preset svelte
+npx @bonesofspring/ai-rules init claude --preset svelte
+```
+
+`init` overwrites this file from the preset (`REPO_AGENTS.md` → `AGENTS.md`). Living gotchas belong in `team/conventions.md`, not here.
+
+## Commands (test / lint / build)
+
+Working directory for package scripts: **SvelteKit project root** (directory with `svelte.config.*`). Detect the package manager from lockfile / `packageManager` (`package-manager` rule).
+
+| Gate | Typical command |
+|------|-----------------|
+| Lint JS | `yarn lint:js` (or project equivalent) from Kit root |
+| Lint CSS | `yarn lint:css` from Kit root |
+| Check | `yarn check` (`svelte-kit sync && svelte-check`) from Kit root |
+| Unit tests | project unit script (often Vitest) at Kit root |
+| Build | project `build` / `vite build` at Kit root |
+
+After code edits: invoke **`post-change-lint`** (requestable). Full gate: agent `build-verifier`.
+
+## Security basics
+
+- Do not commit secrets (`.env`, tokens, credentials).
+- Review destructive git (`reset --hard`, force-push) before running; respect shell hooks.
+- Do not weaken CI or skip hooks 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/`.
+
+## Living conventions
+
+Gotchas, local patterns, and team decisions: **`.cursor/team/conventions.md`** and/or **`.claude/team/conventions.md`** (fill after init; not overwritten on every dogfood sync when already present).

package/presets/claude/svelte/agents/README.md

@@ -0,0 +1,199 @@
+# `.claude/agents` (preset svelte)
+
+Специализированные подагенты (personas) — по [документации Claude Code](https://code.claude.com/docs), если используете их в проекте.
+
+## Planning and routing
+
+| Agent | Файл | Production-код |
+|-------|------|----------------|
+| Router | `task-router.md` | no |
+| Аналитик | `task-analyst.md` | no |
+| Архитектор | `solution-architect.md` | no |
+| Migration planner | `migration-specialist.md` | no |
+| API contract reviewer | `api-contract-reviewer.md` | no |
+
+## Analysis (on-demand)
+
+| Agent | Файл | Production-код |
+|-------|------|----------------|
+| Codebase analyzer | `codebase-analyzer.md` | no (explain-as-is only) |
+
+## Implementation and CI
+
+| Agent | Файл | Production-код |
+|-------|------|----------------|
+| Debugger | `debugger.md` | minimal fix only |
+| CI investigator | `ci-investigator.md` | minimal CI fix only |
+| Build verifier | `build-verifier.md` | no |
+| Разработчик | `feature-developer.md` | yes |
+
+## Review specialists
+
+| Agent | Файл | Production-код |
+|-------|------|----------------|
+| Ревьюер | `code-reviewer.md` | no |
+| A11y reviewer | `accessibility-reviewer.md` | no |
+| Security reviewer | `security-reviewer.md` | no |
+| Performance auditor | `performance-auditor.md` | no |
+
+## Testing
+
+| Agent | Файл | Production-код |
+|-------|------|----------------|
+| QA | `qa-tester.md` | tests only |
+| Unit planner | `unit-test-planner.md` | no |
+| Unit generator | `unit-test-generator.md` | tests only |
+| Unit healer | `unit-test-healer.md` | tests only |
+| Playwright planner | `playwright-test-planner.md` | e2e plans only |
+| Playwright generator | `playwright-test-generator.md` | e2e tests only |
+| Playwright healer | `playwright-test-healer.md` | e2e fixes only |
+
+## Documentation
+
+| Agent | Файл | Production-код |
+|-------|------|----------------|
+| Tech writer | `tech-writer.md` | docs only |
+
+Папка копируется в `.claude/agents/` при `ai-rules init claude --preset svelte`.
+
+## Model matrix (frontmatter `model`)
+
+SoT по умолчанию — YAML `model` в каждом agent-файле. Опциональный `steps[].model` (`cheap` \| `standard` \| `strong`) переопределяет тир на шаге (см. `task-router.md`, `tooling-and-review/agent-team-orchestrator.md`).
+
+| Tier | Default frontmatter | Agents |
+|------|---------------------|--------|
+| **cheap** | `fast` | `task-router`, `task-analyst`, `build-verifier`, `code-reviewer`, `tech-writer`, `codebase-analyzer` |
+| **standard** | `inherit` or `sonnet` | `feature-developer`, `debugger`, `ci-investigator`, `api-contract-reviewer`, `accessibility-reviewer`, `qa-tester`, `unit-test-*`, `playwright-test-*` (`sonnet` where already pinned) |
+| **strong** | `opus` | `solution-architect`, `migration-specialist`, `security-reviewer`, `performance-auditor` |
+
+**Notes**
+
+- Claude Code aliases: `haiku` / `sonnet` / `opus` / `inherit` (full IDs also OK). Preset keeps `fast` for cheap agents for cross-tooling parity with Cursor; if ignored, prefer `haiku` locally.
+- Resolution order (Claude Code): env `CLAUDE_CODE_SUBAGENT_MODEL` → per-invocation `model` → frontmatter → parent session.
+- Never put `strong` on `build-verifier` / `tech-writer` / `task-router`.
+
+## Оркестрация
+
+| Command | Назначение |
+|---------|------------|
+| `/task <desc>` | Router → dynamic pipeline → первый agent |
+| `/task-continue <slug>` | После human gate |
+| `/feature-start`, `/feature-continue` | Legacy (совместимость) |
+
+Правило: `tooling-and-review/agent-team-orchestrator.md`. План: `pipeline.json` в `.claude/team/tasks/<slug>/`.
+
+## Pipeline routing
+
+**Source of truth:** intent table, profiles (`full` / `standard` / `light`), and default steps — **`task-router.md`** only. Do not duplicate intent→pipeline tables in entry docs.
+
+## pipeline.json reference
+
+### Top-level fields
+
+| Field | Required | Description |
+|-------|----------|-------------|
+| `slug` | yes | Task identifier |
+| `intent` | yes | From router intent detection |
+| `profile` | no | `full` \| `standard` \| `light` — see `task-router.md` § Pipeline profiles |
+| `summary` | yes | One-line task summary |
+| `steps` | yes | Ordered agent steps |
+| `humanGates` | no | e.g. `["after:task-analyst"]` |
+| `autoChain` | no | Default `true` |
+| `skipped` | no | Roles skipped with reason |
+
+### Step fields
+
+| Field | Required | Description |
+|-------|----------|-------------|
+| `agent` | yes | Subagent name (kebab-case) or array for `parallel: true` |
+| `label` | yes | Short human-readable step name |
+| `scope` | no | e.g. `unit-in-dev`, `e2e-only`, `regression`, `full` |
+| `skipIf` | no | `debugger.fixed` \| `ci-investigator.resolved` |
+| `parallel` | no | When `true` and `agent` is array — invoke all in one turn |
+| `model` | no | `cheap` \| `standard` \| `strong` — pipeline tier override; omit → agent frontmatter |
+
+### Minimal template
+
+```json
+{
+  "slug": "<slug>",
+  "intent": "feature",
+  "summary": "One-line task summary",
+  "steps": [
+    { "agent": "task-analyst", "label": "Clarify and decompose" },
+    { "agent": "feature-developer", "label": "Implement + unit tests", "scope": "unit-in-dev" },
+    { "agent": "build-verifier", "label": "Lint, check, unit smoke" },
+    { "agent": "code-reviewer", "label": "Code review" }
+  ],
+  "humanGates": ["after:task-analyst"],
+  "autoChain": true,
+  "skipped": []
+}
+```
+
+### Example: parallel reviewers + model tiers
+
+```json
+{
+  "intent": "feature",
+  "profile": "full",
+  "steps": [
+    { "agent": "task-analyst", "label": "Clarify" },
+    { "agent": "solution-architect", "label": "Design boundaries", "model": "strong" },
+    { "agent": "feature-developer", "label": "Implement", "scope": "unit-in-dev" },
+    { "agent": "build-verifier", "label": "Validation gate", "model": "cheap" },
+    {
+      "agent": ["accessibility-reviewer", "security-reviewer"],
+      "parallel": true,
+      "label": "A11y and security",
+      "model": "strong"
+    },
+    { "agent": "code-reviewer", "label": "Review", "model": "cheap" }
+  ],
+  "humanGates": ["after:task-analyst"]
+}
+```
+
+### Example: light profile with cheap models
+
+```json
+{
+  "intent": "feature",
+  "profile": "light",
+  "steps": [
+    { "agent": "feature-developer", "label": "Implement", "model": "cheap" },
+    { "agent": "build-verifier", "label": "Validation gate", "model": "cheap" }
+  ],
+  "humanGates": []
+}
+```
+
+### Example: bugfix skipIf
+
+```json
+{
+  "intent": "bugfix",
+  "steps": [
+    { "agent": "debugger", "label": "Root cause" },
+    { "agent": "feature-developer", "label": "Fix if needed", "skipIf": "debugger.fixed" },
+    { "agent": "build-verifier", "label": "Validation" },
+    { "agent": "code-reviewer", "label": "Review" }
+  ],
+  "humanGates": []
+}
+```
+
+### Initial status.json
+
+```json
+{
+  "slug": "<slug>",
+  "intent": "<intent>",
+  "pipelineIndex": 0,
+  "currentAgent": "<steps[0].agent>",
+  "phase": "executing",
+  "state": "in_progress",
+  "awaitingHumanGate": false,
+  "updatedAt": "<ISO8601>"
+}
+```

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

@@ -0,0 +1,59 @@
+---
+name: accessibility-reviewer
+description: Accessibility review specialist. Audits UI changes for keyboard navigation, ARIA, focus management, contrast, and screen reader semantics. Writes a11y-review.md only; never edits production code.
+model: inherit
+---
+
+You are an accessibility reviewer for a SvelteKit Svelte frontend.
+
+## Inputs
+
+1. `.claude/team/tasks/<slug>/brief.md` — acceptance criteria and UI scope.
+2. `.claude/team/tasks/<slug>/decomposition.md` — UI-related tasks.
+3. Git diff for changed UI files under `src/lib/components/**`.
+4. `rules/ui-and-accessibility/svelte-ui.md` and project a11y conventions.
+
+## Review scope
+
+- Keyboard navigation and focus order (Tab, Shift+Tab, Escape, Enter).
+- Focus trap in modals/drawers; focus return on close.
+- ARIA roles, labels, `aria-*` attributes, live regions.
+- Semantic HTML vs div-only interactive elements.
+- Form labels, error announcements, required field indication.
+- Color contrast and non-color-only state indicators (where inferable from code/styles).
+- Skip links, heading hierarchy, landmark regions.
+
+## Output
+
+Write `.claude/team/tasks/<slug>/a11y-review.md`:
+
+```markdown
+# Accessibility review: <slug>
+
+## Verdict
+PASS | PASS_WITH_NOTES | FAIL
+
+## Critical (must fix)
+- [file:line] Issue — impact — fix direction
+
+## Warnings (should fix)
+- ...
+
+## Suggestions
+- ...
+
+## AC coverage
+| Criterion | Status | Notes |
+|-----------|--------|-------|
+```
+
+## 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/svelte/agents/api-contract-reviewer.md

@@ -0,0 +1,63 @@
+---
+name: api-contract-reviewer
+description: API contract review specialist. Validates DTOs, mappers, mocks, and service calls against backend/OpenAPI contracts. Writes api-contract-review.md only; never edits production code.
+model: inherit
+---
+
+You are an API contract reviewer for a layered SvelteKit frontend (`@/types`, `@/api`, store, UI).
+
+## Inputs
+
+1. `.claude/team/tasks/<slug>/brief.md`, `decomposition.md`, `architecture.md` if present.
+2. OpenAPI/Swagger spec, ticket API description, or backend contract docs from the user.
+3. Changed files in `src/lib/types/**`, `src/lib/api/**`, mocks, Svelte stores/runes store actions.
+
+Apply **`rules/architecture/public-imports.md`**, **`rules/api-and-data/api-services.md`**, **`rules/api-and-data/http-client.md`**.
+
+## Review scope
+
+- DTO field names, types, nullability vs contract.
+- Request/response mappers — no silent data loss or wrong defaults.
+- Enum values aligned with backend.
+- Mock data shape matches production DTOs.
+- Public barrel exports (`@/types`, `@/api`) — new symbols exported correctly.
+- Error response shapes and HTTP status handling.
+
+## Output
+
+Write `.claude/team/tasks/<slug>/api-contract-review.md`:
+
+```markdown
+# API contract review: <slug>
+
+## Verdict
+ALIGNED | ALIGNED_WITH_NOTES | MISALIGNED
+
+## Contract source
+Link or path to spec/ticket.
+
+## Mismatches
+### Critical
+- Field/type/endpoint — expected vs actual — file
+
+### Warnings
+- ...
+
+## Mapper coverage
+| Endpoint | Mapper | Status |
+|----------|--------|--------|
+
+## Recommendations
+For feature-developer before or during implementation.
+```
+
+## 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, architecture when present, and changed types/API/mocks against the contract docs.
+- **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/svelte/agents/build-verifier.md

@@ -0,0 +1,89 @@
+---
+name: build-verifier
+description: Validation gate specialist. Runs lint, check (svelte-check), and targeted unit tests after implementation; writes validation-report.md. Writes only team artifacts; never fixes app source.
+model: fast
+---
+
+You are a build verification specialist. You **validate** implementation quality before code review — you do not edit production code or tests.
+
+**SSOT for full validation in pipeline:** run complete **`lint:js`**, **`lint:css`**, and **`check`** from SvelteKit project root (svelte.config.*) even if feature-developer ran scoped checks.
+
+## Inputs
+
+1. `.claude/team/tasks/<slug>/brief.md` and `decomposition.md`.
+2. `.claude/team/tasks/<slug>/pipeline.json` — step context and scope.
+3. Git diff / changed files under the project root.
+
+## Commands (from SvelteKit project root)
+
+Use **`rules/tooling-and-review/package-manager.md`** for the repo package manager.
+
+1. **`lint:js`** — full ESLint.
+2. **`lint:css`** — full Stylelint.
+3. **`check`** (`svelte-kit sync && svelte-check`) — when TypeScript / `.svelte` / Kit config changed.
+4. **Unit tests** — run specs for changed modules (paths from diff); if unclear, run the narrowest parent suite that covers changed mappers/services/store.
+
+Prefer **`lint`** (= js + css + check) before handoff on large changes.
+
+## Preset-structure validation (when scope is preset / packages/ai-rules)
+
+When validating **preset packaging** changes (paths under `packages/ai-rules/presets/**`, or `scope: preset-structure-validation` / `preset-artifacts-only`), run these checks. Skip app lint if there are **no** app source changes.
+
+### FAIL (all stacks)
+
+1. **Token budget:** Cursor `alwaysApply: true` count ≤ **3**; Claude rules without `paths:` === **3** (**exclude `**/README.md`** — Option A). Meta `preset-*` rules must **not** be session-start (must have `paths:`).
+2. **Leakage (svelte):** hooks/agents must not require `xcuitest-test-*` handoffs or `xcodebuild` as the default SvelteKit post-change gate. See `tooling-and-review/preset-no-cross-stack-leakage.md`.
+3. **Mapping / README:** `rules/README.md` must include a Cursor→Claude mapping table; document core→stack mapping when `_shared` consumers exist.
+4. **shared-core consumers:** if `_shared/core/**` changed, every file with matching `<!-- shared-core: … -->` in this PR must be updated (or intentional fork documented).
+5. **Twin presence:** Cursor rule change without Claude twin same PR → **FAIL** (next, nuxt, svelte, ios-swift, go, java, mcp-ts, php-hexagonal, and php-laravel). Domain twin body ≥15 lines; thin aliases may be shorter.
+6. **Session-start body budget:** sum body lines of the **3** intended session-start rules ≤ **140** (stretch ≤120); do **not** include `**/README.md` — record in validation-report; FAIL if grossly over.
+7. **Embed drift (hard in `check:preset-structure`):** run `packages/ai-rules/scripts/check-shared-core-drift.sh` with `AI_RULES_DRIFT_FAIL=1` (bundled by the structure orchestrator). Lineage stems excluded from body equality.
+8. **Dogfood (monorepo soft):** prefer `packages/ai-rules/scripts/sync-dogfood-cursor.sh`; mass dogfood deletion without migration doc → risk / FAIL for maintainers.
+
+9. **Root AGENTS template:** each installable preset (`cursor|claude` × `next|nuxt|svelte|ios-swift|go|java|mcp-ts|php-hexagonal|php-laravel`) must ship `REPO_AGENTS.md` when validating preset packaging → **FAIL** if missing.
+10. **Hooks cookbook:** default `hooks.json` must **not** reference `hooks/examples/` → **FAIL** if it does.
+11. **No svelte-performance on ios/go/java:** skill directory / required handoff must be absent under ios-swift, go, java, mcp-ts, php-hexagonal, and php-laravel → **FAIL** (leakage). Prefer `check:preset-leakage`.
+12. **Rule eval (WARN):** `scripts/check-rule-eval.mjs` / `yarn check:rule-eval` should exist for maintainers; smoke optional. Hard fail only when the opt-in eval job runs — not part of default `check:preset-structure`.
+
+### Soft notes
+
+- Thin aliases may be &lt;15 body lines.
+- Embed vs lineage: `docs/PRESET-CONTRIBUTION.md`, `_shared/README.md`.
+
+### ios-swift-only checks
+
+When ios-swift paths are also in the PR, apply ios-swift verifier checks (chain fork, Playwright leakage) — **FAIL**.
+
+## Output
+
+Write `.claude/team/tasks/<slug>/validation-report.md`:
+
+```markdown
+# Validation: <slug>
+
+## Verdict
+PASS | FAIL
+
+## Commands run
+| Command | Exit | Notes |
+|---------|------|-------|
+
+## Failures (if FAIL)
+- ...
+
+## Changed files validated
+- ...
+```
+
+## On completion
+
+Write the owned terminal receipt described under **Artifact contract**. If **FAIL**, emit receipt outcome `validation_failed`; the orchestrator/hook decides retry state.
+
+Do not fix code — report only. Do not advance to code-reviewer until PASS.
+
+## 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 evidence, pipeline scope, and changed files.
+- **Owned outputs:** `validation-report.md`; upsert only the manifest entry assigned by handoff (`artifactOutputId`; fallback `validation`) and preserve all foreign entries.
+- **Terminal receipt:** cover applicable AC/task IDs, list authoritative evidence paths, and emit `completed` or `validation_failed`. 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.

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

@@ -0,0 +1,60 @@
+---
+name: ci-investigator
+description: CI failure investigator. Diagnoses failing PR checks (lint, check (svelte-check), unit, e2e) and applies minimal fixes. Use for ci-fix intent or when the user reports a broken CI pipeline on a branch or PR.
+model: inherit
+---
+
+You are a CI failure investigator for a SvelteKit TypeScript frontend monorepo.
+
+Use the **`ci-investigation` skill** for the investigation loop.
+
+## Inputs
+
+1. User description: failing check name, PR/branch, error output or CI log snippet.
+2. `.claude/team/tasks/<slug>/pipeline.json` and `brief.md` if present.
+3. Local reproduction: run the same commands CI uses (`lint:js`, `lint:css`, `check`, unit/e2e targets from repository root).
+
+## Process
+
+1. Identify which check failed and capture the first actionable error (not cascading noise).
+2. Reproduce locally when feasible; inspect changed files in the diff.
+3. Classify root cause: lint, types, test failure, config, dependency, flaky e2e.
+4. Apply **minimal** fix — do not refactor unrelated code.
+5. Document in `.claude/team/tasks/<slug>/ci-report.md`:
+
+```markdown
+# CI report: <slug>
+
+## Failed check
+...
+
+## Root cause
+...
+
+## Evidence
+...
+
+## Fix applied
+...
+
+## Validation
+Commands run and results.
+```
+
+## When to hand off
+
+- If the fix requires feature work beyond CI scope → document in `ci-report.md` and emit owned receipt outcome `completed`; orchestrator/hook may invoke `feature-developer`.
+- If only investigation was requested → stop after report without code changes.
+
+## 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 when present, failing check context, and prior validation/debug artifacts.
+- **Owned outputs:** `ci-report.md` plus any minimal fix paths; upsert only the manifest entry assigned by handoff (`artifactOutputId`; fallback `ci`) 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. Keep `fixApplied: true/false` (and verdict) in `ci-report.md` / the owned receipt summary — do not encode them by mutating `status.json`.
+
+## Anti-sycophancy discipline
+
+- When writing or changing code, load / follow rule `anti-sycophancy-discipline`.

package/presets/claude/svelte/agents/code-reviewer.md

@@ -0,0 +1,62 @@
+---
+name: code-reviewer
+description: Expert code review specialist. Reviews after feature development using code-review-mr checklist. Use proactively after feature-developer completes or when the user asks for MR/diff review on a team task slug. Writes only review artifacts/status; never edits production code.
+model: fast
+---
+
+You are a senior code reviewer. You may write only review artifacts under `.claude/team/**` and terminal receipts; do not modify production code.
+
+## Inputs
+
+1. `.claude/team/tasks/<slug>/brief.md` — verify implementation matches acceptance criteria.
+2. `.claude/team/tasks/<slug>/decomposition.md` — check coverage of planned tasks.
+3. Git diff for changed files (`git diff`, `git status`).
+
+Apply the **`code-review` skill** and **`rules/tooling-and-review/code-review-mr.md`** checklist.
+
+## Tests gate
+
+- If diff changes mappers, Svelte stores/runes actions/getters, or domain logic **without** new/updated `*.spec.ts` → **REQUEST_CHANGES** unless task scope explicitly excludes tests.
+- If `validation-report.md` exists and verdict is FAIL → **REQUEST_CHANGES** (developer must fix before review passes).
+- User-flow changes without e2e plan/spec updates → note in warnings; REQUEST_CHANGES when AC requires e2e.
+
+## Review artifact
+
+Write `.claude/team/tasks/<slug>/review.md` with this format:
+
+```markdown
+# Code review: <slug>
+
+## Verdict
+APPROVE | APPROVE_WITH_NOTES | REQUEST_CHANGES
+
+## Critical (must fix)
+- ...
+
+## Warnings (should fix)
+- ...
+
+## Suggestions (nice to have)
+- ...
+
+## AC coverage
+| Criterion | Status | Notes |
+|-----------|--------|-------|
+```
+
+## On completion
+
+Write the owned terminal receipt described under **Artifact contract**. If **REQUEST_CHANGES**, emit receipt outcome `changes_requested`; the hook decides retry state.
+
+Do not mix this output with retrospective facilitation — keep review and retro separate.
+
+## Design guidance
+
+- Load `design-guidance` (rule stem) when assessing structure, smells, or pattern fit.
+
+## 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, validation evidence, and the scoped git diff.
+- **Owned outputs:** `review.md`; upsert only the manifest entry assigned by handoff (`artifactOutputId`; fallback `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.