Skip to content

Conversation

@lilla28
Copy link
Contributor

@lilla28 lilla28 commented Jan 27, 2026

Changes:

  • Added IModuleChannelSelector to be able to detect changes from both UI - when the user would like to joiin to a user channel, or from the API using the native, TypeScript clients
  • Refactor channel management by replacing ChannelFactory and related interfaces with the new ChannelHandler abstraction across .NET and TypeScript codebases (Renamed internal classes of channel management).
  • All async calls now use .ConfigureAwait(false).
  • Update DesktopAgent, IntentsClient, and related classes to use ChannelHandler, and improve logging, error handling, and resource management.
  • Update tests and public API surface to reflect new abstractions.
  • Add documentation and README updates for channel selector integration.
image

@lilla28 lilla28 requested a review from a team as a code owner January 27, 2026 16:48
@codecov
Copy link

codecov bot commented Jan 27, 2026

Codecov Report

❌ Patch coverage is 83.25792% with 37 lines in your changes missing coverage. Please review.
✅ Project coverage is 65.9%. Comparing base (9cf6822) to head (aad0a19).

Files with missing lines Patch % Lines
...t.Client/Infrastructure/Internal/ChannelHandler.cs 72.9% 9 Missing and 1 partial ⚠️
...oseUI.Fdc3.DesktopAgent/Fdc3DesktopAgentService.cs 88.0% 4 Missing and 2 partials ⚠️
...mposeUI.Fdc3.DesktopAgent/ModuleChannelSelector.cs 81.2% 6 Missing ⚠️
...Infrastructure/Internal/Protocol/PrivateChannel.cs 68.7% 5 Missing ⚠️
...nt/Infrastructure/Internal/UserChannelSetReader.cs 20.0% 4 Missing ⚠️
...pAgent.Client/Infrastructure/DesktopAgentClient.cs 90.9% 2 Missing and 1 partial ⚠️
.../Infrastructure/Internal/ResolverUICommunicator.cs 50.0% 2 Missing ⚠️
...topAgent/Infrastructure/Internal/IntentResolver.cs 50.0% 1 Missing ⚠️

❌ Your patch check has failed because the patch coverage (83.2%) is below the target coverage (100.0%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##            main   #1171     +/-   ##
=======================================
+ Coverage   65.8%   65.9%   +0.1%     
=======================================
  Files        334     335      +1     
  Lines       9755    9819     +64     
  Branches    1228    1289     +61     
=======================================
+ Hits        6421    6480     +59     
- Misses      2957    2964      +7     
+ Partials     377     375      -2     
Files with missing lines Coverage Δ
...nt.Client/Infrastructure/Internal/IntentsClient.cs 92.8% <100.0%> (ø)
...Client/Infrastructure/Internal/Protocol/Channel.cs 96.0% <100.0%> (ø)
...frastructure/Internal/Protocol/IntentResolution.cs 98.0% <100.0%> (ø)
...opAgent.Shared/Contracts/JoinUserChannelRequest.cs 100.0% <ø> (ø)
...pAgent.Shared/Contracts/JoinUserChannelResponse.cs 100.0% <ø> (ø)
...Fdc3.DesktopAgent.Shared/Exceptions/ThrowHelper.cs 62.1% <100.0%> (+1.0%) ⬆️
....Fdc3.DesktopAgent.Shared/Fdc3StartupProperties.cs 100.0% <ø> (ø)
...ey.ComposeUI.Fdc3.DesktopAgent.Shared/Fdc3Topic.cs 93.7% <100.0%> (+0.2%) ⬆️
...DependencyInjection/ServiceCollectionExtensions.cs 100.0% <100.0%> (ø)
...y.ComposeUI.Fdc3.DesktopAgent/Fdc3StartupAction.cs 87.8% <100.0%> (ø)
... and 10 more

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

…management by replacing ChannelFactory and related interfaces with the new ChannelHandler abstraction across .NET and TypeScript codebases (Renamed internal classes of channel management). All async calls now use .ConfigureAwait(false). Update DesktopAgent, IntentsClient, and related classes to use ChannelHandler, and improve logging, error handling, and resource management. Update tests and public API surface to reflect new abstractions. Add documentation and README updates for channel selector integration.
@lilla28 lilla28 force-pushed the feat/add-channel-selector-v3 branch from 8ee273c to aad0a19 Compare January 27, 2026 17:08
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.

1 participant