Apply suggestions from code review

- simplify code comments
- use `removeprefix` instead of `strip`

src/zarr/v3/group.py

@@ -312,13 +312,11 @@ async def members(self) -> AsyncGenerator[tuple[str, AsyncArray | AsyncGroup], N
                 try:
                     # TODO: performance optimization -- load children concurrently   
                     child = await self.getitem(key)
-                    # keyerror is raised when `key``names an object in the store
-                    # in which case `subkey` cannot be the name of a sub-array or sub-group.
                     yield key, child
                 except KeyError:
-                    # keyerror is raised when `subkey` names an object (in the object storage sense),
-                    # as opposed to a prefix, in the store under the prefix associated with this group
-                    # in which case `subkey` cannot be the name of a sub-array or sub-group.
+                    # keyerror is raised when `key` names an object (in the object storage sense),
+                    # as opposed to a prefix, in the store under the prefix associated with this group.
+                    # In this case, `key` cannot be the name of a sub-array or sub-group.
                     logger.warning(
                         "Object at %s is not recognized as a component of a Zarr hierarchy.", key
                     )

src/zarr/v3/store/memory.py

@@ -80,4 +80,4 @@ async def list_prefix(self, prefix: str) -> AsyncGenerator[str, None]:
     async def list_dir(self, prefix: str) -> AsyncGenerator[str, None]:
         for key in self._store_dict:
             if key.startswith(prefix + "/") and key != prefix:
-                yield key.strip(prefix + "/").rsplit("/", maxsplit=1)[0]
+                yield key.removeprefix(prefix + "/").rsplit("/", maxsplit=1)[0]