Refactor duplicated code: Add SetProperty helper and generic dialog base classes

[Copilot]

Identified and eliminated ~280 lines of duplicated code across ViewModels, primarily repetitive property setter boilerplate (1073+ instances) and duplicate command initialization patterns.

Changes

Infrastructure

• Added PropertyChangedBase.SetProperty<T>() - Generic helper using EqualityComparer<T> and CallerMemberName
• Added DialogViewModelBase<T> - Generic base for OK/Cancel dialog ViewModels
• Added ConnectDialogViewModelBase<T> - Generic base for Connect/Cancel dialog ViewModels

Refactored ViewModels (15 files)

Dialog ViewModels: DropdownViewModel, IPAddressViewModel, IPAddressAndSubnetmaskViewModel, HostsFileEditorEntryViewModel, OKCancelInfoMessageViewModel, CredentialsPasswordViewModel, CredentialsSetPasswordViewModel, CredentialsChangePasswordViewModel, CredentialsPasswordProfileFileViewModel, PortProfilesViewModel, SNMPOIDProfilesViewModel

Connect ViewModels: RemoteDesktopConnectViewModel, PuTTYConnectViewModel

Property simplification: PingMonitorHostViewModel (13 properties), multiple others (70+ total)

Example

Before (6 lines):

public string Name
{
    get => _name;
    set
    {
        if (value == _name)
            return;
        _name = value;
        OnPropertyChanged();
    }
}

After (1 line):

public string Name
{
    get => _name;
    set => SetProperty(ref _name, value);
}

Future Opportunities

Remaining ViewModels with similar patterns: AWSSessionManagerConnectViewModel, TigerVNCConnectViewModel, WiFiConnectViewModel, PowerShellConnectViewModel, WebConsoleConnectViewModel

Large files with many property setters: SettingsInfo.cs (271), ProfileViewModel.cs (160), GroupViewModel.cs (96)

Original prompt

Find and refactor duplicated code

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.