e2ec-it/sp-request-code-review
crush/skills/sp-request-code-review/SKILL.md
Request review between tasks and before merge. Use git SHAs to scope changes and produce actionable findings.
development
Updated Apr 21, 2026
$ install --global
skillsauthnpx skillsauth add e2ec-it/pub-xifan-crush-best-practices sp-request-code-reviewInstall this skill globally with one command. Works with Claude Code, Cursor, and Windsurf.
Security Scan Results
3 of 9 scanners reported clean
Some scanners were skipped, did not run, or reported a non-clean status. Review each row below.
3
Scanners Passed
9
Scanners in report
Clean
TrivyContainer and dependency vulnerability scanner
95%
Clean
SemgrepStatic code analysis for vulnerabilities
95%
Clean
mcp-scan (Snyk)Model Context Protocol security validation
95%
Skipped
Snyk (dep)Open source security scanning
50%
Skipped
Socket.devSupply chain security analysis
50%
Skipped
VirusTotalMulti-engine malware detection
50%
Skipped
CrowdStrikeAdvanced threat intelligence
50%
Skipped
OSV-ScannerOpen Source Vulnerability database check
50%
Skipped
OWASP Dep-Check
50%
Last scanned: Apr 21, 2026, 6:32 PM125.5s1 file scanned
SKILL.md
- name:
- sp-request-code-review
- description:
- Request review between tasks and before merge. Use git SHAs to scope changes and produce actionable findings.
Superpowers: Requesting Code Review (Crush Adaptation)
Based on obra/superpowers requesting-code-review skill. (MIT)
Upstream reference: skills/requesting-code-review/SKILL.md
When
- After each task (especially in sp-subagent-dev)
- Before merge / PR
- When stuck
How
- Determine diff range (example):
BASE_SHA=$(git rev-parse HEAD~1)
HEAD_SHA=$(git rev-parse HEAD)
git diff --stat "$BASE_SHA..$HEAD_SHA"
- Review along two axes:
- Spec compliance: matches plan/requirements, no extra scope
- Code quality: tests, readability, error handling, security
- Severity labels: Critical / Important / Minor
- Fix Critical immediately
- Fix Important before proceeding
Related Skills
e2ec-it/sp-writing-plans
content-media
VerifiedTrustedCommunity
SUPERPOWERS-style implementation planning: turn an approved design into 2–5 minute tasks with exact file paths, commands, verifications, and frequent commits.
SKILL.mdUpdated Apr 21, 2026
e2ec-it/sp-tdd
development
VerifiedTrustedCommunity
SUPERPOWERS TDD enforcement: NO production code without a failing test first. Delete code written before tests.
SKILL.mdUpdated Apr 21, 2026
e2ec-it/sp-subagent-dev
development
VerifiedTrustedCommunity
Execute plan per task with role separation and review loops (spec then quality). In Crush, simulate subagents via role prompts / new sessions.
SKILL.mdUpdated Apr 21, 2026
e2ec-it/sp-git-worktrees
testing
VerifiedTrustedCommunity
Create isolated worktree for feature work; verify ignore rules; run baseline tests before work.
SKILL.mdUpdated Apr 21, 2026