fix #4097

Author: lgc2333

astrbot/core/star/context.py

@@ -368,25 +368,10 @@ async def send_message(
     def add_llm_tools(self, *tools: FunctionTool) -> None:
         """添加 LLM 工具。"""
         tool_name = {tool.name for tool in self.provider_manager.llm_tools.func_list}
-        module_path = ""
         for tool in tools:
-            if not module_path:
-                _parts = []
-                module_part = tool.__module__.split(".")
-                flags = ["packages", "plugins"]
-                for i, part in enumerate(module_part):
-                    _parts.append(part)
-                    if part in flags and i + 1 < len(module_part):
-                        _parts.append(module_part[i + 1])
-                        break
-                tool.handler_module_path = ".".join(_parts)
-                module_path = tool.handler_module_path
-            else:
-                tool.handler_module_path = module_path
-            logger.info(
-                f"plugin(module_path {module_path}) added LLM tool: {tool.name}"
-            )
-
+            # Set `handler_module_path` moved to astrbot/core/star/star_manager.py,
+            # search `ft.__module__ == metadata.module_path`
+            logger.info(f"plugin added LLM tool: {tool.name} ({tool.__module__})")
             if tool.name in tool_name:
                 logger.warning("替换已存在的 LLM 工具: " + tool.name)
                 self.provider_manager.llm_tools.remove_func(tool.name)

astrbot/core/star/star_manager.py

@@ -515,15 +515,15 @@ async def load(self, specified_module_path=None, specified_dir_name=None):
                             need_apply = [func_tool]
 
                         for ft in need_apply:
-                            if (
-                                ft.handler
-                                and ft.handler.__module__ == metadata.module_path
-                            ):
+                            if ft.handler:
+                                if ft.handler.__module__ == metadata.module_path:
+                                    ft.handler_module_path = metadata.module_path
+                                    ft.handler = functools.partial(
+                                        ft.handler,
+                                        metadata.star_cls,  # type: ignore
+                                    )
+                            elif ft.__module__ == metadata.module_path:
                                 ft.handler_module_path = metadata.module_path
-                                ft.handler = functools.partial(
-                                    ft.handler,
-                                    metadata.star_cls,  # type: ignore
-                                )
                             if ft.name in inactivated_llm_tools:
                                 ft.active = False