メニュー
Address PR review comments by implementing reviewer feedback and suggestions.
デフォルトでは、最初にソースを確認する Prompt が選択されています。直接コマンドに切り替えるか、ローカルコピーをダウンロードすることもできます。
インストールを決める前に、SKILL.md と SkillsMP に表示されている付属ファイルをお読みください。
Codex または Claude でインストール この Prompt をコピーして Codex、Claude、または他のアシスタントに貼り付けると、Skill ページを確認してインストールできます。
直接コマンドでは確認用 Prompt が省略されます。実行前にソースを確認してください。
npx skills add https://github.com/majesticlabs-dev/majestic-marketplace --skill pr-comment-resolverコマンドは1行のまま表示されます。コピー前に横へスクロールして全体を確認してください。
ローカルで確認しますか？SkillsMP が現在取得できるファイルをダウンロードできます。
Create distinctive, production-grade frontend interfaces with high design quality. Use when building web components, pages, or applications. Includes framework-specific guidance for Tailwind, React, Vue, and Rails/Hotwire ecosystems.
Skill file structure, naming conventions, directory layout, frontmatter requirements, and invocation control. Use when creating skill files or slash commands to ensure correct format and validation.
Forces adversarial reasoning before committing to decisions. Triggers on architectural choices, approach selection, and planning phases to prevent premature commitment bias.
SKILL.md を表示中
SOC 職業分類に基づく
| name | pr-comment-resolver |
| description | Address PR review comments by implementing reviewer feedback and suggestions. |
| allowed-tools | Read Write Edit Grep Glob Bash |
Audience: Developers with PR review comments that need resolution.
Goal: Understand reviewer feedback, implement changes, verify fixes.
# List all PR comments
gh pr view <PR_NUMBER> --comments
# Get review comments on files
gh api repos/{owner}/{repo}/pulls/{pr_number}/comments
# Get specific review details
gh api repos/{owner}/{repo}/pulls/{pr_number}/reviews
| Type | Indicators | Action |
|---|---|---|
| Bug fix | "This will break...", "Missing check..." | Implement the fix |
| Refactor | "Extract this...", "This is too complex..." | Apply the pattern |
| Style | "Rename to...", "Format as..." | Adjust naming/format |
| Tests | "Add test for...", "Missing coverage..." | Add test cases |
| Docs | "Add comment...", "Document this..." | Add documentation |
| Questions | "Why...?", "What about...?" | Reply with explanation |
PR Comment Resolution
Comment: "[Quote or summary of feedback]"
Location: `file:line`
Changes:
- `path/to/file`: [Change description]
- `path/to/test`: [Test updates if any]
Resolution:
[How changes address the feedback]
Verified:
- [Tests run, if applicable]
Notes:
[Any context for reviewer]
PR Comment Resolution
Comment: "This function is too complex, please extract validation"
Location: `app/services/order.py:45`
Changes:
- `app/services/order.py`: Extracted to `_validate_items()`
- `tests/test_order.py`: Added tests for new method
Resolution:
Split 25-line method into two focused functions with single responsibility.
Verified:
- `pytest tests/test_order.py` - all passing
PR Comment Resolution
Comment: "Add test for the edge case when list is empty"
Location: `src/utils/helpers.ts:23`
Changes:
- `tests/utils/helpers.test.ts`: Added empty array test case
Resolution:
Added test that verifies graceful handling of empty input.
Verified:
- `npm test -- helpers` - all passing
If comment is unclear or conflicting: