Skip to content

fix(mango_layouts): use correct mmsg syntax for layout retrieval and dispatch - #451

Closed
DNI9 wants to merge 1 commit into
noctalia-dev:mainfrom
DNI9:fix-mango-layouts-ipc
Closed

fix(mango_layouts): use correct mmsg syntax for layout retrieval and dispatch#451
DNI9 wants to merge 1 commit into
noctalia-dev:mainfrom
DNI9:fix-mango-layouts-ipc

Conversation

@DNI9

@DNI9 DNI9 commented Aug 23, 2026

Copy link
Copy Markdown

Plugin

  • Id: ezequiel/mango_layouts
  • New plugin
  • Update to an existing plugin (version bumped in plugin.toml)

What it does

Fixes broken mmsg IPC commands. Replaces invalid mmsg get all-monitors | jq with mmsg -g | awk to parse active monitor layout, and fixes mmsg dispatch to mmsg -d setlayout.

External dependencies

mmsg, awk, jq (existing dependencies).

Testing

Tested reading the layout and updating it via mmsg commands.

  • Tested on Niri
  • Tested on Hyprland
  • Tested on Sway
  • Tested on another compositor: MangoWC
  • Noctalia version tested against: 5.0.0
  • Plugin API level: 14

Screenshots / Videos

N/A - bug fix for background command

Checklist

  • The directory name matches the part of id after the / in plugin.toml exactly.
  • It ships plugin.toml, README.md, thumbnail.webp, and translations/en.json.
  • README.md follows the README template, documents every entry id and dependency, and includes exact panel IPC commands and launcher prefixes where applicable.
  • I created thumbnail.webp with the thumbnail generator.
  • version follows semver and is bumped in this PR; plugin_api is the oldest API level this plugin requires.
  • Every non-English translation in this PR uses a locale supported by Noctalia core, and I can read, write, and understand that language well enough to review and maintain it (no unreviewed machine/LLM translations).
  • I did not edit catalog.toml; CI generates it.
  • This PR touches exactly one plugin directory.

Code review attestation

  • The code is readable and not obfuscated, minified, or generated.
  • It does not download and execute remote code.
  • Every network call, filesystem write, and spawned process is something the description above accounts for.
  • I have the right to publish this code under the license declared in plugin.toml.

…dispatch

mmsg does not output JSON or support 'get all-monitors' and 'dispatch'
subcommands. Switched to 'mmsg -g' with awk for parsing active monitor
layout, and 'mmsg -d setlayout' for applying layouts.
@github-actions

Copy link
Copy Markdown
Contributor

This pull request was automatically closed because its description no longer contains
every part of the pull request template
that this repository requires.

Missing:

  • the template marker line <!-- noctalia-pr-template:v1 -->
  • the ## Plugin heading
  • the ## What it does heading
  • the ## External dependencies heading
  • the ## Testing heading
  • the ## Screenshots / Videos heading
  • the ## Checklist heading
  • the ## Code review attestation heading
  • the - **Id:** field
  • the - **Noctalia version tested against:** field
  • the - **Plugin API level:** field
  • the checklist entry: New plugin
  • the checklist entry: Update to an existing plugin (version bumped in plugin.toml)
  • the checklist entry: Tested on Niri
  • the checklist entry: Tested on Hyprland
  • the checklist entry: Tested on Sway
  • the checklist entry: Tested on another compositor:
  • the checklist entry: The directory name matches the part of id after the / in plugin.toml exactly.
  • the checklist entry: It ships plugin.toml, README.md, thumbnail.webp, and translations/en.json.
  • the checklist entry: README.md follows the README template, documents every entry id and dependency, and includes exact panel IPC commands and launcher prefixes where applicable.
  • the checklist entry: I created thumbnail.webp with the thumbnail generator.
  • the checklist entry: version follows semver and is bumped in this PR; plugin_api is the oldest API level this plugin requires.
  • the checklist entry: Every non-English translation in this PR uses a locale supported by Noctalia core, and I can read, write, and understand that language well enough to review and maintain it (no unreviewed machine/LLM translations).
  • the checklist entry: I did not edit catalog.toml; CI generates it.
  • the checklist entry: This PR touches exactly one plugin directory.
  • the checklist entry: The code is readable and not obfuscated, minified, or generated.
  • the checklist entry: It does not download and execute remote code.
  • the checklist entry: Every network call, filesystem write, and spawned process is something the description above accounts for.
  • the checklist entry: I have the right to publish this code under the license declared in plugin.toml.
  • exactly one checked plugin type: New plugin or Update to an existing plugin
  • at least one checked compositor testing entry

Please add the items listed above back to the description, keeping their exact wording, then
reopen the pull request. Reopening re-runs this check. Draft pull requests may leave boxes
unchecked. Before a pull request is ready for review, exactly one plugin type, at least one
tested compositor, and every item under Checklist and Code review attestation must be checked.

@github-actions github-actions Bot closed this Aug 23, 2026
@github-actions

Copy link
Copy Markdown
Contributor

CC @ezequielgk

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.

1 participant