Skip to content

ci-analysis skill: MSBuild guidance, merge commit shortcut, MCP alternatives#124359

Merged
lewing merged 2 commits intodotnet:mainfrom
lewing:skill/ci-analysis-msbuild-guidance
Feb 13, 2026
Merged

ci-analysis skill: MSBuild guidance, merge commit shortcut, MCP alternatives#124359
lewing merged 2 commits intodotnet:mainfrom
lewing:skill/ci-analysis-msbuild-guidance

Conversation

@lewing
Copy link
Member

@lewing lewing commented Feb 12, 2026

Follow-up to #124240 with three improvements to the ci-analysis skill:

Changes

MSBuild cross-platform guidance (build-progression-analysis.md)

  • Added anti-pattern warning about MSBuild property path separator differences (; vs :) when comparing binlogs across Windows/Linux Helix queues
  • This is a common false positive in build progression analysis

Merge commit shortcut for target SHA extraction (build-progression-analysis.md)

  • Added Step 2b shortcut: gh api repos/{owner}/{repo}/git/commits/{sourceVersion} --jq '.parents[0].sha'
  • Extracts target branch HEAD from the merge commit's first parent — much simpler than parsing checkout logs
  • Noted caveat: only works for the latest build (GitHub recomputes merge ref on each push)
  • Added get_commit MCP tool as alternative when available

Inline MCP tool alternatives (all 4 files)

  • Added pull_request_read as alternative to gh pr checks in SKILL.md
  • Added search_issues MCP note in azdo-helix-reference.md
  • Added get_commit MCP note in build-progression-analysis.md
  • Reframed azure-cli.md with one-sentence MCP-first preamble

All changes are minimal inline additions — no structural changes to the skill.

Copilot AI review requested due to automatic review settings February 12, 2026 22:08
@github-actions github-actions bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Feb 12, 2026
Copy link
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 enhances the ci-analysis skill with three focused improvements to documentation, providing better guidance for CI failure investigation workflows. The changes add shortcuts for extracting merge commit information, clarify when to use MCP tools vs CLI alternatives, and add cross-platform warnings for MSBuild investigations.

Changes:

  • Added merge commit shortcut using GitHub API to simplify target branch HEAD extraction for current builds
  • Added inline MCP tool alternative references (get_commit, pull_request_read, search_issues) throughout documentation
  • Added MSBuild cross-platform anti-pattern warning about verifying changes across all platforms

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
.github/skills/ci-analysis/references/build-progression-analysis.md Added Step 2b shortcut for extracting target branch HEAD via GitHub API merge commit; added get_commit MCP alternative; added MSBuild cross-platform anti-pattern warning
.github/skills/ci-analysis/references/azure-cli.md Added MCP-first preamble noting AzDO MCP tools handle most queries with Azure CLI as fallback
.github/skills/ci-analysis/references/azdo-helix-reference.md Added parenthetical note to use search_issues MCP tool when available, otherwise gh CLI
.github/skills/ci-analysis/SKILL.md Added pull_request_read MCP tool as alternative to gh pr checks in PR Analysis Mode workflow description; clarified MCP tools provide structured access when available

@lewing lewing force-pushed the skill/ci-analysis-msbuild-guidance branch from 44df0b4 to cca6013 Compare February 12, 2026 22:16
@lewing lewing requested a review from steveisok February 12, 2026 22:29
@lewing lewing added area-skills Agent Skills and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Feb 12, 2026
@lewing lewing merged commit 55f6dc4 into dotnet:main Feb 13, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-skills Agent Skills

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants