@jakkrichm/create-nexus-devflow 2.0.11 → 2.0.13
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/bin/create-nexus-devflow.js +15 -3
- package/lib/update.js +15 -1
- package/package.json +1 -1
- package/template/.agents/skills/70-release/SKILL.md +2 -0
- package/template/.agents/skills/ci/SKILL.md +25 -78
- package/template/.agents/skills/commit/SKILL.md +39 -43
- package/template/.agents/skills/debug/SKILL.md +43 -104
- package/template/.agents/skills/deploy/SKILL.md +37 -65
- package/template/.agents/skills/insight/SKILL.md +27 -116
- package/template/.agents/skills/preview/SKILL.md +24 -103
- package/template/.agents/skills/review/SKILL.md +53 -37
- package/template/.agents/skills/rollback/SKILL.md +1 -0
- package/template/.agents/skills/security-review/SKILL.md +44 -147
- package/template/.agents/skills/simplify/SKILL.md +48 -57
- package/template/.agents/skills/test/SKILL.md +63 -51
- package/template/.claude/skills/70-release/SKILL.md +2 -0
- package/template/.claude/skills/ci/SKILL.md +25 -78
- package/template/.claude/skills/commit/SKILL.md +39 -43
- package/template/.claude/skills/debug/SKILL.md +43 -104
- package/template/.claude/skills/deploy/SKILL.md +37 -65
- package/template/.claude/skills/insight/SKILL.md +27 -116
- package/template/.claude/skills/preview/SKILL.md +24 -103
- package/template/.claude/skills/review/SKILL.md +53 -37
- package/template/.claude/skills/rollback/SKILL.md +1 -0
- package/template/.claude/skills/security-review/SKILL.md +44 -147
- package/template/.claude/skills/simplify/SKILL.md +48 -57
- package/template/.claude/skills/test/SKILL.md +63 -51
- package/template/devflow/history/HISTORY.md +27 -0
- package/template/devflow/reference/running-id-contract.md +1 -1
- package/template/.agents/skills/9arm-skills/README.md +0 -51
- package/template/.agents/skills/9arm-skills/debug-mantra/SKILL.md +0 -86
- package/template/.agents/skills/9arm-skills/management-talk/SKILL.md +0 -79
- package/template/.agents/skills/9arm-skills/post-mortem/SKILL.md +0 -71
- package/template/.agents/skills/9arm-skills/scrutinize/SKILL.md +0 -72
- package/template/.agents/skills/browser-testing-with-devtools/SKILL.md +0 -302
- package/template/.agents/skills/ci-cd-and-automation/SKILL.md +0 -390
- package/template/.agents/skills/code-review-and-quality/SKILL.md +0 -392
- package/template/.agents/skills/code-simplification/SKILL.md +0 -331
- package/template/.agents/skills/debugging-and-error-recovery/SKILL.md +0 -298
- package/template/.agents/skills/deployment-procedures/SKILL.md +0 -241
- package/template/.agents/skills/deprecation-and-migration/SKILL.md +0 -206
- package/template/.agents/skills/diagnosing-bugs/SKILL.md +0 -93
- package/template/.agents/skills/git-workflow-and-versioning/SKILL.md +0 -300
- package/template/.agents/skills/human-review-decisions/SKILL.md +0 -74
- package/template/.agents/skills/idea-refine/SKILL.md +0 -178
- package/template/.agents/skills/idea-refine/examples.md +0 -238
- package/template/.agents/skills/idea-refine/frameworks.md +0 -99
- package/template/.agents/skills/idea-refine/refinement-criteria.md +0 -113
- package/template/.agents/skills/idea-refine/scripts/idea-refine.sh +0 -15
- package/template/.agents/skills/incremental-implementation/SKILL.md +0 -248
- package/template/.agents/skills/insight-capture/SKILL.md +0 -84
- package/template/.agents/skills/intelligent-routing/SKILL.md +0 -176
- package/template/.agents/skills/md2html/SKILL.md +0 -154
- package/template/.agents/skills/md2html/components.md +0 -505
- package/template/.agents/skills/md2html/template.html +0 -1152
- package/template/.agents/skills/planning-and-task-breakdown/SKILL.md +0 -239
- package/template/.agents/skills/pr-review/SKILL.md +0 -143
- package/template/.agents/skills/pr-review-analysis/SKILL.md +0 -89
- package/template/.agents/skills/preview-local-check/SKILL.md +0 -59
- package/template/.agents/skills/release-git-operations/SKILL.md +0 -97
- package/template/.agents/skills/review-followup-routing/SKILL.md +0 -98
- package/template/.agents/skills/security-and-hardening/SKILL.md +0 -349
- package/template/.agents/skills/security-and-hardening/security-checklist.md +0 -134
- package/template/.agents/skills/shipping-and-launch/SKILL.md +0 -311
- package/template/.agents/skills/silent-failure-audit/SKILL.md +0 -73
- package/template/.agents/skills/spec-orchestration/SKILL.md +0 -91
- package/template/.agents/skills/specialist-agent-routing/SKILL.md +0 -107
- package/template/.agents/skills/test-driven-development/SKILL.md +0 -422
- package/template/.agents/skills/test-driven-development/testing-patterns.md +0 -236
- package/template/.agents/skills/test-execution-and-coverage/SKILL.md +0 -56
- package/template/.agents/skills/using-agent-skills/SKILL.md +0 -171
- package/template/.agents/skills/verification-orchestration/SKILL.md +0 -68
- package/template/.agents/skills/vulnerability-scanner/SKILL.md +0 -276
- package/template/.agents/skills/vulnerability-scanner/checklists.md +0 -121
- package/template/.agents/skills/vulnerability-scanner/scripts/security_scan.py +0 -458
- package/template/.agents/skills/wiki/SKILL.md +0 -149
- package/template/.agents/skills/workflow-documentation-sync/SKILL.md +0 -87
- package/template/.claude/skills/9arm-skills/README.md +0 -51
- package/template/.claude/skills/9arm-skills/debug-mantra/SKILL.md +0 -86
- package/template/.claude/skills/9arm-skills/management-talk/SKILL.md +0 -79
- package/template/.claude/skills/9arm-skills/post-mortem/SKILL.md +0 -71
- package/template/.claude/skills/9arm-skills/scrutinize/SKILL.md +0 -72
- package/template/.claude/skills/browser-testing-with-devtools/SKILL.md +0 -302
- package/template/.claude/skills/ci-cd-and-automation/SKILL.md +0 -390
- package/template/.claude/skills/code-review-and-quality/SKILL.md +0 -392
- package/template/.claude/skills/code-simplification/SKILL.md +0 -331
- package/template/.claude/skills/debugging-and-error-recovery/SKILL.md +0 -298
- package/template/.claude/skills/deployment-procedures/SKILL.md +0 -241
- package/template/.claude/skills/deprecation-and-migration/SKILL.md +0 -206
- package/template/.claude/skills/diagnosing-bugs/SKILL.md +0 -93
- package/template/.claude/skills/git-workflow-and-versioning/SKILL.md +0 -300
- package/template/.claude/skills/human-review-decisions/SKILL.md +0 -74
- package/template/.claude/skills/idea-refine/SKILL.md +0 -178
- package/template/.claude/skills/idea-refine/examples.md +0 -238
- package/template/.claude/skills/idea-refine/frameworks.md +0 -99
- package/template/.claude/skills/idea-refine/refinement-criteria.md +0 -113
- package/template/.claude/skills/idea-refine/scripts/idea-refine.sh +0 -15
- package/template/.claude/skills/incremental-implementation/SKILL.md +0 -248
- package/template/.claude/skills/insight-capture/SKILL.md +0 -84
- package/template/.claude/skills/intelligent-routing/SKILL.md +0 -176
- package/template/.claude/skills/md2html/SKILL.md +0 -154
- package/template/.claude/skills/md2html/components.md +0 -505
- package/template/.claude/skills/md2html/template.html +0 -1152
- package/template/.claude/skills/planning-and-task-breakdown/SKILL.md +0 -239
- package/template/.claude/skills/pr-review/SKILL.md +0 -143
- package/template/.claude/skills/pr-review-analysis/SKILL.md +0 -89
- package/template/.claude/skills/preview-local-check/SKILL.md +0 -59
- package/template/.claude/skills/release-git-operations/SKILL.md +0 -97
- package/template/.claude/skills/review-followup-routing/SKILL.md +0 -98
- package/template/.claude/skills/security-and-hardening/SKILL.md +0 -349
- package/template/.claude/skills/security-and-hardening/security-checklist.md +0 -134
- package/template/.claude/skills/shipping-and-launch/SKILL.md +0 -311
- package/template/.claude/skills/silent-failure-audit/SKILL.md +0 -73
- package/template/.claude/skills/spec-orchestration/SKILL.md +0 -91
- package/template/.claude/skills/specialist-agent-routing/SKILL.md +0 -107
- package/template/.claude/skills/test-driven-development/SKILL.md +0 -422
- package/template/.claude/skills/test-driven-development/testing-patterns.md +0 -236
- package/template/.claude/skills/test-execution-and-coverage/SKILL.md +0 -56
- package/template/.claude/skills/using-agent-skills/SKILL.md +0 -171
- package/template/.claude/skills/verification-orchestration/SKILL.md +0 -68
- package/template/.claude/skills/vulnerability-scanner/SKILL.md +0 -276
- package/template/.claude/skills/vulnerability-scanner/checklists.md +0 -121
- package/template/.claude/skills/vulnerability-scanner/scripts/security_scan.py +0 -458
- package/template/.claude/skills/wiki/SKILL.md +0 -149
- package/template/.claude/skills/workflow-documentation-sync/SKILL.md +0 -87
|
@@ -1,239 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: planning-and-task-breakdown
|
|
3
|
-
description: "[Devflow] Breaks work into ordered tasks. Use when you have a spec or clear requirements and need to break work into implementable tasks. Use when a task feels too large to start, when you need to estimate scope, or when parallel work is possible."
|
|
4
|
-
---
|
|
5
|
-
|
|
6
|
-
# Planning and Task Breakdown
|
|
7
|
-
|
|
8
|
-
## Overview
|
|
9
|
-
|
|
10
|
-
Decompose work into small, verifiable tasks with explicit acceptance criteria. Good task breakdown is the difference between an agent that completes work reliably and one that produces a tangled mess. Every task should be small enough to implement, test, and verify in a single focused session.
|
|
11
|
-
|
|
12
|
-
## Task Breakdown Principles
|
|
13
|
-
|
|
14
|
-
1. **Small, Focused Tasks**: Each task should take 2-5 minutes of model effort. One clear outcome per task.
|
|
15
|
-
2. **Clear Verification**: Every task must have a "Verify:" section.
|
|
16
|
-
3. **Logical Ordering**: Dependencies must be identified. Critical path first.
|
|
17
|
-
4. **Principle: Keep It SHORT**: Max 5-10 clear tasks. If more, break into multiple plans.
|
|
18
|
-
|
|
19
|
-
## When to Use
|
|
20
|
-
|
|
21
|
-
- You have a spec and need to break it into implementable units
|
|
22
|
-
- A task feels too large or vague to start
|
|
23
|
-
- Work needs to be parallelized across multiple agents or sessions
|
|
24
|
-
- You need to communicate scope to a human
|
|
25
|
-
- The implementation order isn't obvious
|
|
26
|
-
|
|
27
|
-
In DevFlow 2.0, this skill primarily supports `30-plan`.
|
|
28
|
-
|
|
29
|
-
**When NOT to use:** Single-file changes with obvious scope, or when the spec already contains well-defined tasks.
|
|
30
|
-
|
|
31
|
-
## The Planning Process
|
|
32
|
-
|
|
33
|
-
### Step 1: Enter Plan Mode
|
|
34
|
-
|
|
35
|
-
Before writing any code, operate in read-only mode:
|
|
36
|
-
|
|
37
|
-
- Read the spec and relevant codebase sections
|
|
38
|
-
- Identify existing patterns and conventions
|
|
39
|
-
- Map dependencies between components
|
|
40
|
-
- Note risks and unknowns
|
|
41
|
-
|
|
42
|
-
**Do NOT write code during planning.** The output is a plan document, not implementation.
|
|
43
|
-
|
|
44
|
-
### Step 2: Identify the Dependency Graph
|
|
45
|
-
|
|
46
|
-
Map what depends on what:
|
|
47
|
-
|
|
48
|
-
```
|
|
49
|
-
Database schema
|
|
50
|
-
│
|
|
51
|
-
├── API models/types
|
|
52
|
-
│ │
|
|
53
|
-
│ ├── API endpoints
|
|
54
|
-
│ │ │
|
|
55
|
-
│ │ └── Frontend API client
|
|
56
|
-
│ │ │
|
|
57
|
-
│ │ └── UI components
|
|
58
|
-
│ │
|
|
59
|
-
│ └── Validation logic
|
|
60
|
-
│
|
|
61
|
-
└── Seed data / migrations
|
|
62
|
-
```
|
|
63
|
-
|
|
64
|
-
Implementation order follows the dependency graph bottom-up: build foundations first.
|
|
65
|
-
|
|
66
|
-
### Step 3: Slice Vertically
|
|
67
|
-
|
|
68
|
-
Instead of building all the database, then all the API, then all the UI — build one complete feature path at a time:
|
|
69
|
-
|
|
70
|
-
**Bad (horizontal slicing):**
|
|
71
|
-
```
|
|
72
|
-
Task 1: Build entire database schema
|
|
73
|
-
Task 2: Build all API endpoints
|
|
74
|
-
Task 3: Build all UI components
|
|
75
|
-
Task 4: Connect everything
|
|
76
|
-
```
|
|
77
|
-
|
|
78
|
-
**Good (vertical slicing):**
|
|
79
|
-
```
|
|
80
|
-
Task 1: User can create an account (schema + API + UI for registration)
|
|
81
|
-
Task 2: User can log in (auth schema + API + UI for login)
|
|
82
|
-
Task 3: User can create a task (task schema + API + UI for creation)
|
|
83
|
-
Task 4: User can view task list (query + API + UI for list view)
|
|
84
|
-
```
|
|
85
|
-
|
|
86
|
-
Each vertical slice delivers working, testable functionality.
|
|
87
|
-
|
|
88
|
-
### Step 4: Write Tasks
|
|
89
|
-
|
|
90
|
-
Each task follows this structure:
|
|
91
|
-
|
|
92
|
-
```markdown
|
|
93
|
-
## Task [N]: [Short descriptive title]
|
|
94
|
-
|
|
95
|
-
**Description:** One paragraph explaining what this task accomplishes.
|
|
96
|
-
|
|
97
|
-
**Acceptance criteria:**
|
|
98
|
-
- [ ] [Specific, testable condition]
|
|
99
|
-
- [ ] [Specific, testable condition]
|
|
100
|
-
|
|
101
|
-
**Verification:**
|
|
102
|
-
- [ ] Tests pass: `npm test -- --grep "feature-name"`
|
|
103
|
-
- [ ] Build succeeds: `npm run build`
|
|
104
|
-
- [ ] Manual check: [description of what to verify]
|
|
105
|
-
|
|
106
|
-
**Dependencies:** [Task numbers this depends on, or "None"]
|
|
107
|
-
|
|
108
|
-
**Files likely touched:**
|
|
109
|
-
- `src/path/to/file.ts`
|
|
110
|
-
- `tests/path/to/test.ts`
|
|
111
|
-
|
|
112
|
-
**Estimated scope:** [Small: 1-2 files | Medium: 3-5 files | Large: 5+ files]
|
|
113
|
-
```
|
|
114
|
-
|
|
115
|
-
### Step 5: Order and Checkpoint
|
|
116
|
-
|
|
117
|
-
Arrange tasks so that:
|
|
118
|
-
|
|
119
|
-
1. Dependencies are satisfied (build foundation first)
|
|
120
|
-
2. Each task leaves the system in a working state
|
|
121
|
-
3. Verification checkpoints occur after every 2-3 tasks
|
|
122
|
-
4. High-risk tasks are early (fail fast)
|
|
123
|
-
|
|
124
|
-
Add explicit checkpoints:
|
|
125
|
-
|
|
126
|
-
```markdown
|
|
127
|
-
## Checkpoint: After Tasks 1-3
|
|
128
|
-
- [ ] All tests pass
|
|
129
|
-
- [ ] Application builds without errors
|
|
130
|
-
- [ ] Core user flow works end-to-end
|
|
131
|
-
- [ ] Review with human before proceeding
|
|
132
|
-
```
|
|
133
|
-
|
|
134
|
-
## Task Sizing Guidelines
|
|
135
|
-
|
|
136
|
-
| Size | Files | Scope | Example |
|
|
137
|
-
|------|-------|-------|---------|
|
|
138
|
-
| **XS** | 1 | Single function or config change | Add a validation rule |
|
|
139
|
-
| **S** | 1-2 | One component or endpoint | Add a new API endpoint |
|
|
140
|
-
| **M** | 3-5 | One feature slice | User registration flow |
|
|
141
|
-
| **L** | 5-8 | Multi-component feature | Search with filtering and pagination |
|
|
142
|
-
| **XL** | 8+ | **Too large — break it down further** | — |
|
|
143
|
-
|
|
144
|
-
If a task is L or larger, it should be broken into smaller tasks. An agent performs best on S and M tasks.
|
|
145
|
-
|
|
146
|
-
**When to break a task down further:**
|
|
147
|
-
- It would take more than one focused session (roughly 2+ hours of agent work)
|
|
148
|
-
- You cannot describe the acceptance criteria in 3 or fewer bullet points
|
|
149
|
-
- It touches two or more independent subsystems (e.g., auth and billing)
|
|
150
|
-
- You find yourself writing "and" in the task title (a sign it is two tasks)
|
|
151
|
-
|
|
152
|
-
## DevFlow 2.0 Stage Contract
|
|
153
|
-
|
|
154
|
-
In DevFlow 2.0, every plan must remain understandable from markdown alone inside `devflow/runs/{ID}-*/`.
|
|
155
|
-
|
|
156
|
-
1. **`plan.md`**: The primary planning artifact following the template below.
|
|
157
|
-
2. The markdown plan must still work even if no dashboard or JSON reader is available.
|
|
158
|
-
|
|
159
|
-
## Plan Document Template
|
|
160
|
-
|
|
161
|
-
```markdown
|
|
162
|
-
# Implementation Plan: [Feature/Project Name]
|
|
163
|
-
|
|
164
|
-
## Overview
|
|
165
|
-
[One paragraph summary of what we're building]
|
|
166
|
-
|
|
167
|
-
## Architecture Decisions
|
|
168
|
-
- [Key decision 1 and rationale]
|
|
169
|
-
- [Key decision 2 and rationale]
|
|
170
|
-
|
|
171
|
-
## Task List
|
|
172
|
-
|
|
173
|
-
### Phase 1: Foundation
|
|
174
|
-
- [ ] Task 1: ...
|
|
175
|
-
- [ ] Task 2: ...
|
|
176
|
-
|
|
177
|
-
### Checkpoint: Foundation
|
|
178
|
-
- [ ] Tests pass, builds clean
|
|
179
|
-
|
|
180
|
-
### Phase 2: Core Features
|
|
181
|
-
- [ ] Task 3: ...
|
|
182
|
-
- [ ] Task 4: ...
|
|
183
|
-
|
|
184
|
-
### Checkpoint: Core Features
|
|
185
|
-
- [ ] End-to-end flow works
|
|
186
|
-
|
|
187
|
-
### Phase 3: Polish
|
|
188
|
-
- [ ] Task 5: ...
|
|
189
|
-
- [ ] Task 6: ...
|
|
190
|
-
|
|
191
|
-
### Checkpoint: Complete
|
|
192
|
-
- [ ] All acceptance criteria met
|
|
193
|
-
- [ ] Ready for review
|
|
194
|
-
|
|
195
|
-
## Risks and Mitigations
|
|
196
|
-
| Risk | Impact | Mitigation |
|
|
197
|
-
|------|--------|------------|
|
|
198
|
-
| [Risk] | [High/Med/Low] | [Strategy] |
|
|
199
|
-
|
|
200
|
-
## Open Questions
|
|
201
|
-
- [Question needing human input]
|
|
202
|
-
```
|
|
203
|
-
|
|
204
|
-
## Parallelization Opportunities
|
|
205
|
-
|
|
206
|
-
When multiple agents or sessions are available:
|
|
207
|
-
|
|
208
|
-
- **Safe to parallelize:** Independent feature slices, tests for already-implemented features, documentation
|
|
209
|
-
- **Must be sequential:** Database migrations, shared state changes, dependency chains
|
|
210
|
-
- **Needs coordination:** Features that share an API contract (define the contract first, then parallelize)
|
|
211
|
-
|
|
212
|
-
## Common Rationalizations
|
|
213
|
-
|
|
214
|
-
| Rationalization | Reality |
|
|
215
|
-
|---|---|
|
|
216
|
-
| "I'll figure it out as I go" | That's how you end up with a tangled mess and rework. 10 minutes of planning saves hours. |
|
|
217
|
-
| "The tasks are obvious" | Write them down anyway. Explicit tasks surface hidden dependencies and forgotten edge cases. |
|
|
218
|
-
| "Planning is overhead" | Planning is the task. Implementation without a plan is just typing. |
|
|
219
|
-
| "I can hold it all in my head" | Context windows are finite. Written plans survive session boundaries and compaction. |
|
|
220
|
-
|
|
221
|
-
## Red Flags
|
|
222
|
-
|
|
223
|
-
- Starting implementation without a written task list
|
|
224
|
-
- Tasks that say "implement the feature" without acceptance criteria
|
|
225
|
-
- No verification steps in the plan
|
|
226
|
-
- All tasks are XL-sized
|
|
227
|
-
- No checkpoints between tasks
|
|
228
|
-
- Dependency order isn't considered
|
|
229
|
-
|
|
230
|
-
## Verification
|
|
231
|
-
|
|
232
|
-
Before starting implementation, confirm:
|
|
233
|
-
|
|
234
|
-
- [ ] Every task has acceptance criteria
|
|
235
|
-
- [ ] Every task has a verification step
|
|
236
|
-
- [ ] Task dependencies are identified and ordered correctly
|
|
237
|
-
- [ ] No task touches more than ~5 files
|
|
238
|
-
- [ ] Checkpoints exist between major phases
|
|
239
|
-
- [ ] The human has reviewed and approved the plan
|
|
@@ -1,143 +0,0 @@
|
|
|
1
|
-
---name: pr-review
|
|
2
|
-
|
|
3
|
-
description: "[Devflow] PR Review Addon - Review a pull request using GitHub review prompt patterns and the credited 9arm-skills/scrutinize discipline."
|
|
4
|
-
---
|
|
5
|
-
|
|
6
|
-
# Phase 55: PR Review Addon
|
|
7
|
-
|
|
8
|
-
## Target: $ARGUMENTS
|
|
9
|
-
|
|
10
|
-
Use this workflow when you need a structured PR review without creating or changing the PR. It can review a local diff, a PR URL or number, or a task-linked branch.
|
|
11
|
-
|
|
12
|
-
In DevFlow 2.0, this remains a supporting review workflow. It should feed corrective work back into `40-implement` or clear the path toward `60-report` and then `70-release`.
|
|
13
|
-
|
|
14
|
-
Primary behavior now lives in the `pr-review-analysis` skill. Keep this workflow as the compatibility wrapper and user-facing review prompt surface.
|
|
15
|
-
|
|
16
|
-
## Prompt Sources
|
|
17
|
-
|
|
18
|
-
Adapted from:
|
|
19
|
-
|
|
20
|
-
- `github/QA_REVIEW_SYSTEM_PROMPT.md`
|
|
21
|
-
- `github/pr_reviewer.md`
|
|
22
|
-
- `github/pr_parallel_orchestrator.md`
|
|
23
|
-
- `github/pr_orchestrator.md`
|
|
24
|
-
- `github/pr_quality_agent.md`
|
|
25
|
-
- `github/pr_logic_agent.md`
|
|
26
|
-
- `github/pr_security_agent.md`
|
|
27
|
-
- `github/pr_structural.md`
|
|
28
|
-
- `github/pr_codebase_fit_agent.md`
|
|
29
|
-
- `github/pr_finding_validator.md`
|
|
30
|
-
- `github/partials/full_context_analysis.md`
|
|
31
|
-
|
|
32
|
-
Additional credited discipline:
|
|
33
|
-
|
|
34
|
-
- `.agents/skills/9arm-skills/scrutinize/SKILL.md`
|
|
35
|
-
- Source pack: `9arm-skills`
|
|
36
|
-
- Credit: `thananon/9arm-skills`
|
|
37
|
-
- Upstream: https://github.com/thananon/9arm-skills
|
|
38
|
-
- Adapted for: Antigravity IDE / Nexus-DevFlow
|
|
39
|
-
|
|
40
|
-
## Process
|
|
41
|
-
|
|
42
|
-
### 1. Load Review Context
|
|
43
|
-
|
|
44
|
-
Read the available context:
|
|
45
|
-
|
|
46
|
-
- PR diff or local `git diff`
|
|
47
|
-
- changed filenames
|
|
48
|
-
- linked stage artifacts under `devflow/runs/{ID}-*/`
|
|
49
|
-
- `20-spec.md`, `30-plan.md`, `40-implement.md`, `50-verify.md`, `60-report.md`, and `70-release.md` when relevant
|
|
50
|
-
- project patterns and existing code around the changed files
|
|
51
|
-
- legacy JSON only when migration context is still genuinely relevant
|
|
52
|
-
|
|
53
|
-
### 2. Select Review Lanes
|
|
54
|
-
|
|
55
|
-
Start with the `scrutinize` lens before detailed lane review:
|
|
56
|
-
|
|
57
|
-
- Intent check: should this change exist in this form?
|
|
58
|
-
- Smaller alternative: can the same outcome be reached with less code, lower risk, or existing patterns?
|
|
59
|
-
- Actual path trace: follow runtime behavior beyond the diff.
|
|
60
|
-
- Precision check: were assumptions explicit, edits surgical, success criteria verified, and uncertainty surfaced instead of guessed?
|
|
61
|
-
|
|
62
|
-
Then select only the review lanes that match the scope. Do not run every lane by default; the review should be proportional to the change.
|
|
63
|
-
|
|
64
|
-
| Lane | Use When |
|
|
65
|
-
| :--- | :--- |
|
|
66
|
-
| Correctness / bugs | Always, unless the user explicitly asks for a narrow non-code review. |
|
|
67
|
-
| Type safety | Typed code, public types, schema changes, casts, `any`, or boundary parsing changed. |
|
|
68
|
-
| Maintainability | The change adds duplication, cross-file patterns, API shape, migration bridges, or technical debt. |
|
|
69
|
-
| Simplicity | The change adds abstractions, indirection, configuration, generalized utilities, or complex control flow. |
|
|
70
|
-
| Testability | Important logic is mixed with IO, hidden state, time, random values, constructors, globals, or hard-coded dependencies. |
|
|
71
|
-
| Coverage | New or changed behavior should have tests, especially bug fixes and business logic. |
|
|
72
|
-
| Docs accuracy | Commands, setup, workflows, schemas, public APIs, examples, or user-visible behavior changed. |
|
|
73
|
-
| AGENTS.md adherence | An `AGENTS.md` or equivalent project instruction file exists for the changed scope. |
|
|
74
|
-
| Security | Auth, authorization, secrets, user input, external data, dependencies, or persistence boundaries changed. |
|
|
75
|
-
| Performance | Queries, loops, rendering, data volume, hot paths, or synchronous work changed. |
|
|
76
|
-
|
|
77
|
-
When a lane is skipped, briefly note why if the omission could surprise the reader. For tiny reviews, it is acceptable to state only the lanes used.
|
|
78
|
-
|
|
79
|
-
### 3. Validate Findings
|
|
80
|
-
|
|
81
|
-
Before reporting a finding:
|
|
82
|
-
|
|
83
|
-
- verify it against the changed code
|
|
84
|
-
- check whether it affects new code or pre-existing code
|
|
85
|
-
- ensure it is actionable
|
|
86
|
-
- include file and line references when possible
|
|
87
|
-
- avoid style-only comments unless they hide real risk
|
|
88
|
-
- keep each finding in its strongest matching lane; do not report the same issue twice under different labels
|
|
89
|
-
- drop findings that are only suspicions, preferences, or theoretical improvements without a concrete failure mode
|
|
90
|
-
- for AGENTS.md findings, quote the exact project instruction being violated
|
|
91
|
-
- for docs findings, show the mismatch between what the docs say and what the code now does
|
|
92
|
-
|
|
93
|
-
### 4. Output Findings First
|
|
94
|
-
|
|
95
|
-
Use code-review style:
|
|
96
|
-
|
|
97
|
-
1. Findings ordered by severity
|
|
98
|
-
2. Open questions or assumptions
|
|
99
|
-
3. Short summary
|
|
100
|
-
4. Test gaps or residual risk
|
|
101
|
-
|
|
102
|
-
When saving a substantial report, include a short `Source Discipline` section crediting `9arm-skills/scrutinize` and, where useful, brief `Intent Check`, `Smaller Alternative Considered`, and `Actual Path Trace` sections. Preserve the required PR review template headings.
|
|
103
|
-
|
|
104
|
-
## Output
|
|
105
|
-
|
|
106
|
-
Return a PR review. Do not modify files unless the user explicitly asks for fixes.
|
|
107
|
-
|
|
108
|
-
**MANDATORY RULE:** Before saving a PR review report, inspect `.agent/resources/schemas/pr_review.template.md` and use its required headings and table structure.
|
|
109
|
-
|
|
110
|
-
Before reporting completion, run:
|
|
111
|
-
|
|
112
|
-
Review `{report_path}` against `pr_review.template.md`, keep the required headings, and remove placeholder text before completion.
|
|
113
|
-
|
|
114
|
-
Replace any placeholder or template text with concrete findings, file references, risk, and verification gaps.
|
|
115
|
-
|
|
116
|
-
**MANDATORY RULE:** If a running ID is linked to this PR, always save this review report to `pr_review.md` inside that workspace directory when possible:
|
|
117
|
-
|
|
118
|
-
```text
|
|
119
|
-
devflow/runs/{ID}-*/pr_review.md
|
|
120
|
-
```
|
|
121
|
-
|
|
122
|
-
## Relationship To DevFlow 2.0
|
|
123
|
-
|
|
124
|
-
- Classification: Companion command
|
|
125
|
-
- Mainline status: Verification and release support command, not a numbered stage
|
|
126
|
-
- Typical entry points: `50-verify`, `PR`, `Agent code-reviewer`
|
|
127
|
-
- Typical handoff targets: `PR-Followup`, `60-report`, `70-release`, `Wiki`
|
|
128
|
-
|
|
129
|
-
## Sources
|
|
130
|
-
|
|
131
|
-
- `AGENTS.md`
|
|
132
|
-
- `.agent/resources/schemas/pr_review.template.md`
|
|
133
|
-
- Related commands: `50-verify`, `PR`, `PR-Followup`, `Agent`, `60-report`, `70-release`, `Wiki`
|
|
134
|
-
|
|
135
|
-
## Next Workflow Recommendation
|
|
136
|
-
|
|
137
|
-
- **Primary**: `40-implement {ID}` when review finds required fixes, or `60-report {ID}` when the review is clean and the run needs the final summary refreshed before release execution.
|
|
138
|
-
- **Why**: PR review either creates corrective implementation work or clears the path toward final summary alignment and release-facing packaging.
|
|
139
|
-
- **Alternatives**:
|
|
140
|
-
- `70-release` when the report is already aligned and only release execution remains
|
|
141
|
-
- `Wiki` when the review establishes a durable convention or risk pattern
|
|
142
|
-
- `PR-Followup {target}` when the PR already has comments that need classification and response
|
|
143
|
-
|
|
@@ -1,89 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: pr-review-analysis
|
|
3
|
-
description: "[Devflow] Review pull requests or local diffs with structured finding discipline, selective review lanes, and Nexus-DevFlow report templates. Use when the team needs a substantial review without turning review into a numbered workflow stage."
|
|
4
|
-
---
|
|
5
|
-
|
|
6
|
-
# PR Review Analysis
|
|
7
|
-
|
|
8
|
-
## Overview
|
|
9
|
-
|
|
10
|
-
This skill is the shared behavior layer behind `PR-Review`.
|
|
11
|
-
|
|
12
|
-
It preserves the stronger PR review UX from the legacy workflow while fitting DevFlow 2.0 as a non-mainline support capability.
|
|
13
|
-
|
|
14
|
-
## Related Foundation Skills
|
|
15
|
-
|
|
16
|
-
This skill should reuse and align with:
|
|
17
|
-
|
|
18
|
-
- `.agents/skills/9arm-skills/scrutinize/SKILL.md`
|
|
19
|
-
- `.agent/resources/schemas/pr_review.template.md`
|
|
20
|
-
|
|
21
|
-
## When to Use
|
|
22
|
-
|
|
23
|
-
- after `50-verify` when the change needs structured review before `60-report` and `70-release`
|
|
24
|
-
- when a local diff, branch, or pull request needs findings-first analysis
|
|
25
|
-
- when the user wants review without directly editing code
|
|
26
|
-
|
|
27
|
-
Do not use this skill as a substitute for implementation, debugging, or formal release packaging.
|
|
28
|
-
|
|
29
|
-
## Process
|
|
30
|
-
|
|
31
|
-
### 1. Load Review Context
|
|
32
|
-
|
|
33
|
-
Read the minimum relevant context:
|
|
34
|
-
|
|
35
|
-
- PR diff or local `git diff`
|
|
36
|
-
- changed files
|
|
37
|
-
- linked stage artifacts under the running workspace when available
|
|
38
|
-
- nearby project patterns and affected runtime paths
|
|
39
|
-
|
|
40
|
-
### 2. Apply Scrutiny First
|
|
41
|
-
|
|
42
|
-
Start with the `scrutinize` discipline:
|
|
43
|
-
|
|
44
|
-
- intent check
|
|
45
|
-
- smaller or safer alternative
|
|
46
|
-
- actual runtime path trace
|
|
47
|
-
- precision and evidence check
|
|
48
|
-
|
|
49
|
-
### 3. Select Review Lanes
|
|
50
|
-
|
|
51
|
-
Use only the lanes that match scope:
|
|
52
|
-
|
|
53
|
-
- correctness
|
|
54
|
-
- maintainability
|
|
55
|
-
- simplicity
|
|
56
|
-
- testability and coverage
|
|
57
|
-
- docs accuracy
|
|
58
|
-
- AGENTS or instruction adherence
|
|
59
|
-
- security
|
|
60
|
-
- performance
|
|
61
|
-
|
|
62
|
-
### 4. Validate Findings
|
|
63
|
-
|
|
64
|
-
Before reporting a finding:
|
|
65
|
-
|
|
66
|
-
- verify it against the changed code
|
|
67
|
-
- confirm whether it is new or pre-existing
|
|
68
|
-
- keep it actionable
|
|
69
|
-
- include file and line references whenever possible
|
|
70
|
-
|
|
71
|
-
### 5. Save Reusable Report
|
|
72
|
-
|
|
73
|
-
When saving a substantial report, preserve:
|
|
74
|
-
|
|
75
|
-
```text
|
|
76
|
-
devflow/runs/{ID}-*/pr_review.md
|
|
77
|
-
```
|
|
78
|
-
|
|
79
|
-
Use `.agent/resources/schemas/pr_review.template.md` and replace all placeholder text with real findings, questions, risks, and evidence gaps.
|
|
80
|
-
|
|
81
|
-
## Output
|
|
82
|
-
|
|
83
|
-
Return:
|
|
84
|
-
|
|
85
|
-
- findings ordered by severity
|
|
86
|
-
- open questions or assumptions
|
|
87
|
-
- residual risks or test gaps
|
|
88
|
-
- where the review report was saved
|
|
89
|
-
- the recommended next route such as `40-implement`, `70-release`, or `PR-Followup`
|
|
@@ -1,59 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: preview-local-check
|
|
3
|
-
description: "[Devflow] Manage local preview, smoke-check, and temporary runtime verification before formal verification. Use when implementation needs a quick visual or runtime check, when a reviewer needs a local URL, or when a change should be inspected before 50-verify."
|
|
4
|
-
---
|
|
5
|
-
|
|
6
|
-
# Preview Local Check
|
|
7
|
-
|
|
8
|
-
## Overview
|
|
9
|
-
|
|
10
|
-
This skill handles preview management and lightweight runtime checks without pretending that preview success is the same as full verification success.
|
|
11
|
-
|
|
12
|
-
Use it to:
|
|
13
|
-
|
|
14
|
-
- start or stop a local preview server
|
|
15
|
-
- report preview health and URL
|
|
16
|
-
- run a quick smoke check before formal verification
|
|
17
|
-
- give humans a concrete local artifact to inspect
|
|
18
|
-
|
|
19
|
-
## When to Use
|
|
20
|
-
|
|
21
|
-
- During `40-implement` when a quick runtime check is useful
|
|
22
|
-
- During `50-verify` when the change needs visual or interactive confirmation
|
|
23
|
-
- When a reviewer asks for a local preview URL
|
|
24
|
-
|
|
25
|
-
Do not use this skill as a replacement for test evidence, verification evidence, or release readiness.
|
|
26
|
-
|
|
27
|
-
## Process
|
|
28
|
-
|
|
29
|
-
### 1. Detect Preview Path
|
|
30
|
-
|
|
31
|
-
- identify the correct project-local preview command
|
|
32
|
-
- detect the likely app type and default port
|
|
33
|
-
- separate preview management from deployment logic
|
|
34
|
-
|
|
35
|
-
### 2. Manage State
|
|
36
|
-
|
|
37
|
-
- support `status`, `start`, `stop`, `restart`, and `check`
|
|
38
|
-
- report port conflicts clearly
|
|
39
|
-
- offer a safe alternate port when possible
|
|
40
|
-
|
|
41
|
-
### 3. Report Health
|
|
42
|
-
|
|
43
|
-
Always report:
|
|
44
|
-
|
|
45
|
-
- URL
|
|
46
|
-
- project path
|
|
47
|
-
- app type
|
|
48
|
-
- health status
|
|
49
|
-
- blockers or warnings
|
|
50
|
-
|
|
51
|
-
### 4. Route Back
|
|
52
|
-
|
|
53
|
-
- return to `50-verify` when preview confirms the change is ready for formal checks
|
|
54
|
-
- return to `40-implement` when preview reveals implementation issues
|
|
55
|
-
- use `Debug` when the preview failure needs RCA
|
|
56
|
-
|
|
57
|
-
## Output
|
|
58
|
-
|
|
59
|
-
Return a concise preview status or action summary. When useful, save notes into the current stage artifact rather than inventing a separate contract.
|
|
@@ -1,97 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: release-git-operations
|
|
3
|
-
description: "[Devflow] Handle commit, pull request, merge, deployment, and changelog work around Release. Use when packaging verified work for sharing, integration, deployment, or release communication."
|
|
4
|
-
---
|
|
5
|
-
|
|
6
|
-
# Release Git Operations
|
|
7
|
-
|
|
8
|
-
## Overview
|
|
9
|
-
|
|
10
|
-
This skill groups release-adjacent git and deployment behavior that used to be spread across multiple companion workflows.
|
|
11
|
-
|
|
12
|
-
It is the shared behavior layer behind:
|
|
13
|
-
|
|
14
|
-
- `Commit`
|
|
15
|
-
- `PR`
|
|
16
|
-
- `Merge`
|
|
17
|
-
- `Deploy`
|
|
18
|
-
- `Changelog`
|
|
19
|
-
|
|
20
|
-
## Related Foundation Skills
|
|
21
|
-
|
|
22
|
-
This skill should reuse and align with:
|
|
23
|
-
|
|
24
|
-
- `.agents/skills/git-workflow-and-versioning/SKILL.md`
|
|
25
|
-
- `.agents/skills/shipping-and-launch/SKILL.md`
|
|
26
|
-
- `.agents/skills/deployment-procedures/SKILL.md`
|
|
27
|
-
|
|
28
|
-
## When to Use
|
|
29
|
-
|
|
30
|
-
- After `50-verify` when verified work is ready for packaging
|
|
31
|
-
- During `70-release` when the team needs commit, PR, merge, deployment, or changelog support after the report is already aligned
|
|
32
|
-
- When release communication must be built from markdown-first stage artifacts
|
|
33
|
-
|
|
34
|
-
## Supported Modes
|
|
35
|
-
|
|
36
|
-
- `commit`
|
|
37
|
-
- `pr`
|
|
38
|
-
- `merge`
|
|
39
|
-
- `deploy`
|
|
40
|
-
- `changelog`
|
|
41
|
-
|
|
42
|
-
## Process
|
|
43
|
-
|
|
44
|
-
### 1. Confirm Release Context
|
|
45
|
-
|
|
46
|
-
Read the minimum required artifacts:
|
|
47
|
-
|
|
48
|
-
- `verify.md`
|
|
49
|
-
- `release.md`
|
|
50
|
-
- `report.md` when relevant
|
|
51
|
-
- git state and repository policy when relevant
|
|
52
|
-
|
|
53
|
-
### 2. Run The Appropriate Mode
|
|
54
|
-
|
|
55
|
-
#### Commit
|
|
56
|
-
|
|
57
|
-
- stage the intended scope
|
|
58
|
-
- use disciplined commit messaging
|
|
59
|
-
- preserve branch safety rules
|
|
60
|
-
|
|
61
|
-
#### PR
|
|
62
|
-
|
|
63
|
-
- build PR narrative from markdown-first artifacts
|
|
64
|
-
- preserve repository template fidelity
|
|
65
|
-
- route follow-up review work cleanly
|
|
66
|
-
|
|
67
|
-
#### Merge
|
|
68
|
-
|
|
69
|
-
- confirm verification and review gates are satisfied
|
|
70
|
-
- integrate safely using repository policy
|
|
71
|
-
- route conflicts back to implementation or follow-up
|
|
72
|
-
|
|
73
|
-
#### Deploy
|
|
74
|
-
|
|
75
|
-
- apply shipping and launch discipline
|
|
76
|
-
- verify platform-specific readiness
|
|
77
|
-
- keep rollback and health checks explicit
|
|
78
|
-
|
|
79
|
-
#### Changelog
|
|
80
|
-
|
|
81
|
-
- summarize user-facing changes from stage artifacts and git evidence
|
|
82
|
-
- avoid leaking low-value internal detail
|
|
83
|
-
|
|
84
|
-
### 3. Route Back
|
|
85
|
-
|
|
86
|
-
- return to `70-release` when packaging is still in progress
|
|
87
|
-
- return to `60-report` when communication is the remaining task
|
|
88
|
-
- return to `40-implement`, `50-verify`, or `PR-Followup` if release blockers are discovered
|
|
89
|
-
|
|
90
|
-
## Output
|
|
91
|
-
|
|
92
|
-
Return:
|
|
93
|
-
|
|
94
|
-
- mode executed
|
|
95
|
-
- artifacts or git actions produced
|
|
96
|
-
- risks, blockers, or follow-up needs
|
|
97
|
-
- recommended next stage or companion command
|