Proof Portal
Omarchy
ProbeLabsviewing a historical runA proof layer — requirements, tests and verified fixes — for two of Omarchy's subsystems: the application menu (launcher scripts, QML model, JSONC config, search and selection) and the lock screen (lock scripts, QML, PAM authentication). Scope is deliberately limited to those components of omacom/omarchy; the rest of the distribution is not covered.
A summon over an open uninstall confirmation leaves it up, and Enter meant for the new prompt uninstalls the app
This defect has been fixed and verified.
Description
The issue as recorded.
Delete on an app row opens the uninstall confirmation. Uninstall is the preselected button (shell/Ui/ConfirmDialog.qml selectedIndex 1). Only onOpenedChanged clears the confirmation state, when the menu closes. A summon goes through open() into openDmenu, or into openRoute and openExistingMenu. These replace the rows while opened stays true, and they never clear deleteConfirmOpen or deleteTarget.
The dialog stays on top of the new rows and still takes every key. Enter then runs confirmDelete. It cancels the summoned request, so an omarchy-menu-select caller exits 1, and it removes the app. A live run on upstream 821ae589 (headless sway and Quickshell) confirmed it. That run lost the desktop file of the app for a select prompt, an input prompt and a route summon.
Affected requirements
The requirement(s) this issue violates — click through to the spec.
Severity, explained
Why this is rated the way it is — and the scoring signals behind the rating (each ⓘ explains the term).
- why this rating
- Risk-rated
- risk area
- Data Integrity
- CVE surface
- None
How it's proven
The reproducer — an actual test that drives the real code and shows the issue happening. Run it yourself, or peek the test and the source it covers.
bash test/shell.d/menu-uninstall-confirm-test.sh
Reproduction steps
Technical steps for your engineers to confirm the issue by hand.
bash test/shell.d/menu-uninstall-confirm-test.sh
What protects you, and the fix
What limits your exposure today, and the planned remediation.
What protects you now
Only a summon that arrives while the confirmation shows triggers it, for example an omarchy-menu-select caller such as the Super+K keybindings list. Escape or Cancel on the confirmation before the summon avoids it. Menu keybinds that toggle close the menu, and that clears the confirmation.
The fix
Fixed by the upstream change that makes a summon cancel a pending confirmation in open() (SW-REQ-261002-VJR1); mirror pr-A.
Blast radius
If you touch this issue, what else may need re-checking — the requirements it affects and the code and tests that hang off them. Historical view: authored trace links only — automatically derived links aren't reconstructible for past runs.
Touch this finding and you re-check 2 requirements · 1 code files · 9 tests.
- Menu.qml
- menu-compositor-test.sh
- seq-delete-confirm-cancel-button.events
- seq-delete-confirm-enter.events
- seq-delete-confirm-escape.events
- seq-delete-confirm-then-input-enter.events
- seq-delete-confirm-then-menu-enter.events
- seq-delete-confirm-then-select-enter.events
- harness.mjs
- and 1 more
Per-requirement evidence
For each requirement this finding touches: the implementing code, verifying tests, and proof obligations that discharge it.
Per-requirement evidence
For each requirement this finding touches: the implementing code, verifying tests, and proof obligations that discharge it.
Evidence trail
The raw evidence manifests behind this finding — superseded by the resolved reproducer above, kept here for traceability.
Evidence trail
The raw evidence manifests behind this finding — superseded by the resolved reproducer above, kept here for traceability.
- proof/evidence/ki-menu-open-leaves-confirm.yaml
Change history
Every recorded revision of this finding's source file — when it was added, edited, or re-classified, with the diff for each change.
Discussions
Discuss this with the proof team. Nothing changes in your audit automatically — you open a request and a staff member records any outcome inside the thread.