johno/beads
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

chore: remove dead deprecated wrapper functions

Browse Source 
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 <noreply@anthropic.com>
This commit is contained in:
Steve Yegge
2025-12-21 15:49:55 -08:00
parent 7d43a13600
commit 0c10eab873
1 changed files with 0 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files
cmd/bd/deletion_tracking.go
-14
View File
@@ -136,20 +136,6 @@ func merge3WayAndPruneDeletions(ctx context.Context, store storage.Storage, json
return true, nil 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 // getSnapshotStats returns statistics about the snapshot files
// Deprecated: Use SnapshotManager.GetStats() instead // Deprecated: Use SnapshotManager.GetStats() instead
func getSnapshotStats(jsonlPath string) (baseCount, leftCount int, baseExists, leftExists bool) { func getSnapshotStats(jsonlPath string) (baseCount, leftCount int, baseExists, leftExists bool) {