UI vertical-slice migration with hexagonal Common/ ports

[jschick04]

Reorganizes the EventLogExpert.UI project into a vertical-slice architecture with hexagonal Common/ ports for cross-cutting infrastructure. Builds on top of the eventing-restructure branch (#518).

Highlights

Hexagonal Common/ ports (R2-R8, R30-R33):

• Modal/Alert broker extraction (interface segregation: IInlineAlertHostBroker)
• System/File/Clipboard ports moved to Common/
• IPreferencesProvider, Versioning, AppTitle, IMainThreadService, IWindowsIdentityProvider, LoggingMiddleware, DisplayExtensions all relocated to Common/<Domain>/ slices

Vertical slice promotion (R9-R20):
StatusBar, DebugLog, Banner, Modal, Alerts, Settings, Menu, Database, Update (incl. Deployment + ReleaseNotes), LogTable (was EventTable), EventLog, Filter (incl. FilterCache, FilterGroup, FilterPane).

Renames at the type/file level:

• EventTable -> LogTable slice + LogView record + LogTablePane / LogTabBar components
• CopyType -> EventCopyFormat
• ModalAlertDialogService -> AlertDialogService
• GitReleaseModel -> GitHubRelease
• SavedFilter/SavedFilterGroup/FilterDraft/DateFilter clarifications
• FilterDraft.FromFilterModel -> FromSavedFilter

Typed ID records (replaces raw Guid on internal IDs):

• EventLogId (existing pattern)
• BannerId (Banner slice)
• UpgradeBatchId (Database.Upgrade event flow)
• StatusActivityId (StatusBar loading-tracker)

Internalizations / surface tightening:

• SavedFilterJsonConverter -> internal
• FolderPickerHelper -> internal
• GitHubReleaseNormalizer (already internal) relocated next to its sole consumer (Update/)

Methodology

Atomic commit-per-slice with the eventing-PR conventions:

• Common// requires >=2-file gate (R5 was a documented exception per discussion)
• One type per file
• Hexagonal ports/adapters
• Per-commit hygiene gates: build (0/0), dotnet format whitespace --verify-no-changes, dotnet format style --diagnostics IDE0005 IDE0065, dotnet test, ReSharper cleanup
• Multi-reviewer panel before each major design (codex / rubber-duck / gpt-5.5 / opus-xhigh)

Test baseline (locked)

1610 unit + 235 integration. All gates green at HEAD.

Notes

• LogTable rename brings the EventTable slice into the consistent UI naming system (*Layout / *Pane / *Bar / *Host).
• EventLogExpert.Components is a separate assembly without IVT to UI; some types remain public because Components consumes them. Filter-internal candidates that depend on EditableFilterRowBase are tracked for the follow-up filter-rework task.
• Per arch reviewer: 100% namespace/folder consistency post-promotion. Two cross-slice cycles flagged for the filter-rework follow-up; not addressed here to keep this PR scoped to the migration itself.

[Copilot]

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

[Copilot]

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.