---
name: rsmdt-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."
license: "MIT"
metadata:
  decimalai-skill-id: "ce6a6b0c-97a3-4094-bde0-b4d0b0983138"
  decimalai-version-id: "ffa08c9d-dea8-4ecc-ad55-8b8d3eff2a00"
  decimalai-version: "1"
  decimalai-body-sha256: "eea3a270d4e1f5f67ed575d9e82d074ffdb65c267f1594094d167328efeca74a"
  decimalai-name: "rsmdt/code-quality-review"
  decimalai-format: "1"
  decimalai-invocation: "model"
  decimalai-category: "code-review"
  decimalai-source: "https://app.decimal.ai/s/rsmdt-code-quality-review@1/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.