Skip to content

fix: redis instrumentation fixes#24

Merged
sohankshirsagar merged 7 commits intomainfrom
sohan/redis-fixes
Jan 13, 2026
Merged

fix: redis instrumentation fixes#24
sohankshirsagar merged 7 commits intomainfrom
sohan/redis-fixes

Conversation

@sohankshirsagar
Copy link
Contributor

@sohankshirsagar sohankshirsagar commented Jan 12, 2026

Note

Redis instrumentation

  • Trace Pipeline.execute and Pipeline.immediate_execute_command (sync and async); add async record paths and replay for pipelines/WATCH
  • Centralize input_value building for commands/pipelines; improve mock matching; refine response deserialization
  • Serialize bytes as tagged UTF-8/hex and add corresponding deserialization

E2E tests

  • Add endpoints and requests for MGET/MSET, pipeline (transaction/no-transaction), async pipeline, binary data, and WATCH transaction; enable nonexistent-key GET test

Core/proto serialization

  • Use null_value=0 for betterproto Value when handling None across types and span serialization

Tooling

  • generate_manifest.py: include pythonVersion: ">=3.12"

Written by Cursor Bugbot for commit c0325b5. This will update automatically on new commits. Configure here.

Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 6 files

@sohankshirsagar sohankshirsagar merged commit 435dec2 into main Jan 13, 2026
14 checks passed
@sohankshirsagar sohankshirsagar deleted the sohan/redis-fixes branch January 13, 2026 00:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants