Code Review Quality
This gives you a framework for code reviews that actually catch bugs instead of bikeshedding style preferences. It enforces prioritized feedback (blocker/major/minor/suggestion), includes templates for common issues like SQL injection and missing error handling, and coordinates four QE agents to scan security, performance, and coverage in parallel. The standout feature is minimum findings enforcement: if a review scores below 3.0 on weighted findings, it automatically runs a devils advocate agent as a meta-reviewer to dig deeper. Honest take: the 400 line review limit and "ask questions not commands" guidance are the most immediately useful parts for human reviewers, while the agent orchestration handles the tedious security scanning you'd otherwise miss.
- skills.sh installs
- 1.6K
- GitHub stars
- 524
- last updated
- Aug 21, 2026
Install
$ npx -y skills add proffesor-for-testing/agentic-qe --skill code-review-quality --agent claude-codeInstalls into .claude/skills of the current project.
Files
Code Review Quality
<default_to_action> When reviewing code or establishing review practices:
- PRIORITIZE feedback: 🔴 Blocker (must fix) → 🟡 Major → 🟢 Minor → 💡 Suggestion
- FOCUS on: Bugs, security, testability, maintainability (not style preferences)
- ASK questions over commands: "Have you considered...?" > "Change this to..."
- PROVIDE context: Why this matters, not just what to change
- LIMIT scope: Review < 400 lines at a time for effectiveness
Quick Review Checklist:
- Logic: Does it work correctly? Edge cases handled?
- Security: Input validation? Auth checks? Injection risks?
- Testability: Can this be tested? Is it tested?
- Maintainability: Clear naming? Single responsibility? DRY?
- Performance: O(n²) loops? N+1 queries? Memory leaks?
Critical Success Factors:
- Review the code, not the person
- Catching bugs > nitpicking style
- Fast feedback (< 24h) > thorough feedback </default_to_action>
Quick Reference Card
When to Use
- PR code reviews
- Pair programming feedback
- Establishing team review standards
- Mentoring developers
Sponsors
Agent CaffeineSponsoredClose your lid, come back to finished work
SnugSponsoredThe clipboard manager for the AI eraPromote your product hereReach 125k+ visitors monthly
WaxSponsoredBlock anything with supervision, without erasing your iPhonePromote your product hereReach 125k+ visitors monthly
knobsSponsoredSee your site on any phone, any day, in any languageDetails
More from proffesor-for-testing/agentic-qe
All 13Recommended
More Testing & QA
build-loop-codexbuildgreatproducts/builder-osUse when building features with **Codex** (OpenAI Codex CLI) in any codebase and the work should go through a disciplined build → review → test → fix loop. Trig1.5Kskills.sh installsbuild-loop-cursorbuildgreatproducts/builder-osUse when building features with **Cursor** in any codebase and the work should go through a disciplined build → review → test → fix loop. Triggers on "run the b1.5Kskills.sh installs
rstest-best-practicesrstackjs/agent-skillsRstest best practices for config, CLI workflow, test writing, mocking, snapshot testing, DOM testing, coverage, multi-project setup, CI integration, performance1.5Kskills.sh installs
tlc-spec-driventech-leads-club/agent-skillsFeature planning and implementation with 4 adaptive phases (Specify, Design, Tasks, Execute). Auto-sizes depth by complexity. Writes testable requirements in EA1.3Kskills.sh installs
paid-ads-strategykostja94/marketing-skillsWhen the user wants to plan paid ads strategy, allocate ad budget, or choose paid channels. Also use when the user mentions "paid ads," "paid media," "PPC," "SE1.3Kskills.sh installs
write-testsneolabhq/context-engineering-kitAdd missing test coverage for your local code changes by generating new test files (covers uncommitted and untracked changes, or the latest commit if everything1.3Kskills.sh installs