-
由 Will Chen 提交于
## Summary - Replace generic review prompts with three distinct reviewer personas (correctness, code-health, UX) - Each reviewer has a specialized focus area for more targeted feedback - Update workflow trigger to include skills directory changes ## Test plan - Verify the PR review workflow triggers correctly on PR changes - Confirm each reviewer persona produces relevant, focused feedback
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2644" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Refactors multi-pr-review to use three specialized reviewer personas—Correctness, Code Health, and UX—for more focused, actionable PR feedback. Updates the GitHub workflow to call the new skill. - **Refactors** - Added persona templates: correctness-reviewer.md, code-health-reviewer.md, ux-reviewer.md. - Updated SKILL.md with persona roles, severity guidance, and new references. - Removed old generic prompts: review_prompt_default.md and review_prompt_code_health.md. - Switched workflow command to /dyad:multi-pr-review. <sup>Written for commit f761c88cec8a91be810ca55f66db39b5b7acfa4e. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Medium Risk** > Changes affect the automated PR review pipeline and remove prompt files that the orchestrator may still reference, which could break CI-based reviews if not updated in lockstep. > > **Overview** > Updates the `dyad:multi-pr-review` documentation/spec to describe **three specialized sub-agent personas** (Correctness, Code Health, UX) and expands severity guidance to explicitly include UX blockers/degradations. > > Adds new persona reference docs (`correctness-reviewer.md`, `code-health-reviewer.md`, `ux-reviewer.md`) and removes the prior generic prompt templates (`review_prompt_default.md`, `review_prompt_code_health.md`). The PR review workflow is also updated to run `/dyad:multi-pr-review` instead of the previous command. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit f761c88cec8a91be810ca55f66db39b5b7acfa4e. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> Co-authored-by:Claude Opus 4.5 <noreply@anthropic.com>