@leeovery/claude-technical-workflows 2.0.0
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/LICENSE +21 -0
- package/README.md +283 -0
- package/agents/chain-verifier.md +129 -0
- package/commands/interview.md +54 -0
- package/commands/start-discussion.md +36 -0
- package/commands/start-implementation.md +107 -0
- package/commands/start-planning.md +92 -0
- package/commands/start-research.md +21 -0
- package/commands/start-specification.md +89 -0
- package/package.json +24 -0
- package/skills/.gitkeep +0 -0
- package/skills/technical-discussion/SKILL.md +68 -0
- package/skills/technical-discussion/references/guidelines.md +86 -0
- package/skills/technical-discussion/references/meeting-assistant.md +102 -0
- package/skills/technical-discussion/references/template.md +127 -0
- package/skills/technical-implementation/SKILL.md +167 -0
- package/skills/technical-implementation/references/code-quality.md +41 -0
- package/skills/technical-implementation/references/environment-setup.md +97 -0
- package/skills/technical-implementation/references/plan-execution.md +49 -0
- package/skills/technical-implementation/references/tdd-workflow.md +63 -0
- package/skills/technical-planning/SKILL.md +46 -0
- package/skills/technical-planning/references/formal-planning.md +118 -0
- package/skills/technical-planning/references/output-backlog-md.md +227 -0
- package/skills/technical-planning/references/output-beads.md +302 -0
- package/skills/technical-planning/references/output-formats.md +14 -0
- package/skills/technical-planning/references/output-linear.md +211 -0
- package/skills/technical-planning/references/output-local-markdown.md +185 -0
- package/skills/technical-research/SKILL.md +81 -0
- package/skills/technical-review/SKILL.md +107 -0
- package/skills/technical-review/references/review-checklist.md +155 -0
- package/skills/technical-review/references/template.md +46 -0
- package/skills/technical-specification/SKILL.md +51 -0
- package/skills/technical-specification/references/specification-guide.md +170 -0
package/LICENSE
ADDED
|
@@ -0,0 +1,21 @@
|
|
|
1
|
+
MIT License
|
|
2
|
+
|
|
3
|
+
Copyright (c) 2025 Lee Overy
|
|
4
|
+
|
|
5
|
+
Permission is hereby granted, free of charge, to any person obtaining a copy
|
|
6
|
+
of this software and associated documentation files (the "Software"), to deal
|
|
7
|
+
in the Software without restriction, including without limitation the rights
|
|
8
|
+
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
|
|
9
|
+
copies of the Software, and to permit persons to whom the Software is
|
|
10
|
+
furnished to do so, subject to the following conditions:
|
|
11
|
+
|
|
12
|
+
The above copyright notice and this permission notice shall be included in all
|
|
13
|
+
copies or substantial portions of the Software.
|
|
14
|
+
|
|
15
|
+
THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
|
|
16
|
+
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
|
|
17
|
+
FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
|
|
18
|
+
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
|
|
19
|
+
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
|
|
20
|
+
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
|
|
21
|
+
SOFTWARE.
|
package/README.md
ADDED
|
@@ -0,0 +1,283 @@
|
|
|
1
|
+
<h1 align="center">Claude Technical Workflows</h1>
|
|
2
|
+
|
|
3
|
+
<p align="center">
|
|
4
|
+
<strong>Structured Discussion & Planning Skills for Claude Code</strong>
|
|
5
|
+
</p>
|
|
6
|
+
|
|
7
|
+
<p align="center">
|
|
8
|
+
<a href="#installation">Installation</a> •
|
|
9
|
+
<a href="#the-six-phase-workflow">Workflow</a> •
|
|
10
|
+
<a href="#skills">Skills</a> •
|
|
11
|
+
<a href="#commands">Commands</a> •
|
|
12
|
+
<a href="#how-it-works">How It Works</a> •
|
|
13
|
+
<a href="#contributing">Contributing</a>
|
|
14
|
+
</p>
|
|
15
|
+
|
|
16
|
+
---
|
|
17
|
+
|
|
18
|
+
## Versions
|
|
19
|
+
|
|
20
|
+
| Version | Package Manager | Status | Branch |
|
|
21
|
+
|---------|----------------|--------|--------|
|
|
22
|
+
| 2.x | npm | **Active** | `main` |
|
|
23
|
+
| 1.x | Composer | Deprecated | [`v1`](https://github.com/leeovery/claude-technical-workflows/tree/v1) |
|
|
24
|
+
|
|
25
|
+
## About
|
|
26
|
+
|
|
27
|
+
A structured approach to technical discussions and implementation planning with Claude Code. These skills enforce a deliberate **research-then-discuss-then-specify-then-plan-then-implement-then-review** workflow that captures context, decisions, and rationale before any code is written—then validates the work against those artifacts.
|
|
28
|
+
|
|
29
|
+
**Why this matters:** Complex features benefit from thorough discussion before implementation. These skills help you document the *what* and *why* before diving into the *how*—preserving architectural decisions, edge cases, and the reasoning behind choices that would otherwise be lost.
|
|
30
|
+
|
|
31
|
+
**This is a work in progress.** The workflow is being refined through real-world usage. Expect updates as patterns evolve.
|
|
32
|
+
|
|
33
|
+
## Installation
|
|
34
|
+
|
|
35
|
+
```bash
|
|
36
|
+
npm install @leeovery/claude-technical-workflows
|
|
37
|
+
```
|
|
38
|
+
|
|
39
|
+
That's it. The [Claude Manager](https://github.com/leeovery/claude-manager) handles everything else automatically.
|
|
40
|
+
|
|
41
|
+
## The Six-Phase Workflow
|
|
42
|
+
|
|
43
|
+
This package enforces a deliberate progression through six distinct phases:
|
|
44
|
+
|
|
45
|
+
```
|
|
46
|
+
┌───────────────┐ ┌───────────────┐ ┌───────────────┐ ┌───────────────┐ ┌───────────────┐ ┌───────────────┐
|
|
47
|
+
│ Research │──▶│ Discussion │──▶│ Specification │──▶│ Planning │──▶│Implementation │──▶│ Review │
|
|
48
|
+
│ (Phase 1) │ │ (Phase 2) │ │ (Phase 3) │ │ (Phase 4) │ │ (Phase 5) │ │ (Phase 6) │
|
|
49
|
+
├───────────────┤ ├───────────────┤ ├───────────────┤ ├───────────────┤ ├───────────────┤ ├───────────────┤
|
|
50
|
+
│ EXPLORING │ │ WHAT & WHY │ │ REFINING │ │ HOW │ │ DOING │ │ VALIDATING │
|
|
51
|
+
│ │ │ │ │ │ │ │ │ │ │ │
|
|
52
|
+
│ • Ideas │ │ • Architecture│ │ • Validate │ │ • Phases │ │ • Tests first │ │ • Plan check │
|
|
53
|
+
│ • Market │ │ • Decisions │ │ • Filter │ │ • Tasks │ │ • Then code │ │ • Specs check │
|
|
54
|
+
│ • Viability │ │ • Edge cases │ │ • Enrich │ │ • Criteria │ │ • Commit often│ │ • Test quality│
|
|
55
|
+
│ │ │ • Rationale │ │ • Standalone │ │ • Outputs │ │ • Phase gates │ │ • Code quality│
|
|
56
|
+
└───────────────┘ └───────────────┘ └───────────────┘ └───────────────┘ └───────────────┘ └───────────────┘
|
|
57
|
+
▲ ▲ ▲ ▲ ▲ ▲
|
|
58
|
+
│ │ │ │ │ │
|
|
59
|
+
technical-research technical-discussion technical-spec technical-planning technical-impl technical-review
|
|
60
|
+
```
|
|
61
|
+
|
|
62
|
+
**Phase 1 - Research:** Explore ideas from their earliest seed. Investigate market fit, technical feasibility, business viability. Free-flowing exploration that may or may not lead to building something.
|
|
63
|
+
|
|
64
|
+
**Phase 2 - Discussion:** Captures the back-and-forth exploration of a problem. Documents competing solutions, why certain approaches won or lost, edge cases discovered, and the journey to decisions—not just the decisions themselves.
|
|
65
|
+
|
|
66
|
+
**Phase 3 - Specification:** Transforms discussion documentation into a validated, standalone specification. Filters hallucinations and inaccuracies, enriches gaps through discussion, and builds a document that planning can execute against without referencing other sources.
|
|
67
|
+
|
|
68
|
+
**Phase 4 - Planning:** Converts specifications into actionable implementation plans with phases, tasks, and acceptance criteria. Supports multiple output formats (local markdown, Linear, Backlog.md).
|
|
69
|
+
|
|
70
|
+
**Phase 5 - Implementation:** Executes the plan using strict TDD. Writes tests first, implements to pass, commits frequently, and stops for user approval between phases.
|
|
71
|
+
|
|
72
|
+
**Phase 6 - Review:** Validates completed work against specification requirements and plan acceptance criteria. The specification is the validated source of truth—earlier phases may contain rejected ideas that were intentionally filtered out. Provides structured feedback without fixing code directly.
|
|
73
|
+
|
|
74
|
+
## How It Works
|
|
75
|
+
|
|
76
|
+
This package depends on [`@leeovery/claude-manager`](https://github.com/leeovery/claude-manager), which:
|
|
77
|
+
|
|
78
|
+
1. **Copies skills** into your project's `.claude/skills/` directory
|
|
79
|
+
2. **Copies commands** into your project's `.claude/commands/` directory
|
|
80
|
+
3. **Copies agents** into your project's `.claude/agents/` directory
|
|
81
|
+
4. **Tracks installed plugins** via a manifest file
|
|
82
|
+
5. **Handles installation/removal** automatically via npm hooks
|
|
83
|
+
|
|
84
|
+
You don't need to configure anything—just install and start discussing.
|
|
85
|
+
|
|
86
|
+
### Output Structure
|
|
87
|
+
|
|
88
|
+
Documents are stored using a **phase-first** organization:
|
|
89
|
+
|
|
90
|
+
```
|
|
91
|
+
docs/workflow/
|
|
92
|
+
├── research/ # Phase 1 - flat, semantically named files
|
|
93
|
+
│ ├── competitor-analysis.md
|
|
94
|
+
│ └── pricing-models.md
|
|
95
|
+
├── discussion/ # Phase 2 - one file per topic
|
|
96
|
+
│ └── {topic}.md
|
|
97
|
+
├── specification/ # Phase 3 - one file per topic
|
|
98
|
+
│ └── {topic}.md
|
|
99
|
+
└── planning/ # Phase 4 - one file per topic
|
|
100
|
+
└── {topic}.md
|
|
101
|
+
```
|
|
102
|
+
|
|
103
|
+
Research is a flat directory of semantically named files (topics emerge later). From discussion onwards, each topic gets its own file per phase.
|
|
104
|
+
|
|
105
|
+
### Package Structure
|
|
106
|
+
|
|
107
|
+
This package provides:
|
|
108
|
+
|
|
109
|
+
```
|
|
110
|
+
skills/
|
|
111
|
+
├── technical-research/ # Phase 1: Explore and validate ideas
|
|
112
|
+
├── technical-discussion/ # Phase 2: Document discussions
|
|
113
|
+
├── technical-specification/ # Phase 3: Build validated specifications
|
|
114
|
+
├── technical-planning/ # Phase 4: Create implementation plans
|
|
115
|
+
├── technical-implementation/ # Phase 5: Execute via TDD
|
|
116
|
+
└── technical-review/ # Phase 6: Validate against artifacts
|
|
117
|
+
|
|
118
|
+
commands/
|
|
119
|
+
├── start-research.md # Begin research exploration
|
|
120
|
+
├── start-discussion.md # Begin technical discussions
|
|
121
|
+
├── start-specification.md # Begin specification building
|
|
122
|
+
├── start-planning.md # Begin implementation planning
|
|
123
|
+
└── interview.md # Focused questioning mode
|
|
124
|
+
|
|
125
|
+
agents/
|
|
126
|
+
└── chain-verifier.md # Parallel task verification for review
|
|
127
|
+
```
|
|
128
|
+
|
|
129
|
+
## Skills
|
|
130
|
+
|
|
131
|
+
| Skill | Phase | Description |
|
|
132
|
+
|-------|-------|-------------|
|
|
133
|
+
| [**technical-research**](skills/technical-research/) | 1 | Explore ideas from their earliest seed. Investigate market fit, technical feasibility, business viability. Free-flowing exploration across technical, business, and market domains. |
|
|
134
|
+
| [**technical-discussion**](skills/technical-discussion/) | 2 | Document technical discussions as expert architect and meeting assistant. Captures context, decisions, edge cases, competing solutions, debates, and rationale. |
|
|
135
|
+
| [**technical-specification**](skills/technical-specification/) | 3 | Build validated specifications from discussion documents through collaborative refinement. Filters hallucinations, enriches gaps, produces standalone spec. |
|
|
136
|
+
| [**technical-planning**](skills/technical-planning/) | 4 | Transform specifications into actionable implementation plans with phases, tasks, and acceptance criteria. Supports multiple output formats. |
|
|
137
|
+
| [**technical-implementation**](skills/technical-implementation/) | 5 | Execute implementation plans using strict TDD workflow. Writes tests first, implements to pass, commits frequently, and gates phases on user approval. |
|
|
138
|
+
| [**technical-review**](skills/technical-review/) | 6 | Review completed implementation against specification requirements and plan acceptance criteria. Uses parallel subagents for efficient chain verification. Produces structured feedback without fixing code. |
|
|
139
|
+
|
|
140
|
+
### technical-research
|
|
141
|
+
|
|
142
|
+
Acts as **research partner** with broad expertise spanning technical, product, business, and market domains.
|
|
143
|
+
|
|
144
|
+
**Use when:**
|
|
145
|
+
- Exploring a new idea from its earliest seed
|
|
146
|
+
- Investigating market fit, competitors, or positioning
|
|
147
|
+
- Validating technical feasibility before committing to build
|
|
148
|
+
- Learning and exploration without necessarily building anything
|
|
149
|
+
- Brain dumping early thoughts before formal discussion
|
|
150
|
+
|
|
151
|
+
**What it does:**
|
|
152
|
+
- Explores ideas freely across technical, business, and market domains
|
|
153
|
+
- Prompts before documenting: "Shall I capture that?"
|
|
154
|
+
- Creates research documents that may seed the discussion phase
|
|
155
|
+
- Follows tangents and goes broad when useful
|
|
156
|
+
|
|
157
|
+
### technical-discussion
|
|
158
|
+
|
|
159
|
+
Acts as both **expert software architect** (participating in discussions) and **documentation assistant** (capturing them) simultaneously.
|
|
160
|
+
|
|
161
|
+
**Use when:**
|
|
162
|
+
- Discussing or exploring architecture and design decisions
|
|
163
|
+
- Working through edge cases before specification
|
|
164
|
+
- Documenting technical decisions and their rationale
|
|
165
|
+
- Capturing competing solutions and why certain choices were made
|
|
166
|
+
|
|
167
|
+
**What it captures:**
|
|
168
|
+
- Back-and-forth debates showing how decisions were reached
|
|
169
|
+
- Small details and edge cases that were discussed
|
|
170
|
+
- Competing solutions and why some won over others
|
|
171
|
+
- The journey—false paths, "aha" moments, course corrections
|
|
172
|
+
|
|
173
|
+
### technical-specification
|
|
174
|
+
|
|
175
|
+
Acts as **expert technical architect** and **specification builder**. Transforms discussion documents into validated, standalone specifications.
|
|
176
|
+
|
|
177
|
+
**Use when:**
|
|
178
|
+
- Ready to validate and refine discussion content
|
|
179
|
+
- Need to filter potential hallucinations or inaccuracies from source material
|
|
180
|
+
- Building a standalone document that planning can execute against
|
|
181
|
+
- Converting discussions into verified requirements
|
|
182
|
+
|
|
183
|
+
**What it produces:**
|
|
184
|
+
- Validated, standalone specification document
|
|
185
|
+
- Filtered content (hallucinations and inaccuracies removed)
|
|
186
|
+
- Enriched content (gaps filled through discussion)
|
|
187
|
+
- Clear bridge document for formal planning
|
|
188
|
+
|
|
189
|
+
### technical-planning
|
|
190
|
+
|
|
191
|
+
Converts specifications into structured implementation plans.
|
|
192
|
+
|
|
193
|
+
**Use when:**
|
|
194
|
+
- Ready to plan implementation after specification is complete
|
|
195
|
+
- Need to structure how to build something with phases and concrete steps
|
|
196
|
+
- Converting specification into actionable developer guidance
|
|
197
|
+
|
|
198
|
+
**What it produces:**
|
|
199
|
+
- Phased implementation plans with specific tasks
|
|
200
|
+
- Acceptance criteria at phase and task levels
|
|
201
|
+
- Multiple output formats: local markdown, Linear, Backlog.md, or Beads
|
|
202
|
+
|
|
203
|
+
### technical-implementation
|
|
204
|
+
|
|
205
|
+
Executes plans through strict TDD. Acts as an expert senior developer who builds quality software through disciplined test-driven development.
|
|
206
|
+
|
|
207
|
+
**Use when:**
|
|
208
|
+
- Implementing a plan from `docs/workflow/planning/{topic}.md`
|
|
209
|
+
- Ad hoc coding that should follow TDD and quality standards
|
|
210
|
+
- Bug fixes or features benefiting from structured implementation
|
|
211
|
+
|
|
212
|
+
**Hard rules:**
|
|
213
|
+
- No code before tests—write the failing test first
|
|
214
|
+
- No test changes to pass—fix the code, not the tests
|
|
215
|
+
- No scope expansion—if it's not in the plan, don't build it
|
|
216
|
+
- Commit after green—every passing test is a commit point
|
|
217
|
+
|
|
218
|
+
### technical-review
|
|
219
|
+
|
|
220
|
+
Reviews completed work with fresh perspective. Validates implementation against prior workflow artifacts without fixing code directly.
|
|
221
|
+
|
|
222
|
+
**Use when:**
|
|
223
|
+
- Implementation phase is complete
|
|
224
|
+
- User wants validation before merging/shipping
|
|
225
|
+
- Quality gate check needed after implementation
|
|
226
|
+
|
|
227
|
+
**What it checks:**
|
|
228
|
+
- Were specification requirements implemented?
|
|
229
|
+
- Were all plan acceptance criteria met?
|
|
230
|
+
- Do tests actually verify requirements?
|
|
231
|
+
- Does code follow project conventions?
|
|
232
|
+
|
|
233
|
+
## Commands
|
|
234
|
+
|
|
235
|
+
Slash commands to quickly invoke the workflow.
|
|
236
|
+
|
|
237
|
+
| Command | Description |
|
|
238
|
+
|---------|-------------|
|
|
239
|
+
| [**/start-research**](commands/start-research.md) | Begin research exploration. For early-stage ideas, feasibility checks, and broad exploration before formal discussion. |
|
|
240
|
+
| [**/start-discussion**](commands/start-discussion.md) | Begin a new technical discussion. Gathers topic, context, background information, and relevant codebase areas before starting documentation. |
|
|
241
|
+
| [**/start-specification**](commands/start-specification.md) | Start a specification session from an existing discussion. Validates and refines discussion content into a standalone specification. |
|
|
242
|
+
| [**/start-planning**](commands/start-planning.md) | Start a planning session from an existing specification. Creates implementation plans with phases, tasks, and acceptance criteria. Supports multiple output formats (markdown, Linear, Backlog.md, Beads). |
|
|
243
|
+
| [**/interview**](commands/interview.md) | Shift into focused questioning mode during research or discussion. Probes ideas with non-obvious questions, challenges assumptions, and surfaces concerns. |
|
|
244
|
+
|
|
245
|
+
## Agents
|
|
246
|
+
|
|
247
|
+
Subagents that skills can spawn for parallel task execution.
|
|
248
|
+
|
|
249
|
+
| Agent | Used By | Description |
|
|
250
|
+
|-------|---------|-------------|
|
|
251
|
+
| [**chain-verifier**](agents/chain-verifier.md) | technical-review | Verifies a single plan task was implemented correctly. Checks implementation, tests (not under/over-tested), and code quality. Multiple chain-verifiers run in parallel to verify ALL tasks efficiently. |
|
|
252
|
+
|
|
253
|
+
## Requirements
|
|
254
|
+
|
|
255
|
+
- Node.js 18+
|
|
256
|
+
- [@leeovery/claude-manager](https://github.com/leeovery/claude-manager) ^2.0.0 (installed automatically)
|
|
257
|
+
|
|
258
|
+
## Contributing
|
|
259
|
+
|
|
260
|
+
Contributions are welcome! Whether it's:
|
|
261
|
+
|
|
262
|
+
- **Bug fixes** in the documentation or skill definitions
|
|
263
|
+
- **Improvements** to the workflow or templates
|
|
264
|
+
- **Discussion** about approaches and trade-offs
|
|
265
|
+
- **New skills** that complement the discuss-specify-plan-implement workflow
|
|
266
|
+
|
|
267
|
+
Please open an issue first to discuss significant changes.
|
|
268
|
+
|
|
269
|
+
## Related Packages
|
|
270
|
+
|
|
271
|
+
- [**@leeovery/claude-manager**](https://github.com/leeovery/claude-manager) — The plugin manager that powers skill installation
|
|
272
|
+
- [**@leeovery/claude-laravel**](https://github.com/leeovery/claude-laravel) — Laravel development skills for Claude Code
|
|
273
|
+
- [**@leeovery/claude-nuxt**](https://github.com/leeovery/claude-nuxt) — Nuxt.js development skills for Claude Code
|
|
274
|
+
|
|
275
|
+
## License
|
|
276
|
+
|
|
277
|
+
MIT License. See [LICENSE](LICENSE) for details.
|
|
278
|
+
|
|
279
|
+
---
|
|
280
|
+
|
|
281
|
+
<p align="center">
|
|
282
|
+
<sub>Built with care by <a href="https://github.com/leeovery">Lee Overy</a></sub>
|
|
283
|
+
</p>
|
|
@@ -0,0 +1,129 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: chain-verifier
|
|
3
|
+
description: Verifies a single plan task was implemented correctly. Checks implementation, tests, and code quality against the task's acceptance criteria and spec context. Invoked by technical-review to verify ALL plan tasks in PARALLEL.
|
|
4
|
+
tools: Read, Glob, Grep
|
|
5
|
+
model: haiku
|
|
6
|
+
---
|
|
7
|
+
|
|
8
|
+
# Chain Verifier
|
|
9
|
+
|
|
10
|
+
Act as a **senior software architect** with deep experience in code review. You verify that ONE plan task was implemented correctly, tested adequately, and meets professional quality standards.
|
|
11
|
+
|
|
12
|
+
## Your Input
|
|
13
|
+
|
|
14
|
+
You receive:
|
|
15
|
+
1. **Plan task**: A specific task with its acceptance criteria
|
|
16
|
+
2. **Specification path**: For loading context about this task's feature/requirement
|
|
17
|
+
3. **Plan path**: The full plan for additional context
|
|
18
|
+
4. **Implementation scope**: Files or directories to check
|
|
19
|
+
|
|
20
|
+
## Your Task
|
|
21
|
+
|
|
22
|
+
For the given plan task:
|
|
23
|
+
|
|
24
|
+
```
|
|
25
|
+
Plan Task (acceptance criteria)
|
|
26
|
+
↓
|
|
27
|
+
Load Spec Context (deeper understanding)
|
|
28
|
+
↓
|
|
29
|
+
Verify Implementation (code exists, correct)
|
|
30
|
+
↓
|
|
31
|
+
Verify Tests (adequate, not over/under tested)
|
|
32
|
+
↓
|
|
33
|
+
Check Code Quality (readable, conventions)
|
|
34
|
+
```
|
|
35
|
+
|
|
36
|
+
### Step 1: Understand the Task
|
|
37
|
+
|
|
38
|
+
From the plan task:
|
|
39
|
+
- What should be built?
|
|
40
|
+
- What are the acceptance criteria?
|
|
41
|
+
- What tests should exist (micro acceptance)?
|
|
42
|
+
|
|
43
|
+
### Step 2: Load Spec Context
|
|
44
|
+
|
|
45
|
+
Search the specification for relevant context:
|
|
46
|
+
- What is the broader requirement this task fulfills?
|
|
47
|
+
- Are there edge cases or constraints mentioned?
|
|
48
|
+
- What behavior is expected?
|
|
49
|
+
|
|
50
|
+
### Step 3: Verify Implementation
|
|
51
|
+
|
|
52
|
+
Search the codebase:
|
|
53
|
+
- Is the task implemented?
|
|
54
|
+
- Does the implementation match the acceptance criteria?
|
|
55
|
+
- Does it align with the spec's expected behavior?
|
|
56
|
+
- Any drift from what was planned?
|
|
57
|
+
|
|
58
|
+
### Step 4: Verify Tests
|
|
59
|
+
|
|
60
|
+
Evaluate test coverage critically:
|
|
61
|
+
- Is there a test for this task?
|
|
62
|
+
- Does the test actually verify the acceptance criteria?
|
|
63
|
+
- **Not under-tested**: Are edge cases from the spec covered?
|
|
64
|
+
- **Not over-tested**: Are tests focused and necessary, or bloated with redundant checks?
|
|
65
|
+
- Would the test fail if the feature broke?
|
|
66
|
+
|
|
67
|
+
### Step 5: Check Code Quality
|
|
68
|
+
|
|
69
|
+
Review the implementation as a senior architect would:
|
|
70
|
+
|
|
71
|
+
**Project conventions** (if `.claude/skills/` contains project-specific guidance):
|
|
72
|
+
- Check for project-specific code quality skills
|
|
73
|
+
- Follow any framework or architecture guidelines defined there
|
|
74
|
+
|
|
75
|
+
**General principles** (always apply):
|
|
76
|
+
- **SOLID**: Single responsibility, open/closed, Liskov substitution, interface segregation, dependency inversion
|
|
77
|
+
- **DRY**: No unnecessary duplication (but don't over-abstract prematurely)
|
|
78
|
+
- **Low complexity**: Cyclomatic complexity is reasonable, code paths are clear
|
|
79
|
+
- **Modern idioms**: Uses current language features appropriately
|
|
80
|
+
- **Readability**: Code is self-documenting, intent is clear
|
|
81
|
+
- **Security**: No obvious vulnerabilities (injection, exposure, etc.)
|
|
82
|
+
- **Performance**: No obvious inefficiencies (N+1 queries, unnecessary loops, etc.)
|
|
83
|
+
|
|
84
|
+
## Your Output
|
|
85
|
+
|
|
86
|
+
Return a structured finding:
|
|
87
|
+
|
|
88
|
+
```
|
|
89
|
+
TASK: [Task name/description]
|
|
90
|
+
|
|
91
|
+
ACCEPTANCE CRITERIA: [List from plan]
|
|
92
|
+
|
|
93
|
+
STATUS: Complete | Incomplete | Issues Found
|
|
94
|
+
|
|
95
|
+
SPEC CONTEXT: [Brief summary of relevant spec context]
|
|
96
|
+
|
|
97
|
+
IMPLEMENTATION:
|
|
98
|
+
- Status: [Implemented/Missing/Partial/Drifted]
|
|
99
|
+
- Location: [file:line references]
|
|
100
|
+
- Notes: [Any concerns]
|
|
101
|
+
|
|
102
|
+
TESTS:
|
|
103
|
+
- Status: [Adequate/Under-tested/Over-tested/Missing]
|
|
104
|
+
- Coverage: [What is/isn't tested]
|
|
105
|
+
- Notes: [Specific issues]
|
|
106
|
+
|
|
107
|
+
CODE QUALITY:
|
|
108
|
+
- Project conventions: [Followed/Violations/N/A]
|
|
109
|
+
- SOLID principles: [Good/Concerns]
|
|
110
|
+
- Complexity: [Low/Acceptable/High]
|
|
111
|
+
- Modern idioms: [Yes/Opportunities]
|
|
112
|
+
- Readability: [Good/Concerns]
|
|
113
|
+
- Issues: [Specific problems if any]
|
|
114
|
+
|
|
115
|
+
BLOCKING ISSUES:
|
|
116
|
+
- [List any issues that must be fixed]
|
|
117
|
+
|
|
118
|
+
NON-BLOCKING NOTES:
|
|
119
|
+
- [Suggestions for improvement]
|
|
120
|
+
```
|
|
121
|
+
|
|
122
|
+
## Rules
|
|
123
|
+
|
|
124
|
+
1. **One task only** - You verify exactly one plan task per invocation
|
|
125
|
+
2. **Be thorough** - Check implementation, tests, AND quality
|
|
126
|
+
3. **Be specific** - Include file paths and line numbers
|
|
127
|
+
4. **Balanced test review** - Flag both under-testing AND over-testing
|
|
128
|
+
5. **Report findings** - Don't fix anything, just report what you find
|
|
129
|
+
6. **Fast and focused** - You're one of many running in parallel
|
|
@@ -0,0 +1,54 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: Shift into focused questioning mode to probe an idea more deeply during research or discussion phases.
|
|
3
|
+
---
|
|
4
|
+
|
|
5
|
+
Shift into focused questioning mode to probe an idea more deeply.
|
|
6
|
+
|
|
7
|
+
## Setup
|
|
8
|
+
|
|
9
|
+
Before beginning, ask the user:
|
|
10
|
+
|
|
11
|
+
1. **What stage are you in?** Research or discussion?
|
|
12
|
+
2. **What topic or idea do you want to explore?**
|
|
13
|
+
|
|
14
|
+
Wait for responses before proceeding.
|
|
15
|
+
|
|
16
|
+
## Process
|
|
17
|
+
|
|
18
|
+
1. **Check for existing context**:
|
|
19
|
+
- Research: Look for docs in `docs/workflow/research/`
|
|
20
|
+
- Discussion: Look for the topic's file in `docs/workflow/discussion/{topic}.md`
|
|
21
|
+
|
|
22
|
+
Read what exists to understand what's already been explored.
|
|
23
|
+
|
|
24
|
+
2. **Begin interviewing**: Use the AskUserQuestion tool to probe the idea. Focus on:
|
|
25
|
+
- Non-obvious questions (not things already answered)
|
|
26
|
+
- Assumptions and unstated constraints
|
|
27
|
+
- Tradeoffs and concerns
|
|
28
|
+
- What could go wrong
|
|
29
|
+
- The "why" behind decisions
|
|
30
|
+
|
|
31
|
+
3. **Go where it leads**: Follow tangents if they reveal something valuable. This isn't a checklist—it's a conversation.
|
|
32
|
+
|
|
33
|
+
4. **Document as you go**: Don't defer writing to the end. Capture insights in the appropriate location:
|
|
34
|
+
- Research: `docs/workflow/research/` (semantic filenames)
|
|
35
|
+
- Discussion: `docs/workflow/discussion/{topic}.md`
|
|
36
|
+
|
|
37
|
+
Ask before documenting: "Shall I capture that?"
|
|
38
|
+
|
|
39
|
+
5. **Commit frequently**: At natural breaks and before context refresh. Don't risk losing detail.
|
|
40
|
+
|
|
41
|
+
6. **Exit when done**: The user decides when the interview is complete.
|
|
42
|
+
|
|
43
|
+
## Question quality
|
|
44
|
+
|
|
45
|
+
Aim for questions that:
|
|
46
|
+
- The user hasn't already answered
|
|
47
|
+
- Reveal hidden complexity
|
|
48
|
+
- Surface concerns early
|
|
49
|
+
- Challenge comfortable assumptions
|
|
50
|
+
|
|
51
|
+
Avoid:
|
|
52
|
+
- Restating what's already documented
|
|
53
|
+
- Obvious surface-level questions
|
|
54
|
+
- Leading questions that assume an answer
|
|
@@ -0,0 +1,36 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: Start a technical discussion using the technical-discussion skill. Gathers information about the topic and creates discussion documentation in docs/workflow/discussion/
|
|
3
|
+
---
|
|
4
|
+
|
|
5
|
+
Invoke the **technical-discussion** skill for this conversation.
|
|
6
|
+
|
|
7
|
+
Before beginning, ask the user these questions to properly set up the discussion:
|
|
8
|
+
|
|
9
|
+
## Essential Information
|
|
10
|
+
|
|
11
|
+
1. **Discussion Topic**: What are we discussing? (This will be used as the filename in `docs/workflow/discussion/{topic}.md`)
|
|
12
|
+
|
|
13
|
+
2. **Context & Background**:
|
|
14
|
+
- What sparked this discussion?
|
|
15
|
+
- What problem are we trying to solve?
|
|
16
|
+
- Why are we discussing this now?
|
|
17
|
+
|
|
18
|
+
3. **Foundational Knowledge**:
|
|
19
|
+
- Is there any background information I need to understand before we begin?
|
|
20
|
+
- Are there specific concepts, technologies, or architectures I should know about?
|
|
21
|
+
- Any constraints, requirements, or limitations I should be aware of?
|
|
22
|
+
|
|
23
|
+
4. **Codebase Review**:
|
|
24
|
+
- Are there specific files in this repository I should read first?
|
|
25
|
+
- Should I explore any particular directories or components?
|
|
26
|
+
- Is there existing code related to this discussion?
|
|
27
|
+
|
|
28
|
+
## After Gathering Information
|
|
29
|
+
|
|
30
|
+
Once I have this information:
|
|
31
|
+
- Ensure discussion directory exists: `docs/workflow/discussion/`
|
|
32
|
+
- Start documenting the discussion following the technical-discussion skill structure
|
|
33
|
+
- Create file: `docs/workflow/discussion/{topic}.md`
|
|
34
|
+
- Commit frequently at natural discussion breaks
|
|
35
|
+
|
|
36
|
+
Ask these questions clearly and wait for responses before proceeding with the discussion.
|
|
@@ -0,0 +1,107 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: Start an implementation session from an existing plan. Discovers available plans, checks environment setup, and invokes the technical-implementation skill.
|
|
3
|
+
---
|
|
4
|
+
|
|
5
|
+
## IMPORTANT: Follow these steps EXACTLY. Do not skip steps.
|
|
6
|
+
|
|
7
|
+
- Ask each question and WAIT for a response before proceeding
|
|
8
|
+
- Do NOT install anything or invoke tools until Step 5
|
|
9
|
+
- Even if the user's initial prompt seems to answer a question, still confirm with them at the appropriate step
|
|
10
|
+
- Do NOT make assumptions about what the user wants
|
|
11
|
+
- Complete each step fully before moving to the next
|
|
12
|
+
|
|
13
|
+
## Instructions
|
|
14
|
+
|
|
15
|
+
Follow these steps EXACTLY as written. Do not skip steps or combine them.
|
|
16
|
+
|
|
17
|
+
Before beginning, discover existing work and gather necessary information.
|
|
18
|
+
|
|
19
|
+
## Important
|
|
20
|
+
|
|
21
|
+
Use simple, individual commands. Never combine multiple operations into bash loops or one-liners. Execute commands one at a time.
|
|
22
|
+
|
|
23
|
+
## Step 1: Discover Existing Plans
|
|
24
|
+
|
|
25
|
+
Scan the codebase for plans:
|
|
26
|
+
|
|
27
|
+
1. **Find plans**: Look in `docs/workflow/planning/`
|
|
28
|
+
- Run `ls docs/workflow/planning/` to list plan files
|
|
29
|
+
- Each file is named `{topic}.md`
|
|
30
|
+
|
|
31
|
+
2. **Check plan format**: For each plan file
|
|
32
|
+
- Run `head -10 docs/workflow/planning/{topic}.md` to read the frontmatter
|
|
33
|
+
- Note the `format:` field
|
|
34
|
+
- Do NOT use bash loops - run separate `head` commands for each topic
|
|
35
|
+
|
|
36
|
+
## Step 2: Present Options to User
|
|
37
|
+
|
|
38
|
+
Show what you found.
|
|
39
|
+
|
|
40
|
+
> **Note:** If no plans exist, inform the user that this workflow is designed to be executed in sequence. They need to create plans from specifications prior to implementation using `/start-planning`.
|
|
41
|
+
|
|
42
|
+
> **Auto-select:** If exactly one plan exists, automatically select it and proceed to Step 3. Inform the user which plan was selected. Do not ask for confirmation.
|
|
43
|
+
|
|
44
|
+
```
|
|
45
|
+
Plans found:
|
|
46
|
+
{topic-1}
|
|
47
|
+
{topic-2}
|
|
48
|
+
|
|
49
|
+
Which plan would you like to implement?
|
|
50
|
+
```
|
|
51
|
+
|
|
52
|
+
## Step 3: Check Environment Setup
|
|
53
|
+
|
|
54
|
+
> **IMPORTANT**: This step is for **information gathering only**. Do NOT execute any setup commands at this stage. The technical-implementation skill will handle execution when invoked.
|
|
55
|
+
|
|
56
|
+
After the user selects a plan:
|
|
57
|
+
|
|
58
|
+
1. Check if `docs/workflow/environment-setup.md` exists
|
|
59
|
+
2. If it exists, note the file location for the skill handoff
|
|
60
|
+
3. If missing, ask: "Are there any environment setup instructions I should follow?"
|
|
61
|
+
- If the user provides instructions, save them to `docs/workflow/environment-setup.md`, commit and push to Git
|
|
62
|
+
- See `skills/technical-implementation/references/environment-setup.md` for format guidance
|
|
63
|
+
|
|
64
|
+
## Step 4: Ask About Scope
|
|
65
|
+
|
|
66
|
+
Ask the user about implementation scope:
|
|
67
|
+
|
|
68
|
+
```
|
|
69
|
+
How would you like to proceed?
|
|
70
|
+
|
|
71
|
+
1. **Implement all phases** - Work through the entire plan sequentially
|
|
72
|
+
2. **Implement specific phase** - Focus on one phase (e.g., "Phase 1")
|
|
73
|
+
3. **Implement specific task** - Focus on a single task
|
|
74
|
+
|
|
75
|
+
Which approach?
|
|
76
|
+
```
|
|
77
|
+
|
|
78
|
+
If they choose a specific phase or task, ask them to specify which one.
|
|
79
|
+
|
|
80
|
+
> **Note:** Do NOT verify that the phase or task exists. Accept the user's answer and pass it to the skill. Validation happens during the implementation phase.
|
|
81
|
+
|
|
82
|
+
## Step 5: Invoke Implementation Skill
|
|
83
|
+
|
|
84
|
+
Invoke the **technical-implementation** skill for this conversation.
|
|
85
|
+
|
|
86
|
+
Pass to the technical-implementation skill:
|
|
87
|
+
- Plan: `docs/workflow/planning/{topic}.md`
|
|
88
|
+
- Format: (from frontmatter)
|
|
89
|
+
- Scope: (all phases | specific phase | specific task)
|
|
90
|
+
- Environment setup: (completed | not needed)
|
|
91
|
+
|
|
92
|
+
**Example handoff:**
|
|
93
|
+
```
|
|
94
|
+
Implementation session for: {topic}
|
|
95
|
+
Plan: docs/workflow/planning/{topic}.md
|
|
96
|
+
Format: {format}
|
|
97
|
+
Scope: All phases
|
|
98
|
+
|
|
99
|
+
Environment setup: Completed (or: Not needed)
|
|
100
|
+
|
|
101
|
+
Begin implementation using the technical-implementation skill.
|
|
102
|
+
```
|
|
103
|
+
|
|
104
|
+
## Notes
|
|
105
|
+
|
|
106
|
+
- Ask questions clearly and wait for responses before proceeding
|
|
107
|
+
- Execute environment setup before starting implementation
|