loom-kernel 2.20.0__tar.gz → 2.20.2__tar.gz

{loom_kernel-2.20.0 → loom_kernel-2.20.2}/CHANGELOG.md

@@ -485,6 +485,10 @@
 
 ### rest
 
+- **rest:** a `RestInterface` whose `prefix` contains `{` now raises
+  `InterfaceCompilationError` at compile time, naming the interface and the
+  prefix; declare the placeholder in the route `path` instead (a catch-all
+  `prefix="/{tenant}"` becomes `prefix="/"` with `path="/{tenant}"`).
 - **rest:** auto-CRUD mounts no `/{id}` route (get, update, delete) for a
   model with a composite primary key, since one path segment cannot carry the
   key; naming one of them in `include` raises `ValueError` when the routes are
@@ -602,6 +606,15 @@
 
 ### core
 
+- **core:** the relation steps and the `load_many` projection steps of a read
+  profile run one after another on the request's session. They ran with
+  `asyncio.gather` on that one session: when one step failed while another was
+  still waiting, on Postgres with asyncpg the caller got `InterfaceError:
+  cannot perform operation: another operation is in progress` instead of the
+  step's error, and the pooled connection was left in a manually started
+  transaction, so the next request on it failed until `engine.dispose()`. The
+  number of statements is unchanged, at most one per step.
+
 - **core:** cursor pagination and the by-id operations of the SQLAlchemy
   repository use the full primary key of a model whose key is composite, such
   as a `RowScoped` model keyed by its boundary column and its own id. They
@@ -769,6 +782,11 @@
 
 ### rest
 
+- **rest:** a path placeholder in `RestInterface.prefix` was dropped. The route
+  was registered on prefix + path, but path parameters were read from the
+  route `path` only, so the use case never received the prefix parameter (a
+  required one answered `500`) and OpenAPI did not list it. Such a prefix is
+  now refused at compile time.
 - **rest:** an authenticator can report an outage instead of answering `500`.
   An exception raised inside `AuthenticationMiddleware` never reached
   Starlette's exception handlers, so a custom authenticator whose identity

{loom_kernel-2.20.0 → loom_kernel-2.20.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: loom-kernel
-Version: 2.20.0
+Version: 2.20.2
 Summary: Loom Python project
 License-File: LICENSE
 Classifier: Programming Language :: Python :: 3

{loom_kernel-2.20.0 → loom_kernel-2.20.2}/src/loom/core/backend/core_model.py

@@ -21,7 +21,6 @@ The ORM class (``ProductSA``) is kept exclusively for writes and Alembic.
 
 from __future__ import annotations
 
-import asyncio
 from collections.abc import Sequence
 from dataclasses import dataclass
 from typing import TYPE_CHECKING, Any
@@ -202,8 +201,9 @@ class CoreModel:
     ) -> list[Any]:
         """Execute ``stmt``, batch-load relations, run projections, return ``list[Struct]``.
 
-        Relations declared in the profile are fetched in parallel via
-        ``asyncio.gather``.  Projection steps run after relation loading.
+        Relations declared in the profile are loaded one after another on
+        ``session``, with at most one statement each.  Projection steps run
+        after relation loading.
 
         Args:
             session: Active async SQLAlchemy session.
@@ -278,10 +278,7 @@ async def _load_relations(
     steps: tuple[CoreRelationStep, ...],
     rows: list[Any],
 ) -> dict[str, dict[Any, Any]]:
-    if not steps:
-        return {}
-    results = await asyncio.gather(*(_fetch_relation_batch(session, step, rows) for step in steps))
-    return {step.attr: data for step, data in zip(steps, results, strict=False)}
+    return {step.attr: await _fetch_relation_batch(session, step, rows) for step in steps}
 
 
 def _make_adapter(

{loom_kernel-2.20.0 → loom_kernel-2.20.2}/src/loom/core/projection/runtime.py

@@ -1,6 +1,5 @@
 from __future__ import annotations
 
-import asyncio
 import inspect
 from collections.abc import Mapping, Sequence
 from dataclasses import dataclass
@@ -231,18 +230,12 @@ async def _execute_backend_steps(
     parent_ids: Sequence[object],
     values_by_index: dict[int, dict[str, Any]],
 ) -> None:
-    if not steps:
-        return
-
-    backend_results = await asyncio.gather(
-        *(step.loader.load_many(backend_context, parent_ids) for step in steps)
-    )
-    for step, rows in zip(steps, backend_results, strict=False):
+    for step in steps:
         _merge_rows(
             values_by_index=values_by_index,
             parent_ids=parent_ids,
             field_name=step.name,
-            rows=rows,
+            rows=await step.loader.load_many(backend_context, parent_ids),
             default=step.projection.default,
         )
 

{loom_kernel-2.20.0 → loom_kernel-2.20.2}/src/loom/rest/compiler.py

@@ -423,6 +423,12 @@ class RestInterfaceCompiler:
             raise InterfaceCompilationError(
                 f"{iface_name}: 'prefix' must be a non-empty string (e.g. '/users')."
             )
+        if "{" in interface.prefix:
+            raise InterfaceCompilationError(
+                f"{iface_name}: 'prefix' {interface.prefix!r} contains a path "
+                "placeholder.  Placeholders in 'prefix' are not bound to the use case; "
+                "put the placeholder in the route 'path' instead."
+            )
         if not interface.routes:
             raise InterfaceCompilationError(
                 f"{iface_name}: 'routes' is empty.  Declare at least one RestRoute to expose."

{loom_kernel-2.20.0 → loom_kernel-2.20.2}/src/loom/rest/model.py

@@ -132,6 +132,8 @@ class RestInterface(Generic[T]):
 
     Attributes:
         prefix: URL prefix for all routes in this interface (e.g. ``"/users"``).
+            Must not contain a path placeholder; declare it in the route
+            ``path``.
         tags: OpenAPI tags applied to every route.
         auto: When ``True``, signals intent to use standard CRUD URL conventions.
         include: Whitelist of CRUD operation names to expose when ``auto=True``.