Skip to content

feat: Add Docker Container Snapshot Manager integration #345

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

feat: Add Docker Container Snapshot Manager integration #345

wants to merge 6 commits into from

Conversation

mingolladaniele
Copy link

@mingolladaniele mingolladaniele commented Aug 16, 2025 •
edited
Loading

Completed Tasks ✅

  • Draft PR created in CUA repo
  • ARCHITECTURE.md with concise technical overview
  • DockerSnapshotProvider wired to CUA callbacks
  • Integration tests with real CUA agent verified
  • CI compliance validated (Black, Ruff, pytest)
  • Named Volume Support - Tar-based backup/restore for named volumes
  • Bind Mount Detection - User warnings for non-portable bind mounts
  • Volume Type Identification - Full volume analysis and handling
  • Critical MyPy Fixes - All blocking type errors resolved
  • CUA Volume Example - Complete working demo with RUN_START/RUN_END triggers

Remaining MVP Requirements

Enhanced Restore Modes

  • Replace mode: stop/remove original, create replacement
  • Minimal container healthcheck after restore
  • Error handling for restore failures

Deterministic Retention Policy

  • Age-first cleanup (max_age then max_count)
  • Deterministic ordering for snapshot removal
  • Focused unit test for retention logic

Final Polish

  • CI validation in pipeline
  • Documentation review and finalization

Technical Debt (Non-blocking)

  • MyPy type annotation cleanup (1 false positive remaining)
  • CLI function return type annotations
  • Test function type hints

@mingolladaniele
feat: Add Docker Container Snapshot Manager integration
663e3ed 
- Add snapshot-manager module to CUA Python libs
- Implements snapshot-based state management for CUA Agent SDK
- Features: Docker commit snapshots, JSON metadata, CLI, callback integration
- Complete with architecture docs, examples, and comprehensive test suite
- Added to main CUA README modules table
- Ready for team review and CI validation

Components:
- SnapshotManager: Central orchestrator with retention policies
- DockerSnapshotProvider: Efficient container snapshots via docker commit
- FileSystemSnapshotStorage: JSON metadata with indexing
- SnapshotCallback: CUA Agent SDK lifecycle integration
- CLI: Complete command-line interface for all operations
- Examples: Basic usage and full CUA Agent integration demos
- Tests: Unit and integration test coverage
- Architecture: Detailed design decisions and ASCII diagram
@mingolladaniele
fix: CI compliance for snapshot-manager module
2bd1b88 
- Fixed Black code formatting across all files
- Fixed Ruff linting issues
- Updated Python requirements to >=3.12 for compatibility
- All 9 tests passing successfully
- PDM environment setup complete
- Ready for CI validation in draft PR
@mingolladaniele
feat: Enhance CUA Snapshot Manager with callback integration and CI i…
fd11ac2 
…mprovements
@mingolladaniele
feat: Implement volume management in FileSystemSnapshotStorage
672ed90
@mingolladaniele
feat: Add CUA volume snapshot example and enhance volume management
f309254
@mingolladaniele
feat: Finalize core MVP features for CUA Snapshot Manager
465d447
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mingolladaniele