From 0c10eab8735601439f1a001e3f737be75407b293 Mon Sep 17 00:00:00 2001 From: Steve Yegge Date: Sun, 21 Dec 2025 15:49:55 -0800 Subject: [PATCH] chore: remove dead deprecated wrapper functions MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Remove cleanupSnapshots() and validateSnapshotConsistency() from deletion_tracking.go - both were marked deprecated and never called. The other two deprecated functions (getSnapshotStats, initializeSnapshotsIfNeeded) are still used by tests and production code respectively. Filed bd-xsl9 to track removal of legacy autoflush dual-path code. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 --- cmd/bd/deletion_tracking.go | 14 -------------- 1 file changed, 14 deletions(-) diff --git a/cmd/bd/deletion_tracking.go b/cmd/bd/deletion_tracking.go index 58cf5898..9faa9645 100644 --- a/cmd/bd/deletion_tracking.go +++ b/cmd/bd/deletion_tracking.go @@ -136,20 +136,6 @@ func merge3WayAndPruneDeletions(ctx context.Context, store storage.Storage, json return true, nil } -// cleanupSnapshots removes the snapshot files and their metadata -// Deprecated: Use SnapshotManager.Cleanup() instead -func cleanupSnapshots(jsonlPath string) error { - sm := NewSnapshotManager(jsonlPath) - return sm.Cleanup() -} - -// validateSnapshotConsistency checks if snapshot files are consistent -// Deprecated: Use SnapshotManager.Validate() instead -func validateSnapshotConsistency(jsonlPath string) error { - sm := NewSnapshotManager(jsonlPath) - return sm.Validate() -} - // getSnapshotStats returns statistics about the snapshot files // Deprecated: Use SnapshotManager.GetStats() instead func getSnapshotStats(jsonlPath string) (baseCount, leftCount int, baseExists, leftExists bool) {