opencodekit 0.6.0 → 0.6.2
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/dist/index.js +1 -1
- package/dist/template/.opencode/AGENTS.md +4 -1
- package/dist/template/.opencode/command/skill-create.md +3 -3
- package/dist/template/.opencode/command/skill-optimize.md +1 -1
- package/dist/template/.opencode/dcp.jsonc +63 -41
- package/dist/template/.opencode/opencode.json +5 -4
- package/dist/template/.opencode/package.json +2 -2
- package/dist/template/.opencode/pickle-thinker.jsonc +11 -0
- package/dist/template/.opencode/plugin/skill.ts +275 -0
- package/dist/template/.opencode/{skills → skill}/accessibility-audit/SKILL.md +5 -0
- package/dist/template/.opencode/{skills → skill}/brainstorming/SKILL.md +2 -2
- package/dist/template/.opencode/{skills → skill}/design-system-audit/SKILL.md +5 -0
- package/dist/template/.opencode/{skills → skill}/executing-plans/SKILL.md +13 -2
- package/dist/template/.opencode/{skills → skill}/frontend-aesthetics/SKILL.md +5 -0
- package/dist/template/.opencode/{skills → skill}/mockup-to-code/SKILL.md +5 -0
- package/dist/template/.opencode/{skills → skill}/requesting-code-review/SKILL.md +16 -6
- package/dist/template/.opencode/{skills → skill}/subagent-driven-development/SKILL.md +38 -17
- package/dist/template/.opencode/{skills → skill}/systematic-debugging/SKILL.md +28 -18
- package/dist/template/.opencode/{skills → skill}/testing-skills-with-subagents/SKILL.md +1 -1
- package/dist/template/.opencode/{skills → skill}/ui-ux-research/SKILL.md +5 -0
- package/dist/template/.opencode/{skills → skill}/visual-analysis/SKILL.md +5 -0
- package/dist/template/.opencode/{skills → skill}/writing-plans/SKILL.md +3 -3
- package/dist/template/.opencode/{skills → skill}/writing-skills/SKILL.md +101 -41
- package/package.json +1 -1
- package/dist/template/.opencode/plugin/superpowers.ts +0 -271
- package/dist/template/.opencode/superpowers/.claude/settings.local.json +0 -141
- package/dist/template/.opencode/superpowers/.claude-plugin/marketplace.json +0 -20
- package/dist/template/.opencode/superpowers/.claude-plugin/plugin.json +0 -13
- package/dist/template/.opencode/superpowers/.codex/INSTALL.md +0 -35
- package/dist/template/.opencode/superpowers/.codex/superpowers-bootstrap.md +0 -33
- package/dist/template/.opencode/superpowers/.codex/superpowers-codex +0 -267
- package/dist/template/.opencode/superpowers/.github/FUNDING.yml +0 -3
- package/dist/template/.opencode/superpowers/.opencode/INSTALL.md +0 -135
- package/dist/template/.opencode/superpowers/.opencode/plugin/superpowers.js +0 -215
- package/dist/template/.opencode/superpowers/LICENSE +0 -21
- package/dist/template/.opencode/superpowers/README.md +0 -165
- package/dist/template/.opencode/superpowers/RELEASE-NOTES.md +0 -493
- package/dist/template/.opencode/superpowers/agents/code-reviewer.md +0 -48
- package/dist/template/.opencode/superpowers/commands/brainstorm.md +0 -5
- package/dist/template/.opencode/superpowers/commands/execute-plan.md +0 -5
- package/dist/template/.opencode/superpowers/commands/write-plan.md +0 -5
- package/dist/template/.opencode/superpowers/docs/README.codex.md +0 -153
- package/dist/template/.opencode/superpowers/docs/README.opencode.md +0 -234
- package/dist/template/.opencode/superpowers/docs/plans/2025-11-22-opencode-support-design.md +0 -294
- package/dist/template/.opencode/superpowers/docs/plans/2025-11-22-opencode-support-implementation.md +0 -1095
- package/dist/template/.opencode/superpowers/hooks/hooks.json +0 -15
- package/dist/template/.opencode/superpowers/hooks/session-start.sh +0 -34
- package/dist/template/.opencode/superpowers/lib/skills-core.js +0 -208
- package/dist/template/.opencode/superpowers/tests/opencode/run-tests.sh +0 -165
- package/dist/template/.opencode/superpowers/tests/opencode/setup.sh +0 -73
- package/dist/template/.opencode/superpowers/tests/opencode/test-plugin-loading.sh +0 -81
- package/dist/template/.opencode/superpowers/tests/opencode/test-priority.sh +0 -198
- package/dist/template/.opencode/superpowers/tests/opencode/test-skills-core.sh +0 -440
- package/dist/template/.opencode/superpowers/tests/opencode/test-tools.sh +0 -104
- /package/dist/template/.opencode/{skills → skill}/condition-based-waiting/SKILL.md +0 -0
- /package/dist/template/.opencode/{skills → skill}/condition-based-waiting/example.ts +0 -0
- /package/dist/template/.opencode/{skills → skill}/defense-in-depth/SKILL.md +0 -0
- /package/dist/template/.opencode/{skills → skill}/dispatching-parallel-agents/SKILL.md +0 -0
- /package/dist/template/.opencode/{skills → skill}/finishing-a-development-branch/SKILL.md +0 -0
- /package/dist/template/.opencode/{skills → skill}/gemini-large-context/SKILL.md +0 -0
- /package/dist/template/.opencode/{skills → skill}/receiving-code-review/SKILL.md +0 -0
- /package/dist/template/.opencode/{skills/requesting-code-review/code-reviewer.md → skill/requesting-code-review/review.md} +0 -0
- /package/dist/template/.opencode/{skills → skill}/root-cause-tracing/SKILL.md +0 -0
- /package/dist/template/.opencode/{skills → skill}/root-cause-tracing/find-polluter.sh +0 -0
- /package/dist/template/.opencode/{skills → skill}/sharing-skills/SKILL.md +0 -0
- /package/dist/template/.opencode/{skills → skill}/systematic-debugging/CREATION-LOG.md +0 -0
- /package/dist/template/.opencode/{skills → skill}/systematic-debugging/test-academic.md +0 -0
- /package/dist/template/.opencode/{skills → skill}/systematic-debugging/test-pressure-1.md +0 -0
- /package/dist/template/.opencode/{skills → skill}/systematic-debugging/test-pressure-2.md +0 -0
- /package/dist/template/.opencode/{skills → skill}/systematic-debugging/test-pressure-3.md +0 -0
- /package/dist/template/.opencode/{skills → skill}/test-driven-development/SKILL.md +0 -0
- /package/dist/template/.opencode/{skills → skill}/testing-anti-patterns/SKILL.md +0 -0
- /package/dist/template/.opencode/{skills → skill}/testing-skills-with-subagents/examples/CLAUDE_MD_TESTING.md +0 -0
- /package/dist/template/.opencode/{skills → skill}/using-git-worktrees/SKILL.md +0 -0
- /package/dist/template/.opencode/{skills → skill}/using-superpowers/SKILL.md +0 -0
- /package/dist/template/.opencode/{skills → skill}/verification-before-completion/SKILL.md +0 -0
- /package/dist/template/.opencode/{skills → skill}/writing-skills/anthropic-best-practices.md +0 -0
- /package/dist/template/.opencode/{skills → skill}/writing-skills/graphviz-conventions.dot +0 -0
- /package/dist/template/.opencode/{skills → skill}/writing-skills/persuasion-principles.md +0 -0
|
@@ -1,22 +1,24 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: requesting-code-review
|
|
3
|
-
description: Use when completing tasks, implementing major features, or before merging to verify work meets requirements - dispatches
|
|
3
|
+
description: Use when completing tasks, implementing major features, or before merging to verify work meets requirements - dispatches review subagent to review implementation against plan or requirements before proceeding
|
|
4
4
|
---
|
|
5
5
|
|
|
6
6
|
# Requesting Code Review
|
|
7
7
|
|
|
8
|
-
Dispatch
|
|
8
|
+
Dispatch review subagent to catch issues before they cascade.
|
|
9
9
|
|
|
10
10
|
**Core principle:** Review early, review often.
|
|
11
11
|
|
|
12
12
|
## When to Request Review
|
|
13
13
|
|
|
14
14
|
**Mandatory:**
|
|
15
|
+
|
|
15
16
|
- After each task in subagent-driven development
|
|
16
17
|
- After completing major feature
|
|
17
18
|
- Before merge to main
|
|
18
19
|
|
|
19
20
|
**Optional but valuable:**
|
|
21
|
+
|
|
20
22
|
- When stuck (fresh perspective)
|
|
21
23
|
- Before refactoring (baseline check)
|
|
22
24
|
- After fixing complex bug
|
|
@@ -24,16 +26,18 @@ Dispatch superpowers:code-reviewer subagent to catch issues before they cascade.
|
|
|
24
26
|
## How to Request
|
|
25
27
|
|
|
26
28
|
**1. Get git SHAs:**
|
|
29
|
+
|
|
27
30
|
```bash
|
|
28
31
|
BASE_SHA=$(git rev-parse HEAD~1) # or origin/main
|
|
29
32
|
HEAD_SHA=$(git rev-parse HEAD)
|
|
30
33
|
```
|
|
31
34
|
|
|
32
|
-
**2. Dispatch
|
|
35
|
+
**2. Dispatch review subagent:**
|
|
33
36
|
|
|
34
|
-
Use Task tool with
|
|
37
|
+
Use Task tool with review agent type, fill template at `review.md`
|
|
35
38
|
|
|
36
39
|
**Placeholders:**
|
|
40
|
+
|
|
37
41
|
- `{WHAT_WAS_IMPLEMENTED}` - What you just built
|
|
38
42
|
- `{PLAN_OR_REQUIREMENTS}` - What it should do
|
|
39
43
|
- `{BASE_SHA}` - Starting commit
|
|
@@ -41,6 +45,7 @@ Use Task tool with superpowers:code-reviewer type, fill template at `code-review
|
|
|
41
45
|
- `{DESCRIPTION}` - Brief summary
|
|
42
46
|
|
|
43
47
|
**3. Act on feedback:**
|
|
48
|
+
|
|
44
49
|
- Fix Critical issues immediately
|
|
45
50
|
- Fix Important issues before proceeding
|
|
46
51
|
- Note Minor issues for later
|
|
@@ -56,7 +61,7 @@ You: Let me request code review before proceeding.
|
|
|
56
61
|
BASE_SHA=$(git log --oneline | grep "Task 1" | head -1 | awk '{print $1}')
|
|
57
62
|
HEAD_SHA=$(git rev-parse HEAD)
|
|
58
63
|
|
|
59
|
-
[Dispatch
|
|
64
|
+
[Dispatch review subagent]
|
|
60
65
|
WHAT_WAS_IMPLEMENTED: Verification and repair functions for conversation index
|
|
61
66
|
PLAN_OR_REQUIREMENTS: Task 2 from docs/plans/deployment-plan.md
|
|
62
67
|
BASE_SHA: a7981ec
|
|
@@ -77,29 +82,34 @@ You: [Fix progress indicators]
|
|
|
77
82
|
## Integration with Workflows
|
|
78
83
|
|
|
79
84
|
**Subagent-Driven Development:**
|
|
85
|
+
|
|
80
86
|
- Review after EACH task
|
|
81
87
|
- Catch issues before they compound
|
|
82
88
|
- Fix before moving to next task
|
|
83
89
|
|
|
84
90
|
**Executing Plans:**
|
|
91
|
+
|
|
85
92
|
- Review after each batch (3 tasks)
|
|
86
93
|
- Get feedback, apply, continue
|
|
87
94
|
|
|
88
95
|
**Ad-Hoc Development:**
|
|
96
|
+
|
|
89
97
|
- Review before merge
|
|
90
98
|
- Review when stuck
|
|
91
99
|
|
|
92
100
|
## Red Flags
|
|
93
101
|
|
|
94
102
|
**Never:**
|
|
103
|
+
|
|
95
104
|
- Skip review because "it's simple"
|
|
96
105
|
- Ignore Critical issues
|
|
97
106
|
- Proceed with unfixed Important issues
|
|
98
107
|
- Argue with valid technical feedback
|
|
99
108
|
|
|
100
109
|
**If reviewer wrong:**
|
|
110
|
+
|
|
101
111
|
- Push back with technical reasoning
|
|
102
112
|
- Show code/tests that prove it works
|
|
103
113
|
- Request clarification
|
|
104
114
|
|
|
105
|
-
See template at: requesting-code-review/
|
|
115
|
+
See template at: requesting-code-review/review.md
|
|
@@ -12,17 +12,20 @@ Execute plan by dispatching fresh subagent per task, with code review after each
|
|
|
12
12
|
## Overview
|
|
13
13
|
|
|
14
14
|
**vs. Executing Plans (parallel session):**
|
|
15
|
+
|
|
15
16
|
- Same session (no context switch)
|
|
16
17
|
- Fresh subagent per task (no context pollution)
|
|
17
18
|
- Code review after each task (catch issues early)
|
|
18
19
|
- Faster iteration (no human-in-loop between tasks)
|
|
19
20
|
|
|
20
21
|
**When to use:**
|
|
22
|
+
|
|
21
23
|
- Staying in this session
|
|
22
24
|
- Tasks are mostly independent
|
|
23
25
|
- Want continuous progress with quality gates
|
|
24
26
|
|
|
25
27
|
**When NOT to use:**
|
|
28
|
+
|
|
26
29
|
- Need to review plan first (use executing-plans)
|
|
27
30
|
- Tasks are tightly coupled (manual execution better)
|
|
28
31
|
- Plan needs revision (brainstorm first)
|
|
@@ -38,6 +41,7 @@ Read plan file, create TodoWrite with all tasks.
|
|
|
38
41
|
For each task:
|
|
39
42
|
|
|
40
43
|
**Dispatch fresh subagent:**
|
|
44
|
+
|
|
41
45
|
```
|
|
42
46
|
Task tool (general-purpose):
|
|
43
47
|
description: "Implement Task N: [task name]"
|
|
@@ -60,16 +64,16 @@ Task tool (general-purpose):
|
|
|
60
64
|
|
|
61
65
|
### 3. Review Subagent's Work
|
|
62
66
|
|
|
63
|
-
**Dispatch
|
|
64
|
-
|
|
65
|
-
|
|
66
|
-
|
|
67
|
-
|
|
68
|
-
|
|
69
|
-
|
|
70
|
-
|
|
71
|
-
|
|
72
|
-
|
|
67
|
+
**Dispatch review subagent:**
|
|
68
|
+
|
|
69
|
+
Use template at requesting-code-review/review.md
|
|
70
|
+
|
|
71
|
+
WHAT_WAS_IMPLEMENTED: [from subagent's report]
|
|
72
|
+
PLAN_OR_REQUIREMENTS: Task N from [plan-file]
|
|
73
|
+
BASE_SHA: [commit before task]
|
|
74
|
+
HEAD_SHA: [current commit]
|
|
75
|
+
DESCRIPTION: [task summary]
|
|
76
|
+
|
|
73
77
|
```
|
|
74
78
|
|
|
75
79
|
**Code reviewer returns:** Strengths, Issues (Critical/Important/Minor), Assessment
|
|
@@ -77,13 +81,17 @@ Task tool (superpowers:code-reviewer):
|
|
|
77
81
|
### 4. Apply Review Feedback
|
|
78
82
|
|
|
79
83
|
**If issues found:**
|
|
84
|
+
|
|
80
85
|
- Fix Critical issues immediately
|
|
81
86
|
- Fix Important issues before next task
|
|
82
87
|
- Note Minor issues
|
|
83
88
|
|
|
84
89
|
**Dispatch follow-up subagent if needed:**
|
|
90
|
+
|
|
85
91
|
```
|
|
92
|
+
|
|
86
93
|
"Fix issues from code review: [list issues]"
|
|
94
|
+
|
|
87
95
|
```
|
|
88
96
|
|
|
89
97
|
### 5. Mark Complete, Next Task
|
|
@@ -94,7 +102,8 @@ Task tool (superpowers:code-reviewer):
|
|
|
94
102
|
|
|
95
103
|
### 6. Final Review
|
|
96
104
|
|
|
97
|
-
After all tasks complete, dispatch final
|
|
105
|
+
After all tasks complete, dispatch final review:
|
|
106
|
+
|
|
98
107
|
- Reviews entire implementation
|
|
99
108
|
- Checks all plan requirements met
|
|
100
109
|
- Validates overall architecture
|
|
@@ -102,13 +111,15 @@ After all tasks complete, dispatch final code-reviewer:
|
|
|
102
111
|
### 7. Complete Development
|
|
103
112
|
|
|
104
113
|
After final review passes:
|
|
105
|
-
|
|
106
|
-
-
|
|
114
|
+
|
|
115
|
+
- Announce: "I'm using finishing-a-development-branch skill to complete this work."
|
|
116
|
+
- **REQUIRED SUB-SKILL:** Use use_skill("finishing-a-development-branch")
|
|
107
117
|
- Follow that skill to verify tests, present options, execute choice
|
|
108
118
|
|
|
109
119
|
## Example Workflow
|
|
110
120
|
|
|
111
121
|
```
|
|
122
|
+
|
|
112
123
|
You: I'm using Subagent-Driven Development to execute this plan.
|
|
113
124
|
|
|
114
125
|
[Load plan, create TodoWrite]
|
|
@@ -118,7 +129,7 @@ Task 1: Hook installation script
|
|
|
118
129
|
[Dispatch implementation subagent]
|
|
119
130
|
Subagent: Implemented install-hook with tests, 5/5 passing
|
|
120
131
|
|
|
121
|
-
[Get git SHAs, dispatch
|
|
132
|
+
[Get git SHAs, dispatch review]
|
|
122
133
|
Reviewer: Strengths: Good test coverage. Issues: None. Ready.
|
|
123
134
|
|
|
124
135
|
[Mark Task 1 complete]
|
|
@@ -128,7 +139,7 @@ Task 2: Recovery modes
|
|
|
128
139
|
[Dispatch implementation subagent]
|
|
129
140
|
Subagent: Added verify/repair, 8/8 tests passing
|
|
130
141
|
|
|
131
|
-
[Dispatch
|
|
142
|
+
[Dispatch review]
|
|
132
143
|
Reviewer: Strengths: Solid. Issues (Important): Missing progress reporting
|
|
133
144
|
|
|
134
145
|
[Dispatch fix subagent]
|
|
@@ -139,51 +150,61 @@ Fix subagent: Added progress every 100 conversations
|
|
|
139
150
|
...
|
|
140
151
|
|
|
141
152
|
[After all tasks]
|
|
142
|
-
[Dispatch final
|
|
153
|
+
[Dispatch final review]
|
|
143
154
|
Final reviewer: All requirements met, ready to merge
|
|
144
155
|
|
|
145
156
|
Done!
|
|
157
|
+
|
|
146
158
|
```
|
|
147
159
|
|
|
148
160
|
## Advantages
|
|
149
161
|
|
|
150
162
|
**vs. Manual execution:**
|
|
163
|
+
|
|
151
164
|
- Subagents follow TDD naturally
|
|
152
165
|
- Fresh context per task (no confusion)
|
|
153
166
|
- Parallel-safe (subagents don't interfere)
|
|
154
167
|
|
|
155
168
|
**vs. Executing Plans:**
|
|
169
|
+
|
|
156
170
|
- Same session (no handoff)
|
|
157
171
|
- Continuous progress (no waiting)
|
|
158
172
|
- Review checkpoints automatic
|
|
159
173
|
|
|
160
174
|
**Cost:**
|
|
175
|
+
|
|
161
176
|
- More subagent invocations
|
|
162
177
|
- But catches issues early (cheaper than debugging later)
|
|
163
178
|
|
|
164
179
|
## Red Flags
|
|
165
180
|
|
|
166
181
|
**Never:**
|
|
182
|
+
|
|
167
183
|
- Skip code review between tasks
|
|
168
184
|
- Proceed with unfixed Critical issues
|
|
169
185
|
- Dispatch multiple implementation subagents in parallel (conflicts)
|
|
170
186
|
- Implement without reading plan task
|
|
171
187
|
|
|
172
188
|
**If subagent fails task:**
|
|
189
|
+
|
|
173
190
|
- Dispatch fix subagent with specific instructions
|
|
174
191
|
- Don't try to fix manually (context pollution)
|
|
175
192
|
|
|
176
193
|
## Integration
|
|
177
194
|
|
|
178
195
|
**Required workflow skills:**
|
|
196
|
+
|
|
179
197
|
- **writing-plans** - REQUIRED: Creates the plan that this skill executes
|
|
180
198
|
- **requesting-code-review** - REQUIRED: Review after each task (see Step 3)
|
|
181
199
|
- **finishing-a-development-branch** - REQUIRED: Complete development after all tasks (see Step 7)
|
|
182
200
|
|
|
183
201
|
**Subagents must use:**
|
|
202
|
+
|
|
184
203
|
- **test-driven-development** - Subagents follow TDD for each task
|
|
185
204
|
|
|
186
205
|
**Alternative workflow:**
|
|
206
|
+
|
|
187
207
|
- **executing-plans** - Use for parallel session instead of same-session execution
|
|
188
208
|
|
|
189
|
-
See
|
|
209
|
+
See review template: requesting-code-review/review.md
|
|
210
|
+
```
|
|
@@ -24,6 +24,7 @@ If you haven't completed Phase 1, you cannot propose fixes.
|
|
|
24
24
|
## When to Use
|
|
25
25
|
|
|
26
26
|
Use for ANY technical issue:
|
|
27
|
+
|
|
27
28
|
- Test failures
|
|
28
29
|
- Bugs in production
|
|
29
30
|
- Unexpected behavior
|
|
@@ -32,6 +33,7 @@ Use for ANY technical issue:
|
|
|
32
33
|
- Integration issues
|
|
33
34
|
|
|
34
35
|
**Use this ESPECIALLY when:**
|
|
36
|
+
|
|
35
37
|
- Under time pressure (emergencies make guessing tempting)
|
|
36
38
|
- "Just one quick fix" seems obvious
|
|
37
39
|
- You've already tried multiple fixes
|
|
@@ -39,6 +41,7 @@ Use for ANY technical issue:
|
|
|
39
41
|
- You don't fully understand the issue
|
|
40
42
|
|
|
41
43
|
**Don't skip when:**
|
|
44
|
+
|
|
42
45
|
- Issue seems simple (simple bugs have root causes too)
|
|
43
46
|
- You're in a hurry (rushing guarantees rework)
|
|
44
47
|
- Manager wants it fixed NOW (systematic is faster than thrashing)
|
|
@@ -74,6 +77,7 @@ You MUST complete each phase before proceeding to the next.
|
|
|
74
77
|
**WHEN system has multiple components (CI → build → signing, API → service → database):**
|
|
75
78
|
|
|
76
79
|
**BEFORE proposing fixes, add diagnostic instrumentation:**
|
|
80
|
+
|
|
77
81
|
```
|
|
78
82
|
For EACH component boundary:
|
|
79
83
|
- Log what data enters component
|
|
@@ -87,6 +91,7 @@ You MUST complete each phase before proceeding to the next.
|
|
|
87
91
|
```
|
|
88
92
|
|
|
89
93
|
**Example (multi-layer system):**
|
|
94
|
+
|
|
90
95
|
```bash
|
|
91
96
|
# Layer 1: Workflow
|
|
92
97
|
echo "=== Secrets available in workflow: ==="
|
|
@@ -111,7 +116,7 @@ You MUST complete each phase before proceeding to the next.
|
|
|
111
116
|
|
|
112
117
|
**WHEN error is deep in call stack:**
|
|
113
118
|
|
|
114
|
-
**REQUIRED SUB-SKILL:** Use
|
|
119
|
+
**REQUIRED SUB-SKILL:** Use use_skill("root-cause-tracing") for backward tracing technique
|
|
115
120
|
|
|
116
121
|
**Quick version:**
|
|
117
122
|
- Where does bad value originate?
|
|
@@ -176,7 +181,7 @@ You MUST complete each phase before proceeding to the next.
|
|
|
176
181
|
- Automated test if possible
|
|
177
182
|
- One-off test script if no framework
|
|
178
183
|
- MUST have before fixing
|
|
179
|
-
- **REQUIRED SUB-SKILL:** Use
|
|
184
|
+
- **REQUIRED SUB-SKILL:** Use use_skill("test-driven-development") for writing proper failing tests
|
|
180
185
|
|
|
181
186
|
2. **Implement Single Fix**
|
|
182
187
|
- Address the root cause identified
|
|
@@ -215,6 +220,7 @@ You MUST complete each phase before proceeding to the next.
|
|
|
215
220
|
## Red Flags - STOP and Follow Process
|
|
216
221
|
|
|
217
222
|
If you catch yourself thinking:
|
|
223
|
+
|
|
218
224
|
- "Quick fix for now, investigate later"
|
|
219
225
|
- "Just try changing X and see if it works"
|
|
220
226
|
- "Add multiple changes, run tests"
|
|
@@ -234,6 +240,7 @@ If you catch yourself thinking:
|
|
|
234
240
|
## your human partner's Signals You're Doing It Wrong
|
|
235
241
|
|
|
236
242
|
**Watch for these redirections:**
|
|
243
|
+
|
|
237
244
|
- "Is that not happening?" - You assumed without verifying
|
|
238
245
|
- "Will it show us...?" - You should have added evidence gathering
|
|
239
246
|
- "Stop guessing" - You're proposing fixes without understanding
|
|
@@ -244,25 +251,25 @@ If you catch yourself thinking:
|
|
|
244
251
|
|
|
245
252
|
## Common Rationalizations
|
|
246
253
|
|
|
247
|
-
| Excuse
|
|
248
|
-
|
|
249
|
-
| "Issue is simple, don't need process"
|
|
250
|
-
| "Emergency, no time for process"
|
|
251
|
-
| "Just try this first, then investigate"
|
|
252
|
-
| "I'll write test after confirming fix works" | Untested fixes don't stick. Test first proves it.
|
|
253
|
-
| "Multiple fixes at once saves time"
|
|
254
|
-
| "Reference too long, I'll adapt the pattern" | Partial understanding guarantees bugs. Read it completely.
|
|
255
|
-
| "I see the problem, let me fix it"
|
|
256
|
-
| "One more fix attempt" (after 2+ failures)
|
|
254
|
+
| Excuse | Reality |
|
|
255
|
+
| -------------------------------------------- | ----------------------------------------------------------------------- |
|
|
256
|
+
| "Issue is simple, don't need process" | Simple issues have root causes too. Process is fast for simple bugs. |
|
|
257
|
+
| "Emergency, no time for process" | Systematic debugging is FASTER than guess-and-check thrashing. |
|
|
258
|
+
| "Just try this first, then investigate" | First fix sets the pattern. Do it right from the start. |
|
|
259
|
+
| "I'll write test after confirming fix works" | Untested fixes don't stick. Test first proves it. |
|
|
260
|
+
| "Multiple fixes at once saves time" | Can't isolate what worked. Causes new bugs. |
|
|
261
|
+
| "Reference too long, I'll adapt the pattern" | Partial understanding guarantees bugs. Read it completely. |
|
|
262
|
+
| "I see the problem, let me fix it" | Seeing symptoms ≠ understanding root cause. |
|
|
263
|
+
| "One more fix attempt" (after 2+ failures) | 3+ failures = architectural problem. Question pattern, don't fix again. |
|
|
257
264
|
|
|
258
265
|
## Quick Reference
|
|
259
266
|
|
|
260
|
-
| Phase
|
|
261
|
-
|
|
262
|
-
| **1. Root Cause**
|
|
263
|
-
| **2. Pattern**
|
|
264
|
-
| **3. Hypothesis**
|
|
265
|
-
| **4. Implementation** | Create test, fix, verify
|
|
267
|
+
| Phase | Key Activities | Success Criteria |
|
|
268
|
+
| --------------------- | ------------------------------------------------------ | --------------------------- |
|
|
269
|
+
| **1. Root Cause** | Read errors, reproduce, check changes, gather evidence | Understand WHAT and WHY |
|
|
270
|
+
| **2. Pattern** | Find working examples, compare | Identify differences |
|
|
271
|
+
| **3. Hypothesis** | Form theory, test minimally | Confirmed or new hypothesis |
|
|
272
|
+
| **4. Implementation** | Create test, fix, verify | Bug resolved, tests pass |
|
|
266
273
|
|
|
267
274
|
## When Process Reveals "No Root Cause"
|
|
268
275
|
|
|
@@ -278,10 +285,12 @@ If systematic investigation reveals issue is truly environmental, timing-depende
|
|
|
278
285
|
## Integration with Other Skills
|
|
279
286
|
|
|
280
287
|
**This skill requires using:**
|
|
288
|
+
|
|
281
289
|
- **root-cause-tracing** - REQUIRED when error is deep in call stack (see Phase 1, Step 5)
|
|
282
290
|
- **test-driven-development** - REQUIRED for creating failing test case (see Phase 4, Step 1)
|
|
283
291
|
|
|
284
292
|
**Complementary skills:**
|
|
293
|
+
|
|
285
294
|
- **defense-in-depth** - Add validation at multiple layers after finding root cause
|
|
286
295
|
- **condition-based-waiting** - Replace arbitrary timeouts identified in Phase 2
|
|
287
296
|
- **verification-before-completion** - Verify fix worked before claiming success
|
|
@@ -289,6 +298,7 @@ If systematic investigation reveals issue is truly environmental, timing-depende
|
|
|
289
298
|
## Real-World Impact
|
|
290
299
|
|
|
291
300
|
From debugging sessions:
|
|
301
|
+
|
|
292
302
|
- Systematic approach: 15-30 minutes to fix
|
|
293
303
|
- Random fixes approach: 2-3 hours of thrashing
|
|
294
304
|
- First-time fix rate: 95% vs 40%
|
|
@@ -13,7 +13,7 @@ You run scenarios without the skill (RED - watch agent fail), write skill addres
|
|
|
13
13
|
|
|
14
14
|
**Core principle:** If you didn't watch an agent fail without the skill, you don't know if the skill prevents the right failures.
|
|
15
15
|
|
|
16
|
-
**REQUIRED BACKGROUND:** You MUST understand
|
|
16
|
+
**REQUIRED BACKGROUND:** You MUST understand test-driven-development before using this skill. That skill defines the fundamental RED-GREEN-REFACTOR cycle. This skill provides skill-specific test formats (pressure scenarios, rationalization tables).
|
|
17
17
|
|
|
18
18
|
**Complete worked example:** See examples/CLAUDE_MD_TESTING.md for a full test campaign testing CLAUDE.md documentation variants.
|
|
19
19
|
|
|
@@ -1,3 +1,8 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: ui-ux-research
|
|
3
|
+
description: Use when analyzing UI patterns across codebases, comparing design system implementations, auditing UI consistency, or understanding existing patterns before implementation
|
|
4
|
+
---
|
|
5
|
+
|
|
1
6
|
# UI/UX Research Skill
|
|
2
7
|
|
|
3
8
|
Research UI patterns, design systems, and user experience across codebases.
|
|
@@ -1,3 +1,8 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: visual-analysis
|
|
3
|
+
description: Use when analyzing images, screenshots, UI mockups, or visual content for extracting information, comparing designs, or assessing visual properties like colors, typography, and layout
|
|
4
|
+
---
|
|
5
|
+
|
|
1
6
|
# Visual Analysis Skill
|
|
2
7
|
|
|
3
8
|
Analyze images, screenshots, and visual content.
|
|
@@ -34,7 +34,7 @@ Assume they are a skilled developer, but know almost nothing about our toolset o
|
|
|
34
34
|
```markdown
|
|
35
35
|
# [Feature Name] Implementation Plan
|
|
36
36
|
|
|
37
|
-
> **For Claude:** REQUIRED SUB-SKILL: Use
|
|
37
|
+
> **For Claude:** REQUIRED SUB-SKILL: Use use_skill("executing-plans") to implement this plan task-by-task.
|
|
38
38
|
|
|
39
39
|
**Goal:** [One sentence describing what this builds]
|
|
40
40
|
|
|
@@ -111,11 +111,11 @@ After saving the plan, offer execution choice:
|
|
|
111
111
|
**Which approach?"**
|
|
112
112
|
|
|
113
113
|
**If Subagent-Driven chosen:**
|
|
114
|
-
- **REQUIRED SUB-SKILL:** Use
|
|
114
|
+
- **REQUIRED SUB-SKILL:** Use use_skill("subagent-driven-development")
|
|
115
115
|
- Stay in this session
|
|
116
116
|
- Fresh subagent per task + code review
|
|
117
117
|
|
|
118
118
|
**If Parallel Session chosen:**
|
|
119
119
|
- Guide them to open new session in worktree
|
|
120
|
-
- **REQUIRED SUB-SKILL:** New session uses
|
|
120
|
+
- **REQUIRED SUB-SKILL:** New session uses use_skill("executing-plans")
|
|
121
121
|
```
|