depend on http-client-python

.chronus/changes/depend_on_typespec-2024-8-25-19-12-11.md

@@ -0,0 +1,8 @@
+---
+changeKind: feature
+packages:
+  - "@autorest/python"
+  - "@azure-tools/typespec-python"
+---
+
+Depend completely on `@typespec/http-client-python`

.gitignore

@@ -126,4 +126,6 @@ node_modules/
 # Generated test folders
 test/services/*/_generated
 **/autorest.python/generator
-**/autorest.python/scripts/eng
+**/autorest.python/eng/scripts
+**/typespec-python/generator
+**/typespec-python/eng/scripts

eng/pipelines/ci-template.yml

@@ -75,21 +75,21 @@ steps:
       displayName: List installed packages
       workingDirectory: $(Build.SourcesDirectory)/autorest.python/packages/${{parameters.folderName}}
 
-    - script: pnpm run lint --command eslint --skipWarning ${{ parameters.skipWarning }}
-      displayName: Eslint
-      workingDirectory: $(Build.SourcesDirectory)/autorest.python/packages/${{parameters.folderName}}
+    # - script: pnpm run lint --command eslint --skipWarning ${{ parameters.skipWarning }}
+    #   displayName: Eslint
+    #   workingDirectory: $(Build.SourcesDirectory)/autorest.python/packages/${{parameters.folderName}}
 
-    - script: pnpm run lint --command pylint
-      displayName: Pylint
-      workingDirectory: $(Build.SourcesDirectory)/autorest.python/packages/${{parameters.folderName}}
+    # - script: pnpm run lint --command pylint
+    #   displayName: Pylint
+    #   workingDirectory: $(Build.SourcesDirectory)/autorest.python/packages/${{parameters.folderName}}
 
-    - script: pnpm run lint --command mypy
-      displayName: Mypy
-      workingDirectory: $(Build.SourcesDirectory)/autorest.python/packages/${{parameters.folderName}}
+    # - script: pnpm run lint --command mypy
+    #   displayName: Mypy
+    #   workingDirectory: $(Build.SourcesDirectory)/autorest.python/packages/${{parameters.folderName}}
 
-    - script: pnpm run lint --command pyright
-      displayName: Pyright
-      workingDirectory: $(Build.SourcesDirectory)/autorest.python/packages/${{parameters.folderName}}
+    # - script: pnpm run lint --command pyright
+    #   displayName: Pyright
+    #   workingDirectory: $(Build.SourcesDirectory)/autorest.python/packages/${{parameters.folderName}}
 
     - script: pnpm run format
       displayName: Black

package.json

@@ -31,20 +31,20 @@
   "homepage": "https://github.com/Azure/autorest.python#readme",
   "devDependencies": {
     "@actions/github": "6.0.0",
-    "@azure-tools/cadl-ranch": "~0.14.5",
+    "@azure-tools/cadl-ranch": "~0.14.6",
     "@chronus/chronus": "^0.12.1",
-    "@chronus/github": "^0.4.3",
-    "@eslint/js": "^9.10.0",
+    "@chronus/github": "^0.4.4",
+    "@eslint/js": "^9.11.1",
     "@typespec/prettier-plugin-typespec": "~0.60.0",
     "autorest": "~3.7.1",
     "cross-env": "^7.0.3",
-    "eslint": "^9.10.0",
+    "eslint": "^9.11.1",
     "eslint-plugin-import": "^2.30.0",
     "eslint-plugin-unicorn": "^55.0.0",
-    "typescript-eslint": "^8.5.0",
     "prettier": "~3.3.3",
     "syncpack": "^13.0.0",
-    "typescript": "~5.6.2"
+    "typescript": "~5.6.2",
+    "typescript-eslint": "^8.7.0"
   },
   "syncpack": {
     "dependencyTypes": [

packages/autorest.python/package.json

@@ -8,9 +8,9 @@
     "build": "tsx ./scripts/copy-generator.ts --force",
     "install": "tsx ./scripts/copy-generator.ts && node ./scripts/run-python3.js ./scripts/install.py",
     "debug": "node ./scripts/run-python3.js ./scripts/start.py --debug",
-    "test": "tsx ./scripts/eng/run-tests.ts --validFolders azure/legacy azure/version-tolerant vanilla/legacy vanilla/version-tolerant dpg/version-tolerant",
-    "lint": "tsx ./scripts/eng/lint.ts --folderName autorest --skipEslint true",
-    "format": "tsx ./scripts/eng/format.ts"
+    "test": "tsx ./eng/scripts/ci/run-tests.ts --validFolders azure/legacy azure/version-tolerant vanilla/legacy vanilla/version-tolerant dpg/version-tolerant",
+    "lint": "tsx ./eng/scripts/ci/lint.ts --folderName autorest",
+    "format": "tsx ./eng/scripts/ci/format.ts --pythonPath ./node_modules/@typespec/http-client-python/venv"
   },
   "main": "index.js",
   "repository": {
@@ -34,10 +34,10 @@
     "tsx": "4.17.0"
   },
   "devDependencies": {
-    "@microsoft.azure/autorest.testserver": "^3.3.46",
-    "typescript": "~5.1.3",
-    "@azure-tools/typespec-python": "workspace:^",
-    "chalk": "5.3.0"
+    "@microsoft.azure/autorest.testserver": "^3.3.50",
+    "@typespec/http-client-python": "0.1.0",
+    "chalk": "5.3.0",
+    "typescript": "~5.1.6"
   },
   "files": [
     "autorest/**/*.py",

packages/autorest.python/samples/specification/azure_key_credential/generated/azure/key/credential/sample/_configuration.py

@@ -14,7 +14,7 @@
 from ._version import VERSION
 
 
-class AutoRestHeadTestServiceConfiguration:  # pylint: disable=too-many-instance-attributes,name-too-long
+class AutoRestHeadTestServiceConfiguration:  # pylint: disable=too-many-instance-attributes
     """Configuration for AutoRestHeadTestService.
 
     Note that all parameters used to create this instance are saved as instance

packages/autorest.python/samples/specification/azure_key_credential/generated/azure/key/credential/sample/_serialization.py

@@ -1,4 +1,3 @@
-# pylint: disable=too-many-lines
 # --------------------------------------------------------------------------
 #
 # Copyright (c) Microsoft Corporation. All rights reserved.

packages/autorest.python/samples/specification/azure_key_credential/generated/azure/key/credential/sample/aio/_configuration.py

@@ -14,7 +14,7 @@
 from .._version import VERSION
 
 
-class AutoRestHeadTestServiceConfiguration:  # pylint: disable=too-many-instance-attributes,name-too-long
+class AutoRestHeadTestServiceConfiguration:  # pylint: disable=too-many-instance-attributes
     """Configuration for AutoRestHeadTestService.
 
     Note that all parameters used to create this instance are saved as instance

packages/autorest.python/samples/specification/azure_key_credential/generated/azure/key/credential/sample/aio/operations/_operations.py

@@ -1,4 +1,3 @@
-# pylint: disable=too-many-lines,too-many-statements
 # coding=utf-8
 # --------------------------------------------------------------------------
 # Copyright (c) Microsoft Corporation. All rights reserved.
@@ -7,7 +6,7 @@
 # Changes may cause incorrect behavior and will be lost if the code is regenerated.
 # --------------------------------------------------------------------------
 import sys
-from typing import Any, Callable, Dict, Optional, Type, TypeVar
+from typing import Any, Callable, Dict, Optional, TypeVar
 
 from azure.core.exceptions import (
     ClientAuthenticationError,
@@ -30,7 +29,7 @@
 if sys.version_info >= (3, 9):
     from collections.abc import MutableMapping
 else:
-    from typing import MutableMapping  # type: ignore  # pylint: disable=ungrouped-imports
+    from typing import MutableMapping  # type: ignore
 T = TypeVar("T")
 ClsType = Optional[Callable[[PipelineResponse[HttpRequest, AsyncHttpResponse], T, Dict[str, Any]], Any]]
 
@@ -53,14 +52,14 @@ def __init__(self, *args, **kwargs) -> None:
         self._deserialize = input_args.pop(0) if input_args else kwargs.pop("deserializer")
 
     @distributed_trace_async
-    async def head200(self, **kwargs: Any) -> None:  # pylint: disable=inconsistent-return-statements
+    async def head200(self, **kwargs: Any) -> None:
         """Return 200 status code if successful.
 
         :return: None
         :rtype: None
         :raises ~azure.core.exceptions.HttpResponseError:
         """
-        error_map: MutableMapping[int, Type[HttpResponseError]] = {  # pylint: disable=unsubscriptable-object
+        error_map: MutableMapping = {
             401: ClientAuthenticationError,
             404: ResourceNotFoundError,
             409: ResourceExistsError,
@@ -94,14 +93,14 @@ async def head200(self, **kwargs: Any) -> None:  # pylint: disable=inconsistent-
             return cls(pipeline_response, None, {})  # type: ignore
 
     @distributed_trace_async
-    async def head204(self, **kwargs: Any) -> None:  # pylint: disable=inconsistent-return-statements
+    async def head204(self, **kwargs: Any) -> None:
         """Return 204 status code if successful.
 
         :return: None
         :rtype: None
         :raises ~azure.core.exceptions.HttpResponseError:
         """
-        error_map: MutableMapping[int, Type[HttpResponseError]] = {  # pylint: disable=unsubscriptable-object
+        error_map: MutableMapping = {
             401: ClientAuthenticationError,
             404: ResourceNotFoundError,
             409: ResourceExistsError,
@@ -135,14 +134,14 @@ async def head204(self, **kwargs: Any) -> None:  # pylint: disable=inconsistent-
             return cls(pipeline_response, None, {})  # type: ignore
 
     @distributed_trace_async
-    async def head404(self, **kwargs: Any) -> None:  # pylint: disable=inconsistent-return-statements
+    async def head404(self, **kwargs: Any) -> None:
         """Return 404 status code if successful.
 
         :return: None
         :rtype: None
         :raises ~azure.core.exceptions.HttpResponseError:
         """
-        error_map: MutableMapping[int, Type[HttpResponseError]] = {  # pylint: disable=unsubscriptable-object
+        error_map: MutableMapping = {
             401: ClientAuthenticationError,
             404: ResourceNotFoundError,
             409: ResourceExistsError,

packages/autorest.python/samples/specification/azure_key_credential/generated/azure/key/credential/sample/operations/_operations.py

@@ -1,4 +1,3 @@
-# pylint: disable=too-many-lines,too-many-statements
 # coding=utf-8
 # --------------------------------------------------------------------------
 # Copyright (c) Microsoft Corporation. All rights reserved.
@@ -7,7 +6,7 @@
 # Changes may cause incorrect behavior and will be lost if the code is regenerated.
 # --------------------------------------------------------------------------
 import sys
-from typing import Any, Callable, Dict, Optional, Type, TypeVar
+from typing import Any, Callable, Dict, Optional, TypeVar
 
 from azure.core.exceptions import (
     ClientAuthenticationError,
@@ -26,7 +25,7 @@
 if sys.version_info >= (3, 9):
     from collections.abc import MutableMapping
 else:
-    from typing import MutableMapping  # type: ignore  # pylint: disable=ungrouped-imports
+    from typing import MutableMapping  # type: ignore
 T = TypeVar("T")
 ClsType = Optional[Callable[[PipelineResponse[HttpRequest, HttpResponse], T, Dict[str, Any]], Any]]
 
@@ -80,7 +79,7 @@ def head200(self, **kwargs: Any) -> None:  # pylint: disable=inconsistent-return
         :rtype: None
         :raises ~azure.core.exceptions.HttpResponseError:
         """
-        error_map: MutableMapping[int, Type[HttpResponseError]] = {  # pylint: disable=unsubscriptable-object
+        error_map: MutableMapping = {
             401: ClientAuthenticationError,
             404: ResourceNotFoundError,
             409: ResourceExistsError,
@@ -121,7 +120,7 @@ def head204(self, **kwargs: Any) -> None:  # pylint: disable=inconsistent-return
         :rtype: None
         :raises ~azure.core.exceptions.HttpResponseError:
         """
-        error_map: MutableMapping[int, Type[HttpResponseError]] = {  # pylint: disable=unsubscriptable-object
+        error_map: MutableMapping = {
             401: ClientAuthenticationError,
             404: ResourceNotFoundError,
             409: ResourceExistsError,
@@ -162,7 +161,7 @@ def head404(self, **kwargs: Any) -> None:  # pylint: disable=inconsistent-return
         :rtype: None
         :raises ~azure.core.exceptions.HttpResponseError:
         """
-        error_map: MutableMapping[int, Type[HttpResponseError]] = {  # pylint: disable=unsubscriptable-object
+        error_map: MutableMapping = {
             401: ClientAuthenticationError,
             404: ResourceNotFoundError,
             409: ResourceExistsError,

packages/autorest.python/samples/specification/basic/generated/azure/basic/sample/_configuration.py

@@ -13,7 +13,7 @@
 from ._version import VERSION
 
 
-class AutoRestHeadTestServiceConfiguration:  # pylint: disable=too-many-instance-attributes,name-too-long
+class AutoRestHeadTestServiceConfiguration:  # pylint: disable=too-many-instance-attributes
     """Configuration for AutoRestHeadTestService.
 
     Note that all parameters used to create this instance are saved as instance

packages/autorest.python/samples/specification/basic/generated/azure/basic/sample/_serialization.py

@@ -1,4 +1,3 @@
-# pylint: disable=too-many-lines
 # --------------------------------------------------------------------------
 #
 # Copyright (c) Microsoft Corporation. All rights reserved.

packages/autorest.python/samples/specification/basic/generated/azure/basic/sample/aio/_configuration.py

@@ -13,7 +13,7 @@
 from .._version import VERSION
 
 
-class AutoRestHeadTestServiceConfiguration:  # pylint: disable=too-many-instance-attributes,name-too-long
+class AutoRestHeadTestServiceConfiguration:  # pylint: disable=too-many-instance-attributes
     """Configuration for AutoRestHeadTestService.
 
     Note that all parameters used to create this instance are saved as instance

packages/autorest.python/samples/specification/basic/generated/azure/basic/sample/aio/operations/_operations.py

@@ -1,4 +1,3 @@
-# pylint: disable=too-many-lines,too-many-statements
 # coding=utf-8
 # --------------------------------------------------------------------------
 # Copyright (c) Microsoft Corporation. All rights reserved.
@@ -7,7 +6,7 @@
 # Changes may cause incorrect behavior and will be lost if the code is regenerated.
 # --------------------------------------------------------------------------
 import sys
-from typing import Any, Callable, Dict, Optional, Type, TypeVar
+from typing import Any, Callable, Dict, Optional, TypeVar
 
 from azure.core.exceptions import (
     ClientAuthenticationError,
@@ -30,7 +29,7 @@
 if sys.version_info >= (3, 9):
     from collections.abc import MutableMapping
 else:
-    from typing import MutableMapping  # type: ignore  # pylint: disable=ungrouped-imports
+    from typing import MutableMapping  # type: ignore
 T = TypeVar("T")
 ClsType = Optional[Callable[[PipelineResponse[HttpRequest, AsyncHttpResponse], T, Dict[str, Any]], Any]]
 
@@ -53,14 +52,14 @@ def __init__(self, *args, **kwargs) -> None:
         self._deserialize = input_args.pop(0) if input_args else kwargs.pop("deserializer")
 
     @distributed_trace_async
-    async def head200(self, **kwargs: Any) -> None:  # pylint: disable=inconsistent-return-statements
+    async def head200(self, **kwargs: Any) -> None:
         """Return 200 status code if successful.
 
         :return: None
         :rtype: None
         :raises ~azure.core.exceptions.HttpResponseError:
         """
-        error_map: MutableMapping[int, Type[HttpResponseError]] = {  # pylint: disable=unsubscriptable-object
+        error_map: MutableMapping = {
             401: ClientAuthenticationError,
             404: ResourceNotFoundError,
             409: ResourceExistsError,
@@ -94,14 +93,14 @@ async def head200(self, **kwargs: Any) -> None:  # pylint: disable=inconsistent-
             return cls(pipeline_response, None, {})  # type: ignore
 
     @distributed_trace_async
-    async def head204(self, **kwargs: Any) -> None:  # pylint: disable=inconsistent-return-statements
+    async def head204(self, **kwargs: Any) -> None:
         """Return 204 status code if successful.
 
         :return: None
         :rtype: None
         :raises ~azure.core.exceptions.HttpResponseError:
         """
-        error_map: MutableMapping[int, Type[HttpResponseError]] = {  # pylint: disable=unsubscriptable-object
+        error_map: MutableMapping = {
             401: ClientAuthenticationError,
             404: ResourceNotFoundError,
             409: ResourceExistsError,
@@ -135,14 +134,14 @@ async def head204(self, **kwargs: Any) -> None:  # pylint: disable=inconsistent-
             return cls(pipeline_response, None, {})  # type: ignore
 
     @distributed_trace_async
-    async def head404(self, **kwargs: Any) -> None:  # pylint: disable=inconsistent-return-statements
+    async def head404(self, **kwargs: Any) -> None:
         """Return 404 status code if successful.
 
         :return: None
         :rtype: None
         :raises ~azure.core.exceptions.HttpResponseError:
         """
-        error_map: MutableMapping[int, Type[HttpResponseError]] = {  # pylint: disable=unsubscriptable-object
+        error_map: MutableMapping = {
             401: ClientAuthenticationError,
             404: ResourceNotFoundError,
             409: ResourceExistsError,