Bug fixes

py_hamt/ipfszarr3.py

@@ -115,7 +115,12 @@ def supports_deletes(self) -> bool:
         return not self.hamt.read_only
 
     async def delete(self, key: str) -> None:
-        del self.hamt[key]
+        try:
+            del self.hamt[key]
+        # It's fine if the key was not in the HAMT
+        # Sometimes zarr v3 calls deletes on keys that don't exist (or have already been deleted) for some reason
+        except KeyError:
+            return
 
     @property
     def supports_listing(self) -> bool:

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "py-hamt"
-version = "2.4.0"
+version = "2.4.1"
 description = "HAMT implementation for an abstract backing store"
 readme = "README.md"
 requires-python = ">=3.12"
@@ -10,7 +10,7 @@ dependencies = [
     "multiformats[full]>=0.3.1.post4",
     "pycryptodome>=3.21.0",
     "requests>=2.32.3",
-    "zarr",
+    "zarr>=3.0.6",
 ]
 
 [build-system]
@@ -28,6 +28,6 @@ dev = [
     "snakeviz>=2.2.0",
     "pandas>=2.2.3",
     "numpy>=2.1.3",
-    "xarray>=2025.1.2",
     "pytest-asyncio>=0.25.3",
+    "xarray>=2025.3.0",
 ]

tests/test_zarr_ipfs.py

@@ -160,6 +160,8 @@ async def test_write_read(random_zarr_dataset: tuple[str, xr.Dataset]):
     # now remove that metadata file and then add it back
     ipfszarr3 = IPFSZarr3(ipfszarr3.hamt, read_only=False)  # make a writable version
     await ipfszarr3.delete("zarr.json")
+    # doing a duplicate delete should not result in an error
+    await ipfszarr3.delete("zarr.json")
     ipfszarr3_keys: set[str] = set()
     async for k in ipfszarr3.list():
         ipfszarr3_keys.add(k)
@@ -201,7 +203,8 @@ def test_encryption(random_zarr_dataset: tuple[str, xr.Dataset]):
 
     ds = xr.open_zarr(
         store=IPFSZarr3(
-            HAMT(store=IPFSStore(), root_node_id=hamt.root_node_id, read_only=True)
+            HAMT(store=IPFSStore(), root_node_id=ipfszarr3.hamt.root_node_id),
+            read_only=True,
         )
     )
     print(ds)