facetkit 0.1.0__tar.gz → 0.2.0__tar.gz

{facetkit-0.1.0 → facetkit-0.2.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: facetkit
-Version: 0.1.0
+Version: 0.2.0
 Summary: A composable dependency container with passive UI and service facets.
 Author-email: "Daniel R. Vásquez Montes" <dev.DanielR@gmail.composable>
 License: MIT
@@ -143,7 +143,7 @@ app.unmount_facet("cli")
 - **Duplicate keys** — registering the same name again overwrites the previous entry (last wins).
 - **Unmount** — `unmount_facet` clears that facet's registries. Attached components are not automatically detached; call `remove_component` first if you need a full teardown.
 - **Facet names** — the mount name is arbitrary (`app.mount_facet("cli", CliFacet())`). The library does not enforce that the name matches the facet type.
-- **Missing facets** — `app.facets["cli"]` raises `KeyError`; `app.get("facets.cli")` returns `None`.
+- **Missing facets** — `app.facets["cli"]` raises `KeyError`; `app.get("facets.cli")` raises `PathAccessError`; `app.get("facets.cli", default=None)` returns `None`.
 
 ## Introspection with `get()`
 
@@ -158,14 +158,19 @@ app.get("facets.cli.commands")     # command registry
 app.get("facets.web.routes.users") # a single route entry
 ```
 
-Missing paths return `default` (or `None`):
+Without `default`, resolution errors propagate (missing paths, glom failures, etc.):
 
 ```python
-app.get("facets.missing")              # None
-app.get("facets.missing", default={})  # {}
+app.get("facets.cli.commands")     # works when the cli facet is mounted
+app.get("facets.missing")          # raises glom.PathAccessError
 ```
 
-`get()` also returns `default` on any internal error while resolving a path, not only missing keys. That keeps introspection safe but can hide bugs — prefer direct attribute access when you want exceptions to surface.
+Pass `default` for optional lookups — any resolution error returns that value:
+
+```python
+app.get("facets.missing", default=None)  # None
+app.get("facets.missing", default={})    # {}
+```
 
 Direct dict access also works when you know a facet is mounted:
 

{facetkit-0.1.0 → facetkit-0.2.0}/README.md

@@ -117,7 +117,7 @@ app.unmount_facet("cli")
 - **Duplicate keys** — registering the same name again overwrites the previous entry (last wins).
 - **Unmount** — `unmount_facet` clears that facet's registries. Attached components are not automatically detached; call `remove_component` first if you need a full teardown.
 - **Facet names** — the mount name is arbitrary (`app.mount_facet("cli", CliFacet())`). The library does not enforce that the name matches the facet type.
-- **Missing facets** — `app.facets["cli"]` raises `KeyError`; `app.get("facets.cli")` returns `None`.
+- **Missing facets** — `app.facets["cli"]` raises `KeyError`; `app.get("facets.cli")` raises `PathAccessError`; `app.get("facets.cli", default=None)` returns `None`.
 
 ## Introspection with `get()`
 
@@ -132,14 +132,19 @@ app.get("facets.cli.commands")     # command registry
 app.get("facets.web.routes.users") # a single route entry
 ```
 
-Missing paths return `default` (or `None`):
+Without `default`, resolution errors propagate (missing paths, glom failures, etc.):
 
 ```python
-app.get("facets.missing")              # None
-app.get("facets.missing", default={})  # {}
+app.get("facets.cli.commands")     # works when the cli facet is mounted
+app.get("facets.missing")          # raises glom.PathAccessError
 ```
 
-`get()` also returns `default` on any internal error while resolving a path, not only missing keys. That keeps introspection safe but can hide bugs — prefer direct attribute access when you want exceptions to surface.
+Pass `default` for optional lookups — any resolution error returns that value:
+
+```python
+app.get("facets.missing", default=None)  # None
+app.get("facets.missing", default={})    # {}
+```
 
 Direct dict access also works when you know a facet is mounted:
 

{facetkit-0.1.0 → facetkit-0.2.0}/facetkit/__init__.py

@@ -38,4 +38,4 @@ __all__ = [
     "WebFacet",
 ]
 
-__version__ = "0.1.0"
+__version__ = "0.2.0"

{facetkit-0.1.0 → facetkit-0.2.0}/facetkit/container.py

@@ -3,13 +3,15 @@
 
 import glom
 
-from typing import Dict, Any, Optional
-from facetkit.types import Component, Facet
-
-#===============================================================================
-# DEFINITIONS
-
-class Container:
+from typing import Dict, Any
+from facetkit.types import Component, Facet
+
+#===============================================================================
+# DEFINITIONS
+
+_UNSET = object()
+
+class Container:
 
     def __init__(self, config: Dict[str, Any]):
         self.config     : Dict[str, Any]       = config
@@ -18,16 +20,22 @@ class Container:
 
     # Public API ===============================================================
 
-    def get(self, path: str, default: Any = None) -> Any:
-        """Retrieve from container using glom."""
-
-        if not path: return self
-        try:
-            return glom.glom(self.__dict__, path)
-        except (glom.PathAccessError, glom.GlomError):
-            return default
-        except Exception:
-            return default
+    def get(self, path: str, default: Any = _UNSET) -> Any:
+        """Retrieve from container using glom.
+
+        Raises glom errors when *default* is omitted. Pass *default* to
+        return that value instead when the path cannot be resolved.
+        """
+
+        if not path: return self
+        try:
+            return glom.glom(self.__dict__, path)
+        except (glom.PathAccessError, glom.GlomError):
+            if default is _UNSET: raise
+            return default
+        except Exception:
+            if default is _UNSET: raise
+            return default
 
     def mount_facet(self, name: str, facet: Facet) -> None:
         if name in self.facets: self.unmount_facet(name)

{facetkit-0.1.0 → facetkit-0.2.0}/tests/test_container.py

@@ -44,8 +44,28 @@ class TestContainerGet:
     def test_missing_path_returns_default(self, container):
         assert container.get("config.missing", default="fallback") == "fallback"
 
-    def test_missing_path_returns_none_by_default(self, container):
-        assert container.get("config.missing") is None
+    def test_missing_path_returns_none_when_default_is_none(self, container):
+        assert container.get("config.missing", default=None) is None
+
+    def test_missing_path_raises_without_default(self, container):
+        with pytest.raises(glom.PathAccessError):
+            container.get("config.missing")
+
+    def test_glom_error_raises_without_default(self, container, monkeypatch):
+        def raise_glom_error(*_args, **_kwargs):
+            raise glom.GlomError("boom")
+
+        monkeypatch.setattr(container_module.glom, "glom", raise_glom_error)
+        with pytest.raises(glom.GlomError):
+            container.get("config.app.name")
+
+    def test_unexpected_exception_raises_without_default(self, container, monkeypatch):
+        def raise_runtime_error(*_args, **_kwargs):
+            raise RuntimeError("unexpected")
+
+        monkeypatch.setattr(container_module.glom, "glom", raise_runtime_error)
+        with pytest.raises(RuntimeError):
+            container.get("config.app.name")
 
     def test_glom_error_returns_default(self, container, monkeypatch):
         def raise_glom_error(*_args, **_kwargs):