Repository navigation
fix(pr-review-fixer): add gh-missing fallback via mcp__github__ tools - #5
Merged
Merged
Conversation
pr-review-fixer hard-depended on the gh CLI, which isn't installed in remote/cloud sessions (GitHub access there goes through mcp__github__* tools instead). Adds a gh-availability check and an MCP-based fallback path for fetch/resolve/reply, disclosed to its own gh-missing.md (only loaded when gh turns out to be missing) rather than inlined for every reader. Ports chiptus/UpLine#503's SKILL.md change into this repo's copy of the skill, already deduped per that PR's follow-up review. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01776GRzBiT1RkWHD7vyctrp
chiptus
commented
Sep 16, 2026
…p html_url Ports UpLine#503's follow-up review fixes into this repo's copy: - Phase 1's gh-availability check trimmed from six sentences to two; the auto-exec caveat was guarding against a documentation bug, not an agent-behavior risk, and doesn't apply once the check is plain prose with no `!` in it. - Both gh branches now explicitly ask the user for the PR number when it isn't already clear from context, rather than assuming it's always derivable. - gh-missing.md's Phase 1 now keeps html_url alongside author/body, which Phase 4 needs to build a reply. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01776GRzBiT1RkWHD7vyctrp
chiptus
added a commit
that referenced
this pull request
Sep 28, 2026
…#5) * fix(pr-review-fixer): add gh-missing fallback via mcp__github__ tools pr-review-fixer hard-depended on the gh CLI, which isn't installed in remote/cloud sessions (GitHub access there goes through mcp__github__* tools instead). Adds a gh-availability check and an MCP-based fallback path for fetch/resolve/reply, disclosed to its own gh-missing.md (only loaded when gh turns out to be missing) rather than inlined for every reader. Ports chiptus/UpLine#503's SKILL.md change into this repo's copy of the skill, already deduped per that PR's follow-up review. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01776GRzBiT1RkWHD7vyctrp * fix(pr-review-fixer): trim gh-check paragraph, ask for PR number, keep html_url Ports UpLine#503's follow-up review fixes into this repo's copy: - Phase 1's gh-availability check trimmed from six sentences to two; the auto-exec caveat was guarding against a documentation bug, not an agent-behavior risk, and doesn't apply once the check is plain prose with no `!` in it. - Both gh branches now explicitly ask the user for the PR number when it isn't already clear from context, rather than assuming it's always derivable. - gh-missing.md's Phase 1 now keeps html_url alongside author/body, which Phase 4 needs to build a reply. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01776GRzBiT1RkWHD7vyctrp --------- Co-authored-by: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Ports chiptus/UpLine#503 into this repo's copy of
pr-review-fixer:ghisn't installed in remote/cloud sessions (GitHub access there goes throughmcp__github__*tools instead), so the skill hard-failed oncommand not found. Adds agh-availability check and an MCP-based fallback path for fetch/resolve/reply, disclosed to its owngh-missing.md(only loaded whenghturns out to be missing) rather than inlined for every reader. Already includes the dedup fix applied as a follow-up on mattpocock#503 (dropped a sentence duplicated betweenSKILL.mdandgh-missing.md).Verification
claude plugin validate . --strictpasses.gh-missing path: read throughgh-missing.mdagainstmcp__github__pull_request_read's actual response shape (get_review_comments/get_reviews/get_comments) to confirm the reconstructed{threads, reviews, issueComments}matches what Phase 2+ expects.gh-available path is unchanged (original script calls untouched).🤖 Generated with Claude Code
https://claude.ai/code/session_01776GRzBiT1RkWHD7vyctrp
Generated by Claude Code