@jakkrichm/create-nexus-devflow 2.0.24 → 2.0.27
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 +1 -1
- package/dist/bin/create-nexus-devflow.d.ts +18 -1
- package/dist/bin/create-nexus-devflow.js +439 -11
- package/dist/bin/create-nexus-devflow.js.map +1 -1
- package/dist/lib/command-catalog.d.ts +11 -0
- package/dist/lib/command-catalog.js +63 -0
- package/dist/lib/command-catalog.js.map +1 -0
- package/dist/lib/current-work.js +94 -48
- package/dist/lib/current-work.js.map +1 -1
- package/dist/lib/dashboard-page.d.ts +2 -0
- package/dist/lib/dashboard-page.js +201 -0
- package/dist/lib/dashboard-page.js.map +1 -0
- package/dist/lib/dashboard-snapshot.d.ts +49 -0
- package/dist/lib/dashboard-snapshot.js +96 -0
- package/dist/lib/dashboard-snapshot.js.map +1 -0
- package/dist/lib/dashboard.d.ts +2 -0
- package/dist/lib/dashboard.js +360 -8
- package/dist/lib/dashboard.js.map +1 -1
- package/dist/lib/discoveries.d.ts +19 -0
- package/dist/lib/discoveries.js +81 -0
- package/dist/lib/discoveries.js.map +1 -0
- package/dist/lib/doctor.d.ts +26 -0
- package/dist/lib/doctor.js +474 -0
- package/dist/lib/doctor.js.map +1 -0
- package/dist/lib/findings.d.ts +26 -2
- package/dist/lib/findings.js +165 -1
- package/dist/lib/findings.js.map +1 -1
- package/dist/lib/gatekeeper.d.ts +19 -0
- package/dist/lib/gatekeeper.js +77 -0
- package/dist/lib/gatekeeper.js.map +1 -0
- package/dist/lib/git-hooks.d.ts +16 -0
- package/dist/lib/git-hooks.js +78 -0
- package/dist/lib/git-hooks.js.map +1 -0
- package/dist/lib/history.d.ts +6 -1
- package/dist/lib/history.js +98 -11
- package/dist/lib/history.js.map +1 -1
- package/dist/lib/ideas.d.ts +26 -0
- package/dist/lib/ideas.js +179 -0
- package/dist/lib/ideas.js.map +1 -0
- package/dist/lib/status.d.ts +2 -0
- package/dist/lib/status.js +60 -6
- package/dist/lib/status.js.map +1 -1
- package/dist/lib/update.d.ts +9 -3
- package/dist/lib/update.js +129 -22
- package/dist/lib/update.js.map +1 -1
- package/dist/lib/version-check.d.ts +22 -0
- package/dist/lib/version-check.js +59 -0
- package/dist/lib/version-check.js.map +1 -0
- package/dist/lib/workflow-state.d.ts +23 -0
- package/dist/lib/workflow-state.js +112 -0
- package/dist/lib/workflow-state.js.map +1 -0
- package/dist/scripts/prepare-template.js +3 -3
- package/package.json +1 -1
- package/template/{.claude/skills/00-discover → .agents/skills/00-explore}/SKILL.md +10 -10
- package/template/.agents/skills/10-define/SKILL.md +4 -4
- package/template/.agents/skills/20-spec/SKILL.md +2 -3
- package/template/.agents/skills/30-plan/SKILL.md +2 -3
- package/template/.agents/skills/40-execute/SKILL.md +2 -2
- package/template/.agents/skills/50-verify/SKILL.md +2 -2
- package/template/.agents/skills/60-report/SKILL.md +3 -3
- package/template/{.claude/skills/70-release → .agents/skills/70-deliver}/SKILL.md +7 -13
- package/template/.agents/skills/adopt/SKILL.md +136 -73
- package/template/.agents/skills/audit/SKILL.md +275 -0
- package/template/.agents/skills/autopilot/SKILL.md +228 -147
- package/template/.agents/skills/brainstorm/SKILL.md +3 -3
- package/template/.agents/skills/brief/SKILL.md +74 -51
- package/template/.agents/skills/check/SKILL.md +96 -76
- package/template/.agents/skills/ci/SKILL.md +140 -61
- package/template/.agents/skills/complete/SKILL.md +156 -101
- package/template/.agents/skills/convert-any-to-md/SKILL.md +2 -2
- package/template/.agents/skills/debug/SKILL.md +124 -49
- package/template/.agents/skills/devflow/SKILL.md +16 -10
- package/template/.agents/skills/discovery/SKILL.md +166 -0
- package/template/.agents/skills/doctor/SKILL.md +188 -72
- package/template/.agents/skills/feature/SKILL.md +195 -102
- package/template/.agents/skills/fix/SKILL.md +41 -90
- package/template/.agents/skills/idea/SKILL.md +3 -3
- package/template/.agents/skills/implement/SKILL.md +189 -46
- package/template/.agents/skills/onboard/SKILL.md +215 -85
- package/template/.agents/skills/overview/SKILL.md +51 -85
- package/template/.agents/skills/prototype/SKILL.md +82 -27
- package/template/.agents/skills/release/SKILL.md +160 -0
- package/template/.agents/skills/report-html/SKILL.md +2 -2
- package/template/.agents/skills/rollback/SKILL.md +123 -77
- package/template/.agents/skills/status/SKILL.md +109 -0
- package/template/.agents/skills/test/SKILL.md +2 -2
- package/template/.agents/skills/tests/SKILL.md +126 -0
- package/template/.agents/skills/try/SKILL.md +77 -65
- package/template/{.agents/skills/00-discover → .claude/skills/00-explore}/SKILL.md +9 -9
- package/template/.claude/skills/10-define/SKILL.md +2 -2
- package/template/.claude/skills/20-spec/SKILL.md +1 -1
- package/template/.claude/skills/30-plan/SKILL.md +1 -1
- package/template/.claude/skills/60-report/SKILL.md +1 -1
- package/template/{.agents/skills/70-release → .claude/skills/70-deliver}/SKILL.md +6 -12
- package/template/.claude/skills/adopt/SKILL.md +2 -2
- package/template/.claude/skills/audit/SKILL.md +141 -0
- package/template/.claude/skills/autopilot/SKILL.md +74 -52
- package/template/.claude/skills/brief/SKILL.md +67 -45
- package/template/.claude/skills/ci/SKILL.md +1 -1
- package/template/.claude/skills/complete/SKILL.md +1 -1
- package/template/.claude/skills/debug/SKILL.md +1 -1
- package/template/.claude/skills/devflow/SKILL.md +7 -7
- package/template/.claude/skills/discovery/SKILL.md +145 -0
- package/template/.claude/skills/doctor/SKILL.md +1 -1
- package/template/.claude/skills/feature/SKILL.md +67 -18
- package/template/.claude/skills/fix/SKILL.md +1 -1
- package/template/.claude/skills/idea/SKILL.md +2 -2
- package/template/.claude/skills/onboard/SKILL.md +3 -3
- package/template/.claude/skills/overview/SKILL.md +39 -88
- package/template/.claude/skills/prototype/SKILL.md +1 -1
- package/template/.claude/skills/release/SKILL.md +171 -0
- package/template/.claude/skills/rollback/SKILL.md +2 -2
- package/template/AGENTS.md +5 -5
- package/template/devflow/build-plan.md +40 -0
- package/template/devflow/context/ai-interaction.md +4 -4
- package/template/devflow/context/coding-standards.md +3 -3
- package/template/devflow/context/current-stage.md +1 -1
- package/template/devflow/context/findings.md +1 -1
- package/template/devflow/history/HISTORY.md +2 -2
- package/template/devflow/project-plan.md +27 -0
- package/template/devflow/reference/mockup.html +476 -0
- package/template/devflow/reference/project-overview-template.md +39 -0
- package/template/devflow/reference/running-id-contract.md +24 -2
|
@@ -1,175 +1,256 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: autopilot
|
|
3
|
-
description: "[
|
|
3
|
+
description: "[devflow][B] Optional explicit Blueprint mode for one bounded spec/build/check/audit pass. It can pick or resume the current feature, write the spec when needed, create or reuse the branch, implement small steps, run build/tests/checks, create checkpoint commits after passing steps, audit changed code, repair confirmed high-severity findings, and stop with a review packet. It never completes, merges, pushes, deploys, publishes, sends, or performs destructive actions without explicit approval. Use only when the user explicitly runs /autopilot, invokes $autopilot, or directly asks for Autopilot."
|
|
4
4
|
---
|
|
5
5
|
|
|
6
|
-
# autopilot -
|
|
6
|
+
# autopilot - optional Blueprint loop
|
|
7
7
|
|
|
8
8
|
Where this sits in the workflow:
|
|
9
9
|
|
|
10
|
-
|
|
11
|
-
|
|
12
|
-
|
|
13
|
-
are we?) build, verify, walkthrough) merge with approval)
|
|
14
|
-
report)
|
|
15
|
-
```
|
|
10
|
+
/status -> [autopilot] -> review packet -> /complete
|
|
11
|
+
(where (spec, build, (human review, (log, commit,
|
|
12
|
+
are we?) check, audit) fixes if needed) merge with approval)
|
|
16
13
|
|
|
17
|
-
Autopilot is an explicit opt-in
|
|
14
|
+
Autopilot is an explicit opt-in path. It uses the same Blueprint files and the
|
|
15
|
+
same quality gates, but it does not stop after every normal review point. A
|
|
16
|
+
single user request is permission to run one bounded loop until the feature is
|
|
17
|
+
ready for review, blocked, or unsafe to continue.
|
|
18
18
|
|
|
19
|
-
It does **not** replace the normal
|
|
19
|
+
It does **not** replace the normal workflow. `/feature`, `/implement`, `/check`,
|
|
20
|
+
and `/complete` remain the conservative default.
|
|
20
21
|
|
|
21
|
-
Do not suggest Autopilot as the default next action. Use it only when the user
|
|
22
|
+
Do not suggest Autopilot as the default next action. Use it only when the user
|
|
23
|
+
explicitly asks for it.
|
|
22
24
|
|
|
23
|
-
The explicit Autopilot request is permission to create checkpoint commits on the
|
|
25
|
+
The explicit Autopilot request is permission to create checkpoint commits on the
|
|
26
|
+
feature or fix branch after passing implementation steps. It is not permission to
|
|
27
|
+
merge, push, deploy, publish, send, delete data, or run destructive actions.
|
|
24
28
|
|
|
25
29
|
## Input
|
|
26
30
|
|
|
27
31
|
Common forms:
|
|
28
32
|
|
|
29
|
-
-
|
|
30
|
-
-
|
|
31
|
-
-
|
|
32
|
-
|
|
33
|
+
- No argument: resume the current feature if one exists, otherwise target the next
|
|
34
|
+
unchecked build-plan item.
|
|
35
|
+
- A number or name: target that build-plan feature, for example `/autopilot 3` or
|
|
36
|
+
`$autopilot "directory listing"`.
|
|
37
|
+
- `fix "<issue>"`: write and build an ad-hoc fix spec.
|
|
38
|
+
- `resume`: continue the current feature on its existing branch.
|
|
33
39
|
|
|
34
|
-
If the requested target conflicts with a
|
|
40
|
+
If the requested target conflicts with a feature already in progress, stop and
|
|
41
|
+
ask which one should win. Do not overwrite `devflow/context/current-feature.md`
|
|
42
|
+
silently.
|
|
35
43
|
|
|
36
|
-
|
|
37
|
-
|
|
44
|
+
Rollback is intentionally excluded from Autopilot. If the request is a rollback
|
|
45
|
+
or `current-feature.md` is marked `Type: Rollback`, stop and direct the user to
|
|
46
|
+
the reviewed `/implement` path. Reversing completed work requires the explicit
|
|
47
|
+
dependency and conflict gates in `/rollback` and `/implement`.
|
|
38
48
|
|
|
39
|
-
|
|
40
|
-
|
|
41
|
-
## Step 1 - Preflight & Safety Check
|
|
49
|
+
## Step 1 - preflight like /status
|
|
42
50
|
|
|
43
|
-
Read the
|
|
51
|
+
Read the same state `/status` reads:
|
|
44
52
|
|
|
45
|
-
- `AGENTS.md`
|
|
53
|
+
- `AGENTS.md`
|
|
54
|
+
- `devflow/project-plan.md`
|
|
55
|
+
- `devflow/build-plan.md`
|
|
46
56
|
- `devflow/context/project-overview.md`
|
|
47
|
-
- `devflow/context/current-
|
|
57
|
+
- `devflow/context/current-feature.md`
|
|
58
|
+
- `devflow/context/findings.md`
|
|
48
59
|
- `devflow/context/coding-standards.md`
|
|
49
60
|
- `devflow/context/ai-interaction.md`
|
|
50
|
-
- `devflow/context/findings.md`
|
|
51
61
|
- git branch, status, and recent log
|
|
52
62
|
|
|
53
|
-
|
|
54
|
-
|
|
55
|
-
1. The repo is not a git repository.
|
|
56
|
-
2. The working tree is dirty with uncommitted changes unrelated to this run.
|
|
57
|
-
3. `current-stage.md` has an active run and the user requested a different target without resolving conflict.
|
|
58
|
-
4. The task requires architectural, financial, auth, billing, or destructive decisions not documented in the context.
|
|
59
|
-
|
|
60
|
-
---
|
|
61
|
-
|
|
62
|
-
## Step 2 - Choose or Write Specification (`20-spec`)
|
|
63
|
-
|
|
64
|
-
1. If `devflow/runs/{running-id}-{slug}20-spec.md` already exists, resume it.
|
|
65
|
-
2. If no spec exists:
|
|
66
|
-
- Ensure `10-define.md` exists with locked scope and allocated Running ID.
|
|
67
|
-
- Write `20-spec.md` following the DevFlow specification schema.
|
|
68
|
-
- Critique and red-team the spec (edge cases, unhappy paths, testable criteria).
|
|
69
|
-
- Apply fixes to the spec.
|
|
70
|
-
|
|
71
|
-
---
|
|
72
|
-
|
|
73
|
-
## Step 3 - Branch Setup & Implementation Plan (`30-plan`)
|
|
74
|
-
|
|
75
|
-
1. **Branch Management**:
|
|
76
|
-
- Feature: `feature/{slug}-{running-id}`
|
|
77
|
-
- Fix: `fix/{slug}-{running-id}`
|
|
78
|
-
- Switch to or create the working branch. Never run Autopilot directly on `main` or `master`.
|
|
79
|
-
2. **Planning**:
|
|
80
|
-
- Write `devflow/runs/{running-id}-{slug}30-plan.md`.
|
|
81
|
-
- Seed `checklists/implementation-checklist.md` and `checklists/verification-checklist.md`.
|
|
82
|
-
|
|
83
|
-
---
|
|
84
|
-
|
|
85
|
-
## Step 4 - Implement in Small Increments (`40-execute`)
|
|
86
|
-
|
|
87
|
-
Work through the implementation checklist in order. Each step must remain reviewable.
|
|
88
|
-
|
|
89
|
-
For every subtask:
|
|
90
|
-
|
|
91
|
-
1. Implement only that scoped unit.
|
|
92
|
-
2. Run relevant verification (unit tests, typecheck, lint, build).
|
|
93
|
-
3. If UI is involved, inspect behavior, verify console errors and network calls.
|
|
94
|
-
4. Self-review diff against `coding-standards.md`.
|
|
95
|
-
5. Fix issues and rerun failed checks.
|
|
96
|
-
6. Mark the task checked `[x]` in `checklists/implementation-checklist.md`.
|
|
97
|
-
7. **Create a Checkpoint Commit** on the feature/fix branch for the passing step:
|
|
98
|
-
```bash
|
|
99
|
-
git add <modified-files> devflow/runs/{running-id}-{slug}/checklists/implementation-checklist.md
|
|
100
|
-
git commit -m "feat({running-id}): checkpoint <concise step description>"
|
|
101
|
-
```
|
|
102
|
-
8. Write `devflow/runs/{running-id}-{slug}40-execute.md`.
|
|
103
|
-
|
|
104
|
-
---
|
|
105
|
-
|
|
106
|
-
## Step 5 - Senior QA Verification (`50-verify`)
|
|
63
|
+
Then decide whether it is safe to run.
|
|
107
64
|
|
|
108
|
-
|
|
109
|
-
- Framework integrity (`npm run check` or equivalent)
|
|
110
|
-
- Static contracts (`npm run check:static` when applicable)
|
|
111
|
-
- Project test suite (Unit tests, integration tests)
|
|
112
|
-
- Build / Package smoke tests
|
|
113
|
-
2. Update `checklists/verification-checklist.md` with concrete evidence.
|
|
114
|
-
3. Write `devflow/runs/{running-id}-{slug}50-verify.md` with QA verdict (`PASS` / `FAIL`).
|
|
115
|
-
|
|
116
|
-
---
|
|
117
|
-
|
|
118
|
-
## Step 6 - Targeted Quality Audit & Repair
|
|
119
|
-
|
|
120
|
-
Review diffs and inspect `devflow/context/findings.md`:
|
|
121
|
-
|
|
122
|
-
1. **Repair confirmed P0 and P1 findings** within the current run scope.
|
|
123
|
-
2. Update finding status in `devflow/context/findings.md`.
|
|
124
|
-
3. Rerun verification tests after repairs.
|
|
125
|
-
4. Create a checkpoint commit for the fix.
|
|
126
|
-
5. If a P0/P1 finding cannot be repaired within scope or fails twice consecutively, stop immediately and report.
|
|
127
|
-
|
|
128
|
-
---
|
|
129
|
-
|
|
130
|
-
## Step 7 - Delivery Digest & Review Packet (`60-report`)
|
|
131
|
-
|
|
132
|
-
1. Write `devflow/runs/{running-id}-{slug}60-report.md`.
|
|
133
|
-
2. Render standalone HTML dashboard `devflow/runs/{running-id}-{slug}60-report.html` (via `md2html` or report generator).
|
|
134
|
-
3. Update `devflow/context/current-stage.md` to indicate ready for `70-release`.
|
|
135
|
-
4. Stop with a concise **Review Packet Dashboard** for human approval.
|
|
136
|
-
|
|
137
|
-
---
|
|
138
|
-
|
|
139
|
-
## 🛑 Strict Hard Stops (Never Exceed)
|
|
140
|
-
|
|
141
|
-
Stop immediately and report to the user instead of continuing when Autopilot would need to:
|
|
142
|
-
|
|
143
|
-
- Commit directly to `main` or `master`, merge branches, delete branches, or force push.
|
|
144
|
-
- Run `git push` to remote repositories.
|
|
145
|
-
- Deploy to staging/production or publish packages to npm/registries.
|
|
146
|
-
- Delete data, drop databases, or run irreversible destructive migrations.
|
|
147
|
-
- Make product, business, or architecture decisions not specified in the context.
|
|
148
|
-
- Continue after two failed attempts to fix the same issue (Two-attempt hard stop).
|
|
149
|
-
|
|
150
|
-
---
|
|
151
|
-
|
|
152
|
-
## Output Review Packet Format
|
|
153
|
-
|
|
154
|
-
When Autopilot finishes successfully, output a scannable review packet:
|
|
155
|
-
|
|
156
|
-
```markdown
|
|
157
|
-
### 🛸 Autopilot Execution Summary: [{running-id}]
|
|
158
|
-
|
|
159
|
-
- **Branch**: `{branch-name}`
|
|
160
|
-
- **Target Run**: `{running-id} - {title}`
|
|
161
|
-
- **Artifacts Generated**:
|
|
162
|
-
- Spec: `devflow/runs/{id}20-spec.md`
|
|
163
|
-
- Plan: `devflow/runs/{id}30-plan.md`
|
|
164
|
-
- Implement Evidence: `devflow/runs/{id}40-execute.md`
|
|
165
|
-
- QA Verify Report: `devflow/runs/{id}50-verify.md`
|
|
166
|
-
- Digest Report: `devflow/runs/{id}60-report.md`
|
|
167
|
-
- HTML Dashboard: `devflow/runs/{id}60-report.html`
|
|
168
|
-
- **Validation & Tests**: `All Passed (Green)`
|
|
169
|
-
- **Checkpoint Commits**: `{count} commits created on {branch-name}`
|
|
170
|
-
- **Manual QA Walkthrough**: Run `try {running-id}` for human review guide
|
|
65
|
+
Stop before changing files when:
|
|
171
66
|
|
|
172
|
-
|
|
173
|
-
|
|
174
|
-
|
|
175
|
-
|
|
67
|
+
- The repo is not a git repo.
|
|
68
|
+
- The working tree is dirty and there is no current feature tying those changes
|
|
69
|
+
to this run.
|
|
70
|
+
- `current-feature.md` has real work and the user requested a different target.
|
|
71
|
+
- `project-overview.md` is missing or stale and the planning docs are not clear
|
|
72
|
+
enough to regenerate it.
|
|
73
|
+
- The next feature is visual or replication-heavy and no design reference exists.
|
|
74
|
+
- The task needs product, data, auth, billing, or destructive decisions the docs
|
|
75
|
+
do not answer.
|
|
76
|
+
|
|
77
|
+
If the only issue is that `project-overview.md` is stale and the plans are clear,
|
|
78
|
+
regenerate it using the `/overview` behavior and continue. Include that in the
|
|
79
|
+
final packet.
|
|
80
|
+
|
|
81
|
+
## Step 2 - choose or write the spec
|
|
82
|
+
|
|
83
|
+
If `devflow/context/current-feature.md` already contains an active spec,
|
|
84
|
+
resume it. Read checked steps and continue from the first unchecked step.
|
|
85
|
+
|
|
86
|
+
If there is no active spec:
|
|
87
|
+
|
|
88
|
+
1. Use the `/feature` behavior for a planned feature, or `/fix` behavior for a
|
|
89
|
+
requested fix.
|
|
90
|
+
2. Write `devflow/context/current-feature.md`.
|
|
91
|
+
3. Red-team the spec before building:
|
|
92
|
+
- missing unhappy paths
|
|
93
|
+
- oversized steps
|
|
94
|
+
- undefined contracts
|
|
95
|
+
- missing design reference
|
|
96
|
+
- scope creep
|
|
97
|
+
- vague done-whens
|
|
98
|
+
- missing testing plan when `AGENTS.md` declares a test command
|
|
99
|
+
4. Apply the spec fixes.
|
|
100
|
+
|
|
101
|
+
Autopilot may continue past this spec gate because the user explicitly invoked
|
|
102
|
+
Autopilot. Still report what the critique changed in the final packet.
|
|
103
|
+
|
|
104
|
+
## Step 3 - create or reuse the branch
|
|
105
|
+
|
|
106
|
+
Use the same branch rules as `/implement`:
|
|
107
|
+
|
|
108
|
+
- Feature: `feature/<name>`
|
|
109
|
+
- Fix: `fix/<name>`
|
|
110
|
+
|
|
111
|
+
If the branch already exists, switch to it only if it matches the active spec.
|
|
112
|
+
If switching branches would strand unrelated dirty work, stop and report the
|
|
113
|
+
problem.
|
|
114
|
+
|
|
115
|
+
## Step 4 - implement in small steps
|
|
116
|
+
|
|
117
|
+
Work through the spec's build steps in order. Each step must remain reviewable.
|
|
118
|
+
Unlike `/implement`, do not pause for user approval after each passing step. The
|
|
119
|
+
review happens at the final packet unless a hard stop is hit.
|
|
120
|
+
|
|
121
|
+
For every step:
|
|
122
|
+
|
|
123
|
+
1. Implement only that step.
|
|
124
|
+
2. Run the relevant verification:
|
|
125
|
+
- the exact `Verify` command from `AGENTS.md`, when declared
|
|
126
|
+
- otherwise the build, relevant test, lint, and typecheck commands already
|
|
127
|
+
documented by the project
|
|
128
|
+
- browser, CLI, API, or app-level evidence for behavioral done-whens
|
|
129
|
+
3. If UI is involved, inspect the running app when possible. Prefer Playwright if
|
|
130
|
+
it is already installed or declared. Capture screenshots when they add useful
|
|
131
|
+
evidence. Check for console errors and failed requests.
|
|
132
|
+
4. Self-review the diff for the step:
|
|
133
|
+
- does it match the spec?
|
|
134
|
+
- did it add scope?
|
|
135
|
+
- is the error path handled?
|
|
136
|
+
- did it follow `coding-standards.md`?
|
|
137
|
+
- are tests present for new in-scope logic when the test gate is on?
|
|
138
|
+
5. Fix obvious issues and rerun the failed checks.
|
|
139
|
+
6. Mark the step checked in `current-feature.md` only after the step passes.
|
|
140
|
+
7. Create a checkpoint commit on the feature or fix branch for the passing step.
|
|
141
|
+
Include the code, tests, and the updated `current-feature.md` checkbox. Use a
|
|
142
|
+
conventional message such as `feat: checkpoint mock snapshot route` or
|
|
143
|
+
`fix: checkpoint stale service filter`. Keep the message about the step, not
|
|
144
|
+
about Autopilot.
|
|
145
|
+
|
|
146
|
+
Do not batch the whole feature into one large diff. If a step gets too large,
|
|
147
|
+
split the step in `current-feature.md` and continue with the first smaller step.
|
|
148
|
+
|
|
149
|
+
## Step 5 - acceptance check
|
|
150
|
+
|
|
151
|
+
After all implementation steps are checked, run the `/check` behavior for the
|
|
152
|
+
feature when any done-when is behavioral, visual, or integration-facing.
|
|
153
|
+
|
|
154
|
+
For pure library or CLI work, build plus tests and representative command output
|
|
155
|
+
may be enough. Be explicit about the evidence used.
|
|
156
|
+
|
|
157
|
+
## Step 6 - targeted quality audit and repair
|
|
158
|
+
|
|
159
|
+
After the acceptance check, apply the `/audit current` behavior to the active
|
|
160
|
+
feature, its diff, and the nearby code affected by the change. This is a targeted
|
|
161
|
+
feature audit, not a repository-wide cleanup pass. Findings are recorded in
|
|
162
|
+
`devflow/context/findings.md` with durable IDs and statuses, as `/audit`
|
|
163
|
+
defines; the ledger reports status and never scopes what the audit examines.
|
|
164
|
+
|
|
165
|
+
For every finding:
|
|
166
|
+
|
|
167
|
+
1. Validate it against the actual code, spec, tests, `coding-standards.md`, and
|
|
168
|
+
local project patterns. An audit finding is evidence to investigate, not an
|
|
169
|
+
automatic instruction to edit.
|
|
170
|
+
2. Repair confirmed P0 and P1 findings when the fix stays inside the approved
|
|
171
|
+
feature scope and does not require a product or architecture decision. Set
|
|
172
|
+
the repaired finding to `fixed` in the ledger, never `closed`.
|
|
173
|
+
3. Report P2 and P3 findings in the final packet. Fix them only when the change
|
|
174
|
+
is small, directly caused by the current feature, and clearly required by the
|
|
175
|
+
project standards.
|
|
176
|
+
4. If a confirmed P0 or P1 finding cannot be repaired safely within scope, stop
|
|
177
|
+
and report it. Do not present the feature as ready for `/complete`.
|
|
178
|
+
|
|
179
|
+
After any audit repair:
|
|
180
|
+
|
|
181
|
+
1. Rerun the documented `Verify` command when present; otherwise rerun the
|
|
182
|
+
affected build, lint, typecheck, and test commands.
|
|
183
|
+
2. Rerun the acceptance evidence affected by the repair.
|
|
184
|
+
3. Recheck the repaired area using the same targeted audit criteria. When that
|
|
185
|
+
recheck confirms the original defect is gone and the repair introduced no
|
|
186
|
+
new one, move the `fixed` finding to `closed` under the `/audit` close
|
|
187
|
+
conditions and name it in the packet. An unrelated new finding gets its own
|
|
188
|
+
ledger entry and does not keep the repaired one open.
|
|
189
|
+
4. Create a checkpoint commit only after the repair and its checks pass.
|
|
190
|
+
|
|
191
|
+
Use the existing two-attempt hard stop for repeated repair failures. Do not widen
|
|
192
|
+
the feature into a general refactor, silently suppress a finding, or turn this
|
|
193
|
+
step into a full-project hardening pass. A broader cleanup remains a separate
|
|
194
|
+
`/audit` followed by planned `/fix` work.
|
|
195
|
+
|
|
196
|
+
## Step 7 - final review packet
|
|
197
|
+
|
|
198
|
+
Stop with a concise review packet. Keep it useful enough for `/complete` but not
|
|
199
|
+
a full audit report:
|
|
200
|
+
|
|
201
|
+
- branch name
|
|
202
|
+
- target feature or fix
|
|
203
|
+
- whether the spec was created or resumed
|
|
204
|
+
- what the spec critique changed
|
|
205
|
+
- changed files and why each changed
|
|
206
|
+
- build/test/check commands run, with pass or fail
|
|
207
|
+
- screenshots or output paths, when relevant
|
|
208
|
+
- how to try it manually, or a pointer to `/try` for the full walkthrough
|
|
209
|
+
- checkpoint commits created
|
|
210
|
+
- self-review findings
|
|
211
|
+
- targeted audit scope and findings
|
|
212
|
+
- audit repairs made and checks rerun
|
|
213
|
+
- P0/P1 findings still `open` or `fixed` in `devflow/context/findings.md`,
|
|
214
|
+
which block `/complete`
|
|
215
|
+
- unresolved risks or skipped checks
|
|
216
|
+
- exact next action
|
|
217
|
+
|
|
218
|
+
If everything is green, the next action is usually: review the diff, then run
|
|
219
|
+
`/try` if you want a manual walkthrough, then `/complete`.
|
|
220
|
+
|
|
221
|
+
If something failed, name the failing check and the next fix target.
|
|
222
|
+
|
|
223
|
+
## Hard Stops
|
|
224
|
+
|
|
225
|
+
Stop immediately and report instead of continuing when Autopilot would need to:
|
|
226
|
+
|
|
227
|
+
- commit on `main`, merge, delete a branch, push, deploy, publish, or send
|
|
228
|
+
anything
|
|
229
|
+
- delete data, reset a database, run irreversible migrations, kill processes, or
|
|
230
|
+
change system settings
|
|
231
|
+
- install dependencies or use network access without the current tool's approval
|
|
232
|
+
flow
|
|
233
|
+
- make a product decision not covered by the docs
|
|
234
|
+
- continue after two failed fix attempts on the same issue
|
|
235
|
+
- hide, skip, or hand-wave a failing check
|
|
236
|
+
|
|
237
|
+
## Rules
|
|
238
|
+
|
|
239
|
+
- One Autopilot run handles one feature or one fix.
|
|
240
|
+
- Autopilot creates checkpoint commits on the feature or fix branch after passing
|
|
241
|
+
steps.
|
|
242
|
+
- Autopilot audits the active feature and affected code, not the entire project.
|
|
243
|
+
- A P0 or P1 finding left `open` or `fixed` in `devflow/context/findings.md`
|
|
244
|
+
blocks readiness for `/complete`. The ledger is what makes this enforceable.
|
|
245
|
+
- Autopilot stops before `/complete`. It never merges.
|
|
246
|
+
- The Blueprint files remain the state machine. Keep
|
|
247
|
+
`current-feature.md` accurate as steps complete.
|
|
248
|
+
- Follow `coding-standards.md`, `ai-interaction.md`, and `AGENTS.md`.
|
|
249
|
+
- Prefer fewer, higher-quality changes over broad coverage.
|
|
250
|
+
- Report uncertainty plainly. A blocked run is useful if it tells the truth.
|
|
251
|
+
|
|
252
|
+
## Formatting
|
|
253
|
+
|
|
254
|
+
Format the output to match the project's conventions in
|
|
255
|
+
`devflow/context/ai-interaction.md`: concise, scannable markdown, with lists for
|
|
256
|
+
enumerations and tables for matrices rather than dense paragraphs.
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: brainstorm
|
|
3
|
-
description: "[
|
|
3
|
+
description: "[devflow][B] Companion skill for structured divergent and convergent ideation, generating 2-3 viable options with trade-off analysis before committing to delivery."
|
|
4
4
|
argument-hint: "{topic, feature, or discovery-id}"
|
|
5
5
|
---
|
|
6
6
|
|
|
@@ -58,5 +58,5 @@ Construct a structured evaluation table:
|
|
|
58
58
|
|
|
59
59
|
## Next Workflow Recommendation
|
|
60
60
|
|
|
61
|
-
- **Primary**: `00-
|
|
62
|
-
- **Inbox**: `idea` to record into `devflow/ideas.md`
|
|
61
|
+
- **Primary**: `00-explore {discovery_id}` (if resuming discovery) or `10-define` / `feature` (if ready to specify)
|
|
62
|
+
- **Inbox**: `idea` to record into `devflow/ideas.md`
|
|
@@ -1,77 +1,100 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: brief
|
|
3
|
-
description: "[
|
|
3
|
+
description: "[devflow][B] Read-only briefing on an upcoming build-plan feature before you spec it. With no argument, briefs the next unchecked item; given a number or name, briefs that one. Reads build-plan.md and project-overview.md and explains what the feature is, what it depends on, what it will touch, how big it is, and whether it will split - without writing anything. Use when the user runs /brief, asks what the next feature involves, wants to understand or preview a feature before /feature, or is deciding what to build or reorder next."
|
|
4
4
|
---
|
|
5
5
|
|
|
6
|
-
# brief -
|
|
6
|
+
# brief - understand a feature before you spec it
|
|
7
7
|
|
|
8
8
|
Where this sits in the workflow:
|
|
9
9
|
|
|
10
|
-
|
|
11
|
-
|
|
12
|
-
|
|
13
|
-
explainer) contract)
|
|
14
|
-
```
|
|
10
|
+
build-plan + overview -> [brief] -> /feature -> /implement
|
|
11
|
+
(what exists) (read-only (spec it) (build it)
|
|
12
|
+
explainer)
|
|
15
13
|
|
|
16
|
-
This skill answers one
|
|
14
|
+
This skill answers one question: *what does this feature actually involve, before
|
|
15
|
+
I commit to spec'ing it?* It reads the two files that describe the work and prints
|
|
16
|
+
a short briefing so you can decide whether to spec it now, reorder it, split it,
|
|
17
|
+
or clear a blocker first. It is the read-only precursor to `/feature`.
|
|
17
18
|
|
|
18
|
-
It
|
|
19
|
+
It never writes anything: no spec, no edits to `build-plan.md`, no branch, no
|
|
20
|
+
commit. `/feature` is the mutating step that turns a briefing into a spec; this
|
|
21
|
+
one just explains.
|
|
19
22
|
|
|
20
|
-
|
|
23
|
+
How it differs from its neighbors:
|
|
21
24
|
|
|
22
|
-
|
|
23
|
-
|
|
24
|
-
-
|
|
25
|
-
|
|
26
|
-
- **topic / concern**: e.g. `brief "database migration"`.
|
|
25
|
+
- `/status` reports the *whole project*: progress, current work, git, next action.
|
|
26
|
+
`/brief` zooms into *one feature* and explains it in depth.
|
|
27
|
+
- `/feature` *writes* the spec (and may split the item in `build-plan.md`).
|
|
28
|
+
`/brief` previews what `/feature` would tackle, changing nothing.
|
|
27
29
|
|
|
28
|
-
|
|
30
|
+
## Input
|
|
29
31
|
|
|
30
|
-
|
|
32
|
+
A feature from `build-plan.md`, by number or name - e.g. `/brief 4` or
|
|
33
|
+
`/brief "validator"`.
|
|
31
34
|
|
|
32
|
-
|
|
35
|
+
**With no argument, brief the next one** - the first unchecked leaf in
|
|
36
|
+
`build-plan.md`, the same target `/feature` would pick.
|
|
33
37
|
|
|
34
|
-
|
|
35
|
-
|
|
36
|
-
|
|
37
|
-
4. **Dependencies**: What previously completed runs (`devflow/runs/`) or modules must be in place first.
|
|
38
|
+
If `build-plan.md` is still a placeholder stub or the overview is missing, say so
|
|
39
|
+
plainly and point at `/overview` (or filling the plans) rather than inventing a
|
|
40
|
+
briefing.
|
|
38
41
|
|
|
39
|
-
##
|
|
42
|
+
## What it reads
|
|
40
43
|
|
|
41
|
-
|
|
44
|
+
Gather these, then synthesize. Don't dump file contents; explain.
|
|
42
45
|
|
|
43
|
-
|
|
44
|
-
|
|
45
|
-
|
|
46
|
-
|
|
47
|
-
|
|
48
|
-
|
|
46
|
+
1. **The target** - the feature's line in `devflow/build-plan.md`, plus whether
|
|
47
|
+
a parent was already split into sub-items (`4a`, `4b`, ...).
|
|
48
|
+
2. **Full context** - `devflow/context/project-overview.md`: the data model,
|
|
49
|
+
routes/endpoints, stack, UI/UX, conventions, and open questions that touch this
|
|
50
|
+
feature.
|
|
51
|
+
3. **What already exists** - earlier checked build-plan items and, if useful, git
|
|
52
|
+
history, to ground the dependency read (what must be in place first, what this
|
|
53
|
+
unblocks later).
|
|
54
|
+
4. **Design reference** - if `prototypes/` exists and the feature is UI-facing,
|
|
55
|
+
note which mockups apply (that `/feature` will link them and port `theme.css`).
|
|
49
56
|
|
|
50
|
-
##
|
|
57
|
+
## Output
|
|
51
58
|
|
|
52
|
-
|
|
59
|
+
A short, scannable briefing, not a wall of text. Aim for something like:
|
|
53
60
|
|
|
54
|
-
|
|
55
|
-
|
|
61
|
+
Feature 5 - Submission draft flow
|
|
62
|
+
What: maintainers submit a skill by GitHub URL (primary) or zip (fallback);
|
|
63
|
+
drafts are saved before validation runs.
|
|
64
|
+
Depends on: apps/api must be stood up first (this is the first feature that
|
|
65
|
+
needs the backend); builds on the Skill / SkillVersion shapes from feature 3.
|
|
66
|
+
Unblocks: queue-backed validation (6) and the publishable flow (7).
|
|
67
|
+
Touches: new User, Submission, SkillVersion tables (Drizzle); GitHub OAuth;
|
|
68
|
+
POST /submissions; R2 snapshot upload; a React upload island (client).
|
|
69
|
+
Size: large - likely splits into 5a (OAuth + maintainer profile), 5b
|
|
70
|
+
(GitHub-URL submission + draft record), 5c (zip upload + R2).
|
|
71
|
+
Reference: no prototype for this flow; upload.html covers the later validation
|
|
72
|
+
panel, not this form.
|
|
73
|
+
Open questions: apps/api (Hono) is not scaffolded yet - resolve before spec'ing.
|
|
56
74
|
|
|
57
|
-
|
|
58
|
-
- **Depends On**: Prerequisite modules, runs, or infrastructure.
|
|
59
|
-
- **Unblocks**: Downstream capabilities enabled by this feature.
|
|
60
|
-
- **Touches**: Anticipated files, schemas, endpoints, and UI views.
|
|
61
|
-
- **Estimated Scope & Size**: `[S / M / L / XL]` (and split recommendation if XL).
|
|
62
|
-
- **Key Risks & Open Questions**: Technical or product unknowns that must be settled in `20-spec`.
|
|
75
|
+
Next: run /feature 5 to spec 5a, or clear the apps/api blocker first.
|
|
63
76
|
|
|
64
|
-
|
|
65
|
-
|
|
66
|
-
|
|
77
|
+
Adapt the lines to the feature; drop any that don't apply. Always end with a
|
|
78
|
+
single **Next** action - usually `/feature N` to spec it, but `/overview` if the
|
|
79
|
+
plans aren't ready, `/prototype` if it's UI-facing and the look isn't locked, or
|
|
80
|
+
"clear X first" when a dependency blocks it.
|
|
67
81
|
|
|
68
82
|
## Rules
|
|
69
83
|
|
|
70
|
-
- **
|
|
71
|
-
-
|
|
72
|
-
|
|
73
|
-
- **
|
|
74
|
-
|
|
75
|
-
|
|
76
|
-
|
|
77
|
-
|
|
84
|
+
- **Read-only, always.** Never write a file, never edit `build-plan.md` or
|
|
85
|
+
`current-feature.md`, never branch, commit, install, or build. To act on the
|
|
86
|
+
briefing, the user runs `/feature` next.
|
|
87
|
+
- **Explain, don't spec.** Size, dependencies, and a likely sub-split are the
|
|
88
|
+
value here; the actual build steps are `/feature`'s job. Don't write step lists.
|
|
89
|
+
- **Trace to the plans.** Everything in the briefing comes from `build-plan.md`
|
|
90
|
+
and `project-overview.md`. Don't invent scope; if something is underspecified,
|
|
91
|
+
say so and flag it as a question for `/feature` or `/overview`.
|
|
92
|
+
- **Be honest about gaps.** If the plans are a stub, the overview is stale, or a
|
|
93
|
+
dependency isn't built yet, say that plainly - catching a blocker before
|
|
94
|
+
spec'ing is half the value.
|
|
95
|
+
|
|
96
|
+
## Formatting
|
|
97
|
+
|
|
98
|
+
Format the output to match the project's conventions in
|
|
99
|
+
`devflow/context/ai-interaction.md`: concise, scannable markdown, with lists for
|
|
100
|
+
enumerations and tables for matrices rather than dense paragraphs.
|