Skip to content

v1.4.0: Config consolidation, DB drop, DuckDB-first plans, lock wait chart, clipboard fix#224

Merged
erikdarlingdata merged 1 commit intodevfrom
feature/consolidate-config-changes
Feb 22, 2026
Merged

v1.4.0: Config consolidation, DB drop, DuckDB-first plans, lock wait chart, clipboard fix#224
erikdarlingdata merged 1 commit intodevfrom
feature/consolidate-config-changes

Conversation

@erikdarlingdata
Copy link
Owner

Summary

Additional fixes discovered during testing

  • Added 7 missing columns + Download Plan button to procedure stats grid
  • Fixed plan download button text changing to "Save" instead of staying "Download"
  • Filtered zero-activity rows from query/procedure stats with HAVING clause
  • Fixed "System.Windows.Controls.StackPanel" appearing in clipboard copy headers (both Dashboard and Lite)

Test plan

  • Dashboard: Configuration Changes tab shows three sub-tabs with correct data
  • Dashboard: Remove Server dialog offers DB drop option, works with and without checkbox
  • Lite: Lock wait trend chart renders multi-series data correctly
  • Lite: Procedure stats grid shows all columns with non-zero data
  • Lite: Plan downloads try DuckDB first, fall back to live server
  • Both apps: Copy All Rows shows proper column headers, not StackPanel type names
  • Both apps: Build succeeds with 0 errors, 0 warnings

Closes #185, #201, #219, #141

🤖 Generated with Claude Code

…kDB-first plans, lock wait trend, clipboard fix, procedure stats parity

Issues addressed:
- #185: Consolidate Server/Database/Trace Flag config change tabs into single ConfigChangesContent UserControl with sub-tabs (Dashboard)
- #201: Add option to drop PerformanceMonitor database when removing a server, with confirmation dialog (Dashboard)
- #219: Try DuckDB cache first for query plan downloads before falling back to live server (Lite)
- #141: Add lock waits trend chart to Blocking > Trends tab with multi-series per LCK% wait type (Lite)

Additional fixes:
- Fix clipboard copy: "Copy All Rows" and "Export to CSV" context menu items now extract TextBlock text from StackPanel column headers instead of showing "System.Windows.Controls.StackPanel" (both apps, 22 call sites)
- Add missing columns to Lite procedure stats grid: Total Spills, Min/Max CPU, Min/Max Duration, Plan Handle, and Download Plan button with DuckDB-first lookup
- Filter out zero-activity rows from query stats and procedure stats grids (HAVING clause on delta sums)
- Make query stats grid plan download DuckDB-first (was live-server only)
- Fix plan download button text resetting to "Save" instead of "Download"

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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