simple-module-hosting 0.0.25__tar.gz → 0.0.26__tar.gz

{simple_module_hosting-0.0.25 → simple_module_hosting-0.0.26}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: simple_module_hosting
-Version: 0.0.25
+Version: 0.0.26
 Summary: FastAPI + Inertia.js host runtime for simple_module — app_builder, middleware stack, CLI (sm / simple-module), scaffolding
 Project-URL: Homepage, https://github.com/antosubash/simple_module_python
 Project-URL: Repository, https://github.com/antosubash/simple_module_python
@@ -26,8 +26,8 @@ Requires-Dist: fastapi-inertia>=1.0
 Requires-Dist: fastapi>=0.115
 Requires-Dist: httpx>=0.27
 Requires-Dist: jinja2>=3.1
-Requires-Dist: simple-module-core==0.0.25
-Requires-Dist: simple-module-db==0.0.25
+Requires-Dist: simple-module-core==0.0.26
+Requires-Dist: simple-module-db==0.0.26
 Requires-Dist: starlette>=0.44
 Requires-Dist: tomlkit>=0.13
 Requires-Dist: uvicorn[standard]>=0.34

{simple_module_hosting-0.0.25 → simple_module_hosting-0.0.26}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "simple_module_hosting"
-version = "0.0.25"
+version = "0.0.26"
 description = "FastAPI + Inertia.js host runtime for simple_module — app_builder, middleware stack, CLI (sm / simple-module), scaffolding"
 readme = "README.md"
 license = "MIT"
@@ -26,8 +26,8 @@ dependencies = [
     "fastapi-inertia>=1.0",
     "httpx>=0.27",
     "jinja2>=3.1",
-    "simple_module_core==0.0.25",
-    "simple_module_db==0.0.25",
+    "simple_module_core==0.0.26",
+    "simple_module_db==0.0.26",
     "starlette>=0.44",
     "tomlkit>=0.13",
     "uvicorn[standard]>=0.34",

{simple_module_hosting-0.0.25 → simple_module_hosting-0.0.26}/simple_module_hosting/_phase_helpers.py

@@ -32,6 +32,8 @@ from simple_module_hosting._error_handlers import (
     request_validation_error_handler,
     unhandled_exception_handler,
 )
+from simple_module_hosting._host_services import _HostServices
+from simple_module_hosting.host_settings import HostSettings
 from simple_module_hosting.i18n_middleware import LocaleMiddleware
 from simple_module_hosting.middleware import (
     CorrelationIdMiddleware,
@@ -165,6 +167,32 @@ def mount_module_static_dirs(app: FastAPI, modules: list) -> None:
             )
 
 
+def register_host_settings(app: FastAPI) -> None:
+    """Register host-level settings under ``package="host"`` (DB-backed).
+
+    The Settings module must already have run ``register_settings`` — topo
+    order puts it early, since its ``meta.depends_on`` is empty. When the
+    Settings module isn't enabled there's no registry to register against, so
+    this skips quietly.
+
+    ``settings.registration`` is resolved via importlib rather than a plain
+    ``from settings.registration import ...``: the SM009 coupling check is
+    AST-based and forbids any static import of a plugin package name from
+    within ``framework/*``. Dynamic resolution keeps the framework AST
+    plugin-free while still hitting the real helper at runtime.
+    """
+    if not hasattr(app.state, "settings"):
+        return
+
+    import importlib
+
+    register_module_settings = importlib.import_module(
+        "settings.registration"
+    ).register_module_settings
+
+    register_module_settings(app, "host", HostSettings, lambda s: _HostServices(settings=s))
+
+
 def check_settings_registration(app: FastAPI, modules: list) -> list[Diagnostic]:
     """SM012: warn if a module overrides register_settings but added nothing to app.state.
 

{simple_module_hosting-0.0.25 → simple_module_hosting-0.0.26}/simple_module_hosting/app_builder.py

@@ -10,6 +10,7 @@ from contextlib import asynccontextmanager
 from pathlib import Path
 
 from fastapi import FastAPI
+from simple_module_core.design_packs import DesignPackRegistry
 from simple_module_core.diagnostics import DiagnosticLevel, print_diagnostics, run_diagnostics
 from simple_module_core.discovery import discover_modules, topological_sort
 from simple_module_core.events import EventBus
@@ -22,7 +23,6 @@ from simple_module_core.services import Services
 from simple_module_db.listeners import register_listeners
 from simple_module_db.session import init_db
 
-from simple_module_hosting._host_services import _HostServices
 from simple_module_hosting._inertia_setup import setup_inertia
 from simple_module_hosting._phase_helpers import (
     attach_public_routes,
@@ -30,10 +30,10 @@ from simple_module_hosting._phase_helpers import (
     install_middleware,
     mount_module_static_dirs,
     register_exception_handlers,
+    register_host_settings,
     wire_module_routes,
 )
 from simple_module_hosting.health import router as health_router
-from simple_module_hosting.host_settings import HostSettings
 from simple_module_hosting.i18n_manifest import build_i18n_registry, emit_frontend_types
 from simple_module_hosting.migrations import check_migrations
 from simple_module_hosting.settings import Settings
@@ -165,6 +165,7 @@ def create_app(settings: Settings | None = None) -> FastAPI:
     event_bus = EventBus()
     health_registry = HealthRegistry()
     public_route_registry = PublicRouteRegistry()
+    design_pack_registry = DesignPackRegistry()
 
     @asynccontextmanager
     async def lifespan(app: FastAPI) -> AsyncGenerator[None, None]:
@@ -203,25 +204,7 @@ def create_app(settings: Settings | None = None) -> FastAPI:
     for mod in modules:
         mod.register_settings(app)
 
-    # Register host-level settings under package="host" (DB-backed). The
-    # Settings module must already have run register_settings (topo order
-    # puts it early; its meta.depends_on = [] so it's among the first).
-    # When the Settings module isn't enabled, there's no registry to
-    # register against — skip quietly.
-    #
-    # We resolve `settings.registration` via importlib rather than a plain
-    # `from settings.registration import ...`: the SM009 coupling check is
-    # AST-based and forbids any static import of a plugin package name
-    # from within framework/* code. Dynamic resolution keeps the framework
-    # AST plugin-free while still hitting the real helper at runtime.
-    if hasattr(app.state, "settings"):
-        import importlib
-
-        _register_module_settings = importlib.import_module(
-            "settings.registration"
-        ).register_module_settings
-
-        _register_module_settings(app, "host", HostSettings, lambda s: _HostServices(settings=s))
+    register_host_settings(app)
 
     if settings.is_development:
         settings_diagnostics = check_settings_registration(app, modules)
@@ -236,17 +219,23 @@ def create_app(settings: Settings | None = None) -> FastAPI:
         _register_event_handlers(mod, event_bus, app)
         mod.register_health_checks(health_registry)
         mod.register_public_routes(public_route_registry)
+        mod.register_design_packs(design_pack_registry)
 
     attach_public_routes(app, settings, public_route_registry)
 
+    # Branding reads the packs off app.state directly: its API validates a
+    # submitted slug before persisting it, and its view builds the dropdown.
+    app.state.design_packs = design_pack_registry
+
     logger.info(
         "Registered %d menu items, %d permissions, %d feature flags, "
-        "%d health checks, %d public routes",
+        "%d health checks, %d public routes, %d design packs",
         len(menu_registry.all_items),
         len(perm_registry.all_permissions),
         len(ff_registry.all_flags),
         len(health_registry.all_checks),
         len(public_route_registry.routes),
+        len(design_pack_registry.all()),
     )
 
     # ── Phase 6: Initialize database ───────────────────────
@@ -291,6 +280,7 @@ def create_app(settings: Settings | None = None) -> FastAPI:
         feature_flags=ff_registry,
         health_registry=health_registry,
         public_routes=public_route_registry,
+        design_packs=design_pack_registry,
         i18n_registry=i18n_registry,
         inertia_config=inertia_config,
         modules=tuple(modules),

simple_module_hosting-0.0.26/tests/test_design_packs_wiring.py

@@ -0,0 +1,52 @@
+"""``create_app`` must collect module design packs onto ``app.state.design_packs``.
+
+Branding reads this registry twice — the view builds its dropdown from it, and
+the API validates a submitted slug against it. If the hook were never called,
+or the populated registry never reached ``app.state``, branding would offer an
+empty dropdown and reject every pack, with nothing in the logs to explain why.
+"""
+
+from __future__ import annotations
+
+import simple_module_hosting.app_builder as app_builder
+from simple_module_core.design_packs import DesignPack, DesignPackRegistry
+from simple_module_core.module import ModuleBase, ModuleMeta
+from simple_module_hosting.app_builder import create_app
+
+_PACK = DesignPack(value="pack-fixture", label="Pack Fixture")
+
+
+class _PackModule(ModuleBase):
+    """A module whose only contribution is one design pack."""
+
+    meta = ModuleMeta(name="PackFixture")
+
+    def register_design_packs(self, registry: DesignPackRegistry) -> None:
+        registry.register(_PACK)
+
+
+def test_registry_is_published_on_app_state(settings):
+    app = create_app(settings)
+    assert isinstance(app.state.design_packs, DesignPackRegistry)
+
+
+def test_module_registered_pack_reaches_app_state(monkeypatch, settings):
+    real_discover = app_builder.discover_modules
+
+    def _with_pack_module(*args, **kwargs):
+        return [*real_discover(*args, **kwargs), _PackModule()]
+
+    monkeypatch.setattr(app_builder, "discover_modules", _with_pack_module)
+
+    app = create_app(settings)
+
+    assert app.state.design_packs.has("pack-fixture")
+    assert _PACK in app.state.design_packs.all()
+
+
+def test_registry_is_also_reachable_through_services(settings):
+    # ``app.state.sm`` is the aggregate every other registry is published on;
+    # design packs should not be the one exception a module author has to
+    # remember a different lookup for.
+    app = create_app(settings)
+    assert app.state.sm.design_packs is app.state.design_packs