code-reviewer

Validate pull requests for zero new SonarCloud issues before merge.

Updated Nov 8, 2022
One-click install
npx skills add https://github.com/gvinokur/qatar-prode --skill code-reviewer-gvinokur
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: code-reviewer
Source: https://github.com/gvinokur/qatar-prode/tree/main/.claude/skills/code-reviewer
Command: npx skills add https://github.com/gvinokur/qatar-prode --skill code-reviewer-gvinokur

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Prevents merging pull requests that introduce new SonarCloud issues or stale documentation by performing a strict, reproducible pre-merge validation workflow and a mandatory documentation audit before marking work complete.

Core Features & Use Cases

  • SonarCloud Enforcement: Fetches SonarCloud analysis for the PR, requires 0 new issues of any severity, and verifies coverage thresholds on new code.
  • Plan Reconciliation: Compares the story plan and commit history to actual code changes and records amendments when deviations are found.
  • Pre-Merge Documentation Audit: Performs a mandatory, line-level accuracy check of CODE-STRUCTURE layer files against changed source files (Section 7.5), blocking story completion until the audit is clean.
  • CI and Deployment Verification: Waits for CI/Vercel, validates preview deployment accessibility, and uses helper scripts to collect checks and Sonar reports.
  • Use Case: When a reviewer sees "code looks good" after Vercel preview testing, run this Skill to move the PR out of draft, run SonarCloud, resolve any issues, perform the documentation audit, and produce a final validation summary.

Quick Start

Run the validation flow after the user confirms "code looks good" so the skill can mark the PR ready, wait for CI and SonarCloud, enforce zero new issues, and perform the pre-merge documentation audit.

Frequently Asked Questions about code-reviewer

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

FAQPage Schema
How do I enforce zero new SonarCloud issues before merging a pull request?

To enforce zero new SonarCloud issues before merging, you can run a pre-merge validation workflow that fetches SonarCloud analysis for the PR, requires zero new issues of any severity, and verifies coverage thresholds on new code.

What is a pre-merge documentation audit for code changes?

A pre-merge documentation audit is a line-level accuracy check comparing CODE-STRUCTURE layer files against changed source files, blocking story completion until the documentation accurately reflects the code deviations.

How do I reconcile a story plan with actual git commit history?

Plan reconciliation compares the story plan and commit history to actual code changes, recording amendments when deviations are found so that the final pre-merge state matches the intended feature scope.

Does this pre-merge validation work with Vercel preview deployments?

Yes, the validation workflow works with Vercel preview deployments by waiting for CI and Vercel builds, validating preview deployment accessibility, and collecting checks before running the SonarCloud quality gate enforcement.

What information do I need to run SonarCloud quality gates on a feature branch?

Running SonarCloud quality gates requires the PR number, worktree path, access to SonarCloud and CI results, and the repository CODE-STRUCTURE files to perform plan reconciliation, coverage analysis, and the documentation audit.

When should I not use automated code review quality gates?

Automated code review quality gates should not be used before the user explicitly confirms the code looks good after Vercel preview testing, as the workflow depends on this confirmation to move the PR out of draft and initiate final validation.