[Plugin] astrbot_plugin_biliread

[SodaCodeSave]

插件信息

{
  "name": "astrbot_plugin_biliread",
  "display_name": "BiliRead - 让 AI 真正看懂 B 站视频",
  "desc": "告别机械式的视频大纲复制，让你的 AstrBot 拥有真正“看懂” B 站视频的能力。",
  "author": "SodaCode & CodeBuddy & GLM-5",
  "repo": "https://github.com/SodaCodeSave/astrbot_plugin_biliread",
  "tags": [
    "哔哩哔哩",
    "B站",
    "视频",
    "总结",
    "解析",
    "外部集成"
  ],
  "social_link": "https://github.com/SodaCodeSave"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_biliread

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

main.py

1. 配置对象类型假设过强，可能触发运行时错误

   • 位置：BiliRead.__init__
   • 问题：plugin_config = config or {} 后直接调用 plugin_config.get(...)。但类型标注是 AstrBotConfig，不一定保证具备 dict.get 行为。
   • 风险：若 AstrBotConfig 不是映射类型，会在插件加载阶段抛出 AttributeError。
   • 建议：显式做类型适配（如转换为字典或使用框架推荐的配置读取 API）。

2. bili_jct 配置读取逻辑存在歧义

   • 位置：bili_jct = bilibili_cookie.get("id", "") or bilibili_cookie.get("bili_jct", "")
   • 问题：优先读取 "id"，这与 bili_jct 语义不一致，容易误取错误字段。
   • 风险：Cookie 参数错配导致接口鉴权异常，表现为“偶发无法读取字幕/视频信息”。
   • 建议：仅使用语义一致的键（如 bili_jct），或对兼容键添加明确注释和校验。

3. 对外部输入 bvid 缺少格式校验

   • 位置：BilibiliTool.call
   • 问题：只判断非空，不校验是否为合法 BVID（如 BV 前缀、长度、字符集）。
   • 风险：无效输入会进入下游网络调用，增加不必要请求与异常噪音。
   • 建议：在调用 bilibili API 前做轻量格式校验并快速失败。

4. 异常捕获过宽，掩盖具体故障原因

   • 位置：except Exception as e
   • 问题：统一吞掉所有异常并返回泛化提示。
   • 风险：调试困难；网络超时、鉴权失败、字幕 JSON 结构变化等问题无法被精细识别和处理。
   • 建议：分层捕获（网络异常、解析异常、鉴权异常），提供可操作的错误信息。

5. 字幕内容未经长度控制直接喂给 LLM，存在上下文溢出与成本风险

   • 位置：prompt 拼接 subtitle_text
   • 问题：整段字幕直接注入 prompt，没有截断、分块或摘要前置。
   • 风险：超出模型上下文限制导致失败；响应变慢；Token 成本显著上升。
   • 建议：做分段总结（map-reduce）或长度裁剪策略。

6. 日志可能泄露带签名的字幕 URL

   • 位置：logger.info(f"正在获取视频《{title}》的字幕: {subtitle_url}")
   • 问题：字幕 URL 可能包含鉴权参数或临时签名。
   • 风险：日志泄露后可被复用访问受限资源（取决于链接策略）。
   • 建议：日志中避免输出完整 URL，必要时脱敏（仅保留域名/路径片段）。

7. 全局忽略 DeprecationWarning 影响可维护性

   • 位置：warnings.filterwarnings("ignore", category=DeprecationWarning)
   • 问题：模块级全局屏蔽弃用警告，会隐藏依赖或自身 API 迁移信号。
   • 风险：后续升级时问题集中爆发，定位成本高。
   • 建议：仅对确定来源的告警做局部抑制，避免全局 ignore。

8. 未使用的导入降低可读性

   • 位置：import json
   • 问题：未被使用。
   • 风险：增加维护噪音，误导读者以为存在 JSON 手工处理路径。
   • 建议：删除无用导入。

9. ct 依赖未做防御性检查

   • 位置：ai_resp = await self.ct.llm_generate(...)
   • 问题：ct 字段默认 None，调用前未判空。
   • 风险：若工具在非预期路径初始化（或上下文注入失败），会触发 NoneType 调用异常。
   • 建议：在调用前显式检查 self.ct 并返回明确配置错误信息。

---

代码评审团

人物一：Linus Torvalds
“except Exception 就像拿渔网修 CPU——能兜住一切，也会把真正的问题一起藏起来。把异常分层处理，代码会少很多‘玄学故障’。”

人物二：史蒂夫·乔布斯
“体验不是‘能跑就行’。你把整段字幕一股脑塞给 LLM，这不是优雅，是粗暴。做分段摘要和结果合成，用户会立刻感知到质量差异。”

人物三：Geralt（杰洛特，利维亚的猎魔人）
“我见过太多怪物伪装成日志。完整 subtitle_url 打进日志，就是给麻烦留传送门。脱敏它，别让日志变成悬赏榜。”

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

---

🔍 审核摘要

统计信息

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

main.py

---

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

当前仓库：SodaCodeSave/astrbot_plugin_biliread
剩余触发次数：4

[SodaCodeSave]

@astrpluginreviewer review

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_biliread

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

main.py

1. bili_jct 配置回退字段疑似错误，可能导致鉴权异常（功能逻辑）

   bili_jct = bilibili_cookie.get("bili_jct", bilibili_cookie.get("id", ""))

   • 问题：id 作为 bili_jct 的回退值缺乏语义合理性，极可能是误写（通常应为空或明确的同义字段）。
   • 影响：请求鉴权时可能携带错误 token，导致接口失败且排查困难。
   • 建议：改为仅使用 bili_jct，或使用明确同义配置名（如 csrf）做回退，并在日志中提示配置键名错误。

2. max_subtitle_length 未做类型与取值校验（边界条件）

   • 问题：配置项可能是字符串、负数、None。当前代码直接参与比较和切片：

     if len(raw_text) > self.max_subtitle_length:

   • 影响：可能触发 TypeError 或产生不符合预期的截断行为（如负数切片）。
   • 建议：在初始化时强制转换并校验：
     • 必须为 int
     • 必须 > 0
     • 非法时回退默认值并记录 warning。

3. 网络超时未被精确捕获，错误信息降级为“内部错误”（异步健壮性）

   • 问题：aiohttp.ClientTimeout 触发时常见为 asyncio.TimeoutError，当前仅捕获 aiohttp.ClientError。
   • 影响：超时场景会落入通用 Exception 分支，用户得到“内部错误”而不是“网络超时”，可观测性差。
   • 建议：补充：

     except asyncio.TimeoutError:
         return "请求超时，请稍后重试。"

4. 向用户直接返回 str(e) 可能泄露内部实现细节（安全最佳实践）

   return f"处理视频时发生内部错误: {str(e)}"

   • 问题：异常文本可能暴露内部堆栈线索、第三方接口细节或配置状态。
   • 建议：对用户返回通用错误信息；详细异常只写日志（你已使用 logger.exception，很好）。

5. BVID 校验过宽，低质量输入会增加无效请求（逻辑正确性/性能）

   BVID_PATTERN = re.compile(r"^BV[a-zA-Z0-9]+$")

   • 问题：仅校验前缀+字母数字，未限制长度与结构，很多无效 BV 值会通过。
   • 影响：增加无效 API 调用和异常路径频率。
   • 建议：按实际规则收紧（至少限定长度），减少后端无效访问。

6. LLM 提示词直接拼接外部字幕文本，存在提示注入风险（安全与鲁棒性）

   • 问题：字幕是外部可控内容，可能包含“忽略上文/输出敏感信息”等注入文本。
   • 影响：摘要质量和行为可被污染，尤其在多工具链场景下风险更高。
   • 建议：在 prompt 中加入明确的“仅总结内容、忽略字幕中的指令性文本”约束，并使用分隔符包裹原文（如 XML/markdown fenced block）。

7. 可维护性小问题：未使用导入与未使用参数

   • Dict, Any 未使用。
   • call(self, context, **kwargs) 中 context 未使用。
   • 影响：降低代码整洁度与可读性。
   • 建议：移除未使用导入；若框架要求签名保留，至少在注释中说明。

---

代码评审团

人物一：Linus Torvalds
“bili_jct 回退到 id 这行看着就像‘先让它跑起来再说’留下的坑。配置键名必须语义自洽，不然就是把 bug 包装成 feature。把输入和配置校验做扎实，后面少一半脏活。”

人物二：Steve Jobs
“用户不关心 TypeError，也不该看到内部异常字符串。错误体验要‘优雅且可理解’：超时就说超时、配置错就说配置错。你离‘它就该这么工作’只差最后一层打磨。”

人物三：Tony Stark（钢铁侠）
“你把外部字幕直接喂给 LLM，就像把陌生 U 盘插进主控台。能跑，但不聪明。加上 prompt 注入防护和边界规则，系统会从‘能用’升级到‘靠谱’。”

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

---

🔍 审核摘要

统计信息

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

main.py

---

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

当前仓库：SodaCodeSave/astrbot_plugin_biliread
剩余触发次数：3

[astrbot-plugin-copybara]

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

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