jackyu66gitandCursor
7ab9add5dd
落地埋点 ingest/数据看板、首页宫格 CMS 与测评上下架；含账号引导、问答流式与免责声明去重，以及 review P1 审计同事务修复。 Co-authored-by: Cursor <cursoragent@cursor.com>
28 lines
830 B
Markdown
28 lines
830 B
Markdown
# CODE_REVIEW — ECR-007
|
||
|
||
**Verdict:** Approve（Feature Complete · Ops-B）
|
||
|
||
Date: 2026-08-07
|
||
|
||
## Checklist
|
||
|
||
- [x] Handler → Service → Repository；ingest 挂 DeviceAuth；聚合挂 AdminAuth
|
||
- [x] Feature Spec Active + BD-2026-007 Approved
|
||
- [x] OpenAPI：`POST /analytics/events` · `GET /admin/analytics/*`
|
||
- [x] Migration `000013_analytics_ops` · ERD 同步
|
||
- [x] 文件 ≤400；integration + H5 单测绿
|
||
- [x] props 白名单；无 PII 采集
|
||
- [x] 未引入 Kafka/ClickHouse
|
||
|
||
## Architecture Regression
|
||
|
||
- [x] 无 Handler 直连 DB
|
||
- [x] Admin / Device 鉴权面隔离
|
||
- [x] 未简化分层
|
||
|
||
## Residual（不阻断）
|
||
|
||
- ingest 逐条 INSERT（量上来再批量 COPY / 日汇总表）
|
||
- admin「数据」未进 `@yuxingu/sdk`（与 Ops-A 一致用 thin client）
|
||
- Ops-C/D/E 另开 ECR
|