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 5d4fb9aOct 2, 2026, 11:50 AMquattro-proofBack to current
All requirements
RequirementSW-REQ-260922-EFNRSoftwareReview

swapProviderRows drops exactly the rows carrying the re-run provider's providerMenu and appends the new batch, leaving static children and other providers' rows untouched.

This requirement changed after its last recorded review, so approval is stale. Automated checks pass.
PriorityshallTypeguaranteeCategoryfunctionalComponentmenuAssuranceCFindingsLowworst open

Specification

The requirement exactly as authored — its complete prose text and, where present, the formal FRETish sentence it compiles to.

Description

swapProviderRows drops exactly the rows carrying the re-run provider's providerMenu and appends the new batch, leaving static children and other providers' rows untouched.

FRETish formula
when provider_reran the menu_model shall eventually satisfy previous_batch_replaced
View full formal model

Rationale & tags

Why this requirement exists, and how it is categorised.

MenuModel.js swapProviderRows lines 140-165; a just-disabled plugin disappears from the Enable list on the next run.

Verification & provenance

How this requirement was checked: the review trail, edit history, and the machine-analysis status terms (each ⓘ explains what it means).

Assurance levelC
Formalizationvalid
Realizabilityrealizable
Vacuitychecked_ok
Strategyfretish

Review

Status
in_review
Reviewer
Kimi Zero Warnings · AI agent
Reviewed
Sep 27, 2026, 21:25 UTC

History

Created
Sep 22, 2026, 13:34 UTC · Kimi Dogfood · AI agent
Modified
Sep 27, 2026, 21:25 UTC · Kimi Zero Warnings · AI agent

Hazard review

Reviewed Oct 1, 2026, 21:26 UTCby agent:claude-baseline-passcatalog v1.11.0
  • scenarioreviewedboundaryedge_caseconcurrentconcurrency_scale

    Worst case: a provider rerun leaves its previous batch behind - a just-disabled plugin still listed - or the swap erases another provider's rows or the menu's static children; the providerMenu===menuId filter drops exactly the re-run provider's rows and the fresh-map rebuild copies everything else verbatim, so static children and other providers' rows cannot be touched by construction. boundary: an empty incoming batch (provider now yields nothing) drops the previous batch and appends nothing - the subject's core edge; a row colliding with a static child's id is silently dropped in favor of the static row, the deliberate trust precedence. edge_case: duplicate ids within one batch keep the first occurrence, rows without an id are dropped by the guard, and stamping providerMenu/order onto incoming rows aliases the caller's objects - model-owned mutation, safe because the caller does not reuse the batch. concurrent: the caller serializes provider applications through the requestSerial/applySerial discipline (a stale batch cannot displace a newer one at this layer), which this function relies on rather than re-implements - recorded as the composition boundary. Catalog 1.11.0 re-review: input_domain not applicable, provider rows reach it already parsed. concurrency_scale applied, providers re-run while an earlier batch is still in the model; each swap replaces exactly that provider rows.

  • propertyreviewedidempotencyatomicity

    Swapping the same batch twice yields the same model (the second run drops the first's rows and re-appends identical ones), and the swap is wholesale: the function builds a complete fresh items/order pair and returns it, so the caller installs either the old model or the new one - never a half-swapped menu with some old provider rows and some new.

  • structuralnot applicable

    Array filters and a fresh plain-object map; no memory, pointer, encoding, or numeric surface. The plain-object-as-map exposure to protocol-string ids is the same same-user config hazard dispositioned under 7NPE's malformed_input row and is not re-graded here.

  • domainnot applicable

    In-process model rebuild; providers deliver rows through the shell's own plugin channel owned by the provider-loading requirements, and nothing here touches files, network, crypto, or privilege.

Change history

Every recorded revision of this requirement's source file — newest first, each with its commit message and the diff for that change.

Review history

Human and AI-agent approvals of this requirement — the 'why was this approved' lineage, each with the reviewer's justification and the code it cites.

  1. Kimi Zero Warnings · AI agentApprovedSpec conformanceSep 27, 2026 · 5 days agoREVIEW-49

    Read swapProviderRows. The first pass drops exactly the rows whose providerMenu equals the re-run menu id, keeping static children and other providers' rows in order; the second pass appends the new batch, stamping providerMenu and order, skipping ids already present. The provider_reran condition models the swap invocation; previous_batch_replaced is the drop-then-append pair. Formula conforms to the code.

    Cited code (1)

Open known issues

Findings currently open against this requirement — issues its verification surfaced that are not resolved yet. Each links to the full finding.

Obligations

What this requirement must witness to be considered satisfied — the required evidence, and the tests that discharge each one.

The tests that discharge each obligation aren't available for a historical run

The evidence matrix behind each obligation comes from the live audit index, which can't be rebuilt for a past commit. Nothing here means unknown — not that the requirement has no obligations.

Formula evidence

The formal formula behind this requirement, the variables it is written over, and the tests that exercise it (each term is explained inline).

FRETish formula

when provider_reran the menu_model shall eventually satisfy previous_batch_replaced

Witnesses· 2 scenarios total

  • menu-test.sh:1
    exercises 2 condition scenarios

MC/DC truth table· 3 rows

Each row assigns the formula's conditions (T/F) and shows the Result— the formula's value for that input row, not a test pass/fail. A row proves a condition when flipping only that condition flips the outcome. The test that covers each row is linked.

Covereda test exercises this rowExempteda reviewed mcdc:ignoreNo-actionfalse-result row satisfied by designUncoveredneeds a covering test
#previous_batch_replacedprovider_reranResultProvesCovering test
1FFTprovider_reran
2FTFprevious_batch_replacedExempted · defensive — the swap drops every row tagged with the rerunning provider's menu id unconditionally; keeping the previous batch needs that filter removed (reviewed: REVIEW-M8)
3TTTprevious_batch_replaced

Its place

How this requirement connects — what proves it, what it affects, and what it rests on. Authored links only here; automatically derived links come from the audit index.

Loading graph…

Trace evidence

The concrete artifacts linked to this requirement — implementing code, verifying tests, documents, and the findings raised against it.

Impact

Blast radius — authored trace links only (automatically derived links come from the audit index and aren't shown here).

If you change this

Requirements
0
Files
2
Tests
2
At-risk contracts
0

Files to re-check (2)

  • Menu.qmlshell/plugins/menu/Menu.qml
  • MenuModel.jsshell/plugins/menu/MenuModel.js

Tests to re-run (2)

  • menumodel-replay.test.mjstest/node/menumodel-replay.test.mjs
  • menu-test.shtest/shell.d/menu-test.sh

What this rests on

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