openspec-sp-code-review

Conduct checklist-based code reviews and classify issues as Critical, Important, or Suggestions.

Updated Mar 31, 2026
One-click install
npx skills add https://github.com/cenmiao/openspec-pro --skill openspec-sp-code-review
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: openspec-sp-code-review
Source: https://github.com/cenmiao/openspec-pro/tree/main/.claude/skills/openspec-sp-code-review
Command: npx skills add https://github.com/cenmiao/openspec-pro --skill openspec-sp-code-review

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

This Skill provides a repeatable, checklist-driven process to catch design, quality, security, testing, and performance issues before code is merged, reducing regressions and review inconsistency.

Core Features & Use Cases

  • Structured Checklist: Covers plan alignment, code quality, architecture, documentation, testing, security, and performance for comprehensive reviews.
  • Issue Classification: Produces outputs categorized as Critical, Important, and Suggestions to prioritize fixes.
  • Integration Scenarios: Use for pull request reviews, pre-merge checks, major refactors, feature completions, and gating high-risk changes.

Quick Start

Ask the assistant to review the user authentication module using the checklist and produce a structured report summarizing critical issues, important fixes, and suggestions.

Frequently Asked Questions about openspec-sp-code-review

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I perform a structured code review for a pull request?

A structured code review applies a checklist to verify plan alignment, code quality, architecture, documentation, testing, security, and performance, producing a report that classifies issues as Critical, Important, or Suggestions.

What is checklist-driven code review and when should I use it?

Checklist-driven code review systematically evaluates implemented changes against predefined criteria to catch design and security issues. Use it for pre-merge checks, feature implementations, major refactors, and routine development tasks to reduce regressions.

How do I categorize code review issues by severity before merging?

You can categorize code review issues by classifying them into Critical, Important, and Suggestions based on their impact on architecture, security, testing, and performance to prioritize fixes before merging.

Can I use a checklist to verify architecture and security in refactored code?

Yes, you can apply a structured checklist to major refactors to verify code quality, architecture, documentation, testing, and security, ensuring high-risk changes are safe for gating and merging.

What is the best way to ensure code quality and testing coverage before a merge?

The best way to ensure code quality and testing coverage before a merge is to conduct a repeatable, checklist-based review that verifies plan alignment, security, and performance, reducing review inconsistency and regressions.