graphddb-runtime 1.0.0__tar.gz → 1.0.2__tar.gz

{graphddb_runtime-1.0.0 → graphddb_runtime-1.0.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: graphddb-runtime
-Version: 1.0.0
+Version: 1.0.2
 Summary: Thin DynamoDB executor for GraphDDB-generated Python repositories (single-operation core, issue #44).
 License: MIT
 Requires-Python: >=3.9

{graphddb_runtime-1.0.0 → graphddb_runtime-1.0.2}/graphddb_runtime/filters.py

@@ -148,6 +148,18 @@ def _compile_field(ctx: _Ctx, field: str, condition: Any) -> str:
 
 
 def _compile_node(ctx: _Ctx, node: Any) -> str:
+    # Structural primitives (notExists / attributeExists / attributeNotExists) may be
+    # nested inside an and/or/not group, not only at the top level (issue #330). Render
+    # them here; otherwise the loop treats them as a field named "attributeNotExists"
+    # with an equality operand — a never-satisfied clause.
+    if node.get("notExists") is True:
+        return f"attribute_not_exists({ctx.name_alias('PK')})"
+    ae = node.get("attributeExists")
+    if isinstance(ae, str) and ae:
+        return f"attribute_exists({ctx.name_alias(ae)})"
+    ane = node.get("attributeNotExists")
+    if isinstance(ane, str) and ane:
+        return f"attribute_not_exists({ctx.name_alias(ane)})"
     clauses: List[str] = []
     for key, value in node.items():
         if value is None:

{graphddb_runtime-1.0.0 → graphddb_runtime-1.0.2}/graphddb_runtime/runtime.py

@@ -218,6 +218,14 @@ def _resolve_condition_tree(node: Any, params: Mapping[str, Any]) -> Any:
     typed caller-param values, and returns a tree shaped exactly like a
     declarative ``filter`` so :func:`compile_filter` compiles it identically.
     """
+    # Structural primitives (notExists / attributeExists / attributeNotExists) may be
+    # nested in an and/or/not group (issue #330); pass them through verbatim.
+    if node.get("notExists") is True:
+        return {"notExists": True}
+    if isinstance(node.get("attributeExists"), str):
+        return {"attributeExists": node["attributeExists"]}
+    if isinstance(node.get("attributeNotExists"), str):
+        return {"attributeNotExists": node["attributeNotExists"]}
     out: Dict[str, Any] = {}
     for key, value in node.items():
         if key in ("and", "or"):
@@ -2319,6 +2327,31 @@ class GraphDDBRuntime:
                 command_id, spec, params, names, values, sets
             )
 
+            # #333: an UpdateItem that CREATES a row (create-if-absent) must carry the
+            # model's key-source field attributes, like PutItem does. SET each field a
+            # `keyCondition` template references from the command's key params. A
+            # key-source field is immutable for a given key, so a plain SET to the
+            # key-derived value is a safe no-op on existing rows and populates it on
+            # create. Without this, a row first created by an update lacks e.g. `child`,
+            # so a projection of that field returns empty. (Skipped when add_only — the
+            # #301 fallback only fires on rows that already exist.)
+            seen_ks: set = set()
+            for tmpl in spec["keyCondition"].values():
+                for m in _PLACEHOLDER_RE.finditer(str(tmpl)):
+                    field = m.group(0)[1:-1]
+                    if "." in field or field in seen_ks or field not in params:
+                        continue
+                    raw = params[field]
+                    if raw is None:
+                        continue
+                    seen_ks.add(field)
+                    idx = len(seen_ks) - 1
+                    n = f"#ks{idx}"
+                    v = f":ks{idx}"
+                    names[n] = field
+                    values[v] = self._serializer.serialize(raw)
+                    sets.append(f"{n} = {v}")
+
         # #301: atomic-`ADD` clauses (`ADD #a :d`, a caller `param.number()` delta).
         # Built AFTER the SET so an ADD-only expression (`add_only`, or an update
         # whose SET change set is empty) is legal — DynamoDB's `SET … ADD …` is one

{graphddb_runtime-1.0.0 → graphddb_runtime-1.0.2}/graphddb_runtime/transactions.py

@@ -108,6 +108,16 @@ def _resolve_condition_tree(
     Mirrors the runtime's resolver but element-aware, producing a tree shaped
     like a declarative ``filter`` for :func:`compile_filter`.
     """
+    # Structural primitives (notExists / attributeExists / attributeNotExists) carry a
+    # static field name / flag and no param leaf; they may be nested in an and/or/not
+    # group (issue #330). Pass them through verbatim, else the default branch iterates
+    # the string field name as operator entries.
+    if node.get("notExists") is True:
+        return {"notExists": True}
+    if isinstance(node.get("attributeExists"), str):
+        return {"attributeExists": node["attributeExists"]}
+    if isinstance(node.get("attributeNotExists"), str):
+        return {"attributeNotExists": node["attributeNotExists"]}
     out: Dict[str, Any] = {}
     for key, value in node.items():
         if key in ("and", "or"):

{graphddb_runtime-1.0.0 → graphddb_runtime-1.0.2}/graphddb_runtime.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: graphddb-runtime
-Version: 1.0.0
+Version: 1.0.2
 Summary: Thin DynamoDB executor for GraphDDB-generated Python repositories (single-operation core, issue #44).
 License: MIT
 Requires-Python: >=3.9

{graphddb_runtime-1.0.0 → graphddb_runtime-1.0.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "graphddb-runtime"
-version = "1.0.0"
+version = "1.0.2"
 description = "Thin DynamoDB executor for GraphDDB-generated Python repositories (single-operation core, issue #44)."
 readme = "README.md"
 requires-python = ">=3.9"

{graphddb_runtime-1.0.0 → graphddb_runtime-1.0.2}/tests/test_atomic_add_command.py

@@ -168,6 +168,22 @@ def test_mark_fallback_add_only_on_condition_fail():
     assert any(v == {"N": "1"} for v in fvals.values()), fvals
 
 
+def test_mark_stores_key_source_fields():
+    """#333: an UpdateItem that CREATES the row must persist the key-source field
+    attributes (part/child) via a plain SET — like PutItem — so a row first created
+    by the mark carries them and a projection of e.g. ``child`` is not empty."""
+    client = _MarkClient(fail_first=False)
+    rt = _mark_runtime(client)
+
+    rt.execute_command("StatsTreeCmd__mark", _MARK_PARAMS)
+
+    req = client.updates[0]
+    assert _has_set_clause(req["UpdateExpression"]), req["UpdateExpression"]
+    names = set(req["ExpressionAttributeNames"].values())
+    for field in ("part", "child"):
+        assert field in names, f"key-source field {field!r} must be SET (#333); {names}"
+
+
 def test_mark_non_ccf_error_propagates():
     """A NON-condition error on the primary must NOT be swallowed by the two-stage —
     it propagates and no ADD-only fallback fires. Parity with the Rust

{graphddb_runtime-1.0.0 → graphddb_runtime-1.0.2}/tests/test_unit.py

@@ -253,9 +253,12 @@ def test_update_item_builds_set_expression():
     method, req = client.calls[0]
     assert method == "update_item"
     assert plain(req["Key"]) == {"PK": "USER#alice", "SK": "PROFILE"}
-    assert req["UpdateExpression"] == "SET #c0 = :c0"
-    assert req["ExpressionAttributeNames"] == {"#c0": "status"}
-    assert plain(req["ExpressionAttributeValues"]) == {":c0": "disabled"}
+    # #333: the update also SETs the key-source field(s) the PK/SK templates
+    # reference (here `userId`, from `USER#{userId}`) so a row first created by this
+    # update carries them like a PutItem would — a no-op on an existing row.
+    assert req["UpdateExpression"] == "SET #c0 = :c0, #ks0 = :ks0"
+    assert req["ExpressionAttributeNames"] == {"#c0": "status", "#ks0": "userId"}
+    assert plain(req["ExpressionAttributeValues"]) == {":c0": "disabled", ":ks0": "alice"}
 
 
 def test_update_item_rederives_affected_gsi_key():
@@ -303,7 +306,8 @@ def test_update_item_does_not_touch_unaffected_gsi():
     rt.execute_command("UserConfCommands__disableUser", {"userId": "alice", "status": "disabled"})
     _method, req = client.calls[0]
     assert "GSI1PK" not in req["ExpressionAttributeNames"].values()
-    assert req["UpdateExpression"] == "SET #c0 = :c0"
+    # #333: the change SET plus the key-source SET (`userId`); no GSI re-derive.
+    assert req["UpdateExpression"] == "SET #c0 = :c0, #ks0 = :ks0"
 
 
 def test_put_item_injects_key_attributes():