# 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] 审计与写操作同事务  

## Closed decision（2026-08-07）

```text
Load Agent Profile: REVIEWER
Decision: Approve → Closed
Evidence: TEST_REPORT/ECR-008.md · P1 已修 · Acceptance 全勾 · ess-validate --phase review PASS
Residual: P2 可跟进项不阻断 Closed；部署属 RELEASE_MANAGER + Human
```