@jakkrichm/create-nexus-devflow 2.0.12 → 2.0.13

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (124) hide show
  1. package/lib/update.js +15 -1
  2. package/package.json +1 -1
  3. package/template/.agents/skills/70-release/SKILL.md +2 -0
  4. package/template/.agents/skills/ci/SKILL.md +25 -78
  5. package/template/.agents/skills/commit/SKILL.md +39 -43
  6. package/template/.agents/skills/debug/SKILL.md +43 -104
  7. package/template/.agents/skills/deploy/SKILL.md +37 -65
  8. package/template/.agents/skills/insight/SKILL.md +27 -116
  9. package/template/.agents/skills/preview/SKILL.md +24 -103
  10. package/template/.agents/skills/review/SKILL.md +53 -37
  11. package/template/.agents/skills/rollback/SKILL.md +1 -0
  12. package/template/.agents/skills/security-review/SKILL.md +44 -147
  13. package/template/.agents/skills/simplify/SKILL.md +48 -57
  14. package/template/.agents/skills/test/SKILL.md +63 -51
  15. package/template/.claude/skills/70-release/SKILL.md +2 -0
  16. package/template/.claude/skills/ci/SKILL.md +25 -78
  17. package/template/.claude/skills/commit/SKILL.md +39 -43
  18. package/template/.claude/skills/debug/SKILL.md +43 -104
  19. package/template/.claude/skills/deploy/SKILL.md +37 -65
  20. package/template/.claude/skills/insight/SKILL.md +27 -116
  21. package/template/.claude/skills/preview/SKILL.md +24 -103
  22. package/template/.claude/skills/review/SKILL.md +53 -37
  23. package/template/.claude/skills/rollback/SKILL.md +1 -0
  24. package/template/.claude/skills/security-review/SKILL.md +44 -147
  25. package/template/.claude/skills/simplify/SKILL.md +48 -57
  26. package/template/.claude/skills/test/SKILL.md +63 -51
  27. package/template/devflow/history/HISTORY.md +27 -0
  28. package/template/devflow/reference/running-id-contract.md +1 -1
  29. package/template/.agents/skills/9arm-skills/README.md +0 -51
  30. package/template/.agents/skills/9arm-skills/debug-mantra/SKILL.md +0 -86
  31. package/template/.agents/skills/9arm-skills/management-talk/SKILL.md +0 -79
  32. package/template/.agents/skills/9arm-skills/post-mortem/SKILL.md +0 -71
  33. package/template/.agents/skills/9arm-skills/scrutinize/SKILL.md +0 -72
  34. package/template/.agents/skills/browser-testing-with-devtools/SKILL.md +0 -302
  35. package/template/.agents/skills/ci-cd-and-automation/SKILL.md +0 -390
  36. package/template/.agents/skills/code-review-and-quality/SKILL.md +0 -392
  37. package/template/.agents/skills/code-simplification/SKILL.md +0 -331
  38. package/template/.agents/skills/debugging-and-error-recovery/SKILL.md +0 -298
  39. package/template/.agents/skills/deployment-procedures/SKILL.md +0 -241
  40. package/template/.agents/skills/deprecation-and-migration/SKILL.md +0 -206
  41. package/template/.agents/skills/diagnosing-bugs/SKILL.md +0 -93
  42. package/template/.agents/skills/git-workflow-and-versioning/SKILL.md +0 -300
  43. package/template/.agents/skills/human-review-decisions/SKILL.md +0 -74
  44. package/template/.agents/skills/idea-refine/SKILL.md +0 -178
  45. package/template/.agents/skills/idea-refine/examples.md +0 -238
  46. package/template/.agents/skills/idea-refine/frameworks.md +0 -99
  47. package/template/.agents/skills/idea-refine/refinement-criteria.md +0 -113
  48. package/template/.agents/skills/idea-refine/scripts/idea-refine.sh +0 -15
  49. package/template/.agents/skills/incremental-implementation/SKILL.md +0 -248
  50. package/template/.agents/skills/insight-capture/SKILL.md +0 -84
  51. package/template/.agents/skills/intelligent-routing/SKILL.md +0 -176
  52. package/template/.agents/skills/md2html/SKILL.md +0 -154
  53. package/template/.agents/skills/md2html/components.md +0 -505
  54. package/template/.agents/skills/md2html/template.html +0 -1152
  55. package/template/.agents/skills/planning-and-task-breakdown/SKILL.md +0 -239
  56. package/template/.agents/skills/pr-review/SKILL.md +0 -143
  57. package/template/.agents/skills/pr-review-analysis/SKILL.md +0 -89
  58. package/template/.agents/skills/preview-local-check/SKILL.md +0 -59
  59. package/template/.agents/skills/release-git-operations/SKILL.md +0 -97
  60. package/template/.agents/skills/review-followup-routing/SKILL.md +0 -98
  61. package/template/.agents/skills/security-and-hardening/SKILL.md +0 -349
  62. package/template/.agents/skills/security-and-hardening/security-checklist.md +0 -134
  63. package/template/.agents/skills/shipping-and-launch/SKILL.md +0 -311
  64. package/template/.agents/skills/silent-failure-audit/SKILL.md +0 -73
  65. package/template/.agents/skills/spec-orchestration/SKILL.md +0 -91
  66. package/template/.agents/skills/specialist-agent-routing/SKILL.md +0 -107
  67. package/template/.agents/skills/test-driven-development/SKILL.md +0 -422
  68. package/template/.agents/skills/test-driven-development/testing-patterns.md +0 -236
  69. package/template/.agents/skills/test-execution-and-coverage/SKILL.md +0 -56
  70. package/template/.agents/skills/using-agent-skills/SKILL.md +0 -171
  71. package/template/.agents/skills/verification-orchestration/SKILL.md +0 -68
  72. package/template/.agents/skills/vulnerability-scanner/SKILL.md +0 -276
  73. package/template/.agents/skills/vulnerability-scanner/checklists.md +0 -121
  74. package/template/.agents/skills/vulnerability-scanner/scripts/security_scan.py +0 -458
  75. package/template/.agents/skills/wiki/SKILL.md +0 -149
  76. package/template/.agents/skills/workflow-documentation-sync/SKILL.md +0 -87
  77. package/template/.claude/skills/9arm-skills/README.md +0 -51
  78. package/template/.claude/skills/9arm-skills/debug-mantra/SKILL.md +0 -86
  79. package/template/.claude/skills/9arm-skills/management-talk/SKILL.md +0 -79
  80. package/template/.claude/skills/9arm-skills/post-mortem/SKILL.md +0 -71
  81. package/template/.claude/skills/9arm-skills/scrutinize/SKILL.md +0 -72
  82. package/template/.claude/skills/browser-testing-with-devtools/SKILL.md +0 -302
  83. package/template/.claude/skills/ci-cd-and-automation/SKILL.md +0 -390
  84. package/template/.claude/skills/code-review-and-quality/SKILL.md +0 -392
  85. package/template/.claude/skills/code-simplification/SKILL.md +0 -331
  86. package/template/.claude/skills/debugging-and-error-recovery/SKILL.md +0 -298
  87. package/template/.claude/skills/deployment-procedures/SKILL.md +0 -241
  88. package/template/.claude/skills/deprecation-and-migration/SKILL.md +0 -206
  89. package/template/.claude/skills/diagnosing-bugs/SKILL.md +0 -93
  90. package/template/.claude/skills/git-workflow-and-versioning/SKILL.md +0 -300
  91. package/template/.claude/skills/human-review-decisions/SKILL.md +0 -74
  92. package/template/.claude/skills/idea-refine/SKILL.md +0 -178
  93. package/template/.claude/skills/idea-refine/examples.md +0 -238
  94. package/template/.claude/skills/idea-refine/frameworks.md +0 -99
  95. package/template/.claude/skills/idea-refine/refinement-criteria.md +0 -113
  96. package/template/.claude/skills/idea-refine/scripts/idea-refine.sh +0 -15
  97. package/template/.claude/skills/incremental-implementation/SKILL.md +0 -248
  98. package/template/.claude/skills/insight-capture/SKILL.md +0 -84
  99. package/template/.claude/skills/intelligent-routing/SKILL.md +0 -176
  100. package/template/.claude/skills/md2html/SKILL.md +0 -154
  101. package/template/.claude/skills/md2html/components.md +0 -505
  102. package/template/.claude/skills/md2html/template.html +0 -1152
  103. package/template/.claude/skills/planning-and-task-breakdown/SKILL.md +0 -239
  104. package/template/.claude/skills/pr-review/SKILL.md +0 -143
  105. package/template/.claude/skills/pr-review-analysis/SKILL.md +0 -89
  106. package/template/.claude/skills/preview-local-check/SKILL.md +0 -59
  107. package/template/.claude/skills/release-git-operations/SKILL.md +0 -97
  108. package/template/.claude/skills/review-followup-routing/SKILL.md +0 -98
  109. package/template/.claude/skills/security-and-hardening/SKILL.md +0 -349
  110. package/template/.claude/skills/security-and-hardening/security-checklist.md +0 -134
  111. package/template/.claude/skills/shipping-and-launch/SKILL.md +0 -311
  112. package/template/.claude/skills/silent-failure-audit/SKILL.md +0 -73
  113. package/template/.claude/skills/spec-orchestration/SKILL.md +0 -91
  114. package/template/.claude/skills/specialist-agent-routing/SKILL.md +0 -107
  115. package/template/.claude/skills/test-driven-development/SKILL.md +0 -422
  116. package/template/.claude/skills/test-driven-development/testing-patterns.md +0 -236
  117. package/template/.claude/skills/test-execution-and-coverage/SKILL.md +0 -56
  118. package/template/.claude/skills/using-agent-skills/SKILL.md +0 -171
  119. package/template/.claude/skills/verification-orchestration/SKILL.md +0 -68
  120. package/template/.claude/skills/vulnerability-scanner/SKILL.md +0 -276
  121. package/template/.claude/skills/vulnerability-scanner/checklists.md +0 -121
  122. package/template/.claude/skills/vulnerability-scanner/scripts/security_scan.py +0 -458
  123. package/template/.claude/skills/wiki/SKILL.md +0 -149
  124. 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