-
Notifications
You must be signed in to change notification settings - Fork 13
release: Bump to 0.0.8 and loosen redisvl dep #67
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR bumps the package to v0.0.8 with updated project metadata, loosens the redisvl
dependency constraint, refactors how JSON commands are enqueued (removing misplaced await
s), and adds/removes type ignores across several Redis interactions.
- Bump to version 0.0.8, update description, keywords, and loosen dependencies in
pyproject.toml
- Refactor
pipeline.json().set
calls to enqueue operations instead of awaiting them - Adjust type hints and ignore comments, and tweak API signature of
get_next_version
Reviewed Changes
Copilot reviewed 9 out of 10 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
tests/test_async_aget_tuple_checkpoint_id.py | Reformatted assert statements into multi-line style |
pyproject.toml | Bumped version, updated description/keywords, loosened redisvl and other deps |
langgraph/store/redis/aio.py | Removed redundant # type: ignore on TTL config; added ignore on JSON.get calls |
langgraph/store/redis/init.py | Cleaned up JSON parsing ignore and TTL condition parentheses |
langgraph/checkpoint/redis/base.py | Changed get_next_version signature (removed ChannelProtocol ), added ignore on writes loop |
langgraph/checkpoint/redis/ashallow.py | Switched await pipeline.json().set to pipeline.json().set to properly enqueue ops |
langgraph/checkpoint/redis/aio.py | Removed unused imports, refactored JSON set calls, added ignore comments |
langgraph/checkpoint/redis/init.py | Removed unnecessary # type: ignore on step assignment |
.github/workflows/test.yml | Updated Redis version matrix from 8.0-M03 to 8.0.2 |
Comments suppressed due to low confidence (3)
langgraph/checkpoint/redis/base.py:386
- Changing
get_next_version
to drop theChannelProtocol
parameter in favor ofchannel: None = None
is a breaking API change. Consider preserving backward compatibility or documenting the change and bumping the major version accordingly.
def get_next_version(self, current: Optional[str], channel: None = None) -> str:
pyproject.toml:21
- The
redis
andpython-ulid
dependencies were removed. Verify that no parts of the codebase still import or rely on these packages to prevent runtime errors.
python = ">=3.9,<3.14"
langgraph/checkpoint/redis/ashallow.py:93
- In
_write_obj_tx
, thepipeline.json().set
calls are no longer awaited but also not executed here; ensure that the caller invokespipeline.execute()
after this function to actually send the commands to Redis.
pipe.json().set(key, "$.channel", write_obj["channel"])
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Double-check your changes within "cluster mode" code paths. We intentionally avoid pipelines in these paths and instead execute single commands.
49202e7
to
980946a
Compare
Release 0.0.8 version of
langgraph-checkpoint-redis
. This release fixes a few known bugs and also updates the dependency requirements to be more flexible. We also focus on fixing a few mypy issues introduced inredis-py
6.2.0 version.