Repository navigation
Update Copilot SDK to 1.0.28 with separate preview and stable commits - #273
Merged
Merged
Conversation
Audit Connector display-name additions and preserve identifier-based MCP denial. Verify the bundled 1.0.105-1 native runtime and refresh third-party notices. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Use the exact NuGet package's bundled runtime 1.0.105 rather than the release-note snapshot label. Retain the preview Connector identity regression coverage. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🔵 Needs a closer look
The native runtime upgrade lacks win-x64 installed-package startup, model-turn, and shutdown validation; ARM64 probes and app E2E results do not establish compatibility across that package/runtime boundary.
No open issues 🎉
What changed in this PR
Updates RepoSyncRadar’s embedded GitHub Copilot SDK from 1.0.27 to 1.0.28, through separate preview and stable commits, while preserving the existing restrictive MCP approval policy.
Changes:
- Updates the SDK dependency and bundled-runtime assertion to
1.0.105. - Refreshes third-party notices and their matching test.
- Adds regression coverage for Connector display-label serialization, identity preservation, and MCP denial.
| File | Description |
|---|---|
tests/RepoSyncRadar.App.Tests/Copilot/RadarPermissionPolicyTests.cs |
Tests that Connector labels preserve identity and do not change MCP denial. |
tests/RepoSyncRadar.App.Tests/Copilot/CopilotInProcessRuntimeTests.cs |
Expects bundled runtime version 1.0.105. |
tests/RepoSyncRadar.App.Tests/Components/ThirdPartyNoticesPanelTests.cs |
Expects SDK version 1.0.28 in notices. |
src/RepoSyncRadar.App/Settings/ThirdPartyNotices.cs |
Updates the displayed SDK version. |
Directory.Packages.props |
Pins GitHub.Copilot.SDK to 1.0.28. |
Review effort: Balanced
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
GitHub.Copilot.SDKfrom1.0.27to1.0.28in the requested two-commit sequence:1462dd2— update to1.0.28-preview.0, refresh notices/runtime assertions, and add Connector identity/approval regressions.6d11175— promote to stable1.0.28and its exact bundled runtime.1.0.105-1, while stable bundles1.0.105despite its release prose mentioning1.0.105-1; (3) generated RPC APIs and build targets did not change, so existing native alias/publish refresh workarounds remain necessary.ServerName, and deny MCP requests even when read-only, managed-approval-required, or labeled like an approved radar tool/docs host.Evidence matrix
9bc718cb608c4cd3b63e0e71c637dcf1ebfd5409fa108e71ee3025663e0dac20f2a071055d3a6b14; preview-to-stable diff changes only Node version files1.0.27,1.0.28-preview.0,1.0.28.nuspec, bundled props, build targets, net10 XML docs1.0.104→1.0.105-1→1.0.105; dependencies/targets unchanged; preview/stable XML identical. Native status tests confirmed both updated pins.9301f9874504241ceb199febd252b7411a92bd52; both new packages reference845c22879d426535d97901908014b807e0b68d0fSession.cs,Client.cs,Types.cs,Generated/SessionEvents.cs,Generated/Rpc.csElicitationE2ETests,SessionEventSerializationTests,PermissionHandlerTestsAdoption matrix
PermissionRequestMcp.ServerDisplayNamePermissionPromptRequestMcp.ServerDisplayNameElicitationRequestedData.ElicitationSourceDisplayNameElicitationContext.ElicitationSourceDisplayNameSamplingRequestedData.ServerDisplayNameAutomatic benefits are the native runtime refresh and additive event/handler metadata support in the app SDK adapter. No .NET SDK/compiler/test-tooling dependencies changed. Experimental warning boundaries, manual permissions, policy isolation, and typed structured drafts remain unchanged.
Related Issues
N/A - no issue to close
Scope
Validation
dotnet build RepoSyncRadar.sln -warnaserror— executed asdotnet build RepoSyncRadar.sln -warnaserror --verbosity quiet; passed, zero warnings/errors.dotnet test RepoSyncRadar.sln -- --filter-not-trait Category=Manual— executed with the documented run-level timeout:dotnet test RepoSyncRadar.sln --timeout 10m -- --filter-not-trait Category=Manual; failed: 1,585 passed, 2 failed. App 833/833, Integration 55/55, E2E 27/27; Core 670 passed/2 failed.dotnet test tests\RepoSyncRadar.App.Tests\RepoSyncRadar.App.Tests.csproj --no-build -- --filter-class RepoSyncRadar.App.Tests.Copilot.CopilotInProcessRuntimeTests --filter-class RepoSyncRadar.App.Tests.Copilot.RadarPermissionPolicyTests --filter-class RepoSyncRadar.App.Tests.Components.ThirdPartyNoticesPanelTests— preview passed 47/47, including runtime identity/hash and native policy-schema inventory. The same classes passed within the stable full App suite. Stable App suite also passed repeated-publish equal-metadata asset refresh coverage.The two failures are existing, unchanged Core tests:
TcpPreviewPortAllocatorTests.AllocateSingle_Skips_Listening_Preferred_PortandAllocateComparison_Skips_Pair_When_After_Port_Is_Listening. Both reproduced usingdotnet test tests\RepoSyncRadar.Core.Tests\RepoSyncRadar.Core.Tests.csproj --no-build -- --filter-class RepoSyncRadar.Core.Tests.Services.Preview.TcpPreviewPortAllocatorTests(2 passed/2 failed). Core source/tests have no diff against the base; unrelated fixes are intentionally excluded.dotnet restore RepoSyncRadar.sln --verbosity quietandgit diff --checkpassed. An initial pipe-delimited class selector ran zero tests; it was corrected to repeated--filter-classselectors before recording focused success.Risk And Rollback
1.0.27, runtime assertions, and notices together; keep existing alias/publish invalidation targets intact.Release Notes
Release note draft:
Updated the embedded GitHub Copilot SDK to 1.0.28 and its bundled runtime to 1.0.105. Connector display metadata does not change RepoSyncRadar's restrictive tool-approval policy.