mirror of
https://github.com/langchain-ai/langchain.git
synced 2026-10-05 09:25:14 +03:00
Juan Ezquerro LLanes
ff46bb478b
Fixes #40985 --- `ChatOllama(reasoning="xhigh")` currently raises a Pydantic `ValidationError` instead of reaching Ollama, because the `ollama` floor still admits clients that reject model-defined thinking levels. `ChatOllama.reasoning` is `bool | str | None` and `_chat_params()` forwards the string verbatim as `think`. Since ollama 0.6.3 (ollama/ollama-python#744) `ChatRequest.think` is `Optional[Union[bool, str]]`, matching the Ollama Go backend. But the declared range `ollama>=0.6.1,<1.0.0` still admits 0.6.1 and 0.6.2, where the annotation is `Optional[Union[bool, Literal["low", "medium", "high"]]]` and `"xhigh"` or `"max"` fails validation client-side. Verified both directions on the same snippet — `ChatOllama(reasoning="xhigh")._chat_params(...)["think"] == "xhigh"`, then constructing `ChatRequest(..., think=...)`: - ollama 0.6.2 → `ValidationError` on `think.bool` and `think.literal['low','medium','high']` - ollama 0.6.3 → validates This raises the floor only; no `ChatOllama` code path changes. Narrowing `reasoning` inside `ChatOllama` instead would reject levels the Ollama backend accepts and duplicate type logic the client now owns. Per the PR template I'm opening this alongside the issue rather than pre-approved, so please assign it to me if the approach looks right. ## Release note `langchain-ollama` now requires `ollama>=0.6.3`, fixing a `ValidationError` when `ChatOllama` is configured with a model-defined reasoning level such as `"xhigh"` or `"max"`.
125 lines
3.8 KiB
TOML
125 lines
3.8 KiB
TOML
[build-system]
|
|
requires = ["hatchling"]
|
|
build-backend = "hatchling.build"
|
|
|
|
[project]
|
|
name = "langchain-ollama"
|
|
description = "An integration package connecting Ollama and LangChain"
|
|
license = { text = "MIT" }
|
|
readme = "README.md"
|
|
classifiers = [
|
|
"Development Status :: 5 - Production/Stable",
|
|
"Intended Audience :: Developers",
|
|
"License :: OSI Approved :: MIT License",
|
|
"Programming Language :: Python :: 3",
|
|
"Programming Language :: Python :: 3.10",
|
|
"Programming Language :: Python :: 3.11",
|
|
"Programming Language :: Python :: 3.12",
|
|
"Programming Language :: Python :: 3.13",
|
|
"Programming Language :: Python :: 3.14",
|
|
"Topic :: Scientific/Engineering :: Artificial Intelligence",
|
|
]
|
|
|
|
version = "1.1.0"
|
|
requires-python = ">=3.10.0,<4.0.0"
|
|
dependencies = [
|
|
"ollama>=0.6.3,<1.0.0",
|
|
"langchain-core>=1.4.7,<2.0.0",
|
|
]
|
|
|
|
[project.urls]
|
|
Homepage = "https://docs.langchain.com/oss/python/integrations/providers/ollama"
|
|
Documentation = "https://reference.langchain.com/python/integrations/langchain_ollama/"
|
|
Repository = "https://github.com/langchain-ai/langchain"
|
|
Issues = "https://github.com/langchain-ai/langchain/issues"
|
|
Changelog = "https://github.com/langchain-ai/langchain/releases?q=%22langchain-ollama%22"
|
|
Twitter = "https://x.com/langchain_oss"
|
|
Slack = "https://www.langchain.com/join-community"
|
|
Reddit = "https://www.reddit.com/r/LangChain/"
|
|
|
|
[dependency-groups]
|
|
test = [
|
|
"pytest>=9.0.3,<10.0.0",
|
|
"pytest-asyncio>=1.3.0,<2.0.0",
|
|
"pytest-socket>=0.7.0,<1.0.0",
|
|
"pytest-watcher>=0.4.3,<1.0.0",
|
|
"pytest-xdist>=3.6.1,<4.0.0",
|
|
"syrupy>=5.0.0,<6.0.0",
|
|
"langchain-tests>=1.1.9,<2.0.0",
|
|
]
|
|
test_integration = []
|
|
lint = ["ruff>=0.13.1,<0.14.0"]
|
|
dev = []
|
|
typing = [
|
|
"ty>=0.0.1,<1.0.0",
|
|
]
|
|
|
|
[tool.uv]
|
|
constraint-dependencies = ["pygments>=2.20.0"] # CVE-2026-4539
|
|
|
|
[tool.uv.sources]
|
|
langchain-core = { path = "../../core", editable = true }
|
|
langchain-tests = { path = "../../standard-tests", editable = true }
|
|
|
|
[tool.ruff.format]
|
|
docstring-code-format = true
|
|
docstring-code-line-length = 100
|
|
|
|
[tool.ruff.lint]
|
|
select = ["ALL"]
|
|
ignore = [
|
|
"COM812", # Messes with the formatter
|
|
"ISC001", # Messes with the formatter
|
|
"PERF203", # Rarely useful
|
|
"S112", # Rarely useful
|
|
"RUF012", # Doesn't play well with Pydantic
|
|
"SLF001", # Private member access
|
|
"FIX002", # TODOs
|
|
"TD002", # TODO authors
|
|
"TD003", # TODO missing url
|
|
"TC002", # Incorrect type-checking block
|
|
"TC003", # Incorrect type-checking block
|
|
"PLR0912", # Too many branches
|
|
"PLR0915", # Too many statements
|
|
"C901", # Function too complex
|
|
"FBT001", # Boolean function param
|
|
"ERA001", # Commented-out code
|
|
|
|
# TODO
|
|
"ANN401",
|
|
]
|
|
unfixable = ["B028"] # People should intentionally tune the stacklevel
|
|
|
|
[tool.ruff.lint.pydocstyle]
|
|
convention = "google"
|
|
ignore-var-parameters = true # ignore missing documentation for *args and **kwargs parameters
|
|
|
|
[tool.ruff.lint.flake8-tidy-imports]
|
|
ban-relative-imports = "all"
|
|
|
|
[tool.ruff.lint.per-file-ignores]
|
|
"tests/**" = ["D"] # ignore docstring checks for tests
|
|
|
|
[tool.coverage.run]
|
|
omit = ["tests/*"]
|
|
|
|
[tool.pytest.ini_options]
|
|
addopts = "--snapshot-warn-unused --strict-markers --strict-config --durations=5"
|
|
markers = [
|
|
"compile: mark placeholder test used to compile integration tests without running them",
|
|
]
|
|
asyncio_mode = "auto"
|
|
|
|
[tool.ruff.lint.extend-per-file-ignores]
|
|
"tests/**/*.py" = [
|
|
"S101", # Tests need assertions
|
|
"S105", # False positive on dict key "token" in logprobs assertions
|
|
"S311", # Standard pseudo-random generators are not suitable for cryptographic purposes
|
|
"ARG001", # Unused function arguments in tests (e.g. kwargs)
|
|
"PLR2004", # Magic value in comparisons
|
|
"PT011", # `pytest.raises()` is too broad
|
|
]
|
|
"scripts/*.py" = [
|
|
"INP001", # Not a package
|
|
]
|