Skip to content

chore: switch org with bt asynchronously#75

Merged
Cedric / ViaDézo1er (viadezo1er) merged 2 commits into
mainfrom
cedric/switch-auth-in-background
Jun 12, 2026
Merged

chore: switch org with bt asynchronously#75
Cedric / ViaDézo1er (viadezo1er) merged 2 commits into
mainfrom
cedric/switch-auth-in-background

Conversation

@viadezo1er

@viadezo1er Cedric / ViaDézo1er (viadezo1er) commented Jun 6, 2026

Copy link
Copy Markdown
Collaborator

Skips 0.5s of waiting time (average over 50 runs) by launching bt switch/auth login in the background.
Still makes the user wait a bit if bt switch/auth login is slower than the claude/codex smoke test, which does not happen in practice.

earns a few seconds of waiting time
makes the user wait if it's slower than the claude/codex smoke test

@lforst Luca Forstner (lforst) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

making the world a better place one micro optimization at a time.

I still wouldn't remove the .env() call though!

return yargs([])
.scriptName("braintrust-setup")
.usage("$0 [options]")
.env("BRAINTRUST")

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Intentional? I added this.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Oops
Fixed in 46995a6

@viadezo1er Cedric / ViaDézo1er (viadezo1er) merged commit e4b3df9 into main Jun 12, 2026
13 checks passed
@viadezo1er Cedric / ViaDézo1er (viadezo1er) deleted the cedric/switch-auth-in-background branch June 12, 2026 23:55
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