Skip to content

Fix delete_syftbox to clean up all state#137

Merged
koenvanderveen merged 1 commit intomainfrom
koen/fix-delete-syftbox-cleanup
Feb 17, 2026
Merged

Fix delete_syftbox to clean up all state#137
koenvanderveen merged 1 commit intomainfrom
koen/fix-delete-syftbox-cleanup

Conversation

@koenvanderveen
Copy link
Collaborator

Summary

  • Clear in-memory caches (DO/DS event caches, file hashes) by calling _clear_caches() during deletion
  • Delete local filesystem cache directories (-events/, -event-messages/) that live alongside the syftbox folder
  • Add checkpoint and rolling state file patterns to orphaned file search, so orphaned checkpoint/rolling state files are caught
  • Clear dataset_collection_folder_id_cache in reset_caches() which was previously missed

Test plan

  • Unit tests pass (179 passed, 1 skipped)

🤖 Generated with Claude Code

…kpoints, dataset cache)

- Clear in-memory caches (DO/DS event caches, file hashes) via _clear_caches()
- Delete local filesystem cache directories (-events/, -event-messages/)
- Add checkpoint and rolling state patterns to orphaned file search
- Clear dataset_collection_folder_id_cache in reset_caches()

🤖 Generated with [Claude Code](https://claude.com/claude-code)
@koenvanderveen koenvanderveen merged commit 12258f8 into main Feb 17, 2026
3 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

Comments