digitalkin 1.0.4.dev0__tar.gz → 1.0.4.dev1__tar.gz

{digitalkin-1.0.4.dev0 → digitalkin-1.0.4.dev1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: digitalkin
-Version: 1.0.4.dev0
+Version: 1.0.4.dev1
 Summary: SDK to build kin used in DigitalKin
 Author-email: "DigitalKin.ai" <contact@digitalkin.ai>
 License: Attribution-NonCommercial-ShareAlike 4.0 International

{digitalkin-1.0.4.dev0 → digitalkin-1.0.4.dev1}/pyproject.toml

@@ -41,7 +41,7 @@
         "pydantic>=2.12.4",
         "redis[hiredis]>=7.4.0,<9",
     ]
-    version = "1.0.4.dev0"
+    version = "1.0.4.dev1"
 
     [project.optional-dependencies]
         agno = [ "agno>=2.6" ]

{digitalkin-1.0.4.dev0 → digitalkin-1.0.4.dev1}/src/digitalkin/__version__.py

@@ -5,4 +5,4 @@ from importlib.metadata import PackageNotFoundError, version
 try:
     __version__ = version("digitalkin")
 except PackageNotFoundError:
-    __version__ = "1.0.4.dev0"
+    __version__ = "1.0.4.dev1"

{digitalkin-1.0.4.dev0 → digitalkin-1.0.4.dev1}/src/digitalkin/services/setup/default_setup.py

@@ -18,6 +18,7 @@ from digitalkin.services.setup.setup_strategy import (
     SetupVersionPage,
 )
 from digitalkin.utils.json_structure import JsonStructure
+from digitalkin.utils.setup_content_validator import SetupContentValidator
 
 
 class DefaultSetup(SetupStrategy):
@@ -101,6 +102,7 @@ class DefaultSetup(SetupStrategy):
         Raises:
             ValueError: If name or content is invalid.
         """
+        SetupContentValidator.reject_oversized_output_format_spec(setup_dict.get("content") or {})
         setup_id = self._new_id()
         try:
             setup = SetupData(
@@ -158,6 +160,7 @@ class DefaultSetup(SetupStrategy):
         if not name or not isinstance(content, dict):
             msg = "setup_id, name and content (object) are required"
             raise ValueError(msg)
+        SetupContentValidator.reject_oversized_output_format_spec(content)
         setup.name = name
         # A new revision rather than an in-place edit, matching UpdateSetup on the wire.
         history = self.versions.setdefault(setup.id, [setup.current_setup_version])

{digitalkin-1.0.4.dev0 → digitalkin-1.0.4.dev1}/src/digitalkin/services/setup/grpc_setup.py

@@ -20,6 +20,7 @@ from digitalkin.services.setup.exceptions import SetupServiceError
 from digitalkin.services.setup.setup_strategy import SetupData, SetupStrategy, SetupVersionData, SetupVersionPage
 from digitalkin.utils.json_structure import JsonStructure
 from digitalkin.utils.proto_utils import ProtoUtils
+from digitalkin.utils.setup_content_validator import SetupContentValidator
 
 
 class GrpcSetup(SetupStrategy, GrpcClientWrapper):
@@ -191,13 +192,16 @@ class GrpcSetup(SetupStrategy, GrpcClientWrapper):
             The created setup with its initial version and structure.
 
         Raises:
-            ValueError: If name or content is missing.
+            ValueError: If name or content is missing, or output_format_spec is oversized.
             ServerError: If gRPC operation fails.
             SetupServiceError: If the server reports failure or an unexpected error occurs.
         """
         if not setup_dict.get("name") or not isinstance(setup_dict.get("content"), dict):
             msg = "name and content (object) are required"
             raise ValueError(msg)
+        # Outside handle_grpc_errors: an input guard must stay a ValueError, not become a
+        # SetupServiceError via the catch-all.
+        SetupContentValidator.reject_oversized_output_format_spec(setup_dict["content"])
         async with self.handle_grpc_errors("Setup Creation"):
             content_struct = Struct()
             content_struct.update(setup_dict["content"])
@@ -235,7 +239,7 @@ class GrpcSetup(SetupStrategy, GrpcClientWrapper):
             The updated setup with its current version and structure.
 
         Raises:
-            ValueError: If setup_id, name or content is missing.
+            ValueError: If setup_id, name or content is missing, or output_format_spec is oversized.
             ServerError: If gRPC operation fails.
             SetupServiceError: If the server reports failure or an unexpected error occurs.
         """
@@ -246,6 +250,7 @@ class GrpcSetup(SetupStrategy, GrpcClientWrapper):
         ):
             msg = "setup_id, name and content (object) are required"
             raise ValueError(msg)
+        SetupContentValidator.reject_oversized_output_format_spec(setup_dict["content"])
         async with self.handle_grpc_errors("Setup Update"):
             content_struct = Struct()
             content_struct.update(setup_dict["content"])

{digitalkin-1.0.4.dev0 → digitalkin-1.0.4.dev1}/src/digitalkin/utils/setup_content_validator.py

@@ -39,6 +39,8 @@ class SetupContentValidator:
     _STRICT_SCALARS: ClassVar[tuple[type, ...]] = (int, float, bool)
     _FIRST_PRINTABLE: ClassVar[int] = 0x20  # code points below this are C0 control characters.
     _LAST_BMP: ClassVar[int] = 0xFFFF  # code points above this are astral (non-BMP, e.g. emoji).
+    _OUTPUT_FORMAT_SPEC_KEY: ClassVar[str] = "output_format_spec"
+    _OUTPUT_FORMAT_SPEC_MAX: ClassVar[int] = 4096  # at or past this the written setup is unusable.
     _NUMERIC_CONSTRAINTS: ClassVar[dict[str, str]] = {
         "minimum": "ge",
         "maximum": "le",
@@ -113,6 +115,45 @@ class SetupContentValidator:
                 stack.extend((path, item) for item in node)
         return content
 
+    @classmethod
+    def reject_oversized_output_format_spec(cls, content: dict[str, Any]) -> dict[str, Any]:
+        """Refuse an ``output_format_spec`` that reaches the length the setup cannot carry.
+
+        The write itself succeeds, so the breakage surfaces later as an unusable setup with no
+        diagnostic pointing back at the field. Reject up front, naming the path and the actual
+        size. Recurses through nested objects and arrays; a non-string value is left to the
+        schema check, which owns typing.
+
+        Args:
+            content: The setup ``content`` about to be written.
+
+        Returns:
+            The content unchanged when every ``output_format_spec`` fits.
+
+        Raises:
+            ValueError: An ``output_format_spec`` string is at or over the limit.
+        """
+        stack: list[tuple[str, Any]] = [("", content)]
+        while stack:
+            path, node = stack.pop()
+            if isinstance(node, dict):
+                for key, value in node.items():
+                    where = f"{path}.{key}" if path else key
+                    if (
+                        key == cls._OUTPUT_FORMAT_SPEC_KEY
+                        and isinstance(value, str)
+                        and len(value) >= cls._OUTPUT_FORMAT_SPEC_MAX
+                    ):
+                        msg = (
+                            f"content field {where!r} is {len(value)} characters; it must stay under "
+                            f"{cls._OUTPUT_FORMAT_SPEC_MAX} or the setup breaks"
+                        )
+                        raise ValueError(msg)
+                    stack.append((where, value))
+            elif isinstance(node, list):
+                stack.extend((path, item) for item in node)
+        return content
+
     @classmethod
     def validate(cls, content: dict[str, Any], schema: dict[str, Any]) -> None:
         """Validate ``content`` against ``schema``.

{digitalkin-1.0.4.dev0 → digitalkin-1.0.4.dev1}/src/digitalkin.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: digitalkin
-Version: 1.0.4.dev0
+Version: 1.0.4.dev1
 Summary: SDK to build kin used in DigitalKin
 Author-email: "DigitalKin.ai" <contact@digitalkin.ai>
 License: Attribution-NonCommercial-ShareAlike 4.0 International