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
|
409e26614f
|
refactor(business): 编辑出参取校验返回值省重查，submit_change 批量预载
- update_receipt 取 _ensure_shipment_of_business 返回值组装出参
（shipment_id 已赋为 payload 新值，与 get_with_refs 重查同源），
省一次三表 join；update_shipment 同理取锁定行的 business_no
- 删 business_repository.count_items（无任何调用的死代码，删除保护
实际走 has_docs）
- submit_change 校验段定义与岗位各一条 IN 批量预载（逐条查询在批次
扩大时是写侧 N+1）；岗位字典键 casefold 归一保持与单条版 ci 校对
等价的命中语义；旧值留痕保持逐条（窗口函数优化留待 M3 一并设计）
Co-Authored-By: Claude Code <noreply@anthropic.com>
|
2026-10-15 09:32:35 +08:00 |
|
liuwenxin
|
e6c2c71d1b
|
fix(business): 行金额乘积显式两位固化，计提比例对负天数钳制
- 明细行金额（数量2位×单价4位可产 6 位小数）直写 DECIMAL(14,2) 被
MySQL 静默舍入且回显漂移（创建响应读内存值、复查读库值不一致）；
显式 quantize HALF_UP 固化，舍入方向与 MySQL 一致（业务/发货同改）
- 先款后货回款（payment_days 为负）线性外插会算出高于起始值的计提
比例（多算绩效）；钳制到 0 天按起始比例计提，payment_days 出参仍
照实输出负值（口径：2026-10-14 用户裁决）
- 新增 formula 负天数钳制单测与 quantize 回显/库值一致性集成用例
Co-Authored-By: Claude Code <noreply@anthropic.com>
|
2026-10-14 18:58:30 +08:00 |
|
liuwenxin
|
a0086554b1
|
refactor(core): 1062 唯一键冲突判定收口至 core/db_errors
六个 service 各自复制的 _is_duplicate_entry（口径已漂移：rating/product
用 _ERR_DUP_ENTRY 常量、其余四处写字面量 1062）收敛为共享函数；
is_duplicate_entry 语义与文案注释各处保留，非 1062 完整性错误仍原样
上抛由全局处理器收敛。
Co-Authored-By: Claude Code <noreply@anthropic.com>
|
2026-10-14 15:17:00 +08:00 |
|
liuwenxin
|
5a3778a424
|
fix(business): 唯一键冲突重试改为审计上下文内循环，单条审计无递归
三处 create（业务/发货/回款）的 return _retry_create 是吞异常后正常
退出：外层审计把已回滚的首次尝试记成 success（假成功），重入整个
create_* 违反 audit_service「不支持嵌套」约定，且 _NO_RETRY 死常量
导致持续冲突时无界递归至 RecursionError。改为 audit 内 for 循环
（上限 _NO_RETRY），rollback 后重新锁行与取号，成功/最终失败各恰好
一条审计。新增集成用例锁定两条路径的日志增量。
Co-Authored-By: Claude Code <noreply@anthropic.com>
|
2026-10-14 15:09:05 +08:00 |
|
liuwenxin
|
4aea2be103
|
feat(business): 发货管理接口（发货单/明细/运费/汇总回写/删除保护）
- FOR UPDATE 锁业务行防发货/回款并发丢失更新；终态业务禁新发货
- 行金额服务层按数量×单价固化；明细行 remark 按设计定稿移除（冒烟修复）
Co-Authored-By: Claude Code <noreply@anthropic.com>
|
2026-10-13 12:25:33 +08:00 |
|