Skip to content

fix: detect current Codex settings sidebar#46

Draft
CreepsoOff wants to merge 1 commit into
b-nnett:mainfrom
CreepsoOff:fix/settings-sidebar-current-codex
Draft

fix: detect current Codex settings sidebar#46
CreepsoOff wants to merge 1 commit into
b-nnett:mainfrom
CreepsoOff:fix/settings-sidebar-current-codex

Conversation

@CreepsoOff

Copy link
Copy Markdown
Contributor

Summary

Fixes Codex++ Settings sidebar injection on the current Codex Desktop Settings UI.

The previous detection logic depended on older Settings sidebar assumptions, especially the presence of Back to app. Current Codex Settings no longer always exposes that label, and the available Settings entries can vary by platform, region, and feature availability.

This updates settings-injector.ts to detect the Settings sidebar by scoring known navigation labels instead of requiring Back to app.

Settings variants

The detection is intentionally score-based rather than an exact full Settings list. Some entries can vary, for example Computer use / Computer Use may appear for some users but not others.

The matcher normalizes case, spacing, and diacritics, and only requires:

  • at least 2 core labels: General, Appearance, Configuration, Personalization
  • at least 3 total known labels, including optional labels such as MCP servers, Git, Usage, Browser use, Computer use, Worktrees, Skills, Plugins, Connections, etc.

Tested

Tested locally on Windows with:

OpenAI.Codex_26.429.8261.0_x64__2p2nqsd0c76g0

Build command:

npm run build --workspace @codex-plusplus/runtime

Result:

The Settings sidebar now correctly shows Codex++ / Config / Tweaks / Tweak Store.

Successful logs:

[settings-injector] settings surface {"visible":true,"reason":"sidebar-found"}
[settings-injector] nav group injected {"outerTag":"NAV"}
[settings-injector] pages group synced {"count":3}

Known follow-up

The injector can still emit noisy sidebar not found logs while Settings is not mounted or during remounts. This PR focuses on restoring functional sidebar detection.

shaneholloman pushed a commit to shaneholloman/codex-plusplus that referenced this pull request May 14, 2026
Includes PR b-nnett#46 settings sidebar detection fix.

Co-authored-by: CreepsoOff <51055703+CrepsoOff@users.noreply.github.com>
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.

1 participant