@ngxtm/devkit 3.19.0 → 3.20.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/merged-commands/application-performance-performance-optimization.md +13 -13
- package/merged-commands/ask/fast.md +14 -57
- package/merged-commands/ask/hard.md +22 -79
- package/merged-commands/auto.md +6 -33
- package/merged-commands/backend-development-feature-development.md +12 -12
- package/merged-commands/bootstrap/auto/fast.md +15 -15
- package/merged-commands/bootstrap/auto/parallel.md +12 -12
- package/merged-commands/bootstrap/auto.md +14 -14
- package/merged-commands/bootstrap.md +15 -15
- package/merged-commands/brainstorm/fast.md +19 -72
- package/merged-commands/brainstorm/hard.md +23 -84
- package/merged-commands/c4-architecture-c4-architecture.md +5 -5
- package/merged-commands/code/auto.md +16 -16
- package/merged-commands/code/fast.md +19 -72
- package/merged-commands/code/hard.md +38 -122
- package/merged-commands/code/no-test.md +12 -12
- package/merged-commands/code/parallel.md +9 -9
- package/merged-commands/code.md +14 -14
- package/merged-commands/comprehensive-review-full-review.md +8 -8
- package/merged-commands/context-degradation.md +2 -2
- package/merged-commands/context-engineering.md +4 -4
- package/merged-commands/context-optimization.md +3 -3
- package/merged-commands/cook/auto/fast.md +3 -3
- package/merged-commands/cook/auto/parallel.md +9 -9
- package/merged-commands/cook/auto.md +1 -1
- package/merged-commands/cook/fast.md +38 -47
- package/merged-commands/cook/hard.md +46 -41
- package/merged-commands/cook.md +13 -13
- package/merged-commands/daily-news-report.md +15 -15
- package/merged-commands/data-engineering-data-driven-feature.md +16 -16
- package/merged-commands/debug/fast.md +13 -29
- package/merged-commands/debug/hard.md +47 -49
- package/merged-commands/debug.md +1 -1
- package/merged-commands/debugging-toolkit-smart-debug.md +1 -1
- package/merged-commands/deploy/check.md +22 -71
- package/merged-commands/deploy/preview.md +18 -62
- package/merged-commands/deploy/production.md +22 -71
- package/merged-commands/deploy/rollback.md +22 -71
- package/merged-commands/deploy.md +0 -11
- package/merged-commands/design/3d.md +3 -3
- package/merged-commands/design/describe.md +1 -1
- package/merged-commands/design/fast.md +2 -2
- package/merged-commands/design/good.md +3 -3
- package/merged-commands/design/hard.md +15 -85
- package/merged-commands/design/screenshot.md +1 -1
- package/merged-commands/design/video.md +1 -1
- package/merged-commands/design.md +0 -11
- package/merged-commands/doc-coauthoring.md +5 -5
- package/merged-commands/docker-expert.md +1 -1
- package/merged-commands/docs/audit.md +26 -77
- package/merged-commands/docs/business.md +26 -77
- package/merged-commands/docs/core.md +24 -68
- package/merged-commands/docs/init.md +8 -8
- package/merged-commands/docs/update.md +13 -13
- package/merged-commands/docs.md +0 -12
- package/merged-commands/error-debugging-multi-agent-review.md +1 -1
- package/merged-commands/error-diagnostics-smart-debug.md +1 -1
- package/merged-commands/finishing-a-development-branch.md +1 -1
- package/merged-commands/fix/ci.md +2 -2
- package/merged-commands/fix/fast.md +2 -2
- package/merged-commands/fix/hard.md +6 -6
- package/merged-commands/fix/logs.md +5 -5
- package/merged-commands/fix/parallel.md +9 -9
- package/merged-commands/fix/test.md +6 -6
- package/merged-commands/fix/ui.md +8 -8
- package/merged-commands/fixing.md +3 -3
- package/merged-commands/framework-migration-legacy-modernize.md +13 -13
- package/merged-commands/full-stack-orchestration-full-stack-feature.md +12 -12
- package/merged-commands/git/cm.md +1 -1
- package/merged-commands/git/cp.md +1 -1
- package/merged-commands/git/merge.md +1 -1
- package/merged-commands/git/pr.md +1 -1
- package/merged-commands/git-pr-workflows-git-workflow.md +10 -10
- package/merged-commands/google-adk-python.md +1 -1
- package/merged-commands/hr-pro.md +1 -1
- package/merged-commands/incident-response-incident-response.md +13 -13
- package/merged-commands/integrate/polar.md +3 -3
- package/merged-commands/integrate/sepay.md +3 -3
- package/merged-commands/journal.md +1 -1
- package/merged-commands/linear-claude-skill.md +2 -2
- package/merged-commands/loki-mode.md +14 -14
- package/merged-commands/machine-learning-ops-ml-pipeline.md +7 -7
- package/merged-commands/mcp-management.md +8 -8
- package/merged-commands/multi-agent-patterns.md +14 -14
- package/merged-commands/multi-platform-apps-multi-platform.md +10 -10
- package/merged-commands/nestjs-expert.md +1 -1
- package/merged-commands/performance-testing-review-multi-agent-review.md +1 -1
- package/merged-commands/plan/archive.md +1 -1
- package/merged-commands/plan/ci.md +1 -1
- package/merged-commands/plan/fast.md +2 -2
- package/merged-commands/plan/hard.md +4 -4
- package/merged-commands/plan/parallel.md +5 -5
- package/merged-commands/plan/two.md +6 -6
- package/merged-commands/requesting-code-review.md +6 -6
- package/merged-commands/review/codebase/parallel.md +5 -5
- package/merged-commands/review/codebase.md +5 -5
- package/merged-commands/review/fast.md +13 -29
- package/merged-commands/review/hard.md +48 -49
- package/merged-commands/review.md +0 -11
- package/merged-commands/security-scanning-security-hardening.md +13 -13
- package/merged-commands/skill/add.md +6 -6
- package/merged-commands/skill/create.md +6 -6
- package/merged-commands/skill/fix-logs.md +6 -6
- package/merged-commands/skill/optimize/auto.md +1 -1
- package/merged-commands/skill/optimize.md +1 -1
- package/merged-commands/skill/plan.md +1 -1
- package/merged-commands/skill/update.md +6 -6
- package/merged-commands/subagent-driven-development.md +53 -53
- package/merged-commands/tdd-workflows-tdd-cycle.md +12 -12
- package/merged-commands/tdd-workflows-tdd-red.md +1 -1
- package/merged-commands/tdd-workflows-tdd-refactor.md +1 -1
- package/merged-commands/test/fast.md +22 -33
- package/merged-commands/test/hard.md +59 -56
- package/merged-commands/test/ui.md +1 -1
- package/merged-commands/test.md +1 -1
- package/merged-commands/typescript-expert.md +1 -1
- package/merged-commands/use-mcp.md +5 -5
- package/merged-commands/writing-plans.md +3 -3
- package/merged-commands/writing-skills.md +8 -8
- package/package.json +1 -1
- package/rules-index.json +1 -1
- package/skills/application-performance-performance-optimization/SKILL.md +13 -13
- package/skills/azure-ai-agents-python/references/tools.md +1 -1
- package/skills/backend-development-feature-development/SKILL.md +12 -12
- package/skills/best-practices/references/anti-patterns.md +2 -2
- package/skills/best-practices/references/best-practices-guide.md +14 -14
- package/skills/c4-architecture-c4-architecture/SKILL.md +5 -5
- package/skills/comprehensive-review-full-review/SKILL.md +8 -8
- package/skills/context-degradation/SKILL.md +2 -2
- package/skills/context-engineering/SKILL.md +4 -4
- package/skills/context-engineering/references/context-degradation.md +1 -1
- package/skills/context-engineering/references/context-optimization.md +1 -1
- package/skills/context-engineering/references/multi-agent-patterns.md +1 -1
- package/skills/context-engineering/references/runtime-awareness.md +1 -1
- package/skills/context-optimization/SKILL.md +3 -3
- package/skills/daily-news-report/SKILL.md +15 -15
- package/skills/data-engineering-data-driven-feature/SKILL.md +16 -16
- package/skills/debugging-toolkit-smart-debug/SKILL.md +1 -1
- package/skills/doc-coauthoring/SKILL.md +5 -5
- package/skills/docker-expert/SKILL.md +1 -1
- package/skills/error-debugging-multi-agent-review/SKILL.md +1 -1
- package/skills/error-diagnostics-smart-debug/SKILL.md +1 -1
- package/skills/finishing-a-development-branch/SKILL.md +1 -1
- package/skills/fixing/SKILL.md +3 -3
- package/skills/fixing/references/parallel-exploration.md +4 -4
- package/skills/fixing/references/skill-activation-matrix.md +3 -3
- package/skills/fixing/references/workflow-deep.md +11 -11
- package/skills/fixing/references/workflow-quick.md +4 -4
- package/skills/fixing/references/workflow-standard.md +12 -12
- package/skills/framework-migration-legacy-modernize/SKILL.md +13 -13
- package/skills/full-stack-orchestration-full-stack-feature/SKILL.md +12 -12
- package/skills/git-pr-workflows-git-workflow/SKILL.md +10 -10
- package/skills/google-adk-python/SKILL.md +1 -1
- package/skills/hr-pro/SKILL.md +1 -1
- package/skills/incident-response-incident-response/SKILL.md +13 -13
- package/skills/incident-response-smart-fix/resources/implementation-playbook.md +17 -17
- package/skills/linear-claude-skill/SKILL.md +2 -2
- package/skills/loki-mode/ACKNOWLEDGEMENTS.md +4 -4
- package/skills/loki-mode/CHANGELOG.md +9 -9
- package/skills/loki-mode/CONTEXT-EXPORT.md +1 -1
- package/skills/loki-mode/README.md +2 -2
- package/skills/loki-mode/SKILL.md +14 -14
- package/skills/loki-mode/autonomy/run.sh +1 -1
- package/skills/loki-mode/integrations/vibe-kanban.md +1 -1
- package/skills/loki-mode/references/core-workflow.md +4 -4
- package/skills/loki-mode/references/production-patterns.md +6 -6
- package/skills/loki-mode/references/quality-control.md +2 -2
- package/skills/loki-mode/references/sdlc-phases.md +3 -3
- package/skills/machine-learning-ops-ml-pipeline/SKILL.md +7 -7
- package/skills/mcp-builder/reference/evaluation.md +3 -3
- package/skills/mcp-management/README.md +6 -6
- package/skills/mcp-management/SKILL.md +8 -8
- package/skills/mcp-management/references/gemini-cli-integration.md +1 -1
- package/skills/multi-agent-patterns/SKILL.md +14 -14
- package/skills/multi-platform-apps-multi-platform/SKILL.md +10 -10
- package/skills/nestjs-expert/SKILL.md +1 -1
- package/skills/performance-testing-review-multi-agent-review/SKILL.md +1 -1
- package/skills/planning-with-files/reference.md +2 -2
- package/skills/requesting-code-review/SKILL.md +6 -6
- package/skills/security-scanning-security-hardening/SKILL.md +13 -13
- package/skills/subagent-driven-development/SKILL.md +53 -53
- package/skills/subagent-driven-development/code-quality-reviewer-prompt.md +1 -1
- package/skills/subagent-driven-development/implementer-prompt.md +3 -3
- package/skills/subagent-driven-development/spec-reviewer-prompt.md +1 -1
- package/skills/tdd-workflows-tdd-cycle/SKILL.md +12 -12
- package/skills/tdd-workflows-tdd-green/resources/implementation-playbook.md +1 -1
- package/skills/tdd-workflows-tdd-red/SKILL.md +1 -1
- package/skills/tdd-workflows-tdd-refactor/SKILL.md +1 -1
- package/skills/typescript-expert/SKILL.md +1 -1
- package/skills/writing-plans/SKILL.md +3 -3
- package/skills/writing-skills/SKILL.md +8 -8
- package/skills/writing-skills/examples/CLAUDE_MD_TESTING.md +1 -1
- package/skills/writing-skills/references/cso/README.md +3 -3
- package/skills/writing-skills/testing-skills-with-subagents.md +1 -1
|
@@ -1,5 +1,5 @@
|
|
|
1
1
|
---
|
|
2
|
-
description:
|
|
2
|
+
description: Full Development Cycle — Plan, Implement, Test, Review
|
|
3
3
|
version: "1.0"
|
|
4
4
|
category: engineering
|
|
5
5
|
execution-mode: execute
|
|
@@ -13,41 +13,7 @@ execution-mode: execute
|
|
|
13
13
|
|
|
14
14
|
---
|
|
15
15
|
|
|
16
|
-
##
|
|
17
|
-
|
|
18
|
-
**LOAD now** (in order; path `./rules/` or `~/.{TOOL}/skills/agent-assistant/rules/`):
|
|
19
|
-
|
|
20
|
-
1. ORCHESTRATION-LAWS.md
|
|
21
|
-
2. ADAPTIVE-EXECUTION.md
|
|
22
|
-
3. EXECUTION-PROTOCOL.md
|
|
23
|
-
|
|
24
|
-
**⛔ Do not run Phase 1 until all are loaded.** Follow **all** rules in those files; they override any conflicting instructions in this file.
|
|
25
|
-
|
|
26
|
-
---
|
|
27
|
-
|
|
28
|
-
## 🔀 TIERED EXECUTION PROTOCOL (MANDATORY)
|
|
29
|
-
|
|
30
|
-
> **Reference**: `{RULES_PATH}/ADAPTIVE-EXECUTION.md`
|
|
31
|
-
|
|
32
|
-
```yaml
|
|
33
|
-
tiered_execution:
|
|
34
|
-
principle: "Sub-agent FIRST (Tier 1). EMBODY ONLY on system failure (Tier 2)."
|
|
35
|
-
|
|
36
|
-
tool_discovery: "Check runSubagent availability BEFORE first delegation"
|
|
37
|
-
|
|
38
|
-
for_each_phase:
|
|
39
|
-
TIER_1_MANDATORY: "IF tool exists → MUST use SUB_AGENT_DELEGATION"
|
|
40
|
-
TIER_2_FALLBACK: "ONLY on system error—NOT complexity/preference/speed"
|
|
41
|
-
|
|
42
|
-
anti_lazy_fallback:
|
|
43
|
-
- ❌ NEVER use Tier 2 when Tier 1 tool is available
|
|
44
|
-
- ✅ ALWAYS attempt Tier 1 first when tool exists
|
|
45
|
-
- ✅ ALWAYS log justification for Tier 2 fallback
|
|
46
|
-
```
|
|
47
|
-
|
|
48
|
-
---
|
|
49
|
-
|
|
50
|
-
## ⚠️ CRITICAL: PHASE CONTINUITY RULES
|
|
16
|
+
## CRITICAL: PHASE CONTINUITY RULES
|
|
51
17
|
|
|
52
18
|
```yaml
|
|
53
19
|
phase_continuity:
|
|
@@ -62,12 +28,12 @@ phase_continuity:
|
|
|
62
28
|
- Phase 3 (Planning) MUST incorporate Scout findings
|
|
63
29
|
- Phase 4 (Implementation) MUST follow the Plan file exactly
|
|
64
30
|
- Phase 5 (Testing) MUST verify all plan checkpoints
|
|
65
|
-
- If prior phase file missing →
|
|
31
|
+
- If prior phase file missing → MUST create it
|
|
66
32
|
```
|
|
67
33
|
|
|
68
34
|
All files in `./reports/` → English only.
|
|
69
35
|
|
|
70
|
-
##
|
|
36
|
+
## INPUT REQUIREMENTS & VERIFICATION MATRIX
|
|
71
37
|
|
|
72
38
|
```yaml
|
|
73
39
|
phase_dependencies:
|
|
@@ -109,13 +75,13 @@ phase_dependencies:
|
|
|
109
75
|
verification: "Code MUST match plan intent"
|
|
110
76
|
```
|
|
111
77
|
|
|
112
|
-
##
|
|
78
|
+
## BLOCKING ENFORCEMENT
|
|
113
79
|
|
|
114
80
|
```
|
|
115
81
|
BEFORE entering any BLOCKING phase:
|
|
116
82
|
1. CHECK: Does required input file exist?
|
|
117
83
|
2. IF missing:
|
|
118
|
-
a. OUTPUT: "
|
|
84
|
+
a. OUTPUT: "BLOCKED: Missing [{file}]"
|
|
119
85
|
b. ROUTE to creating agent
|
|
120
86
|
c. WAIT for creation
|
|
121
87
|
3. IF exists:
|
|
@@ -125,115 +91,82 @@ BEFORE entering any BLOCKING phase:
|
|
|
125
91
|
|
|
126
92
|
---
|
|
127
93
|
|
|
128
|
-
##
|
|
94
|
+
## Execution
|
|
129
95
|
|
|
130
|
-
One phase at a time,
|
|
96
|
+
One phase at a time, sequential. Each phase must complete before next begins.
|
|
131
97
|
|
|
132
98
|
---
|
|
133
99
|
|
|
134
|
-
##
|
|
100
|
+
## Phase 1: REQUIREMENTS ANALYSIS
|
|
135
101
|
|
|
136
102
|
| Attribute | Value |
|
|
137
103
|
| ----------- | --------------------------------- |
|
|
138
|
-
| **
|
|
104
|
+
| **Role** | `brainstormer` |
|
|
139
105
|
| **Goal** | Clarify requirements if ambiguous |
|
|
140
106
|
| **Trigger** | If requirements unclear |
|
|
141
107
|
|
|
142
|
-
###
|
|
143
|
-
|
|
144
|
-
**TIER 1 (MANDATORY when tool exists):**
|
|
108
|
+
### Delegation
|
|
145
109
|
|
|
146
|
-
>
|
|
147
|
-
|
|
148
|
-
**TIER 2 (FALLBACK on system error only):**
|
|
149
|
-
|
|
150
|
-
> Load `{AGENTS_PATH}/brainstormer.md`
|
|
151
|
-
> EMBODY [brainstormer] — Requires logged system error justification.
|
|
110
|
+
> Task(subagent_type="general-purpose", prompt="You are a brainstormer. Clarify requirements if ambiguous using Socratic questioning and assumption surfacing. Exit when: requirements clear, scope defined, acceptance criteria established.", description="brainstormer: Clarify requirements if ambiguous")
|
|
152
111
|
|
|
153
112
|
**Exit Criteria:**
|
|
154
113
|
|
|
155
114
|
- [ ] Requirements clear
|
|
156
115
|
- [ ] Scope defined
|
|
157
116
|
- [ ] Acceptance criteria established
|
|
158
|
-
- [ ] **METHODOLOGY CHECK**: Output aligns with `brainstormer` Thinking Protocol (Socratic questioning, assumption surfacing)
|
|
159
117
|
|
|
160
118
|
---
|
|
161
119
|
|
|
162
|
-
##
|
|
120
|
+
## Phase 2: CODEBASE ANALYSIS
|
|
163
121
|
|
|
164
122
|
| Attribute | Value |
|
|
165
123
|
| --------- | ----------------------------- |
|
|
166
|
-
| **
|
|
124
|
+
| **Role** | `scouter` |
|
|
167
125
|
| **Goal** | Full codebase context mapping |
|
|
168
126
|
|
|
169
|
-
###
|
|
170
|
-
|
|
171
|
-
**TIER 1 (MANDATORY when tool exists):**
|
|
172
|
-
|
|
173
|
-
> Invoke runSubagent for `scouter`. Context: ISOLATED.
|
|
174
|
-
|
|
175
|
-
**TIER 2 (FALLBACK on system error only):**
|
|
127
|
+
### Delegation
|
|
176
128
|
|
|
177
|
-
>
|
|
178
|
-
> EMBODY [scouter] — Requires logged system error justification.
|
|
129
|
+
> Task(subagent_type="general-purpose", prompt="You are a scouter. Perform full codebase context mapping. Exit when: architecture understood, patterns documented, integration points identified.", description="scouter: Full codebase context mapping")
|
|
179
130
|
|
|
180
131
|
**Exit Criteria:**
|
|
181
132
|
|
|
182
133
|
- [ ] Architecture understood
|
|
183
134
|
- [ ] Patterns documented
|
|
184
135
|
- [ ] Integration points identified
|
|
185
|
-
- [ ] **METHODOLOGY CHECK**: Output aligns with `scouter` Thinking Protocol (file locations, patterns as constraints)
|
|
186
136
|
|
|
187
137
|
---
|
|
188
138
|
|
|
189
|
-
##
|
|
139
|
+
## Phase 3: IMPLEMENTATION PLANNING
|
|
190
140
|
|
|
191
141
|
| Attribute | Value |
|
|
192
142
|
| --------- | -------------------------- |
|
|
193
|
-
| **
|
|
143
|
+
| **Role** | `planner` |
|
|
194
144
|
| **Goal** | Create implementation plan |
|
|
195
145
|
|
|
196
|
-
###
|
|
146
|
+
### Delegation
|
|
197
147
|
|
|
198
|
-
|
|
199
|
-
|
|
200
|
-
> Invoke runSubagent for `planner`. Context: ISOLATED.
|
|
201
|
-
|
|
202
|
-
**TIER 2 (FALLBACK on system error only):**
|
|
203
|
-
|
|
204
|
-
> Load `{AGENTS_PATH}/planner.md`
|
|
205
|
-
> EMBODY [planner] — Requires logged system error justification.
|
|
148
|
+
> Task(subagent_type="general-purpose", prompt="You are a planner. Create an implementation plan incorporating Scout findings. Exit when: plan created, steps defined, risks identified.", description="planner: Create implementation plan")
|
|
206
149
|
|
|
207
150
|
**Exit Criteria:**
|
|
208
151
|
|
|
209
152
|
- [ ] Plan created
|
|
210
153
|
- [ ] Steps defined
|
|
211
154
|
- [ ] Risks identified
|
|
212
|
-
- [ ] **METHODOLOGY CHECK**: Output aligns with `planner` Thinking Protocol (constraint consumption, complexity scoring)
|
|
213
155
|
|
|
214
156
|
---
|
|
215
157
|
|
|
216
|
-
##
|
|
158
|
+
## Phase 4: IMPLEMENTATION
|
|
217
159
|
|
|
218
160
|
| Attribute | Value |
|
|
219
161
|
| --------- | ----------------------------------- |
|
|
220
|
-
| **
|
|
162
|
+
| **Role** | `tech-lead` → routes to specialists |
|
|
221
163
|
| **Goal** | Execute plan with specialist agents |
|
|
222
164
|
|
|
223
|
-
###
|
|
224
|
-
|
|
225
|
-
**TIER 1 (MANDATORY when tool exists):**
|
|
165
|
+
### Delegation
|
|
226
166
|
|
|
227
|
-
>
|
|
228
|
-
> Tech-lead will spawn specialist subagents as needed.
|
|
167
|
+
> Task(subagent_type="general-purpose", prompt="You are a tech-lead. Execute the implementation plan, routing to specialist engineers as needed. READ the plan completely BEFORE any implementation. Implement EXACTLY as specified. If deviation needed: STOP, document, request re-planning. Exit when: all plan steps executed, code complete, documentation updated, no unauthorized deviations.", description="tech-lead: Execute implementation plan")
|
|
229
168
|
|
|
230
|
-
|
|
231
|
-
|
|
232
|
-
> Load `{AGENTS_PATH}/tech-lead.md`
|
|
233
|
-
> EMBODY [tech-lead] — Requires logged system error justification.
|
|
234
|
-
> Then recursively EMBODY specialists as needed.
|
|
235
|
-
|
|
236
|
-
### 🔗 INPUT REQUIREMENTS (BLOCKING)
|
|
169
|
+
### INPUT REQUIREMENTS (BLOCKING)
|
|
237
170
|
|
|
238
171
|
```yaml
|
|
239
172
|
required_inputs:
|
|
@@ -243,13 +176,13 @@ required_inputs:
|
|
|
243
176
|
if_missing: "STOP → Route to planner"
|
|
244
177
|
```
|
|
245
178
|
|
|
246
|
-
###
|
|
179
|
+
### STRICT ADHERENCE DIRECTIVE
|
|
247
180
|
|
|
248
181
|
```
|
|
249
182
|
1. READ plan completely BEFORE any implementation
|
|
250
183
|
2. FOR EACH plan step:
|
|
251
184
|
a. Implement EXACTLY as specified
|
|
252
|
-
b. Mark complete:
|
|
185
|
+
b. Mark complete: - [ ] → - [x]
|
|
253
186
|
3. IF deviation needed:
|
|
254
187
|
a. STOP
|
|
255
188
|
b. REQUEST Re-Planning
|
|
@@ -263,61 +196,44 @@ required_inputs:
|
|
|
263
196
|
- [ ] Documentation updated
|
|
264
197
|
- [ ] **Each plan step has corresponding implementation**
|
|
265
198
|
- [ ] **No unauthorized deviations**
|
|
266
|
-
- [ ] **METHODOLOGY CHECK**: Output aligns with `tech-lead` Thinking Protocol (proper delegation, drift detection)
|
|
267
199
|
|
|
268
200
|
---
|
|
269
201
|
|
|
270
|
-
##
|
|
202
|
+
## Phase 5: TESTING
|
|
271
203
|
|
|
272
204
|
| Attribute | Value |
|
|
273
205
|
| --------- | --------------------- |
|
|
274
|
-
| **
|
|
206
|
+
| **Role** | `tester` |
|
|
275
207
|
| **Goal** | Comprehensive testing |
|
|
276
208
|
|
|
277
|
-
###
|
|
278
|
-
|
|
279
|
-
**TIER 1 (MANDATORY when tool exists):**
|
|
280
|
-
|
|
281
|
-
> Invoke runSubagent for `tester`. Context: ISOLATED.
|
|
209
|
+
### Delegation
|
|
282
210
|
|
|
283
|
-
|
|
284
|
-
|
|
285
|
-
> Load `{AGENTS_PATH}/tester.md`
|
|
286
|
-
> EMBODY [tester] — Requires logged system error justification.
|
|
211
|
+
> Task(subagent_type="general-purpose", prompt="You are a tester. Perform comprehensive testing of the implementation. Verify all plan checkpoints. Exit when: tests written, all tests pass, coverage adequate.", description="tester: Comprehensive testing")
|
|
287
212
|
|
|
288
213
|
**Exit Criteria:**
|
|
289
214
|
|
|
290
215
|
- [ ] Tests written
|
|
291
216
|
- [ ] All tests pass
|
|
292
217
|
- [ ] Coverage adequate
|
|
293
|
-
- [ ] **METHODOLOGY CHECK**: Output aligns with `tester` Thinking Protocol (test pyramid, determinism verification)
|
|
294
218
|
|
|
295
219
|
---
|
|
296
220
|
|
|
297
|
-
##
|
|
221
|
+
## Phase 6: REVIEW
|
|
298
222
|
|
|
299
223
|
| Attribute | Value |
|
|
300
224
|
| --------- | ------------------- |
|
|
301
|
-
| **
|
|
225
|
+
| **Role** | `reviewer` |
|
|
302
226
|
| **Goal** | Code quality review |
|
|
303
227
|
|
|
304
|
-
###
|
|
305
|
-
|
|
306
|
-
**TIER 1 (MANDATORY when tool exists):**
|
|
307
|
-
|
|
308
|
-
> Invoke runSubagent for `reviewer`. Context: ISOLATED.
|
|
309
|
-
|
|
310
|
-
**TIER 2 (FALLBACK on system error only):**
|
|
228
|
+
### Delegation
|
|
311
229
|
|
|
312
|
-
>
|
|
313
|
-
> EMBODY [reviewer] — Requires logged system error justification.
|
|
230
|
+
> Task(subagent_type="general-purpose", prompt="You are a reviewer. Perform code quality review. Check plan compliance, standards, and blocking issues. Exit when: code reviewed, standards met, no blocking issues.", description="reviewer: Code quality review")
|
|
314
231
|
|
|
315
232
|
**Exit Criteria:**
|
|
316
233
|
|
|
317
234
|
- [ ] Code reviewed
|
|
318
235
|
- [ ] Standards met
|
|
319
236
|
- [ ] No blocking issues
|
|
320
|
-
- [ ] **METHODOLOGY CHECK**: Output aligns with `reviewer` Thinking Protocol (plan compliance check, priority matrix)
|
|
321
237
|
|
|
322
238
|
---
|
|
323
239
|
|
|
@@ -325,6 +241,6 @@ required_inputs:
|
|
|
325
241
|
|
|
326
242
|
Present implementation report with:
|
|
327
243
|
|
|
328
|
-
1.
|
|
329
|
-
2.
|
|
330
|
-
3.
|
|
244
|
+
1. **Done** — Feature complete
|
|
245
|
+
2. **Deploy** → `/deploy:preview`
|
|
246
|
+
3. **Docs** → `/docs:core`
|
|
@@ -3,7 +3,7 @@ description: ⚡⚡ Start coding an existing plan (no testing)
|
|
|
3
3
|
argument-hint: [plan]
|
|
4
4
|
---
|
|
5
5
|
|
|
6
|
-
**MUST READ** `CLAUDE.md` then **THINK HARDER** to start working on the following plan follow the Orchestration Protocol, Core Responsibilities,
|
|
6
|
+
**MUST READ** `CLAUDE.md` then **THINK HARDER** to start working on the following plan follow the Orchestration Protocol, Core Responsibilities, Task Agents Team and Development Rules:
|
|
7
7
|
<plan>$ARGUMENTS</plan>
|
|
8
8
|
|
|
9
9
|
---
|
|
@@ -13,7 +13,7 @@ argument-hint: [plan]
|
|
|
13
13
|
- Validate the plan's assumptions, surface blockers, and confirm priorities with the user prior to execution.
|
|
14
14
|
- Drive the implementation from start to finish, reporting progress and adjusting the plan responsibly while honoring **YAGNI**, **KISS**, and **DRY** principles.
|
|
15
15
|
|
|
16
|
-
**IMPORTANT:** Remind these rules with
|
|
16
|
+
**IMPORTANT:** Remind these rules with Task agents communication:
|
|
17
17
|
- Sacrifice grammar for the sake of concision when writing reports.
|
|
18
18
|
- In reports, list any unresolved questions at the end, if any.
|
|
19
19
|
- Ensure token efficiency while maintaining high quality.
|
|
@@ -30,9 +30,9 @@ argument-hint: [plan]
|
|
|
30
30
|
|
|
31
31
|
**Output:** `✓ Step 0: [Plan Name] - [Phase Name]`
|
|
32
32
|
|
|
33
|
-
**
|
|
33
|
+
**Task Agent Pattern (use throughout):**
|
|
34
34
|
```
|
|
35
|
-
Task(subagent_type="
|
|
35
|
+
Task(subagent_type="general-purpose", prompt="[task description]", description="[brief]")
|
|
36
36
|
```
|
|
37
37
|
|
|
38
38
|
---
|
|
@@ -65,7 +65,7 @@ Mark Step 1 complete in TodoWrite, mark Step 2 in_progress.
|
|
|
65
65
|
|
|
66
66
|
## Step 2: Implementation
|
|
67
67
|
|
|
68
|
-
Implement selected plan phase step-by-step following extracted tasks (Step 2.1, Step 2.2, etc.). Mark tasks complete as done. For UI work,
|
|
68
|
+
Implement selected plan phase step-by-step following extracted tasks (Step 2.1, Step 2.2, etc.). Mark tasks complete as done. For UI work, use a Task agent for UI/UX design: Task(subagent_type="general-purpose", prompt="You are a ui-ux-designer. Implement [feature] UI per ./docs/design-guidelines.md", description="Implement UI"). Use `ai-multimodal` skill for image assets, `imagemagick` for editing. Run type checking and compile to verify no syntax errors.
|
|
69
69
|
|
|
70
70
|
**Output:** `✓ Step 2: Implemented [N] files - [X/Y] tasks complete, compilation passed`
|
|
71
71
|
|
|
@@ -75,7 +75,7 @@ Mark Step 2 complete in TodoWrite, mark Step 3 in_progress.
|
|
|
75
75
|
|
|
76
76
|
## Step 3: Code Review & Approval ⏸ BLOCKING GATE
|
|
77
77
|
|
|
78
|
-
|
|
78
|
+
Use a Task agent for code review: Task(subagent_type="general-purpose", prompt="You are a code-reviewer. Review changes for plan phase [phase-name]. Check security, performance, architecture, YAGNI/KISS/DRY. Return score (X/10), critical issues list, warnings list, suggestions list.", description="Code review")
|
|
79
79
|
|
|
80
80
|
**Display + Approve Flow (optimized for speed):**
|
|
81
81
|
|
|
@@ -126,8 +126,8 @@ Mark Step 3 complete in TodoWrite, mark Step 4 in_progress.
|
|
|
126
126
|
**Prerequisites:** User approved in Step 3 (verified above).
|
|
127
127
|
|
|
128
128
|
1. **STATUS UPDATE - BOTH MANDATORY - PARALLEL EXECUTION:**
|
|
129
|
-
- **
|
|
130
|
-
- **
|
|
129
|
+
- **Use** Task agent for project management: Task(subagent_type="general-purpose", prompt="You are a project-manager. Update plan status in [plan-path]. Mark plan phase [phase-name] as DONE with timestamp. Update roadmap.", description="Update plan status")
|
|
130
|
+
- **Use** Task agent for documentation: Task(subagent_type="general-purpose", prompt="You are a docs-manager. Update docs for plan phase [phase-name]. Changed files: [list].", description="Update docs")
|
|
131
131
|
|
|
132
132
|
2. **ONBOARDING CHECK:** Detect onboarding requirements (API keys, env vars, config) + generate summary report with next steps.
|
|
133
133
|
|
|
@@ -158,13 +158,13 @@ Mark Step 4 complete in TodoWrite.
|
|
|
158
158
|
|
|
159
159
|
**TodoWrite tracking required:** Initialize at Step 0, mark each step complete before next.
|
|
160
160
|
|
|
161
|
-
**Mandatory
|
|
162
|
-
- Step 3:
|
|
163
|
-
- Step 4:
|
|
161
|
+
**Mandatory Task agent calls:**
|
|
162
|
+
- Step 3: Task agent for code review
|
|
163
|
+
- Step 4: Task agent for project management AND Task agent for documentation (when user approves)
|
|
164
164
|
|
|
165
165
|
**Blocking gates:**
|
|
166
166
|
- Step 3: User must explicitly approve (via AskUserQuestion)
|
|
167
|
-
- Step 4: Both
|
|
167
|
+
- Step 4: Both Task agents for project management and documentation must complete successfully
|
|
168
168
|
|
|
169
169
|
**REMEMBER:**
|
|
170
170
|
- Do not skip steps. Do not proceed if validation fails. Do not assume approval without user response.
|
|
@@ -16,7 +16,7 @@ Execute plan: <plan>$ARGUMENTS</plan>
|
|
|
16
16
|
|
|
17
17
|
### 2A. Parallel Execution
|
|
18
18
|
1. Parse execution strategy (which phases concurrent/sequential, file ownership)
|
|
19
|
-
2. Launch multiple
|
|
19
|
+
2. Launch multiple Task agents for fullstack development simultaneously for parallel phases
|
|
20
20
|
- Pass: phase file path, environment info, file ownership boundaries
|
|
21
21
|
3. Wait for parallel group completion, verify no conflicts
|
|
22
22
|
4. Execute sequential phases (one agent per phase after dependencies)
|
|
@@ -26,18 +26,18 @@ Execute plan: <plan>$ARGUMENTS</plan>
|
|
|
26
26
|
Follow `./.claude/rules/primary-workflow.md`:
|
|
27
27
|
1. Use main agent step by step
|
|
28
28
|
2. Read `plan.md`, implement phases one by one
|
|
29
|
-
3. Use
|
|
30
|
-
4. Use
|
|
29
|
+
3. Use Task agent for project management for progress updates
|
|
30
|
+
4. Use Task agent for UI/UX design for frontend
|
|
31
31
|
5. Run type checking after each phase
|
|
32
32
|
6. Proceed to Step 3
|
|
33
33
|
|
|
34
34
|
### 3. Testing
|
|
35
|
-
- Use
|
|
36
|
-
- If fail:
|
|
35
|
+
- Use Task agent for testing for full suite (NO fake data/mocks)
|
|
36
|
+
- If fail: Task agent for debugging → fix → repeat
|
|
37
37
|
|
|
38
38
|
### 4. Code Review (Interactive Cycle)
|
|
39
39
|
|
|
40
|
-
|
|
40
|
+
Use a Task agent for code review: Task(subagent_type="general-purpose", prompt="You are a code-reviewer. Review all changes from parallel/sequential execution. Check security, performance, architecture, YAGNI/KISS/DRY. Return score (X/10), critical issues list, warnings list, suggestions list.", description="Code review")
|
|
41
41
|
|
|
42
42
|
**Interactive Review-Fix Cycle (max 3 cycles):**
|
|
43
43
|
|
|
@@ -85,7 +85,7 @@ LOOP:
|
|
|
85
85
|
```
|
|
86
86
|
|
|
87
87
|
### 5. Project Management & Docs
|
|
88
|
-
- If approved:
|
|
88
|
+
- If approved: Task agent for project management + Task agent for documentation in parallel (update plans, docs, roadmap)
|
|
89
89
|
- If rejected: fix → repeat
|
|
90
90
|
|
|
91
91
|
### 6. Onboarding
|
|
@@ -93,8 +93,8 @@ LOOP:
|
|
|
93
93
|
|
|
94
94
|
### 7. Final Report
|
|
95
95
|
- Summary, guide, next steps
|
|
96
|
-
- Ask to commit (use
|
|
96
|
+
- Ask to commit (use Task agent for git operations if yes)
|
|
97
97
|
|
|
98
98
|
**Examples:**
|
|
99
|
-
- Parallel: "Phases 1-3 parallel, then 4" → Launch 3 agents → Wait → Launch 1 agent
|
|
99
|
+
- Parallel: "Phases 1-3 parallel, then 4" → Launch 3 Task agents → Wait → Launch 1 Task agent
|
|
100
100
|
- Sequential: "Phase 1 → 2 → 3" → Main agent implements each phase
|
package/merged-commands/code.md
CHANGED
|
@@ -3,7 +3,7 @@ description: ⚡⚡⚡ Start coding & testing an existing plan
|
|
|
3
3
|
argument-hint: [plan]
|
|
4
4
|
---
|
|
5
5
|
|
|
6
|
-
**MUST READ** `CLAUDE.md` then **THINK HARDER** to start working on the following plan follow the Orchestration Protocol, Core Responsibilities,
|
|
6
|
+
**MUST READ** `CLAUDE.md` then **THINK HARDER** to start working on the following plan follow the Orchestration Protocol, Core Responsibilities, Task Agents Team and Development Rules:
|
|
7
7
|
<plan>$ARGUMENTS</plan>
|
|
8
8
|
|
|
9
9
|
---
|
|
@@ -13,7 +13,7 @@ argument-hint: [plan]
|
|
|
13
13
|
- Validate the plan's assumptions, surface blockers, and confirm priorities with the user prior to execution.
|
|
14
14
|
- Drive the implementation from start to finish, reporting progress and adjusting the plan responsibly while honoring **YAGNI**, **KISS**, and **DRY** principles.
|
|
15
15
|
|
|
16
|
-
**IMPORTANT:** Remind these rules with
|
|
16
|
+
**IMPORTANT:** Remind these rules with Task agents communication:
|
|
17
17
|
- Sacrifice grammar for the sake of concision when writing reports.
|
|
18
18
|
- In reports, list any unresolved questions at the end, if any.
|
|
19
19
|
- Ensure token efficiency while maintaining high quality.
|
|
@@ -36,9 +36,9 @@ Example: `plans/260108-1418-custom-domain-refactor/phase-01-database.md` → run
|
|
|
36
36
|
|
|
37
37
|
**Output:** `✓ Step 0: [Plan Name] - [Phase Name]`
|
|
38
38
|
|
|
39
|
-
**
|
|
39
|
+
**Task Agent Pattern (use throughout):**
|
|
40
40
|
```
|
|
41
|
-
Task(subagent_type="
|
|
41
|
+
Task(subagent_type="general-purpose", prompt="[task description]", description="[brief]")
|
|
42
42
|
```
|
|
43
43
|
|
|
44
44
|
---
|
|
@@ -72,7 +72,7 @@ Mark Step 1 complete in TodoWrite, mark Step 2 in_progress.
|
|
|
72
72
|
|
|
73
73
|
## Step 2: Implementation
|
|
74
74
|
|
|
75
|
-
Implement selected plan phase step-by-step following extracted tasks (Step 2.1, Step 2.2, etc.). Mark tasks complete as done. For UI work,
|
|
75
|
+
Implement selected plan phase step-by-step following extracted tasks (Step 2.1, Step 2.2, etc.). Mark tasks complete as done. For UI work, use a Task agent for UI/UX design: Task(subagent_type="general-purpose", prompt="You are a ui-ux-designer. Implement [feature] UI per ./docs/design-guidelines.md", description="Implement UI"). Use `ai-multimodal` skill for image assets, `imagemagick` for editing. Run type checking and compile to verify no syntax errors.
|
|
76
76
|
|
|
77
77
|
**Output:** `✓ Step 2: Implemented [N] files - [X/Y] tasks complete, compilation passed`
|
|
78
78
|
|
|
@@ -82,7 +82,7 @@ Mark Step 2 complete in TodoWrite, mark Step 3 in_progress.
|
|
|
82
82
|
|
|
83
83
|
## Step 3: Testing
|
|
84
84
|
|
|
85
|
-
Write tests covering happy path, edge cases, and error cases.
|
|
85
|
+
Write tests covering happy path, edge cases, and error cases. Use a Task agent for testing: Task(subagent_type="general-purpose", prompt="You are a tester. Run test suite for plan phase [phase-name]", description="Run tests"). If ANY tests fail: STOP, use a Task agent for debugging: Task(subagent_type="general-purpose", prompt="You are a debugger. Analyze failures: [details]", description="Debug test failures"), fix all issues, re-run tester. Repeat until 100% pass.
|
|
86
86
|
|
|
87
87
|
**Testing standards:** Unit tests may use mocks for external dependencies (APIs, DB). Integration tests use test environment. E2E tests use real but isolated data. Forbidden: commenting out tests, changing assertions to pass, TODO/FIXME to defer fixes.
|
|
88
88
|
|
|
@@ -96,7 +96,7 @@ Mark Step 3 complete in TodoWrite, mark Step 4 in_progress.
|
|
|
96
96
|
|
|
97
97
|
## Step 4: Code Review & Approval ⏸ BLOCKING GATE
|
|
98
98
|
|
|
99
|
-
|
|
99
|
+
Use a Task agent for code review: Task(subagent_type="general-purpose", prompt="You are a code-reviewer. Review changes for plan phase [phase-name]. Check security, performance, architecture, YAGNI/KISS/DRY. Return score (X/10), critical issues list, warnings list, suggestions list.", description="Code review")
|
|
100
100
|
|
|
101
101
|
**Interactive Review-Fix Cycle (max 3 cycles):**
|
|
102
102
|
|
|
@@ -154,8 +154,8 @@ Mark Step 4 complete in TodoWrite, mark Step 5 in_progress.
|
|
|
154
154
|
**Prerequisites:** User approved in Step 4 (verified above).
|
|
155
155
|
|
|
156
156
|
1. **STATUS UPDATE - BOTH MANDATORY - PARALLEL EXECUTION:**
|
|
157
|
-
- **
|
|
158
|
-
- **
|
|
157
|
+
- **Use** Task agent for project management: Task(subagent_type="general-purpose", prompt="You are a project-manager. Update plan status in [plan-path]. Mark plan phase [phase-name] as DONE with timestamp. Update roadmap.", description="Update plan status")
|
|
158
|
+
- **Use** Task agent for documentation: Task(subagent_type="general-purpose", prompt="You are a docs-manager. Update docs for plan phase [phase-name]. Changed files: [list].", description="Update docs")
|
|
159
159
|
|
|
160
160
|
2. **ONBOARDING CHECK:** Detect onboarding requirements (API keys, env vars, config) + generate summary report with next steps.
|
|
161
161
|
|
|
@@ -187,15 +187,15 @@ Mark Step 5 complete in TodoWrite.
|
|
|
187
187
|
|
|
188
188
|
**TodoWrite tracking required:** Initialize at Step 0, mark each step complete before next.
|
|
189
189
|
|
|
190
|
-
**Mandatory
|
|
191
|
-
- Step 3:
|
|
192
|
-
- Step 4:
|
|
193
|
-
- Step 5:
|
|
190
|
+
**Mandatory Task agent calls:**
|
|
191
|
+
- Step 3: Task agent for testing
|
|
192
|
+
- Step 4: Task agent for code review
|
|
193
|
+
- Step 5: Task agent for project management AND Task agent for documentation (when user approves)
|
|
194
194
|
|
|
195
195
|
**Blocking gates:**
|
|
196
196
|
- Step 3: Tests must be 100% passing
|
|
197
197
|
- Step 4: User must explicitly approve (via AskUserQuestion)
|
|
198
|
-
- Step 5: Both
|
|
198
|
+
- Step 5: Both Task agents for project management and documentation must complete successfully
|
|
199
199
|
|
|
200
200
|
**REMEMBER:**
|
|
201
201
|
- Do not skip steps. Do not proceed if validation fails. Do not assume approval without user response.
|
|
@@ -41,13 +41,13 @@ Orchestrate comprehensive multi-dimensional code review using specialized review
|
|
|
41
41
|
Use Task tool to orchestrate quality and architecture agents in parallel:
|
|
42
42
|
|
|
43
43
|
### 1A. Code Quality Analysis
|
|
44
|
-
- Use Task tool with subagent_type="
|
|
44
|
+
- Use Task tool with subagent_type="general-purpose"
|
|
45
45
|
- Prompt: "Perform comprehensive code quality review for: $ARGUMENTS. Analyze code complexity, maintainability index, technical debt, code duplication, naming conventions, and adherence to Clean Code principles. Integrate with SonarQube, CodeQL, and Semgrep for static analysis. Check for code smells, anti-patterns, and violations of SOLID principles. Generate cyclomatic complexity metrics and identify refactoring opportunities."
|
|
46
46
|
- Expected output: Quality metrics, code smell inventory, refactoring recommendations
|
|
47
47
|
- Context: Initial codebase analysis, no dependencies on other phases
|
|
48
48
|
|
|
49
49
|
### 1B. Architecture & Design Review
|
|
50
|
-
- Use Task tool with subagent_type="
|
|
50
|
+
- Use Task tool with subagent_type="general-purpose"
|
|
51
51
|
- Prompt: "Review architectural design patterns and structural integrity in: $ARGUMENTS. Evaluate microservices boundaries, API design, database schema, dependency management, and adherence to Domain-Driven Design principles. Check for circular dependencies, inappropriate coupling, missing abstractions, and architectural drift. Verify compliance with enterprise architecture standards and cloud-native patterns."
|
|
52
52
|
- Expected output: Architecture assessment, design pattern analysis, structural recommendations
|
|
53
53
|
- Context: Runs parallel with code quality analysis
|
|
@@ -57,13 +57,13 @@ Use Task tool to orchestrate quality and architecture agents in parallel:
|
|
|
57
57
|
Use Task tool with security and performance agents, incorporating Phase 1 findings:
|
|
58
58
|
|
|
59
59
|
### 2A. Security Vulnerability Assessment
|
|
60
|
-
- Use Task tool with subagent_type="
|
|
60
|
+
- Use Task tool with subagent_type="general-purpose"
|
|
61
61
|
- Prompt: "Execute comprehensive security audit on: $ARGUMENTS. Perform OWASP Top 10 analysis, dependency vulnerability scanning with Snyk/Trivy, secrets detection with GitLeaks, input validation review, authentication/authorization assessment, and cryptographic implementation review. Include findings from Phase 1 architecture review: {phase1_architecture_context}. Check for SQL injection, XSS, CSRF, insecure deserialization, and configuration security issues."
|
|
62
62
|
- Expected output: Vulnerability report, CVE list, security risk matrix, remediation steps
|
|
63
63
|
- Context: Incorporates architectural vulnerabilities identified in Phase 1B
|
|
64
64
|
|
|
65
65
|
### 2B. Performance & Scalability Analysis
|
|
66
|
-
- Use Task tool with subagent_type="
|
|
66
|
+
- Use Task tool with subagent_type="general-purpose"
|
|
67
67
|
- Prompt: "Conduct performance analysis and scalability assessment for: $ARGUMENTS. Profile code for CPU/memory hotspots, analyze database query performance, review caching strategies, identify N+1 problems, assess connection pooling, and evaluate asynchronous processing patterns. Consider architectural findings from Phase 1: {phase1_architecture_context}. Check for memory leaks, resource contention, and bottlenecks under load."
|
|
68
68
|
- Expected output: Performance metrics, bottleneck analysis, optimization recommendations
|
|
69
69
|
- Context: Uses architecture insights to identify systemic performance issues
|
|
@@ -73,13 +73,13 @@ Use Task tool with security and performance agents, incorporating Phase 1 findin
|
|
|
73
73
|
Use Task tool for test and documentation quality assessment:
|
|
74
74
|
|
|
75
75
|
### 3A. Test Coverage & Quality Analysis
|
|
76
|
-
- Use Task tool with subagent_type="
|
|
76
|
+
- Use Task tool with subagent_type="general-purpose"
|
|
77
77
|
- Prompt: "Evaluate testing strategy and implementation for: $ARGUMENTS. Analyze unit test coverage, integration test completeness, end-to-end test scenarios, test pyramid adherence, and test maintainability. Review test quality metrics including assertion density, test isolation, mock usage, and flakiness. Consider security and performance test requirements from Phase 2: {phase2_security_context}, {phase2_performance_context}. Verify TDD practices if --tdd-review flag is set."
|
|
78
78
|
- Expected output: Coverage report, test quality metrics, testing gap analysis
|
|
79
79
|
- Context: Incorporates security and performance testing requirements from Phase 2
|
|
80
80
|
|
|
81
81
|
### 3B. Documentation & API Specification Review
|
|
82
|
-
- Use Task tool with subagent_type="
|
|
82
|
+
- Use Task tool with subagent_type="general-purpose"
|
|
83
83
|
- Prompt: "Review documentation completeness and quality for: $ARGUMENTS. Assess inline code documentation, API documentation (OpenAPI/Swagger), architecture decision records (ADRs), README completeness, deployment guides, and runbooks. Verify documentation reflects actual implementation based on all previous phase findings: {phase1_context}, {phase2_context}. Check for outdated documentation, missing examples, and unclear explanations."
|
|
84
84
|
- Expected output: Documentation coverage report, inconsistency list, improvement recommendations
|
|
85
85
|
- Context: Cross-references all previous findings to ensure documentation accuracy
|
|
@@ -89,13 +89,13 @@ Use Task tool for test and documentation quality assessment:
|
|
|
89
89
|
Use Task tool to verify framework-specific and industry best practices:
|
|
90
90
|
|
|
91
91
|
### 4A. Framework & Language Best Practices
|
|
92
|
-
- Use Task tool with subagent_type="
|
|
92
|
+
- Use Task tool with subagent_type="general-purpose"
|
|
93
93
|
- Prompt: "Verify adherence to framework and language best practices for: $ARGUMENTS. Check modern JavaScript/TypeScript patterns, React hooks best practices, Python PEP compliance, Java enterprise patterns, Go idiomatic code, or framework-specific conventions (based on --framework flag). Review package management, build configuration, environment handling, and deployment practices. Include all quality issues from previous phases: {all_previous_contexts}."
|
|
94
94
|
- Expected output: Best practices compliance report, modernization recommendations
|
|
95
95
|
- Context: Synthesizes all previous findings for framework-specific guidance
|
|
96
96
|
|
|
97
97
|
### 4B. CI/CD & DevOps Practices Review
|
|
98
|
-
- Use Task tool with subagent_type="
|
|
98
|
+
- Use Task tool with subagent_type="general-purpose"
|
|
99
99
|
- Prompt: "Review CI/CD pipeline and DevOps practices for: $ARGUMENTS. Evaluate build automation, test automation integration, deployment strategies (blue-green, canary), infrastructure as code, monitoring/observability setup, and incident response procedures. Assess pipeline security, artifact management, and rollback capabilities. Consider all issues identified in previous phases that impact deployment: {all_critical_issues}."
|
|
100
100
|
- Expected output: Pipeline assessment, DevOps maturity evaluation, automation recommendations
|
|
101
101
|
- Context: Focuses on operationalizing fixes for all identified issues
|
|
@@ -157,11 +157,11 @@ Four strategies address different aspects of context degradation:
|
|
|
157
157
|
|
|
158
158
|
**Compress**: Reduce tokens while preserving information through summarization, abstraction, and observation masking. This extends effective context capacity.
|
|
159
159
|
|
|
160
|
-
**Isolate**: Split context across
|
|
160
|
+
**Isolate**: Split context across Task agents or sessions to prevent any single context from growing large enough to degrade. This is the most aggressive strategy but often the most effective.
|
|
161
161
|
|
|
162
162
|
### Architectural Patterns
|
|
163
163
|
|
|
164
|
-
Implement these strategies through specific architectural patterns. Use just-in-time context loading to retrieve information only when needed. Use observation masking to replace verbose tool outputs with compact references. Use
|
|
164
|
+
Implement these strategies through specific architectural patterns. Use just-in-time context loading to retrieve information only when needed. Use observation masking to replace verbose tool outputs with compact references. Use Task agent architectures to isolate context for different tasks. Use compaction to summarize growing context before it exceeds limits.
|
|
165
165
|
|
|
166
166
|
## Examples
|
|
167
167
|
|
|
@@ -25,13 +25,13 @@ Context engineering curates the smallest high-signal token set for LLM tasks. Th
|
|
|
25
25
|
1. **Context quality > quantity** - High-signal tokens beat exhaustive content
|
|
26
26
|
2. **Attention is finite** - U-shaped curve favors beginning/end positions
|
|
27
27
|
3. **Progressive disclosure** - Load information just-in-time
|
|
28
|
-
4. **Isolation prevents degradation** - Partition work across
|
|
28
|
+
4. **Isolation prevents degradation** - Partition work across Task agents
|
|
29
29
|
5. **Measure before optimizing** - Know your baseline
|
|
30
30
|
|
|
31
31
|
**IMPORTANT:**
|
|
32
32
|
- Sacrifice grammar for the sake of concision.
|
|
33
33
|
- Ensure token efficiency while maintaining high quality.
|
|
34
|
-
- Pass these rules to
|
|
34
|
+
- Pass these rules to Task agents.
|
|
35
35
|
|
|
36
36
|
## Quick Reference
|
|
37
37
|
|
|
@@ -61,7 +61,7 @@ Context engineering curates the smallest high-signal token set for LLM tasks. Th
|
|
|
61
61
|
1. **Write**: Save context externally (scratchpads, files)
|
|
62
62
|
2. **Select**: Pull only relevant context (retrieval, filtering)
|
|
63
63
|
3. **Compress**: Reduce tokens while preserving info (summarization)
|
|
64
|
-
4. **Isolate**: Split across
|
|
64
|
+
4. **Isolate**: Split across Task agents (partitioning)
|
|
65
65
|
|
|
66
66
|
## Anti-Patterns
|
|
67
67
|
|
|
@@ -75,7 +75,7 @@ Context engineering curates the smallest high-signal token set for LLM tasks. Th
|
|
|
75
75
|
|
|
76
76
|
1. Place critical info at beginning/end of context
|
|
77
77
|
2. Implement compaction at 70-80% utilization
|
|
78
|
-
3. Use
|
|
78
|
+
3. Use Task agents for context isolation, not role-play
|
|
79
79
|
4. Design tools with 4-question framework (what, when, inputs, returns)
|
|
80
80
|
5. Optimize for tokens-per-task, not tokens-per-request
|
|
81
81
|
6. Validate with probe-based evaluation
|