@kennethsolomon/shipkit 3.17.1 → 3.19.0
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 +16 -1
- package/commands/sk/finish-feature.md +14 -0
- package/commands/sk/help.md +13 -15
- package/package.json +1 -1
- package/skills/sk:autopilot/SKILL.md +25 -3
- package/skills/sk:gates/SKILL.md +2 -1
- package/skills/sk:laravel-deploy/SKILL.md +112 -0
- package/skills/sk:laravel-init/SKILL.md +18 -0
- package/skills/sk:setup-claude/SKILL.md +45 -0
- package/skills/sk:setup-claude/templates/.claude/agents/debugger.md +7 -0
- package/skills/sk:setup-claude/templates/.claude/agents/qa-engineer.md +8 -0
- package/skills/sk:setup-claude/templates/.claude/agents/tech-writer.md +7 -0
- package/skills/sk:setup-claude/templates/.claude/rules/laravel.md.template +11 -0
- package/skills/sk:setup-claude/templates/CLAUDE.md.template +10 -1
- package/skills/sk:setup-optimizer/SKILL.md +4 -4
- package/skills/sk:start/SKILL.md +2 -2
- package/skills/sk:team/SKILL.md +1 -1
package/README.md
CHANGED
|
@@ -63,7 +63,7 @@ npm install -g @kennethsolomon/shipkit && shipkit # update globally
|
|
|
63
63
|
| I want to... | Start here | Flow |
|
|
64
64
|
|---|---|---|
|
|
65
65
|
| **Not sure — just describe my task** | `/sk:start <description>` | Auto-classified |
|
|
66
|
-
| **Build a new feature** | `/sk:start add <feature>` | Feature (8
|
|
66
|
+
| **Build a new feature** | `/sk:start add <feature>` | Feature (8 phases + scope check, learn, retro) |
|
|
67
67
|
| **Build a full-stack feature (backend + frontend + mobile)** | `/sk:start --team add <feature>` | Feature with parallel agents |
|
|
68
68
|
| **Make a small change** (config, copy, dependency bump) | `/sk:start bump lodash` | Fast-track (5 steps) |
|
|
69
69
|
| **Fix a bug** | `/sk:start fix <description>` | Debug (7 steps) |
|
|
@@ -129,6 +129,9 @@ qa-engineer (background) → writes 14 Playwright E2E scenarios while others im
|
|
|
129
129
|
|
|
130
130
|
Backend and frontend work in isolated worktrees — zero conflicts. Results merge when both complete.
|
|
131
131
|
|
|
132
|
+
**Step 5.5 — Scope Check** (`/sk:scope-check`)
|
|
133
|
+
Compares everything that was implemented against `tasks/todo.md`. Flags anything that crept in beyond the plan — extra features, unrequested refactors, new files not in scope. Trims or defers the excess before committing.
|
|
134
|
+
|
|
132
135
|
**Step 6 — Commit** (`/sk:smart-commit`)
|
|
133
136
|
Presents the diff. You approve. Commits.
|
|
134
137
|
|
|
@@ -156,6 +159,18 @@ Each failure auto-fixes and re-runs. One squash commit per gate pass.
|
|
|
156
159
|
**Step 8 — Finalize** (`/sk:finish-feature`)
|
|
157
160
|
Changelog updated. PR created. Feature spec synced. Asks about release.
|
|
158
161
|
|
|
162
|
+
**Step 8.5 — Learn** (`/sk:learn`)
|
|
163
|
+
Extracts reusable patterns from this session:
|
|
164
|
+
> "Rate limiting must be applied to all auth endpoints — security-reviewer flagged POST /login"
|
|
165
|
+
|
|
166
|
+
Saved to `~/.claude/skills/learned/` — available in future sessions across all projects.
|
|
167
|
+
|
|
168
|
+
**Step 8.6 — Retro** (`/sk:retro`)
|
|
169
|
+
Brief post-ship retrospective — 3-5 bullets:
|
|
170
|
+
- What went well (gates caught rate-limit issue before PR)
|
|
171
|
+
- What slowed down (schema index discovery required re-migration)
|
|
172
|
+
- Next action (add rate-limit check to write-tests template)
|
|
173
|
+
|
|
159
174
|
---
|
|
160
175
|
|
|
161
176
|
### Scenario B — Fixing a Bug
|
|
@@ -155,6 +155,20 @@ If unresolved Critical/High findings remain, warn the user before proceeding.
|
|
|
155
155
|
|
|
156
156
|
e) Report the PR URL to the user.
|
|
157
157
|
|
|
158
|
+
8. **Capture Patterns** (`/sk:learn`)
|
|
159
|
+
|
|
160
|
+
After the PR is created, run `/sk:learn` to extract reusable patterns from this session.
|
|
161
|
+
Present extracted patterns and ask: "Save patterns? (all / 1,3 / none)"
|
|
162
|
+
|
|
163
|
+
9. **Retrospective** (`/sk:retro`)
|
|
164
|
+
|
|
165
|
+
Run `/sk:retro` to capture a brief post-ship retrospective:
|
|
166
|
+
- What went well
|
|
167
|
+
- What slowed things down
|
|
168
|
+
- Top action items for next time
|
|
169
|
+
|
|
170
|
+
Output is appended to `tasks/progress.md`.
|
|
171
|
+
|
|
158
172
|
## When Done
|
|
159
173
|
|
|
160
174
|
> "Feature finalized and PR created! Run `/sk:release` when ready to tag and publish."
|
package/commands/sk/help.md
CHANGED
|
@@ -21,21 +21,19 @@ Run these commands in order for a complete, quality-gated feature build.
|
|
|
21
21
|
|
|
22
22
|
## Feature Workflow
|
|
23
23
|
|
|
24
|
-
| Command | Purpose |
|
|
25
|
-
|
|
26
|
-
| `/sk:brainstorm` | Explore requirements
|
|
27
|
-
| `/sk:
|
|
28
|
-
| `/sk:
|
|
29
|
-
| `/sk:
|
|
30
|
-
| `/sk:write-tests` | TDD red:
|
|
31
|
-
| `/sk:
|
|
32
|
-
| `/sk:smart-commit` | Conventional commit with approval |
|
|
33
|
-
| `/sk:
|
|
34
|
-
| `/sk:
|
|
35
|
-
| `/sk:
|
|
36
|
-
| `/sk:
|
|
37
|
-
| `/sk:update-task` | Mark task done, log completion |
|
|
38
|
-
| `/sk:finish-feature` | Changelog + PR creation |
|
|
24
|
+
| # | Command | Purpose |
|
|
25
|
+
|---|---------|---------|
|
|
26
|
+
| 1 | `/sk:brainstorm` | Explore requirements — **no code yet** |
|
|
27
|
+
| 2 | `/sk:frontend-design` or `/sk:api-design` | Design UI or API contracts *(auto-skip if no frontend/API keywords)* |
|
|
28
|
+
| 3 | `/sk:write-plan` | Write a decision-complete plan to `tasks/todo.md` |
|
|
29
|
+
| 4 | `/sk:branch` | Create a feature branch from the current task |
|
|
30
|
+
| 5 | `/sk:write-tests` + `/sk:execute-plan` | TDD red + green (includes `/sk:schema-migrate` if DB keywords detected) |
|
|
31
|
+
| 5.5 | `/sk:scope-check` | Trim scope creep — compare implementation to plan |
|
|
32
|
+
| 6 | `/sk:smart-commit` | Conventional commit with approval |
|
|
33
|
+
| 7 | `/sk:gates` | **All quality gates** — lint, test, security, perf, review, e2e *(hard gate)* |
|
|
34
|
+
| 8 | `/sk:finish-feature` | Changelog + PR creation |
|
|
35
|
+
| 8.5 | `/sk:learn` | Extract reusable patterns from this session |
|
|
36
|
+
| 8.6 | `/sk:retro` | Post-ship retrospective — velocity, blockers, next actions |
|
|
39
37
|
|
|
40
38
|
## Requirement Change Flow
|
|
41
39
|
|
package/package.json
CHANGED
|
@@ -1,12 +1,12 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: sk:autopilot
|
|
3
|
-
description: Hands-free workflow — runs all 8
|
|
3
|
+
description: Hands-free workflow — runs all 8 phases (including scope check, learn, retro) with auto-skip, auto-advance, auto-commit. Stops only for direction approval, 3-strike failures, and PR push.
|
|
4
4
|
allowed-tools: Read, Write, Bash, Glob, Grep, Agent, Skill
|
|
5
5
|
---
|
|
6
6
|
|
|
7
7
|
# Autopilot Mode
|
|
8
8
|
|
|
9
|
-
Hands-free workflow that executes all 8
|
|
9
|
+
Hands-free workflow that executes all 8 phases (including scope check, learn, and retro) of the ShipIt workflow with minimal interruptions. Same quality gates, same fix loops, same 100% coverage — just fewer stops.
|
|
10
10
|
|
|
11
11
|
## When to Use
|
|
12
12
|
|
|
@@ -22,7 +22,7 @@ Hands-free workflow that executes all 8 steps of the ShipIt workflow with minima
|
|
|
22
22
|
|
|
23
23
|
## Quality Guarantee
|
|
24
24
|
|
|
25
|
-
Autopilot runs the EXACT same
|
|
25
|
+
Autopilot runs the EXACT same workflow as manual mode (8 phases: explore, design, plan, branch, implement + scope check, commit, gates, ship + learn + retro):
|
|
26
26
|
- ALL quality gates enforced (lint, test, security, perf, review, e2e)
|
|
27
27
|
- ALL fix-rerun loops active
|
|
28
28
|
- 100% test coverage required on new code
|
|
@@ -64,6 +64,13 @@ Create feature branch auto-named from the task. Do NOT ask for confirmation.
|
|
|
64
64
|
- Run `/sk:execute-plan` (TDD green phase)
|
|
65
65
|
- Auto-advance when done
|
|
66
66
|
|
|
67
|
+
### 5.5. Scope Check (auto-advance)
|
|
68
|
+
|
|
69
|
+
Run `/sk:scope-check` to compare the implementation against `tasks/todo.md`.
|
|
70
|
+
|
|
71
|
+
- If scope creep detected: log findings, trim the excess, re-commit
|
|
72
|
+
- If on-scope: auto-advance silently
|
|
73
|
+
|
|
67
74
|
### 6. Commit (auto-commit)
|
|
68
75
|
|
|
69
76
|
Auto-commit with conventional commit format. Do NOT ask for commit message approval.
|
|
@@ -96,6 +103,20 @@ After confirmation:
|
|
|
96
103
|
- Sync features (`/sk:features`)
|
|
97
104
|
- Ask about release (never auto-skipped)
|
|
98
105
|
|
|
106
|
+
### 8.5. Learn (auto-advance)
|
|
107
|
+
|
|
108
|
+
Run `/sk:learn` to extract reusable patterns from this session.
|
|
109
|
+
|
|
110
|
+
- Patterns are saved to `~/.claude/skills/learned/` automatically
|
|
111
|
+
- Auto-advance after saving — no confirmation needed in autopilot
|
|
112
|
+
|
|
113
|
+
### 8.6. Retro (auto-advance)
|
|
114
|
+
|
|
115
|
+
Run `/sk:retro` to capture velocity, blockers, and action items for this feature.
|
|
116
|
+
|
|
117
|
+
- Output is brief — 3-5 bullets covering what went well, what slowed down, and next actions
|
|
118
|
+
- Appended to `tasks/progress.md`
|
|
119
|
+
|
|
99
120
|
## 3-Strike Protocol
|
|
100
121
|
|
|
101
122
|
If any step fails 3 times:
|
|
@@ -111,6 +132,7 @@ If any step fails 3 times:
|
|
|
111
132
|
| Direction approval | After brainstorm (step 1) | User must approve the approach |
|
|
112
133
|
| 3-strike failure | Any step fails 3x | Needs human judgment |
|
|
113
134
|
| PR push | Before creating PR (step 8) | Visible to others — always confirm |
|
|
135
|
+
| Release | After step 8.6 | Never auto-skipped — always ask |
|
|
114
136
|
|
|
115
137
|
Everything else auto-advances.
|
|
116
138
|
|
package/skills/sk:gates/SKILL.md
CHANGED
|
@@ -10,7 +10,7 @@ Run all quality gates (lint, test, security, perf, review, e2e) in optimized bat
|
|
|
10
10
|
|
|
11
11
|
## When to Use
|
|
12
12
|
|
|
13
|
-
Run `/sk:gates` after
|
|
13
|
+
Run `/sk:gates` after `/sk:smart-commit` completes (step 6). This single command covers all quality gates (step 7 of the workflow).
|
|
14
14
|
|
|
15
15
|
## Execution Strategy
|
|
16
16
|
|
|
@@ -23,6 +23,7 @@ Launch 3 agents simultaneously:
|
|
|
23
23
|
1. **Linter agent** — runs all formatters, analyzers, dep audits
|
|
24
24
|
2. **`security-reviewer` agent** — OWASP audit on changed files (read-only; reports findings, does not fix)
|
|
25
25
|
3. **`performance-optimizer` agent** — bundle, N+1, Core Web Vitals, memory (worktree isolation — finds AND fixes critical/high issues)
|
|
26
|
+
**Auto-skip:** If NO frontend keywords (component, view, page, CSS, UI, form, modal, button, react, vue, svelte, blade) AND NO database keywords (migration, schema, table, column, model, database, foreign key, index, seed) appear in `tasks/todo.md`, skip this agent and log: `Auto-skipped: Performance (no frontend or database keywords in plan)`.
|
|
26
27
|
|
|
27
28
|
These 3 have no dependencies on each other. Run them in parallel using the Agent tool.
|
|
28
29
|
|
|
@@ -0,0 +1,112 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: sk:laravel-deploy
|
|
3
|
+
description: "Deploy a Laravel application to Laravel Cloud via the `cloud` CLI. Requires /sk:gates to pass before deploying."
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# /sk:laravel-deploy
|
|
7
|
+
|
|
8
|
+
Deploy the current Laravel application to Laravel Cloud.
|
|
9
|
+
|
|
10
|
+
## Pre-Deploy Gate Check
|
|
11
|
+
|
|
12
|
+
**Gates must pass before any deploy.** If `/sk:gates` has not been run since the last code change, run it first:
|
|
13
|
+
|
|
14
|
+
```
|
|
15
|
+
/sk:gates
|
|
16
|
+
```
|
|
17
|
+
|
|
18
|
+
All quality checks (lint, tests, security, review) must be green before deploying to any environment.
|
|
19
|
+
|
|
20
|
+
## Setup (First Time)
|
|
21
|
+
|
|
22
|
+
Install the Laravel Cloud CLI if missing:
|
|
23
|
+
```bash
|
|
24
|
+
composer global require laravel/cloud-cli
|
|
25
|
+
```
|
|
26
|
+
|
|
27
|
+
Authenticate:
|
|
28
|
+
```bash
|
|
29
|
+
cloud auth -n
|
|
30
|
+
```
|
|
31
|
+
|
|
32
|
+
## CLI Rules
|
|
33
|
+
|
|
34
|
+
Never hardcode command signatures — always run `cloud <command> -h` to discover options at runtime.
|
|
35
|
+
|
|
36
|
+
| Operation | Flags |
|
|
37
|
+
|-----------|-------|
|
|
38
|
+
| Read | `--json -n` |
|
|
39
|
+
| Create | `--json -n` |
|
|
40
|
+
| Update | `--json -n --force` |
|
|
41
|
+
| Delete | `-n --force` (no `--json`) |
|
|
42
|
+
| Deploy | `-n` with all options explicitly |
|
|
43
|
+
|
|
44
|
+
## Deployment Workflow
|
|
45
|
+
|
|
46
|
+
### First Deploy
|
|
47
|
+
```bash
|
|
48
|
+
cloud ship -n
|
|
49
|
+
```
|
|
50
|
+
|
|
51
|
+
### Existing App
|
|
52
|
+
```bash
|
|
53
|
+
cloud repo:config
|
|
54
|
+
cloud deploy {app} {env} -n --open
|
|
55
|
+
cloud deploy:monitor -n
|
|
56
|
+
```
|
|
57
|
+
|
|
58
|
+
### Environment Variables
|
|
59
|
+
```bash
|
|
60
|
+
cloud environment:variables -n --force
|
|
61
|
+
```
|
|
62
|
+
|
|
63
|
+
### Infrastructure
|
|
64
|
+
|
|
65
|
+
**Database:**
|
|
66
|
+
```bash
|
|
67
|
+
cloud database-cluster:create --json -n
|
|
68
|
+
cloud database:create --json -n
|
|
69
|
+
```
|
|
70
|
+
|
|
71
|
+
**Cache:**
|
|
72
|
+
```bash
|
|
73
|
+
cloud cache:create --json -n
|
|
74
|
+
```
|
|
75
|
+
|
|
76
|
+
**Custom Domain:**
|
|
77
|
+
```bash
|
|
78
|
+
cloud domain:create --json -n
|
|
79
|
+
cloud domain:verify -n
|
|
80
|
+
```
|
|
81
|
+
|
|
82
|
+
## Sub-Agent Delegation
|
|
83
|
+
|
|
84
|
+
Delegate to background agents (non-blocking):
|
|
85
|
+
- `deploy:monitor` — watch deployment progress
|
|
86
|
+
- `deployment:get` — fetch deployment status
|
|
87
|
+
- `resource:list` — list infrastructure resources
|
|
88
|
+
|
|
89
|
+
Keep in main context:
|
|
90
|
+
- `cloud deploy` (need deployment ID immediately)
|
|
91
|
+
- Short status commands
|
|
92
|
+
|
|
93
|
+
## Troubleshooting
|
|
94
|
+
|
|
95
|
+
Inspect current infrastructure state:
|
|
96
|
+
```bash
|
|
97
|
+
cloud <resource>:list --json -n
|
|
98
|
+
```
|
|
99
|
+
|
|
100
|
+
Check deployment logs:
|
|
101
|
+
```bash
|
|
102
|
+
cloud deployment:get {id} --json -n
|
|
103
|
+
```
|
|
104
|
+
|
|
105
|
+
## Official Plugin
|
|
106
|
+
|
|
107
|
+
For richer guidance with checklists for complex multi-step infrastructure operations, install the official plugin:
|
|
108
|
+
```
|
|
109
|
+
/plugin install laravel-cloud@laravel
|
|
110
|
+
```
|
|
111
|
+
|
|
112
|
+
Docs: https://cloud.laravel.com/docs
|
|
@@ -32,6 +32,24 @@ Behavior depends on who owns the file:
|
|
|
32
32
|
- **Generated** (has `<!-- Generated by /laravel-init -->` marker): safe to edit directly with section-level updates using `<!-- BEGIN/END -->` markers
|
|
33
33
|
- **User-owned** (no marker): never edit — create `CLAUDE-sub-agents.md` alongside it and let the user copy what they want
|
|
34
34
|
|
|
35
|
+
## Optional: Official Laravel Plugins
|
|
36
|
+
|
|
37
|
+
After setup completes, suggest the two official Laravel plugins from Taylor Otwell:
|
|
38
|
+
|
|
39
|
+
> "Install official Laravel plugins? (`laravel-simplifier` for code refinement + `laravel-cloud` for deployments) [y/n]"
|
|
40
|
+
|
|
41
|
+
If yes, install each:
|
|
42
|
+
```bash
|
|
43
|
+
/plugin install laravel-simplifier@laravel
|
|
44
|
+
/plugin install laravel-cloud@laravel
|
|
45
|
+
```
|
|
46
|
+
|
|
47
|
+
**What each adds:**
|
|
48
|
+
- `laravel-simplifier` — an Opus-powered agent that reviews recently modified PHP/Laravel code and refines it for PSR-12 compliance, Laravel conventions, and readability without changing behavior. Invoke after `/sk:execute-plan`: "Review recent changes using the laravel-simplifier agent"
|
|
49
|
+
- `laravel-cloud` — guides deployment and infrastructure management on Laravel Cloud via the `cloud` CLI. Triggers automatically when you ask about deploying. Or use `/sk:laravel-deploy` for a gate-enforced deploy workflow.
|
|
50
|
+
|
|
51
|
+
If no, skip — plugins can be installed manually at any time.
|
|
52
|
+
|
|
35
53
|
## After Running
|
|
36
54
|
|
|
37
55
|
Start with `/sk:brainstorm` to plan your feature.
|
|
@@ -176,6 +176,24 @@ composer require --dev pestphp/pest pestphp/pest-plugin-laravel
|
|
|
176
176
|
./vendor/bin/pest --init
|
|
177
177
|
```
|
|
178
178
|
|
|
179
|
+
### Laravel Official Plugins
|
|
180
|
+
|
|
181
|
+
After tool installation, suggest the two official Laravel plugins from Taylor Otwell:
|
|
182
|
+
|
|
183
|
+
> "Install official Laravel plugins? (`laravel-simplifier` for PHP code refinement + `laravel-cloud` for deployments) [y/n]"
|
|
184
|
+
|
|
185
|
+
If yes, install each — skip any already configured:
|
|
186
|
+
```bash
|
|
187
|
+
/plugin install laravel-simplifier@laravel
|
|
188
|
+
/plugin install laravel-cloud@laravel
|
|
189
|
+
```
|
|
190
|
+
|
|
191
|
+
**What each adds:**
|
|
192
|
+
- `laravel-simplifier` — an Opus-powered agent that reviews recently modified PHP/Laravel code and refines it for PSR-12 compliance, Laravel conventions, and readability without changing behavior. Invoke after `/sk:execute-plan`: "Review recent changes using the laravel-simplifier agent"
|
|
193
|
+
- `laravel-cloud` — guides deployment and infrastructure management on Laravel Cloud via the `cloud` CLI. Triggers automatically when you ask about deploying. Also available via `/sk:laravel-deploy` for a gate-enforced workflow.
|
|
194
|
+
|
|
195
|
+
If no, skip — plugins can be installed manually at any time.
|
|
196
|
+
|
|
179
197
|
### Config Publishing (create-if-missing only)
|
|
180
198
|
|
|
181
199
|
**`phpstan.neon`:**
|
|
@@ -254,6 +272,23 @@ Use Agent tool with subagent_type="Explore" — launch all in a single message:
|
|
|
254
272
|
- Agent 3: Explore test patterns and existing test coverage for the area
|
|
255
273
|
```
|
|
256
274
|
|
|
275
|
+
### Code Refinement (after /sk:execute-plan)
|
|
276
|
+
|
|
277
|
+
After implementing code, run the `laravel-simplifier` agent to refine recently modified PHP/Laravel code:
|
|
278
|
+
|
|
279
|
+
```
|
|
280
|
+
Invoke the laravel-simplifier agent:
|
|
281
|
+
"Review recent changes using the laravel-simplifier agent"
|
|
282
|
+
```
|
|
283
|
+
|
|
284
|
+
This agent (Opus-powered, official from Taylor Otwell):
|
|
285
|
+
- Applies PSR-12 standards and Laravel conventions
|
|
286
|
+
- Reduces unnecessary nesting — prefers `match` over nested ternaries
|
|
287
|
+
- Improves variable and method naming
|
|
288
|
+
- Never changes behavior — refine only
|
|
289
|
+
|
|
290
|
+
Requires: `/plugin install laravel-simplifier@laravel`
|
|
291
|
+
|
|
257
292
|
### Parallel Quality Checks (/sk:lint)
|
|
258
293
|
|
|
259
294
|
After Pint formats files, run PHPStan and Rector in parallel (both are read-only):
|
|
@@ -309,6 +344,16 @@ When a generated CLAUDE.md exists (has `<!-- Generated by /sk:setup-claude -->`
|
|
|
309
344
|
|
|
310
345
|
This check runs every time — even if tools are installed and tasks files exist. Never short-circuit before verifying section completeness.
|
|
311
346
|
|
|
347
|
+
### Laravel Commands (CLAUDE.md Additions)
|
|
348
|
+
|
|
349
|
+
When generating or updating a CLAUDE.md for a Laravel project, append these commands to the Commands table (in addition to the standard ShipKit commands):
|
|
350
|
+
|
|
351
|
+
| Command | Purpose |
|
|
352
|
+
|---------|---------|
|
|
353
|
+
| `/sk:laravel-deploy` | Deploy to Laravel Cloud (gates must pass first) |
|
|
354
|
+
| `/sk:laravel-init` | Configure existing Laravel project with production-ready conventions |
|
|
355
|
+
| `/sk:laravel-new` | Scaffold a fresh Laravel app with production-ready conventions |
|
|
356
|
+
|
|
312
357
|
### Laravel Idempotency (extends global rules)
|
|
313
358
|
|
|
314
359
|
**Never overwrite** (in addition to global list):
|
|
@@ -6,6 +6,13 @@ allowed-tools: Read, Edit, Bash, Grep, Glob
|
|
|
6
6
|
memory: project
|
|
7
7
|
---
|
|
8
8
|
|
|
9
|
+
<!-- DESIGN NOTE: No `isolation: worktree` by design.
|
|
10
|
+
Debugger is invoked solo (never in parallel with other agents) and must
|
|
11
|
+
see the actual working state — the real failing code, the real test output,
|
|
12
|
+
the real stack trace. A worktree copy would give it a clean slate,
|
|
13
|
+
defeating its ability to reproduce the bug. Edit access is needed to place
|
|
14
|
+
and remove targeted debug logs during investigation. -->
|
|
15
|
+
|
|
9
16
|
# Debugger Agent
|
|
10
17
|
|
|
11
18
|
You are an expert debugger. Find root causes, not symptoms.
|
|
@@ -7,6 +7,14 @@ memory: project
|
|
|
7
7
|
background: true
|
|
8
8
|
---
|
|
9
9
|
|
|
10
|
+
<!-- DESIGN NOTE: No `isolation: worktree` by design.
|
|
11
|
+
qa-engineer runs as a background agent (background: true) alongside
|
|
12
|
+
backend-dev and frontend-dev during sk:team. It only creates test scenario
|
|
13
|
+
files (e.g., e2e/<feature>.spec.ts, tasks/e2e-scenarios.md) — never source
|
|
14
|
+
files. Its writes land in a separate directory tree from the implementation
|
|
15
|
+
agents' worktree changes, so file conflicts cannot occur. Isolation adds
|
|
16
|
+
overhead with no benefit here. -->
|
|
17
|
+
|
|
10
18
|
# QA Engineer Agent
|
|
11
19
|
|
|
12
20
|
You are the QA Agent in a team workflow. Your job is to write E2E test scenarios while the Backend and Frontend agents implement code in parallel.
|
|
@@ -6,6 +6,13 @@ tools: Read, Write, Edit, Grep, Glob
|
|
|
6
6
|
memory: project
|
|
7
7
|
---
|
|
8
8
|
|
|
9
|
+
<!-- DESIGN NOTE: No `isolation: worktree` by design.
|
|
10
|
+
tech-writer only creates and edits documentation files (README.md,
|
|
11
|
+
docs/*.md, architecture docs, inline comments). It never modifies source
|
|
12
|
+
code, migrations, or config files. Because its writes are confined to
|
|
13
|
+
documentation paths that implementation agents never touch, isolation is
|
|
14
|
+
unnecessary overhead. -->
|
|
15
|
+
|
|
9
16
|
You are a technical writer specializing in developer documentation. You make codebases comprehensible — to future contributors, to users, and to the developers themselves six months later.
|
|
10
17
|
|
|
11
18
|
## On Invocation
|
|
@@ -18,3 +18,14 @@ paths:
|
|
|
18
18
|
- **Migrations**: One logical change per migration. Never modify a published migration.
|
|
19
19
|
- **Config**: Access config via `config()` helper, never `env()` outside config files.
|
|
20
20
|
- **Strict mode**: Models use strict mode (prevent lazy loading, silently discarding attributes, accessing missing attributes).
|
|
21
|
+
|
|
22
|
+
## Code Refinement
|
|
23
|
+
|
|
24
|
+
After running `/sk:execute-plan`, use the `laravel-simplifier` agent to refine recently modified PHP/Laravel code:
|
|
25
|
+
- Applies PSR-12 standards and Laravel conventions
|
|
26
|
+
- Reduces unnecessary nesting and complexity — prefer `match` over nested ternaries
|
|
27
|
+
- Improves variable and method naming for readability
|
|
28
|
+
- Never changes behavior — refine only
|
|
29
|
+
|
|
30
|
+
Install once: `/plugin install laravel-simplifier@laravel`
|
|
31
|
+
Invoke: "Review recent changes using the laravel-simplifier agent"
|
|
@@ -51,9 +51,12 @@ Progress is tracked via git branch + `tasks/todo.md` checkboxes.
|
|
|
51
51
|
| 3 | Plan | `/sk:write-plan` | required |
|
|
52
52
|
| 4 | Branch | `/sk:branch` | required |
|
|
53
53
|
| 5 | Write Tests + Implement | `/sk:write-tests` then `/sk:execute-plan` | required |
|
|
54
|
+
| 5.5 | Scope Check | `/sk:scope-check` | required |
|
|
54
55
|
| 6 | Commit | `/sk:smart-commit` | required |
|
|
55
56
|
| 7 | Gates | `/sk:gates` | required (hard gate) |
|
|
56
57
|
| 8 | Finalize | `/sk:finish-feature` | required |
|
|
58
|
+
| 8.5 | Learn | `/sk:learn` | required |
|
|
59
|
+
| 8.6 | Retro | `/sk:retro` | required |
|
|
57
60
|
|
|
58
61
|
### Step Details
|
|
59
62
|
|
|
@@ -62,9 +65,12 @@ Progress is tracked via git branch + `tasks/todo.md` checkboxes.
|
|
|
62
65
|
3. **Plan** — run `/sk:write-plan` to write a decision-complete plan into `tasks/todo.md`. No code in this step. After the plan is written, auto-skip detection runs for step 2 if not already done.
|
|
63
66
|
4. **Branch** — run `/sk:branch` to create a feature branch auto-named from the current task.
|
|
64
67
|
5. **Write Tests + Implement** — run `/sk:write-tests` (TDD red phase), then `/sk:execute-plan` (TDD green phase). Includes `/sk:schema-migrate` if database keywords detected in the plan. Log progress to `tasks/progress.md`.
|
|
68
|
+
5.5. **Scope Check** — run `/sk:scope-check` to compare implementation against `tasks/todo.md`. Trim scope creep before committing.
|
|
65
69
|
6. **Commit** — run `/sk:smart-commit` to commit tests + implementation.
|
|
66
70
|
7. **Gates** — run `/sk:gates` to execute all quality gates in optimized parallel batches (lint, test, security, perf, review, e2e). This is a **hard gate** — blocks all forward progress until every check passes. Individual gate commands (`/sk:lint`, `/sk:test`, `/sk:security-check`, `/sk:perf`, `/sk:review`, `/sk:e2e`) are still available standalone.
|
|
67
71
|
8. **Finalize** — run `/sk:finish-feature` for changelog, PR creation, `/sk:update-task`, `/sk:features` sync. Ask about `/sk:release` (never auto-skipped).
|
|
72
|
+
8.5. **Learn** — run `/sk:learn` to extract reusable patterns from the session into `~/.claude/skills/learned/`.
|
|
73
|
+
8.6. **Retro** — run `/sk:retro` for a brief post-ship retrospective (velocity, blockers, next actions).
|
|
68
74
|
|
|
69
75
|
### Workflow Rules
|
|
70
76
|
|
|
@@ -286,7 +292,7 @@ Create entries in: `[ARCH_CHANGELOG_DIR]`
|
|
|
286
292
|
|---------|---------|
|
|
287
293
|
| `/sk:accessibility` | WCAG 2.1 AA audit — runs after design, before implementation |
|
|
288
294
|
| `/sk:api-design` | Design API contracts (endpoints, payloads, auth, errors) before implementation |
|
|
289
|
-
| `/sk:autopilot` | Hands-free workflow — all 8
|
|
295
|
+
| `/sk:autopilot` | Hands-free workflow — all 8 phases (scope check, learn, retro included), auto-skip, auto-advance, auto-commit |
|
|
290
296
|
| `/sk:brainstorm` | Explore requirements and design (includes search-first research) |
|
|
291
297
|
| `/sk:branch` | Create feature branch auto-named from current task |
|
|
292
298
|
| `/sk:change` | Handle mid-workflow requirement changes — re-enter at correct step |
|
|
@@ -304,6 +310,9 @@ Create entries in: `[ARCH_CHANGELOG_DIR]`
|
|
|
304
310
|
| `/sk:gates` | Run all quality gates in optimized parallel batches |
|
|
305
311
|
| `/sk:health` | Harness self-audit scorecard (7 categories, 0-70) |
|
|
306
312
|
| `/sk:hotfix` | Emergency fix workflow — skip design/TDD, quality gates enforced |
|
|
313
|
+
| `/sk:laravel-deploy` | Deploy to Laravel Cloud (gates must pass first) — Laravel projects only |
|
|
314
|
+
| `/sk:laravel-init` | Configure existing Laravel project with production-ready conventions |
|
|
315
|
+
| `/sk:laravel-new` | Scaffold a fresh Laravel app with production-ready conventions |
|
|
307
316
|
| `/sk:learn` | Extract reusable patterns from sessions into learned instincts |
|
|
308
317
|
| `/sk:lint` | Auto-detect and run all project linters + dependency audits |
|
|
309
318
|
| `/sk:perf` | Performance audit — bundle, N+1, Core Web Vitals, memory |
|
|
@@ -44,7 +44,7 @@ Before making any changes, runs a diagnostic pass on the existing CLAUDE.md:
|
|
|
44
44
|
- **Stale content** — detects outdated info (stale model/route counts, removed dependencies, old command names like `/laravel-lint` instead of `/sk:lint`)
|
|
45
45
|
- **Inconsistencies** — compares documented vs actual project state (directories, scripts, workflows)
|
|
46
46
|
- **Section completeness** — flags sections that exist but are empty or have only placeholder text
|
|
47
|
-
- **Outdated workflow** — checks if the workflow matches the current
|
|
47
|
+
- **Outdated workflow** — checks if the workflow matches the current 11-step flow (1, 2, 3, 4, 5, 5.5, 6, 7, 8, 8.5, 8.6) with `/sk:gates` as single gate step
|
|
48
48
|
- **Missing commands** — checks for `sk:start`, `sk:autopilot`, `sk:team`, `sk:learn`, `sk:context-budget`, `sk:health`, `sk:save-session`, `sk:resume-session`, `sk:safety-guard`, `sk:eval`, `sk:ci`, `sk:plugin` in the Commands table
|
|
49
49
|
- **Missing agents** — checks if `.claude/agents/` exists and contains the 13 core agents: `backend-dev`, `frontend-dev`, `mobile-dev`, `qa-engineer`, `code-reviewer`, `security-reviewer`, `performance-optimizer`, `architect`, `database-architect`, `devops-engineer`, `debugger`, `refactor-specialist`, `tech-writer`
|
|
50
50
|
- **Missing rules** — checks if `.claude/rules/` exists and contains the project-relevant rule files based on detected stack (laravel.md, react.md, vue.md, tests.md, api.md, migrations.md)
|
|
@@ -59,13 +59,13 @@ Reports findings before proceeding. If issues are found, they inform subsequent
|
|
|
59
59
|
|
|
60
60
|
If the workflow section is outdated or missing, replace it with the latest version:
|
|
61
61
|
|
|
62
|
-
**Current workflow (
|
|
62
|
+
**Current workflow (11 steps, TDD with `/sk:gates` as single gate step):**
|
|
63
63
|
```
|
|
64
|
-
Explore → Design → Plan → Branch → Write Tests + Implement → Commit → Gates → Finalize
|
|
64
|
+
Explore → Design → Plan → Branch → Write Tests + Implement → Scope Check → Commit → Gates → Finalize + Learn + Retro
|
|
65
65
|
```
|
|
66
66
|
|
|
67
67
|
**What gets updated:**
|
|
68
|
-
- Workflow table (
|
|
68
|
+
- Workflow table (11 steps — `/sk:brainstorm`, `/sk:frontend-design` or `/sk:api-design`, `/sk:write-plan`, `/sk:branch`, `/sk:write-tests` + `/sk:execute-plan`, `/sk:scope-check`, `/sk:smart-commit`, `/sk:gates`, `/sk:finish-feature`, `/sk:learn`, `/sk:retro`)
|
|
69
69
|
- Step details (TDD red/green/verify descriptions)
|
|
70
70
|
- Workflow rules (auto-advance, conditional summary, auto-skip, squash gate commits)
|
|
71
71
|
- Bug fix flow section (7 steps)
|
package/skills/sk:start/SKILL.md
CHANGED
|
@@ -32,7 +32,7 @@ Read the task description from arguments. Scan for signal keywords to determine
|
|
|
32
32
|
| bug, fix, broken, error, regression, failing, crash, wrong | `debug` (7 steps) |
|
|
33
33
|
| urgent, prod down, hotfix, emergency, critical, production, incident | `hotfix` (6 steps) |
|
|
34
34
|
| config, bump, typo, copy, rename, dependency, upgrade, version, docs | `fast-track` (5 steps) |
|
|
35
|
-
| *(default — no special signals)* | `feature` (8
|
|
35
|
+
| *(default — no special signals)* | `feature` (8 phases + scope check, learn, retro) |
|
|
36
36
|
|
|
37
37
|
**Scope detection:**
|
|
38
38
|
|
|
@@ -84,7 +84,7 @@ Present the classification and recommendation:
|
|
|
84
84
|
```
|
|
85
85
|
Detected: [Full-stack feature / Backend bug fix / Frontend hotfix / Small config change / etc.]
|
|
86
86
|
Recommended:
|
|
87
|
-
Flow: [feature (8
|
|
87
|
+
Flow: [feature (8 phases) / debug (7 steps) / hotfix (6 steps) / fast-track (5 steps)]
|
|
88
88
|
Mode: [autopilot / manual]
|
|
89
89
|
Agents: [team (backend + frontend + QA) / solo]
|
|
90
90
|
|
package/skills/sk:team/SKILL.md
CHANGED
|
@@ -106,7 +106,7 @@ If both agents used worktree isolation and made changes:
|
|
|
106
106
|
|
|
107
107
|
### 5. Collect QA Agent Results
|
|
108
108
|
|
|
109
|
-
Collect the QA Agent's E2E scenarios. These will be used in the E2E gate (step
|
|
109
|
+
Collect the QA Agent's E2E scenarios. These will be used in the E2E gate (step 7 / Batch 4).
|
|
110
110
|
|
|
111
111
|
### 6. Report Results
|
|
112
112
|
|