Do not expose mixin operation group

packages/autorest.python/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Release
 
+## 6.36.0
+
+### Features
+
+- Make mixin operations classes private to remove from documentation
+
+
 ## 6.35.5
 
 No changes, version bump only.

packages/autorest.python/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@autorest/python",
-  "version": "6.35.5",
+  "version": "6.36.0",
   "description": "The Python extension for generators in AutoRest.",
   "scripts": {
     "start": "node ./scripts/run-python3.js ./scripts/start.py",
@@ -29,7 +29,7 @@
   },
   "homepage": "https://github.com/Azure/autorest.python/blob/main/README.md",
   "dependencies": {
-    "@typespec/http-client-python": "0.13.0-dev.3",
+    "@typespec/http-client-python": "~0.13.0",
     "@autorest/system-requirements": "~1.0.2",
     "fs-extra": "~11.2.0",
     "tsx": "~4.19.1"
@@ -47,4 +47,4 @@
     "requirements.txt",
     "generator/"
   ]
-}
+}

packages/autorest.python/samples/specification/directives/generated/azure/directives/sample/_client.py

@@ -15,11 +15,11 @@
 from azure.core.rest import HttpRequest, HttpResponse
 
 from ._configuration import PollingPagingExampleConfiguration
-from ._operations import PollingPagingExampleOperationsMixin
+from ._operations._operations import _PollingPagingExampleOperationsMixin
 from ._utils.serialization import Deserializer, Serializer
 
 
-class PollingPagingExample(PollingPagingExampleOperationsMixin):  # pylint: disable=client-accepts-api-version-keyword
+class PollingPagingExample(_PollingPagingExampleOperationsMixin):  # pylint: disable=client-accepts-api-version-keyword
     """Show polling and paging generation.
 
     :keyword endpoint: Service URL. Default value is "http://localhost:3000".

packages/autorest.python/samples/specification/directives/generated/azure/directives/sample/_operations/__init__.py

@@ -12,14 +12,11 @@
 if TYPE_CHECKING:
     from ._patch import *  # pylint: disable=unused-wildcard-import
 
-from ._operations import PollingPagingExampleOperationsMixin  # type: ignore
 
 from ._patch import __all__ as _patch_all
 from ._patch import *
 from ._patch import patch_sdk as _patch_sdk
 
-__all__ = [
-    "PollingPagingExampleOperationsMixin",
-]
+__all__ = []
 __all__.extend([p for p in _patch_all if p not in __all__])  # pyright: ignore
 _patch_sdk()

packages/autorest.python/samples/specification/directives/generated/azure/directives/sample/_operations/_operations.py

@@ -72,7 +72,7 @@ def build_polling_paging_example_basic_paging_request(**kwargs: Any) -> HttpRequ
     return HttpRequest(method="GET", url=_url, headers=_headers, **kwargs)
 
 
-class PollingPagingExampleOperationsMixin(
+class _PollingPagingExampleOperationsMixin(
     ClientMixinABC[PipelineClient[HttpRequest, HttpResponse], PollingPagingExampleConfiguration]
 ):
 

packages/autorest.python/samples/specification/directives/generated/azure/directives/sample/aio/_client.py

@@ -16,10 +16,10 @@
 
 from .._utils.serialization import Deserializer, Serializer
 from ._configuration import PollingPagingExampleConfiguration
-from ._operations import PollingPagingExampleOperationsMixin
+from ._operations._operations import _PollingPagingExampleOperationsMixin
 
 
-class PollingPagingExample(PollingPagingExampleOperationsMixin):  # pylint: disable=client-accepts-api-version-keyword
+class PollingPagingExample(_PollingPagingExampleOperationsMixin):  # pylint: disable=client-accepts-api-version-keyword
     """Show polling and paging generation.
 
     :keyword endpoint: Service URL. Default value is "http://localhost:3000".

packages/autorest.python/samples/specification/directives/generated/azure/directives/sample/aio/_operations/__init__.py

@@ -12,14 +12,11 @@
 if TYPE_CHECKING:
     from ._patch import *  # pylint: disable=unused-wildcard-import
 
-from ._operations import PollingPagingExampleOperationsMixin  # type: ignore
 
 from ._patch import __all__ as _patch_all
 from ._patch import *
 from ._patch import patch_sdk as _patch_sdk
 
-__all__ = [
-    "PollingPagingExampleOperationsMixin",
-]
+__all__ = []
 __all__.extend([p for p in _patch_all if p not in __all__])  # pyright: ignore
 _patch_sdk()

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

@@ -43,7 +43,7 @@
 ClsType = Optional[Callable[[PipelineResponse[HttpRequest, AsyncHttpResponse], T, Dict[str, Any]], Any]]
 
 
-class PollingPagingExampleOperationsMixin(
+class _PollingPagingExampleOperationsMixin(
     ClientMixinABC[AsyncPipelineClient[HttpRequest, AsyncHttpResponse], PollingPagingExampleConfiguration]
 ):
 

packages/autorest.python/test/azure/legacy/AcceptanceTests/test_lro_parameterized_endpoints.py

@@ -45,13 +45,15 @@ def test_poll_with_constant_parameterized_endpoints(client):
 
 
 def test_operation_groups():
-    from lrowithparameterizedendpoints.operations import LROWithParamaterizedEndpointsOperationsMixin
+    from lrowithparameterizedendpoints.operations._lro_with_paramaterized_endpoints_operations import (
+        _LROWithParamaterizedEndpointsOperationsMixin,
+    )
 
     with pytest.raises(ImportError):
         from lrowithparameterizedendpoints.operations import _lro_with_paramaterized_endpoints_operations_py3
 
     from lrowithparameterizedendpoints.operations._lro_with_paramaterized_endpoints_operations import (
-        LROWithParamaterizedEndpointsOperationsMixin as LROWithParamaterizedEndpointsOperationsMixinPy2,
+        _LROWithParamaterizedEndpointsOperationsMixin as _LROWithParamaterizedEndpointsOperationsMixinPy2,
     )
 
-    assert LROWithParamaterizedEndpointsOperationsMixin == LROWithParamaterizedEndpointsOperationsMixinPy2
+    assert _LROWithParamaterizedEndpointsOperationsMixin == _LROWithParamaterizedEndpointsOperationsMixinPy2

packages/autorest.python/test/azure/legacy/Expected/AcceptanceTests/AzureReport/azurereport/_auto_rest_report_service_for_azure.py

@@ -17,11 +17,11 @@
 from . import models as _models
 from ._configuration import AutoRestReportServiceForAzureConfiguration
 from ._utils.serialization import Deserializer, Serializer
-from .operations import AutoRestReportServiceForAzureOperationsMixin
+from .operations._auto_rest_report_service_for_azure_operations import _AutoRestReportServiceForAzureOperationsMixin
 
 
 class AutoRestReportServiceForAzure(
-    AutoRestReportServiceForAzureOperationsMixin
+    _AutoRestReportServiceForAzureOperationsMixin
 ):  # pylint: disable=client-accepts-api-version-keyword
     """Test Infrastructure for AutoRest.
 

packages/autorest.python/test/azure/legacy/Expected/AcceptanceTests/AzureReport/azurereport/aio/_auto_rest_report_service_for_azure.py

@@ -17,11 +17,11 @@
 from .. import models as _models
 from .._utils.serialization import Deserializer, Serializer
 from ._configuration import AutoRestReportServiceForAzureConfiguration
-from .operations import AutoRestReportServiceForAzureOperationsMixin
+from .operations._auto_rest_report_service_for_azure_operations import _AutoRestReportServiceForAzureOperationsMixin
 
 
 class AutoRestReportServiceForAzure(
-    AutoRestReportServiceForAzureOperationsMixin
+    _AutoRestReportServiceForAzureOperationsMixin
 ):  # pylint: disable=client-accepts-api-version-keyword
     """Test Infrastructure for AutoRest.
 

packages/autorest.python/test/azure/legacy/Expected/AcceptanceTests/AzureReport/azurereport/aio/operations/__init__.py

@@ -12,14 +12,11 @@
 if TYPE_CHECKING:
     from ._patch import *  # pylint: disable=unused-wildcard-import
 
-from ._auto_rest_report_service_for_azure_operations import AutoRestReportServiceForAzureOperationsMixin  # type: ignore
 
 from ._patch import __all__ as _patch_all
 from ._patch import *
 from ._patch import patch_sdk as _patch_sdk
 
-__all__ = [
-    "AutoRestReportServiceForAzureOperationsMixin",
-]
+__all__ = []
 __all__.extend([p for p in _patch_all if p not in __all__])  # pyright: ignore
 _patch_sdk()

packages/autorest.python/test/azure/legacy/Expected/AcceptanceTests/AzureReport/azurereport/aio/operations/_auto_rest_report_service_for_azure_operations.py

@@ -31,7 +31,7 @@
 ClsType = Optional[Callable[[PipelineResponse[HttpRequest, AsyncHttpResponse], T, Dict[str, Any]], Any]]
 
 
-class AutoRestReportServiceForAzureOperationsMixin(  # pylint: disable=name-too-long
+class _AutoRestReportServiceForAzureOperationsMixin(
     ClientMixinABC[AsyncPipelineClient[HttpRequest, AsyncHttpResponse], AutoRestReportServiceForAzureConfiguration]
 ):
 

packages/autorest.python/test/azure/legacy/Expected/AcceptanceTests/AzureReport/azurereport/operations/__init__.py

@@ -12,14 +12,11 @@
 if TYPE_CHECKING:
     from ._patch import *  # pylint: disable=unused-wildcard-import
 
-from ._auto_rest_report_service_for_azure_operations import AutoRestReportServiceForAzureOperationsMixin  # type: ignore
 
 from ._patch import __all__ as _patch_all
 from ._patch import *
 from ._patch import patch_sdk as _patch_sdk
 
-__all__ = [
-    "AutoRestReportServiceForAzureOperationsMixin",
-]
+__all__ = []
 __all__.extend([p for p in _patch_all if p not in __all__])  # pyright: ignore
 _patch_sdk()

packages/autorest.python/test/azure/legacy/Expected/AcceptanceTests/AzureReport/azurereport/operations/_auto_rest_report_service_for_azure_operations.py

@@ -53,7 +53,7 @@ def build_get_report_request(*, qualifier: Optional[str] = None, **kwargs: Any)
     return HttpRequest(method="GET", url=_url, params=_params, headers=_headers, **kwargs)
 
 
-class AutoRestReportServiceForAzureOperationsMixin(  # pylint: disable=name-too-long
+class _AutoRestReportServiceForAzureOperationsMixin(
     ClientMixinABC[PipelineClient[HttpRequest, HttpResponse], AutoRestReportServiceForAzureConfiguration]
 ):
 

packages/autorest.python/test/azure/legacy/Expected/AcceptanceTests/LroWithParameterizedEndpoints/lrowithparameterizedendpoints/_lro_with_paramaterized_endpoints.py

@@ -17,11 +17,11 @@
 from . import models as _models
 from ._configuration import LROWithParamaterizedEndpointsConfiguration
 from ._utils.serialization import Deserializer, Serializer
-from .operations import LROWithParamaterizedEndpointsOperationsMixin
+from .operations._lro_with_paramaterized_endpoints_operations import _LROWithParamaterizedEndpointsOperationsMixin
 
 
 class LROWithParamaterizedEndpoints(
-    LROWithParamaterizedEndpointsOperationsMixin
+    _LROWithParamaterizedEndpointsOperationsMixin
 ):  # pylint: disable=client-accepts-api-version-keyword
     """Test Infrastructure for AutoRest.
 

packages/autorest.python/test/azure/legacy/Expected/AcceptanceTests/LroWithParameterizedEndpoints/lrowithparameterizedendpoints/aio/_lro_with_paramaterized_endpoints.py

@@ -17,11 +17,11 @@
 from .. import models as _models
 from .._utils.serialization import Deserializer, Serializer
 from ._configuration import LROWithParamaterizedEndpointsConfiguration
-from .operations import LROWithParamaterizedEndpointsOperationsMixin
+from .operations._lro_with_paramaterized_endpoints_operations import _LROWithParamaterizedEndpointsOperationsMixin
 
 
 class LROWithParamaterizedEndpoints(
-    LROWithParamaterizedEndpointsOperationsMixin
+    _LROWithParamaterizedEndpointsOperationsMixin
 ):  # pylint: disable=client-accepts-api-version-keyword
     """Test Infrastructure for AutoRest.
 

packages/autorest.python/test/azure/legacy/Expected/AcceptanceTests/LroWithParameterizedEndpoints/lrowithparameterizedendpoints/aio/operations/__init__.py

@@ -12,14 +12,11 @@
 if TYPE_CHECKING:
     from ._patch import *  # pylint: disable=unused-wildcard-import
 
-from ._lro_with_paramaterized_endpoints_operations import LROWithParamaterizedEndpointsOperationsMixin  # type: ignore
 
 from ._patch import __all__ as _patch_all
 from ._patch import *
 from ._patch import patch_sdk as _patch_sdk
 
-__all__ = [
-    "LROWithParamaterizedEndpointsOperationsMixin",
-]
+__all__ = []
 __all__.extend([p for p in _patch_all if p not in __all__])  # pyright: ignore
 _patch_sdk()

packages/autorest.python/test/azure/legacy/Expected/AcceptanceTests/LroWithParameterizedEndpoints/lrowithparameterizedendpoints/aio/operations/_lro_with_paramaterized_endpoints_operations.py

@@ -38,7 +38,7 @@
 ClsType = Optional[Callable[[PipelineResponse[HttpRequest, AsyncHttpResponse], T, Dict[str, Any]], Any]]
 
 
-class LROWithParamaterizedEndpointsOperationsMixin(  # pylint: disable=name-too-long
+class _LROWithParamaterizedEndpointsOperationsMixin(
     ClientMixinABC[AsyncPipelineClient[HttpRequest, AsyncHttpResponse], LROWithParamaterizedEndpointsConfiguration]
 ):
 

packages/autorest.python/test/azure/legacy/Expected/AcceptanceTests/LroWithParameterizedEndpoints/lrowithparameterizedendpoints/operations/__init__.py

@@ -12,14 +12,11 @@
 if TYPE_CHECKING:
     from ._patch import *  # pylint: disable=unused-wildcard-import
 
-from ._lro_with_paramaterized_endpoints_operations import LROWithParamaterizedEndpointsOperationsMixin  # type: ignore
 
 from ._patch import __all__ as _patch_all
 from ._patch import *
 from ._patch import patch_sdk as _patch_sdk
 
-__all__ = [
-    "LROWithParamaterizedEndpointsOperationsMixin",
-]
+__all__ = []
 __all__.extend([p for p in _patch_all if p not in __all__])  # pyright: ignore
 _patch_sdk()

packages/autorest.python/test/azure/legacy/Expected/AcceptanceTests/LroWithParameterizedEndpoints/lrowithparameterizedendpoints/operations/_lro_with_paramaterized_endpoints_operations.py

@@ -74,7 +74,7 @@ def build_poll_with_constant_parameterized_endpoints_request(  # pylint: disable
     return HttpRequest(method="POST", url=_url, headers=_headers, **kwargs)
 
 
-class LROWithParamaterizedEndpointsOperationsMixin(  # pylint: disable=name-too-long
+class _LROWithParamaterizedEndpointsOperationsMixin(
     ClientMixinABC[PipelineClient[HttpRequest, HttpResponse], LROWithParamaterizedEndpointsConfiguration]
 ):
 

packages/autorest.python/test/azure/legacy/Expected/AcceptanceTests/SecurityAadSwagger/securityaadswagger/_autorest_security_aad.py

@@ -19,13 +19,13 @@
 
 from ._configuration import AutorestSecurityAadConfiguration
 from ._utils.serialization import Deserializer, Serializer
-from .operations import AutorestSecurityAadOperationsMixin
+from .operations._autorest_security_aad_operations import _AutorestSecurityAadOperationsMixin
 
 if TYPE_CHECKING:
     from azure.core.credentials import TokenCredential
 
 
-class AutorestSecurityAad(AutorestSecurityAadOperationsMixin):  # pylint: disable=client-accepts-api-version-keyword
+class AutorestSecurityAad(_AutorestSecurityAadOperationsMixin):  # pylint: disable=client-accepts-api-version-keyword
     """Autorest Security Aad REST APIs.
 
     :param credential: Credential needed for the client to connect to Azure. Required.

packages/autorest.python/test/azure/legacy/Expected/AcceptanceTests/SecurityAadSwagger/securityaadswagger/aio/_autorest_security_aad.py

@@ -19,13 +19,13 @@
 
 from .._utils.serialization import Deserializer, Serializer
 from ._configuration import AutorestSecurityAadConfiguration
-from .operations import AutorestSecurityAadOperationsMixin
+from .operations._autorest_security_aad_operations import _AutorestSecurityAadOperationsMixin
 
 if TYPE_CHECKING:
     from azure.core.credentials_async import AsyncTokenCredential
 
 
-class AutorestSecurityAad(AutorestSecurityAadOperationsMixin):  # pylint: disable=client-accepts-api-version-keyword
+class AutorestSecurityAad(_AutorestSecurityAadOperationsMixin):  # pylint: disable=client-accepts-api-version-keyword
     """Autorest Security Aad REST APIs.
 
     :param credential: Credential needed for the client to connect to Azure. Required.

packages/autorest.python/test/azure/legacy/Expected/AcceptanceTests/SecurityAadSwagger/securityaadswagger/aio/operations/__init__.py

@@ -12,14 +12,11 @@
 if TYPE_CHECKING:
     from ._patch import *  # pylint: disable=unused-wildcard-import
 
-from ._autorest_security_aad_operations import AutorestSecurityAadOperationsMixin  # type: ignore
 
 from ._patch import __all__ as _patch_all
 from ._patch import *
 from ._patch import patch_sdk as _patch_sdk
 
-__all__ = [
-    "AutorestSecurityAadOperationsMixin",
-]
+__all__ = []
 __all__.extend([p for p in _patch_all if p not in __all__])  # pyright: ignore
 _patch_sdk()

packages/autorest.python/test/azure/legacy/Expected/AcceptanceTests/SecurityAadSwagger/securityaadswagger/aio/operations/_autorest_security_aad_operations.py

@@ -31,7 +31,7 @@
 ClsType = Optional[Callable[[PipelineResponse[HttpRequest, AsyncHttpResponse], T, Dict[str, Any]], Any]]
 
 
-class AutorestSecurityAadOperationsMixin(
+class _AutorestSecurityAadOperationsMixin(
     ClientMixinABC[AsyncPipelineClient[HttpRequest, AsyncHttpResponse], AutorestSecurityAadConfiguration]
 ):
 

packages/autorest.python/test/azure/legacy/Expected/AcceptanceTests/SecurityAadSwagger/securityaadswagger/operations/__init__.py

@@ -12,14 +12,11 @@
 if TYPE_CHECKING:
     from ._patch import *  # pylint: disable=unused-wildcard-import
 
-from ._autorest_security_aad_operations import AutorestSecurityAadOperationsMixin  # type: ignore
 
 from ._patch import __all__ as _patch_all
 from ._patch import *
 from ._patch import patch_sdk as _patch_sdk
 
-__all__ = [
-    "AutorestSecurityAadOperationsMixin",
-]
+__all__ = []
 __all__.extend([p for p in _patch_all if p not in __all__])  # pyright: ignore
 _patch_sdk()

packages/autorest.python/test/azure/legacy/Expected/AcceptanceTests/SecurityAadSwagger/securityaadswagger/operations/_autorest_security_aad_operations.py

@@ -40,7 +40,7 @@ def build_head_request(**kwargs: Any) -> HttpRequest:
     return HttpRequest(method="HEAD", url=_url, **kwargs)
 
 
-class AutorestSecurityAadOperationsMixin(
+class _AutorestSecurityAadOperationsMixin(
     ClientMixinABC[PipelineClient[HttpRequest, HttpResponse], AutorestSecurityAadConfiguration]
 ):