moicle 2.2.3 → 2.3.1
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 +14 -4
- package/assets/commands/marketing.md +7 -7
- package/assets/skills/docs/sync/SKILL.md +7 -7
- package/assets/skills/docs/write/SKILL.md +8 -8
- package/assets/skills/feature/api/SKILL.md +9 -9
- package/assets/skills/feature/deprecate/SKILL.md +3 -3
- package/assets/skills/feature/new/SKILL.md +10 -10
- package/assets/skills/feature/refactor/SKILL.md +8 -8
- package/assets/skills/feature/track/SKILL.md +157 -0
- package/assets/skills/fix/hotfix/SKILL.md +12 -12
- package/assets/skills/fix/incident/SKILL.md +6 -6
- package/assets/skills/fix/pr-comment/SKILL.md +11 -11
- package/assets/skills/fix/root-cause/SKILL.md +6 -6
- package/assets/skills/marketing/content/SKILL.md +5 -5
- package/assets/skills/marketing/logo/SKILL.md +4 -4
- package/assets/skills/marketing/seo-blog/SKILL.md +5 -5
- package/assets/skills/marketing/video/SKILL.md +3 -3
- package/assets/skills/research/onboarding/SKILL.md +7 -7
- package/assets/skills/research/spike/SKILL.md +10 -10
- package/assets/skills/research/web/SKILL.md +8 -8
- package/assets/skills/review/architect/SKILL.md +12 -12
- package/assets/skills/review/branch/SKILL.md +6 -6
- package/assets/skills/review/pr/SKILL.md +8 -8
- package/assets/skills/review/tdd/SKILL.md +5 -5
- package/bin/cli.js +4 -2
- package/dist/commands/disable.d.ts.map +1 -1
- package/dist/commands/disable.js +44 -59
- package/dist/commands/disable.js.map +1 -1
- package/dist/commands/enable.d.ts.map +1 -1
- package/dist/commands/enable.js +45 -61
- package/dist/commands/enable.js.map +1 -1
- package/dist/commands/install/cursor-editor.d.ts +3 -0
- package/dist/commands/install/cursor-editor.d.ts.map +1 -0
- package/dist/commands/install/cursor-editor.js +114 -0
- package/dist/commands/install/cursor-editor.js.map +1 -0
- package/dist/commands/install/cursor-transform.d.ts +3 -0
- package/dist/commands/install/cursor-transform.d.ts.map +1 -0
- package/dist/commands/install/cursor-transform.js +25 -0
- package/dist/commands/install/cursor-transform.js.map +1 -0
- package/dist/commands/install/index.d.ts.map +1 -1
- package/dist/commands/install/index.js +5 -1
- package/dist/commands/install/index.js.map +1 -1
- package/dist/commands/install/prompts.d.ts +1 -1
- package/dist/commands/install/prompts.d.ts.map +1 -1
- package/dist/commands/install/prompts.js +1 -0
- package/dist/commands/install/prompts.js.map +1 -1
- package/dist/commands/install/skill-editor.d.ts.map +1 -1
- package/dist/commands/install/skill-editor.js +1 -12
- package/dist/commands/install/skill-editor.js.map +1 -1
- package/dist/commands/install/transform.d.ts +1 -0
- package/dist/commands/install/transform.d.ts.map +1 -1
- package/dist/commands/install/transform.js +6 -0
- package/dist/commands/install/transform.js.map +1 -1
- package/dist/commands/install/usage.d.ts.map +1 -1
- package/dist/commands/install/usage.js +21 -9
- package/dist/commands/install/usage.js.map +1 -1
- package/dist/commands/install/write-if-changed.d.ts +4 -0
- package/dist/commands/install/write-if-changed.d.ts.map +1 -0
- package/dist/commands/install/write-if-changed.js +14 -0
- package/dist/commands/install/write-if-changed.js.map +1 -0
- package/dist/commands/list.d.ts.map +1 -1
- package/dist/commands/list.js +41 -3
- package/dist/commands/list.js.map +1 -1
- package/dist/commands/postinstall.d.ts.map +1 -1
- package/dist/commands/postinstall.js +2 -1
- package/dist/commands/postinstall.js.map +1 -1
- package/dist/commands/status.d.ts.map +1 -1
- package/dist/commands/status.js +53 -4
- package/dist/commands/status.js.map +1 -1
- package/dist/commands/uninstall.d.ts.map +1 -1
- package/dist/commands/uninstall.js +91 -4
- package/dist/commands/uninstall.js.map +1 -1
- package/dist/utils/editor-constants.d.ts +5 -0
- package/dist/utils/editor-constants.d.ts.map +1 -0
- package/dist/utils/editor-constants.js +5 -0
- package/dist/utils/editor-constants.js.map +1 -0
- package/dist/utils/editor-items.d.ts +11 -0
- package/dist/utils/editor-items.d.ts.map +1 -0
- package/dist/utils/editor-items.js +57 -0
- package/dist/utils/editor-items.js.map +1 -0
- package/dist/utils/symlink.d.ts +1 -0
- package/dist/utils/symlink.d.ts.map +1 -1
- package/dist/utils/symlink.js +7 -5
- package/dist/utils/symlink.js.map +1 -1
- package/package.json +1 -1
package/README.md
CHANGED
|
@@ -16,7 +16,7 @@ A toolkit to bootstrap and accelerate project development with Claude Code throu
|
|
|
16
16
|
|
|
17
17
|
- **16 AI Agents** - 6 developer agents + 10 utility agents
|
|
18
18
|
- **4 Commands** - Wizards for bootstrap, brainstorm, documentation, and marketing
|
|
19
|
-
- **
|
|
19
|
+
- **22 Skills** - Auto-triggered workflows for the full SDLC (feature, bug, review, release, ops, content)
|
|
20
20
|
- **9 Architecture References** - DDD + Hexagonal + stack-specific patterns
|
|
21
21
|
|
|
22
22
|
|
|
@@ -25,9 +25,11 @@ A toolkit to bootstrap and accelerate project development with Claude Code throu
|
|
|
25
25
|
- [x] Claude
|
|
26
26
|
- [x] Codex CLI
|
|
27
27
|
- [x] Antigravity
|
|
28
|
-
- [
|
|
28
|
+
- [x] Cursor
|
|
29
29
|
- [ ] Windsurf
|
|
30
30
|
|
|
31
|
+
Older MoiCle versions merged agents into `~/.cursor/AGENTS.md`. Re-run `moicle install --target cursor` for native `.cursor/rules/*.mdc` layout; you may delete legacy `AGENTS.md` manually.
|
|
32
|
+
|
|
31
33
|
## Installation
|
|
32
34
|
|
|
33
35
|
```bash
|
|
@@ -46,6 +48,10 @@ moicle install --target codex --global
|
|
|
46
48
|
# Install for Antigravity
|
|
47
49
|
moicle install --target antigravity --global
|
|
48
50
|
|
|
51
|
+
# Install for Cursor (rules, commands, skills, architecture)
|
|
52
|
+
moicle install --target cursor --global
|
|
53
|
+
moicle install --target cursor --project
|
|
54
|
+
|
|
49
55
|
# Choose:
|
|
50
56
|
# 1. Pick Claude Code, Codex CLI, or Antigravity
|
|
51
57
|
# 2. Pick global or project scope
|
|
@@ -60,7 +66,9 @@ moicle install --target antigravity --global
|
|
|
60
66
|
| `moicle install --project` | Install to ./.claude/ (copies) |
|
|
61
67
|
| `moicle install --target codex --global` | Install Codex skills + architecture to ~/.codex/ |
|
|
62
68
|
| `moicle install --target codex --project` | Install Codex skills + architecture to ./.codex/ |
|
|
63
|
-
| `moicle
|
|
69
|
+
| `moicle install --target cursor --global` | Install Cursor rules, commands, skills to ~/.cursor/ |
|
|
70
|
+
| `moicle install --target cursor --project` | Install Cursor assets to ./.cursor/ |
|
|
71
|
+
| `moicle list --target cursor` | List Cursor rules, commands, and skills |
|
|
64
72
|
| `moicle status` | Show enabled/disabled status |
|
|
65
73
|
| `moicle enable <item>` | Enable an agent/command/skill |
|
|
66
74
|
| `moicle disable <item>` | Disable an agent/command/skill |
|
|
@@ -116,7 +124,7 @@ moicle install --target antigravity --global
|
|
|
116
124
|
| `/brainstorm` | Brainstorm ideas with 6 frameworks |
|
|
117
125
|
| `/doc` | Scan project and generate documentation |
|
|
118
126
|
|
|
119
|
-
### Skills (
|
|
127
|
+
### Skills (22)
|
|
120
128
|
|
|
121
129
|
Skills are grouped by a `<group>-` prefix. Type `/<group>-` then `Tab` in Claude Code to see all skills in a group.
|
|
122
130
|
|
|
@@ -128,6 +136,7 @@ Skills are grouped by a `<group>-` prefix. Type `/<group>-` then `Tab` in Claude
|
|
|
128
136
|
| `/feature-refactor` | Restructure existing module to DDD or improve internals |
|
|
129
137
|
| `/feature-api` | Add a new endpoint or integrate an external API |
|
|
130
138
|
| `/feature-deprecate` | Safely sunset a feature, API, or module |
|
|
139
|
+
| `/feature-track` | Plan + approve, then run a tracked loop: branch → checklist → per-item test/implement/verify/commit |
|
|
131
140
|
|
|
132
141
|
**`/fix-*` — Bugs & Incidents**
|
|
133
142
|
|
|
@@ -184,6 +193,7 @@ When more than one skill could fit, use this matrix:
|
|
|
184
193
|
| Want to verify DDD compliance only | `/review-architect` | `/review-pr` (broader scope) |
|
|
185
194
|
| Don't know the right solution yet | `/research-web` | `/research-spike` (skip if you can decide from docs) |
|
|
186
195
|
| Need to validate an idea by building | `/research-spike` | `/feature-new` (commit only after spike) |
|
|
196
|
+
| Driving a multi-step task as a checklist with commit-per-step | `/feature-track` | `/feature-new` (single full DDD feature), `/review-tdd` (one unit, no branch/commit) |
|
|
187
197
|
| Writing README / API docs by hand | `/docs-write` | `/docs-sync` (overkill for single file) |
|
|
188
198
|
| Generating full docs site from codebase | `/docs-sync` | `/docs-write` (manual is slower) |
|
|
189
199
|
|
|
@@ -30,13 +30,13 @@ Let the user choose which areas to focus on:
|
|
|
30
30
|
Which marketing areas do you want to plan?
|
|
31
31
|
|
|
32
32
|
1. Logo & Brand Identity — Design logo, colors, typography, brand guidelines
|
|
33
|
-
Triggers skill: /marketing
|
|
33
|
+
Triggers skill: /marketing-logo
|
|
34
34
|
|
|
35
35
|
2. Video Content Strategy — Plan video series, scripts, production, publishing
|
|
36
|
-
Triggers skill: /marketing
|
|
36
|
+
Triggers skill: /marketing-video
|
|
37
37
|
|
|
38
38
|
3. Content Writing Strategy — Blog posts, social media, SEO, newsletter
|
|
39
|
-
Triggers skill: /marketing
|
|
39
|
+
Triggers skill: /marketing-content
|
|
40
40
|
|
|
41
41
|
4. All of the Above — Complete marketing plan (Recommended)
|
|
42
42
|
|
|
@@ -55,19 +55,19 @@ MARKETING PLAN WORKFLOW
|
|
|
55
55
|
|
|
56
56
|
Phase 1: Brand Foundation
|
|
57
57
|
│
|
|
58
|
-
├── Execute: /marketing
|
|
58
|
+
├── Execute: /marketing-logo skill
|
|
59
59
|
│ └── Output: Brand guidelines, color palette, typography
|
|
60
60
|
│
|
|
61
61
|
▼
|
|
62
62
|
Phase 2: Content Strategy
|
|
63
63
|
│
|
|
64
|
-
├── Execute: /marketing
|
|
64
|
+
├── Execute: /marketing-content skill
|
|
65
65
|
│ └── Output: Content pillars, blog plan, social media plan, newsletter
|
|
66
66
|
│
|
|
67
67
|
▼
|
|
68
68
|
Phase 3: Video Content
|
|
69
69
|
│
|
|
70
|
-
├── Execute: /marketing
|
|
70
|
+
├── Execute: /marketing-video skill
|
|
71
71
|
│ └── Output: Video series, scripts, production specs, calendar
|
|
72
72
|
│
|
|
73
73
|
▼
|
|
@@ -85,7 +85,7 @@ Phase 5: Launch Plan
|
|
|
85
85
|
|
|
86
86
|
### Execution Notes:
|
|
87
87
|
- Run skills sequentially — brand identity informs content and video decisions
|
|
88
|
-
- Pass brand guidelines (colors, voice, tone) from /marketing
|
|
88
|
+
- Pass brand guidelines (colors, voice, tone) from /marketing-logo into /marketing-content and /marketing-video
|
|
89
89
|
- Ensure consistency across all outputs
|
|
90
90
|
|
|
91
91
|
## Step 4: Create Unified Calendar
|
|
@@ -12,9 +12,9 @@ Generate a complete `docs/` site (business overview, architecture, use cases wit
|
|
|
12
12
|
- ✅ Re-generate the whole `docs/` site from current code
|
|
13
13
|
- ✅ Existing docs have drifted; want batch sync with review loop
|
|
14
14
|
- ✅ Project has no structured docs at all
|
|
15
|
-
- ❌ Authoring a single doc by hand (README / API.md only) → use `/docs
|
|
16
|
-
- ❌ Understand the codebase, not document it → use `/research
|
|
17
|
-
- ❌ Adding doc for one endpoint → use `/feature
|
|
15
|
+
- ❌ Authoring a single doc by hand (README / API.md only) → use `/docs-write`
|
|
16
|
+
- ❌ Understand the codebase, not document it → use `/research-onboarding`
|
|
17
|
+
- ❌ Adding doc for one endpoint → use `/feature-api` Phase 4
|
|
18
18
|
|
|
19
19
|
## Read Architecture First
|
|
20
20
|
|
|
@@ -229,10 +229,10 @@ For each iteration:
|
|
|
229
229
|
|
|
230
230
|
| When | Use |
|
|
231
231
|
|------|-----|
|
|
232
|
-
| Author single doc by hand | `/docs
|
|
233
|
-
| Onboard self / new dev | `/research
|
|
234
|
-
| Doc only one new endpoint | `/feature
|
|
235
|
-
| Architecture itself needs review | `/review
|
|
232
|
+
| Author single doc by hand | `/docs-write` |
|
|
233
|
+
| Onboard self / new dev | `/research-onboarding` |
|
|
234
|
+
| Doc only one new endpoint | `/feature-api` Phase 4 |
|
|
235
|
+
| Architecture itself needs review | `/review-architect` |
|
|
236
236
|
|
|
237
237
|
## Recommended Agents
|
|
238
238
|
|
|
@@ -5,15 +5,15 @@ description: Documentation workflow for authoring project documentation manually
|
|
|
5
5
|
|
|
6
6
|
# Documentation Workflow
|
|
7
7
|
|
|
8
|
-
Hand-author project docs (README, API.md, ARCHITECTURE.md, CONTRIBUTING.md) with full control over voice and structure. For automated batch generation of a whole `docs/` tree with a review loop, use `/docs
|
|
8
|
+
Hand-author project docs (README, API.md, ARCHITECTURE.md, CONTRIBUTING.md) with full control over voice and structure. For automated batch generation of a whole `docs/` tree with a review loop, use `/docs-sync` instead.
|
|
9
9
|
|
|
10
10
|
## When to use this skill
|
|
11
11
|
|
|
12
12
|
- ✅ Authoring or updating a specific document
|
|
13
13
|
- ✅ Need opinionated prose, not just structure
|
|
14
14
|
- ✅ Doc is small / scoped — a single file or section
|
|
15
|
-
- ❌ Want automated multi-doc generation → use `/docs
|
|
16
|
-
- ❌ Just need API reference from OpenAPI → use `/feature
|
|
15
|
+
- ❌ Want automated multi-doc generation → use `/docs-sync`
|
|
16
|
+
- ❌ Just need API reference from OpenAPI → use `/feature-api` Phase 4
|
|
17
17
|
|
|
18
18
|
## Read Architecture First
|
|
19
19
|
|
|
@@ -108,7 +108,7 @@ For each doc define:
|
|
|
108
108
|
|
|
109
109
|
### Minimal skeletons
|
|
110
110
|
|
|
111
|
-
Use these as starting points. For full templates, see `/docs
|
|
111
|
+
Use these as starting points. For full templates, see `/docs-sync` (which generates the full set).
|
|
112
112
|
|
|
113
113
|
**README.md** (≤80 lines for most projects)
|
|
114
114
|
```markdown
|
|
@@ -258,10 +258,10 @@ If any issue → return to Phase 3 for that doc, fix, re-review.
|
|
|
258
258
|
|
|
259
259
|
| When | Use |
|
|
260
260
|
|------|-----|
|
|
261
|
-
| Generate full doc site from codebase | `/docs
|
|
262
|
-
| Document a newly added API endpoint | `/feature
|
|
263
|
-
| Onboard self / new dev | `/research
|
|
264
|
-
| Write blog / social content | `/marketing
|
|
261
|
+
| Generate full doc site from codebase | `/docs-sync` |
|
|
262
|
+
| Document a newly added API endpoint | `/feature-api` Phase 4 |
|
|
263
|
+
| Onboard self / new dev | `/research-onboarding` |
|
|
264
|
+
| Write blog / social content | `/marketing-content` |
|
|
265
265
|
|
|
266
266
|
## Recommended Agents
|
|
267
267
|
|
|
@@ -13,8 +13,8 @@ End-to-end workflow for designing, implementing, testing, and documenting APIs
|
|
|
13
13
|
- ✅ Integrating a third-party API (Stripe, OpenAI, etc.) into the system
|
|
14
14
|
- ✅ Replacing or upgrading an existing API integration
|
|
15
15
|
- ❌ Just need a one-off HTTP call in a script → use Bash directly
|
|
16
|
-
- ❌ Need to research which API to use → use `/research
|
|
17
|
-
- ❌ Building a whole new domain → use `/feature
|
|
16
|
+
- ❌ Need to research which API to use → use `/research-web` first
|
|
17
|
+
- ❌ Building a whole new domain → use `/feature-new` (which calls this skill internally for the API surface)
|
|
18
18
|
|
|
19
19
|
---
|
|
20
20
|
|
|
@@ -160,7 +160,7 @@ GET /resource?cursor=<opaque>&limit=<int 1..100, default 20>
|
|
|
160
160
|
|
|
161
161
|
### What to update
|
|
162
162
|
1. **OpenAPI spec** committed to repo (`openapi.yaml` or per-resource files)
|
|
163
|
-
2. **API.md** — append the new endpoint(s) (or refer to `/docs
|
|
163
|
+
2. **API.md** — append the new endpoint(s) (or refer to `/docs-write` skill for full re-author)
|
|
164
164
|
3. **CHANGELOG.md** — note breaking / additive changes
|
|
165
165
|
4. **README.md** — if this changes quick-start
|
|
166
166
|
|
|
@@ -201,11 +201,11 @@ Create a resource. Idempotent via `Idempotency-Key` header.
|
|
|
201
201
|
|
|
202
202
|
## Phase 5: REVIEW LOOP
|
|
203
203
|
|
|
204
|
-
Run `/review
|
|
204
|
+
Run `/review-architect` for the touched domain. Loop until score ≥ B.
|
|
205
205
|
|
|
206
206
|
```
|
|
207
207
|
LOOP:
|
|
208
|
-
1. Run /review
|
|
208
|
+
1. Run /review-architect {stack} {domain}
|
|
209
209
|
2. Fix violations
|
|
210
210
|
3. Re-run tests + build
|
|
211
211
|
4. GOTO 1 (until score ≥ B)
|
|
@@ -257,10 +257,10 @@ LOOP:
|
|
|
257
257
|
| When | Use |
|
|
258
258
|
|------|-----|
|
|
259
259
|
| Designing the API contract from scratch | `@api-designer` agent |
|
|
260
|
-
| Adding the endpoint as part of a larger feature | `/feature
|
|
261
|
-
| Writing tests for the endpoint | `/review
|
|
262
|
-
| Documenting the full API surface | `/docs
|
|
263
|
-
| Reviewing architecture compliance | `/review
|
|
260
|
+
| Adding the endpoint as part of a larger feature | `/feature-new` (calls this internally) |
|
|
261
|
+
| Writing tests for the endpoint | `/review-tdd` |
|
|
262
|
+
| Documenting the full API surface | `/docs-write` |
|
|
263
|
+
| Reviewing architecture compliance | `/review-architect` |
|
|
264
264
|
|
|
265
265
|
---
|
|
266
266
|
|
|
@@ -255,10 +255,10 @@ class OldWidget { ... }
|
|
|
255
255
|
|
|
256
256
|
| When | Use |
|
|
257
257
|
|------|-----|
|
|
258
|
-
| Adding the replacement first | `/feature
|
|
258
|
+
| Adding the replacement first | `/feature-new` |
|
|
259
259
|
| Restructuring the code that uses the deprecated thing | `refactor` |
|
|
260
|
-
| Documenting the migration guide | `/docs
|
|
261
|
-
| Reviewing the removal PR | `/review
|
|
260
|
+
| Documenting the migration guide | `/docs-write` |
|
|
261
|
+
| Reviewing the removal PR | `/review-pr` |
|
|
262
262
|
|
|
263
263
|
---
|
|
264
264
|
|
|
@@ -15,9 +15,9 @@ Build a new feature following DDD layers with rule checks per phase and a final
|
|
|
15
15
|
- ✅ Feature spans multiple DDD layers (domain + app + infra)
|
|
16
16
|
- ✅ The approach is well-understood (no major research / prototype needed)
|
|
17
17
|
- ✅ You want automated architecture review at the end
|
|
18
|
-
- ❌ Quick bug fix → use `/fix
|
|
19
|
-
- ❌ Don't know the right approach yet → `/research
|
|
20
|
-
- ❌ Restructuring existing code → use `/feature
|
|
18
|
+
- ❌ Quick bug fix → use `/fix-hotfix`
|
|
19
|
+
- ❌ Don't know the right approach yet → `/research-web` or `/research-spike` first
|
|
20
|
+
- ❌ Restructuring existing code → use `/feature-refactor`
|
|
21
21
|
|
|
22
22
|
## Read Architecture First
|
|
23
23
|
|
|
@@ -205,7 +205,7 @@ Build in order: **value objects → entities → events → ports → usecases**
|
|
|
205
205
|
|
|
206
206
|
```
|
|
207
207
|
LOOP:
|
|
208
|
-
1. /review
|
|
208
|
+
1. /review-architect {stack} {domain}
|
|
209
209
|
2. IF violations severity ≥ MEDIUM:
|
|
210
210
|
fix all → build → tests → GOTO 1
|
|
211
211
|
3. IF score ≥ B → BREAK
|
|
@@ -253,12 +253,12 @@ LOOP:
|
|
|
253
253
|
|
|
254
254
|
| When | Use |
|
|
255
255
|
|------|-----|
|
|
256
|
-
| Don't know approach yet | `/research
|
|
257
|
-
| Want to validate by prototyping | `/research
|
|
258
|
-
| Adding only an endpoint | `/feature
|
|
259
|
-
| Restructuring existing module | `/feature
|
|
260
|
-
| Architecture check (called automatically in Phase 7) | `/review
|
|
261
|
-
| Write tests inline (TDD style) | `/review
|
|
256
|
+
| Don't know approach yet | `/research-web` → then this skill |
|
|
257
|
+
| Want to validate by prototyping | `/research-spike` → then this skill |
|
|
258
|
+
| Adding only an endpoint | `/feature-api` |
|
|
259
|
+
| Restructuring existing module | `/feature-refactor` |
|
|
260
|
+
| Architecture check (called automatically in Phase 7) | `/review-architect` |
|
|
261
|
+
| Write tests inline (TDD style) | `/review-tdd` |
|
|
262
262
|
|
|
263
263
|
## Recommended Agents
|
|
264
264
|
|
|
@@ -15,9 +15,9 @@ Restructure existing code into DDD layers, or fix drift in an existing DDD modul
|
|
|
15
15
|
- ✅ Migrating a legacy module into DDD layers
|
|
16
16
|
- ✅ Existing DDD module has drifted (fat controller, anemic entity, mixed concerns)
|
|
17
17
|
- ✅ Splitting one domain into multiple bounded contexts
|
|
18
|
-
- ❌ Building a brand-new feature → use `/feature
|
|
18
|
+
- ❌ Building a brand-new feature → use `/feature-new`
|
|
19
19
|
- ❌ Just renaming files / variables → just do it
|
|
20
|
-
- ❌ Fixing a bug → use `/fix
|
|
20
|
+
- ❌ Fixing a bug → use `/fix-hotfix` or `/fix-root-cause`
|
|
21
21
|
|
|
22
22
|
## Read Architecture First
|
|
23
23
|
|
|
@@ -196,11 +196,11 @@ grep -r "{old_import_path}" --include="*.{ext}" . && echo "FAIL: stale imports"
|
|
|
196
196
|
|
|
197
197
|
## Review Loop
|
|
198
198
|
|
|
199
|
-
After Phase 6, call `/review
|
|
199
|
+
After Phase 6, call `/review-architect {stack} {domain}`. Loop until score ≥ B.
|
|
200
200
|
|
|
201
201
|
```
|
|
202
202
|
LOOP:
|
|
203
|
-
1. /review
|
|
203
|
+
1. /review-architect {stack} {domain}
|
|
204
204
|
2. IF violations severity ≥ MEDIUM:
|
|
205
205
|
fix all → full build → all tests → GOTO 1
|
|
206
206
|
3. IF score ≥ B → BREAK
|
|
@@ -252,10 +252,10 @@ LOOP:
|
|
|
252
252
|
|
|
253
253
|
| When | Use |
|
|
254
254
|
|------|-----|
|
|
255
|
-
| Building from scratch (no existing code) | `/feature
|
|
256
|
-
| Just adding tests to untested code | `/review
|
|
257
|
-
| Reviewing the refactor before merging | `/review
|
|
258
|
-
| Final architecture check (called automatically in review loop) | `/review
|
|
255
|
+
| Building from scratch (no existing code) | `/feature-new` |
|
|
256
|
+
| Just adding tests to untested code | `/review-tdd` |
|
|
257
|
+
| Reviewing the refactor before merging | `/review-branch` |
|
|
258
|
+
| Final architecture check (called automatically in review loop) | `/review-architect` |
|
|
259
259
|
|
|
260
260
|
## Recommended Agents
|
|
261
261
|
|
|
@@ -0,0 +1,157 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: feature-track
|
|
3
|
+
description: Tracked development loop — first brainstorm + plan and get user approval, THEN checkout a branch, build a task checklist, and loop each item through test → implement → verify → fix → mark done → commit (one commit per item). Use when user says "bắt đầu loop", "làm track này", "tracked loop", "checklist driven dev", "run the loop", "checkout branch and start track", "brainstorm rồi làm track", "implement with checklist and commit per task".
|
|
4
|
+
args: "[TASK_DESCRIPTION]"
|
|
5
|
+
---
|
|
6
|
+
|
|
7
|
+
# Tracked Development Loop
|
|
8
|
+
|
|
9
|
+
Run a multi-step task as a controlled loop: **brainstorm + plan first, get approval, THEN** checkout a branch, turn the plan into a task checklist, and drive each item through test → implement → verify → fix → mark done → commit — **one commit per item**.
|
|
10
|
+
|
|
11
|
+
**ARGUMENTS:** `<task description>` — e.g., `wallet savings account`, `add rate limiting to auth`, `migrate orders module to events`
|
|
12
|
+
|
|
13
|
+
## When to use this skill
|
|
14
|
+
|
|
15
|
+
- ✅ A task that splits into several verifiable steps you want to drive as a checklist
|
|
16
|
+
- ✅ You want a clean history — one commit per completed step, easy to revert
|
|
17
|
+
- ✅ You want test-first discipline on each step, on a dedicated branch
|
|
18
|
+
- ❌ One-line / obvious change → just do it
|
|
19
|
+
- ❌ Urgent production bug → use `/fix-hotfix`
|
|
20
|
+
- ❌ Single full DDD feature across all layers → use `/feature-new`
|
|
21
|
+
- ❌ Driving ONE unit test-first with no branch/checklist → use `/review-tdd`
|
|
22
|
+
|
|
23
|
+
## Read Architecture First
|
|
24
|
+
|
|
25
|
+
Detect stack via `~/.claude/architecture/_shared/stack-detection.md`. The stack doc tells you:
|
|
26
|
+
- Test command + test file location convention
|
|
27
|
+
- Build + lint commands (for the VERIFY step)
|
|
28
|
+
- Mocking pattern for ports / dependencies
|
|
29
|
+
|
|
30
|
+
---
|
|
31
|
+
|
|
32
|
+
## Workflow
|
|
33
|
+
|
|
34
|
+
```
|
|
35
|
+
0 BRAINSTORM & PLAN ─▶ [USER APPROVES] ─▶ 1 SETUP ─▶ ┌─ 2 LOOP per item ──────────────────────┐ ─▶ 3 REPORT
|
|
36
|
+
(understand, /brainstorm,(last step of (branch + │ TEST → IMPLEMENT → VERIFY → FIX → │
|
|
37
|
+
plan + draft checklist) planning) checklist) │ MARK DONE → COMMIT │
|
|
38
|
+
│ └ fail ─▶ FIX ─▶ VERIFY (to pass) │
|
|
39
|
+
└─────────── next pending item ──────────┘
|
|
40
|
+
```
|
|
41
|
+
|
|
42
|
+
> Core rule: **brainstorm and plan, the LAST planning step is user approval, THEN start the loop.** Never jump straight to branch/checklist.
|
|
43
|
+
|
|
44
|
+
---
|
|
45
|
+
|
|
46
|
+
## Phase 0: BRAINSTORM & PLAN
|
|
47
|
+
|
|
48
|
+
**Goal:** understand the task, pick the simplest viable approach, draft the checklist — and get the user to approve before touching anything.
|
|
49
|
+
|
|
50
|
+
### Steps
|
|
51
|
+
1. **Detect stack** — load the matching architecture doc for test/build/lint commands and conventions.
|
|
52
|
+
2. **Understand the task** — read the relevant code/context. If `[TASK_DESCRIPTION]` is ambiguous (scope, acceptance criteria, edge cases), ask to clarify (AskUserQuestion). Don't assume.
|
|
53
|
+
3. **Brainstorm the approach — invoke `/brainstorm`** to generate and compare options for the task (First Principles, SCAMPER, Working Backwards, 5 Whys…), then pick the simplest that works. For a trivial/obvious task a quick single-framework pass is enough — don't overengineer, but don't skip the step.
|
|
54
|
+
4. **Draft the plan** — settle:
|
|
55
|
+
- the chosen approach,
|
|
56
|
+
- a proposed branch name (`feat/<slug>`),
|
|
57
|
+
- the **draft checklist** — each item = one verifiable unit = one test passing + one commit,
|
|
58
|
+
- files affected + how each item is tested.
|
|
59
|
+
5. **Present the plan and WAIT for approval.** This is the last step of planning. No branch, no code, no `TaskCreate` yet.
|
|
60
|
+
|
|
61
|
+
### Gate
|
|
62
|
+
- [ ] Stack detected
|
|
63
|
+
- [ ] Requirements clear (clarified if they weren't)
|
|
64
|
+
- [ ] Approach brainstormed via `/brainstorm` (simplest viable picked)
|
|
65
|
+
- [ ] Plan + draft checklist presented
|
|
66
|
+
- [ ] **User APPROVED the plan**
|
|
67
|
+
|
|
68
|
+
---
|
|
69
|
+
|
|
70
|
+
## Phase 1: SETUP
|
|
71
|
+
|
|
72
|
+
**Goal:** turn the approved plan into a branch + a live checklist. Runs only AFTER approval.
|
|
73
|
+
|
|
74
|
+
### Steps
|
|
75
|
+
1. **Checkout a new branch** with the approved name (`feat/<slug>`) off the current base. Never run the loop on `master` / `main`.
|
|
76
|
+
2. **Create the checklist** with `TaskCreate` — one task per checklist item, in order, matching the approved plan.
|
|
77
|
+
|
|
78
|
+
### Gate
|
|
79
|
+
- [ ] New branch created and checked out
|
|
80
|
+
- [ ] Checklist created via `TaskCreate`, matching the approved plan
|
|
81
|
+
|
|
82
|
+
---
|
|
83
|
+
|
|
84
|
+
## Phase 2: LOOP (per item)
|
|
85
|
+
|
|
86
|
+
Take the next `pending` item from the task list and run it through the full cycle. Repeat until no `pending` items remain.
|
|
87
|
+
|
|
88
|
+
1. **MARK IN_PROGRESS** — `TaskUpdate` the item → `in_progress`.
|
|
89
|
+
2. **TEST (RED)** — write a unit test describing the item's behavior. Run it and confirm it **fails for the right reason** (code missing / assertion fails), not a typo or import error.
|
|
90
|
+
3. **IMPLEMENT (GREEN)** — write the minimal code to make the test pass. Follow an existing reference module so it matches the codebase style.
|
|
91
|
+
4. **VERIFY** — actually run it: the item's test + build + lint (commands per stack). Never claim pass without running.
|
|
92
|
+
5. **FIX** — if anything fails, fix at the **root, never swallow the error** (handle exceptions explicitly — no empty catch). Then go back to VERIFY until green.
|
|
93
|
+
6. **MARK DONE** — `TaskUpdate` the item → `completed`.
|
|
94
|
+
7. **COMMIT** — commit just this item with a conventional message (`feat:` / `fix:` / `test:` / `refactor:` …). Commit only — **do not push automatically**.
|
|
95
|
+
|
|
96
|
+
### Gate (per item, before moving on)
|
|
97
|
+
- [ ] Test green
|
|
98
|
+
- [ ] Build + lint green
|
|
99
|
+
- [ ] Task marked `completed`
|
|
100
|
+
- [ ] Item committed (one commit)
|
|
101
|
+
|
|
102
|
+
---
|
|
103
|
+
|
|
104
|
+
## Phase 3: Final Report
|
|
105
|
+
|
|
106
|
+
```markdown
|
|
107
|
+
## Track Complete: {task}
|
|
108
|
+
|
|
109
|
+
Branch: `feat/{slug}`
|
|
110
|
+
|
|
111
|
+
| # | Item | Test(s) | Commit | Status |
|
|
112
|
+
|---|------|---------|--------|--------|
|
|
113
|
+
| 1 | {item} | {test name} | {hash} `feat: …` | ✅ |
|
|
114
|
+
| 2 | {item} | {test name} | {hash} `feat: …` | ✅ |
|
|
115
|
+
|
|
116
|
+
### Totals
|
|
117
|
+
- Items: {N} done / {N}
|
|
118
|
+
- Commits: {N}
|
|
119
|
+
- Tests added: {N}, all green
|
|
120
|
+
|
|
121
|
+
### Next
|
|
122
|
+
- Push + open PR, or run `/review-branch` first
|
|
123
|
+
```
|
|
124
|
+
|
|
125
|
+
---
|
|
126
|
+
|
|
127
|
+
## Hard Rules
|
|
128
|
+
|
|
129
|
+
- **Plan + user approval FIRST** — no branch, no code until the plan is approved.
|
|
130
|
+
- **Branch first** — never run the loop on `master` / `main`.
|
|
131
|
+
- **RED before GREEN** — no production code that a failing test didn't drive.
|
|
132
|
+
- **Verify by running real commands** — never assert pass from inspection alone.
|
|
133
|
+
- **Never swallow errors** in FIX — handle exceptions explicitly, fix the root cause.
|
|
134
|
+
- **One commit per item** — don't squash steps together, don't push automatically.
|
|
135
|
+
- **DB operations (migrate / seed / drop)** inside any item: **CONFIRM with the user before running — every time, even if a previous item was already confirmed.**
|
|
136
|
+
- **Simple first** — don't overengineer an item.
|
|
137
|
+
|
|
138
|
+
---
|
|
139
|
+
|
|
140
|
+
## Related Skills
|
|
141
|
+
|
|
142
|
+
| When | Use |
|
|
143
|
+
|------|-----|
|
|
144
|
+
| Need idea frameworks for Phase 0 | `/brainstorm` |
|
|
145
|
+
| Drive ONE unit deeper test-first | `/review-tdd` |
|
|
146
|
+
| Full DDD feature across all layers | `/feature-new` |
|
|
147
|
+
| Self-review the branch after the loop | `/review-branch` |
|
|
148
|
+
| Urgent prod bug, no checklist | `/fix-hotfix` |
|
|
149
|
+
|
|
150
|
+
## Recommended Agents
|
|
151
|
+
|
|
152
|
+
| Phase | Agent | Purpose |
|
|
153
|
+
|-------|-------|---------|
|
|
154
|
+
| BRAINSTORM / PLAN | `@clean-architect` | Approach + decomposition |
|
|
155
|
+
| TEST | `@test-writer` | Failing test per item |
|
|
156
|
+
| IMPLEMENT | Stack-specific dev agent | Minimal implementation |
|
|
157
|
+
| VERIFY / FIX | `@code-reviewer` | Catch issues before commit |
|
|
@@ -12,9 +12,9 @@ Fast-track bug fix with a rollback plan. Use when the cause is identifiable in u
|
|
|
12
12
|
- ✅ Bug is reproducible and root cause is clear (or quickly identifiable)
|
|
13
13
|
- ✅ Need a fix shipped fast with a rollback plan
|
|
14
14
|
- ✅ Severity ranges from low (minor UI) to high (feature broken)
|
|
15
|
-
- ❌ Production is down right now → use `/fix
|
|
16
|
-
- ❌ Bug has been "fixed" multiple times and keeps returning → use `/fix
|
|
17
|
-
- ❌ Building a new feature → use `/feature
|
|
15
|
+
- ❌ Production is down right now → use `/fix-incident` first
|
|
16
|
+
- ❌ Bug has been "fixed" multiple times and keeps returning → use `/fix-root-cause`
|
|
17
|
+
- ❌ Building a new feature → use `/feature-new`
|
|
18
18
|
|
|
19
19
|
## Read Architecture First
|
|
20
20
|
|
|
@@ -22,7 +22,7 @@ Read `ddd-architecture.md` + stack-specific doc — the fix must land in the rig
|
|
|
22
22
|
|
|
23
23
|
## Severity
|
|
24
24
|
|
|
25
|
-
Use `~/.claude/architecture/_shared/severity-levels.md` (incident table). Hotfix typically covers P2-P4. P1 → start with `/fix
|
|
25
|
+
Use `~/.claude/architecture/_shared/severity-levels.md` (incident table). Hotfix typically covers P2-P4. P1 → start with `/fix-incident` for triage / comms.
|
|
26
26
|
|
|
27
27
|
---
|
|
28
28
|
|
|
@@ -61,7 +61,7 @@ IDENTIFY → REPRODUCE → FIX → VERIFY → DEPLOY
|
|
|
61
61
|
- [ ] Error captured verbatim
|
|
62
62
|
- [ ] Reproduction steps known (or "intermittent" noted)
|
|
63
63
|
- [ ] Severity assigned (P2 / P3 / P4)
|
|
64
|
-
- [ ] Root cause identified (use `/fix
|
|
64
|
+
- [ ] Root cause identified (use `/fix-root-cause` if 5 Whys doesn't converge)
|
|
65
65
|
|
|
66
66
|
---
|
|
67
67
|
|
|
@@ -91,7 +91,7 @@ IDENTIFY → REPRODUCE → FIX → VERIFY → DEPLOY
|
|
|
91
91
|
### Rules
|
|
92
92
|
- Fix at the **root cause**, not the symptom
|
|
93
93
|
- Land the fix in the right layer (boundary validation → handler/DTO; business rule → usecase; data shape → infra mapper)
|
|
94
|
-
- **Add a regression test first** (RED), then make it pass (GREEN). See `/review
|
|
94
|
+
- **Add a regression test first** (RED), then make it pass (GREEN). See `/review-tdd`.
|
|
95
95
|
- Don't refactor on the fix — separate PR
|
|
96
96
|
- If the fix requires schema migration, plan it as 2 deploys (compatible code first, then migration)
|
|
97
97
|
|
|
@@ -131,7 +131,7 @@ For data-shape bugs (null where not expected, type mismatch from external API):
|
|
|
131
131
|
## Phase 5: DEPLOY
|
|
132
132
|
|
|
133
133
|
### Pre-deploy checklist
|
|
134
|
-
- [ ] PR reviewed (`/review
|
|
134
|
+
- [ ] PR reviewed (`/review-branch` self + `/review-pr` from teammate)
|
|
135
135
|
- [ ] Rollback plan documented (see below)
|
|
136
136
|
- [ ] On-call notified
|
|
137
137
|
- [ ] CHANGELOG entry
|
|
@@ -215,11 +215,11 @@ Document **before** deploying:
|
|
|
215
215
|
|
|
216
216
|
| When | Use |
|
|
217
217
|
|------|-----|
|
|
218
|
-
| Production is down right now | `/fix
|
|
219
|
-
| Bug keeps coming back after fixes | `/fix
|
|
220
|
-
| Need to write regression test first | `/review
|
|
221
|
-
| Bug is on an open PR | `/fix
|
|
222
|
-
| Self-review before opening PR | `/review
|
|
218
|
+
| Production is down right now | `/fix-incident` first, then hotfix |
|
|
219
|
+
| Bug keeps coming back after fixes | `/fix-root-cause` |
|
|
220
|
+
| Need to write regression test first | `/review-tdd` |
|
|
221
|
+
| Bug is on an open PR | `/fix-pr-comment` |
|
|
222
|
+
| Self-review before opening PR | `/review-branch` |
|
|
223
223
|
|
|
224
224
|
## Recommended Agents
|
|
225
225
|
|
|
@@ -12,8 +12,8 @@ Structured workflow for handling production incidents from triage to postmortem.
|
|
|
12
12
|
- ✅ Production is down or degraded right now
|
|
13
13
|
- ✅ Users are impacted, on-call has been paged
|
|
14
14
|
- ✅ Need to coordinate response, mitigation, comms, and postmortem
|
|
15
|
-
- ❌ Bug is reported but no users impacted → use `/fix
|
|
16
|
-
- ❌ Already mitigated, need to find root cause → use `/fix
|
|
15
|
+
- ❌ Bug is reported but no users impacted → use `/fix-hotfix`
|
|
16
|
+
- ❌ Already mitigated, need to find root cause → use `/fix-root-cause`
|
|
17
17
|
- ❌ Postmortem only, no active incident → jump to Phase 5
|
|
18
18
|
|
|
19
19
|
---
|
|
@@ -205,7 +205,7 @@ Live walkthrough (5-10 min): outgoing IC walks incoming IC through the timeline
|
|
|
205
205
|
**Goal:** apply the permanent fix and verify.
|
|
206
206
|
|
|
207
207
|
### Actions
|
|
208
|
-
1. Confirm root cause (use `/fix
|
|
208
|
+
1. Confirm root cause (use `/fix-root-cause` workflow if not obvious)
|
|
209
209
|
2. Implement permanent fix following stack architecture
|
|
210
210
|
3. Add test that would have caught this
|
|
211
211
|
4. Deploy fix through normal pipeline (don't skip CI even under pressure — unless explicitly authorized)
|
|
@@ -340,10 +340,10 @@ Postmortem: {date} — link to come
|
|
|
340
340
|
|
|
341
341
|
| When | Use |
|
|
342
342
|
|------|-----|
|
|
343
|
-
| Bug reported but no active incident | `/fix
|
|
344
|
-
| Root cause unclear, need deep investigation | `/fix
|
|
343
|
+
| Bug reported but no active incident | `/fix-hotfix` |
|
|
344
|
+
| Root cause unclear, need deep investigation | `/fix-root-cause` |
|
|
345
345
|
| Need to roll back deployment | (use ops runbook, not a skill) |
|
|
346
|
-
| Documenting postmortem as a doc | `/docs
|
|
346
|
+
| Documenting postmortem as a doc | `/docs-write` |
|
|
347
347
|
|
|
348
348
|
---
|
|
349
349
|
|
|
@@ -15,9 +15,9 @@ Fetch all review comments on an open PR, address each, push, and respond back to
|
|
|
15
15
|
- ✅ Reviewer left comments and you need to address each one
|
|
16
16
|
- ✅ Want to track which comments are resolved vs still open
|
|
17
17
|
- ✅ Need structured responses posted back to GitHub
|
|
18
|
-
- ❌ You are the reviewer → use `/review
|
|
19
|
-
- ❌ No PR yet → use `/review
|
|
20
|
-
- ❌ Fix is a brand-new bug surfaced by reviewer → `/fix
|
|
18
|
+
- ❌ You are the reviewer → use `/review-pr`
|
|
19
|
+
- ❌ No PR yet → use `/review-branch`
|
|
20
|
+
- ❌ Fix is a brand-new bug surfaced by reviewer → `/fix-hotfix` first, then this skill
|
|
21
21
|
|
|
22
22
|
---
|
|
23
23
|
|
|
@@ -86,15 +86,15 @@ Build a triage table:
|
|
|
86
86
|
- Fix in order: must-fix → should-fix → discussion outcomes
|
|
87
87
|
- One concern per commit (`fix(handler): validate input in DTO per #pr-comment-1`)
|
|
88
88
|
- After each batch: run build + lint + tests locally
|
|
89
|
-
- Re-run `/review
|
|
89
|
+
- Re-run `/review-branch` before pushing
|
|
90
90
|
|
|
91
91
|
### When the fix changes architecture
|
|
92
|
-
If a comment requests something structural (move logic between layers, add a port), use `/feature
|
|
92
|
+
If a comment requests something structural (move logic between layers, add a port), use `/feature-refactor` for that subtree, then come back here to respond.
|
|
93
93
|
|
|
94
94
|
### Gate
|
|
95
95
|
- [ ] All must-fix items addressed
|
|
96
96
|
- [ ] Build + lint + tests green
|
|
97
|
-
- [ ] Self-review with `/review
|
|
97
|
+
- [ ] Self-review with `/review-branch` clean
|
|
98
98
|
|
|
99
99
|
---
|
|
100
100
|
|
|
@@ -162,7 +162,7 @@ gh pr edit $PR --add-reviewer {original_reviewer}
|
|
|
162
162
|
- **Reply to every comment** — silence reads as ignoring
|
|
163
163
|
- **Push back politely** when you have a real reason; never just close threads
|
|
164
164
|
- **One concern per commit** — easier for reviewer to verify
|
|
165
|
-
- **Run `/review
|
|
165
|
+
- **Run `/review-branch` before pushing** — catch easy issues before reviewer round 2
|
|
166
166
|
- **Don't argue style** when the team has a linter — let the tool decide
|
|
167
167
|
|
|
168
168
|
---
|
|
@@ -171,10 +171,10 @@ gh pr edit $PR --add-reviewer {original_reviewer}
|
|
|
171
171
|
|
|
172
172
|
| When | Use |
|
|
173
173
|
|------|-----|
|
|
174
|
-
| You are the reviewer | `/review
|
|
175
|
-
| Self-review before pushing again | `/review
|
|
176
|
-
| Reviewer asked for architectural refactor | `/feature
|
|
177
|
-
| Reviewer surfaced a bug needing investigation | `/fix
|
|
174
|
+
| You are the reviewer | `/review-pr` |
|
|
175
|
+
| Self-review before pushing again | `/review-branch` |
|
|
176
|
+
| Reviewer asked for architectural refactor | `/feature-refactor` |
|
|
177
|
+
| Reviewer surfaced a bug needing investigation | `/fix-root-cause` |
|
|
178
178
|
|
|
179
179
|
## Recommended Agents
|
|
180
180
|
|