[Plugin] astrbot_plugin_bookkeeper

[IwannaYuJie]

插件信息

{
  "name": "astrbot_plugin_bookkeeper",
  "display_name": "自动记账",
  "desc": "AI 智能记账插件，支持自动记账、白名单管理、定时账单推送与手动查询。",
  "author": "IwannaYuJie",
  "repo": "https://github.com/IwannaYuJie/astrbot_plugin_bookkeeper",
  "tags": [
    "记账",
    "生活助手",
    "财务"
  ],
  "social_link": "https://github.com/IwannaYuJie"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_bookkeeper

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

main.py

1. 配置变更可能无法持久化（功能正确性 / 可维护性）

   • 位置: __init__ + _save_config
   • 问题: self.config 在 config is None 时会被设置为普通 {}，而 _save_config() 仅在 isinstance(self.config, AstrBotConfig) 时才调用 save_config()。
   • 影响: 运行期通过命令修改的配置（如 book auto off、白名单、定时设置）在部分场景可能不会落盘，重启后丢失。
   • 建议: 明确保证 self.config 始终是可持久化对象；或在非 AstrBotConfig 分支给出告警/兜底持久化逻辑。

2. 金额使用 float 存储与计算，不适合账务场景（逻辑正确性）

   • 位置: bookkeeper_add_expense / _append_record / _render_bill / _render_summary / _safe_float
   • 问题: 虽然 _normalize_amount 用了 Decimal 量化，但最终仍转成 float 存储和汇总。
   • 影响: 累计统计可能出现浮点误差（例如 0.1 + 0.2 类问题），长期账单会出现“分角级”偏差。
   • 建议: 全链路使用 Decimal（存储字符串或最小货币单位整数），展示时再格式化。

3. 数据异常被静默吞掉并当成 0.0（潜在缺陷）

   • 位置: _safe_float 及其调用链
   • 问题: 非法金额值被直接转换为 0.0，不会显式暴露数据损坏。
   • 影响:
     • 汇总结果被悄悄污染；
     • 删除逻辑（fallback 匹配）可能因“同为 0.0”提高误匹配概率。
   • 建议: 对非法金额做告警日志并跳过该条，或在加载数据时进行一次校验修复。

4. 月度定时在小月可能“整月不触发”（边界条件）

   • 位置: _build_monthly_cron_expression
   • 问题: 允许设置 29/30/31 号，但 cron 表达式在无该日期的月份通常不会触发。
   • 影响: 用户配置“每月 31 号推送”时，2/4/6/9/11 月会漏推送。
   • 建议: 增加策略选项（如“超出当月天数时改为月末”），或在设置时明确警告此行为。

5. “仅记支出”依赖提示词约束，缺少服务端硬校验（安全/业务一致性）

   • 位置: on_llm_request 与 bookkeeper_add_expense
   • 问题: on_llm_request 中有 policy，但真正落库工具函数只校验 amount > 0 和白名单，不校验“是否支出语义”。
   • 影响: 在提示注入、模型误判或工具误调用时，退款/收入等仍可能被写入，造成账本污染。
   • 建议: 在 bookkeeper_add_expense 增加服务端业务规则（关键词黑名单/调用来源标记/二次确认机制）。

6. 时间格式校验与文案不一致（可用性）

   • 位置: _parse_hhmm + 命令提示 “请使用 HH:MM”
   • 问题: 当前允许 7:5 通过，但提示文案暗示应为两位制 HH:MM。
   • 影响: 用户体验不一致，容易产生“看起来错但能保存”的困惑。
   • 建议: 若文案坚持 HH:MM，请改为严格正则校验（^\d{2}:\d{2}$）。

---

代码评审团

人物一：Linus Torvalds（直给型）
你这记账逻辑主干挺实在，但“账务用 float”这事就像拿锤子修精密钟表——能动，但不该。建议把金额类型一次性收敛到 Decimal，别让每个函数都在猜“这次会不会精度漂移”。

人物二：Ada Lovelace（结构与正确性导向）
我最担心的是“错误被悄悄归零”。_safe_float -> 0.0 会让异常数据伪装成正常记录，统计看上去还“很优雅”。建议将异常显式化：要么拒绝、要么标记、要么隔离，不要静默同化。

人物三：Tony Stark（产品体验+工程落地）
月报 31 号在小月失踪，这就是典型“上线后用户来追你”的 bug。加一个“自动落到月末”的开关，顺便在 book monthly 命令里回显策略。功能立刻从“能用”升级到“靠谱”。

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

---

🔍 审核摘要

统计信息

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

main.py

---

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

[astrbot-plugin-copybara]

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

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