liuwenxin/PayrollManagementSystem
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
91 Commits 9 Branches 0 Tags 9 MiB
a536608b53
Commit Graph

5 Commits

Author SHA1 Message Date
liuwenxin
028a94a6eb refactor(m3): settlement_service 三段收尾——生成/聚合/调整单拆分 
code-review 约定合规项续（calc_receipt 之后的剩余超长函数）：

- generate（~120 行）→ 编排 + 六段私有函数：_guard_regeneration（重生成
  准入）、_release_previous_round（解绑回填+旧聚合作废）、_upsert_draft
  （草稿头表新建/重置）、_build_items（=_bucket_rows + _item_of 记账）与
  _apply_totals（头表三合计）
- _build_manager_aggregates（~110 行）→ _ShareParams 冻结 dataclass +
  _manager_share_params（月末取参/百分数口径/#12 开关同源）/ _collect_team_buckets
  （含 #12「个人户行归经理本人」分支）/ _add_team_performance / _aggregate_row；
  全公司基数循环外算一次（口径逐行固定，行为等价）
- create_adjustment（~90 行）→ _adjustment_amount（恒正）/ _wire_adjustment_approval
  （submit_order 分流与留痕，含「不得手工插 ApprovalStep」外键教训注释）/
  _adjustment_out（出参前置到 commit 之前）

行为零变化；#12 参数化（include_manager_personal 过滤、个人户行按
personnel_id 直归经理、snapshot 的 manager_personal_included）语义原样保留。
验证：重构后月结集成 15 例 + 月结单测 5 例全绿（81s 内无并发干扰的一轮）；
全量回归待 88 会话 calc_service 收口后复跑（其进行中的拆行改动与测试
并发互清导致临时 401，非本重构所致）

Co-Authored-By: Claude Code <noreply@anthropic.com>
 2026-10-19 12:52:36 +08:00
liuwenxin
45c268b146 fix(m3): code-review P2——金额校验、参数解析与运维脚本容错 
- 调整单金额恒正：schema 正则拦格式（非负两位小数、≤12 位整数防
  NUMERIC(14,2) 溢出）+ 服务层 Decimal>0 语义校验——负数会反转补发/
  扣减语义，'abc' 原样入库会以 500 暴露
- 分界天数参数解析：int(parse_fraction(...)) 替代 int(参数原文)——
  '360.0' 是 INT 参数的合法入库值（Fraction 分母为 1），直接 int(str)
  抛 ValueError 会让整月核算 500
- logo 旧文件清理容错：OSError 只记 warning——裸抛会把成功的 logo 更新
  记成失败审计并 500（库中 logo_path 却已是新值，审计与事实相反）
- health_check.sh：Redis 探活改容器内环境变量展开（密码不进宿主进程
  命令行，模式同 backup_full.sh）；compose redis 补 REDIS_PASSWORD
- restore_drill.sh：行数核对扩到全部 32 张业务表（原 16 张漏 V7 四表、
  二次分配等核心表，备份漏表演练全绿）；查询失败回显 '?' 一律判 ✗
  （原 '?'=='?' 判一致，表缺失时误报「备份可恢复」）
- 并入 88 会话的 #12 实现：manager.personal.included 开关消费
  （_build_manager_aggregates 参数化，测试见 828b573）
- 测试：调整单金额恒正 5 稿、INT 参数小数原文核算 200 回归；单测 469

Co-Authored-By: Claude Code <noreply@anthropic.com>
 2026-10-19 12:10:29 +08:00
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
cb4f38dfd9 fix(m3): code-review P0 六项——算错钱与不可用 
- 经理聚合分成放大 100 倍：参数按百分数存储（12.5=12.5%），parse_fraction
  后调用方又 ×100、函数内部再 ÷100，净效应「绩效×12.5」；去掉调用方
  ×100，rate_text 改用实际参数原文（原硬编码 12.5%+5% 参数调整后失真）
- 审批联动只认终审：财务节点通过（1→2）不再提前 mark_locked/调整单生效，
  否则领导驳回后 locked_at/locked_by 残留、两级审批形同虚设；驳回在任何
  节点即时联动；mark_rejected 补清锁定字段
- 调整单新建不可用：ENABLE_CODE_BY_TYPE 未登记 type=3 → is_enabled 恒 400；
  补登记 approval.adjustment.enabled。另修复叠加缺陷——开关关闭分支手工
  ApprovalStep(order_id=调整单id)，而该外键指向 approval_orders，必然
  errno 1452；统一改走 submit_order（自动通过留痕由它负责）
- 重算双重计提：业务级一次性计提的「进度登记」回款无生效台账行，重算会
  凭空落新行；无旧行一律 400
- 前端重算传错 id：台账行主键当 receipt_id 传给重算接口，会作废无关回款
  的台账；改传 row.receipt_id（聚合行防御）
- quantize 显式 ROUND_HALF_UP：9 处金额 quantize 缺省银行家舍入，
  x.xx5 与四舍五入口径差 1 分（docstring 却声称 HALF_UP）

测试：显式控制审批开关（V8 起种子默认开启，用例不再依赖默认值）；
新增回归 4 例（经理聚合比例/调整单即生效/进度登记重算 400/财务节点不提前
锁定），test_calc+test_settlements 22 passed

Co-Authored-By: Claude Code <noreply@anthropic.com>
 2026-10-19 11:25:21 +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