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 c901ca9Oct 4, 2026, 01:35 AMpr/14155Back to current
Back to findings
Known issueKI-MENU-APPS-SORT-LOCALE

The Apps menu lists names that start with an accented letter after Z

FixedFixedLow

This defect has been fixed and verified.

Closure: Fix witnessed

Description

The issue as recorded.

The Apps menu sorts its rows in rebuildDisplay with its own comparator, because DesktopEntries can reorder the apps. On upstream a85e29ab the comparator compares the lowercased names with < and >, that is by UTF-16 code unit. So every name that starts with a letter above U+007A sorts after z. The search sort in the same function uses localeCompare. AppSearch.sortedEntries, which gives the same list to plugins, compares the names in the same way. App names come from the Name of each desktop entry for the user's language. On a French desktop, Écrans and Éditeur de texte show after Zed at the end of the list. A check in the real shell with LANG=fr_FR.UTF-8 on a85e29ab showed this order. On a Russian, Greek or Chinese desktop, the names in that script also show after Z, where the locale puts them first. Every app stays in the list and search still finds it.

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
Correctness
Security classification
Not a security surface
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.

Known issue not reproducedprofileknown_issue_reproducer
Reproducer test
Run it yourself
bash test/shell.d/menu-apps-order-test.sh
Covers (2)
Last run Oct 3, 2026, 6:11 PM

Reproduction steps

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

bash test/shell.d/menu-apps-order-test.sh

What protects you, and the fix

What limits your exposure today, and the planned remediation.

What protects you now

Every app stays in the list, and search finds an app by its name in any order. Only the place of names that start outside a to z is wrong. That covers names with an accented Latin first letter, and on a Russian, Greek or Chinese desktop the names in that script. On an English desktop most names are plain ASCII.

The fix

Fixed by PR #14155, which makes the Apps comparator and AppSearch.sortedEntries order the names with localeCompare (SW-REQ-261003-B7ZA).

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 2 requirements · 3 code files · 21 tests.

Code files (3)
  • Menu.qml
  • MenuModel.js
  • AppSearch.js
Tests (21)
  • menumodel-replay.test.mjs
  • menu-test.sh
  • seq-apps-ascii.events
  • seq-apps-case-tie.events
  • seq-apps-decomposed.events
  • seq-apps-encoding.events
  • seq-apps-first-row.events
  • seq-apps-french-reordered.events
  • and 13 more

Per-requirement evidence

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

Implementing code (5)
  • shell/plugins/menu/Menu.qml
    mergeAppRows
  • shell/plugins/menu/Menu.qml
    mergeProviderRows
  • shell/plugins/menu/Menu.qml
    slugify
  • shell/plugins/menu/MenuModel.js
    mergeAppRows
  • shell/plugins/menu/MenuModel.js
    swapProviderRows
Tests & evidence (2)
  • test/node/menumodel-replay.test.mjs
  • test/shell.d/menu-test.sh
Proof obligations (1)
atomicity

Operation either fully completes or has no effect.

  • ✓negative (required) — test/shell.d/menu-test.sh:921
  • ✓nominal (required) — test/shell.d/menu-test.sh:911
Implementing code (2)
  • shell/plugins/menu/Menu.qml
    rebuildDisplay
  • shell/services/AppSearch.js
    sortedEntries
Tests & evidence (19)
  • test/bdiff/corpus/lifecycle/seq-apps-ascii.events
  • test/bdiff/corpus/lifecycle/seq-apps-case-tie.events
  • test/bdiff/corpus/lifecycle/seq-apps-decomposed.events
  • test/bdiff/corpus/lifecycle/seq-apps-encoding.events
  • test/bdiff/corpus/lifecycle/seq-apps-first-row.events
  • test/bdiff/corpus/lifecycle/seq-apps-french-reordered.events
  • test/bdiff/corpus/lifecycle/seq-apps-french.events
  • test/bdiff/corpus/lifecycle/seq-apps-german.events
  • test/bdiff/corpus/lifecycle/seq-apps-locale-el.events
  • test/bdiff/corpus/lifecycle/seq-apps-locale-ru.events
  • test/bdiff/corpus/lifecycle/seq-apps-locale-zh.events
  • test/bdiff/corpus/lifecycle/seq-apps-nonlatin.events
  • test/bdiff/corpus/lifecycle/seq-apps-plugin-list.events
  • test/bdiff/corpus/lifecycle/seq-apps-plugin-tie.events
  • test/bdiff/corpus/lifecycle/seq-apps-root-control.events
  • test/bdiff/corpus/lifecycle/seq-apps-symbols.events
  • test/bdiff/harness.mjs
  • test/shell.d/menu-apps-order-bdiff-test.sh
  • test/shell.d/menu-apps-order-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-apps-sort-locale.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