feat: accept content in upload_translation_memory_file() and deprecate input_file

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

Author: hive-bot

CHANGELOG.md

@@ -143,6 +143,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   `expired`, and the plain string for any other value. Members are strings
   that compare equal to the API value, so existing comparisons with strings
   and the `TranslationMemoryJobResult.STATUS_*` constants keep working.
+- Added the `content` parameter name to `upload_translation_memory_file()`
+  for the file content, replacing `input_file`. Positional calls are
+  unaffected.
 
 ### Changed
 - HTTP 404 responses raise `NotFoundException` instead of `DeepLException`
@@ -214,6 +217,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - The argument names `glossary_dicts`, `glossary_dict`, `csv_data` and
   `glossary_id` of the multilingual glossary functions are deprecated, use
   `dictionaries`, `dictionary`, `csv` and `glossary`.
+- The `input_file` keyword of `upload_translation_memory_file()` is
+  deprecated in favour of `content` and will be removed in the next major
+  version. Passing it emits a `DeprecationWarning`.
 
 ### Fixed
 - HTTP 401 responses now raise `AuthorizationException`, like 403. Before they

README.md

@@ -1235,6 +1235,12 @@ an upload URL that the file must be uploaded to before processing starts, then
 `create_translation_memory_import()` takes the file name and size, and the
 optional keyword arguments `content_type`, `display_name` and
 `extra_body_parameters`, with the same meaning as above.
+`upload_translation_memory_file()` takes the import job (or its upload URL),
+the file `content` as bytes or a binary file object, and an optional
+`content_type` (default `application/xml`) that must match the one declared
+when creating the job. The keyword `input_file` is a deprecated alias of
+`content`: it emits a `DeprecationWarning` and will be removed in the next
+major version.
 
 ```python
 created = deepl_client.create_translation_memory_import(
@@ -1243,7 +1249,7 @@ created = deepl_client.create_translation_memory_import(
     display_name="Legal TM",
 )
 with open("/path/to/legal.tmx", "rb") as input_file:
-    deepl_client.upload_translation_memory_file(created, input_file)
+    deepl_client.upload_translation_memory_file(created, content=input_file)
 
 # The job reports "awaiting_input" both before the upload and for a while
 # afterwards, until the API detects it; this polls through that status.

deepl/deepl_client.py

@@ -2068,19 +2068,40 @@ def create_translation_memory_import(
     def upload_translation_memory_file(
         self,
         translation_memory_import: Union[str, TranslationMemoryImport],
-        input_file: Union[BinaryIO, bytes, Any],
+        content: Union[BinaryIO, bytes, Any] = None,
         content_type: str = "application/xml",
+        *,
+        input_file: Union[BinaryIO, bytes, Any] = None,
     ) -> None:
         """Uploads a TMX file to the upload URL of an import job, which starts
         processing.
 
         :param translation_memory_import: TranslationMemoryImport returned by
             create_translation_memory_import(), or its upload URL.
-        :param input_file: TMX file content as bytes or a file-like object
+        :param content: TMX file content as bytes or a file-like object
             opened in binary mode.
         :param content_type: (Optional) MIME type of the file. Must match the
             content_type declared when the import job was created.
+        :param input_file: Deprecated alias of content.
+
+            .. deprecated::
+                Use content instead. input_file will be removed in the next
+                major version.
         """
+        if input_file is not None:
+            if content is not None:
+                raise ValueError("pass either content or input_file, not both")
+            warnings.warn(
+                "input_file is deprecated, use content",
+                DeprecationWarning,
+                stacklevel=2,
+            )
+            content = input_file
+        if content is None:
+            raise TypeError(
+                "upload_translation_memory_file() missing required argument: "
+                "'content'"
+            )
         if isinstance(translation_memory_import, TranslationMemoryImport):
             upload_url = translation_memory_import.upload_url
         else:
@@ -2091,17 +2112,17 @@ def upload_translation_memory_file(
         # Read a file-like object into bytes up front. The request is prepared
         # once and reused across retries, so a stream body would already be at
         # EOF on the second attempt and upload nothing.
-        if hasattr(input_file, "read"):
-            input_file = input_file.read()
+        if hasattr(content, "read"):
+            content = content.read()
 
-        status_code, content = self._asset_call(
+        status_code, response_content = self._asset_call(
             "PUT",
             upload_url,
-            data=input_file,
+            data=content,
             headers={"Content-Type": content_type},
         )
         self._raise_for_asset_status(
-            status_code, content, "uploading translation memory file"
+            status_code, response_content, "uploading translation memory file"
         )
 
     def create_translation_memory_export(

tests/offline/test_translation_memories.py

@@ -329,10 +329,28 @@ def test_sends_given_content_type_when_uploading_file(mocked):
     )
 
     make_deepl_client().upload_translation_memory_file(
-        UPLOAD_URL, b"<tmx/>", content_type="text/xml"
+        UPLOAD_URL, content=b"<tmx/>", content_type="text/xml"
     )
 
 
+def test_warns_when_uploading_file_with_input_file_keyword(mocked):
+    mocked.put(UPLOAD_URL, match=storage_request("application/xml", "<tmx/>"))
+
+    with pytest.warns(
+        DeprecationWarning, match="input_file is deprecated, use content"
+    ):
+        make_deepl_client().upload_translation_memory_file(
+            UPLOAD_URL, input_file=b"<tmx/>"
+        )
+
+
+def test_raises_when_uploading_file_with_content_and_input_file():
+    with pytest.raises(ValueError, match="either content or input_file"):
+        make_deepl_client().upload_translation_memory_file(
+            UPLOAD_URL, b"a", input_file=b"b"
+        )
+
+
 def test_raises_when_uploading_file_without_url():
     with pytest.raises(ValueError, match="upload_url must not be empty"):
         make_deepl_client().upload_translation_memory_file("", b"x")