rappel 0.4.1__py3-none-win_amd64.whl → 0.4.5__py3-none-win_amd64.whl

rappel/actions.py

@@ -1,9 +1,11 @@
 import inspect
 from dataclasses import dataclass
+from functools import wraps
 from typing import Any, Callable, Optional, TypeVar, overload
 
 from proto import messages_pb2 as pb2
 
+from .dependencies import provide_dependencies
 from .registry import AsyncAction, registry
 from .serialization import dumps, loads
 
@@ -64,17 +66,42 @@ def action(
     *,
     name: Optional[str] = None,
 ) -> Callable[[TAsync], TAsync] | TAsync:
-    """Decorator for registering async actions."""
+    """Decorator for registering async actions.
+
+    Actions decorated with @action will automatically resolve Depend() markers
+    when called directly (e.g., during pytest runs where workflows bypass the
+    gRPC bridge).
+    """
 
     def decorator(target: TAsync) -> TAsync:
         if not inspect.iscoroutinefunction(target):
             raise TypeError(f"action '{target.__name__}' must be defined with 'async def'")
         action_name = name or target.__name__
         action_module = target.__module__
+
+        @wraps(target)
+        async def wrapper(*args: Any, **kwargs: Any) -> Any:
+            # Convert positional args to kwargs based on the signature
+            sig = inspect.signature(target)
+            params = list(sig.parameters.keys())
+            for i, arg in enumerate(args):
+                if i < len(params):
+                    kwargs[params[i]] = arg
+
+            # Resolve dependencies using the same mechanism as execute_action
+            async with provide_dependencies(target, kwargs) as call_kwargs:
+                return await target(**call_kwargs)
+
+        # Copy over the original function's attributes for introspection
+        wrapper.__wrapped__ = target  # type: ignore[attr-defined]
+        wrapper.__rappel_action_name__ = action_name  # type: ignore[attr-defined]
+        wrapper.__rappel_action_module__ = action_module  # type: ignore[attr-defined]
+
+        # Register the original function (not the wrapper) so execute_action
+        # doesn't double-resolve dependencies
         registry.register(action_module, action_name, target)
-        target.__rappel_action_name__ = action_name
-        target.__rappel_action_module__ = action_module
-        return target
+
+        return wrapper  # type: ignore[return-value]
 
     if func is not None:
         return decorator(func)

rappel/dependencies.py

@@ -7,26 +7,40 @@ from typing import Annotated, Any, AsyncIterator, Callable, Optional, get_args,
 
 
 @dataclass(frozen=True)
-class Depend:
-    """Marker for dependency injection, mirroring FastAPI's Depends syntax."""
+class DependMarker:
+    """Internal marker for dependency injection."""
 
     dependency: Optional[Callable[..., Any]] = None
     use_cache: bool = True
 
 
-def _depend_from_annotation(annotation: Any) -> Depend | None:
+def Depend(  # noqa: N802
+    dependency: Optional[Callable[..., Any]] = None,
+    *,
+    use_cache: bool = True,
+) -> Any:
+    """Marker for dependency injection, mirroring FastAPI's Depends syntax.
+
+    Returns Any to allow usage as a default parameter value:
+        def my_func(service: MyService = Depend(get_service)):
+            ...
+    """
+    return DependMarker(dependency=dependency, use_cache=use_cache)
+
+
+def _depend_from_annotation(annotation: Any) -> DependMarker | None:
     origin = get_origin(annotation)
     if origin is not Annotated:
         return None
     metadata = get_args(annotation)[1:]
     for meta in metadata:
-        if isinstance(meta, Depend):
+        if isinstance(meta, DependMarker):
             return meta
     return None
 
 
-def _dependency_marker(parameter: inspect.Parameter) -> Depend | None:
-    if isinstance(parameter.default, Depend):
+def _dependency_marker(parameter: inspect.Parameter) -> DependMarker | None:
+    if isinstance(parameter.default, DependMarker):
         return parameter.default
     return _depend_from_annotation(parameter.annotation)
 
@@ -69,7 +83,7 @@ class _DependencyResolver:
             raise TypeError(f"Missing required parameter '{name}' for {func_name}")
         return call_kwargs
 
-    async def _resolve_dependency(self, marker: Depend) -> Any:
+    async def _resolve_dependency(self, marker: DependMarker) -> Any:
         dependency = marker.dependency
         if dependency is None:
             raise TypeError("Depend requires a dependency callable")

rappel/registry.py

@@ -29,6 +29,32 @@ class ActionRegistry:
         self._actions: dict[str, _ActionEntry] = {}
         self._lock = RLock()
 
+    def _source_fingerprint(self, func: AsyncAction) -> tuple[str | None, str | None]:
+        func_any: Any = func
+        try:
+            code = func_any.__code__
+        except AttributeError:
+            return (None, None)
+        try:
+            qualname = func_any.__qualname__
+        except AttributeError:
+            qualname = None
+        filename = code.co_filename
+        if not isinstance(filename, str):
+            filename = None
+        if qualname is not None and not isinstance(qualname, str):
+            qualname = None
+        return (filename, qualname)
+
+    def _is_same_action_definition(self, existing: AsyncAction, new: AsyncAction) -> bool:
+        if existing is new:
+            return True
+        existing_fingerprint = self._source_fingerprint(existing)
+        new_fingerprint = self._source_fingerprint(new)
+        if existing_fingerprint == (None, None) or new_fingerprint == (None, None):
+            return False
+        return existing_fingerprint == new_fingerprint
+
     def register(self, module: str, name: str, func: AsyncAction) -> None:
         """Register an action with its module and name.
 
@@ -38,11 +64,16 @@ class ActionRegistry:
             func: The async function to execute.
 
         Raises:
-            ValueError: If an action with the same module:name is already registered.
+            ValueError: If an action with the same module:name is already registered
+                with a different implementation.
         """
         key = _make_key(module, name)
         with self._lock:
-            if key in self._actions:
+            existing = self._actions.get(key)
+            if existing is not None:
+                if self._is_same_action_definition(existing.func, func):
+                    self._actions[key] = _ActionEntry(module=module, name=name, func=func)
+                    return
                 raise ValueError(f"action '{module}:{name}' already registered")
             self._actions[key] = _ActionEntry(module=module, name=name, func=func)
 

{rappel-0.4.1.dist-info → rappel-0.4.5.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: rappel
-Version: 0.4.1
+Version: 0.4.5
 Summary: Distributed & durable background events in Python
 Requires-Python: >=3.10
 Requires-Dist: googleapis-common-protos>=1.72.0
@@ -155,19 +155,26 @@ If you have a particular workflow that you think should be working but isn't yet
 
 The main rappel configuration is done through env vars, which is what you'll typically use in production when using a docker deployment pipeline. If we can't find an environment parameter we will fallback to looking for an .env that specifies it within your local filesystem.
 
+These are the primary environment parameters that you'll likely want to customize for your deployment:
+
 | Environment Variable | Description | Default | Example |
 |---------------------|-------------|---------|---------|
-| `DATABASE_URL` | PostgreSQL connection string for the rappel server | (required) | `postgresql://user:pass@localhost:5433/rappel` |
-| `RAPPEL_HTTP_ADDR` | HTTP bind address for `rappel-bridge` | `127.0.0.1:24117` | `0.0.0.0:24117` |
-| `RAPPEL_GRPC_ADDR` | gRPC bind address for `rappel-bridge` | HTTP port + 1 | `0.0.0.0:24118` |
+| `RAPPEL_DATABASE_URL` | PostgreSQL connection string for the rappel server | (required on bridge &workers ) | `postgresql://user:pass@localhost:5433/rappel` |
 | `RAPPEL_WORKER_COUNT` | Number of Python worker processes | `num_cpus` | `8` |
 | `RAPPEL_CONCURRENT_PER_WORKER` | Max concurrent actions per worker | `10` | `20` |
 | `RAPPEL_USER_MODULE` | Python module preloaded into each worker | none | `my_app.actions` |
 | `RAPPEL_POLL_INTERVAL_MS` | Poll interval for the dispatch loop (ms) | `100` | `50` |
-| `RAPPEL_BATCH_SIZE` | Max actions fetched per poll | `workers * concurrent_per_worker` | `200` |
 | `RAPPEL_WEBAPP_ENABLED` | Enable the web dashboard | `false` | `true` |
 | `RAPPEL_WEBAPP_ADDR` | Web dashboard bind address | `0.0.0.0:24119` | `0.0.0.0:8080` |
 
+We expect that you won't need to modify the following env parameters, but we provide them for convenience:
+
+| Environment Variable | Description | Default | Example |
+|---------------------|-------------|---------|---------|
+| `RAPPEL_HTTP_ADDR` | HTTP bind address for `rappel-bridge` | `127.0.0.1:24117` | `0.0.0.0:24117` |
+| `RAPPEL_GRPC_ADDR` | gRPC bind address for `rappel-bridge` | HTTP port + 1 | `0.0.0.0:24118` |
+| `RAPPEL_BATCH_SIZE` | Max actions fetched per poll | `workers * concurrent_per_worker` | `200` |
+
 ## Philosophy
 
 Background jobs in webapps are so frequently used that they should really be a primitive of your fullstack library: database, backend, frontend, _and_ background jobs. Otherwise you're stuck in a situation where users either have to always make blocking requests to an API or you spin up ephemeral tasks that will be killed during re-deployments or an accidental docker crash.

{rappel-0.4.1.dist-info → rappel-0.4.5.dist-info}/RECORD

@@ -7,26 +7,26 @@ proto/messages_pb2.pyi,sha256=lzv_RubGC8ZNVwh70FQr6DI9MECKHbvmVrL0hcBUDPc,39801
 proto/messages_pb2_grpc.py,sha256=6MhQxDSSTSiW55qZh7nIRtiEsuBp5t9tCbcZxiuWcF4,16252
 proto/messages_pb2_grpc.pyi,sha256=dAauApqvlu5Ij7JncrOIhKgcQZJEeV8FrdoJydMbfRo,12702
 rappel/__init__.py,sha256=2jnekFrCK42PbFxBiXKLa5wzzYcIy7VQ1zFizv1ZUpo,1391
-rappel/actions.py,sha256=wUcmBCLjMb-406T7N5S-uR73p9grKbfomXLNjsQvFpo,2735
+rappel/actions.py,sha256=Zf3klKHNNl9UjgEkETMQWbn4wS2Ya-6JGnaRxUo-yW8,3955
 rappel/bridge.py,sha256=t-j6k3rHO84qIoOB_VFCUJ87RHKwDDHejxXkNsfQfi0,8219
-rappel/dependencies.py,sha256=40NiWV9fsn-_G5XivCUz2OMNoYC2H6cdVAQG7ziEhS0,5347
+rappel/dependencies.py,sha256=OkgxNG1mXMGepxmDMPxBwqd8Y4ebyIMoxOTbb33FQ9M,5789
 rappel/exceptions.py,sha256=mLkNf1a44rbRxuFZp9saiSp0Ec3Y4gXPXI7Ozkhdt1U,348
 rappel/formatter.py,sha256=mNLJ24nkl5zN-zXe_ObpZGeC__Pyv-7J-iPde-wLvbM,3223
 rappel/ir_builder.py,sha256=wBLdp0pndpBSmzLXGIVGZJ6p4k4OlNi30xErY2Z7ZDk,128070
 rappel/logger.py,sha256=auFfr0gdVP9J53OiADhJlCSdS9As-1DA14WNtAqgv3Y,1157
-rappel/registry.py,sha256=AZftT8r-Lsb0JgX6vbbnydqegheYXViYsCzF_NoqX40,2302
+rappel/registry.py,sha256=xrUNZqeKtwuqKkHzvzCmNsMtO34m-REQBWdcZyIirMQ,3632
 rappel/schedule.py,sha256=k_DD2BZBnJCK60VoOLlIJh6QHIIMiEnSOKsUSRved_I,9464
 rappel/serialization.py,sha256=k4uq_Wka5ptVYp0SIjXD5zflbmgxUV05pNmQs-b5i8o,8028
 rappel/worker.py,sha256=0iZkruFbGBXLLZNmENqxGF4f-ot5v9m0HzWNt8pT24s,7027
 rappel/workflow.py,sha256=Jv6SzPhLxS6oNfiKFCbp7GroWK8rGY1pMd8JqoVEPDM,8270
 rappel/workflow_runtime.py,sha256=GBLaAi1MB7rQmTtL7adAJt9rDLoWS9n52aD1YcJA-ZI,4403
-rappel/bin/boot-rappel-singleton.exe,sha256=_qbWkDd4tfd4WkfMW6-m1Z1SxFtGE06UXgZO024TVrU,5104128
-rappel/bin/rappel-bridge.exe,sha256=kOGR0ENIJ_c3qjpz85yFEhaVnBmvTezclXchCk-Z79g,9680896
-rappel/bin/start-workers.exe,sha256=wKdShH7oKt0Gmhkvkhx0xBOR1TvPD-CytRBvyli0cAw,15662592
-rappel-0.4.1.dist-info/METADATA,sha256=xUr4y0umv-V16uvqR9UO3LmaswP-eGo-DTiUjYl5Lbk,15381
-rappel-0.4.1.dist-info/entry_points.txt,sha256=h9D-AufOUWpdE7XjnyZyQCc-kER-ZIKj1Jryc1JNL_I,53
-rappel-0.4.1.dist-info/RECORD,,
-rappel-0.4.1.data/scripts/boot-rappel-singleton.exe,sha256=_qbWkDd4tfd4WkfMW6-m1Z1SxFtGE06UXgZO024TVrU,5104128
-rappel-0.4.1.data/scripts/rappel-bridge.exe,sha256=kOGR0ENIJ_c3qjpz85yFEhaVnBmvTezclXchCk-Z79g,9680896
-rappel-0.4.1.data/scripts/start-workers.exe,sha256=wKdShH7oKt0Gmhkvkhx0xBOR1TvPD-CytRBvyli0cAw,15662592
-rappel-0.4.1.dist-info/WHEEL,sha256=phIoPJnECdbLLKrTiGU1mv92w_v6wBxRMAejLbbrKno,94
+rappel/bin/boot-rappel-singleton.exe,sha256=FeJdZiFVoesqrpG8_ESSyXGrVkSDB4dsqdXta51fygI,5104128
+rappel/bin/rappel-bridge.exe,sha256=JpnSY7RYd1-NO3kPfuiS__g6tTMdUjQMe18rmsf-Wls,9680896
+rappel/bin/start-workers.exe,sha256=nCvJWiNIrGAwlxzd0hFVJjI6MEaGDQlRXwPwmjxmj8A,15667712
+rappel-0.4.5.dist-info/METADATA,sha256=ddaik4XZC1PT3HUbybTfnCyu9lUINfi4fd8JmBbcIEA,15738
+rappel-0.4.5.dist-info/entry_points.txt,sha256=h9D-AufOUWpdE7XjnyZyQCc-kER-ZIKj1Jryc1JNL_I,53
+rappel-0.4.5.dist-info/RECORD,,
+rappel-0.4.5.data/scripts/boot-rappel-singleton.exe,sha256=FeJdZiFVoesqrpG8_ESSyXGrVkSDB4dsqdXta51fygI,5104128
+rappel-0.4.5.data/scripts/rappel-bridge.exe,sha256=JpnSY7RYd1-NO3kPfuiS__g6tTMdUjQMe18rmsf-Wls,9680896
+rappel-0.4.5.data/scripts/start-workers.exe,sha256=nCvJWiNIrGAwlxzd0hFVJjI6MEaGDQlRXwPwmjxmj8A,15667712
+rappel-0.4.5.dist-info/WHEEL,sha256=phIoPJnECdbLLKrTiGU1mv92w_v6wBxRMAejLbbrKno,94