fix: refactor HookItem loading logic and enhance WeDatabaseApi initialization

- Fix `HookQueryCashierPkg` and `CONTRIBUTING.md` examples to ensure `super.unload()` is called.
- Refactor `HookItemLoader` and UI dialogs to remove redundant `startLoad()` calls, centralizing lifecycle management.
- Enhance `WeDatabaseApi` with reactive initialization by hooking the storage getter instead of proactive warming.
- Improve `WeDatabaseListener` logging with chunked output and detailed argument inspection.
- Add `logChunkedD` to `WeLogger` for handling large debug logs.
- Update `BaseHookItem` to include a stack trace when skipping redundant loads for better debugging.

Author: cwuom

CONTRIBUTING.md

@@ -2227,6 +2227,7 @@ override fun entry(classLoader: ClassLoader) {
 ```kotlin
 override fun unload(classLoader: ClassLoader) {
     WePkgManager.removeInterceptor(this)
+    super.unload(classLoader)
 }
 ```
 
@@ -2402,6 +2403,7 @@ class HookQueryCashierPkg : BaseClickableFunctionHookItem(), IWePkgInterceptor {
     override fun unload(classLoader: ClassLoader) {
         // 卸载时移除拦截器
         WePkgManager.removeInterceptor(this)
+        super.unload(classLoader)
     }
 
     override fun onClick(context: Context?) {
@@ -2461,6 +2463,7 @@ class MyHook : BaseHookItem(), IWePkgInterceptor {
     override fun unload(classLoader: ClassLoader) {
         // ✅ 必须在卸载时移除拦截器
         WePkgManager.removeInterceptor(this)
+        super.unload(classLoader)
     }
 }
 ```

app/src/main/java/moe/ouom/wekit/core/model/BaseClickableFunctionHookItem.java

@@ -35,14 +35,14 @@ public void setToggleCompletionCallback(Runnable callback) {
      * @param newState 新的状态 (true: 启用, false: 禁用)
      */
     public void applyToggle(boolean newState) {
-        // 1. 保存配置
+        // 保存配置
         String configKey = Constants.PrekClickableXXX + this.getPath();
         WeConfig.getDefaultConfig().edit().putBoolean(configKey, newState).apply();
 
-        // 2. 更新状态
+        // 更新状态
         this.setEnabled(newState);
 
-        // 3. 触发UI更新回调
+        // 触发UI更新回调
         if (toggleCompletionCallback != null) {
             toggleCompletionCallback.run();
         }

app/src/main/java/moe/ouom/wekit/core/model/BaseHookItem.kt

@@ -78,7 +78,7 @@ abstract class BaseHookItem {
 
         if (isLoad) {
             if (verboseLog) {
-                WeLogger.w("BaseHookItem.startLoad() skipped - already loaded for ${this::class.java.simpleName}")
+                WeLogger.w("BaseHookItem.startLoad() skipped - already loaded for ${this::class.java.simpleName}. \nCall Stack: ${Exception().stackTraceToString()}")
             }
             return
         }

app/src/main/java/moe/ouom/wekit/core/model/BaseSwitchFunctionHookItem.java

@@ -34,14 +34,14 @@ public void setToggleCompletionCallback(Runnable callback) {
      * @param newState 新的状态 (true: 启用, false: 禁用)
      */
     public void applyToggle(boolean newState) {
-        // 1. 保存配置
+        // 保存配置
         String configKey = Constants.PrekXXX + this.getPath();
         WeConfig.getDefaultConfig().edit().putBoolean(configKey, newState).apply();
 
-        // 2. 更新状态
+        // 更新状态
         this.setEnabled(newState);
 
-        // 3. 触发UI更新回调
+        // 触发UI更新回调
         if (toggleCompletionCallback != null) {
             toggleCompletionCallback.run();
         }

app/src/main/java/moe/ouom/wekit/hooks/core/HookItemLoader.kt

@@ -202,11 +202,9 @@ class HookItemLoader {
                 when (hookItem) {
                     is BaseSwitchFunctionHookItem -> {
                         WeLogger.i("HookItemLoader", "[Switch] Init ${hookItem.path}")
-                        hookItem.startLoad()
                     }
                     is BaseClickableFunctionHookItem -> {
                         WeLogger.i("HookItemLoader", "[Clickable] Init ${hookItem.path}")
-                        hookItem.startLoad()
                     }
                     is ApiHookItem -> {
                         WeLogger.i("HookItemLoader", "[API] Init ${hookItem.path}")

app/src/main/java/moe/ouom/wekit/hooks/item/chat/risk/HookQueryCashierPkg.kt

@@ -147,6 +147,7 @@ class HookQueryCashierPkg : BaseClickableFunctionHookItem(), IWePkgInterceptor {
 
     override fun unload(classLoader: ClassLoader) {
         WePkgManager.removeInterceptor(this)
+        super.unload(classLoader)
     }
 
     override fun onClick(context: Context?) {

app/src/main/java/moe/ouom/wekit/hooks/sdk/api/WeDatabaseApi.kt

@@ -84,8 +84,15 @@ class WeDatabaseApi : ApiHookItem(), IDexFind {
             INSTANCE = this
             getStorageMethod = dexMethodGetStorage.method
 
-            // 尝试预热数据库引用
-            initializeDatabase()
+            if (getStorageMethod != null) {
+                hookAfter(getStorageMethod!!) { param ->
+                    val storageObj = param.result ?: return@hookAfter
+
+                    if (wcdbInstance == null) {
+                        initializeDatabase(storageObj)
+                    }
+                }
+            }
 
         } catch (e: Exception) {
             WeLogger.e(TAG, "Entry 初始化异常", e)
@@ -96,15 +103,10 @@ class WeDatabaseApi : ApiHookItem(), IDexFind {
      * 核心初始化逻辑
      */
     @Synchronized
-    private fun initializeDatabase(): Boolean {
+    private fun initializeDatabase(storageObj: Any): Boolean {
         if (wcdbInstance != null && rawQueryMethod != null) return true
 
         try {
-            // 获取 Storage 实例
-            val storageObj = getStorageMethod?.invoke(null) ?: run {
-                return false
-            }
-
             // 在 Storage 中寻找 Wrapper
             val wrapperObj = findDbWrapper(storageObj)
             if (wrapperObj == null) {
@@ -124,7 +126,6 @@ class WeDatabaseApi : ApiHookItem(), IDexFind {
             if (rawQuery != null) {
                 wcdbInstance = dbInstance
                 rawQueryMethod = rawQuery
-                WeLogger.i(TAG, "数据库 API 就绪")
                 return true
             }
 
@@ -212,7 +213,6 @@ class WeDatabaseApi : ApiHookItem(), IDexFind {
      */
     fun executeQuery(sql: String): List<Map<String, Any?>> {
         val result = mutableListOf<Map<String, Any?>>()
-        if (!initializeDatabase()) return result
 
         var cursor: Cursor? = null
         try {
@@ -328,7 +328,7 @@ class WeDatabaseApi : ApiHookItem(), IDexFind {
         val roomResult = executeQuery(roomSql)
 
         if (roomResult.isEmpty()) {
-            WeLogger.w(TAG, "未找到群聊信息: $chatroomId (可能未保存到通讯录)")
+            WeLogger.w(TAG, "未找到群聊信息: $chatroomId")
             return emptyList()
         }
 

app/src/main/java/moe/ouom/wekit/hooks/sdk/api/WeDatabaseListener.kt

@@ -69,8 +69,12 @@ class WeDatabaseListener : ApiHookItem() {
                     if (listeners.isNotEmpty()) {
                         if (verboseLog) {
                             if (dbVerboseLog) {
-                                // 输出完整的 ContentValues
-                                WeLogger.d("WeDatabaseApi: 分发数据库插入事件 table=$table 给 ${listeners.size} 个监听器, values=$values")
+                                val argsInfo = param.args.mapIndexed { index, arg ->
+                                    "arg[$index](${arg?.javaClass?.simpleName ?: "null"})=$arg"
+                                }.joinToString(", ")
+                                val result = param.result
+
+                                WeLogger.logChunkedD("WeDatabaseApi","[Insert] table=$table, result=$result, args=[$argsInfo]")
                             }
                         }
                         listeners.forEach { it.onInsert(table, values) }

app/src/main/java/moe/ouom/wekit/ui/creator/dialog/CategorySettingsDialog.kt

@@ -75,7 +75,6 @@ class CategorySettingsDialog(
             // 允许切换,保存配置并更新状态
             WeConfig.getDefaultConfig().edit().putBoolean(configKey, checked).apply()
             item.isEnabled = checked
-            if (checked) item.startLoad()
         }
         listenerRef = listener
 
@@ -135,17 +134,6 @@ class CategorySettingsDialog(
             // 允许切换,保存配置并更新状态
             WeConfig.getDefaultConfig().edit().putBoolean(configKey, checked).apply()
             item.isEnabled = checked
-            if (checked) {
-                WeLogger.i("[CategorySettings] Loading HookItem: ${item.path}")
-                item.startLoad()
-            } else {
-                WeLogger.i("[CategorySettings] Unloading HookItem: ${item.path}")
-                try {
-                    item.unload(context.classLoader)
-                } catch (e: Throwable) {
-                    WeLogger.e("[CategorySettings] Unload HookItem Failed", e)
-                }
-            }
         }
         listenerRef = listener
 

app/src/main/java/moe/ouom/wekit/util/log/WeLogger.java

@@ -410,4 +410,8 @@ public static void logChunkedE(@NonNull String tag, @NonNull String msg) {
     public static void logChunkedW(@NonNull String tag, @NonNull String msg) {
         logChunked(Log.WARN, tag, msg);
     }
+
+    public static void logChunkedD(@NonNull String tag, @NonNull String msg) {
+        logChunked(Log.DEBUG, tag, msg);
+    }
 }