trodo-python 2.18.2__tar.gz → 2.19.0__tar.gz

{trodo_python-2.18.2 → trodo_python-2.19.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: trodo-python
-Version: 2.18.2
+Version: 2.19.0
 Summary: Trodo Analytics SDK for Python — server-side event tracking
 License: ISC
 Keywords: analytics,tracking,trodo,server-side

{trodo_python-2.18.2 → trodo_python-2.19.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "trodo-python"
-version = "2.18.2"
+version = "2.19.0"
 description = "Trodo Analytics SDK for Python — server-side event tracking"
 readme = "README.md"
 license = { text = "ISC" }

{trodo_python-2.18.2 → trodo_python-2.19.0}/tests/test_prompts.py

@@ -106,12 +106,41 @@ def test_compile_carries_model_and_tools():
     assert out.response_format is None
 
 
-def test_compile_renders_missing_variable_empty():
-    # `who` has no value and no default -> empty, not a raise. Nothing is required.
-    out = compile_prompt(PROMPT, {})
+def test_compile_raises_for_required_variable_with_no_value():
+    # `who` is required with no default. Catching it here is the point: the
+    # alternative is paying for a call to a prompt that says "Hello ".
+    with pytest.raises(CompileError, match="'who': required"):
+        compile_prompt(PROMPT, {})
+
+
+def test_compile_treats_none_and_empty_string_as_no_value():
+    # Both render as nothing, the exact outcome `required` exists to stop.
+    with pytest.raises(CompileError, match="'who': required"):
+        compile_prompt(PROMPT, {"who": None})
+    with pytest.raises(CompileError, match="'who': required"):
+        compile_prompt(PROMPT, {"who": ""})
+
+
+def test_compile_renders_missing_optional_variable_empty():
+    # `required` defaults to False, so a declaration that doesn't ask for it
+    # behaves exactly as before -- no stored prompt changes meaning.
+    optional = {**PROMPT, "variables": [
+        {"name": "persona", "type": "string", "default": "a greeter"},
+        {"name": "who", "type": "string"},
+    ]}
+    out = compile_prompt(optional, {})
     assert out.messages[1]["content"][0] == {"type": "text", "text": "Hello "}
 
 
+def test_compile_lets_a_default_satisfy_required():
+    with_default = {**PROMPT, "variables": [
+        {"name": "persona", "type": "string", "default": "a greeter"},
+        {"name": "who", "type": "string", "required": True, "default": "world"},
+    ]}
+    out = compile_prompt(with_default, {})
+    assert out.messages[1]["content"][0] == {"type": "text", "text": "Hello world"}
+
+
 def test_compile_rejects_undeclared_variable():
     # Passing a variable the prompt doesn't declare is the one variable mistake
     # compile still catches (it's almost always a typo/rename).

{trodo_python-2.18.2 → trodo_python-2.19.0}/trodo/__init__.py

@@ -41,7 +41,7 @@ Downstream microservice (join the caller's run instead of making a new one):
 
 from __future__ import annotations
 
-__version__ = "2.18.0"
+__version__ = "2.19.0"
 
 from typing import Any, Callable, Dict, List, Optional, Union
 

{trodo_python-2.18.2 → trodo_python-2.19.0}/trodo/prompts/compile.py

@@ -74,8 +74,12 @@ def build_scope(
 ) -> Dict[str, Any]:
     """Build the render scope from declarations + caller values.
 
-    Resolution order: caller value → declared default → required-and-absent
-    raises → optional-and-absent renders empty.
+    Resolution order: caller value -> declared default -> required-and-absent
+    raises -> optional-and-absent renders empty.
+
+    `required` is enforced HERE, in your process, before the model call -- Trodo
+    only stores the declaration. A variable with a default can never fail this
+    check, because the default is itself a value.
     """
     errors: List[str] = []
     scope: Dict[str, Any] = {}
@@ -100,11 +104,24 @@ def build_scope(
         has = name in values
         value = values.get(name) if has else None
 
-        # Missing value -> the declared default, or empty. Nothing is
-        # "required": compile always succeeds with whatever the caller provides.
+        # `required` asks a narrower question than default-substitution does:
+        # did the caller supply something that will actually render? None and ""
+        # both render as nothing, the very outcome the flag exists to prevent,
+        # so neither counts. (0 and False do -- they render.)
+        supplied = has and value is not None and value != ""
+
+        # Missing value -> the declared default, or empty.
         if not has or value is None:
             value = default if has_default else ([] if type_ == "messages" else "")
 
+        # A default always satisfies the requirement, so the two never both
+        # apply; the editor won't let you author both, but a hand-written
+        # declaration can.
+        required = v.get("required") if isinstance(v, dict) else getattr(v, "required", None)
+        if not supplied and required is True and not has_default:
+            errors.append(f"variable '{name}': required, but no value was provided")
+            continue
+
         scope[name] = _coerce(value, type_, name, errors)
 
     # Passing something the prompt doesn't declare is nearly always a rename

{trodo_python-2.18.2 → trodo_python-2.19.0}/trodo/prompts/types.py

@@ -29,6 +29,16 @@ class PromptVariable:
     regex, with no defaults and no required-ness. The declaration is what lets
     ``compile()`` fail before the model call rather than shipping a literal
     ``{{typo}}`` and finding out from the bill.
+
+    ``required`` is enforced by ``compile()``, which runs in YOUR process —
+    Trodo only stores the declaration. Nothing server-side rejects a call for a
+    missing variable, and nothing rejects a version for declaring one.
+
+    Marking a variable required means the caller must pass a value that will
+    actually render: an omitted key, ``None`` and ``""`` all fail, since all
+    three render as nothing (``0`` and ``False`` pass). A variable with
+    a ``default`` can never fail the check — the default is a value — so the two
+    are mutually exclusive in practice.
     """
 
     name: str

{trodo_python-2.18.2 → trodo_python-2.19.0}/trodo_python.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: trodo-python
-Version: 2.18.2
+Version: 2.19.0
 Summary: Trodo Analytics SDK for Python — server-side event tracking
 License: ISC
 Keywords: analytics,tracking,trodo,server-side