add class underscore

Author: Max Wang

examples/basic/installation_example.py

@@ -84,9 +84,9 @@ def validate_imports():
 
         print(f"  ✅ Core config: DataverseConfig")
 
-        from PowerPlatform.Dataverse.data._odata import ODataClient
+        from PowerPlatform.Dataverse.data._odata import _ODataClient
 
-        print(f"  ✅ Data layer: ODataClient")
+        print(f"  ✅ Data layer: _ODataClient")
 
         # Test Azure Identity import
         from azure.identity import InteractiveBrowserCredential

src/PowerPlatform/Dataverse/client.py

@@ -7,9 +7,9 @@
 
 from azure.core.credentials import TokenCredential
 
-from .core._auth import AuthManager
+from .core._auth import _AuthManager
 from .core.config import DataverseConfig
-from .data._odata import ODataClient
+from .data._odata import _ODataClient
 
 
 class DataverseClient:
@@ -18,7 +18,7 @@ class DataverseClient:
 
     This client provides a simple, stable interface for interacting with Dataverse environments
     through the Web API. It handles authentication via Azure Identity and delegates HTTP operations
-    to an internal :class:`~PowerPlatform.Dataverse.data._odata.ODataClient`.
+    to an internal :class:`~PowerPlatform.Dataverse.data._odata._ODataClient`.
 
     Key capabilities:
         - OData CRUD operations: create, read, update, delete records
@@ -74,25 +74,25 @@ def __init__(
         credential: TokenCredential,
         config: Optional[DataverseConfig] = None,
     ) -> None:
-        self.auth = AuthManager(credential)
+        self.auth = _AuthManager(credential)
         self._base_url = (base_url or "").rstrip("/")
         if not self._base_url:
             raise ValueError("base_url is required.")
         self._config = config or DataverseConfig.from_env()
-        self._odata: Optional[ODataClient] = None
+        self._odata: Optional[_ODataClient] = None
 
-    def _get_odata(self) -> ODataClient:
+    def _get_odata(self) -> _ODataClient:
         """
         Get or create the internal OData client instance.
 
         This method implements lazy initialization of the low-level OData client,
         deferring construction until the first API call.
 
         :return: The lazily-initialized low-level client used to perform HTTP requests.
-        :rtype: ~PowerPlatform.Dataverse.data._odata.ODataClient
+        :rtype: ~PowerPlatform.Dataverse.data._odata._ODataClient
         """
         if self._odata is None:
-            self._odata = ODataClient(
+            self._odata = _ODataClient(
                 self.auth,
                 self._base_url,
                 self._config,

src/PowerPlatform/Dataverse/core/_auth.py

@@ -4,8 +4,8 @@
 """
 Authentication helpers for Dataverse.
 
-This module provides :class:`~PowerPlatform.Dataverse.core._auth.AuthManager`, a thin wrapper over any Azure Identity
-``TokenCredential`` for acquiring OAuth2 access tokens, and :class:`~PowerPlatform.Dataverse.core._auth.TokenPair` for
+This module provides :class:`~PowerPlatform.Dataverse.core._auth._AuthManager`, a thin wrapper over any Azure Identity
+``TokenCredential`` for acquiring OAuth2 access tokens, and :class:`~PowerPlatform.Dataverse.core._auth._TokenPair` for
 storing the acquired token alongside its scope.
 """
 
@@ -17,7 +17,7 @@
 
 
 @dataclass
-class TokenPair:
+class _TokenPair:
     """
     Container for an OAuth2 access token and its associated resource scope.
 
@@ -31,7 +31,7 @@ class TokenPair:
     access_token: str
 
 
-class AuthManager:
+class _AuthManager:
     """
     Azure Identity-based authentication manager for Dataverse.
 
@@ -45,15 +45,15 @@ def __init__(self, credential: TokenCredential) -> None:
             raise TypeError("credential must implement azure.core.credentials.TokenCredential.")
         self.credential: TokenCredential = credential
 
-    def _acquire_token(self, scope: str) -> TokenPair:
+    def _acquire_token(self, scope: str) -> _TokenPair:
         """
         Acquire an access token for the specified OAuth2 scope.
 
         :param scope: OAuth2 scope string, typically ``"https://<org>.crm.dynamics.com/.default"``.
         :type scope: :class:`str`
         :return: Token pair containing the scope and access token.
-        :rtype: ~PowerPlatform.Dataverse.core._auth.TokenPair
+        :rtype: ~PowerPlatform.Dataverse.core._auth._TokenPair
         :raises ~azure.core.exceptions.ClientAuthenticationError: If token acquisition fails.
         """
         token = self.credential.get_token(scope)
-        return TokenPair(resource=scope, access_token=token.token)
+        return _TokenPair(resource=scope, access_token=token.token)

src/PowerPlatform/Dataverse/core/_http.py

@@ -4,7 +4,7 @@
 """
 HTTP client with automatic retry logic and timeout handling.
 
-This module provides :class:`~PowerPlatform.Dataverse.core._http.HttpClient`, a wrapper
+This module provides :class:`~PowerPlatform.Dataverse.core._http._HttpClient`, a wrapper
 around the requests library that adds configurable retry behavior for transient
 network errors and intelligent timeout management based on HTTP method types.
 """
@@ -17,7 +17,7 @@
 import requests
 
 
-class HttpClient:
+class _HttpClient:
     """
     HTTP client with configurable retry logic and timeout handling.
 

src/PowerPlatform/Dataverse/data/_odata.py

@@ -14,8 +14,8 @@
 from datetime import datetime, timezone
 import importlib.resources as ir
 
-from ..core._http import HttpClient
-from ._upload import ODataFileUpload
+from ..core._http import _HttpClient
+from ._upload import _ODataFileUpload
 from ..core.errors import *
 from ..core._error_codes import (
     _http_subcode,
@@ -37,7 +37,7 @@
 _GUID_RE = re.compile(r"[0-9a-fA-F]{8}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{12}")
 
 
-class ODataClient(ODataFileUpload):
+class _ODataClient(_ODataFileUpload):
     """Dataverse Web API client: CRUD, SQL-over-API, and table metadata helpers."""
 
     @staticmethod
@@ -82,7 +82,7 @@ def __init__(
         Sets up authentication, base URL, configuration, and internal caches.
 
         :param auth: Authentication manager providing ``_acquire_token(scope)`` that returns an object with ``access_token``.
-        :type auth: ~PowerPlatform.Dataverse.core._auth.AuthManager
+        :type auth: ~PowerPlatform.Dataverse.core._auth._AuthManager
         :param base_url: Organization base URL (e.g. ``"https://<org>.crm.dynamics.com"``).
         :type base_url: ``str``
         :param config: Optional Dataverse configuration (HTTP retry, backoff, timeout, language code). If omitted ``DataverseConfig.from_env()`` is used.
@@ -100,7 +100,7 @@ def __init__(
                 "PowerPlatform.Dataverse.core.config", fromlist=["DataverseConfig"]
             ).DataverseConfig.from_env()
         )
-        self._http = HttpClient(
+        self._http = _HttpClient(
             retries=self.config.http_retries,
             backoff=self.config.http_backoff,
             timeout=self.config.http_timeout,

src/PowerPlatform/Dataverse/data/_upload.py

@@ -8,7 +8,7 @@
 from typing import Optional
 
 
-class ODataFileUpload:
+class _ODataFileUpload:
     """File upload capabilities (small + chunk) with auto selection."""
 
     def _upload_file(

tests/unit/core/test_http_errors.py

@@ -4,7 +4,7 @@
 import pytest
 from PowerPlatform.Dataverse.core.errors import HttpError
 from PowerPlatform.Dataverse.core._error_codes import HTTP_404, HTTP_429, HTTP_500
-from PowerPlatform.Dataverse.data._odata import ODataClient
+from PowerPlatform.Dataverse.data._odata import _ODataClient
 
 
 class DummyAuth:
@@ -49,7 +49,7 @@ def json_fail():
         return r
 
 
-class MockClient(ODataClient):
+class MockClient(_ODataClient):
     def __init__(self, responses):
         super().__init__(DummyAuth(), "https://org.example", None)
         self._http = DummyHTTP(responses)

tests/unit/data/test_enum_optionset_payload.py

@@ -4,7 +4,7 @@
 import pytest
 from enum import Enum, IntEnum
 
-from PowerPlatform.Dataverse.data._odata import ODataClient
+from PowerPlatform.Dataverse.data._odata import _ODataClient
 
 
 class DummyAuth:
@@ -16,18 +16,18 @@ class T:
 
 
 class DummyConfig:
-    """Minimal config stub providing attributes ODataClient.__init__ expects."""
+    """Minimal config stub providing attributes _ODataClient.__init__ expects."""
 
     def __init__(self, language_code=1033):
         self.language_code = language_code
-        # HTTP settings referenced during ODataClient construction
+        # HTTP settings referenced during _ODataClient construction
         self.http_retries = 0
         self.http_backoff = 0
         self.http_timeout = 5
 
 
 def _make_client(lang=1033):
-    return ODataClient(DummyAuth(), "https://org.example", DummyConfig(language_code=lang))
+    return _ODataClient(DummyAuth(), "https://org.example", DummyConfig(language_code=lang))
 
 
 def _labels_for(option):

tests/unit/data/test_logical_crud.py

@@ -3,7 +3,7 @@
 
 import types
 import pytest
-from PowerPlatform.Dataverse.data._odata import ODataClient
+from PowerPlatform.Dataverse.data._odata import _ODataClient
 from PowerPlatform.Dataverse.core.errors import MetadataError
 
 
@@ -43,7 +43,7 @@ def json_func():
         return resp
 
 
-class MockableClient(ODataClient):
+class MockableClient(_ODataClient):
     def __init__(self, responses):
         super().__init__(DummyAuth(), "https://org.example", None)
         self._http = DummyHTTPClient(responses)

tests/unit/data/test_sql_parse.py

@@ -2,7 +2,7 @@
 # Licensed under the MIT license.
 
 import pytest
-from PowerPlatform.Dataverse.data._odata import ODataClient
+from PowerPlatform.Dataverse.data._odata import _ODataClient
 
 
 class DummyAuth:
@@ -14,7 +14,7 @@ class T:
 
 
 def _client():
-    return ODataClient(DummyAuth(), "https://org.example", None)
+    return _ODataClient(DummyAuth(), "https://org.example", None)
 
 
 def test_basic_from():