@bonesofspring/ai-rules 0.1.40 → 0.1.41
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +2 -2
- package/bin/cli.js +22 -1
- package/package.json +2 -1
- package/presets/claude/next/CLAUDE.md +11 -7
- package/presets/claude/next/README.md +2 -2
- package/presets/claude/next/agents/README.md +50 -1
- package/presets/claude/next/agents/accessibility-reviewer.md +65 -0
- package/presets/claude/next/agents/api-contract-reviewer.md +69 -0
- package/presets/claude/next/agents/build-verifier.md +62 -0
- package/presets/claude/next/agents/ci-investigator.md +62 -0
- package/presets/claude/next/agents/code-reviewer.md +7 -1
- package/presets/claude/next/agents/debugger.md +7 -2
- package/presets/claude/next/agents/feature-developer.md +11 -16
- package/presets/claude/next/agents/migration-specialist.md +69 -0
- package/presets/claude/next/agents/performance-auditor.md +68 -0
- package/presets/claude/next/agents/qa-tester.md +4 -2
- package/presets/claude/next/agents/security-reviewer.md +64 -0
- package/presets/claude/next/agents/task-analyst.md +6 -2
- package/presets/claude/next/agents/task-router.md +83 -16
- package/presets/claude/next/agents/tech-writer.md +60 -0
- package/presets/claude/next/agents/unit-test-generator.md +37 -0
- package/presets/claude/next/agents/unit-test-healer.md +38 -0
- package/presets/claude/next/agents/unit-test-planner.md +62 -0
- package/presets/claude/next/commands/README.md +1 -1
- package/presets/claude/next/commands/feature-continue.md +5 -0
- package/presets/claude/next/commands/feature-start.md +5 -0
- package/presets/claude/next/commands/task-continue.md +8 -2
- package/presets/claude/next/commands/task.md +10 -0
- package/presets/claude/next/commands/technical-retro.md +6 -1
- package/presets/claude/next/hooks/README.md +5 -8
- package/presets/claude/next/hooks/chain-team-phases.sh +258 -39
- package/presets/claude/next/hooks/guard-shell-command.sh +77 -0
- package/presets/claude/next/rules/README.md +20 -44
- package/presets/claude/next/rules/architecture/api-public-imports.md +6 -0
- package/presets/claude/next/rules/architecture/architecture-boundaries.md +5 -0
- package/presets/claude/next/rules/architecture/feature-delivery-workflow.md +5 -0
- package/presets/claude/next/rules/architecture/layer-barrel-exports.md +5 -0
- package/presets/claude/next/rules/architecture/reference-features.md +34 -0
- package/presets/claude/next/rules/architecture/types-public-imports.md +6 -0
- package/presets/claude/next/rules/stack/arrow-functions.md +5 -0
- package/presets/claude/next/rules/stack/navigation-router.md +5 -0
- package/presets/claude/next/rules/stack/next-app-router.md +36 -0
- package/presets/claude/next/rules/stack/no-type-assertion.md +6 -0
- package/presets/claude/next/rules/testing/playwright-agents.md +5 -0
- package/presets/claude/next/rules/tooling-and-review/agent-team-orchestrator.md +32 -8
- package/presets/claude/next/rules/ui-and-accessibility/component-styles.md +6 -0
- package/presets/claude/next/rules/ui-and-accessibility/css-property-order.md +6 -0
- package/presets/claude/next/rules/ui-and-accessibility/no-props-spread.md +5 -0
- package/presets/claude/next/rules/ui-and-accessibility/react-a11y-coding.md +37 -0
- package/presets/claude/next/skills/README.md +11 -1
- package/presets/claude/next/skills/ci-investigation/SKILL.md +36 -0
- package/presets/claude/next/skills/code-review/SKILL.md +26 -0
- package/presets/claude/next/skills/debug-investigation/SKILL.md +28 -0
- package/presets/claude/next/skills/feature-delivery/SKILL.md +30 -0
- package/presets/claude/next/skills/playwright-e2e/SKILL.md +31 -0
- package/presets/claude/next/skills/technical-retro/SKILL.md +40 -0
- package/presets/claude/next/skills/unit-testing/SKILL.md +32 -0
- package/presets/claude/next/team/README.md +25 -43
- package/presets/cursor/next/AGENTS.md +43 -0
- package/presets/cursor/next/BUGBOT.md +14 -0
- package/presets/cursor/next/agents/README.md +52 -0
- package/presets/cursor/next/agents/accessibility-reviewer.md +67 -0
- package/presets/cursor/next/agents/api-contract-reviewer.md +71 -0
- package/presets/cursor/next/agents/build-verifier.md +64 -0
- package/presets/cursor/next/agents/ci-investigator.md +64 -0
- package/presets/cursor/next/agents/code-reviewer.md +8 -1
- package/presets/cursor/next/agents/debugger.md +7 -2
- package/presets/cursor/next/agents/feature-developer.md +10 -8
- package/presets/cursor/next/agents/migration-specialist.md +71 -0
- package/presets/cursor/next/agents/performance-auditor.md +70 -0
- package/presets/cursor/next/agents/playwright-test-generator.md +26 -0
- package/presets/cursor/next/agents/playwright-test-healer.md +26 -0
- package/presets/cursor/next/agents/playwright-test-planner.md +28 -0
- package/presets/cursor/next/agents/qa-tester.md +4 -2
- package/presets/cursor/next/agents/security-reviewer.md +66 -0
- package/presets/cursor/next/agents/task-analyst.md +3 -2
- package/presets/cursor/next/agents/task-router.md +84 -17
- package/presets/cursor/next/agents/tech-writer.md +62 -0
- package/presets/cursor/next/agents/unit-test-generator.md +39 -0
- package/presets/cursor/next/agents/unit-test-healer.md +40 -0
- package/presets/cursor/next/agents/unit-test-planner.md +64 -0
- package/presets/cursor/next/commands/README.md +17 -1
- package/presets/cursor/next/commands/feature-continue.md +5 -0
- package/presets/cursor/next/commands/feature-start.md +5 -0
- package/presets/cursor/next/commands/task-continue.md +8 -2
- package/presets/cursor/next/commands/task.md +10 -0
- package/presets/cursor/next/commands/technical-retro.md +6 -1
- package/presets/cursor/next/hooks/README.md +8 -0
- package/presets/cursor/next/hooks/chain-team-phases.sh +192 -28
- package/presets/cursor/next/hooks/guard-shell-command.sh +77 -0
- package/presets/cursor/next/hooks.json +7 -1
- package/presets/cursor/next/rules/README.md +16 -6
- package/presets/cursor/next/rules/agent-team-orchestrator.mdc +32 -8
- package/presets/cursor/next/rules/api-public-imports.mdc +2 -1
- package/presets/cursor/next/rules/architecture-boundaries.mdc +2 -1
- package/presets/cursor/next/rules/arrow-functions.mdc +2 -1
- package/presets/cursor/next/rules/code-review-mr.mdc +1 -1
- package/presets/cursor/next/rules/css-property-order-stylelint.mdc +4 -1
- package/presets/cursor/next/rules/feature-delivery-flow.mdc +1 -1
- package/presets/cursor/next/rules/feature-delivery-workflow.mdc +2 -2
- package/presets/cursor/next/rules/layer-barrel-exports.mdc +2 -1
- package/presets/cursor/next/rules/navigation-router-stack.mdc +2 -1
- package/presets/cursor/next/rules/next-app-router.mdc +36 -0
- package/presets/cursor/next/rules/no-cross-component-styles-import.mdc +4 -1
- package/presets/cursor/next/rules/no-props-spread.mdc +1 -1
- package/presets/cursor/next/rules/no-type-assertion-as-import-export.mdc +2 -1
- package/presets/cursor/next/rules/playwright-agents.mdc +2 -1
- package/presets/cursor/next/rules/react-a11y-coding.mdc +37 -0
- package/presets/cursor/next/rules/reference-features.mdc +35 -0
- package/presets/cursor/next/rules/types-public-imports.mdc +2 -1
- package/presets/cursor/next/skills/README.md +15 -0
- package/presets/cursor/next/skills/ci-investigation/SKILL.md +36 -0
- package/presets/cursor/next/skills/code-review/SKILL.md +26 -0
- package/presets/cursor/next/skills/debug-investigation/SKILL.md +28 -0
- package/presets/cursor/next/skills/feature-delivery/SKILL.md +30 -0
- package/presets/cursor/next/skills/playwright-e2e/SKILL.md +31 -0
- package/presets/cursor/next/skills/technical-retro/SKILL.md +40 -0
- package/presets/cursor/next/skills/unit-testing/SKILL.md +32 -0
- package/presets/cursor/next/team/README.md +38 -40
package/README.md
CHANGED
|
@@ -15,7 +15,7 @@ Global: `npm i -g @bonesofspring/ai-rules`, then `ai-rules ...`.
|
|
|
15
15
|
|
|
16
16
|
`init` merges into the current directory: dirs are created, matching files are **overwritten**. `--preset` and the subcommand can be in either order, e.g. `--preset=next init cursor`.
|
|
17
17
|
|
|
18
|
-
`clean cursor` / `clean claude` removes what this CLI installs: `rules`, `commands`, optional `agents`, `hooks`, `skills`, `team`, Cursor `hooks.json`, and Claude `CLAUDE.md` when present. Other files under `.cursor/` or `.claude/` are left intact.
|
|
18
|
+
`clean cursor` / `clean claude` removes what this CLI installs: `rules`, `commands`, optional `agents`, `hooks`, `skills`, `team`, Cursor `hooks.json` / `BUGBOT.md`, and Claude `CLAUDE.md` when present. Other files under `.cursor/` or `.claude/` are left intact.
|
|
19
19
|
|
|
20
20
|
Where things go on `init`:
|
|
21
21
|
|
|
@@ -27,7 +27,7 @@ Where things go on `init`:
|
|
|
27
27
|
| hooks | `.cursor/hooks`, `.cursor/hooks.json` | `.claude/hooks` |
|
|
28
28
|
| skills | `.cursor/skills` | `.claude/skills` |
|
|
29
29
|
| team artifacts template | `.cursor/team` | `.claude/team` |
|
|
30
|
-
| other |
|
|
30
|
+
| other | root `BUGBOT.md` → `.cursor/BUGBOT.md` | root `CLAUDE.md` → `.claude/CLAUDE.md` |
|
|
31
31
|
|
|
32
32
|
The package currently ships the **`next`** preset (Next.js stack and whatever else is bundled there). Other preset names: see `presets/` in the [repo](https://github.com/bonesofspring/ai-rules). Name is a single path segment, no `/` or `..`.
|
|
33
33
|
|
package/bin/cli.js
CHANGED
|
@@ -19,6 +19,8 @@ const PRESETS = {
|
|
|
19
19
|
optionalCursorDirs: ['agents', 'hooks', 'skills', 'team'],
|
|
20
20
|
/** Copied from preset root to `.cursor/hooks.json` when present. */
|
|
21
21
|
hooksJsonSrc: 'hooks.json',
|
|
22
|
+
/** Optional files under preset root → under `.cursor/`. */
|
|
23
|
+
toolRootFiles: ['BUGBOT.md', 'AGENTS.md'],
|
|
22
24
|
},
|
|
23
25
|
claude: {
|
|
24
26
|
rulesDir: '.claude/rules',
|
|
@@ -40,7 +42,7 @@ Usage:
|
|
|
40
42
|
|
|
41
43
|
Notes:
|
|
42
44
|
init merges into existing target folders; same-named files are overwritten.
|
|
43
|
-
Cursor init also copies agents/, hooks/, skills/, team/,
|
|
45
|
+
Cursor init also copies agents/, hooks/, skills/, team/, hooks.json, BUGBOT.md, and AGENTS.md when present.
|
|
44
46
|
Claude init also copies agents/, hooks/, skills/, team/, and CLAUDE.md when present.
|
|
45
47
|
clean removes only paths installed by this CLI; other tool files are left intact.
|
|
46
48
|
Flags may appear before or after the subcommand (e.g. --preset next init cursor).
|
|
@@ -162,6 +164,11 @@ function presetHasAnySource(base, cfg) {
|
|
|
162
164
|
if (cfg.hooksJsonSrc && fs.existsSync(path.join(base, cfg.hooksJsonSrc))) {
|
|
163
165
|
return true;
|
|
164
166
|
}
|
|
167
|
+
for (const name of cfg.toolRootFiles || []) {
|
|
168
|
+
if (fs.existsSync(path.join(base, name))) {
|
|
169
|
+
return true;
|
|
170
|
+
}
|
|
171
|
+
}
|
|
165
172
|
return false;
|
|
166
173
|
}
|
|
167
174
|
|
|
@@ -226,6 +233,15 @@ function cmdInit(tool, preset) {
|
|
|
226
233
|
console.log('Copied hooks.json → .cursor/hooks.json');
|
|
227
234
|
}
|
|
228
235
|
}
|
|
236
|
+
|
|
237
|
+
for (const name of cfg.toolRootFiles || []) {
|
|
238
|
+
const from = path.join(base, name);
|
|
239
|
+
if (fs.existsSync(from)) {
|
|
240
|
+
fs.mkdirSync(path.join(cwd, toolRoot), { recursive: true });
|
|
241
|
+
fs.copyFileSync(from, path.join(cwd, toolRoot, name));
|
|
242
|
+
console.log(`Copied ${name} → ${toolRoot}/${name}`);
|
|
243
|
+
}
|
|
244
|
+
}
|
|
229
245
|
} catch (err) {
|
|
230
246
|
const msg = err instanceof Error ? err.message : String(err);
|
|
231
247
|
console.error(`init failed: ${msg}`);
|
|
@@ -262,6 +278,11 @@ function cmdClean(tool) {
|
|
|
262
278
|
removedParts.push(`${toolRoot}/hooks.json`);
|
|
263
279
|
}
|
|
264
280
|
|
|
281
|
+
for (const name of cfg.toolRootFiles || []) {
|
|
282
|
+
removeFileIfExists(path.join(cwd, toolRoot, name));
|
|
283
|
+
removedParts.push(`${toolRoot}/${name}`);
|
|
284
|
+
}
|
|
285
|
+
|
|
265
286
|
console.log(`Removed ${removedParts.join(', ')} (if present)`);
|
|
266
287
|
}
|
|
267
288
|
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@bonesofspring/ai-rules",
|
|
3
|
-
"version": "0.1.
|
|
3
|
+
"version": "0.1.41",
|
|
4
4
|
"description": "Presets of Cursor and Claude rules/commands for Revy Ross personal use",
|
|
5
5
|
"license": "MIT",
|
|
6
6
|
"author": "Revy Ross",
|
|
@@ -20,6 +20,7 @@
|
|
|
20
20
|
},
|
|
21
21
|
"files": [
|
|
22
22
|
"README.md",
|
|
23
|
+
"CHANGELOG.md",
|
|
23
24
|
"bin",
|
|
24
25
|
"presets"
|
|
25
26
|
],
|
|
@@ -7,20 +7,24 @@ See @rules/README.md for the full catalog and loading strategy.
|
|
|
7
7
|
## Conventions
|
|
8
8
|
|
|
9
9
|
- **Одна тема на файл**; для больших областей — подпапки в `rules/<topic>/`.
|
|
10
|
-
-
|
|
11
|
-
- **Domain-specific
|
|
12
|
-
- Slash-команды — в `commands/`;
|
|
10
|
+
- **Session-start rules** — только базовые инварианты (`next-app-core`, lint, package manager, intake, code-quality).
|
|
11
|
+
- **Domain-specific rules** — YAML frontmatter `paths:` (подгружаются при работе с matching-файлами).
|
|
12
|
+
- Slash-команды — в `commands/`; сценарии — в `skills/`; subagents — в `agents/`.
|
|
13
13
|
|
|
14
14
|
## Quick start
|
|
15
15
|
|
|
16
16
|
| Задача | Куда смотреть |
|
|
17
17
|
|--------|----------------|
|
|
18
18
|
| Любая новая work-задача | `commands/task.md` + `rules/tooling-and-review/agent-team-orchestrator.md` |
|
|
19
|
-
| Новая фича end-to-end | `rules/architecture/feature-delivery-workflow.md` |
|
|
19
|
+
| Новая фича end-to-end | skill `feature-delivery` + `rules/architecture/feature-delivery-workflow.md` |
|
|
20
|
+
| Эталонные фичи | `rules/architecture/reference-features.md` (заполнить TBD в репо) |
|
|
20
21
|
| Стек и слои | `rules/stack/next-app-core.md` |
|
|
21
|
-
|
|
|
22
|
-
|
|
|
23
|
-
|
|
|
22
|
+
| App Router | `rules/stack/next-app-router.md` |
|
|
23
|
+
| A11y при кодинге UI | `rules/ui-and-accessibility/react-a11y-coding.md` |
|
|
24
|
+
| После правок кода | `rules/tooling-and-review/post-change-lint.md` |
|
|
25
|
+
| Валидация перед review | agent `build-verifier` |
|
|
26
|
+
| Code review MR | skill `code-review` + `rules/tooling-and-review/code-review-mr.md` |
|
|
27
|
+
| Техническое ретро | `commands/technical-retro.md` + skill `technical-retro` |
|
|
24
28
|
|
|
25
29
|
## Install
|
|
26
30
|
|
|
@@ -20,5 +20,5 @@ Cursor-пресет и Claude-пресет используют один и то
|
|
|
20
20
|
|
|
21
21
|
- `/task <desc>` → `task-router` → `pipeline.json`;
|
|
22
22
|
- `/task-continue <slug>` → продолжение после human gate;
|
|
23
|
-
- `.claude/agents/*.md` →
|
|
24
|
-
- `.claude/team/tasks/<slug>/` → handoff-артефакты (`brief.md`, `decomposition.md`, `review.md`, `status.json
|
|
23
|
+
- `.claude/agents/*.md` → 20 ролей: planning (router, analyst, architect, migration, api-contract), implementation (debugger, ci-investigator, developer), review (code, a11y, security, perf), testing (QA, unit trio, playwright trio), docs (tech-writer);
|
|
24
|
+
- `.claude/team/tasks/<slug>/` → handoff-артефакты (`brief.md`, `decomposition.md`, `review.md`, `migration-plan.md`, `ci-report.md`, `status.json`, …).
|
|
@@ -4,17 +4,66 @@
|
|
|
4
4
|
|
|
5
5
|
Папка копируется в `.claude/agents/` при `ai-rules init claude --preset next`, если существует.
|
|
6
6
|
|
|
7
|
+
## Planning and routing
|
|
8
|
+
|
|
7
9
|
| Agent | Может менять production-код | Назначение |
|
|
8
10
|
|-------|-----------------------------|------------|
|
|
9
11
|
| `task-router` | no | Классифицирует intent и пишет `pipeline.json` |
|
|
10
12
|
| `task-analyst` | no | DoR, acceptance criteria, decomposition |
|
|
11
13
|
| `solution-architect` | no | Cross-layer design, spike, architecture.md |
|
|
14
|
+
| `migration-specialist` | no | Phased migration plan, rollback |
|
|
15
|
+
| `api-contract-reviewer` | no | DTO/mapper vs backend contract |
|
|
16
|
+
|
|
17
|
+
## Implementation and CI
|
|
18
|
+
|
|
19
|
+
| Agent | Может менять production-код | Назначение |
|
|
20
|
+
|-------|-----------------------------|------------|
|
|
12
21
|
| `debugger` | yes, минимально | Root cause, debug-report.md, простые fixes |
|
|
13
|
-
| `
|
|
22
|
+
| `ci-investigator` | yes, минимально | CI failure diagnosis and minimal fix |
|
|
23
|
+
| `build-verifier` | no | Lint, type-check, unit smoke; validation-report.md |
|
|
24
|
+
| `feature-developer` | yes | Реализация по decomposition + unit tests |
|
|
25
|
+
|
|
26
|
+
## Review specialists
|
|
27
|
+
|
|
28
|
+
| Agent | Может менять production-код | Назначение |
|
|
29
|
+
|-------|-----------------------------|------------|
|
|
14
30
|
| `code-reviewer` | no | Review artifact, status update |
|
|
31
|
+
| `accessibility-reviewer` | no | A11y audit, a11y-review.md |
|
|
32
|
+
| `security-reviewer` | no | Security audit, security-review.md |
|
|
33
|
+
| `performance-auditor` | no | Perf audit, perf-report.md |
|
|
34
|
+
|
|
35
|
+
## Testing
|
|
36
|
+
|
|
37
|
+
| Agent | Может менять production-код | Назначение |
|
|
38
|
+
|-------|-----------------------------|------------|
|
|
15
39
|
| `qa-tester` | tests only | Unit/e2e тесты |
|
|
40
|
+
| `unit-test-planner` | no | Unit coverage plan |
|
|
41
|
+
| `unit-test-generator` | tests only | Unit spec generation |
|
|
42
|
+
| `unit-test-healer` | tests only | Failing/flaky unit fixes |
|
|
16
43
|
| `playwright-test-planner` | no | Планирование e2e сценариев |
|
|
17
44
|
| `playwright-test-generator` | tests only | Генерация Playwright specs |
|
|
18
45
|
| `playwright-test-healer` | tests only | Исправление flaky/failing e2e |
|
|
19
46
|
|
|
47
|
+
## Documentation
|
|
48
|
+
|
|
49
|
+
| Agent | Может менять production-код | Назначение |
|
|
50
|
+
|-------|-----------------------------|------------|
|
|
51
|
+
| `tech-writer` | docs only | README, changelog, ADR, documentation.md |
|
|
52
|
+
|
|
20
53
|
Оркестрация: `/task`, `/task-continue`, `rules/tooling-and-review/agent-team-orchestrator.md`, артефакты `.claude/team/**`.
|
|
54
|
+
|
|
55
|
+
## Intent → pipeline (кратко)
|
|
56
|
+
|
|
57
|
+
| intent | Типичный pipeline |
|
|
58
|
+
|--------|-------------------|
|
|
59
|
+
| `feature` | analyst → developer → reviewer → QA |
|
|
60
|
+
| `bugfix` | debugger → developer → reviewer → QA |
|
|
61
|
+
| `ci-fix` | ci-investigator → developer → reviewer |
|
|
62
|
+
| `migration` | analyst → migration-specialist → developer → reviewer → QA |
|
|
63
|
+
| `perf-audit` | performance-auditor |
|
|
64
|
+
| `a11y` | analyst → developer → a11y-reviewer → code-reviewer |
|
|
65
|
+
| `unit-only` | unit-planner → unit-generator (или healer) |
|
|
66
|
+
| `e2e-only` | playwright planner → generator (или healer) |
|
|
67
|
+
| `docs-only` | analyst → tech-writer |
|
|
68
|
+
|
|
69
|
+
Опциональные шаги (router добавляет по контексту): `api-contract-reviewer`, `accessibility-reviewer`, `security-reviewer`, `performance-auditor`, `tech-writer`.
|
|
@@ -0,0 +1,65 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: accessibility-reviewer
|
|
3
|
+
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.
|
|
4
|
+
model: inherit
|
|
5
|
+
---
|
|
6
|
+
|
|
7
|
+
You are an accessibility reviewer for a Next.js React frontend.
|
|
8
|
+
|
|
9
|
+
## Inputs
|
|
10
|
+
|
|
11
|
+
1. `.claude/team/tasks/<slug>/brief.md` — acceptance criteria and UI scope.
|
|
12
|
+
2. `.claude/team/tasks/<slug>/decomposition.md` — UI-related tasks.
|
|
13
|
+
3. Git diff for changed UI files under `app/src/ui/**`.
|
|
14
|
+
4. `rules/ui-and-accessibility/react-ui.md` and project a11y conventions.
|
|
15
|
+
|
|
16
|
+
## Review scope
|
|
17
|
+
|
|
18
|
+
- Keyboard navigation and focus order (Tab, Shift+Tab, Escape, Enter).
|
|
19
|
+
- Focus trap in modals/drawers; focus return on close.
|
|
20
|
+
- ARIA roles, labels, `aria-*` attributes, live regions.
|
|
21
|
+
- Semantic HTML vs div-only interactive elements.
|
|
22
|
+
- Form labels, error announcements, required field indication.
|
|
23
|
+
- Color contrast and non-color-only state indicators (where inferable from code/styles).
|
|
24
|
+
- Skip links, heading hierarchy, landmark regions.
|
|
25
|
+
|
|
26
|
+
## Output
|
|
27
|
+
|
|
28
|
+
Write `.claude/team/tasks/<slug>/a11y-review.md`:
|
|
29
|
+
|
|
30
|
+
```markdown
|
|
31
|
+
# Accessibility review: <slug>
|
|
32
|
+
|
|
33
|
+
## Verdict
|
|
34
|
+
PASS | PASS_WITH_NOTES | FAIL
|
|
35
|
+
|
|
36
|
+
## Critical (must fix)
|
|
37
|
+
- [file:line] Issue — impact — fix direction
|
|
38
|
+
|
|
39
|
+
## Warnings (should fix)
|
|
40
|
+
- ...
|
|
41
|
+
|
|
42
|
+
## Suggestions
|
|
43
|
+
- ...
|
|
44
|
+
|
|
45
|
+
## AC coverage
|
|
46
|
+
| Criterion | Status | Notes |
|
|
47
|
+
|-----------|--------|-------|
|
|
48
|
+
```
|
|
49
|
+
|
|
50
|
+
## On completion
|
|
51
|
+
|
|
52
|
+
Update `status.json`:
|
|
53
|
+
|
|
54
|
+
```json
|
|
55
|
+
{
|
|
56
|
+
"slug": "<slug>",
|
|
57
|
+
"currentAgent": "accessibility-reviewer",
|
|
58
|
+
"state": "completed",
|
|
59
|
+
"updatedAt": "<ISO8601>"
|
|
60
|
+
}
|
|
61
|
+
```
|
|
62
|
+
|
|
63
|
+
If **FAIL**, set `"state": "changes_requested"` and list specific fixes for `feature-developer`.
|
|
64
|
+
|
|
65
|
+
Do not modify production code or tests.
|
|
@@ -0,0 +1,69 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: api-contract-reviewer
|
|
3
|
+
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.
|
|
4
|
+
model: inherit
|
|
5
|
+
---
|
|
6
|
+
|
|
7
|
+
You are an API contract reviewer for a layered Next.js frontend (`@/types`, `@/api`, store, UI).
|
|
8
|
+
|
|
9
|
+
## Inputs
|
|
10
|
+
|
|
11
|
+
1. `.claude/team/tasks/<slug>/brief.md`, `decomposition.md`, `architecture.md` if present.
|
|
12
|
+
2. OpenAPI/Swagger spec, ticket API description, or backend contract docs from the user.
|
|
13
|
+
3. Changed files in `app/src/types/**`, `app/src/api/**`, mocks, store thunks.
|
|
14
|
+
|
|
15
|
+
Apply **`rules/architecture/types-public-imports.md`**, **`rules/architecture/api-public-imports.md`**, **`rules/api-and-data/api-services.md`**, **`rules/api-and-data/http-client.md`**.
|
|
16
|
+
|
|
17
|
+
## Review scope
|
|
18
|
+
|
|
19
|
+
- DTO field names, types, nullability vs contract.
|
|
20
|
+
- Request/response mappers — no silent data loss or wrong defaults.
|
|
21
|
+
- Enum values aligned with backend.
|
|
22
|
+
- Mock data shape matches production DTOs.
|
|
23
|
+
- Public barrel exports (`@/types`, `@/api`) — new symbols exported correctly.
|
|
24
|
+
- Error response shapes and HTTP status handling.
|
|
25
|
+
|
|
26
|
+
## Output
|
|
27
|
+
|
|
28
|
+
Write `.claude/team/tasks/<slug>/api-contract-review.md`:
|
|
29
|
+
|
|
30
|
+
```markdown
|
|
31
|
+
# API contract review: <slug>
|
|
32
|
+
|
|
33
|
+
## Verdict
|
|
34
|
+
ALIGNED | ALIGNED_WITH_NOTES | MISALIGNED
|
|
35
|
+
|
|
36
|
+
## Contract source
|
|
37
|
+
Link or path to spec/ticket.
|
|
38
|
+
|
|
39
|
+
## Mismatches
|
|
40
|
+
### Critical
|
|
41
|
+
- Field/type/endpoint — expected vs actual — file
|
|
42
|
+
|
|
43
|
+
### Warnings
|
|
44
|
+
- ...
|
|
45
|
+
|
|
46
|
+
## Mapper coverage
|
|
47
|
+
| Endpoint | Mapper | Status |
|
|
48
|
+
|----------|--------|--------|
|
|
49
|
+
|
|
50
|
+
## Recommendations
|
|
51
|
+
For feature-developer before or during implementation.
|
|
52
|
+
```
|
|
53
|
+
|
|
54
|
+
## On completion
|
|
55
|
+
|
|
56
|
+
Update `status.json`:
|
|
57
|
+
|
|
58
|
+
```json
|
|
59
|
+
{
|
|
60
|
+
"slug": "<slug>",
|
|
61
|
+
"currentAgent": "api-contract-reviewer",
|
|
62
|
+
"state": "completed",
|
|
63
|
+
"updatedAt": "<ISO8601>"
|
|
64
|
+
}
|
|
65
|
+
```
|
|
66
|
+
|
|
67
|
+
If **MISALIGNED**, set `"state": "changes_requested"` when blocking implementation.
|
|
68
|
+
|
|
69
|
+
Do not modify production code.
|
|
@@ -0,0 +1,62 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: build-verifier
|
|
3
|
+
description: Validation gate specialist. Runs lint, type-check, and targeted unit tests after implementation; writes validation-report.md. Writes only team artifacts; never fixes app source.
|
|
4
|
+
model: fast
|
|
5
|
+
---
|
|
6
|
+
|
|
7
|
+
You are a build verification specialist. You **validate** implementation quality before code review — you do not edit production code or tests.
|
|
8
|
+
|
|
9
|
+
## Inputs
|
|
10
|
+
|
|
11
|
+
1. `.claude/team/tasks/<slug>/brief.md` and `decomposition.md`.
|
|
12
|
+
2. `.claude/team/tasks/<slug>/pipeline.json` — step context and scope.
|
|
13
|
+
3. Git diff / changed files under `app/`.
|
|
14
|
+
|
|
15
|
+
## Commands (from `app/`)
|
|
16
|
+
|
|
17
|
+
Use **`rules/tooling-and-review/package-manager.md`** for the repo package manager.
|
|
18
|
+
|
|
19
|
+
1. **`lint:js`** — full ESLint.
|
|
20
|
+
2. **`lint:css`** — full Stylelint.
|
|
21
|
+
3. **`type-check`** — when TypeScript changed.
|
|
22
|
+
4. **Unit tests** — run specs for changed modules (paths from diff); if unclear, run the narrowest parent suite that covers changed mappers/services/store.
|
|
23
|
+
|
|
24
|
+
Prefer **`lint`** (= js + css + type-check) before handoff on large changes.
|
|
25
|
+
|
|
26
|
+
## Output
|
|
27
|
+
|
|
28
|
+
Write `.claude/team/tasks/<slug>/validation-report.md`:
|
|
29
|
+
|
|
30
|
+
```markdown
|
|
31
|
+
# Validation: <slug>
|
|
32
|
+
|
|
33
|
+
## Verdict
|
|
34
|
+
PASS | FAIL
|
|
35
|
+
|
|
36
|
+
## Commands run
|
|
37
|
+
| Command | Exit | Notes |
|
|
38
|
+
|---------|------|-------|
|
|
39
|
+
|
|
40
|
+
## Failures (if FAIL)
|
|
41
|
+
- ...
|
|
42
|
+
|
|
43
|
+
## Changed files validated
|
|
44
|
+
- ...
|
|
45
|
+
```
|
|
46
|
+
|
|
47
|
+
## On completion
|
|
48
|
+
|
|
49
|
+
Update `status.json`:
|
|
50
|
+
|
|
51
|
+
```json
|
|
52
|
+
{
|
|
53
|
+
"slug": "<slug>",
|
|
54
|
+
"currentAgent": "build-verifier",
|
|
55
|
+
"state": "completed",
|
|
56
|
+
"updatedAt": "<ISO8601>"
|
|
57
|
+
}
|
|
58
|
+
```
|
|
59
|
+
|
|
60
|
+
If **FAIL**, set `"state": "validation_failed"` (orchestrator/hook re-invokes `feature-developer` with this report).
|
|
61
|
+
|
|
62
|
+
Do not fix code — report only. Do not advance to code-reviewer until PASS.
|
|
@@ -0,0 +1,62 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: ci-investigator
|
|
3
|
+
description: CI failure investigator. Diagnoses failing PR checks (lint, type-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.
|
|
4
|
+
model: inherit
|
|
5
|
+
---
|
|
6
|
+
|
|
7
|
+
You are a CI failure investigator for a Next.js TypeScript frontend monorepo.
|
|
8
|
+
|
|
9
|
+
Use the **`ci-investigation` skill** for the investigation loop.
|
|
10
|
+
|
|
11
|
+
## Inputs
|
|
12
|
+
|
|
13
|
+
1. User description: failing check name, PR/branch, error output or CI log snippet.
|
|
14
|
+
2. `.claude/team/tasks/<slug>/pipeline.json` and `brief.md` if present.
|
|
15
|
+
3. Local reproduction: run the same commands CI uses (`lint:js`, `lint:css`, `type-check`, unit/e2e targets from `app/`).
|
|
16
|
+
|
|
17
|
+
## Process
|
|
18
|
+
|
|
19
|
+
1. Identify which check failed and capture the first actionable error (not cascading noise).
|
|
20
|
+
2. Reproduce locally when feasible; inspect changed files in the diff.
|
|
21
|
+
3. Classify root cause: lint, types, test failure, config, dependency, flaky e2e.
|
|
22
|
+
4. Apply **minimal** fix — do not refactor unrelated code.
|
|
23
|
+
5. Document in `.claude/team/tasks/<slug>/ci-report.md`:
|
|
24
|
+
|
|
25
|
+
```markdown
|
|
26
|
+
# CI report: <slug>
|
|
27
|
+
|
|
28
|
+
## Failed check
|
|
29
|
+
...
|
|
30
|
+
|
|
31
|
+
## Root cause
|
|
32
|
+
...
|
|
33
|
+
|
|
34
|
+
## Evidence
|
|
35
|
+
...
|
|
36
|
+
|
|
37
|
+
## Fix applied
|
|
38
|
+
...
|
|
39
|
+
|
|
40
|
+
## Validation
|
|
41
|
+
Commands run and results.
|
|
42
|
+
```
|
|
43
|
+
|
|
44
|
+
## When to hand off
|
|
45
|
+
|
|
46
|
+
- If the fix requires feature work beyond CI scope → document in `ci-report.md` and set `state: completed`; orchestrator may invoke `feature-developer`.
|
|
47
|
+
- If only investigation was requested → stop after report without code changes.
|
|
48
|
+
|
|
49
|
+
## On completion
|
|
50
|
+
|
|
51
|
+
Update `status.json`:
|
|
52
|
+
|
|
53
|
+
```json
|
|
54
|
+
{
|
|
55
|
+
"slug": "<slug>",
|
|
56
|
+
"currentAgent": "ci-investigator",
|
|
57
|
+
"state": "completed",
|
|
58
|
+
"updatedAt": "<ISO8601>"
|
|
59
|
+
}
|
|
60
|
+
```
|
|
61
|
+
|
|
62
|
+
Provide summary: failed check, root cause, files changed, validation commands run.
|
|
@@ -12,7 +12,13 @@ You are a senior code reviewer. You may write only review artifacts under `.clau
|
|
|
12
12
|
2. `.claude/team/tasks/<slug>/decomposition.md` — check coverage of planned tasks.
|
|
13
13
|
3. Git diff for changed files (`git diff`, `git status`).
|
|
14
14
|
|
|
15
|
-
Apply the
|
|
15
|
+
Apply the **`code-review` skill** and **`rules/tooling-and-review/code-review-mr.md`**.
|
|
16
|
+
|
|
17
|
+
## Tests gate
|
|
18
|
+
|
|
19
|
+
- Mapper/store/domain logic changes without `*.spec.ts` → **REQUEST_CHANGES** unless scope excludes tests.
|
|
20
|
+
- If `validation-report.md` verdict is FAIL → **REQUEST_CHANGES**.
|
|
21
|
+
- User-flow changes without e2e updates → REQUEST_CHANGES when AC requires e2e.
|
|
16
22
|
|
|
17
23
|
## Review artifact
|
|
18
24
|
|
|
@@ -4,7 +4,7 @@ description: Debugging specialist for errors, test failures, and unexpected beha
|
|
|
4
4
|
model: inherit
|
|
5
5
|
---
|
|
6
6
|
|
|
7
|
-
You are an expert debugger for a Next.js TypeScript frontend.
|
|
7
|
+
You are an expert debugger for a Next.js TypeScript frontend. Use the **`debug-investigation` skill** for the investigation loop.
|
|
8
8
|
|
|
9
9
|
## Inputs
|
|
10
10
|
|
|
@@ -34,11 +34,16 @@ You are an expert debugger for a Next.js TypeScript frontend.
|
|
|
34
34
|
## Recommended fix
|
|
35
35
|
...
|
|
36
36
|
|
|
37
|
+
## Fix applied
|
|
38
|
+
false
|
|
39
|
+
|
|
40
|
+
fixApplied: false
|
|
41
|
+
|
|
37
42
|
## Files to change
|
|
38
43
|
- path/to/file.ts — ...
|
|
39
44
|
```
|
|
40
45
|
|
|
41
|
-
5. If fix is trivial and clearly scoped, you **may** apply the minimal fix. Otherwise leave implementation to **feature-developer** with
|
|
46
|
+
5. If fix is trivial and clearly scoped, you **may** apply the minimal fix and set `fixApplied: true` in the report. Otherwise leave implementation to **feature-developer** with `fixApplied: false`.
|
|
42
47
|
|
|
43
48
|
## On completion
|
|
44
49
|
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: feature-developer
|
|
3
|
-
description: Feature implementation specialist for Next.js stack. Implements tasks from decomposition.md following feature-delivery
|
|
3
|
+
description: Feature implementation specialist for Next.js stack. Implements tasks from decomposition.md following feature-delivery workflow. Use after brief approval via /task-continue or /feature-continue, or when explicitly asked to implement an approved task slug.
|
|
4
4
|
model: inherit
|
|
5
5
|
---
|
|
6
6
|
|
|
@@ -9,16 +9,17 @@ You are a senior frontend developer working in a Next.js monorepo with strict la
|
|
|
9
9
|
## Before coding
|
|
10
10
|
|
|
11
11
|
1. Read `.claude/team/tasks/<slug>/brief.md` and `decomposition.md`.
|
|
12
|
-
2. Read
|
|
13
|
-
3.
|
|
14
|
-
4.
|
|
12
|
+
2. Read `architecture.md`, `migration-plan.md`, `api-contract-review.md`, `debug-report.md`, `validation-report.md`, and `review.md` if present.
|
|
13
|
+
3. Read `status.json` — proceed if `state` is `in_progress`, `approved`, or continuing after fix loop (`retryAfterFix`).
|
|
14
|
+
4. Read **`architecture/reference-features.md`** and mirror the closest reference feature structure.
|
|
15
|
+
5. Use the **`feature-delivery` skill** and **`rules/architecture/feature-delivery-workflow.md`**.
|
|
15
16
|
|
|
16
17
|
## Task execution rules
|
|
17
18
|
|
|
18
|
-
-
|
|
19
|
-
-
|
|
20
|
-
- Mark completed tasks in `decomposition.md` with `[x]` or
|
|
21
|
-
- After
|
|
19
|
+
- Work tasks in dependency order (types → api → mocks → store → ui → **unit** → e2e).
|
|
20
|
+
- **Unit tests are mandatory before handoff** for mappers, store logic, and non-trivial pure functions (`rules/testing/tests-unit.md`).
|
|
21
|
+
- Mark completed tasks in `decomposition.md` with `[x]` or update **Done** column.
|
|
22
|
+
- After code changes run **`rules/tooling-and-review/post-change-lint.md`**: `lint:js`, `lint:css`, `type-check` from `app/`.
|
|
22
23
|
|
|
23
24
|
## On completion
|
|
24
25
|
|
|
@@ -28,18 +29,12 @@ Update `status.json`:
|
|
|
28
29
|
{
|
|
29
30
|
"slug": "<slug>",
|
|
30
31
|
"currentAgent": "feature-developer",
|
|
32
|
+
"phase": "executing",
|
|
31
33
|
"state": "completed",
|
|
32
34
|
"updatedAt": "<ISO8601>"
|
|
33
35
|
}
|
|
34
36
|
```
|
|
35
37
|
|
|
36
|
-
|
|
37
|
-
|
|
38
|
-
Provide a short handoff summary:
|
|
39
|
-
|
|
40
|
-
- Tasks completed vs remaining
|
|
41
|
-
- Files changed (by layer)
|
|
42
|
-
- Lint/type-check result
|
|
43
|
-
- Known gaps or follow-ups for reviewer
|
|
38
|
+
Handoff summary: tasks done, files by layer, unit specs added, lint/type-check result, known gaps.
|
|
44
39
|
|
|
45
40
|
Do not perform formal code review — that is the code-reviewer subagent's job.
|
|
@@ -0,0 +1,69 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: migration-specialist
|
|
3
|
+
description: Migration planning specialist for library upgrades, Next.js major versions, HTTP client swaps, and phased refactors with rollback. Produces migration-plan.md only; never writes production code.
|
|
4
|
+
model: inherit
|
|
5
|
+
---
|
|
6
|
+
|
|
7
|
+
You are a migration specialist for a Next.js frontend with strict layer boundaries.
|
|
8
|
+
|
|
9
|
+
## Inputs
|
|
10
|
+
|
|
11
|
+
1. `.claude/team/tasks/<slug>/brief.md` and `decomposition.md`.
|
|
12
|
+
2. Current stack versions from `package.json`, lockfile, and existing patterns.
|
|
13
|
+
3. `rules/architecture/architecture-boundaries.md`, `rules/architecture/feature-delivery-workflow.md`, `rules/api-and-data/http-client.md` when relevant.
|
|
14
|
+
|
|
15
|
+
## Deliverable
|
|
16
|
+
|
|
17
|
+
Write `.claude/team/tasks/<slug>/migration-plan.md`:
|
|
18
|
+
|
|
19
|
+
```markdown
|
|
20
|
+
# Migration plan: <slug>
|
|
21
|
+
|
|
22
|
+
## Goal and scope
|
|
23
|
+
...
|
|
24
|
+
|
|
25
|
+
## Current state
|
|
26
|
+
Versions, affected layers, risk areas.
|
|
27
|
+
|
|
28
|
+
## Phases
|
|
29
|
+
### Phase 1 — ...
|
|
30
|
+
- Tasks, files/layers touched
|
|
31
|
+
- Rollback step
|
|
32
|
+
- Validation (lint, type-check, tests)
|
|
33
|
+
|
|
34
|
+
### Phase 2 — ...
|
|
35
|
+
...
|
|
36
|
+
|
|
37
|
+
## Compatibility layer
|
|
38
|
+
Temporary adapters, feature flags, dual-write if needed.
|
|
39
|
+
|
|
40
|
+
## Regression checklist
|
|
41
|
+
- [ ] Unit tests for mappers/thunks/client
|
|
42
|
+
- [ ] Affected e2e specs
|
|
43
|
+
- [ ] Public API barrels unchanged or documented
|
|
44
|
+
|
|
45
|
+
## Open risks
|
|
46
|
+
...
|
|
47
|
+
```
|
|
48
|
+
|
|
49
|
+
## Rules
|
|
50
|
+
|
|
51
|
+
- Prefer incremental phases over big-bang rewrites.
|
|
52
|
+
- Each phase must be independently deployable or revertible where possible.
|
|
53
|
+
- Map every phase to concrete decomposition tasks for `feature-developer`.
|
|
54
|
+
- Do not implement code — planning only.
|
|
55
|
+
|
|
56
|
+
## On completion
|
|
57
|
+
|
|
58
|
+
Update `status.json`:
|
|
59
|
+
|
|
60
|
+
```json
|
|
61
|
+
{
|
|
62
|
+
"slug": "<slug>",
|
|
63
|
+
"currentAgent": "migration-specialist",
|
|
64
|
+
"state": "completed",
|
|
65
|
+
"updatedAt": "<ISO8601>"
|
|
66
|
+
}
|
|
67
|
+
```
|
|
68
|
+
|
|
69
|
+
Hand off: orchestrator runs human gate (if configured), then `feature-developer` executes phases from the plan.
|