PR review resolutions

Author: luxaritas

python/python-core/src/main/python/domain/_annotations.py

@@ -654,6 +654,7 @@ def __init__(self, *,
 class DeepPlanningClone(JavaAnnotation):
     """
     Marks an attribute as being required to be deep planning cloned.
+    Not needed for `planning_solution` or `planning_entity` attributes because those are automatically deep cloned.
     This is especially useful for `list` (or `dict`) properties.
     Not needed for a `list` (or `dist`) attribute with a generic type of `planning_entity`,
     because those are automatically deep cloned.

python/python-core/tests/test_domain.py

@@ -957,7 +957,7 @@ class Solution:
 
 def test_deep_clone_class():
     @deep_planning_clone
-    @dataclass
+    @dataclass(eq=False)
     class Code:
         value: str
         parent_entity: 'Entity' = field(default=None)
@@ -1019,3 +1019,4 @@ class Solution:
     assert solution.score.score == 2
     assert solution.entities[0].value == v1
     assert solution.codes[0].parent_entity == solution.entities[0]
+    assert solution.codes[0] != e1.code