Make email attachments' names simpler (Skyvern-AI#208)

skyvern/forge/sdk/workflow/models/block.py

@@ -9,7 +9,6 @@
 from pathlib import Path
 from tempfile import NamedTemporaryFile
 from typing import Annotated, Any, Literal, Union
-from urllib.parse import urlparse
 
 import filetype
 import structlog
@@ -822,10 +821,11 @@ async def _build_email_message(
                     extension = None
 
                 maintype, subtype = ctype.split("/", 1)
-                attachment_filename = urlparse(filename).path.replace("/", "_")
+                attachment_path = Path(path)
+                attachment_filename = attachment_path.name
 
                 # Check if the filename has an extension
-                if not Path(attachment_filename).suffix:
+                if not attachment_path.suffix:
                     # If no extension, guess it based on the MIME type
                     if extension:
                         attachment_filename += f".{extension}"