修复比价记录机型与 ROM 版本展示 #183

2026-07-27T14:37:31+08:00

linkeyu commented

2026-07-27 14:37:31 +08:00

改动说明

比价记录列表和详情接口补充可读机型名
列表接口返回 ROM 大版本
保留原始设备编码，方便排查
增加列表与详情接口回归测试

验证

后端测试：20 项通过

## 改动说明 - 比价记录列表和详情接口补充可读机型名 - 列表接口返回 ROM 大版本 - 保留原始设备编码，方便排查 - 增加列表与详情接口回归测试 ## 验证 - 后端测试：20 项通过

linkeyu added 1 commit 2026-07-27 14:37:32 +08:00

修复：补全比价记录设备信息展示 973c440a8a

为比价记录列表与详情补充可读机型名和 ROM 版本字段，并增加接口回归测试。

linkeyu changed title from ~~????????? ROM ????~~ to 修复比价记录机型与 ROM 版本展示

2026-07-27 14:38:15 +08:00

guke commented

2026-07-27 16:37:51 +08:00

🤖 review-pr 深审结论

🟢 可以合（置信度 0.9）。已在隔离 worktree 实跑验证:对当前 main（58d609e）干净合入;pytest -k comparison 3 passed(含新测试 test_comparison_records_show_readable_device_and_rom_version 单跑 1 passed);V2166BA → vivo Y77e 映射确在 _DEVICE_MARKETING_NAMES。

改动

新增 _attach_comparison_device_details(),给每条记录挂瞬态 device_model_name = _device_marketing_name(device_model),沿用既有 _attach_user_info / ad_revenue_yuan 瞬态挂载范式;list 与 detail 两处入口都补挂。
rom_version 从 AdminComparisonDetail 提升到 AdminComparisonListItem（Detail 继承,详情侧不丢),并加 device_model_name。未知机型 → device_model_name=None,前端回退原始编码,不影响。

⚠️ 唯一需协调:与 #184 的落地顺序
兄弟 PR #184（补充是否下单状态)改同 3 文件。实测 merge #184 ↔ #183 会在 queries.py（helper 定义 + list_comparison_records 的 attach 调用)与 tests/test_admin_read.py（测试插入锚点)冲突,但都是「各加一个独立函数/调用/测试」的机械冲突,device_model_name 与 ordered 互不干扰 —— 后落地者 keep-both 即可,无逻辑不兼容。前端侧 #87 ↔ #88 则自动合并无冲突。

搭配前端 admin-web #87,建议后端先上(前端对缺字段优雅降级)。

## 🤖 review-pr 深审结论 🟢 **可以合**（置信度 0.9）。已在隔离 worktree 实跑验证:对当前 main（58d609e）干净合入;`pytest -k comparison` 3 passed(含新测试 test_comparison_records_show_readable_device_and_rom_version 单跑 1 passed);`V2166BA` → `vivo Y77e` 映射确在 _DEVICE_MARKETING_NAMES。 **改动** - 新增 `_attach_comparison_device_details()`,给每条记录挂瞬态 device_model_name = _device_marketing_name(device_model),沿用既有 _attach_user_info / ad_revenue_yuan 瞬态挂载范式;list 与 detail 两处入口都补挂。 - rom_version 从 AdminComparisonDetail 提升到 AdminComparisonListItem（Detail 继承,详情侧不丢),并加 device_model_name。未知机型 → device_model_name=None,前端回退原始编码,不影响。 **⚠️ 唯一需协调:与 #184 的落地顺序** 兄弟 PR #184（补充是否下单状态)改同 3 文件。实测 merge #184 ↔ #183 会在 queries.py（helper 定义 + list_comparison_records 的 attach 调用)与 tests/test_admin_read.py（测试插入锚点)冲突,但都是「各加一个独立函数/调用/测试」的机械冲突,device_model_name 与 ordered 互不干扰 —— 后落地者 keep-both 即可,无逻辑不兼容。前端侧 #87 ↔ #88 则自动合并无冲突。搭配前端 admin-web #87,建议后端先上(前端对缺字段优雅降级)。

guke added 1 commit 2026-07-27 16:38:53 +08:00

Merge branch 'main' into codex/comparison-device-display ab034d3c15

guke merged commit 36ce18a250 into main

2026-07-27 16:38:58 +08:00

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: WonderableAI/shaguabijia-app-server#183