sequant 1.0.0 → 1.1.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 +12 -8
- package/dist/src/commands/doctor.d.ts.map +1 -1
- package/dist/src/commands/doctor.js +70 -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 +209 -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 +69 -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 +195 -0
- package/dist/src/commands/init.test.js.map +1 -0
- 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/package.json +4 -3
- 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 +16 -16
- package/templates/skills/clean/SKILL.md +2 -2
- package/templates/skills/docs/SKILL.md +32 -34
- package/templates/skills/exec/SKILL.md +17 -25
- package/templates/skills/fullsolve/SKILL.md +18 -16
- package/templates/skills/loop/SKILL.md +8 -5
- package/templates/skills/qa/SKILL.md +22 -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 +4 -2
- package/templates/skills/reflect/references/documentation-tiers.md +3 -3
- package/templates/skills/security-review/references/security-checklists.md +10 -8
- package/templates/skills/solve/SKILL.md +109 -155
- package/templates/skills/spec/SKILL.md +2 -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 +6 -5
- package/templates/skills/testgen/SKILL.md +0 -1
- package/templates/skills/verify/SKILL.md +5 -5
- package/templates/skills/reflect/scripts/workflow-queries.ts +0 -165
|
@@ -151,8 +151,8 @@ if (user.role !== 'admin') throw new ForbiddenError()
|
|
|
151
151
|
|
|
152
152
|
**Bad:**
|
|
153
153
|
```typescript
|
|
154
|
-
const
|
|
155
|
-
// Missing: .
|
|
154
|
+
const item = await db.items.findUnique({ where: { id: itemId } })
|
|
155
|
+
// Missing: ownership verification (e.g., where: { id: itemId, owner_id: user.id })
|
|
156
156
|
```
|
|
157
157
|
|
|
158
158
|
### AUTHZ-4: Horizontal Privilege Escalation
|
|
@@ -174,12 +174,12 @@ const shop = await db.shops.findUnique({ where: { id: shopId } })
|
|
|
174
174
|
|
|
175
175
|
**How to Verify:**
|
|
176
176
|
```bash
|
|
177
|
-
grep -r "audit\|logAction\|
|
|
177
|
+
grep -r "audit\|logAction\|createAuditLog" lib/ app/admin/
|
|
178
178
|
```
|
|
179
179
|
|
|
180
180
|
**Good:**
|
|
181
181
|
```typescript
|
|
182
|
-
await logAuditEvent({ action: '
|
|
182
|
+
await logAuditEvent({ action: 'approve_item', actor: userId, target: itemId })
|
|
183
183
|
```
|
|
184
184
|
|
|
185
185
|
---
|
|
@@ -217,18 +217,20 @@ const { name } = req.body // No validation
|
|
|
217
217
|
|
|
218
218
|
**How to Verify:**
|
|
219
219
|
```bash
|
|
220
|
-
grep -r "\.from\(" lib/ app/ | grep -v "supabase"
|
|
221
220
|
grep -r "raw\|execute\|query" lib/ app/
|
|
222
221
|
```
|
|
223
222
|
|
|
224
223
|
**Good:**
|
|
225
224
|
```typescript
|
|
226
|
-
|
|
225
|
+
// Using ORM with parameterized queries
|
|
226
|
+
db.items.findUnique({ where: { id: itemId } })
|
|
227
|
+
// Or query builder with parameters
|
|
228
|
+
db.from('items').select('*').eq('id', itemId)
|
|
227
229
|
```
|
|
228
230
|
|
|
229
231
|
**Bad:**
|
|
230
232
|
```typescript
|
|
231
|
-
db.query(`SELECT * FROM
|
|
233
|
+
db.query(`SELECT * FROM items WHERE id = ${itemId}`)
|
|
232
234
|
```
|
|
233
235
|
|
|
234
236
|
### API-4: Rate Limiting
|
|
@@ -290,7 +292,7 @@ if (file.size > 5 * 1024 * 1024) throw Error
|
|
|
290
292
|
**Requirement:** Sensitive data must be encrypted in database.
|
|
291
293
|
|
|
292
294
|
**How to Verify:**
|
|
293
|
-
-
|
|
295
|
+
- Verify database uses encryption at rest (most cloud providers enable by default)
|
|
294
296
|
- Check for additional encryption on highly sensitive fields
|
|
295
297
|
|
|
296
298
|
### DATA-2: Encryption in Transit
|
|
@@ -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,143 @@ 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 |
|
|
110
|
-
|
|
111
|
-
### Recommended Command
|
|
112
|
-
|
|
113
|
-
Since issues #152 and #154 require UI testing, we'll add the `/test` phase for all issues:
|
|
114
|
-
|
|
115
|
-
\`\`\`bash
|
|
116
|
-
PHASES=spec,exec,test,qa npx tsx --env-file=.env.local scripts/dev/execute-issues.ts 152 153 154
|
|
117
|
-
\`\`\`
|
|
118
|
-
|
|
119
|
-
### Explanation
|
|
120
|
-
|
|
121
|
-
- **Parallel execution**: All 3 issues run simultaneously
|
|
122
|
-
- **Custom phases**: `spec,exec,test,qa` includes browser testing
|
|
123
|
-
- **Logs**: Check `/tmp/claude-issue-{152,153,154}.log` for progress
|
|
72
|
+
| Issue | Title | Labels | Workflow |
|
|
73
|
+
|-------|-------|--------|----------|
|
|
74
|
+
| #152 | Add user dashboard | ui, enhancement | Full (with /test) |
|
|
75
|
+
| #153 | Fix API validation | backend, bug | Standard |
|
|
124
76
|
|
|
125
|
-
###
|
|
77
|
+
### Recommended Workflow
|
|
126
78
|
|
|
127
|
-
For
|
|
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
|
+
```
|
|
128
86
|
|
|
129
|
-
|
|
130
|
-
|
|
131
|
-
|
|
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
|
+
```
|
|
132
93
|
|
|
133
|
-
|
|
94
|
+
### Full Workflow Option
|
|
134
95
|
|
|
135
|
-
|
|
96
|
+
For comprehensive quality with automatic fix iterations:
|
|
97
|
+
```bash
|
|
98
|
+
/fullsolve 152
|
|
99
|
+
```
|
|
136
100
|
|
|
137
|
-
|
|
101
|
+
### CLI Command
|
|
138
102
|
|
|
139
|
-
|
|
140
|
-
|
|
141
|
-
|
|
103
|
+
Run from terminal (useful for automation/CI):
|
|
104
|
+
```bash
|
|
105
|
+
sequant run 152 # Single issue
|
|
106
|
+
sequant run 152 153 # Multiple issues
|
|
107
|
+
```
|
|
142
108
|
|
|
143
|
-
###
|
|
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
|
+
```
|
|
144
114
|
|
|
145
|
-
|
|
115
|
+
## Workflow Selection Logic
|
|
146
116
|
|
|
147
|
-
|
|
148
|
-
|
|
149
|
-
|
|
150
|
-
|
|
117
|
+
### Standard Workflow (Most Issues)
|
|
118
|
+
```
|
|
119
|
+
/spec → /exec → /qa
|
|
120
|
+
```
|
|
151
121
|
|
|
152
|
-
|
|
122
|
+
### UI Feature Workflow
|
|
123
|
+
```
|
|
124
|
+
/spec → /exec → /test → /qa
|
|
125
|
+
```
|
|
153
126
|
|
|
154
|
-
|
|
155
|
-
|
|
156
|
-
|
|
127
|
+
### Bug Fix Workflow (Simple)
|
|
128
|
+
```
|
|
129
|
+
/exec → /qa
|
|
130
|
+
```
|
|
131
|
+
Skip `/spec` if the bug is well-defined and straightforward.
|
|
157
132
|
|
|
158
|
-
|
|
133
|
+
### Complex Feature Workflow
|
|
134
|
+
```
|
|
135
|
+
/fullsolve <issue>
|
|
159
136
|
```
|
|
137
|
+
Runs complete workflow with automatic fix iterations.
|
|
160
138
|
|
|
161
|
-
##
|
|
139
|
+
## Quick Reference
|
|
162
140
|
|
|
163
|
-
|
|
164
|
-
|
|
165
|
-
|
|
166
|
-
|
|
167
|
-
|
|
168
|
-
|
|
169
|
-
|
|
170
|
-
6. **Display output**: Show issue table + recommended command + explanation
|
|
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 |
|
|
171
148
|
|
|
172
|
-
##
|
|
149
|
+
## CLI Alternative
|
|
173
150
|
|
|
174
|
-
|
|
175
|
-
```bash
|
|
176
|
-
npx tsx --env-file=.env.local scripts/dev/execute-issues.ts 145 146 147
|
|
177
|
-
```
|
|
178
|
-
No `PHASES` env var needed - default is `spec,exec,qa`
|
|
151
|
+
Use `sequant run` for batch execution from the command line:
|
|
179
152
|
|
|
180
|
-
### All UI Issues
|
|
181
153
|
```bash
|
|
182
|
-
|
|
183
|
-
|
|
154
|
+
# Run workflow for single issue
|
|
155
|
+
sequant run 152
|
|
184
156
|
|
|
185
|
-
|
|
186
|
-
|
|
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.
|
|
157
|
+
# Multiple issues in parallel
|
|
158
|
+
sequant run 152 153 154
|
|
188
159
|
|
|
189
|
-
|
|
190
|
-
|
|
160
|
+
# Sequential execution (respects dependencies)
|
|
161
|
+
sequant run 152 153 --sequential
|
|
191
162
|
|
|
192
|
-
|
|
193
|
-
|
|
194
|
-
npx tsx --env-file=.env.local scripts/dev/execute-issues.ts --batch "152 153" --batch "154"
|
|
195
|
-
```
|
|
163
|
+
# Custom phases
|
|
164
|
+
sequant run 152 --phases spec,exec,qa
|
|
196
165
|
|
|
197
|
-
|
|
166
|
+
# Dry run (shows what would execute)
|
|
167
|
+
sequant run 152 --dry-run
|
|
168
|
+
```
|
|
198
169
|
|
|
199
|
-
|
|
170
|
+
## Edge Cases
|
|
200
171
|
|
|
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
|
|
172
|
+
### Multiple Issues with Different Types
|
|
205
173
|
|
|
206
|
-
|
|
207
|
-
- Simple bug fixes with clear scope
|
|
208
|
-
- Documentation-only changes
|
|
209
|
-
- User explicitly wants quick execution
|
|
174
|
+
When solving multiple issues with mixed types, recommend running them separately:
|
|
210
175
|
|
|
211
|
-
|
|
176
|
+
```markdown
|
|
177
|
+
These issues have different requirements. Run separately:
|
|
212
178
|
|
|
213
|
-
|
|
179
|
+
**UI Issues (with browser testing):**
|
|
180
|
+
/fullsolve 152
|
|
214
181
|
|
|
215
|
-
|
|
216
|
-
|
|
217
|
-
|
|
182
|
+
**Backend Issues:**
|
|
183
|
+
/fullsolve 153
|
|
184
|
+
```
|
|
218
185
|
|
|
219
|
-
|
|
220
|
-
- Batch processing many issues (speed priority)
|
|
221
|
-
- Issues with long-running test suites
|
|
222
|
-
- Simple documentation changes
|
|
186
|
+
### Issue with Dependencies
|
|
223
187
|
|
|
224
|
-
|
|
225
|
-
```bash
|
|
226
|
-
npx tsx scripts/dev/analyze-hook-logs.ts --tests
|
|
227
|
-
```
|
|
188
|
+
If issues depend on each other:
|
|
228
189
|
|
|
229
|
-
|
|
190
|
+
```markdown
|
|
191
|
+
⚠️ Issue #154 depends on #153. Run in order:
|
|
230
192
|
|
|
231
|
-
|
|
232
|
-
|
|
233
|
-
|
|
234
|
-
|
|
235
|
-
- `QUALITY_LOOP=true`: Auto-fix test/QA failures, **auto-includes `/testgen` after `/spec`**
|
|
236
|
-
- `MAX_ITERATIONS`: Max fix attempts per phase (default: 3)
|
|
237
|
-
- **Smart tests: Enabled by default** - auto-runs related tests after file edits
|
|
238
|
-
- `--no-smart-tests`: Disable smart tests (faster but no auto-regression detection)
|
|
239
|
-
- Parallel execution: Multiple issues run simultaneously
|
|
240
|
-
- Batch mode: `--batch "N M"` runs batches sequentially
|
|
241
|
-
- `--env-file=.env.local`: **Required** for database logging (workflow analytics)
|
|
242
|
-
- Logs: `/tmp/claude-issue-<N>.log`, `/tmp/claude-tests.log` (smart test results)
|
|
193
|
+
1. /fullsolve 153
|
|
194
|
+
2. (Wait for PR merge)
|
|
195
|
+
3. /fullsolve 154
|
|
196
|
+
```
|
|
@@ -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)**
|
|
@@ -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
|
|
@@ -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
|
```
|