From 6972d7cf3eb9b65fd7286bd90bc07c68208f985a Mon Sep 17 00:00:00 2001
From: kanarus <kanarus786@gmail.com>
Date: Wed, 27 Aug 2025 15:50:38 +0900
Subject: [PATCH] refactor(fangs): improve `FangAction` doc and tidy code
 format

---
 ohkami/src/fang/middleware/util.rs | 23 +++++++++++------------
 1 file changed, 11 insertions(+), 12 deletions(-)

diff --git a/ohkami/src/fang/middleware/util.rs b/ohkami/src/fang/middleware/util.rs
index 4f018e2a8..6f008ec9c 100644
--- a/ohkami/src/fang/middleware/util.rs
+++ b/ohkami/src/fang/middleware/util.rs
@@ -70,49 +70,48 @@ use crate::openapi;
 pub trait FangAction: Clone + SendSyncOnNative + 'static {
     // Here not using `-> impl SendOnNativeFuture` for
     // rust-analyzer's completion.
-    // Currently rust-analyzer can complete `-> Future` methods
-    // as `async fn ...` **only when** it returns exactly one of:
-    // 
-    // * `-> impl Future<Output = T>`
-    // * `-> impl Future<Output = T> + Send`
-    // * `-> impl Future<Output = T> + Send + 'lifetime`
-    // 
-    // so `-> impl SendOnNativeFuture<T>` prevents his completion...
+    // Currently rust-analyzer can perform completion for `-> impl Future` methods
+    // as `async fn ...` **only when** it returns exactly `impl Future (+ something)*`,
+    // and he can't do it for `-> impl SendOnNativeFuture<T>`.
 
-    #[cfg(any(feature="rt_worker",))]
     /// *fore fang*, that bites a request before a handler.
     /// 
     /// ### default
     /// just return `Ok(())`
+    #[cfg(any(feature="rt_worker",))]
     #[allow(unused_variables)]
+    #[inline(always)]
     fn fore<'a>(&'a self, req: &'a mut Request) -> impl Future<Output = Result<(), Response>> {
         async {Ok(())}
     }
-    #[cfg(not(any(feature="rt_worker",)))]
     /// *fore fang*, that bites a request before a handler.
     /// 
     /// ### default
     /// just return `Ok(())`
+    #[cfg(not(any(feature="rt_worker",)))]
     #[allow(unused_variables)]
+    #[inline(always)]
     fn fore<'a>(&'a self, req: &'a mut Request) -> impl Future<Output = Result<(), Response>> + Send {
         async {Ok(())}
     }
 
-    #[cfg(any(feature="rt_worker",))]
     /// *back fang*, that bites a generated response.
     /// 
     /// ### default
     /// just return `()`
+    #[cfg(any(feature="rt_worker",))]
     #[allow(unused_variables)]
+    #[inline(always)]
     fn back<'a>(&'a self, res: &'a mut Response) -> impl Future<Output = ()> {
         async {}
     }
-    #[cfg(not(any(feature="rt_worker",)))]
     /// *back fang*, that bites a generated response.
     /// 
     /// ### default
     /// just return `()`
+    #[cfg(not(any(feature="rt_worker",)))]
     #[allow(unused_variables)]
+    #[inline(always)]
     fn back<'a>(&'a self, res: &'a mut Response) -> impl Future<Output = ()> + Send {
         async {}
     }