Conversation
|
🧪 Testing To try out this version of the SDK, run: Expires at: Sun, 08 Feb 2026 00:55:35 GMT |
0aca8a4 to
10e2542
Compare
10e2542 to
7d93621
Compare
| self, | ||
| *, | ||
| container_tag: str, | ||
| threshold: float, | ||
| q: str | Omit = omit, | ||
| # Use the following arguments if you need to pass additional parameters to the API that aren't available via kwargs. | ||
| # The extra values given here take precedence over values defined on the client or passed to this method. |
There was a problem hiding this comment.
Bug: The profile method now requires a threshold parameter without a default value. This is a breaking change that will cause existing calls to fail after upgrading.
Severity: CRITICAL
🔍 Detailed Analysis
The profile() method in both the synchronous and asynchronous clients has been updated to require a new threshold parameter without providing a default value. Code that uses an older version of the SDK and calls client.profile(container_tag="...") will fail with a TypeError: profile() missing 1 required keyword-only argument: 'threshold' after upgrading. This breaking change was introduced in a minor version release, which can lead to unexpected runtime failures in production environments for users who follow semantic versioning.
💡 Suggested Fix
To maintain backward compatibility, add a default value to the threshold parameter, for example, threshold: float = 0.0. Alternatively, release this as a major version and clearly document the breaking change and migration steps in the CHANGELOG.
🤖 Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent.
Verify if this is a real issue. If it is, propose a fix; if not, explain why it's not
valid.
Location: src/supermemory/_client.py#L294-L300
Potential issue: The `profile()` method in both the synchronous and asynchronous clients
has been updated to require a new `threshold` parameter without providing a default
value. Code that uses an older version of the SDK and calls
`client.profile(container_tag="...")` will fail with a `TypeError: profile() missing 1
required keyword-only argument: 'threshold'` after upgrading. This breaking change was
introduced in a minor version release, which can lead to unexpected runtime failures in
production environments for users who follow semantic versioning.
Did we get this right? 👍 / 👎 to inform future reviews.
Reference ID: 8373668
|
🤖 Release is at https://github.com/supermemoryai/python-sdk/releases/tag/v3.15.0 🌻 |
Automated Release PR
3.15.0 (2026-01-09)
Full Changelog: v3.14.0...v3.15.0
Features
Documentation
This pull request is managed by Stainless's GitHub App.
The semver version number is based on included commit messages. Alternatively, you can manually set the version number in the title of this pull request.
For a better experience, it is recommended to use either rebase-merge or squash-merge when merging this pull request.
🔗 Stainless website
📚 Read the docs
🙋 Reach out for help or questions