Skip to content

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

Merged
merged 10 commits into from
Jun 25, 2025
Merged

Conversation

tylerhutcherson
Copy link
Contributor

@tylerhutcherson tylerhutcherson commented Jun 24, 2025

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 in redis-py 6.2.0 version.

@tylerhutcherson tylerhutcherson changed the title feat: loosen redisvl dependency and release 0.8.0 release: Bump to 0.0.8 and loosen redisvl dep Jun 24, 2025
@abrookins abrookins requested a review from Copilot June 24, 2025 22:26
Copy link

@Copilot Copilot AI left a 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 awaits), 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 the ChannelProtocol parameter in favor of channel: 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 and python-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, the pipeline.json().set calls are no longer awaited but also not executed here; ensure that the caller invokes pipeline.execute() after this function to actually send the commands to Redis.
        pipe.json().set(key, "$.channel", write_obj["channel"])

Copy link
Contributor

@abrookins abrookins left a 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.

@tylerhutcherson tylerhutcherson added the enhancement New feature or request label Jun 25, 2025
@tylerhutcherson tylerhutcherson merged commit 78222c9 into main Jun 25, 2025
19 checks passed
@tylerhutcherson tylerhutcherson deleted the release/0.0.8 branch June 25, 2025 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants