Use Azure Open AI model in NodeJS OpenAI Sample #181
Merged
Conversation
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
- Add openai-config.ts for conditional OpenAI/Azure OpenAI configuration - Use setDefaultOpenAIClient with AzureOpenAI for Azure endpoints - Use chat_completions API for Azure OpenAI compatibility - Update .env.template with Azure OpenAI environment variables - Dynamic model/deployment name selection based on provider
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the NodeJS OpenAI sample agent to support using Azure OpenAI deployments (in addition to standard OpenAI) via environment-based configuration.
Changes:
- Added an OpenAI/Azure OpenAI configuration module and wired it into agent creation (including model/deployment selection).
- Adjusted environment loading and hosting behavior (production detection, health endpoint, bind host).
- Expanded the
.env.templateto document Azure OpenAI settings.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| nodejs/openai/sample-agent/src/openai-config.ts | New module to configure @openai/agents for Azure OpenAI vs standard OpenAI based on env vars. |
| nodejs/openai/sample-agent/src/client.ts | Initializes OpenAI/Azure config and passes the selected model/deployment into the Agent. |
| nodejs/openai/sample-agent/src/agent.ts | Loads .env earlier to ensure env vars are available during agent initialization. |
| nodejs/openai/sample-agent/src/index.ts | Updates production/development detection, adds /api/health, and binds server to 0.0.0.0. |
| nodejs/openai/sample-agent/.env.template | Adds Azure OpenAI configuration variables and model selection settings. |
- Add 'openai' as direct dependency in package.json - Check AZURE_OPENAI_DEPLOYMENT in isAzureOpenAI() function - Throw clear error if AZURE_OPENAI_DEPLOYMENT is missing - Remove appId from health endpoint to reduce information disclosure
- Remove direct openai dependency (use transitive from @openai/agents) - Use require() and 'any' type to bypass version compatibility issues
pontemonti
reviewed
Jan 23, 2026
- Remove isProduction variable, use only isDevelopment - Auth is required for everything except NODE_ENV=development - Addresses pontemonti's review comment about ambiguous state
JesuTerraz
approved these changes
Jan 23, 2026
pontemonti
approved these changes
Jan 23, 2026
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Uh oh!
There was an error while loading. Please reload this page.