Skip to content

fix: support new Hyprland lua config parser#13

Merged
MuNeNiCK merged 2 commits into
MuNeNiCK:mainfrom
LuukBlankenstijn:fix/support-new-hyprland-config-parser
Jun 15, 2026
Merged

fix: support new Hyprland lua config parser#13
MuNeNiCK merged 2 commits into
MuNeNiCK:mainfrom
LuukBlankenstijn:fix/support-new-hyprland-config-parser

Conversation

@LuukBlankenstijn

Copy link
Copy Markdown
Contributor

As described in #12 the new Hyprland Lua config parser does not support the keyword monitor command anymore. This PR fixes this by first trying the old behavior of keyword monitor and if this fails, tries to use the new Lua api.

Closes #12.

@LuukBlankenstijn

Copy link
Copy Markdown
Contributor Author

I have been using this fix locally for a while, but I saw someone made an issue for it so here a PR.

@MuNeNiCK

MuNeNiCK commented Jun 15, 2026

Copy link
Copy Markdown
Owner

Thanks for the fix. I added one follow-up commit with unit tests for the Hyprland Lua monitor fallback, including the generated headless-output rule, non-numeric scale handling, Lua string escaping, malformed rule rejection, and non-legacy parser error detection.

Verified the merge result locally with:

  • cargo fmt --check
  • cargo test hyprland
  • cargo test
  • cargo clippy --all-targets -- -D warnings

I will merge this once the GitHub check finishes.

@MuNeNiCK
MuNeNiCK merged commit 069f320 into MuNeNiCK:main Jun 15, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update Hyprland IPC usage for non-legacy parsers

2 participants