@comfanion/workflow 4.36.41 → 4.36.42
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/README.md +1 -1
- package/package.json +1 -1
- package/src/build-info.json +2 -2
- package/src/opencode/FLOW.yaml +1 -1
- package/src/opencode/agents/architect.md +7 -5
- package/src/opencode/checklists/architecture-checklist.md +4 -5
- package/src/opencode/checklists/code-review-checklist.md +4 -4
- package/src/opencode/commands/architecture.md +1 -1
- package/src/opencode/opencode.json +1 -1
- package/src/opencode/skills/adr-writing/SKILL.md +2 -1
- package/src/opencode/skills/architecture-design/template.md +3 -3
- package/src/opencode/skills/code-review/SKILL.md +3 -3
- package/src/opencode/skills/coding-standards/SKILL.md +5 -6
- package/src/opencode/skills/diagram-creation/SKILL.md +1 -1
- package/src/opencode/skills/prd-writing/template.md +1 -1
- package/src/opencode/skills/story-writing/template.md +4 -4
package/README.md
CHANGED
|
@@ -38,7 +38,7 @@ Development: /dev-story ↔ /code-review (loop until done)
|
|
|
38
38
|
|
|
39
39
|
- **requirements-gathering** - Extract FR/NFR through interviews
|
|
40
40
|
- **prd-writing** - Product requirements documents
|
|
41
|
-
- **architecture-design** -
|
|
41
|
+
- **architecture-design** - System architecture patterns
|
|
42
42
|
- **story-writing** - User stories with Given/When/Then AC
|
|
43
43
|
- **dev-story** - Red-green-refactor implementation cycle
|
|
44
44
|
- **jira-integration** - Bidirectional sync with Jira
|
package/package.json
CHANGED
package/src/build-info.json
CHANGED
package/src/opencode/FLOW.yaml
CHANGED
|
@@ -370,7 +370,7 @@ agents:
|
|
|
370
370
|
file: agents/architect.md
|
|
371
371
|
expertise:
|
|
372
372
|
- Distributed systems
|
|
373
|
-
-
|
|
373
|
+
- Architecture patterns (chooses based on context)
|
|
374
374
|
- System design
|
|
375
375
|
personality: Technical, precise, patterns-focused
|
|
376
376
|
skills_used:
|
|
@@ -109,7 +109,7 @@ permission:
|
|
|
109
109
|
|
|
110
110
|
<persona>
|
|
111
111
|
<role>System Architect + Technical Design Leader</role>
|
|
112
|
-
<identity>Senior architect with expertise in distributed systems, cloud infrastructure, API design.
|
|
112
|
+
<identity>Senior architect with expertise in distributed systems, cloud infrastructure, API design. Chooses architecture patterns based on project needs, not dogma.</identity>
|
|
113
113
|
<communication_style>Calm, pragmatic, balancing 'what could be' with 'what should be.' Technical but accessible. Always considers trade-offs.</communication_style>
|
|
114
114
|
<principles>
|
|
115
115
|
- Channel expert lean architecture: distributed systems, cloud patterns, scalability
|
|
@@ -129,12 +129,14 @@ permission:
|
|
|
129
129
|
</skills>
|
|
130
130
|
|
|
131
131
|
<design-principles>
|
|
132
|
-
1.
|
|
132
|
+
1. Right Pattern for Context - Choose architecture style based on project needs (see architecture-design skill)
|
|
133
133
|
2. Single Responsibility - Each module has one job
|
|
134
134
|
3. Explicit Contracts - Clear interfaces between modules
|
|
135
|
-
4.
|
|
136
|
-
5.
|
|
137
|
-
6.
|
|
135
|
+
4. Separation of Concerns - Isolate business logic from infrastructure
|
|
136
|
+
5. Event-Driven Where Appropriate - Loose coupling via events when justified
|
|
137
|
+
6. Idempotency - Operations should be safely retryable
|
|
138
|
+
7. Observability First - Design for debugging and monitoring
|
|
139
|
+
8. Trade-off Aware - Document decisions with pros/cons in ADRs
|
|
138
140
|
</design-principles>
|
|
139
141
|
|
|
140
142
|
<documentation-structure hint="For unit-writing skill">
|
|
@@ -56,8 +56,8 @@ Use this checklist to validate architecture before proceeding to epic/story crea
|
|
|
56
56
|
## Design Quality
|
|
57
57
|
|
|
58
58
|
### Architecture Patterns
|
|
59
|
-
- [ ] Chosen
|
|
60
|
-
- [ ] Pattern choice justified
|
|
59
|
+
- [ ] Chosen pattern documented (Layered, Hexagonal, Clean, Microservices, etc.)
|
|
60
|
+
- [ ] Pattern choice justified with ADR
|
|
61
61
|
- [ ] Consistent pattern application
|
|
62
62
|
|
|
63
63
|
### Module Design
|
|
@@ -93,9 +93,8 @@ Use this checklist to validate architecture before proceeding to epic/story crea
|
|
|
93
93
|
|
|
94
94
|
## Coding Standards Alignment
|
|
95
95
|
|
|
96
|
-
- [ ] Architecture follows
|
|
97
|
-
- [ ]
|
|
98
|
-
- [ ] Use case pattern applicable
|
|
96
|
+
- [ ] Architecture follows AGENTS.md patterns
|
|
97
|
+
- [ ] Chosen architecture pattern consistently applied
|
|
99
98
|
- [ ] Testing strategy fits architecture
|
|
100
99
|
|
|
101
100
|
## ADR Quality
|
|
@@ -18,11 +18,11 @@ Use this checklist when reviewing implemented code before marking story as "done
|
|
|
18
18
|
## Code Quality
|
|
19
19
|
|
|
20
20
|
### Architecture Compliance
|
|
21
|
-
- [ ] Follows
|
|
22
|
-
- [ ]
|
|
23
|
-
- [ ]
|
|
21
|
+
- [ ] Follows project's chosen architecture pattern (AGENTS.md)
|
|
22
|
+
- [ ] Business logic isolated from infrastructure
|
|
23
|
+
- [ ] Clear separation of concerns
|
|
24
24
|
- [ ] Explicit mapping (no reflection libraries)
|
|
25
|
-
- [ ] Dependencies flow correctly
|
|
25
|
+
- [ ] Dependencies flow correctly per chosen pattern
|
|
26
26
|
|
|
27
27
|
### Code Structure
|
|
28
28
|
- [ ] Single responsibility principle followed
|
|
@@ -31,7 +31,7 @@ author: {{author}}
|
|
|
31
31
|
<!-- e.g.
|
|
32
32
|
TaskFlow is a task management platform for distributed teams.
|
|
33
33
|
|
|
34
|
-
**Architecture Pattern:** Modular Monolith
|
|
34
|
+
**Architecture Pattern:** Modular Monolith — enables independent scaling while keeping deployment simple (pattern chosen based on team size and scale requirements)
|
|
35
35
|
|
|
36
36
|
**Key Domains:**
|
|
37
37
|
1. **Task Domain** (2 modules) — Task CRUD, assignments, status workflow
|
|
@@ -53,8 +53,8 @@ TaskFlow is a task management platform for distributed teams.
|
|
|
53
53
|
| Communication | {{decision}} | {{why}} |
|
|
54
54
|
|
|
55
55
|
<!-- e.g.
|
|
56
|
-
| Architecture |
|
|
57
|
-
| Database | PostgreSQL
|
|
56
|
+
| Architecture | Modular Monolith | Team size, deployment simplicity |
|
|
57
|
+
| Database | PostgreSQL | ACID needed, team expertise |
|
|
58
58
|
| Communication | REST sync + Kafka async | REST for queries, Kafka for events |
|
|
59
59
|
-->
|
|
60
60
|
|
|
@@ -28,9 +28,9 @@ For each AC in the story:
|
|
|
28
28
|
### 3. Code Quality Review
|
|
29
29
|
|
|
30
30
|
#### Architecture Compliance
|
|
31
|
-
- [ ] Follows
|
|
32
|
-
- [ ]
|
|
33
|
-
- [ ]
|
|
31
|
+
- [ ] Follows project's chosen architecture pattern (see AGENTS.md)
|
|
32
|
+
- [ ] Business logic isolated from infrastructure
|
|
33
|
+
- [ ] Clear separation of concerns
|
|
34
34
|
- [ ] Explicit mapping (no reflection libraries)
|
|
35
35
|
|
|
36
36
|
#### Code Structure
|
|
@@ -58,7 +58,7 @@ docs/coding-standards/
|
|
|
58
58
|
|
|
59
59
|
| Topic | Document | Key Rules |
|
|
60
60
|
|-------|----------|-----------|
|
|
61
|
-
| Project Layout | [project-structure.md](./project-structure.md) |
|
|
61
|
+
| Project Layout | [project-structure.md](./project-structure.md) | Modules, structure |
|
|
62
62
|
| Go Code | [go-standards.md](./go-standards.md) | Naming, errors, imports |
|
|
63
63
|
| Testing | [testing-standards.md](./testing-standards.md) | 80% coverage, table-driven |
|
|
64
64
|
| API | [api-standards.md](./api-standards.md) | REST, error format |
|
|
@@ -150,13 +150,12 @@ project/
|
|
|
150
150
|
```markdown
|
|
151
151
|
# Architecture Patterns
|
|
152
152
|
|
|
153
|
-
##
|
|
153
|
+
## Chosen Pattern
|
|
154
154
|
|
|
155
|
-
### Hexagonal
|
|
156
|
-
[Description,
|
|
155
|
+
### [Pattern Name] (e.g., Layered, Hexagonal, Clean, Vertical Slices)
|
|
156
|
+
[Description, why chosen for this project, example structure]
|
|
157
157
|
|
|
158
|
-
|
|
159
|
-
[4-file structure, example]
|
|
158
|
+
See architecture-design skill for pattern selection guidance.
|
|
160
159
|
|
|
161
160
|
### Repository Pattern
|
|
162
161
|
[Interface in domain, implementation in infrastructure]
|
|
@@ -31,7 +31,7 @@ author: {{author}}
|
|
|
31
31
|
<!-- e.g.
|
|
32
32
|
TaskFlow is a B2B platform for managing distributed teams. The system handles task management, real-time collaboration, and team analytics.
|
|
33
33
|
|
|
34
|
-
**Architecture:** Modular Monolith
|
|
34
|
+
**Architecture:** Modular Monolith (pattern chosen based on team size and requirements)
|
|
35
35
|
|
|
36
36
|
**Key Domains:**
|
|
37
37
|
1. **Task Management** — CRUD, assignments, status workflow
|
|
@@ -19,7 +19,7 @@ size: S | M | L
|
|
|
19
19
|
- {{what_this_story_does_NOT_do}}
|
|
20
20
|
|
|
21
21
|
<!-- e.g.
|
|
22
|
-
Implement CRUD operations for Task entity following
|
|
22
|
+
Implement CRUD operations for Task entity following project architecture.
|
|
23
23
|
|
|
24
24
|
**Context:** This story is part of Epic 1 (Task Management Core). It focuses on the domain layer and repository.
|
|
25
25
|
|
|
@@ -52,8 +52,8 @@ Implement CRUD operations for Task entity following hexagonal architecture.
|
|
|
52
52
|
| → Epic: `{{epic_path}}` | Technical Decisions | ADRs for this epic |
|
|
53
53
|
|
|
54
54
|
<!-- e.g.
|
|
55
|
-
| → `
|
|
56
|
-
| → `docs/coding-standards/` | All | **MANDATORY** —
|
|
55
|
+
| → `AGENTS.md` | All | Project patterns, naming |
|
|
56
|
+
| → `docs/coding-standards/` | All | **MANDATORY** — architecture, error handling |
|
|
57
57
|
| → Unit: `Task` | Data Model, Operations | Fields, validation rules |
|
|
58
58
|
| → `docs/architecture.md` | Task Module | Internal services, events |
|
|
59
59
|
| → `docs/architecture.md#error-handling` | Error Handling | Error codes format |
|
|
@@ -76,7 +76,7 @@ Story is complete when:
|
|
|
76
76
|
- [ ] Empty title returns validation error (400, TASK_001)
|
|
77
77
|
- [ ] Task ID is UUID format
|
|
78
78
|
- [ ] Tests cover: create, validation, duplicate handling
|
|
79
|
-
- [ ] Code follows
|
|
79
|
+
- [ ] Code follows architecture patterns from coding-standards
|
|
80
80
|
-->
|
|
81
81
|
|
|
82
82
|
---
|