Skip to content

Skill to test changes using just-built MSBuild#13202

Merged
baronfel merged 2 commits intodotnet:mainfrom
rainersigwald:bootstrap-skill
Feb 4, 2026
Merged

Skill to test changes using just-built MSBuild#13202
baronfel merged 2 commits intodotnet:mainfrom
rainersigwald:bootstrap-skill

Conversation

@rainersigwald
Copy link
Copy Markdown
Member

@rainersigwald rainersigwald commented Feb 4, 2026

A standard part of the inner developer loop in this repo is to build MSBuild with a change, then run it against a repro project or projects to validate that MSBuild is generally working and the change affects the behavior you expect.

Adding a skill to (hopefully!) make that crystal clear to agents.

A standard part of the inner developer loop in this repo is to build
MSBuild with a change, then run it against a repro project or projects
to validate that MSBuild is generally working and the change affects the
behavior you expect.

Adding a skill to (hopefully!) make that crystal clear to agents.
@rainersigwald rainersigwald requested a review from a team as a code owner February 4, 2026 20:28
Copilot AI review requested due to automatic review settings February 4, 2026 20:28
@rainersigwald rainersigwald self-assigned this Feb 4, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a new skill document to guide developers through testing bug reproductions using the locally-built MSBuild (bootstrap). The skill provides step-by-step instructions for building MSBuild, running repro projects against the built version, and troubleshooting common issues.

Changes:

  • Added .github/skills/use-bootstrap-msbuild/SKILL.md with comprehensive guidance on using the bootstrap MSBuild for testing bug fixes

Copy link
Copy Markdown
Member

@baronfel baronfel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lovely to have! Will CCA use this automatically, or is there some kind of registration step that needs to happen so that agents see this?

@baronfel
Copy link
Copy Markdown
Member

baronfel commented Feb 4, 2026

Also, a nit - this won't work with Claude Code due to its location expectations. It would be ideal to have a way to easily reuse skill definitions across the major players.

@rainersigwald
Copy link
Copy Markdown
Member Author

Also, a nit - this won't work with Claude Code due to its location expectations. It would be ideal to have a way to easily reuse skill definitions across the major players.

I looked at docs a bit and as far as I can tell there is an interoperable location but it's .claude/skills/ and I don't want anyone to yell at me for checking that in on work time 😅

@baronfel
Copy link
Copy Markdown
Member

baronfel commented Feb 4, 2026

@rainersigwald just slap a symlink real quick - this is what Viktor and I did for his skills repo to lower the cost

@rainersigwald
Copy link
Copy Markdown
Member Author

Lovely to have! Will CCA use this automatically, or is there some kind of registration step that needs to happen so that agents see this?

AFAIK it will be known to any Copilot invocation after this is checked in (the directory structure is the registration). I did /skills reload in Copilot CLI after having it write the first draft of this and it sees it.

It's not totally clear to me though how well things will pick up on and use it. I think we'll have to try it and see.

@baronfel baronfel enabled auto-merge (squash) February 4, 2026 21:36
@baronfel baronfel merged commit da989a5 into dotnet:main Feb 4, 2026
9 checks passed
JanProvaznik pushed a commit to JanProvaznik/msbuild that referenced this pull request Feb 25, 2026
A standard part of the inner developer loop in this repo is to build
MSBuild with a change, then run it against a repro project or projects
to validate that MSBuild is generally working and the change affects the
behavior you expect.

Adding a skill to (hopefully!) make that crystal clear to agents.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants