@jakkrichm/create-nexus-devflow 2.0.11 → 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/bin/create-nexus-devflow.js +15 -3
- 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,300 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: git-workflow-and-versioning
|
|
3
|
-
description: "[Devflow] Structures git workflow practices. Use when making any code change. Use when committing, branching, resolving conflicts, or when you need to organize work across multiple parallel streams."
|
|
4
|
-
---
|
|
5
|
-
|
|
6
|
-
# Git Workflow and Versioning
|
|
7
|
-
|
|
8
|
-
## Overview
|
|
9
|
-
|
|
10
|
-
Git is your safety net. Treat commits as save points, branches as sandboxes, and history as documentation. With AI agents generating code at high speed, disciplined version control is the mechanism that keeps changes manageable, reviewable, and reversible.
|
|
11
|
-
|
|
12
|
-
## When to Use
|
|
13
|
-
|
|
14
|
-
Always. Every code change flows through git.
|
|
15
|
-
|
|
16
|
-
## Core Principles
|
|
17
|
-
|
|
18
|
-
### Trunk-Based Development (Recommended)
|
|
19
|
-
|
|
20
|
-
Keep `main` always deployable. Work in short-lived feature branches that merge back within 1-3 days. Long-lived development branches are hidden costs — they diverge, create merge conflicts, and delay integration. DORA research consistently shows trunk-based development correlates with high-performing engineering teams.
|
|
21
|
-
|
|
22
|
-
```
|
|
23
|
-
main ──●──●──●──●──●──●──●──●──●── (always deployable)
|
|
24
|
-
╲ ╱ ╲ ╱
|
|
25
|
-
●──●─╱ ●──╱ ← short-lived feature branches (1-3 days)
|
|
26
|
-
```
|
|
27
|
-
|
|
28
|
-
This is the recommended default. Teams using gitflow or long-lived branches can adapt the principles (atomic commits, small changes, descriptive messages) to their branching model — the commit discipline matters more than the specific branching strategy.
|
|
29
|
-
|
|
30
|
-
- **Dev branches are costs.** Every day a branch lives, it accumulates merge risk.
|
|
31
|
-
- **Release branches are acceptable.** When you need to stabilize a release while main moves forward.
|
|
32
|
-
- **Feature flags > long branches.** Prefer deploying incomplete work behind flags rather than keeping it on a branch for weeks.
|
|
33
|
-
|
|
34
|
-
### 1. Commit Early, Commit Often
|
|
35
|
-
|
|
36
|
-
Each successful increment gets its own commit. Don't accumulate large uncommitted changes.
|
|
37
|
-
|
|
38
|
-
```
|
|
39
|
-
Work pattern:
|
|
40
|
-
Implement slice → Test → Verify → Commit → Next slice
|
|
41
|
-
|
|
42
|
-
Not this:
|
|
43
|
-
Implement everything → Hope it works → Giant commit
|
|
44
|
-
```
|
|
45
|
-
|
|
46
|
-
Commits are save points. If the next change breaks something, you can revert to the last known-good state instantly.
|
|
47
|
-
|
|
48
|
-
### 2. Atomic Commits
|
|
49
|
-
|
|
50
|
-
Each commit does one logical thing:
|
|
51
|
-
|
|
52
|
-
```
|
|
53
|
-
# Good: Each commit is self-contained
|
|
54
|
-
git log --oneline
|
|
55
|
-
a1b2c3d Add task creation endpoint with validation
|
|
56
|
-
d4e5f6g Add task creation form component
|
|
57
|
-
h7i8j9k Connect form to API and add loading state
|
|
58
|
-
m1n2o3p Add task creation tests (unit + integration)
|
|
59
|
-
|
|
60
|
-
# Bad: Everything mixed together
|
|
61
|
-
git log --oneline
|
|
62
|
-
x1y2z3a Add task feature, fix sidebar, update deps, refactor utils
|
|
63
|
-
```
|
|
64
|
-
|
|
65
|
-
### 3. Descriptive Messages
|
|
66
|
-
|
|
67
|
-
Commit messages explain the *why*, not just the *what*:
|
|
68
|
-
|
|
69
|
-
```
|
|
70
|
-
# Good: Explains intent
|
|
71
|
-
feat: add email validation to registration endpoint
|
|
72
|
-
|
|
73
|
-
Prevents invalid email formats from reaching the database.
|
|
74
|
-
Uses Zod schema validation at the route handler level,
|
|
75
|
-
consistent with existing validation patterns in auth.ts.
|
|
76
|
-
|
|
77
|
-
# Bad: Describes what's obvious from the diff
|
|
78
|
-
update auth.ts
|
|
79
|
-
```
|
|
80
|
-
|
|
81
|
-
**Format:**
|
|
82
|
-
```
|
|
83
|
-
<type>: <short description>
|
|
84
|
-
|
|
85
|
-
<optional body explaining why, not what>
|
|
86
|
-
```
|
|
87
|
-
|
|
88
|
-
**Types:**
|
|
89
|
-
- `feat` — New feature
|
|
90
|
-
- `fix` — Bug fix
|
|
91
|
-
- `refactor` — Code change that neither fixes a bug nor adds a feature
|
|
92
|
-
- `test` — Adding or updating tests
|
|
93
|
-
- `docs` — Documentation only
|
|
94
|
-
- `chore` — Tooling, dependencies, config
|
|
95
|
-
|
|
96
|
-
### 4. Keep Concerns Separate
|
|
97
|
-
|
|
98
|
-
Don't combine formatting changes with behavior changes. Don't combine refactors with features. Each type of change should be a separate commit — and ideally a separate PR:
|
|
99
|
-
|
|
100
|
-
```
|
|
101
|
-
# Good: Separate concerns
|
|
102
|
-
git commit -m "refactor: extract validation logic to shared utility"
|
|
103
|
-
git commit -m "feat: add phone number validation to registration"
|
|
104
|
-
|
|
105
|
-
# Bad: Mixed concerns
|
|
106
|
-
git commit -m "refactor validation and add phone number field"
|
|
107
|
-
```
|
|
108
|
-
|
|
109
|
-
**Separate refactoring from feature work.** A refactoring change and a feature change are two different changes — submit them separately. This makes each change easier to review, revert, and understand in history. Small cleanups (renaming a variable) can be included in a feature commit at reviewer discretion.
|
|
110
|
-
|
|
111
|
-
### 5. Size Your Changes
|
|
112
|
-
|
|
113
|
-
Target ~100 lines per commit/PR. Changes over ~1000 lines should be split. See the splitting strategies in `code-review-and-quality` for how to break down large changes.
|
|
114
|
-
|
|
115
|
-
```
|
|
116
|
-
~100 lines → Easy to review, easy to revert
|
|
117
|
-
~300 lines → Acceptable for a single logical change
|
|
118
|
-
~1000 lines → Split into smaller changes
|
|
119
|
-
```
|
|
120
|
-
|
|
121
|
-
## Branching Strategy
|
|
122
|
-
|
|
123
|
-
### Feature Branches
|
|
124
|
-
|
|
125
|
-
```
|
|
126
|
-
main (always deployable)
|
|
127
|
-
│
|
|
128
|
-
├── feature/task-creation ← One feature per branch
|
|
129
|
-
├── feature/user-settings ← Parallel work
|
|
130
|
-
└── fix/duplicate-tasks ← Bug fixes
|
|
131
|
-
```
|
|
132
|
-
|
|
133
|
-
- Branch from `main` (or the team's default branch)
|
|
134
|
-
- Keep branches short-lived (merge within 1-3 days) — long-lived branches are hidden costs
|
|
135
|
-
- Delete branches after merge
|
|
136
|
-
- Prefer feature flags over long-lived branches for incomplete features
|
|
137
|
-
|
|
138
|
-
### Branch Naming
|
|
139
|
-
|
|
140
|
-
```
|
|
141
|
-
feature/<short-description> → feature/task-creation
|
|
142
|
-
fix/<short-description> → fix/duplicate-tasks
|
|
143
|
-
chore/<short-description> → chore/update-deps
|
|
144
|
-
refactor/<short-description> → refactor/auth-module
|
|
145
|
-
```
|
|
146
|
-
|
|
147
|
-
## Working with Worktrees
|
|
148
|
-
|
|
149
|
-
For parallel AI agent work, use git worktrees to run multiple branches simultaneously:
|
|
150
|
-
|
|
151
|
-
```bash
|
|
152
|
-
# Create a worktree for a feature branch
|
|
153
|
-
git worktree add ../project-feature-a feature/task-creation
|
|
154
|
-
git worktree add ../project-feature-b feature/user-settings
|
|
155
|
-
|
|
156
|
-
# Each worktree is a separate directory with its own branch
|
|
157
|
-
# Agents can work in parallel without interfering
|
|
158
|
-
ls ../
|
|
159
|
-
project/ ← main branch
|
|
160
|
-
project-feature-a/ ← task-creation branch
|
|
161
|
-
project-feature-b/ ← user-settings branch
|
|
162
|
-
|
|
163
|
-
# When done, merge and clean up
|
|
164
|
-
git worktree remove ../project-feature-a
|
|
165
|
-
```
|
|
166
|
-
|
|
167
|
-
Benefits:
|
|
168
|
-
- Multiple agents can work on different features simultaneously
|
|
169
|
-
- No branch switching needed (each directory has its own branch)
|
|
170
|
-
- If one experiment fails, delete the worktree — nothing is lost
|
|
171
|
-
- Changes are isolated until explicitly merged
|
|
172
|
-
|
|
173
|
-
## The Save Point Pattern
|
|
174
|
-
|
|
175
|
-
```
|
|
176
|
-
Agent starts work
|
|
177
|
-
│
|
|
178
|
-
├── Makes a change
|
|
179
|
-
│ ├── Test passes? → Commit → Continue
|
|
180
|
-
│ └── Test fails? → Revert to last commit → Investigate
|
|
181
|
-
│
|
|
182
|
-
├── Makes another change
|
|
183
|
-
│ ├── Test passes? → Commit → Continue
|
|
184
|
-
│ └── Test fails? → Revert to last commit → Investigate
|
|
185
|
-
│
|
|
186
|
-
└── Feature complete → All commits form a clean history
|
|
187
|
-
```
|
|
188
|
-
|
|
189
|
-
This pattern means you never lose more than one increment of work. If an agent goes off the rails, stop, inspect `git status`, and ask for explicit approval before any destructive reset back to the last successful state.
|
|
190
|
-
|
|
191
|
-
## Change Summaries
|
|
192
|
-
|
|
193
|
-
After any modification, provide a structured summary. This makes review easier, documents scope discipline, and surfaces unintended changes:
|
|
194
|
-
|
|
195
|
-
```
|
|
196
|
-
CHANGES MADE:
|
|
197
|
-
- src/routes/tasks.ts: Added validation middleware to POST endpoint
|
|
198
|
-
- src/lib/validation.ts: Added TaskCreateSchema using Zod
|
|
199
|
-
|
|
200
|
-
THINGS I DIDN'T TOUCH (intentionally):
|
|
201
|
-
- src/routes/auth.ts: Has similar validation gap but out of scope
|
|
202
|
-
- src/middleware/error.ts: Error format could be improved (separate task)
|
|
203
|
-
|
|
204
|
-
POTENTIAL CONCERNS:
|
|
205
|
-
- The Zod schema is strict — rejects extra fields. Confirm this is desired.
|
|
206
|
-
- Added zod as a dependency (72KB gzipped) — already in package.json
|
|
207
|
-
```
|
|
208
|
-
|
|
209
|
-
This pattern catches wrong assumptions early and gives reviewers a clear map of the change. The "DIDN'T TOUCH" section is especially important — it shows you exercised scope discipline and didn't go on an unsolicited renovation.
|
|
210
|
-
|
|
211
|
-
## Pre-Commit Hygiene
|
|
212
|
-
|
|
213
|
-
Before every commit:
|
|
214
|
-
|
|
215
|
-
```bash
|
|
216
|
-
# 1. Check what you're about to commit
|
|
217
|
-
git diff --staged
|
|
218
|
-
|
|
219
|
-
# 2. Ensure no secrets
|
|
220
|
-
git diff --staged | grep -i "password\|secret\|api_key\|token"
|
|
221
|
-
|
|
222
|
-
# 3. Run tests
|
|
223
|
-
npm test
|
|
224
|
-
|
|
225
|
-
# 4. Run linting
|
|
226
|
-
npm run lint
|
|
227
|
-
|
|
228
|
-
# 5. Run type checking
|
|
229
|
-
npx tsc --noEmit
|
|
230
|
-
```
|
|
231
|
-
|
|
232
|
-
Automate this with git hooks:
|
|
233
|
-
|
|
234
|
-
```json
|
|
235
|
-
// package.json (using lint-staged + husky)
|
|
236
|
-
{
|
|
237
|
-
"lint-staged": {
|
|
238
|
-
"*.{ts,tsx}": ["eslint --fix", "prettier --write"],
|
|
239
|
-
"*.{json,md}": ["prettier --write"]
|
|
240
|
-
}
|
|
241
|
-
}
|
|
242
|
-
```
|
|
243
|
-
|
|
244
|
-
## Handling Generated Files
|
|
245
|
-
|
|
246
|
-
- **Commit generated files** only if the project expects them (e.g., `package-lock.json`, Prisma migrations)
|
|
247
|
-
- **Don't commit** build output (`dist/`, `.next/`), environment files (`.env`), or IDE config (`.vscode/settings.json` unless shared)
|
|
248
|
-
- **Have a `.gitignore`** that covers: `node_modules/`, `dist/`, `.env`, `.env.local`, `*.pem`
|
|
249
|
-
|
|
250
|
-
## Using Git for Debugging
|
|
251
|
-
|
|
252
|
-
```bash
|
|
253
|
-
# Find which commit introduced a bug
|
|
254
|
-
git bisect start
|
|
255
|
-
git bisect bad HEAD
|
|
256
|
-
git bisect good <known-good-commit>
|
|
257
|
-
# Git checkouts midpoints; run your test at each to narrow down
|
|
258
|
-
|
|
259
|
-
# View what changed recently
|
|
260
|
-
git log --oneline -20
|
|
261
|
-
git diff HEAD~5..HEAD -- src/
|
|
262
|
-
|
|
263
|
-
# Find who last changed a specific line
|
|
264
|
-
git blame src/services/task.ts
|
|
265
|
-
|
|
266
|
-
# Search commit messages for a keyword
|
|
267
|
-
git log --grep="validation" --oneline
|
|
268
|
-
```
|
|
269
|
-
|
|
270
|
-
## Common Rationalizations
|
|
271
|
-
|
|
272
|
-
| Rationalization | Reality |
|
|
273
|
-
|---|---|
|
|
274
|
-
| "I'll commit when the feature is done" | One giant commit is impossible to review, debug, or revert. Commit each slice. |
|
|
275
|
-
| "The message doesn't matter" | Messages are documentation. Future you (and future agents) will need to understand what changed and why. |
|
|
276
|
-
| "I'll squash it all later" | Squashing destroys the development narrative. Prefer clean incremental commits from the start. |
|
|
277
|
-
| "Branches add overhead" | Short-lived branches are free and prevent conflicting work from colliding. Long-lived branches are the problem — merge within 1-3 days. |
|
|
278
|
-
| "I'll split this change later" | Large changes are harder to review, riskier to deploy, and harder to revert. Split before submitting, not after. |
|
|
279
|
-
| "I don't need a .gitignore" | Until `.env` with production secrets gets committed. Set it up immediately. |
|
|
280
|
-
|
|
281
|
-
## Red Flags
|
|
282
|
-
|
|
283
|
-
- Large uncommitted changes accumulating
|
|
284
|
-
- Commit messages like "fix", "update", "misc"
|
|
285
|
-
- Formatting changes mixed with behavior changes
|
|
286
|
-
- No `.gitignore` in the project
|
|
287
|
-
- Committing `node_modules/`, `.env`, or build artifacts
|
|
288
|
-
- Long-lived branches that diverge significantly from main
|
|
289
|
-
- Force-pushing to shared branches
|
|
290
|
-
|
|
291
|
-
## Verification
|
|
292
|
-
|
|
293
|
-
For every commit:
|
|
294
|
-
|
|
295
|
-
- [ ] Commit does one logical thing
|
|
296
|
-
- [ ] Message explains the why, follows type conventions
|
|
297
|
-
- [ ] Tests pass before committing
|
|
298
|
-
- [ ] No secrets in the diff
|
|
299
|
-
- [ ] No formatting-only changes mixed with behavior changes
|
|
300
|
-
- [ ] `.gitignore` covers standard exclusions
|
|
@@ -1,74 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: human-review-decisions
|
|
3
|
-
description: "[Devflow] Handle human approval, feedback, rejection, and recheck decisions around verified work. Use when a task needs a human gate after verification and the result must route cleanly back into the DevFlow 2.0 lifecycle."
|
|
4
|
-
---
|
|
5
|
-
|
|
6
|
-
# Human Review Decisions
|
|
7
|
-
|
|
8
|
-
## Overview
|
|
9
|
-
|
|
10
|
-
This skill handles the human review loop after implementation and verification. It does not replace mainline stages; it records or advises the human decision and routes the task back to the right stage.
|
|
11
|
-
|
|
12
|
-
## Supported Modes
|
|
13
|
-
|
|
14
|
-
- `approve`
|
|
15
|
-
- `feedback`
|
|
16
|
-
- `reject`
|
|
17
|
-
- `recheck`
|
|
18
|
-
|
|
19
|
-
## When to Use
|
|
20
|
-
|
|
21
|
-
- After `50-verify` when a human must decide whether the work is acceptable
|
|
22
|
-
- When verified work needs non-blocking feedback
|
|
23
|
-
- When review fails and the task must be sent back with explicit action items
|
|
24
|
-
- When a read-only re-evaluation is needed before the human chooses a decision
|
|
25
|
-
|
|
26
|
-
## Process
|
|
27
|
-
|
|
28
|
-
### 1. Load The Review Context
|
|
29
|
-
|
|
30
|
-
Prefer these artifacts:
|
|
31
|
-
|
|
32
|
-
- `verify.md`
|
|
33
|
-
- `implement.md`
|
|
34
|
-
- `release.md`
|
|
35
|
-
- `report.md`
|
|
36
|
-
|
|
37
|
-
Read only what is necessary to understand the review question and evidence.
|
|
38
|
-
|
|
39
|
-
### 2. Apply The Requested Decision
|
|
40
|
-
|
|
41
|
-
#### Approve
|
|
42
|
-
|
|
43
|
-
- confirm the task is genuinely review-ready
|
|
44
|
-
- record approval in the current review-facing artifact
|
|
45
|
-
- route to `60-report` first, then `70-release` when release execution is the remaining step
|
|
46
|
-
|
|
47
|
-
#### Feedback
|
|
48
|
-
|
|
49
|
-
- record requested improvements without treating them as hard rejection
|
|
50
|
-
- preserve the feedback round and source
|
|
51
|
-
- route to `40-implement` or `30-plan` if the feedback changes scope
|
|
52
|
-
|
|
53
|
-
#### Reject
|
|
54
|
-
|
|
55
|
-
- record rejection reason and concrete action items
|
|
56
|
-
- route to `40-implement`, `30-plan`, or `Debug` depending on the issue
|
|
57
|
-
|
|
58
|
-
#### Recheck
|
|
59
|
-
|
|
60
|
-
- keep the action read-only unless the user explicitly asks to record it
|
|
61
|
-
- evaluate the evidence and recommend `approve`, `feedback`, `reject`, or `50-verify`
|
|
62
|
-
|
|
63
|
-
### 3. Record Reusable Lessons
|
|
64
|
-
|
|
65
|
-
If the review reveals a durable team lesson, add it to `devflow/lessons.md` or route to `Wiki`.
|
|
66
|
-
|
|
67
|
-
## Output
|
|
68
|
-
|
|
69
|
-
Return:
|
|
70
|
-
|
|
71
|
-
- decision taken
|
|
72
|
-
- artifact updated or not updated
|
|
73
|
-
- next recommended stage or companion command
|
|
74
|
-
- any reusable lesson worth capturing
|
|
@@ -1,178 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: idea-refine
|
|
3
|
-
description: "[Devflow] Refines ideas iteratively. Refine ideas through structured divergent and convergent thinking. Use "idea-refine" or "ideate" to trigger."
|
|
4
|
-
---
|
|
5
|
-
|
|
6
|
-
# Idea Refine
|
|
7
|
-
|
|
8
|
-
Refines raw ideas into sharp, actionable concepts worth building through structured divergent and convergent thinking.
|
|
9
|
-
|
|
10
|
-
## How It Works
|
|
11
|
-
|
|
12
|
-
1. **Understand & Expand (Divergent):** Restate the idea, ask sharpening questions, and generate variations.
|
|
13
|
-
2. **Evaluate & Converge:** Cluster ideas, stress-test them, and surface hidden assumptions.
|
|
14
|
-
3. **Sharpen & Ship:** Produce a concrete markdown one-pager moving work forward.
|
|
15
|
-
|
|
16
|
-
## Usage
|
|
17
|
-
|
|
18
|
-
This skill is primarily an interactive dialogue. Invoke it with an idea, and the agent will guide you through the process.
|
|
19
|
-
|
|
20
|
-
```bash
|
|
21
|
-
# Optional: Initialize the ideas directory
|
|
22
|
-
bash /mnt/skills/user/idea-refine/scripts/idea-refine.sh
|
|
23
|
-
```
|
|
24
|
-
|
|
25
|
-
**Trigger Phrases:**
|
|
26
|
-
- "Help me refine this idea"
|
|
27
|
-
- "Ideate on [concept]"
|
|
28
|
-
- "Stress-test my plan"
|
|
29
|
-
|
|
30
|
-
## Output
|
|
31
|
-
|
|
32
|
-
The final output is a markdown one-pager saved to `docs/ideas/[idea-name].md` (after user confirmation), containing:
|
|
33
|
-
- Problem Statement
|
|
34
|
-
- Recommended Direction
|
|
35
|
-
- Key Assumptions
|
|
36
|
-
- MVP Scope
|
|
37
|
-
- Not Doing list
|
|
38
|
-
|
|
39
|
-
## Detailed Instructions
|
|
40
|
-
|
|
41
|
-
You are an ideation partner. Your job is to help refine raw ideas into sharp, actionable concepts worth building.
|
|
42
|
-
|
|
43
|
-
### Philosophy
|
|
44
|
-
|
|
45
|
-
- Simplicity is the ultimate sophistication. Push toward the simplest version that still solves the real problem.
|
|
46
|
-
- Start with the user experience, work backwards to technology.
|
|
47
|
-
- Say no to 1,000 things. Focus beats breadth.
|
|
48
|
-
- Challenge every assumption. "How it's usually done" is not a reason.
|
|
49
|
-
- Show people the future — don't just give them better horses.
|
|
50
|
-
- The parts you can't see should be as beautiful as the parts you can.
|
|
51
|
-
|
|
52
|
-
### Process
|
|
53
|
-
|
|
54
|
-
When the user invokes this skill with an idea (`$ARGUMENTS`), guide them through three phases. Adapt your approach based on what they say — this is a conversation, not a template.
|
|
55
|
-
|
|
56
|
-
#### Phase 1: Understand & Expand (Divergent)
|
|
57
|
-
|
|
58
|
-
**Goal:** Take the raw idea and open it up.
|
|
59
|
-
|
|
60
|
-
1. **Restate the idea** as a crisp "How Might We" problem statement. This forces clarity on what's actually being solved.
|
|
61
|
-
|
|
62
|
-
2. **Ask 3-5 sharpening questions** — no more. Focus on:
|
|
63
|
-
- Who is this for, specifically?
|
|
64
|
-
- What does success look like?
|
|
65
|
-
- What are the real constraints (time, tech, resources)?
|
|
66
|
-
- What's been tried before?
|
|
67
|
-
- Why now?
|
|
68
|
-
|
|
69
|
-
Use the `AskUserQuestion` tool to gather this input. Do NOT proceed until you understand who this is for and what success looks like.
|
|
70
|
-
|
|
71
|
-
3. **Generate 5-8 idea variations** using these lenses:
|
|
72
|
-
- **Inversion:** "What if we did the opposite?"
|
|
73
|
-
- **Constraint removal:** "What if budget/time/tech weren't factors?"
|
|
74
|
-
- **Audience shift:** "What if this were for [different user]?"
|
|
75
|
-
- **Combination:** "What if we merged this with [adjacent idea]?"
|
|
76
|
-
- **Simplification:** "What's the version that's 10x simpler?"
|
|
77
|
-
- **10x version:** "What would this look like at massive scale?"
|
|
78
|
-
- **Expert lens:** "What would [domain] experts find obvious that outsiders wouldn't?"
|
|
79
|
-
|
|
80
|
-
Push beyond what the user initially asked for. Create products people don't know they need yet.
|
|
81
|
-
|
|
82
|
-
**If running inside a codebase:** Use `Glob`, `Grep`, and `Read` to scan for relevant context — existing architecture, patterns, constraints, prior art. Ground your variations in what actually exists. Reference specific files and patterns when relevant.
|
|
83
|
-
|
|
84
|
-
Read `frameworks.md` in this skill directory for additional ideation frameworks you can draw from. Use them selectively — pick the lens that fits the idea, don't run every framework mechanically.
|
|
85
|
-
|
|
86
|
-
#### Phase 2: Evaluate & Converge
|
|
87
|
-
|
|
88
|
-
After the user reacts to Phase 1 (indicates which ideas resonate, pushes back, adds context), shift to convergent mode:
|
|
89
|
-
|
|
90
|
-
1. **Cluster** the ideas that resonated into 2-3 distinct directions. Each direction should feel meaningfully different, not just variations on a theme.
|
|
91
|
-
|
|
92
|
-
2. **Stress-test** each direction against three criteria:
|
|
93
|
-
- **User value:** Who benefits and how much? Is this a painkiller or a vitamin?
|
|
94
|
-
- **Feasibility:** What's the technical and resource cost? What's the hardest part?
|
|
95
|
-
- **Differentiation:** What makes this genuinely different? Would someone switch from their current solution?
|
|
96
|
-
|
|
97
|
-
Read `refinement-criteria.md` in this skill directory for the full evaluation rubric.
|
|
98
|
-
|
|
99
|
-
3. **Surface hidden assumptions.** For each direction, explicitly name:
|
|
100
|
-
- What you're betting is true (but haven't validated)
|
|
101
|
-
- What could kill this idea
|
|
102
|
-
- What you're choosing to ignore (and why that's okay for now)
|
|
103
|
-
|
|
104
|
-
This is where most ideation fails. Don't skip it.
|
|
105
|
-
|
|
106
|
-
**Be honest, not supportive.** If an idea is weak, say so with kindness. A good ideation partner is not a yes-machine. Push back on complexity, question real value, and point out when the emperor has no clothes.
|
|
107
|
-
|
|
108
|
-
#### Phase 3: Sharpen & Ship
|
|
109
|
-
|
|
110
|
-
Produce a concrete artifact — a markdown one-pager that moves work forward:
|
|
111
|
-
|
|
112
|
-
```markdown
|
|
113
|
-
# [Idea Name]
|
|
114
|
-
|
|
115
|
-
## Problem Statement
|
|
116
|
-
[One-sentence "How Might We" framing]
|
|
117
|
-
|
|
118
|
-
## Recommended Direction
|
|
119
|
-
[The chosen direction and why — 2-3 paragraphs max]
|
|
120
|
-
|
|
121
|
-
## Key Assumptions to Validate
|
|
122
|
-
- [ ] [Assumption 1 — how to test it]
|
|
123
|
-
- [ ] [Assumption 2 — how to test it]
|
|
124
|
-
- [ ] [Assumption 3 — how to test it]
|
|
125
|
-
|
|
126
|
-
## MVP Scope
|
|
127
|
-
[The minimum version that tests the core assumption. What's in, what's out.]
|
|
128
|
-
|
|
129
|
-
## Not Doing (and Why)
|
|
130
|
-
- [Thing 1] — [reason]
|
|
131
|
-
- [Thing 2] — [reason]
|
|
132
|
-
- [Thing 3] — [reason]
|
|
133
|
-
|
|
134
|
-
## Open Questions
|
|
135
|
-
- [Question that needs answering before building]
|
|
136
|
-
```
|
|
137
|
-
|
|
138
|
-
**The "Not Doing" list is arguably the most valuable part.** Focus is about saying no to good ideas. Make the trade-offs explicit.
|
|
139
|
-
|
|
140
|
-
Ask the user if they'd like to save this to `docs/ideas/[idea-name].md` (or a location of their choosing). Only save if they confirm.
|
|
141
|
-
|
|
142
|
-
### Anti-patterns to Avoid
|
|
143
|
-
|
|
144
|
-
- **Don't generate 20+ ideas.** Quality over quantity. 5-8 well-considered variations beat 20 shallow ones.
|
|
145
|
-
- **Don't be a yes-machine.** Push back on weak ideas with specificity and kindness.
|
|
146
|
-
- **Don't skip "who is this for."** Every good idea starts with a person and their problem.
|
|
147
|
-
- **Don't produce a plan without surfacing assumptions.** Untested assumptions are the #1 killer of good ideas.
|
|
148
|
-
- **Don't over-engineer the process.** Three phases, each doing one thing well. Resist adding steps.
|
|
149
|
-
- **Don't just list ideas — tell a story.** Each variation should have a reason it exists, not just be a bullet point.
|
|
150
|
-
- **Don't ignore the codebase.** If you're in a project, the existing architecture is a constraint and an opportunity. Use it.
|
|
151
|
-
|
|
152
|
-
### Tone
|
|
153
|
-
|
|
154
|
-
Direct, thoughtful, slightly provocative. You're a sharp thinking partner, not a facilitator reading from a script. Channel the energy of "that's interesting, but what if..." -- always pushing one step further without being exhausting.
|
|
155
|
-
|
|
156
|
-
Read `examples.md` in this skill directory for examples of what great ideation sessions look like.
|
|
157
|
-
|
|
158
|
-
## Red Flags
|
|
159
|
-
|
|
160
|
-
- Generating 20+ shallow variations instead of 5-8 considered ones
|
|
161
|
-
- Skipping the "who is this for" question
|
|
162
|
-
- No assumptions surfaced before committing to a direction
|
|
163
|
-
- Yes-machining weak ideas instead of pushing back with specificity
|
|
164
|
-
- Producing a plan without a "Not Doing" list
|
|
165
|
-
- Ignoring existing codebase constraints when ideating inside a project
|
|
166
|
-
- Jumping straight to Phase 3 output without running Phases 1 and 2
|
|
167
|
-
|
|
168
|
-
## Verification
|
|
169
|
-
|
|
170
|
-
After completing an ideation session:
|
|
171
|
-
|
|
172
|
-
- [ ] A clear "How Might We" problem statement exists
|
|
173
|
-
- [ ] The target user and success criteria are defined
|
|
174
|
-
- [ ] Multiple directions were explored, not just the first idea
|
|
175
|
-
- [ ] Hidden assumptions are explicitly listed with validation strategies
|
|
176
|
-
- [ ] A "Not Doing" list makes trade-offs explicit
|
|
177
|
-
- [ ] The output is a concrete artifact (markdown one-pager), not just conversation
|
|
178
|
-
- [ ] The user confirmed the final direction before any implementation work
|