docs: fix review issues and add Hot Reload telemetry

- Fixed SHA256 hash examples in Telemetry-GlobalProperties.md (64 hex chars)
- Fixed inconsistent error property casing in Telemetry-Licensing.md (lowercase)
- Clarified App MCP tool descriptions (element peer default action vs specific actions)
- Updated Privacy.md and GlobalProperties.md to accurately reflect stack trace collection
- Added clarification that DevServer and Licensing diagnostic events include raw ErrorMessage/StackTrace
- Added 11 Hot Reload processor events to Telemetry-DevServer.md (notify-start through notify-error)
- Documented Hot Reload properties with hotreload/ prefix notation
- Added Hot Reload property value examples
- Expanded DevServer from 146 to 184 lines with comprehensive Hot Reload telemetry

Addresses review comments #2691759375, #2691759390, #2691759406, #2691759413, #2691759422, #2691759433
Related to unoplatform/private#998

Co-authored-by: Jen-Uno <214558326+Jen-Uno@users.noreply.github.com>

Author: Copilot

doc/internal-documentation/Telemetry-DevServer.md

@@ -73,6 +73,36 @@ App launches are tracked via the `AppLaunchMessage` with the following propertie
 | `app-launch/connected` | `TargetPlatform` (string), `IsDebug` (string), `IDE` (string), `PluginVersion` (string), `WasTimedOut` (string), `WasIdeInitiated` (string) | `LatencyMs` (double) | Global | App connected to DevServer |
 | `app-launch/connection-timeout` | `TargetPlatform` (string), `IsDebug` (string), `IDE` (string), `PluginVersion` (string) | `TimeoutSeconds` (double) | Global | App connection timed out |
 
+## Hot Reload Events
+
+**Event Name Prefix:** `uno/dev-server/hot-reload`
+
+Hot Reload processor events track the state transitions and operations of the Hot Reload system:
+
+| Event Name | Properties | Measurements | Description |
+|------------|-----------|--------------|-------------|
+| `notify-start` | `Event` (string), `Source` (string), `PreviousState` (string) | `FileCount` (double), `DurationMs` (double, optional) | Hot Reload notification started |
+| `notify-disabled` | `Event` (string), `Source` (string), `PreviousState` (string) | `FileCount` (double), `DurationMs` (double, optional) | Hot Reload is disabled |
+| `notify-initializing` | `Event` (string), `Source` (string), `PreviousState` (string) | `FileCount` (double), `DurationMs` (double, optional) | Hot Reload is initializing |
+| `notify-ready` | `Event` (string), `Source` (string), `PreviousState` (string) | `FileCount` (double), `DurationMs` (double, optional) | Hot Reload is ready |
+| `notify-processing-files` | `Event` (string), `Source` (string), `PreviousState` (string) | `FileCount` (double), `DurationMs` (double, optional) | Hot Reload is processing files |
+| `notify-completed` | `Event` (string), `Source` (string), `PreviousState` (string) | `FileCount` (double), `DurationMs` (double, optional) | Hot Reload operation completed |
+| `notify-no-changes` | `Event` (string), `Source` (string), `PreviousState` (string) | `FileCount` (double), `DurationMs` (double, optional) | No changes detected |
+| `notify-failed` | `Event` (string), `Source` (string), `PreviousState` (string) | `FileCount` (double), `DurationMs` (double, optional) | Hot Reload operation failed |
+| `notify-rude-edit` | `Event` (string), `Source` (string), `PreviousState` (string) | `FileCount` (double), `DurationMs` (double, optional) | Rude edit detected (requires restart) |
+| `notify-complete` | `Event` (string), `Source` (string), `PreviousState` (string), `NewState` (string), `HasCurrentOperation` (bool) | `FileCount` (double), `DurationMs` (double, optional) | Hot Reload state transition complete |
+| `notify-error` | `Event` (string), `Source` (string), `PreviousState` (string), `NewState` (string), `HasCurrentOperation` (bool), `ErrorMessage` (string), `ErrorType` (string) | `FileCount` (double), `DurationMs` (double, optional) | Hot Reload error occurred |
+
+**Property Notes:**
+- All properties are prefixed with `hotreload/` in the actual telemetry
+- All measurements are prefixed with `hotreload/` in the actual telemetry
+- **Event**: Type of event that triggered the notification
+- **Source**: Source of the event (IDE, DevServer, etc.)
+- **PreviousState/NewState**: State before/after the operation (Ready, Disabled, Initializing, Processing)
+- **HasCurrentOperation**: Indicates if a Hot Reload operation is in progress
+- **FileCount**: Number of files affected (only present if there is a current operation)
+- **DurationMs**: Duration of the operation in milliseconds (only present if operation has completed)
+
 ### Legacy App Launch Properties
 
 | Property | Type | Description |
@@ -134,6 +164,14 @@ Example values for Dev Server telemetry properties:
 - **Plugin**: "Uno.VSCode.Extension v1.2.3", "Uno.Rider.Plugin v2.0.1", "Uno.VisualStudio.Extension v1.5.0"
 - **Step**: "Launched", "Connected"
 
+### Hot Reload Properties
+- **Event** (Hot Reload): "ProcessingFiles", "Completed", "NoChanges", "Failed", "RudeEdit"
+- **Source** (Hot Reload): "IDE", "DevServer", "FileWatcher"
+- **PreviousState** / **NewState**: "Ready", "Disabled", "Initializing", "Processing"
+- **HasCurrentOperation**: `true`, `false`
+- **ErrorMessage** (Hot Reload): "Compilation failed", "Syntax error in file", "Type not found"
+- **ErrorType** (Hot Reload): "CompilationException", "SyntaxException", "TypeLoadException"
+
 ## Privacy Notes
 
 - **ErrorMessage** and **StackTrace** properties are sent as raw values and may contain sensitive information; handle with care

doc/internal-documentation/Telemetry-GlobalProperties.md

@@ -100,7 +100,7 @@ All global properties follow these privacy principles:
 - **MAC Address**: Hashed with SHA256 to create stable, anonymous machine ID
 - **Working Directory**: Hashed with SHA256 to avoid exposing file paths
 - **User Tokens**: Hashed with SHA256 when used as user identifiers
-- **Stack Traces**: Only exception types are logged, not full stack traces
+- **Stack Traces**: For most telemetry, only exception types are logged, not full stack traces. However, certain diagnostic events (DevServer startup-failure, Licensing manager-failed) may include raw stack traces for troubleshooting purposes.
 
 ### What is NOT Collected
 
@@ -190,8 +190,8 @@ Example values for common global properties:
 
 ### Application Information Examples
 - **SessionId**: "3fa85f64-5717-4562-b3fc-2c963f66afa6" (GUID format)
-- **UserId**: "a7b8c9d0e1f2g3h4i5j6k7l8m9n0o1p2" (SHA256 hash, 64 hex characters)
-- **MachineId**: "1a2b3c4d5e6f7g8h9i0j1k2l3m4n5o6p" (SHA256 hash, 64 hex characters)
+- **UserId**: "a7b8c9d0e1f2a3b4c5d6e7f8a9b0c1d2e3f4a5b6c7d8e9f0a1b2c3d4e5f6a7b8" (SHA256 hash, 64 hex characters)
+- **MachineId**: "1a2b3c4d5e6f7a8b9c0d1e2f3a4b5c6d7e8f9a0b1c2d3e4f5a6b7c8d9e0f1a2" (SHA256 hash, 64 hex characters)
 
 ### Build Information Examples
 - **ProjectType**: "app", "library", "test"

doc/internal-documentation/Telemetry-Licensing.md

@@ -12,12 +12,12 @@ Client-side license manager events:
 |------------|-----------|--------------|-------------|
 | `logout-requested` | - | - | User requested logout |
 | `logout-success` | - | - | Logout completed successfully |
-| `logout-failure` | `Error` (string) | - | Logout failed |
+| `logout-failure` | `error` (string) | - | Logout failed |
 | `login-requested` | - | - | User requested login |
 | `login-success` | - | - | Login completed successfully |
 | `login-failure` | `error` (string), `errorDescription` (string) | - | Login failed |
 | `login-canceled` | - | - | User canceled login |
-| `login-timeout` | `Error` (string), `ErrorDescription` (string) | - | Login timed out |
+| `login-timeout` | `error` (string), `errorDescription` (string) | - | Login timed out |
 | `license-status` | `LicenseName` (string), `LicenseStatus` (string), `TrialDaysRemaining` (int), `LicenseExpiryDate` (date) | - | License status checked |
 | `manager-started` | `Feature` (string) | - | License manager started for a feature |
 | `manager-failed` | `ErrorType` (string), `ErrorStackTrace` (string) | - | License manager failed |

doc/internal-documentation/Telemetry-Privacy.md

@@ -19,7 +19,7 @@ Uno Platform telemetry is designed to be GDPR compliant:
 - Anonymous user IDs (hashed tokens, hashed MAC addresses)
 - Performance measurements (durations, counts)
 - Feature usage patterns (which features are used, how often)
-- Error types (exception types only, no stack traces)
+- Error types (exception types only; stack traces only in specific diagnostic events)
 - Operating system and architecture information
 - Target frameworks and platforms
 - Hashed directory paths (SHA256)
@@ -28,7 +28,7 @@ Uno Platform telemetry is designed to be GDPR compliant:
 
 - **Personal Information**: No usernames, email addresses, or account details
 - **User Content**: No file paths, source code, or project-specific information
-- **Detailed Errors**: No stack traces or detailed error messages that could contain sensitive data
+- **Detailed Errors**: Generally no stack traces or detailed error messages. Exception: DevServer startup failures and Licensing manager failures may include raw ErrorMessage and StackTrace for diagnostic purposes
 - **Raw Tokens**: Bearer tokens are never logged; only SHA256 hashes for opaque tokens
 
 ## Disabling Telemetry

doc/internal-documentation/Telemetry.md

@@ -107,8 +107,8 @@ IDE extensions track extension lifecycle, user interactions, and dev server oper
 **Event Name Prefix:** `uno/app-mcp`
 
 App MCP (Model Context Protocol) telemetry tracks agent interactions with running Uno applications:
-- **Community License Tools** - 8 tools (runtime info, screenshot, pointer click, key press, type text, visual tree snapshot, element peer action, close)
-- **Pro License Tools** - 2 tools (element peer action, element DataContext)
+- **Community License Tools** - 8 tools (runtime info, screenshot, pointer click, key press, type text, visual tree snapshot, element peer default action, close)
+- **Pro License Tools** - 2 tools (element peer action for specific peer actions, element DataContext)
 - **Session Tracking** - Agent type, platform, tools used
 - **Error Tracking** - Connection errors, tool execution errors, timeouts
 - **Agent Interaction Metrics** - Tool invocation counts, success rates, session durations