graphddb-runtime 0.7.7__tar.gz → 0.7.8__tar.gz

{graphddb_runtime-0.7.7 → graphddb_runtime-0.7.8}/PKG-INFO

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

{graphddb_runtime-0.7.7 → graphddb_runtime-0.7.8}/graphddb_runtime/transactions.py

@@ -50,11 +50,27 @@ def _resolve_template(
 
 
 def _resolve_value(
-    template: str,
+    template: Any,
     params: Mapping[str, Any],
     element: Optional[Mapping[str, Any]],
 ) -> Any:
-    """Resolve a value template, keeping the typed value for a whole placeholder."""
+    """Resolve a value leaf, preserving its type (issue #245).
+
+    A non-string leaf (an ``int`` / ``bool`` typed literal, from the #245
+    serializer) passes through verbatim, so it lands in DynamoDB as ``N`` /
+    ``BOOL``. A string leaf is a template: a whole placeholder keeps the bound
+    param's type; a composite / plain string resolves to a string.
+
+    Scope note (#245): only integer / boolean literals keep their type in this
+    runtime. A ``float`` literal (e.g. ``rate: 0.5``) passes through here but then
+    fails LOUD at serialize time — boto3's ``TypeSerializer`` raises
+    ``TypeError: Float types are not supported`` (the same pre-existing constraint
+    that applies to ``float`` *params*), so a float is never silently corrupted.
+    Making floats round-trip requires a ``Decimal`` conversion pass — a broader
+    change deliberately OUT of #245 scope (a follow-up).
+    """
+    if not isinstance(template, str):
+        return template
     if _WHOLE_PLACEHOLDER_RE.match(template):
         name = template[1:-1]
         if name.startswith("item."):

{graphddb_runtime-0.7.7 → graphddb_runtime-0.7.8}/graphddb_runtime.egg-info/PKG-INFO

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

{graphddb_runtime-0.7.7 → graphddb_runtime-0.7.8}/pyproject.toml

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

{graphddb_runtime-0.7.7 → graphddb_runtime-0.7.8}/tests/test_unit.py

@@ -634,6 +634,63 @@ def test_transaction_exceeding_25_after_expansion_raises():
         )
 
 
+# ── issue #245: value-position literals keep their type (N / BOOL, not S) ─────
+def test_resolve_value_passes_typed_literal_through():
+    """A non-string leaf (number / boolean typed literal, #245) is returned
+    verbatim, while a string leaf is resolved as a template."""
+    from graphddb_runtime.transactions import _resolve_value
+
+    assert _resolve_value(0, {}, None) == 0
+    assert isinstance(_resolve_value(0, {}, None), int)
+    assert _resolve_value(True, {}, None) is True
+    # A whole-placeholder string still keeps the bound param's type.
+    assert _resolve_value("{v}", {"v": 42}, None) == 42
+    # A composite string resolves to a string.
+    assert _resolve_value("V#{v}", {"v": 42}, None) == "V#42"
+
+
+def test_transaction_put_serializes_numeric_literal_as_N():
+    """End-to-end through ``TransactionExpander._build_item``: a ``version: 0``
+    numeric literal in a put item is serialized as an ``N`` attribute (not ``"0"``
+    as an ``S``), the #245 regression the whole fix targets."""
+    from graphddb_runtime.transactions import TransactionExpander
+
+    rt = make_runtime(FakeClient())
+    # Register a chunk-like modeled entity (mirrors the TS ChunkModel).
+    rt._entities["ChunkModel"] = {
+        "table": "UserPermissions",
+        "physicalName": "UserPermissions",
+        "prefix": "CHUNK",
+        "fields": {
+            "id": {"type": "string"},
+            "version": {"type": "number"},
+            "active": {"type": "boolean"},
+        },
+        "key": {
+            "inputFields": ["id"],
+            "pkTemplate": "CHUNK#{id}",
+            "skTemplate": "META",
+        },
+        "gsis": [],
+        "relations": {},
+    }
+    expander = TransactionExpander(rt)
+    spec = {
+        "type": "Put",
+        "tableName": "UserPermissions",
+        "entity": "ChunkModel",
+        # The #245 serializer emits number / boolean literals verbatim (native JSON).
+        "item": {"id": "{id}", "version": 0, "active": True},
+    }
+    built = expander._build_item(spec, {"id": "c1"}, None)
+    item = built["Put"]["Item"]
+    # DynamoDB attribute-value form: N for the number, BOOL for the boolean, S for id.
+    assert item["version"] == {"N": "0"}
+    assert item["active"] == {"BOOL": True}
+    assert item["id"] == {"S": "c1"}
+    assert item["PK"] == {"S": "CHUNK#c1"}
+
+
 def test_transaction_validates_array_param():
     rt = make_runtime(FakeClient())
     with pytest.raises(ParameterValidationError):