 左辰勇
|
70aa8dcde2
|
fix(alembic): 合并 88f2380 引入的双 head,恢复 upgrade head 可用
88f2380 把 origin/main 合进本分支后,两条迁移线在 git 上汇合了、在 alembic 图上
却没有:本分支的 ad_reward_boost_round_id → drop_signin_boost_record 与 main 的
merge_active_phone 同从 comparison_llm_cost 分叉,此后再无迁移 revise
drop_signin_boost_record,它成了悬空的第二个 head。
后果:`alembic upgrade head`(单数)报 "Multiple head revisions are present"。
按 CLAUDE.md,run.sh 启动即自动迁移 —— app server 直接起不来,CI/线上同样报错。
用标准 merge revision 收敛(而非改 down_revision 指向),保留两侧作者文件不动。
纯 merge,无表结构/数据改动。
验证:alembic heads 恰好一个 merge_signin_boost_main;空库 alembic upgrade head
全链路跑通,current 停在 (head) (mergepoint);ruff 通过;pytest 435 passed
(6 个既有失败,与本次改动前一致)。
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
2026-07-21 20:27:14 +08:00 |
|
|
左辰勇
|
de1fd58749
|
feat(ad): 膨胀弹窗改用服务端权威金额 + 本轮累计口径,下线 signin_boost
要守住的不变量:弹窗数字 == 本轮实际到账之和 == 用户看到的余额涨幅。三者对不上,
用户就会认为少发了钱(走查现象:弹窗 240、余额只涨 40)。
- reward-result:按 ad_session_id 查本次实发金币,替代余额差 / coin_per_ad 估算。
S2S 异步未到账返 200+pending 而非 404(404 只表示路由不存在,混在一起客户端没法
区分「后端没部署」和「再等等」);同 session 多条时显式优先 granted——客户端先报
closed_early、S2S 后到时,granted 反而是后写的。
- boost_round_id:客户端经 mediaExtra 透传「这条广告属于哪一轮膨胀」,穿山甲 S2S 原样
带回后随发奖记录落库。**纯标签,不参与发奖判定**。reward-result 新增 round_coin,按
(user_id, boost_round_id) 对 granted 记录求和。之所以由服务端求和而非客户端自己累加
——客户端进程被杀/重建后本地累计会丢,发奖记录不会。
· 求和恒带 user_id:轮 id 是客户端生成的,不带就等于让任何人拿别人的轮 id 查别人发了多少。
· 本条非 granted(capped 等)时仍返本轮累计、该条按 0 计,让限额 toast 有数可显。
· test-grant 加可选 boost_round_id:它不经 S2S 拿不到 extra,不补则 debug 包验不了累计。
· 客户端复用同一轮 id 只会把展示数字滚大,求和的是已发生的记录,不产生新入账,无资损。
- 下线 signin_boost(签到膨胀):它按固定 3000 金币发、与广告实际收益脱钩,产品确认从来
不是设计内的口径——奖励只有「签到」和「看视频」两种。签到弹窗的「看广告膨胀」改与福利页
看视频同走 reward_video(按 eCPM 公式)。摘除回调分支、POST /signin/boost、
SigninBoostRecord、signin_boost_coin 配置,并 drop signin_boost_record 表。
**coin_transaction.biz_type='signin_boost' 的历史流水保留不动**——钱是真发过的,账必须
留得住;admin 大盘那两项改从金币流水统计(一次膨胀 = 一笔,与原口径等价),继续能查回历史。
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
2026-07-20 16:55:25 +08:00 |
|
guke