[Plugin] astrbot_plugin_HonorOfKings

[i12cu4]

插件信息

{
  "name": "astrbot_plugin_HonorOfKings",
  "display_name": "温雅的农药查询工具",
  "desc": "查询王者荣耀账号的近期战绩/实时上榜战力/账号资料等功能",
  "author": "温雅",
  "repo": "https://github.com/i12cu4/astrbot_plugin_HonorOfKings",
  "tags": [
    "王者荣耀",
    "查询工具"
  ],
  "social_link": "https://github.com/i12cu4"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_HonorOfKings

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

main.py

1. 潜在 AttributeError（配置读取链式 .get）
   self.conf.get("prefix").get("enable")、self.conf.get("comment").get("enable") 等写法在 prefix/comment 配置缺失时会对 None 调用 .get()，导致插件初始化直接异常。

2. 数据目录名与注册名不一致，易造成维护混乱
   @register 名称是 astrbot_plugin_honorofkings，但 StarTools.get_data_dir("astrbot_plugin_gok") 使用了不同目录键，后续迁移、排障、用户备份时容易错位。

3. 生命周期属性未预先定义，terminate() 存在属性不存在风险
   self.gokfun、self.sql_db 在 __init__ 中未初始化为 None；若 initialize() 中途失败，terminate() 访问它们可能触发 AttributeError。

4. _call_with_auto_args 参数转换失败时可能传入 inspect._empty
   类型转换失败分支中直接 call_args.append(p.default)；若参数本身无默认值，会把 inspect._empty 传给 handler，产生隐蔽运行时错误（而非明确报“参数错误”）。

5. 消息发送封装函数异常捕获范围不完整
   plain_msg / T2I_image_msg / image_msg / T2I_image_and_plain_msg 都在 try 之前先执行 data = await action()；若 action 抛异常，将绕过统一错误回复逻辑。

6. 异常类型未导入：ProviderNotFoundError
except ProviderNotFoundError 但文件未导入该符号，触发该分支时会因名称未定义导致二次异常。

7. 未使用导入项影响可读性
   MessageEventResult、MessageChain 未使用，增加噪音。

---

init.py

（未发现可报告问题）

---

fun_basic.py

1. extract_fields 过度吞异常，且静默失败
   except Exception 后直接返回 []，无日志、无错误上下文，排障困难，且会掩盖真实数据问题。

2. 无效异常变量
   except Exception as e: 中 e 未使用，代码味道不佳。

3. 类型标注缺失
   extract_fields 形参/返回值未标注，降低静态检查与维护效率（在 Python 3.10 项目中尤其可改进）。

---

gok_data.py

1. 敏感信息泄露风险（令牌写入日志）
   logger.debug(f"获取应天API令牌成功。{self.ytapi_token}")、nyapi_token 同理，直接输出 token，不符合安全最佳实践。

2. ID 判定阈值不一致导致逻辑偏差
   select() 用 >=100000000，get_gokid() 用 >=10000000；同一输入在不同功能里可能被不同规则处理，出现“查得到/查不到”不一致。

3. 异常捕获范围过宽且写法冗余
   多处 except (ValueError, TypeError, Exception)，Exception 已覆盖前两者；过宽捕获导致真实异常被误判为业务分支。

4. 数据库操作异常处理不完整
   add/update/delete/get_gokid 仅捕获 FileNotFoundError，但真实数据库异常通常是 IntegrityError/OperationalError 等，当前会直接抛出到上层。

5. fmt_num 使用裸 except
except: 会吞掉所有异常（含非预期系统异常），影响可诊断性与健壮性。

6. 未使用导入项较多，降低可维护性
   例如 datetime（顶部）、base64、Union/List、load_template 等未使用。

---

request.py

1. 默认关闭 SSL 校验（高风险）
   ssl_verify=False 默认值会让 HTTPS 请求在默认配置下不校验证书，存在中间人攻击风险。

2. 日志可能泄漏敏感请求参数
   _request 中 debug 记录完整 params/json_data，若包含 key/apikey/token 将在日志暴露。

3. 异步取消信号可能被吞掉
   _request 使用 except Exception 直接返回 None，会误吞 CancelledError，破坏异步任务取消语义。

4. 分页日志页码有偏移
   current_page += 1 后再打印“已获取第 X 页”，日志与实际已完成页不一致。

5. 未使用导入项
   Union 未使用。

---

sqlite.py

1. SQL 注入面：表名/条件字符串直接拼接
   INSERT INTO {table}、WHERE {where} 依赖字符串格式化，若调用方传入非受控内容存在注入风险（参数化仅覆盖 values，未覆盖标识符/子句）。

2. 连接状态未校验
   execute/fetch_* 默认假设 self.conn 已初始化；若未先 connect()，会触发 NoneType 异常，缺少明确错误提示。

3. close() 后未清理连接引用
   关闭后未将 self.conn = None，后续状态判断与复用语义不清晰，易引发误用。

4. 类型标注与实际使用不一致
   db_path: str，但上层传入 Path；虽通常可运行，但静态类型与接口契约不一致。

---

代码评审团

人物一：Linus Torvalds（直球型）
“你这插件最像‘能跑就行’阶段：except Exception 到处兜底，等于把错误信息扔垃圾桶。建议先把异常分层：网络、参数、数据库分别处理，别让所有错误都变成‘参数错误或执行失败’这种黑盒提示。”

人物二：Steve Jobs（产品洁癖型）
“用户体验会被‘偶发崩溃+模糊提示’杀死。action() 异常没统一兜住、配置项缺失直接炸、ID 规则前后不一致——这些都不是功能问题，是体验一致性问题。把输入规则和错误反馈做成统一标准，产品质感会立刻提升。”

人物三：福尔摩斯 Sherlock Holmes（推理型）
“我观察到两条关键线索：其一，日志里出现 token；其二，HTTP 关闭证书校验。两者叠加意味着：即使业务逻辑正确，系统仍暴露在‘可观测性泄密 + 链路劫持’双重风险中。先修安全基线，再谈功能迭代。”

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

---

🔍 审核摘要

统计信息

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

main.py
__init__.py
fun_basic.py
gok_data.py
request.py
sqlite.py

---

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

当前仓库：i12cu4/astrbot_plugin_HonorOfKings
剩余触发次数：4

[astrbot-plugin-copybara]

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

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