Update Omarchy skill for Quattro#6310
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the Omarchy agent skill documentation to reflect Omarchy 4 (“quattro”) Hyprland Lua configuration, so agents point users at the active *.lua modules (and Omarchy’s Lua helpers) instead of legacy *.conf files that may reload without effect.
Changes:
- Clarifies
/usr/share/omarchyas packaged runtime files (or a linked dev checkout), not a Git-managed working tree. - Updates Hyprland configuration guidance to reference
hyprland.lua+ the Lua override modules (bindings.lua,monitors.lua, etc.) and Omarchy helper APIs. - Refreshes keybinding/monitor/window-rule examples and updates the canonical Hyprland window rules documentation link.
Tip
If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or run gh pr ready --undo.
Click "Ready for review" or run gh pr ready to reengage.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
46e7e3b to
e4087fc
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
Comments suppressed due to low confidence (1)
default/omarchy-skill/SKILL.md:353
- The rebind checklist still uses legacy
.confterminology (“unbind directive” / “bind”). Since this section now documents the Lua helpers, the checklist should reference callinghl.unbind(...)beforeo.bind(...)to match the examples and avoid confusion.
3. Inform the user what the key was previously bound to
e4087fc to
945ad39
Compare
945ad39 to
5dead48
Compare
| | `omarchy pkg` | Package management | `omarchy pkg install <pkg>` | | ||
| | `omarchy setup` | Initial setup tasks | `omarchy setup fingerprint` | | ||
| | `omarchy pkg` | Package management | `omarchy pkg add <pkg>` | | ||
| | `omarchy setup` | Initial setup tasks | `omarchy setup security fingerprint` | |
The skill still described pre-Quattro .conf files and workflows, sending agents to inactive config files and stale commands. Now covers the Lua config layout, o.bind/hl.unbind/hl.monitor forms, .d hook directories with omarchy hook install, plugin cloning, and corrected command routes. Distilled from #6310. Co-Authored-By: ssupt <149246609+ssupt@users.noreply.github.com> Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Thanks — this caught real staleness. The essential corrections (Lua config layout, o.bind/hl.unbind/hl.monitor syntax, .d hook directories, plugin cloning, command route fixes) landed in 1feb025 with co-author credit, trimmed to keep the skill succinct. One note: |
The skill still described pre-Quattro .conf files and workflows, sending agents to inactive config files and stale commands. Now covers the Lua config layout, o.bind/hl.unbind/hl.monitor forms, .d hook directories with omarchy hook install, plugin cloning, and corrected command routes. Distilled from basecamp#6310. Co-Authored-By: ssupt <149246609+ssupt@users.noreply.github.com> Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Summary
Why
The Omarchy skill is the mandatory guide for end-user desktop customization, but parts of it still described pre-Quattro files and workflows. That could send agents to inactive Hyprland
.conffiles, hidden support commands, incomplete hook paths, or packaged shell plugin source.This update aligns the full set of customization scenarios claimed by the skill with the current
quattrobranch while keeping Omarchy source-development workflows explicitly out of scope.Verification
./test/cligit diff --checkomarchy commands --json./test/clicompleted successfully. Its theme test emitted the expected sandbox-only/run/user/1000lock warning, but the suite exited 0.