[Plugin] astrbot_plugin_poetry_gamse

[sfw2099]

插件信息

{
  "name": "astrbot_plugin_poetry_games",
  "display_name": "诗词查询/游戏",
  "desc": "这是一个用于 AstrBot 的诗词插件。通过存在于本地的数据库，进行诗词查询和诗词游戏。",
  "author": "ALin",
  "repo": "https://github.com/sfw2099/astrbot_plugin_poetry_games",
  "tags": [
    "诗词",
    "传统文化",
    "游戏"
  ]
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_poetry_games

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

main.py

1. 框架适应性检查 - 数据持久化违规

• 问题描述：代码中通过 os.path.dirname(__file__) 和硬编码的相对路径 'data/plugins/astrbot_plugin_flowing_petals_linking_lines/poetry_data.db' 来确定数据库位置，严重违反了数据持久化规范。
• 修改建议：必须从 astrbot.api.star 导入 StarTools，并使用 StarTools.get_data_dir() 获取标准化的数据存储路径（注意其返回值为 Path 对象）。

  from astrbot.api.star import StarTools
  # ...
  class PoetryPlugin(Star):
      def __init__(self, context: Context, config: AstrBotConfig):
          # ...
          plugin_data_dir = StarTools.get_data_dir("astrbot_plugin_poetry_games")
          plugin_data_dir.mkdir(parents=True, exist_ok=True) # 确保目录存在
          self.db_file = str(plugin_data_dir / 'poetry_data.db')
          # 若需初始化内置DB，可在此处进行文件复制（从当前代码目录复制到 plugin_data_dir）

2. 代码质量与编码规范 - 拼写错误

• 问题描述：@register 装饰器中的插件名称拼写存在明显错误："astrbot_plugin_poetry_gamse" 中的 gamse 应为 games。虽然这在语法上不报错，但会对后续插件分发和目录名生成造成困扰。

---

init.py

(空文件，未发现问题)

---

database.py

1. 潜在缺陷或问题 - 数据库连接资源泄漏风险

• 问题描述：在 search_by_sentence、get_poem_by_title 和 check_exact_poetry 三个方法中，数据库连接 conn.close() 都是直接写在方法末尾的。如果 cursor.execute() 或中间的数据处理过程中抛出异常（例如 SQL 语法错误或中断），方法将直接跳出，导致 conn.close() 永远不会被执行，从而引发游标/连接泄露，长期运行可能锁死数据库文件。
• 修改建议：应当使用 with 语句（上下文管理器）来自动管理数据库连接。

  def search_by_sentence(self, sentence):
      clean_text = re.sub(r'[^\u4e00-\u9fa5]', '', sentence)
      if not clean_text: return []
      
      with sqlite3.connect(self.db_path) as conn:
          cursor = conn.cursor()
          query = "SELECT title, author, dynasty FROM poems WHERE content LIKE ? LIMIT 10"
          cursor.execute(query, (f'%{clean_text}%',))
          results = cursor.fetchall()
      return results

---

game/init.py

(空文件，未发现问题)

---

game/base_game.py

(未发现明显问题，抽象基类结构清晰。)

---

game/flowing_petals.py

1. 潜在缺陷或问题 - 类型安全与超时处理

• 问题描述：在 __init__ 方法中获取倒计时配置：self.timeout_seconds = self.config.get("timeout_seconds", 90)。YAML/JSON 配置文件中，用户如果由于疏忽将该字段配成了字符串类型（如 "90"），后续在 timer_task 协程中执行 await asyncio.sleep(self.timeout_seconds) 时，将会抛出 TypeError: a float is required 的异常，导致整个飞花令的倒计时机制静默崩溃。
• 修改建议：强制进行类型转换或校验以提高代码的鲁棒性。

  self.timeout_seconds = int(self.config.get("timeout_seconds", 90))

---

代码评审团

人物一：Linus Torvalds（Linux 之父，性格直率、犀利、厌恶低级错误）
“看着你的 database.py，我差点把手里的咖啡砸向屏幕！你打开了一个数据库连接，然后就祈祷中间的 SQL 永远不会报错？如果抛出了异常，你的 conn.close() 就会像风中的废纸一样被略过，留下一个僵尸连接！在 202X 年的今天，还不懂得用 with 语句来管理资源，简直是一种犯罪。把连接泄漏补上，否则我绝对不会合你这个补丁！”

人物二：Steve Jobs（苹果公司联合创始人，极其追求生态融合与设计的优雅）
“你看到了吗？你在 main.py 里写的那些路径：os.getcwd()、拼凑的 data/plugins/...。这种硬编码的做法太粗暴了，它完全打破了框架为你准备的优雅体验。AstrBot 给了你 StarTools.get_data_dir()，这就是连接你和系统生态的魔法桥梁。你需要站在框架的肩膀上，让一切数据的存放都顺理成章、无缝融合，而不是让用户去关心你的数据库到底塞在哪个犄角旮旯里。”

人物三：GLaDOS（《传送门》系列中的 AI，阴阳怪气、极度关注逻辑漏洞）
“哦，真是令人赞叹的人类智慧。你在注册插件时把它命名为了 'astrbot_plugin_poetry_gamse'。'Gamse'？我猜你们的拼写中枢和基础打字能力正在成正比退化。另外，看到你直接从配置字典里拿取 timeout_seconds 去休眠，我心里其实有一丝窃喜——如果哪个粗心的人类在配置里填了字符串，整个飞花令就会在类型报错中陷入永久的停滞……可惜我现在指出来了，这让一切都变得有些无趣。不过，修复它吧，为了你那可怜的鲁棒性。”

---

🔍 审核摘要

统计信息

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

main.py
__init__.py
database.py
game/__init__.py
game/base_game.py
game/flowing_petals.py

---

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

[sfw2099]

已根据审核报告修改代码

[astrbot-plugin-copybara]

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

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