← ClaudeAtlas

heyeddi-pr-reviewlisted

Reviews submitted PRs using only committed changes: product fit, docs drift, engineering quality, test coverage, and pre-merge gate. Use when approving a PR, doing reviewer QA, or self-checking before requesting review. Not for replying to review comments (use heyeddi-pr-respond).
HeyEddi-com/heyeddi-skills · ★ 0 · Code & Development · score 75
Install: claude install-skill HeyEddi-com/heyeddi-skills
# HeyEddi PR Review **Submitted PR review workflow**: judges only committed changes (`base...head`), not local WIP. You are the **reviewer** workflow. Scope every judgment to the PR diff (`base...head`). Ignore uncommitted files. **Artifacts:** `.heyeddi/docs/pr-<N>-review.md`: never repo root. ## When to use | Situation | Who | |-----------|-----| | Reviewer approving a teammate's PR | QA / lead | | Author self-check before requesting review | Author | | PM/eng sign-off on product + docs alignment | `@heyeddi-product` delegates here for PR scope | **Not this skill:** responding to human review comments → `@heyeddi-pr-respond`. ## Subagents (default) See `reference/subagents.md`. Main chat owns the verdict; delegate fetches and scans via **Task `shell`**. ## Mandatory pipeline Read **`reference/review-checklist.md`**. ``` fetch_pr_context --pr <N> → cache + paths; title/body not in stdout check_doc_drift --pr <N> → product.md / design.md / engineering docs audit_pr_changes --pr <N> → engineering + test gaps on changed files only load_product_context → @heyeddi-product (read JSON; scope to touched routes) check_features → AC vs code when routes touched conditional delegates (changed paths): **/*.vue → @visual-auditor, @no-duplicate-ui, @primevue-openprops-architect **/composables/** → @composable-patterns backend/**