Enhance type hinting for optional imports in persistent storage modules

- Added type ignore comments for optional imports of plyvel and rocksdb to improve type checking.
- Updated the RocksDBDatastoreSync class to include a type ignore comment for the dynamically imported _rocksdb module, ensuring better compatibility with type checkers.

Author: yashksaini-coder

libp2p/peer/persistent/datastore/rocksdb_sync.py

@@ -111,7 +111,8 @@ def _ensure_connection(self) -> None:
                             setattr(opts, key, value)
 
                     # Open RocksDB database
-                    self.db = self._rocksdb.DB(str(self.path), opts)
+                    # pyrefly can't infer types for dynamically imported modules
+                    self.db = self._rocksdb.DB(str(self.path), opts)  # type: ignore[missing-attribute]
 
     def get(self, key: bytes) -> bytes | None:
         """Retrieve a value by key."""

libp2p/peer/persistent/factory.py

@@ -10,19 +10,19 @@
 
 # Optional imports - check availability once at module level
 try:
-    import plyvel
+    import plyvel  # type: ignore[import-untyped]
 
     PLYVEL_AVAILABLE = True
 except ImportError:
-    plyvel = None
+    plyvel = None  # type: ignore[assignment]
     PLYVEL_AVAILABLE = False
 
 try:
-    import rocksdb
+    import rocksdb  # type: ignore[import-untyped]
 
     ROCKSDB_AVAILABLE = True
 except ImportError:
-    rocksdb = None
+    rocksdb = None  # type: ignore[assignment]
     ROCKSDB_AVAILABLE = False
 
 from .async_.peerstore import AsyncPersistentPeerStore