[Plugin] astrbot_plugin_inaturalist_search

[CecilyGao]

插件信息

{
  "name": "astrbot_plugin_inaturalist_search",
  "display_name": "iNaturalist自然观察数据查询插件",
  "desc": "一个基于 iNaturalist API 的 AstrBot 插件，支持物种分类信息查询和观察记录搜索。",
  "author": "CecilyGao",
  "repo": "https://github.com/CecilyGao/astrbot_plugin_inaturalist_search",
  "tags": [
    "自然",
    "生物",
    "生态",
    "照片",
    "观察",
    "科学",
    "记录",
    "数据库"
  ],
  "social_link": "https://github.com/CecilyGao"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_inaturalist_search

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

main.py

1. 日志模块使用不符合框架要求（高优先级）

   • 代码同时使用了：
     • import logging + logging.getLogger(...)
     • from astrbot.core import logger
   • 但规范要求 logger 必须且只能从 astrbot.api 导入。
   • 这会导致日志体系不统一，且违反你给的框架适配硬性约束。

2. HTML 模板中大量 CSS 属性值缺失，渲染结果存在明显异常风险（高优先级）

   • TAXON_TEMPLATE / OBSERVATIONS_TEMPLATE 里多处样式是类似：
     • background-color:
     • color:
     • border: 1px solid
   • 这些是不完整 CSS，可能导致样式失效、布局错乱，甚至影响图片渲染服务稳定性（输出图不可读）。

3. ina observations [数量] 未做上限控制，存在滥用与性能风险（高优先级）

   • 用户输入数字后直接作为 per_page 请求参数。
   • 极端值会导致：
     • 对外部 API 产生高负载请求；
     • 本地组装转发消息节点过多，消息发送压力陡增；
     • 在群聊中造成刷屏和延迟。
   • 建议限制范围（如 1~20）并做非法值兜底。

4. int(bot_id) 存在潜在崩溃点（中优先级）

   • 在 _handle_observations 和 get_inaturalist_observations_tool 中，bot_id = event.get_self_id() or "1000000" 后直接 int(bot_id)。
   • 若某些适配器返回非纯数字 ID（或异常值），会抛 ValueError，导致整个处理流程中断。
   • 该处无局部异常保护。

5. 每次请求都新建 aiohttp.ClientSession，异步环境下连接复用不足（中优先级）

   • search_taxon / search_observations 每次调用都 ClientSession()。
   • 高频调用场景会增加连接建立成本，影响吞吐和响应时延。
   • 更推荐在插件生命周期内复用 session（并在卸载/退出时关闭）。

6. 存在明显重复逻辑，维护成本偏高（中优先级）

   • _handle_observations 与 get_inaturalist_observations_tool 在“样本提取 + 转发节点构建”几乎完全重复。
   • _handle_taxon 与 get_inaturalist_taxon_tool 也有大量重复。
   • 同一逻辑多处拷贝后续容易出现“一个修了另一个漏改”的一致性问题。

7. 配置项 inat_user / inat_password 读取后未使用（低优先级）

   • 当前实现未进行鉴权调用，也未消费这两个配置。
   • 容易误导维护者，以为插件有认证链路；同时会引入“存了但没用”的敏感信息管理负担。

8. render_observations_info 中写入了未使用字段 date（低优先级）

   • 循环里给每个 sample 增加 s['date']，但模板读取的是 sample.observed_on。
   • 属于无效代码，增加理解噪音，并且对入参产生不必要的原地修改。

---

代码评审团

人物一：Linus Torvalds（直接型、工程洁癖）
“你这功能是能跑，但日志系统两套混着来是典型‘先跑起来再说’风格。基础设施不统一，后面排障会很痛苦。先把 logger 规范统一，再谈功能优化。”

人物二：Steve Jobs（产品体验导向）
“模板里一堆 color: 空值就像 UI 设计稿里写‘这里先空着’然后直接上线。用户看到的是结果，不是解释。先保证渲染稳定和可读，再追求花哨样式。”

人物三：Deadpool（嘴碎但抓重点）
“ina obs 99999 猫？你这不是查询，是在邀请大家一起打爆 API。加个 limit 上限吧，顺便把重复代码抽函数，不然你未来的自己会追着现在的自己打。”

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

---

🔍 审核摘要

统计信息

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

main.py

---

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

[astrbot-plugin-copybara]

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

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