Conversation
| except Exception as e: | ||
| raise RuntimeError("Could not authenticate with the Mistral AI API.") from e | ||
|
|
||
| def _list_models( |
There was a problem hiding this comment.
This function gets updated in the PR for the LLM Selector to filter out duplicate model IDs.
There was a problem hiding this comment.
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.mistralpackage with category/icon setup and a Mistral AI Authenticator node emitting a dedicated authentication port type. - Registered the new provider by importing
models.mistralinsrc/knime_llm.py. - Added
langchain-mistralaito 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.
iusethemouse
left a comment
There was a problem hiding this comment.
Looks good to me, also played around with it locally and checked validation 👍 Good to merge
No description provided.