Skip to content

Add Copilot skill for investigating VS integration test failures#83002

Merged
akhera99 merged 2 commits intodotnet:mainfrom
akhera99:add_integration_test_skill
Apr 6, 2026
Merged

Add Copilot skill for investigating VS integration test failures#83002
akhera99 merged 2 commits intodotnet:mainfrom
akhera99:add_integration_test_skill

Conversation

@akhera99
Copy link
Copy Markdown
Member

@akhera99 akhera99 commented Mar 31, 2026

Adds a new Copilot skill that analyzes roslyn-integration-CI pipeline failures:

  • Fetches build timeline and identifies failed/canceled VS_Integration_* jobs
  • Parses test runner logs for VSIX install status, test progress, and VS/.NET versions
  • Downloads and analyzes published artifacts (exception XMLs, screenshots, MEF errors)
  • Merges timeout + exception signals to identify root causes like runtime mismatches
  • Outputs structured JSON summary for agent reasoning
Microsoft Reviewers: Open in CodeFlow

…test failures

Adds a new Copilot skill that analyzes roslyn-integration-CI pipeline failures:
- Fetches build timeline and identifies failed/canceled VS_Integration_* jobs
- Parses test runner logs for VSIX install status, test progress, and VS/.NET versions
- Downloads and analyzes published artifacts (exception XMLs, screenshots, MEF errors)
- Merges timeout + exception signals to identify root causes like runtime mismatches
- Outputs structured JSON summary for agent reasoning

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@akhera99 akhera99 marked this pull request as ready for review April 6, 2026 19:59
@akhera99 akhera99 requested a review from a team as a code owner April 6, 2026 19:59

if ($job.name -match 'VS_Integration') {
# Check if this was a timeout (canceled after running > 140 min)
if ($job.result -eq 'canceled' -and $durationMin -ne 'N/A' -and $durationMin -gt 140) {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This check is a bit dangerous if we ever change the timeout.


# --- Merge signals into root cause ---

# Best case: Exception XML explains WHY tests hung (e.g., assembly load failure causing timeout)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is this something that needs to be in the script or are we better off just letting the LLM figure it out? My worry is if this might direct it in a bad direction...

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

good point, changed

@akhera99 akhera99 merged commit fca1217 into dotnet:main Apr 6, 2026
3 of 4 checks passed
@dotnet-policy-service dotnet-policy-service bot added this to the Next milestone Apr 6, 2026
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.

2 participants