[Bug] turn_on_plugin 不在 AstrBot 重启路径中，inactivated_llm_tools 幽灵条目随重启累积

[irmia2026]

What happened / 发生了什么

turn_off_plugin 将工具名写入持久化黑名单 inactivated_llm_tools（SQLite preferences 表），但恢复路径仅有一条：在 WebUI 中手动点击「启用插件」按钮。AstrBot 正常重启（stop() → load()）、reload()、restart() 均不经过 turn_on_plugin，导致黑名单条目随每次停用操作不断累积，重启后无差别将所有工具设为关闭状态。

---

根因：写入和恢复路径不对称

下面是 inactivated_llm_tools 的完整数据流：

┌──────────────────────────────────────────────────┐
│ 写入（唯一路径）                                   │
│ star_manager.py:1712-1716 — turn_off_plugin       │
│   sp.global_put("inactivated_llm_tools", [...])   │
│   → 持久化到 SQLite preferences 表                 │
├──────────────────────────────────────────────────┤
│ 读取 + 应用                                        │
│ star_manager.py:891 — load()                      │
│   inactivated_llm_tools = sp.global_get(...)      │
│ star_manager.py:1091 — load()                     │
│   if ft.name in inactivated_llm_tools:            │
│       ft.active = False                           │
│   （只应用黑名单，从不清理）                         │
├──────────────────────────────────────────────────┤
│ 恢复（唯一路径）                                    │
│ dashboard/routes/plugin.py:1902                   │
│   → turn_on_plugin                               │
│   → 遍历 func_list，清理匹配的条目                  │
│   → 仅 WebUI 手动「启用插件」触发                   │
└──────────────────────────────────────────────────┘

关键缺口在 load()：它读取黑名单、应用黑名单，但从不清理那些"对应插件已启用"的条目。load() 在所有插件加载完成后应该检查：某个插件是启用的 → 它的工具不应该在黑名单里 → 清除对应条目。

此外，以下常见操作路径完全绕过了 turn_on_plugin：

操作	是否调 turn_on_plugin
AstrBot 重启（stop() → load()）	❌
reload()（_unbind_plugin → _terminate_plugin → load()）	❌
restart()	❌
WebUI「启用插件」按钮	✅（唯一）

---

累积效应

每次停用-重新启用循环如果是通过 AstrBot 重启（而非 WebUI 按钮）完成的，就会在 DB 中留下 N 个幽灵条目。多次累积后，inactivated_llm_tools 列表持续增长，load() 在 L1091 无差别应用——即使用户手动逐个打开过所有工具，重启后它们又全被关闭。

---

涉及文件

文件	行号	说明
astrbot/core/star/star_manager.py	870-1249	load()：读取并应用黑名单，但缺少末尾清理步骤
astrbot/core/star/star_manager.py	1679-1718	turn_off_plugin：唯一写入点
astrbot/core/star/star_manager.py	1767-1791	turn_on_plugin：唯一清理点（但不在重启路径中）
astrbot/dashboard/routes/plugin.py	1902	WebUI 触发 turn_on_plugin 的唯一入口

---

Expected behavior / 期望行为

load() 在所有插件加载完成后，遍历所有 activated=True 的插件已注册的工具，将匹配条目从 inactivated_llm_tools 中清除并恢复 active：

# load() 末尾，所有 star 加载完成后（仅 full reload 时执行）
# 遍历 star_registry 而非 star_map，因为前者按注册顺序保留
for star_meta in star_registry:
    if not star_meta.activated or not star_meta.module_path:
        continue
    plugin_path = star_meta.module_path
    for func_tool in llm_tools.func_list:
        mp = func_tool.handler_module_path
        if not mp:
            continue
        if mp.startswith(plugin_path):
            if func_tool.name in inactivated_llm_tools:
                inactivated_llm_tools.remove(func_tool.name)
                func_tool.active = True
await sp.global_put("inactivated_llm_tools", inactivated_llm_tools)

注意：此修复仅针对"因插件整体停用而写入黑名单的工具"。由于 inactivated_llm_tools 是扁平列表，不区分"因插件停用而关闭"和"用户单独手动关闭"，用户单独关闭的工具也会在重启后被恢复。这是 inactivated_llm_tools 架构层面的缺陷（turn_on_plugin 目前同样有此问题），彻底的修复需要将两套语义分开存储（例如新增 inactivated_plugins 作为独立判断依据），不在本 issue 范围内。

---

Reproduce / 如何复现？

1. 部署一个注册了函数工具的插件
2. WebUI 停用该插件 → turn_off_plugin 将 N 个工具名写入 DB
3. WebUI 重新启用插件 → 工具恢复正常（这一步走了 turn_on_plugin，清理成功）
4. 重启 AstrBot（不走 turn_on_plugin）→ load() 从 DB 读取黑名单 → 工具被设为 active=False
5. 用户手动在「函数工具管理」中重新打开工具
6. 再次重启 AstrBot → load() 再次读取 → 工具再次被关闭

黑名单条目不会被任何重启路径清理，只能通过 WebUI 启用插件来清除。

---

AstrBot version, deployment method (e.g., Windows Docker Desktop deployment), provider used, and messaging platform used. / AstrBot 版本、部署方式（如 Windows Docker Desktop 部署）、使用的提供商、使用的消息平台适配器

项	值
AstrBot 版本	v4.24.2
Python 版本	3.12.12
操作系统	Windows 11

OS

Windows

Logs / 报错日志

无显式报错。

Are you willing to submit a PR? / 你愿意提交 PR 吗？

• Yes!

Code of Conduct

• I have read and agree to abide by the project's Code of Conduct。

[irmia2026]

@dosubot 帮忙分析一下这个 bug，看看根因分析和修复建议是否正确。

[dosubot]

你的根因分析完全正确 ✅，代码验证了完整的数据流。

---

问题确认：

1. 写入路径：turn_off_plugin 将工具名追加到 inactivated_llm_tools 并持久化到 SQLite [1]
2. 应用路径：load() 在启动时读取 inactivated_llm_tools 并对匹配工具设 active=False，但从不清理过期条目 [2] [3]
3. 恢复路径：turn_on_plugin 是唯一从黑名单中移除条目的代码路径 [4]，且仅由 WebUI /api/plugin/on 端点触发 [5]
4. 重启路径缺口：reload() 不调用 turn_on_plugin——实际上反过来，turn_on_plugin 内部调用 reload() [6]。因此重启/reload 操作完全绕过清理逻辑

---

关于修复建议：

你提出的在 load() 末尾对 activated=True 的插件清理黑名单条目的方案是合理的最小修复。需注意的点你也已指出：

• ⚠️ inactivated_llm_tools 是扁平列表，无法区分"因插件停用而关闭"和"用户单独手动关闭"这两种语义。你的修复会在重启时恢复用户单独关闭的工具——这与当前 turn_on_plugin 的行为一致，所以不算引入新问题，但确实是架构层面需后续解决的设计缺陷。

另外注意到你在 #8582 报告了一个密切相关的问题（停用插件保存配置时无条件 reload() 导致 func_list 被清空），两个 issue 的幽灵条目累积机制互相加剧——建议修复时一并考虑。

期待你的 PR 🎉

To reply, just mention @dosu.

---

Share context across your team and agents. Try Dosu.

[NayukiChiba]

你的这个issue是跟#8583, #8582 是连在一起的吗

[irmia2026]

@NayukiChiba 其实是四个 issue，分别对应不同的相邻 bug——#8578 是根因（handler_module_path 路径格式不一致导致 startswith 匹配不上），#8582 和 #8583 是途中发现的（reload 误清 func_list、startswith 方向写反），#8584 是更底层的结构问题（重启路径缺少黑名单清理）。合在一起是 inactivated_llm_tools 从写入到清理的完整链路缺陷。