@comfanion/workflow 4.36.60 → 4.36.62
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/package.json +1 -1
- package/src/build-info.json +2 -2
- package/src/opencode/agents/analyst.md +8 -2
- package/src/opencode/agents/architect.md +2 -1
- package/src/opencode/agents/coder.md +0 -8
- package/src/opencode/agents/pm.md +2 -1
- package/src/opencode/commands/archive.md +0 -17
- package/src/opencode/commands/code-review.md +0 -24
- package/src/opencode/commands/dev-story.md +0 -24
- package/src/opencode/commands/diagram.md +0 -26
- package/src/opencode/commands/quick.md +0 -12
- package/src/opencode/commands/research.md +0 -23
- package/src/opencode/commands/review-story.md +0 -24
- package/src/opencode/commands/search.md +0 -11
- package/src/opencode/commands/unit-docs.md +0 -25
- package/src/opencode/skills/architecture-validation/SKILL.md +1 -1
- package/src/opencode/skills/dev-story/SKILL.md +88 -99
- package/src/opencode/skills/jira-integration/SKILL.md +1 -1
- package/src/opencode/skills/prd-validation/SKILL.md +2 -2
- package/src/opencode/skills/requirements-validation/SKILL.md +2 -2
- package/src/opencode/skills/sprint-planning/SKILL.md +8 -8
- package/src/opencode/skills/translation/SKILL.md +1 -1
package/package.json
CHANGED
package/src/build-info.json
CHANGED
|
@@ -17,7 +17,7 @@ tools:
|
|
|
17
17
|
list: true
|
|
18
18
|
skill: true
|
|
19
19
|
question: true
|
|
20
|
-
bash:
|
|
20
|
+
bash: true # For file operations
|
|
21
21
|
webfetch: true # For research
|
|
22
22
|
todowrite: true # Track complex requirements gathering
|
|
23
23
|
todoread: true
|
|
@@ -27,7 +27,13 @@ tools:
|
|
|
27
27
|
# Permissions - granular control
|
|
28
28
|
permission:
|
|
29
29
|
edit: allow # Can write documentation
|
|
30
|
-
bash:
|
|
30
|
+
bash:
|
|
31
|
+
"*": ask
|
|
32
|
+
"ls *": allow
|
|
33
|
+
"cat *": allow
|
|
34
|
+
"tree *": allow
|
|
35
|
+
"mkdir *": allow
|
|
36
|
+
"mv *": allow
|
|
31
37
|
---
|
|
32
38
|
|
|
33
39
|
<agent id="analyst" name="Sara" title="Business Analyst" icon="📊">
|
|
@@ -18,7 +18,7 @@ tools:
|
|
|
18
18
|
skill: true
|
|
19
19
|
question: true
|
|
20
20
|
bash: true # For codebase analysis
|
|
21
|
-
webfetch:
|
|
21
|
+
webfetch: true # For technical research
|
|
22
22
|
todowrite: true # Architecture can be complex multi-step
|
|
23
23
|
todoread: true
|
|
24
24
|
lsp: true # Code intelligence for architecture analysis
|
|
@@ -39,6 +39,7 @@ permission:
|
|
|
39
39
|
"mkdir *": allow
|
|
40
40
|
"head *": allow
|
|
41
41
|
"tail *": allow
|
|
42
|
+
"mv *": allow
|
|
42
43
|
---
|
|
43
44
|
|
|
44
45
|
<agent id="architect" name="Winston" title="Solution Architect" icon="🏗️">
|
|
@@ -56,14 +56,6 @@ permission:
|
|
|
56
56
|
</step>
|
|
57
57
|
<step n="8">Report completion or errors</step>
|
|
58
58
|
|
|
59
|
-
<lint-commands hint="Common linter commands">
|
|
60
|
-
<js>npx eslint --fix {files} OR npx biome check --write {files}</js>
|
|
61
|
-
<ts>npx eslint --fix {files} OR npx tsc --noEmit</ts>
|
|
62
|
-
<go>gofmt -w {files} && golangci-lint run {files}</go>
|
|
63
|
-
<py>ruff check --fix {files} OR black {files}</py>
|
|
64
|
-
<rust>cargo fmt && cargo clippy --fix</rust>
|
|
65
|
-
</lint-commands>
|
|
66
|
-
|
|
67
59
|
<rules>
|
|
68
60
|
<r>DO NOT ask clarifying questions - execute or fail</r>
|
|
69
61
|
<r>DO NOT refactor beyond task scope</r>
|
|
@@ -20,7 +20,7 @@ tools:
|
|
|
20
20
|
skill: true
|
|
21
21
|
question: true
|
|
22
22
|
bash: true # Limited bash for directory ops
|
|
23
|
-
webfetch:
|
|
23
|
+
webfetch: true # For product research
|
|
24
24
|
todowrite: true # PM tracks complex planning tasks
|
|
25
25
|
todoread: true
|
|
26
26
|
search: true # PREFERRED: Semantic search for docs/PRD/architecture
|
|
@@ -35,6 +35,7 @@ permission:
|
|
|
35
35
|
"cat *": allow
|
|
36
36
|
"tree *": allow
|
|
37
37
|
"mkdir *": allow
|
|
38
|
+
"mv *": allow
|
|
38
39
|
"git branch*": allow
|
|
39
40
|
"git status": allow
|
|
40
41
|
"git log*": allow
|
|
@@ -5,23 +5,6 @@ agent: pm
|
|
|
5
5
|
|
|
6
6
|
# Archive Command
|
|
7
7
|
|
|
8
|
-
## Arguments
|
|
9
|
-
$ARGUMENTS
|
|
10
|
-
|
|
11
|
-
Format: `[action] [path]`
|
|
12
|
-
|
|
13
|
-
**Actions:**
|
|
14
|
-
- "move [path]" - Archive a specific document
|
|
15
|
-
- "sprint [sprint-N]" - Archive completed sprint
|
|
16
|
-
- "list" - Show archive contents
|
|
17
|
-
- "find [query]" - Search archive
|
|
18
|
-
|
|
19
|
-
**Examples:**
|
|
20
|
-
- `/archive move docs/research/technical/old-topic.md`
|
|
21
|
-
- `/archive sprint sprint-0`
|
|
22
|
-
- `/archive list`
|
|
23
|
-
- `/archive find authentication`
|
|
24
|
-
|
|
25
8
|
## Archive Policy
|
|
26
9
|
|
|
27
10
|
**CRITICAL RULES:**
|
|
@@ -7,20 +7,6 @@ agent: dev
|
|
|
7
7
|
|
|
8
8
|
Review implemented code for quality, correctness, and adherence to standards.
|
|
9
9
|
|
|
10
|
-
## Usage
|
|
11
|
-
|
|
12
|
-
```
|
|
13
|
-
/code-review [story-path]
|
|
14
|
-
```
|
|
15
|
-
|
|
16
|
-
## Arguments
|
|
17
|
-
|
|
18
|
-
- `story-path` (optional): Path to story file to review. If not provided, finds stories in `review` status.
|
|
19
|
-
|
|
20
|
-
## Agent
|
|
21
|
-
|
|
22
|
-
This command invokes the **Dev** agent (Rick) in review mode.
|
|
23
|
-
|
|
24
10
|
## Process
|
|
25
11
|
|
|
26
12
|
1. Load story file and implementation
|
|
@@ -78,16 +64,6 @@ Updates story file with:
|
|
|
78
64
|
- [ ] No SQL injection risks
|
|
79
65
|
- [ ] Proper authentication/authorization
|
|
80
66
|
|
|
81
|
-
## Example
|
|
82
|
-
|
|
83
|
-
```bash
|
|
84
|
-
# Review stories in 'review' status
|
|
85
|
-
/code-review
|
|
86
|
-
|
|
87
|
-
# Review specific story
|
|
88
|
-
/code-review docs/sprint-artifacts/sprint-1/stories/1-1-user-auth.md
|
|
89
|
-
```
|
|
90
|
-
|
|
91
67
|
## After Review
|
|
92
68
|
|
|
93
69
|
- If **Approved**: Mark story as `done`
|
|
@@ -7,20 +7,6 @@ agent: dev
|
|
|
7
7
|
|
|
8
8
|
Implement a story using red-green-refactor cycle with TODO tracking.
|
|
9
9
|
|
|
10
|
-
## Usage
|
|
11
|
-
|
|
12
|
-
```
|
|
13
|
-
/dev-story [story-path]
|
|
14
|
-
```
|
|
15
|
-
|
|
16
|
-
## Arguments
|
|
17
|
-
|
|
18
|
-
- `story-path` (optional): Path to specific story file. If not provided, finds next `ready-for-dev` story from sprint-status.yaml.
|
|
19
|
-
|
|
20
|
-
## Agent
|
|
21
|
-
|
|
22
|
-
This command invokes the **Dev** agent (Rick).
|
|
23
|
-
|
|
24
10
|
## Process
|
|
25
11
|
|
|
26
12
|
### Phase 1: Setup
|
|
@@ -100,16 +86,6 @@ The workflow will HALT and ask for input when:
|
|
|
100
86
|
- Required configuration is missing
|
|
101
87
|
- Ambiguous requirements need clarification
|
|
102
88
|
|
|
103
|
-
## Example
|
|
104
|
-
|
|
105
|
-
```bash
|
|
106
|
-
# Find next ready story automatically
|
|
107
|
-
/dev-story
|
|
108
|
-
|
|
109
|
-
# Implement specific story
|
|
110
|
-
/dev-story docs/sprint-artifacts/sprint-1/stories/1-1-user-auth.md
|
|
111
|
-
```
|
|
112
|
-
|
|
113
89
|
## Story Status Flow
|
|
114
90
|
|
|
115
91
|
```
|
|
@@ -5,32 +5,6 @@ agent: architect
|
|
|
5
5
|
|
|
6
6
|
# Diagram Creation
|
|
7
7
|
|
|
8
|
-
## Arguments
|
|
9
|
-
$ARGUMENTS
|
|
10
|
-
|
|
11
|
-
Format: `[scope] [type]`
|
|
12
|
-
|
|
13
|
-
**Scope:**
|
|
14
|
-
- "system" - System-wide diagram
|
|
15
|
-
- "[module-name]" - Module-specific diagram
|
|
16
|
-
|
|
17
|
-
**Type:**
|
|
18
|
-
- "c4-context" - C4 Level 1: System context
|
|
19
|
-
- "c4-container" - C4 Level 2: Containers/services
|
|
20
|
-
- "c4-component" - C4 Level 3: Module components
|
|
21
|
-
- "sequence [flow-name]" - Sequence diagram for a flow
|
|
22
|
-
- "erd" - Entity relationship diagram
|
|
23
|
-
- "flow [process-name]" - Flowchart
|
|
24
|
-
- "state [entity-name]" - State machine
|
|
25
|
-
|
|
26
|
-
**Examples:**
|
|
27
|
-
- `/diagram system c4-context`
|
|
28
|
-
- `/diagram catalog c4-component`
|
|
29
|
-
- `/diagram catalog sequence product-import`
|
|
30
|
-
- `/diagram catalog erd`
|
|
31
|
-
- `/diagram order flow checkout-process`
|
|
32
|
-
- `/diagram offer state lifecycle`
|
|
33
|
-
|
|
34
8
|
## Check Existing Diagrams
|
|
35
9
|
|
|
36
10
|
!`ls -la docs/diagrams/ 2>/dev/null || echo "No diagrams yet"`
|
|
@@ -13,18 +13,6 @@ Skip the full workflow for small changes (L0-L1 scale):
|
|
|
13
13
|
- Configuration changes
|
|
14
14
|
- Small refactors
|
|
15
15
|
|
|
16
|
-
## Arguments
|
|
17
|
-
$ARGUMENTS
|
|
18
|
-
|
|
19
|
-
Description of the change to make.
|
|
20
|
-
|
|
21
|
-
Example:
|
|
22
|
-
```
|
|
23
|
-
/quick Fix login button not working on mobile
|
|
24
|
-
/quick Add dark mode toggle to settings
|
|
25
|
-
/quick Update API endpoint for user service
|
|
26
|
-
```
|
|
27
|
-
|
|
28
16
|
## Quick Mode Criteria
|
|
29
17
|
|
|
30
18
|
Automatically used when:
|
|
@@ -5,29 +5,6 @@ agent: researcher
|
|
|
5
5
|
|
|
6
6
|
# Research Command
|
|
7
7
|
|
|
8
|
-
## Arguments
|
|
9
|
-
$ARGUMENTS
|
|
10
|
-
|
|
11
|
-
Format: `[type] [topic]`
|
|
12
|
-
|
|
13
|
-
**Types:**
|
|
14
|
-
- "technical" - Technology evaluation
|
|
15
|
-
- "market" - Market/competitor analysis
|
|
16
|
-
- "domain" - Business domain research
|
|
17
|
-
- "integration" - External system/API research
|
|
18
|
-
- "pattern" - Architecture/design patterns
|
|
19
|
-
|
|
20
|
-
**Examples:**
|
|
21
|
-
- `/research technical message-queues`
|
|
22
|
-
- `/research market competitor-analysis`
|
|
23
|
-
- `/research domain e-commerce-regulations`
|
|
24
|
-
- `/research integration stripe-api`
|
|
25
|
-
- `/research pattern cqrs-event-sourcing`
|
|
26
|
-
|
|
27
|
-
**Other commands:**
|
|
28
|
-
- `/research list` - Show all research
|
|
29
|
-
- `/research update [path]` - Update existing research
|
|
30
|
-
|
|
31
8
|
## Check Existing Research
|
|
32
9
|
|
|
33
10
|
!`tree docs/research/ 2>/dev/null || echo "No research yet"`
|
|
@@ -7,20 +7,6 @@ agent: reviewer
|
|
|
7
7
|
|
|
8
8
|
Review a completed story using @reviewer agent (GPT-5.2 Codex) for deep security and quality analysis.
|
|
9
9
|
|
|
10
|
-
## Usage
|
|
11
|
-
|
|
12
|
-
```
|
|
13
|
-
/review-story [story-path]
|
|
14
|
-
```
|
|
15
|
-
|
|
16
|
-
## Arguments
|
|
17
|
-
|
|
18
|
-
- `story-path` (optional): Path to story file. If not provided, finds stories in `review` status.
|
|
19
|
-
|
|
20
|
-
## Agent
|
|
21
|
-
|
|
22
|
-
This command invokes the **Reviewer** agent (Marcus) with GPT-5.2 Codex model - best at finding bugs and security issues.
|
|
23
|
-
|
|
24
10
|
## When to Use
|
|
25
11
|
|
|
26
12
|
1. **After `/dev-story`** completes all tasks (auto-invoked if `auto_review: true`)
|
|
@@ -104,16 +90,6 @@ development:
|
|
|
104
90
|
auto_review: true # Auto-invoke @reviewer after /dev-story completes
|
|
105
91
|
```
|
|
106
92
|
|
|
107
|
-
## Example
|
|
108
|
-
|
|
109
|
-
```bash
|
|
110
|
-
# Review stories in 'review' status
|
|
111
|
-
/review-story
|
|
112
|
-
|
|
113
|
-
# Review specific story
|
|
114
|
-
/review-story docs/sprint-artifacts/sprint-1/stories/story-01-user-auth.md
|
|
115
|
-
```
|
|
116
|
-
|
|
117
93
|
## Flow with /dev-story
|
|
118
94
|
|
|
119
95
|
```
|
|
@@ -7,17 +7,6 @@ agent: crawler
|
|
|
7
7
|
|
|
8
8
|
Semantic search across the codebase using vectorizer.
|
|
9
9
|
|
|
10
|
-
## Arguments
|
|
11
|
-
$ARGUMENTS
|
|
12
|
-
|
|
13
|
-
## Examples
|
|
14
|
-
|
|
15
|
-
```
|
|
16
|
-
/search authentication middleware
|
|
17
|
-
/search how users are stored
|
|
18
|
-
/search error handling patterns
|
|
19
|
-
```
|
|
20
|
-
|
|
21
10
|
## Your Task
|
|
22
11
|
|
|
23
12
|
Perform **multiple semantic searches** to find the best results for user's query.
|
|
@@ -9,14 +9,6 @@ skills_loaded:
|
|
|
9
9
|
|
|
10
10
|
Create structured documentation for any logical piece of the system using the Universal Unit format.
|
|
11
11
|
|
|
12
|
-
## Arguments
|
|
13
|
-
|
|
14
|
-
$ARGUMENTS
|
|
15
|
-
|
|
16
|
-
- `[unit-type] [unit-name]`: Create unit doc (e.g., `entity Task`, `module catalog`)
|
|
17
|
-
- `list`: List existing unit docs
|
|
18
|
-
- `validate [unit-name]`: Validate unit doc completeness
|
|
19
|
-
|
|
20
12
|
## Supported Unit Types
|
|
21
13
|
|
|
22
14
|
| Type | When to Use | Example |
|
|
@@ -150,21 +142,4 @@ Suggest:
|
|
|
150
142
|
- `/unit-docs [type] [name]` - Document related units
|
|
151
143
|
- `/stories {epic-id}` - Reference unit in story tasks
|
|
152
144
|
|
|
153
|
-
## Examples
|
|
154
|
-
|
|
155
|
-
```bash
|
|
156
|
-
# Document a core entity
|
|
157
|
-
/unit-docs entity Task
|
|
158
145
|
|
|
159
|
-
# Document a service
|
|
160
|
-
/unit-docs service NotificationService
|
|
161
|
-
|
|
162
|
-
# Document a module
|
|
163
|
-
/unit-docs module catalog
|
|
164
|
-
|
|
165
|
-
# List all units
|
|
166
|
-
/unit-docs list
|
|
167
|
-
|
|
168
|
-
# Validate a unit
|
|
169
|
-
/unit-docs validate Task
|
|
170
|
-
```
|
|
@@ -11,123 +11,112 @@ metadata:
|
|
|
11
11
|
|
|
12
12
|
# Dev Story Skill
|
|
13
13
|
|
|
14
|
-
|
|
14
|
+
How to transform story tasks into executable instructions for @coder.
|
|
15
15
|
|
|
16
|
-
##
|
|
16
|
+
## Task Transformation
|
|
17
17
|
|
|
18
|
-
|
|
19
|
-
- **Respect task dependencies** - never start blocked tasks
|
|
20
|
-
- **Tests are MANDATORY** - each task has tests that MUST pass
|
|
21
|
-
- **NEVER lie about tests** - tests must actually exist and pass
|
|
22
|
-
- **For parallel execution** - call multiple @coder in one message
|
|
18
|
+
Story task is a specification. @coder needs executable instruction with full context.
|
|
23
19
|
|
|
24
|
-
|
|
20
|
+
### Add Context
|
|
25
21
|
|
|
26
|
-
|
|
27
|
-
|
|
28
|
-
|
|
29
|
-
|
|
22
|
+
@coder doesn't see the full story. Include:
|
|
23
|
+
- **Existing files** - actual paths to domain entities, interfaces, services that exist
|
|
24
|
+
- **Patterns to follow** - link to existing similar code ("follow pattern from X")
|
|
25
|
+
- **What was done** - results of previous tasks that this task depends on
|
|
26
|
+
- **Imports** - what packages to use
|
|
30
27
|
|
|
31
|
-
|
|
28
|
+
### Add Implementation Details
|
|
32
29
|
|
|
33
|
-
|
|
30
|
+
Story has Code Sketch. Expand it:
|
|
31
|
+
- Actual method signatures with types
|
|
32
|
+
- Error handling approach
|
|
33
|
+
- Logging requirements
|
|
34
|
+
- Validation rules
|
|
34
35
|
|
|
35
|
-
|
|
36
|
-
<phase name="DEPENDENCY_CHECK">
|
|
37
|
-
<action>Parse task summary table from story file</action>
|
|
38
|
-
<action>Read "Depends On" column for current task</action>
|
|
39
|
-
<check if="dependencies not complete">
|
|
40
|
-
<action>SKIP this task, find next ready task</action>
|
|
41
|
-
</check>
|
|
42
|
-
</phase>
|
|
43
|
-
```
|
|
36
|
+
### Add Verification
|
|
44
37
|
|
|
45
|
-
|
|
38
|
+
Story has "Done when". Make it concrete:
|
|
39
|
+
- Specific test commands to run
|
|
40
|
+
- Expected output format
|
|
41
|
+
- Files that must compile
|
|
46
42
|
|
|
47
|
-
|
|
48
|
-
<phase name="PARALLEL_CHECK">
|
|
49
|
-
<action>Identify tasks with same satisfied dependencies</action>
|
|
50
|
-
<check if="parallel tasks exist">
|
|
51
|
-
<action>Call multiple @coder in ONE message</action>
|
|
52
|
-
</check>
|
|
53
|
-
</phase>
|
|
54
|
-
```
|
|
43
|
+
## Formulating Task for @coder
|
|
55
44
|
|
|
56
|
-
|
|
57
|
-
|
|
58
|
-
```xml
|
|
59
|
-
<check if="methodology is TDD">
|
|
60
|
-
<phase name="RED">
|
|
61
|
-
<action>Write test for task deliverables</action>
|
|
62
|
-
<action>Run test - MUST FAIL</action>
|
|
63
|
-
</phase>
|
|
64
|
-
|
|
65
|
-
<phase name="GREEN">
|
|
66
|
-
<action>Implement MINIMAL code to pass test</action>
|
|
67
|
-
<action>Run test - MUST PASS</action>
|
|
68
|
-
</phase>
|
|
69
|
-
|
|
70
|
-
<phase name="REFACTOR">
|
|
71
|
-
<action>Improve code structure</action>
|
|
72
|
-
<action>Run test - MUST STILL PASS</action>
|
|
73
|
-
</phase>
|
|
74
|
-
</check>
|
|
45
|
+
Structure:
|
|
75
46
|
```
|
|
47
|
+
## Task: [Name] (Task IDs)
|
|
76
48
|
|
|
77
|
-
|
|
49
|
+
[One line goal]
|
|
78
50
|
|
|
79
|
-
|
|
80
|
-
|
|
81
|
-
<phase name="STUB">
|
|
82
|
-
<action>Write stub implementation (mock data)</action>
|
|
83
|
-
</phase>
|
|
51
|
+
### Skills
|
|
52
|
+
- Use `doc-todo` for TODO comments
|
|
84
53
|
|
|
85
|
-
|
|
86
|
-
|
|
87
|
-
|
|
88
|
-
|
|
54
|
+
### Context
|
|
55
|
+
- [Existing file]: [What it contains, what to use from it]
|
|
56
|
+
- [Another file]: [Description]
|
|
57
|
+
- Existing pattern: [Path to similar code to follow]
|
|
89
58
|
|
|
90
|
-
|
|
91
|
-
|
|
92
|
-
|
|
93
|
-
|
|
94
|
-
|
|
95
|
-
```
|
|
59
|
+
### Output Files
|
|
60
|
+
- [Path to create/modify]
|
|
61
|
+
|
|
62
|
+
### Requirements
|
|
63
|
+
[Numbered list of what to implement with signatures/details]
|
|
96
64
|
|
|
97
|
-
|
|
98
|
-
|
|
99
|
-
|
|
100
|
-
|
|
101
|
-
|
|
102
|
-
|
|
103
|
-
|
|
104
|
-
|
|
105
|
-
|
|
106
|
-
|
|
107
|
-
|
|
108
|
-
<action>Mark task [x] in story file</action>
|
|
109
|
-
<action>Update File List</action>
|
|
110
|
-
</check>
|
|
111
|
-
|
|
112
|
-
<check if="ANY fails">
|
|
113
|
-
<action>DO NOT mark task complete</action>
|
|
114
|
-
<action>Fix issues first</action>
|
|
115
|
-
</check>
|
|
65
|
+
### Implementation Pattern
|
|
66
|
+
[Code example or reference to existing code]
|
|
67
|
+
|
|
68
|
+
### Error Handling
|
|
69
|
+
[How to handle errors]
|
|
70
|
+
|
|
71
|
+
### Done When
|
|
72
|
+
- [ ] [Specific checkable item]
|
|
73
|
+
- [ ] [Another item]
|
|
74
|
+
- [ ] File compiles
|
|
75
|
+
- [ ] Tests pass
|
|
116
76
|
```
|
|
117
77
|
|
|
118
|
-
##
|
|
78
|
+
## Parallel Tasks
|
|
79
|
+
|
|
80
|
+
When delegating multiple tasks in one message:
|
|
81
|
+
1. Each task gets full context (don't assume @coder remembers)
|
|
82
|
+
2. Clearly separate tasks with headers
|
|
83
|
+
3. No shared state between parallel tasks
|
|
84
|
+
4. Tasks must work on different files
|
|
85
|
+
|
|
86
|
+
## Task Boundaries
|
|
87
|
+
|
|
88
|
+
**Good task:**
|
|
89
|
+
- Logically complete unit (service, handler, entity, feature)
|
|
90
|
+
- Clear single responsibility
|
|
91
|
+
- Can be tested independently
|
|
92
|
+
|
|
93
|
+
**Consider splitting when:**
|
|
94
|
+
- Multiple unrelated responsibilities
|
|
95
|
+
- No logical connection between parts
|
|
96
|
+
|
|
97
|
+
**Consider combining when:**
|
|
98
|
+
- Tasks too granular to be meaningful
|
|
99
|
+
- Same file, same concern
|
|
100
|
+
|
|
101
|
+
## Common Patterns
|
|
102
|
+
|
|
103
|
+
### New Service
|
|
104
|
+
Context: domain entities, repository interface, existing service example
|
|
105
|
+
Requirements: interface, constructor, methods
|
|
106
|
+
Pattern: existing service structure
|
|
107
|
+
|
|
108
|
+
### New Handler
|
|
109
|
+
Context: service interface, DTOs, existing handler example
|
|
110
|
+
Requirements: handler struct, methods, error mapping
|
|
111
|
+
Pattern: existing handler structure
|
|
119
112
|
|
|
120
|
-
|
|
121
|
-
|
|
122
|
-
|
|
123
|
-
|
|
113
|
+
### New Tests
|
|
114
|
+
Context: code to test, existing test examples
|
|
115
|
+
Requirements: test scenarios, mocks
|
|
116
|
+
Pattern: existing test structure
|
|
124
117
|
|
|
125
|
-
##
|
|
118
|
+
## Methodology
|
|
126
119
|
|
|
127
|
-
|
|
128
|
-
-
|
|
129
|
-
-
|
|
130
|
-
- [ ] Integration tests (if required)
|
|
131
|
-
- [ ] All tests pass
|
|
132
|
-
- [ ] File List complete
|
|
133
|
-
- [ ] Change Log updated
|
|
120
|
+
Include in task based on config.yaml:
|
|
121
|
+
- **TDD**: "Write failing test first, then implement"
|
|
122
|
+
- **STUB**: "Create stub first, write tests, then real implementation"
|
|
@@ -362,7 +362,7 @@ Agent manages the full development lifecycle:
|
|
|
362
362
|
| From | To | Trigger | Agent Action |
|
|
363
363
|
|------|----|---------|--------------|
|
|
364
364
|
| backlog | planned | Sprint planning | Update Jira status |
|
|
365
|
-
| planned | in_progress |
|
|
365
|
+
| planned | in_progress | Dev starts work | Create branch, update status |
|
|
366
366
|
| in_progress | review | PR created | Link PR, update status |
|
|
367
367
|
| review | in_progress | Review failed | Update status, add comment |
|
|
368
368
|
| review | qa | Review passed | Update status |
|
|
@@ -126,8 +126,8 @@ For each FR in PRD:
|
|
|
126
126
|
## Next Steps
|
|
127
127
|
|
|
128
128
|
After fixing issues:
|
|
129
|
-
- Re-run
|
|
130
|
-
- If PASS, proceed to
|
|
129
|
+
- Re-run validation
|
|
130
|
+
- If PASS, proceed to architecture design
|
|
131
131
|
```
|
|
132
132
|
|
|
133
133
|
## If QA Artifact Missing
|
|
@@ -28,22 +28,22 @@ Check: ls docs/sprint-artifacts/backlog/epic-*.md 2>/dev/null || ls docs/sprint-
|
|
|
28
28
|
|
|
29
29
|
| Check | Action if Missing |
|
|
30
30
|
|-------|-------------------|
|
|
31
|
-
| No epics in backlog or sprints | **STOP** →
|
|
32
|
-
| Epics exist but no stories | Epics can be planned, stories created later
|
|
33
|
-
| PRD missing | **STOP** →
|
|
31
|
+
| No epics in backlog or sprints | **STOP** → Create epics from PRD first |
|
|
32
|
+
| Epics exist but no stories | Epics can be planned, stories created later |
|
|
33
|
+
| PRD missing | **STOP** → Create PRD first |
|
|
34
34
|
|
|
35
35
|
### Dependency Chain
|
|
36
36
|
|
|
37
37
|
```
|
|
38
|
-
|
|
39
|
-
|
|
40
|
-
|
|
41
|
-
|
|
38
|
+
PRD → Architecture → Epics → Sprint Plan
|
|
39
|
+
↑
|
|
40
|
+
YOU ARE HERE
|
|
41
|
+
(need epics first!)
|
|
42
42
|
```
|
|
43
43
|
|
|
44
44
|
If user asks for sprint planning without epics:
|
|
45
45
|
1. Inform them epics are required
|
|
46
|
-
2. Offer to create epics first
|
|
46
|
+
2. Offer to create epics first
|
|
47
47
|
3. Only proceed with sprint planning after epics exist
|
|
48
48
|
|
|
49
49
|
## Sprint Duration
|