Skip to content

fix(tui): correct codex onboarding command#127

Merged
BunsDev merged 1 commit into
mainfrom
fix/onboarding-codex-login-copy
Jul 4, 2026
Merged

fix(tui): correct codex onboarding command#127
BunsDev merged 1 commit into
mainfrom
fix/onboarding-codex-login-copy

Conversation

@BunsDev

@BunsDev BunsDev commented Jul 4, 2026

Copy link
Copy Markdown
Member

Summary

  • Correct the Codex first-run onboarding command to the supported coven-code codex login surface.
  • Keeps the visible onboarding copy aligned with the current CLI auth commands.

Test Plan

  • CARGO_TARGET_DIR=/Users/buns/Documents/GitHub/OpenCoven/coven-code/src-rust/target cargo test --package claurst-tui --lib provider_setup_ -- --nocapture
  • CARGO_TARGET_DIR=/Users/buns/Documents/GitHub/OpenCoven/coven-code/src-rust/target cargo test --package claurst-tui --lib provider_picker_prioritizes -- --nocapture
  • CARGO_TARGET_DIR=/Users/buns/Documents/GitHub/OpenCoven/coven-code/src-rust/target cargo check --workspace
  • CARGO_TARGET_DIR=/Users/buns/Documents/GitHub/OpenCoven/coven-code/src-rust/target cargo build --package claurst --bin coven-code
  • tmux smoke with isolated HOME/COVEN_HOME confirmed the first-run provider overlay shows coven-code codex login.

Copilot AI review requested due to automatic review settings July 4, 2026 11:58
@vercel

vercel Bot commented Jul 4, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview Jul 4, 2026 11:58am

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the TUI onboarding provider-setup overlay so the Codex first-run instructions match the currently supported CLI authentication surface.

Changes:

  • Replace the Codex onboarding setup command from coven-code auth login --provider codex to coven-code codex login.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 166 to 170
ProviderEntry {
name: "Codex",
tagline: " gpt-5.2-codex via ChatGPT login",
setup: "coven-code auth login --provider codex",
setup: "coven-code codex login",
setup_suffix: "",
@BunsDev BunsDev merged commit 17472f5 into main Jul 4, 2026
3 checks passed
@BunsDev BunsDev deleted the fix/onboarding-codex-login-copy branch July 4, 2026 12:31
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