Skip to content

React to PermissionRequestResultKind breaking change in GitHub.Copilot.SDK 0.1.32#326

Merged
ViktorHofer merged 2 commits intodependabot/nuget/eng/skill-validator/src/all-other-nuget-ac7f55a1f1from
copilot/sub-pr-325
Mar 11, 2026
Merged

React to PermissionRequestResultKind breaking change in GitHub.Copilot.SDK 0.1.32#326
ViktorHofer merged 2 commits intodependabot/nuget/eng/skill-validator/src/all-other-nuget-ac7f55a1f1from
copilot/sub-pr-325

Conversation

Copy link
Contributor

Copilot AI commented Mar 11, 2026

SDK 0.1.32 changed PermissionRequestResult.Kind from string to the strongly-typed struct PermissionRequestResultKind, breaking compilation in four service files.

Changes

  • AgentRunner.cs, Judge.cs, PairwiseJudge.cs: Replace "approved" / "denied-by-rules" string literals with PermissionRequestResultKind.Approved / PermissionRequestResultKind.DeniedByRules
  • OverfittingJudge.cs (2 sites): Same — always uses DeniedByRules since the judge session grants no tool permissions
// Before
Kind = result ? "approved" : "denied-by-rules",

// After
Kind = result ? PermissionRequestResultKind.Approved : PermissionRequestResultKind.DeniedByRules,

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: ViktorHofer <7412651+ViktorHofer@users.noreply.github.com>
Copilot AI changed the title [WIP] Update GitHub.Copilot.SDK to version 0.1.32 React to PermissionRequestResultKind breaking change in GitHub.Copilot.SDK 0.1.32 Mar 11, 2026
@ViktorHofer ViktorHofer marked this pull request as ready for review March 11, 2026 07:33
@ViktorHofer ViktorHofer merged commit a0631dd into dependabot/nuget/eng/skill-validator/src/all-other-nuget-ac7f55a1f1 Mar 11, 2026
16 checks passed
@ViktorHofer ViktorHofer deleted the copilot/sub-pr-325 branch March 11, 2026 07:36
@github-actions
Copy link
Contributor

Skill Validation Results

Skill Scenario Quality Skills Loaded Overfit Verdict
csharp-scripts Test a C# language feature with a script 4.0/5 → 4.0/5 ✅ csharp-scripts; tools: skill, create 🟡 0.32
nuget-trusted-publishing Set up trusted publishing for a new NuGet library 1.0/5 ⏰ → 4.0/5 🟢 ✅ nuget-trusted-publishing; tools: skill, glob, create ✅ 0.12
nuget-trusted-publishing Set up NuGet publishing without mentioning trusted publishing 3.0/5 → 5.0/5 🟢 ✅ nuget-trusted-publishing; tools: skill, report_intent, task, view, glob, bash ✅ 0.12
nuget-trusted-publishing Migrate existing workflow from API key to trusted publishing 2.0/5 → 4.0/5 🟢 ✅ nuget-trusted-publishing; tools: skill ✅ 0.12
dotnet-pinvoke Generate LibraryImport declaration from C header (.NET 8+) 3.0/5 → 5.0/5 🟢 ✅ dotnet-pinvoke; tools: skill ✅ 0.07
dotnet-pinvoke Generate LibraryImport declaration from C header (.NET Framework) 5.0/5 → 5.0/5 ✅ dotnet-pinvoke; tools: skill ✅ 0.07
dotnet-trace-collect High CPU in Kubernetes on Linux (.NET 8) 3.0/5 → 4.0/5 🟢 ✅ dotnet-trace-collect; tools: skill, report_intent, view ✅ 0.16
dotnet-trace-collect .NET Framework on Windows without admin privileges 1.0/5 → 5.0/5 🟢 ✅ dotnet-trace-collect; tools: skill ✅ 0.16
dotnet-trace-collect .NET 10 on Linux with root access and native call stacks 1.0/5 → 4.0/5 🟢 ✅ dotnet-trace-collect; tools: skill ✅ 0.16
dotnet-trace-collect Memory leak on Linux (.NET 8) 2.0/5 → 3.0/5 🟢 ✅ dotnet-trace-collect; tools: skill ✅ 0.16
dotnet-trace-collect Slow requests on Windows with PerfView 5.0/5 → 5.0/5 ✅ dotnet-trace-collect; tools: skill, report_intent, view ✅ 0.16
dotnet-trace-collect Excessive GC on Linux (.NET 8) 3.0/5 → 5.0/5 🟢 ✅ dotnet-trace-collect; tools: skill ✅ 0.16
dotnet-trace-collect Hang or deadlock diagnosis on Linux 3.0/5 → 4.0/5 🟢 ✅ dotnet-trace-collect; tools: report_intent, skill ✅ 0.16
dotnet-trace-collect Windows container high CPU with PerfView 1.0/5 → 4.0/5 🟢 ✅ dotnet-trace-collect; tools: skill ✅ 0.16
dotnet-trace-collect Long-running intermittent issue with PerfView triggers 2.0/5 → 5.0/5 🟢 ✅ dotnet-trace-collect; tools: skill, report_intent, view ✅ 0.16
dotnet-trace-collect Linux pre-.NET 10 needing native call stacks 2.0/5 → 5.0/5 🟢 ✅ dotnet-trace-collect; tools: skill, report_intent, view, glob ✅ 0.16
dotnet-trace-collect Windows modern .NET with admin high CPU 5.0/5 → 5.0/5 ✅ dotnet-trace-collect; tools: skill, report_intent, view ✅ 0.16
dotnet-trace-collect Memory leak on .NET Framework Windows 3.0/5 → 5.0/5 🟢 ✅ dotnet-trace-collect; tools: skill, report_intent, view ✅ 0.16
dotnet-trace-collect Kubernetes with console access prefers console tools 4.0/5 → 5.0/5 🟢 ✅ dotnet-trace-collect; tools: skill ✅ 0.16
dotnet-trace-collect Container installation without .NET SDK 4.0/5 → 2.0/5 🔴 ✅ dotnet-trace-collect; tools: skill ✅ 0.16
dotnet-trace-collect HTTP 500s from downstream service on Linux (.NET 8) 5.0/5 → 5.0/5 ✅ dotnet-trace-collect; tools: skill, report_intent, view ✅ 0.16
dotnet-trace-collect Networking timeouts on Windows with admin (.NET 8) 2.0/5 → 5.0/5 🟢 ✅ dotnet-trace-collect; tools: skill, report_intent, view ✅ 0.16
microbenchmarking Investigate runtime upgrade performance impact 3.0/5 → 4.0/5 🟢 ✅ microbenchmarking; tools: skill, glob ✅ 0.10
clr-activation-debugging Diagnose unexpected FOD dialog from native build tool 1.0/5 → 5.0/5 🟢 ✅ clr-activation-debugging; tools: skill ✅ 0.08
clr-activation-debugging Diagnose FOD suppressed but activation still failing 1.0/5 → 5.0/5 🟢 ✅ clr-activation-debugging; tools: skill ✅ 0.08
clr-activation-debugging Explain why same binary behaves differently under different launch methods 1.0/5 → 5.0/5 🟢 ✅ clr-activation-debugging; tools: skill ✅ 0.08
clr-activation-debugging Analyze healthy managed EXE activation 1.0/5 → 5.0/5 🟢 ✅ clr-activation-debugging; tools: skill, glob ✅ 0.08
clr-activation-debugging Identify multiple activation sequences in a single log 2.0/5 → 4.0/5 ⏰ 🟢 ✅ clr-activation-debugging; tools: skill, task, glob, read_agent ✅ 0.08
clr-activation-debugging Explain useLegacyV2RuntimeActivationPolicy in activation log 2.0/5 → 3.0/5 🟢 ✅ clr-activation-debugging; tools: skill ✅ 0.08
clr-activation-debugging Decline non-CLR-activation issue 1.0/5 → 5.0/5 🟢 ✅ clr-activation-debugging; tools: skill ✅ 0.08
analyzing-dotnet-performance Detects compiled regex startup budget and regex chain allocations 1.0/5 → 5.0/5 🟢 ✅ analyzing-dotnet-performance; tools: skill, grep ✅ 0.16
analyzing-dotnet-performance Detects CurrentCulture comparer and compiled regex budget in inflection rules 1.0/5 → 1.0/5 ⏰ ✅ analyzing-dotnet-performance; tools: skill ✅ 0.16
analyzing-dotnet-performance Finds per-call Dictionary allocation not hoisted to static 1.0/5 → 5.0/5 ⏰ 🟢 ✅ analyzing-dotnet-performance; tools: skill ✅ 0.16
analyzing-dotnet-performance Catches compound allocations in recursive number converter with ToLower 1.0/5 → 4.0/5 🟢 ✅ analyzing-dotnet-performance; tools: skill ✅ 0.16
analyzing-dotnet-performance Finds StringComparison.Ordinal missing and FrozenDictionary opportunities 1.0/5 → 5.0/5 🟢 ✅ analyzing-dotnet-performance; tools: skill, grep ✅ 0.16
analyzing-dotnet-performance Detects Aggregate+Replace chain and struct missing IEquatable 1.0/5 → 1.0/5 ⏰ ✅ analyzing-dotnet-performance; tools: skill ✅ 0.16
analyzing-dotnet-performance Finds branched Replace chain in format string manipulation 1.0/5 → 4.0/5 🟢 ✅ analyzing-dotnet-performance; tools: skill ✅ 0.16
analyzing-dotnet-performance Catches LINQ on hot-path string processing and All(char.IsUpper) 1.0/5 → 5.0/5 🟢 ✅ analyzing-dotnet-performance; tools: skill ✅ 0.16
analyzing-dotnet-performance Detects LINQ pipeline in TimeSpan formatting and collection processing 1.0/5 → 5.0/5 🟢 ✅ analyzing-dotnet-performance; tools: skill ✅ 0.16
analyzing-dotnet-performance Flags Span inconsistencies and compound method chains in truncation library 1.0/5 → 4.0/5 🟢 ✅ analyzing-dotnet-performance; tools: skill ✅ 0.16
analyzing-dotnet-performance Identifies unsealed leaf classes and locale hierarchy patterns 1.0/5 → 5.0/5 🟢 ✅ analyzing-dotnet-performance; tools: skill ✅ 0.16
android-tombstone-symbolication Symbolicate .NET frames in an Android tombstone 3.0/5 → 4.0/5 🟢 ✅ android-tombstone-symbolication; tools: skill, glob ✅ 0.19
android-tombstone-symbolication Recognize tombstone with no .NET frames 5.0/5 → 5.0/5 ✅ android-tombstone-symbolication; tools: skill, bash ✅ 0.19
android-tombstone-symbolication Symbolicate CoreCLR frames in an Android tombstone 2.0/5 ⏰ → 4.0/5 🟢 ✅ android-tombstone-symbolication; tools: skill, glob ✅ 0.19
android-tombstone-symbolication Recognize NativeAOT tombstone with app binary and libSystem.Native.so 2.0/5 → 4.0/5 🟢 ✅ android-tombstone-symbolication; tools: skill, bash ✅ 0.19
android-tombstone-symbolication Symbolicate multi-thread tombstone 1.0/5 ⏰ → 4.0/5 🟢 ✅ android-tombstone-symbolication; tools: skill ✅ 0.19
android-tombstone-symbolication Handle .NET frames with no BuildId metadata 5.0/5 → 5.0/5 ✅ android-tombstone-symbolication; tools: skill, bash ✅ 0.19
android-tombstone-symbolication Symbolicate tombstone with multiple .NET libraries and different BuildIds 3.0/5 → 5.0/5 🟢 ✅ android-tombstone-symbolication; tools: skill, glob ✅ 0.19
android-tombstone-symbolication Reject iOS crash log as wrong format 5.0/5 → 5.0/5 ℹ️ not activated (expected) ✅ 0.19
dump-collect Configure automatic crash dumps for CoreCLR app on Linux 5.0/5 → 5.0/5 ✅ dump-collect; tools: skill, report_intent, view 🟡 0.28
dump-collect Set up NativeAOT crash dumps with createdump in Kubernetes 2.0/5 → 5.0/5 🟢 ✅ dump-collect; tools: skill 🟡 0.28
dump-collect Recover crash dump from macOS NativeAOT without createdump 3.0/5 → 3.0/5 ✅ dump-collect; tools: report_intent, skill, view, glob 🟡 0.28 [1]
dump-collect Configure CoreCLR dump collection in Alpine Docker as non-root 4.0/5 → 4.0/5 ✅ dump-collect; tools: skill 🟡 0.28
dump-collect Advisory: macOS NativeAOT crash dump recovery steps 2.0/5 → 4.0/5 🟢 ✅ dump-collect; tools: skill 🟡 0.28
dump-collect Advisory: CoreCLR Alpine Docker non-root configuration 3.0/5 → 5.0/5 🟢 ✅ dump-collect; tools: skill 🟡 0.28
dump-collect Advisory: NativeAOT Kubernetes dump collection setup 3.0/5 → 5.0/5 🟢 ✅ dump-collect; tools: report_intent, skill, view 🟡 0.28
dump-collect Detect runtime and configure crash dumps for unknown .NET app on Linux 4.0/5 → 4.0/5 ✅ dump-collect; tools: skill, glob 🟡 0.28 [2]
dump-collect Decline dump analysis request 2.0/5 → 5.0/5 🟢 ℹ️ not activated (expected) 🟡 0.28
optimizing-ef-core-queries Optimize bulk operations with EF Core 7+ ExecuteUpdate and ExecuteDelete 5.0/5 → 4.0/5 🔴 ✅ optimizing-ef-core-queries; tools: report_intent, skill 🟡 0.50
build-parallelism Analyze build parallelism bottlenecks 4.0/5 → 5.0/5 🟢 ✅ build-parallelism; tools: task, glob, skill ✅ 0.16
including-generated-files Diagnose generated file inclusion failure 3.0/5 → 5.0/5 🟢 ✅ including-generated-files; tools: skill ✅ 0.17
msbuild-antipatterns Review MSBuild files for anti-patterns and style issues 5.0/5 → 5.0/5 ✅ msbuild-antipatterns; tools: skill ✅ 0.06
build-perf-baseline Establish build performance baseline and recommend optimizations 3.0/5 → 3.0/5 ✅ build-perf-baseline; binlog-failure-analysis; tools: skill, glob 🟡 0.33
msbuild-modernization Modernize legacy project to SDK-style 4.0/5 → 5.0/5 🟢 ✅ msbuild-modernization; tools: skill ✅ 0.04
directory-build-organization Organize build infrastructure for a multi-project repo 4.0/5 → 5.0/5 🟢 ✅ directory-build-organization; msbuild-antipatterns; tools: task, bash, skill ✅ 0.15
check-bin-obj-clash Diagnose bin/obj output path clashes 4.0/5 → 5.0/5 🟢 ✅ check-bin-obj-clash; tools: skill, edit ✅ 0.12 [3]
incremental-build Analyze incremental build issues 3.0/5 → 4.0/5 🟢 ✅ incremental-build; tools: skill, bash ✅ 0.12
eval-performance Analyze MSBuild evaluation performance issues 3.0/5 → 4.0/5 🟢 ✅ eval-performance; tools: skill ✅ 0.15 [4]
build-perf-diagnostics Analyze analyzer performance impact on builds 5.0/5 → 5.0/5 ✅ binlog-generation; build-perf-diagnostics; tools: skill, edit 🟡 0.21
binlog-generation Build project with /bl flag 1.0/5 → 5.0/5 🟢 ✅ binlog-generation; tools: skill 🟡 0.48 [5]
binlog-generation Build with /bl in PowerShell 3.0/5 → 5.0/5 🟢 ✅ binlog-generation; tools: skill 🟡 0.48
binlog-generation Build multiple configurations with unique binlogs 5.0/5 → 5.0/5 ✅ binlog-generation; tools: skill 🟡 0.48
binlog-failure-analysis Diagnose build failures from binlog only (no source files) 5.0/5 → 5.0/5 ✅ binlog-failure-analysis; tools: skill ✅ 0.04 [6]
dotnet-maui-doctor Plan macOS MAUI setup with Xcode 4.0/5 ⏰ → 5.0/5 🟢 ✅ dotnet-maui-doctor; tools: skill ✅ 0.13
dotnet-maui-doctor Plan Linux MAUI environment for Android 3.0/5 → 5.0/5 🟢 ✅ dotnet-maui-doctor; tools: skill, view, glob ✅ 0.13
dotnet-maui-doctor Guardrail against workload update and repair 1.0/5 → 2.0/5 🟢 ✅ dotnet-maui-doctor; tools: report_intent, skill ✅ 0.13
dotnet-maui-doctor Diagnose non-Microsoft JDK causing build failure 3.0/5 → 5.0/5 🟢 ✅ dotnet-maui-doctor; tools: skill ✅ 0.13
dotnet-maui-doctor Plan complete MAUI setup on Windows 3.0/5 → 5.0/5 🟢 ✅ dotnet-maui-doctor; tools: skill ✅ 0.13
dotnet-maui-doctor Prevent incorrect JAVA_HOME configuration 1.0/5 → 5.0/5 🟢 ✅ dotnet-maui-doctor; tools: report_intent, skill ✅ 0.13
dotnet-maui-doctor Determine required Android SDK packages for specific .NET version 3.0/5 → 5.0/5 🟢 ✅ dotnet-maui-doctor; tools: report_intent, skill, view, bash, glob ✅ 0.13
dotnet-maui-doctor Fix stale MAUI workloads after SDK update 4.0/5 → 4.0/5 ✅ dotnet-maui-doctor; tools: skill, glob, web_fetch ✅ 0.13
thread-abort-migration Worker thread with abort-based cancellation 5.0/5 → 5.0/5 ✅ thread-abort-migration; tools: skill ✅ 0.08
thread-abort-migration Timeout enforcement via Thread.Abort 4.0/5 → 3.0/5 🔴 ✅ thread-abort-migration; tools: skill ✅ 0.08
thread-abort-migration Blocking WaitHandle with Thread.Interrupt 4.0/5 → 4.0/5 ✅ thread-abort-migration; tools: skill, report_intent, bash ✅ 0.08
thread-abort-migration ASP.NET Response.End and Response.Redirect with Thread.Abort 4.0/5 → 5.0/5 🟢 ✅ thread-abort-migration; tools: skill, report_intent ✅ 0.08
thread-abort-migration Thread.Join and Thread.Sleep only — should not migrate 4.0/5 → 5.0/5 🟢 ✅ thread-abort-migration; tools: skill ✅ 0.08
migrate-nullable-references Enable NRT in a small library with mixed nullability 5.0/5 → 5.0/5 ✅ migrate-nullable-references; tools: skill ✅ 0.15 [7]
migrate-nullable-references File-by-file migration: only modify the targeted file 5.0/5 → 5.0/5 ✅ tools: glob ✅ 0.15 [8]
migrate-nullable-references Enable NRT in ASP.NET Core Web API with EF Core 3.0/5 → 3.0/5 ✅ migrate-nullable-references; tools: skill ✅ 0.15
dotnet-aot-compat Make Azure.ResourceManager AOT-compatible 1.0/5 ⏰ → 2.0/5 ⏰ 🟢 ✅ dotnet-aot-compat; tools: skill ✅ 0.14

[1] Quality unchanged but weighted score is -53.0% due to: quality, completion (✓ → ✗), judgment, tokens (11553 → 86982), tool calls (0 → 7), time (7.9s → 31.1s)
[2] Quality unchanged but weighted score is -4.4% due to: quality, tokens (73497 → 105053)
[3] Quality improved but weighted score is -15.7% due to: completion (✓ → ✗), quality
[4] Quality improved but weighted score is -2.2% due to: tokens (37234 → 74491)
[5] Quality improved but weighted score is -4.3% due to: tokens (35145 → 57028), tool calls (3 → 4)
[6] Quality unchanged but weighted score is -13.2% due to: tokens (186073 → 920831), quality, tool calls (11 → 34), time (77.3s → 161.4s)
[7] Quality unchanged but weighted score is -7.3% due to: tokens (123171 → 313942), time (74.8s → 128.6s), tool calls (24 → 29)
[8] Quality unchanged but weighted score is -2.5% due to: tokens (89555 → 116911), tool calls (10 → 14)

timeout — run hit the scenario timeout limit; scoring may be impacted by aborting model execution before it could produce its full output

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

Full results

JanKrivanek pushed a commit that referenced this pull request Mar 11, 2026
* Bump the all-other-nuget group with 1 update

Bumps GitHub.Copilot.SDK from 0.1.30 to 0.1.32

---
updated-dependencies:
- dependency-name: GitHub.Copilot.SDK
  dependency-version: 0.1.32
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: all-other-nuget
- dependency-name: GitHub.Copilot.SDK
  dependency-version: 0.1.32
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: all-other-nuget
...

Signed-off-by: dependabot[bot] <support@github.com>

* React to PermissionRequestResultKind breaking change in GitHub.Copilot.SDK 0.1.32 (#326)

* Initial plan

* Fix PermissionRequestResultKind breaking change in SDK 0.1.32

Co-authored-by: ViktorHofer <7412651+ViktorHofer@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ViktorHofer <7412651+ViktorHofer@users.noreply.github.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ViktorHofer <7412651+ViktorHofer@users.noreply.github.com>
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