prefect 3.7.2.dev1__tar.gz → 3.7.2.dev2__tar.gz

{prefect-3.7.2.dev1 → prefect-3.7.2.dev2}/PKG-INFO

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

{prefect-3.7.2.dev1 → prefect-3.7.2.dev2}/src/prefect/AGENTS.md

@@ -30,6 +30,7 @@ There is no formal public/private boundary beyond the `_` prefix convention. Mod
 - **`ProcessPoolTaskRunner` instances may be deserialized without `__init__` running.** When a runner is pickled and restored in a subprocess, `__init__` is not called, so instance attributes set there may be absent. Access any such attribute via `getattr(self, "_attr", default)` rather than `self._attr` directly — both in the property getter and in `duplicate()`. The `subprocess_message_processor_factories` property demonstrates the required pattern.
 - **`ThreadPoolTaskRunner` is cloudpickled when a flow run is dispatched to a subprocess.** `threading.Lock` and other non-picklable thread primitives must be dropped in `__getstate__` and rebuilt in `__setstate__`. Any new instance state added to this class must be evaluated for picklability.
 - **Nested task submissions on a bounded `ThreadPoolTaskRunner` can deadlock.** When a worker task submits children and blocks on `.result()` while all `max_workers` threads are busy, the pool starves. `_warn_if_nested_submit_would_deadlock` detects this and emits a one-time warning — preserve this detection when changing pool management.
+- **Result storage resolves through three tiers, potentially making an API call.** `get_default_result_storage()` checks: (1) `PREFECT_DEFAULT_RESULT_STORAGE_BLOCK` setting, (2) server-configured default block (API call to the server's Configuration store), (3) local storage path. The API call is silently suppressed on any HTTP error. Engines use `_get_default_persist_result()` — not `should_persist_result()` — at context setup time to auto-enable persistence when a server default block is configured. Do not swap `_get_default_persist_result()` back to `should_persist_result()` in engine initialization paths; `should_persist_result()` does not consult the server default.
 - **`ResultRecordMetadata` tolerates unknown serializer types.** When loading persisted metadata, an unrecognized serializer `type` is converted to an `UnknownSerializer` placeholder rather than raising `ValidationError`. This allows inspecting result metadata when the serializer implementation is unavailable in the current environment. However, known serializer types with invalid fields still raise `ValidationError`. `UnknownSerializer.dumps()` and `UnknownSerializer.loads()` raise `RuntimeError` — the tolerance is for inspection only, not actual serialization/deserialization.
 - **Flow-run suspension is enforced at orchestration boundaries.** Suspension is delivered via `FlowRunSuspensionRequest` and raised with `raise_if_flow_run_suspension_requested()`. When changing engines, futures, task runners, or generator execution, check before returning control to flow user code or proposing a flow state that could leave `Suspended`; keep sync/async paths aligned and avoid per-boundary API reads.
 

prefect-3.7.2.dev2/src/prefect/_build_info.py

@@ -0,0 +1,5 @@
+# Generated by versioningit
+__version__ = "3.7.2.dev2"
+__build_date__ = "2026-05-19 09:19:13.443109+00:00"
+__git_commit__ = "68244724cf7a168f14f81bdf032689c912a404de"
+__dirty__ = False

{prefect-3.7.2.dev1 → prefect-3.7.2.dev2}/src/prefect/blocks/notifications.py

@@ -57,7 +57,15 @@ class AbstractAppriseNotificationBlock(NotificationBlock, ABC):
         body: str,
         subject: str | None = None,
     ) -> None:
-        with LogEavesdropper("apprise", level=logging.DEBUG) as eavesdropper:
+        apprise_logger = logging.getLogger("apprise")
+        root_logger = logging.getLogger()
+
+        if apprise_logger.level == logging.NOTSET:
+            log_level = root_logger.getEffectiveLevel()
+        else:
+            log_level = max(apprise_logger.level, root_logger.getEffectiveLevel())
+
+        with LogEavesdropper("apprise", level=log_level) as eavesdropper:
             result = await self._apprise_client.async_notify(  # pyright: ignore[reportUnknownMemberType] incomplete type hints in apprise
                 body=body,
                 title=subject or "",
@@ -72,7 +80,15 @@ class AbstractAppriseNotificationBlock(NotificationBlock, ABC):
         body: str,
         subject: str | None = None,
     ) -> None:
-        with LogEavesdropper("apprise", level=logging.DEBUG) as eavesdropper:
+        apprise_logger = logging.getLogger("apprise")
+        root_logger = logging.getLogger()
+
+        if apprise_logger.level == logging.NOTSET:
+            log_level = root_logger.getEffectiveLevel()
+        else:
+            log_level = max(apprise_logger.level, root_logger.getEffectiveLevel())
+
+        with LogEavesdropper("apprise", level=log_level) as eavesdropper:
             result = self._apprise_client.notify(  # pyright: ignore[reportUnknownMemberType] incomplete type hints in apprise
                 body=body,
                 title=subject or "",

{prefect-3.7.2.dev1 → prefect-3.7.2.dev2}/src/prefect/bundles/__init__.py

@@ -593,8 +593,23 @@ def execute_bundle_in_subprocess(
 
     # Install dependencies if necessary
     if dependencies := bundle.get("dependencies"):
+        dep_lines = [line.strip() for line in dependencies.split("\n") if line.strip()]
+        validated_deps: list[str] = []
+        for dep_line in dep_lines:
+            if dep_line.startswith("-"):
+                raise ValueError(
+                    f"Invalid dependency (command-line flags are not allowed): {dep_line!r}"
+                )
+            try:
+                Requirement(dep_line)
+            except InvalidRequirement as e:
+                raise ValueError(
+                    f"Invalid PEP 508 dependency specifier: {dep_line!r}"
+                ) from e
+            validated_deps.append(dep_line)
+
         subprocess.check_call(
-            [_get_uv_path(), "pip", "install", *dependencies.split("\n")],
+            [_get_uv_path(), "pip", "install", *validated_deps],
             # Copy the current environment to ensure we install into the correct venv
             env=os.environ,
         )

{prefect-3.7.2.dev1 → prefect-3.7.2.dev2}/src/prefect/runner/_scheduled_run_poller.py

@@ -126,7 +126,6 @@ class ScheduledRunPoller:
             ),
         )
 
-        submitted_ids: set[UUID] = set()
         for flow_run in submittable_flow_runs:
             if flow_run.id in self._submitting_flow_run_ids:
                 continue
@@ -135,13 +134,12 @@ class ScheduledRunPoller:
             except anyio.WouldBlock:
                 break  # sorted: no earlier run fits
             self._submitting_flow_run_ids.add(flow_run.id)
-            submitted_ids.add(flow_run.id)
             task_group.start_soon(self._submit_run, flow_run, task_group, slot_token)
 
         skipped_count = sum(
             1
             for r in submittable_flow_runs
-            if r.id not in self._submitting_flow_run_ids and r.id not in submitted_ids
+            if r.id not in self._submitting_flow_run_ids
         )
         if skipped_count > 0:
             self._logger.info("%d scheduled runs skipped (at capacity)", skipped_count)

{prefect-3.7.2.dev1 → prefect-3.7.2.dev2}/src/prefect/server/AGENTS.md

@@ -45,6 +45,8 @@ alembic_revision("description")      # Create a new migration
 
 - **`_find_block_schema_via_checksum` dict-index miss is definitive — no linear-scan fallback.** In `models/block_schemas.py`, passing a `checksum_index` dict makes lookups O(1); a miss returns `None` without consulting the row list. Any code calling `_construct_full_block_schema` in a bulk loop must pre-build a `checksum_index` once and thread it through all recursive calls — omitting the kwarg on a recursive call silently downgrades every lookup to an O(N) scan, restoring O(N²) overall cost. The index uses first-wins semantics to match `next()` scan order.
 
+- **`read_server_default_result_storage` bypasses the generic `Configuration` cache intentionally.** In `models/storage_defaults.py`, the read path uses a raw `sa.select` query instead of `models/configuration.read_configuration()` to avoid serving stale cached data when another server process updates the setting. Do not refactor this to use the generic `read_configuration()` helper — the cache isolation is the desired behavior.
+
 - **`GlobalConcurrencyLimitResponse.active_slots` is computed at read time, not stored.** The DB column holds the raw accumulated count; the API joins `active_slots_after_decay()` (in `models/concurrency_limits_v2.py`) on every read. Never return ORM model fields directly from `api/concurrency_limits_v2.py` — use `_global_concurrency_limit_response()` which pulls `active_slots` from the JOIN result. `PATCH` operations do not persist the decayed count; after an update, `active_slots` in the DB remains the pre-decay value.
 
 - **`force=True` routes through `MinimalFlowPolicy`, not a true bypass.** `MinimalFlowPolicy` is intentionally lightweight but still includes guards — notably `PreventResultDataLoss`, which rejects COMPLETED→COMPLETED transitions that would discard persisted result data. Do not assume `force=True` skips all orchestration rules.

{prefect-3.7.2.dev1 → prefect-3.7.2.dev2}/src/prefect/workers/_worker_channel/_sync.py

@@ -16,7 +16,7 @@ from prefect.client.base import ServerType
 from prefect.client.orchestration import PrefectClient
 from prefect.client.schemas.actions import WorkPoolCreate, WorkPoolUpdate
 from prefect.client.schemas.objects import WorkerMetadata, WorkPool
-from prefect.exceptions import ObjectNotFound
+from prefect.exceptions import ObjectAlreadyExists, ObjectNotFound
 from prefect.workers._cleanup import WorkerCleanupExecutor
 from prefect.workers._worker_channel._protocol import WorkerChannelProtocolHandler
 from prefect.workers._worker_channel._state import (
@@ -310,8 +310,21 @@ class WorkPoolWorkerChannel:
                 if self.base_job_template is not None:
                     wp.base_job_template = self.base_job_template
 
-                work_pool = await self._client.create_work_pool(work_pool=wp)
-                self._logger.info(f"Work pool {self.work_pool_name!r} created.")
+                try:
+                    work_pool = await self._client.create_work_pool(work_pool=wp)
+                    self._logger.info(f"Work pool {self.work_pool_name!r} created.")
+                except ObjectAlreadyExists:
+                    # Another worker created the pool between our read and
+                    # create. Re-read so we can continue with the existing
+                    # pool rather than failing setup.
+                    self._logger.debug(
+                        "Work pool %r was created concurrently by another "
+                        "worker; re-reading.",
+                        self.work_pool_name,
+                    )
+                    work_pool = await self._client.read_work_pool(
+                        work_pool_name=self.work_pool_name
+                    )
             else:
                 self._logger.warning(f"Work pool {self.work_pool_name!r} not found!")
                 if self.base_job_template is not None:

prefect-3.7.2.dev1/src/prefect/_build_info.py

@@ -1,5 +0,0 @@
-# Generated by versioningit
-__version__ = "3.7.2.dev1"
-__build_date__ = "2026-05-18 09:21:21.610305+00:00"
-__git_commit__ = "25ff793e11f525e8f7043525165e0ded57abaea9"
-__dirty__ = False