Skip to content

Drop conflicting auth keys from the Codex requesty provider on configure - #34

Merged
lavicosa merged 1 commit into
mainfrom
devin/1790608551-codex-launch-provider-id
Sep 29, 2026
Merged

lavicosa merged 1 commit into
mainfrom
devin/1790608551-codex-launch-provider-id

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Fixes this user report on requesty codex:

Error loading config.toml: model_providers.requesty: provider auth cannot be combined with env_key

A hand-written [model_providers.requesty] can set env_key = "OPENAI_API_KEY", which is what our Codex docs describe. Our auth.command then lands on that same table and Codex refuses it. Codex also refuses auth combined with experimental_bearer_token or requires_openai_auth. This happens both when the -c overrides from requesty codex are merged in and when requesty configure merges into the file.

[model_providers.requesty] is our own section, so configureMerge now deletes those three keys from it after merging:

delete(config["model_providers"]["requesty"], "env_key" | "experimental_bearer_token" | "requires_openai_auth")

Everything else is still merged, and the file is still backed up first.

Launch is unchanged and writes nothing to disk. A user who only runs requesty codex without re-configuring will still see the error until they run configure, or remove the key by hand.

Test added: TestCodexHarnessConfigureMergeDropsConflictingProviderAuth

Link to Devin session: https://app.devin.ai/sessions/c3f712f027814498ab963003b0d22e9a
Open in Devin Desktop: https://app.devin.ai/desktop/session/c3f712f027814498ab963003b0d22e9a?variant=devin

@github-actions

github-actions Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

AI Cost

No spend detected for branch devin/1790608551-codex-launch-provider-id in the last 30 days.

Updated 2026-09-29T10:36Z · last 30 days · data from Requesty

@github-actions github-actions Bot added the cost:$0.00 AI cost for this branch label Sep 28, 2026
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@devin-ai-integration devin-ai-integration Bot changed the title Launch Codex under its own provider id and drop auth conflicts on configure Drop conflicting auth keys from the Codex requesty provider on configure Sep 28, 2026
@lavicosa

Copy link
Copy Markdown
Collaborator

Squash your commits.

@devin-ai-integration
devin-ai-integration Bot force-pushed the devin/1790608551-codex-launch-provider-id branch from 638b93e to 89f1efd Compare September 28, 2026 17:05
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

Squashed into a single commit (89f1efd).

Comment thread internal/harnesses/codex.go
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration
devin-ai-integration Bot force-pushed the devin/1790608551-codex-launch-provider-id branch from 89f1efd to e6e9a08 Compare September 29, 2026 10:36
@lavicosa
lavicosa merged commit bbb6e50 into main Sep 29, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cost:$0.00 AI cost for this branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant