@autorest/python 6.4.10 → 6.4.12

package/autorest/codegen/models/client.py

@@ -146,8 +146,14 @@ class Client(_ClientConfigBase[ClientGlobalParameterList]):
         retval = add_to_pylint_disable("", "client-accepts-api-version-keyword")
         if len(self.operation_groups) > 6:
             retval = add_to_pylint_disable(retval, "too-many-instance-attributes")
-        if len(self.name) > 40:
-            retval = add_to_pylint_disable(retval, "name-too-long")
+        return retval
+
+    @property
+    def url_pylint_disable(self) -> str:
+        # if the url is too long
+        retval = ""
+        if len(self.url) > 85:
+            retval = add_to_pylint_disable(retval, "line-too-long")
         return retval
 
     @property

package/autorest/codegen/models/code_model.py

@@ -184,7 +184,10 @@ class CodeModel:  # pylint: disable=too-many-public-methods
         """All of the model types in this class"""
         if not self._model_types:
             self._model_types = [
-                t for t in self.types_map.values() if isinstance(t, ModelType)
+                t
+                for t in self.types_map.values()
+                if isinstance(t, ModelType)
+                and not (self.options["models_mode"] == "dpg" and t.page_result_model)
             ]
         return self._model_types
 

package/autorest/codegen/models/model_type.py

@@ -74,6 +74,7 @@ class ModelType(  # pylint: disable=abstract-method
         self._got_polymorphic_subtypes = False
         self.internal: bool = self.yaml_data.get("internal", False)
         self.snake_case_name: str = self.yaml_data["snakeCaseName"]
+        self.page_result_model: bool = self.yaml_data.get("pageResultModel", False)
 
     @property
     def is_xml(self) -> bool:
@@ -129,7 +130,7 @@ class ModelType(  # pylint: disable=abstract-method
         # don't add additional properties, because there's not really a concept of
         # additional properties in the template
         representation = {
-            f'"{prop.rest_api_name}"': prop.get_json_template_representation(
+            f'"{prop.wire_name}"': prop.get_json_template_representation(
                 optional=optional,
                 client_default_value_declaration=client_default_value_declaration,
                 description=description,
@@ -142,16 +143,14 @@ class ModelType(  # pylint: disable=abstract-method
         }
         if self.discriminator and self.discriminator_value:
             representation[
-                f'"{self.discriminator.rest_api_name}"'
+                f'"{self.discriminator.wire_name}"'
             ] = f'"{self.discriminator_value}"'
 
         # once we've finished, we want to reset created_json_template_representation to false
         # so we can call it again
         self._created_json_template_representation = False
         optional_keys = [
-            f'"{p.rest_api_name}"'
-            for p in self.properties
-            if getattr(p, "optional", False)
+            f'"{p.wire_name}"' for p in self.properties if getattr(p, "optional", False)
         ]
         return OrderedDict(
             sorted(

package/autorest/codegen/models/operation.py

@@ -130,8 +130,6 @@ class OperationBase(  # pylint: disable=too-many-public-methods
         if self.response_type_annotation(async_mode=False) == "None":
             # doesn't matter if it's async or not
             retval = add_to_pylint_disable(retval, "inconsistent-return-statements")
-        if len(self.name) > 40:
-            retval = add_to_pylint_disable(retval, "name-too-long")
         return retval
 
     def cls_type_annotation(self, *, async_mode: bool) -> str:

package/autorest/codegen/models/paging_operation.py

@@ -62,17 +62,17 @@ class PagingOperationBase(OperationBase[PagingResponseType]):
         self.pager_sync: str = yaml_data["pagerSync"]
         self.pager_async: str = yaml_data["pagerAsync"]
 
-    def _get_attr_name(self, rest_api_name: str) -> str:
+    def _get_attr_name(self, wire_name: str) -> str:
         response = self.responses[0]
         try:
             return next(
                 p.client_name
                 for p in cast(ModelType, response.type).properties
-                if p.rest_api_name == rest_api_name
+                if p.wire_name == wire_name
             )
         except StopIteration as exc:
             raise ValueError(
-                f"Can't find a matching property in response for {rest_api_name}"
+                f"Can't find a matching property in response for {wire_name}"
             ) from exc
 
     def get_pager(self, async_mode: bool) -> str:
@@ -80,21 +80,21 @@ class PagingOperationBase(OperationBase[PagingResponseType]):
 
     @property
     def continuation_token_name(self) -> Optional[str]:
-        rest_api_name = self.yaml_data["continuationTokenName"]
-        if not rest_api_name:
+        wire_name = self.yaml_data["continuationTokenName"]
+        if not wire_name:
             # That's an ok scenario, it just means no next page possible
             return None
         if self.code_model.options["models_mode"] == "msrest":
-            return self._get_attr_name(rest_api_name)
-        return rest_api_name
+            return self._get_attr_name(wire_name)
+        return wire_name
 
     @property
     def item_name(self) -> str:
-        rest_api_name = self.yaml_data["itemName"]
+        wire_name = self.yaml_data["itemName"]
         if self.code_model.options["models_mode"] == "msrest":
             # we don't use the paging model for dpg
-            return self._get_attr_name(rest_api_name)
-        return rest_api_name
+            return self._get_attr_name(wire_name)
+        return wire_name
 
     @property
     def item_type(self) -> ModelType:
@@ -156,7 +156,14 @@ class PagingOperationBase(OperationBase[PagingResponseType]):
                 "azure.core.utils", "case_insensitive_dict", ImportType.AZURECORE
             )
         if self.code_model.options["models_mode"] == "dpg":
+            relative_path = "..." if async_mode else ".."
             file_import.merge(self.item_type.imports(**kwargs))
+            if self.default_error_deserialization or any(
+                r.type for r in self.responses
+            ):
+                file_import.add_submodule_import(
+                    f"{relative_path}_model_base", "_deserialize", ImportType.LOCAL
+                )
         return file_import
 
 

package/autorest/codegen/models/parameter.py

@@ -64,7 +64,7 @@ class _ParameterBase(
         type: BaseType,
     ) -> None:
         super().__init__(yaml_data, code_model)
-        self.rest_api_name: str = yaml_data["restApiName"]
+        self.wire_name: str = yaml_data["wireName"]
         self.client_name: str = self.yaml_data["clientName"]
         self.optional: bool = self.yaml_data["optional"]
         self.location: ParameterLocation = self.yaml_data["location"]
@@ -342,7 +342,7 @@ class Parameter(_ParameterBase):
 
     @property
     def in_method_signature(self) -> bool:
-        return not (self.rest_api_name == "Accept" or self.grouped_by or self.flattened)
+        return not (self.wire_name == "Accept" or self.grouped_by or self.flattened)
 
     @property
     def full_client_name(self) -> str:
@@ -356,7 +356,7 @@ class Parameter(_ParameterBase):
 
     @property
     def is_content_type(self) -> bool:
-        return bool(self.rest_api_name) and self.rest_api_name.lower() == "content-type"
+        return bool(self.wire_name) and self.wire_name.lower() == "content-type"
 
     @property
     def method_location(  # pylint: disable=too-many-return-statements
@@ -399,7 +399,7 @@ class ClientParameter(Parameter):
 
     @property
     def is_host(self) -> bool:
-        return self.rest_api_name == "$host"
+        return self.wire_name == "$host"
 
     @property
     def method_location(self) -> ParameterMethodLocation:
@@ -423,7 +423,7 @@ class ConfigParameter(Parameter):
 
     @property
     def is_host(self) -> bool:
-        return self.rest_api_name == "$host"
+        return self.wire_name == "$host"
 
     @property
     def method_location(self) -> ParameterMethodLocation:

package/autorest/codegen/models/property.py

@@ -24,7 +24,7 @@ class Property(BaseModel):  # pylint: disable=too-many-instance-attributes
         type: BaseType,
     ) -> None:
         super().__init__(yaml_data, code_model)
-        self.rest_api_name: str = self.yaml_data["restApiName"]
+        self.wire_name: str = self.yaml_data["wireName"]
         self.client_name: str = self.yaml_data["clientName"]
         self.type = type
         self.optional: bool = self.yaml_data["optional"]

package/autorest/codegen/models/request_builder.py

@@ -16,7 +16,6 @@ from typing import (
 from abc import abstractmethod
 
 from .base_builder import BaseBuilder
-from .utils import add_to_pylint_disable
 from .parameter_list import (
     RequestBuilderParameterList,
     OverloadedRequestBuilderParameterList,
@@ -57,12 +56,6 @@ class RequestBuilderBase(BaseBuilder[ParameterListType]):
         self.method: str = yaml_data["method"]
         self.want_tracing = False
 
-    @property
-    def pylint_disable(self) -> str:
-        if len(self.name) > 40:
-            return add_to_pylint_disable("", "name-too-long")
-        return ""
-
     def response_type_annotation(self, **kwargs) -> str:
         return "HttpRequest"
 

package/autorest/codegen/models/response.py

@@ -26,7 +26,7 @@ class ResponseHeader(BaseModel):
         type: BaseType,
     ) -> None:
         super().__init__(yaml_data, code_model)
-        self.rest_api_name: str = yaml_data["restApiName"]
+        self.wire_name: str = yaml_data["wireName"]
         self.type = type
 
     @property
@@ -59,6 +59,13 @@ class Response(BaseModel):
         self.type = type
         self.nullable = yaml_data.get("nullable")
 
+    @property
+    def result_property(self) -> str:
+        field = self.yaml_data.get("resultProperty")
+        if field:
+            return f'.get("{field}")'
+        return ""
+
     def get_polymorphic_subtypes(self, polymorphic_subtypes: List["ModelType"]) -> None:
         if self.type:
             self.type.get_polymorphic_subtypes(polymorphic_subtypes)

package/autorest/codegen/serializers/__init__.py

@@ -48,6 +48,16 @@ _PACKAGE_FILES = [
 
 _REGENERATE_FILES = {"setup.py", "MANIFEST.in"}
 
+# extract sub folders. For example, source_file_path is like:
+# "xxx/resource-manager/Microsoft.XX/stable/2023-04-01/examples/Compute/createOrUpdate/AKSCompute.json",
+# and we want to extract the sub folders after "examples/", which is "compute/create_or_update"
+def _sample_output_path(source_file_path: str) -> Path:
+    posix_path = Path(source_file_path).as_posix()
+    if "examples/" in posix_path:
+        after_examples = Path(posix_path.split("examples/", maxsplit=1)[-1]).parent
+        return Path("/".join([to_snake_case(i) for i in after_examples.parts]))
+    return Path("")
+
 
 class JinjaSerializer(ReaderAndWriter):  # pylint: disable=abstract-method
     def __init__(
@@ -568,7 +578,7 @@ class JinjaSerializer(ReaderAndWriter):  # pylint: disable=abstract-method
                         file_name = to_snake_case(extract_sample_name(file)) + ".py"
                         try:
                             self.write_file(
-                                out_path / file_name,
+                                out_path / _sample_output_path(file) / file_name,
                                 SampleSerializer(
                                     code_model=self.code_model,
                                     env=env,

package/autorest/codegen/serializers/builder_serializer.py

@@ -200,7 +200,7 @@ def _serialize_multipart_body(builder: BuilderType) -> List[str]:
     retval.append("# Construct form data")
     retval.append(f"_{body_param.client_name} = {{")
     for param in body_param.entries:
-        retval.append(f'    "{param.rest_api_name}": {param.client_name},')
+        retval.append(f'    "{param.wire_name}": {param.client_name},')
     retval.append("}")
     return retval
 
@@ -396,7 +396,7 @@ class _BuilderBaseSerializer(Generic[BuilderType]):  # pylint: disable=abstract-
             # we just assume one kind of polymorphic body for input
             discriminator_name = cast(
                 Property, polymorphic_subtypes[0].discriminator
-            ).rest_api_name
+            ).wire_name
             template.append(
                 "# The input is polymorphic. The following are possible polymorphic "
                 f'inputs based off discriminator "{discriminator_name}":'
@@ -425,7 +425,7 @@ class _BuilderBaseSerializer(Generic[BuilderType]):  # pylint: disable=abstract-
     def _serialize_parameter(self, param: Parameter, kwarg_name: str) -> List[str]:
         set_parameter = "_{}['{}'] = {}".format(
             kwarg_name,
-            param.rest_api_name,
+            param.wire_name,
             self.parameter_serializer.serialize_parameter(param, self.serializer_name),
         )
         if not param.optional:
@@ -480,7 +480,7 @@ class RequestBuilderSerializer(
                     if param.location == ParameterLocation.HEADER
                     else "params"
                 )
-                return f"_{kwarg_dict}.pop('{param.rest_api_name}', {param_type.get_declaration()})"
+                return f"_{kwarg_dict}.pop('{param.wire_name}', {param_type.get_declaration()})"
             return f"{param_type.get_declaration()}"
 
         return [
@@ -614,7 +614,7 @@ class _OperationSerializer(
                 # we just assume one kind of polymorphic body for input
                 discriminator_name = cast(
                     Property, polymorphic_subtypes[0].discriminator
-                ).rest_api_name
+                ).wire_name
                 retval.append(
                     "# The response is polymorphic. The following are possible polymorphic "
                     f'responses based off discriminator "{discriminator_name}":'
@@ -792,7 +792,7 @@ class _OperationSerializer(
                 and not next(
                     p
                     for p in builder.parameters
-                    if p.rest_api_name.lower() == "content-type"
+                    if p.wire_name.lower() == "content-type"
                 ).optional
             ):
                 content_types = "'" + "', '".join(body_param.content_types) + "'"
@@ -1043,8 +1043,8 @@ class _OperationSerializer(
     ) -> List[str]:
         retval: List[str] = [
             (
-                f"response_headers['{response_header.rest_api_name}']=self._deserialize("
-                f"'{response_header.serialization_type}', response.headers.get('{response_header.rest_api_name}'))"
+                f"response_headers['{response_header.wire_name}']=self._deserialize("
+                f"'{response_header.serialization_type}', response.headers.get('{response_header.wire_name}'))"
             )
             for response_header in response.headers
         ]
@@ -1075,7 +1075,9 @@ class _OperationSerializer(
                 deserialize_code.append(
                     f"    {response.type.type_annotation(is_operation_file=True)},{pylint_disable}"
                 )
-                deserialize_code.append("    response.json()")
+                deserialize_code.append(
+                    f"    response.json(){response.result_property}"
+                )
                 deserialize_code.append(")")
             else:
                 deserialized_value = (

package/autorest/codegen/serializers/model_serializer.py

@@ -188,7 +188,7 @@ class MsrestModelSerializer(_ModelSerializer):
                 _ModelSerializer.escape_dot(n) for n in prop.flattened_names
             )
         else:
-            attribute_key = _ModelSerializer.escape_dot(prop.rest_api_name)
+            attribute_key = _ModelSerializer.escape_dot(prop.wire_name)
         if prop.type.xml_serialization_ctxt:
             xml_metadata = f", 'xml': {{{prop.type.xml_serialization_ctxt}}}"
         else:
@@ -253,8 +253,8 @@ class DpgModelSerializer(_ModelSerializer):
     @staticmethod
     def declare_property(prop: Property) -> str:
         args = []
-        if prop.client_name != prop.rest_api_name or prop.is_discriminator:
-            args.append(f'name="{prop.rest_api_name}"')
+        if prop.client_name != prop.wire_name or prop.is_discriminator:
+            args.append(f'name="{prop.wire_name}"')
         if prop.readonly:
             args.append("readonly=True")
         if prop.client_default_value is not None:

package/autorest/codegen/serializers/parameter_serializer.py

@@ -95,7 +95,7 @@ class ParameterSerializer:
         retval.extend(
             [
                 '    "{}": {},'.format(
-                    path_parameter.rest_api_name,
+                    path_parameter.wire_name,
                     self.serialize_parameter(path_parameter, serializer_name),
                 )
                 for path_parameter in parameters
@@ -143,7 +143,7 @@ class ParameterSerializer:
                         else "params"
                     )
                     default_value = (
-                        f"_{kwarg_dict}.pop('{kwarg.rest_api_name}', {default_value})"
+                        f"_{kwarg_dict}.pop('{kwarg.wire_name}', {default_value})"
                     )
                 retval.append(
                     f"{kwarg.client_name}: {type_annot} = kwargs.pop('{kwarg.client_name}', "

package/autorest/codegen/serializers/sample_serializer.py

@@ -5,7 +5,7 @@
 # license information.
 # --------------------------------------------------------------------------
 import logging
-from typing import Dict, Any, Union
+from typing import Dict, Any, Union, Tuple
 from jinja2 import Environment
 
 from autorest.codegen.models.credential_types import AzureKeyCredentialType
@@ -58,7 +58,7 @@ class SampleSerializer:
             if (
                 not param.client_default_value
                 and not param.optional
-                and param.rest_api_name in self.sample["parameters"]
+                and param.wire_name in self.sample["parameters"]
             ):
                 imports.merge(param.type.imports_for_sample())
         return FileImportSerializer(imports, True)
@@ -82,7 +82,7 @@ class SampleSerializer:
         client_params = {
             p.client_name: special_param.get(
                 p.client_name,
-                f'"{self.sample["parameters"].get(p.rest_api_name) or p.client_name.upper()}"',
+                f'"{self.sample["parameters"].get(p.wire_name) or p.client_name.upper()}"',
             )
             for p in params_positional
         }
@@ -107,7 +107,7 @@ class SampleSerializer:
         failure_info = "fail to find required param named {}"
         operation_params = {}
         for param in params_positional:
-            name = param.rest_api_name
+            name = param.wire_name
             param_value = self.sample["parameters"].get(name)
             if not param.optional:
                 if not param_value:
@@ -122,17 +122,25 @@ class SampleSerializer:
             return ""
         return f".{self.operation_group.property_name}"
 
-    def _operation_result(self) -> str:
+    def _operation_result(self) -> Tuple[str, str]:
+        is_response_none = "None" in self.operation.response_type_annotation(
+            async_mode=False
+        )
         lro = ".result()"
-        paging = "\n    for item in response:\n        print(item)"
-        normal_print = "\n    print(response)"
+        if is_response_none:
+            paging, normal_print, return_var = "", "", ""
+        else:
+            paging = "\n    for item in response:\n        print(item)"
+            normal_print = "\n    print(response)"
+            return_var = "response = "
+
         if self.operation.operation_type == "paging":
-            return paging
+            return paging, return_var
         if self.operation.operation_type == "lro":
-            return lro + normal_print
+            return lro + normal_print, return_var
         if self.operation.operation_type == "lropaging":
-            return lro + paging
-        return normal_print
+            return lro + paging, return_var
+        return normal_print, return_var
 
     def _operation_name(self) -> str:
         return f".{self.operation.name}"
@@ -144,14 +152,16 @@ class SampleSerializer:
         return ""
 
     def serialize(self) -> str:
+        operation_result, return_var = self._operation_result()
         return self.env.get_template("sample.py.jinja2").render(
             code_model=self.code_model,
             file_name=self.file_name,
-            operation_result=self._operation_result(),
+            operation_result=operation_result,
             operation_params=self._operation_params(),
             operation_group_name=self._operation_group_name(),
             operation_name=self._operation_name(),
             imports=self._imports(),
             client_params=self._client_params(),
             origin_file=self._origin_file(),
+            return_var=return_var,
         )

package/autorest/codegen/templates/client.py.jinja2

@@ -7,7 +7,7 @@
         super().__init__()
     {% endif %}
         {% if client.has_parameterized_host %}
-        {{ serializer.host_variable_name }} = {{ keywords.escape_str(client.url) }}
+        {{ serializer.host_variable_name }} = {{ keywords.escape_str(client.url) }}{{ client.url_pylint_disable }}
         {% endif %}
         {{ serializer.initialize_config() }}
         {{ serializer.initialize_pipeline_client(async_mode) }}

package/autorest/codegen/templates/sample.py.jinja2

@@ -31,7 +31,7 @@ def main():
         {% endfor %}
     )
 
-    response = client{{ operation_group_name }}{{ operation_name }}(
+    {{ return_var }}client{{ operation_group_name }}{{ operation_name }}(
         {% for key, value in operation_params.items() %}
         {{ key }}={{ value|indent(8) }},
         {% endfor %}

package/autorest/m4reformatter/__init__.py

@@ -117,7 +117,7 @@ def update_property(
         client_name = "additional_properties1"
     return {
         "clientName": client_name,
-        "restApiName": yaml_data["serializedName"],
+        "wireName": yaml_data["serializedName"],
         "flattenedNames": yaml_data.get("flattenedNames", []),
         "type": update_type(yaml_data["schema"]),
         "optional": not yaml_data.get("required"),
@@ -158,7 +158,7 @@ def fill_model(
         properties.append(
             {
                 "clientName": "additional_properties",
-                "restApiName": "",
+                "wireName": "",
                 "type": update_type(dict_parents[0]),
                 "optional": True,
                 "description": "Unmatched properties from the message are deserialized to this collection.",
@@ -333,7 +333,7 @@ def filter_out_paging_next_operation(
 
 def update_response_header(yaml_data: Dict[str, Any]) -> Dict[str, Any]:
     return {
-        "restApiName": yaml_data["header"],
+        "wireName": yaml_data["header"],
         "type": update_type(yaml_data["schema"]),
     }
 
@@ -470,7 +470,7 @@ class M4Reformatter(
             "optional": not yaml_data.get("required", False),
             "description": yaml_data["language"]["default"]["description"],
             "clientName": client_name,
-            "restApiName": yaml_data["language"]["default"].get("serializedName"),
+            "wireName": yaml_data["language"]["default"].get("serializedName"),
             "clientDefaultValue": yaml_data.get("clientDefaultValue"),
             "location": location,
             "groupedBy": grouped_by,
@@ -499,7 +499,7 @@ class M4Reformatter(
                 group_name, yaml_data, body_type, content_types=content_types
             )
             for parameter in overload["parameters"]:
-                if parameter["restApiName"].lower() == "content-type":
+                if parameter["wireName"].lower() == "content-type":
                     parameter["clientDefaultValue"] = overload["bodyParameter"][
                         "defaultContentType"
                     ]
@@ -606,7 +606,7 @@ class M4Reformatter(
         operation["itemType"] = next(
             p["type"]
             for p in returned_response_object["type"]["properties"]
-            if p["restApiName"] == operation["itemName"]
+            if p["wireName"] == operation["itemName"]
         )
         if yaml_data["language"]["default"]["paging"].get("nextLinkOperation"):
             operation["nextOperation"] = self.update_operation(
@@ -724,7 +724,7 @@ class M4Reformatter(
                 body_param["clientDefaultValue"] = body_type["value"]
         body_param["flattened"] = flattened
         body_param["isPartialBody"] = is_partial_body
-        body_param["restApiName"] = body_param["restApiName"] or to_lower_camel_case(
+        body_param["wireName"] = body_param["wireName"] or to_lower_camel_case(
             body_param["clientName"]
         )
         return body_param
@@ -742,7 +742,7 @@ class M4Reformatter(
             "optional": not first_value.get("required", False),
             "description": description,
             "clientName": client_name,
-            "restApiName": client_name,
+            "wireName": client_name,
             "clientDefaultValue": None,
             "location": "Method",
             "type": KNOWN_TYPES["anydict"],
@@ -1120,7 +1120,7 @@ class M4Reformatter(
             "description": "Credential needed for the client to connect to Azure.",
             "clientName": "credential",
             "location": "other",
-            "restApiName": "credential",
+            "wireName": "credential",
             "implementation": "Client",
             "skipUrlEncoding": True,
             "inOverload": False,

package/autorest/preprocess/__init__.py

@@ -85,7 +85,7 @@ def add_overload(
 
     # update content type to be an overloads content type
     content_type_param = next(
-        p for p in overload["parameters"] if p["restApiName"].lower() == "content-type"
+        p for p in overload["parameters"] if p["wireName"].lower() == "content-type"
     )
     content_type_param["inOverload"] = True
     content_type_param["inDocstring"] = True
@@ -126,7 +126,7 @@ def add_overloads_for_body_param(yaml_data: Dict[str, Any]) -> None:
                 add_overload(yaml_data, body_type, for_flatten_params=True)
             )
     content_type_param = next(
-        p for p in yaml_data["parameters"] if p["restApiName"].lower() == "content-type"
+        p for p in yaml_data["parameters"] if p["wireName"].lower() == "content-type"
     )
     content_type_param["inOverload"] = False
     content_type_param["inOverriden"] = True
@@ -143,7 +143,7 @@ def _remove_paging_maxpagesize(yaml_data: Dict[str, Any]) -> None:
     yaml_data["parameters"] = [
         p
         for p in yaml_data.get("parameters", [])
-        if p["restApiName"].lower() not in ["maxpagesize", "$maxpagesize"]
+        if p["wireName"].lower() not in ["maxpagesize", "$maxpagesize"]
     ]
 
 
@@ -212,7 +212,9 @@ class PreProcessPlugin(YamlUpdatePlugin):  # pylint: disable=abstract-method
     def update_types(self, yaml_data: List[Dict[str, Any]]) -> None:
         for type in yaml_data:
             for property in type.get("properties", []):
-                property["description"] = update_description(property["description"])
+                property["description"] = update_description(
+                    property.get("description", "")
+                )
                 property["clientName"] = self.pad_reserved_words(
                     property["clientName"].lower(), PadType.PROPERTY
                 )
@@ -264,7 +266,7 @@ class PreProcessPlugin(YamlUpdatePlugin):  # pylint: disable=abstract-method
         return self.update_operation
 
     def update_parameter(self, yaml_data: Dict[str, Any]) -> None:
-        yaml_data["description"] = update_description(yaml_data["description"])
+        yaml_data["description"] = update_description(yaml_data.get("description", ""))
         if not (
             yaml_data["location"] == "header"
             and yaml_data["clientName"] in ("content_type", "accept")

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@autorest/python",
-  "version": "6.4.10",
+  "version": "6.4.12",
   "description": "The Python extension for generators in AutoRest.",
   "main": "index.js",
   "repository": {
@@ -19,11 +19,11 @@
   },
   "homepage": "https://github.com/Azure/autorest.python/blob/main/README.md",
   "dependencies": {
-    "@autorest/system-requirements": "~1.0.0"
+    "@autorest/system-requirements": "~1.0.2"
   },
   "devDependencies": {
     "@microsoft.azure/autorest.testserver": "^3.3.46",
-    "typescript": "^4.8.3"
+    "typescript": "^5.0.4"
   },
   "files": [
     "autorest/**/*.py",