Proof Portal

Project overview

Omarchy

ProbeLabsviewing a historical run

A 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.

Viewing historical run 67b0e44Oct 1, 2026, 12:54 PMquattro-proofBack to current
Back to findings
Known issueKI-MENU-OPEN-QUADRATIC

Opening a select menu walks its rows quadratically (omacom/omarchy#10601)

OpenOpenLow

Low-severity issue, currently open.

Description

The issue as recorded.

rebuildDmenuDisplay (shell/plugins/menu/Menu.qml) appends rows to displayModel one at a time. The visibleRowsHeight binding evaluates again after each append. Each pass calls foldedListHeight over the full row list. A menu of N rows thus costs about N(N+1)/2 row visits. 231 rows take 232 height passes and 27,027 row visits. 462 rows take 463 passes and 107,415 visits. The Super+K keybindings menu is the realistic large case. Upstream issue: omacom/omarchy#10601. Report intake validated it (report cmulr8ysi0jwr1gw4fjmt9khq). Upstream PR 10631 does not change the counts.

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).

LowRated severity — the impact if this issue is exploited or hit.
why this rating
Risk-rated
risk area
Performance
Security classification
Not a security surface
CVE surface
None

Where it is

The code the issue lives in — peek the affected function inline to see it in context.

  • Menu.qml rebuildDmenuDisplay + visibleRowsHeight binding (omarchy-menu-select summon)

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.

Known issue reproducedprofileknown_issue_reproducerprovesSW-REQ-260922-B757
Reproducer test
Run it yourself
sh test/reports/report-cmulr8ysi0jwr1gw4fjmt9khq.sh
Covers (2)
Last run Oct 1, 2026, 11:52 AM

Reproduction steps

Technical steps for your engineers to confirm the issue by hand.

sh test/reports/report-cmulr8ysi0jwr1gw4fjmt9khq.sh

What protects you, and the fix

What limits your exposure today, and the planned remediation.

What protects you now

Menus of typical size (tens of rows) open without a visible delay. The cost grows only for very long select lists such as the keybindings menu

The fix

Append the rows in one model update, or defer the height binding until the rebuild ends. Also add a requirement that bounds the menu-open cost; none exists today

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.

Tracing blast radius…

Touch this finding and you re-check 1 requirements · 1 code files · 1 tests.

Requirements (1)
Code files (1)
  • Menu.qml
Tests (1)
  • menu-compositor-test.sh

Per-requirement evidence

For each requirement this finding touches: the implementing code, verifying tests, and proof obligations that discharge it.

Implementing code (2)
  • shell/plugins/menu/Menu.qml
    dmenuRowListHeight
  • shell/plugins/menu/Menu.qml
    runAction
Tests & evidence (1)
  • test/shell.d/menu-compositor-test.sh

Evidence trail

The raw evidence manifests behind this finding — superseded by the resolved reproducer above, kept here for traceability.

  • proof/evidence/ki-menu-open-quadratic.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.

Sign in to discuss this with the proof team.Sign in