maiconfz--review-fix-ship--code-reviewer

Reviews scoped git diffs for design, maintainability, and missing tests.

3|1|Updated May 2, 2026
One-click install
npx skills add https://github.com/agents-repo/registry --skill maiconfz-review-fix-ship-code-reviewer-agents-repo
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: maiconfz--review-fix-ship--code-reviewer
Source: https://github.com/agents-repo/registry/tree/main/.cursor/skills/maiconfz/review-fix-ship/maiconfz--review-fix-ship--code-reviewer
Command: npx skills add https://github.com/agents-repo/registry --skill maiconfz-review-fix-ship-code-reviewer-agents-repo

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve? It provides a general-quality review of a scoped git diff, catching design issues, maintainability problems, API contract drift, and missing tests before code is merged. ## Core Features & Use Cases - Scoped Diff Review: Reviews branch changes (merge-base with the default branch) or uncommitted working-tree changes. - Severity-Ordered Findings: Emits a findings table sorted by severity with file and line locations. - Clear Boundaries: Focuses on quality concerns while leaving deep bug hunting and security analysis to specialist reviewers. - Use Case: Before merging a feature branch, run this review to surface design flaws, unclear regressions, and missing test coverage in the diff. ## Quick Start Ask the assistant to review the current branch changes for design and maintainability issues and return a severity-ordered findings table.

Frequently Asked Questions about maiconfz--review-fix-ship--code-reviewer

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

FAQPage Schema
How do I review a git diff for code quality before merging?▼

Run this review on the scoped diff, which defaults to branch changes computed from the merge-base with the default branch. It returns a findings table sorted by severity with file and line locations covering design, maintainability, and missing tests.

What does a general code quality review cover?▼

It covers design, maintainability, API contract drift, missing tests, and unclear regressions in the scoped diff. It does not deep-hunt correctness bugs or security issues, which belong to dedicated bug-reviewer and security-reviewer specialists.

Can I review uncommitted changes instead of branch changes?▼

Yes, pass uncommitted-changes as the diff input to review the local working tree, including staged and unstaged changes. The default scope is branch-changes based on the merge-base with the default branch.

Does the code reviewer edit or fix files automatically?▼

No, it is review-only and must not edit, create, or delete files, nor commit, push, or merge. It outputs a findings table or a one-line no-findings status, leaving fixes to the user or other agents.

When should I use a security or bug reviewer instead?▼

Use specialist reviewers when you need deep correctness bug hunting or security vulnerability analysis. This reviewer only mentions obvious bugs or security issues in passing and focuses on general quality concerns.