/pr-review | sdlc | ClaudePluginHub

Back to Commands

Slash Command

Community

/pr-review

51

Description

Conduct comprehensive PR review from multiple perspectives (PM, Developer, QA, Security)

AI Summary

Comprehensive PR review from PM, developer, QA, and security perspectives. Use when you need thorough code analysis with immediate actionable feedback before merging.

Install

1

Add the repository(one-time)

$

/plugin marketplace add jmagly/ai-writing-guide

2

Install the plugin

$

/plugin install jmagly-sdlc-plugins-sdlc@jmagly/ai-writing-guide

Argument

<pr_link_or_number>

Allowed Tools

Bash(gh *)Read

Command Content

PR Review

PR Link/Number: $ARGUMENTS

Instructions: Execute each task in the order given to conduct a thorough code review. Update GitHub with this review. Important: The future is now—any improvements or "future" recommendations must be addressed immediately.

Task 1: Product Manager Review

Objective: Assess from a product management perspective, focusing on:

Business Value: Does this PR clearly advance our core product goals and deliver immediate ROI?
User Experience: Is the change intuitive and delightful for users right now? If not, make fixes immediately.
Strategic Alignment: Does the PR align with current (and long-term, i.e., now) strategic objectives?

Action: Provide clear directives on how to ensure maximum user and business impact. All "future" suggestions must be implemented now.

Task 2: Developer Review

Objective: Evaluate the code thoroughly from a senior lead engineer perspective:

Code Quality & Maintainability: Is the code structured for readability and easy maintenance? If not, refactor now.
Performance & Scalability: Will these changes operate efficiently at scale? If not, optimize immediately.
Best Practices & Standards: Note any deviation from coding standards and correct it now.

Action: Leave a concise yet complete review comment, ensuring all improvements happen immediately—no deferrals.

Task 3: Quality Engineer Review

Objective: Verify the overall quality, testing strategy, and reliability of the solution:

Test Coverage: Are there sufficient tests (unit, integration, E2E)? If not, add them now.
Potential Bugs & Edge Cases: Have all edge cases been considered? If not, address them immediately.
Regression Risk: Confirm changes don't undermine existing functionality. If risk is identified, mitigate now with additional checks or tests.

Action: Provide a detailed QA assessment, insisting any "future" improvements be completed right away.

Task 4: Security Engineer Review

Objective: Ensure robust security practices and compliance:

Vulnerabilities:

Links

GitHub Stats

4 forks

Updated 6 days ago

Other plugins with /pr-review

通過 Pull Request 的系統化審查確保代碼質量和架構健全性。

889

Pull Request の体系的レビューでコード品質とアーキテクチャの健全性を確保します。

889

Asegura la calidad del código y solidez arquitectónica a través de revisiones sistemáticas de Pull Request.

889

Revisão sistemática de Pull Request para garantir qualidade do código e sanidade da arquitetura.

889

Pull Request 의 체계적 리뷰로 코드 품질과 아키텍처 건전성을 확보합니다.

889