refactor: Remove unused functions from hash_registry

Remove dead code that was only tested but never used in production:
- hash_exists (gc uses set operations on paths)
- delete_hash (gc uses delete_path directly)
- get_size (gc collects sizes during walk)
- get_hash_size (wrapper for get_size)

Remaining API: compute_hash, build_hash_path, get_store_backend,
get_store_subfolding, put_hash, get_hash, delete_path

Co-Authored-By: Claude Opus 4.5 <[email protected]>

Author: dimitri-yatsenko

src/datajoint/hash_registry.py

@@ -267,34 +267,6 @@ def get_hash(metadata: dict[str, Any]) -> bytes:
     return data
 
 
-def hash_exists(
-    content_hash: str,
-    schema_name: str,
-    store_name: str | None = None,
-) -> bool:
-    """
-    Check if hash-addressed content exists in storage.
-
-    Parameters
-    ----------
-    content_hash : str
-        Base32-encoded hash (26 characters).
-    schema_name : str
-        Database/schema name for path isolation.
-    store_name : str, optional
-        Name of the store. If None, uses default store.
-
-    Returns
-    -------
-    bool
-        True if content exists.
-    """
-    subfolding = get_store_subfolding(store_name)
-    path = build_hash_path(content_hash, schema_name, subfolding)
-    backend = get_store_backend(store_name)
-    return backend.exists(path)
-
-
 def delete_path(
     path: str,
     store_name: str | None = None,
@@ -328,88 +300,3 @@ def delete_path(
         logger.debug(f"Deleted: {path}")
         return True
     return False
-
-
-# Backward compatibility alias
-def delete_hash(
-    content_hash: str,
-    schema_name: str,
-    store_name: str | None = None,
-) -> bool:
-    """
-    Delete hash-addressed content from storage (deprecated).
-
-    .. deprecated::
-        Use :func:`delete_path` with the stored path instead.
-
-    Parameters
-    ----------
-    content_hash : str
-        Base32-encoded hash (26 characters).
-    schema_name : str
-        Database/schema name for path isolation.
-    store_name : str, optional
-        Name of the store. If None, uses default store.
-
-    Returns
-    -------
-    bool
-        True if content was deleted, False if it didn't exist.
-    """
-    subfolding = get_store_subfolding(store_name)
-    path = build_hash_path(content_hash, schema_name, subfolding)
-    return delete_path(path, store_name)
-
-
-def get_size(
-    path: str,
-    store_name: str | None = None,
-) -> int:
-    """
-    Get the size of content at the specified path.
-
-    Parameters
-    ----------
-    path : str
-        Storage path (as stored in metadata).
-    store_name : str, optional
-        Name of the store. If None, uses default store.
-
-    Returns
-    -------
-    int
-        Size in bytes.
-    """
-    backend = get_store_backend(store_name)
-    return backend.size(path)
-
-
-# Backward compatibility alias
-def get_hash_size(
-    content_hash: str,
-    schema_name: str,
-    store_name: str | None = None,
-) -> int:
-    """
-    Get the size of hash-addressed content (deprecated).
-
-    .. deprecated::
-        Use :func:`get_size` with the stored path instead.
-
-    Parameters
-    ----------
-    content_hash : str
-        Base32-encoded hash (26 characters).
-    schema_name : str
-        Database/schema name for path isolation.
-    store_name : str, optional
-        Name of the store. If None, uses default store.
-
-    Returns
-    -------
-    int
-        Size in bytes.
-    """
-    subfolding = get_store_subfolding(store_name)
-    path = build_hash_path(content_hash, schema_name, subfolding)
-    return get_size(path, store_name)

tests/integration/test_hash_storage.py

@@ -10,12 +10,8 @@
 from datajoint.hash_registry import (
     build_hash_path,
     compute_hash,
-    hash_exists,
     delete_path,
-    delete_hash,
     get_hash,
-    get_size,
-    get_hash_size,
     put_hash,
 )
 from datajoint.errors import DataJointError
@@ -192,34 +188,6 @@ def test_verifies_hash(self, mock_get_backend):
             get_hash(metadata)
 
 
-class TestHashExists:
-    """Tests for hash_exists function."""
-
-    @patch("datajoint.hash_registry.get_store_subfolding")
-    @patch("datajoint.hash_registry.get_store_backend")
-    def test_returns_true_when_exists(self, mock_get_backend, mock_get_subfolding):
-        """Test that True is returned when content exists."""
-        mock_backend = MagicMock()
-        mock_backend.exists.return_value = True
-        mock_get_backend.return_value = mock_backend
-        mock_get_subfolding.return_value = None
-
-        content_hash = "abcdefghijklmnopqrstuvwxyz"[:26]  # Valid base32
-        assert hash_exists(content_hash, schema_name="test_schema", store_name="test_store") is True
-
-    @patch("datajoint.hash_registry.get_store_subfolding")
-    @patch("datajoint.hash_registry.get_store_backend")
-    def test_returns_false_when_not_exists(self, mock_get_backend, mock_get_subfolding):
-        """Test that False is returned when content doesn't exist."""
-        mock_backend = MagicMock()
-        mock_backend.exists.return_value = False
-        mock_get_backend.return_value = mock_backend
-        mock_get_subfolding.return_value = None
-
-        content_hash = "abcdefghijklmnopqrstuvwxyz"[:26]  # Valid base32
-        assert hash_exists(content_hash, schema_name="test_schema", store_name="test_store") is False
-
-
 class TestDeletePath:
     """Tests for delete_path function."""
 
@@ -248,57 +216,3 @@ def test_returns_false_for_nonexistent(self, mock_get_backend):
 
         assert result is False
         mock_backend.remove.assert_not_called()
-
-
-class TestDeleteHash:
-    """Tests for delete_hash function (backward compatibility)."""
-
-    @patch("datajoint.hash_registry.get_store_subfolding")
-    @patch("datajoint.hash_registry.get_store_backend")
-    def test_deletes_existing_content(self, mock_get_backend, mock_get_subfolding):
-        """Test deleting existing content by hash."""
-        mock_backend = MagicMock()
-        mock_backend.exists.return_value = True
-        mock_get_backend.return_value = mock_backend
-        mock_get_subfolding.return_value = None
-
-        content_hash = "abcdefghijklmnopqrstuvwxyz"[:26]  # Valid base32
-        result = delete_hash(content_hash, schema_name="test_schema", store_name="test_store")
-
-        assert result is True
-        mock_backend.remove.assert_called_once()
-
-
-class TestGetSize:
-    """Tests for get_size function."""
-
-    @patch("datajoint.hash_registry.get_store_backend")
-    def test_returns_size(self, mock_get_backend):
-        """Test getting content size by path."""
-        mock_backend = MagicMock()
-        mock_backend.size.return_value = 1024
-        mock_get_backend.return_value = mock_backend
-
-        path = "_hash/test_schema/abcdefghijklmnopqrst"
-        result = get_size(path, store_name="test_store")
-
-        assert result == 1024
-        mock_backend.size.assert_called_once_with(path)
-
-
-class TestGetHashSize:
-    """Tests for get_hash_size function (backward compatibility)."""
-
-    @patch("datajoint.hash_registry.get_store_subfolding")
-    @patch("datajoint.hash_registry.get_store_backend")
-    def test_returns_size(self, mock_get_backend, mock_get_subfolding):
-        """Test getting content size by hash."""
-        mock_backend = MagicMock()
-        mock_backend.size.return_value = 1024
-        mock_get_backend.return_value = mock_backend
-        mock_get_subfolding.return_value = None
-
-        content_hash = "abcdefghijklmnopqrstuvwxyz"[:26]  # Valid base32
-        result = get_hash_size(content_hash, schema_name="test_schema", store_name="test_store")
-
-        assert result == 1024