@jakkrichm/create-nexus-devflow 2.0.12 → 2.0.13
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/lib/update.js +15 -1
- package/package.json +1 -1
- package/template/.agents/skills/70-release/SKILL.md +2 -0
- package/template/.agents/skills/ci/SKILL.md +25 -78
- package/template/.agents/skills/commit/SKILL.md +39 -43
- package/template/.agents/skills/debug/SKILL.md +43 -104
- package/template/.agents/skills/deploy/SKILL.md +37 -65
- package/template/.agents/skills/insight/SKILL.md +27 -116
- package/template/.agents/skills/preview/SKILL.md +24 -103
- package/template/.agents/skills/review/SKILL.md +53 -37
- package/template/.agents/skills/rollback/SKILL.md +1 -0
- package/template/.agents/skills/security-review/SKILL.md +44 -147
- package/template/.agents/skills/simplify/SKILL.md +48 -57
- package/template/.agents/skills/test/SKILL.md +63 -51
- package/template/.claude/skills/70-release/SKILL.md +2 -0
- package/template/.claude/skills/ci/SKILL.md +25 -78
- package/template/.claude/skills/commit/SKILL.md +39 -43
- package/template/.claude/skills/debug/SKILL.md +43 -104
- package/template/.claude/skills/deploy/SKILL.md +37 -65
- package/template/.claude/skills/insight/SKILL.md +27 -116
- package/template/.claude/skills/preview/SKILL.md +24 -103
- package/template/.claude/skills/review/SKILL.md +53 -37
- package/template/.claude/skills/rollback/SKILL.md +1 -0
- package/template/.claude/skills/security-review/SKILL.md +44 -147
- package/template/.claude/skills/simplify/SKILL.md +48 -57
- package/template/.claude/skills/test/SKILL.md +63 -51
- package/template/devflow/history/HISTORY.md +27 -0
- package/template/devflow/reference/running-id-contract.md +1 -1
- package/template/.agents/skills/9arm-skills/README.md +0 -51
- package/template/.agents/skills/9arm-skills/debug-mantra/SKILL.md +0 -86
- package/template/.agents/skills/9arm-skills/management-talk/SKILL.md +0 -79
- package/template/.agents/skills/9arm-skills/post-mortem/SKILL.md +0 -71
- package/template/.agents/skills/9arm-skills/scrutinize/SKILL.md +0 -72
- package/template/.agents/skills/browser-testing-with-devtools/SKILL.md +0 -302
- package/template/.agents/skills/ci-cd-and-automation/SKILL.md +0 -390
- package/template/.agents/skills/code-review-and-quality/SKILL.md +0 -392
- package/template/.agents/skills/code-simplification/SKILL.md +0 -331
- package/template/.agents/skills/debugging-and-error-recovery/SKILL.md +0 -298
- package/template/.agents/skills/deployment-procedures/SKILL.md +0 -241
- package/template/.agents/skills/deprecation-and-migration/SKILL.md +0 -206
- package/template/.agents/skills/diagnosing-bugs/SKILL.md +0 -93
- package/template/.agents/skills/git-workflow-and-versioning/SKILL.md +0 -300
- package/template/.agents/skills/human-review-decisions/SKILL.md +0 -74
- package/template/.agents/skills/idea-refine/SKILL.md +0 -178
- package/template/.agents/skills/idea-refine/examples.md +0 -238
- package/template/.agents/skills/idea-refine/frameworks.md +0 -99
- package/template/.agents/skills/idea-refine/refinement-criteria.md +0 -113
- package/template/.agents/skills/idea-refine/scripts/idea-refine.sh +0 -15
- package/template/.agents/skills/incremental-implementation/SKILL.md +0 -248
- package/template/.agents/skills/insight-capture/SKILL.md +0 -84
- package/template/.agents/skills/intelligent-routing/SKILL.md +0 -176
- package/template/.agents/skills/md2html/SKILL.md +0 -154
- package/template/.agents/skills/md2html/components.md +0 -505
- package/template/.agents/skills/md2html/template.html +0 -1152
- package/template/.agents/skills/planning-and-task-breakdown/SKILL.md +0 -239
- package/template/.agents/skills/pr-review/SKILL.md +0 -143
- package/template/.agents/skills/pr-review-analysis/SKILL.md +0 -89
- package/template/.agents/skills/preview-local-check/SKILL.md +0 -59
- package/template/.agents/skills/release-git-operations/SKILL.md +0 -97
- package/template/.agents/skills/review-followup-routing/SKILL.md +0 -98
- package/template/.agents/skills/security-and-hardening/SKILL.md +0 -349
- package/template/.agents/skills/security-and-hardening/security-checklist.md +0 -134
- package/template/.agents/skills/shipping-and-launch/SKILL.md +0 -311
- package/template/.agents/skills/silent-failure-audit/SKILL.md +0 -73
- package/template/.agents/skills/spec-orchestration/SKILL.md +0 -91
- package/template/.agents/skills/specialist-agent-routing/SKILL.md +0 -107
- package/template/.agents/skills/test-driven-development/SKILL.md +0 -422
- package/template/.agents/skills/test-driven-development/testing-patterns.md +0 -236
- package/template/.agents/skills/test-execution-and-coverage/SKILL.md +0 -56
- package/template/.agents/skills/using-agent-skills/SKILL.md +0 -171
- package/template/.agents/skills/verification-orchestration/SKILL.md +0 -68
- package/template/.agents/skills/vulnerability-scanner/SKILL.md +0 -276
- package/template/.agents/skills/vulnerability-scanner/checklists.md +0 -121
- package/template/.agents/skills/vulnerability-scanner/scripts/security_scan.py +0 -458
- package/template/.agents/skills/wiki/SKILL.md +0 -149
- package/template/.agents/skills/workflow-documentation-sync/SKILL.md +0 -87
- package/template/.claude/skills/9arm-skills/README.md +0 -51
- package/template/.claude/skills/9arm-skills/debug-mantra/SKILL.md +0 -86
- package/template/.claude/skills/9arm-skills/management-talk/SKILL.md +0 -79
- package/template/.claude/skills/9arm-skills/post-mortem/SKILL.md +0 -71
- package/template/.claude/skills/9arm-skills/scrutinize/SKILL.md +0 -72
- package/template/.claude/skills/browser-testing-with-devtools/SKILL.md +0 -302
- package/template/.claude/skills/ci-cd-and-automation/SKILL.md +0 -390
- package/template/.claude/skills/code-review-and-quality/SKILL.md +0 -392
- package/template/.claude/skills/code-simplification/SKILL.md +0 -331
- package/template/.claude/skills/debugging-and-error-recovery/SKILL.md +0 -298
- package/template/.claude/skills/deployment-procedures/SKILL.md +0 -241
- package/template/.claude/skills/deprecation-and-migration/SKILL.md +0 -206
- package/template/.claude/skills/diagnosing-bugs/SKILL.md +0 -93
- package/template/.claude/skills/git-workflow-and-versioning/SKILL.md +0 -300
- package/template/.claude/skills/human-review-decisions/SKILL.md +0 -74
- package/template/.claude/skills/idea-refine/SKILL.md +0 -178
- package/template/.claude/skills/idea-refine/examples.md +0 -238
- package/template/.claude/skills/idea-refine/frameworks.md +0 -99
- package/template/.claude/skills/idea-refine/refinement-criteria.md +0 -113
- package/template/.claude/skills/idea-refine/scripts/idea-refine.sh +0 -15
- package/template/.claude/skills/incremental-implementation/SKILL.md +0 -248
- package/template/.claude/skills/insight-capture/SKILL.md +0 -84
- package/template/.claude/skills/intelligent-routing/SKILL.md +0 -176
- package/template/.claude/skills/md2html/SKILL.md +0 -154
- package/template/.claude/skills/md2html/components.md +0 -505
- package/template/.claude/skills/md2html/template.html +0 -1152
- package/template/.claude/skills/planning-and-task-breakdown/SKILL.md +0 -239
- package/template/.claude/skills/pr-review/SKILL.md +0 -143
- package/template/.claude/skills/pr-review-analysis/SKILL.md +0 -89
- package/template/.claude/skills/preview-local-check/SKILL.md +0 -59
- package/template/.claude/skills/release-git-operations/SKILL.md +0 -97
- package/template/.claude/skills/review-followup-routing/SKILL.md +0 -98
- package/template/.claude/skills/security-and-hardening/SKILL.md +0 -349
- package/template/.claude/skills/security-and-hardening/security-checklist.md +0 -134
- package/template/.claude/skills/shipping-and-launch/SKILL.md +0 -311
- package/template/.claude/skills/silent-failure-audit/SKILL.md +0 -73
- package/template/.claude/skills/spec-orchestration/SKILL.md +0 -91
- package/template/.claude/skills/specialist-agent-routing/SKILL.md +0 -107
- package/template/.claude/skills/test-driven-development/SKILL.md +0 -422
- package/template/.claude/skills/test-driven-development/testing-patterns.md +0 -236
- package/template/.claude/skills/test-execution-and-coverage/SKILL.md +0 -56
- package/template/.claude/skills/using-agent-skills/SKILL.md +0 -171
- package/template/.claude/skills/verification-orchestration/SKILL.md +0 -68
- package/template/.claude/skills/vulnerability-scanner/SKILL.md +0 -276
- package/template/.claude/skills/vulnerability-scanner/checklists.md +0 -121
- package/template/.claude/skills/vulnerability-scanner/scripts/security_scan.py +0 -458
- package/template/.claude/skills/wiki/SKILL.md +0 -149
- package/template/.claude/skills/workflow-documentation-sync/SKILL.md +0 -87
|
@@ -1,311 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: shipping-and-launch
|
|
3
|
-
description: "[Devflow] Prepares production launches. Use when preparing to deploy to production. Use when you need a pre-launch checklist, when setting up monitoring, when planning a staged rollout, or when you need a rollback strategy."
|
|
4
|
-
---
|
|
5
|
-
|
|
6
|
-
# Shipping and Launch
|
|
7
|
-
|
|
8
|
-
## Overview
|
|
9
|
-
|
|
10
|
-
Ship with confidence. The goal is not just to deploy — it's to deploy safely, with monitoring in place, a rollback plan ready, and a clear understanding of what success looks like. Every launch should be reversible, observable, and incremental.
|
|
11
|
-
|
|
12
|
-
## When to Use
|
|
13
|
-
|
|
14
|
-
- Deploying a feature to production for the first time
|
|
15
|
-
- Releasing a significant change to users
|
|
16
|
-
- Migrating data or infrastructure
|
|
17
|
-
- Opening a beta or early access program
|
|
18
|
-
- Any deployment that carries risk (all of them)
|
|
19
|
-
|
|
20
|
-
In DevFlow 2.0, this skill most often supports `70-release` and assumes `60-report` is already aligned as the communication baseline.
|
|
21
|
-
|
|
22
|
-
## The Pre-Launch Checklist
|
|
23
|
-
|
|
24
|
-
### Code Quality
|
|
25
|
-
|
|
26
|
-
- [ ] All tests pass (unit, integration, e2e)
|
|
27
|
-
- [ ] Build succeeds with no warnings
|
|
28
|
-
- [ ] Lint and type checking pass
|
|
29
|
-
- [ ] Code reviewed and approved
|
|
30
|
-
- [ ] No TODO comments that should be resolved before launch
|
|
31
|
-
- [ ] No `console.log` debugging statements in production code
|
|
32
|
-
- [ ] Error handling covers expected failure modes
|
|
33
|
-
|
|
34
|
-
### Security
|
|
35
|
-
|
|
36
|
-
- [ ] No secrets in code or version control
|
|
37
|
-
- [ ] `npm audit` shows no critical or high vulnerabilities
|
|
38
|
-
- [ ] Input validation on all user-facing endpoints
|
|
39
|
-
- [ ] Authentication and authorization checks in place
|
|
40
|
-
- [ ] Security headers configured (CSP, HSTS, etc.)
|
|
41
|
-
- [ ] Rate limiting on authentication endpoints
|
|
42
|
-
- [ ] CORS configured to specific origins (not wildcard)
|
|
43
|
-
|
|
44
|
-
### Performance
|
|
45
|
-
|
|
46
|
-
- [ ] Core Web Vitals within "Good" thresholds
|
|
47
|
-
- [ ] No N+1 queries in critical paths
|
|
48
|
-
- [ ] Images optimized (compression, responsive sizes, lazy loading)
|
|
49
|
-
- [ ] Bundle size within budget
|
|
50
|
-
- [ ] Database queries have appropriate indexes
|
|
51
|
-
- [ ] Caching configured for static assets and repeated queries
|
|
52
|
-
|
|
53
|
-
### Accessibility
|
|
54
|
-
|
|
55
|
-
- [ ] Keyboard navigation works for all interactive elements
|
|
56
|
-
- [ ] Screen reader can convey page content and structure
|
|
57
|
-
- [ ] Color contrast meets WCAG 2.1 AA (4.5:1 for text)
|
|
58
|
-
- [ ] Focus management correct for modals and dynamic content
|
|
59
|
-
- [ ] Error messages are descriptive and associated with form fields
|
|
60
|
-
- [ ] No accessibility warnings in axe-core or Lighthouse
|
|
61
|
-
|
|
62
|
-
### Infrastructure
|
|
63
|
-
|
|
64
|
-
- [ ] Environment variables set in production
|
|
65
|
-
- [ ] Database migrations applied (or ready to apply)
|
|
66
|
-
- [ ] DNS and SSL configured
|
|
67
|
-
- [ ] CDN configured for static assets
|
|
68
|
-
- [ ] Logging and error reporting configured
|
|
69
|
-
- [ ] Health check endpoint exists and responds
|
|
70
|
-
|
|
71
|
-
### Documentation
|
|
72
|
-
|
|
73
|
-
- [ ] README updated with any new setup requirements
|
|
74
|
-
- [ ] API documentation current
|
|
75
|
-
- [ ] ADRs written for any architectural decisions
|
|
76
|
-
- [ ] Changelog updated
|
|
77
|
-
- [ ] User-facing documentation updated (if applicable)
|
|
78
|
-
|
|
79
|
-
## Feature Flag Strategy
|
|
80
|
-
|
|
81
|
-
Ship behind feature flags to decouple deployment from release:
|
|
82
|
-
|
|
83
|
-
```typescript
|
|
84
|
-
// Feature flag check
|
|
85
|
-
const flags = await getFeatureFlags(userId);
|
|
86
|
-
|
|
87
|
-
if (flags.taskSharing) {
|
|
88
|
-
// New feature: task sharing
|
|
89
|
-
return <TaskSharingPanel task={task} />;
|
|
90
|
-
}
|
|
91
|
-
|
|
92
|
-
// Default: existing behavior
|
|
93
|
-
return null;
|
|
94
|
-
```
|
|
95
|
-
|
|
96
|
-
**Feature flag lifecycle:**
|
|
97
|
-
|
|
98
|
-
```
|
|
99
|
-
1. DEPLOY with flag OFF → Code is in production but inactive
|
|
100
|
-
2. ENABLE for team/beta → Internal testing in production environment
|
|
101
|
-
3. GRADUAL ROLLOUT → 5% → 25% → 50% → 100% of users
|
|
102
|
-
4. MONITOR at each stage → Watch error rates, performance, user feedback
|
|
103
|
-
5. CLEAN UP → Remove flag and dead code path after full rollout
|
|
104
|
-
```
|
|
105
|
-
|
|
106
|
-
**Rules:**
|
|
107
|
-
- Every feature flag has an owner and an expiration date
|
|
108
|
-
- Clean up flags within 2 weeks of full rollout
|
|
109
|
-
- Don't nest feature flags (creates exponential combinations)
|
|
110
|
-
- Test both flag states (on and off) in CI
|
|
111
|
-
|
|
112
|
-
## Staged Rollout
|
|
113
|
-
|
|
114
|
-
### The Rollout Sequence
|
|
115
|
-
|
|
116
|
-
```
|
|
117
|
-
1. DEPLOY to staging
|
|
118
|
-
└── Full test suite in staging environment
|
|
119
|
-
└── Manual smoke test of critical flows
|
|
120
|
-
|
|
121
|
-
2. DEPLOY to production (feature flag OFF)
|
|
122
|
-
└── Verify deployment succeeded (health check)
|
|
123
|
-
└── Check error monitoring (no new errors)
|
|
124
|
-
|
|
125
|
-
3. ENABLE for team (flag ON for internal users)
|
|
126
|
-
└── Team uses the feature in production
|
|
127
|
-
└── 24-hour monitoring window
|
|
128
|
-
|
|
129
|
-
4. CANARY rollout (flag ON for 5% of users)
|
|
130
|
-
└── Monitor error rates, latency, user behavior
|
|
131
|
-
└── Compare metrics: canary vs. baseline
|
|
132
|
-
└── 24-48 hour monitoring window
|
|
133
|
-
└── Advance only if all thresholds pass (see table below)
|
|
134
|
-
|
|
135
|
-
5. GRADUAL increase (25% -> 50% -> 100%)
|
|
136
|
-
└── Same monitoring at each step
|
|
137
|
-
└── Ability to roll back to previous percentage at any point
|
|
138
|
-
|
|
139
|
-
6. FULL rollout (flag ON for all users)
|
|
140
|
-
└── Monitor for 1 week
|
|
141
|
-
└── Clean up feature flag
|
|
142
|
-
```
|
|
143
|
-
|
|
144
|
-
### Rollout Decision Thresholds
|
|
145
|
-
|
|
146
|
-
Use these thresholds to decide whether to advance, hold, or roll back at each stage:
|
|
147
|
-
|
|
148
|
-
| Metric | Advance (green) | Hold and investigate (yellow) | Roll back (red) |
|
|
149
|
-
|--------|-----------------|-------------------------------|-----------------|
|
|
150
|
-
| Error rate | Within 10% of baseline | 10-100% above baseline | >2x baseline |
|
|
151
|
-
| P95 latency | Within 20% of baseline | 20-50% above baseline | >50% above baseline |
|
|
152
|
-
| Client JS errors | No new error types | New errors at <0.1% of sessions | New errors at >0.1% of sessions |
|
|
153
|
-
| Business metrics | Neutral or positive | Decline <5% (may be noise) | Decline >5% |
|
|
154
|
-
|
|
155
|
-
### When to Roll Back
|
|
156
|
-
|
|
157
|
-
Roll back immediately if:
|
|
158
|
-
- Error rate increases by more than 2x baseline
|
|
159
|
-
- P95 latency increases by more than 50%
|
|
160
|
-
- User-reported issues spike
|
|
161
|
-
- Data integrity issues detected
|
|
162
|
-
- Security vulnerability discovered
|
|
163
|
-
|
|
164
|
-
## Monitoring and Observability
|
|
165
|
-
|
|
166
|
-
### What to Monitor
|
|
167
|
-
|
|
168
|
-
```
|
|
169
|
-
Application metrics:
|
|
170
|
-
├── Error rate (total and by endpoint)
|
|
171
|
-
├── Response time (p50, p95, p99)
|
|
172
|
-
├── Request volume
|
|
173
|
-
├── Active users
|
|
174
|
-
└── Key business metrics (conversion, engagement)
|
|
175
|
-
|
|
176
|
-
Infrastructure metrics:
|
|
177
|
-
├── CPU and memory utilization
|
|
178
|
-
├── Database connection pool usage
|
|
179
|
-
├── Disk space
|
|
180
|
-
├── Network latency
|
|
181
|
-
└── Queue depth (if applicable)
|
|
182
|
-
|
|
183
|
-
Client metrics:
|
|
184
|
-
├── Core Web Vitals (LCP, INP, CLS)
|
|
185
|
-
├── JavaScript errors
|
|
186
|
-
├── API error rates from client perspective
|
|
187
|
-
└── Page load time
|
|
188
|
-
```
|
|
189
|
-
|
|
190
|
-
### Error Reporting
|
|
191
|
-
|
|
192
|
-
```typescript
|
|
193
|
-
// Set up error boundary with reporting
|
|
194
|
-
class ErrorBoundary extends React.Component {
|
|
195
|
-
componentDidCatch(error: Error, info: React.ErrorInfo) {
|
|
196
|
-
// Report to error tracking service
|
|
197
|
-
reportError(error, {
|
|
198
|
-
componentStack: info.componentStack,
|
|
199
|
-
userId: getCurrentUser()?.id,
|
|
200
|
-
page: window.location.pathname,
|
|
201
|
-
});
|
|
202
|
-
}
|
|
203
|
-
|
|
204
|
-
render() {
|
|
205
|
-
if (this.state.hasError) {
|
|
206
|
-
return <ErrorFallback onRetry={() => this.setState({ hasError: false })} />;
|
|
207
|
-
}
|
|
208
|
-
return this.props.children;
|
|
209
|
-
}
|
|
210
|
-
}
|
|
211
|
-
|
|
212
|
-
// Server-side error reporting
|
|
213
|
-
app.use((err: Error, req: Request, res: Response, next: NextFunction) => {
|
|
214
|
-
reportError(err, {
|
|
215
|
-
method: req.method,
|
|
216
|
-
url: req.url,
|
|
217
|
-
userId: req.user?.id,
|
|
218
|
-
});
|
|
219
|
-
|
|
220
|
-
// Don't expose internals to users
|
|
221
|
-
res.status(500).json({
|
|
222
|
-
error: { code: 'INTERNAL_ERROR', message: 'Something went wrong' },
|
|
223
|
-
});
|
|
224
|
-
});
|
|
225
|
-
```
|
|
226
|
-
|
|
227
|
-
### Post-Launch Verification
|
|
228
|
-
|
|
229
|
-
In the first hour after launch:
|
|
230
|
-
|
|
231
|
-
```
|
|
232
|
-
1. Check health endpoint returns 200
|
|
233
|
-
2. Check error monitoring dashboard (no new error types)
|
|
234
|
-
3. Check latency dashboard (no regression)
|
|
235
|
-
4. Test the critical user flow manually
|
|
236
|
-
5. Verify logs are flowing and readable
|
|
237
|
-
6. Confirm rollback mechanism works (dry run if possible)
|
|
238
|
-
```
|
|
239
|
-
|
|
240
|
-
## Rollback Strategy
|
|
241
|
-
|
|
242
|
-
Every deployment needs a rollback plan before it happens:
|
|
243
|
-
|
|
244
|
-
```markdown
|
|
245
|
-
## Rollback Plan for [Feature/Release]
|
|
246
|
-
|
|
247
|
-
### Trigger Conditions
|
|
248
|
-
- Error rate > 2x baseline
|
|
249
|
-
- P95 latency > [X]ms
|
|
250
|
-
- User reports of [specific issue]
|
|
251
|
-
|
|
252
|
-
### Rollback Steps
|
|
253
|
-
1. Disable feature flag (if applicable)
|
|
254
|
-
OR
|
|
255
|
-
1. Deploy previous version: `git revert <commit> && git push`
|
|
256
|
-
2. Verify rollback: health check, error monitoring
|
|
257
|
-
3. Communicate: notify team of rollback
|
|
258
|
-
|
|
259
|
-
### Database Considerations
|
|
260
|
-
- Migration [X] has a rollback: `npx prisma migrate rollback`
|
|
261
|
-
- Data inserted by new feature: [preserved / cleaned up]
|
|
262
|
-
|
|
263
|
-
### Time to Rollback
|
|
264
|
-
- Feature flag: < 1 minute
|
|
265
|
-
- Redeploy previous version: < 5 minutes
|
|
266
|
-
- Database rollback: < 15 minutes
|
|
267
|
-
```
|
|
268
|
-
## See Also
|
|
269
|
-
|
|
270
|
-
- For security pre-launch checks, see `references/security-checklist.md`
|
|
271
|
-
- For performance pre-launch checklist, see `references/performance-checklist.md`
|
|
272
|
-
- For accessibility verification before launch, see `references/accessibility-checklist.md`
|
|
273
|
-
|
|
274
|
-
## Common Rationalizations
|
|
275
|
-
|
|
276
|
-
| Rationalization | Reality |
|
|
277
|
-
|---|---|
|
|
278
|
-
| "It works in staging, it'll work in production" | Production has different data, traffic patterns, and edge cases. Monitor after deploy. |
|
|
279
|
-
| "We don't need feature flags for this" | Every feature benefits from a kill switch. Even "simple" changes can break things. |
|
|
280
|
-
| "Monitoring is overhead" | Not having monitoring means you discover problems from user complaints instead of dashboards. |
|
|
281
|
-
| "We'll add monitoring later" | Add it before launch. You can't debug what you can't see. |
|
|
282
|
-
| "Rolling back is admitting failure" | Rolling back is responsible engineering. Shipping a broken feature is the failure. |
|
|
283
|
-
|
|
284
|
-
## Red Flags
|
|
285
|
-
|
|
286
|
-
- Deploying without a rollback plan
|
|
287
|
-
- No monitoring or error reporting in production
|
|
288
|
-
- Big-bang releases (everything at once, no staging)
|
|
289
|
-
- Feature flags with no expiration or owner
|
|
290
|
-
- No one monitoring the deploy for the first hour
|
|
291
|
-
- Production environment configuration done by memory, not code
|
|
292
|
-
- "It's Friday afternoon, let's ship it"
|
|
293
|
-
|
|
294
|
-
## Verification
|
|
295
|
-
|
|
296
|
-
Before deploying:
|
|
297
|
-
|
|
298
|
-
- [ ] Pre-launch checklist completed (all sections green)
|
|
299
|
-
- [ ] Feature flag configured (if applicable)
|
|
300
|
-
- [ ] Rollback plan documented
|
|
301
|
-
- [ ] Monitoring dashboards set up
|
|
302
|
-
- [ ] Team notified of deployment
|
|
303
|
-
|
|
304
|
-
After deploying:
|
|
305
|
-
|
|
306
|
-
- [ ] Health check returns 200
|
|
307
|
-
- [ ] Error rate is normal
|
|
308
|
-
- [ ] Latency is normal
|
|
309
|
-
- [ ] Critical user flow works
|
|
310
|
-
- [ ] Logs are flowing
|
|
311
|
-
- [ ] Rollback tested or verified ready
|
|
@@ -1,73 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: silent-failure-audit
|
|
3
|
-
description: "[Devflow] Audits error handling, logging, fallbacks, and observability for silent failures. Use after adding catch blocks, fallback logic, retry handling, optional chaining, or user-facing failure paths."
|
|
4
|
-
---
|
|
5
|
-
|
|
6
|
-
# Silent Failure Audit
|
|
7
|
-
|
|
8
|
-
Use this skill to find errors that disappear, get logged without user impact, or fall back in ways that make failures hard to debug.
|
|
9
|
-
|
|
10
|
-
## Core Rule
|
|
11
|
-
|
|
12
|
-
Every important failure must be visible to the right audience: logged with context for maintainers, surfaced with actionable guidance for users, or intentionally propagated to a caller that can handle it.
|
|
13
|
-
|
|
14
|
-
## When To Use
|
|
15
|
-
|
|
16
|
-
- Code adds or changes try/catch, `.catch`, callbacks, result handling, or retry logic.
|
|
17
|
-
- A fallback, default value, mock, optional chaining, or null coalescing branch could hide a failure.
|
|
18
|
-
- A reviewer needs to assess whether error handling is observable and actionable.
|
|
19
|
-
- A production incident involved confusing, missing, or misleading errors.
|
|
20
|
-
|
|
21
|
-
## Audit Scope
|
|
22
|
-
|
|
23
|
-
Review:
|
|
24
|
-
|
|
25
|
-
- Empty or broad catch blocks.
|
|
26
|
-
- Logs that omit operation, identifiers, input shape, or original error.
|
|
27
|
-
- User-facing failures with vague or unactionable messages.
|
|
28
|
-
- Fallbacks that silently substitute data or behavior.
|
|
29
|
-
- Retry exhaustion paths.
|
|
30
|
-
- Optional chaining or default values that mask required data.
|
|
31
|
-
|
|
32
|
-
## Failure Patterns
|
|
33
|
-
|
|
34
|
-
| Pattern | Severity | Why it matters |
|
|
35
|
-
|---|---|---|
|
|
36
|
-
| Empty catch block | Critical | The failure vanishes completely. |
|
|
37
|
-
| Log and continue with no user signal | High | Operators may see logs, but users think the action worked. |
|
|
38
|
-
| Silent default/null return | High | Callers cannot distinguish no data from failed data. |
|
|
39
|
-
| Broad catch-all around unrelated work | High | Unexpected bugs are converted into normal control flow. |
|
|
40
|
-
| Fallback without explicit visibility | Medium | Users and maintainers cannot tell degraded mode is active. |
|
|
41
|
-
| Generic message only | Medium | Users cannot recover and maintainers lack breadcrumbs. |
|
|
42
|
-
|
|
43
|
-
## Output Format
|
|
44
|
-
|
|
45
|
-
```markdown
|
|
46
|
-
## Silent Failure Audit: [Scope]
|
|
47
|
-
|
|
48
|
-
### Scope
|
|
49
|
-
- **Reviewing**: [PR diff / file / feature]
|
|
50
|
-
- **Error handlers found**: [count]
|
|
51
|
-
- **Files**: [files]
|
|
52
|
-
|
|
53
|
-
### Findings
|
|
54
|
-
#### [Severity] [Title]
|
|
55
|
-
**Location**: `path/file.ts:45`
|
|
56
|
-
**Pattern**: [Empty catch / silent fallback / generic message]
|
|
57
|
-
**Problem**: [What is hidden]
|
|
58
|
-
**User or operator impact**: [Concrete impact]
|
|
59
|
-
**Required fix**: [Specific behavior/logging/message/propagation change]
|
|
60
|
-
|
|
61
|
-
### Positive Patterns
|
|
62
|
-
- `path/file.ts:88` logs operation context and surfaces an actionable user message.
|
|
63
|
-
|
|
64
|
-
### Verdict
|
|
65
|
-
[Pass / Needs fixes / Critical issues]
|
|
66
|
-
```
|
|
67
|
-
|
|
68
|
-
## Verification
|
|
69
|
-
|
|
70
|
-
- Every critical or high finding identifies the hidden failure and concrete user/operator impact.
|
|
71
|
-
- Required fixes preserve useful context from the original error.
|
|
72
|
-
- Fallbacks are documented, observable, and intentional.
|
|
73
|
-
- Passing audits mention the positive patterns that justify the pass.
|
|
@@ -1,91 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: spec-orchestration
|
|
3
|
-
description: "[Devflow] Coordinate Brainstorm, Research, PRD, competitor context, and spec critique before the work re-enters the numbered mainline. Use when a broad idea needs orchestration rather than a straight pass."
|
|
4
|
-
---
|
|
5
|
-
|
|
6
|
-
# Spec Orchestration
|
|
7
|
-
|
|
8
|
-
## Overview
|
|
9
|
-
|
|
10
|
-
This skill is the shared behavior layer behind `Spec-Orchestrate`.
|
|
11
|
-
|
|
12
|
-
It keeps the valuable orchestration UX from the legacy prompt while making sure DevFlow 2.0 still routes back into the numbered mainline cleanly.
|
|
13
|
-
|
|
14
|
-
## Related Assets
|
|
15
|
-
|
|
16
|
-
This skill should reuse and align with:
|
|
17
|
-
|
|
18
|
-
- `.agent/resources/schemas/spec_orchestration.template.md`
|
|
19
|
-
- `Brainstorm`
|
|
20
|
-
- `Research`
|
|
21
|
-
- `Spec-Research`
|
|
22
|
-
- `Competitor`
|
|
23
|
-
- `PRD`
|
|
24
|
-
|
|
25
|
-
## When to Use
|
|
26
|
-
|
|
27
|
-
- when an idea is too broad for one straight pass through Discover, Define, and Spec
|
|
28
|
-
- when multiple evidence lanes are needed before a real delivery contract can be written
|
|
29
|
-
- when the user needs orchestration, not autonomous execution
|
|
30
|
-
|
|
31
|
-
## Process
|
|
32
|
-
|
|
33
|
-
### 1. Clarify The Idea
|
|
34
|
-
|
|
35
|
-
Identify:
|
|
36
|
-
|
|
37
|
-
- target user
|
|
38
|
-
- core problem
|
|
39
|
-
- desired outcome
|
|
40
|
-
- constraints
|
|
41
|
-
- acceptance signals
|
|
42
|
-
|
|
43
|
-
### 2. Choose Only The Needed Support
|
|
44
|
-
|
|
45
|
-
Route proportionally:
|
|
46
|
-
|
|
47
|
-
- `Brainstorm` for option generation
|
|
48
|
-
- `Research` or `Spec-Research` for external or technical proof
|
|
49
|
-
- `Competitor` for market context
|
|
50
|
-
- `PRD` for product framing
|
|
51
|
-
- `10-define` for scope and decision locking
|
|
52
|
-
- `20-spec` for delivery contract writing
|
|
53
|
-
|
|
54
|
-
Do not stack every support surface by default.
|
|
55
|
-
|
|
56
|
-
### 3. Critique Before Execution
|
|
57
|
-
|
|
58
|
-
Check for:
|
|
59
|
-
|
|
60
|
-
- missing acceptance criteria
|
|
61
|
-
- unverified dependencies
|
|
62
|
-
- unclear boundaries
|
|
63
|
-
- hidden risks
|
|
64
|
-
- ambiguous UX, API, or data behavior
|
|
65
|
-
- wrong sequencing between framing and execution
|
|
66
|
-
|
|
67
|
-
### 4. Save A Reusable Report
|
|
68
|
-
|
|
69
|
-
Write the orchestration report to:
|
|
70
|
-
|
|
71
|
-
```text
|
|
72
|
-
devflow/reports/{date}-{slug}-spec-orchestration.md
|
|
73
|
-
```
|
|
74
|
-
|
|
75
|
-
Use `.agent/resources/schemas/spec_orchestration.template.md` and replace placeholder text fully.
|
|
76
|
-
|
|
77
|
-
### 5. Route Back To Mainline
|
|
78
|
-
|
|
79
|
-
- `10-define` when the idea still needs scope decisions
|
|
80
|
-
- `20-spec` when enough evidence exists to write the delivery contract
|
|
81
|
-
- `30-plan` only when the spec is already solid
|
|
82
|
-
|
|
83
|
-
## Output
|
|
84
|
-
|
|
85
|
-
Return:
|
|
86
|
-
|
|
87
|
-
- what is already stable
|
|
88
|
-
- what still needs proof or discovery
|
|
89
|
-
- the supporting commands chosen
|
|
90
|
-
- where the orchestration report was saved
|
|
91
|
-
- the exact next command
|
|
@@ -1,107 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: specialist-agent-routing
|
|
3
|
-
description: "[Devflow] Route a request to the right specialist agent and preserve the DevFlow 2.0 stage-first contract. Use when the user wants direct specialist judgment on a file, folder, workspace, or concern."
|
|
4
|
-
---
|
|
5
|
-
|
|
6
|
-
# Specialist Agent Routing
|
|
7
|
-
|
|
8
|
-
## Overview
|
|
9
|
-
|
|
10
|
-
This skill is the shared behavior layer behind `Agent`.
|
|
11
|
-
|
|
12
|
-
It keeps direct specialist invocation available for advanced use, but makes clear that the owning workflow or mainline stage still controls lifecycle state.
|
|
13
|
-
|
|
14
|
-
## When to Use
|
|
15
|
-
|
|
16
|
-
- when the user explicitly wants a specialist persona
|
|
17
|
-
- when a stage needs deeper focused judgment on a bounded target
|
|
18
|
-
- when a file, folder, running ID workspace, or codebase concern maps better to one expert than to a whole workflow
|
|
19
|
-
|
|
20
|
-
## Core Rule
|
|
21
|
-
|
|
22
|
-
`Agent` is not a mainline stage and should not replace the owning workflow.
|
|
23
|
-
|
|
24
|
-
Use it to add focused expertise, then route back to the correct lifecycle step.
|
|
25
|
-
|
|
26
|
-
## Input Shape
|
|
27
|
-
|
|
28
|
-
```text
|
|
29
|
-
Agent {AGENT_NAME} {TARGET}
|
|
30
|
-
```
|
|
31
|
-
|
|
32
|
-
Examples:
|
|
33
|
-
|
|
34
|
-
```text
|
|
35
|
-
Agent requirements-engineer devflow/runs/007/spec.md
|
|
36
|
-
Agent codebase-explorer src/services/
|
|
37
|
-
Agent code-reviewer devflow/runs/007
|
|
38
|
-
```
|
|
39
|
-
|
|
40
|
-
## Process
|
|
41
|
-
|
|
42
|
-
### 1. Load The Specialist
|
|
43
|
-
|
|
44
|
-
Read:
|
|
45
|
-
|
|
46
|
-
- `.agent/agents/{AGENT_NAME}.md`
|
|
47
|
-
- the minimum necessary target context
|
|
48
|
-
- relevant stage artifacts when the target belongs to a running ID
|
|
49
|
-
|
|
50
|
-
### 2. Preserve Stage-First Contract
|
|
51
|
-
|
|
52
|
-
- prefer `discover.md`, `define.md`, `spec.md`, `plan.md`, `implement.md`, `verify.md`, `release.md`, and `report.md`
|
|
53
|
-
- do not silently mutate legacy JSON contracts
|
|
54
|
-
- if the agent is only reviewing, return recommendations or save a report instead of pretending to execute the whole lifecycle
|
|
55
|
-
|
|
56
|
-
### 3. Apply Specialist Discipline
|
|
57
|
-
|
|
58
|
-
Examples:
|
|
59
|
-
|
|
60
|
-
- `code-reviewer` -> correctness, risk, maintainability, findings-first review
|
|
61
|
-
- `prp-core-debugger` -> RCA and validation evidence
|
|
62
|
-
- `requirements-engineer` -> scope, ambiguity, acceptance criteria
|
|
63
|
-
- `codebase-explorer` -> architecture, pattern, and data flow investigation
|
|
64
|
-
|
|
65
|
-
### 4. Save A Reusable Report When Needed
|
|
66
|
-
|
|
67
|
-
For substantial outputs, save to:
|
|
68
|
-
|
|
69
|
-
```text
|
|
70
|
-
devflow/reports/{date}-{agent-name}-{timestamp}.md
|
|
71
|
-
```
|
|
72
|
-
|
|
73
|
-
Use `.agent/resources/schemas/agent_report.template.md` unless the invoking workflow provides a more specific template.
|
|
74
|
-
|
|
75
|
-
### 5. Route Back
|
|
76
|
-
|
|
77
|
-
Return to the owning step:
|
|
78
|
-
|
|
79
|
-
- `10-define`
|
|
80
|
-
- `20-spec`
|
|
81
|
-
- `30-plan`
|
|
82
|
-
- `40-implement`
|
|
83
|
-
- `50-verify`
|
|
84
|
-
- `70-release`
|
|
85
|
-
- `60-report`
|
|
86
|
-
- `PR-Review`
|
|
87
|
-
- `Research`
|
|
88
|
-
|
|
89
|
-
## Skills That Should Not Be Invoked As Agents
|
|
90
|
-
|
|
91
|
-
These former narrow agents now belong in the skill layer:
|
|
92
|
-
|
|
93
|
-
- `code-simplification`
|
|
94
|
-
- `type-design`
|
|
95
|
-
- `silent-failure-audit`
|
|
96
|
-
|
|
97
|
-
Invoke them through the responsible workflow or agent instead.
|
|
98
|
-
|
|
99
|
-
## Output
|
|
100
|
-
|
|
101
|
-
Return:
|
|
102
|
-
|
|
103
|
-
- agent used
|
|
104
|
-
- target reviewed
|
|
105
|
-
- concise findings or changes
|
|
106
|
-
- report path when a substantial report was written
|
|
107
|
-
- exact recommended next route
|