Skip to content

Restore removed brokered service descriptors for hot reload#83184

Merged
dibarbet merged 1 commit intodotnet:mainfrom
dibarbet:servicebroker_notificationservice
Apr 15, 2026
Merged

Restore removed brokered service descriptors for hot reload#83184
dibarbet merged 1 commit intodotnet:mainfrom
dibarbet:servicebroker_notificationservice

Conversation

@dibarbet
Copy link
Copy Markdown
Member

@dibarbet dibarbet commented Apr 15, 2026

Re-add HotReloadSessionNotificationService, ManagedHotReloadAgentManagerService, and GenericHotReloadAgentManagerService descriptors and registrations that were removed in PR #83018. These services are still used by the XAML in the C# extension, and without them the logs are spammed with errors like

2026-04-15 13:19:47.388 [debug] [serviceBroker/connect] [BrokeredServiceTraceListener] ServiceBroker Start: 0 : 
2026-04-15 13:19:47.388 [debug] [serviceBroker/connect] [BrokeredServiceTraceListener] Requesting proxy to "Microsoft.VisualStudio.Debugger.HotReloadSessionNotificationService (0.1)"
2026-04-15 13:19:47.388 [debug] [serviceBroker/connect] [BrokeredServiceTraceListener] ServiceBroker Warning: 2 : 
2026-04-15 13:19:47.389 [debug] [serviceBroker/connect] [BrokeredServiceTraceListener] Request for proxy to "Microsoft.VisualStudio.Debugger.HotReloadSessionNotificationService (0.1)" is declined: NotLocallyRegistered.
2026-04-15 13:19:47.389 [debug] [serviceBroker/connect] [BrokeredServiceTraceListener] ServiceBroker Stop: 0 : 

See https://devdiv.visualstudio.com/DevDiv/_git/VS?path=%2Fsrc%2FXaml%2FDiagnostics%2FSource%2FCodeAnalysisDiagnostics%2FLanguageService%2FServiceBrokerWrapper.cs&_a=contents&version=GBmain

Microsoft Reviewers: Open in CodeFlow

Re-add HotReloadSessionNotificationService, ManagedHotReloadAgentManagerService,
and GenericHotReloadAgentManagerService descriptors and registrations that were
removed in PR dotnet#83018. These services are still used by the XAML diagnostics
ServiceBrokerWrapper in VS, causing NotLocallyRegistered errors in DevKit logs.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dibarbet dibarbet requested a review from a team as a code owner April 15, 2026 20:20
@dibarbet dibarbet requested a review from tmat April 15, 2026 20:20
@dibarbet dibarbet merged commit e8c0d14 into dotnet:main Apr 15, 2026
27 checks passed
@dibarbet dibarbet deleted the servicebroker_notificationservice branch April 15, 2026 21:54
@dotnet-policy-service dotnet-policy-service Bot added this to the Next milestone Apr 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants