fix(component): compare field values in BaseComponent.__eq__

bool() on a generator is always True, so any two same-class components
compared equal. Use all() to actually evaluate the per-field comparisons.

Author: FarhanAliRaza

packages/reflex-base/src/reflex_base/components/component.py

@@ -346,7 +346,7 @@ def __eq__(self, value: Any) -> bool:
         Returns:
             Whether the component is equal to the value.
         """
-        return type(self) is type(value) and bool(
+        return type(self) is type(value) and all(
             getattr(self, key) == getattr(value, key) for key in self.get_fields()
         )
 

tests/units/components/test_component.py

@@ -2301,3 +2301,36 @@ def test_ref():
     assert id_component._render().props["ref"].equals(Var("ref_custom_id"))
 
     assert "ref" not in rx.box()._render().props
+
+
+def test_component_equality_compares_fields():
+    """``BaseComponent.__eq__`` must compare field values, not just class identity.
+
+    HTML ``Element`` subclasses (``rx.box`` etc.) override ``__eq__`` to compare by
+    tag only, so this test uses a plain ``Component`` subclass to exercise the
+    base implementation.
+    """
+
+    class EqProbe(Component):
+        tag = "EqProbe"
+        label: str = ""
+
+    class OtherProbe(Component):
+        tag = "OtherProbe"
+        label: str = ""
+
+    a = EqProbe.create(label="x")
+    b = EqProbe.create(label="x")
+    c = EqProbe.create(label="y")
+
+    assert a == b
+    assert a != c
+
+    parent_same = EqProbe.create(EqProbe.create(label="leaf"), label="root")
+    parent_other = EqProbe.create(EqProbe.create(label="leaf"), label="root")
+    parent_diff = EqProbe.create(EqProbe.create(label="other"), label="root")
+    assert parent_same == parent_other
+    assert parent_same != parent_diff
+
+    assert EqProbe.create() != OtherProbe.create()
+    assert EqProbe.create() != "not a component"