Remove standalone Telegram/Naver clients from public API

Drop top-level re-exports + __all__ entries for Telegram/Naver
(datamaxi) and AsyncTelegram/AsyncNaver (datamaxi.aio); bind aio
mount imports privately so the names aren't importable. Impl classes
kept, reachable only via maxi.telegram / maxi.naver. Tests source the
classes from their package modules / mounts.

Closes #188

Author: martinkersner

datamaxi/__init__.py

@@ -1,6 +1,4 @@
 from datamaxi.resources import Datamaxi  # noqa: F401
-from datamaxi.telegram import Telegram  # noqa: F401
-from datamaxi.naver import Naver  # noqa: F401
 from datamaxi.lib.constants import (  # noqa: F401
     SPOT,
     FUTURES,
@@ -46,8 +44,6 @@
 
 __all__ = [
     "Datamaxi",
-    "Telegram",
-    "Naver",
     "SPOT",
     "FUTURES",
     "USD",

datamaxi/aio/__init__.py

@@ -16,12 +16,10 @@
 
 Mirrors the full sync surface (``cex.*``, ``funding_rate``, ``forex``,
 ``premium``, ``liquidation``, ``open_interest``, ``margin_borrow``,
-``index_price``, ``telegram``, ``naver``). The standalone
-``AsyncTelegram`` / ``AsyncNaver`` classes stay exported for back-compat.
-Reuses
-the sync client's endpoint resolution and error handling (``datamaxi._dispatch``)
-and the shared DataFrame / ResponseMeta helpers, so the two clients can't drift
-on request building or error semantics.
+``index_price``, ``telegram``, ``naver``). Reuses the sync client's endpoint
+resolution and error handling (``datamaxi._dispatch``) and the shared DataFrame
+/ ResponseMeta helpers, so the two clients can't drift on request building or
+error semantics.
 """
 
 from typing import Any
@@ -45,8 +43,8 @@
 from datamaxi.aio.open_interest import AsyncOpenInterest
 from datamaxi.aio.margin_borrow import AsyncMarginBorrow
 from datamaxi.aio.index_price import AsyncIndexPrice
-from datamaxi.aio.telegram import AsyncTelegram
-from datamaxi.aio.naver import AsyncNaver
+from datamaxi.aio.telegram import AsyncTelegram as _AsyncTelegram
+from datamaxi.aio.naver import AsyncNaver as _AsyncNaver
 
 
 class AsyncDatamaxi:
@@ -70,8 +68,8 @@ def __init__(self, api_key=None, **kwargs: Any):
         self.open_interest = AsyncOpenInterest(api)
         self.margin_borrow = AsyncMarginBorrow(api)
         self.index_price = AsyncIndexPrice(api)
-        self.telegram = AsyncTelegram(api=api)
-        self.naver = AsyncNaver(api=api)
+        self.telegram = _AsyncTelegram(api=api)
+        self.naver = _AsyncNaver(api=api)
 
     async def aclose(self):
         await self._api.aclose()
@@ -90,8 +88,6 @@ def __repr__(self):
 
 __all__ = [
     "AsyncDatamaxi",
-    "AsyncTelegram",
-    "AsyncNaver",
     "AsyncAPI",
     "AsyncResource",
     "AsyncCex",

tests/conftest.py

@@ -9,7 +9,7 @@
 
 import pytest
 
-from datamaxi import Datamaxi, Telegram, Naver
+from datamaxi import Datamaxi
 from datamaxi.api import API
 from datamaxi.error import ServerError
 
@@ -76,11 +76,11 @@ def datamaxi():
 
 @pytest.fixture(scope="module")
 def telegram():
-    """Create Telegram client for live tests."""
-    return Telegram(api_key=API_KEY, base_url=BASE_URL, timeout=TIMEOUT)
+    """Telegram resource (mounted) for live tests."""
+    return Datamaxi(api_key=API_KEY, base_url=BASE_URL, timeout=TIMEOUT).telegram
 
 
 @pytest.fixture(scope="module")
 def naver():
-    """Create Naver client for live tests."""
-    return Naver(api_key=API_KEY, base_url=BASE_URL, timeout=TIMEOUT)
+    """Naver resource (mounted) for live tests."""
+    return Datamaxi(api_key=API_KEY, base_url=BASE_URL, timeout=TIMEOUT).naver

tests/test_async_resources.py

@@ -11,7 +11,9 @@
 
 httpx = pytest.importorskip("httpx")
 
-from datamaxi.aio import AsyncDatamaxi, AsyncTelegram, AsyncNaver  # noqa: E402
+from datamaxi.aio import AsyncDatamaxi  # noqa: E402
+from datamaxi.aio.telegram import AsyncTelegram  # noqa: E402
+from datamaxi.aio.naver import AsyncNaver  # noqa: E402
 
 BASE_URL = "https://api.datamaxiplus.com"
 
@@ -171,6 +173,13 @@ async def run():
     assert _run(run()) == _CHANNELS
 
 
+def test_standalone_async_clients_not_top_level_importable():
+    import datamaxi.aio
+
+    assert not hasattr(datamaxi.aio, "AsyncTelegram")
+    assert not hasattr(datamaxi.aio, "AsyncNaver")
+
+
 def test_async_telegram_naver_mounted_reuse_shared_session():
     c = _dm()
     assert isinstance(c.telegram, AsyncTelegram)

tests/test_naver.py

@@ -6,7 +6,8 @@
 import pytest
 from urllib.parse import urlparse, parse_qs
 
-from datamaxi import Datamaxi, Naver
+from datamaxi import Datamaxi
+from datamaxi.naver import Naver
 from datamaxi.error import ClientError, ServerError
 from tests.util import mock_http_response
 

tests/test_telegram.py

@@ -5,7 +5,8 @@
 import pytest
 from urllib.parse import urlparse, parse_qs
 
-from datamaxi import Datamaxi, Telegram
+from datamaxi import Datamaxi
+from datamaxi.telegram import Telegram
 from datamaxi.error import ClientError, ServerError
 from tests.util import mock_http_response
 
@@ -125,3 +126,10 @@ def test_telegram_mounted_messages_work():
     res, next_request = maxi.telegram.messages(channel_name="alpha")
     assert res == _MESSAGES
     assert callable(next_request)
+
+
+def test_standalone_clients_not_top_level_importable():
+    import datamaxi
+
+    assert not hasattr(datamaxi, "Telegram")
+    assert not hasattr(datamaxi, "Naver")