Stop adding a HEY row to the Omarchy menu - #509
Conversation
The Omarchy menu belongs to the user and Omarchy. hey setup omarchy no longer writes a row into ~/.config/omarchy/extensions/omarchy-menu.jsonc, and it takes out the marker block earlier releases added, leaving the rest of the file untouched. --remove still strips it too. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
There was a problem hiding this comment.
Platform pattern review found no security or privacy issues in the changed lines. Setup no longer writes the Omarchy menu and only removes a prior marker-delimited block; that is a local config cleanup, not an unauthenticated route, identity, TLS, logging, or HTML/CSP change.
Sent by Cursor Security Agent: Security Reviewer
There was a problem hiding this comment.
Copilot review overview
🟢 Approved
The focused implementation matches the stated behavior and includes appropriate regression coverage.
Review effort: Balanced
Findings: None
What changed in this PR
Stops Omarchy setup from adding a HEY menu row while removing legacy managed rows safely.
Changes:
- Removes menu-row installation logic.
- Cleans up legacy managed menu blocks.
- Updates tests, help text, wizard output, and documentation.
[!TIP]
If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or rungh pr ready --undo.
Click "Ready for review" or rungh pr readyto reengage.
| File | Description |
|---|---|
internal/cmd/setup.go |
Updates the wizard summary. |
internal/cmd/omarchy.go |
Removes menu installation and performs legacy cleanup. |
internal/cmd/omarchy_test.go |
Tests menu preservation and legacy removal. |
docs/omarchy.md |
Documents the revised integration behavior. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.


hey setup omarchy(and the fullhey setupwizard) wrote a marker-delimitedHEYrow into~/.config/omarchy/extensions/omarchy-menu.jsonc. The Omarchy menu belongs to the user and Omarchy, so hey shouldn't add to it.--removestill strips the block.docs/omarchy.mdno longer mention a menu row.Tests
removed, a second run says nothing about the menu, and--removeon the clean file reportsabsent.go test ./...passes.🤖 Generated with Claude Code