Release v2.5.0: version bumps, changelog, bug fixes found during release testing (#766)

erikdarlingdata · claude · web-flow · commit 9a63521624da · 2026-03-30T21:10:56.000-04:00
- Bump all project versions to 2.5.0
- Add v2.5.0 changelog entry and README sync (Overview tab, query heatmap, webhooks, chart drill-down)
- Fix PERCENTILE_CONT missing OVER() in FinOps VM right-sizing query
- Fix database_id column reference on AWS RDS sys.dm_db_persisted_sku_features
- Fix FinOps right-click copy/export broken on all Dashboard grids (PlacementTarget was row, not grid)
- Add server name to FinOps recommendation error logs
- Add null guard for _dbSizesFilterMgr race condition

Co-authored-by: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,92 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [2.5.0] - 2026-03-30
+
+### Important
+
+- **InstallerGui retired**: The standalone GUI installer has been removed. Installation, upgrade, and uninstall are now handled directly from the Dashboard's Add Server dialog, powered by the new Installer.Core shared library. The CLI installer continues to work as before. ([#755])
+
+### Added
+
+- **Dashboard integrated installer** — Add Server dialog now installs, upgrades, and uninstalls PerformanceMonitor directly, replacing the standalone InstallerGui ([#755])
+- **Installer.Core shared library** — shared installation logic used by both the CLI installer and Dashboard ([#755])
+- **Overview tab** for Lite with 2x2 resource chart grid (CPU, Memory, Wait Stats, TempDB) ([#689])
+- **Chart drill-down** on CPU, Memory, TempDB, Blocking, and Deadlock charts in both Dashboard and Lite — right-click any chart point to jump to Active Queries for that time window ([#682])
+- **Grid-to-slicer overlay** for Query Stats, Procedure Stats, and Query Store tabs — click a row to overlay its trend on the slicer chart ([#683])
+- **Query heatmap** tab in both Dashboard and Lite — visual heat map of query activity over time ([#739], [#743])
+- **Webhook notifications** for alerts — configurable webhook endpoint for alert delivery ([#725])
+- **Per-server collector schedule intervals** — customize collection frequency per server ([#703])
+- **Investigate button** in Critical Issues grid — jump directly to relevant tab from an alert ([#684])
+- **Dismiss Selected** context menu and View Log sidebar button for alert management ([#718], [#740])
+- **Alert archival awareness** — dismissed_archive_alerts sidecar table, source column for live vs archived alerts, stale-data indicator, structured telemetry ([#718])
+- **Dashboard read-only connection intent** — connections use `ApplicationIntent=ReadOnly` where supported ([#728])
+- FUNDING.yml for GitHub Sponsors ([#752])
+
+### Changed
+
+- **Installer architecture** refactored: CLI installer is now a thin wrapper over Installer.Core ([#755])
+- **DuckDB memory capped** at 2 GB during parquet compaction to prevent out-of-memory on large archives ([#758])
+- **Text rendering** improved with `TextOptions.TextFormattingMode="Display"` for sharper text ([#710])
+- **installation_history version columns** widened from nvarchar(255) to nvarchar(512) to handle long @@VERSION strings ([#712])
+
+### Fixed
+
+- **Memory leaks in Lite** — delta cache, event handlers, and chart helpers properly disposed ([#758])
+- **Doomed transaction errors** in delta framework and ensure_collection_table — ROLLBACK now occurs before error logging ([#756])
+- **XACT_STATE check** added after third-party stored procedure calls (sp_HumanEventsBlockViewer, sp_BlitzLock) to prevent doomed transaction errors ([#695])
+- **CREATE DATABASE failure** when model database has large default file sizes ([#676])
+- **CPU metrics mixed** for different Azure SQL databases on the same logical server ([#680])
+- **Azure SQL DB vCore** FinOps calculations incorrect for serverless/vCore tiers ([#736])
+- **Webhook alert recording** not persisting correctly ([#726])
+- **Drill-down timezone** misalignment between chart and detail view ([#747], [#750])
+- **Drill-down refresh** losing context on auto-refresh ([#744])
+- **Drill-down target** incorrectly routing Memory to Memory Grants instead of Active Queries ([#706])
+- **Heatmap colorbar stacking** when switching between servers ([#746])
+- **Display mode pickers** not reflecting current state on tab switch ([#751])
+- **Slicer custom range** handling and sub-hour display issues ([#704])
+- **Overlay selection** lost on Dashboard auto-refresh ([#683])
+- **Numeric values** in alert details treated as strings instead of numbers ([#732])
+- **FinOps VM right-sizing** query error — `PERCENTILE_CONT` missing required `OVER()` clause
+- **FinOps Enterprise features** query error on AWS RDS — `database_id` column not present in `sys.dm_db_persisted_sku_features` on RDS
+- **FinOps right-click copy** broken on all Dashboard FinOps grids — context menu walked to row instead of grid
+- **FinOps recommendation error logs** now include server name for easier troubleshooting
+
+### Deprecated
+
+- **InstallerGui** — removed from the solution and build pipeline. Use the Dashboard or CLI installer instead. ([#755])
+
+[#676]: https://github.com/erikdarlingdata/PerformanceMonitor/issues/676
+[#680]: https://github.com/erikdarlingdata/PerformanceMonitor/issues/680
+[#682]: https://github.com/erikdarlingdata/PerformanceMonitor/issues/682
+[#683]: https://github.com/erikdarlingdata/PerformanceMonitor/issues/683
+[#684]: https://github.com/erikdarlingdata/PerformanceMonitor/issues/684
+[#689]: https://github.com/erikdarlingdata/PerformanceMonitor/issues/689
+[#695]: https://github.com/erikdarlingdata/PerformanceMonitor/issues/695
+[#703]: https://github.com/erikdarlingdata/PerformanceMonitor/issues/703
+[#704]: https://github.com/erikdarlingdata/PerformanceMonitor/issues/704
+[#706]: https://github.com/erikdarlingdata/PerformanceMonitor/issues/706
+[#710]: https://github.com/erikdarlingdata/PerformanceMonitor/issues/710
+[#712]: https://github.com/erikdarlingdata/PerformanceMonitor/issues/712
+[#718]: https://github.com/erikdarlingdata/PerformanceMonitor/issues/718
+[#725]: https://github.com/erikdarlingdata/PerformanceMonitor/pull/725
+[#726]: https://github.com/erikdarlingdata/PerformanceMonitor/pull/726
+[#728]: https://github.com/erikdarlingdata/PerformanceMonitor/pull/728
+[#732]: https://github.com/erikdarlingdata/PerformanceMonitor/pull/732
+[#736]: https://github.com/erikdarlingdata/PerformanceMonitor/pull/736
+[#739]: https://github.com/erikdarlingdata/PerformanceMonitor/pull/739
+[#740]: https://github.com/erikdarlingdata/PerformanceMonitor/issues/740
+[#743]: https://github.com/erikdarlingdata/PerformanceMonitor/pull/743
+[#744]: https://github.com/erikdarlingdata/PerformanceMonitor/pull/744
+[#746]: https://github.com/erikdarlingdata/PerformanceMonitor/pull/746
+[#747]: https://github.com/erikdarlingdata/PerformanceMonitor/pull/747
+[#750]: https://github.com/erikdarlingdata/PerformanceMonitor/pull/750
+[#751]: https://github.com/erikdarlingdata/PerformanceMonitor/pull/751
+[#752]: https://github.com/erikdarlingdata/PerformanceMonitor/pull/752
+[#755]: https://github.com/erikdarlingdata/PerformanceMonitor/issues/755
+[#756]: https://github.com/erikdarlingdata/PerformanceMonitor/issues/756
+[#758]: https://github.com/erikdarlingdata/PerformanceMonitor/issues/758
+
 ## [2.4.0] - 2026-03-23
 
 ### Important
diff --git a/Dashboard/Controls/FinOpsContent.xaml.cs b/Dashboard/Controls/FinOpsContent.xaml.cs
@@ -630,7 +630,8 @@ private async Task LoadDatabaseSizesAsync()
                     }
                 }
 
-                _dbSizesFilterMgr!.UpdateData(data);
+                if (_dbSizesFilterMgr == null) return;
+                _dbSizesFilterMgr.UpdateData(data);
                 UpdateDbSizeCountUI();
             }
             catch (Exception ex)
@@ -901,11 +902,22 @@ private async void ServerInventoryRefresh_Click(object sender, RoutedEventArgs e
         // Copy / Export Context Menu Handlers
         // ============================================
 
+        private static DataGrid? FindParentDataGrid(DependencyObject? element)
+        {
+            while (element != null)
+            {
+                if (element is DataGrid dg) return dg;
+                element = VisualTreeHelper.GetParent(element);
+            }
+            return null;
+        }
+
         private void CopyCell_Click(object sender, RoutedEventArgs e)
         {
             if (sender is MenuItem menuItem && menuItem.Parent is ContextMenu contextMenu)
             {
-                if (contextMenu.PlacementTarget is DataGrid grid && grid.CurrentCell.Column != null)
+                var grid = FindParentDataGrid(contextMenu.PlacementTarget);
+                if (grid != null && grid.CurrentCell.Column != null)
                 {
                     var cellContent = TabHelpers.GetCellContent(grid, grid.CurrentCell);
                     if (!string.IsNullOrEmpty(cellContent))
@@ -921,7 +933,8 @@ private void CopyRow_Click(object sender, RoutedEventArgs e)
         {
             if (sender is MenuItem menuItem && menuItem.Parent is ContextMenu contextMenu)
             {
-                if (contextMenu.PlacementTarget is DataGrid grid && grid.SelectedItem != null)
+                var grid = FindParentDataGrid(contextMenu.PlacementTarget);
+                if (grid != null && grid.SelectedItem != null)
                 {
                     var rowText = TabHelpers.GetRowAsText(grid, grid.SelectedItem);
                     if (!string.IsNullOrEmpty(rowText))
@@ -937,7 +950,8 @@ private void CopyAllRows_Click(object sender, RoutedEventArgs e)
         {
             if (sender is MenuItem menuItem && menuItem.Parent is ContextMenu contextMenu)
             {
-                if (contextMenu.PlacementTarget is DataGrid grid)
+                var grid = FindParentDataGrid(contextMenu.PlacementTarget);
+                if (grid != null)
                 {
                     var sb = new StringBuilder();
 
@@ -975,7 +989,8 @@ private void ExportToCsv_Click(object sender, RoutedEventArgs e)
         {
             if (sender is MenuItem menuItem && menuItem.Parent is ContextMenu contextMenu)
             {
-                if (contextMenu.PlacementTarget is DataGrid grid)
+                var grid = FindParentDataGrid(contextMenu.PlacementTarget);
+                if (grid != null)
                 {
                     var dialog = new SaveFileDialog
                     {
diff --git a/Dashboard/Dashboard.csproj b/Dashboard/Dashboard.csproj
@@ -7,10 +7,10 @@
     <StartupObject>PerformanceMonitorDashboard.Program</StartupObject>
     <AssemblyName>PerformanceMonitorDashboard</AssemblyName>
     <Product>SQL Server Performance Monitor Dashboard</Product>
-    <Version>2.4.1</Version>
-    <AssemblyVersion>2.4.1.0</AssemblyVersion>
-    <FileVersion>2.4.1.0</FileVersion>
-    <InformationalVersion>2.4.1</InformationalVersion>
+    <Version>2.5.0</Version>
+    <AssemblyVersion>2.5.0.0</AssemblyVersion>
+    <FileVersion>2.5.0.0</FileVersion>
+    <InformationalVersion>2.5.0</InformationalVersion>
     <Company>Darling Data, LLC</Company>
     <Copyright>Copyright © 2026 Darling Data, LLC</Copyright>
     <ApplicationIcon>EDD.ico</ApplicationIcon>
diff --git a/Dashboard/Services/DatabaseService.FinOps.cs b/Dashboard/Services/DatabaseService.FinOps.cs
@@ -1841,11 +1841,18 @@ public async Task<List<FinOpsRecommendation>> GetFinOpsRecommendationsAsync(deci
 
                 if (edition.Contains("Enterprise", StringComparison.OrdinalIgnoreCase))
                 {
-                    using var featCmd = new SqlCommand(@"
-SELECT
-    DB_NAME(database_id) AS database_name,
-    feature_name
-FROM sys.dm_db_persisted_sku_features", connection);
+                    var hasDatabaseId = false;
+                    using (var colCheck = new SqlCommand(
+                        "SELECT COL_LENGTH('sys.dm_db_persisted_sku_features', 'database_id')", connection))
+                    {
+                        colCheck.CommandTimeout = 10;
+                        hasDatabaseId = await colCheck.ExecuteScalarAsync() is not null and not DBNull;
+                    }
+
+                    var featSql = hasDatabaseId
+                        ? "SELECT DB_NAME(database_id) AS database_name, feature_name FROM sys.dm_db_persisted_sku_features"
+                        : "SELECT N'(unknown)' AS database_name, feature_name FROM sys.dm_db_persisted_sku_features";
+                    using var featCmd = new SqlCommand(featSql, connection);
                     featCmd.CommandTimeout = 30;
 
                     var features = new List<string>();
@@ -1909,7 +1916,7 @@ public async Task<List<FinOpsRecommendation>> GetFinOpsRecommendationsAsync(deci
             }
             catch (Exception ex)
             {
-                Logger.Error($"Recommendation check failed (Enterprise features): {ex.Message}", ex);
+                Logger.Error($"[{ServerLabel}] Recommendation check failed (Enterprise features): {ex.Message}", ex);
             }
 
             // 2. CPU right-sizing score
@@ -1953,7 +1960,7 @@ FROM report.finops_utilization_efficiency AS v
             }
             catch (Exception ex)
             {
-                Logger.Error($"Recommendation check failed (CPU right-sizing): {ex.Message}", ex);
+                Logger.Error($"[{ServerLabel}] Recommendation check failed (CPU right-sizing): {ex.Message}", ex);
             }
 
             // 3. Memory right-sizing score
@@ -1997,7 +2004,7 @@ ORDER BY ms.collection_time DESC
             }
             catch (Exception ex)
             {
-                Logger.Error($"Recommendation check failed (Memory right-sizing): {ex.Message}", ex);
+                Logger.Error($"[{ServerLabel}] Recommendation check failed (Memory right-sizing): {ex.Message}", ex);
             }
 
             // 4. Unused index cost quantification
@@ -2019,7 +2026,7 @@ ORDER BY ms.collection_time DESC
             }
             catch (Exception ex)
             {
-                Logger.Error($"Recommendation check failed (Index analysis): {ex.Message}", ex);
+                Logger.Error($"[{ServerLabel}] Recommendation check failed (Index analysis): {ex.Message}", ex);
             }
 
             // 5. Compression savings estimator
@@ -2086,7 +2093,7 @@ size_mb DESC
             }
             catch (Exception ex)
             {
-                Logger.Error($"Recommendation check failed (Compression): {ex.Message}", ex);
+                Logger.Error($"[{ServerLabel}] Recommendation check failed (Compression): {ex.Message}", ex);
             }
 
             // 6. Dormant database detection with cost impact
@@ -2122,7 +2129,7 @@ size_mb DESC
             }
             catch (Exception ex)
             {
-                Logger.Error($"Recommendation check failed (Dormant databases): {ex.Message}", ex);
+                Logger.Error($"[{ServerLabel}] Recommendation check failed (Dormant databases): {ex.Message}", ex);
             }
 
             // 7. Dev/test workload detection
@@ -2159,7 +2166,7 @@ FROM sys.databases
             }
             catch (Exception ex)
             {
-                Logger.Error($"Recommendation check failed (Dev/test detection): {ex.Message}", ex);
+                Logger.Error($"[{ServerLabel}] Recommendation check failed (Dev/test detection): {ex.Message}", ex);
             }
 
             // 11. Maintenance window efficiency — jobs running long
@@ -2206,7 +2213,7 @@ HAVING SUM(CAST(is_running_long AS int)) >= 3
             }
             catch (Exception ex)
             {
-                Logger.Error($"Recommendation check failed (Maintenance window): {ex.Message}", ex);
+                Logger.Error($"[{ServerLabel}] Recommendation check failed (Maintenance window): {ex.Message}", ex);
             }
 
             // 12. VM right-sizing — prescriptive core/memory targets
@@ -2215,7 +2222,7 @@ HAVING SUM(CAST(is_running_long AS int)) >= 3
                 using var vmCmd = new SqlCommand(@"
 SET TRANSACTION ISOLATION LEVEL READ UNCOMMITTED;
 SELECT
-    p95_cpu = (SELECT PERCENTILE_CONT(0.95) WITHIN GROUP (ORDER BY cus.sqlserver_cpu_utilization)
+    p95_cpu = (SELECT TOP (1) PERCENTILE_CONT(0.95) WITHIN GROUP (ORDER BY cus.sqlserver_cpu_utilization) OVER ()
                FROM collect.cpu_utilization_stats AS cus
                WHERE cus.collection_time >= DATEADD(DAY, -7, SYSDATETIME())),
     cpu_count = (SELECT si.cpu_count FROM sys.dm_os_sys_info AS si),
@@ -2291,7 +2298,7 @@ FROM sys.dm_os_performance_counters AS pc
             }
             catch (Exception ex)
             {
-                Logger.Error($"Recommendation check failed (VM right-sizing): {ex.Message}", ex);
+                Logger.Error($"[{ServerLabel}] Recommendation check failed (VM right-sizing): {ex.Message}", ex);
             }
 
             // 13. Storage tier optimization — flag databases with low IO latency
@@ -2353,7 +2360,7 @@ ORDER BY
             }
             catch (Exception ex)
             {
-                Logger.Error($"Recommendation check failed (Storage tier): {ex.Message}", ex);
+                Logger.Error($"[{ServerLabel}] Recommendation check failed (Storage tier): {ex.Message}", ex);
             }
 
             // 14. Reserved capacity candidates — stable CPU utilization
@@ -2398,7 +2405,7 @@ HAVING COUNT(*) >= 24
             }
             catch (Exception ex)
             {
-                Logger.Error($"Recommendation check failed (Reserved capacity): {ex.Message}", ex);
+                Logger.Error($"[{ServerLabel}] Recommendation check failed (Reserved capacity): {ex.Message}", ex);
             }
 
             return recommendations;
diff --git a/Dashboard/Services/DatabaseService.cs b/Dashboard/Services/DatabaseService.cs
@@ -37,6 +37,12 @@ public DatabaseService(string connectionString)
         /// </summary>
         public string ConnectionString => _connectionString;
 
+        /// <summary>
+        /// Server name extracted from the connection string, for use in log messages.
+        /// </summary>
+        private string ServerLabel =>
+            new SqlConnectionStringBuilder(_connectionString).DataSource ?? "(unknown)";
+
         /// <summary>
         /// Opens a throttled database connection. The semaphore is released when the connection is disposed.
         /// </summary>
diff --git a/Installer.Core/Installer.Core.csproj b/Installer.Core/Installer.Core.csproj
@@ -7,10 +7,10 @@
     <RootNamespace>Installer.Core</RootNamespace>
     <AssemblyName>Installer.Core</AssemblyName>
     <Product>SQL Server Performance Monitor Installer Core</Product>
-    <Version>2.4.1</Version>
-    <AssemblyVersion>2.4.1.0</AssemblyVersion>
-    <FileVersion>2.4.1.0</FileVersion>
-    <InformationalVersion>2.4.1</InformationalVersion>
+    <Version>2.5.0</Version>
+    <AssemblyVersion>2.5.0.0</AssemblyVersion>
+    <FileVersion>2.5.0.0</FileVersion>
+    <InformationalVersion>2.5.0</InformationalVersion>
     <Company>Darling Data, LLC</Company>
     <Copyright>Copyright (c) 2026 Darling Data, LLC</Copyright>
     <EnableNETAnalyzers>true</EnableNETAnalyzers>
diff --git a/Installer/PerformanceMonitorInstaller.csproj b/Installer/PerformanceMonitorInstaller.csproj
@@ -20,10 +20,10 @@
     <!-- Application metadata -->
     <AssemblyName>PerformanceMonitorInstaller</AssemblyName>
     <Product>SQL Server Performance Monitor Installer</Product>
-    <Version>2.4.1</Version>
-    <AssemblyVersion>2.4.1.0</AssemblyVersion>
-    <FileVersion>2.4.1.0</FileVersion>
-    <InformationalVersion>2.4.1</InformationalVersion>
+    <Version>2.5.0</Version>
+    <AssemblyVersion>2.5.0.0</AssemblyVersion>
+    <FileVersion>2.5.0.0</FileVersion>
+    <InformationalVersion>2.5.0</InformationalVersion>
     <Company>Darling Data, LLC</Company>
     <Copyright>Copyright © 2026 Darling Data, LLC</Copyright>
     <Description>Installation utility for SQL Server Performance Monitor - Supports SQL Server 2016-2025</Description>
diff --git a/Lite/PerformanceMonitorLite.csproj b/Lite/PerformanceMonitorLite.csproj
@@ -8,10 +8,10 @@
     <AssemblyName>PerformanceMonitorLite</AssemblyName>
     <RootNamespace>PerformanceMonitorLite</RootNamespace>
     <Product>SQL Server Performance Monitor Lite</Product>
-    <Version>2.4.1</Version>
-    <AssemblyVersion>2.4.1.0</AssemblyVersion>
-    <FileVersion>2.4.1.0</FileVersion>
-    <InformationalVersion>2.4.1</InformationalVersion>
+    <Version>2.5.0</Version>
+    <AssemblyVersion>2.5.0.0</AssemblyVersion>
+    <FileVersion>2.5.0.0</FileVersion>
+    <InformationalVersion>2.5.0</InformationalVersion>
     <Company>Darling Data, LLC</Company>
     <Copyright>Copyright © 2026 Darling Data, LLC</Copyright>
     <Description>Lightweight SQL Server performance monitoring - no installation required on target servers</Description>
diff --git a/README.md b/README.md

Original file line number	Diff line number	Diff line change
`@@ -630,7 +630,8 @@ private async Task LoadDatabaseSizesAsync()`
`630`	`630`	`}`
`631`	`631`	`}`
`632`	`632`
`633`		`- _dbSizesFilterMgr!.UpdateData(data);`
	`633`	`+ if (_dbSizesFilterMgr == null) return;`
	`634`	`+ _dbSizesFilterMgr.UpdateData(data);`
`634`	`635`	`UpdateDbSizeCountUI();`
`635`	`636`	`}`
`636`	`637`	`catch (Exception ex)`
`@@ -901,11 +902,22 @@ private async void ServerInventoryRefresh_Click(object sender, RoutedEventArgs e`
`901`	`902`	`// Copy / Export Context Menu Handlers`
`902`	`903`	`// ============================================`
`903`	`904`
	`905`	`+ private static DataGrid? FindParentDataGrid(DependencyObject? element)`
	`906`	`+ {`
	`907`	`+ while (element != null)`
	`908`	`+ {`
	`909`	`+ if (element is DataGrid dg) return dg;`
	`910`	`+ element = VisualTreeHelper.GetParent(element);`
	`911`	`+ }`
	`912`	`+ return null;`
	`913`	`+ }`
	`914`	`+`
`904`	`915`	`private void CopyCell_Click(object sender, RoutedEventArgs e)`
`905`	`916`	`{`
`906`	`917`	`if (sender is MenuItem menuItem && menuItem.Parent is ContextMenu contextMenu)`
`907`	`918`	`{`
`908`		`- if (contextMenu.PlacementTarget is DataGrid grid && grid.CurrentCell.Column != null)`
	`919`	`+ var grid = FindParentDataGrid(contextMenu.PlacementTarget);`
	`920`	`+ if (grid != null && grid.CurrentCell.Column != null)`
`909`	`921`	`{`
`910`	`922`	`var cellContent = TabHelpers.GetCellContent(grid, grid.CurrentCell);`
`911`	`923`	`if (!string.IsNullOrEmpty(cellContent))`
`@@ -921,7 +933,8 @@ private void CopyRow_Click(object sender, RoutedEventArgs e)`
`921`	`933`	`{`
`922`	`934`	`if (sender is MenuItem menuItem && menuItem.Parent is ContextMenu contextMenu)`
`923`	`935`	`{`
`924`		`- if (contextMenu.PlacementTarget is DataGrid grid && grid.SelectedItem != null)`
	`936`	`+ var grid = FindParentDataGrid(contextMenu.PlacementTarget);`
	`937`	`+ if (grid != null && grid.SelectedItem != null)`
`925`	`938`	`{`
`926`	`939`	`var rowText = TabHelpers.GetRowAsText(grid, grid.SelectedItem);`
`927`	`940`	`if (!string.IsNullOrEmpty(rowText))`
`@@ -937,7 +950,8 @@ private void CopyAllRows_Click(object sender, RoutedEventArgs e)`
`937`	`950`	`{`
`938`	`951`	`if (sender is MenuItem menuItem && menuItem.Parent is ContextMenu contextMenu)`
`939`	`952`	`{`
`940`		`- if (contextMenu.PlacementTarget is DataGrid grid)`
	`953`	`+ var grid = FindParentDataGrid(contextMenu.PlacementTarget);`
	`954`	`+ if (grid != null)`
`941`	`955`	`{`
`942`	`956`	`var sb = new StringBuilder();`
`943`	`957`
`@@ -975,7 +989,8 @@ private void ExportToCsv_Click(object sender, RoutedEventArgs e)`
`975`	`989`	`{`
`976`	`990`	`if (sender is MenuItem menuItem && menuItem.Parent is ContextMenu contextMenu)`
`977`	`991`	`{`
`978`		`- if (contextMenu.PlacementTarget is DataGrid grid)`
	`992`	`+ var grid = FindParentDataGrid(contextMenu.PlacementTarget);`
	`993`	`+ if (grid != null)`
`979`	`994`	`{`
`980`	`995`	`var dialog = new SaveFileDialog`
`981`	`996`	`{`
Original file line number	Diff line number	Diff line change
`@@ -1841,11 +1841,18 @@ public async Task<List<FinOpsRecommendation>> GetFinOpsRecommendationsAsync(deci`
`1841`	`1841`
`1842`	`1842`	`if (edition.Contains("Enterprise", StringComparison.OrdinalIgnoreCase))`
`1843`	`1843`	`{`
`1844`		`- using var featCmd = new SqlCommand(@"`
`1845`		`-SELECT`
`1846`		`- DB_NAME(database_id) AS database_name,`
`1847`		`- feature_name`
`1848`		`-FROM sys.dm_db_persisted_sku_features", connection);`
	`1844`	`+ var hasDatabaseId = false;`
	`1845`	`+ using (var colCheck = new SqlCommand(`
	`1846`	`+ "SELECT COL_LENGTH('sys.dm_db_persisted_sku_features', 'database_id')", connection))`
	`1847`	`+ {`
	`1848`	`+ colCheck.CommandTimeout = 10;`
	`1849`	`+ hasDatabaseId = await colCheck.ExecuteScalarAsync() is not null and not DBNull;`
	`1850`	`+ }`
	`1851`	`+`
	`1852`	`+ var featSql = hasDatabaseId`
	`1853`	`+ ? "SELECT DB_NAME(database_id) AS database_name, feature_name FROM sys.dm_db_persisted_sku_features"`
	`1854`	`+ : "SELECT N'(unknown)' AS database_name, feature_name FROM sys.dm_db_persisted_sku_features";`
	`1855`	`+ using var featCmd = new SqlCommand(featSql, connection);`
`1849`	`1856`	`featCmd.CommandTimeout = 30;`
`1850`	`1857`
`1851`	`1858`	`var features = new List<string>();`
`@@ -1909,7 +1916,7 @@ public async Task<List<FinOpsRecommendation>> GetFinOpsRecommendationsAsync(deci`
`1909`	`1916`	`}`
`1910`	`1917`	`catch (Exception ex)`
`1911`	`1918`	`{`
`1912`		`- Logger.Error($"Recommendation check failed (Enterprise features): {ex.Message}", ex);`
	`1919`	`+ Logger.Error($"[{ServerLabel}] Recommendation check failed (Enterprise features): {ex.Message}", ex);`
`1913`	`1920`	`}`
`1914`	`1921`
`1915`	`1922`	`// 2. CPU right-sizing score`
`@@ -1953,7 +1960,7 @@ FROM report.finops_utilization_efficiency AS v`
`1953`	`1960`	`}`
`1954`	`1961`	`catch (Exception ex)`
`1955`	`1962`	`{`
`1956`		`- Logger.Error($"Recommendation check failed (CPU right-sizing): {ex.Message}", ex);`
	`1963`	`+ Logger.Error($"[{ServerLabel}] Recommendation check failed (CPU right-sizing): {ex.Message}", ex);`
`1957`	`1964`	`}`
`1958`	`1965`
`1959`	`1966`	`// 3. Memory right-sizing score`
`@@ -1997,7 +2004,7 @@ ORDER BY ms.collection_time DESC`
`1997`	`2004`	`}`
`1998`	`2005`	`catch (Exception ex)`
`1999`	`2006`	`{`
`2000`		`- Logger.Error($"Recommendation check failed (Memory right-sizing): {ex.Message}", ex);`
	`2007`	`+ Logger.Error($"[{ServerLabel}] Recommendation check failed (Memory right-sizing): {ex.Message}", ex);`
`2001`	`2008`	`}`
`2002`	`2009`
`2003`	`2010`	`// 4. Unused index cost quantification`
`@@ -2019,7 +2026,7 @@ ORDER BY ms.collection_time DESC`
`2019`	`2026`	`}`
`2020`	`2027`	`catch (Exception ex)`
`2021`	`2028`	`{`
`2022`		`- Logger.Error($"Recommendation check failed (Index analysis): {ex.Message}", ex);`
	`2029`	`+ Logger.Error($"[{ServerLabel}] Recommendation check failed (Index analysis): {ex.Message}", ex);`
`2023`	`2030`	`}`
`2024`	`2031`
`2025`	`2032`	`// 5. Compression savings estimator`
`@@ -2086,7 +2093,7 @@ size_mb DESC`
`2086`	`2093`	`}`
`2087`	`2094`	`catch (Exception ex)`
`2088`	`2095`	`{`
`2089`		`- Logger.Error($"Recommendation check failed (Compression): {ex.Message}", ex);`
	`2096`	`+ Logger.Error($"[{ServerLabel}] Recommendation check failed (Compression): {ex.Message}", ex);`
`2090`	`2097`	`}`
`2091`	`2098`
`2092`	`2099`	`// 6. Dormant database detection with cost impact`
`@@ -2122,7 +2129,7 @@ size_mb DESC`
`2122`	`2129`	`}`
`2123`	`2130`	`catch (Exception ex)`
`2124`	`2131`	`{`
`2125`		`- Logger.Error($"Recommendation check failed (Dormant databases): {ex.Message}", ex);`
	`2132`	`+ Logger.Error($"[{ServerLabel}] Recommendation check failed (Dormant databases): {ex.Message}", ex);`
`2126`	`2133`	`}`
`2127`	`2134`
`2128`	`2135`	`// 7. Dev/test workload detection`
`@@ -2159,7 +2166,7 @@ FROM sys.databases`
`2159`	`2166`	`}`
`2160`	`2167`	`catch (Exception ex)`
`2161`	`2168`	`{`
`2162`		`- Logger.Error($"Recommendation check failed (Dev/test detection): {ex.Message}", ex);`
	`2169`	`+ Logger.Error($"[{ServerLabel}] Recommendation check failed (Dev/test detection): {ex.Message}", ex);`
`2163`	`2170`	`}`
`2164`	`2171`
`2165`	`2172`	`// 11. Maintenance window efficiency — jobs running long`
`@@ -2206,7 +2213,7 @@ HAVING SUM(CAST(is_running_long AS int)) >= 3`
`2206`	`2213`	`}`
`2207`	`2214`	`catch (Exception ex)`
`2208`	`2215`	`{`
`2209`		`- Logger.Error($"Recommendation check failed (Maintenance window): {ex.Message}", ex);`
	`2216`	`+ Logger.Error($"[{ServerLabel}] Recommendation check failed (Maintenance window): {ex.Message}", ex);`
`2210`	`2217`	`}`
`2211`	`2218`
`2212`	`2219`	`// 12. VM right-sizing — prescriptive core/memory targets`
`@@ -2215,7 +2222,7 @@ HAVING SUM(CAST(is_running_long AS int)) >= 3`
`2215`	`2222`	`using var vmCmd = new SqlCommand(@"`
`2216`	`2223`	`SET TRANSACTION ISOLATION LEVEL READ UNCOMMITTED;`
`2217`	`2224`	`SELECT`
`2218`		`- p95_cpu = (SELECT PERCENTILE_CONT(0.95) WITHIN GROUP (ORDER BY cus.sqlserver_cpu_utilization)`
	`2225`	`+ p95_cpu = (SELECT TOP (1) PERCENTILE_CONT(0.95) WITHIN GROUP (ORDER BY cus.sqlserver_cpu_utilization) OVER ()`
`2219`	`2226`	`FROM collect.cpu_utilization_stats AS cus`
`2220`	`2227`	`WHERE cus.collection_time >= DATEADD(DAY, -7, SYSDATETIME())),`
`2221`	`2228`	`cpu_count = (SELECT si.cpu_count FROM sys.dm_os_sys_info AS si),`
`@@ -2291,7 +2298,7 @@ FROM sys.dm_os_performance_counters AS pc`
`2291`	`2298`	`}`
`2292`	`2299`	`catch (Exception ex)`
`2293`	`2300`	`{`
`2294`		`- Logger.Error($"Recommendation check failed (VM right-sizing): {ex.Message}", ex);`
	`2301`	`+ Logger.Error($"[{ServerLabel}] Recommendation check failed (VM right-sizing): {ex.Message}", ex);`
`2295`	`2302`	`}`
`2296`	`2303`
`2297`	`2304`	`// 13. Storage tier optimization — flag databases with low IO latency`
`@@ -2353,7 +2360,7 @@ ORDER BY`
`2353`	`2360`	`}`
`2354`	`2361`	`catch (Exception ex)`
`2355`	`2362`	`{`
`2356`		`- Logger.Error($"Recommendation check failed (Storage tier): {ex.Message}", ex);`
	`2363`	`+ Logger.Error($"[{ServerLabel}] Recommendation check failed (Storage tier): {ex.Message}", ex);`
`2357`	`2364`	`}`
`2358`	`2365`
`2359`	`2366`	`// 14. Reserved capacity candidates — stable CPU utilization`
`@@ -2398,7 +2405,7 @@ HAVING COUNT(*) >= 24`
`2398`	`2405`	`}`
`2399`	`2406`	`catch (Exception ex)`
`2400`	`2407`	`{`
`2401`		`- Logger.Error($"Recommendation check failed (Reserved capacity): {ex.Message}", ex);`
	`2408`	`+ Logger.Error($"[{ServerLabel}] Recommendation check failed (Reserved capacity): {ex.Message}", ex);`
`2402`	`2409`	`}`
`2403`	`2410`
`2404`	`2411`	`return recommendations;`