feat(compare/coupon): trace_id 统一由后端签发,前端不再本地生成 #210

Merged
marco merged 2 commits from feat-unify-trace-id-backend-issued into main 2026-07-31 23:11:36 +08:00

2 Commits

Author SHA1 Message Date
marco 93bf991fe1 feat(coupon): /coupon/step 每帧响应顶层回显 trace_id,补齐领券链路
对齐 compare 链路(_forward 的 setdefault 注入,每步响应都带 trace_id):coupon_step
透传响应在 resp.json() 后 setdefault 注入本次任务 trace_id,客户端任一帧都能从响应
拿到全链 id。**只回显请求里带的、不 mint**——step 是循环接口,每帧签新 id 会把一次
任务打散;领券 trace_id 的唯一签发点保持在 /coupon/session (status=started)。带
isinstance dict + 非空 trace_id 双防御(pricebot 响应顶层本无 trace_id 字段,setdefault
不会覆盖任何上游值)。

顺手修复 test_coupon_proxy.py 一个 pre-existing 红测试:coupon_step 转发早已改为
content=raw 原始字节透传,但 fake_post 仍只捕获 json= 参数 → captured["json"] 恒
None、body 转发断言一直失败(git stash 验证不带本轮改动同样红)。fake 改捕 content、
断言按字节 json.loads 后对比;transparently-passes-through 断言更新为"透传 + 顶层
回显 trace_id"新契约。

领券相关测试 22 passed。

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
2026-07-31 23:08:43 +08:00
marco a73309ac07 feat(compare/coupon): trace_id 统一由后端签发,前端不再本地生成
一次比价/领券的 trace_id 改由后端签发,让前端 SLS 运行日志(trace_id 索引列)、
app-server 比价记录与领券流水、pricebot trace 目录/run.log/trace_url 全链共用同一个
id 查到底(此前前端各业务自己 randomUUID,虽同链但非后端签发、也无单一签发点)。

- POST /api/v1/compare/start(预占额度,任务第一个请求,签发与建 running 行合一):
  请求 trace_id 改可选,缺省时服务端签发 uuid;响应新增 trace_id 字段返回(签发的或
  回显客户端带来的)。客户端带值则沿用——老客户端兼容 + 同 trace 重试幂等。
- POST /api/v1/coupon/session:started 帧缺 trace_id 时签发并随响应返回(签发不依赖
  写库成功);非 started 帧缺 trace_id 不签发、不写库(收尾没有 id 只能是异常调用,
  签发新 id 只会造出查不到发起信息的孤儿行)。新增 CouponSessionOut 响应模型——原
  dict[str,bool] 注解无法承载字符串 trace_id,FastAPI 响应校验会炸。
- 测试:更新 2 处旧断言(响应体多出 trace_id 字段),新增 compare 不带 id 签发用例 +
  coupon started 签发/回显、终尾缺 id 跳过写库 3 个用例。全量 30 passed + ruff clean。

配合 shaguabijia-app-android 同名分支 feat-unify-trace-id-backend-issued 的前端换源改动。

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
2026-07-31 21:31:24 +08:00