chore: add review-fix-merge PR review skills - #128
Merged
Conversation
Two project skills for reviewing + merging PRs (force-added past the .claude/ gitignore so they live with the repo rather than only locally): - review-fix-merge-pr: single-reviewer loop — review a PR's diff, fix findings, loop review<->fix until clean, then squash-merge. Bundles a parameterized Workflow script (review-fix-loop.js). - review-fix-merge-pr-advanced: multi-lens variant — parallel reviewers each backed by a specialized skill (security-audit, performance, code-quality, testing-strategy, correctness), adversarial verification of findings before fixing, loop, then merge (advanced-review-fix-loop.js). Co-Authored-By: Claude Opus 4.8 (1M context) <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
1 participant
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.
Adds two project skills for reviewing + merging PRs, so they live with the repo instead of only in a local
.claude/.Note:
.claude/is gitignored (line 26), so these were force-added (git add -f). They stay tracked and their edits show normally; new files added inside these skill dirs later would need anothergit add -f.Skills
review-fix-merge-pr— single-reviewer loop: review a PR's diff → fix findings → loop review↔fix until clean → squash-merge. Bundles a parameterizedWorkflowscript (review-fix-loop.js).review-fix-merge-pr-advanced— multi-lens variant: parallel reviewers each backed by a specialized skill (security-audit,performance,code-quality,testing-strategy, correctness), adversarial verification of each finding before fixing, loop, then merge (advanced-review-fix-loop.js).Both scripts are syntax-checked; invoked via
/review-merge-pr...or by asking to review-and-merge a PR.🤖 Generated with Claude Code