Skip to content

Commit cea6647

Browse files
belumumeclaude
andcommitted
fix: resolve PR review findings from cubic + coderabbit
- Phase 7 opening: intensity-aware ("run required phases") not unconditional - research-prompts.md: "Verify non-library claims" to avoid overriding doc-first - research-prompts.md: model existence check uses HuggingFace API, not pip - review-protocols.md: final gate prerequisite is intensity-aware Identified by cubic and coderabbit (PR #5 review). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 658dec5 commit cea6647

File tree

3 files changed

+6
-6
lines changed

3 files changed

+6
-6
lines changed

deep-brainstorming/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -173,7 +173,7 @@ This protects against context compaction losing approved work.
173173

174174
## Phase 7: Three-Phase Review
175175

176-
After assembling the spec, run three sequential review phases. Each phase is a separate agent with NO context from prior phases.
176+
After assembling the spec, run the review phases required by your selected intensity level. Each phase is a separate agent with NO context from prior phases.
177177

178178
**Phase R1 — Verify-fix:** Standard review for consistency, completeness, formatting. Fix issues, verify fixes are clean.
179179

deep-brainstorming/references/research-prompts.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ Research the best approaches for the [DOMAIN AREA] requirements above.
4747
3. Rank approaches by quality-of-fit for THESE specific requirements
4848
4. Note any requirements that NO current tool fully addresses
4949
50-
Verify all claims via web search. Do not rely on training data for version numbers,
50+
Verify non-library claims via web search. Do not rely on training data for version numbers,
5151
benchmarks, or feature availability. If you cannot verify a claim, mark it as UNVERIFIED.
5252
5353
Do NOT recommend tools because they are popular. Recommend them because they are
@@ -176,9 +176,9 @@ Use when an agent reports "X doesn't exist" or "X is not available." NEVER accep
176176
**Do NOT dispatch another agent.** Verify locally:
177177

178178
```bash
179-
# Model existence
180-
pip index versions <package-name>
181-
# or: HuggingFace API query
179+
# Model existence (HuggingFace models, not PyPI packages)
180+
# HuggingFace API: curl -s https://huggingface.co/api/models/<org>/<model> | head
181+
# or: huggingface-cli search <model-name>
182182

183183
# Package existence
184184
pip index versions <package-name>

deep-brainstorming/references/review-protocols.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,7 @@ For each finding:
147147

148148
## "Objectively Best?" Final Gate
149149

150-
After all three review phases pass, apply this final quality gate.
150+
After all required review phases for your selected intensity pass, apply this final quality gate.
151151

152152
### Protocol
153153

0 commit comments

Comments
 (0)