[side-quest] Add node: Recovering When a Reviewer Skill Isn't Applied - #3126
Draft
github-actions[bot] wants to merge 1 commit into
Draft
[side-quest] Add node: Recovering When a Reviewer Skill Isn't Applied#3126github-actions[bot] wants to merge 1 commit into
github-actions[bot] wants to merge 1 commit into
Conversation
- Move the 'skill not applied' recovery checklist out of Step 14b's ready-for-review testing section into a new optional side quest (side-quest-14b-01-skill-recovery.md). - Replace the extracted checklist in 14b-pr-reviewer-workflow.md with a short summary and an Optional Side Quest callout linking to the new file. - List the new side quest in workshop/README.md under Optional Side Quests, noting it branches from Step 14b. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
github-actions
Bot
added
documentation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
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.
Caution
Protected files were modified in this change.
This pull request is in
request_reviewmode and requires explicit human scrutiny before merge.Protected files:
README.mdMined
workshop/14b-pr-reviewer-workflow.md, which embedded a full troubleshooting/recovery checklist for when a PR reviewer's inlinepr-revieweragent skips itspr-review-standardsskill (missing skills directory, reinforcing the instruction, recompiling) inside its "Test the Ready-for-Review Trigger" section.Side quest extracted: Side Quest: Recovering When a Reviewer Skill Isn't Applied — a self-contained detour covering how to recognize the symptom (a completed run with a generic, non-evidence-based review), confirm
.github/skills/agentic-workflows/is pushed, reinforce the skill-discovery instruction via/agentic-workflows, and re-verify.Routing changes:
14b-pr-reviewer-workflow.mdkeeps a one-sentence summary of the failure symptom and adds anOptional Side Questcallout (multi-line<details>pattern, matching existing workshop conventions) linking to the new file. The main testing path is otherwise unchanged.14b-pr-reviewer-workflow.md.workshop/README.mdlists the new side quest under Optional Side Quests, noting it branches from Step 14b.No YAML frontmatter workflow snippets appear in the new file, so no
gh aw compile --validatestep was needed.Warning
Firewall blocked 1 domain
The following domain was blocked by the firewall during workflow execution:
registry.npmjs.orgTo allow these domains, add them to the
network.allowedlist in your workflow frontmatter:See Network Configuration for more information.