Skip to content

Memory fix#212

Merged
m1rl0k merged 4 commits intotestfrom
memory-fix
Jan 27, 2026
Merged

Memory fix#212
m1rl0k merged 4 commits intotestfrom
memory-fix

Conversation

@m1rl0k
Copy link
Collaborator

@m1rl0k m1rl0k commented Jan 27, 2026

No description provided.

Update _redis_get_json_by_key to use execute_command for GET, handle string-to-bytes conversion, and properly decompress and decode Redis values before JSON parsing.
Introduces a separate Redis client with decode_responses=False to properly handle binary (LZ4-compressed) data. Updates relevant functions to use this binary client for get operations, ensuring correct data retrieval and decompression.
Replaced the Business Source License 1.1 with the Context-Engine Source Available License 1.0 in the LICENSE file, updating terms to restrict use to personal, non-commercial purposes and clarifying commercial licensing requirements. Updated package.json to reference the new license.
@m1rl0k m1rl0k merged commit 8395dfd into test Jan 27, 2026
@kiloconnect
Copy link

kiloconnect bot commented Jan 27, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (4 files)
  • LICENSE - License text update
  • package.json - License field update
  • scripts/workspace_state.py - Redis binary client implementation
  • vscode-extension/context-engine-uploader/package-lock.json - Generated file

@augmentcode
Copy link

augmentcode bot commented Jan 27, 2026

🤖 Augment PR Summary

Summary: This PR addresses Redis-backed workspace state memory/correctness issues when storing LZ4-compressed payloads, and updates repository licensing metadata.

Changes:

  • Replaced the repository LICENSE text with the new “Context-Engine Source Available License 1.0”.
  • Updated package.json to reference the license via “SEE LICENSE IN LICENSE”.
  • Added a dedicated Redis “binary” client (decode_responses=False) for fetching compressed state values.
  • Switched Redis JSON reads (_redis_get_json, _redis_get_json_by_key) to use GET via the binary client to avoid UTF-8 decoding of binary/LZ4 payloads.
  • Ensured _redis_get_json_by_key now runs values through the LZ4 decompression path before JSON decoding.

Technical Notes: Read-paths now consistently treat Redis values as bytes and only decode to UTF-8 after optional LZ4 decompression, reducing unnecessary string allocations and preventing decode failures on binary payloads.

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

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

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

@m1rl0k m1rl0k deleted the memory-fix branch January 29, 2026 11:27
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.

1 participant