PR Comment Resolver Skill

PR Comment Resolver

Audience: Developers with PR review comments that need resolution.

Goal: Understand reviewer feedback, implement changes, verify fixes.

PR Comment Commands

# 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

Comment Types

| 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 |

Workflow

Read - Get comment content and context
Locate - Find the code being discussed
Understand - Note constraints/preferences
Implement - Make requested changes
Verify - Run tests if code changed
Report - Summarize resolution

Implementation Principles

Address the specific feedback
Maintain existing code style
Don't break other functionality
Follow project conventions
Ask if feedback is ambiguous

Resolution Report

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]

Example Resolutions

Refactoring Request

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

Missing Test Coverage

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

When Blocked

If comment is unclear or conflicting:

State your interpretation
Explain any constraints
Suggest alternatives if needed
Ask for clarification

Agent Skills: PR Comment Resolver

Install this agent skill to your local

Skill Files