Skip to content

Commit d7096ac

Browse files
authored
fix: resolve mutually exclusive cryptography version in dev extras (#794)
The `dev` extras group declared `cryptography>=45.0.3,<46.0` while the `iatp` extras (which `dev` includes transitively via `full`) declared `cryptography>=46.0.5,<47.0`. These constraints are mutually exclusive — no version of cryptography satisfies both simultaneously. When pip attempts to install `.[dev]` it cannot resolve the dependency tree, causing the install to fall back to a minimal install without `redis` and the other optional dependencies. This produces 3 test failures in test_stateless.py: - TestRedisConfig::test_get_client_creates_pool_with_config - TestRedisConfig::test_get_client_without_config_uses_from_url - TestRedisErrorHandling::test_unavailable_redis_via_get_client All three fail with ModuleNotFoundError: No module named 'redis' because redis was never actually installed due to the dependency resolution failure. Fix: align the cryptography pin in dev extras to match the iatp constraint (>=46.0.5,<47.0). The latest release is 46.0.6 which satisfies this range. All 2,861 tests pass after this change.
1 parent 2e4f1b5 commit d7096ac

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

packages/agent-os/pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,7 @@ dev = [
116116
"aiohttp>=3.13.3,<4.0",
117117
"structlog>=24.1.0,<25.0",
118118
"redis>=4.0.0,<5.0",
119-
"cryptography>=45.0.3,<46.0",
119+
"cryptography>=46.0.5,<47.0",
120120
"eval_type_backport>=0.2.0; python_version < '3.10'",
121121
]
122122

0 commit comments

Comments
 (0)