feat: add highlight_type to files.completeUploadExternal and files_upload_v2

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

Author: zimeg

slack_sdk/web/async_client.py

@@ -4037,6 +4037,7 @@ async def files_upload_v2(
         title: Optional[str] = None,
         alt_txt: Optional[str] = None,
         snippet_type: Optional[str] = None,
+        highlight_type: Optional[str] = None,
         # To upload multiple files at a time
         file_uploads: Optional[List[Dict[str, Any]]] = None,
         channel: Optional[str] = None,
@@ -4081,6 +4082,7 @@ async def files_upload_v2(
                     "title": title,
                     "alt_txt": alt_txt,
                     "snippet_type": snippet_type,
+                    "highlight_type": highlight_type,
                 }
             )
             files.append(f)
@@ -4118,7 +4120,7 @@ async def files_upload_v2(
 
         # step3: files.completeUploadExternal with all the sets of (file_id + title)
         completion = await self.files_completeUploadExternal(
-            files=[{"id": f["file_id"], "title": f["title"]} for f in files],
+            files=[{"id": f["file_id"], "title": f["title"], "highlight_type": f.get("highlight_type")} for f in files],
             channel_id=channel,
             channels=channels,
             initial_comment=initial_comment,

slack_sdk/web/client.py

@@ -4027,6 +4027,7 @@ def files_upload_v2(
         title: Optional[str] = None,
         alt_txt: Optional[str] = None,
         snippet_type: Optional[str] = None,
+        highlight_type: Optional[str] = None,
         # To upload multiple files at a time
         file_uploads: Optional[List[Dict[str, Any]]] = None,
         channel: Optional[str] = None,
@@ -4071,6 +4072,7 @@ def files_upload_v2(
                     "title": title,
                     "alt_txt": alt_txt,
                     "snippet_type": snippet_type,
+                    "highlight_type": highlight_type,
                 }
             )
             files.append(f)
@@ -4108,7 +4110,7 @@ def files_upload_v2(
 
         # step3: files.completeUploadExternal with all the sets of (file_id + title)
         completion = self.files_completeUploadExternal(
-            files=[{"id": f["file_id"], "title": f["title"]} for f in files],
+            files=[{"id": f["file_id"], "title": f["title"], "highlight_type": f.get("highlight_type")} for f in files],
             channel_id=channel,
             channels=channels,
             initial_comment=initial_comment,

slack_sdk/web/internal_utils.py

@@ -379,6 +379,7 @@ def _to_v2_file_upload_item(upload_file: Dict[str, Any]) -> Dict[str, Optional[A
         "title": title,
         "alt_txt": upload_file.get("alt_txt"),
         "snippet_type": upload_file.get("snippet_type"),
+        "highlight_type": upload_file.get("highlight_type"),
     }
 
 

slack_sdk/web/legacy_client.py

@@ -3962,6 +3962,7 @@ def files_upload_v2(
         title: Optional[str] = None,
         alt_txt: Optional[str] = None,
         snippet_type: Optional[str] = None,
+        highlight_type: Optional[str] = None,
         # To upload multiple files at a time
         file_uploads: Optional[List[Dict[str, Any]]] = None,
         channel: Optional[str] = None,
@@ -4006,6 +4007,7 @@ def files_upload_v2(
                     "title": title,
                     "alt_txt": alt_txt,
                     "snippet_type": snippet_type,
+                    "highlight_type": highlight_type,
                 }
             )
             files.append(f)
@@ -4043,7 +4045,7 @@ def files_upload_v2(
 
         # step3: files.completeUploadExternal with all the sets of (file_id + title)
         completion = self.files_completeUploadExternal(
-            files=[{"id": f["file_id"], "title": f["title"]} for f in files],
+            files=[{"id": f["file_id"], "title": f["title"], "highlight_type": f.get("highlight_type")} for f in files],
             channel_id=channel,
             channels=channels,
             initial_comment=initial_comment,

tests/slack_sdk/web/test_internal_utils.py

@@ -133,6 +133,13 @@ def test_files_upload_v2_issue_1356(self):
         file_io_item = _to_v2_file_upload_item({"file": file_io, "filename": "foo.txt"})
         assert file_io_item.get("filename") == "foo.txt"
 
+    def test_to_v2_file_upload_item_passes_through_highlight_type(self):
+        item = _to_v2_file_upload_item({"content": "test", "filename": "image.png", "highlight_type": "png"})
+        assert item.get("highlight_type") == "png"
+
+        item_without = _to_v2_file_upload_item({"content": "test", "filename": "image.png"})
+        assert item_without.get("highlight_type") is None
+
     def test_to_v2_file_upload_item_can_accept_file_as_path(self):
         filepath = "tests/slack_sdk/web/test_internal_utils.py"
         upload_item_str = _to_v2_file_upload_item({"file": filepath})