[Plugin] astrbot_plugin_warframe_helper

[moemoli]

插件信息

{
  "name": "astrbot_plugin_warframe_helper",
  "display_name": "Warframe助手",
  "desc": "国际服Warframe游戏信息查询",
  "author": "moemoli",
  "repo": "https://github.com/moemoli/astrbot_plugin_warframe_helper",
  "tags": ["warframe","星际战甲"],
  "social_link": ""
}

插件检查清单

• 我的插件经过完整的测试
• 我的插件不包含恶意代码
• 我已阅读并同意遵守该项目的 行为准则。

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_warframe_helper

您好！我已经对你提交的插件代码进行了初步自动化审核，作为初步参考:

main.py

1. 可维护性问题：单类职责过重（God Object）
   WarframeHelperPlugin 同时承担了配置解析、依赖装配、命令路由、QQ webhook 特化逻辑、结果提取与发送策略，方法数量和分支非常多，后续改动风险高（容易“牵一发动全身”）。

2. 异常处理过于宽泛，掩盖真实错误
   多处 except Exception: pass/return（如初始化配置、should_call_llm、多处发送逻辑）会吞掉关键上下文，线上问题难排查。

3. 重复逻辑较多
   大量命令方法都在重复“调用命令 -> QQ markdown fallback -> yield result”流程，重复代码显著增加维护成本和出错概率。

4. wf_help 中参数分支无实际作用
   sub = ... 后 if not sub or sub in {...}: pass，该分支不改变行为，属于无效代码，增加阅读噪音。

---

clients/drop_data_client.py

1. 数据持久化目录不符合规范约束
   使用 get_astrbot_plugin_data_path() 并手工拼接 "astrbot_plugin_warframe_helper/drop_data"，未采用要求的 StarTools.get_data_dir()。这会增加路径耦合与迁移风险。

2. 缓存容器无清理策略，存在内存增长风险
   _mem_relic_detail 按 key 累积，只有 TTL 校验、无主动淘汰，长运行下可能持续膨胀。

3. 异步流程中大量同步磁盘读写 + 宽泛异常吞噬
   read_text/write_text 与 except Exception: pass 组合，会让缓存损坏、权限问题等长期隐形化，排障困难。

---

clients/market_client.py

1. 缓存无上限，潜在内存泄漏
   _orders_cache、_riven_cache 只按 TTL 命中判断，不做容量控制/清理。高基数查询场景下会持续增长。

2. 异常恢复策略信息不足
   网络错误统一返回空列表，调用方难区分“真无数据”与“请求失败”，会导致用户侧误判。

---

clients/public_export_client.py

1. 数据持久化目录不符合规范约束
   与 drop_data_client.py 同样问题：未使用 StarTools.get_data_dir()，而是手工固定子路径。

2. 内存缓存结构可能无限增长
   _mem_exports、_mem_unique_name_maps、_mem_localized_to_en 等无容量约束，长期运行有内存压力风险。

3. search_mod 中条件分支是无效逻辑
   if any(...): out.append(r) else: out.append(r) 两个分支完全一致，疑似遗漏过滤逻辑或历史残留代码。

---

clients/worldstate_client.py

1. platform 参数实际未生效（功能正确性）
   _get_worldstate 虽接收并缓存分平台 key，但请求 URL 不随 platform 变化，等价于所有平台读同一份数据，可能与接口预期不一致。

2. 周期计算默认 epoch=0，结果可能系统性偏差
   地球/希图斯/魔胎/奥布/双衍周期依赖环境变量 epoch，默认 0 时大概率与真实轮转不对齐，属于高影响逻辑风险。

3. 告警型日志可能造成日志污染
   fetch_arbitration 每次调用都 logger.warning(...)，在高频命令下会产生大量重复噪声。

---

components/event_ttl_cache.py

1. 缓存仅“读时清理”，无定期淘汰机制
   _data 会随 key 增长；若很多 key 不再访问，过期数据不会自动回收，存在内存累积风险。

2. 过度宽泛异常吞噬
   put/get 的异常处理直接 return，会隐藏 key 生成或状态序列化问题。

---

components/qq_official_webhook.py

1. 逻辑错误：send_markdown_notice_interaction 误依赖 keyboard 模板
   该方法是“notice”发送，却使用 if not self._keyboard_template_id: return 作为前置条件；即使 markdown 模板可用，也会被错误短路。

2. 交互路径存在“静默失败”风险
   多个 _for_interaction 发送函数要求 reply_to_msg_id 必填，否则直接返回；调用链中若未传入，会无提示失败，用户体验差且难排查。

3. 交互 hook 侵入性高，副作用风险
   _maybe_hook_interactions 直接覆写 bot.on_interaction_create，并在非分页交互也继续调用自定义 handler，可能影响其他插件/模块的交互处理语义。

4. 重复代码严重
   多个发送方法重复构建 route/payload 和消息源分支，维护成本高，后续修 bug 易出现“改一处漏多处”。

---

handlers/qq_interaction.py

1. 翻页状态更新时机不合理
   在确认“是否有更多结果”之前就先把 page += 1 回写缓存；当 top 为空时缓存页码已前移，导致状态漂移（连续点击 next 会越来越偏）。

2. 失败路径缺乏回滚
   网络失败/渲染失败后不恢复旧页码，用户后续翻页体验可能异常。

---

handlers/wm_pick.py

1. 业务关联校验不足
   仅校验“消息里有 Reply + 有缓存”，未校验 Reply 目标是否确为对应 /wm 结果消息，可能误触发旧缓存数据。

---

代码评审团

人物一：Linus Torvalds（直接、硬核）
“main.py 这不是插件类，这是‘瑞士军刀航母’。功能能跑不代表结构健康。把 QQ 发送策略、命令分发、结果解析拆开，否则每次改一个命令都像在内核热补丁。”

人物二：Steve Jobs（产品体验导向）
“用户点击‘下一页’却静默没反应，这是体验灾难。qq_official_webhook.py 的交互发送链路需要明确失败反馈，不要让用户猜发生了什么。”

人物三：Geralt of Rivia（冷静、实战派）
“怪物不在明处，而在缓存里：多处 map 缓存无上限、只读时清理。短战没事，长战必出事。给缓存加容量和淘汰策略，能少很多‘诡异崩溃’。”

Disclaimer: 以上评审内容由 AI 自动生成，所涉及人物形象与现实无关，不代表真实人物观点。如果给出的建议无关痛痒请忽略。

---

🔍 审核摘要

统计信息

• 仓库文件总数: 38 个 Python 文件
• 已选择审核文件: 15 / 38
  已发送至 AI 审核的文件清单

main.py
constants.py
helpers.py
http_utils.py
clients/__init__.py
clients/drop_data_client.py
clients/market_client.py
clients/public_export_client.py
clients/worldstate_client.py
components/__init__.py
components/event_ttl_cache.py
components/qq_official_webhook.py
handlers/__init__.py
handlers/qq_interaction.py
handlers/wm_pick.py

注意：由于项目规模或Token限制，本次仅审核了部分优先文件。以上报告内容由 AI 直接生成。

---

此报告由AI自动生成，旨在提供初步反馈和改进建议，不能完全替代人工审核。最终决策以社区维护者的人工审核为准。目前自动审核（仓库地址）处于试验阶段，如遇问题请向维护者反馈。

[astrbot-plugin-copybara]

插件信息已同步到集合仓库。
PR: AstrBotDevs/AstrBot_Plugins_Collection#266
已自动合并 PR #266

该插件已上架。
This plugin is now published.