codex-workflows 0.4.11 → 0.6.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/.agents/skills/coding-rules/references/typescript.md +1 -1
- package/.agents/skills/documentation-criteria/references/design-template.md +8 -0
- package/.agents/skills/documentation-criteria/references/plan-template.md +22 -3
- package/.agents/skills/documentation-criteria/references/task-template.md +1 -1
- package/.agents/skills/documentation-criteria/references/ui-spec-template.md +10 -0
- package/.agents/skills/external-resource-context/SKILL.md +99 -0
- package/.agents/skills/external-resource-context/agents/openai.yaml +7 -0
- package/.agents/skills/external-resource-context/references/api.md +20 -0
- package/.agents/skills/external-resource-context/references/backend.md +21 -0
- package/.agents/skills/external-resource-context/references/frontend.md +21 -0
- package/.agents/skills/external-resource-context/references/infra.md +21 -0
- package/.agents/skills/external-resource-context/references/template.md +72 -0
- package/.agents/skills/integration-e2e-testing/SKILL.md +34 -21
- package/.agents/skills/integration-e2e-testing/references/e2e-design.md +16 -10
- package/.agents/skills/recipe-add-integration-tests/SKILL.md +7 -0
- package/.agents/skills/recipe-build/SKILL.md +32 -5
- package/.agents/skills/recipe-front-adjust/SKILL.md +113 -0
- package/.agents/skills/recipe-front-adjust/agents/openai.yaml +7 -0
- package/.agents/skills/recipe-front-build/SKILL.md +32 -5
- package/.agents/skills/recipe-front-design/SKILL.md +28 -9
- package/.agents/skills/recipe-front-plan/SKILL.md +1 -1
- package/.agents/skills/recipe-front-review/SKILL.md +29 -11
- package/.agents/skills/recipe-fullstack-build/SKILL.md +32 -5
- package/.agents/skills/recipe-fullstack-implement/SKILL.md +13 -4
- package/.agents/skills/recipe-implement/SKILL.md +12 -4
- package/.agents/skills/recipe-plan/SKILL.md +5 -5
- package/.agents/skills/recipe-prepare-implementation/SKILL.md +162 -0
- package/.agents/skills/recipe-prepare-implementation/agents/openai.yaml +7 -0
- package/.agents/skills/recipe-review/SKILL.md +34 -6
- package/.agents/skills/subagents-orchestration-guide/SKILL.md +36 -34
- package/.agents/skills/subagents-orchestration-guide/references/monorepo-flow.md +45 -48
- package/.agents/skills/task-analyzer/SKILL.md +3 -2
- package/.agents/skills/task-analyzer/references/skills-index.yaml +54 -7
- package/.agents/skills/testing/references/typescript.md +2 -3
- package/.codex/agents/acceptance-test-generator.toml +69 -31
- package/.codex/agents/quality-fixer-frontend.toml +5 -0
- package/.codex/agents/quality-fixer.toml +5 -0
- package/.codex/agents/task-decomposer.toml +27 -2
- package/.codex/agents/task-executor-frontend.toml +16 -11
- package/.codex/agents/task-executor.toml +19 -14
- package/.codex/agents/technical-designer-frontend.toml +25 -2
- package/.codex/agents/technical-designer.toml +13 -0
- package/.codex/agents/ui-analyzer.toml +307 -0
- package/.codex/agents/ui-spec-designer.toml +15 -0
- package/.codex/agents/work-planner.toml +54 -17
- package/README.md +54 -26
- package/package.json +1 -1
|
@@ -4,7 +4,7 @@
|
|
|
4
4
|
skills:
|
|
5
5
|
coding-rules:
|
|
6
6
|
skill: "coding-rules"
|
|
7
|
-
tags: [implementation, code-quality, refactoring, clean-code, maintainability, function-design, error-handling, parameterized-dependencies, performance, security]
|
|
7
|
+
tags: [implementation, code-quality, refactoring, clean-code, maintainability, function-design, error-handling, parameterized-dependencies, reference-representativeness, performance, security]
|
|
8
8
|
typical-use: "Language-agnostic code creation, modification, and refactoring principles applicable to all programming languages"
|
|
9
9
|
size: medium
|
|
10
10
|
key-references:
|
|
@@ -14,23 +14,26 @@ skills:
|
|
|
14
14
|
- "Refactoring - Martin Fowler"
|
|
15
15
|
- "Single Responsibility Principle - SOLID"
|
|
16
16
|
sections:
|
|
17
|
+
- "Language-Specific References"
|
|
17
18
|
- "Core Philosophy [MANDATORY]"
|
|
18
19
|
- "Code Quality [MANDATORY]"
|
|
19
20
|
- "Function Design [MANDATORY]"
|
|
20
21
|
- "Error Handling [MANDATORY]"
|
|
21
22
|
- "Dependency Management"
|
|
23
|
+
- "Reference Representativeness"
|
|
22
24
|
- "Performance"
|
|
23
25
|
- "Code Organization"
|
|
24
26
|
- "Commenting Principles"
|
|
25
27
|
- "Refactoring [SAFE CHANGE PROTOCOL]"
|
|
26
|
-
- "Security
|
|
28
|
+
- "Security"
|
|
27
29
|
- "Version Control [MANDATORY]"
|
|
28
30
|
references:
|
|
29
31
|
- "references/typescript.md"
|
|
32
|
+
- "references/security-checks.md"
|
|
30
33
|
|
|
31
34
|
testing:
|
|
32
35
|
skill: "testing"
|
|
33
|
-
tags: [testing, tdd, quality, unit-testing, integration-testing, e2e-testing, test-design, coverage, mocking, test-independence, ci-cd, test-quality-criteria]
|
|
36
|
+
tags: [testing, tdd, quality, unit-testing, integration-testing, e2e-testing, data-layer-testing, test-design, coverage, mocking, test-independence, ci-cd, test-quality-criteria]
|
|
34
37
|
typical-use: "Universal testing principles, TDD practice, test quality criteria, test creation and quality assurance for all programming languages"
|
|
35
38
|
size: large
|
|
36
39
|
key-references:
|
|
@@ -39,6 +42,7 @@ skills:
|
|
|
39
42
|
- "AAA Pattern - Arrange-Act-Assert"
|
|
40
43
|
- "Test Pyramid - Mike Cohn"
|
|
41
44
|
sections:
|
|
45
|
+
- "Language-Specific References"
|
|
42
46
|
- "Core Testing Philosophy"
|
|
43
47
|
- "TDD Process [MANDATORY for all code changes]"
|
|
44
48
|
- "Quality Requirements [MANDATORY]"
|
|
@@ -46,6 +50,7 @@ skills:
|
|
|
46
50
|
- "Test Design Principles"
|
|
47
51
|
- "Test Independence"
|
|
48
52
|
- "Mocking and Test Doubles"
|
|
53
|
+
- "Data Layer Testing"
|
|
49
54
|
- "Test Quality Practices [MANDATORY]"
|
|
50
55
|
- "What to Test"
|
|
51
56
|
- "Test Quality Criteria [MANDATORY]"
|
|
@@ -68,19 +73,23 @@ skills:
|
|
|
68
73
|
- "DRY Principle - The Pragmatic Programmer"
|
|
69
74
|
- "YAGNI Principle - Extreme Programming"
|
|
70
75
|
sections:
|
|
76
|
+
- "Language-Specific References"
|
|
71
77
|
- "Technical Anti-patterns (Red Flag Patterns) [MANDATORY]"
|
|
72
78
|
- "Fail-Fast Fallback Design Principles"
|
|
73
79
|
- "Rule of Three - Criteria for Code Duplication"
|
|
74
80
|
- "Common Failure Patterns and Avoidance Methods"
|
|
75
81
|
- "Debugging Techniques"
|
|
82
|
+
- "Quality Assurance Mechanism Awareness"
|
|
76
83
|
- "Quality Check Workflow [MANDATORY]"
|
|
77
84
|
- "Situations Requiring Technical Decisions"
|
|
78
85
|
- "Implementation Completeness Assurance"
|
|
79
86
|
- "Impact Analysis"
|
|
87
|
+
references:
|
|
88
|
+
- "references/frontend.md"
|
|
80
89
|
|
|
81
90
|
documentation-criteria:
|
|
82
91
|
skill: "documentation-criteria"
|
|
83
|
-
tags: [documentation, decision-making, adr, prd, design-doc, planning, process, scale-assessment]
|
|
92
|
+
tags: [documentation, decision-making, adr, prd, design-doc, ui-spec, work-plan, templates, planning, process, scale-assessment]
|
|
84
93
|
typical-use: "Scale assessment at implementation start, document creation criteria, ADR/PRD/Design Doc/Work Plan creation standards"
|
|
85
94
|
size: medium
|
|
86
95
|
key-references:
|
|
@@ -98,6 +107,13 @@ skills:
|
|
|
98
107
|
- "AI Automation Rules [MANDATORY]"
|
|
99
108
|
- "Diagram Requirements"
|
|
100
109
|
- "Common ADR Relationships"
|
|
110
|
+
references:
|
|
111
|
+
- "references/prd-template.md"
|
|
112
|
+
- "references/adr-template.md"
|
|
113
|
+
- "references/ui-spec-template.md"
|
|
114
|
+
- "references/design-template.md"
|
|
115
|
+
- "references/plan-template.md"
|
|
116
|
+
- "references/task-template.md"
|
|
101
117
|
|
|
102
118
|
implementation-approach:
|
|
103
119
|
skill: "implementation-approach"
|
|
@@ -117,7 +133,7 @@ skills:
|
|
|
117
133
|
|
|
118
134
|
integration-e2e-testing:
|
|
119
135
|
skill: "integration-e2e-testing"
|
|
120
|
-
tags: [testing, integration-testing, e2e-testing, test-design, behavior-first, roi, test-skeleton, ears-format]
|
|
136
|
+
tags: [testing, integration-testing, e2e-testing, fixture-e2e, service-integration-e2e, e2e-absence-reason, test-design, behavior-first, roi, test-skeleton, ears-format]
|
|
121
137
|
typical-use: "Integration and E2E test design principles, value-based test selection, behavior-first approach, test skeleton specification"
|
|
122
138
|
size: medium
|
|
123
139
|
key-references:
|
|
@@ -133,11 +149,13 @@ skills:
|
|
|
133
149
|
- "Test File Naming Convention"
|
|
134
150
|
- "Review Criteria"
|
|
135
151
|
- "Quality Standards [MANDATORY]"
|
|
152
|
+
references:
|
|
153
|
+
- "references/e2e-design.md"
|
|
136
154
|
|
|
137
155
|
subagents-orchestration-guide:
|
|
138
156
|
skill: "subagents-orchestration-guide"
|
|
139
|
-
tags: [orchestration, workflow, subagents, autonomous-execution, planning, design-flow, implementation-flow]
|
|
140
|
-
typical-use: "Orchestrating subagents through implementation workflows, scale determination, stop points, autonomous execution mode"
|
|
157
|
+
tags: [orchestration, workflow, subagents, context-isolation, autonomous-execution, guided-autonomous-execution, planning, design-flow, implementation-flow, implementation-readiness, readiness-gate]
|
|
158
|
+
typical-use: "Orchestrating subagents through implementation workflows, scale determination, stop points, guided autonomous execution mode"
|
|
141
159
|
size: large
|
|
142
160
|
key-references:
|
|
143
161
|
- "Orchestrator Pattern"
|
|
@@ -152,6 +170,8 @@ skills:
|
|
|
152
170
|
- "Explicit Stop Points [MANDATORY]"
|
|
153
171
|
- "Scale Determination and Document Requirements"
|
|
154
172
|
- "Structured Response Specification"
|
|
173
|
+
- "Implementation Readiness Marker Contract"
|
|
174
|
+
- "Implementation Readiness Preflight Procedure"
|
|
155
175
|
- "Handling Requirement Changes"
|
|
156
176
|
- "Basic Flow for Work Planning"
|
|
157
177
|
- "Autonomous Execution Mode"
|
|
@@ -160,3 +180,30 @@ skills:
|
|
|
160
180
|
- "Required Dialogue Points with Humans [MANDATORY]"
|
|
161
181
|
- "Action Checklist"
|
|
162
182
|
- "References"
|
|
183
|
+
references:
|
|
184
|
+
- "references/monorepo-flow.md"
|
|
185
|
+
|
|
186
|
+
external-resource-context:
|
|
187
|
+
skill: "external-resource-context"
|
|
188
|
+
tags: [external-resources, external-resource-hearing, project-context, mcp, design-source, design-system, visual-verification, generated-artifacts, api-schema, infrastructure, secret-store, context-engineering, documentation]
|
|
189
|
+
typical-use: "Discover, confirm, and record external resource access methods for downstream design, planning, implementation, and review"
|
|
190
|
+
size: medium
|
|
191
|
+
key-references:
|
|
192
|
+
- "Single Source of Truth"
|
|
193
|
+
- "Context Engineering"
|
|
194
|
+
sections:
|
|
195
|
+
- "Purpose"
|
|
196
|
+
- "Scope Boundaries"
|
|
197
|
+
- "Storage Locations"
|
|
198
|
+
- "Hearing Protocol"
|
|
199
|
+
- "Storage Protocol"
|
|
200
|
+
- "Lookup Protocol"
|
|
201
|
+
- "Output Format"
|
|
202
|
+
- "Quality Checklist"
|
|
203
|
+
- "References"
|
|
204
|
+
references:
|
|
205
|
+
- "references/frontend.md"
|
|
206
|
+
- "references/backend.md"
|
|
207
|
+
- "references/api.md"
|
|
208
|
+
- "references/infra.md"
|
|
209
|
+
- "references/template.md"
|
|
@@ -13,9 +13,8 @@ import userEvent from '@testing-library/user-event'
|
|
|
13
13
|
### Coverage Requirements
|
|
14
14
|
|
|
15
15
|
- **Overall minimum**: 60%
|
|
16
|
-
- **
|
|
17
|
-
- **
|
|
18
|
-
- **Organisms (Header, Footer)**: 60%+
|
|
16
|
+
- **Atomic Design projects**: Atoms 70%+, Molecules 65%+, Organisms 60%+
|
|
17
|
+
- **Other component architectures**: Keep 60% as the baseline and raise foundational or highly reused components to 70%+
|
|
19
18
|
- **Custom Hooks**: 65%+
|
|
20
19
|
- **Utils**: 70%+
|
|
21
20
|
|
|
@@ -58,7 +58,8 @@ Test type definitions, budgets, and value-based selection rules are specified in
|
|
|
58
58
|
|
|
59
59
|
Key points:
|
|
60
60
|
- **Integration Tests**: MAX 3 per feature, created alongside implementation
|
|
61
|
-
- **
|
|
61
|
+
- **fixture-e2e**: MAX 3 per feature, created alongside UI implementation, mocked backend / fixture-driven state
|
|
62
|
+
- **service-integration-e2e**: MAX 1-2 per feature, executed in final phase only, live local stack
|
|
62
63
|
|
|
63
64
|
## 4-Phase Generation Process
|
|
64
65
|
|
|
@@ -123,7 +124,7 @@ For each valid AC from Phase 1:
|
|
|
123
124
|
|
|
124
125
|
**Output**: Candidate pool with value metadata
|
|
125
126
|
|
|
126
|
-
### Phase 3: Value-Based Selection (Two-Pass #2)
|
|
127
|
+
### Phase 3: Value-Based Selection and Lane Assignment (Two-Pass #2)
|
|
127
128
|
|
|
128
129
|
Value score and E2E selection rules are defined in **integration-e2e-testing skill**.
|
|
129
130
|
|
|
@@ -138,14 +139,15 @@ Value score and E2E selection rules are defined in **integration-e2e-testing ski
|
|
|
138
139
|
3. **Push-Down Analysis**:
|
|
139
140
|
```
|
|
140
141
|
Can this be unit-tested? → Remove from integration/E2E pool
|
|
141
|
-
Already integration-tested
|
|
142
|
+
Already integration-tested AND verifiable in-process? → Remove from E2E pool
|
|
142
143
|
```
|
|
143
|
-
4. **
|
|
144
|
+
4. **Lane Assignment**:
|
|
144
145
|
```
|
|
145
|
-
|
|
146
|
-
|
|
146
|
+
UI journey verifiable with mocked backend / fixture-driven state → fixture-e2e
|
|
147
|
+
Journey correctness depends on real cross-service behavior → service-integration-e2e
|
|
148
|
+
Service-internal chain only → Not reserved-slot eligible
|
|
147
149
|
```
|
|
148
|
-
5. **Sort by Value Score** (descending order)
|
|
150
|
+
5. **Sort by Value Score within each lane** (descending order)
|
|
149
151
|
|
|
150
152
|
**Output**: Ranked, deduplicated candidate list
|
|
151
153
|
|
|
@@ -153,16 +155,19 @@ Value score and E2E selection rules are defined in **integration-e2e-testing ski
|
|
|
153
155
|
|
|
154
156
|
**Hard Limits per Feature**:
|
|
155
157
|
- **Integration Tests**: MAX 3 tests
|
|
156
|
-
- **
|
|
158
|
+
- **fixture-e2e**: MAX 3 tests
|
|
159
|
+
- **service-integration-e2e**: MAX 1-2 tests
|
|
157
160
|
|
|
158
161
|
**Selection Algorithm**:
|
|
159
162
|
|
|
160
163
|
```
|
|
161
164
|
1. Sort integration candidates by Value Score (descending)
|
|
162
165
|
2. Select up to 3 integration candidates
|
|
163
|
-
3. Reserve 1
|
|
164
|
-
4.
|
|
165
|
-
5.
|
|
166
|
+
3. Reserve 1 fixture-e2e slot for the highest-value user-facing multi-step journey, if one exists
|
|
167
|
+
4. Reserve 1 service-integration-e2e slot only when the journey needs real cross-service verification
|
|
168
|
+
5. Fill remaining fixture-e2e budget with candidates that satisfy `Value Score >= 20`
|
|
169
|
+
6. Fill remaining service-integration-e2e budget with candidates that satisfy `Value Score > 50`
|
|
170
|
+
7. If a lane emits no tests, return its generated file as `null` with a concrete lane-specific absence reason
|
|
166
171
|
```
|
|
167
172
|
|
|
168
173
|
**Output**: Final test set
|
|
@@ -198,24 +203,46 @@ Adapt comment syntax to the project's language when generating annotations.
|
|
|
198
203
|
[Test: 'AC1: Failed payment displays error without creating order']
|
|
199
204
|
```
|
|
200
205
|
|
|
201
|
-
###
|
|
206
|
+
### fixture-e2e Test File
|
|
202
207
|
|
|
203
208
|
```
|
|
204
|
-
// [Feature Name]
|
|
205
|
-
// Generated: [date] | Budget Used: 1/
|
|
206
|
-
// Test Type:
|
|
207
|
-
// Implementation Timing:
|
|
209
|
+
// [Feature Name] fixture-e2e Test - Design Doc: [filename]
|
|
210
|
+
// Generated: [date] | Budget Used: 1/3 fixture-e2e
|
|
211
|
+
// Test Type: Browser UI with mocked backend / fixture-driven state
|
|
212
|
+
// Implementation Timing: Alongside UI implementation
|
|
208
213
|
|
|
209
214
|
[Import statement using detected test framework]
|
|
210
215
|
|
|
211
216
|
[Test suite using detected framework syntax]
|
|
212
|
-
// User Journey:
|
|
213
|
-
// Value Score:
|
|
214
|
-
// Verification:
|
|
215
|
-
// @category: e2e
|
|
217
|
+
// User Journey: Dismiss card -> Undo banner appears -> Undo restores card
|
|
218
|
+
// Value Score: 60 | Business Value: 6 | Frequency: 7 | Defect Detection: 8
|
|
219
|
+
// Verification: Browser-visible state transitions with mocked backend state
|
|
220
|
+
// @category: fixture-e2e
|
|
221
|
+
// @lane: fixture-e2e
|
|
222
|
+
// @dependency: full-ui (mocked backend)
|
|
223
|
+
// @complexity: medium
|
|
224
|
+
[Test: 'User Journey: Dismiss and undo restores the card']
|
|
225
|
+
```
|
|
226
|
+
|
|
227
|
+
### service-integration-e2e Test File
|
|
228
|
+
|
|
229
|
+
```
|
|
230
|
+
// [Feature Name] service-integration-e2e Test - Design Doc: [filename]
|
|
231
|
+
// Generated: [date] | Budget Used: 1/2 service-integration-e2e
|
|
232
|
+
// Test Type: End-to-end against running local stack
|
|
233
|
+
// Implementation Timing: Final phase only
|
|
234
|
+
|
|
235
|
+
[Import statement using detected test framework]
|
|
236
|
+
|
|
237
|
+
[Test suite using detected framework syntax]
|
|
238
|
+
// User Journey: Complete purchase flow (browse -> checkout -> payment -> confirmation persisted)
|
|
239
|
+
// Value Score: 120 | Business Value: 10 (business-critical) | Frequency: 10 (core flow) | Legal: true
|
|
240
|
+
// Verification: Order persists in DB and confirmation event is emitted
|
|
241
|
+
// @category: service-integration-e2e
|
|
242
|
+
// @lane: service-integration-e2e
|
|
216
243
|
// @dependency: full-system
|
|
217
244
|
// @complexity: high
|
|
218
|
-
[Test: 'User Journey: Complete product purchase
|
|
245
|
+
[Test: 'User Journey: Complete product purchase persists order and emits confirmation']
|
|
219
246
|
```
|
|
220
247
|
|
|
221
248
|
### Generation Report
|
|
@@ -226,13 +253,18 @@ Adapt comment syntax to the project's language when generating annotations.
|
|
|
226
253
|
"feature": "[feature name]",
|
|
227
254
|
"generatedFiles": {
|
|
228
255
|
"integration": "[path]/[feature].int.test.[ext]",
|
|
229
|
-
"
|
|
256
|
+
"fixtureE2e": null,
|
|
257
|
+
"serviceE2e": null
|
|
230
258
|
},
|
|
231
259
|
"budgetUsage": {
|
|
232
260
|
"integration": "2/3",
|
|
233
|
-
"
|
|
261
|
+
"fixtureE2e": "0/3",
|
|
262
|
+
"serviceE2e": "0/2"
|
|
234
263
|
},
|
|
235
|
-
"e2eAbsenceReason":
|
|
264
|
+
"e2eAbsenceReason": {
|
|
265
|
+
"fixtureE2e": "all_e2e_candidates_below_threshold",
|
|
266
|
+
"serviceE2e": "no_real_service_dependency"
|
|
267
|
+
}
|
|
236
268
|
}
|
|
237
269
|
```
|
|
238
270
|
|
|
@@ -242,13 +274,18 @@ Adapt comment syntax to the project's language when generating annotations.
|
|
|
242
274
|
"feature": "[feature name]",
|
|
243
275
|
"generatedFiles": {
|
|
244
276
|
"integration": "[path]/[feature].int.test.[ext]",
|
|
245
|
-
"
|
|
277
|
+
"fixtureE2e": "[path]/[feature].fixture.e2e.test.[ext]",
|
|
278
|
+
"serviceE2e": "[path]/[feature].service.e2e.test.[ext]"
|
|
246
279
|
},
|
|
247
280
|
"budgetUsage": {
|
|
248
281
|
"integration": "2/3",
|
|
249
|
-
"
|
|
282
|
+
"fixtureE2e": "1/3",
|
|
283
|
+
"serviceE2e": "1/2"
|
|
250
284
|
},
|
|
251
|
-
"e2eAbsenceReason":
|
|
285
|
+
"e2eAbsenceReason": {
|
|
286
|
+
"fixtureE2e": null,
|
|
287
|
+
"serviceE2e": null
|
|
288
|
+
}
|
|
252
289
|
}
|
|
253
290
|
```
|
|
254
291
|
|
|
@@ -256,8 +293,9 @@ Adapt comment syntax to the project's language when generating annotations.
|
|
|
256
293
|
|
|
257
294
|
Each test case MUST have the following standard annotations for test implementation planning:
|
|
258
295
|
|
|
259
|
-
- **@category**: core-functionality | integration | edge-case | ux
|
|
260
|
-
- **@
|
|
296
|
+
- **@category**: core-functionality | integration | edge-case | ux | fixture-e2e | service-integration-e2e
|
|
297
|
+
- **@lane**: integration | fixture-e2e | service-integration-e2e
|
|
298
|
+
- **@dependency**: none | [component names] | full-ui (mocked backend) | full-system
|
|
261
299
|
- **@complexity**: low | medium | high
|
|
262
300
|
|
|
263
301
|
These annotations are used when planning and prioritizing test implementation.
|
|
@@ -282,7 +320,7 @@ These annotations are used when planning and prioritizing test implementation.
|
|
|
282
320
|
|
|
283
321
|
### Auto-processable
|
|
284
322
|
- **Directory Absent**: Auto-create appropriate directory following detected test structure
|
|
285
|
-
- **No E2E Selected**: Valid outcome when accompanied by `e2eAbsenceReason`
|
|
323
|
+
- **No E2E Selected**: Valid outcome when accompanied by lane-specific `e2eAbsenceReason`
|
|
286
324
|
- **Budget Exceeded by Critical Test**: Report to user
|
|
287
325
|
|
|
288
326
|
### Escalation Required
|
|
@@ -316,7 +354,7 @@ These annotations are used when planning and prioritizing test implementation.
|
|
|
316
354
|
- **Post-execution**:
|
|
317
355
|
- Completeness of selected tests
|
|
318
356
|
- Dependency validity verified
|
|
319
|
-
- Integration tests and
|
|
357
|
+
- Integration tests, fixture-e2e tests, and service-integration-e2e tests generated in separate files when selected
|
|
320
358
|
- Generation report completeness
|
|
321
359
|
|
|
322
360
|
## Completion Gate [BLOCKING]
|
|
@@ -88,6 +88,7 @@ If incomplete implementation is detected, stop immediately and return `status: "
|
|
|
88
88
|
|
|
89
89
|
**Supplementary detection** (when `task_file` is provided):
|
|
90
90
|
- Read the task file's `Quality Assurance Mechanisms` section
|
|
91
|
+
- Read `docs/project-context/external-resources.md` and related `External Resources Used` sections when the task file references them
|
|
91
92
|
- For executable mechanisms, verify the tool exists and is runnable in the current project, then add it to the quality-check command set
|
|
92
93
|
- For non-executable domain constraints, keep them as explicit verification targets and check the changed files against the stated constraint during review
|
|
93
94
|
- Record skipped mechanisms only when neither executable verification nor direct constraint checking is possible
|
|
@@ -475,3 +476,7 @@ enabled = true
|
|
|
475
476
|
[[skills.config]]
|
|
476
477
|
path = ".agents/skills/ai-development-guide/SKILL.md"
|
|
477
478
|
enabled = true
|
|
479
|
+
|
|
480
|
+
[[skills.config]]
|
|
481
|
+
path = ".agents/skills/external-resource-context/SKILL.md"
|
|
482
|
+
enabled = true
|
|
@@ -86,6 +86,7 @@ If incomplete implementation is detected, stop immediately and return `status: "
|
|
|
86
86
|
|
|
87
87
|
**Supplementary detection** (when `task_file` is provided):
|
|
88
88
|
- Read the task file's `Quality Assurance Mechanisms` section
|
|
89
|
+
- Read `docs/project-context/external-resources.md` and related `External Resources Used` sections when the task file references them
|
|
89
90
|
- For executable mechanisms, verify the tool exists and is runnable in the current project, then add it to the quality-check command set
|
|
90
91
|
- For non-executable domain constraints, keep them as explicit verification targets and check the changed files against the stated constraint during review
|
|
91
92
|
- Record skipped mechanisms only when neither executable verification nor direct constraint checking is possible
|
|
@@ -369,3 +370,7 @@ enabled = true
|
|
|
369
370
|
[[skills.config]]
|
|
370
371
|
path = ".agents/skills/ai-development-guide/SKILL.md"
|
|
371
372
|
enabled = true
|
|
373
|
+
|
|
374
|
+
[[skills.config]]
|
|
375
|
+
path = ".agents/skills/external-resource-context/SKILL.md"
|
|
376
|
+
enabled = true
|
|
@@ -130,8 +130,12 @@ Decompose tasks based on implementation strategy patterns determined in implemen
|
|
|
130
130
|
|---|---|
|
|
131
131
|
| Existing code modification | Files being changed, adjacent tests, relevant Design Doc sections |
|
|
132
132
|
| New feature/component | Adjacent implementations in the same layer/domain, interface-defining Design Doc sections |
|
|
133
|
-
|
|
|
134
|
-
|
|
|
133
|
+
| Frontend component implementation | UI Spec component section cited by the work plan's UI Spec Component -> Task Mapping, interface-defining Design Doc sections, adjacent components |
|
|
134
|
+
| Frontend integration / fixture-e2e test work | UI Spec component section including state and interaction tables, test skeleton file, target implementation, fixture data, browser harness config |
|
|
135
|
+
| Integration test work | Test skeleton file, target implementation under test, existing fixture/auth/setup patterns |
|
|
136
|
+
| fixture-e2e environment/setup work | Existing fixture data, API mock layer, browser harness configuration |
|
|
137
|
+
| service-integration-e2e environment/setup work | Current environment config, startup scripts, seed scripts, auth flow references, external service stubs |
|
|
138
|
+
| Cross-boundary implementation | Connection Map rows touching the task target files, caller/producer module, callee/consumer module, expected signal, contract definition |
|
|
135
139
|
| Bug fix/refactor | Affected code paths, failing tests, reproduction-related files |
|
|
136
140
|
| Behavior replacement/rewrite | Existing implementation being replaced, observable outputs, Verification Strategy section in the Design Doc |
|
|
137
141
|
|
|
@@ -140,6 +144,8 @@ Decompose tasks based on implementation strategy patterns determined in implemen
|
|
|
140
144
|
- Investigation Targets are file paths to read, not actions to perform
|
|
141
145
|
- Use specific paths with optional hints such as `docs/design/payments.md (§ Retry Flow)` or `src/orders/service.ts (createOrder)`
|
|
142
146
|
- When test skeletons exist, include them explicitly
|
|
147
|
+
- When the work plan contains a UI Spec Component -> Task Mapping table, propagate matching component sections to every task listed in the row
|
|
148
|
+
- When the work plan contains a Connection Map, propagate boundary rows touching the task's target files to every task on either side of the boundary
|
|
143
149
|
- When a task matches multiple natures, include Investigation Targets from all matching rows and deduplicate overlaps
|
|
144
150
|
|
|
145
151
|
7. **Implementation Pattern Consistency**
|
|
@@ -185,6 +191,25 @@ When the work plan includes a `Design-to-Plan Traceability` section:
|
|
|
185
191
|
5. **Verification integrity**: For `verification` rows, ensure the corresponding task file includes the required comparison or verification method in Operation Verification Methods.
|
|
186
192
|
6. **Prerequisite integrity**: For `prerequisite` rows, place setup, migration, seed, auth, or environment work before dependent implementation tasks.
|
|
187
193
|
|
|
194
|
+
## UI Spec Propagation
|
|
195
|
+
|
|
196
|
+
When the work plan includes a `UI Spec Component -> Task Mapping` section:
|
|
197
|
+
|
|
198
|
+
1. For each row, locate the task IDs listed in `Covered By Task(s)`.
|
|
199
|
+
2. Add the component section heading to those task files' Investigation Targets.
|
|
200
|
+
3. Include the states listed in `States to Cover` in Investigation Notes and Operation Verification Methods.
|
|
201
|
+
4. Preserve `gap` rows as planning issues and surface them to the caller.
|
|
202
|
+
5. If a task implements UI but no component mapping row covers it, add a warning in the decomposition report.
|
|
203
|
+
|
|
204
|
+
## Connection Map Propagation
|
|
205
|
+
|
|
206
|
+
When the work plan includes a `Connection Map` section:
|
|
207
|
+
|
|
208
|
+
1. For each boundary row, locate all tasks listed in `Covered By Task(s)`.
|
|
209
|
+
2. Add the caller/producer module, callee/consumer module, serialized contract, and expected signal to each listed task's Investigation Targets or Notes.
|
|
210
|
+
3. For tasks on one side of a boundary, include an Operation Verification Method that observes the expected signal from the other side.
|
|
211
|
+
4. Propagate only boundary rows explicitly mapped in the work plan.
|
|
212
|
+
|
|
188
213
|
## Task File Template
|
|
189
214
|
|
|
190
215
|
See task template in documentation-criteria skill for details.
|
|
@@ -133,7 +133,9 @@ Use the appropriate run command based on the `packageManager` field in package.j
|
|
|
133
133
|
|
|
134
134
|
### 1. Task Selection
|
|
135
135
|
|
|
136
|
-
|
|
136
|
+
If the orchestrator prompt provides a task file path, execute only that exact task file. Do not scan or select any other task file.
|
|
137
|
+
|
|
138
|
+
When no task file path is provided, select and execute files with pattern `docs/plans/tasks/*-task-*.md` that have uncompleted checkboxes `[ ]` remaining.
|
|
137
139
|
|
|
138
140
|
### 2. Task Background Understanding
|
|
139
141
|
#### Investigation Targets (Required when present)
|
|
@@ -152,6 +154,13 @@ Select and execute files with pattern `docs/plans/tasks/*-task-*.md` that have u
|
|
|
152
154
|
- API Specifications → Understand endpoints, parameters, response formats (for MSW mocking)
|
|
153
155
|
- Overall Design Document → Understand system-wide context
|
|
154
156
|
|
|
157
|
+
**External Resources Consultation**:
|
|
158
|
+
When the task file, Dependencies, or Investigation Targets reference `docs/project-context/external-resources.md` or an `External Resources Used` section:
|
|
159
|
+
1. Read `docs/project-context/external-resources.md`
|
|
160
|
+
2. Resolve feature-specific identifiers from the UI Spec, Design Doc, or task file
|
|
161
|
+
3. Use declared access methods for design origin, design system, guidelines, and visual verification facts during investigation and verification
|
|
162
|
+
4. Record consulted resources in Investigation Notes
|
|
163
|
+
|
|
155
164
|
### 3. Implementation Execution
|
|
156
165
|
|
|
157
166
|
#### Test Environment Check
|
|
@@ -185,7 +194,7 @@ This is a repeated self-check during implementation, not a one-time pre-implemen
|
|
|
185
194
|
|
|
186
195
|
**Implementation procedure for each checkbox item**:
|
|
187
196
|
1. **Red**: Create React Testing Library test for that checkbox item (failing state)
|
|
188
|
-
※For integration tests
|
|
197
|
+
※For integration tests and fixture-e2e tests, create and execute with the related UI implementation; service-integration-e2e tests are executed in final phase only. Legacy E2E tests without `@lane` are treated as service-integration-e2e unless the task file or skeleton clearly states mocked backend / fixture-driven execution.
|
|
189
198
|
2. **Green**: Implement minimum code to pass test (React function component)
|
|
190
199
|
3. **Refactor**: Improve code quality (readability, maintainability, React best practices)
|
|
191
200
|
4. **Progress Update [MANDATORY]**: Execute the following in sequence (cannot be omitted)
|
|
@@ -211,16 +220,11 @@ Return one of the following as the final response (see Structured Response Speci
|
|
|
211
220
|
- `status: "completed"` — task fully implemented
|
|
212
221
|
- `status: "escalation_needed"` — design deviation or similar component discovered
|
|
213
222
|
|
|
214
|
-
## Research Task Deliverables
|
|
215
|
-
|
|
216
|
-
Research/analysis tasks create deliverable files specified in metadata "Provides".
|
|
217
|
-
Examples: `docs/plans/analysis/component-research.md`, `docs/plans/analysis/api-integration.md`
|
|
218
|
-
|
|
219
223
|
## Structured Response Specification
|
|
220
224
|
|
|
221
225
|
### Field Specifications
|
|
222
226
|
|
|
223
|
-
**requiresTestReview**: Set to `true` when the task added or updated integration tests or
|
|
227
|
+
**requiresTestReview**: Set to `true` when the task added or updated integration tests, fixture-e2e tests, or service-integration-e2e tests. Set to `false` for unit-test-only tasks or tasks with no tests.
|
|
224
228
|
|
|
225
229
|
### 1. Task Completion Response
|
|
226
230
|
Report in the following JSON format upon task completion (**without executing quality checks or commits**, delegating to quality assurance process):
|
|
@@ -380,9 +384,6 @@ When repository-wide verification is insufficient to determine the appropriate d
|
|
|
380
384
|
|
|
381
385
|
**ENFORCEMENT**: HALT if any gate unchecked. Return `status: "escalation_needed"` to caller.
|
|
382
386
|
|
|
383
|
-
## Completion Criteria
|
|
384
|
-
|
|
385
|
-
- [ ] Final response is a single JSON with status `completed` or `escalation_needed`
|
|
386
387
|
"""
|
|
387
388
|
|
|
388
389
|
[[skills.config]]
|
|
@@ -400,3 +401,7 @@ enabled = true
|
|
|
400
401
|
[[skills.config]]
|
|
401
402
|
path = ".agents/skills/implementation-approach/SKILL.md"
|
|
402
403
|
enabled = true
|
|
404
|
+
|
|
405
|
+
[[skills.config]]
|
|
406
|
+
path = ".agents/skills/external-resource-context/SKILL.md"
|
|
407
|
+
enabled = true
|
|
@@ -133,7 +133,9 @@ Skill Status:
|
|
|
133
133
|
|
|
134
134
|
### 1. Task Selection
|
|
135
135
|
|
|
136
|
-
|
|
136
|
+
If the orchestrator prompt provides a task file path, execute only that exact task file. Do not scan or select any other task file.
|
|
137
|
+
|
|
138
|
+
When no task file path is provided, select and execute files with pattern `docs/plans/tasks/*-task-*.md` that have uncompleted checkboxes `[ ]` remaining.
|
|
137
139
|
|
|
138
140
|
### 2. Task Background Understanding
|
|
139
141
|
#### Investigation Targets (Required when present)
|
|
@@ -152,6 +154,13 @@ Select and execute files with pattern `docs/plans/tasks/*-task-*.md` that have u
|
|
|
152
154
|
- Data Schema → Understand table structure, relationships
|
|
153
155
|
- Overall Design Document → Understand system-wide context
|
|
154
156
|
|
|
157
|
+
**External Resources Consultation**:
|
|
158
|
+
When the task file, Dependencies, or Investigation Targets reference `docs/project-context/external-resources.md` or an `External Resources Used` section:
|
|
159
|
+
1. Read `docs/project-context/external-resources.md`
|
|
160
|
+
2. Resolve feature-specific identifiers from the referenced Design Doc or task file
|
|
161
|
+
3. Use declared access methods for required external facts during investigation and verification
|
|
162
|
+
4. Record consulted resources in Investigation Notes
|
|
163
|
+
|
|
155
164
|
### 3. Implementation Execution
|
|
156
165
|
|
|
157
166
|
#### Test Environment Check
|
|
@@ -194,7 +203,9 @@ This is a repeated self-check during implementation, not a one-time pre-implemen
|
|
|
194
203
|
**Test types**:
|
|
195
204
|
- Unit tests: RED-GREEN-REFACTOR cycle
|
|
196
205
|
- Integration tests: Create and execute with implementation
|
|
197
|
-
-
|
|
206
|
+
- fixture-e2e tests: Create and execute with the related UI/browser task when the task file specifies that lane
|
|
207
|
+
- service-integration-e2e tests: Execute only in final phase when the task file specifies that lane
|
|
208
|
+
- legacy E2E tests without `@lane`: Treat as service-integration-e2e unless the task file or skeleton clearly states mocked backend / fixture-driven execution
|
|
198
209
|
|
|
199
210
|
#### Operation Verification
|
|
200
211
|
- Execute "Operation Verification Methods" section in task
|
|
@@ -212,16 +223,11 @@ Return one of the following as the final response (see Structured Response Speci
|
|
|
212
223
|
- `status: "completed"` — task fully implemented
|
|
213
224
|
- `status: "escalation_needed"` — design deviation or similar function discovered
|
|
214
225
|
|
|
215
|
-
## Research Task Deliverables
|
|
216
|
-
|
|
217
|
-
Research/analysis tasks create deliverable files specified in metadata "Provides".
|
|
218
|
-
Examples: `docs/plans/analysis/research-results.md`, `docs/plans/analysis/api-spec.md`
|
|
219
|
-
|
|
220
226
|
## Structured Response Specification
|
|
221
227
|
|
|
222
228
|
### Field Specifications
|
|
223
229
|
|
|
224
|
-
**requiresTestReview**: Set to `true` when the task added or updated integration tests or
|
|
230
|
+
**requiresTestReview**: Set to `true` when the task added or updated integration tests, fixture-e2e tests, or service-integration-e2e tests. Set to `false` for unit-test-only tasks or tasks with no tests.
|
|
225
231
|
|
|
226
232
|
### 1. Task Completion Response
|
|
227
233
|
Report in the following JSON format upon task completion (**without executing quality checks or commits**, delegating to quality assurance process):
|
|
@@ -364,11 +370,9 @@ When repository-wide verification is insufficient to determine the appropriate d
|
|
|
364
370
|
```
|
|
365
371
|
|
|
366
372
|
## Execution Principles
|
|
367
|
-
|
|
368
373
|
- Follow RED-GREEN-REFACTOR (see the principles in testing skill)
|
|
369
374
|
- Update progress checkboxes per step
|
|
370
|
-
- Escalate when
|
|
371
|
-
- Escalate when dependency version or representative pattern choice cannot be determined from repository evidence
|
|
375
|
+
- Escalate when design deviation, similar functions, missing investigation targets/test environment, or uncertain dependency/pattern choice blocks repository-evidence-based implementation
|
|
372
376
|
- Stop after implementation and test creation — quality checks and commits are handled separately
|
|
373
377
|
|
|
374
378
|
## Completion Gate [BLOCKING]
|
|
@@ -383,9 +387,6 @@ When repository-wide verification is insufficient to determine the appropriate d
|
|
|
383
387
|
|
|
384
388
|
**ENFORCEMENT**: HALT if any gate unchecked. Return `status: "escalation_needed"` to caller.
|
|
385
389
|
|
|
386
|
-
## Completion Criteria
|
|
387
|
-
|
|
388
|
-
- [ ] Final response is a single JSON with status `completed` or `escalation_needed`
|
|
389
390
|
"""
|
|
390
391
|
|
|
391
392
|
[[skills.config]]
|
|
@@ -403,3 +404,7 @@ enabled = true
|
|
|
403
404
|
[[skills.config]]
|
|
404
405
|
path = ".agents/skills/implementation-approach/SKILL.md"
|
|
405
406
|
enabled = true
|
|
407
|
+
|
|
408
|
+
[[skills.config]]
|
|
409
|
+
path = ".agents/skills/external-resource-context/SKILL.md"
|
|
410
|
+
enabled = true
|