Skip to content

fix(menubar): order tray % metrics by providers_order#92

Merged
prakersh merged 1 commit into
onllm-dev:mainfrom
yesme:fix/menubar-tray-order-follows-providers-order
Jul 23, 2026
Merged

fix(menubar): order tray % metrics by providers_order#92
prakersh merged 1 commit into
onllm-dev:mainfrom
yesme:fix/menubar-tray-order-follows-providers-order

Conversation

@yesme

@yesme yesme commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Summary

macOS menubar tray percentages ignored the configured provider list order and followed selected_quotas click order instead.

Repro

  • Menubar provider order: Codex → Anthropic → Grok
  • Selected tray quotas (click order): Codex, Grok, Anthropic
  • Shown: 17%·15%·7% (Codex·Grok·Anthropic)
  • Expected: 17%·7%·15% (Codex·Anthropic·Grok)

Root cause

TrayTitle walks status_display.selected_quotas in storage order. That array is filled when the user toggles quotas and is not kept in sync with providers_order.

Fix

During Settings.Normalize(), re-sort selected_quotas by providers_order when mode is multi_provider. Every consumer of normalized settings (tray title, preferences API) then matches the menubar list order.

Test plan

  • go test ./internal/menubar -run 'TrayTitleFollows|OrderSelections'
  • Branch based on onllm-dev/onWatch@main, single commit only
  • With live menubar: set order Codex / Anthropic / Grok and confirm tray reads left-to-right as that order

Related

User-facing menubar provider ordering settings; companion of tray multi-provider status display (selected_quotas, max 3).

Tray title used selected_quotas click order, so configuring provider
list as Codex → Anthropic → Grok still showed 17%·15%·7% (Codex·Grok·Anthropic).

Re-sort multi_provider selected_quotas by providers_order during
Settings.Normalize so tray title matches the menubar list order.

Co-authored-by: Grok 4.5 (high effort) <302482056+a-grok-build-bot[bot]@users.noreply.github.com>
@prakersh
prakersh merged commit 4e0012b into onllm-dev:main Jul 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants