hyprctl binds to also show modkeys (string) instead of just modmask (int)

[msmafra]

Description

This bind uses $mainMod (SUPER) plus SHIFT plus RETURN and shows modmask 65 as result

❯ hyprctl binds -j | jq -r '.[] | select(.description |contains("Alacritty"))'
{
  "locked": false,
  "mouse": false,
  "release": false,
  "repeat": false,
  "non_consuming": false,
  "has_description": true,
  "modmask": 65,
  "submap": "",
  "key": "RETURN",
  "keycode": 0,
  "catch_all": false,
  "description": "Opens Alacritty terminal",
  "dispatcher": "exec",
  "arg": "alacritty"
}

It would be very nice for it to also show the modkeys

❯ hyprctl binds -j | jq -r '.[] | select(.description |contains("manager"))'
{
  "locked": false,
  "mouse": false,
  "release": false,
  "repeat": false,
  "non_consuming": false,
  "has_description": true,
  "modmask": 64,
  "modkeys":"SUPER SHIFT",
  "submap": "",
  "key": "E",
  "keycode": 0,
  "catch_all": false,
  "description": "Opens your file manager",
  "dispatcher": "exec",
  "arg": "nautilus"
}

"modkeys":"SUPER SHIFT",

or

"modkeys": [
                 "SUPER",
                 "SHIFT"
],

[vaxerski]

but why? Modmask is what you want.

[msmafra]

but why? Modmask is what you want.

Sorry, my intention is/was not to replace modmask, but to add modkeys. It would allow to a much clear/easier keybinds generation with hyprctl binds, making it a simpler dynamic thing, for example. Even more now with the description for binds.
It would be quicker to check the keybinds with hyprctl -j binds | jq <some_filter_here>

[msmafra]

Like so

{
  "modmask": 72,
  "modname": "SUPER ALT",
  "locked": false,
  "mouse": false,
  "release": false,
  "repeat": false,
  "non_consuming": false,
  "has_description": true,
  "submap": "",
  "key": "comma",
  "keycode": 0,
  "catch_all": false,
  "description": "Move current workspace to another monitor decrementally",
  "dispatcher": "movecurrentworkspacetomonitor",
  "arg": "-1"
},
{
  "modmask": 77,
  "modname": "SUPER SHIFT ALT CTRL",
  "locked": false,
  "mouse": false,
  "release": false,
  "repeat": false,
  "non_consuming": false,
  "has_description": true,
  "submap": "",
  "key": "END",
  "keycode": 0,
  "catch_all": false,
  "description": "Exits Hyprland by terminating the user sessions",
  "dispatcher": "exec",
  "arg": "loginctl terminate-user \"\""
}

[vaxerski]

I am generally against duplicating data that can be easily parsed.