fix: add missing 'name' field to ollama._types.ListResponse.Model #563
+1
−0
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull Request: add missing 'name' field to ollama._types.ListResponse.Model
What this does
The REST endpoint
/api/tags
already returns a field called"name"
for every model:But the Python client class
ListResponse.Model
only maps these keys:Because the extra
name
key is not listed, Pydantic discards it when the JSON is deserialized.This PR simply adds:
to
ListResponse.Model
so the SDK keeps the same data that the server already sends.Why it matters
Some downstream libraries (for example mem0) do:
Today
model.get("name")
always returnsNone
, causing unnecessary pulls or runtime errors.With this tiny change those projects work out of the box.
Backwards compatibility
None
.Files changed
ollama/_types.py
– add one line insideListResponse.Model
.Verification
That’s all.