design-review
Design Review
Overview
Use this skill to convert subjective design feedback into prioritized, evidence-based decisions that unblock implementation.
Scope Boundaries
- Use this skill when the task matches the trigger condition described in
description. - Do not use this skill when the primary task falls outside this skill's domain.
Shared References
- Severity calibration guidance:
references/design-review-severity-guidance.md
Templates And Assets
- Findings template:
assets/design-review-findings-template.md
- Approval decision log:
assets/design-review-decision-log-template.md
Inputs To Gather
- Review target (screens, flows, interaction states) and intended users.
- Acceptance criteria and non-negotiable constraints.
- Accessibility and localization expectations for in-scope surfaces.
- Delivery timeline and risk tolerance for unresolved issues.
Deliverables
- Findings list with severity, user impact, engineering impact, and owner.
- Explicit approval decision: approved, conditional approval, or rejected.
- Remediation plan with due dates and verification method.
- Residual risk statement for any deferred non-blocker issues.
Quick Example
- Blocker: keyboard focus order breaks checkout completion.
- Major: ambiguous copy causes wrong destructive action selection.
- Minor: visual rhythm inconsistency without measurable task impact.
- Decision: reject due to blocker; re-review only affected flow after fix.
Quality Standard
- Findings are reproducible and linked to concrete evidence.
- Severity reflects impact, not reviewer preference.
- Blockers are separated from improvements and tracked independently.
- Approval decision is auditable and tied to explicit criteria.
Workflow
- Confirm review scope, criteria, and decision authority.
- Evaluate critical user journeys before secondary surfaces.
- Log findings with impact, severity, and remediation guidance in
assets/design-review-findings-template.md. - Resolve or defer issues using explicit risk acceptance rules and
references/design-review-severity-guidance.md. - Publish final decision and follow-up actions in
assets/design-review-decision-log-template.md.
Failure Conditions
- Stop when review scope or acceptance criteria are ambiguous.
- Stop when critical flows cannot be validated end-to-end.
- Escalate when blocker findings remain open near implementation handoff.
More from kentoshimizu/sw-agent-skills
graph-algorithms
Graph algorithm workflow for modeling entities/relations and selecting traversal, path, ordering, or flow strategies. Use when correctness or performance depends on graph representation and algorithm choice; do not use for schema-only modeling or deployment topology planning.
14bash-style-guide
Style, review, and refactoring standards for Bash shell scripting. Trigger when `.sh` files, files with `#!/usr/bin/env bash` or `#!/bin/bash`, or CI workflow blocks with `shell: bash` are created, modified, or reviewed and Bash-specific quality controls (quoting safety, error handling, portability, readability) must be enforced. Do not use for generic POSIX `sh`, PowerShell, or language-specific application style rules. In multi-language pull requests, run together with other applicable `*-style-guide` skills.
11architecture-clean-architecture
Clean Architecture workflow for enforcing dependency direction, stable domain boundaries, and use-case-centered application design. Use when teams must separate business rules from frameworks and delivery mechanisms; do not use for isolated module cleanup without boundary implications.
11powershell-style-guide
Style, review, and refactoring standards for PowerShell scripting. Trigger when `.ps1`, `.psm1`, `.psd1` files, or CI workflow blocks with `shell: pwsh` or `shell: powershell` are created, modified, or reviewed and PowerShell-specific quality controls (error handling, parameter validation, readability, operational safety) must be enforced. Do not use for Bash, generic POSIX `sh`, or language-specific application style rules. In multi-language pull requests, run together with other applicable `*-style-guide` skills.
10github-codeowners-management
Govern CODEOWNERS rules so review routing reflects real ownership and risk boundaries on GitHub. Use when repository ownership mapping or mandatory reviewer rules must be defined, updated, or audited; do not use for non-GitHub runtime architecture or data-layer design.
9security-authentication
Security workflow for authentication architecture, credential lifecycle, and session/token assurance. Use when login, identity proofing, MFA, or session security decisions are required; do not use for authorization policy design or non-security quality tuning.
9