[Plugin] astrbot_plugin_steamachievement_query

[Murokivot]

插件信息

{
  "name": "astrbot_plugin_steamachievement_query",
  "display_name": "SteamHunter成就查询",
  "desc": "快速查询Steam成就积分、成就数、积分排名等信息的简易插件。",
  "author": "Muroki",
  "repo": "https://github.com/Murokivot/astrbot_plugin_steamachievement_query",
  "tags": [
    "Steam"
  ]
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_steamachievement_query

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

main.py

1. 数据持久化目录不规范（框架适配问题）

   • self.cache_path = Path("/AstrBot/data/steam_achievement_cache.json") 使用了硬编码绝对路径。
   • 按要求，插件持久化数据应通过 StarTools.get_data_dir() 获取插件数据目录，避免部署环境路径不一致与权限问题。

2. 缓存逻辑存在明显功能错误

   • 在命中缓存后立即执行：

     if key in cache:
         del cache[key]
         self._save_cache(cache)

     这会导致每次查询都强制失效缓存，缓存机制实际被禁用。
   • self.cache_expire 定义了但从未参与判定，缓存过期策略未实现。

3. has_data 判定逻辑有误，容易误报“有数据”

   • 当前逻辑把 username 也纳入核心字段：

     core_fields = [data["points"], data["achievements"], data["games_played"], data["username"]]
     data["has_data"] = any(x not in ("0", "", "未知") for x in core_fields)

   • 只要用户名被解析到（通常都会有），即便统计值全是 0，也会被判定为有数据。

4. 时长解析 _parse_ymd_from_playtime 有逻辑缺陷

   • day 的 ¾ 分支里计算了 hours，但该变量未用于最终 total_hours，导致 0.75 天增量被丢弃。
   • 正则中的 .25/.5/.75 写法未转义点号，. 会匹配任意字符，存在误匹配风险。
   • 先计算 total_hours 后再归一化 months >= 12，显示年月天与小时值可能不一致。

5. Playtime 抓取正则过于宽泛，易抓错字段

   • playtime_match 在整页文本中搜索首个 years/months/days，很可能匹配到“最近更新 xx days ago”等非游戏时长文本，导致时长错误。

6. 用户名兜底解析逻辑不可靠

   • 兜底正则写死了 "3 hours ago"：

     re.search(r"(\w+)\s+3 hours ago", ...)

   • 仅在特定文案下有效，且对用户名字符集支持不足（\w+ 不支持很多昵称格式）。

7. HTTP 状态码处理不完整

   • 除了 404 + 非数字ID 的分支外，其它非 200 情况未显式处理，可能把错误页 HTML 当正常页面继续解析，导致误判。

8. 异常处理过宽，掩盖真实问题

   • _parse_update_time、_parse_country 使用裸 except:，会吞掉所有异常（包括编码/结构变化等关键问题），不利于排错和维护。

9. 异步网络性能可优化（潜在瓶颈）

   • _fetch_steam_data 每次调用都新建 aiohttp.ClientSession()，高频调用下会增加连接建立开销。
   • 在异步环境中更推荐复用会话（如插件生命周期内维护单例 session）。

10. 无效/未使用成员影响可维护性

    • self.steam_api_key 未使用。
    • cache_expire 未使用。
    • AstrMessageEvent 被导入但在函数签名中未使用类型标注。
      这会增加阅读负担、误导后续维护者。

---

代码评审团

人物一：Linus Torvalds（暴躁务实流）
“你这缓存代码是‘先缓存再删除’的反向工程艺术吗？缓存命中就删，等于没写。先把行为修正确保功能可信，再谈花里胡哨。建议：实现真正的 TTL 判断，命中则直接返回，不命中再请求。”

人物二：史蒂夫·乔布斯（产品体验流）
“用户只关心一件事：结果准不准。你现在的 Playtime 正则可能抓到‘几天前更新’，这会让输出看起来像真数据但其实错了，最伤信任。建议：只在明确的 Playtime DOM 节点内解析，不要全页盲扫。”

人物三：托尼·斯塔克（工程极客流）
“每次请求都 new 一个 ClientSession？这是拿性能当反应堆燃料烧。建议：把 session 提升为插件级资源，初始化一次，terminate 时关闭；并补全非 200 状态处理，失败就快速返回，别拿错误页做数据源。”

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

---

🔍 审核摘要

统计信息

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

main.py

---

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

当前仓库：Murokivot/astrbot_plugin_steamachievement_query
剩余触发次数：4