What problem does it solve? Reviewing DHIS2 Core pull requests requires deep knowledge of platform-specific conventions (layering, ACL, transactions, Flyway migrations, test goals) that generic code reviews routinely miss, letting critical defects like missing authority checks or non-idempotent migrations slip through. ## Core Features & Use Cases - Mandatory Findings Scan: Checks every diff against 17 high-signal DHIS2 defect patterns, from missing @RequiresAuthority to SQL injection and boolean config misreads, each with a fixed severity. - DHIS2-Specific Checklists: Enforces rules for layering (Controller -> Service -> Store), transaction boundaries, caching via CacheProvider, Flyway migration idempotency, OpenAPI annotations, and banned imports. - Structured Review Report: Produces a standardized output with Strengths, severity-tiered Issues (Critical/Important/Minor), Recommendations, and a merge verdict. - Use Case: A maintainer receives a PR adding a new REST endpoint with a schema change. The skill flags the missing negative (403) auth test, the non-idempotent Flyway migration, and the absent WOW coordination entry before approval. ## Quick Start Review the current DHIS2 Core pull request diff against the platform conventions and produce a severity-tiered review report with a merge verdict.
