[Plugin] astrbot_plugin_mailer

[FFFold]

插件信息

{
  "name": "astrbot_plugin_mailer",
  "display_name": "学会发邮件",
  "desc": "为Agent提供一个受控的 SMTP 电子邮件发送工具，支持 HTML 格式、内嵌图片以及附件功能。",
  "author": "Fold",
  "repo": "https://github.com/FFFold/astrbot_plugin_mailer",
  "tags": [
    "email",
    "电子邮件",
    "smtp"
  ]
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_mailer

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

main.py

1. send_email 工具参数 Schema 与实际校验逻辑不一致（功能正确性）

   • FunctionTool 的 required 仅有 ["to", "subject"]，但 MailRequest.from_payload() 强制要求 text_body 或 html_body 至少一个非空。
   • 结果是：LLM 可能按 Schema 生成“合法”调用却在运行时抛 ValueError，造成工具调用不稳定。

2. SMTP 网络调用缺少超时控制（异步环境稳定性）

   • _send_mail -> send_message -> aiosmtplib.send(...) 未设置 timeout。
   • 在异步环境中，网络异常时可能长时间挂起，影响事件循环中其它任务的响应性。

3. SMTP 端口缺少范围校验（边界条件）

   • _smtp_settings() 中 port=int(...) 后未校验端口是否在 1~65535。
   • 错误配置会延迟到更晚阶段才报错，定位成本更高。

4. mail_config_check 返回完整允许目录列表（信息暴露风险）

   • 命令返回 allowed_attachment_roots 的绝对路径，属于主机路径信息。
   • 尽管有管理员权限限制，仍建议最小化暴露（如仅显示数量或脱敏路径）。

---

init.py

（未发现问题）

---

mailer/init.py

（未发现问题）

---

mailer/message_builder.py

1. html_to_text 使用正则剥离 HTML，文本回退质量和正确性有限（功能边界）
   • 正则方式无法正确处理 HTML 实体、复杂结构、<style>/<script> 等场景。
   • 当只提供 html_body 时，纯文本备选正文可能出现语义丢失或异常空白，影响邮件可读性与兼容性。

---

mailer/models.py

1. 邮箱校验强度偏弱（数据质量）
   • 依赖 parseaddr + "@" in parsed 的判定较宽松，可能放过格式上不够规范的地址。
   • 可能导致后续 SMTP 阶段失败（而不是在输入校验阶段尽早失败）。

---

mailer/smtp_client.py

1. SMTP 发送缺少超时参数（异步网络 I/O 风险）
   • aiosmtplib.send(...) 未显式设置 timeout。
   • 在网络抖动或服务器异常时，调用时长不可控，影响插件整体可用性。

---

tests/conftest.py

1. 路径层级硬编码脆弱（可维护性）
   • WORKSPACE_ROOT = PLUGIN_ROOT.parents[4] 依赖固定目录深度。
   • 在不同 CI/本地目录结构下容易触发 IndexError 或导入路径错误。

---

tests/test_message_builder.py

（未发现问题）

---

tests/test_models.py

（未发现问题）

---

tests/test_plugin.py

1. 测试对 sys.modules 的全局注入未回滚（测试隔离性）
   • _install_astrbot_stubs() 修改全局模块缓存，且没有 teardown/恢复。
   • 可能污染其它测试模块，导致顺序相关问题和“幽灵失败”。

---

tests/test_smtp_client.py

（未发现问题）

---

代码评审团

人物一：Linus Torvalds
“接口契约要一致，别让 Schema 说‘可以’，运行时却说‘不行’。send_email 的 required 字段和 MailRequest 的约束不一致，这就是典型的‘文档驱动 bug 生成器’。把契约统一，很多线上噪音会直接消失。”

人物二：Grace Hopper
“网络是最不可靠的外设。异步并不等于不会卡住：没有 timeout 的 SMTP 调用，本质上还是可能把系统拖慢。给发送链路加上超时（必要时重试与错误分级），你会得到更可预测的系统行为。”

人物三：Tony Stark（钢铁侠）
“测试里全局改 sys.modules 还不还原，就像在实验室里改了反应堆参数转头就走——下一位同事会‘惊喜连连’。把 stub 放进 fixture 并做清理，测试隔离性会立刻提升一个时代。”

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

---

🔍 审核摘要

统计信息

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

main.py
__init__.py
mailer/__init__.py
mailer/message_builder.py
mailer/models.py
mailer/smtp_client.py
tests/conftest.py
tests/test_message_builder.py
tests/test_models.py
tests/test_plugin.py
tests/test_smtp_client.py

---

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

当前仓库：FFFold/astrbot_plugin_mailer
剩余触发次数：4

[FFFold]

@astrpluginreviewer review

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_mailer

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

main.py

1. 配置布尔值解析存在隐性逻辑错误风险
   use_tls/use_starttls/enable_llm_tool/require_tool_confirmation/allow_unsafe_all_attachment_paths 等字段使用了 bool(...) 强转。
   若配置源给出字符串（如 "false"、"0"），在 Python 中都会被解析为 True，会导致安全策略或 SMTP 行为与预期相反（例如误开 TLS 或误开任意路径附件）。

2. 多个“应为列表”的安全配置缺少类型校验，可能被字符串逐字符解析
   如 allowed_sender_ids/allowed_domains/blocked_domains/allowed_attachment_roots，当前直接迭代配置值。
   当配置被错误写成字符串时，会按字符迭代，造成白名单/黑名单判定异常（功能正确性和安全策略都会受影响）。

3. 附件大小限制配置缺少合法区间校验
   max_attachment_size_mb、max_total_attachment_size_mb 仅做 int(...) 转换，未校验是否为正数。
   若被配置为 0 或负数，会触发“所有文件都超限”等异常行为，错误定位困难。

4. 可维护性问题：存在无效变量与重复计算
   mail_config_check 中 roots 变量构造后未使用；同时 _allowed_roots() 被重复调用。
   这会增加维护噪音，也容易在后续修改时引入不一致。

---

mailer/models.py

1. 邮箱正则对域名标签约束不完整，可能放过明显非法域名
   EMAIL_RE 中域名标签使用 [A-Za-z0-9-]+，未限制“标签不能以 - 开头或结尾”。
   例如 user@-bad-.com 这类地址可能通过校验，后续在 SMTP 阶段失败（输入校验与实际可投递性不一致）。

---

tests/test_plugin.py

1. 在测试中直接使用 asyncio.run(...) 存在事件循环冲突风险
   若测试环境已存在运行中的事件循环（常见于启用异步插件/工具链时），asyncio.run 会抛 RuntimeError。
   这会导致测试在不同 CI/本地环境表现不一致（潜在稳定性缺陷）。

---

代码评审团

人物一：Linus Torvalds
“bool("false") == True 这种坑，属于‘看起来没问题，线上最要命’。配置解析要强类型，不然你写的是安全策略，运行的是玄学策略。”

人物二：Steve Jobs
“安全配置项如果允许‘字符串误入’，就是糟糕的产品体验：用户以为关了功能，其实没关。配置系统也需要‘直觉一致性’——输入什么就得到什么。”

人物三：福尔摩斯（Sherlock Holmes）
“asyncio.run 在测试里像一枚定时炸弹：平时安静，遇到已有事件循环就爆。可重复、可迁移的测试，必须先排除环境依赖型失败。”

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

---

🔍 审核摘要

统计信息

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

main.py
__init__.py
mailer/__init__.py
mailer/message_builder.py
mailer/models.py
mailer/smtp_client.py
tests/conftest.py
tests/test_message_builder.py
tests/test_models.py
tests/test_plugin.py
tests/test_smtp_client.py

---

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

当前仓库：FFFold/astrbot_plugin_mailer
剩余触发次数：3

[FFFold]

@astrpluginreviewer review

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_mailer

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

main.py

1. 默认权限策略偏宽，存在被滥用发送邮件风险

   • 问题：_check_sender_allowed 在 security.allowed_sender_ids 为空时，等价于“允许所有发送者”；而 send_email_tool 本身没有 @filter.permission_type 保护（工具路径通常也不走 command 权限装饰器）。
   • 影响：若 LLM 工具可被普通用户触发，可能造成未授权发信、垃圾邮件或对外滥发。
   • 建议：将默认策略改为“拒绝所有，需显式配置白名单后启用”，或至少增加独立配置开关（如 security.require_sender_allowlist=True）并在未配置时拒绝发送。

2. 高风险配置开关可导致任意本地文件外传

   • 问题：security.allow_unsafe_all_attachment_paths=True 时，附件路径仅校验“文件存在”，会绕过根目录限制。
   • 影响：一旦工具被误触发或被滥用，可读取并外传进程可访问的任意本地文件。
   • 建议：即使保留该开关，也应叠加更强保护（仅管理员/仅命令路径/二次确认/审计日志），并建议在代码层对该模式输出显式高危告警日志。

3. 存在未使用变量，降低可维护性

   • 问题：_resolve_safe_path 中 security_cfg = self._security_config() 未被使用。
   • 影响：增加阅读噪音，易误导维护者。
   • 建议：删除未使用变量，保持函数最小必要实现。

---

mailer/models.py

1. 输入校验存在“falsy 值被吞掉”的逻辑漏洞（类型检查可被绕过）
   • 问题：from_payload 中多处使用 payload.get(...) or ...：
     • text_body = payload.get("text_body") or ""
     • html_body = payload.get("html_body") or ""
     • reply_to = payload.get("reply_to") or ""
     • attachments_payload = payload.get("attachments") or []
     • inline_payload = payload.get("inline_images") or []
   • 影响：例如 reply_to=0、text_body=False、attachments={}（空 dict）等非法类型会被静默转成默认值，从而绕过原本应触发的类型错误。
   • 建议：改为“先判断 key 是否存在，再做严格类型校验”，不要用 or 做兜底（仅在 is None 时兜底）。

---

tests/conftest.py

1. 全局修改 sys.path 且不回滚，测试隔离性不足
   • 问题：模块导入时直接向 sys.path 注入路径，且未在会话结束后恢复。
   • 影响：可能污染其他测试模块的导入行为，导致用例之间互相影响（尤其在大型测试集或并行场景）。
   • 建议：用 fixture 管理并在 teardown 中回滚，或改用更可控的测试启动参数/环境配置。

---

代码评审团

人物一：Linus Torvalds
“默认允许所有人发邮件？这就像把 chmod 777 当最佳实践。权限模型应该默认收紧，再按需放开。尤其你还有个‘允许任意附件路径’开关，组合起来风险直线上天。先把默认策略变成 deny-by-default。”

人物二：Grace Hopper
“or 兜底在配置解析里很好写，但也最容易把坏数据‘洗白’。类型系统和校验逻辑的价值就在于早失败（fail fast）。建议把 None 与‘错误类型’严格区分，别让 False/0/{} 悄悄溜过去。”

人物三：Tony Stark（钢铁侠）
“测试里直接改 sys.path，短期爽，长期痛。等测试规模上来，这种全局副作用会让你 debug 到凌晨。把路径注入放进可回滚 fixture，给未来的你留条活路。”

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

---

🔍 审核摘要

统计信息

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

main.py
__init__.py
mailer/__init__.py
mailer/message_builder.py
mailer/models.py
mailer/smtp_client.py
tests/conftest.py
tests/test_message_builder.py
tests/test_models.py
tests/test_plugin.py
tests/test_smtp_client.py

---

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

当前仓库：FFFold/astrbot_plugin_mailer
剩余触发次数：2

[FFFold]

AI reviewer意见已按需修复，请维护者审阅