converts the radius-code-review prompt into a skill#11961
Conversation
Signed-off-by: Nell Shamrell <nellshamrell@gmail.com>
There was a problem hiding this comment.
Pull request overview
Converts the existing radius.code-review.prompt.md prompt into a structured skill under .github/skills/radius-code-review/, registering it in the Copilot instructions index. The new skill preserves the workflow for producing PR analysis, review markdown, and a GitHub API posting script, while improving documentation structure and tightening script generation guidance (e.g., resolving head commit SHA instead of using HEAD).
Changes:
- Add new skill
radius-code-reviewwith SKILL.md describing inputs, steps, outputs, and a reference posting script. - Remove the legacy
radius.code-review.prompt.mdprompt file. - Register the new skill in
.github/copilot-instructions.md.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| .github/skills/radius-code-review/SKILL.md | New skill describing the PR code-review workflow and posting script template. |
| .github/prompts/radius.code-review.prompt.md | Removed; superseded by the new skill. |
| .github/copilot-instructions.md | Adds the new skill to the skills index. |
nellshamrell
left a comment
There was a problem hiding this comment.
Clean format/packaging migration. The new skill preserves the prompt workflow and adds real improvements (commit-SHA resolution, Step 4 doc-impact assessment, Quality Checklist, default event: COMMENT). Frontmatter and structure match the existing contributing-docs-updater skill, and relative links resolve.
Non-blocking issues to address:
- PR description is unfilled — please pick a "Type of change" option (maintenance/refactor) and clean up the
Fixes:line. - Step 5 says "prefer
jq" but the embedded reference script parses JSON withgrep/sedand builds payloads via string interpolation. Either update the reference to usejqor relax the guidance. - Confirm in the PR description that removing the
.prompt.mdentry point is intentional — VS Code Copilot Chat users invoking the old prompt by name will no longer find it.
Contributor doc impact: none. The change is scoped to .github/ Copilot tooling.
… be invoked in the same way as when it used to be a prompt Signed-off-by: Nell Shamrell <nellshamrell@gmail.com>
…NUMBER - Rewrite Step 5 reference script to use jq for SHA extraction, payload construction, and review-id parsing (no more grep/sed parsing of API responses or string-interpolated JSON). - Accept PR_NUMBER, REPO_OWNER, REPO_NAME from environment so the generated script can be re-run against a different PR without edits. - Add explicit jq prerequisite and fail-fast checks for jq and PR_NUMBER. - Tighten Quality Checklist to a concrete "passes shellcheck -x" item. Signed-off-by: Nell Shamrell <nellshamrell@gmail.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
29a86d1 to
6455ec5
Compare
nellshamrell
left a comment
There was a problem hiding this comment.
This is a clean migration from a large prompt file to a reusable skill, and the prompt shim preserves the existing invocation path. The skill also addresses prior feedback around jq, head SHA resolution, and environment-driven script configuration.
I found one blocking issue in the reference review-posting script. The inline comment object uses line by itself, but GitHub review comments need a valid diff location form. The safest fix is to either use position as shown in the create-review API examples, or include the line-side fields expected by the API, such as line plus side: "RIGHT" for comments on added/current lines. Without that, generated scripts can fail with a 422 validation response when posting inline comments.
Contributor doc impact: no updates are needed under docs/contributing/ or docs/architecture/. The PR changes Copilot workflow customization only, and the contributor-facing .github/copilot-instructions.md skill registration is already included.
Signed-off-by: Nell Shamrell <nellshamrell@gmail.com>
Adds a short subsection to the code-reviewing guide noting that the radius-code-review Copilot skill is available as an optional aid, with explicit reminders that the human reviewer still owns the review and must verify any AI-generated feedback against the actual diff. Signed-off-by: Nell Shamrell <nellshamrell@gmail.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Adds a short subsection to the pull-request contributing guide noting that contributors can run the radius-code-review skill against their own PR to generate an initial AI-assisted review before requesting human review, with explicit caveats that the output is a draft and that a human reviewer is still required for merge. Signed-off-by: Nell Shamrell <nellshamrell@gmail.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…s-code-review Both the reviewer and contributor guides now list prerequisites (`gh`, `jq`, Copilot CLI or VS Code Copilot Chat with prompt files enabled) and show the exact invocation for each path, instead of referring obliquely to "the shim" or "the equivalent VS Code prompt". Signed-off-by: Nell Shamrell <nellshamrell@gmail.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
47d7439 to
befbe12
Compare
nicolejms
left a comment
There was a problem hiding this comment.
Looks good. Thanks for doing this. I think keeping the prompt that references the skill is ok so the prompt will continue to work.


Description
Converts the existing
radius.code-review.prompt.mdCopilot prompt into a structured skill under.github/skills/radius-code-review/, and registers it in the Copilot skills index. The skill preserves the full PR-review workflow (per-file analysis, review markdown, GitHub API posting script) and adds:HEAD)contributing-docs-updaterskillevent: COMMENTas the default review eventA thin
.github/prompts/radius.code-review.prompt.mdshim is retained so existing VS Code Copilot Chat invocations continue to work — the shim delegates to the SKILL.md so there is no duplication.Type of change
Contributor checklist
eng/design-notes/in this repository, if new APIs are being introduced.