What problem does it solve? Before opening a pull request or refactoring legacy code, developers need a consistent, evidence-based check that their Java code follows the project's SOLID principles, packaging rules, and testability standards. This Skill runs a structured review and produces a report of violations with concrete fix suggestions, instead of relying on ad-hoc manual review. ## Core Features & Use Cases - SOLID and packaging audit: Checks SRP, OCP, LSP, ISP, DIP, lego-brick package hierarchy, and external-client wrapping rules with cited line numbers as evidence. - Cross-cutting convention checks: Flags exception handling, logging, constants, concurrency, reactive boundary, and configuration violations against project conventions. - Test quality and testability analysis: Detects meaningless assertions, missing negative-path tests, static business logic, global state, hidden dependencies, and constructor side effects, plus quantitative thresholds (complexity, method length, coverage). - Use Case: Before opening a PR, run the review on your changed files to get a PASS/FAIL/WARN report per checklist item, a summary of top fixes, and pointers to the follow-up skills that apply each fix. ## Quick Start Ask the AI to run the quality-review skill on the current class or changed files and produce the structured findings report before opening the pull request.
