Repository navigation
feat(skills): adjudicate posted pull request review findings - #20
Merged
Merged
Conversation
Add the adjudicate-review-outcomes skill for picking up review findings already posted into a pull request conversation, deciding FIX or REFUSE for each with stated reasoning, and publishing one disposition comment back onto that same pull request. The skill leads with the disposition comment because that record is the deliverable: it goes on the same pull request, covers every finding, and an unposted record means the run is not done. It then states the blocker/major/minor/nit priority rule and the nine rules a run must satisfy, each with the reason it exists. Register the skill in the documentation audience inventory and declare its load trigger in AGENTS.md section 7.
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.
Transplants commit 25e3d71 onto feat/omp-adaptor unchanged.
The adjudicate-review-outcomes skill was authored on fm/adjudicate-review-outcomes-skill, which was cut from an unrelated base and was never opened or merged, leaving the skill invisible to every running session. Only that single commit is wanted here, so it is cherry-picked rather than merging the branch.
Contents, identical to the original commit:
.agents/skills/adjudicate-review-outcomes/SKILL.md(new, 87 lines), byte-identical togit show 25e3d719:.agents/skills/adjudicate-review-outcomes/SKILL.mdAGENTS.md: two lines in section 7 declaring the load triggerdocs/documentation-audiences.json: four lines registering the skill surfaceNo existing entry was removed on either side.
tests/fm-documentation-audiences.test.shandtests/fm-ensure-agents-md.test.shpass.