prefect-client 3.8.8.dev2__tar.gz → 3.8.8.dev4__tar.gz

{prefect_client-3.8.8.dev2 → prefect_client-3.8.8.dev4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: prefect-client
-Version: 3.8.8.dev2
+Version: 3.8.8.dev4
 Summary: Workflow orchestration and management.
 Project-URL: Changelog, https://github.com/PrefectHQ/prefect/releases
 Project-URL: Documentation, https://docs.prefect.io

prefect_client-3.8.8.dev4/src/prefect/_build_info.py

@@ -0,0 +1,5 @@
+# Generated by versioningit
+__version__ = "3.8.8.dev4"
+__build_date__ = "2026-10-06 08:20:50.566787+00:00"
+__git_commit__ = "69b2c7f719ab9aca457ea7059fd159dca7aa1f74"
+__dirty__ = False

{prefect_client-3.8.8.dev2 → prefect_client-3.8.8.dev4}/src/prefect/_internal/concurrency/services.py

@@ -57,6 +57,9 @@ if hasattr(os, "register_at_fork"):
 class _QueueServiceBase(abc.ABC, Generic[T]):
     _instances: dict[int, Self] = {}
     _instance_lock = threading.Lock()
+    # Keep cache access separate from startup serialization: instance() waits for
+    # the global loop, whose service cleanup must be able to evict instances.
+    _instance_cache_lock = threading.Lock()
     _max_queue_size: int = 0  # 0 means unbounded
 
     def __init__(self, *args: Hashable) -> None:
@@ -96,6 +99,7 @@ class _QueueServiceBase(abc.ABC, Generic[T]):
         """Reset class-level state after fork() to prevent deadlocks in child process."""
         cls._instances.clear()
         cls._instance_lock = threading.Lock()
+        cls._instance_cache_lock = threading.Lock()
 
     def start(self) -> None:
         logger.debug("Starting service %r", self)
@@ -281,7 +285,8 @@ class _QueueServiceBase(abc.ABC, Generic[T]):
         """
         futures: list[concurrent.futures.Future[bool]] = []
         with cls._instance_lock:
-            instances = tuple(cls._instances.values())
+            with cls._instance_cache_lock:
+                instances = tuple(cls._instances.values())
 
             for instance in instances:
                 futures.append(instance._drain(at_exit=at_exit))
@@ -312,28 +317,44 @@ class _QueueServiceBase(abc.ABC, Generic[T]):
         """
         with cls._instance_lock:
             key = hash((cls, *args))
-            if key not in cls._instances:
-                cls._instances[key] = cls._new_instance(*args)
+            with cls._instance_cache_lock:
+                instance = cls._instances.get(key)
+            if instance is None or instance._stopped:
+                instance = cls._new_instance(*args)
 
-            return cls._instances[key]
+            return instance
 
     def _remove_instance(self):
-        self._instances.pop(self._key, None)
+        # Only evict this instance; a newer instance may already be cached under
+        # the same key
+        with self._instance_cache_lock:
+            if self._instances.get(self._key) is self:
+                self._instances.pop(self._key, None)
 
     @classmethod
     def _new_instance(cls, *args: Hashable) -> Self:
         """
-        Create and start a new instance of the service.
+        Create, cache, and start a new instance of the service.
         """
         instance = cls(*args)
 
-        # If already on the global loop, just start it here to avoid deadlock
-        if threading.get_ident() == get_global_loop().thread.ident:
-            instance.start()
+        # Cache before starting so that a service which stops while starting, e.g.
+        # because its lifespan failed, evicts itself instead of being cached after
+        # it has already stopped
+        with cls._instance_cache_lock:
+            cls._instances[instance._key] = instance
 
-        # Otherwise, bind the service to the global loop
-        else:
-            from_sync.call_soon_in_loop_thread(create_call(instance.start)).result()
+        try:
+            # If already on the global loop, just start it here to avoid deadlock
+            if threading.get_ident() == get_global_loop().thread.ident:
+                instance.start()
+
+            # Otherwise, bind the service to the global loop
+            else:
+                from_sync.call_soon_in_loop_thread(create_call(instance.start)).result()
+        except BaseException:
+            instance._remove_instance()
+            raise
 
         return instance
 

{prefect_client-3.8.8.dev2 → prefect_client-3.8.8.dev4}/src/prefect/blocks/notifications.py

@@ -751,6 +751,7 @@ class MattermostWebhook(AbstractAppriseNotificationBlock):
             token=self.token.get_secret_value(),
             fullpath=self.path,
             host=self.hostname,
+            user=self.botname,
             targets=self.channels,
             include_image=self.include_image,
             port=self.port,

{prefect_client-3.8.8.dev2 → prefect_client-3.8.8.dev4}/src/prefect/cache_policies.py

@@ -1,7 +1,7 @@
 import inspect
 import sys
 from copy import deepcopy
-from dataclasses import dataclass, field
+from dataclasses import dataclass, field, replace
 from logging import Logger
 from pathlib import Path
 from typing import (
@@ -255,7 +255,7 @@ class CompoundCachePolicy(CachePolicy):
 
         if inputs_policies:
             new = Inputs(exclude=[other])
-            return CompoundCachePolicy(policies=[*self.policies, new])
+            return replace(self, policies=[*self.policies, new])
         else:
             # no dependency on inputs already
             return self
@@ -404,7 +404,7 @@ class Inputs(CachePolicy):
     def __sub__(self, other: str) -> "CachePolicy":
         if not isinstance(other, str):  # type: ignore[reportUnnecessaryIsInstance]
             raise TypeError("Can only subtract strings from key policies.")
-        return Inputs(exclude=self.exclude + [other])
+        return replace(self, exclude=self.exclude + [other])
 
 
 INPUTS = Inputs()

{prefect_client-3.8.8.dev2 → prefect_client-3.8.8.dev4}/src/prefect/client/base.py

@@ -1,4 +1,5 @@
 import copy
+import math
 import sys
 import threading
 import time
@@ -7,6 +8,7 @@ from collections import defaultdict
 from collections.abc import AsyncGenerator, Awaitable, MutableMapping
 from contextlib import asynccontextmanager
 from datetime import datetime, timezone
+from email.utils import parsedate_to_datetime
 from logging import Logger
 from typing import TYPE_CHECKING, Any, Callable, Optional, Protocol, runtime_checkable
 
@@ -177,6 +179,31 @@ class PrefectResponse(httpx.Response):
         return new_response
 
 
+def _parse_retry_after(value: str) -> float | None:
+    """
+    Parse a `Retry-After` header value into a number of seconds to wait.
+
+    Supports both forms allowed by RFC 9110: delay-seconds and HTTP-date. Returns
+    `None` if the value cannot be interpreted so callers can fall back to their
+    default backoff.
+    """
+    try:
+        seconds = float(value)
+    except ValueError:
+        try:
+            retry_at = parsedate_to_datetime(value)
+        except (TypeError, ValueError, OverflowError):
+            return None
+        if retry_at.tzinfo is None:
+            retry_at = retry_at.replace(tzinfo=timezone.utc)
+        seconds = (retry_at - datetime.now(timezone.utc)).total_seconds()
+
+    if not math.isfinite(seconds):
+        return None
+
+    return max(0.0, seconds)
+
+
 class PrefectHttpxAsyncClient(httpx.AsyncClient):
     """
     A Prefect wrapper for the async httpx client with support for retry-after headers
@@ -293,7 +320,9 @@ class PrefectHttpxAsyncClient(httpx.AsyncClient):
                     return response
 
                 if "Retry-After" in response.headers:
-                    retry_seconds = float(response.headers["Retry-After"])
+                    retry_seconds = _parse_retry_after(response.headers["Retry-After"])
+
+            retry_after_from_header = retry_seconds is not None
 
             # Use an exponential back-off if not set in a header
             if retry_seconds is None:
@@ -302,7 +331,7 @@ class PrefectHttpxAsyncClient(httpx.AsyncClient):
             # Add jitter
             jitter_factor = PREFECT_CLIENT_RETRY_JITTER_FACTOR.value()
             if retry_seconds > 0 and jitter_factor > 0:
-                if response is not None and "Retry-After" in response.headers:
+                if retry_after_from_header:
                     # Always wait for _at least_ retry seconds if requested by the API
                     retry_seconds = bounded_poisson_interval(
                         retry_seconds, retry_seconds * (1 + jitter_factor)
@@ -563,7 +592,9 @@ class PrefectHttpxSyncClient(httpx.Client):
                     return response
 
                 if "Retry-After" in response.headers:
-                    retry_seconds = float(response.headers["Retry-After"])
+                    retry_seconds = _parse_retry_after(response.headers["Retry-After"])
+
+            retry_after_from_header = retry_seconds is not None
 
             # Use an exponential back-off if not set in a header
             if retry_seconds is None:
@@ -572,7 +603,7 @@ class PrefectHttpxSyncClient(httpx.Client):
             # Add jitter
             jitter_factor = PREFECT_CLIENT_RETRY_JITTER_FACTOR.value()
             if retry_seconds > 0 and jitter_factor > 0:
-                if response is not None and "Retry-After" in response.headers:
+                if retry_after_from_header:
                     # Always wait for _at least_ retry seconds if requested by the API
                     retry_seconds = bounded_poisson_interval(
                         retry_seconds, retry_seconds * (1 + jitter_factor)

{prefect_client-3.8.8.dev2 → prefect_client-3.8.8.dev4}/src/prefect/runner/runner.py

@@ -241,7 +241,14 @@ class Runner:
 
         if name and ("/" in name or "%" in name):
             raise ValueError("Runner name cannot contain '/' or '%'")
-        self.name: str = Path(name).stem if name is not None else f"runner-{uuid4()}"
+        if name is None:
+            self.name: str = f"runner-{uuid4()}"
+        else:
+            # Only strip the suffix when a filename was actually passed; a `.` is
+            # legal in a runner name and `Path().stem` would otherwise treat
+            # everything after the last one as a file extension. Mirrors
+            # `RunnerDeployment.validate_name`.
+            self.name = Path(name).stem if name.endswith(".py") else name
         self._logger: "logging.Logger" = get_logger("runner")
 
         self.started: bool = False

{prefect_client-3.8.8.dev2 → prefect_client-3.8.8.dev4}/src/prefect/task_engine.py

@@ -444,8 +444,6 @@ class SyncTaskRunEngine(BaseTaskRunEngine[P, R]):
             Callable[["Task[P, Coroutine[Any, Any, R]]", TaskRun, State[R]], bool]
         ] = self.task.retry_condition_fn
 
-        failure_type = "exception" if isinstance(exc_or_state, Exception) else "state"
-
         if not self.task_run:
             raise ValueError("Task run is not set")
         try:
@@ -453,9 +451,13 @@ class SyncTaskRunEngine(BaseTaskRunEngine[P, R]):
                 f"Running `retry_condition_fn` check {retry_condition!r} for task"
                 f" {self.task.name!r}"
             )
-            state = Failed(
-                data=exc_or_state,
-                message=f"Task run encountered unexpected {failure_type}: {repr(exc_or_state)}",
+            state = (
+                exc_or_state
+                if isinstance(exc_or_state, State)
+                else Failed(
+                    data=exc_or_state,
+                    message=f"Task run encountered unexpected exception: {repr(exc_or_state)}",
+                )
             )
             if inspect.iscoroutinefunction(retry_condition):
                 should_retry = run_coro_as_sync(
@@ -674,6 +676,10 @@ class SyncTaskRunEngine(BaseTaskRunEngine[P, R]):
                     if isinstance(self.task.retry_delay_seconds, Sequence)
                     else self.task.retry_delay_seconds
                 )
+                if self.task.retry_jitter_factor and delay > 0:
+                    delay = clamped_poisson_interval(
+                        delay, clamping_factor=self.task.retry_jitter_factor
+                    )
                 new_state = AwaitingRetry(
                     scheduled_time=prefect.types._datetime.now("UTC")
                     + timedelta(seconds=delay)
@@ -1056,8 +1062,6 @@ class AsyncTaskRunEngine(BaseTaskRunEngine[P, R]):
             Callable[["Task[P, Coroutine[Any, Any, R]]", TaskRun, State[R]], bool]
         ] = self.task.retry_condition_fn
 
-        failure_type = "exception" if isinstance(exc_or_state, Exception) else "state"
-
         if not self.task_run:
             raise ValueError("Task run is not set")
         try:
@@ -1065,9 +1069,13 @@ class AsyncTaskRunEngine(BaseTaskRunEngine[P, R]):
                 f"Running `retry_condition_fn` check {retry_condition!r} for task"
                 f" {self.task.name!r}"
             )
-            state = Failed(
-                data=exc_or_state,
-                message=f"Task run encountered unexpected {failure_type}: {repr(exc_or_state)}",
+            state = (
+                exc_or_state
+                if isinstance(exc_or_state, State)
+                else Failed(
+                    data=exc_or_state,
+                    message=f"Task run encountered unexpected exception: {repr(exc_or_state)}",
+                )
             )
             if inspect.iscoroutinefunction(retry_condition):
                 should_retry = await retry_condition(self.task, self.task_run, state)
@@ -1301,6 +1309,10 @@ class AsyncTaskRunEngine(BaseTaskRunEngine[P, R]):
                     if isinstance(self.task.retry_delay_seconds, Sequence)
                     else self.task.retry_delay_seconds
                 )
+                if self.task.retry_jitter_factor and delay > 0:
+                    delay = clamped_poisson_interval(
+                        delay, clamping_factor=self.task.retry_jitter_factor
+                    )
                 new_state = AwaitingRetry(
                     scheduled_time=prefect.types._datetime.now("UTC")
                     + timedelta(seconds=delay)

{prefect_client-3.8.8.dev2 → prefect_client-3.8.8.dev4}/src/prefect/tasks.py

@@ -792,6 +792,7 @@ class Task(Generic[P, R]):
             name=name or self.name,
             description=description or self.description,
             tags=tags or copy(self.tags),
+            version=self.version,
             cache_policy=cache_policy
             if cache_policy is not NotSet
             else self._user_cache_policy,
@@ -844,6 +845,8 @@ class Task(Generic[P, R]):
             on_completion=on_completion or self.on_completion_hooks,
             on_failure=on_failure or self.on_failure_hooks,
             on_running=on_running or self.on_running_hooks,
+            on_rollback=copy(self.on_rollback_hooks),
+            on_commit=copy(self.on_commit_hooks),
             retry_condition_fn=retry_condition_fn or self.retry_condition_fn,
             viz_return_value=viz_return_value or self.viz_return_value,
             asset_deps=asset_deps or self.asset_deps,

{prefect_client-3.8.8.dev2 → prefect_client-3.8.8.dev4}/src/prefect/utilities/schema_tools/validation.py

@@ -251,8 +251,11 @@ def process_properties(
         _fix_tuple_items(schema)
 
         if "properties" in schema:
-            required_fields = schema.get("required", [])
-            process_properties(schema["properties"], required_fields)
+            process_properties(
+                schema["properties"],
+                schema.get("required", []),
+                allow_none_with_default,
+            )
 
 
 def preprocess_schema(

{prefect_client-3.8.8.dev2 → prefect_client-3.8.8.dev4}/src/prefect/utilities/templating/__init__.py

@@ -401,8 +401,15 @@ async def resolve_block_document_references(
         value: Any = data
 
         if len(data) == 1 and "value" in data:
-            # only resolve the value if the keypath is not already pointing to "value"
-            if not (value_keypath and value_keypath[0].startswith("value")):
+            # only resolve the value if the keypath is not already pointing to "value";
+            # a key that merely starts with "value" (e.g. "values") is inside it
+            if not (
+                value_keypath
+                and (
+                    value_keypath[0] == "value"
+                    or value_keypath[0].startswith(("value.", "value["))
+                )
+            ):
                 data = value = value["value"]
 
         if value_keypath:

prefect_client-3.8.8.dev2/src/prefect/_build_info.py

@@ -1,5 +0,0 @@
-# Generated by versioningit
-__version__ = "3.8.8.dev2"
-__build_date__ = "2026-10-01 08:22:06.424502+00:00"
-__git_commit__ = "aa05fef1cff77741202fe9429c951351cbb7c892"
-__dirty__ = False