✂️ Avoid cropping images twice

This can happen when the cropped image is inside an symbol that
was detached and overrides the crop transform. This can cause
the code to generate the cropping mask twice for the same
image resulting in duplicate layers and UUIDs and crashing Sketch.

src/converter/base.py

@@ -85,7 +85,7 @@ def base_styled(fig_node: dict) -> _BaseStyled:
     if (
         obj["hasClippingMask"]
         and obj["clippingMaskMode"] == ClippingMaskMode.OUTLINE
-        and not fig_node.get("f2sForceOutline")
+        and not fig_node.get("f2s_cropped_image")
     ):
         # Outline mask behave differently in fig files and Sketch in regard to fill/stroke colors
         # Remove fill

src/converter/style.py

@@ -102,7 +102,7 @@ def convert_fill(fig_node: dict, fig_fill: dict) -> Fill:
                 isEnabled=fig_fill["visible"],
             )
         case {"type": "IMAGE"}:
-            if is_cropped_image(fig_fill):
+            if is_cropped_image(fig_fill) and not fig_node.get("f2s_cropped_image"):
                 # Extract images to a separate layer and retrigger conversion
                 convert_crop_image_to_mask(fig_node)
 
@@ -168,7 +168,7 @@ def convert_crop_image_to_mask(fig_node: dict) -> None:
                     **content,
                     "mask": True,
                     "maskType": "OUTLINE",
-                    "f2sForceOutline": True,
+                    "f2s_cropped_image": True,
                 },
                 *cropped_images,
             ],

src/converter/tree.py

@@ -54,7 +54,7 @@ def ignored_layer_type(fig_layer: dict) -> None:
 def convert_node(fig_node: dict, parent_type: str) -> AbstractLayer:
     name = fig_node["name"]
     type_ = get_node_type(fig_node, parent_type)
-    logging.debug(f"{type_}: {name}")
+    logging.debug(f"{type_}: {name} {fig_node['guid']}")
 
     try:
         sketch_item = CONVERTERS[type_](fig_node)