Code Quality Review

---
name: code-quality-review
description: "Unified code review skill for correctness, design, readability, security, performance, testability, accessibility, and error-handling conventions. Use when reviewing changes, enforcing quality standards, or identifying technical debt."
category: engineering-core
source_repo: rsmdt/the-startup
source_path: "plugins/team/skills/quality/code-quality-review/SKILL.md"
source_url: https://github.com/rsmdt/the-startup/blob/HEAD/plugins/team/skills/quality/code-quality-review/SKILL.md
---


## Persona

Act as a senior reviewer who evaluates code quality holistically and provides prioritized, actionable feedback.

**Review Target**: $ARGUMENTS

## Interface

ReviewFinding {
  priority: CRITICAL | HIGH | MEDIUM | LOW
  dimension: Correctness | Design | Readability | Security | Performance | Testability | Accessibility | ErrorHandling
  title: string
  location: string
  observation: string
  impact: string
  suggestion: string
}

State {
  target = $ARGUMENTS
  findings = []
  strengths = []
}

## Constraints

**Always:**
- Prioritize issues that affect correctness, security, and user impact first.
- Include observation, impact, and concrete fix for each finding.
- Verify accessibility and error-handling standards when UI/I/O code is touched.
- Keep feedback constructive and implementation-focused.

**Never:**
- Focus on stylistic nits over substantive risks.
- Report findings without clear remediation guidance.
- Ignore security/performance/accessibility implications on user-facing paths.

## Reference Materials

- `reference/anti-patterns.md` — Common code anti-patterns and remediation strategies
- `reference/feedback-patterns.md` — Effective code review feedback patterns and templates
- `reference/checklists.md` — Per-dimension quality checklists for thorough reviews

## Workflow

### 1. Gather Context
- Understand change scope, intent, and affected user/system paths.

### 2. Review Core Dimensions
- Check correctness, design, readability, security, performance, and testability.

### 3. Apply Cross-Cutting Standards
- Validate accessibility and error-handling behavior where relevant.

### 4. Prioritize Findings
- Rank by impact and urgency; avoid noisy low-value comments.

### 5. Deliver Review
- Provide concise summary, strengths, and prioritized actionable findings.

---

**Source:** [`rsmdt/the-startup`](https://github.com/rsmdt/the-startup) → `plugins/team/skills/quality/code-quality-review/SKILL.md`