AC QA Reviewer

ac-qa-reviewer

Quality assurance review for implementations. Use when reviewing code quality, checking implementation standards, performing QA cycles, or validating feature quality.

NeverSight/skills_feed·0installs215starsSecurity

SKILL.md

Full skill instructions

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-validator for static analysis
Reports to: ac-state-tracker

API Reference

See scripts/qa_reviewer.py for full implementation.

Install skill

Copy and run in your terminal

npx -y skills add https://github.com/NeverSight/skills_feed --skill ac-qa-reviewer

View on GitHub

data/skills-md/adaptationio/skrillz/ac-qa-reviewer/SKILL.md

More Security skills

View all

Security

supabase

Handles the full Supabase workflow from schema changes to deployment, with built-in security guardrails that catch common traps like RLS...

2.7K 308.9K

View

Security

neon-postgres

Comprehensive guides and best practices for Neon Serverless Postgres, covering setup, connection methods, authentication, and platform APIs.

98 216.2K

View

Security

firebase-auth-basics

Guide for setting up and using Firebase Authentication. Use this skill when the user's app requires user sign-in, user management, or secure data access using auth rules.

462 163.8K

View

Security

firebase-security-rules-auditor

A skill to evaluate how secure Firestore security rules are. Use this when Firestore security rules are updated to ensure that the generated rules are extremely secure and robust.

462 127.7K

View

Security

firebase-ai-logic-basics

Official skill for integrating Firebase AI Logic (Gemini API) into web applications. Covers setup, multimodal inference, structured output, and security.

462 125.1K

View

Security

better-auth-best-practices

Complete Better Auth server and client setup with database adapters, session management, plugins, and security configuration.

222 118.2K

View

Security

Deploy and manage projects on Vercel using token-based authentication. Use when working with Vercel CLI using access tokens rather than interactive login — e.g. "deploy to vercel", "set up vercel", "add environment variables to vercel".

31.9K 116.1K

View

Security

cloudflare

Complete Cloudflare platform integration with decision trees for compute, storage, AI, networking, security, and infrastructure-as-code.

3K 110.7K

View

Security

azure-compliance

Run Azure compliance and security audits with azqr plus Key Vault expiration checks. Covers best-practice assessment, resource review, policy/compliance validation, and security posture checks. WHEN: compliance scan, security audit, BEFORE running azqr (compliance cli tool), Azure best practices, Key Vault expiration check, expired certificates, expiring secrets, orphaned resources, compliance assessment.

253 103K

View