Please address the comments from this code review:

## Overall Comments
- The hardcoded modifier bit values (1, 2, 4, 8, 16, 64) in `WaylandHelper::depressed_key_modifiers`/`locked_key_modifiers` would be clearer and less error-prone if expressed via a named enum or `constexpr` bitmask constants shared with `WaylandClient`.
- The `input_key` modifier handling currently ignores `EventPhase::Moved`; if this phase is not expected for key events, consider asserting/logging, or otherwise documenting the assumption to avoid silent no-ops.
- The logic `current_locked_modifiers_ ^= locked_modifier` on key press assumes all lock keys toggle; if additional lock-like keys are added later this might be fragile—consider centralizing the toggle semantics or restricting it explicitly to caps/num lock to make the behavior clearer.

## Individual Comments

### Comment 1
<location path="source/MaaWlRootsControlUnit/Client/WaylandHelper.cpp" line_range="45-54" />
<code_context>
     }
 }

+int WaylandHelper::depressed_key_modifiers(int key) {
+    switch (key) {
+        case KEY_LEFTSHIFT:
+        case KEY_RIGHTSHIFT:
+            return 1;
+        case KEY_CAPSLOCK:
+            return 2;
+        case KEY_LEFTCTRL:
+        case KEY_RIGHTCTRL:
+            return 4;
+        case KEY_LEFTALT:
+        case KEY_RIGHTALT:
+            return 8; // Mod1
+        case KEY_NUMLOCK:
+            return 16; // Mod2
+        case KEY_LEFTMETA:
+        case KEY_RIGHTMETA:
+            return 64; // Mod4
+        default:
+            return 0;
</code_context>
<issue_to_address>
**suggestion:** Replace magic modifier bit values with named constants or a strongly-typed enum/bitmask.

The hard-coded bit values (1, 2, 4, 8, 16, 64) obscure which logical modifier each represents and make it harder to check alignment with xkb/common. Please introduce `constexpr` named constants or a scoped enum used as a bitmask (e.g., Shift, Lock, Control, Mod1, Mod2, Mod4) and use those instead, so future changes (adding Mod3/Mod5 or adjusting bit positions) are easier and safer.

Suggested implementation:

```cpp
}
}

namespace {
constexpr int MOD_SHIFT   = 1;
constexpr int MOD_LOCK    = 2;
constexpr int MOD_CONTROL = 4;
constexpr int MOD1        = 8;   // Typically Alt
constexpr int MOD2        = 16;  // Typically NumLock
constexpr int MOD4        = 64;  // Typically Meta/Super
}

int WaylandHelper::depressed_key_modifiers(int key) {

```

```cpp
int WaylandHelper::depressed_key_modifiers(int key) {
    switch (key) {
        case KEY_LEFTSHIFT:
        case KEY_RIGHTSHIFT:
            return MOD_SHIFT;
        case KEY_CAPSLOCK:
            return MOD_LOCK;
        case KEY_LEFTCTRL:
        case KEY_RIGHTCTRL:
            return MOD_CONTROL;
        case KEY_LEFTALT:
        case KEY_RIGHTALT:
            return MOD1;  // Mod1 (Alt)
        case KEY_NUMLOCK:
            return MOD2;  // Mod2 (NumLock)
        case KEY_LEFTMETA:
        case KEY_RIGHTMETA:
            return MOD4;  // Mod4 (Meta/Super)
        default:
            return 0;
    }
}

```

```cpp
int WaylandHelper::locked_key_modifiers(int key) {
    switch (key) {
        case KEY_CAPSLOCK:
            return MOD_LOCK;
        case KEY_NUMLOCK:
            return MOD2; // Mod2 (NumLock)
        default:

```
</issue_to_address>