# Quality Review

> Check changed code against the resolved dotbabel quality policy, then review semantic risks that tools cannot judge. Triggers on: "quality review", "quality check", "review code quality".

- Skill: `kaiohenricunha/quality-review` (Agent Skill, multi-file: 4 files)
- Install (CLI): `npx skillmds@latest add kaiohenricunha/quality-review`
- Raw SKILL.md: https://api.skillmd.com/api/skills/kaiohenricunha/quality-review/raw
- Safety review: pending
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Coding & Dev Tools
- Author: kaiohenricunha (https://skillmd.com/u/kaiohenricunha)
- Updated: 2026-09-17
- Page: https://skillmd.com/skills/kaiohenricunha/quality-review

---


# Quality review

Use this workflow for changed-code quality checks. Do not use it as a substitute for a focused security audit.

## Workflow

1. Run `dotbabel quality explain` before a policy-sensitive change. Use `--rule <id>` for one rule.
2. Run `dotbabel quality detect`. Resolve an ambiguous tool choice through `.dotbabel.json`; do not guess.
3. Run `dotbabel quality check --profile <profile> --base <base>`. Use `fast` during edits and `pr` before a pull request.
4. Review the changed code for the semantic rules listed by `quality explain`.
5. Report failures, warnings, unavailable capabilities, improvements, and policy-file changes.

Do not claim that an unsupported or unavailable result passed. Do not install a missing tool.

## Review duties

- Simplify control flow before you split code for a metric.
- Add behavioral tests for success, failure, and boundary behavior.
- Reject coverage padding and tests that only repeat implementation details.
- Review new suppressions for a narrow scope and an engineering reason.
- Review concurrency ownership, cancellation, error handling, unsafe types, assertions, and speculative abstractions.
- Prefer a legacy improvement over unrelated repository cleanup.

Read [references/policy.md](references/policy.md) when a baseline, exception, or unavailable result affects the verdict.
Read [references/languages.md](references/languages.md) for language-specific semantic review.
Read [references/ci.md](references/ci.md) when you configure CI or local attestation.