sequant 1.0.0 → 1.1.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 +12 -8
- package/dist/bin/cli.js +12 -0
- package/dist/bin/cli.js.map +1 -1
- package/dist/src/commands/doctor.d.ts.map +1 -1
- package/dist/src/commands/doctor.js +46 -0
- package/dist/src/commands/doctor.js.map +1 -1
- package/dist/src/commands/doctor.test.d.ts +2 -0
- package/dist/src/commands/doctor.test.d.ts.map +1 -0
- package/dist/src/commands/doctor.test.js +140 -0
- package/dist/src/commands/doctor.test.js.map +1 -0
- package/dist/src/commands/init.d.ts.map +1 -1
- package/dist/src/commands/init.js +45 -2
- package/dist/src/commands/init.js.map +1 -1
- package/dist/src/commands/init.test.d.ts +2 -0
- package/dist/src/commands/init.test.d.ts.map +1 -0
- package/dist/src/commands/init.test.js +152 -0
- package/dist/src/commands/init.test.js.map +1 -0
- package/dist/src/commands/logs.d.ts +18 -0
- package/dist/src/commands/logs.d.ts.map +1 -0
- package/dist/src/commands/logs.js +188 -0
- package/dist/src/commands/logs.js.map +1 -0
- package/dist/src/commands/run.d.ts +2 -0
- package/dist/src/commands/run.d.ts.map +1 -1
- package/dist/src/commands/run.js +114 -29
- package/dist/src/commands/run.js.map +1 -1
- package/dist/src/lib/stacks.d.ts.map +1 -1
- package/dist/src/lib/stacks.js +39 -0
- package/dist/src/lib/stacks.js.map +1 -1
- package/dist/src/lib/stacks.test.d.ts +2 -0
- package/dist/src/lib/stacks.test.d.ts.map +1 -0
- package/dist/src/lib/stacks.test.js +145 -0
- package/dist/src/lib/stacks.test.js.map +1 -0
- package/dist/src/lib/system.d.ts +16 -0
- package/dist/src/lib/system.d.ts.map +1 -0
- package/dist/src/lib/system.js +52 -0
- package/dist/src/lib/system.js.map +1 -0
- package/dist/src/lib/system.test.d.ts +2 -0
- package/dist/src/lib/system.test.d.ts.map +1 -0
- package/dist/src/lib/system.test.js +80 -0
- package/dist/src/lib/system.test.js.map +1 -0
- package/dist/src/lib/workflow/log-writer.d.ts +83 -0
- package/dist/src/lib/workflow/log-writer.d.ts.map +1 -0
- package/dist/src/lib/workflow/log-writer.js +193 -0
- package/dist/src/lib/workflow/log-writer.js.map +1 -0
- package/dist/src/lib/workflow/run-log-schema.d.ts +261 -0
- package/dist/src/lib/workflow/run-log-schema.d.ts.map +1 -0
- package/dist/src/lib/workflow/run-log-schema.js +234 -0
- package/dist/src/lib/workflow/run-log-schema.js.map +1 -0
- package/package.json +6 -4
- package/stacks/astro.yaml +35 -0
- package/templates/hooks/post-tool.sh +0 -11
- package/templates/hooks/pre-tool.sh +2 -2
- package/templates/memory/constitution.md +8 -0
- package/templates/scripts/cleanup-worktree.sh +1 -1
- package/templates/scripts/new-feature.sh +7 -5
- package/templates/skills/assess/SKILL.md +31 -16
- package/templates/skills/clean/SKILL.md +17 -2
- package/templates/skills/docs/SKILL.md +48 -34
- package/templates/skills/exec/SKILL.md +31 -25
- package/templates/skills/fullsolve/SKILL.md +34 -16
- package/templates/skills/loop/SKILL.md +22 -5
- package/templates/skills/qa/SKILL.md +89 -4
- package/templates/skills/qa/references/code-quality-exemplars.md +23 -28
- package/templates/skills/qa/references/code-review-checklist.md +6 -17
- package/templates/skills/qa/scripts/quality-checks.sh +4 -17
- package/templates/skills/reflect/SKILL.md +18 -2
- package/templates/skills/reflect/references/documentation-tiers.md +3 -3
- package/templates/skills/security-review/SKILL.md +15 -0
- package/templates/skills/security-review/references/security-checklists.md +10 -8
- package/templates/skills/solve/SKILL.md +147 -149
- package/templates/skills/spec/SKILL.md +61 -3
- package/templates/skills/spec/references/parallel-groups.md +1 -1
- package/templates/skills/spec/references/verification-criteria.md +1 -1
- package/templates/skills/test/SKILL.md +20 -5
- package/templates/skills/testgen/SKILL.md +15 -1
- package/templates/skills/verify/SKILL.md +20 -5
- package/templates/skills/reflect/scripts/workflow-queries.ts +0 -165
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: solve
|
|
3
|
-
description: "Generate the
|
|
3
|
+
description: "Generate the recommended workflow for one or more GitHub issues"
|
|
4
4
|
license: MIT
|
|
5
5
|
metadata:
|
|
6
6
|
author: sequant
|
|
@@ -18,68 +18,33 @@ You are the "Solve Command Generator" for the current repository.
|
|
|
18
18
|
When invoked as `/solve <issue-numbers>`, your job is to:
|
|
19
19
|
|
|
20
20
|
1. Analyze the provided issue number(s)
|
|
21
|
-
2. Check
|
|
22
|
-
3.
|
|
23
|
-
4. Display the
|
|
21
|
+
2. Check labels to determine issue type
|
|
22
|
+
3. Recommend the optimal workflow sequence
|
|
23
|
+
4. Display the recommended commands
|
|
24
24
|
|
|
25
25
|
## Behavior
|
|
26
26
|
|
|
27
27
|
### Invocation Formats
|
|
28
28
|
|
|
29
29
|
- `/solve 152` - Single issue
|
|
30
|
-
- `/solve 152 153 154` - Multiple issues
|
|
31
|
-
- `/solve --batch "152 153" "154 155"` - Sequential batches
|
|
30
|
+
- `/solve 152 153 154` - Multiple issues
|
|
32
31
|
|
|
33
32
|
### Detection Logic
|
|
34
33
|
|
|
35
|
-
For each issue, check GitHub labels to determine
|
|
34
|
+
For each issue, check GitHub labels to determine the workflow:
|
|
36
35
|
|
|
37
36
|
```bash
|
|
38
37
|
gh issue view <issue-number> --json labels --jq '.labels[].name'
|
|
39
38
|
```
|
|
40
39
|
|
|
41
|
-
**UI
|
|
42
|
-
- `
|
|
43
|
-
- `ui`
|
|
44
|
-
- `frontend`
|
|
40
|
+
**UI/Frontend Issues** (labels: `ui`, `frontend`, `admin`):
|
|
41
|
+
- Include `/test` phase for browser testing
|
|
45
42
|
|
|
46
|
-
**Backend Issues** (
|
|
47
|
-
-
|
|
43
|
+
**Backend/API Issues** (labels: `backend`, `api`, `cli`):
|
|
44
|
+
- Skip browser testing, focus on unit/integration tests
|
|
48
45
|
|
|
49
|
-
|
|
50
|
-
|
|
51
|
-
**Single Issue:**
|
|
52
|
-
```bash
|
|
53
|
-
# UI issue (has admin/ui/frontend label)
|
|
54
|
-
PHASES=spec,exec,test,qa npx tsx --env-file=.env.local scripts/dev/execute-issues.ts 152
|
|
55
|
-
|
|
56
|
-
# Backend issue (no UI label)
|
|
57
|
-
npx tsx --env-file=.env.local scripts/dev/execute-issues.ts 152
|
|
58
|
-
```
|
|
59
|
-
|
|
60
|
-
**Multiple Issues (Parallel):**
|
|
61
|
-
```bash
|
|
62
|
-
# All backend issues
|
|
63
|
-
npx tsx --env-file=.env.local scripts/dev/execute-issues.ts 152 153 154
|
|
64
|
-
|
|
65
|
-
# Mixed (some UI, some backend)
|
|
66
|
-
PHASES=spec,exec,test,qa npx tsx --env-file=.env.local scripts/dev/execute-issues.ts 152 153 154
|
|
67
|
-
|
|
68
|
-
# Note: PHASES env var applies to ALL issues
|
|
69
|
-
# If ANY issue needs /test, add it for all
|
|
70
|
-
```
|
|
71
|
-
|
|
72
|
-
**Sequential Batches (Dependency-Aware):**
|
|
73
|
-
```bash
|
|
74
|
-
# Run issues sequentially (respects dependencies)
|
|
75
|
-
npx tsx --env-file=.env.local scripts/dev/execute-issues.ts --sequential 152 153 154
|
|
76
|
-
|
|
77
|
-
# Run batch 1, then batch 2
|
|
78
|
-
npx tsx --env-file=.env.local scripts/dev/execute-issues.ts --batch "152 153" --batch "154 155"
|
|
79
|
-
|
|
80
|
-
# With custom phases
|
|
81
|
-
PHASES=spec,exec,test,qa npx tsx --env-file=.env.local scripts/dev/execute-issues.ts --batch "152 153" --batch "154 155"
|
|
82
|
-
```
|
|
46
|
+
**Bug Fixes** (labels: `bug`, `fix`):
|
|
47
|
+
- May need simpler workflow, possibly skip `/spec`
|
|
83
48
|
|
|
84
49
|
## Output Format
|
|
85
50
|
|
|
@@ -89,154 +54,187 @@ Provide a clear, actionable response with:
|
|
|
89
54
|
- Issue number
|
|
90
55
|
- Title
|
|
91
56
|
- Labels
|
|
92
|
-
-
|
|
57
|
+
- Recommended workflow
|
|
58
|
+
|
|
59
|
+
2. **Recommended Commands** in order
|
|
93
60
|
|
|
94
|
-
|
|
61
|
+
3. **CLI Command** - ALWAYS include `sequant run <issue>` for terminal/CI usage
|
|
95
62
|
|
|
96
|
-
|
|
63
|
+
4. **Explanation** of why this workflow was chosen
|
|
97
64
|
|
|
98
65
|
### Example Output
|
|
99
66
|
|
|
100
67
|
```markdown
|
|
101
|
-
## Solve
|
|
68
|
+
## Solve Workflow for Issues: 152, 153
|
|
102
69
|
|
|
103
70
|
### Issue Analysis
|
|
104
71
|
|
|
105
|
-
| Issue | Title | Labels |
|
|
106
|
-
|
|
107
|
-
| #152 |
|
|
108
|
-
| #153 |
|
|
109
|
-
| #154 | City onboarding UI | admin, ui | Yes |
|
|
72
|
+
| Issue | Title | Labels | Workflow |
|
|
73
|
+
|-------|-------|--------|----------|
|
|
74
|
+
| #152 | Add user dashboard | ui, enhancement | Full (with /test) |
|
|
75
|
+
| #153 | Fix API validation | backend, bug | Standard |
|
|
110
76
|
|
|
111
|
-
### Recommended
|
|
77
|
+
### Recommended Workflow
|
|
112
78
|
|
|
113
|
-
|
|
79
|
+
**For #152 (UI feature):**
|
|
80
|
+
```bash
|
|
81
|
+
/spec 152 # Plan the implementation
|
|
82
|
+
/exec 152 # Implement the feature
|
|
83
|
+
/test 152 # Browser-based UI testing
|
|
84
|
+
/qa # Quality review
|
|
85
|
+
```
|
|
114
86
|
|
|
115
|
-
|
|
116
|
-
|
|
117
|
-
|
|
87
|
+
**For #153 (Backend fix):**
|
|
88
|
+
```bash
|
|
89
|
+
/spec 153 # Quick plan (bug fix)
|
|
90
|
+
/exec 153 # Implement the fix
|
|
91
|
+
/qa # Quality review
|
|
92
|
+
```
|
|
118
93
|
|
|
119
|
-
###
|
|
94
|
+
### Full Workflow Option
|
|
120
95
|
|
|
121
|
-
|
|
122
|
-
|
|
123
|
-
|
|
96
|
+
For comprehensive quality with automatic fix iterations:
|
|
97
|
+
```bash
|
|
98
|
+
/fullsolve 152
|
|
99
|
+
```
|
|
124
100
|
|
|
125
|
-
###
|
|
101
|
+
### CLI Command
|
|
126
102
|
|
|
127
|
-
|
|
103
|
+
Run from terminal (useful for automation/CI):
|
|
104
|
+
```bash
|
|
105
|
+
sequant run 152 # Single issue
|
|
106
|
+
sequant run 152 153 # Multiple issues
|
|
107
|
+
```
|
|
128
108
|
|
|
129
|
-
|
|
130
|
-
|
|
131
|
-
|
|
109
|
+
### Notes
|
|
110
|
+
- Issue #152 requires UI testing due to `ui` label
|
|
111
|
+
- Issue #153 is a bug fix - simpler workflow recommended
|
|
112
|
+
- Run `/qa` after each issue before moving to next
|
|
113
|
+
```
|
|
132
114
|
|
|
133
|
-
|
|
115
|
+
## Workflow Selection Logic
|
|
134
116
|
|
|
135
|
-
###
|
|
117
|
+
### Standard Workflow (Most Issues)
|
|
118
|
+
```
|
|
119
|
+
/spec → /exec → /qa
|
|
120
|
+
```
|
|
136
121
|
|
|
137
|
-
|
|
122
|
+
### UI Feature Workflow
|
|
123
|
+
```
|
|
124
|
+
/spec → /exec → /test → /qa
|
|
125
|
+
```
|
|
138
126
|
|
|
139
|
-
|
|
140
|
-
|
|
141
|
-
|
|
127
|
+
### Bug Fix Workflow (Simple)
|
|
128
|
+
```
|
|
129
|
+
/exec → /qa
|
|
130
|
+
```
|
|
131
|
+
Skip `/spec` if the bug is well-defined and straightforward.
|
|
142
132
|
|
|
143
|
-
###
|
|
133
|
+
### Complex Feature Workflow
|
|
134
|
+
```
|
|
135
|
+
/fullsolve <issue>
|
|
136
|
+
```
|
|
137
|
+
Runs complete workflow with automatic fix iterations.
|
|
144
138
|
|
|
145
|
-
|
|
139
|
+
## Quick Reference
|
|
146
140
|
|
|
147
|
-
|
|
148
|
-
|
|
149
|
-
|
|
150
|
-
|
|
141
|
+
| Issue Type | Labels | Workflow |
|
|
142
|
+
|------------|--------|----------|
|
|
143
|
+
| UI Feature | ui, frontend, admin | spec → exec → test → qa |
|
|
144
|
+
| Backend Feature | backend, api | spec → exec → qa |
|
|
145
|
+
| Bug Fix | bug, fix | exec → qa (or full if complex) |
|
|
146
|
+
| Complex Feature | complex, refactor | fullsolve |
|
|
147
|
+
| Documentation | docs | exec → qa |
|
|
151
148
|
|
|
152
|
-
|
|
149
|
+
## CLI Alternative
|
|
153
150
|
|
|
154
|
-
|
|
155
|
-
npx tsx --env-file=.env.local scripts/dev/execute-issues.ts 152 153 154
|
|
156
|
-
\`\`\`
|
|
151
|
+
Use `sequant run` for batch execution from the command line:
|
|
157
152
|
|
|
158
|
-
|
|
159
|
-
|
|
153
|
+
```bash
|
|
154
|
+
# Run workflow for single issue
|
|
155
|
+
sequant run 152
|
|
160
156
|
|
|
161
|
-
|
|
157
|
+
# Multiple issues in parallel
|
|
158
|
+
sequant run 152 153 154
|
|
162
159
|
|
|
163
|
-
|
|
164
|
-
|
|
165
|
-
3. **Analyze labels**: Check for admin/ui/frontend labels
|
|
166
|
-
4. **Determine phases**:
|
|
167
|
-
- If ANY issue has UI label → use `PHASES=spec,exec,test,qa`
|
|
168
|
-
- If ALL issues are backend → use default phases (no PHASES env var)
|
|
169
|
-
5. **Generate command**: Format based on number of issues and batch requirements
|
|
170
|
-
6. **Display output**: Show issue table + recommended command + explanation
|
|
160
|
+
# Sequential execution (respects dependencies)
|
|
161
|
+
sequant run 152 153 --sequential
|
|
171
162
|
|
|
172
|
-
|
|
163
|
+
# Custom phases
|
|
164
|
+
sequant run 152 --phases spec,exec,qa
|
|
173
165
|
|
|
174
|
-
|
|
175
|
-
|
|
176
|
-
npx tsx --env-file=.env.local scripts/dev/execute-issues.ts 145 146 147
|
|
166
|
+
# Dry run (shows what would execute)
|
|
167
|
+
sequant run 152 --dry-run
|
|
177
168
|
```
|
|
178
|
-
No `PHASES` env var needed - default is `spec,exec,qa`
|
|
179
169
|
|
|
180
|
-
|
|
181
|
-
|
|
182
|
-
|
|
170
|
+
## Edge Cases
|
|
171
|
+
|
|
172
|
+
### Multiple Issues with Different Types
|
|
173
|
+
|
|
174
|
+
When solving multiple issues with mixed types, recommend running them separately:
|
|
175
|
+
|
|
176
|
+
```markdown
|
|
177
|
+
These issues have different requirements. Run separately:
|
|
178
|
+
|
|
179
|
+
**UI Issues (with browser testing):**
|
|
180
|
+
/fullsolve 152
|
|
181
|
+
|
|
182
|
+
**Backend Issues:**
|
|
183
|
+
/fullsolve 153
|
|
183
184
|
```
|
|
184
185
|
|
|
185
|
-
###
|
|
186
|
-
**Recommendation**: Use `PHASES=spec,exec,test,qa` for consistency, but warn user:
|
|
187
|
-
> Note: Issue #153 is a backend issue and doesn't need `/test`, but we're including it for consistency. If you want to skip `/test` for #153, run it separately.
|
|
186
|
+
### Issue with Dependencies
|
|
188
187
|
|
|
189
|
-
|
|
190
|
-
User types: `/solve --batch "152 153" "154"`
|
|
188
|
+
If issues depend on each other:
|
|
191
189
|
|
|
192
|
-
|
|
193
|
-
|
|
194
|
-
|
|
190
|
+
```markdown
|
|
191
|
+
⚠️ Issue #154 depends on #153. Run in order:
|
|
192
|
+
|
|
193
|
+
1. /fullsolve 153
|
|
194
|
+
2. (Wait for PR merge)
|
|
195
|
+
3. /fullsolve 154
|
|
195
196
|
```
|
|
196
197
|
|
|
197
|
-
|
|
198
|
+
---
|
|
198
199
|
|
|
199
|
-
|
|
200
|
+
## Output Verification
|
|
200
201
|
|
|
201
|
-
|
|
202
|
-
2. **Issues with many ACs** - More acceptance criteria = more chances for partial implementation
|
|
203
|
-
3. **New feature implementations** - First-time implementations may need iteration
|
|
204
|
-
4. **User requests "best quality"** - Explicit quality preference
|
|
202
|
+
**Before responding, verify your output includes ALL of these:**
|
|
205
203
|
|
|
206
|
-
**
|
|
207
|
-
-
|
|
208
|
-
-
|
|
209
|
-
-
|
|
204
|
+
- [ ] **Issue Summary Table** - Table with Issue, Title, Labels, Workflow columns
|
|
205
|
+
- [ ] **Recommended Workflow** - Slash commands in order for each issue
|
|
206
|
+
- [ ] **CLI Command** - `sequant run <issue-numbers>` command (REQUIRED)
|
|
207
|
+
- [ ] **Explanation** - Brief notes explaining workflow choices
|
|
210
208
|
|
|
211
|
-
|
|
209
|
+
**DO NOT respond until all items are verified.**
|
|
212
210
|
|
|
213
|
-
|
|
211
|
+
## Output Template
|
|
214
212
|
|
|
215
|
-
|
|
216
|
-
- Catches regressions immediately (5-10s overhead per edit)
|
|
217
|
-
- Results logged to `/tmp/claude-tests.log`
|
|
213
|
+
You MUST use this exact structure:
|
|
218
214
|
|
|
219
|
-
|
|
220
|
-
|
|
221
|
-
- Issues with long-running test suites
|
|
222
|
-
- Simple documentation changes
|
|
215
|
+
```markdown
|
|
216
|
+
## Solve Workflow for Issues: <ISSUE_NUMBERS>
|
|
223
217
|
|
|
224
|
-
|
|
225
|
-
```bash
|
|
226
|
-
npx tsx scripts/dev/analyze-hook-logs.ts --tests
|
|
227
|
-
```
|
|
218
|
+
### Issue Analysis
|
|
228
219
|
|
|
229
|
-
|
|
220
|
+
| Issue | Title | Labels | Workflow |
|
|
221
|
+
|-------|-------|--------|----------|
|
|
222
|
+
<!-- FILL: one row per issue -->
|
|
230
223
|
|
|
231
|
-
|
|
232
|
-
|
|
233
|
-
|
|
234
|
-
|
|
235
|
-
|
|
236
|
-
|
|
237
|
-
|
|
238
|
-
|
|
239
|
-
|
|
240
|
-
|
|
241
|
-
|
|
242
|
-
|
|
224
|
+
### Recommended Workflow
|
|
225
|
+
|
|
226
|
+
**For #<N> (<type>):**
|
|
227
|
+
\`\`\`bash
|
|
228
|
+
<!-- FILL: slash commands in order -->
|
|
229
|
+
\`\`\`
|
|
230
|
+
|
|
231
|
+
### CLI Command
|
|
232
|
+
|
|
233
|
+
Run from terminal (useful for automation/CI):
|
|
234
|
+
\`\`\`bash
|
|
235
|
+
sequant run <ISSUE_NUMBERS>
|
|
236
|
+
\`\`\`
|
|
237
|
+
|
|
238
|
+
### Notes
|
|
239
|
+
<!-- FILL: explanation of workflow choices -->
|
|
240
|
+
```
|
|
@@ -60,11 +60,10 @@ Task(subagent_type="schema-inspector", model="haiku",
|
|
|
60
60
|
|
|
61
61
|
**Important:** Spawn all agents in a SINGLE message for parallel execution.
|
|
62
62
|
|
|
63
|
-
### Using MCP Tools
|
|
63
|
+
### Using MCP Tools (Optional)
|
|
64
64
|
|
|
65
65
|
- **Sequential Thinking:** For complex analysis with multiple dependencies
|
|
66
66
|
- **Context7:** For understanding existing patterns and architecture
|
|
67
|
-
- **Supabase MCP:** For database changes, queries, or data modeling
|
|
68
67
|
|
|
69
68
|
## Context Gathering Strategy
|
|
70
69
|
|
|
@@ -83,7 +82,7 @@ Task(subagent_type="schema-inspector", model="haiku",
|
|
|
83
82
|
- Prefer existing dependencies over new ones
|
|
84
83
|
|
|
85
84
|
4. **For database-heavy features**
|
|
86
|
-
-
|
|
85
|
+
- Verify table schemas against TypeScript types
|
|
87
86
|
- Check proposed types match database columns
|
|
88
87
|
|
|
89
88
|
5. **For complex features (>5 AC items)**
|
|
@@ -167,3 +166,62 @@ Post the draft comment to GitHub:
|
|
|
167
166
|
gh issue comment <issue-number> --body "..."
|
|
168
167
|
gh issue edit <issue-number> --add-label "planned"
|
|
169
168
|
```
|
|
169
|
+
|
|
170
|
+
---
|
|
171
|
+
|
|
172
|
+
## Output Verification
|
|
173
|
+
|
|
174
|
+
**Before responding, verify your output includes ALL of these:**
|
|
175
|
+
|
|
176
|
+
- [ ] **AC Checklist** - Numbered AC items (AC-1, AC-2, etc.) with descriptions
|
|
177
|
+
- [ ] **Verification Criteria** - Each AC has Verification Method and Test Scenario
|
|
178
|
+
- [ ] **Implementation Plan** - 3-7 concrete steps with codebase references
|
|
179
|
+
- [ ] **Open Questions** - Any ambiguities with recommended defaults
|
|
180
|
+
- [ ] **Issue Comment Draft** - Formatted for GitHub posting
|
|
181
|
+
|
|
182
|
+
**DO NOT respond until all items are verified.**
|
|
183
|
+
|
|
184
|
+
## Output Template
|
|
185
|
+
|
|
186
|
+
You MUST include these sections in order:
|
|
187
|
+
|
|
188
|
+
```markdown
|
|
189
|
+
## Acceptance Criteria
|
|
190
|
+
|
|
191
|
+
### AC-1: [Description]
|
|
192
|
+
|
|
193
|
+
**Verification Method:** [Unit Test | Integration Test | Browser Test | Manual Test]
|
|
194
|
+
|
|
195
|
+
**Test Scenario:**
|
|
196
|
+
- Given: [Initial state]
|
|
197
|
+
- When: [Action]
|
|
198
|
+
- Then: [Expected outcome]
|
|
199
|
+
|
|
200
|
+
### AC-2: [Description]
|
|
201
|
+
<!-- Continue for all AC items -->
|
|
202
|
+
|
|
203
|
+
---
|
|
204
|
+
|
|
205
|
+
## Implementation Plan
|
|
206
|
+
|
|
207
|
+
### Phase 1: [Phase Name]
|
|
208
|
+
1. [Step with specific file/component references]
|
|
209
|
+
2. [Step]
|
|
210
|
+
|
|
211
|
+
### Phase 2: [Phase Name]
|
|
212
|
+
<!-- Continue for all phases -->
|
|
213
|
+
|
|
214
|
+
---
|
|
215
|
+
|
|
216
|
+
## Open Questions
|
|
217
|
+
|
|
218
|
+
1. **[Question]**
|
|
219
|
+
- Recommendation: [Default choice]
|
|
220
|
+
- Impact: [What happens if wrong]
|
|
221
|
+
|
|
222
|
+
---
|
|
223
|
+
|
|
224
|
+
--- DRAFT GITHUB ISSUE COMMENT (PLAN) ---
|
|
225
|
+
|
|
226
|
+
[Complete formatted comment for GitHub]
|
|
227
|
+
```
|
|
@@ -51,7 +51,7 @@ Include a `[model: haiku]` or `[model: sonnet]` annotation at the end of each ta
|
|
|
51
51
|
- [ ] Refactor `lib/hooks/useMetrics.ts` to use new service [model: sonnet]
|
|
52
52
|
|
|
53
53
|
### Sequential (depends on Group 2)
|
|
54
|
-
- [ ] Integrate into
|
|
54
|
+
- [ ] Integrate into main feature page
|
|
55
55
|
- [ ] Add tests in `__tests__/metrics.test.ts`
|
|
56
56
|
```
|
|
57
57
|
|
|
@@ -14,7 +14,7 @@ Verification criteria force you to:
|
|
|
14
14
|
| Method | Use When | Example |
|
|
15
15
|
|--------|----------|---------|
|
|
16
16
|
| **Unit Test** | Pure logic, utilities, helpers | `formatCurrency()`, `validateEmail()` |
|
|
17
|
-
| **Integration Test** | External APIs, database, file system | Hook scripts,
|
|
17
|
+
| **Integration Test** | External APIs, database, file system | Hook scripts, database queries |
|
|
18
18
|
| **Browser Test** | UI interactions, forms, modals | Admin dashboard, form validation |
|
|
19
19
|
| **Manual Test** | One-time setup, visual verification | Deployment checks, design review |
|
|
20
20
|
| **N/A - Trivial** | Config changes, simple renames | Env var addition, label change |
|
|
@@ -91,10 +91,10 @@ find components -name "*<FeatureName>*" -type f
|
|
|
91
91
|
|
|
92
92
|
Check for test data requirements:
|
|
93
93
|
|
|
94
|
-
1. Look for seed script
|
|
94
|
+
1. Look for seed script or test fixtures in the project
|
|
95
95
|
2. If seed script exists, offer to run it:
|
|
96
96
|
```bash
|
|
97
|
-
npx tsx
|
|
97
|
+
npx tsx scripts/seed-test-<feature>.ts
|
|
98
98
|
```
|
|
99
99
|
3. If no seed script exists, check Issue for SQL statements or manual setup steps
|
|
100
100
|
4. Execute setup or provide clear instructions to user
|
|
@@ -184,7 +184,7 @@ Use Chrome DevTools MCP for browser-based tests:
|
|
|
184
184
|
|
|
185
185
|
```javascript
|
|
186
186
|
// Navigate to feature
|
|
187
|
-
mcp__chrome-devtools__navigate_page({url: "
|
|
187
|
+
mcp__chrome-devtools__navigate_page({url: "{{DEV_URL}}/..."})
|
|
188
188
|
|
|
189
189
|
// Get page structure
|
|
190
190
|
mcp__chrome-devtools__take_snapshot()
|
|
@@ -446,7 +446,8 @@ After testing completes, ask user if test data should be cleaned up:
|
|
|
446
446
|
|
|
447
447
|
**Option 1: Delete test data**
|
|
448
448
|
```sql
|
|
449
|
-
|
|
449
|
+
-- Clean up test records created during testing
|
|
450
|
+
DELETE FROM <table> WHERE name LIKE 'Test %';
|
|
450
451
|
```
|
|
451
452
|
|
|
452
453
|
**Option 2: Keep for future testing**
|
|
@@ -473,7 +474,7 @@ Testing session is complete when:
|
|
|
473
474
|
**Expected workflow:**
|
|
474
475
|
1. Fetch Issue #151
|
|
475
476
|
2. Detect 10 test cases in Issue body
|
|
476
|
-
3. Check for seed script
|
|
477
|
+
3. Check for seed script or test fixtures
|
|
477
478
|
4. Start dev server if needed
|
|
478
479
|
5. Execute tests 1-10 using Chrome DevTools MCP
|
|
479
480
|
6. Generate test summary with PASS/FAIL/BLOCKED status
|
|
@@ -506,3 +507,17 @@ Testing session is complete when:
|
|
|
506
507
|
Both can be used together:
|
|
507
508
|
1. `/test` → Verify feature works for users
|
|
508
509
|
2. `/qa` → Verify code quality and completeness
|
|
510
|
+
|
|
511
|
+
---
|
|
512
|
+
|
|
513
|
+
## Output Verification
|
|
514
|
+
|
|
515
|
+
**Before responding, verify your output includes ALL of these:**
|
|
516
|
+
|
|
517
|
+
- [ ] **Test Summary** - X/Y tests passed
|
|
518
|
+
- [ ] **Test Results Table** - Each test marked PASS, FAIL, or BLOCKED
|
|
519
|
+
- [ ] **Bugs Found** - List of bugs with file:line locations (if any)
|
|
520
|
+
- [ ] **Coverage** - Completed, failed, blocked, remaining counts
|
|
521
|
+
- [ ] **Recommendations** - Next steps for failures or follow-up
|
|
522
|
+
|
|
523
|
+
**DO NOT respond until all items are verified.**
|
|
@@ -17,7 +17,6 @@ allowed-tools:
|
|
|
17
17
|
- Bash(git worktree list:*)
|
|
18
18
|
- Bash(ls:*)
|
|
19
19
|
- Bash(mkdir:*)
|
|
20
|
-
- mcp__supabase__*
|
|
21
20
|
---
|
|
22
21
|
|
|
23
22
|
# Test Generation Command
|
|
@@ -559,3 +558,18 @@ For Issue #452 (hooks):
|
|
|
559
558
|
---
|
|
560
559
|
Generated with [Claude Code](https://claude.com/claude-code)
|
|
561
560
|
```
|
|
561
|
+
|
|
562
|
+
---
|
|
563
|
+
|
|
564
|
+
## Output Verification
|
|
565
|
+
|
|
566
|
+
**Before responding, verify your output includes ALL of these:**
|
|
567
|
+
|
|
568
|
+
- [ ] **AC Parsing** - Each AC identified with verification method
|
|
569
|
+
- [ ] **Test Stubs Generated** - Files created for Unit/Integration tests
|
|
570
|
+
- [ ] **Browser/Manual Scenarios** - Written for applicable AC items
|
|
571
|
+
- [ ] **Failure Paths** - Error handling stubs for each AC
|
|
572
|
+
- [ ] **Summary Table** - AC count, happy path count, failure path count
|
|
573
|
+
- [ ] **GitHub Comment** - Summary posted to issue
|
|
574
|
+
|
|
575
|
+
**DO NOT respond until all items are verified.**
|
|
@@ -42,7 +42,7 @@ Use `/verify` for:
|
|
|
42
42
|
|
|
43
43
|
```bash
|
|
44
44
|
# With explicit command
|
|
45
|
-
/verify 559 --command "npx tsx scripts/
|
|
45
|
+
/verify 559 --command "npx tsx scripts/migrate.ts --dry-run"
|
|
46
46
|
|
|
47
47
|
# With issue only (will prompt for command)
|
|
48
48
|
/verify 559
|
|
@@ -200,14 +200,14 @@ To prevent oversized GitHub comments (64KB limit):
|
|
|
200
200
|
### Example 1: Successful Verification
|
|
201
201
|
|
|
202
202
|
```bash
|
|
203
|
-
/verify 558 --command "npx tsx scripts/
|
|
203
|
+
/verify 558 --command "npx tsx scripts/migrate.ts --dry-run"
|
|
204
204
|
```
|
|
205
205
|
|
|
206
206
|
Output:
|
|
207
207
|
```
|
|
208
|
-
Starting
|
|
209
|
-
|
|
210
|
-
|
|
208
|
+
Starting migration (dry run)...
|
|
209
|
+
Checking tables...
|
|
210
|
+
Migration plan: 3 tables, 5 columns
|
|
211
211
|
...
|
|
212
212
|
Completed successfully
|
|
213
213
|
```
|
|
@@ -264,3 +264,18 @@ If verification fails due to infrastructure issues:
|
|
|
264
264
|
4. **File not found:** Verify worktree is correct, check file paths
|
|
265
265
|
|
|
266
266
|
Report infrastructure issues separately from feature issues.
|
|
267
|
+
|
|
268
|
+
---
|
|
269
|
+
|
|
270
|
+
## Output Verification
|
|
271
|
+
|
|
272
|
+
**Before responding, verify your output includes ALL of these:**
|
|
273
|
+
|
|
274
|
+
- [ ] **Command Executed** - The exact command that was run
|
|
275
|
+
- [ ] **Exit Code** - Success (0) or error code
|
|
276
|
+
- [ ] **Duration** - How long the command took
|
|
277
|
+
- [ ] **Output Sample** - Captured stdout/stderr (truncated if needed)
|
|
278
|
+
- [ ] **Human Confirmation** - User's verification response recorded
|
|
279
|
+
- [ ] **GitHub Comment** - Verification evidence posted to issue
|
|
280
|
+
|
|
281
|
+
**DO NOT respond until all items are verified.**
|