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
|
e7e47d88bc
|
feat(settlement): 月度结算快照生成与审批锁定，调整单（补发/扣减）与审批联动
- settlement_service：generate（确认回款口径 #8 策略点=有效台账行按回款日
归月、经理聚合 #12 暂口径基数不含经理个人户、月结 uk 一行一月、草稿/
驳回可重生成、已审定/待审 400）→ submit（审批开关分流，关=自动通过即
锁定、开=待财务复核）→ approval_service.act 扩 order_type=2/3 钩子
（通过=mark_locked/调整单生效；驳回=回草稿/作废，延迟导入防循环）
- 锁定拦截：回款新增写路径校验回款月已审定 400（比需求「提示」取硬，
一期安全侧；M2 已核算禁改防线先于其生效）
- 调整单：新建（TZ 单号，开关分流自动通过留痕/走审批）+ 列表
- schemas settlement：金额 Decimal 声明（str 声明遇 Decimal 入参拒绝）
- 测试 6 例：生成回填/无记录 400/已审定重生成 400/关自动锁定/锁定后回款
禁改/开两节点流转至锁定（finance→leader）
Co-Authored-By: Claude Code <noreply@anthropic.com>
|
2026-10-17 00:33:31 +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
|
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
|
a02b6c3ec8
|
feat(business): 回款管理接口（批次号/手动指定批次/已核算禁改/合计回写）
- 批次 HK 生成；shipment_id 同业务强校验（统一失败面防探测）
- 已核算回款禁改禁删（M3 防重锚点）；payment_days 派生（发货日首日=第 1 天）
Co-Authored-By: Claude Code <noreply@anthropic.com>
|
2026-10-13 12:25:34 +08:00 |
|