Skip to content

Refactor blob capture and extraction to support capture folder and post-rotation handling#31

Merged
effective-range merged 2 commits into
mainfrom
blobmemory
May 25, 2026
Merged

Refactor blob capture and extraction to support capture folder and post-rotation handling#31
effective-range merged 2 commits into
mainfrom
blobmemory

Conversation

@effective-range
Copy link
Copy Markdown
Contributor

  • Updated BlobFsCapture to accept a capture_folder parameter for storing rotated blobs.
  • Modified rotate method to move the current blob to the specified capture folder.
  • Enhanced CompositeBlobCapture to manage post-rotation blobs and ensure follow-up frames are captured correctly.
  • Adjusted BlobExtractor to handle multiple blob files, including extracting from tar archives.
  • Updated tests to verify new behavior for blob rotation, extraction, and completion handler calls.
  • Refactored argument parsing in add_args to include --blob-capture-folder option.

…st-rotation handling

- Updated `BlobFsCapture` to accept a `capture_folder` parameter for storing rotated blobs.
- Modified `rotate` method to move the current blob to the specified capture folder.
- Enhanced `CompositeBlobCapture` to manage post-rotation blobs and ensure follow-up frames are captured correctly.
- Adjusted `BlobExtractor` to handle multiple blob files, including extracting from tar archives.
- Updated tests to verify new behavior for blob rotation, extraction, and completion handler calls.
- Refactored argument parsing in `add_args` to include `--blob-capture-folder` option.
@github-actions
Copy link
Copy Markdown

ghost commented May 25, 2026

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  common_utility
  capture.py 426, 431, 513-514
Project Total  

This report was generated by python-coverage-comment-action

@effective-range effective-range merged commit fbf87ad into main May 25, 2026
7 checks passed
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