Repository navigation
Drop conflicting auth keys from the Codex requesty provider on configure - #34
Merged
Merged
Conversation
devin-ai-integration
Bot
requested review from
lavicosa,
nickc128,
requesty-JohnCosta27 and
requesty-ditti
as code owners
September 28, 2026 15:16
AI CostNo spend detected for branch Updated 2026-09-29T10:36Z · last 30 days · data from Requesty |
Contributor
Author
|
I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".
|
Collaborator
|
Squash your commits. |
devin-ai-integration
Bot
force-pushed
the
devin/1790608551-codex-launch-provider-id
branch
from
September 28, 2026 17:05
638b93e to
89f1efd
Compare
Contributor
Author
|
Squashed into a single commit (89f1efd). |
lavicosa
reviewed
Sep 29, 2026
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
devin-ai-integration
Bot
force-pushed
the
devin/1790608551-codex-launch-provider-id
branch
from
September 29, 2026 10:36
89f1efd to
e6e9a08
Compare
lavicosa
approved these changes
Sep 29, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes this user report on
requesty codex:A hand-written
[model_providers.requesty]can setenv_key = "OPENAI_API_KEY", which is what our Codex docs describe. Ourauth.commandthen lands on that same table and Codex refuses it. Codex also refusesauthcombined withexperimental_bearer_tokenorrequires_openai_auth. This happens both when the-coverrides fromrequesty codexare merged in and whenrequesty configuremerges into the file.[model_providers.requesty]is our own section, soconfigureMergenow deletes those three keys from it after merging:Everything else is still merged, and the file is still backed up first.
Launchis unchanged and writes nothing to disk. A user who only runsrequesty codexwithout re-configuring will still see the error until they run configure, or remove the key by hand.Test added:
TestCodexHarnessConfigureMergeDropsConflictingProviderAuthLink to Devin session: https://app.devin.ai/sessions/c3f712f027814498ab963003b0d22e9a
Open in Devin Desktop: https://app.devin.ai/desktop/session/c3f712f027814498ab963003b0d22e9a?variant=devin