@jakkrichm/create-nexus-devflow 2.17.3 → 2.17.4
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/package.json +1 -1
- package/template/.agents/skills/bughunter/SKILL.md +1 -1
- package/template/.agents/skills/devflow/SKILL.md +3 -2
- package/template/.agents/skills/fix/SKILL.md +3 -3
- package/template/.agents/skills/implement/SKILL.md +36 -118
- package/template/.claude/skills/bughunter/SKILL.md +1 -1
- package/template/.claude/skills/devflow/SKILL.md +3 -2
- package/template/.claude/skills/fix/SKILL.md +3 -3
- package/template/.claude/skills/implement/SKILL.md +36 -118
- package/template/AGENTS.md +16 -38
- package/template/devflow/context/ai-interaction.md +15 -14
- package/template/devflow/context/coding-standards.md +5 -4
- package/template/devflow/context/glossary.md +6 -6
- package/template/devflow/reference/help-playbook-protocol.md +33 -0
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: bughunter
|
|
3
|
-
description: "[devflow]
|
|
3
|
+
description: "[devflow] Investigate security vulnerabilities with evidence. Use for /bughunter or an authorized security review of code or endpoints."
|
|
4
4
|
argument-hint: "[{target, vuln-class, or topic}]"
|
|
5
5
|
---
|
|
6
6
|
|
|
@@ -36,7 +36,8 @@ When invoked without an argument (or when determining the next step), inspect:
|
|
|
36
36
|
|
|
37
37
|
### Default State Recommendations
|
|
38
38
|
- If no run is active and user wants to start a feature -> Recommend `/feature <name>`.
|
|
39
|
-
- If
|
|
39
|
+
- If the user requests a bugfix plan -> Recommend `/fix <bug>`.
|
|
40
|
+
- If the user requests a completed fix -> preserve the implementation and verification scope; draft the required spec internally and continue to `/implement` using existing authorization.
|
|
40
41
|
- If no run is active and user has pending ideas in `devflow/ideas.md` -> Highlight `/feature IDEA-xxx` or `/discovery IDEA-xxx`.
|
|
41
42
|
- If no run is active and user wants deep architectural exploration -> Recommend `/discovery`.
|
|
42
43
|
- If user asks to check system health -> Recommend `doctor`.
|
|
@@ -48,7 +49,7 @@ When invoked without an argument (or when determining the next step), inspect:
|
|
|
48
49
|
| User Intent / Request Type | Recommended Skill | Normal Name / Alias | Lifecycle Path |
|
|
49
50
|
| :--- | :--- | :--- | :--- |
|
|
50
51
|
| **"Spec new feature / living spec"** | `feature` | `/feature` / `/spec` | **Unified**: `/feature` -> `/implement` -> `/check` -> `/complete` |
|
|
51
|
-
| **"
|
|
52
|
+
| **"Draft a bugfix / small-change plan"** | `fix` | `/fix` | **Unified**: `/fix` -> `/implement` -> `/check` -> `/complete` |
|
|
52
53
|
| **"Capture quick idea / thought"** | `idea` | `/idea` | **Companion**: Enriches & saves to `devflow/ideas.md` |
|
|
53
54
|
| **"Pre-delivery exploration / research"**| `discovery`| `/discovery` | **Pre-Flight**: `discovery` -> `/feature` |
|
|
54
55
|
| **"Socratic alignment / ADR / glossary"**| `grill` | `/grill` / `/align` | **Pre-Flight**: `grill` -> `/feature` |
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: fix
|
|
3
|
-
description: "[devflow]
|
|
3
|
+
description: "[devflow] Draft a task-isolated spec for a bug or small change. Use when invoking /fix or requesting a fix plan."
|
|
4
4
|
argument-hint: "[{title or issue-description}]"
|
|
5
5
|
---
|
|
6
6
|
|
|
@@ -23,7 +23,7 @@ it creates a dedicated run folder at `devflow/context/{xxx-slug}/` (e.g. `059-fi
|
|
|
23
23
|
## Input
|
|
24
24
|
|
|
25
25
|
A description of the bug or change, for example `/fix "password reset email never
|
|
26
|
-
sends"`.
|
|
26
|
+
sends"`. Use the reported problem as input when the user asks for a fix plan. A request to diagnose a problem belongs to `/debug`; a request to fix and verify it retains its full delivery scope.
|
|
27
27
|
|
|
28
28
|
The input may also be a finding ID from `devflow/context/{xxx-slug}/findings.md` (or previous findings), alone
|
|
29
29
|
or with a description, for example `/fix F-03`.
|
|
@@ -46,7 +46,7 @@ then write a short spec to `devflow/context/{xxx-slug}/spec.md`. Also initialize
|
|
|
46
46
|
big to read. Each ends with an observable "done when".
|
|
47
47
|
- **Verify** - how to confirm it's fixed (what to click or test).
|
|
48
48
|
|
|
49
|
-
|
|
49
|
+
For an explicit `/fix` or a planning-only request, present the spec as the completed deliverable and identify `/implement` as the next command. For an existing request to implement and verify the fix, preserve that authorization: use this spec as the internal handoff and continue through `/implement` without an extra approval pause. Ask only for unresolved material scope, design, or permission decisions.
|
|
50
50
|
|
|
51
51
|
## Rules
|
|
52
52
|
|
|
@@ -1,10 +1,10 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: implement
|
|
3
|
-
description: "[devflow]
|
|
3
|
+
description: "[devflow] Build and verify a task-isolated spec. Use for /implement or requests to implement an approved plan."
|
|
4
4
|
argument-hint: "[{run-id, number, or name}] [--ticket <NN | path>]"
|
|
5
5
|
---
|
|
6
6
|
|
|
7
|
-
# implement - build the target spec
|
|
7
|
+
# implement - build and verify the target spec
|
|
8
8
|
|
|
9
9
|
**Context reuse:** Reuse any required file already loaded in project instructions or the current session. Read it again only if absent, changed, or exact current bytes or line references are needed.
|
|
10
10
|
|
|
@@ -20,17 +20,23 @@ Where this sits in the workflow:
|
|
|
20
20
|
|
|
21
21
|
`/feature`, `/fix`, or `/rollback` wrote the spec to
|
|
22
22
|
`devflow/context/{xxx-slug}/spec.md` and stopped.
|
|
23
|
-
|
|
24
|
-
|
|
25
|
-
|
|
26
|
-
|
|
27
|
-
|
|
28
|
-
|
|
23
|
+
Build the selected spec through its acceptance criteria and required verification, then present the configured review handoff. Preserve authorization already given for this work. Read only relevant conventions and reuse loaded context.
|
|
24
|
+
|
|
25
|
+
## Review cadence and checkpoints
|
|
26
|
+
|
|
27
|
+
Read `workflow.stepReview` and `workflow.checkpointCommits` from `devflow/config.json` independently; defaults are `feature` and `disabled`.
|
|
28
|
+
|
|
29
|
+
- `stepReview: "feature"`: continue through all authorized steps without asking Continue after each step. Record evidence as work proceeds and present one final review packet.
|
|
30
|
+
- `stepReview: "every"`: after a working step, show its diff and evidence, then wait for approval before the next step. A walkthrough is available on request.
|
|
31
|
+
- `checkpointCommits: "disabled"`: omit ordinary checkpoint offers.
|
|
32
|
+
- `checkpointCommits: "enabled"`: offer an optional checkpoint at the configured review boundary (per-step for `every`, final packet for `feature`). Show the exact candidate and obtain commit authorization before committing. The setting itself is not authorization.
|
|
33
|
+
|
|
34
|
+
Independent-review checkpoints follow their separate contract below. Review cadence never authorizes commit, merge, push, or deployment.
|
|
29
35
|
|
|
30
36
|
## Multi-Run Target & Ticket-Driven Target Resolution
|
|
31
37
|
|
|
32
|
-
- **Given an ID or name** (e.g. `/implement 12`, `/implement 012`, `/implement kanban`) -> locates the matching run folder `devflow/context/{xxx-slug}/`,
|
|
33
|
-
- **With no argument** (`/implement`) -> checks current
|
|
38
|
+
- **Given an ID or name** (e.g. `/implement 12`, `/implement 012`, `/implement kanban`) -> locates the matching run folder `devflow/context/{xxx-slug}/`, selects the branch matching the run type and configured prefix, and loads only that run's `spec.md`.
|
|
39
|
+
- **With no argument** (`/implement`) -> checks the current feature, fix, or rollback branch against the configured prefixes, or auto-picks if only 1 spec is active in `devflow/context/`, or prompts the user if multiple specs are queued.
|
|
34
40
|
- **Ticket Input (`--ticket <NN | path>`)**: Targets a specific tracer-bullet ticket directly (e.g. `/implement 12 --ticket 01`, `/implement --ticket 02`, `/implement --ticket issues/01-auth.md`, or `/implement 01` when run-id is not ambiguous).
|
|
35
41
|
|
|
36
42
|
### Ticket Discovery Hierarchy
|
|
@@ -80,11 +86,7 @@ receipt. After the final Verify pass, an active gate routes to
|
|
|
80
86
|
`current` freshness may proceed toward `/complete`. This skill never approves
|
|
81
87
|
its own work or rewrites receipt evidence.
|
|
82
88
|
|
|
83
|
-
For
|
|
84
|
-
record the result in the step evidence. Use `browseros-neo` for the interactive
|
|
85
|
-
handoff when it is available and the done-when is visual or behavioral. When the
|
|
86
|
-
script or MCP is absent, report that exact limitation; do not silently install a
|
|
87
|
-
runner or claim browser evidence.
|
|
89
|
+
For browser-facing changes, run focused browser tests during iteration when available. Run the required `npm run test:browser` gate once before handoff if the spec or project requires it. Use `browseros-neo` for visual or behavioral evidence when available and relevant. Record missing tooling accurately; install a runner only when setup is in the authorized scope.
|
|
88
90
|
|
|
89
91
|
**Resuming?** If the spec already has some build steps checked off (`- [x]`), this
|
|
90
92
|
feature was started earlier and interrupted (often a cleared context). The spec and
|
|
@@ -95,8 +97,7 @@ starting over.
|
|
|
95
97
|
|
|
96
98
|
## Step 1 - branch
|
|
97
99
|
|
|
98
|
-
Create
|
|
99
|
-
`fix/{xxx-slug}` for a fix, or `rollback/{xxx-slug}` for a Type: Rollback spec. If the
|
|
100
|
+
Create or select the branch for the spec using `git.featureBranchPrefix`, `git.fixBranchPrefix`, or `git.rollbackBranchPrefix` from config (defaults: `feature/`, `fix/`, `rollback/`). Preserve unrelated working-tree changes before switching. If the
|
|
100
101
|
project isn't a git repo yet, say so and ask the user to run `git init` first;
|
|
101
102
|
the loop needs branches. On resume, the branch already exists - check it out
|
|
102
103
|
instead of creating a new one.
|
|
@@ -105,99 +106,20 @@ instead of creating a new one.
|
|
|
105
106
|
|
|
106
107
|
When implementing a rollback task, follow the exact safety procedure in `reference/rollback-implementation.md`.
|
|
107
108
|
|
|
108
|
-
## Step 2 -
|
|
109
|
-
|
|
110
|
-
Work
|
|
111
|
-
|
|
112
|
-
1.
|
|
113
|
-
|
|
114
|
-
|
|
115
|
-
|
|
116
|
-
|
|
117
|
-
|
|
118
|
-
|
|
119
|
-
|
|
120
|
-
|
|
121
|
-
|
|
122
|
-
- The error message identifies missing behavior, rather than invalid configuration
|
|
123
|
-
- The failure reflects the behavior under test, rather than a bug in the test itself
|
|
124
|
-
|
|
125
|
-
**Green-Phase Discipline**: Write the minimum code needed to pass the test; add only the options/features it requires (YAGNI constraint).
|
|
126
|
-
|
|
127
|
-
> 📖 For anti-patterns and Bad vs Good examples, read [`tdd-anti-patterns.md`](tdd-anti-patterns.md).
|
|
128
|
-
|
|
129
|
-
2. Implement just that step: the smallest change that satisfies its "done when."
|
|
130
|
-
3. Show the **diff**, not whole files.
|
|
131
|
-
4. **Explain it, and prove it.** Give a short summary: what the step delivered,
|
|
132
|
-
one line per changed file on what it does and why, then confirm the step's
|
|
133
|
-
"done when" is met with empirical evidence (test pass output, build output, or screenshot). This summary is the comprehension gate, so keep it concrete, not
|
|
134
|
-
vague. Include a short **How to try it** note when the step has a manual
|
|
135
|
-
path: the command, URL, click, endpoint, or output the user can check.
|
|
136
|
-
5. **Verify the step.** If `AGENTS.md` declares a `Verify` command, run that exact
|
|
137
|
-
command as the automated gate. It is only an umbrella for checks the project
|
|
138
|
-
actually has, so do not invent tests or other checks to satisfy it. If no
|
|
139
|
-
`Verify` command exists, run the documented build command and the test command
|
|
140
|
-
when the project declares one. A step that adds logic must ship a passing test
|
|
141
|
-
in the same diff when the test gate is on, and the suite must be green before
|
|
142
|
-
the step is approved (see the Testing gate in `coding-standards.md`). UI and
|
|
143
|
-
integration-only steps ride on screenshot plus build evidence. Run a focused
|
|
144
|
-
test separately when it gives faster feedback, then use `Verify` as the final
|
|
145
|
-
automated gate. For UI or integration done-whens, prefer Playwright when it is
|
|
146
|
-
already installed or declared in `AGENTS.md`; do not add it silently for an
|
|
147
|
-
unrelated feature. Create focused test files next to the source they cover,
|
|
148
|
-
per `coding-standards.md`. Never install a runner mid-step unless the current
|
|
149
|
-
spec is explicitly the unit-testing setup itself (for example `/fix "add unit
|
|
150
|
-
testing"`). If a step surfaces non-trivial logic the spec did not foresee, add
|
|
151
|
-
a focused test then, or note why not. When a step's done-when is behavioral (a
|
|
152
|
-
click, a download, or a flow across screens), run `/check` to prove it against
|
|
153
|
-
the running app rather than eyeballing it.
|
|
154
|
-
5. **Iterate until it works.** If it fails or the user wants changes, revise the
|
|
155
|
-
step (re-prompt or hand-edit the code), show the updated diff, and re-test.
|
|
156
|
-
Repeat until it works and the user approves. Nothing is committed until the
|
|
157
|
-
user is happy with the step.
|
|
158
|
-
6. **Mark it done, then prompt to move on.** Once the step is approved, check that
|
|
159
|
-
step off (`- [x]`) in `devflow/context/{xxx-slug}/spec.md` so progress survives a context
|
|
160
|
-
clear. If the step repaired a finding tracked in
|
|
161
|
-
`devflow/context/{xxx-slug}/findings.md`, set that finding's status to `fixed` now too
|
|
162
|
-
and note the repair in its **Resolution** line. Never set `closed`: a repair
|
|
163
|
-
is re-reviewed by `/audit` before it clears, because a fix can introduce a
|
|
164
|
-
worse defect than the one it removed. Then offer a short choice, noting that checkpoints are optional since
|
|
165
|
-
`/complete` makes the real feature-level commit. Use the current tool's short
|
|
166
|
-
user-input prompt when available; when you've just produced a long block to
|
|
167
|
-
read (a deep explanation, a big
|
|
168
|
-
walk-through), ask in plain text instead, so the prompt doesn't cover what the
|
|
169
|
-
user is still reading:
|
|
170
|
-
- **Continue** (default) - roll into the next step without committing.
|
|
171
|
-
- **Commit checkpoint** - commit just this step on the branch with a
|
|
172
|
-
conventional message (a cheap rollback point).
|
|
173
|
-
- **Walk me through it** - give a deeper, line-level explanation of the new or
|
|
174
|
-
changed code (why this approach, what each part does, any gotchas), then
|
|
175
|
-
re-ask this checkpoint prompt. A loop-back, not a terminal choice.
|
|
176
|
-
- **Stop here** - pause the loop so the user can review or come back later.
|
|
177
|
-
|
|
178
|
-
On **Continue** or after **Commit checkpoint**, go to the next step. On **Walk
|
|
179
|
-
me through it**, explain in depth and then re-ask this prompt in plain text (the
|
|
180
|
-
explanation is long, so a modal would cover it). On **Stop here**, stop and say
|
|
181
|
-
where things stand: the branch is intact; run `/implement` again to resume, or
|
|
182
|
-
`/complete` to wrap up what's built so far.
|
|
183
|
-
|
|
184
|
-
### Ticket-Driven TDD Execution (when working from a Ticket)
|
|
185
|
-
|
|
186
|
-
When executing a tracer-bullet ticket (via `--ticket` or auto-frontier):
|
|
187
|
-
1. **Extract Scope**: Read *What to build* and *Acceptance criteria* checklist (`- [ ]`) from the active ticket file.
|
|
188
|
-
2. **Strict TDD Cycle per Criterion**:
|
|
189
|
-
- **🔴 RED**: Write the unit/integration test for the criterion first at the agreed seam. Execute the test command and verify the expected failing assertion.
|
|
190
|
-
- **🟢 GREEN**: Implement only the minimal code in the source file necessary to make the test pass. Re-run test and show passing output.
|
|
191
|
-
- **🔵 REFACTOR**: Refactor and format cleanly, verifying that 100% of tests remain green.
|
|
192
|
-
3. **Verify Gate**: Run the project's Verify command (`npm run check` or documented test suite) to ensure whole-system health before concluding the ticket.
|
|
193
|
-
4. **Bidirectional State Sync**:
|
|
194
|
-
- Mark each completed criterion `- [x]` in the ticket file.
|
|
195
|
-
- Once all criteria in the ticket pass, update the ticket status to `**Status:** done` (or `Status: done`).
|
|
196
|
-
- Mirror the completed work, status, and diff evidence into `devflow/context/{xxx-slug}/spec.md` (under `## 📋 3. Execution Plan & TDD Checklist` and `## ⚡ 4. Implementation Log & Evidence`) so the Living Spec remains the authoritative single source of truth for `/check`, `/audit`, and `/complete`.
|
|
197
|
-
|
|
198
|
-
Never batch the whole thing into one diff. If a step's diff is too big to read,
|
|
199
|
-
split it. The documented `Verify` command, or the fallback build and tests, must
|
|
200
|
-
pass before any commit.
|
|
109
|
+
## Step 2 - implement and verify
|
|
110
|
+
|
|
111
|
+
Work in small reviewable diffs, keeping progress in the selected spec.
|
|
112
|
+
|
|
113
|
+
1. For logic or behavior changes with the test gate on, use Red-Green-Refactor: write a focused failing test, confirm the assertion fails for the missing behavior, implement the smallest change, then refactor with affected tests green. Preserve the user's existing work. Consult [TDD anti-patterns](tdd-anti-patterns.md) when test design needs it.
|
|
114
|
+
2. Match the step's done-when and run focused checks. Documentation or formatting-only edits use structural checks and review; they do not require artificial unit tests. Capture relevant runtime or browser proof for behavioral criteria, following the selected Check policy.
|
|
115
|
+
3. Fix failures caused by the change and rerun affected checks without requesting authorization again. Broaden verification when dependencies, failures, or risk justify it.
|
|
116
|
+
4. Record the diff summary and evidence. Mark a step complete once its done-when passes and any configured per-step review has been approved. A repaired finding becomes `fixed` with resolution evidence; only a subsequent audit can close it. Apply the review cadence above before proceeding.
|
|
117
|
+
|
|
118
|
+
At the final handoff, run the exact documented Verify command and all required gates. If there is no Verify command, use the project's declared build and test commands. Reuse a passing result for unchanged inputs within this pass; rerun affected gates after subsequent edits. Do not add tests or tools just to inflate the verification matrix.
|
|
119
|
+
|
|
120
|
+
### Ticket execution
|
|
121
|
+
|
|
122
|
+
When a ticket is selected, use its acceptance criteria as the steps above. Mark completed criteria in the ticket, mark it `done` after its criteria and required verification pass, and mirror status and evidence into the run's spec. A ticket handoff runs required gates once; criteria within the ticket use focused checks. A request to finish the whole run continues through the dependency frontier until the authorized scope is verified.
|
|
201
123
|
|
|
202
124
|
After final Verify and required Check pass, set the active spec to `verified`
|
|
203
125
|
with every completed box checked. Then resolve independent review before the
|
|
@@ -270,23 +192,19 @@ named area. If the feature spans too many distinct areas for one useful pass,
|
|
|
270
192
|
name the sections first and let the user choose where to begin. Remain read-only
|
|
271
193
|
unless the user separately requests changes.
|
|
272
194
|
|
|
273
|
-
|
|
274
|
-
The sole exception is exactly one immutable independent-review checkpoint after
|
|
275
|
-
showing its exact candidate and receiving current explicit commit approval.
|
|
276
|
-
Configuration never supplies that approval. Never merge, push, deploy, publish,
|
|
195
|
+
Ordinary checkpoint commits are available only through the configured review boundary and explicit candidate approval above. The immutable independent-review checkpoint also requires explicit candidate approval. Work-level completion commits belong to `/complete`. Configuration never supplies commit approval. Never merge, push, deploy, publish,
|
|
277
196
|
or start unrelated work from this skill.
|
|
278
197
|
|
|
279
198
|
## Rules
|
|
280
199
|
|
|
281
|
-
-
|
|
200
|
+
- Keep diffs small and apply the configured review cadence.
|
|
282
201
|
- Explain every change in plain English. Understanding the code is the point.
|
|
283
202
|
- Iterate on the branch until each step works; never commit code the user hasn't
|
|
284
203
|
approved.
|
|
285
204
|
- Follow `devflow/context/coding-standards.md` (server vs client, scope user-owned queries
|
|
286
205
|
by the authenticated user id, validate inputs, and so on).
|
|
287
|
-
-
|
|
288
|
-
|
|
289
|
-
- Per-step commits are optional checkpoints. The work-level commit, the merge,
|
|
206
|
+
- Keep the spec aligned with the authorized scope. Correct routine omissions with evidence; ask only when a material scope or design decision is unresolved.
|
|
207
|
+
- Configured checkpoint offers are optional. The work-level commit, the merge,
|
|
290
208
|
and any push are `/complete`'s job.
|
|
291
209
|
- For Type: Rollback, reverse only the approved product diff and preserve all
|
|
292
210
|
protected Blueprint paths.
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: bughunter
|
|
3
|
-
description: "[devflow]
|
|
3
|
+
description: "[devflow] Investigate security vulnerabilities with evidence. Use for /bughunter or an authorized security review of code or endpoints."
|
|
4
4
|
argument-hint: "[{target, vuln-class, or topic}]"
|
|
5
5
|
---
|
|
6
6
|
|
|
@@ -36,7 +36,8 @@ When invoked without an argument (or when determining the next step), inspect:
|
|
|
36
36
|
|
|
37
37
|
### Default State Recommendations
|
|
38
38
|
- If no run is active and user wants to start a feature -> Recommend `/feature <name>`.
|
|
39
|
-
- If
|
|
39
|
+
- If the user requests a bugfix plan -> Recommend `/fix <bug>`.
|
|
40
|
+
- If the user requests a completed fix -> preserve the implementation and verification scope; draft the required spec internally and continue to `/implement` using existing authorization.
|
|
40
41
|
- If no run is active and user has pending ideas in `devflow/ideas.md` -> Highlight `/feature IDEA-xxx` or `/discovery IDEA-xxx`.
|
|
41
42
|
- If no run is active and user wants deep architectural exploration -> Recommend `/discovery`.
|
|
42
43
|
- If user asks to check system health -> Recommend `doctor`.
|
|
@@ -48,7 +49,7 @@ When invoked without an argument (or when determining the next step), inspect:
|
|
|
48
49
|
| User Intent / Request Type | Recommended Skill | Normal Name / Alias | Lifecycle Path |
|
|
49
50
|
| :--- | :--- | :--- | :--- |
|
|
50
51
|
| **"Spec new feature / living spec"** | `feature` | `/feature` / `/spec` | **Unified**: `/feature` -> `/implement` -> `/check` -> `/complete` |
|
|
51
|
-
| **"
|
|
52
|
+
| **"Draft a bugfix / small-change plan"** | `fix` | `/fix` | **Unified**: `/fix` -> `/implement` -> `/check` -> `/complete` |
|
|
52
53
|
| **"Capture quick idea / thought"** | `idea` | `/idea` | **Companion**: Enriches & saves to `devflow/ideas.md` |
|
|
53
54
|
| **"Pre-delivery exploration / research"**| `discovery`| `/discovery` | **Pre-Flight**: `discovery` -> `/feature` |
|
|
54
55
|
| **"Socratic alignment / ADR / glossary"**| `grill` | `/grill` / `/align` | **Pre-Flight**: `grill` -> `/feature` |
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: fix
|
|
3
|
-
description: "[devflow]
|
|
3
|
+
description: "[devflow] Draft a task-isolated spec for a bug or small change. Use when invoking /fix or requesting a fix plan."
|
|
4
4
|
argument-hint: "[{title or issue-description}]"
|
|
5
5
|
---
|
|
6
6
|
|
|
@@ -23,7 +23,7 @@ it creates a dedicated run folder at `devflow/context/{xxx-slug}/` (e.g. `059-fi
|
|
|
23
23
|
## Input
|
|
24
24
|
|
|
25
25
|
A description of the bug or change, for example `/fix "password reset email never
|
|
26
|
-
sends"`.
|
|
26
|
+
sends"`. Use the reported problem as input when the user asks for a fix plan. A request to diagnose a problem belongs to `/debug`; a request to fix and verify it retains its full delivery scope.
|
|
27
27
|
|
|
28
28
|
The input may also be a finding ID from `devflow/context/{xxx-slug}/findings.md` (or previous findings), alone
|
|
29
29
|
or with a description, for example `/fix F-03`.
|
|
@@ -46,7 +46,7 @@ then write a short spec to `devflow/context/{xxx-slug}/spec.md`. Also initialize
|
|
|
46
46
|
big to read. Each ends with an observable "done when".
|
|
47
47
|
- **Verify** - how to confirm it's fixed (what to click or test).
|
|
48
48
|
|
|
49
|
-
|
|
49
|
+
For an explicit `/fix` or a planning-only request, present the spec as the completed deliverable and identify `/implement` as the next command. For an existing request to implement and verify the fix, preserve that authorization: use this spec as the internal handoff and continue through `/implement` without an extra approval pause. Ask only for unresolved material scope, design, or permission decisions.
|
|
50
50
|
|
|
51
51
|
## Rules
|
|
52
52
|
|
|
@@ -1,10 +1,10 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: implement
|
|
3
|
-
description: "[devflow]
|
|
3
|
+
description: "[devflow] Build and verify a task-isolated spec. Use for /implement or requests to implement an approved plan."
|
|
4
4
|
argument-hint: "[{run-id, number, or name}] [--ticket <NN | path>]"
|
|
5
5
|
---
|
|
6
6
|
|
|
7
|
-
# implement - build the target spec
|
|
7
|
+
# implement - build and verify the target spec
|
|
8
8
|
|
|
9
9
|
**Context reuse:** Reuse any required file already loaded in project instructions or the current session. Read it again only if absent, changed, or exact current bytes or line references are needed.
|
|
10
10
|
|
|
@@ -20,17 +20,23 @@ Where this sits in the workflow:
|
|
|
20
20
|
|
|
21
21
|
`/feature`, `/fix`, or `/rollback` wrote the spec to
|
|
22
22
|
`devflow/context/{xxx-slug}/spec.md` and stopped.
|
|
23
|
-
|
|
24
|
-
|
|
25
|
-
|
|
26
|
-
|
|
27
|
-
|
|
28
|
-
|
|
23
|
+
Build the selected spec through its acceptance criteria and required verification, then present the configured review handoff. Preserve authorization already given for this work. Read only relevant conventions and reuse loaded context.
|
|
24
|
+
|
|
25
|
+
## Review cadence and checkpoints
|
|
26
|
+
|
|
27
|
+
Read `workflow.stepReview` and `workflow.checkpointCommits` from `devflow/config.json` independently; defaults are `feature` and `disabled`.
|
|
28
|
+
|
|
29
|
+
- `stepReview: "feature"`: continue through all authorized steps without asking Continue after each step. Record evidence as work proceeds and present one final review packet.
|
|
30
|
+
- `stepReview: "every"`: after a working step, show its diff and evidence, then wait for approval before the next step. A walkthrough is available on request.
|
|
31
|
+
- `checkpointCommits: "disabled"`: omit ordinary checkpoint offers.
|
|
32
|
+
- `checkpointCommits: "enabled"`: offer an optional checkpoint at the configured review boundary (per-step for `every`, final packet for `feature`). Show the exact candidate and obtain commit authorization before committing. The setting itself is not authorization.
|
|
33
|
+
|
|
34
|
+
Independent-review checkpoints follow their separate contract below. Review cadence never authorizes commit, merge, push, or deployment.
|
|
29
35
|
|
|
30
36
|
## Multi-Run Target & Ticket-Driven Target Resolution
|
|
31
37
|
|
|
32
|
-
- **Given an ID or name** (e.g. `/implement 12`, `/implement 012`, `/implement kanban`) -> locates the matching run folder `devflow/context/{xxx-slug}/`,
|
|
33
|
-
- **With no argument** (`/implement`) -> checks current
|
|
38
|
+
- **Given an ID or name** (e.g. `/implement 12`, `/implement 012`, `/implement kanban`) -> locates the matching run folder `devflow/context/{xxx-slug}/`, selects the branch matching the run type and configured prefix, and loads only that run's `spec.md`.
|
|
39
|
+
- **With no argument** (`/implement`) -> checks the current feature, fix, or rollback branch against the configured prefixes, or auto-picks if only 1 spec is active in `devflow/context/`, or prompts the user if multiple specs are queued.
|
|
34
40
|
- **Ticket Input (`--ticket <NN | path>`)**: Targets a specific tracer-bullet ticket directly (e.g. `/implement 12 --ticket 01`, `/implement --ticket 02`, `/implement --ticket issues/01-auth.md`, or `/implement 01` when run-id is not ambiguous).
|
|
35
41
|
|
|
36
42
|
### Ticket Discovery Hierarchy
|
|
@@ -80,11 +86,7 @@ receipt. After the final Verify pass, an active gate routes to
|
|
|
80
86
|
`current` freshness may proceed toward `/complete`. This skill never approves
|
|
81
87
|
its own work or rewrites receipt evidence.
|
|
82
88
|
|
|
83
|
-
For
|
|
84
|
-
record the result in the step evidence. Use `browseros-neo` for the interactive
|
|
85
|
-
handoff when it is available and the done-when is visual or behavioral. When the
|
|
86
|
-
script or MCP is absent, report that exact limitation; do not silently install a
|
|
87
|
-
runner or claim browser evidence.
|
|
89
|
+
For browser-facing changes, run focused browser tests during iteration when available. Run the required `npm run test:browser` gate once before handoff if the spec or project requires it. Use `browseros-neo` for visual or behavioral evidence when available and relevant. Record missing tooling accurately; install a runner only when setup is in the authorized scope.
|
|
88
90
|
|
|
89
91
|
**Resuming?** If the spec already has some build steps checked off (`- [x]`), this
|
|
90
92
|
feature was started earlier and interrupted (often a cleared context). The spec and
|
|
@@ -95,8 +97,7 @@ starting over.
|
|
|
95
97
|
|
|
96
98
|
## Step 1 - branch
|
|
97
99
|
|
|
98
|
-
Create
|
|
99
|
-
`fix/{xxx-slug}` for a fix, or `rollback/{xxx-slug}` for a Type: Rollback spec. If the
|
|
100
|
+
Create or select the branch for the spec using `git.featureBranchPrefix`, `git.fixBranchPrefix`, or `git.rollbackBranchPrefix` from config (defaults: `feature/`, `fix/`, `rollback/`). Preserve unrelated working-tree changes before switching. If the
|
|
100
101
|
project isn't a git repo yet, say so and ask the user to run `git init` first;
|
|
101
102
|
the loop needs branches. On resume, the branch already exists - check it out
|
|
102
103
|
instead of creating a new one.
|
|
@@ -105,99 +106,20 @@ instead of creating a new one.
|
|
|
105
106
|
|
|
106
107
|
When implementing a rollback task, follow the exact safety procedure in `reference/rollback-implementation.md`.
|
|
107
108
|
|
|
108
|
-
## Step 2 -
|
|
109
|
-
|
|
110
|
-
Work
|
|
111
|
-
|
|
112
|
-
1.
|
|
113
|
-
|
|
114
|
-
|
|
115
|
-
|
|
116
|
-
|
|
117
|
-
|
|
118
|
-
|
|
119
|
-
|
|
120
|
-
|
|
121
|
-
|
|
122
|
-
- The error message identifies missing behavior, rather than invalid configuration
|
|
123
|
-
- The failure reflects the behavior under test, rather than a bug in the test itself
|
|
124
|
-
|
|
125
|
-
**Green-Phase Discipline**: Write the minimum code needed to pass the test; add only the options/features it requires (YAGNI constraint).
|
|
126
|
-
|
|
127
|
-
> 📖 For anti-patterns and Bad vs Good examples, read [`tdd-anti-patterns.md`](tdd-anti-patterns.md).
|
|
128
|
-
|
|
129
|
-
2. Implement just that step: the smallest change that satisfies its "done when."
|
|
130
|
-
3. Show the **diff**, not whole files.
|
|
131
|
-
4. **Explain it, and prove it.** Give a short summary: what the step delivered,
|
|
132
|
-
one line per changed file on what it does and why, then confirm the step's
|
|
133
|
-
"done when" is met with empirical evidence (test pass output, build output, or screenshot). This summary is the comprehension gate, so keep it concrete, not
|
|
134
|
-
vague. Include a short **How to try it** note when the step has a manual
|
|
135
|
-
path: the command, URL, click, endpoint, or output the user can check.
|
|
136
|
-
5. **Verify the step.** If `AGENTS.md` declares a `Verify` command, run that exact
|
|
137
|
-
command as the automated gate. It is only an umbrella for checks the project
|
|
138
|
-
actually has, so do not invent tests or other checks to satisfy it. If no
|
|
139
|
-
`Verify` command exists, run the documented build command and the test command
|
|
140
|
-
when the project declares one. A step that adds logic must ship a passing test
|
|
141
|
-
in the same diff when the test gate is on, and the suite must be green before
|
|
142
|
-
the step is approved (see the Testing gate in `coding-standards.md`). UI and
|
|
143
|
-
integration-only steps ride on screenshot plus build evidence. Run a focused
|
|
144
|
-
test separately when it gives faster feedback, then use `Verify` as the final
|
|
145
|
-
automated gate. For UI or integration done-whens, prefer Playwright when it is
|
|
146
|
-
already installed or declared in `AGENTS.md`; do not add it silently for an
|
|
147
|
-
unrelated feature. Create focused test files next to the source they cover,
|
|
148
|
-
per `coding-standards.md`. Never install a runner mid-step unless the current
|
|
149
|
-
spec is explicitly the unit-testing setup itself (for example `/fix "add unit
|
|
150
|
-
testing"`). If a step surfaces non-trivial logic the spec did not foresee, add
|
|
151
|
-
a focused test then, or note why not. When a step's done-when is behavioral (a
|
|
152
|
-
click, a download, or a flow across screens), run `/check` to prove it against
|
|
153
|
-
the running app rather than eyeballing it.
|
|
154
|
-
5. **Iterate until it works.** If it fails or the user wants changes, revise the
|
|
155
|
-
step (re-prompt or hand-edit the code), show the updated diff, and re-test.
|
|
156
|
-
Repeat until it works and the user approves. Nothing is committed until the
|
|
157
|
-
user is happy with the step.
|
|
158
|
-
6. **Mark it done, then prompt to move on.** Once the step is approved, check that
|
|
159
|
-
step off (`- [x]`) in `devflow/context/{xxx-slug}/spec.md` so progress survives a context
|
|
160
|
-
clear. If the step repaired a finding tracked in
|
|
161
|
-
`devflow/context/{xxx-slug}/findings.md`, set that finding's status to `fixed` now too
|
|
162
|
-
and note the repair in its **Resolution** line. Never set `closed`: a repair
|
|
163
|
-
is re-reviewed by `/audit` before it clears, because a fix can introduce a
|
|
164
|
-
worse defect than the one it removed. Then offer a short choice, noting that checkpoints are optional since
|
|
165
|
-
`/complete` makes the real feature-level commit. Use the current tool's short
|
|
166
|
-
user-input prompt when available; when you've just produced a long block to
|
|
167
|
-
read (a deep explanation, a big
|
|
168
|
-
walk-through), ask in plain text instead, so the prompt doesn't cover what the
|
|
169
|
-
user is still reading:
|
|
170
|
-
- **Continue** (default) - roll into the next step without committing.
|
|
171
|
-
- **Commit checkpoint** - commit just this step on the branch with a
|
|
172
|
-
conventional message (a cheap rollback point).
|
|
173
|
-
- **Walk me through it** - give a deeper, line-level explanation of the new or
|
|
174
|
-
changed code (why this approach, what each part does, any gotchas), then
|
|
175
|
-
re-ask this checkpoint prompt. A loop-back, not a terminal choice.
|
|
176
|
-
- **Stop here** - pause the loop so the user can review or come back later.
|
|
177
|
-
|
|
178
|
-
On **Continue** or after **Commit checkpoint**, go to the next step. On **Walk
|
|
179
|
-
me through it**, explain in depth and then re-ask this prompt in plain text (the
|
|
180
|
-
explanation is long, so a modal would cover it). On **Stop here**, stop and say
|
|
181
|
-
where things stand: the branch is intact; run `/implement` again to resume, or
|
|
182
|
-
`/complete` to wrap up what's built so far.
|
|
183
|
-
|
|
184
|
-
### Ticket-Driven TDD Execution (when working from a Ticket)
|
|
185
|
-
|
|
186
|
-
When executing a tracer-bullet ticket (via `--ticket` or auto-frontier):
|
|
187
|
-
1. **Extract Scope**: Read *What to build* and *Acceptance criteria* checklist (`- [ ]`) from the active ticket file.
|
|
188
|
-
2. **Strict TDD Cycle per Criterion**:
|
|
189
|
-
- **🔴 RED**: Write the unit/integration test for the criterion first at the agreed seam. Execute the test command and verify the expected failing assertion.
|
|
190
|
-
- **🟢 GREEN**: Implement only the minimal code in the source file necessary to make the test pass. Re-run test and show passing output.
|
|
191
|
-
- **🔵 REFACTOR**: Refactor and format cleanly, verifying that 100% of tests remain green.
|
|
192
|
-
3. **Verify Gate**: Run the project's Verify command (`npm run check` or documented test suite) to ensure whole-system health before concluding the ticket.
|
|
193
|
-
4. **Bidirectional State Sync**:
|
|
194
|
-
- Mark each completed criterion `- [x]` in the ticket file.
|
|
195
|
-
- Once all criteria in the ticket pass, update the ticket status to `**Status:** done` (or `Status: done`).
|
|
196
|
-
- Mirror the completed work, status, and diff evidence into `devflow/context/{xxx-slug}/spec.md` (under `## 📋 3. Execution Plan & TDD Checklist` and `## ⚡ 4. Implementation Log & Evidence`) so the Living Spec remains the authoritative single source of truth for `/check`, `/audit`, and `/complete`.
|
|
197
|
-
|
|
198
|
-
Never batch the whole thing into one diff. If a step's diff is too big to read,
|
|
199
|
-
split it. The documented `Verify` command, or the fallback build and tests, must
|
|
200
|
-
pass before any commit.
|
|
109
|
+
## Step 2 - implement and verify
|
|
110
|
+
|
|
111
|
+
Work in small reviewable diffs, keeping progress in the selected spec.
|
|
112
|
+
|
|
113
|
+
1. For logic or behavior changes with the test gate on, use Red-Green-Refactor: write a focused failing test, confirm the assertion fails for the missing behavior, implement the smallest change, then refactor with affected tests green. Preserve the user's existing work. Consult [TDD anti-patterns](tdd-anti-patterns.md) when test design needs it.
|
|
114
|
+
2. Match the step's done-when and run focused checks. Documentation or formatting-only edits use structural checks and review; they do not require artificial unit tests. Capture relevant runtime or browser proof for behavioral criteria, following the selected Check policy.
|
|
115
|
+
3. Fix failures caused by the change and rerun affected checks without requesting authorization again. Broaden verification when dependencies, failures, or risk justify it.
|
|
116
|
+
4. Record the diff summary and evidence. Mark a step complete once its done-when passes and any configured per-step review has been approved. A repaired finding becomes `fixed` with resolution evidence; only a subsequent audit can close it. Apply the review cadence above before proceeding.
|
|
117
|
+
|
|
118
|
+
At the final handoff, run the exact documented Verify command and all required gates. If there is no Verify command, use the project's declared build and test commands. Reuse a passing result for unchanged inputs within this pass; rerun affected gates after subsequent edits. Do not add tests or tools just to inflate the verification matrix.
|
|
119
|
+
|
|
120
|
+
### Ticket execution
|
|
121
|
+
|
|
122
|
+
When a ticket is selected, use its acceptance criteria as the steps above. Mark completed criteria in the ticket, mark it `done` after its criteria and required verification pass, and mirror status and evidence into the run's spec. A ticket handoff runs required gates once; criteria within the ticket use focused checks. A request to finish the whole run continues through the dependency frontier until the authorized scope is verified.
|
|
201
123
|
|
|
202
124
|
After final Verify and required Check pass, set the active spec to `verified`
|
|
203
125
|
with every completed box checked. Then resolve independent review before the
|
|
@@ -270,23 +192,19 @@ named area. If the feature spans too many distinct areas for one useful pass,
|
|
|
270
192
|
name the sections first and let the user choose where to begin. Remain read-only
|
|
271
193
|
unless the user separately requests changes.
|
|
272
194
|
|
|
273
|
-
|
|
274
|
-
The sole exception is exactly one immutable independent-review checkpoint after
|
|
275
|
-
showing its exact candidate and receiving current explicit commit approval.
|
|
276
|
-
Configuration never supplies that approval. Never merge, push, deploy, publish,
|
|
195
|
+
Ordinary checkpoint commits are available only through the configured review boundary and explicit candidate approval above. The immutable independent-review checkpoint also requires explicit candidate approval. Work-level completion commits belong to `/complete`. Configuration never supplies commit approval. Never merge, push, deploy, publish,
|
|
277
196
|
or start unrelated work from this skill.
|
|
278
197
|
|
|
279
198
|
## Rules
|
|
280
199
|
|
|
281
|
-
-
|
|
200
|
+
- Keep diffs small and apply the configured review cadence.
|
|
282
201
|
- Explain every change in plain English. Understanding the code is the point.
|
|
283
202
|
- Iterate on the branch until each step works; never commit code the user hasn't
|
|
284
203
|
approved.
|
|
285
204
|
- Follow `devflow/context/coding-standards.md` (server vs client, scope user-owned queries
|
|
286
205
|
by the authenticated user id, validate inputs, and so on).
|
|
287
|
-
-
|
|
288
|
-
|
|
289
|
-
- Per-step commits are optional checkpoints. The work-level commit, the merge,
|
|
206
|
+
- Keep the spec aligned with the authorized scope. Correct routine omissions with evidence; ask only when a material scope or design decision is unresolved.
|
|
207
|
+
- Configured checkpoint offers are optional. The work-level commit, the merge,
|
|
290
208
|
and any push are `/complete`'s job.
|
|
291
209
|
- For Type: Rollback, reverse only the approved product diff and preserve all
|
|
292
210
|
protected Blueprint paths.
|
package/template/AGENTS.md
CHANGED
|
@@ -11,14 +11,22 @@ This project uses **Nexus-DevFlow**, an agentic workflow layer supporting **The
|
|
|
11
11
|
|
|
12
12
|
To start a new project, scaffold the application first in an empty folder, then run `npx nexus-devflow` (or `npm create @jakkrichm/nexus-devflow`) to overlay DevFlow onto your codebase.
|
|
13
13
|
|
|
14
|
-
##
|
|
14
|
+
## Load context for the task
|
|
15
15
|
|
|
16
|
-
|
|
17
|
-
|
|
18
|
-
- `devflow/
|
|
19
|
-
- `devflow/context/
|
|
20
|
-
- `devflow/context/
|
|
21
|
-
- `devflow/context/
|
|
16
|
+
Reuse instructions already loaded. Read relevant sections, rather than every document before every edit:
|
|
17
|
+
|
|
18
|
+
- `devflow/config.json` for workflow policy and quality-gate decisions.
|
|
19
|
+
- `devflow/context/project-overview.md` for project purpose, architecture, and cross-module boundaries.
|
|
20
|
+
- `devflow/context/coding-standards.md` for conventions affecting changed code.
|
|
21
|
+
- `devflow/context/ai-interaction.md` for workflow interaction and review cadence.
|
|
22
|
+
- `devflow/context/glossary.md` when domain terminology matters.
|
|
23
|
+
- `devflow/context/{xxx-slug}/` for the selected run's spec, stage, findings, and review receipt.
|
|
24
|
+
|
|
25
|
+
## Task intent and authorization
|
|
26
|
+
|
|
27
|
+
Explicit user scope takes precedence over skill guidance, within runtime permissions and safety constraints. A request to draft a spec ends at the spec; a request to implement and verify continues through those authorized steps. A skill transition is not a new approval requirement. Ask when a material decision or authorization is missing, reuse existing authorization, and identify the exact instruction if it requires a pause. Preserve separate authorization for commit, merge, push, and deployment.
|
|
28
|
+
|
|
29
|
+
Use focused checks during edits, and run the documented required gates before handoff. Repeat passing checks only after relevant changes, failures, or unresolved concerns. Apply TDD to logic and behavior changes; documentation-only edits need appropriate structural or review evidence.
|
|
22
30
|
|
|
23
31
|
## Project configuration
|
|
24
32
|
|
|
@@ -50,37 +58,7 @@ Unused adapter families can be removed. Codex, Antigravity, GitHub Copilot, and
|
|
|
50
58
|
3. **Google Antigravity & Claude Code**: Native skill engines automatically discover and surface `.agents/skills/` and `.claude/skills/`.
|
|
51
59
|
4. **State-Aware Inspection**: When unsure what to do next, invoke `devflow` to automatically inspect active task directories in `devflow/context/{xxx-slug}/`, `devflow/discoveries/`, and `devflow/ideas.md`.
|
|
52
60
|
5. **Default Artifact & Communication Language (Thai)**: All generated markdown stage artifacts (`spec.md`, `discovery.md`, etc.) and user communication MUST default to **Thai (`th`)**, while code, technical terms, file paths, and identifiers remain in English.
|
|
53
|
-
6. **
|
|
54
|
-
Whenever any skill or command is invoked with a help keyword or flag (e.g., `/bughunter help`, `bughunter --help`, `/archify -h`, `/check ?`, `/feature help`):
|
|
55
|
-
- **Execution Safety Gate**: DO NOT execute the skill's primary or mutating actions (do not launch scans/attacks, do not modify source code, and do not initialize new task runs).
|
|
56
|
-
- **Dynamic Workspace Inspection**: Read the target skill definition (`.agents/skills/<skill>/SKILL.md`, `.claude/skills/<skill>/SKILL.md`, and any vendored resources such as `devflow/.vendor/<skill>/`), then scan the active workspace (`devflow/context/project-overview.md`, `package.json`, tech stack, active tasks, endpoints, and file hierarchy).
|
|
57
|
-
- **Persistent HTML Playbook & Token-Saving Cache**:
|
|
58
|
-
- **Output Location & Light Theme Mandate**: Compile and persist the tailored playbook as a standalone, styled HTML report at `devflow/docs/playbooks/{skill}.html` adhering to the **Paper/Ink Light Theme Standard** (`data-theme="light"`, teal accent, ruler corner, terminal mockup UI, and diagram slot per `devflow/docs/playbooks/template.html`). Never generate dark-mode-first documentation.
|
|
59
|
-
- **First Run**: Render the comprehensive 5W1H playbook in chat and write the complete HTML report to `devflow/docs/playbooks/{skill}.html`.
|
|
60
|
-
- **Subsequent Invocations (Delta Updates & Zero Token Bloat)**: If `devflow/docs/playbooks/{skill}.html` already exists:
|
|
61
|
-
- Inspect the existing HTML file and compute the diff/delta against the current repository state (e.g., new file paths, modified endpoints, or newly installed vendor assets).
|
|
62
|
-
- DO NOT repeat unchanged boilerplate or full static content in chat.
|
|
63
|
-
- Provide a concise **Delta Summary** in chat (highlighting what changed, new recommendations, or newly available commands).
|
|
64
|
-
- In-place update `devflow/docs/playbooks/{skill}.html` with the fresh timestamp and state.
|
|
65
|
-
- Provide a direct clickable link to [devflow/docs/playbooks/{skill}.html](file:///d:/devtools/nexus-devflow/devflow/docs/playbooks/{skill}.html) for browser viewing.
|
|
66
|
-
- **Render the User-Friendly 5W1H Playbook**: Emit an easy-to-read, practical guide organized around the **5W 1H Framework** (rendered on initial run or updated in the HTML playbook):
|
|
67
|
-
1. **What (Concept & Capabilities)**: Clear, jargon-free summary of what this skill does, its core concept, and its bundled tools/capabilities.
|
|
68
|
-
2. **Why (Problems Solved & Key Benefits)**: The problems it solves, key advantages, and why you should use it instead of ad-hoc manual work.
|
|
69
|
-
3. **Who (Target Roles & Personas)**: Target audience or engineering role (e.g., Feature Developer, Senior QA, Security Auditor, Solo Dev, or Tech Lead).
|
|
70
|
-
4. **Where (Boundaries & Artifact Locations)**: Target workspace boundaries, affected directories, and generated artifact locations (e.g., `devflow/context/`, `devflow/discoveries/`, `findings.md`).
|
|
71
|
-
5. **When (Timing & Prerequisites/Next Steps)**:
|
|
72
|
-
- **Timing & Triggers**: Practical situations or milestones when this skill should be invoked.
|
|
73
|
-
- **Run Before (Prerequisites)**: What to check, prepare, or run beforehand to get the best result.
|
|
74
|
-
- **Run After (Next Steps)**: What DevFlow command or lifecycle stage should follow next (e.g., handing off to `/fix`, `/feature`, or `/complete`).
|
|
75
|
-
6. **How (Execution Lifecycle & Real Command Examples)**:
|
|
76
|
-
- **How It Works (Lifecycle)**: Simple, plain-language breakdown of the execution steps or phases.
|
|
77
|
-
- **Available Commands & Options**: Common sub-commands, arguments, and flags with brief explanations.
|
|
78
|
-
- **Safety, Guardrails & Precautions**: Operational boundaries, credential hygiene, read-only vs. mutating constraints, and explicit special safety constraints / precautions.
|
|
79
|
-
7. **Diagram Slot & Architecture Flow**:
|
|
80
|
-
- **Real Diagram Mandate**: MUST embed a real, verified vector SVG or interactive diagram reflecting actual skill lifecycle, state transitions, or threat models in Light Theme (`data-theme="light"`).
|
|
81
|
-
- **Thai Language & Legible Typography**: Architecture and flow diagrams must describe workflows and nodes in Thai language while preserving technical identifiers, paths, and commands in English. Typography must be sufficiently sized and legible for Thai script (Headers 13–14px, box content/descriptions 11–12px, line spacing 18–22px) to prevent tone marks and vowels from clipping or overlapping.
|
|
82
|
-
- **Strictly No Placeholders**: NEVER output empty placeholder cards, generic instruction text, or phrases like *"when invoking archify..."*. Every playbook must ship with a real visual architectural diagram.
|
|
83
|
-
- Always conclude the response with: `"Help menu displayed successfully"`.
|
|
61
|
+
6. **Contextual Help**: When a DevFlow command is invoked with `help`, `--help`, `-h`, or `?`, read [the playbook protocol](devflow/reference/help-playbook-protocol.md). Produce its help deliverable without running the command's primary action or creating a task run.
|
|
84
62
|
|
|
85
63
|
---
|
|
86
64
|
|
|
@@ -8,7 +8,7 @@
|
|
|
8
8
|
|
|
9
9
|
- **Be Concise and Direct**: State conclusions, status, and findings first; provide supporting context afterward.
|
|
10
10
|
- **Explain Non-Obvious Decisions Briefly**: Highlight architectural trade-offs, edge-case rationale, or safety considerations in 1–2 sentences.
|
|
11
|
-
- **Ask
|
|
11
|
+
- **Authorization**: Reuse explicit authorization already given for the requested work. Ask before destructive actions or material architectural changes outside that scope. Identify the instruction requiring a pause; routine implementation choices do not require renewed approval.
|
|
12
12
|
- **Don't Add Unplanned Scope**: Stick strictly to the Acceptance Criteria defined in the spec. Avoid adding "nice-to-have" features that were not requested.
|
|
13
13
|
- **Preserve Existing Codebase Patterns**: Respect existing file structure, typing patterns, naming conventions, and deep module boundaries.
|
|
14
14
|
|
|
@@ -21,8 +21,8 @@ Format every response for fast scanning and readability:
|
|
|
21
21
|
- **Real Markdown, Not Prose Walls**: Use bold labels, concise bullet points, and blank lines between blocks.
|
|
22
22
|
- **Enumerations Are Lists**: Numbered or bulleted lists for sequential steps or findings, never inline runs crammed into paragraphs.
|
|
23
23
|
- **Tables for Comparative Matrices**: Use markdown tables when comparing status, options, test results, or trade-offs.
|
|
24
|
-
- **Backticks for Code References**: Wrap file paths, variable names, functions, CLI flags, and commands in backticks (e.g. `
|
|
25
|
-
- **Clickable File Links**: Use
|
|
24
|
+
- **Backticks for Code References**: Wrap file paths, variable names, functions, CLI flags, and commands in backticks (e.g. `devflow/context/{xxx-slug}/spec.md`, `npm run check`).
|
|
25
|
+
- **Clickable File Links**: Use the active client's supported local-file link format with the actual resolved path.
|
|
26
26
|
- **Lead With the Result**: State pass/fail status or completed action before presenting logs.
|
|
27
27
|
|
|
28
28
|
---
|
|
@@ -97,7 +97,7 @@ After the first successful `/overview`, DevFlow offers a reviewed local commit f
|
|
|
97
97
|
## 4. Strict TDD & Two-Stage Review Interaction Rules
|
|
98
98
|
|
|
99
99
|
### 🔴🟢 Strict TDD Execution Discipline
|
|
100
|
-
|
|
100
|
+
For logic and behavior changes during `/implement`, follow the spec's test decision:
|
|
101
101
|
- **Show Red Phase**: First execute tests to demonstrate expected failure *before* adding production code.
|
|
102
102
|
- **Show Green Phase**: Add minimal production code, re-run tests, and report pass rate.
|
|
103
103
|
- **Show Refactor Phase**: Polish and clean up with zero test regression.
|
|
@@ -111,10 +111,10 @@ During `/check`:
|
|
|
111
111
|
|
|
112
112
|
### ⚡ Review Cadence & Implementation Loop
|
|
113
113
|
- **Efficient Default (`stepReview: "feature"`)**: Presents one comprehensive review packet after all implementation steps of the feature complete, with step checkpoint commits disabled (`checkpointCommits: "disabled"`). This minimizes review fatigue and saves context window tokens.
|
|
114
|
-
- **Guided Review (`stepReview: "every"`)**: Available for high-risk changes, pair programming, or instructional sessions. Pauses for user approval after each small step
|
|
114
|
+
- **Guided Review (`stepReview: "every"`)**: Available for high-risk changes, pair programming, or instructional sessions. Pauses for user approval after each small step. Offer optional checkpoints only if `checkpointCommits: "enabled"`.
|
|
115
115
|
- **Post-Implementation Walkthrough**: After the final review packet, `/implement` always offers an optional read-only walkthrough of the finished code, regardless of review cadence or checkpoint settings.
|
|
116
116
|
- **Independent Review Execution**: Regular and Continuous independent review default to `when-sensitive`, while `review.independentExecution` defaults to `automatic` (spawning an isolated reviewer child when the active adapter supports it). Setting `review.independentExecution: "manual"` produces the fresh-session handoff.
|
|
117
|
-
- **Configuration**: Managed directly in `devflow/config.json`.
|
|
117
|
+
- **Configuration**: Managed directly in `devflow/config.json`. The two workflow keys are independent: `every` does not imply checkpoints are enabled, and `enabled` never grants commit authorization. Use the implementation skill's cadence contract for all four combinations.
|
|
118
118
|
|
|
119
119
|
---
|
|
120
120
|
|
|
@@ -131,16 +131,17 @@ During `/check`:
|
|
|
131
131
|
|
|
132
132
|
Progress lives in persistent files, not in transient chat history:
|
|
133
133
|
|
|
134
|
-
- In
|
|
134
|
+
- In the selected run: `devflow/context/{xxx-slug}/spec.md` maintains ticked checklist boxes `- [x]`.
|
|
135
135
|
- In Git: Commits, branches, and working tree maintain the code history.
|
|
136
|
-
- When starting a fresh session after a context clear, run
|
|
136
|
+
- When starting a fresh session after a context clear, resolve the run from an explicit ID or current branch, then inspect `devflow/context/{xxx-slug}/stage.md` and its spec to pick up immediately from the next pending step.
|
|
137
137
|
|
|
138
138
|
---
|
|
139
139
|
|
|
140
|
-
## 6.
|
|
140
|
+
## 6. Multi-Run Isolation
|
|
141
141
|
|
|
142
|
-
-
|
|
143
|
-
-
|
|
142
|
+
- Multiple specs may be queued in `devflow/context/{xxx-slug}/` without blocking new drafts.
|
|
143
|
+
- Select a run by explicit ID, matching branch, or the sole active candidate. Ask only when the target remains ambiguous.
|
|
144
|
+
- Keep each run's spec, stage, findings, and branch separate. Protect unrelated working-tree changes before switching; use an isolated worktree when concurrent implementation requires it.
|
|
144
145
|
|
|
145
146
|
---
|
|
146
147
|
|
|
@@ -177,14 +178,14 @@ Progress lives in persistent files, not in transient chat history:
|
|
|
177
178
|
Nexus-DevFlow natively supports extending workflow capabilities with **Third-Party Skills** (installed via `nexus-devflow skill add <repo-or-path>`):
|
|
178
179
|
|
|
179
180
|
### 🧩 1. Discovery & Inventory
|
|
180
|
-
-
|
|
181
|
+
- Reuse the available skill catalog. Inspect the active adapter or run `nexus-devflow skill list` only when a required capability is not already known.
|
|
181
182
|
- Installed third-party skills maintain their exact original directory structure, references, assets, scripts, and `SKILL.md`.
|
|
182
183
|
|
|
183
184
|
### 🎯 2. Cross-Command Delegation & Genuine Execution
|
|
184
|
-
- **Verbatim & Original Prompt Fidelity**: When a DevFlow stage requires specialized domain expertise (e.g., editorial diagramming, system mapping, domain modeling, cloud provisioning, UI benchmarking), the AI agent must read the third-party skill's `SKILL.md` (e.g. `.agents/skills/archify/SKILL.md` or `.agents/skills/diagram-design/SKILL.md`) and
|
|
185
|
+
- **Verbatim & Original Prompt Fidelity**: When a DevFlow stage requires specialized domain expertise (e.g., editorial diagramming, system mapping, domain modeling, cloud provisioning, UI benchmarking), the AI agent must read the third-party skill's `SKILL.md` (e.g. `.agents/skills/archify/SKILL.md` or `.agents/skills/diagram-design/SKILL.md`) and apply the relevant workflow within the user's requested scope. Explicit user instructions take precedence over skill guidance; reuse existing authorization rather than adding approval gates.
|
|
185
186
|
- **Workflow Command Integration Map**:
|
|
186
187
|
- **`/discovery`**: When macro roadmap planning or micro pre-flight exploration involves system architecture, IT landscape, data pipelines, sequence flows, or user journeys, check for `archify` (for interactive system maps, data flows, and sequence traces) or `diagram-design` (for editorial and business layouts).
|
|
187
|
-
- **`/feature` & `/fix`**:
|
|
188
|
+
- **`/feature` & `/fix`**: When a diagram materially clarifies Section 2 (`## 📐 2. Technical Spec & Contracts`), use `archify` for interactive component diagrams, state machines, and sequence traces, or `diagram-design` for database ER models.
|
|
188
189
|
- **`/implement`**: Seamlessly consumes Tracer-Bullet Tickets generated by Matt Pocock's `/to-tickets` (`.scratch/<feature>/issues/`, `devflow/context/{xxx-slug}/tickets/`), resolving the unblocked Frontier and executing with Strict TDD.
|
|
189
190
|
- **`/brainstorm` & `/grill`**: Leverage visual trade-off matrices (Quadrant, Radar spider) or domain models (UML class, ER).
|
|
190
191
|
- **`/prototype` & `/report-html`**: Embed standalone HTML/SVG assets seamlessly.
|
|
@@ -119,12 +119,13 @@ nexus-devflow/
|
|
|
119
119
|
|
|
120
120
|
Testing is a core quality gate in Nexus-DevFlow, not an afterthought:
|
|
121
121
|
|
|
122
|
-
- **Unit Test Mandate**:
|
|
122
|
+
- **Unit Test Mandate**: Logic or behavior changes must include meaningful automated coverage when the spec's test gate is enabled. Documentation, prompt wording, or formatting-only changes use structural checks and review instead of artificial unit tests.
|
|
123
123
|
- **Strict TDD (Red-Green-Refactor) Protocol**:
|
|
124
124
|
1. **🔴 RED (Test First)**: Always write automated tests *before* writing or modifying functional logic. Run the test command and verify that it fails for the expected reason.
|
|
125
125
|
2. **🟢 GREEN (Minimal Code)**: Write only the minimal production code necessary to make the failing test pass. Run the test command and verify 100% green pass.
|
|
126
126
|
3. **🔵 REFACTOR (Clean & Robust)**: Refactor code for readability, performance, and DRY/YAGNI discipline while ensuring all tests stay green.
|
|
127
127
|
- *Code Deletion / Reversion Rule*: If functional code is created without a prior failing test for behavior changes, it must be reverted or immediately backed by tests before continuing.
|
|
128
|
+
- **Verification Scope**: Run focused checks during edits and required project gates before handoff. Repeat passing checks only after relevant changes, new failures, or unresolved risk.
|
|
128
129
|
- **Test Framework**: Use Node.js native test runner executed via `tsx --test test/*.test.ts` under `packages/create-nexus-devflow/`.
|
|
129
130
|
- **Test Design (AAA Pattern)**:
|
|
130
131
|
- Structure each test case cleanly: **Arrange** (setup fixtures/mock directories), **Act** (execute function), **Assert** (verify invariants).
|
|
@@ -137,7 +138,7 @@ Testing is a core quality gate in Nexus-DevFlow, not an afterthought:
|
|
|
137
138
|
2. *Interactive AI Visual QA (MCP)*: Live DOM and styling inspection via MCP `browseros-neo` (`http://127.0.0.1:9010/mcp`) during `/check` and `/try`.
|
|
138
139
|
- **Two-Stage Review Pattern (Verification Gate)**:
|
|
139
140
|
- **Stage 1: Spec Fidelity & Acceptance Gate**:
|
|
140
|
-
- Verify 100% conformance against the Single Living Spec (`
|
|
141
|
+
- Verify 100% conformance against the Single Living Spec (`devflow/context/{xxx-slug}/spec.md`).
|
|
141
142
|
- Validate all Acceptance Criteria (ACs) and "Done When" observables without missing requirements or scope creep.
|
|
142
143
|
- Test edge cases and boundary conditions defined in the specification.
|
|
143
144
|
- **Stage 2: Code Quality, Security & Architecture Gate**:
|
|
@@ -145,13 +146,13 @@ Testing is a core quality gate in Nexus-DevFlow, not an afterthought:
|
|
|
145
146
|
- **Lane 2 (Automated Test Suites & Evals)**: `npm test` (Unit tests 100% pass) + `npm run test:routing` (Skill routing accuracy).
|
|
146
147
|
- **Lane 3 (Scrutinize & Security Audit)**: Edge cases, null-safety, 0 secrets, safe inputs.
|
|
147
148
|
- **Lane 4 (Manual / Scenario Proof)**: Concrete walkthrough steps ("Where to go", "What to run", "What to expect").
|
|
148
|
-
- **Findings Ledger State**: 0 blockers (P0/P1) in `devflow/context/findings.md`.
|
|
149
|
+
- **Findings Ledger State**: 0 blockers (P0/P1) in `devflow/context/{xxx-slug}/findings.md`.
|
|
149
150
|
|
|
150
151
|
---
|
|
151
152
|
|
|
152
153
|
## 8. Findings Ledger & Quality Gates (`findings.md`, `review.md`)
|
|
153
154
|
|
|
154
|
-
- All quality defects, security findings, or regression issues identified during review must be logged in `devflow/context/findings.md`.
|
|
155
|
+
- All quality defects, security findings, or regression issues identified during review must be logged in `devflow/context/{xxx-slug}/findings.md`.
|
|
155
156
|
- **Finding State Machine**:
|
|
156
157
|
- `open`: Confirmed issue waiting to be fixed.
|
|
157
158
|
- `fixed`: Repaired in code, pending QA re-verification.
|
|
@@ -4,15 +4,15 @@
|
|
|
4
4
|
|
|
5
5
|
---
|
|
6
6
|
|
|
7
|
-
###
|
|
8
|
-
- **Definition**: เอกสาร Markdown
|
|
9
|
-
- **Constraints**:
|
|
7
|
+
### Task-Isolated Living Spec (`devflow/context/{xxx-slug}/spec.md`)
|
|
8
|
+
- **Definition**: เอกสาร Markdown หลักหนึ่งฉบับต่อ runที่ใช้ขับเคลื่อนและบันทึกวงจรชีวิตของงานพัฒนาตั้งแต่ต้นจนจบ (Single Source of Truth during active delivery)
|
|
9
|
+
- **Constraints**: ร่างหลาย run ได้พร้อมกัน แต่แต่ละ run แยก spec, stage, findings และ branch; เลือกเป้าหมายก่อน implement
|
|
10
10
|
- **Structure**: ครอบคลุม 6 ส่วนหลัก: (1) Define & Boundaries, (2) Technical Spec & Contracts, (3) Execution Plan & TDD Tasks, (4) Implementation Log & Evidence, (5) Multi-Lane Verification Matrix, (6) Release Digest & Retrospective
|
|
11
|
-
- **Aliases / Related**: `
|
|
11
|
+
- **Aliases / Related**: `devflow/context/{xxx-slug}/spec.md`, Living Spec, Feature Spec
|
|
12
12
|
|
|
13
13
|
### Unified Fast-Track
|
|
14
14
|
- **Definition**: รูปแบบการพัฒนาหลักแบบ Single-Track ของ Nexus-DevFlow ที่รวมความสามารถเชิงสถาปัตยกรรมระดับลึก (Deep) และความคล่องตัว (Fast) เข้าด้วยกันผ่าน 4 คำสั่งหลัก: `/feature` (หรือ `/fix`), `/implement`, `/check`, และ `/complete`
|
|
15
|
-
- **Constraints**: ขับเคลื่อนผ่านไฟล์ `
|
|
15
|
+
- **Constraints**: ขับเคลื่อนผ่านไฟล์ `devflow/context/{xxx-slug}/spec.md` และเมื่อเสร็จสิ้นจะถูก Archive เป็นไฟล์ Markdown เดี่ยวใน `devflow/history/{features|fixes|rollbacks}/{xxx-slug}.md`
|
|
16
16
|
- **Aliases / Related**: Fast-Track, Unified Track
|
|
17
17
|
|
|
18
18
|
### Pre-Flight Discovery
|
|
@@ -22,7 +22,7 @@
|
|
|
22
22
|
|
|
23
23
|
### Multi-Lane Verification Matrix
|
|
24
24
|
- **Definition**: ตารางตรวจสอบคุณภาพแบบหลายมิติที่ครอบคลุม Typecheck, Linter, Automated Unit/Integration Tests, และ Manual Proof Evidence ในคำสั่ง `/check`
|
|
25
|
-
- **Constraints**:
|
|
25
|
+
- **Constraints**: ต้องมีหลักฐานผ่านสำหรับทุก Lane ที่ใช้กับงาน ส่วนที่ไม่เกี่ยวข้องให้บันทึก N/A พร้อมเหตุผล และไม่มี P0/P1 Finding ที่ยังค้างอยู่ก่อนที่จะส่งต่อไปยัง `/complete`
|
|
26
26
|
- **Aliases / Related**: QA Matrix, Lane Verification
|
|
27
27
|
|
|
28
28
|
### Release Digest & Retrospective
|
|
@@ -0,0 +1,33 @@
|
|
|
1
|
+
# Contextual Help Playbook
|
|
2
|
+
|
|
3
|
+
**Universal Contextual Help & Playbook Protocol (`help`, `--help`, `-h`, `?`)**:
|
|
4
|
+
Whenever any skill or command is invoked with a help keyword or flag (e.g., `/bughunter help`, `bughunter --help`, `/archify -h`, `/check ?`, `/feature help`):
|
|
5
|
+
- **Execution Safety Gate**: DO NOT execute the skill's primary or mutating actions (do not launch scans/attacks, do not modify source code, and do not initialize new task runs).
|
|
6
|
+
- **Dynamic Workspace Inspection**: Read the target skill in the active adapter. Inspect only workspace files and selected references needed to tailor its examples; reuse current context.
|
|
7
|
+
- **Persistent HTML Playbook & Token-Saving Cache**:
|
|
8
|
+
- **Output Location & Light Theme Mandate**: Compile and persist the tailored playbook as a standalone, styled HTML report at `devflow/docs/playbooks/{skill}.html` adhering to the **Paper/Ink Light Theme Standard** (`data-theme="light"`, teal accent, ruler corner, terminal mockup UI, and a real diagram). Reuse `devflow/docs/playbooks/template.html` when present; otherwise build the standalone page using these conventions.
|
|
9
|
+
- **First Run**: Render the comprehensive 5W1H playbook in chat and write the complete HTML report to `devflow/docs/playbooks/{skill}.html`.
|
|
10
|
+
- **Subsequent Invocations (Delta Updates & Zero Token Bloat)**: If `devflow/docs/playbooks/{skill}.html` already exists:
|
|
11
|
+
- Inspect the existing HTML file and compute the diff/delta against the current repository state (e.g., new file paths, modified endpoints, or newly installed vendor assets).
|
|
12
|
+
- DO NOT repeat unchanged boilerplate or full static content in chat.
|
|
13
|
+
- Provide a concise **Delta Summary** in chat (highlighting what changed, new recommendations, or newly available commands).
|
|
14
|
+
- In-place update `devflow/docs/playbooks/{skill}.html` with the fresh timestamp and state.
|
|
15
|
+
- Provide a clickable link to the actual resolved output file using the active client's supported local-file link format.
|
|
16
|
+
- **Render the User-Friendly 5W1H Playbook**: Emit an easy-to-read, practical guide organized around the **5W 1H Framework** (rendered on initial run or updated in the HTML playbook):
|
|
17
|
+
1. **What (Concept & Capabilities)**: Clear, jargon-free summary of what this skill does, its core concept, and its bundled tools/capabilities.
|
|
18
|
+
2. **Why (Problems Solved & Key Benefits)**: The problems it solves, key advantages, and why you should use it instead of ad-hoc manual work.
|
|
19
|
+
3. **Who (Target Roles & Personas)**: Target audience or engineering role (e.g., Feature Developer, Senior QA, Security Auditor, Solo Dev, or Tech Lead).
|
|
20
|
+
4. **Where (Boundaries & Artifact Locations)**: Target workspace boundaries, affected directories, and generated artifact locations (e.g., `devflow/context/`, `devflow/discoveries/`, `findings.md`).
|
|
21
|
+
5. **When (Timing & Prerequisites/Next Steps)**:
|
|
22
|
+
- **Timing & Triggers**: Practical situations or milestones when this skill should be invoked.
|
|
23
|
+
- **Run Before (Prerequisites)**: What to check, prepare, or run beforehand to get the best result.
|
|
24
|
+
- **Run After (Next Steps)**: What DevFlow command or lifecycle stage should follow next (e.g., handing off to `/fix`, `/feature`, or `/complete`).
|
|
25
|
+
6. **How (Execution Lifecycle & Real Command Examples)**:
|
|
26
|
+
- **How It Works (Lifecycle)**: Simple, plain-language breakdown of the execution steps or phases.
|
|
27
|
+
- **Available Commands & Options**: Common sub-commands, arguments, and flags with brief explanations.
|
|
28
|
+
- **Safety, Guardrails & Precautions**: Operational boundaries, credential hygiene, read-only vs. mutating constraints, and explicit special safety constraints / precautions.
|
|
29
|
+
7. **Diagram Slot & Architecture Flow**:
|
|
30
|
+
- **Real Diagram Mandate**: MUST embed a real, verified vector SVG or interactive diagram reflecting actual skill lifecycle, state transitions, or threat models in Light Theme (`data-theme="light"`).
|
|
31
|
+
- **Thai Language & Legible Typography**: Architecture and flow diagrams must describe workflows and nodes in Thai language while preserving technical identifiers, paths, and commands in English. Typography must be sufficiently sized and legible for Thai script (Headers 13–14px, box content/descriptions 11–12px, line spacing 18–22px) to prevent tone marks and vowels from clipping or overlapping.
|
|
32
|
+
- **Strictly No Placeholders**: NEVER output empty placeholder cards, generic instruction text, or phrases like *"when invoking archify..."*. Every playbook must ship with a real visual architectural diagram.
|
|
33
|
+
- Always conclude the response with: `"Help menu displayed successfully"`.
|