@autorest/python 6.28.3 → 6.29.0

package/generator/build/lib/pygen/black.py

@@ -60,8 +60,19 @@ class BlackScriptPlugin(Plugin):
         except:
             _LOGGER.error("Error: failed to format %s", file)
             raise
-        if len(file_content.splitlines()) > 1000 and "pylint: disable=too-many-lines" not in file_content:
-            file_content = "# pylint: disable=too-many-lines\n" + file_content
+        pylint_disables = []
+        lines = file_content.splitlines()
+        if len(lines) > 0:
+            if "line-too-long" not in lines[0] and any(len(line) > 120 for line in lines):
+                pylint_disables.extend(["line-too-long", "useless-suppression"])
+            if "too-many-lines" not in lines[0] and len(lines) > 1000:
+                pylint_disables.append("too-many-lines")
+            if pylint_disables:
+                file_content = (
+                    "\n".join([lines[0] + ",".join([""] + pylint_disables)] + lines[1:])
+                    if "pylint: disable=" in lines[0]
+                    else f"# pylint: disable={','.join(pylint_disables)}\n" + file_content
+                )
         self.write_file(file, file_content)
 
 

package/generator/build/lib/pygen/codegen/models/client.py

@@ -161,14 +161,6 @@ class Client(_ClientConfigBase[ClientGlobalParameterList]):
             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
     def filename(self) -> str:
         """Name of the file for the client"""

package/generator/build/lib/pygen/codegen/serializers/builder_serializer.py

@@ -498,11 +498,7 @@ class RequestBuilderSerializer(_BuilderBaseSerializer[RequestBuilderType]):
             url_value = _escape_str(builder.url)
         else:
             url_value = f'kwargs.pop("template_url", {_escape_str(builder.url)})'
-        result = "_url = " + url_value
-        # there will be always 4 spaces before the url
-        if len(result) + 4 > 120:
-            return result + "  # pylint: disable=line-too-long"
-        return result
+        return "_url = " + url_value
 
 
 ############################## NORMAL OPERATIONS ##############################
@@ -1005,7 +1001,9 @@ class _OperationSerializer(_BuilderBaseSerializer[OperationType]):
                 retval.extend(deserialize_code)
         return retval
 
-    def handle_error_response(self, builder: OperationType) -> List[str]:   # pylint: disable=too-many-statements, too-many-branches
+    def handle_error_response(  # pylint: disable=too-many-statements, too-many-branches
+        self, builder: OperationType
+    ) -> List[str]:
         async_await = "await " if self.async_mode else ""
         retval = [f"if response.status_code not in {str(builder.success_status_codes)}:"]
         response_read = [
@@ -1042,34 +1040,23 @@ class _OperationSerializer(_BuilderBaseSerializer[OperationType]):
                             )
                         # add build-in error type
                         # TODO: we should decide whether need to this wrapper for customized error type
-                        if status_code == 401:
-                            retval.append(
-                                "        raise ClientAuthenticationError(response=response{}{})".format(
-                                    error_model,
-                                    (", error_format=ARMErrorFormat" if self.code_model.options["azure_arm"] else ""),
-                                )
-                            )
-                        elif status_code == 404:
-                            retval.append(
-                                "        raise ResourceNotFoundError(response=response{}{})".format(
-                                    error_model,
-                                    (", error_format=ARMErrorFormat" if self.code_model.options["azure_arm"] else ""),
-                                )
-                            )
-                        elif status_code == 409:
-                            retval.append(
-                                "        raise ResourceExistsError(response=response{}{})".format(
-                                    error_model,
-                                    (", error_format=ARMErrorFormat" if self.code_model.options["azure_arm"] else ""),
-                                )
-                            )
-                        elif status_code == 304:
+                        status_code_error_map = {
+                            401: "ClientAuthenticationError",
+                            404: "ResourceNotFoundError",
+                            409: "ResourceExistsError",
+                            304: "ResourceNotModifiedError",
+                        }
+                        if status_code in status_code_error_map:
                             retval.append(
-                                "        raise ResourceNotModifiedError(response=response{}{})".format(
+                                "        raise {}(response=response{}{})".format(
+                                    status_code_error_map[cast(int, status_code)],
                                     error_model,
                                     (", error_format=ARMErrorFormat" if self.code_model.options["azure_arm"] else ""),
                                 )
                             )
+                            condition = "if"
+                        else:
+                            condition = "elif"
                 # ranged status code only exist in typespec and will not have multiple status codes
                 else:
                     retval.append(
@@ -1084,15 +1071,13 @@ class _OperationSerializer(_BuilderBaseSerializer[OperationType]):
                                 f"        error = _failsafe_deserialize_xml({type_annotation},  response.text())"
                             )
                         else:
-                            retval.append(
-                                f"        error = _failsafe_deserialize({type_annotation},  response.json())"
-                            )
+                            retval.append(f"        error = _failsafe_deserialize({type_annotation},  response.json())")
                     else:
                         retval.append(
                             f"        error = self._deserialize.failsafe_deserialize({type_annotation}, "
                             "pipeline_response)"
                         )
-                condition = "elif"
+                    condition = "elif"
         # default error handling
         if builder.default_error_deserialization and self.code_model.options["models_mode"]:
             error_model = ", model=error"

package/generator/build/lib/pygen/codegen/serializers/model_serializer.py

@@ -325,13 +325,7 @@ class DpgModelSerializer(_ModelSerializer):
         )
         type_annotation = prop.type_annotation(serialize_namespace=self.serialize_namespace)
         generated_code = f'{prop.client_name}: {type_annotation} = {field}({", ".join(args)})'
-        # there is 4 spaces indentation so original line length limit 120 - 4 = 116
-        pylint_disable = (
-            " # pylint: disable=line-too-long"
-            if len(generated_code) <= 116 < (len(generated_code) + len(type_ignore))
-            else ""
-        )
-        return f"{generated_code}{type_ignore}{pylint_disable}"
+        return f"{generated_code}{type_ignore}"
 
     def initialize_properties(self, model: ModelType) -> List[str]:
         init_args = []

package/generator/build/lib/pygen/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) }}{{ client.url_pylint_disable }}
+        {{ serializer.host_variable_name }} = {{ keywords.escape_str(client.url) }}
         {% endif %}
         {{ serializer.initialize_config() }}
         {{ op_tools.serialize(serializer.initialize_pipeline_client(async_mode)) | indent(8) }}

package/generator/build/lib/pygen/codegen/templates/macros.jinja2

@@ -4,10 +4,6 @@
 {% for line in list_result %}
     {% set prefix = "" if loop.index == 1 else " " %}
     {% set suffix = suffix_string if list_result | length == loop.index %}
-    {% if line | length > 120 %}
-{{ prefix }}{{ line + "  # pylint: disable=line-too-long" }}{{ suffix }}
-    {% else %}
 {{ prefix }}{{ line }}{{ suffix }}
-    {% endif %}
 {% endfor %}
 {% endmacro %}

package/generator/build/lib/pygen/codegen/templates/model_dpg.py.jinja2

@@ -8,16 +8,6 @@
 
     {{ serializer.discriminator_docstring(model) | wordwrap(width=95, break_long_words=False, break_on_hyphens=False, wrapstring='\n    ') }}
     {% endif %}
-    {% if model.has_readonly_or_constant_property %}
-
-    Readonly variables are only populated by the server, and will be ignored when sending a request.
-    {% endif %}
-    {% if (model.properties | selectattr('optional', "equalto", false) | first) is defined %}
-
-    {% if not model.is_usage_output %}
-    All required parameters must be populated in order to send to server.
-    {% endif %}
-    {% endif %}
 
     {% if model.properties != None %}
         {% for p in model.properties %}

package/generator/build/lib/pygen/codegen/templates/operation_tools.jinja2

@@ -12,9 +12,6 @@
     {% for description in serializer.description_and_summary(builder) %}
         {% if description %}
 {% set description = wrap_string(description, wrapstring='\n') %}
-            {% if (serializer.line_too_long(example_template) or ns.line_too_long) and loop.first %}
-# pylint: disable=line-too-long
-            {% endif %}
 {{ '"""' + description if loop.first else description }}
         {% else %}
 

package/generator/build/lib/pygen/codegen/templates/serialization.py.jinja2

@@ -410,7 +410,7 @@ class Model:
         :param function key_extractors: A key extractor function.
         :param str content_type: JSON by default, set application/xml if XML.
         :returns: An instance of this model
-        :raises: DeserializationError if something went wrong
+        :raises DeserializationError: if something went wrong
         :rtype: Self
         """
         deserializer = Deserializer(cls._infer_class_models())
@@ -1360,7 +1360,7 @@ def xml_key_extractor(attr, attr_desc, data):  # pylint: disable=unused-argument
         # Iter and wrapped, should have found one node only (the wrap one)
         if len(children) != 1:
             raise DeserializationError(
-                "Tried to deserialize an array not wrapped, and found several nodes '{}'. Maybe you should declare this array as wrapped?".format(  # pylint: disable=line-too-long
+                "Tried to deserialize an array not wrapped, and found several nodes '{}'. Maybe you should declare this array as wrapped?".format(
                     xml_name
                 )
             )

package/generator/component-detection-pip-report.json

@@ -1,6 +1,6 @@
 {
   "version": "1",
-  "pip_version": "25.0",
+  "pip_version": "25.0.1",
   "install": [
     {
       "download_info": {
@@ -126,9 +126,9 @@
     "implementation_version": "3.8.10",
     "os_name": "posix",
     "platform_machine": "x86_64",
-    "platform_release": "5.15.0-1079-azure",
+    "platform_release": "5.15.0-1081-azure",
     "platform_system": "Linux",
-    "platform_version": "#88~20.04.1-Ubuntu SMP Fri Jan 17 18:28:29 UTC 2025",
+    "platform_version": "#90~20.04.1-Ubuntu SMP Tue Jan 28 05:34:18 UTC 2025",
     "python_full_version": "3.8.10",
     "platform_python_implementation": "CPython",
     "python_version": "3.8",

package/generator/pygen/black.py

@@ -60,8 +60,19 @@ class BlackScriptPlugin(Plugin):
         except:
             _LOGGER.error("Error: failed to format %s", file)
             raise
-        if len(file_content.splitlines()) > 1000 and "pylint: disable=too-many-lines" not in file_content:
-            file_content = "# pylint: disable=too-many-lines\n" + file_content
+        pylint_disables = []
+        lines = file_content.splitlines()
+        if len(lines) > 0:
+            if "line-too-long" not in lines[0] and any(len(line) > 120 for line in lines):
+                pylint_disables.extend(["line-too-long", "useless-suppression"])
+            if "too-many-lines" not in lines[0] and len(lines) > 1000:
+                pylint_disables.append("too-many-lines")
+            if pylint_disables:
+                file_content = (
+                    "\n".join([lines[0] + ",".join([""] + pylint_disables)] + lines[1:])
+                    if "pylint: disable=" in lines[0]
+                    else f"# pylint: disable={','.join(pylint_disables)}\n" + file_content
+                )
         self.write_file(file, file_content)
 
 

package/generator/pygen/codegen/models/client.py

@@ -161,14 +161,6 @@ class Client(_ClientConfigBase[ClientGlobalParameterList]):
             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
     def filename(self) -> str:
         """Name of the file for the client"""

package/generator/pygen/codegen/serializers/builder_serializer.py

@@ -498,11 +498,7 @@ class RequestBuilderSerializer(_BuilderBaseSerializer[RequestBuilderType]):
             url_value = _escape_str(builder.url)
         else:
             url_value = f'kwargs.pop("template_url", {_escape_str(builder.url)})'
-        result = "_url = " + url_value
-        # there will be always 4 spaces before the url
-        if len(result) + 4 > 120:
-            return result + "  # pylint: disable=line-too-long"
-        return result
+        return "_url = " + url_value
 
 
 ############################## NORMAL OPERATIONS ##############################
@@ -1005,7 +1001,9 @@ class _OperationSerializer(_BuilderBaseSerializer[OperationType]):
                 retval.extend(deserialize_code)
         return retval
 
-    def handle_error_response(self, builder: OperationType) -> List[str]:   # pylint: disable=too-many-statements, too-many-branches
+    def handle_error_response(  # pylint: disable=too-many-statements, too-many-branches
+        self, builder: OperationType
+    ) -> List[str]:
         async_await = "await " if self.async_mode else ""
         retval = [f"if response.status_code not in {str(builder.success_status_codes)}:"]
         response_read = [
@@ -1042,34 +1040,23 @@ class _OperationSerializer(_BuilderBaseSerializer[OperationType]):
                             )
                         # add build-in error type
                         # TODO: we should decide whether need to this wrapper for customized error type
-                        if status_code == 401:
-                            retval.append(
-                                "        raise ClientAuthenticationError(response=response{}{})".format(
-                                    error_model,
-                                    (", error_format=ARMErrorFormat" if self.code_model.options["azure_arm"] else ""),
-                                )
-                            )
-                        elif status_code == 404:
-                            retval.append(
-                                "        raise ResourceNotFoundError(response=response{}{})".format(
-                                    error_model,
-                                    (", error_format=ARMErrorFormat" if self.code_model.options["azure_arm"] else ""),
-                                )
-                            )
-                        elif status_code == 409:
-                            retval.append(
-                                "        raise ResourceExistsError(response=response{}{})".format(
-                                    error_model,
-                                    (", error_format=ARMErrorFormat" if self.code_model.options["azure_arm"] else ""),
-                                )
-                            )
-                        elif status_code == 304:
+                        status_code_error_map = {
+                            401: "ClientAuthenticationError",
+                            404: "ResourceNotFoundError",
+                            409: "ResourceExistsError",
+                            304: "ResourceNotModifiedError",
+                        }
+                        if status_code in status_code_error_map:
                             retval.append(
-                                "        raise ResourceNotModifiedError(response=response{}{})".format(
+                                "        raise {}(response=response{}{})".format(
+                                    status_code_error_map[cast(int, status_code)],
                                     error_model,
                                     (", error_format=ARMErrorFormat" if self.code_model.options["azure_arm"] else ""),
                                 )
                             )
+                            condition = "if"
+                        else:
+                            condition = "elif"
                 # ranged status code only exist in typespec and will not have multiple status codes
                 else:
                     retval.append(
@@ -1084,15 +1071,13 @@ class _OperationSerializer(_BuilderBaseSerializer[OperationType]):
                                 f"        error = _failsafe_deserialize_xml({type_annotation},  response.text())"
                             )
                         else:
-                            retval.append(
-                                f"        error = _failsafe_deserialize({type_annotation},  response.json())"
-                            )
+                            retval.append(f"        error = _failsafe_deserialize({type_annotation},  response.json())")
                     else:
                         retval.append(
                             f"        error = self._deserialize.failsafe_deserialize({type_annotation}, "
                             "pipeline_response)"
                         )
-                condition = "elif"
+                    condition = "elif"
         # default error handling
         if builder.default_error_deserialization and self.code_model.options["models_mode"]:
             error_model = ", model=error"

package/generator/pygen/codegen/serializers/model_serializer.py

@@ -325,13 +325,7 @@ class DpgModelSerializer(_ModelSerializer):
         )
         type_annotation = prop.type_annotation(serialize_namespace=self.serialize_namespace)
         generated_code = f'{prop.client_name}: {type_annotation} = {field}({", ".join(args)})'
-        # there is 4 spaces indentation so original line length limit 120 - 4 = 116
-        pylint_disable = (
-            " # pylint: disable=line-too-long"
-            if len(generated_code) <= 116 < (len(generated_code) + len(type_ignore))
-            else ""
-        )
-        return f"{generated_code}{type_ignore}{pylint_disable}"
+        return f"{generated_code}{type_ignore}"
 
     def initialize_properties(self, model: ModelType) -> List[str]:
         init_args = []

package/generator/pygen/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) }}{{ client.url_pylint_disable }}
+        {{ serializer.host_variable_name }} = {{ keywords.escape_str(client.url) }}
         {% endif %}
         {{ serializer.initialize_config() }}
         {{ op_tools.serialize(serializer.initialize_pipeline_client(async_mode)) | indent(8) }}

package/generator/pygen/codegen/templates/macros.jinja2

@@ -4,10 +4,6 @@
 {% for line in list_result %}
     {% set prefix = "" if loop.index == 1 else " " %}
     {% set suffix = suffix_string if list_result | length == loop.index %}
-    {% if line | length > 120 %}
-{{ prefix }}{{ line + "  # pylint: disable=line-too-long" }}{{ suffix }}
-    {% else %}
 {{ prefix }}{{ line }}{{ suffix }}
-    {% endif %}
 {% endfor %}
 {% endmacro %}

package/generator/pygen/codegen/templates/model_dpg.py.jinja2

@@ -8,16 +8,6 @@
 
     {{ serializer.discriminator_docstring(model) | wordwrap(width=95, break_long_words=False, break_on_hyphens=False, wrapstring='\n    ') }}
     {% endif %}
-    {% if model.has_readonly_or_constant_property %}
-
-    Readonly variables are only populated by the server, and will be ignored when sending a request.
-    {% endif %}
-    {% if (model.properties | selectattr('optional', "equalto", false) | first) is defined %}
-
-    {% if not model.is_usage_output %}
-    All required parameters must be populated in order to send to server.
-    {% endif %}
-    {% endif %}
 
     {% if model.properties != None %}
         {% for p in model.properties %}

package/generator/pygen/codegen/templates/operation_tools.jinja2

@@ -12,9 +12,6 @@
     {% for description in serializer.description_and_summary(builder) %}
         {% if description %}
 {% set description = wrap_string(description, wrapstring='\n') %}
-            {% if (serializer.line_too_long(example_template) or ns.line_too_long) and loop.first %}
-# pylint: disable=line-too-long
-            {% endif %}
 {{ '"""' + description if loop.first else description }}
         {% else %}
 

package/generator/pygen/codegen/templates/serialization.py.jinja2

@@ -410,7 +410,7 @@ class Model:
         :param function key_extractors: A key extractor function.
         :param str content_type: JSON by default, set application/xml if XML.
         :returns: An instance of this model
-        :raises: DeserializationError if something went wrong
+        :raises DeserializationError: if something went wrong
         :rtype: Self
         """
         deserializer = Deserializer(cls._infer_class_models())
@@ -1360,7 +1360,7 @@ def xml_key_extractor(attr, attr_desc, data):  # pylint: disable=unused-argument
         # Iter and wrapped, should have found one node only (the wrap one)
         if len(children) != 1:
             raise DeserializationError(
-                "Tried to deserialize an array not wrapped, and found several nodes '{}'. Maybe you should declare this array as wrapped?".format(  # pylint: disable=line-too-long
+                "Tried to deserialize an array not wrapped, and found several nodes '{}'. Maybe you should declare this array as wrapped?".format(
                     xml_name
                 )
             )

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@autorest/python",
-  "version": "6.28.3",
+  "version": "6.29.0",
   "description": "The Python extension for generators in AutoRest.",
   "main": "index.js",
   "repository": {
@@ -19,7 +19,7 @@
   },
   "homepage": "https://github.com/Azure/autorest.python/blob/main/README.md",
   "dependencies": {
-    "@typespec/http-client-python": "~0.6.10",
+    "@typespec/http-client-python": "~0.7.0",
     "@autorest/system-requirements": "~1.0.2",
     "fs-extra": "~11.2.0",
     "tsx": "~4.19.1"