Skip to content

hide openai models codex auth doesn't support; don't use browser for everything#62

Merged
laithrw merged 3 commits into
mainfrom
model-fixes
Jun 5, 2026
Merged

hide openai models codex auth doesn't support; don't use browser for everything#62
laithrw merged 3 commits into
mainfrom
model-fixes

Conversation

@laithrw
Copy link
Copy Markdown
Member

@laithrw laithrw commented Jun 4, 2026

Summary by cubic

Hide unsupported OpenAI models for Codex login, make recommended picks fall back to OpenAI auth when Codex isn’t available, and answer casual questions without starting a browser. This reduces auth errors and skips unnecessary browser sessions.

  • Bug Fixes
    • Codex accounts seed the model search from bundled_codex_login_model_ids(): show only gpt-5.5, gpt-5.4, gpt-5.4-mini; hide gpt-5.5-pro, gpt-5.4-pro, gpt-5.4-nano, gpt-5.3-codex.
    • Recommended quick-picks adapt at runtime via recommended_models_for_codex_availability(): use Codex when logged in; otherwise use OpenAI and open API key auth.
    • OpenRouter now respects API key auth: resolve the key when present, don’t pre-seed its model list, and allow typed raw ids until the key-scoped fetch returns.
    • Python runtime fixes: per-event-loop write lock and safe queue creation to avoid cross-loop write races and “closed loop” queue errors.

Written for commit 994e9bb. Summary will update on new commits.

Review in cubic

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 3 files

Re-trigger cubic

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 issue found across 3 files (changes from recent commits).

Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic

Comment thread crates/browser-use-tui/src/main.rs Outdated
# Conflicts:
#	crates/browser-use-tui/src/main.rs
@laithrw laithrw merged commit 4b63c17 into main Jun 5, 2026
6 checks passed
@laithrw laithrw deleted the model-fixes branch June 5, 2026 14:43
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