test baseten responses endpoint against lingua testcases#327
test baseten responses endpoint against lingua testcases#327Ken Jiang (knjiang) wants to merge 2 commits into
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 31b2635688
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "Codex (@codex) review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "Codex (@codex) address that feedback".
| return executeOpenAIResponses(caseName, payload, { | ||
| ...options, | ||
| baseURL: BASETEN_HOST, | ||
| apiKey: process.env.BASETEN_API_KEY, |
There was a problem hiding this comment.
When BASETEN_API_KEY is unset but OPENAI_API_KEY is set, this passes undefined into executeOpenAIResponses; that helper defaults apiKey to process.env.OPENAI_API_KEY while the base URL is still BASETEN_HOST, so ordinary OpenAI-only capture environments can send an OpenAI key to Baseten and create/overwrite auth-error snapshots. Add an explicit BASETEN_API_KEY check here, like the transform capture path does, instead of falling through to the OpenAI key fallback.
Useful? React with 👍 / 👎.
2 participants
No description provided.