Skip to content

Enabling Azure Foundry model deployments for codex-cli #1381

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

avberaMS2
Copy link

We can use any api-key based foundry model deployments to run codex-cli

Verified against o3, o4-mini, o3-mini azure foundry deployments

Use codex-cli/test_azure.bat to run codex-cli

We can use any api-key based foundry model deployments to run codex-cli

Verified against o3, o4-mini, o3-mini azure foundry deployments

Use codex-cli/test_azure.bat to run codex-cli
Copy link

github-actions bot commented Jun 25, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@avberaMS2
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

github-actions bot added a commit that referenced this pull request Jun 25, 2025
Copy link
Collaborator

@bolinfest bolinfest left a comment

Choose a reason for hiding this comment

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

Now that we have moved to the Rust CLI for releases, we are not considering major changes to the TypeScript codebase.

For the Rust CLI, we are not baking in third-party providers like we were in the TypeScript CLI. If providers need special behavior, it must be done via configuration.

That said, I believe I added a comparable feature for the openai provider in #1487. So if you put together a PR that added a config option like base_url_env_var that takes a string to indicate an environment variable such that:

  • if the corresponding env var exists and is non-empty, then its value is used as base_url
  • otherwise it falls back to base_url

If base_url is empty, then the request fails.

This would also require updating the docs in config.md.

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