brainlessdb 1.0.0__tar.gz → 1.2.0__tar.gz

{brainlessdb-1.0.0 → brainlessdb-1.2.0}/CHANGELOG.md

@@ -3,6 +3,31 @@
 All notable changes to this project will be documented in this file.
 
 
+## [1.2.0] - 2026-11-15
+
+### 🚀 Features
+
+- Plain __allowed_apps__ list grants access to a collection
+   - A class without allows_app() but with __allowed_apps__ (a list of app
+namespaces) is allowed when the namespace is in the list. ucs_data
+buckets declare only __allowed_apps__ and were all rejected with
+AppAccessError. allows_app() still takes precedence. 
+
+
+## [1.1.0] - 2026-11-03
+
+### 🚀 Features
+
+- Ingest() reports unknown keys
+   - Top level keys which are not fields of the struct were silently left out,
+so a typo or a renamed field went unnoticed. They are still left out, but
+now reported as Repair with action "ignored". 
+
+### ⚙️ Miscellaneous Tasks
+
+- Update uv.lock to 1.0.0
+
+
 ## [1.0.0] - 2026-11-03
 
 ### 🚀 Features

{brainlessdb-1.0.0 → brainlessdb-1.2.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: brainlessdb
-Version: 1.0.0
+Version: 1.2.0
 Summary: Typed collections backed by NATS JetStream KV
 Author-email: "INSOFT s.r.o." <helpdesk@insoft.cz>
 License-Expression: MIT

{brainlessdb-1.0.0 → brainlessdb-1.2.0}/brainlessdb/__init__.py

@@ -1,6 +1,6 @@
 """Brainless DB - Typed collections backed by NATS JetStream KV."""
 
-__version__ = "1.0.0"
+__version__ = "1.2.0"
 
 from typing import Any, Optional, TypeVar
 

{brainlessdb-1.0.0 → brainlessdb-1.2.0}/brainlessdb/client.py

@@ -87,11 +87,16 @@ class BrainlessDB:
         """Validate __allowed_apps__. Errors are deferred to start() before
         start, raised immediately after."""
         allows_app = getattr(cls, "allows_app", None)
+        allowed_apps = getattr(cls, "__allowed_apps__", None)
         error = None
-        if allows_app is None:
+        if allows_app is not None:
+            if not allows_app(self._namespace):
+                error = AppAccessError(cls.__name__, self._namespace)
+        elif allowed_apps is not None:
+            if self._namespace not in allowed_apps:
+                error = AppAccessError(cls.__name__, self._namespace)
+        else:
             error = AppAccessError(cls.__name__)
-        elif not allows_app(self._namespace):
-            error = AppAccessError(cls.__name__, self._namespace)
 
         if error is not None:
             if self._started:

{brainlessdb-1.0.0 → brainlessdb-1.2.0}/brainlessdb/ingest.py

@@ -13,9 +13,10 @@ _MISSING = object()
 class Repair(Struct, frozen=True):
     """One value ingest() could not store as given.
 
-    action is "kept" (current value kept), "default" (field default used) or
-    "removed" (list item or dict entry left out). The invalid value itself is
-    not recorded, it may be a password or key.
+    action is "kept" (current value kept), "default" (field default used),
+    "removed" (list item or dict entry left out) or "ignored" (top level key
+    is not a field of the struct). The invalid value itself is not recorded,
+    it may be a password or key.
     """
 
     path: Path
@@ -201,15 +202,21 @@ def repair(
     current is the stored item as builtins: invalid values fall back to it, except
     inside lists and dicts. Values are coerced like msgspec.convert(strict=False),
     e.g. "5060" to 5060. Raises msgspec.ValidationError when data can't be repaired,
-    e.g. a required field without current value is invalid.
+    e.g. a required field without current value is invalid. Top level keys
+    which are not fields of cls are left out and reported as "ignored".
     """
-    repairs: list[Repair] = []
+    fields = {field.encode_name: field for field in msgspec.structs.fields(cls)}
+    tag_field = cls.__struct_config__.tag_field
+    repairs = [
+        Repair((key,), "ignored", f"Object has unknown field `{key}`")
+        for key in data
+        if key not in fields and key != tag_field
+    ]
     try:
         fixed = _repair_struct(data, cls, current, (), repairs, partial=partial)
     except _Unrepairable as ex:
         raise msgspec.ValidationError(ex.error) from None
 
-    fields = {field.encode_name: field for field in msgspec.structs.fields(cls)}
     converted = {
         key: msgspec.to_builtins(msgspec.convert(value, fields[key].type, strict=False))
         for key, value in fixed.items()

{brainlessdb-1.0.0 → brainlessdb-1.2.0}/brainlessdb/struct.py

@@ -172,11 +172,12 @@ class BrainlessBucket(Struct, kw_only=True, tag=True, tag_field="TYPE"):
         """Like update(), but repair invalid values instead of raising.
 
         An invalid value keeps the current one; inside lists and dicts the invalid
-        item is removed (items are not matched by position). Values are coerced,
-        e.g. "5060" to 5060. Returns the repairs made. Raises like update() when
-        data can't be repaired or violates a unique constraint.
+        item is removed (items are not matched by position). Unknown keys are
+        ignored and reported. Values are coerced, e.g. "5060" to 5060. Returns
+        the repairs made. Raises like update() when data can't be repaired or
+        violates a unique constraint.
         """
-        fixed, repairs = repair(_merge_nested(self, data), type(self), to_builtins(self), partial=True)
+        fixed, repairs = repair({**data, **_merge_nested(self, data)}, type(self), to_builtins(self), partial=True)
         self.update(fixed)
         return repairs
 
@@ -227,11 +228,11 @@ class BrainlessRegistry(Struct, kw_only=True, tag=True, tag_field="TYPE", dict=T
     def ingest(self, data: dict) -> list[Repair]:
         """Set fields from data and save, repairing invalid values instead of raising.
 
-        An invalid value keeps the current one. Values are coerced, e.g. "5060"
-        to 5060. Returns the repairs made.
+        An invalid value keeps the current one, unknown keys are ignored and
+        reported. Values are coerced, e.g. "5060" to 5060. Returns the repairs made.
         """
         current = to_builtins(self)
-        fixed, repairs = repair(_merge_nested(self, data), type(self), current, partial=True)
+        fixed, repairs = repair({**data, **_merge_nested(self, data)}, type(self), current, partial=True)
         new = convert({**current, **fixed}, type(self))
         for f in self.__struct_fields__:
             setattr(self, f, getattr(new, f))

{brainlessdb-1.0.0 → brainlessdb-1.2.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "brainlessdb"
-version = "1.0.0"
+version = "1.2.0"
 description = "Typed collections backed by NATS JetStream KV"
 authors = [
     {name = "INSOFT s.r.o.", email = "helpdesk@insoft.cz"}

{brainlessdb-1.0.0 → brainlessdb-1.2.0}/tests/test_client.py

@@ -123,6 +123,28 @@ class TestAppAccess:
         with pytest.raises(RuntimeError, match="must declare __allowed_apps__"):
             await db.start()
 
+    def test_collection_allowed_apps_attribute(self):
+        """__allowed_apps__ without allows_app - namespace in the list is allowed."""
+        from brainlessdb import BrainlessBucket
+
+        class ListedV1(BrainlessBucket):
+            __allowed_apps__ = ["test", "other"]
+
+        db = BrainlessDB(namespace="test")
+        db.collection(ListedV1)
+        assert db._app_errors == []
+
+    def test_collection_allowed_apps_attribute_wrong_app(self):
+        """__allowed_apps__ without allows_app - namespace not in the list is an error."""
+        from brainlessdb import BrainlessBucket
+
+        class ListedV1(BrainlessBucket):
+            __allowed_apps__ = ["other"]
+
+        db = BrainlessDB(namespace="test")
+        db.collection(ListedV1)
+        assert len(db._app_errors) == 1
+
     @pytest.mark.asyncio
     async def test_collection_after_start_wrong_app_raises_immediately(self):
         """Wrong namespace after start — raises immediately."""

{brainlessdb-1.0.0 → brainlessdb-1.2.0}/tests/test_ingest.py

@@ -60,6 +60,12 @@ class TestCollectionIngestAdd:
         assert item.vlan == 10
         assert phones.find(id=1) is item
 
+    @pytest.mark.asyncio
+    async def test_reports_unknown_keys(self, phones):
+        item, repairs = phones.ingest({"id": 1, "name": "a", "unknown": 1}, key="id")
+        assert item.name == "a"
+        assert actions(repairs) == {("unknown",): "ignored"}
+
     @pytest.mark.asyncio
     async def test_invalid_field_gets_default(self, phones):
         item, repairs = phones.ingest({"id": 1, "name": "a", "mac": "nope", "vlan": 5000}, key="id")
@@ -147,9 +153,10 @@ class TestItemIngest:
         assert actions(repairs) == {("vlan",): "kept"}
 
     @pytest.mark.asyncio
-    async def test_ignores_unknown_keys(self, phones):
+    async def test_reports_unknown_keys(self, phones):
         item, _ = phones.ingest({"id": 1, "name": "a"}, key="id")
-        assert item.ingest({"unknown": 1}) == []
+        assert actions(item.ingest({"unknown": 1})) == {("unknown",): "ignored"}
+        assert phones.find(id=1).name == "a"
 
 
 class TestRegistryIngest:
@@ -164,6 +171,17 @@ class TestRegistryIngest:
         assert actions(repairs) == {("host",): "kept"}
         await db.stop()
 
+    @pytest.mark.asyncio
+    async def test_reports_unknown_keys(self):
+        db = BrainlessDB(namespace="test")
+        sip = db.registry(SipConfigV1)
+        await db.start()
+
+        repairs = sip.ingest({"hots": "10.0.0.1", "port": 5070})
+        assert (sip.host, sip.port) == ("127.0.0.1", 5070)
+        assert actions(repairs) == {("hots",): "ignored"}
+        await db.stop()
+
     @pytest.mark.asyncio
     async def test_marks_dirty(self):
         db = BrainlessDB(namespace="test")

{brainlessdb-1.0.0 → brainlessdb-1.2.0}/uv.lock

@@ -17,7 +17,7 @@ wheels = [
 
 [[package]]
 name = "brainlessdb"
-version = "0.17.4"
+version = "1.0.0"
 source = { editable = "." }
 dependencies = [
     { name = "msgspec" },