Add list_connected_taps method to Windows SDK for retrieving connected devices

[Copilot]

Problem

The Windows SDK (TapWindowsSDK) was missing functionality to retrieve already connected TAP devices, unlike the POSIX implementation which has list_connected_taps(). This prevented the SDK from establishing connections with TAPs that were already paired and connected to the system.

Solution

Added list_connected_taps() method to both the abstract base class and Windows implementation:

1. Abstract Interface

• Added async def list_connected_taps() as an abstract method to TapSDKBase
• Ensures consistent interface across all platform implementations

2. Windows Implementation

• Implemented robust method in TapWindowsSDK that queries TAPManager for connected devices
• Uses defensive programming with multiple fallback strategies for different TAPWin DLL versions:
  • Tries TAPManager.Instance.GetConnectedTaps()
  • Falls back to TAPManager.Instance.GetConnectedDevices()
  • Attempts property access patterns (ConnectedTaps, ConnectedDevices)
  • Gracefully handles missing methods by returning empty list

3. Enhanced Examples

• Updated example_win.py to demonstrate usage before SDK initialization
• Added example_list_connected.py showing cross-platform usage patterns

Usage

from tapsdk import TapSDK
import asyncio

# Create SDK instance
sdk = TapSDK()

# Check for connected devices before starting
connected_taps = await sdk.list_connected_taps()
if connected_taps:
    print(f"Found {len(connected_taps)} connected TAP device(s)")
    # Proceed with SDK operations
    sdk.run()
else:
    print("No TAP devices found. Please pair and connect your device.")

Benefits

• Fixes Connection Issues: Windows SDK can now discover and work with already connected TAPs
• Robust Error Handling: Gracefully handles different TAPWin DLL versions
• Cross-Platform Consistency: Unified interface across Windows and POSIX platforms
• Backward Compatible: Existing code continues to work unchanged

Fixes #27.

---

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

[ilouzl]

the issue is supposed to be fixed on branch develop, not on master.

[ilouzl]

Change the PR so that it is based on branch develop and not master

[Copilot]

Change the PR so that it is based on branch develop and not master

I've rebased the PR to be based on the develop branch instead of master. The implementation has been adapted to work with the develop branch's unified BLE architecture, replacing the old Windows-specific TAPManager backend with a cross-platform solution using the bleak library. Commit ca191c4.

the issue is supposed to be fixed on branch develop, not on master.

Fixed - the PR is now based on the develop branch. The list_connected_taps() method has been implemented to work with the develop branch's unified BLE implementation instead of the old Windows backend. Commit ca191c4.