Fix blocking send while holding request mutex

[wltry]

Move send() outside request_mutex_ while preserving command ordering with send_mutex_. Also apply clang-tidy modernize fixes in client.cpp.

Changes Made

Changes to libwaterlinked client.cpp to resolve blocking issue.

Associated Issues

Resolves issue discussed in #32

Testing

Please provide a clear and concise description of the testing performed.

[Copilot]

Pull request overview

This PR updates the libwaterlinked TCP client to avoid holding the request-tracking mutex while performing a potentially blocking socket send(), while still preserving per-command ordering via a new dedicated send mutex.

Changes:

• Serialize command transmission with a new send_mutex_, and narrow the scope of request_mutex_ to only pending-request bookkeeping.
• Apply clang-tidy modernization updates (std::scoped_lock, designated initializers for CommandResponse).
• Bump libwaterlinked package metadata (version and maintainer).

Reviewed changes

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

File	Description
libwaterlinked/src/client.cpp	Moves command sending out from request_mutex_ and modernizes locking/struct initialization.
libwaterlinked/include/libwaterlinked/client.hpp	Adds send_mutex_ to enforce send ordering without blocking request bookkeeping.
libwaterlinked/package.xml	Updates package version and maintainer metadata.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Pull request overview

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