Fix sync for raw files whose display name has no extension

In dynamic folder mode, the display name of an uploaded raw file does not
include the extension (`notes`), but the public ID does
(`abc123.txt`). `_display_path` used the display name as is, so:

- `sync --push` did not match local `notes.txt` with the remote asset
  and uploaded it again on every run;
- `sync --pull` saved the file as `notes`, with no extension.

For raw assets, add the public ID extension to the display name when the
display name does not already end with it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Author: TalLevAmi

cloudinary_cli/utils/api_utils.py

@@ -124,7 +124,13 @@ def _display_path(asset):
     if asset.get("display_name") is None:
         return ""
 
-    if asset["resource_type"] == "raw" or asset["type"] == 'fetch':
+    if asset["resource_type"] == "raw":
+        # The display name of a raw asset can omit the extension that the public ID keeps.
+        normalized_display_name = asset["display_name"]
+        extension = path.splitext(asset["public_id"])[1]
+        if extension and not normalized_display_name.lower().endswith(extension.lower()):
+            normalized_display_name += extension
+    elif asset["type"] == 'fetch':
         normalized_display_name = asset["display_name"]
     else:
         normalized_display_name = ".".join(filter(None, [asset["display_name"], asset.get("format", None)]))

test/test_modules/test_cli_sync.py

@@ -10,7 +10,27 @@
 from test.helper_test import unique_suffix, RESOURCES_DIR, TEST_FILES_DIR, delete_cld_folder_if_exists, retry_assertion, \
     get_request_url, get_params, URLLIB3_REQUEST
 from test.test_modules.test_cli_upload_dir import UPLOAD_MOCK_RESPONSE
-from cloudinary_cli.utils.api_utils import get_folder_mode
+from cloudinary_cli.utils.api_utils import get_folder_mode, _display_path
+
+
+class TestDisplayPath(unittest.TestCase):
+    @staticmethod
+    def _asset(resource_type, public_id, display_name, fmt=None):
+        return {"resource_type": resource_type, "type": "upload", "public_id": public_id,
+                "display_name": display_name, "format": fmt, "asset_folder": "folder"}
+
+    def test_image_display_path_adds_format(self):
+        self.assertEqual("folder/red.png", _display_path(self._asset("image", "abc123", "red", "png")))
+
+    def test_raw_display_path_adds_missing_extension(self):
+        self.assertEqual("folder/notes.txt", _display_path(self._asset("raw", "abc123.txt", "notes")))
+
+    def test_raw_display_path_keeps_existing_extension(self):
+        self.assertEqual("folder/notes.txt", _display_path(self._asset("raw", "abc123.txt", "notes.txt")))
+        self.assertEqual("folder/notes.TXT", _display_path(self._asset("raw", "abc123.txt", "notes.TXT")))
+
+    def test_raw_display_path_without_extension(self):
+        self.assertEqual("folder/notes", _display_path(self._asset("raw", "abc123", "notes")))
 
 
 class TestCLISync(unittest.TestCase):