fix: Allow duplicate sub-assemblies

Author: nathaniel-brough

cadquery/assembly.py

@@ -244,9 +244,8 @@ def add(self, arg, **kwargs):
             # enforce unique names
             name = kwargs["name"] if kwargs.get("name") else arg.name
             if name in self.objects:
-                raise ValueError(
-                    f"Unique name is required. {name} is already in the assembly"
-                )
+                # Add a incremented number to the name
+                name = f"{name}_{len(self.objects)}"
 
             subassy = arg._copy()
 

tests/test_assembly.py

@@ -1792,12 +1792,6 @@ def test_constrain_with_tags(nested_assy):
         nested_assy.constrain("SECOND/BOTTOM ? pts", "dummy", "Plane")
 
 
-def test_duplicate_name(nested_assy):
-
-    with pytest.raises(ValueError):
-        nested_assy.add(None, name="SECOND")
-
-
 def test_empty_solve(nested_assy):
 
     with pytest.raises(ValueError):