Replace _intf to __rid__ for identifying interface object (#194)

* Fix hashing for dict contain list

* Fix linter error

* Remove _rintf, and use __rid__

javascript/package.json

@@ -1,6 +1,6 @@
 {
   "name": "imjoy-rpc",
-  "version": "0.3.28",
+  "version": "0.3.29",
   "description": "Remote procedure calls for ImJoy.",
   "module": "index.js",
   "scripts": {

python/imjoy_rpc/VERSION

@@ -1,3 +1,3 @@
 {
-    "version": "0.3.28"
+    "version": "0.3.29"
 }

python/imjoy_rpc/rpc.py

@@ -861,9 +861,10 @@ def _decode(self, a_object, with_promise):
         if isinstance(a_object, dict) and a_object.get("_rintf"):
             # make the dict hashable
             if isinstance(b_object, dict):
-                b_object["_rintf"] = a_object.get("_rintf")
                 if not isinstance(b_object, dotdict):
                     b_object = dotdict(b_object)
+                # __rid__ is used for hashing the object for removing it afterwards
+                b_object.__rid__ = a_object.get("_rintf")
 
             self._object_weakmap[b_object] = a_object.get("_rintf")
         return b_object

python/imjoy_rpc/utils.py

@@ -30,10 +30,18 @@ class dotdict(dict):  # pylint: disable=invalid-name
     __setattr__ = dict.__setitem__
     __delattr__ = dict.__delitem__
 
+    def __setattr__(self, name, value):
+        """Set the attribute."""
+        # Make an exception for __rid__
+        if name == "__rid__":
+            super().__setattr__("__rid__", value)
+        else:
+            super().__setitem__(name, value)
+
     def __hash__(self):
         """Return the hash."""
-        if "_rintf" in self and type(self["_rintf"]) is str:
-            return hash(self["_rintf"] + _hash_id)
+        if self.__rid__ and type(self.__rid__) is str:
+            return hash(self.__rid__ + _hash_id)
 
         # FIXME: This does not address the issue of inner list
         return hash(tuple(sorted(self.items())))