payi 0.1.0a62__tar.gz → 0.1.0a64__tar.gz

payi-0.1.0a64/.release-please-manifest.json

@@ -0,0 +1,3 @@
+{
+  ".": "0.1.0-alpha.64"
+}

{payi-0.1.0a62 → payi-0.1.0a64}/CHANGELOG.md

@@ -1,5 +1,21 @@
 # Changelog
 
+## 0.1.0-alpha.64 (2025-04-09)
+
+Full Changelog: [v0.1.0-alpha.63...v0.1.0-alpha.64](https://github.com/Pay-i/pay-i-python/compare/v0.1.0-alpha.63...v0.1.0-alpha.64)
+
+### Chores
+
+* **internal:** slight transform perf improvement ([#283](https://github.com/Pay-i/pay-i-python/issues/283)) ([4b3f9e6](https://github.com/Pay-i/pay-i-python/commit/4b3f9e60fbecf99f8ddffed5903f916f3d5445c5))
+
+## 0.1.0-alpha.63 (2025-04-08)
+
+Full Changelog: [v0.1.0-alpha.62...v0.1.0-alpha.63](https://github.com/Pay-i/pay-i-python/compare/v0.1.0-alpha.62...v0.1.0-alpha.63)
+
+### Features
+
+* feat: simplify global instrumentation init ([a3f7f4f](https://github.com/Pay-i/pay-i-python/commit/a3f7f4f22eacb694b648a2da935fd9412c40740e))
+
 ## 0.1.0-alpha.62 (2025-04-07)
 
 Full Changelog: [v0.1.0-alpha.61...v0.1.0-alpha.62](https://github.com/Pay-i/pay-i-python/compare/v0.1.0-alpha.61...v0.1.0-alpha.62)

{payi-0.1.0a62 → payi-0.1.0a64}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: payi
-Version: 0.1.0a62
+Version: 0.1.0a64
 Summary: The official Python library for the payi API
 Project-URL: Homepage, https://github.com/Pay-i/pay-i-python
 Project-URL: Repository, https://github.com/Pay-i/pay-i-python

{payi-0.1.0a62 → payi-0.1.0a64}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "payi"
-version = "0.1.0-alpha.62"
+version = "0.1.0-alpha.64"
 description = "The official Python library for the payi API"
 dynamic = ["readme"]
 license = "Apache-2.0"

{payi-0.1.0a62 → payi-0.1.0a64}/src/payi/_utils/_transform.py

@@ -142,6 +142,10 @@ def _maybe_transform_key(key: str, type_: type) -> str:
     return key
 
 
+def _no_transform_needed(annotation: type) -> bool:
+    return annotation == float or annotation == int
+
+
 def _transform_recursive(
     data: object,
     *,
@@ -184,6 +188,15 @@ def _transform_recursive(
             return cast(object, data)
 
         inner_type = extract_type_arg(stripped_type, 0)
+        if _no_transform_needed(inner_type):
+            # for some types there is no need to transform anything, so we can get a small
+            # perf boost from skipping that work.
+            #
+            # but we still need to convert to a list to ensure the data is json-serializable
+            if is_list(data):
+                return data
+            return list(data)
+
         return [_transform_recursive(d, annotation=annotation, inner_type=inner_type) for d in data]
 
     if is_union_type(stripped_type):
@@ -332,6 +345,15 @@ async def _async_transform_recursive(
             return cast(object, data)
 
         inner_type = extract_type_arg(stripped_type, 0)
+        if _no_transform_needed(inner_type):
+            # for some types there is no need to transform anything, so we can get a small
+            # perf boost from skipping that work.
+            #
+            # but we still need to convert to a list to ensure the data is json-serializable
+            if is_list(data):
+                return data
+            return list(data)
+
         return [await _async_transform_recursive(d, annotation=annotation, inner_type=inner_type) for d in data]
 
     if is_union_type(stripped_type):

{payi-0.1.0a62 → payi-0.1.0a64}/src/payi/_version.py

@@ -1,4 +1,4 @@
 # File generated from our OpenAPI spec by Stainless. See CONTRIBUTING.md for details.
 
 __title__ = "payi"
-__version__ = "0.1.0-alpha.62"  # x-release-please-version
+__version__ = "0.1.0-alpha.64"  # x-release-please-version

{payi-0.1.0a62 → payi-0.1.0a64}/src/payi/lib/helpers.py

@@ -49,6 +49,7 @@ def create_headers(
     use_case_id: Union[str, None] = None,
     use_case_name: Union[str, None] = None,
     use_case_version: Union[int, None] = None,
+    route_as_resource: Union[str, None] = None,
 ) -> Dict[str, str]:
     headers: Dict[str, str] = {}
 
@@ -68,6 +69,8 @@ def create_headers(
         headers.update({ PayiHeaderNames.use_case_name: use_case_name})
     if use_case_version:
         headers.update({ PayiHeaderNames.use_case_version: str(use_case_version)})
+    if route_as_resource:
+        headers.update({ PayiHeaderNames.route_as_resource: route_as_resource})
     return headers
 
 def _resolve_payi_base_url(payi_base_url: Union[str, None]) -> str:

{payi-0.1.0a62 → payi-0.1.0a64}/src/payi/lib/instrument.py

@@ -23,8 +23,8 @@ from .Stopwatch import Stopwatch
 
 class PayiInstrumentConfig(TypedDict, total=False):
     proxy: bool
+    global_instrumentation_enabled: bool
     limit_ids: Optional["list[str]"]
-    request_tags: Optional["list[str]"]
     experience_name: Optional[str]
     experience_id: Optional[str]
     use_case_name: Optional[str]
@@ -40,18 +40,8 @@ class _Context(TypedDict, total=False):
     use_case_id: Optional[str]
     use_case_version: Optional[int]
     limit_ids: Optional['list[str]']
-    request_tags: Optional['list[str]']
     user_id: Optional[str]
 
-class _ParentState(TypedDict, total=False):
-    experience_name: Optional[str]
-    experience_id: Optional[str]
-    use_case_name: Optional[str]
-    use_case_id: Optional[str]
-    use_case_version: Optional[int]
-    limit_ids: Optional['list[str]']
-    request_tags: Optional['list[str]']
-
 class _IsStreaming(Enum):
     false = 0
     true = 1 
@@ -69,14 +59,11 @@ class _PayiInstrumentor:
         prompt_and_response_logger: Optional[
             Callable[[str, "dict[str, str]"], None]
         ] = None,  # (request id, dict of data to store) -> None
+        global_config: Optional[PayiInstrumentConfig] = None,
     ):
         self._payi: Optional[Payi] = payi
         self._apayi: Optional[AsyncPayi] = apayi
 
-        if not self._payi and not self._apayi:
-            self._payi = Payi()
-            self._apayi = AsyncPayi()
-
         self._context_stack: list[_Context] = []  # Stack of context dictionaries
         self._log_prompt_and_response: bool = log_prompt_and_response
         self._prompt_and_response_logger: Optional[Callable[[str, dict[str, str]], None]] = prompt_and_response_logger
@@ -89,6 +76,24 @@ class _PayiInstrumentor:
         else:
             self._instrument_specific(instruments)
 
+        global_instrumentation_enabled = global_config.pop("global_instrumentation_enabled", True) if global_config else True
+
+        if global_instrumentation_enabled:
+            if global_config is None:
+                global_config = {}
+            if "proxy" not in global_config:
+                global_config["proxy"] = False
+
+            # Use default clients if not provided for global ingest instrumentation
+            if not self._payi and not self._apayi and global_config.get("proxy") == False:
+                self._payi = Payi()
+                self._apayi = AsyncPayi()
+
+            context: _Context = {}
+            self._context_stack.append(context)            
+            # init_context will update the currrent context stack location
+            self._init_context(context=context, parentContext={}, **global_config) # type: ignore
+
     def _instrument_all(self) -> None:
         self._instrument_openai()
         self._instrument_anthropic()
@@ -249,90 +254,99 @@ class _PayiInstrumentor:
 
     def _setup_call_func(
         self
-        ) -> 'tuple[_Context, _ParentState]':
+        ) -> 'tuple[_Context, _Context]':
         context: _Context = {}
-        parentState: _ParentState = {}
+        parentContext: _Context = {}
 
         if len(self._context_stack) > 0:
             # copy current context into the upcoming context
             context = self._context_stack[-1].copy()
             context.pop("proxy")
-            parentState["experience_name"] = context.get("experience_name", None)
-            parentState["experience_id"] = context.get("experience_id", None)
-            parentState["use_case_name"] = context.get("use_case_name", None)
-            parentState["use_case_id"] = context.get("use_case_id", None)
-            parentState["use_case_version"] = context.get("use_case_version", None)
-            parentState["limit_ids"] = context.get("limit_ids", None)
-            parentState["request_tags"] = context.get("request_tags", None)
+            parentContext = {**context}
 
-        return (context, parentState)
+        return (context, parentContext)
 
     def _init_context(
         self,
         context: _Context,
-        parentState: _ParentState,
+        parentContext: _Context,
         proxy: bool,
-        limit_ids: Optional["list[str]"],
-        request_tags: Optional["list[str]"],
-        experience_name: Optional[str],
-        experience_id: Optional[str],
-        use_case_name: Optional[str],
-        use_case_id: Optional[str],
-        use_case_version: Optional[int],                
-        user_id: Optional[str],
+        limit_ids: Optional["list[str]"] = None,
+        experience_name: Optional[str] = None,
+        experience_id: Optional[str] = None,
+        use_case_name: Optional[str]= None,
+        use_case_id: Optional[str]= None,
+        use_case_version: Optional[int]= None,                
+        user_id: Optional[str]= None,
         ) -> None:
         context["proxy"] = proxy
 
-        # TODO use case what if caller specified epxerience / use_case ID and no name?
+        parent_experience_name = parentContext.get("experience_name", None)
+        parent_experience_id = parentContext.get("experience_id", None)
 
-        # Handle experience name and ID logic
-        if not experience_name:
+        if experience_name is None:
             # If no experience_name specified, use previous values
-            context["experience_name"] = parentState.get("experience_name", None)
-            context["experience_id"] = parentState.get("experience_id", None)
+            context["experience_name"] = parent_experience_name
+            context["experience_id"] = parent_experience_id
+        elif len(experience_name) == 0:
+            # Empty string explicitly blocks inheriting from the parent state
+            context["experience_name"] = None
+            context["experience_id"] = None
         else:
-            previous_experience_name = parentState.get("experience_name", None)
-            previous_experience_id = parentState.get("experience_id", None)
-
-            # If experience_name is specified
-            if experience_name == previous_experience_name:
+            # Check if experience_name is the same as the previous one
+            if experience_name == parent_experience_name:
                 # Same experience name, use previous ID unless new one specified
                 context["experience_name"] = experience_name
-                context["experience_id"] = experience_id if experience_id else previous_experience_id
+                context["experience_id"] = experience_id if experience_id else parent_experience_id
             else:
                 # Different experience name, use specified ID or generate one
                 context["experience_name"] = experience_name
                 context["experience_id"] = experience_id if experience_id else str(uuid.uuid4())
 
-        # Handle use case name and ID logic
-        if not use_case_name: # TODO use case
+        parent_use_case_name = parentContext.get("use_case_name", None)
+        parent_use_case_id = parentContext.get("use_case_id", None)
+        parent_use_case_version = parentContext.get("use_case_version", None)
+
+        if use_case_name is None:
             # If no use_case_name specified, use previous values
-            context["use_case_name"] = parentState.get("use_case_name", None)
-            context["use_case_id"] = parentState.get("use_case_id", None)
-            context["use_case_version"] = parentState.get("use_case_version", None)
+            context["use_case_name"] = parent_use_case_name
+            context["use_case_id"] = parent_use_case_id
+            context["use_case_version"] = parent_use_case_version
+        elif len(use_case_name) == 0:
+            # Empty string explicitly blocks inheriting from the parent state
+            context["use_case_name"] = None
+            context["use_case_id"] = None
+            context["use_case_version"] = None
         else:
-            previous_use_case_name = parentState.get("use_case_name", None)
-            previous_use_case_id = parentState.get("use_case_id", None)
-            previous_use_case_version = parentState.get("use_case_version", None)
-
-            # If use_case_name is specified
-            if use_case_name == previous_use_case_name:
+            if use_case_name == parent_use_case_name:
                 # Same use case name, use previous ID unless new one specified
                 context["use_case_name"] = use_case_name
-                context["use_case_id"] = use_case_id if use_case_id else previous_use_case_id
-                context["use_case_version"] = use_case_version if use_case_version else previous_use_case_version
+                context["use_case_id"] = use_case_id if use_case_id else parent_use_case_id
+                context["use_case_version"] = use_case_version if use_case_version else parent_use_case_version
             else:
-                # Different experience name, use specified ID or generate one
+                # Different use case name, use specified ID or generate one
                 context["use_case_name"] = use_case_name
                 context["use_case_id"] = use_case_id if use_case_id else str(uuid.uuid4())
-                context["use_case_version"] = use_case_version
-
-        # set any values explicitly passed by the caller, otherwise use what is already in the context
-        if limit_ids:
-            context["limit_ids"] = limit_ids
-        if request_tags:
-            context["request_tags"] = request_tags
-        if user_id:
+                context["use_case_version"] = use_case_version if use_case_version else None
+
+        parent_limit_ids = parentContext.get("limit_ids", None)
+        if limit_ids is None:
+            # use the parent limit_ids if it exists
+            context["limit_ids"] = parent_limit_ids
+        elif len(limit_ids) == 0:
+            # caller passing an empty array explicitly blocks inheriting from the parent state
+            context["limit_ids"] = None
+        else:
+            # union of new and parent lists if the parent context contains limit ids
+            context["limit_ids"] = list(set(limit_ids) | set(parent_limit_ids)) if parent_limit_ids else limit_ids
+
+        if user_id is None:
+            # use the parent user_id if it exists
+            context["user_id"] = parentContext.get("user_id", None)
+        elif len(user_id) == 0:
+            # caller passing an empty string explicitly blocks inheriting from the parent state
+            context["user_id"] = None
+        else:
             context["user_id"] = user_id
 
         self.set_context(context)
@@ -342,7 +356,6 @@ class _PayiInstrumentor:
         func: Any,
         proxy: bool,
         limit_ids: Optional["list[str]"],
-        request_tags: Optional["list[str]"],
         experience_name: Optional[str],
         experience_id: Optional[str],
         use_case_name: Optional[str],
@@ -352,15 +365,14 @@ class _PayiInstrumentor:
         *args: Any,
         **kwargs: Any,
     ) -> Any:
-        context, parentState = self._setup_call_func()
+        context, parentContext = self._setup_call_func()
 
         with self:
             self._init_context(
                 context,
-                parentState,
+                parentContext,
                 proxy, 
                 limit_ids, 
-                request_tags,
                 experience_name,
                 experience_id,
                 use_case_name,
@@ -374,7 +386,6 @@ class _PayiInstrumentor:
         func: Any,
         proxy: bool,
         limit_ids: Optional["list[str]"],
-        request_tags: Optional["list[str]"],
         experience_name: Optional[str],
         experience_id: Optional[str],
         use_case_name: Optional[str],
@@ -384,15 +395,14 @@ class _PayiInstrumentor:
         *args: Any,
         **kwargs: Any,
     ) -> Any:
-        context, parentState = self._setup_call_func()
+        context, parentContext = self._setup_call_func()
 
         with self:
             self._init_context(
                 context,
-                parentState,
+                parentContext,
                 proxy, 
                 limit_ids, 
-                request_tags,
                 experience_name,
                 experience_id,
                 use_case_name,
@@ -493,7 +503,7 @@ class _PayiInstrumentor:
 
         # after _udpate_headers, all metadata to add to ingest is in extra_headers, keyed by the xproxy-xxx header name
         extra_headers = kwargs.get("extra_headers", {})
-        self._update_headers(context, extra_headers)
+        self._update_extra_headers(context, extra_headers)
 
         if context.get("proxy", True):
             if "extra_headers" not in kwargs:
@@ -617,7 +627,7 @@ class _PayiInstrumentor:
 
         # after _udpate_headers, all metadata to add to ingest is in extra_headers, keyed by the xproxy-xxx header name
         extra_headers = kwargs.get("extra_headers", {})
-        self._update_headers(context, extra_headers)
+        self._update_extra_headers(context, extra_headers)
 
         if context.get("proxy", True):
             if "extra_headers" not in kwargs:
@@ -728,88 +738,74 @@ class _PayiInstrumentor:
         return response
 
     @staticmethod
-    def _update_headers(
+    def _update_extra_headers(
         context: _Context,
         extra_headers: "dict[str, str]",
     ) -> None:
-        limit_ids: Optional[list[str]] = context.get("limit_ids")
-        request_tags: Optional[list[str]] = context.get("request_tags")
-        experience_name: Optional[str] = context.get("experience_name")
-        experience_id: Optional[str] = context.get("experience_id")
-        use_case_name: Optional[str] = context.get("use_case_name")
-        use_case_id: Optional[str] = context.get("use_case_id")
-        use_case_version: Optional[int] = context.get("use_case_version")
-        user_id: Optional[str] = context.get("user_id")
-
-        # Merge limits from the decorator and extra headers
-        if limit_ids is not None:
-            existing_limit_ids = extra_headers.get(PayiHeaderNames.limit_ids, None)
-            
-            if not existing_limit_ids:
-                extra_headers[PayiHeaderNames.limit_ids] = ",".join(limit_ids)
+        context_limit_ids: Optional[list[str]] = context.get("limit_ids")
+        context_experience_name: Optional[str] = context.get("experience_name")
+        context_experience_id: Optional[str] = context.get("experience_id")
+        context_use_case_name: Optional[str] = context.get("use_case_name")
+        context_use_case_id: Optional[str] = context.get("use_case_id")
+        context_use_case_version: Optional[int] = context.get("use_case_version")
+        context_user_id: Optional[str] = context.get("user_id")
+
+        # headers_limit_ids = extra_headers.get(PayiHeaderNames.limit_ids, None)
+    
+        # If the caller specifies limit_ids in extra_headers, it takes precedence over the decorator
+        if PayiHeaderNames.limit_ids in extra_headers:
+            headers_limit_ids = extra_headers.get(PayiHeaderNames.limit_ids)
+
+            if headers_limit_ids is None or len(headers_limit_ids) == 0:
+                # headers_limit_ids is empty, remove it from extra_headers
+                extra_headers.pop(PayiHeaderNames.limit_ids, None)
+            else:   
+                # leave the value in extra_headers
+                ...
+        elif context_limit_ids:
+            extra_headers[PayiHeaderNames.limit_ids] = ",".join(context_limit_ids)
+
+        if PayiHeaderNames.user_id in extra_headers:
+            headers_user_id = extra_headers.get(PayiHeaderNames.user_id, None)
+            if headers_user_id is None or len(headers_user_id) == 0:
+                # headers_user_id is empty, remove it from extra_headers
+                extra_headers.pop(PayiHeaderNames.user_id, None)
             else:
-                existing_ids = existing_limit_ids.split(',')
-                combined_ids = list(set(existing_ids + limit_ids))
-                extra_headers[PayiHeaderNames.limit_ids] = ",".join(combined_ids)
-
-        # Merge request from the decorator and extra headers
-        if request_tags is not None:
-            existing_request_tags = extra_headers.get(PayiHeaderNames.request_tags, None)
-
-            if not existing_request_tags:
-                extra_headers[PayiHeaderNames.request_tags] = ",".join(request_tags)
+                # leave the value in extra_headers
+                ...
+        elif context_user_id:
+            extra_headers[PayiHeaderNames.user_id] = context_user_id
+
+        if PayiHeaderNames.use_case_name in extra_headers:
+            headers_use_case_name = extra_headers.get(PayiHeaderNames.use_case_name, None)
+            if headers_use_case_name is None or len(headers_use_case_name) == 0:
+                # headers_use_case_name is empty, remove all use case related headers
+                extra_headers.pop(PayiHeaderNames.use_case_name, None)
+                extra_headers.pop(PayiHeaderNames.use_case_id, None)
+                extra_headers.pop(PayiHeaderNames.use_case_version, None)
             else:
-                existing_tags = existing_request_tags.split(',')
-                combined_tags = list(set(existing_tags + request_tags))
-                extra_headers[PayiHeaderNames.request_tags] = ",".join(combined_tags)
-
-        # inner extra_headers user_id takes precedence over outer decorator user_id
-        if user_id is not None and extra_headers.get(PayiHeaderNames.user_id, None) is None:
-            extra_headers[PayiHeaderNames.user_id] = user_id   
-
-        # inner extra_headers experience_name and experience_id take precedence over outer decorator experience_name and experience_id
-        # if either inner value is specified, ignore outer decorator values
-        if PayiHeaderNames.experience_name not in extra_headers and PayiHeaderNames.experience_id not in extra_headers:
-
-            # use both decorator values
-            if experience_name is not None:
-                extra_headers[PayiHeaderNames.experience_name] = experience_name
-            if experience_id is not None:
-                extra_headers[PayiHeaderNames.experience_id] = experience_id
-        
-        elif PayiHeaderNames.experience_id in extra_headers and PayiHeaderNames.experience_name not in extra_headers:
-            # use the decorator experience name and the inner experience id
-            if experience_name is not None:
-                extra_headers[PayiHeaderNames.experience_name] = experience_name
-        
-        else:
-            # use the inner experience name and id as-is
-            ...
-
-        # inner extra_headers use_casee_name and use_case_id take precedence over outer decorator use_case_name and use_case_id
-        # if either inner value is specified, ignore outer decorator values
-        if PayiHeaderNames.use_case_name not in extra_headers and PayiHeaderNames.use_case_id not in extra_headers:
-
-            # use decorator values
-            if use_case_name is not None:
-                extra_headers[PayiHeaderNames.use_case_name] = use_case_name
-            if use_case_id is not None:
-                extra_headers[PayiHeaderNames.use_case_id] = use_case_id
-            if use_case_version is not None:
-                extra_headers[PayiHeaderNames.use_case_version] = str(use_case_version)
-        
-        elif PayiHeaderNames.use_case_id in extra_headers and PayiHeaderNames.use_case_name not in extra_headers:
-            # use the decorator experience name and the inner experience id
-            if use_case_name is not None:
-                extra_headers[PayiHeaderNames.use_case_name] = use_case_name
-            
-            # use the decorator experience version and the inner experience id
-            if use_case_version is not None:
-                extra_headers[PayiHeaderNames.use_case_version] = str(use_case_version) # TODO use case 
-        
-        else:
-            # use the inner experience name and id as-is
-            ...
+                # leave the value in extra_headers
+                ...
+        elif context_use_case_name:
+            extra_headers[PayiHeaderNames.use_case_name] = context_use_case_name
+            if context_use_case_id is not None:
+                extra_headers[PayiHeaderNames.use_case_id] = context_use_case_id
+            if context_use_case_version is not None:
+                extra_headers[PayiHeaderNames.use_case_version] = str(context_use_case_version)
+
+        if PayiHeaderNames.experience_name in extra_headers:
+            headers_experience_name = extra_headers.get(PayiHeaderNames.experience_name, None)
+            if headers_experience_name is None or len(headers_experience_name) == 0:
+                # headers_experience_name is empty, remove all experience related headers
+                extra_headers.pop(PayiHeaderNames.experience_name, None)
+                extra_headers.pop(PayiHeaderNames.experience_id, None)
+            else:
+                # leave the value in extra_headers
+                ...
+        elif context_experience_name is not None:
+            extra_headers[PayiHeaderNames.experience_name] = context_experience_name
+            if context_experience_id is not None:
+                extra_headers[PayiHeaderNames.experience_id] = context_experience_id
 
     @staticmethod
     def update_for_vision(input: int, units: 'dict[str, Units]') -> int:
@@ -1020,18 +1016,6 @@ def payi_instrument(
             elif isinstance(p, AsyncPayi): # type: ignore
                 apayi_param = p
     
-    global_context: _Context = {}
-
-    if config and len(config) > 0:
-        if "proxy" not in config:
-            config["proxy"] = False
-        global_context = {**config}
-
-    # create the clients on their behalf if not provided for global ingest instrumentation
-    if not payi_param and not apayi_param and len(global_context) > 0 and global_context.get("proxy", False):
-        payi_param = Payi()
-        apayi_param = AsyncPayi()
-    
     # allow for both payi and apayi to be None for the @proxy case
     _instrumentor = _PayiInstrumentor(
         payi=payi_param,
@@ -1039,19 +1023,11 @@ def payi_instrument(
         instruments=instruments,
         log_prompt_and_response=log_prompt_and_response,
         prompt_and_response_logger=prompt_and_response_logger,
+        global_config=config,
     )
 
-    if len(global_context) > 0:
-        if  "use_case_name" in global_context and not "use_case_id" in global_context:
-            global_context["use_case_id"] = str(uuid.uuid4())
-        if "experience_name" in global_context and not "experience_id" in global_context:
-            global_context["experience_id"] = str(uuid.uuid4())
-
-        _instrumentor._context_stack.append(global_context)
-
 def ingest(
     limit_ids: Optional["list[str]"] = None,
-    request_tags: Optional["list[str]"] = None,
     experience_name: Optional[str] = None,
     experience_id: Optional[str] = None,
     use_case_name: Optional[str] = None,
@@ -1070,7 +1046,6 @@ def ingest(
                     func,
                     False,
                     limit_ids,
-                    request_tags,
                     experience_name,
                     experience_id,
                     use_case_name,
@@ -1089,7 +1064,6 @@ def ingest(
                     func,
                     False,
                     limit_ids,
-                    request_tags,
                     experience_name,
                     experience_id,
                     use_case_name,
@@ -1104,7 +1078,6 @@ def ingest(
 
 def proxy(
     limit_ids: Optional["list[str]"] = None,
-    request_tags: Optional["list[str]"] = None,
     experience_name: Optional[str] = None,
     experience_id: Optional[str] = None,
     use_case_id: Optional[str] = None,
@@ -1122,7 +1095,6 @@ def proxy(
                     func,
                     True,
                     limit_ids,
-                    request_tags,
                     experience_name,
                     experience_id,
                     use_case_name,
@@ -1142,7 +1114,6 @@ def proxy(
                     func,
                     True,
                     limit_ids,
-                    request_tags,
                     experience_name,
                     experience_id,
                     use_case_name,

{payi-0.1.0a62 → payi-0.1.0a64}/tests/test_transform.py

@@ -432,3 +432,15 @@ async def test_base64_file_input(use_async: bool) -> None:
     assert await transform({"foo": io.BytesIO(b"Hello, world!")}, TypedDictBase64Input, use_async) == {
         "foo": "SGVsbG8sIHdvcmxkIQ=="
     }  # type: ignore[comparison-overlap]
+
+
+@parametrize
+@pytest.mark.asyncio
+async def test_transform_skipping(use_async: bool) -> None:
+    # lists of ints are left as-is
+    data = [1, 2, 3]
+    assert await transform(data, List[int], use_async) is data
+
+    # iterables of ints are converted to a list
+    data = iter([1, 2, 3])
+    assert await transform(data, Iterable[int], use_async) == [1, 2, 3]

payi-0.1.0a62/.release-please-manifest.json

@@ -1,3 +0,0 @@
-{
-  ".": "0.1.0-alpha.62"
-}