Add more detailed steps for skill testing#37959
Merged
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the Copilot skill authoring/testing guidance and adjusts repository configuration to support more detailed skill testing and CI failure investigation (including shared MCP server settings and Copilot agent setup steps).
Changes:
- Add a shared VS Code MCP server configuration (
.vscode/mcp.json) and stop ignoring the entire.vscode/directory. - Expand the
make-skillskill with additional testing guidance (multi-model testing, anti-patterns, testing patterns references). - Update the Copilot agent setup workflow to check out the repo and attempt to restore the repo-local .NET SDK.
Reviewed changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
.vscode/mcp.json |
Adds shared MCP server definitions for investigation workflows. |
.gitignore |
Stops ignoring .vscode/, allowing MCP config to be committed. |
.github/workflows/copilot-setup-steps.yml |
Adds checkout + .NET restore + environment export for agent sessions. |
.agents/skills/make-skill/references/testing-patterns.md |
New reference doc describing multi-model testing methodology. |
.agents/skills/make-skill/references/anti-patterns.md |
New reference doc cataloging common pitfalls/anti-patterns in skill design. |
.agents/skills/make-skill/SKILL.md |
Adds script-writing guidance and a multi-model testing step pointing to new references. |
.agents/skills/ci-analysis/references/manual-investigation.md |
Minor formatting/spacing tweaks for readability. |
AndriySvyryd
force-pushed
the
CopilotSWE
branch
from
6ac0a0b to
857e2d6
Compare
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
AndriySvyryd
force-pushed
the
CopilotSWE
branch
from
857e2d6 to
84f2657
Compare
roji
approved these changes
Mar 20, 2026
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.
Add shared MCP settings for CI failure investigation
Change CCA setup steps to allow .NET MCP servers