[API Nodes] Send empty request body instead of empty dictionary. (#8)

Author: robinjhuang

comfy_api_nodes/apis/client.py

@@ -298,8 +298,8 @@ def execute(self, client: Optional[ApiClient] = None) -> R:
                     verify_ssl=self.verify_ssl,
                 )
 
-            # Convert request model to dict
-            request_dict = self.request.model_dump(exclude_none=True)
+            # Convert request model to dict, but use None for EmptyRequest
+            request_dict = None if isinstance(self.request, EmptyRequest) else self.request.model_dump(exclude_none=True)
 
             # Debug log for request
             logging.debug(f"[DEBUG] API Request: {self.endpoint.method.value} {self.endpoint.path}")
@@ -315,7 +315,11 @@ def execute(self, client: Optional[ApiClient] = None) -> R:
             )
 
             # Debug log for response
-            logging.debug(f"[DEBUG] API Response: {json.dumps(resp, indent=2)}")
+            logging.debug("=" * 50)
+            logging.debug("[DEBUG] RESPONSE DETAILS:")
+            logging.debug("[DEBUG] Status Code: 200 (Success)")
+            logging.debug(f"[DEBUG] Response Body: {json.dumps(resp, indent=2)}")
+            logging.debug("=" * 50)
 
             # Parse and return the response
             return self._parse_response(resp)

comfy_api_nodes/nodes_api.py

@@ -10,13 +10,6 @@
 import json
 import av
 
-
-def check_auth_token(auth_token):
-    """Verify that an auth token is present."""
-    if auth_token is None:
-        raise Exception("Please login first to use this node.")
-    return auth_token
-
 class IdeogramTextToImage(ComfyNodeABC):
     """
     Generates images synchronously based on a given prompt and optional parameters.
@@ -281,23 +274,26 @@ def generate_video(
         task_result = video_generate_operation.execute()
 
         file_id = task_result.file_id
-
+        if file_id is None:
+            raise Exception("Request was not successful. Missing file ID.")
         file_retrieve_operation = SynchronousOperation(
             endpoint=ApiEndpoint(
                 path="/proxy/minimax/files/retrieve",
                 method=HttpMethod.GET,
                 request_model=EmptyRequest,
                 response_model=MinimaxFileRetrieveResponse,
-                query_params={"file_id": file_id},
+                query_params={"file_id": int(file_id)},
             ),
             request=EmptyRequest(),
             auth_token=auth_token,
         )
         file_result = file_retrieve_operation.execute()
 
         file_url = file_result.file.download_url
-
+        if file_url is None:
+            raise Exception(f"No video was found in the response. Full response: {file_result.model_dump()}")
         logging.info(f"Generated video URL: {file_url}")
+        _, filename = folder_paths.download_url_to_file("comfyapinodes", file_url, folder_paths.get_output_directory())
 
         # Construct the save path
         full_output_folder, filename, counter, subfolder, filename_prefix = (