-
Notifications
You must be signed in to change notification settings - Fork 820
App Launch monitoring #21560
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
App Launch monitoring #21560
Conversation
55889ec to
f5408ee
Compare
3d7a637 to
510adae
Compare
src/Uno.UI.RemoteControl.DevServer.Tests/AppLaunch/AppLaunchIntegrationTests.cs
Outdated
Show resolved
Hide resolved
src/Uno.UI.RemoteControl.DevServer.Tests/AppLaunch/AppLaunchIntegrationTests.cs
Outdated
Show resolved
Hide resolved
src/Uno.UI.RemoteControl.DevServer.Tests/AppLaunch/AppLaunchIntegrationTests.cs
Outdated
Show resolved
Hide resolved
src/Uno.UI.RemoteControl.DevServer.Tests/AppLaunch/AppLaunchIntegrationTests.cs
Outdated
Show resolved
Hide resolved
src/Uno.UI.RemoteControl.DevServer.Tests/AppLaunch/AppLaunchIntegrationTests.cs
Outdated
Show resolved
Hide resolved
src/Uno.UI.RemoteControl.DevServer.Tests/AppLaunch/Given_ApplicationLaunchMonitor.cs
Outdated
Show resolved
Hide resolved
cd914e1 to
3c1b55e
Compare
4bc6faa to
0bba7d7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR implements comprehensive app launch monitoring functionality for the Uno Platform development server. The primary purpose is to correlate "I launched an app" events from IDEs with "that app connected back" events from the runtime, enabling telemetry tracking of application launch success rates and connection latencies.
Key changes:
- Adds
ApplicationLaunchMonitorservice for server-side launch/connection correlation - Implements
VsAppLaunchStateServicefor Visual Studio IDE event coordination - Introduces new messaging protocols for app launch registration and connection reporting
Reviewed Changes
Copilot reviewed 45 out of 45 changed files in this pull request and generated 9 comments.
Show a summary per file
| File | Description |
|---|---|
src/Uno.UI.RemoteControl/RemoteControlClient.cs |
Enhanced remote control client with app identity reporting and improved disposal handling |
src/Uno.UI.RemoteControl/Messages/AppLaunchMessage.cs |
New message type for runtime-to-server app launch notifications |
src/Uno.UI.RemoteControl/Helpers/ApplicationInfoHelper.cs |
Utility for extracting assembly MVID and target platform information |
src/Uno.UI.RemoteControl.VS/EntryPoint.cs |
Integration of app launch tracking components into VS extension |
src/Uno.UI.RemoteControl.VS/AppLaunch/VsAppLaunchStateService.cs |
State machine for correlating VS Play events with build outcomes |
src/Uno.UI.RemoteControl.VS/AppLaunch/VsAppLaunchIdeBridge.cs |
Bridge between VS IDE events and state service |
src/Uno.UI.RemoteControl.VS/AppLaunch/VsAppLaunchStateConsumer.cs |
Consumer that sends app launch data to dev server |
src/Uno.UI.RemoteControl.Server/AppLaunch/ApplicationLaunchMonitor.cs |
Core server-side service for launch/connection correlation with timeout handling |
src/Uno.UI.RemoteControl.Host/RemoteControlServer.cs |
Server integration for processing app launch messages |
src/Uno.UI.RemoteControl.Host/RemoteControlExtensions.cs |
HTTP endpoints for app launch registration |
|
The build 178576 found UI Test snapshots differences: Details
|
|
|
|
🤖 Your WebAssembly Skia Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-21560/wasm-skia-net9/index.html |
|
🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-21560/docs/index.html |
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 48 out of 48 changed files in this pull request and generated 8 comments.
- Reorganize solution filters.
…assembly info read
…odic timers for keep-alive handling - Improves stability by handling disconnections and cancellations correctly.
…keep-alive send exceptions
b16e211 to
602a481
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 52 out of 52 changed files in this pull request and generated 4 comments.
|
🤖 Your WebAssembly Skia Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-21560/wasm-skia-net9/index.html |
|
🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-21560/docs/index.html |
… simplify command line handling - Refactored `Program.cs` to streamline argument processing using `IConfiguration`. - Removed requirement of the ChannelID to be a Guid
|
🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-21560/docs/index.html |
|
🤖 Your WebAssembly Skia Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-21560/wasm-skia-net9/index.html |
|
The build 180222 found UI Test snapshots differences: Details
|
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 54 out of 54 changed files in this pull request and generated 2 comments.
|
🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-21560/docs/index.html |
|
🤖 Your WebAssembly Skia Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-21560/wasm-skia-net9/index.html |
Closes https://github.com/unoplatform/uno-private/issues/1491
Closes https://github.com/unoplatform/uno-private/issues/1492
Closes https://github.com/unoplatform/uno-private/issues/1493
Implements and closes https://github.com/unoplatform/uno-private/issues/1361#issuecomment-2923045864
Feature - Application Launch Monitoring
This PR implements comprehensive app launch monitoring functionality for the Uno Platform development server. The primary purpose is to correlate "I launched an app" events from IDEs with "that app connected back" events from the runtime, enabling telemetry tracking of application launch success rates and connection latencies.
PR Checklist ✅
Please check if your PR fulfills the following requirements:
Screenshots Compare Test Runresults.