refactor(simprint): remove incomplete mutable multi-type methods

Remove get_raw_multi and delete_raw_multi stub methods from
LmdbSimprintIndexMulti. These were incomplete placeholders that
returned empty data or did nothing. The class now correctly implements
only the base SimprintIndexMulti protocol, not SimprintIndexMutableMulti.

Also remove associated tests for these non-functional methods.

Author: titusz

iscc_search/indexes/simprint/lmdb_multi.py

@@ -13,7 +13,6 @@
 
 from iscc_search.indexes.simprint.lmdb_core import LmdbSimprintIndex
 from iscc_search.indexes.simprint.models import (
-    SimprintEntryMulti,
     SimprintEntryRaw,
     SimprintMatchMulti,
     TypeMatchResult,
@@ -82,11 +81,14 @@ def add_raw_multi(self, entries):
         if self.realm_id is None:
             self._extract_realm_id(entries[0].iscc_id)
 
+        # Batch validate all realm_ids upfront (fail fast before processing)
+        for entry in entries:
+            self._validate_realm_id(entry.iscc_id)
+
         # Group entries by type for batch processing
         type_entries = defaultdict(list)  # type: dict[str, list[SimprintEntryRaw]]
 
         for entry in entries:
-            self._validate_realm_id(entry.iscc_id)
             iscc_id_body = entry.iscc_id[2:]  # Strip 2-byte header
 
             for simprint_type, simprints in entry.simprints.items():
@@ -193,45 +195,6 @@ def __contains__(self, iscc_id):
         iscc_id_body = iscc_id[2:]
         return any(iscc_id_body in index for index in self.indexes.values())
 
-    def get_raw_multi(self, iscc_ids):
-        # type: (list[bytes]) -> list[SimprintEntryMulti]
-        """
-        Retrieve indexed entries by their ISCC-IDs across all types.
-
-        :param iscc_ids: Full ISCC-ID digests (10 bytes each)
-        :return: List of entries (empty simprints dict for non-existent ISCC-IDs)
-        """
-        results = []
-
-        for iscc_id in iscc_ids:
-            iscc_id_body = iscc_id[2:]
-            simprints_by_type = {}  # type: dict[str, list[SimprintRaw]]
-
-            # Query each type-specific index
-            for simprint_type, index in self.indexes.items():
-                # LmdbSimprintIndex doesn't have get_raw, so we check containment only
-                # For full implementation, we'd need to add get_raw to LmdbSimprintIndex
-                if iscc_id_body in index:
-                    # Placeholder: actual retrieval would require get_raw method
-                    simprints_by_type[simprint_type] = []
-
-            results.append(SimprintEntryMulti(iscc_id=iscc_id, simprints=simprints_by_type))
-
-        return results
-
-    def delete_raw_multi(self, iscc_ids):
-        # type: (list[bytes]) -> None
-        """
-        Remove assets from all type-specific indexes by their ISCC-IDs.
-
-        :param iscc_ids: Full ISCC-ID digests (10 bytes each)
-        """
-        # Delete from each type-specific index
-        for index in self.indexes.values():
-            # LmdbSimprintIndex doesn't have delete_raw yet
-            # For full implementation, we'd need to add delete_raw to LmdbSimprintIndex
-            pass
-
     def close(self):
         # type: () -> None
         """

tests/test_indexes_simprint_lmdb_multi.py

@@ -307,44 +307,6 @@ def test_contains_invalid_length(temp_index_path):
     index.close()
 
 
-def test_get_raw_multi(temp_index_path, sample_entries):
-    # type: (Path, list[SimprintEntryMulti]) -> None
-    """Test get_raw_multi retrieves entries."""
-    index = LmdbSimprintIndexMulti(str(temp_index_path))
-    index.add_raw_multi(sample_entries)
-
-    results = index.get_raw_multi([sample_entries[0].iscc_id])
-    assert len(results) == 1
-    assert results[0].iscc_id == sample_entries[0].iscc_id
-    index.close()
-
-
-def test_get_raw_multi_non_existing(temp_index_path, sample_entries):
-    # type: (Path, list[SimprintEntryMulti]) -> None
-    """Test get_raw_multi with non-existing ISCC-IDs."""
-    index = LmdbSimprintIndexMulti(str(temp_index_path))
-    index.add_raw_multi(sample_entries)
-
-    non_existing = b"\x00\x10\xaa\xbb\xcc\xdd\xee\xff\x00\x11"
-    results = index.get_raw_multi([non_existing])
-
-    assert len(results) == 1
-    assert results[0].iscc_id == non_existing
-    assert results[0].simprints == {}
-    index.close()
-
-
-def test_delete_raw_multi(temp_index_path, sample_entries):
-    # type: (Path, list[SimprintEntryMulti]) -> None
-    """Test delete_raw_multi removes entries (currently a no-op)."""
-    index = LmdbSimprintIndexMulti(str(temp_index_path))
-    index.add_raw_multi(sample_entries)
-
-    # Delete doesn't raise error (even though not fully implemented)
-    index.delete_raw_multi([sample_entries[0].iscc_id])
-    index.close()
-
-
 def test_close_releases_resources(temp_index_path, sample_entries):
     # type: (Path, list[SimprintEntryMulti]) -> None
     """Test close releases all index resources."""
@@ -475,20 +437,6 @@ def test_search_result_ordering(temp_index_path):
     index.close()
 
 
-def test_delete_raw_multi_with_indexes(temp_index_path, sample_entries):
-    # type: (Path, list[SimprintEntryMulti]) -> None
-    """Test delete_raw_multi with populated indexes."""
-    index = LmdbSimprintIndexMulti(str(temp_index_path))
-    index.add_raw_multi(sample_entries)
-
-    # Ensure indexes exist
-    assert len(index.indexes) > 0
-
-    # Delete should iterate over indexes (even if not fully implemented)
-    index.delete_raw_multi([sample_entries[0].iscc_id])
-    index.close()
-
-
 def test_context_manager_with_exception(temp_index_path, sample_entries):
     # type: (Path, list[SimprintEntryMulti]) -> None
     """Test context manager properly closes even with exception."""