Skip to content

fix(designer): Restore Azure OpenAI as agent model source option#9206

Open
Elaina-Lee wants to merge 2 commits into
mainfrom
fix/restore-azureopenai-agent-model-option
Open

fix(designer): Restore Azure OpenAI as agent model source option#9206
Elaina-Lee wants to merge 2 commits into
mainfrom
fix/restore-azureopenai-agent-model-option

Conversation

@Elaina-Lee
Copy link
Copy Markdown
Contributor

Commit Type

  • feature - New functionality
  • fix - Bug fix
  • refactor - Code restructuring without behavior change
  • perf - Performance improvement
  • docs - Documentation update
  • test - Test-related changes
  • chore - Maintenance/tooling

Risk Level

  • Low - Minor changes, limited scope
  • Medium - Moderate changes, some user impact
  • High - Major changes, significant user/system impact

What & Why

PR #9028 ("fix(designer): Default agent connection type to MicrosoftFoundry") removed AzureOpenAI from the agentModelType dropdown options in the agent loop manifest (agentloop.ts) and changed the default to MicrosoftFoundry.

This restores AzureOpenAI ("Azure OpenAI") as a selectable agent model source. The default remains MicrosoftFoundry — only the option was re-added, not the default.

Note: AzureOpenAI was only ever removed from the dropdown. The x-ms-input-dependencies visibility rules and the connection/serialization logic already continue to handle 'AzureOpenAI' (see the existing values: ['AzureOpenAI', 'MicrosoftFoundry'] entries throughout the manifest), so no other changes are needed.

Impact of Change

  • Users: Azure OpenAI is once again available in the "Agent model source" dropdown when configuring an agent. New agents still default to Foundry Models.
  • Developers: No API changes.
  • System: No architecture, performance, or dependency changes.

Test Plan

  • Unit tests added/updated
  • E2E tests added/updated
  • Manual testing completed
  • Tested in: Standalone designer — confirm "Azure OpenAI" appears in the Agent model source dropdown

Contributors

Screenshots/Videos

PR #9028 removed the AzureOpenAI option from the agentModelType dropdown in the agent loop manifest and changed the default to MicrosoftFoundry. This re-adds Azure OpenAI as a selectable option while keeping MicrosoftFoundry as the default.

AzureOpenAI was only removed from the dropdown; the visibility dependencies and serialization logic already continue to handle it.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 21, 2026 21:52
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 21, 2026

🤖 AI PR Validation Report

PR Review Results

Thank you for your submission! Here's detailed feedback on your PR title and body compliance:

PR Title

  • Current: fix(designer): Restore Azure OpenAI as agent model source option
  • Issue: None — the title is specific, descriptive, and matches the change.
  • Recommendation: No change needed.

Commit Type

  • Properly selected (fix).
  • Only one commit type is checked, which is correct.

Risk Level

  • The selected risk (Low) matches the scope of the diff and the label risk:low.
  • No mismatch detected.

What & Why

  • Current: Clear explanation of the regression, why AzureOpenAI needs to be restored, and what remains unchanged.
  • Issue: None.
  • Recommendation: No change needed.

Impact of Change

  • The impact section is appropriately scoped and consistent with the code change.
  • Recommendation:
    • Users: Correctly notes the dropdown option is restored and the default stays Foundry Models.
    • Developers: Correctly states there are no API changes.
    • System: Correctly states there are no architecture/performance/dependency changes.

Test Plan

  • Unit tests were added/updated in the diff, so the test plan passes.
  • Manual testing is also documented, which is fine.

⚠️ Contributors

  • Blank, which is allowed.
  • Recommendation: Optional: add contributors if PM/Design/others helped shape the change.

⚠️ Screenshots/Videos

  • Blank, which is acceptable for this non-visual change.
  • Recommendation: No screenshots required unless this PR is later considered UI-sensitive.

Summary Table

Section Status Recommendation
Title No changes needed
Commit Type No changes needed
Risk Level No changes needed
What & Why No changes needed
Impact of Change No changes needed
Test Plan Unit tests are present in the diff
Contributors ⚠️ Optional to add contributors if applicable
Screenshots/Videos ⚠️ Not required for this change

Overall: this PR passes review for title and body compliance. The advised risk level remains low, matching the submitter's selection and the risk:low label.


Last updated: Thu, 21 May 2026 22:04:12 GMT

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Restores AzureOpenAI (“Azure OpenAI”) as a selectable Agent model source in the Standard agent loop operation manifest, while keeping the default model source as MicrosoftFoundry.

Changes:

  • Re-added AzureOpenAI to the agentModelType dropdown options in the Standard agent loop manifest.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 21, 2026

📊 Coverage Check

No source files changed in this PR.

@Elaina-Lee Elaina-Lee added the risk:low Low risk change with minimal impact label May 21, 2026
…nAI option

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-validated risk:low Low risk change with minimal impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants