Skip to content

fix: pin upper bound for mistralai dependency#1935

Merged
mkmeral merged 2 commits intostrands-agents:mainfrom
mkmeral:fix/pin-mistralai-upper-bound
Mar 19, 2026
Merged

fix: pin upper bound for mistralai dependency#1935
mkmeral merged 2 commits intostrands-agents:mainfrom
mkmeral:fix/pin-mistralai-upper-bound

Conversation

@mkmeral
Copy link
Contributor

@mkmeral mkmeral commented Mar 19, 2026

Description

Currently Mistral model provider is broken, because we do not have an upper bound in our deps, and they just released 2.0

This pins the upper version of mistralai

Also pins versions for Bedrock Bidi clients, so we don't run into the same issue. We should have upper bounds for everything

Related Issues

#1924

Documentation PR

Type of Change

Bug fix

Testing

How have you tested the change? Verify that the changes do not break functionality or introduce warnings in consuming repositories: agents-docs, agents-tools, agents-cli

  • I ran hatch run prepare

Checklist

  • I have read the CONTRIBUTING document
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature, or no new docs are needed
  • My changes generate no new warnings
  • Any dependent changes have been merged and published

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

mkmeral added 2 commits March 19, 2026 11:26
mistralai 2.0.0 moved the Mistral client class from
mistralai.Mistral to mistralai.client.Mistral, breaking
mypy type checking in CI.
Pin aws_sdk_bedrock_runtime and smithy-aws-core to
>=0.4.0,<1.0.0 to prevent unexpected breaking changes
from pre-1.0 packages.
@codecov
Copy link

codecov bot commented Mar 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@mkmeral mkmeral enabled auto-merge (squash) March 19, 2026 15:34
@mkmeral mkmeral changed the title fix: pin mistralai upper bound fix: pin upper bound for mistralai dependency Mar 19, 2026
@github-actions
Copy link

Assessment: Comment

This PR correctly addresses issue #1924 by pinning mistralai<2.0.0 to avoid breaking changes in the v2.0 SDK.

Review Notes
  • Scope clarity: The PR also modifies bidi dependencies (aws_sdk_bedrock_runtime, smithy-aws-core) which aren't mentioned in the description. Please update the description to explain these changes or consider a separate PR.
  • Testing checklist: The "I ran hatch run prepare" checkbox is unchecked - please ensure all checks pass before merging.

The mistral fix looks good and follows the existing dependency versioning pattern used throughout the project. 👍

@mkmeral mkmeral merged commit ae28397 into strands-agents:main Mar 19, 2026
25 of 27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants