magento2-code-review

Solid

This skill should be used when the user asks to "review this PR/MR", "review this merge request", "review this module", "audit this module before merge", "review this theme", "audit this theme PR", or wants a "full review before release". Orchestrates a PR/MR, module, theme, or full-project code review by running the QA trio (magento2-linter, magento2-security-scan, magento2-performance-audit) and magento2-dev-core's anti-pattern checks at the right scope, then merges their findings into one report using a shared severity scale and stable finding codes. DEPENDENT on magento2-dev-core; invokes magento2-linter, magento2-security-scan, and magento2-performance-audit as needed for the chosen scope.

Code & Development 4 stars 2 forks Updated today MIT

Install

View on GitHub

Quality Score: 83/100

Stars 20%
23
Recency 20%
100
Frontmatter 20%
70
Documentation 15%
100
Issue Health 10%
80
License 10%
100
Description 5%
100

Skill Content

# Magento 2 Code Review Orchestrates a review at one of four scopes — PR/MR, module, theme, or whole project — by calling the existing QA trio and `magento2-dev-core` at the right scope, then merging their output into one report with one severity scale. This skill does not reimplement any check the trio already owns. ## Related Skills **REQUIRED BACKGROUND:** Load `magento2-dev-core` first — its `references/severity-and-codes.md` defines the severity scale and finding codes this skill's report uses. Invokes `magento2-linter` (style/static analysis), `magento2-security-scan` (vulnerability scanning), and `magento2-performance-audit` (runtime/ infrastructure) at whichever scope is in play — see "Scope modes" below. Theme scope additionally cross-references `magento2-hyva-dev` (CSP) and `magento2-frontend-dev` (Luma build/verify); it does not duplicate their content. Together with the trio, this forms the "QA quartet" — the four skills expected to run before a release. ## Scope modes | Mode | Trigger phrase example | File list source | |---|---|---| | Full path | "review this module/project" | module/theme/`app/code` path, as today | | Explicit file list | caller already knows which files | passed straight through | | Local git diff | "review this PR", branch already checked out | `git diff` against a base ref | | Remote fetch | "review MR !123 before I check it out" | GitHub MCP (`pull_request_read`, preferred for GitHub) / `glab mr diff` / `gh pr diff` — **text-pattern c...

Details

Author
ddtcorex
Repository
ddtcorex/dev-skills-hub
Created
3 months ago
Last Updated
today
Language
JavaScript
License
MIT

Integrates with

Similar Skills

Semantically similar based on skill content — not just same category