Skip to content

AP-24074: Add authenticator for Mistral AI API#61

Open
ya-hn wants to merge 1 commit intomasterfrom
enh/AP-24074-mistral-ai-authenticator-node
Open

AP-24074: Add authenticator for Mistral AI API#61
ya-hn wants to merge 1 commit intomasterfrom
enh/AP-24074-mistral-ai-authenticator-node

Conversation

@ya-hn
Copy link
Contributor

@ya-hn ya-hn commented Mar 16, 2026

No description provided.

@ya-hn ya-hn requested a review from a team as a code owner March 16, 2026 17:33
@ya-hn ya-hn requested review from Copilot and knime-ghub-bot and removed request for a team March 16, 2026 17:34
except Exception as e:
raise RuntimeError("Could not authenticate with the Mistral AI API.") from e

def _list_models(
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This function gets updated in the PR for the LLM Selector to filter out duplicate model IDs.

Copy link
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

Introduces a new Mistral AI provider module that registers a KNIME node to create an authentication port object for the Mistral AI API, and wires the module into the main extension entrypoint.

Changes:

  • Added models.mistral package with category/icon setup and a Mistral AI Authenticator node emitting a dedicated authentication port type.
  • Registered the new provider by importing models.mistral in src/knime_llm.py.
  • Added langchain-mistralai to the Pixi environment (and updated the lockfile accordingly).

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/models/mistral/_base.py Defines Mistral icon and KNIME category entry for grouping Mistral nodes.
src/models/mistral/_auth.py Implements Mistral authentication spec/port object and the authenticator node.
src/models/mistral/__init__.py Exposes the authenticator so it registers on import.
src/knime_llm.py Imports models.mistral so nodes are registered in the extension runtime.
pixi.toml Adds langchain-mistralai dependency for the environment.
pixi.lock Lockfile updates to include langchain-mistralai and its resolved dependencies.

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

You can also share your feedback on Copilot code review. Take the survey.

Copy link
Member

@iusethemouse iusethemouse left a comment

Choose a reason for hiding this comment

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

Looks good to me, also played around with it locally and checked validation 👍 Good to merge

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.

3 participants