Skip to content

feat(OpenAI Chat Model Node): Update default model to gpt-4.1-mini #15975

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

Merged
merged 2 commits into from
Jun 5, 2025

Conversation

burivuhster
Copy link
Contributor

@burivuhster burivuhster commented Jun 3, 2025

Summary

Make gpt-4.1-mini model selected by default in OpenAI Chat Model Node.

Related Linear tickets, Github issues, and Community forum posts

Review / Merge checklist

  • PR title and summary are descriptive. (conventions)
  • Docs updated or follow-up ticket created.
  • Tests included.
  • PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

cubic found 1 issue across 1 file. Review it in cubic.dev

React with 👍 or 👎 to teach cubic. Tag @cubic-dev-ai to give specific feedback.

Copy link

codecov bot commented Jun 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

@burivuhster burivuhster changed the title feat(OpenAI Chat Model Node): update default model to gpt-4.1-mini feat(OpenAI Chat Model Node): Update default model to gpt-4.1-mini Jun 3, 2025
@n8n-assistant n8n-assistant bot added the n8n team Authored by the n8n team label Jun 3, 2025
Copy link
Contributor

@jeanpaul jeanpaul left a comment

Choose a reason for hiding this comment

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

Should we not also change the defaults for OpenRouter and the OpenAI node too?

@burivuhster
Copy link
Contributor Author

OpenAI nodes do not have a default, and for the OpenRouter node I changed it as well 👌

@burivuhster burivuhster requested review from jeanpaul, a team and ggozad June 4, 2025 07:52
@shortstacked
Copy link
Contributor

Workflow Test Results 📊 🔴 3 Failed, ⚠️ 3 Warnings, 👍 77 Successful out of 83 total workflows.

Detail: Workflows failing: 237: Workflow contains 1 deleted data. View full workflow run

Tested Ref: 73e6fd32d2cabb02398dc847dfe40d2a22976b33 by @ggozad

❌ Failed Tests (3)

Workflow ID Workflow Name Reason
237 BasicLLMChain:AzureChat Workflow contains 1 deleted data.
52 Coda:Table:getAllColumns getColumn getAllRows getR... Too Many Requests on node Coda15
84 Matrix:Room:create invite kick leave:RoomMember:ge... Too Many Requests on node Matrix1

⚠️ Warnings (3)

Workflow ID Workflow Name Reason
35 Slack:User:getPresence info:UserProfile:get update... Workflow contains new data that previously did not exist.
257 Agent:auto-fix:anthropic Workflow contains new data that previously did not exist.
48 Asana:Project:getAll get:Task:create update move g... Workflow contains new data that previously did not exist.

Copy link
Contributor

github-actions bot commented Jun 5, 2025

⚠️ Some Cypress E2E specs are failing, please fix them before merging

Copy link
Contributor

github-actions bot commented Jun 5, 2025

✅ All Cypress E2E specs passed

@burivuhster burivuhster merged commit 0bea193 into master Jun 5, 2025
56 of 58 checks passed
@burivuhster burivuhster deleted the ai-gpt-4.1-mini-default branch June 5, 2025 11:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
n8n team Authored by the n8n team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants