Skip to content

feat: Add OpenRouter node - #12882

Merged
OlegIvaniv merged 5 commits into
masterfrom
ai_openrouter_node
Jan 28, 2025
Merged

feat: Add OpenRouter node#12882
OlegIvaniv merged 5 commits into
masterfrom
ai_openrouter_node

Conversation

@OlegIvaniv

Copy link
Copy Markdown
Contributor

Summary

Added support for new LLM provider:

  • OpenRouter Chat Model (credentials + 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)

@n8n-assistant n8n-assistant Bot added the n8n team Authored by the n8n team label Jan 28, 2025
@codecov

codecov Bot commented Jan 28, 2025

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 0% with 22 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...des/llms/LmChatOpenRouter/LmChatOpenRouter.node.ts 0.00% 15 Missing ⚠️
...langchain/credentials/OpenRouterApi.credentials.ts 0.00% 7 Missing ⚠️

📢 Thoughts on this report? Let us know!

Comment thread packages/@n8n/nodes-langchain/nodes/llms/LmChatOpenRouter/openrouter.svg Outdated
Comment thread packages/@n8n/nodes-langchain/credentials/OpenRouterApi.credentials.ts Outdated

@OlegIvaniv OlegIvaniv left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks @netroy! I added the base URL to the credential and applied your other suggestions. Can you have another look pls? Once this one is merged, I'll do the same for the Deepseek node(#12873)

@OlegIvaniv
OlegIvaniv requested a review from netroy January 28, 2025 13:32
Comment thread packages/@n8n/nodes-langchain/credentials/OpenRouterApi.credentials.ts Outdated
Comment thread packages/@n8n/nodes-langchain/credentials/OpenRouterApi.credentials.ts Outdated
@OlegIvaniv
OlegIvaniv requested a review from netroy January 28, 2025 14:15
@cypress

cypress Bot commented Jan 28, 2025

Copy link
Copy Markdown

n8n    Run #8991

Run Properties:  status check passed Passed #8991  •  git commit fc181d2f0b: 🌳 🖥️ browsers:node18.12.0-chrome107 🤖 OlegIvaniv 🗃️ e2e/*
Project n8n
Branch Review ai_openrouter_node
Run status status check passed Passed #8991
Run duration 04m 24s
Commit git commit fc181d2f0b: 🌳 🖥️ browsers:node18.12.0-chrome107 🤖 OlegIvaniv 🗃️ e2e/*
Committer Oleg Ivaniv
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 1
Tests that did not run due to a developer annotating a test with .skip  Pending 5
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 432
View all changes introduced in this branch ↗︎

@github-actions

Copy link
Copy Markdown
Contributor

✅ All Cypress E2E specs passed

@OlegIvaniv
OlegIvaniv merged commit dc85b02 into master Jan 28, 2025
@OlegIvaniv
OlegIvaniv deleted the ai_openrouter_node branch January 28, 2025 14:43
@github-actions github-actions Bot mentioned this pull request Jan 29, 2025
@janober

janober commented Jan 29, 2025

Copy link
Copy Markdown
Member

Got released with n8n@1.77.0

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 Released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants