Adapt LSPosedBridge to convention-based hooker discovery (#534)

JingMatrix · web-flow · commit 40f09cf1c765 · 2026-02-13T18:27:16.000+01:00
We update `LSPosedBridge` to align with upstream API changes, which have replaced annotation-based hooker discovery with a naming convention.

The `doHook` implementation has been refactored to:
- Remove dependencies on the deleted `io.github.libxposed.api.annotations` package (`XposedHooker`, `BeforeInvocation`, `AfterInvocation`).
- Scan for public static methods explicitly named `before` and `after` instead of relying on annotations.
- Enforce validation on these named methods to ensure they match the required signatures.

To adapt to this change, existing Hooker classes are refactored by removing the deprecated annotations and renaming their callback methods to `before` and `after` respectively.
diff --git a/core/proguard-rules.pro b/core/proguard-rules.pro
@@ -22,9 +22,12 @@
 -keepclassmembers class org.lsposed.lspd.impl.LSPosedHookCallback {
     public <methods>;
 }
--keep,allowoptimization,allowobfuscation @io.github.libxposed.api.annotations.* class * {
-    @io.github.libxposed.api.annotations.BeforeInvocation <methods>;
-    @io.github.libxposed.api.annotations.AfterInvocation <methods>;
+-keepclassmembers,allowoptimization class ** implements io.github.libxposed.api.XposedInterface$Hooker {
+    public static *** before();
+    public static *** before(io.github.libxposed.api.XposedInterface$BeforeHookCallback);
+    public static void after();
+    public static void after(io.github.libxposed.api.XposedInterface$AfterHookCallback);
+    public static void after(io.github.libxposed.api.XposedInterface$AfterHookCallback, ***);
 }
 -assumenosideeffects class android.util.Log {
     public static *** v(...);
diff --git a/core/src/main/java/org/lsposed/lspd/hooker/AttachHooker.java b/core/src/main/java/org/lsposed/lspd/hooker/AttachHooker.java
@@ -4,14 +4,10 @@
 
 import de.robv.android.xposed.XposedInit;
 import io.github.libxposed.api.XposedInterface;
-import io.github.libxposed.api.annotations.AfterInvocation;
-import io.github.libxposed.api.annotations.XposedHooker;
 
-@XposedHooker
 public class AttachHooker implements XposedInterface.Hooker {
 
-    @AfterInvocation
-    public static void afterHookedMethod(XposedInterface.AfterHookCallback callback) {
+    public static void after(XposedInterface.AfterHookCallback callback) {
         XposedInit.loadModules((ActivityThread) callback.getThisObject());
     }
 }
diff --git a/core/src/main/java/org/lsposed/lspd/hooker/CrashDumpHooker.java b/core/src/main/java/org/lsposed/lspd/hooker/CrashDumpHooker.java
@@ -4,14 +4,10 @@
 import org.lsposed.lspd.util.Utils.Log;
 
 import io.github.libxposed.api.XposedInterface;
-import io.github.libxposed.api.annotations.BeforeInvocation;
-import io.github.libxposed.api.annotations.XposedHooker;
 
-@XposedHooker
 public class CrashDumpHooker implements XposedInterface.Hooker {
 
-    @BeforeInvocation
-    public static void beforeHookedMethod(XposedInterface.BeforeHookCallback callback) {
+    public static void before(XposedInterface.BeforeHookCallback callback) {
         try {
             var e = (Throwable) callback.getArgs()[0];
             LSPosedBridge.log("Crash unexpectedly: " + Log.getStackTraceString(e));
diff --git a/core/src/main/java/org/lsposed/lspd/hooker/HandleSystemServerProcessHooker.java b/core/src/main/java/org/lsposed/lspd/hooker/HandleSystemServerProcessHooker.java
@@ -27,11 +27,8 @@
 import org.lsposed.lspd.util.Hookers;
 
 import io.github.libxposed.api.XposedInterface;
-import io.github.libxposed.api.annotations.AfterInvocation;
-import io.github.libxposed.api.annotations.XposedHooker;
 
 // system_server initialization
-@XposedHooker
 public class HandleSystemServerProcessHooker implements XposedInterface.Hooker {
 
     public interface Callback {
@@ -42,8 +39,7 @@ public interface Callback {
     public static volatile Callback callback = null;
 
     @SuppressLint("PrivateApi")
-    @AfterInvocation
-    public static void afterHookedMethod() {
+    public static void after() {
         Hookers.logD("ZygoteInit#handleSystemServerProcess() starts");
         try {
             // get system_server classLoader
diff --git a/core/src/main/java/org/lsposed/lspd/hooker/LoadedApkCreateCLHooker.java b/core/src/main/java/org/lsposed/lspd/hooker/LoadedApkCreateCLHooker.java
@@ -51,11 +51,8 @@
 import de.robv.android.xposed.callbacks.XC_LoadPackage;
 import io.github.libxposed.api.XposedInterface;
 import io.github.libxposed.api.XposedModuleInterface;
-import io.github.libxposed.api.annotations.AfterInvocation;
-import io.github.libxposed.api.annotations.XposedHooker;
 
 @SuppressLint("BlockedPrivateApi")
-@XposedHooker
 public class LoadedApkCreateCLHooker implements XposedInterface.Hooker {
     private final static Field defaultClassLoaderField;
 
@@ -77,8 +74,7 @@ static void addLoadedApk(LoadedApk loadedApk) {
         loadedApks.add(loadedApk);
     }
 
-    @AfterInvocation
-    public static void afterHookedMethod(XposedInterface.AfterHookCallback callback) {
+    public static void after(XposedInterface.AfterHookCallback callback) {
         LoadedApk loadedApk = (LoadedApk) callback.getThisObject();
 
         if (callback.getArgs()[0] != null || !loadedApks.contains(loadedApk)) {
diff --git a/core/src/main/java/org/lsposed/lspd/hooker/LoadedApkCtorHooker.java b/core/src/main/java/org/lsposed/lspd/hooker/LoadedApkCtorHooker.java
@@ -29,15 +29,11 @@
 import de.robv.android.xposed.XposedHelpers;
 import de.robv.android.xposed.XposedInit;
 import io.github.libxposed.api.XposedInterface;
-import io.github.libxposed.api.annotations.AfterInvocation;
-import io.github.libxposed.api.annotations.XposedHooker;
 
 // when a package is loaded for an existing process, trigger the callbacks as well
-@XposedHooker
 public class LoadedApkCtorHooker implements XposedInterface.Hooker {
 
-    @AfterInvocation
-    public static void afterHookedMethod(XposedInterface.AfterHookCallback callback) {
+    public static void after(XposedInterface.AfterHookCallback callback) {
         Hookers.logD("LoadedApk#<init> starts");
 
         try {
diff --git a/core/src/main/java/org/lsposed/lspd/hooker/OpenDexFileHooker.java b/core/src/main/java/org/lsposed/lspd/hooker/OpenDexFileHooker.java
@@ -6,14 +6,10 @@
 import org.lsposed.lspd.nativebridge.HookBridge;
 
 import io.github.libxposed.api.XposedInterface;
-import io.github.libxposed.api.annotations.AfterInvocation;
-import io.github.libxposed.api.annotations.XposedHooker;
 
-@XposedHooker
 public class OpenDexFileHooker implements XposedInterface.Hooker {
 
-    @AfterInvocation
-    public static void afterHookedMethod(XposedInterface.AfterHookCallback callback) {
+    public static void after(XposedInterface.AfterHookCallback callback) {
         ClassLoader classLoader = null;
         for (var arg : callback.getArgs()) {
             if (arg instanceof ClassLoader) {
diff --git a/core/src/main/java/org/lsposed/lspd/hooker/StartBootstrapServicesHooker.java b/core/src/main/java/org/lsposed/lspd/hooker/StartBootstrapServicesHooker.java
@@ -32,14 +32,10 @@
 import de.robv.android.xposed.callbacks.XC_LoadPackage;
 import io.github.libxposed.api.XposedInterface;
 import io.github.libxposed.api.XposedModuleInterface;
-import io.github.libxposed.api.annotations.BeforeInvocation;
-import io.github.libxposed.api.annotations.XposedHooker;
 
-@XposedHooker
 public class StartBootstrapServicesHooker implements XposedInterface.Hooker {
 
-    @BeforeInvocation
-    public static void beforeHookedMethod() {
+    public static void before() {
         logD("SystemServer#startBootstrapServices() starts");
 
         try {
diff --git a/core/src/main/java/org/lsposed/lspd/impl/LSPosedBridge.java b/core/src/main/java/org/lsposed/lspd/impl/LSPosedBridge.java
@@ -12,9 +12,6 @@
 
 import de.robv.android.xposed.XposedBridge;
 import io.github.libxposed.api.XposedInterface;
-import io.github.libxposed.api.annotations.AfterInvocation;
-import io.github.libxposed.api.annotations.BeforeInvocation;
-import io.github.libxposed.api.annotations.XposedHooker;
 import io.github.libxposed.api.errors.HookFailedError;
 
 public class LSPosedBridge {
@@ -218,16 +215,14 @@ public static void dummyCallback() {
             throw new IllegalArgumentException("Cannot hook Method.invoke");
         } else if (hooker == null) {
             throw new IllegalArgumentException("hooker should not be null!");
-        } else if (hooker.getAnnotation(XposedHooker.class) == null) {
-            throw new IllegalArgumentException("Hooker should be annotated with @XposedHooker");
         }
 
         Method beforeInvocation = null, afterInvocation = null;
         var modifiers = Modifier.PUBLIC | Modifier.STATIC;
         for (var method : hooker.getDeclaredMethods()) {
-            if (method.getAnnotation(BeforeInvocation.class) != null) {
+            if (method.getName().equals("before")) {
                 if (beforeInvocation != null) {
-                    throw new IllegalArgumentException("More than one method annotated with @BeforeInvocation");
+                    throw new IllegalArgumentException("More than one method named before");
                 }
                 boolean valid = (method.getModifiers() & modifiers) == modifiers;
                 var params = method.getParameterTypes();
@@ -237,13 +232,12 @@ public static void dummyCallback() {
                     valid = false;
                 }
                 if (!valid) {
-                    throw new IllegalArgumentException("BeforeInvocation method format is invalid");
+                    throw new IllegalArgumentException("before method format is invalid");
                 }
                 beforeInvocation = method;
-            }
-            if (method.getAnnotation(AfterInvocation.class) != null) {
+            } else if (method.getName().equals("after")) {
                 if (afterInvocation != null) {
-                    throw new IllegalArgumentException("More than one method annotated with @AfterInvocation");
+                    throw new IllegalArgumentException("More than one method named after");
                 }
                 boolean valid = (method.getModifiers() & modifiers) == modifiers;
                 valid &= method.getReturnType().equals(void.class);
@@ -254,13 +248,13 @@ public static void dummyCallback() {
                     valid = false;
                 }
                 if (!valid) {
-                    throw new IllegalArgumentException("AfterInvocation method format is invalid");
+                    throw new IllegalArgumentException("after method format is invalid");
                 }
                 afterInvocation = method;
             }
         }
         if (beforeInvocation == null && afterInvocation == null) {
-            throw new IllegalArgumentException("No method annotated with @BeforeInvocation or @AfterInvocation");
+            throw new IllegalArgumentException("No method named before or after found in " + hooker.getName());
         }
         try {
             if (beforeInvocation == null) {
@@ -271,7 +265,7 @@ public static void dummyCallback() {
                 var ret = beforeInvocation.getReturnType();
                 var params = afterInvocation.getParameterTypes();
                 if (ret != void.class && params.length == 2 && !ret.equals(params[1])) {
-                    throw new IllegalArgumentException("BeforeInvocation and AfterInvocation method format is invalid");
+                    throw new IllegalArgumentException("before and after method format is invalid");
                 }
             }
         } catch (NoSuchMethodException e) {
diff --git a/magisk-loader/src/main/java/org/lsposed/lspd/util/ParasiticManagerSystemHooker.java b/magisk-loader/src/main/java/org/lsposed/lspd/util/ParasiticManagerSystemHooker.java
@@ -12,8 +12,6 @@
 import org.lsposed.lspd.util.Utils;
 
 import io.github.libxposed.api.XposedInterface;
-import io.github.libxposed.api.annotations.AfterInvocation;
-import io.github.libxposed.api.annotations.XposedHooker;
 
 
 public class ParasiticManagerSystemHooker implements HandleSystemServerProcessHooker.Callback {
@@ -33,10 +31,8 @@ public static void beforeHookedMethod(XposedInterface.BeforeHookCallback callbac
         }
     }*/
 
-    @XposedHooker
     private static class Hooker implements XposedInterface.Hooker {
-        @AfterInvocation
-        public static void afterHookedMethod(XposedInterface.AfterHookCallback callback) throws Throwable {
+        public static void after(XposedInterface.AfterHookCallback callback) throws Throwable {
             var intent = (Intent) callback.getArgs()[0];
             if (intent == null) return;
             if (!intent.hasCategory("org.lsposed.manager.LAUNCH_MANAGER")) return;
diff --git a/xposed/libxposed b/xposed/libxposed
@@ -1 +1 @@
-Subproject commit 55efdf9d159195261d7326e9e125965a90025a12
+Subproject commit 64e29bd657ef4d2540b34402f5a988778f29e676

Original file line number	Diff line number	Diff line change
`@@ -4,14 +4,10 @@`
`4`	`4`
`5`	`5`	`import de.robv.android.xposed.XposedInit;`
`6`	`6`	`import io.github.libxposed.api.XposedInterface;`
`7`		`-import io.github.libxposed.api.annotations.AfterInvocation;`
`8`		`-import io.github.libxposed.api.annotations.XposedHooker;`
`9`	`7`
`10`		`-@XposedHooker`
`11`	`8`	`public class AttachHooker implements XposedInterface.Hooker {`
`12`	`9`
`13`		`- @AfterInvocation`
`14`		`- public static void afterHookedMethod(XposedInterface.AfterHookCallback callback) {`
	`10`	`+ public static void after(XposedInterface.AfterHookCallback callback) {`
`15`	`11`	`XposedInit.loadModules((ActivityThread) callback.getThisObject());`
`16`	`12`	`}`
`17`	`13`	`}`
Original file line number	Diff line number	Diff line change
`@@ -12,9 +12,6 @@`
`12`	`12`
`13`	`13`	`import de.robv.android.xposed.XposedBridge;`
`14`	`14`	`import io.github.libxposed.api.XposedInterface;`
`15`		`-import io.github.libxposed.api.annotations.AfterInvocation;`
`16`		`-import io.github.libxposed.api.annotations.BeforeInvocation;`
`17`		`-import io.github.libxposed.api.annotations.XposedHooker;`
`18`	`15`	`import io.github.libxposed.api.errors.HookFailedError;`
`19`	`16`
`20`	`17`	`public class LSPosedBridge {`
`@@ -218,16 +215,14 @@ public static void dummyCallback() {`
`218`	`215`	`throw new IllegalArgumentException("Cannot hook Method.invoke");`
`219`	`216`	`} else if (hooker == null) {`
`220`	`217`	`throw new IllegalArgumentException("hooker should not be null!");`
`221`		`- } else if (hooker.getAnnotation(XposedHooker.class) == null) {`
`222`		`- throw new IllegalArgumentException("Hooker should be annotated with @XposedHooker");`
`223`	`218`	`}`
`224`	`219`
`225`	`220`	`Method beforeInvocation = null, afterInvocation = null;`
`226`	`221`	`var modifiers = Modifier.PUBLIC \| Modifier.STATIC;`
`227`	`222`	`for (var method : hooker.getDeclaredMethods()) {`
`228`		`- if (method.getAnnotation(BeforeInvocation.class) != null) {`
	`223`	`+ if (method.getName().equals("before")) {`
`229`	`224`	`if (beforeInvocation != null) {`
`230`		`- throw new IllegalArgumentException("More than one method annotated with @BeforeInvocation");`
	`225`	`+ throw new IllegalArgumentException("More than one method named before");`
`231`	`226`	`}`
`232`	`227`	`boolean valid = (method.getModifiers() & modifiers) == modifiers;`
`233`	`228`	`var params = method.getParameterTypes();`
`@@ -237,13 +232,12 @@ public static void dummyCallback() {`
`237`	`232`	`valid = false;`
`238`	`233`	`}`
`239`	`234`	`if (!valid) {`
`240`		`- throw new IllegalArgumentException("BeforeInvocation method format is invalid");`
	`235`	`+ throw new IllegalArgumentException("before method format is invalid");`
`241`	`236`	`}`
`242`	`237`	`beforeInvocation = method;`
`243`		`- }`
`244`		`- if (method.getAnnotation(AfterInvocation.class) != null) {`
	`238`	`+ } else if (method.getName().equals("after")) {`
`245`	`239`	`if (afterInvocation != null) {`
`246`		`- throw new IllegalArgumentException("More than one method annotated with @AfterInvocation");`
	`240`	`+ throw new IllegalArgumentException("More than one method named after");`
`247`	`241`	`}`
`248`	`242`	`boolean valid = (method.getModifiers() & modifiers) == modifiers;`
`249`	`243`	`valid &= method.getReturnType().equals(void.class);`
`@@ -254,13 +248,13 @@ public static void dummyCallback() {`
`254`	`248`	`valid = false;`
`255`	`249`	`}`
`256`	`250`	`if (!valid) {`
`257`		`- throw new IllegalArgumentException("AfterInvocation method format is invalid");`
	`251`	`+ throw new IllegalArgumentException("after method format is invalid");`
`258`	`252`	`}`
`259`	`253`	`afterInvocation = method;`
`260`	`254`	`}`
`261`	`255`	`}`
`262`	`256`	`if (beforeInvocation == null && afterInvocation == null) {`
`263`		`- throw new IllegalArgumentException("No method annotated with @BeforeInvocation or @AfterInvocation");`
	`257`	`+ throw new IllegalArgumentException("No method named before or after found in " + hooker.getName());`
`264`	`258`	`}`
`265`	`259`	`try {`
`266`	`260`	`if (beforeInvocation == null) {`
`@@ -271,7 +265,7 @@ public static void dummyCallback() {`
`271`	`265`	`var ret = beforeInvocation.getReturnType();`
`272`	`266`	`var params = afterInvocation.getParameterTypes();`
`273`	`267`	`if (ret != void.class && params.length == 2 && !ret.equals(params[1])) {`
`274`		`- throw new IllegalArgumentException("BeforeInvocation and AfterInvocation method format is invalid");`
	`268`	`+ throw new IllegalArgumentException("before and after method format is invalid");`
`275`	`269`	`}`
`276`	`270`	`}`
`277`	`271`	`} catch (NoSuchMethodException e) {`