ReddRadar
Agent skill
ac-qa-reviewer
Quality assurance review for implementations. Use when reviewing code quality, checking implementation standards, performing QA cycles, or validating feature quality.
Install this agent skill to your Project
npx add-skill https://github.com/majiayu000/claude-skill-registry/tree/main/skills/data/ac-qa-reviewer
SKILL.md
AC QA Reviewer
Perform quality assurance reviews on feature implementations.
Purpose
Reviews completed features for code quality, adherence to standards, security issues, and best practices before marking features as complete.
Quick Start
from scripts.qa_reviewer import QAReviewer
reviewer = QAReviewer(project_dir)
result = await reviewer.review_feature("auth-001")
Review Dimensions
Code Quality
- Clean code principles
- DRY (Don't Repeat Yourself)
- SOLID principles
- Code complexity metrics
Security
- Input validation
- SQL injection prevention
- XSS prevention
- Secure authentication
Performance
- Algorithm efficiency
- Database query optimization
- Memory usage
- Caching opportunities
Testing
- Test coverage
- Test quality
- Edge case coverage
- Integration tests
Documentation
- Code comments
- API documentation
- README updates
- Type annotations
Review Result
{
"feature_id": "auth-001",
"approved": true,
"score": 85,
"dimensions": {
"code_quality": {"score": 90, "issues": []},
"security": {"score": 85, "issues": ["Consider rate limiting"]},
"performance": {"score": 80, "issues": []},
"testing": {"score": 88, "issues": []},
"documentation": {"score": 82, "issues": ["Add docstring to validate_token"]}
},
"blocking_issues": [],
"suggestions": [
"Consider extracting authentication middleware",
"Add rate limiting for login endpoint"
],
"auto_fixable": ["missing_docstring"]
}
QA Workflow
1. SCAN → Static analysis of changed files
2. ANALYZE → Check against quality rules
3. SECURITY → Security-specific checks
4. REVIEW → Contextual code review
5. REPORT → Generate review report
6. FIX → Auto-fix simple issues (optional)
7. APPROVE → Mark as QA passed or request changes
Quality Gates
gates:
minimum_score: 70
blocking_categories:
- security_critical
- test_failures
required_checks:
- linting_passes
- type_checks_pass
- tests_pass
- coverage_minimum
Auto-Fix Capabilities
The reviewer can automatically fix:
- Missing type hints
- Formatting issues
- Simple code style violations
- Missing docstrings (basic)
Integration
- Input: Completed features from
ac-task-executor - Uses:
ac-code-validatorfor static analysis - Reports to:
ac-state-tracker
API Reference
See scripts/qa_reviewer.py for full implementation.
Recommended Agent Skills
Expand your agent's capabilities with these related and highly-rated skills.
majiayu000/claude-skill-registry
agent-ops-spec
Manage specification documents in .agent/specs/. Use when user provides requirements, acceptance criteria, or feature descriptions that need to be tracked and validated against implementation.
majiayu000/claude-skill-registry
agent-ops-state
Maintain .agent state files. Use at session start, after meaningful steps, and before concluding: read/update constitution/memory/focus/issues/baseline consistently.
majiayu000/claude-skill-registry
agent-ops-spec
Manage specification documents in .agent/specs/. Use when user provides requirements, acceptance criteria, or feature descriptions that need to be tracked and validated against implementation.
majiayu000/claude-skill-registry
agent-ops-testing
Test strategy, execution, and coverage analysis. Use when designing tests, running test suites, or analyzing test results beyond baseline checks.
majiayu000/claude-skill-registry
agent-ops-testing
Test strategy, execution, and coverage analysis. Use when designing tests, running test suites, or analyzing test results beyond baseline checks.
majiayu000/claude-skill-registry
agent-ops-state
Maintain .agent state files. Use at session start, after meaningful steps, and before concluding: read/update constitution/memory/focus/issues/baseline consistently.
Didn't find tool you were looking for?