@autorest/python 6.8.0 → 6.9.0

package/autorest/codegen/models/combined_type.py

@@ -43,7 +43,9 @@ class CombinedType(BaseType):
         If list: '[str]'
         If dict: '{str}'
         """
-        raise ValueError("Shouldn't get serialization type of a combinedtype")
+        if not all(t for t in self.types if t.type == "constant"):
+            raise ValueError("Shouldn't get serialization type of a combinedtype")
+        return self.types[0].serialization_type
 
     @property
     def client_default_value(self) -> Any:

package/autorest/codegen/models/paging_operation.py

@@ -15,6 +15,7 @@ from .request_builder import (
 from .imports import ImportType, FileImport, TypingSection
 from .parameter_list import ParameterList
 from .model_type import ModelType
+from .list_type import ListType
 
 if TYPE_CHECKING:
     from .code_model import CodeModel
@@ -63,11 +64,17 @@ class PagingOperationBase(OperationBase[PagingResponseType]):
         self.pager_async: str = yaml_data["pagerAsync"]
 
     def _get_attr_name(self, wire_name: str) -> str:
-        response = self.responses[0]
+        response_type = self.responses[0].type
+        if not response_type:
+            raise ValueError(
+                f"Can't find a matching property in response for {wire_name}"
+            )
+        if response_type.type == "list":
+            response_type = cast(ListType, response_type).element_type
         try:
             return next(
                 p.client_name
-                for p in cast(ModelType, response.type).properties
+                for p in cast(ModelType, response_type).properties
                 if p.wire_name == wire_name
             )
         except StopIteration as exc:

package/autorest/m4reformatter/__init__.py

@@ -1126,10 +1126,7 @@ class M4Reformatter(
             "skipUrlEncoding": True,
             "inOverload": False,
         }
-        if self.version_tolerant:
-            parameters.append(credential)
-        else:
-            parameters.insert(0, credential)
+        parameters.append(credential)
 
     def update_client(self, yaml_data: Dict[str, Any]) -> Dict[str, Any]:
         parameters = self.update_global_parameters(

package/autorest/preprocess/__init__.py

@@ -281,7 +281,8 @@ class PreProcessPlugin(YamlUpdatePlugin):  # pylint: disable=abstract-method
             yaml_data["description"], default_description=yaml_data["name"]
         )
         yaml_data["legacyFilename"] = to_snake_case(yaml_data["name"].replace(" ", "_"))
-        for parameter in yaml_data["parameters"]:
+        parameters = yaml_data["parameters"]
+        for parameter in parameters:
             self.update_parameter(parameter)
             if parameter["clientName"] == "credential":
                 policy = parameter["type"].get("policy")
@@ -294,7 +295,13 @@ class PreProcessPlugin(YamlUpdatePlugin):  # pylint: disable=abstract-method
                     policy["credentialScopes"] = [
                         "https://management.azure.com/.default"
                     ]
-
+        if (
+            (not self.version_tolerant or self.azure_arm)
+            and parameters
+            and parameters[-1]["clientName"] == "credential"
+        ):
+            # we need to move credential to the front in mgmt mode for backcompat reasons
+            yaml_data["parameters"] = [parameters[-1]] + parameters[:-1]
         prop_name = yaml_data["name"]
         if prop_name.endswith("Client"):
             prop_name = prop_name[: len(prop_name) - len("Client")]

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@autorest/python",
-  "version": "6.8.0",
+  "version": "6.9.0",
   "description": "The Python extension for generators in AutoRest.",
   "main": "index.js",
   "repository": {