Fix #370 by adding an alias of next arg (next_) in middleware arguments (#394)

Author: seratch

slack_bolt/kwargs_injection/args.py

@@ -72,6 +72,8 @@ def handle_buttons(ack, respond, logger, context, body, client):
     # middleware
     next: Callable[[], None]
     """`next()` utility function, which tells the middleware chain that it can continue with the next one"""
+    next_: Callable[[], None]
+    """An alias of `next()` for avoiding the Python built-in method overrides in middleware functions"""
 
     def __init__(
         self,
@@ -93,6 +95,9 @@ def __init__(
         ack: Ack,
         say: Say,
         respond: Respond,
+        # As this method is not supposed to be invoked by bolt-python users,
+        # the naming conflict with the built-in one affects
+        # only the internals of this method
         next: Callable[[], None],
         **kwargs  # noqa
     ):
@@ -116,3 +121,4 @@ def __init__(
         self.say: Say = say
         self.respond: Respond = respond
         self.next: Callable[[], None] = next
+        self.next_: Callable[[], None] = next

slack_bolt/kwargs_injection/async_args.py

@@ -71,6 +71,8 @@ async def handle_buttons(ack, respond, logger, context, body, client):
     # middleware
     next: Callable[[], Awaitable[None]]
     """`next()` utility function, which tells the middleware chain that it can continue with the next one"""
+    next_: Callable[[], Awaitable[None]]
+    """An alias of `next()` for avoiding the Python built-in method overrides in middleware functions"""
 
     def __init__(
         self,
@@ -115,3 +117,4 @@ def __init__(
         self.say: AsyncSay = say
         self.respond: AsyncRespond = respond
         self.next: Callable[[], Awaitable[None]] = next
+        self.next_: Callable[[], Awaitable[None]] = next

slack_bolt/kwargs_injection/async_utils.py

@@ -52,6 +52,7 @@ def build_async_required_kwargs(
         "respond": request.context.respond,
         # middleware
         "next": next_func,
+        "next_": next_func,  # for the middleware using Python's built-in `next()` function
     }
     all_available_args["payload"] = (
         all_available_args["options"]

slack_bolt/kwargs_injection/utils.py

@@ -52,6 +52,7 @@ def build_required_kwargs(
         "respond": request.context.respond,
         # middleware
         "next": next_func,
+        "next_": next_func,  # for the middleware using Python's built-in `next()` function
     }
     all_available_args["payload"] = (
         all_available_args["options"]

slack_bolt/listener/listener.py

@@ -45,10 +45,10 @@ def run_middleware(
         for m in self.middleware:
             middleware_state = {"next_called": False}
 
-            def next():
+            def next_():
                 middleware_state["next_called"] = True
 
-            resp = m.process(req=req, resp=resp, next=next)
+            resp = m.process(req=req, resp=resp, next=next_)
             if not middleware_state["next_called"]:
                 # next() was not called in this middleware
                 return (resp, True)

slack_bolt/logger/messages.py

@@ -99,7 +99,7 @@ def warning_unhandled_by_global_middleware(  # type: ignore
     name: str, req: Union[BoltRequest, "AsyncBoltRequest"]  # type: ignore
 ) -> str:  # type: ignore
     return (
-        f"A global middleware ({name}) skipped calling `next()` "
+        f"A global middleware ({name}) skipped calling either `next()` or `next_()` "
         f"without providing a response for the request ({req.body})"
     )
 

slack_bolt/middleware/async_custom_middleware.py

@@ -31,6 +31,9 @@ async def async_process(
         *,
         req: AsyncBoltRequest,
         resp: BoltResponse,
+        # As this method is not supposed to be invoked by bolt-python users,
+        # the naming conflict with the built-in one affects
+        # only the internals of this method
         next: Callable[[], Awaitable[BoltResponse]],
     ) -> BoltResponse:
         return await self.func(

slack_bolt/middleware/async_middleware.py

@@ -14,6 +14,9 @@ async def async_process(
         *,
         req: AsyncBoltRequest,
         resp: BoltResponse,
+        # As this method is not supposed to be invoked by bolt-python users,
+        # the naming conflict with the built-in one affects
+        # only the internals of this method
         next: Callable[[], Awaitable[BoltResponse]],
     ) -> Optional[BoltResponse]:
         """Processes a request data before other middleware and listeners.
@@ -24,6 +27,14 @@ async def simple_middleware(req, resp, next):
                 # do something here
                 await next()
 
+        This `async_process(req, resp, next)` method is supposed to be invoked only inside bolt-python.
+        If you want to avoid the name `next()` in your middleware functions, you can use `next_()` method instead.
+
+            @app.middleware
+            async def simple_middleware(req, resp, next_):
+                # do something here
+                await next_()
+
         Args:
             req: The incoming request
             resp: The response

slack_bolt/middleware/authorization/async_multi_teams_authorization.py

@@ -28,6 +28,9 @@ async def async_process(
         *,
         req: AsyncBoltRequest,
         resp: BoltResponse,
+        # As this method is not supposed to be invoked by bolt-python users,
+        # the naming conflict with the built-in one affects
+        # only the internals of this method
         next: Callable[[], Awaitable[BoltResponse]],
     ) -> BoltResponse:
         if _is_no_auth_required(req):

slack_bolt/middleware/authorization/async_single_team_authorization.py

@@ -22,6 +22,9 @@ async def async_process(
         *,
         req: AsyncBoltRequest,
         resp: BoltResponse,
+        # As this method is not supposed to be invoked by bolt-python users,
+        # the naming conflict with the built-in one affects
+        # only the internals of this method
         next: Callable[[], Awaitable[BoltResponse]],
     ) -> BoltResponse:
         if _is_no_auth_required(req):