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

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

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

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

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

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

@@ -86,7 +86,7 @@ PROMPT = {
     "response_format": None,
     "variables": [
         {"name": "persona", "type": "string", "default": "a greeter"},
-        {"name": "who", "type": "string", "required": True},
+        {"name": "who", "type": "string"},
     ],
 }
 
@@ -106,36 +106,28 @@ def test_compile_carries_model_and_tools():
     assert out.response_format is None
 
 
-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_variable_empty():
+    # `who` has no value and no default. There is deliberately no way to make
+    # that a failure: the rule is caller value -> default -> empty, and it never
+    # needs a fourth case.
+    out = compile_prompt(PROMPT, {})
+    assert out.messages[1]["content"][0] == {"type": "text", "text": "Hello "}
 
 
-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": [
+def test_compile_ignores_a_legacy_required_flag():
+    # Versions in the wild still carry it. It must stay inert.
+    legacy = {**PROMPT, "variables": [
         {"name": "persona", "type": "string", "default": "a greeter"},
         {"name": "who", "type": "string"},
     ]}
-    out = compile_prompt(optional, {})
+    out = compile_prompt(legacy, {})
     assert out.messages[1]["content"][0] == {"type": "text", "text": "Hello "}
 
 
-def test_compile_lets_a_default_satisfy_required():
+def test_compile_applies_a_default_when_omitted():
     with_default = {**PROMPT, "variables": [
         {"name": "persona", "type": "string", "default": "a greeter"},
-        {"name": "who", "type": "string", "required": True, "default": "world"},
+        {"name": "who", "type": "string", "default": "world"},
     ]}
     out = compile_prompt(with_default, {})
     assert out.messages[1]["content"][0] == {"type": "text", "text": "Hello world"}

{trodo_python-2.19.0 → trodo_python-2.20.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.19.0"
+__version__ = "2.20.0"
 
 from typing import Any, Callable, Dict, List, Optional, Union
 

{trodo_python-2.19.0 → trodo_python-2.20.0}/trodo/managers/prompt_manager.py

@@ -56,10 +56,12 @@ def _to_variables(raw: Any) -> List[PromptVariable]:
             out.append(v)
         elif isinstance(v, dict) and isinstance(v.get("name"), str):
             out.append(
+                # Named fields only — a stored version may carry keys this SDK
+                # no longer models (`required`, retired in 2.20.0), and fetching
+                # one must never raise.
                 PromptVariable(
                     name=v["name"],
                     type=v.get("type") or "string",
-                    required=bool(v.get("required")),
                     default=v.get("default"),
                     description=v.get("description"),
                 )

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

@@ -74,12 +74,10 @@ 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 -> 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.
+    A missing value never raises. The one thing this does reject is a value for
+    a variable the prompt does not declare, which is almost always a typo.
     """
     errors: List[str] = []
     scope: Dict[str, Any] = {}
@@ -104,24 +102,13 @@ def build_scope(
         has = name in values
         value = values.get(name) if has else None
 
-        # `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.
+        # A missing value resolves, it never fails: the declared default, or
+        # empty. That IS the point of declaring a default -- there is no third
+        # case where the caller has to have supplied something, and adding one
+        # would turn an empty render into a crash for no gain.
         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.19.0 → trodo_python-2.20.0}/trodo/prompts/types.py

@@ -26,24 +26,18 @@ class PromptVariable:
     """A declared variable.
 
     Every competitor leaves variables undeclared and untyped — discovered by
-    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.
+    regex, and with no defaults. Knowing the declared set is what lets
+    ``compile()`` catch a value passed for a variable that doesn't exist,
+    instead of silently rendering the wrong prompt.
+
+    A missing value never fails. It resolves to the declared ``default``, or to
+    empty — that is what declaring a default is FOR. There is deliberately no
+    ``required`` flag: it would give the author a decision to get wrong and the
+    caller a crash where an empty render would do.
     """
 
     name: str
     type: str = "string"
-    required: bool = False
     default: Any = None
     description: Optional[str] = None
 

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

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