Delete commands.py and remove all Command class coupling from Ops

Author: knipknap

rayforge/builtin_addons/rayforge-addon-laser/tests/producers/test_contour_producer.py

@@ -40,7 +40,7 @@ def get_geo_from_artifact(artifact: WorkPieceArtifact) -> List[Geometry]:
     Ops are flattened, so we reconstruct geometries based on MoveTo commands.
     """
     # Create a single geometry from all ops
-    full_geo = Geometry.from_dict(artifact.ops.to_dict())
+    full_geo = artifact.ops.to_geometry()
     # Split into individual closed loops (contours)
     return full_geo.split_into_contours()
 
@@ -290,7 +290,7 @@ def test_cut_order_inside_outside(laser, dummy_surface, vector_workpiece):
     # The artifact.ops object contains sections. We need to convert to
     # geometries sequentially.
 
-    full_geo = Geometry.from_dict(artifact.ops.to_dict())
+    full_geo = artifact.ops.to_geometry()
     # split_into_contours preserves order of occurrence in command list
     contours = full_geo.split_into_contours()
 
@@ -319,7 +319,7 @@ def test_cut_order_outside_inside(laser, dummy_surface, vector_workpiece):
         generation_id=1,
     )
 
-    full_geo = Geometry.from_dict(artifact.ops.to_dict())
+    full_geo = artifact.ops.to_geometry()
     contours = full_geo.split_into_contours()
 
     assert len(contours) == 2
@@ -441,8 +441,8 @@ def test_overcut_zero_is_noop(laser, dummy_surface, vector_workpiece):
         generation_id=1,
     )
 
-    geo_a = Geometry.from_dict(artifact_a.ops.to_dict())
-    geo_b = Geometry.from_dict(artifact_b.ops.to_dict())
+    geo_a = artifact_a.ops.to_geometry()
+    geo_b = artifact_b.ops.to_geometry()
     assert geo_a == geo_b