# Code Quality Review

> 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. Use when this capability is needed.

- Skill: `tomevault-io/code-quality-review-2` (Agent Skill, multi-file: 2 files)
- Install (CLI): `npx skillmds@latest add tomevault-io/code-quality-review-2`
- Raw SKILL.md: https://api.skillmd.com/api/skills/tomevault-io/code-quality-review-2/raw
- Safety review: pending (external: skill-scanner PASS, skillspector PASS)
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Web & Frontend
- Author: tomevault-io (https://skillmd.com/u/tomevault-io)
- Updated: 2026-09-17
- Page: https://skillmd.com/skills/tomevault-io/code-quality-review-2

---


## 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.

---
> Converted and distributed by [TomeVault](https://tomevault.io/claim/rsmdt) — claim your Tome and manage your conversions.
<!-- tomevault:4.0:skill_md:2026-04-11 -->