liuwenxin
|
76ded6d5ec
|
fix(m3): code-review P1——锁定不变量、入结回填与双轨漂移
- 月结锁定拦截补全：原仅新增回款一处，现覆盖回款编辑（原月+新月，防跨月
搬数）/删除、发货新增（按发货日）/编辑（原月+新月）/删除、核算与重算
（已审定月份金额只走调整单）；拦截先于「被引用/已核算」判定
- 台账 settlement_id 回填落地：generate 在 replace_items 后回填纳入行、
重生成前先解绑（clear_ledger_settlement_id）——不回填/不解绑会让
「未入结」判据失效：重生成漏算整月或把已固化行反复计入
- 经理聚合行落库：原只进内存参与 items 汇总，台账表永远无 calc_type=2
行、报表下钻落空；重生成先作废旧聚合行（deactivate_month_aggregates），
_confirm_scope 限定计算类型避免聚合行混入单笔池
- 月结取参锚定所属月月末（_month_end）：原用 date.today()，补生成历史
月份时分成/月工资口径随生成日漂移；聚合行 receipt_date 同步为月末
- generate/submit 出参固化前置到 commit 之前（commit 后 SELECT 失败会把
已生成的月结记成失败审计，与库中事实相反）
- V9 迁移：business_secondary_allocations 唯一键条件式补齐（幂等裸 SQL，
临时库验证「无键→补」「有键→跳过」两分支），修复旧 V6 升级库拿不到
V6 就地加入的约束；V8 迁移 revision 声明补类型注解（原格式漂移使
schema 同步测试的 head 推断静默漏检 V8）；init-dev 版本号推进至 V9
- V7 留存 SQL 约束/索引名单数改复数（与 alembic op.f 与 init-dev 对齐，
消除 SHOW CREATE TABLE 双轨漂移）
- 测试：显式控制审批开关（不依赖 V8 翻转后的种子默认值）；新增回归 8 例
（回填落库/聚合不重复/月末取参/月末纯函数 5 例/锁定拦截回款与发货与
核算）；全量 773 passed
Co-Authored-By: Claude Code <noreply@anthropic.com>
|
2026-10-19 11:59:49 +08:00 |
|
liuwenxin
|
1c93ea0a4b
|
fix(m3): code-review 15 项修复（13 项落地，2 项依赖分摊语义记待办）
- 审批：删 _flush_order 撞键内部重试（rollback 会回掉同事务业务行造成
静默丢数据，异常改由 create 外层重试整体重来/update 路径审计收敛）；
act() 出参固化前置到 commit 之前（commit 后查询失败会把已通过的审批
记成失败审计）；驳回/撤回清空方案生效行（防被否决方案进 M3 快照）；
keyword 改 contains(autoescape=True) 对齐全库转义口径；撤单抽
withdraw_pending_orders 供编辑清空方案时显式调用
- 业务：删业务遇在途审批单禁删 400、终态审批单随 _hard_delete 清理；
编辑清空方案同步撤回在途单；历史离职参与人编辑放行（冗余姓名固化
语义，仅新增参与人做在职校验）
- 迁移：business_secondary_allocations 补 uk(business_id, personnel_id)
（模型/迁移 py/导出 SQL/init-dev 四方同步，开发库往返重放验证）
- 设置：upload_logo 改同步 def（阻塞操作不卡事件循环）+ file.size 先拦；
company_name schema 剥空白防全空白入库；logo 旧文件清理移到 commit
成功后（防库指旧路径而文件已删断链）；uploadLogo 走 request() 解包
（原直发 service.post 不解包信封，logoUrl 会被置 undefined）
- 备份：restore_drill --drop-exec 先清全部历史 drill 库再演练并即删本轮
（原语义永远删错对象，每轮净增一份整库拷贝）
- 测试：conftest 清理补 system_settings 种子行与 uploads/logo 文件；
新增 5 例（驳回清方案/清空撤单/在途禁删终态随删/离职放行/BOOL 边界）；
全量回归 701 passed
- 遗留 2 项记 AI_CONTEXT（方案版本承载、审批单快照，依赖 ④ 分摊语义）
Co-Authored-By: Claude Code <noreply@anthropic.com>
|
2026-10-16 14:23:53 +08:00 |
|
liuwenxin
|
680661a80b
|
feat(allocation): 二次分配方案录入/替换/终态门槛（分摊计算待澄清另批）
- SecondaryAllocationIn/Out：比例走金额红线口径（str 入参、两位小数、
超 100 拒绝不静默舍入）；参与人 schema 层去重（uk 前置友好提示）
- create/update 同事务写入（replace_allocations，先删后 flush 再插——
uk 表防御写法）；create 放编号重试循环内，rollback 后方案随业务重来
- 人员在职复用 ensure_active（label=分摊参与），personnel_name 冗余固化；
FK RESTRICT：被方案引用人员禁删，方案回显不静默失联
- 合计不强制 100%（需求原文）：详情 allocation_hint 只展示合计/余额，
余额归属待澄清 #2 不落库
- 单测 13 例（比例边界/去重）+ 集成 9 例（回显/替换清空/离职 400/
终态禁改/RESTRICT 保护）
Co-Authored-By: Claude Code <noreply@anthropic.com>
|
2026-10-15 18:31:27 +08:00 |
|