jackyu66gitandCursor
7ab9add5dd
落地埋点 ingest/数据看板、首页宫格 CMS 与测评上下架；含账号引导、问答流式与免责声明去重，以及 review P1 审计同事务修复。 Co-authored-by: Cursor <cursoragent@cursor.com>
35 lines
1.2 KiB
Markdown
35 lines
1.2 KiB
Markdown
# CODE_REVIEW — ECR-008
|
||
|
||
**Verdict:** Approve（Feature Complete · Ops-C · P1 已修）
|
||
|
||
Date: 2026-08-07 · Reviewer pass → P1 fixed
|
||
|
||
## Checklist
|
||
|
||
- [x] Handler → Service → Repository；home DeviceAuth；admin AdminAuth
|
||
- [x] Feature Spec Active + BD-2026-008 Approved
|
||
- [x] OpenAPI home/admin content paths
|
||
- [x] Migration `000014_ops_content` · ERD
|
||
- [x] path/icon 白名单；**审计与变更同事务**（ReplaceAllWithAudit / UpdateStatusWithAudit）
|
||
- [x] C 端 Get/Submit 仅 published scales
|
||
- [x] H5 静态回退；integration 绿（含 audit 断言 + tools restore）
|
||
|
||
## P1 — resolved
|
||
|
||
| ID | Fix |
|
||
|----|-----|
|
||
| P1-1 审计非原子 | `HomeToolsRepo.ReplaceAllWithAudit` · `ScaleRepo.UpdateStatusWithAudit` |
|
||
| P1-2 ↑↓ 破坏 row_index | `ContentPage.move` 仅交换位置并按行重算 sort_order |
|
||
| P1-3 Analytics nil | `requireAnalytics` 统一守卫 |
|
||
|
||
## P2（可跟进）
|
||
|
||
- Scales 直连 ScaleRepo；C 端多暴露字段；icon 白名单三处重复；OpenAPI schema 偏薄
|
||
|
||
## Architecture Regression
|
||
|
||
- [x] 无 Handler 直连 DB
|
||
- [x] Admin / Device 鉴权隔离
|
||
- [x] 未引入非默认中间件
|
||
- [x] 审计与写操作同事务
|