km:quality-review

Review git diffs against ISO/IEC 25010:2023 quality characteristics.

Updated Jan 24, 2026
One-click install
npx skills add https://github.com/koumatsumoto/agent-config --skill km-quality-review
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: km:quality-review
Source: https://github.com/koumatsumoto/agent-config/tree/main/templates/skills/quality-review
Command: npx skills add https://github.com/koumatsumoto/agent-config --skill km-quality-review

SYSTEM DOCUMENTATION & REQUIREMENTS

💡 This Skill includes references (resource) components.

What problem does it solve?

This skill systematizes quality reviews of uncommitted changes by mapping diffs to ISO/IEC 25010:2023 quality characteristics, reducing missed security, reliability, performance, and maintainability issues that typically escape functional-focused reviews.

Core Features & Use Cases

  • Checklist-driven evaluation: Uses a prioritized quality-checklist to decide depth and scope of review per change type (Full / Focused / Quick).
  • Reference-backed judgments: Provides per-characteristic guidance and anti-patterns via a reference directory to reduce false positives and justify findings.
  • Structured reporting: Emits a report-format with categorized findings (CRITICAL/HIGH/MEDIUM/LOW) and a nine-characteristic PASS/WARN/FAIL/SKIP summary to support commit gating and engineering decision-making.
  • Use Case: During a feature branch review, run this workflow on the git diff to detect ownership/authorization gaps, N+1 patterns, unsafe input handling, and regression risks before merging.

Quick Start

Run a checklist-based review of the current git diff using the quality-checklist, consult reference files for ambiguous items, and produce a report that summarizes all nine ISO/IEC 25010 characteristics with evidence-backed findings.

Frequently Asked Questions about km:quality-review

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

FAQPage Schema
How do I review a git diff for security and maintainability issues?

To review a git diff, run a checklist-based analysis against ISO/IEC 25010 characteristics to detect security, maintainability, and reliability issues. This process maps uncommitted file changes to standard quality metrics and generates a structured report with categorized findings.

What is the best way to check uncommitted code changes against ISO/IEC 25010 quality characteristics?

Checking uncommitted changes against ISO/IEC 25010 involves evaluating the git diff across nine quality characteristics like performance and reliability. The review outputs a summary with PASS, WARN, FAIL, or SKIP statuses per characteristic, backed by diff evidence.

How does checklist-driven code review work for feature branch diffs?

Checklist-driven code review for feature branch diffs works by applying a prioritized quality checklist to decide the review scope, then consulting reference files for anti-patterns. It identifies ownership gaps, N+1 patterns, and unsafe input handling before merging.

Can I use automated quality reviews to detect regression risks before merging?

Yes, you can use automated quality reviews to detect regression risks before merging by analyzing uncommitted changes. The review evaluates reliability and security metrics, producing categorized findings from CRITICAL to LOW to support commit gating decisions.

Does code quality review provide traceable evidence for engineering decisions?

Code quality review provides traceable evidence for engineering decisions by mapping categorized findings directly back to specific git diff lines. This ensures that PASS, WARN, and FAIL evaluations are justified by reference-backed guidance and anti-patterns.

When should I avoid using ISO/IEC 25010 for code review?

You should avoid using ISO/IEC 25010 for code review when evaluating purely stylistic or formatting changes, as the checklist focuses on functional characteristics like security, reliability, and performance rather than code style enforcement.