Fix: Memory leaks

[averyhome]

Summary

This PR fixes two memory leak issues:

1. NWPathMonitor was not properly cancelled, causing retain cycles and memory leaks.
2. requestController retained cycles

Changes

• Add monitor.cancel() in appropriate deinit or shutdown logic.
• Fix retain cycle by using [weak self] in closures.
• Add relevant cleanup code to avoid strong references.

Motivation

These issues caused retained cycles in production use, especially with reconnect behavior and long-running sessions.

[home-assistant]

Hi @averyhome

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

[home-assistant]

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

[bgoncal]

Is this PR ready or you plan to add anything else?

[Copilot]

Pull Request Overview

This PR fixes memory leaks by addressing retain cycles in the network monitoring subsystem and connection management.

• Adds proper cleanup for NWPathMonitor to prevent retain cycles
• Converts strong capture to weak capture in request controller closures

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
Source/Internal/HAReconnectManager.swift	Adds deinit method to cancel pathMonitor and prevent memory leaks
Source/Internal/HAConnectionImpl.swift	Changes strong capture to weak capture for requestController to break retain cycle

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}