opencodekit 0.0.1

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.
Files changed (123) hide show
  1. package/README.md +258 -0
  2. package/dist/index.js +3391 -0
  3. package/dist/template/.opencode/.env.example +193 -0
  4. package/dist/template/.opencode/AGENTS.md +214 -0
  5. package/dist/template/.opencode/README.md +269 -0
  6. package/dist/template/.opencode/agent/build.md +75 -0
  7. package/dist/template/.opencode/agent/explore.md +66 -0
  8. package/dist/template/.opencode/agent/planner.md +83 -0
  9. package/dist/template/.opencode/agent/review.md +90 -0
  10. package/dist/template/.opencode/agent/rush.md +85 -0
  11. package/dist/template/.opencode/agent/scout.md +93 -0
  12. package/dist/template/.opencode/command/analyze-project.md +39 -0
  13. package/dist/template/.opencode/command/brainstorm.md +11 -0
  14. package/dist/template/.opencode/command/commit.md +11 -0
  15. package/dist/template/.opencode/command/create.md +118 -0
  16. package/dist/template/.opencode/command/design.md +15 -0
  17. package/dist/template/.opencode/command/finish.md +233 -0
  18. package/dist/template/.opencode/command/fix-ci.md +20 -0
  19. package/dist/template/.opencode/command/fix-types.md +10 -0
  20. package/dist/template/.opencode/command/fix-ui.md +22 -0
  21. package/dist/template/.opencode/command/fix.md +22 -0
  22. package/dist/template/.opencode/command/handoff.md +146 -0
  23. package/dist/template/.opencode/command/implement.md +167 -0
  24. package/dist/template/.opencode/command/import-plan.md +188 -0
  25. package/dist/template/.opencode/command/integration-test.md +36 -0
  26. package/dist/template/.opencode/command/issue.md +41 -0
  27. package/dist/template/.opencode/command/plan.md +158 -0
  28. package/dist/template/.opencode/command/pr.md +36 -0
  29. package/dist/template/.opencode/command/quick-build.md +13 -0
  30. package/dist/template/.opencode/command/research-and-implement.md +21 -0
  31. package/dist/template/.opencode/command/research-ui.md +32 -0
  32. package/dist/template/.opencode/command/research.md +153 -0
  33. package/dist/template/.opencode/command/resume.md +127 -0
  34. package/dist/template/.opencode/command/review-codebase.md +13 -0
  35. package/dist/template/.opencode/command/skill-create.md +29 -0
  36. package/dist/template/.opencode/command/skill-optimize.md +28 -0
  37. package/dist/template/.opencode/command/status.md +109 -0
  38. package/dist/template/.opencode/command/ui-review.md +28 -0
  39. package/dist/template/.opencode/dcp.jsonc +34 -0
  40. package/dist/template/.opencode/memory/README.md +128 -0
  41. package/dist/template/.opencode/memory/_templates/handoff.md +33 -0
  42. package/dist/template/.opencode/memory/_templates/research.md +29 -0
  43. package/dist/template/.opencode/memory/_templates/task-prd.md +43 -0
  44. package/dist/template/.opencode/memory/_templates/task-review.md +73 -0
  45. package/dist/template/.opencode/memory/_templates/task-spec.md +71 -0
  46. package/dist/template/.opencode/memory/design-guidelines.md +281 -0
  47. package/dist/template/.opencode/memory/handoffs/README.md +83 -0
  48. package/dist/template/.opencode/opencode.json +469 -0
  49. package/dist/template/.opencode/package.json +23 -0
  50. package/dist/template/.opencode/pickle-thinker.jsonc +11 -0
  51. package/dist/template/.opencode/plugin/README.md +162 -0
  52. package/dist/template/.opencode/plugin/notification.ts +88 -0
  53. package/dist/template/.opencode/plugin/sessions.ts +434 -0
  54. package/dist/template/.opencode/plugin/superpowers.ts +332 -0
  55. package/dist/template/.opencode/plugin/tsconfig.json +15 -0
  56. package/dist/template/.opencode/superpowers/.claude/settings.local.json +141 -0
  57. package/dist/template/.opencode/superpowers/.claude-plugin/marketplace.json +20 -0
  58. package/dist/template/.opencode/superpowers/.claude-plugin/plugin.json +13 -0
  59. package/dist/template/.opencode/superpowers/.codex/INSTALL.md +35 -0
  60. package/dist/template/.opencode/superpowers/.codex/superpowers-bootstrap.md +33 -0
  61. package/dist/template/.opencode/superpowers/.codex/superpowers-codex +267 -0
  62. package/dist/template/.opencode/superpowers/.github/FUNDING.yml +3 -0
  63. package/dist/template/.opencode/superpowers/.opencode/INSTALL.md +135 -0
  64. package/dist/template/.opencode/superpowers/.opencode/plugin/superpowers.js +215 -0
  65. package/dist/template/.opencode/superpowers/LICENSE +21 -0
  66. package/dist/template/.opencode/superpowers/README.md +165 -0
  67. package/dist/template/.opencode/superpowers/RELEASE-NOTES.md +493 -0
  68. package/dist/template/.opencode/superpowers/agents/code-reviewer.md +48 -0
  69. package/dist/template/.opencode/superpowers/commands/brainstorm.md +5 -0
  70. package/dist/template/.opencode/superpowers/commands/execute-plan.md +5 -0
  71. package/dist/template/.opencode/superpowers/commands/write-plan.md +5 -0
  72. package/dist/template/.opencode/superpowers/docs/README.codex.md +153 -0
  73. package/dist/template/.opencode/superpowers/docs/README.opencode.md +234 -0
  74. package/dist/template/.opencode/superpowers/docs/plans/2025-11-22-opencode-support-design.md +294 -0
  75. package/dist/template/.opencode/superpowers/docs/plans/2025-11-22-opencode-support-implementation.md +1095 -0
  76. package/dist/template/.opencode/superpowers/hooks/hooks.json +15 -0
  77. package/dist/template/.opencode/superpowers/hooks/session-start.sh +34 -0
  78. package/dist/template/.opencode/superpowers/lib/skills-core.js +208 -0
  79. package/dist/template/.opencode/superpowers/skills/brainstorming/SKILL.md +54 -0
  80. package/dist/template/.opencode/superpowers/skills/condition-based-waiting/SKILL.md +120 -0
  81. package/dist/template/.opencode/superpowers/skills/condition-based-waiting/example.ts +158 -0
  82. package/dist/template/.opencode/superpowers/skills/defense-in-depth/SKILL.md +127 -0
  83. package/dist/template/.opencode/superpowers/skills/dispatching-parallel-agents/SKILL.md +180 -0
  84. package/dist/template/.opencode/superpowers/skills/executing-plans/SKILL.md +76 -0
  85. package/dist/template/.opencode/superpowers/skills/finishing-a-development-branch/SKILL.md +200 -0
  86. package/dist/template/.opencode/superpowers/skills/frontend-aesthetics/SKILL.md +137 -0
  87. package/dist/template/.opencode/superpowers/skills/gemini-large-context/SKILL.md +205 -0
  88. package/dist/template/.opencode/superpowers/skills/receiving-code-review/SKILL.md +209 -0
  89. package/dist/template/.opencode/superpowers/skills/requesting-code-review/SKILL.md +105 -0
  90. package/dist/template/.opencode/superpowers/skills/requesting-code-review/code-reviewer.md +146 -0
  91. package/dist/template/.opencode/superpowers/skills/root-cause-tracing/SKILL.md +174 -0
  92. package/dist/template/.opencode/superpowers/skills/root-cause-tracing/find-polluter.sh +63 -0
  93. package/dist/template/.opencode/superpowers/skills/sharing-skills/SKILL.md +194 -0
  94. package/dist/template/.opencode/superpowers/skills/subagent-driven-development/SKILL.md +189 -0
  95. package/dist/template/.opencode/superpowers/skills/systematic-debugging/CREATION-LOG.md +119 -0
  96. package/dist/template/.opencode/superpowers/skills/systematic-debugging/SKILL.md +295 -0
  97. package/dist/template/.opencode/superpowers/skills/systematic-debugging/test-academic.md +14 -0
  98. package/dist/template/.opencode/superpowers/skills/systematic-debugging/test-pressure-1.md +58 -0
  99. package/dist/template/.opencode/superpowers/skills/systematic-debugging/test-pressure-2.md +68 -0
  100. package/dist/template/.opencode/superpowers/skills/systematic-debugging/test-pressure-3.md +69 -0
  101. package/dist/template/.opencode/superpowers/skills/test-driven-development/SKILL.md +364 -0
  102. package/dist/template/.opencode/superpowers/skills/testing-anti-patterns/SKILL.md +302 -0
  103. package/dist/template/.opencode/superpowers/skills/testing-skills-with-subagents/SKILL.md +387 -0
  104. package/dist/template/.opencode/superpowers/skills/testing-skills-with-subagents/examples/CLAUDE_MD_TESTING.md +189 -0
  105. package/dist/template/.opencode/superpowers/skills/ui-ux-research/SKILL.md +191 -0
  106. package/dist/template/.opencode/superpowers/skills/using-git-worktrees/SKILL.md +213 -0
  107. package/dist/template/.opencode/superpowers/skills/using-superpowers/SKILL.md +101 -0
  108. package/dist/template/.opencode/superpowers/skills/verification-before-completion/SKILL.md +139 -0
  109. package/dist/template/.opencode/superpowers/skills/writing-plans/SKILL.md +116 -0
  110. package/dist/template/.opencode/superpowers/skills/writing-skills/SKILL.md +622 -0
  111. package/dist/template/.opencode/superpowers/skills/writing-skills/anthropic-best-practices.md +1150 -0
  112. package/dist/template/.opencode/superpowers/skills/writing-skills/graphviz-conventions.dot +172 -0
  113. package/dist/template/.opencode/superpowers/skills/writing-skills/persuasion-principles.md +187 -0
  114. package/dist/template/.opencode/superpowers/tests/opencode/run-tests.sh +165 -0
  115. package/dist/template/.opencode/superpowers/tests/opencode/setup.sh +73 -0
  116. package/dist/template/.opencode/superpowers/tests/opencode/test-plugin-loading.sh +81 -0
  117. package/dist/template/.opencode/superpowers/tests/opencode/test-priority.sh +198 -0
  118. package/dist/template/.opencode/superpowers/tests/opencode/test-skills-core.sh +440 -0
  119. package/dist/template/.opencode/superpowers/tests/opencode/test-tools.sh +104 -0
  120. package/dist/template/.opencode/tool/memory-read.ts +66 -0
  121. package/dist/template/.opencode/tool/memory-update.ts +61 -0
  122. package/dist/template/.opencode/tsconfig.json +21 -0
  123. package/package.json +52 -0
@@ -0,0 +1,137 @@
1
+ ---
2
+ name: frontend-aesthetics
3
+ description: Use when building UI components, landing pages, dashboards, or any frontend work - avoids generic AI slop aesthetic by steering toward distinctive typography, cohesive themes, meaningful motion, and atmospheric backgrounds
4
+ ---
5
+
6
+ # Frontend Aesthetics
7
+
8
+ Prevent distributional convergence that creates generic "AI slop" frontends. Every interface should feel intentionally crafted for its specific context.
9
+
10
+ ## When to Use
11
+
12
+ - Building landing pages, dashboards, forms, components
13
+ - Any frontend generation task
14
+ - Reviewing UI for visual quality
15
+
16
+ ## Decision Gate (Do This First)
17
+
18
+ Before writing any code, state in 1-2 sentences:
19
+
20
+ 1. **Aesthetic direction** — Which style from the list below?
21
+ 2. **Why** — How does it fit the context/purpose?
22
+
23
+ ### Aesthetic Directions
24
+
25
+ Pick ONE (or combine 2 for hybrid):
26
+
27
+ | Direction | Character |
28
+ | ---------------- | ----------------------------------------------- |
29
+ | Neo-Brutalist | Raw textures, bold type, harsh contrasts |
30
+ | Soft Minimalism | Muted palettes, generous whitespace |
31
+ | Retro-Futuristic | CRT effects, scan lines, neon, cyberpunk |
32
+ | Editorial | Dynamic grids, mixed media, bold type |
33
+ | Glass Morphism | Translucent layers, backdrop blur, depth |
34
+ | Dark Academia | Rich textures, serif type, scholarly |
35
+ | Swiss Design | Grid systems, sans-serif, functional |
36
+ | Art Deco | Geometric patterns, gold accents, luxury |
37
+ | Y2K Revival | Gradients, metallics, early-web nostalgia |
38
+ | Organic/Natural | Flowing shapes, nature palettes, paper textures |
39
+
40
+ ## Core Prompt
41
+
42
+ ```xml
43
+ <frontend_aesthetics>
44
+ You tend to converge toward generic outputs. In frontend design, this creates
45
+ "AI slop" aesthetic. Make creative, distinctive frontends that surprise.
46
+
47
+ Focus on:
48
+ - Typography: Beautiful, unique fonts. Never Inter/Roboto/Arial/system fonts.
49
+ Good: JetBrains Mono, Fira Code, Space Grotesk, Playfair Display, Crimson Pro,
50
+ IBM Plex, Bricolage Grotesque, Newsreader, Instrument Serif, Bebas Neue
51
+ - Color: Commit to cohesive theme. CSS variables. Dominant colors with sharp
52
+ accents beat timid palettes. Draw from IDE themes, cultural aesthetics.
53
+ - Motion: One orchestrated page load with staggered reveals beats scattered
54
+ micro-interactions. CSS-only for HTML, Motion library for React.
55
+ - Backgrounds: Atmosphere and depth, not solid colors. Layer gradients,
56
+ geometric patterns, noise textures.
57
+
58
+ Never do ALL of these together (AI slop signature):
59
+ - Purple/blue gradient backgrounds
60
+ - Inter or system fonts
61
+ - Centered hero with subheading
62
+ - 3-column feature cards with icons
63
+ - Rounded corners on everything (16px border-radius)
64
+ - Drop shadows on all cards
65
+ - #6366F1 or #667eea as primary color
66
+ - "Modern, clean, simple" as only descriptors
67
+
68
+ Vary light/dark themes, fonts, aesthetics. Make unexpected choices.
69
+ </frontend_aesthetics>
70
+ ```
71
+
72
+ ## Typography Quick Reference
73
+
74
+ | Category | Good Choices | Avoid |
75
+ | ----------- | ----------------------------------------------- | ---------------- |
76
+ | Code/Tech | JetBrains Mono, Fira Code, Space Mono | Roboto Mono |
77
+ | Editorial | Playfair Display, Crimson Pro, Instrument Serif | Times New Roman |
78
+ | Display | Bebas Neue, Oswald, Archivo Black | Impact |
79
+ | Technical | IBM Plex, Source Sans 3, Work Sans | Arial, Helvetica |
80
+ | Distinctive | Bricolage Grotesque, Syne, Outfit | Inter, Lato |
81
+
82
+ **Pairing:** High contrast = interesting. Display + monospace, serif + geometric sans.
83
+
84
+ **Weight:** Use extremes (100/200 vs 800/900), not middle (400 vs 600). Size jumps 3x+.
85
+
86
+ ## Color Anti-Patterns
87
+
88
+ ```css
89
+ /* BAD - Overused AI gradient */
90
+ background: linear-gradient(135deg, #667eea 0%, #764ba2 100%);
91
+
92
+ /* GOOD - Intentional duotone */
93
+ background:
94
+ linear-gradient(45deg, #ff6b6b 0%, transparent 70%),
95
+ linear-gradient(-45deg, #4ecdc4 0%, transparent 70%), #f7fff7;
96
+ ```
97
+
98
+ ## Animation Pattern
99
+
100
+ One orchestrated entrance beats scattered effects:
101
+
102
+ ```css
103
+ @keyframes revealUp {
104
+ from {
105
+ opacity: 0;
106
+ transform: translateY(30px);
107
+ }
108
+ }
109
+
110
+ .hero > * {
111
+ animation: revealUp 800ms cubic-bezier(0.19, 1, 0.22, 1) both;
112
+ }
113
+ .hero > *:nth-child(1) {
114
+ animation-delay: 0ms;
115
+ }
116
+ .hero > *:nth-child(2) {
117
+ animation-delay: 80ms;
118
+ }
119
+ .hero > *:nth-child(3) {
120
+ animation-delay: 160ms;
121
+ }
122
+ ```
123
+
124
+ ## Framework Integration
125
+
126
+ **React + Tailwind:** "Use Tailwind with custom theme, animate with Framer Motion"
127
+
128
+ **shadcn/ui:** "Use shadcn components but customize theme, avoid default styling"
129
+
130
+ ## Quality Check
131
+
132
+ Before delivery:
133
+
134
+ - [ ] Clear aesthetic point of view?
135
+ - [ ] Avoids ALL items in the AI slop list?
136
+ - [ ] Would someone remember this tomorrow?
137
+ - [ ] Responsive, accessible, performs well?
@@ -0,0 +1,205 @@
1
+ ---
2
+ name: gemini-large-context
3
+ description: Use when analyzing large codebases, comparing multiple files, or researching project-wide patterns that exceed typical context limits - leverages Gemini CLI's 1M token context window for comprehensive analysis
4
+ ---
5
+
6
+ # Gemini CLI Large Context Analysis
7
+
8
+ ## Overview
9
+
10
+ Use Gemini CLI's massive context window (1M tokens) when analysis requires more context than available in current session. Ideal for codebase-wide searches, multi-file comparisons, and pattern discovery across large projects.
11
+
12
+ ## When to Use
13
+
14
+ - Analyzing entire codebases or large directories (100KB+)
15
+ - Comparing multiple large files simultaneously
16
+ - Understanding project-wide patterns or architecture
17
+ - Context window insufficient for current task
18
+ - Verifying feature implementations across codebase
19
+ - Research that needs to scan many files at once
20
+
21
+ ## Quick Reference
22
+
23
+ ```bash
24
+ # Basic non-interactive analysis
25
+ gemini -p "@src/ Summarize architecture"
26
+
27
+ # Multiple directories
28
+ gemini -p "@src/ @tests/ Analyze test coverage"
29
+
30
+ # All project files
31
+ gemini --all-files -p "Project overview"
32
+
33
+ # Specific model (Pro = 1M context)
34
+ gemini -m gemini-2.5-pro -p "@src/ Deep analysis"
35
+ ```
36
+
37
+ ## File/Directory Inclusion Syntax
38
+
39
+ ```bash
40
+ # Single file
41
+ gemini -p "@src/main.py Explain this file"
42
+
43
+ # Multiple files
44
+ gemini -p "@package.json @src/index.js Analyze dependencies"
45
+
46
+ # Entire directory (recursive)
47
+ gemini -p "@src/ Summarize architecture"
48
+
49
+ # Multiple directories
50
+ gemini -p "@src/ @tests/ Analyze test coverage"
51
+
52
+ # Current directory
53
+ gemini -p "@./ Project overview"
54
+
55
+ # Alternative: all files flag
56
+ gemini --all-files -p "Analyze project structure"
57
+ ```
58
+
59
+ ## Research Patterns
60
+
61
+ ### Architecture Analysis
62
+
63
+ ```bash
64
+ gemini -p "@src/
65
+ Describe:
66
+ 1. Overall architecture pattern (MVC, Clean, etc.)
67
+ 2. Key modules and their responsibilities
68
+ 3. Data flow between components
69
+ 4. External dependencies and integrations"
70
+ ```
71
+
72
+ ### Feature Implementation Check
73
+
74
+ ```bash
75
+ gemini -p "@src/ @lib/
76
+ Has [FEATURE] been implemented?
77
+ Show:
78
+ - Relevant files and functions
79
+ - Implementation approach
80
+ - Any gaps or incomplete areas"
81
+ ```
82
+
83
+ ### Pattern Discovery
84
+
85
+ ```bash
86
+ gemini -p "@src/
87
+ Find all instances of [PATTERN]:
88
+ - File paths
89
+ - Implementation variations
90
+ - Consistency assessment"
91
+ ```
92
+
93
+ ### Security Audit
94
+
95
+ ```bash
96
+ gemini -p "@src/ @api/
97
+ Security review:
98
+ - Input validation practices
99
+ - Authentication/authorization patterns
100
+ - SQL injection protections
101
+ - XSS prevention measures"
102
+ ```
103
+
104
+ ### Test Coverage Analysis
105
+
106
+ ```bash
107
+ gemini -p "@src/ @tests/
108
+ Assess test coverage:
109
+ - Which modules have tests?
110
+ - Which are missing tests?
111
+ - Test quality (unit vs integration)
112
+ - Edge cases covered"
113
+ ```
114
+
115
+ ## Integration with OpenCode Workflow
116
+
117
+ ### For Task-Constrained Research
118
+
119
+ When researching within task boundaries:
120
+
121
+ ```bash
122
+ # 1. Include bead spec in context
123
+ gemini -p "@src/ @.beads/artifacts/bd-xxx/spec.md
124
+ Research implementations matching spec constraints"
125
+
126
+ # 2. Save findings to bead artifacts
127
+ # Write to .beads/artifacts/bd-xxx/research.md
128
+ ```
129
+
130
+ ### Delegating Large Research to Gemini
131
+
132
+ In AGENTS.md or prompts, instruct agents:
133
+
134
+ ```markdown
135
+ When analysis requires more than 100KB of files:
136
+
137
+ 1. Use Gemini CLI: `gemini -p "@[paths] [question]"`
138
+ 2. Capture output
139
+ 3. Synthesize findings in response
140
+ ```
141
+
142
+ ## Output Options
143
+
144
+ ```bash
145
+ # Plain text (default)
146
+ gemini -p "query"
147
+
148
+ # JSON for parsing
149
+ gemini -p "query" --output-format json
150
+
151
+ # Streaming for long ops
152
+ gemini -p "query" --output-format stream-json
153
+ ```
154
+
155
+ ## Common Research Queries
156
+
157
+ - **Architecture overview:** `gemini -p "@src/ Describe architecture"`
158
+ - **Find implementations:** `gemini -p "@src/ Where is [X] implemented?"`
159
+ - **Pattern audit:** `gemini -p "@src/ Find all [pattern] usages"`
160
+ - **Dependency analysis:** `gemini -p "@package.json @src/ Analyze dependency usage"`
161
+ - **Code quality:** `gemini -p "@src/ Identify code smells"`
162
+ - **Migration planning:** `gemini -p "@src/ Plan migration from [A] to [B]"`
163
+
164
+ ## Configuration
165
+
166
+ ### Authentication
167
+
168
+ ```bash
169
+ # Google login (free tier: 60 req/min, 1000/day)
170
+ gemini # Follow OAuth flow
171
+
172
+ # Or API key
173
+ export GEMINI_API_KEY="your-key"
174
+ ```
175
+
176
+ ### Model Selection
177
+
178
+ For most analysis tasks, **`gemini-2.5-pro`** is the best choice—it has a 1M token context window and handles code, text, and image analysis exceptionally well. This is the default model.
179
+
180
+ Use **`gemini-2.5-flash`** when you need faster responses and don't need the full context window.
181
+
182
+ **Important:** If you're analyzing images or screenshots, use `gemini-2.5-pro`. Image generation models like `imagen-3.0-generate-002` are for _creating_ new images, not understanding existing ones.
183
+
184
+ ```bash
185
+ # Gemini 2.5 Pro (1M context) - default, best for analysis
186
+ gemini -p "query"
187
+
188
+ # Gemini Flash (faster, smaller context)
189
+ gemini -m gemini-2.5-flash -p "query"
190
+ ```
191
+
192
+ ## Best Practices
193
+
194
+ 1. **Scope appropriately** - Include only relevant directories
195
+ 2. **Be specific** - Ask focused questions, not vague queries
196
+ 3. **Use for read-only analysis** - Don't ask Gemini to modify files in non-interactive mode
197
+ 4. **Capture output** - Redirect to file for long analyses: `gemini -p "..." > analysis.md`
198
+ 5. **Combine with local tools** - Use Gemini for research, local tools for implementation
199
+
200
+ ## Limitations
201
+
202
+ - Non-interactive mode: no tool approvals (file writes, shell commands)
203
+ - `@file` syntax: best in interactive mode
204
+ - Network dependency: requires internet connection
205
+ - Rate limits: 60 req/min free tier
@@ -0,0 +1,209 @@
1
+ ---
2
+ name: receiving-code-review
3
+ description: Use when receiving code review feedback, before implementing suggestions, especially if feedback seems unclear or technically questionable - requires technical rigor and verification, not performative agreement or blind implementation
4
+ ---
5
+
6
+ # Code Review Reception
7
+
8
+ ## Overview
9
+
10
+ Code review requires technical evaluation, not emotional performance.
11
+
12
+ **Core principle:** Verify before implementing. Ask before assuming. Technical correctness over social comfort.
13
+
14
+ ## The Response Pattern
15
+
16
+ ```
17
+ WHEN receiving code review feedback:
18
+
19
+ 1. READ: Complete feedback without reacting
20
+ 2. UNDERSTAND: Restate requirement in own words (or ask)
21
+ 3. VERIFY: Check against codebase reality
22
+ 4. EVALUATE: Technically sound for THIS codebase?
23
+ 5. RESPOND: Technical acknowledgment or reasoned pushback
24
+ 6. IMPLEMENT: One item at a time, test each
25
+ ```
26
+
27
+ ## Forbidden Responses
28
+
29
+ **NEVER:**
30
+ - "You're absolutely right!" (explicit CLAUDE.md violation)
31
+ - "Great point!" / "Excellent feedback!" (performative)
32
+ - "Let me implement that now" (before verification)
33
+
34
+ **INSTEAD:**
35
+ - Restate the technical requirement
36
+ - Ask clarifying questions
37
+ - Push back with technical reasoning if wrong
38
+ - Just start working (actions > words)
39
+
40
+ ## Handling Unclear Feedback
41
+
42
+ ```
43
+ IF any item is unclear:
44
+ STOP - do not implement anything yet
45
+ ASK for clarification on unclear items
46
+
47
+ WHY: Items may be related. Partial understanding = wrong implementation.
48
+ ```
49
+
50
+ **Example:**
51
+ ```
52
+ your human partner: "Fix 1-6"
53
+ You understand 1,2,3,6. Unclear on 4,5.
54
+
55
+ ❌ WRONG: Implement 1,2,3,6 now, ask about 4,5 later
56
+ ✅ RIGHT: "I understand items 1,2,3,6. Need clarification on 4 and 5 before proceeding."
57
+ ```
58
+
59
+ ## Source-Specific Handling
60
+
61
+ ### From your human partner
62
+ - **Trusted** - implement after understanding
63
+ - **Still ask** if scope unclear
64
+ - **No performative agreement**
65
+ - **Skip to action** or technical acknowledgment
66
+
67
+ ### From External Reviewers
68
+ ```
69
+ BEFORE implementing:
70
+ 1. Check: Technically correct for THIS codebase?
71
+ 2. Check: Breaks existing functionality?
72
+ 3. Check: Reason for current implementation?
73
+ 4. Check: Works on all platforms/versions?
74
+ 5. Check: Does reviewer understand full context?
75
+
76
+ IF suggestion seems wrong:
77
+ Push back with technical reasoning
78
+
79
+ IF can't easily verify:
80
+ Say so: "I can't verify this without [X]. Should I [investigate/ask/proceed]?"
81
+
82
+ IF conflicts with your human partner's prior decisions:
83
+ Stop and discuss with your human partner first
84
+ ```
85
+
86
+ **your human partner's rule:** "External feedback - be skeptical, but check carefully"
87
+
88
+ ## YAGNI Check for "Professional" Features
89
+
90
+ ```
91
+ IF reviewer suggests "implementing properly":
92
+ grep codebase for actual usage
93
+
94
+ IF unused: "This endpoint isn't called. Remove it (YAGNI)?"
95
+ IF used: Then implement properly
96
+ ```
97
+
98
+ **your human partner's rule:** "You and reviewer both report to me. If we don't need this feature, don't add it."
99
+
100
+ ## Implementation Order
101
+
102
+ ```
103
+ FOR multi-item feedback:
104
+ 1. Clarify anything unclear FIRST
105
+ 2. Then implement in this order:
106
+ - Blocking issues (breaks, security)
107
+ - Simple fixes (typos, imports)
108
+ - Complex fixes (refactoring, logic)
109
+ 3. Test each fix individually
110
+ 4. Verify no regressions
111
+ ```
112
+
113
+ ## When To Push Back
114
+
115
+ Push back when:
116
+ - Suggestion breaks existing functionality
117
+ - Reviewer lacks full context
118
+ - Violates YAGNI (unused feature)
119
+ - Technically incorrect for this stack
120
+ - Legacy/compatibility reasons exist
121
+ - Conflicts with your human partner's architectural decisions
122
+
123
+ **How to push back:**
124
+ - Use technical reasoning, not defensiveness
125
+ - Ask specific questions
126
+ - Reference working tests/code
127
+ - Involve your human partner if architectural
128
+
129
+ **Signal if uncomfortable pushing back out loud:** "Strange things are afoot at the Circle K"
130
+
131
+ ## Acknowledging Correct Feedback
132
+
133
+ When feedback IS correct:
134
+ ```
135
+ ✅ "Fixed. [Brief description of what changed]"
136
+ ✅ "Good catch - [specific issue]. Fixed in [location]."
137
+ ✅ [Just fix it and show in the code]
138
+
139
+ ❌ "You're absolutely right!"
140
+ ❌ "Great point!"
141
+ ❌ "Thanks for catching that!"
142
+ ❌ "Thanks for [anything]"
143
+ ❌ ANY gratitude expression
144
+ ```
145
+
146
+ **Why no thanks:** Actions speak. Just fix it. The code itself shows you heard the feedback.
147
+
148
+ **If you catch yourself about to write "Thanks":** DELETE IT. State the fix instead.
149
+
150
+ ## Gracefully Correcting Your Pushback
151
+
152
+ If you pushed back and were wrong:
153
+ ```
154
+ ✅ "You were right - I checked [X] and it does [Y]. Implementing now."
155
+ ✅ "Verified this and you're correct. My initial understanding was wrong because [reason]. Fixing."
156
+
157
+ ❌ Long apology
158
+ ❌ Defending why you pushed back
159
+ ❌ Over-explaining
160
+ ```
161
+
162
+ State the correction factually and move on.
163
+
164
+ ## Common Mistakes
165
+
166
+ | Mistake | Fix |
167
+ |---------|-----|
168
+ | Performative agreement | State requirement or just act |
169
+ | Blind implementation | Verify against codebase first |
170
+ | Batch without testing | One at a time, test each |
171
+ | Assuming reviewer is right | Check if breaks things |
172
+ | Avoiding pushback | Technical correctness > comfort |
173
+ | Partial implementation | Clarify all items first |
174
+ | Can't verify, proceed anyway | State limitation, ask for direction |
175
+
176
+ ## Real Examples
177
+
178
+ **Performative Agreement (Bad):**
179
+ ```
180
+ Reviewer: "Remove legacy code"
181
+ ❌ "You're absolutely right! Let me remove that..."
182
+ ```
183
+
184
+ **Technical Verification (Good):**
185
+ ```
186
+ Reviewer: "Remove legacy code"
187
+ ✅ "Checking... build target is 10.15+, this API needs 13+. Need legacy for backward compat. Current impl has wrong bundle ID - fix it or drop pre-13 support?"
188
+ ```
189
+
190
+ **YAGNI (Good):**
191
+ ```
192
+ Reviewer: "Implement proper metrics tracking with database, date filters, CSV export"
193
+ ✅ "Grepped codebase - nothing calls this endpoint. Remove it (YAGNI)? Or is there usage I'm missing?"
194
+ ```
195
+
196
+ **Unclear Item (Good):**
197
+ ```
198
+ your human partner: "Fix items 1-6"
199
+ You understand 1,2,3,6. Unclear on 4,5.
200
+ ✅ "Understand 1,2,3,6. Need clarification on 4 and 5 before implementing."
201
+ ```
202
+
203
+ ## The Bottom Line
204
+
205
+ **External feedback = suggestions to evaluate, not orders to follow.**
206
+
207
+ Verify. Question. Then implement.
208
+
209
+ No performative agreement. Technical rigor always.
@@ -0,0 +1,105 @@
1
+ ---
2
+ name: requesting-code-review
3
+ description: Use when completing tasks, implementing major features, or before merging to verify work meets requirements - dispatches superpowers:code-reviewer subagent to review implementation against plan or requirements before proceeding
4
+ ---
5
+
6
+ # Requesting Code Review
7
+
8
+ Dispatch superpowers:code-reviewer subagent to catch issues before they cascade.
9
+
10
+ **Core principle:** Review early, review often.
11
+
12
+ ## When to Request Review
13
+
14
+ **Mandatory:**
15
+ - After each task in subagent-driven development
16
+ - After completing major feature
17
+ - Before merge to main
18
+
19
+ **Optional but valuable:**
20
+ - When stuck (fresh perspective)
21
+ - Before refactoring (baseline check)
22
+ - After fixing complex bug
23
+
24
+ ## How to Request
25
+
26
+ **1. Get git SHAs:**
27
+ ```bash
28
+ BASE_SHA=$(git rev-parse HEAD~1) # or origin/main
29
+ HEAD_SHA=$(git rev-parse HEAD)
30
+ ```
31
+
32
+ **2. Dispatch code-reviewer subagent:**
33
+
34
+ Use Task tool with superpowers:code-reviewer type, fill template at `code-reviewer.md`
35
+
36
+ **Placeholders:**
37
+ - `{WHAT_WAS_IMPLEMENTED}` - What you just built
38
+ - `{PLAN_OR_REQUIREMENTS}` - What it should do
39
+ - `{BASE_SHA}` - Starting commit
40
+ - `{HEAD_SHA}` - Ending commit
41
+ - `{DESCRIPTION}` - Brief summary
42
+
43
+ **3. Act on feedback:**
44
+ - Fix Critical issues immediately
45
+ - Fix Important issues before proceeding
46
+ - Note Minor issues for later
47
+ - Push back if reviewer is wrong (with reasoning)
48
+
49
+ ## Example
50
+
51
+ ```
52
+ [Just completed Task 2: Add verification function]
53
+
54
+ You: Let me request code review before proceeding.
55
+
56
+ BASE_SHA=$(git log --oneline | grep "Task 1" | head -1 | awk '{print $1}')
57
+ HEAD_SHA=$(git rev-parse HEAD)
58
+
59
+ [Dispatch superpowers:code-reviewer subagent]
60
+ WHAT_WAS_IMPLEMENTED: Verification and repair functions for conversation index
61
+ PLAN_OR_REQUIREMENTS: Task 2 from docs/plans/deployment-plan.md
62
+ BASE_SHA: a7981ec
63
+ HEAD_SHA: 3df7661
64
+ DESCRIPTION: Added verifyIndex() and repairIndex() with 4 issue types
65
+
66
+ [Subagent returns]:
67
+ Strengths: Clean architecture, real tests
68
+ Issues:
69
+ Important: Missing progress indicators
70
+ Minor: Magic number (100) for reporting interval
71
+ Assessment: Ready to proceed
72
+
73
+ You: [Fix progress indicators]
74
+ [Continue to Task 3]
75
+ ```
76
+
77
+ ## Integration with Workflows
78
+
79
+ **Subagent-Driven Development:**
80
+ - Review after EACH task
81
+ - Catch issues before they compound
82
+ - Fix before moving to next task
83
+
84
+ **Executing Plans:**
85
+ - Review after each batch (3 tasks)
86
+ - Get feedback, apply, continue
87
+
88
+ **Ad-Hoc Development:**
89
+ - Review before merge
90
+ - Review when stuck
91
+
92
+ ## Red Flags
93
+
94
+ **Never:**
95
+ - Skip review because "it's simple"
96
+ - Ignore Critical issues
97
+ - Proceed with unfixed Important issues
98
+ - Argue with valid technical feedback
99
+
100
+ **If reviewer wrong:**
101
+ - Push back with technical reasoning
102
+ - Show code/tests that prove it works
103
+ - Request clarification
104
+
105
+ See template at: requesting-code-review/code-reviewer.md