Remove unused rpc_metadata/rpc_timeout from child interceptor inputs

Remove rpc_metadata and rpc_timeout fields from
UpdateWithStartUpdateWorkflowInput and UpdateWithStartStartWorkflowInput.
These fields were never forwarded to the underlying execute_multi_operation
gRPC call — only the top-level StartWorkflowUpdateWithStartInput fields
are authoritative.

Also remove the corresponding parameters from WithStartWorkflowOperation
since they only served to populate the (now-removed) child input fields.

This is a breaking change for interceptors that accessed rpc_metadata or
rpc_timeout on the child input objects.

Author: brucearctor

temporalio/client/_client.py

@@ -1190,8 +1190,6 @@ async def _start_update_with_start(
             args=temporalio.common._arg_or_args(arg, args),
             headers={},
             ret_type=result_type or result_type_from_type_hint,
-            rpc_metadata=rpc_metadata,
-            rpc_timeout=rpc_timeout,
             wait_for_stage=wait_for_stage,
         )
 

temporalio/client/_interceptor.py

@@ -334,8 +334,6 @@ class UpdateWithStartUpdateWorkflowInput:
     wait_for_stage: WorkflowUpdateStage
     headers: Mapping[str, temporalio.api.common.v1.Payload]
     ret_type: type | None
-    rpc_metadata: Mapping[str, str | bytes]
-    rpc_timeout: timedelta | None
 
 
 @dataclass
@@ -366,8 +364,6 @@ class UpdateWithStartStartWorkflowInput:
     static_details: str | None
     # Type may be absent
     ret_type: type | None
-    rpc_metadata: Mapping[str, str | bytes]
-    rpc_timeout: timedelta | None
     priority: temporalio.common.Priority
     versioning_override: temporalio.common.VersioningOverride | None = None
 
@@ -376,12 +372,9 @@ class UpdateWithStartStartWorkflowInput:
 class StartWorkflowUpdateWithStartInput:
     """Input for :py:meth:`OutboundInterceptor.start_update_with_start_workflow`.
 
-    The top-level ``rpc_metadata`` and ``rpc_timeout`` fields are authoritative
-    for the ``execute_multi_operation`` gRPC call.  The sub-inputs
-    (``start_workflow_input`` and ``update_workflow_input``) also carry their own
-    ``rpc_metadata`` / ``rpc_timeout`` for interceptor introspection, but those
-    values are **not** forwarded to the gRPC call.  Interceptors that wish to set
-    RPC metadata should modify :py:attr:`rpc_metadata` on this object.
+    The ``rpc_metadata`` and ``rpc_timeout`` fields are authoritative for the
+    ``execute_multi_operation`` gRPC call.  Interceptors that wish to set RPC
+    metadata should modify :py:attr:`rpc_metadata` on this object.
     """
 
     start_workflow_input: UpdateWithStartStartWorkflowInput

temporalio/client/_workflow.py

@@ -1065,8 +1065,6 @@ def __init__(
         static_summary: str | None = None,
         static_details: str | None = None,
         start_delay: timedelta | None = None,
-        rpc_metadata: Mapping[str, str | bytes] = {},
-        rpc_timeout: timedelta | None = None,
         priority: temporalio.common.Priority = temporalio.common.Priority.default,
         versioning_override: temporalio.common.VersioningOverride | None = None,
     ) -> None: ...
@@ -1095,8 +1093,6 @@ def __init__(
         static_summary: str | None = None,
         static_details: str | None = None,
         start_delay: timedelta | None = None,
-        rpc_metadata: Mapping[str, str | bytes] = {},
-        rpc_timeout: timedelta | None = None,
         priority: temporalio.common.Priority = temporalio.common.Priority.default,
         versioning_override: temporalio.common.VersioningOverride | None = None,
     ) -> None: ...
@@ -1127,8 +1123,6 @@ def __init__(
         static_summary: str | None = None,
         static_details: str | None = None,
         start_delay: timedelta | None = None,
-        rpc_metadata: Mapping[str, str | bytes] = {},
-        rpc_timeout: timedelta | None = None,
         priority: temporalio.common.Priority = temporalio.common.Priority.default,
         versioning_override: temporalio.common.VersioningOverride | None = None,
     ) -> None: ...
@@ -1159,8 +1153,6 @@ def __init__(
         static_summary: str | None = None,
         static_details: str | None = None,
         start_delay: timedelta | None = None,
-        rpc_metadata: Mapping[str, str | bytes] = {},
-        rpc_timeout: timedelta | None = None,
         priority: temporalio.common.Priority = temporalio.common.Priority.default,
         versioning_override: temporalio.common.VersioningOverride | None = None,
     ) -> None: ...
@@ -1189,8 +1181,6 @@ def __init__(
         static_summary: str | None = None,
         static_details: str | None = None,
         start_delay: timedelta | None = None,
-        rpc_metadata: Mapping[str, str | bytes] = {},
-        rpc_timeout: timedelta | None = None,
         priority: temporalio.common.Priority = temporalio.common.Priority.default,
         versioning_override: temporalio.common.VersioningOverride | None = None,
         stack_level: int = 2,
@@ -1228,8 +1218,6 @@ def __init__(
             start_delay=start_delay,
             headers={},
             ret_type=result_type or result_type_from_run_fn,
-            rpc_metadata=rpc_metadata,
-            rpc_timeout=rpc_timeout,
             priority=priority,
             versioning_override=versioning_override,
         )