[Buckets] Add search param to list_buckets#4130
Conversation
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 79ffc3a. Configure here.
|
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
Wauplin
left a comment
Wauplin
left a comment
There was a problem hiding this comment.
Thanks @alexpouliquen for your PR!
Left a comment regarding tests in the Python library. It's not a list of hard rules but more a set of guidelines we try to follow. These should be in the AGENTS.md but often gets ignored so good to double-check. Other than that, ready to merge!
Co-authored-by: Lucain <[email protected]>
Wauplin
left a comment
Wauplin
left a comment
There was a problem hiding this comment.
Good to merge! First of many? 🎉
|
This PR has been shipped as part of the v1.12.0 release. |
4 participants
Add support for buckets list search params.
100% vibecoded, but should be fine
Note
Low Risk
Small, additive change that only affects bucket-listing queries and CLI flag parsing; main risk is behavior differences in list results when
--search/searchis used.Overview
Adds a
searchfilter for bucket listing in both Python and CLI.HfApi.list_buckets(..., search=...)now forwards asearchquery param to the Hub, andhf buckets list --search ...exposes it with validation to reject use when listing files plus an updated empty-results message. Documentation and CLI reference are updated and new CLI tests cover the flag and error cases.Reviewed by Cursor Bugbot for commit a647a9b. Bugbot is set up for automated code reviews on this repo. Configure here.