[Plugin] astrbot_plugin_bookkeeping

[NONAME00X]

插件信息

{
  "name": "astrbot_plugin_bookkeeping",
  "display_name": "日常记账",
  "desc": "一个功能完善的个人记账插件，支持支出/收入记录、统计分析、AI财务建议等功能。",
  "author": "NONAME",
  "repo": "https://github.com/NONAME00X/astrbot_plugin_bookkeeping",
  "tags": [
    "财务",
    "记账",
    "日常"
  ],
  "social_link": "https://github.com/NONAME00X"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_bookkeeping

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

main.py

1. 数据持久化路径不符合框架规范

   • 问题：当前通过 get_astrbot_data_path() 手动拼接 plugin_data/<plugin_name>，未按要求使用 StarTools.get_data_dir()。
   • 影响：在框架约定、迁移、插件隔离策略变化时，路径兼容性和可维护性会变差。
   • 建议：改为从 astrbot.api.star 导入 StarTools，并使用 StarTools.get_data_dir() 获取插件数据目录（返回 Path 对象）。

2. 以 sender_name 作为文件名存在安全与数据隔离风险

   • 问题：_get_user_file() 直接使用 user_name 拼接文件名：f"{user_name}_bookkeeping.json"。
   • 影响：
     • 可能出现路径穿越或非法文件名字符问题（如包含 ../、/、\、保留字符等）。
     • 不同用户同名会发生账本混写（隐私与数据错乱风险）。
   • 建议：
     • 使用平台稳定唯一 ID（如 user_id）做主键，而不是昵称。
     • 对文件名做白名单清洗（仅保留安全字符）或做哈希映射（如 sha256(user_id)）。

3. “查账详情/删除账单”切片逻辑错误，实际操作的是“最旧20条”

   • 问题位置：query_details() 与 delete_record() 中 records = sorted(..., reverse=True) 后使用 records[-20:]。
   • 影响：
     • 逆序后 [-20:] 取到的是尾部（最旧记录），而非最近 20 条。
     • 用户可能误删非预期账单，功能语义偏差明显。
   • 建议：若想展示/删除最近20条，应使用 records[:20]。并确保“展示序号”和“删除序号”指向同一数据切片。

4. 并发下存在“读-改-写”竞态，可能丢账单

   • 问题：_save_record() 与 delete_record() 都是“先读文件 -> 内存修改 -> 覆盖写回”，没有并发保护。
   • 影响：异步环境中同一用户短时间多条命令并发执行时，后写覆盖前写，导致记录丢失。
   • 建议：按用户维度加 asyncio.Lock，或引入原子更新策略（临时文件+replace）。

5. 写文件非原子操作，异常中断可导致 JSON 损坏

   • 问题：_save_records() 直接写目标文件。
   • 影响：进程中断/异常时可能留下半写文件，后续 json.load 失败并返回空列表，造成“数据看似丢失”。
   • 建议：先写临时文件，再 Path.replace() 原子替换；保留损坏文件备份便于恢复。

6. query_details() 的“序号语义”与全量数据不一致，易引发误解

   • 问题：只展示固定20条且序号从1开始，但文案“账户详情”未提示“仅展示部分记录”。
   • 影响：用户可能以为是全量序号，删除时出现认知偏差。
   • 建议：在输出中明确“仅显示最近20条”，并可提供分页/全量导出命令。

---

代码评审团

人物一：Linus Torvalds（直给型）
“records[-20:] 这个切片看得我血压上来了：你先 reverse 排序，再拿尾巴，拿到的是老古董。逻辑能跑不代表逻辑正确。把 ‘用户想看的最近记录’ 当第一原则，先修这个。”

人物二：Steve Jobs（产品体验型）
“删除账单是高风险操作，你现在的序号系统在用户视角不够‘可预期’。一旦误删，体验就是灾难。把‘显示范围’和‘删除范围’讲清楚，再加确认机制，产品就会从‘能用’变成‘可信’。”

人物三：Geralt of Rivia（谨慎实战型）
“名字当文件名？这像在夜里把钱包放路边。用唯一ID、做安全映射、加锁防并发——这些不是‘可选强化’，是避免数据事故的银剑。”

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

---

🔍 审核摘要

统计信息

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

main.py

---

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

[astrbot-plugin-copybara]

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

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