jackyu66gitandCursor
7ab9add5dd
落地埋点 ingest/数据看板、首页宫格 CMS 与测评上下架；含账号引导、问答流式与免责声明去重，以及 review P1 审计同事务修复。 Co-authored-by: Cursor <cursoragent@cursor.com>
1.2 KiB
1.2 KiB
CODE_REVIEW — ECR-008
Verdict: Approve（Feature Complete · Ops-C · P1 已修）
Date: 2026-08-07 · Reviewer pass → P1 fixed
Checklist
- Handler → Service → Repository；home DeviceAuth；admin AdminAuth
- Feature Spec Active + BD-2026-008 Approved
- OpenAPI home/admin content paths
- Migration
000014_ops_content· ERD - path/icon 白名单；审计与变更同事务（ReplaceAllWithAudit / UpdateStatusWithAudit）
- C 端 Get/Submit 仅 published scales
- 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
- 无 Handler 直连 DB
- Admin / Device 鉴权隔离
- 未引入非默认中间件
- 审计与写操作同事务