All checks were successful
构建并部署到测试环境(无 SSH) / build-and-deploy (push) Successful in 9m26s
用 PRD 2.14 语义整体替换退款佣金「整单全额失效」实现:原佣金保持已发放不变, 回溯事实落在新表 tb_commission_clawback_record 的负数、不可提现明细上。 - 新增成对迁移 000220 建 tb_commission_clawback_record,唯一约束 (refund_id, original_commission_id) 为权威幂等键,附店铺+时间/原佣金/订单索引。 - 回溯用例(internal/service/refund/clawback.go):准入仅由退款申请状态、审批异常 标记与退款方式决定;金额按分整数计算,分母取冻结实收(缺失回落审批尝试)、 分子原路取渠道成功金额,乘法用 math/big 中间量,舍入差自末条起向前补差; 终态判据要求订单佣金已离开待计算且不存在 status IN (1,2,99) 的记录。 - 三层幂等:唯一约束兜底、佣金行行锁 + 钱包乐观锁、commission_deducted 仅作投影 并带 WHERE commission_deducted = false 条件置位;闭合三结果为已回溯、无需回溯、 审批异常转人工。 - 事务内顺序固定:锁提现申请行 → 锁尝试行 → 解冻冻结 → 置驳回 → 插回溯明细 → 扣 balance(允许为负)→ 写负数流水 → 审计;删除旧全额失效写入与其两个审计调用点, refund.invalidate_commission 仅保留常量与注册供历史审计读取。 - 读侧:佣金明细列表 status 筛选透传,两表 UNION ALL 合并分页并以 source ASC 作 末位次序键;新增佣金明细详情接口并同步路由与 OpenAPI 装配。 - 导出:新增 commission_record 场景(白名单、exporter 注册、DTO oneof、DataSource 与列定义),粒度为佣金记录,原佣金与回溯各一行,金额保持分且可为负。 - 新增退款佣金回溯周期补偿任务(@every 1m / MaxRetry(3) / Timeout(10m) / Unique(10m),独立队列),保留启动时补偿扫描,判据与既有实现一致。 Refs: AUG26-012
20 KiB
20 KiB
1. 事实与常量
- 1.1 新增
CommissionStatusClawback = 5与GetCommissionRecordStatusName的「回溯」映射,并同步佣金记录状态常量注释与internal/model/commission.go的说明。 证据:pkg/constants/iot.go:214新增CommissionStatusClawback = 5,:345注释同步为…4=已失效, 5=回溯, 99=待人工修正,:356-357新增case CommissionStatusClawback: return "回溯";internal/model/commission.go:23列注释同步含5-回溯,:12补说明「退款佣金回溯不改动本表」;迁移 000220 同步COMMENT ON COLUMN tb_commission_record.status。 - 1.2 新增回溯审计动作码
refund.clawback_commission(生成回溯明细)与refund.clawback_not_required(终态确无佣金),在internal/infrastructure/audit/registry.go注册;保留refund.invalidate_commission常量与注册以维持历史审计可读,并删除其全部业务调用点。 证据:pkg/constants/audit.go:363-366新增AuditActionRefundClawbackCommission = "refund.clawback_commission"与AuditActionRefundClawbackNotRequired = "refund.clawback_not_required";:360-362旧常量补注释「保留常量与注册仅供历史审计读取,代码中已无调用点」;internal/infrastructure/audit/registry.go:313-314用refundSystemAction定义、:644-645登记、:311/:643保留旧动作注册。旧调用点appendCommissionAudit、recordCommissionFailure已整体删除(internal/service/refund/audit.go现无该常量引用),全仓 grep 仅剩常量定义与注册两处。 - 1.3 核对回溯用例 Audit Event、Domain Ledger、Integration Log 与 Outbox 四类事实的使用决定或 N/A 理由,并在仓库外记录证据(ENG-AUDIT-001)。 证据:四类事实决定记录于仓库外
/tmp/aug26012-evidence/07-audit-facts.md:Audit Event 使用、Domain Ledger 使用、Integration Log N/A(本用例不调用外部系统,理由已写明)、Outbox 复用既有事件。落地实现见internal/service/refund/clawback.go的applyClawback/settleClawback/appendClawbackAudit(审计与资金同事务、AppendAndGet要求成功必达)。
2. Schema 与模型
- 2.1 新增成对迁移建
tb_commission_clawback_record:refund_id、original_commission_id、order_id、shop_id、负数amount、不可提现标识、status、balance_after、原佣金与退款关键快照、created_at;迁移编号按migrations/目录当前最大编号顺延,不预占编号,不修改任何既有迁移。 证据:新增成对迁移migrations/000220_add_commission_clawback_record.up.sql/.down.sql;编号按migrations/当前最大 000219 顺延,未修改任何既有迁移(git status中 migrations 仅新增这两个文件)。建表含refund_id、original_commission_id、order_id、shop_id、order_no、refund_no、commission_source、负数amount、balance_after、withdrawable、status、created_at,及chk_commission_clawback_amount CHECK (amount < 0)、chk_commission_clawback_withdrawable CHECK (withdrawable = FALSE)、chk_commission_clawback_status CHECK (status = 5)。created_at采用TIMESTAMP(不带时区),与另一支tb_commission_record.created_at(实测timestamp without time zone)保持一致:若用timestamptz,UNION ALL会按会话时区改写其中一支,破坏统一排序键与时间筛选口径;S-4 同时把两个回溯分支的占位列由NULL::timestamptz改为NULL::timestamp,消除该耦合(实测【S-4】released_at 跨会话时区稳定:Shanghai/NewYork/UTC 三会话同值)。 - 2.2 迁移内建唯一约束
(refund_id, original_commission_id),以及店铺 + 时间、original_commission_id、order_id索引;down仅删除本 Change 新增对象。 证据:迁移内建CONSTRAINT uk_commission_clawback_refund_original UNIQUE (refund_id, original_commission_id)与三个索引idx_commission_clawback_shop_created (shop_id, created_at DESC, id DESC)、idx_commission_clawback_original (original_commission_id)、idx_commission_clawback_order (order_id);down在存在回溯明细时RAISE EXCEPTION阻断后DROP TABLE,并还原tb_commission_record.status列注释,不触碰既有表与数据。 D-2 修复:down还原的tb_commission_record.status列注释改为 000220 up 之前的真实值状态 1-已入账 2-已失效(最后一次写该注释的是migrations/archive/000029_add_one_time_commission.up.sql:94;000113 只迁移状态值未改注释),使 down 成为 up 的严格逆操作(ENG-MIG-001)。实测:down 后 dbhub 只读核对该注释为状态 1-已入账 2-已失效,up 后为状态 1-已冻结 2-解冻中 3-已发放 4-已失效 5-回溯 99-待人工修正(见01-migration.txt)。 - 2.3 新增回溯明细模型与 DTO(含中文 description 与状态枚举名,ENG-DTO-001)。 证据:新增
internal/model/commission_clawback.go的CommissionClawbackRecord(表名tb_commission_clawback_record);DTO 见internal/model/dto/shop_commission_dto.go的ShopCommissionClawbackItem、ShopCommissionRecordItem(status 描述为1:已冻结, 2:解冻中, 3:已发放, 4:已失效, 5:回溯, 99:待人工修正,与pkg/constants一致)、ShopCommissionRecordDetailReq/Resp,均为中文 description。
3. 写路径替换
- 3.1 实现回溯准入:仅
refund.status = 2且anomaly_flag = 0且(非原路方式或channel_refund_status = 2)时生成;其余状态返回可重试错误,不写明细、不置完成事实;不使用失败分类标记判定准入。 证据:internal/service/refund/clawback.go:clawbackCommission——refund.AnomalyFlag != 0先走转人工闭合;refund.Status != model.RefundStatusApproved返回CodeServiceUnavailable;refund.Method == original_route && ChannelRefundStatus != 2返回可重试错误。未引用RefundFailureReasonIsDefinitive(grep -rn "RefundFailureReasonIsDefinitive" internal/service/refund/为空)。实测见04-write-path.txt:「原路在途不回溯」「渠道失败不回溯」「待审批等待」通过。 准入与终态判据见internal/service/refund/clawback.go:67-97;终态判据除「订单佣金非待计算」外,还要求该订单不存在status IN (1,2,99)的未终态佣金记录(D-1 修复,理由与收敛条件见 design 第 3 条)。收敛性依据实测的全仓状态写点:tb_commission_record.status仅被commission_calculation/service.go:181/:241/:554(置 3)、:220/:581(置 99)、:689-693(入账置 3)、recharge_order/service.go:418(置 3)与shop_commission/service.go:751/:781(人工处置置 4 或 3)写入,从不写入 1/2(遗留枚举,仅历史读取与名称映射用),故现实等待只可能由 99 引起且处置后必然收敛。。 - 3.2 实现金额计算:分母取本次退款冻结实收金额(缺失时回落审批尝试记录),
<= 0时记录可恢复失败且不落库、不改余额;分子原路取渠道明确成功金额、其余取审批退款金额;乘法使用math/big或等价大整数中间量,全程int64分,禁止浮点与溢出。 证据:resolveClawbackDenominator(退款行缺失时回落LatestAttemptID对应尝试的frozen_actual_received_amount)、resolveClawbackNumerator(原路取ChannelRefundAmount,其余取ApprovedRefundAmount);D <= 0记录可恢复失败并返回错误,不落库、不改余额、不置位;乘法用math/big中间量(mulDivFloor/mulDivFloorInt64),全程 int64 分。实测见04-write-path.txt:「冻结实收非正 D=0」「分母回落+原路分子」「乘积极端值 6148914691236517204 / [6148914691236517204 6148914691236517205]」通过。 - 3.3 实现分摊:稳定顺序
original_commission_id ASC;剩余_i、逐条base_i、应回溯总额上限,逐条向下取整,舍入差自末条起向前分摊且每条不超过该条剩余。 证据:allocateClawbackAmounts—— 输入按original_commission_id ASC固定;剩余_i = amount_i − 已生成回溯累计(loadClawedAmounts从明细聚合);base_i = min(floor(amount_i×N/D), 剩余_i);T = min(floor(Σamount_i×N/D), Σ剩余_i);补差自末条向前逐条受剩余_i − base_i限制。实测见04-write-path.txt:「比例与补差 -66/-66/-68 总额-200」「补差跨条向前,末两条剩余=0 被跳过 → 各 -1000 总额-2000」「Σ剩余封顶 总额-2100」通过。 - 3.4 实现后处理闭合的三种结果与置位保护:已生成回溯、终态无需回溯、审批异常转人工;置位
commission_deducted加WHERE commission_deducted = false保护;订单佣金未终态时不写明细、不置位并返回可重试错误。 证据:clawbackCommission三种闭合——已生成回溯(applyClawback,置位WHERE id = ? AND commission_deducted = false并判RowsAffected != 1→CodeConflict)、终态无需回溯(settleClawback+refund.clawback_not_required)、审批异常转人工(settleClawback复用refund.anomaly_flag);订单CommissionStatusPending时返回可重试错误、不写明细、不置位。旧实现两处无保护的Update("commission_deducted", true)已随deductAllCommission整体删除。实测见04-write-path.txt:「未终态等待」「终态确无佣金」「撤销不回溯」「置位保护」通过。 另实测 D-1 修复后的闭合边界(04-write-path.txt):「order=3 + 仅 99 记录」返回可重试错误(err=订单存在未终态佣金记录,等待终态后再回溯)且无明细/未置位/无审计;99 处置为已发放 5000 后生成 -5000 明细并闭合;99 处置为已失效后写「无需回溯」审计并闭合(不永久等待);order=2 确无佣金保持原行为。 - 3.5 删除旧全额失效写入(原佣金置已失效、旧审计前后态与调用点),替换为插入回溯明细;保留
collectPendingWithdrawalRejects、applyPendingWithdrawalRejects、appendWithdrawalRejectAudit,事务内顺序固定为:锁提现申请行 → 锁尝试行 → 解冻冻结 → 置驳回 → 插回溯明细 → 扣balance(允许为负)→ 写负数流水 → 审计。 证据:deductAllCommission/deductSingleCommission/appendCommissionAudit/recordCommissionFailure已整体删除(原佣金置已失效写入、旧审计前后态与全部调用点消失)。applyClawback事务内顺序为:锁原佣金行 → 按店铺升序collectPendingWithdrawalRejects(申请行 → 尝试行)→ 锁佣金钱包行 →applyPendingWithdrawalRejects(解冻 + 置驳回 + 正向流水 + 审计)→ 插回溯明细 → 扣balance(允许为负)→ 写commission_deduct负数流水 → 审计 → 条件置位。collectPendingWithdrawalRejects/applyPendingWithdrawalRejects/appendWithdrawalRejectAudit未重写。实测见04-write-path.txt:「释放先于扣款 frozen 800→0、balance 1000→-4000、提现已拒绝、尝试已释放、原佣金不变」通过。 S-1 修复:本次未生成任何明细时(仅投影陈旧可达成)不写refund.clawback_commission审计,避免clawback_record_count=0 且 clawback_required=true的误导性事实;置位逻辑不变,正常路径(确有明细)审计与 metadata 计数不变(实测【S-1】与【S-1 对照】:正常路径 metadata={"order_id": 540, "clawback_required": true, "clawback_record_count": 2, "clawback_total_amount": -5000})。 - 3.6 新增周期性补偿任务:
@every 1m、MaxRetry(3)、Timeout(10m)、Unique(10m)、独立队列,配套任务类型常量、QueueForTaskType映射与 Handler;保留既有启动时补偿扫描,判据保持status = 2 且 commission_deducted = false。 证据:pkg/constants/constants.go:95新增TaskTypeRefundCommissionRecovery = "refund:commission:recovery",:311纳入QueueForTaskType;internal/infrastructure/commissiondelivery/recovery_task.go新增RefundRecoveryTaskHandler(只重投稳定事件,不直接改资金);cmd/worker/main.go:502-511注册 Handler、:164调用,Scheduler 以@every 1m+MaxRetry(3)+Timeout(10m)+Unique(10m)+Queue(QueueForTaskType(...))注册(与TaskTypeRefundChannelRecovery同形态)。补偿扫描判据与 HEAD 一致、未新增任何列条件:仍是status = 2 AND (commission_deducted = false OR asset_reset = false)(internal/infrastructure/commissiondelivery/event.go的RecoverRefundPostProcessing,同时覆盖资产后处理),仅将退款部分抽出为独立函数供启动扫描(cmd/worker/main.go:413的commissionDelivery.Recover保留不变)与周期任务共用。
4. 读侧与导出
- 4.1 列表支持
status筛选透传(DTO 与 service);读侧两表UNION ALL合并分页,统一排序键created_at DESC, id DESC,筛选与数据范围在合并前各自应用。 证据:DTOinternal/model/dto/shop_commission_dto.go:151新增Status *int;Serviceinternal/service/shop_commission/service.go:283赋值Status: req.Status并补PageSize > MaxPageSize归一化;Store 新增ListLedgerByShopID,两支先各自Count与各自应用数据范围/筛选,再UNION ALL合并后分页。排序与次序键为created_at DESC→id DESC→source ASC:两表自增 ID 空间独立,同一created_at下(created_at DESC, id DESC)不是全序,必须再以source ASC作末位次序键才能在created_at与id完全相同的跨表记录上保证翻页不重不漏(design 第 8 条已同步该理由)。实测见05-read-path.txt:「合并分页 total=3 逐页 size=1 无重复无缺失」「status=5→1 条回溯 / status=3→1 条原佣金」「同秒排序稳定」通过;次序键反例已实测:强制两表插入created_at与id完全相同的记录(id=987654321)后按 size=1 翻页,两行仍不重不漏且次序稳定为[clawback original](【次序键复验】)。 - 4.2 原佣金返回
clawback_records摘要;回溯明细返回原佣金标识、退款单号、负数金额、不可提现标识、回溯后余额与生成时间;越权按既有数据范围返回不存在或空结果。 证据:原佣金行返回clawback_records摘要(ListClawbackSummaries)与clawback_total_amount;回溯行返回original_commission_id、refund_id、refund_no、负数amount、withdrawable、balance_after、status_name=回溯。越权在 Service 用middleware.CanManageShop+ 详情内row.ShopID != req.ShopID双重拦截,错误文本与「不存在」一致。实测见05-read-path.txt:「越权不可枚举」(越权列表/详情失败、不存在返回「佣金明细不存在」、范围内可见 2 条、详情双向定位)通过。 - 4.3 新增佣金明细详情接口,并同步可执行路由、
cmd/api/docs.go与cmd/gendocs/main.go装配。 证据:internal/handler/admin/shop_commission.go:108-134新增GetCommissionRecord;internal/routes/shop.go:162-169新增GET /:shop_id/commission-records/:idRouteSpec(含中文 Description);pkg/openapi/handlers.go:32的BuildDocHandlers已含ShopCommission,cmd/api/docs.go与cmd/gendocs/main.go同构调用该入口(未新增 Handler 类型,故无需改这两处的显式赋值)。实测:go run cmd/gendocs/main.go退出码 0(08-gendocs.txt),docs/admin-openapi.yaml出现/api/admin/shops/{shop_id}/commission-records/{id}。 - 4.4 落地佣金明细导出场景:场景白名单、
internal/exporter注册、DTOoneof、DataSource 与列定义;粒度为佣金记录,原佣金与回溯记录各一行,金额保持分且可为负,冻结创建时筛选条件、操作者与可见范围。 证据:pkg/constants/constants.go:370新增ExportTaskSceneCommissionRecord = "commission_record";internal/exporter/registry.go:39注册NewCommissionRecordDataSource、:76纳入IsSupportedScene白名单;internal/model/dto/export_task_dto.go三处oneof与描述补齐;新增internal/exporter/commission_record_scene.go(列:记录来源/记录ID/代理店铺名称/关联订单号/资产标识/佣金来源/金额(元)/是否可提现/状态/回溯后佣金余额(元)/原佣金记录ID/来源退款单号/佣金入账时间/生成时间)。实测见06-export.txt:「原佣金与回溯各一行,回溯金额 -90.00、余额 -40.00 原样导出,数据范围生效」通过。
5. 验证
- 5.1 按 ENG-TEST-001 在维护者指定的
junhong_cmp_testPostgreSQL 与 Redis DB 6 上,以本地显式DB_*参数执行./scripts/migrate.sh完成 up/down/up;仅创建与删除本 Change 自己的 fixture,禁止重置整个测试库。 证据:见/tmp/aug26012-evidence/01-migration.txt(含 D-2 修正后的 down 注释还原核对):显式DB_*(junhong_cmp_test)执行./scripts/migrate.sh up(220 应用成功)→version返回 220 →down 1回滚 220 →version返回 219 →up重新应用到 220 →version返回 220。全部 fixture 建在回滚事务内,收尾核查tb_commission_clawback_record行数为 0、验证用店铺数为 0、验证用退款单数为 0(dbhub 只读查询,见10-fixture-cleanup.txt),未重置整库。 - 5.2 验证金额:全额与部分回溯、比例与补差、乘法溢出行边界、累计不超过各原佣金剩余、冻结实收非正可恢复失败。 证据:见
04-write-path.txt(17 项全部 PASS):全额回溯(-10000/-25000/-555,余额 64445)、比例与补差(-66/-66/-68 总额-200)、补差跨条向前(末两条剩余=0 被跳过 → 各 -1000 总额-2000)、Σ剩余封顶(-2100)、乘积极端值(单条/多条 int64 max 无溢出)、冻结实收非正(D=0 与 D<0 均不落库不改余额不置位)、分母回落 + 原路分子(-5000)。 - 5.3 验证时序与幂等:佣金未终态等待、终态确无佣金、原路在途不回溯、企业微信通过后撤销不回溯、渠道失败重提后按最终金额回溯一次、重复消费不重复。 证据:同上
04-write-path.txt:佣金未终态等待、终态确无佣金(置位 +refund.clawback_not_required)、原路在途不回溯、撤销不回溯(anomaly_flag=1转人工且闭合)、渠道失败重提后按最终金额回溯一次(-6000)、重复消费不重复(3 次 → 1 明细 1 流水)、唯一约束权威(重复插入被数据库拒绝)、并发消费(已有明细时跳过重复扣款并闭合)。 - 5.4 验证资金与读侧:释放先于扣款且负余额不违约、原佣金状态与金额不变、读侧合并分页与越权、导出负数余额与原样导出。 证据:见
04-write-path.txt的「释放先于扣款且负余额不违约」(frozen 800→0、balance 1000→-4000、提现已拒绝、尝试已释放、原佣金 status/amount/source/released_at 全部不变);05-read-path.txt(4 项 PASS)的合并分页、status 筛选、越权不可枚举、同秒稳定排序;06-export.txt的回溯行金额-90.00与回溯后余额-40.00原样导出、粒度两行、数据范围生效。 - 5.5 运行
gofmt -w、go build ./cmd/api ./cmd/worker、go run cmd/gendocs/main.go、openspec validate add-commission-clawback-records --strict、openspec doctor --json与./scripts/context-health.sh;自动化测试按项目决策为 N/A。 证据:见02-build.txt(退出码逐一显式记录,不经管道过滤):gofmt -l对全部 22 个变更/新增 Go 文件无输出;go build ./cmd/api ./cmd/workerBUILD_EXIT=0(构建输出仅含 go 模块缓存写权限噪音,无编译错误);go vet ./internal/... ./pkg/...VET_EXIT=0 且无输出;go run cmd/gendocs/main.goGENDOCS_EXIT=0。openspec validate add-commission-clawback-records --strict→Change is valid(VALIDATE_EXIT=0);openspec doctor --json→"healthy": true;./scripts/context-health.sh→Context 健康检查通过(CTX_EXIT=0,见03-context-health.txt)。自动化测试按项目决策为 N/A:临时验证脚手架已全部删除,收尾后find . -name '*_test.go' | wc -l为 0、find . -name 'zz_*' | wc -l为 0,仓库未新增测试入口。