Skip to content

Updated documentation and .env.example to include model limitations and API versatility#38

Merged
jack-williams merged 2 commits intomicrosoft:fix/openai-client-configfrom
based-jace:fix/openai-client-config
May 4, 2026
Merged

Updated documentation and .env.example to include model limitations and API versatility#38
jack-williams merged 2 commits intomicrosoft:fix/openai-client-configfrom
based-jace:fix/openai-client-config

Conversation

@based-jace
Copy link
Copy Markdown
Contributor

No description provided.

Jace Medlin (AIM Consulting Addison Group) added 2 commits April 30, 2026 16:44
@jack-williams jack-williams merged commit 0e4bfc1 into microsoft:fix/openai-client-config May 4, 2026
2 checks passed
jack-williams added a commit that referenced this pull request May 4, 2026
)

* Use AzureOpenAI client and clean up env config in chat & image apps

- Switch to the SDK's AzureOpenAI client when VITE_OPENAI_BASE_URL is set so
  Azure auth uses the supported single api-key header path with api-version,
  instead of the base OpenAI client with a manually-injected baseURL.
- Drop the dead process.env.OPENAI_* fallbacks. vite.config.ts defines
  process.env as {}, so those reads were inlined as undefined at build time
  and never did anything.
- Read VITE_OPENAI_API_VERSION (new) and use it when constructing the Azure
  client. Update vite-env.d.ts on both apps with the new optional types.
- Default chat model is now gpt-4.1 to match the rest of the codebase.
- Mirror the .env.example layout across both apps and comment out the optional
  vars so copying .env.example does not silently activate the Azure path with
  a placeholder URL.
- Document the optional VITE_OPENAI_* configuration in the README.

Co-authored-by: Jace Medlin <32235747+based-jace@users.noreply.github.com>

* Updated documentation and .env.example to include model limitations and API versatility (#38)

* Updated documentation and .env.example to include model limitations and API versatility

* Ran prettier on README

---------

Co-authored-by: Jace Medlin (AIM Consulting Addison Group) <v-jacemedlin@gmail.com>

---------

Co-authored-by: Jack Williams <jack.williams@microsoft.com>
Co-authored-by: Jace Medlin <32235747+based-jace@users.noreply.github.com>
Co-authored-by: Jace Medlin (AIM Consulting Addison Group) <v-jacemedlin@gmail.com>
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