fz-mirrors/AstrBot
1
0
Fork 0
mirror of https://github.com/AstrBotDevs/AstrBot synced 2026-07-15 17:30:13 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix: address review feedback - add plugin naming pattern check and edge case handling

Browse Source 
Address feedback from Sourcery AI and Gemini Code Assist:
- Only apply fallback path to known plugin naming patterns (astrbot_plugin_*)
- Add defensive check for None/empty tool.__module__
- Fallback to tool.__module__ if _parts is empty
This commit is contained in:
MUHAMED FAZAL PS
2026-06-08 17:27:00 +05:30
parent 736bc93b2a
commit 992aea9869
1 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
astrbot/core/star/context.py
View File

@@ -491,7 +491,7 @@ class Context:
for tool in tools:
if not module_path:
_parts = []
module_part = tool.__module__.split(".")
module_part = tool.__module__.split(".") if tool.__module__ else []
flags = ["builtin_stars", "plugins"]
found_flag = False
for i, part in enumerate(module_part):
@@ -501,11 +501,13 @@ class Context:
_parts.append("main")
found_flag = True
break
if not found_flag:
if not found_flag and module_part:
# Subdirectory tool: construct path matching star_manager format
# Only apply to known plugin naming patterns (astrbot_plugin_*)
plugin_name = module_part[0]
_parts = ["data", "plugins", plugin_name, "main"]
tool.handler_module_path = ".".join(_parts)
if plugin_name.startswith("astrbot_plugin_"):
_parts = ["data", "plugins", plugin_name, "main"]
tool.handler_module_path = ".".join(_parts) if _parts else tool.__module__ or ""
module_path = tool.handler_module_path
else:
tool.handler_module_path = module_path