Restore package-level __all__ and add DataverseClient to top-level

Re-exports are now declared in each package's __all__ list:
- top-level: DataverseClient, __version__, col, raw, DataverseModel, QueryResult
  (added DataverseClient; the 4 GA convenience symbols remain)
- core: 8 symbols (DataverseConfig, OperationContext, all 5 errors, LogConfig)
- models: 24 symbols
- operations: 13 symbols

Why: analysis of the original 60-warning doc-build report showed warnings
were caused by three independent root causes:
1. External pandas refs (~22) - fixed by doc-repo xrefmap_custom.yml
2. Docstring bugs (~17) - fixed by earlier commits qualifying :class: refs
3. Re-export amplification (~21) - doubled the docstring bugs above

Now that root causes 1 and 2 are resolved, restoring __all__ no longer
amplifies any warnings. The latest docfx report shows 3 warnings (all
since fixed) and no duplicate-target warnings.

Top-level __init__.py also adds DataverseClient (the primary entry point,
matching 'from openai import OpenAI' / 'from anthropic import Anthropic'
SDK conventions).

Tests updated to assert exact __all__ contents per package (catches
accidental drift).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: Abel Milash

src/PowerPlatform/Dataverse/__init__.py

@@ -3,10 +3,21 @@
 
 from importlib.metadata import version
 
-from .models.filters import col, raw
-from .models.protocol import DataverseModel
-from .models.record import QueryResult
-
+# Set __version__ FIRST. Downstream modules (e.g. data/_odata_base.py) import
+# this back from the top-level package, so it must be bound before any
+# transitive import of those modules runs.
 __version__ = version("PowerPlatform-Dataverse-Client")
 
-__all__ = ["__version__", "col", "raw", "DataverseModel", "QueryResult"]
+from .client import DataverseClient  # noqa: E402
+from .models.filters import col, raw  # noqa: E402
+from .models.protocol import DataverseModel  # noqa: E402
+from .models.record import QueryResult  # noqa: E402
+
+__all__ = [
+    "DataverseClient",
+    "DataverseModel",
+    "QueryResult",
+    "__version__",
+    "col",
+    "raw",
+]

src/PowerPlatform/Dataverse/core/__init__.py

@@ -12,4 +12,13 @@
 from .errors import DataverseError, HttpError, MetadataError, SQLParseError, ValidationError
 from .log_config import LogConfig
 
-__all__: list[str] = []
+__all__ = [
+    "DataverseConfig",
+    "DataverseError",
+    "HttpError",
+    "LogConfig",
+    "MetadataError",
+    "OperationContext",
+    "SQLParseError",
+    "ValidationError",
+]

src/PowerPlatform/Dataverse/models/__init__.py

@@ -34,4 +34,29 @@
 from .table_info import AlternateKeyInfo, ColumnInfo, TableInfo
 from .upsert import UpsertItem
 
-__all__: list[str] = []
+__all__ = [
+    "AlternateKeyInfo",
+    "BatchItemResponse",
+    "BatchResult",
+    "CascadeConfiguration",
+    "ColumnInfo",
+    "ColumnProxy",
+    "DataverseModel",
+    "ExpandOption",
+    "FetchXmlQuery",
+    "FilterExpression",
+    "Label",
+    "LocalizedLabel",
+    "LookupAttributeMetadata",
+    "ManyToManyRelationshipMetadata",
+    "OneToManyRelationshipMetadata",
+    "QueryBuilder",
+    "QueryParams",
+    "QueryResult",
+    "Record",
+    "RelationshipInfo",
+    "TableInfo",
+    "UpsertItem",
+    "col",
+    "raw",
+]

src/PowerPlatform/Dataverse/operations/__init__.py

@@ -24,4 +24,18 @@
 from .records import RecordOperations
 from .tables import TableOperations
 
-__all__: list[str] = []
+__all__ = [
+    "BatchDataFrameOperations",
+    "BatchOperations",
+    "BatchQueryOperations",
+    "BatchRecordOperations",
+    "BatchRequest",
+    "BatchTableOperations",
+    "ChangeSet",
+    "ChangeSetRecordOperations",
+    "DataFrameOperations",
+    "FileOperations",
+    "QueryOperations",
+    "RecordOperations",
+    "TableOperations",
+]

tests/unit/test_package_exports.py

@@ -1,21 +1,30 @@
 # Copyright (c) Microsoft Corporation.
 # Licensed under the MIT license.
 
-"""Tests for package-level imports.
+"""Tests for package-level re-exports.
 
-The three sub-packages (``core``, ``models``, ``operations``) deliberately keep
-``__all__`` empty to avoid creating duplicate doc-tool entries for re-exported
-symbols. Symbols are still importable from the package namespace because
-Python's ``from package import Symbol`` does not consult ``__all__``.
+Each package (``PowerPlatform.Dataverse``, ``core``, ``models``, ``operations``)
+re-exports its public symbols and declares them in ``__all__``. This gives users
+short, stable import paths (``from PowerPlatform.Dataverse.models import Record``)
+that survive internal module reorganization.
 
 These tests verify:
-1. ``__all__`` is empty (locks in the design decision).
-2. Every expected public symbol is still importable from the package namespace.
-3. Each imported symbol is the same object as its source definition.
+1. ``__all__`` matches the expected list exactly (catches accidental drift).
+2. Every name in ``__all__`` is importable from the package namespace.
+3. Each re-export is the same object as its source definition.
 """
 
 import unittest
 
+DATAVERSE_EXPECTED = [
+    "DataverseClient",
+    "DataverseModel",
+    "QueryResult",
+    "__version__",
+    "col",
+    "raw",
+]
+
 CORE_EXPECTED = [
     "DataverseConfig",
     "DataverseError",
@@ -71,14 +80,46 @@
 ]
 
 
+class TestDataverseTopLevelExports(unittest.TestCase):
+    """Verify top-level PowerPlatform.Dataverse package exports."""
+
+    def test_all_matches_expected(self):
+        """``__all__`` matches the expected list exactly."""
+        import PowerPlatform.Dataverse as m
+
+        self.assertEqual(sorted(m.__all__), sorted(DATAVERSE_EXPECTED))
+
+    def test_expected_symbols_importable(self):
+        """Every expected public symbol is reachable from the package namespace."""
+        import PowerPlatform.Dataverse as m
+
+        for name in DATAVERSE_EXPECTED:
+            self.assertTrue(hasattr(m, name), f"{name!r} not importable from PowerPlatform.Dataverse")
+
+    def test_identity(self):
+        """Re-exported objects are the same objects as their source definitions."""
+        import PowerPlatform.Dataverse as m
+        from PowerPlatform.Dataverse.client import DataverseClient
+        from PowerPlatform.Dataverse.models.filters import col, raw
+        from PowerPlatform.Dataverse.models.protocol import DataverseModel
+        from PowerPlatform.Dataverse.models.record import QueryResult
+
+        self.assertIs(m.DataverseClient, DataverseClient)
+        self.assertIs(m.DataverseModel, DataverseModel)
+        self.assertIs(m.QueryResult, QueryResult)
+        self.assertIs(m.col, col)
+        self.assertIs(m.raw, raw)
+        self.assertIsInstance(m.__version__, str)
+
+
 class TestCoreExports(unittest.TestCase):
     """Verify package-level imports for PowerPlatform.Dataverse.core."""
 
-    def test_all_is_empty(self):
-        """``__all__`` is deliberately empty to avoid doc-tool duplication."""
+    def test_all_matches_expected(self):
+        """``__all__`` matches the expected list exactly."""
         import PowerPlatform.Dataverse.core as m
 
-        self.assertEqual(m.__all__, [])
+        self.assertEqual(sorted(m.__all__), sorted(CORE_EXPECTED))
 
     def test_expected_symbols_importable(self):
         """Every expected public symbol is reachable from the package namespace."""
@@ -113,11 +154,11 @@ def test_identity(self):
 class TestModelsExports(unittest.TestCase):
     """Verify package-level imports for PowerPlatform.Dataverse.models."""
 
-    def test_all_is_empty(self):
-        """``__all__`` is deliberately empty to avoid doc-tool duplication."""
+    def test_all_matches_expected(self):
+        """``__all__`` matches the expected list exactly."""
         import PowerPlatform.Dataverse.models as m
 
-        self.assertEqual(m.__all__, [])
+        self.assertEqual(sorted(m.__all__), sorted(MODELS_EXPECTED))
 
     def test_expected_symbols_importable(self):
         """Every expected public symbol is reachable from the package namespace."""
@@ -175,11 +216,11 @@ def test_identity(self):
 class TestOperationsExports(unittest.TestCase):
     """Verify package-level imports for PowerPlatform.Dataverse.operations."""
 
-    def test_all_is_empty(self):
-        """``__all__`` is deliberately empty to avoid doc-tool duplication."""
+    def test_all_matches_expected(self):
+        """``__all__`` matches the expected list exactly."""
         import PowerPlatform.Dataverse.operations as m
 
-        self.assertEqual(m.__all__, [])
+        self.assertEqual(sorted(m.__all__), sorted(OPERATIONS_EXPECTED))
 
     def test_expected_symbols_importable(self):
         """Every expected public symbol is reachable from the package namespace."""