Please address the comments from this code review:

## Overall Comments
- Consider changing `send_activate()` to return the resolved target `HWND` (or otherwise share it) so that subsequent calls in the same operation (e.g., `touch_*`, `input_text`, `key_*`, `scroll`) use a consistent window rather than re-resolving `get_active_hwnd()` and potentially sending parts of a gesture/sequence to different windows if focus changes mid-flow.
- In `get_active_hwnd()`, it may be worth handling/logging the case where `GetAncestor(hwnd_, GA_ROOTOWNER)` returns `nullptr` more explicitly instead of falling back to `hwnd_`, to make it easier to diagnose issues when the stored `hwnd_` becomes invalid.

Please address the comments from this code review:

## Overall Comments
- In `touch_up`, `send_activate()` already calls `send_activate_message`, so the additional explicit `send_activate_message(target, use_post)` right after is redundant and should be removed to avoid double activation messages.
- When `send_or_post_w` fails in `touch_move`, `gesture_target_` is left unchanged; consider clearing or otherwise handling `gesture_target_` on failure to avoid subsequent gestures using a stale HWND.

## Individual Comments

### Comment 1
<location path="source/MaaWin32ControlUnit/Input/MessageInput.cpp" line_range="132-141" />
<code_context>
+    return hwnd_;
+}
+
+LPARAM MessageInput::make_mouse_lparam(HWND target, int x, int y)
+{
+    if (target == hwnd_) {
+        return MAKELPARAM(x, y);
+    }
+    POINT pt = { x, y };
+    if (!ClientToScreen(hwnd_, &pt)) {
+        LogError << "ClientToScreen failed in make_mouse_lparam" << VAR(hwnd_) << VAR(GetLastError());
+        return MAKELPARAM(x, y);
+    }
+    if (!ScreenToClient(target, &pt)) {
+        LogError << "ScreenToClient failed in make_mouse_lparam" << VAR(target) << VAR(GetLastError());
+        return MAKELPARAM(x, y);
+    }
+    return MAKELPARAM(pt.x, pt.y);
+}
+
</code_context>
<issue_to_address>
**issue (bug_risk):** Failure paths in make_mouse_lparam may produce coordinates in the wrong space for the target window.

If either `ClientToScreen(hwnd_, &pt)` or `ScreenToClient(target, &pt)` fails, we log but still return `MAKELPARAM(x, y)`, which is in `hwnd_`’s client space while the message is sent to `target`. That means we may send mouse events with coordinates in the wrong space, potentially targeting the wrong location. Consider instead either:
- Returning coordinates in a known, consistent space (e.g., last successfully derived screen coords), or
- Explicitly signaling failure so the caller can avoid sending a message with invalid coordinates.
</issue_to_address>

Please address the comments from this code review:

## Individual Comments

### Comment 1
<location path="source/MaaWin32ControlUnit/Input/MessageInput.cpp" line_range="132-141" />
<code_context>
+    return hwnd_;
+}
+
+LPARAM MessageInput::make_mouse_lparam(HWND target, int x, int y)
+{
+    if (target == hwnd_) {
+        return MAKELPARAM(x, y);
+    }
+    POINT pt = { x, y };
+    if (!ClientToScreen(hwnd_, &pt)) {
+        LogError << "ClientToScreen failed in make_mouse_lparam" << VAR(hwnd_) << VAR(GetLastError());
+        return MAKELPARAM(x, y);
+    }
+    if (!ScreenToClient(target, &pt)) {
+        LogError << "ScreenToClient failed in make_mouse_lparam" << VAR(target) << VAR(GetLastError());
+        return MAKELPARAM(x, y);
+    }
+    return MAKELPARAM(pt.x, pt.y);
+}
+
</code_context>
<issue_to_address>
**suggestion (bug_risk):** Coordinate fallback on conversion failure may produce incorrect mouse positions.

If `ClientToScreen(hwnd_, &pt)` or `ScreenToClient(target, &pt)` fails, we log and fall back to `MAKELPARAM(x, y)`. But at that point `x`/`y` are still client coords for `hwnd_`, not for `target`, so the resulting lParam is incorrect for the target window. Consider instead either returning a sentinel (e.g. `0`) so callers can abort, or returning coordinates in a clearly-defined, best-known space (e.g. screen) rather than misleading client coords for `target`.

Suggested implementation:

```cpp
LPARAM MessageInput::make_mouse_lparam(HWND target, int x, int y)
{
    if (target == hwnd_) {
        return MAKELPARAM(x, y);
    }

    POINT pt = { x, y };
    if (!ClientToScreen(hwnd_, &pt)) {
        LogError << "ClientToScreen failed in make_mouse_lparam" << VAR(hwnd_) << VAR(GetLastError());
        // Return sentinel to signal failure so callers can abort sending an incorrect lParam.
        return 0;
    }
    if (!ScreenToClient(target, &pt)) {
        LogError << "ScreenToClient failed in make_mouse_lparam" << VAR(target) << VAR(GetLastError());
        // Return sentinel to signal failure so callers can abort sending an incorrect lParam.
        return 0;
    }
    return MAKELPARAM(pt.x, pt.y);
}

```

Callers of `MessageInput::make_mouse_lparam` should be updated to:
1. Check the returned `LPARAM` value.
2. If it is `0`, treat that as a failure to convert coordinates and **do not** send the mouse message (or take whatever fallback/abort behavior is appropriate for your codebase).
</issue_to_address>