Fix Windows compilation errors in platform-specific code

[Copilot]

This PR fixes several Windows compilation errors in the src/platform/windows directory that would prevent the library from building on Windows systems.

Issues Fixed

PIMPL Implementation Inconsistencies

The Windows platform code had naming mismatches between header declarations and implementations:

• WindowManager header declared pimpl_ but implementation used impl_
• PIMPL class was named WindowManagerImpl instead of Impl to match forward declaration

Const Correctness Issues

Several Window class getter methods were declared as non-const in the header but should be const since they don't modify object state:

// Before (compilation error)
bool IsAlwaysOnTop();
Point GetPosition();
std::string GetTitle();

// After (fixed)
bool IsAlwaysOnTop() const;
Point GetPosition() const;
std::string GetTitle() const;

Missing Library Linking Directives

Windows platform files were missing required #pragma comment(lib, ...) directives for:

• shell32.lib - Required for Shell_NotifyIcon and other tray icon APIs
• oleacc.lib - Required for accessibility manager functions

Technical Details

The fixes ensure proper compilation on Windows by:

1. Aligning PIMPL naming conventions between headers and implementations
2. Adding const qualifiers to methods that don't modify object state
3. Including necessary Windows library linking pragmas for platform-specific APIs

All changes are minimal and surgical, focusing only on compilation correctness without altering functionality or breaking existing working code.

Testing

The changes have been verified to maintain consistency across all Windows platform files and ensure proper header/implementation alignment.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.