Skip to content

dotnet-msbuild: move AITools.BinlogMcp source to dotnet-tools feed#680

Merged
Evangelink merged 1 commit into
mainfrom
dev/jankrivanek/binlog-mcp-feed
May 21, 2026
Merged

dotnet-msbuild: move AITools.BinlogMcp source to dotnet-tools feed#680
Evangelink merged 1 commit into
mainfrom
dev/jankrivanek/binlog-mcp-feed

Conversation

@JanKrivanek
Copy link
Copy Markdown
Member

Follow-up to #673. The AITools.BinlogMcp package was relocated from the dnceng dotnet-eng public feed to the dnceng dotnet-tools public feed, so the plugin can no longer restore it.

What changed

  • plugins/dotnet-msbuild/plugin.json--add-source URL updated to https://pkgs.dev.azure.com/dnceng/public/_packaging/dotnet-tools/nuget/v3/index.json.
  • plugins/dotnet-msbuild/skills/binlog-failure-analysis/SKILL.md — fallback note updated to reference the dotnet-tools feed.

Verified the package resolves on the new feed:

> dotnet package search AITools.BinlogMcp --source https://pkgs.dev.azure.com/dnceng/public/_packaging/dotnet-tools/nuget/v3/index.json --prerelease
| Package ID        | Latest Version        |
| AITools.BinlogMcp | 1.0.0-preview.26270.7 |

The MCP package was relocated from the dotnet-eng public feed to the dotnet-tools public feed on dnceng. Update plugin.json --add-source URL and the SKILL.md fallback note accordingly.
@github-actions
Copy link
Copy Markdown
Contributor

Skill Coverage Report

Plugin Skill Covered Coverage
dotnet-msbuild directory-build-organization 0/1 0%
dotnet-msbuild msbuild-modernization 6/7 85.7%
⚠️ dotnet-msbuild msbuild-server 7/9 77.8%
dotnet-msbuild resolve-project-references 5/6 83.3%
Uncovered: dotnet-msbuild/directory-build-organization
  • [CodePattern] [MSBuild] (line 122)
Uncovered: dotnet-msbuild/msbuild-modernization
  • [WorkflowStep] Step 6: Remove Unnecessary Boilerplate (line 217)
Uncovered: dotnet-msbuild/msbuild-server
  • [Validation] MSBUILDUSESERVER=1 is set in the shell (line 59)
  • [Validation] Second sequential build is faster than the first (line 60)
Uncovered: dotnet-msbuild/resolve-project-references
  • [Validation] ResolveProjectReferences was not set as the optimization target (line 58)

@JanKrivanek JanKrivanek marked this pull request as ready for review May 21, 2026 10:04
Copilot AI review requested due to automatic review settings May 21, 2026 10:04
@JanKrivanek JanKrivanek requested review from a team and ViktorHofer as code owners May 21, 2026 10:04
@JanKrivanek
Copy link
Copy Markdown
Member Author

/evaluate

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

Updates the dotnet-msbuild plugin configuration and associated documentation to restore AITools.BinlogMcp after it moved from the dnceng dotnet-eng NuGet feed to the dnceng dotnet-tools feed.

Changes:

  • Updated plugins/dotnet-msbuild/plugin.json to use the dotnet-tools feed URL for --add-source.
  • Updated the binlog-failure-analysis skill fallback documentation to reference the dotnet-tools feed.
Show a summary per file
File Description
plugins/dotnet-msbuild/plugin.json Switches AITools.BinlogMcp restore source to the dotnet-tools feed so the MCP server can be installed again.
plugins/dotnet-msbuild/skills/binlog-failure-analysis/SKILL.md Updates the offline/fallback note to reference the new feed name.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 2/2 changed files
  • Comments generated: 0

github-actions Bot added a commit that referenced this pull request May 21, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Skill Validation Results

Skill Scenario Quality Skills Loaded Overfit Verdict
binlog-failure-analysis Diagnose build failures from binlog only (no source files) 3.7/5 ⏰ → 4.3/5 🟢 ✅ binlog-failure-analysis; tools: skill, view, binlog-binlog_overview, binlog-binlog_errors, binlog-binlog_items, binlog-binlog_files, binlog-binlog_search / ⚠️ NOT ACTIVATED ✅ 0.11 [1]

[1] ⚠️ High run-to-run variance (CV=158%) — consider re-running with --runs 5

timeout — run(s) hit the (360s) scenario timeout limit; scoring may be impacted by aborting model execution before it could produce its full output (increase via timeout in eval.yaml)

Model: claude-opus-4.6 | Judge: claude-opus-4.6

🔍 Full Results - additional metrics and failure investigation steps

▶ Sessions Visualisation -- interactive replay of all evaluation sessions

@Evangelink Evangelink merged commit e951d69 into main May 21, 2026
41 checks passed
@Evangelink Evangelink deleted the dev/jankrivanek/binlog-mcp-feed branch May 21, 2026 11:14
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