Skip to content

fix(buzz-agent): OPENAI_BASE_URL fallback under OPENAI_COMPAT_BASE_URL - #3773

Closed
Bartok9 wants to merge 1 commit into
block:mainfrom
Bartok9:bartok9/fix-openai-base-url-fallback
Closed

fix(buzz-agent): OPENAI_BASE_URL fallback under OPENAI_COMPAT_BASE_URL#3773
Bartok9 wants to merge 1 commit into
block:mainfrom
Bartok9:bartok9/fix-openai-base-url-fallback

Conversation

@Bartok9

@Bartok9 Bartok9 commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Fixes #3630 (env path). Related collab notes on #3705.

Summary

When OPENAI_COMPAT_BASE_URL is unset/blank, use OPENAI_BASE_URL, then https://api.openai.com/v1.

Precedence

  1. OPENAI_COMPAT_BASE_URL (Buzz-native)
  2. OPENAI_BASE_URL (common toolbox default)
  3. OpenAI default

(Opposite order in #3705 would let a leftover generic env override intentional compat config.)

Test

cargo test -p buzz-agent --lib resolve_openai_compat_base_url_precedence

Note

Desktop Agent defaults may need a separate seam if they never export either env into the agent process; this PR is the agent-side resolution fix.

Buzz-native OPENAI_COMPAT_BASE_URL still wins when set. If it is unset
or blank, fall back to OPENAI_BASE_URL before api.openai.com (block#3630).
Pure helper + precedence unit tests.

Signed-off-by: Bartok9 <danielrpike9@gmail.com>
@Bartok9

Bartok9 commented Jul 30, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by #3705@Chukwuebuka-2003 flipped the or_else precedence (compat-first, then generic) and added the full env-matrix unit tests on that branch. Closing this residual in favor of theirs. Thanks!

@Bartok9 Bartok9 closed this Jul 30, 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.

buzz-agent ignores custom OPENAI_BASE_URL from Agent defaults, always calls api.openai.com

1 participant