Code review skill

[mthalman]

No description provided.

[Copilot]

Pull request overview

Adds a new .github/skills/code-review agent skill document intended to guide problem-focused pull request reviews in dotnet/sdk.

Changes:

• Introduces a code-review skill with step-by-step workflow (local checkout, context gathering, review, and posting).
• Adds guidance on change categorization, test/impact analysis, and what to flag vs. avoid flagging.

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

[nagilson]

Thanks for investing in this code review skill. Some fundamental questions:

1. Does this get picked up by the Copilot review from GitHub? If not, is there a way to do that?

2. Do you happen to have any demo PR reviews showing the results before and after this skill was introduced? I'm curious how it overall changes the behavior of the PR review. I think there's a lot of useful guidance here though I also tend to find the agents have gotten better over time at doing code review. There's a lot of language to parse here and I've noticed even subtle tweaks can significantly influence the outcome.

I wonder also what influenced the decisions here as many repos are adding their own code review skills such as https://github.com/microsoft/aspire-skills/blob/main/.github/skills/pr-review/SKILL.md.