Skip to content

fix(chatmodels): add newer Azure OpenAI models#6414

Open
pragnyanramtha wants to merge 2 commits into
FlowiseAI:mainfrom
pragnyanramtha:pragnyan/fix-azure-chatopenai-newer-models-6411
Open

fix(chatmodels): add newer Azure OpenAI models#6414
pragnyanramtha wants to merge 2 commits into
FlowiseAI:mainfrom
pragnyanramtha:pragnyan/fix-azure-chatopenai-newer-models-6411

Conversation

@pragnyanramtha
Copy link
Copy Markdown

Summary

  • add the newer GPT-5.4 and GPT-5.5 model options to the Azure ChatOpenAI registry
  • keep Azure ChatOpenAI costs aligned with the existing ChatOpenAI entries
  • add a focused registry test so these newer GPT-5.x options do not drift again

Closes #6411.

Validation

  • node -e "const m=require('./packages/components/models.json'); const names=['gpt-5.5','gpt-5.5-pro','gpt-5.4','gpt-5.4-pro','gpt-5.4-mini','gpt-5.4-nano']; const azure=m.chat.find(p=>p.name==='azureChatOpenAI').models; const open=m.chat.find(p=>p.name==='chatOpenAI').models; for (const n of names) { const a=azure.find(x=>x.name===n); const o=open.find(x=>x.name===n); if (JSON.stringify(a)!==JSON.stringify(o)) throw new Error(n); } console.log('azure model parity ok')"
  • git diff --check
  • pnpm exec prettier --check packages/components/models.json packages/components/src/modelRegistry.test.ts
  • NODE_OPTIONS=--max-old-space-size=4096 pnpm --filter flowise-components test -- modelRegistry.test.ts --runInBand

Note: the first focused Jest attempt hit the default Node heap limit locally; rerunning the same focused suite with a 4096 MB heap completed successfully.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds several GPT-5.x models, including gpt-5.5 and gpt-5.4 variants, to the Azure ChatOpenAI provider within the models configuration. Additionally, a new test suite has been introduced to ensure that these Azure models are correctly aligned with the standard ChatOpenAI provider entries. I have no feedback to provide.

@pragnyanramtha pragnyanramtha marked this pull request as ready for review May 20, 2026 17:50
Copilot AI review requested due to automatic review settings May 20, 2026 17:50
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

Note

Copilot was unable to run its full agentic suite in this review.

Adds newer GPT-5.x model entries to the azureChatOpenAI provider and introduces a regression test to ensure Azure’s registry stays aligned with the chatOpenAI model definitions.

Changes:

  • Add GPT-5.5 / GPT-5.4 variants to azureChatOpenAI in models.json
  • Add a test that checks GPT-5.x Azure entries match corresponding chatOpenAI entries

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
packages/components/src/modelRegistry.test.ts Adds a Jest test asserting Azure ChatOpenAI GPT-5.x models match ChatOpenAI definitions
packages/components/models.json Adds GPT-5.5 and GPT-5.4* model entries to the azureChatOpenAI provider

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/components/src/modelRegistry.test.ts Outdated
Comment thread packages/components/src/modelRegistry.test.ts Outdated
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.

Azure ChatOpenAI Tool does not support models latest openAI models like 5.4-mini and nano (BLOCKER due to model retirements by Azure)

2 participants