refactor: Centralize version management in _version.py and implement lazy loading for APCoreMCP.

Author: tercel

src/apcore_mcp/__init__.py

@@ -16,7 +16,6 @@
 from apcore_mcp.adapters.errors import ErrorMapper
 from apcore_mcp.adapters.id_normalizer import ModuleIDNormalizer
 from apcore_mcp.adapters.schema import SchemaConverter
-from apcore_mcp.apcore_mcp import APCoreMCP
 from apcore_mcp.auth import Authenticator, AuthMiddleware, ClaimMapping, JWTAuthenticator
 from apcore_mcp.constants import ERROR_CODES, MODULE_ID_PATTERN, REGISTRY_EVENTS
 from apcore_mcp.converters.openai import OpenAIConverter
@@ -65,7 +64,19 @@
     "MCP_ELICIT_KEY",
 ]
 
-__version__ = "0.10.0"
+from apcore_mcp._version import VERSION
+
+__version__ = VERSION
+
+
+def __getattr__(name: str) -> object:
+    if name == "APCoreMCP":
+        from apcore_mcp.apcore_mcp import APCoreMCP
+
+        globals()["APCoreMCP"] = APCoreMCP
+        return APCoreMCP
+    raise AttributeError(f"module {__name__!r} has no attribute {name!r}")
+
 
 logger = logging.getLogger(__name__)
 

src/apcore_mcp/_version.py

@@ -0,0 +1,3 @@
+"""Single source of truth for the package version."""
+
+VERSION = "0.10.0"

src/apcore_mcp/apcore_mcp.py

@@ -104,7 +104,7 @@ def __init__(
             logging.getLogger("apcore_mcp").setLevel(getattr(logging, log_level.upper()))
 
         # Resolve backend: str/Path → Registry with discover(), otherwise pass through
-        if isinstance(extensions_dir_or_backend, (str, Path)):
+        if isinstance(extensions_dir_or_backend, str | Path):
             from apcore import Registry
 
             backend: object = Registry(extensions_dir=str(extensions_dir_or_backend))
@@ -153,11 +153,11 @@ def _build_server_components(self) -> tuple[Any, Any, list, Any, Any]:
         Returns:
             Tuple of (server, router, tools, init_options, version).
         """
-        from apcore_mcp import __version__
+        from apcore_mcp._version import VERSION
         from apcore_mcp.server.factory import MCPServerFactory
         from apcore_mcp.server.router import ExecutionRouter
 
-        version = self._version or __version__
+        version = self._version or VERSION
 
         factory = MCPServerFactory()
         server = factory.create_server(name=self._name, version=version)
@@ -194,7 +194,9 @@ def _build_explorer_routes(
         logger.info("Tool Explorer enabled at %s", explorer_prefix)
         return [mount]
 
-    def _build_auth_middleware(self, *, explorer: bool = False, explorer_prefix: str = "/explorer") -> list[tuple[type, dict]] | None:
+    def _build_auth_middleware(
+        self, *, explorer: bool = False, explorer_prefix: str = "/explorer"
+    ) -> list[tuple[type, dict]] | None:
         """Build auth middleware list if authenticator is configured."""
         if self._authenticator is None:
             return None

tests/test_apcore_mcp.py

@@ -9,7 +9,6 @@
 
 from apcore_mcp.apcore_mcp import APCoreMCP
 
-
 # ---------------------------------------------------------------------------
 # Stubs
 # ---------------------------------------------------------------------------
@@ -58,6 +57,7 @@ async def call_async(self, module_id: str, inputs: dict[str, Any]) -> Any:
 # Helpers to build common patches for serve/async_serve
 # ---------------------------------------------------------------------------
 
+
 def _make_serve_patches():
     """Return patches for the lazy imports inside APCoreMCP.serve()."""
     return (
@@ -118,7 +118,7 @@ def test_from_path_object(self) -> None:
 
         mock_registry = StubRegistry()
         with patch("apcore.Registry", return_value=mock_registry) as mock_cls:
-            mcp = APCoreMCP(Path("/tmp/extensions"))
+            APCoreMCP(Path("/tmp/extensions"))
             mock_cls.assert_called_once_with(extensions_dir="/tmp/extensions")
 
     def test_custom_name(self) -> None: