Update .NET 10→11 migration skill for Preview 2 and 3 - #628
Conversation
|
/evaluate |
There was a problem hiding this comment.
Pull request overview
Updates the migrate-dotnet10-to-dotnet11 skill content to cover additional .NET 11 breaking changes through Preview 3, including new reference documentation and expanded evaluation scenarios.
Changes:
- Expanded
SKILL.mdguidance (assessment checklist, breaking-change patterns, verification items) and updated Preview 3 SDK pin example. - Added/updated reference documents for new breaking changes (ASP.NET Core OpenAPI/Microsoft.OpenApi v3, BackgroundService host-stop behavior, ZipArchive CRC32 validation, EF Core/SqlClient, AIA downloads, NativeAOT lib prefix, etc.).
- Added 4 new eval scenarios and updated known external domains to allow new reference URLs.
Show a summary per file
| File | Description |
|---|---|
| tests/dotnet-upgrade/migrate-dotnet10-to-dotnet11/eval.vally.yaml | Adds new evaluation stimuli for Preview 2/3 breaking changes (worker services, EF Core, ASP.NET Core, mTLS/AIA). |
| plugins/dotnet-upgrade/skills/migrate-dotnet10-to-dotnet11/SKILL.md | Updates skill instructions/checklists and reference loading table for Preview 3 coverage. |
| plugins/dotnet-upgrade/skills/migrate-dotnet10-to-dotnet11/references/sdk-msbuild-dotnet10to11.md | Documents new SDK/MSBuild/CLI breaking changes (NETSDK1235, --self-contained parsing). |
| plugins/dotnet-upgrade/skills/migrate-dotnet10-to-dotnet11/references/runtime-jit-dotnet10to11.md | Adds NativeAOT Unix lib prefix output change. |
| plugins/dotnet-upgrade/skills/migrate-dotnet10-to-dotnet11/references/efcore-dotnet10to11.md | Adds EF Core Preview 3 breaking changes (migrations behavior, dependencies, SqlClient, SQLite bundle removal, etc.). |
| plugins/dotnet-upgrade/skills/migrate-dotnet10-to-dotnet11/references/cryptography-dotnet10to11.md | Adds AIA intermediate cert download behavior change documentation. |
| plugins/dotnet-upgrade/skills/migrate-dotnet10-to-dotnet11/references/core-libraries-dotnet10to11.md | Adds core library breaking changes (CRC32 validation, BackgroundService behavior, tar hardlinks, Zstandard in-box). |
| plugins/dotnet-upgrade/skills/migrate-dotnet10-to-dotnet11/references/aspnetcore-dotnet10to11.md | New ASP.NET Core reference doc (OpenAPI.NET v3 upgrade, Blazor Virtualize default change). |
| eng/known-domains.txt | Allows new GitHub repo domain prefix used by the added ASP.NET Core reference sources. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comments suppressed due to low confidence (1)
tests/dotnet-upgrade/migrate-dotnet10-to-dotnet11/eval.vally.yaml:329
- The output-matches regex
(Virtualize|OverscanCount|3|15)is overly permissive: it will pass if the model mentions any "3" (e.g., "Preview 3") or any other incidental number, without actually addressing the Virtualize OverscanCount change. Tighten this check by requiring Virtualize/OverscanCount and the new default value (15) in the same regex (e.g., via lookaheads) or by splitting into multiple, more specificoutput-matchesgraders.
pattern: (OpenApi|Microsoft\.OpenApi|v3|3\.2|upgrade)
- type: output-matches
config:
pattern: (Virtualize|OverscanCount|3|15)
- type: prompt
- Files reviewed: 9/9 changed files
- Comments generated: 3
Skill Coverage Report
Uncovered:
|
Skill Validation Results
[1]
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 |
Add breaking changes from .NET 11 Preview 2 and Preview 3 to the migrate-dotnet10-to-dotnet11 skill. The skill previously covered only Preview 1. Changes: - Add 15 new breaking changes across all reference files - Create new aspnetcore-dotnet10to11.md reference (OpenAPI v3, Blazor) - Update SKILL.md workflow steps, assessment checklist, and patterns - Add 4 new eval scenarios covering high-impact P2/P3 changes - Add github.com/dotnet/aspnetcore to known-domains.txt Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Replace prompt-echoable 'corrupt' in ZipArchive eval regex with 'checksum' - Update EF Core reference note to say 'Preview 1 through Preview 3' - Add missing Source link for Zstandard entry Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
a4d8e2e to
4139cc7
Compare
- Tighten OverscanCount grader regex to require 'OverscanCount' with a number, instead of matching bare '3' anywhere - Wrap Blazor component snippet in backticks so it renders as code Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
/evaluate |
Skill Validation Results
[1] (Isolated) Quality improved but weighted score is -12.4% due to: quality, tokens (13296 → 48909), tool calls (0 → 4)
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 |
Summary
Updates the migrate-dotnet10-to-dotnet11 skill to cover breaking changes through .NET 11 Preview 3 (previously Preview 1 only).
Changes
Reference files — 15 new breaking changes added:
SKILL.md — Updated frontmatter (P1→P3), Step 1 assessment checklist, Step 3 code patterns, Step 4 behavioral changes, Step 5 global.json version, Step 6 verification items, reference tables.
Evals — 4 new scenarios: BackgroundService+ZipCRC32, EF Core SqlClient+Design, ASP.NET Core OpenAPI+Blazor, mTLS AIA certificates.
known-domains.txt — Added \github.com/dotnet/aspnetcore.
Validation