[Plugin] astrbot_plugin_CNInfoCrawler

[Interstellar1217]

插件信息

{
"name": "astrbot_plugin_CNInfoCrawler",
"display_name": "巨潮资讯上市公司公告自动爬取工具",
"version": "1.0.0",
"desc": "CNInfoHedgeCrawler 是基于巨潮资讯的 A 股上市公司公告爬取工具，支持任意关键词搜索、日期范围过滤、PDF 批量下载和企业微信推送。",
"author": "Interstellar1217",
"repo": "https://github.com/Interstellar1217/CNInfoHedgeCrawler",
"tags": ["套期保值", "公告查询", "巨潮资讯", "业绩预告", "金融"],
"social_link": "https://github.com/Interstellar1217"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_CNInfoCrawler

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

astrbot_plugin/main.py

1. 主类结构不符合 main.py 专项规范

   • 要求是存在继承 Star 的插件主类；当前为 CNInfoHedgePlugin(BasePlugin)，不符合指定框架约束。

2. 未采用 filter 事件装饰器体系

   • 文件中没有 from astrbot.api.event import filter，也没有基于 @filter.command(...) 的监听器定义。
   • 当前使用 register() 手动注册命令，与专项检查要求不一致。

3. 异步环境中存在同步网络 I/O 阻塞

   • async def search_handler(...) 内部调用同步的 search_announcements()。
   • search_announcements() -> _fetch_announcement_list() 使用 self.session.post(...)（同步网络请求），会阻塞事件循环。

4. 日志器来源不满足“必须且只能 from astrbot.api import logger”

   • 虽然尝试从 astrbot.api 导入，但 ImportError 分支里自定义了 logger 类，违反“只能使用 astrbot.api logger”的约束。

5. 资源释放风险

   • Session 创建后没有显式关闭逻辑（例如插件卸载时），存在连接资源长期占用风险。

6. 异常处理过宽

   • 多处 except Exception / 裸 except（如时间戳解析处）会掩盖具体错误，增加排障难度。

7. sys.path.insert 修改导入路径可维护性较差

   • 运行时修改 sys.path 增加模块解析不确定性，易引入环境相关问题。

---

dify_plugin/cninfo_hedge/main.py

1. 日志框架不符合约束

   • 使用 from loguru import logger，违反“logger 必须且只能来自 astrbot.api”的规则。

2. main.py 结构不符合专项规范

   • 无继承 Star 的主类。
   • 未导入 filter，无 @filter.xxx 事件监听器结构。

3. 异步环境中的同步网络 I/O 阻塞

   • Session.post(...) 为同步请求；在异步运行场景下会阻塞。
   • 并且有 time.sleep(...)（虽非网络 I/O，但会进一步阻塞执行线程）。

4. 参数边界未兜底

   • max_pages=int(tool_parameters.get("max_pages", 1)) 对非法输入会抛 ValueError，当前未单独处理。

5. 资源释放风险

   • CNInfoHedgeTool 内部 Session 未提供关闭/清理机制。

6. sys.path.insert 动态改路径

   • 增加部署环境耦合，降低可维护性与可移植性。

---

config.py

1. 持久化目录硬编码

   • DATA_DIR = "data"、LOGS_DIR = "logs" 属于硬编码目录；在 AstrBot 插件语境下应通过 StarTools.get_data_dir() 获取规范数据目录。

2. 类型标注不规范（Python 3.10）

   • page_size: int = None 应为 page_size: Optional[int] = None（或 int | None），否则与标注语义不一致。

---

crawler.py

1. 日志框架不符合约束

   • 使用 loguru，违反 logger 来源要求。

2. 异步环境下同步网络 I/O 阻塞严重

   • fetch_announcement_list() 中 self.session.post(...)。
   • download_pdf() 中 self.session.get(...)。
   • crawl_page() 内调用 send_to_wecom(...) 也会触发同步网络请求。
     以上都会阻塞异步执行环境。

3. 持久化路径硬编码

   • 直接使用 Path(config.DATA_DIR) 等路径，未采用 StarTools.get_data_dir() 规范目录。

4. 异常类型可能不匹配

   • download_pdf() 捕获 requests.RequestException，但代码其他处使用 requests.RequestsError；异常体系不一致，可能导致网络异常漏捕获。

5. 统计口径逻辑不准确

   • stats['total_announcements'] += len(downloaded) 实际累计的是“下载成功数”，不是“总公告数”，字段语义与实现不一致。

6. 会话未关闭

   • Session 生命周期未托管，存在资源泄漏风险。

---

util.py

1. 日志框架不符合约束

   • 使用 loguru，不符合 logger 约束。

2. 持久化目录依赖硬编码配置

   • ensure_directories() 使用 config.DATA_DIR/config.LOGS_DIR，未走 StarTools.get_data_dir() 规范路径。

3. 重试装饰器对异步函数不兼容

   • retry_on_failure 仅包装同步函数；若误用于 async def 将失效/行为异常，可维护性风险高。

---

extractors/extractor.py

1. 日志框架不符合约束

   • 使用 loguru，不符合 logger 约束。

2. 日期转换存在潜在时区偏差

   • _format_date 强制用 timezone.utc 解析毫秒时间戳，若业务期望本地日期（如中国时区）可能出现“日期减一天”等偏差。

---

notifiers/notifier.py

1. 日志框架不符合约束

   • 使用 loguru，违反 logger 约束。

2. 异步环境中的同步网络 I/O 阻塞

   • send_to_wecom() 使用 requests.post(...) 同步请求，会阻塞异步执行环境。

3. 持久化路径硬编码

   • Path(config.DATA_DIR) / config.METADATA_FILE 未采用 StarTools.get_data_dir()。

4. 潜在 KeyError

   • _build_markdown() 中 info['sec_name'] 直接索引，若字段缺失会抛异常，鲁棒性不足。

5. sys.path.insert 动态改路径

   • 在 __main__ 中修改导入路径，降低可维护性与部署稳定性。

---

代码评审团

人物一：Linus Torvalds
“最大的问题不是语法，而是执行模型：你把同步网络请求塞进异步环境，这会让并发性能像单线程脚本一样。先把网络调用改成不会阻塞事件循环的模式，再谈扩展性。”

人物二：Steve Jobs
“用户只看到‘卡不卡、稳不稳’。max_pages 一旦输入脏数据就异常、某些字段缺失就 KeyError，这些都在破坏体验。把输入校验和降级处理做精致，产品感会立刻提升。”

人物三：Rick Sanchez（《瑞克和莫蒂》）
“你这日志体系像多宇宙分裂：一会儿 astrbot logger，一会儿 loguru，再来个自定义 logger。统一可观测性，不然出故障时你会在日志宇宙里迷路，Morty 都救不了你。”

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

---

🔍 审核摘要

统计信息

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

astrbot_plugin/main.py
dify_plugin/cninfo_hedge/main.py
config.py
crawler.py
util.py
astrbot_plugin/__init__.py
extractors/__init__.py
extractors/extractor.py
notifiers/__init__.py
notifiers/notifier.py
dify_plugin/cninfo_hedge/__init__.py

---

此报告由AI自动生成，旨在提供初步反馈和改进建议，不能完全替代人工审核。最终决策以社区维护者的人工审核为准。目前自动审核（仓库地址）处于试验阶段，如遇问题请向维护者反馈。评论 @astrpluginreviewer review 来重新触发 review，请不要重复提交插件发布 issue。

当前仓库：Interstellar1217/CNInfoHedgeCrawler
剩余触发次数：4