Skip to content

Add Straico Chat Model Node #15881

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

Closed
wants to merge 10 commits into from

Conversation

nlp-deutschland-de
Copy link

Summary

This PR introduces the LmChatStraico node to integrate Straico's chat-completion models into n8n workflows for AI chains and agents.
Screenshot 2025-05-31 165854

New Files

  • New Files: nodes/llms/LmChatStraico/LmChatStraico.node.ts
  • nodes/llms/LmChatStraico/StraicoChatModel.ts
  • credentials/StraicoApi.credentials.ts
  • nodes/llms/LmChatStraico/straico.svg

Updated Files

nodes/agents/Agent/V1/AgentV1.node.ts:

  • Added lmChatStraico to supported language model connections for AI Agent V1 compatibility.

nodes/agents/Agent/V2/AgentV2.node.ts:

  • Added lmChatStraico to supported language model connections for AI Agent V2 compatibility.

package.json:

  • Registered StraicoApi.credentials.ts and LmChatStraico.node.ts in the credentials and nodes arrays.

Testing

Functional Tests:

  • Tested LmChatStraico node with input “Hi” to confirm successful API calls and valid completions.
  • Verified maxTokens defaults to 1024, resolving HTTP 422 error.
  • Tested invalid inputs (e.g., maxTokens: 0, undefined) to ensure proper defaults and validation.
  • Validated model selection via GET /models and parameter passing to StraicoChatModel.

image

Error Handling:

  • Simulated API failures (HTTP 401, 429, 422, 500) to verify ChatResult error messages.
  • Tested retry logic with invalid baseURL to confirm exponential backoff.

Additional Notes

@CLAassistant
Copy link

CLAassistant commented May 31, 2025

CLA assistant check
All committers have signed the CLA.

Copy link

@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 4 issues across 6 files. Review them in cubic.dev

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

@n8n-assistant n8n-assistant bot added community Authored by a community member in linear Issue or PR has been created in Linear for internal review labels May 31, 2025
@Joffcom
Copy link
Member

Joffcom commented May 31, 2025

Hey @nlp-deutschland-de,

Thanks for the PR, We have created "GHC-2274" as the internal reference to get this reviewed.

One of us will be in touch if there are any changes needed, in most cases this is normally within a couple of weeks but it depends on the current workload of the team.

nlp-deutschland-de and others added 2 commits May 31, 2025 17:35
…traico.node.ts

Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
…traico.node.ts

Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
@Joffcom Joffcom closed this Jun 2, 2025
@Joffcom
Copy link
Member

Joffcom commented Jun 2, 2025

Hey @nlp-deutschland-de,

As covered in our contribution guide we no longer accept new nodes being added from a PR unless requested or cleared by the team first. The best way to do this would be through community nodes which can be published to npm although at the moment support for AI nodes may not be fully functional so it would be a case of waiting a bit longer for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Authored by a community member in linear Issue or PR has been created in Linear for internal review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants