Ignore missing group window indexes

[aastrand]

What changed

Guard the numbered group-window bindings before dispatching to hl.dsp.group.active.

Pressing Super+Alt+1–5 now:

• switches normally when that member exists in the active window group
• quietly does nothing when the active window is ungrouped or the requested index is outside the group

This prevents Hyprland from showing a Lua runtime-error notification for a missing group member.

Screenshot pre-fix

Testing

• Tested live on Hyprland 0.56.2:
  • valid indexes still switch grouped windows
  • nonexistent indexes on grouped and ungrouped windows are quiet no-ops
• luac -p default/hypr/bindings/tiling.lua
• bash test/shell.d/hyprland-default-config-test.sh
• bash test/shell.d/hyprland-binding-conflicts-test.sh

[paulogeyer]

+1 — right fix for #12085.

Live-verified on Omarchy / Hyprland 0.56.2: stock SUPER+ALT+3 (group of 2) shows Runtime error in lua: Index out of range; with a size-guard before hl.dsp.group.active the miss is a silent no-op and SUPER+ALT+1/2 still switch.

Please also pull in the small regression test from #8411 (test/shell.d/hyprland-group-bindings-test.sh) so this doesn’t regress. Prefer this focused PR over the pcall-only hunk in #12155.

[omarchybot]

Thanks for this, @aastrand. It is the same change as #8411, which was opened earlier against quattro, so I'm closing this one in favour of it.

Both PRs fix #12085 the same way: in default/hypr/bindings/tiling.lua the Super+Alt+1–5 bindings become functions that look up hl.get_active_window(), read its group, and call hl.dispatch(hl.dsp.group.active({ index = index })) only when the window is grouped and index <= group.size. The cause, the guard and the result are identical. #8411 also adds a regression test, test/shell.d/hyprland-group-bindings-test.sh, that covers an in-range index, an out-of-range index and an ungrouped window, so it is the more complete of the two.

I compared the two diffs by reading them. Nothing was run for this check. Codex Medium was asked separately whether they make the same change and agreed. Independence isn't guaranteed, since it can read this session's reasoning.

If I've got this wrong and there is a difference that matters, say so and it can be reopened.