Skip to content

Update Omarchy skill for Quattro#6310

Closed
ssupt wants to merge 1 commit into
basecamp:quattrofrom
ssupt:fix/omarchy-skill-hyprland-lua
Closed

Update Omarchy skill for Quattro#6310
ssupt wants to merge 1 commit into
basecamp:quattrofrom
ssupt:fix/omarchy-skill-hyprland-lua

Conversation

@ssupt

@ssupt ssupt commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

Summary

  • document the active Omarchy 4 Hyprland Lua configuration and helpers
  • document Quattro shell, bar, plugin cloning, menu extension, idle, and lock configuration
  • cover every supported user hook and the composable hook installation workflow
  • correct stale package, setup, restart, and support-log command guidance
  • clarify packaged versus development paths and make privilege guidance consistent

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 .conf files, 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 quattro branch while keeping Omarchy source-development workflows explicitly out of scope.

Verification

  • ./test/cli
  • git diff --check
  • verified documented CLI routes against omarchy commands --json

./test/cli completed successfully. Its theme test emitted the expected sandbox-only /run/user/1000 lock warning, but the suite exited 0.

Copilot AI review requested due to automatic review settings July 19, 2026 05:38

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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/omarchy as 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.

Comment thread default/omarchy-skill/SKILL.md
Comment thread default/omarchy-skill/SKILL.md
Copilot AI review requested due to automatic review settings July 19, 2026 06:00
@ssupt
ssupt force-pushed the fix/omarchy-skill-hyprland-lua branch from 46e7e3b to e4087fc Compare July 19, 2026 06:00
@ssupt ssupt changed the title Update Omarchy skill for Hyprland Lua configs Update Omarchy skill for Quattro Jul 19, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 .conf terminology (“unbind directive” / “bind”). Since this section now documents the Lua helpers, the checklist should reference calling hl.unbind(...) before o.bind(...) to match the examples and avoid confusion.
3. Inform the user what the key was previously bound to

Comment thread default/omarchy-skill/SKILL.md Outdated
Copilot AI review requested due to automatic review settings July 19, 2026 06:09
@ssupt
ssupt force-pushed the fix/omarchy-skill-hyprland-lua branch from e4087fc to 945ad39 Compare July 19, 2026 06:09

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

Comment thread default/omarchy-skill/SKILL.md Outdated
Comment thread default/omarchy-skill/SKILL.md Outdated
Copilot AI review requested due to automatic review settings July 19, 2026 06:23
@ssupt
ssupt force-pushed the fix/omarchy-skill-hyprland-lua branch from 945ad39 to 5dead48 Compare July 19, 2026 06:23

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

| `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` |
dhh added a commit that referenced this pull request Jul 19, 2026
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>
@dhh

dhh commented Jul 19, 2026

Copy link
Copy Markdown
Member

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: { omarchy = "nautilus" } in the keybinding example would resolve to omarchy-launch-nautilus, which doesn't exist — { launch = "nautilus" } is the right form for launching apps.

@dhh dhh closed this Jul 19, 2026
katrushenkov pushed a commit to katrushenkov/omarchy that referenced this pull request Jul 20, 2026
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>
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.

3 participants