review 结果
This commit is contained in:
Shen Wei
@@ -1,441 +0,0 @@
|
||||
# Fonrey 全局系统设计 Review 报告
|
||||
|
||||
> **Review 类型**：全量 Review（PRD + DATA_MODEL + TECH_STACK + UI/UX 交叉验证）
|
||||
> **Review 日期**：2026-04-25
|
||||
> **Reviewer**：系统设计 Reviewer（AI 辅助）
|
||||
> **当前阶段**：需求 80% / 数据模型 50% / UI 未开始
|
||||
> **覆盖文档**：8 份 PRD、8 份 DATA_MODEL、3 份 TECH_STACK、2 份 UI/UX
|
||||
> **问题分级**：🔴 Blocker（阻塞开发） / 🟠 Major（必须修复但不阻塞） / 🟡 Minor（建议优化）
|
||||
|
||||
---
|
||||
|
||||
## 〇、执行摘要（Executive Summary）
|
||||
|
||||
### 整体评价
|
||||
|
||||
Fonrey 文档体系**结构完整、深度足够**，DATA_MODEL 与 TECH_STACK 的颗粒度（手机号加密、稀疏权限存储、tsvector 全文检索、物化路径树、append-only 审计）已达"可直接进入编码阶段"的水平，远超一般 SaaS 项目设计文档。但作为**全局系统**，存在 **6 处文档间一致性裂缝**、**4 处多租户隔离的执行细节缺失**、**1 处性能落地空缺**，以及 UI 阶段尚未启动带来的 **5 处验收标准悬空**。
|
||||
|
||||
### 核心问题摘录（Top 8）
|
||||
|
||||
| # | 等级 | 问题 | 维度 |
|
||||
|---|------|------|------|
|
||||
| 1 | 🔴 | **权限 PRD v1.1 与 DATA_MODEL_PERMISSION 的数据范围档位不一致**（PRD 三档 vs 数据模型 5 档 SCOPE + 跨层级 DataScope 叠加） | PRD↔Data |
|
||||
| 2 | 🔴 | **Keyset 分页未在任何正式设计文档落地**（仅 Review 提示词模板提及），89k 房源 + 200 万跟进日志使用 OFFSET 分页将在深翻页时崩溃 | TECH/Data |
|
||||
| 3 | 🟠 | **R2 文件路径无租户隔离命名规范**，所有 `file_key` 字段说明均为"R2 存储路径"，未约束 `tenants/{schema_name}/...` 前缀 | TECH/合规 |
|
||||
| 4 | 🟠 | **Celery 任务的 Schema 切换机制无统一规范**，`tenant_context()` 仅在 PERMISSION 文档某 signal 中出现一次，TECH_STACK.md 未规定异步任务的租户上下文传递契约 | TECH/多租户 |
|
||||
| 5 | 🟠 | **PRD 性能目标（录入耗时 ≤ 30s、配房响应 < 3s）在 TECH_STACK 中无 SLI/SLO 落地**，缺索引清单与 N+1 查询治理策略 | PRD↔TECH |
|
||||
| 6 | 🟠 | **UI 阶段尚未开始**，但 PRD 含 90+ 个"验收标准"涉及具体交互（侧边抽屉、Toast、面包屑、批量操作 Modal），目前组件清单仅覆盖核心列表/表单，缺 Drawer / Stepper / Permission Tree 等关键组件设计 | PRD↔UI |
|
||||
| 7 | 🟠 | **租户注销→数据导出→清除**全链路在 PRD（系统管理 §) 与 DATA_MODEL_PUBLIC（export_tasks/backup_records）中存在职责边界模糊：导出包是否包含 R2 文件实体、清除时序与备份保留期的依赖关系 | PRD↔Data↔合规 |
|
||||
| 8 | 🟠 | **房源并发编辑乐观锁 / 楼盘锁定（4 类锁）的 DDL/版本字段未见**，PRD 多处提到"楼盘锁定不可改商圈"但 `complexes` 表 DDL 未发现 `version` 或 `lock_type` 字段（待 DATA_MODEL_COMPLEX §三 详查） | PRD↔Data |
|
||||
|
||||
### 风险等级分布
|
||||
|
||||
- 🔴 Blocker: **2** 项
|
||||
- 🟠 Major: **14** 项
|
||||
- 🟡 Minor: **9** 项
|
||||
|
||||
---
|
||||
|
||||
## 一、PRD 一致性审查（PRD ↔ PRD）
|
||||
|
||||
### 1.1 数据范围/权限粒度——核心冲突
|
||||
|
||||
| 文档 | 表述 |
|
||||
|------|------|
|
||||
| `PRD/权限管理/权限管理模块PRD.md` v1.1 §3 非目标 | "数据范围控制以**本人 / 本部门 / 全公司**三档为主，本期不含行级权限" |
|
||||
| `PRD/权限管理/权限管理模块PRD.md` v1.1 Story 3 §验收 | 范围型选项："**本人 / 本组 / 本门店 / 本区域 / 全公司**"——五档 |
|
||||
| `DATA_MODEL_PERMISSION.md` §1.2 | SCOPE 五档 + `staff_data_scopes` 跨层级并集叠加（"本组 ∪ 门店B"） |
|
||||
|
||||
🔴 **Blocker P-01**：同一份 PRD 内 §3 与 Story 3 自相矛盾（三档 vs 五档），且 DATA_MODEL 实现的"跨层级 DataScope 叠加"已超出 PRD §3 非目标声明的范畴——**这本质上是行级/对象级权限的弱化形态**。
|
||||
- **责任**：PM 必须先在 PRD v1.2 中：(a) 锁定档位数（推荐五档）；(b) 明确 `staff_data_scopes` 是否为本期范围（若是，§3 非目标第 1 条需改写）；(c) 给出"跨层级叠加"的业务用例与 UI 入口（当前 Story 1 操作列只有 "扩充范围"/"范围"，未说明能否选多个组织节点）。
|
||||
|
||||
### 1.2 模块边界
|
||||
|
||||
🟠 **Major P-02**：**"系统设置"职责分散**。`PRD/权限管理` §1 提到"关联模块：组织人事管理、房源管理、客源管理、**系统设置**"，但 PRD 列表中没有独立的"系统设置 PRD"，仅有 `PRD/系统配置/系统配置.md`（128 行）和 `PRD/系统管理/系统管理模块PRD.md`（594 行，平台运营视角）。`TECH_STACK.md` §1 又提到"本期聚焦首页设置与房源设置（字段标签、必填规则、自定义字段、标签管理）"——**租户级"系统设置"PRD 缺失**。
|
||||
- **责任**：PM 补一份 `PRD/系统设置/租户级系统设置模块PRD.md`，覆盖 lookup_items / 字段标签 / 自定义字段 / 标签管理，否则数据模型中已存在的 `lookup_items`（客源活跃度阈值依赖）配置入口不明。
|
||||
|
||||
🟡 **Minor P-03**：登录 PRD v1.4 §5.5 已迁出数据模型，但**"业务规则"与"数据约束"的交叉**（如 5 次失败锁定 30min、密码历史保留 3 条）在 PRD/技术方案/DATA_MODEL_LOGIN 三处重复定义，未来变更存在三处同步风险。建议 PRD 改为"详见 DATA_MODEL_LOGIN.md §X"引用。
|
||||
|
||||
### 1.3 性能目标完整性
|
||||
|
||||
🟠 **Major P-04**：PRD 中性能目标分布零散且**不可观测**：
|
||||
- 房源 PRD：录入耗时 ≤ 30s（验收无对应 SLI）
|
||||
- 客源 PRD：智能配房响应 < 3s、跟进率 ≥ 60%
|
||||
- 系统管理 PRD：RTO < 2h、灰度 ≤ 5%
|
||||
- 登录 PRD：登录成功率指标缺失
|
||||
- 发布 PRD：自动更新成功率 ≥ 98%
|
||||
|
||||
未见统一的"非功能性需求矩阵"，TECH_STACK 也未承接这些目标。
|
||||
- **责任**：PM 在 `PRD/PRD_MVP.md` 中汇总 NFR 矩阵；架构师在 `TECH_STACK.md` 中映射为 SLI/SLO + 监控埋点。
|
||||
|
||||
### 1.4 状态机一致性
|
||||
|
||||
🟡 **Minor P-05**：房源 `properties.status` 8 态（出售/出租/租售/暂缓/他售/他租/成交/未挂牌）与 PRD 房源生命周期描述未做状态转换图（哪些状态可互转、转出条件），DATA_MODEL_PROPERTY §3 仅枚举值。建议补状态机图。
|
||||
|
||||
---
|
||||
|
||||
## 二、DATA_MODEL 完整性审查
|
||||
|
||||
### 2.1 表覆盖度
|
||||
|
||||
DATA_MODEL 已实施"按模块拆分"（v1.3 索引化），8 个子文档覆盖良好。已确认存在的关键设计：
|
||||
- ✅ public schema 13 表（含 audit_logs 月度分区建议、唯一 current 版本约束）
|
||||
- ✅ 房源 22 张表（含 `sensitive_view` 跟进不可删、`listing_histories` append-only）
|
||||
- ✅ 客源（手机号加密+哈希索引、活跃度阈值由 `lookup_items` 配置）
|
||||
- ✅ 楼盘（`complexes.search_vector` tsvector + GIN）
|
||||
- ✅ 组织（物化路径 + `staff_transfer_logs` append-only）
|
||||
- ✅ 权限 6 表（稀疏存储 + 优先级合并）
|
||||
- ✅ 登录 4 表（90 天审计、3 条历史防重用）
|
||||
|
||||
### 2.2 跨文档一致性
|
||||
|
||||
🔴 **Blocker D-01**：**Keyset 分页缺位**。`prompt/Fonrey_系统设计Review_提示词模板_v1.md` Line 315 明确要求"分页查询是否使用了高效方案（如 Keyset 分页，而非 OFFSET 分页）"，但：
|
||||
- `DATA_MODEL.md` §五 容量与分区规划仅给出表大小估算
|
||||
- `DATA_MODEL_PROPERTY.md`、`DATA_MODEL_CLIENT.md` 的"查询模式参考"章节未见 Keyset 示例
|
||||
- `TECH_STACK.md` 无任何分页规范
|
||||
|
||||
89,000 房源 + 200 万跟进日志使用 OFFSET 分页，第 100 页响应将达秒级。
|
||||
- **责任**：架构师在 `TECH_STACK.md` 新增 §「分页规范」、在 `DATA_MODEL_PROPERTY.md` §6 查询模式参考补 Keyset SQL 模板（`WHERE (created_at, id) < (?, ?) ORDER BY created_at DESC, id DESC LIMIT 21`）。
|
||||
|
||||
🟠 **Major D-02**：**乐观锁/写冲突字段缺失**。
|
||||
- `properties` 主表 PRD 多处提到"维护完成度 ≥ 70% 才能上架"、多人协作场景，但 DATA_MODEL_PROPERTY.md `properties` 表 DDL 未见 `version` / `row_version` 字段。
|
||||
- 楼盘 PRD §"楼盘锁定"提到 4 类锁（信息锁/坐标锁/楼栋锁/合并锁），DATA_MODEL_COMPLEX 是否实现待详查（grep 未在 §三 摘要中发现 `lock_*` 字段）。
|
||||
- **责任**：架构师在 `properties`、`clients`、`complexes` 主表补 `version INTEGER DEFAULT 0`，配合 Django ORM `update(version=F('version')+1).filter(version=expected)` 模式。
|
||||
|
||||
🟠 **Major D-03**：**外键跨 schema 限制的隔离机制说明不足**。`DATA_MODEL_PERMISSION.md` §3.1 已明确 `permission_defs` 放租户 schema 是为了规避 django-tenants 跨 schema FK，但其他类似场景（如 `staff` 引用 `org_units`、`user_accounts.staff_id`）未统一说明。建议在 `DATA_MODEL.md` §一架构决策中补 "**所有 FK 必须在同一 schema 内**" 原则。
|
||||
|
||||
🟠 **Major D-04**：**审计/日志表的分区策略不一致**。
|
||||
- `platform_audit_logs`：建议月度分区 ✅
|
||||
- `permission_change_logs`：月度分区 + 6 个月归档 ✅
|
||||
- `follow_logs`（200 万+）：DATA_MODEL.md §五已建议月度分区，但 DATA_MODEL_PROPERTY §4.5 表 DDL **未给出分区 DDL 或 PARTITION BY 子句**——开发期不分区，未来再分区将带来数据迁移成本。
|
||||
- `login_attempts`（90 天保留）：DATA_MODEL_LOGIN 未给分区或 TTL 清理策略。
|
||||
- **责任**：架构师在每张高写入表的 DDL 中给出 `CREATE TABLE ... PARTITION BY RANGE (created_at)` + `pg_partman` 自动维护方案。
|
||||
|
||||
🟡 **Minor D-05**：`UserAccount.username` 在 `user_accounts` 表的唯一性约束注释提到"在租户 Schema 维度生效"，但若员工跨租户（多平台账号 `StaffAccount` 已支持 Fonrey/58/安居客/网络经纪人），登录 PRD 的"账号体系"是否覆盖这一场景未明。
|
||||
|
||||
🟡 **Minor D-06**：DATA_MODEL.md §一 1.3 关键设计原则中"金额 NUMERIC(12,2) 万元精度"——12,2 表示最大 9999999999.99 万元，是否过度，建议改 NUMERIC(14,2) 元精度（与会计/合同模块对齐）或在文档中说明万元单位的统一约定。
|
||||
|
||||
### 2.3 缺失数据
|
||||
|
||||
🟠 **Major D-07**：以下 PRD 提及但 DATA_MODEL 未见的实体（待 PM/架构师确认是否本期范围）：
|
||||
- 楼盘"价格走势"（PRD 楼盘 §)：DATA_MODEL_COMPLEX 是否含 `complex_price_history` 待查
|
||||
- "市场报盘"（房源 PRD §)：DATA_MODEL_PROPERTY 22 表中无 `market_quotes`
|
||||
- "公客转换"（客源 PRD：私客 → 公客自动转）：DATA_MODEL_CLIENT 是否含触发器/job 待查
|
||||
- "学区关联距离"：DATA_MODEL_COMPLEX 已实现 `complex_schools` N:M 含距离 ✅
|
||||
|
||||
---
|
||||
|
||||
## 三、TECH_STACK 完整性审查
|
||||
|
||||
### 3.1 已覆盖
|
||||
|
||||
✅ HTMX + Alpine.js + Tailwind 禁用清单（Do NOT use 段）
|
||||
✅ App 目录结构（`apps/property/models/` 文件级拆分）
|
||||
✅ Celery + Redis + R2 + Sentry + Grafana
|
||||
✅ Electron 客户端：electron-builder + electron-updater + EV 签名 + SHA256 校验
|
||||
✅ 登录 `accounts` App 依赖关系图、Redis Key 命名规范
|
||||
✅ 权限系统：自定义 Hybrid RBAC、Redis 缓存快照、Override 优先级
|
||||
|
||||
### 3.2 关键缺失
|
||||
|
||||
🟠 **Major T-01**：**Celery 多租户上下文规范缺失**。
|
||||
- `TECH_STACK.md` §3 关键约定第 4 条仅说"耗时任务必须 Celery"，未规定**租户 ID 如何传入 Worker**。
|
||||
- `DATA_MODEL_PERMISSION.md` 行 1075 出现 `tenant_context(instance)` 仅一例，其他模块（导出 89k 房源、智能配房、图片转码）的 schema 切换无统一约定。
|
||||
- 风险：Worker 在错误 schema 下执行查询、跨租户数据污染。
|
||||
- **责任**：架构师在 `TECH_STACK.md` 新增 §「异步任务规范」：(a) 所有 task 第一参数必须为 `tenant_schema_name`；(b) 强制使用装饰器 `@with_tenant_context`；(c) Sentry 上报必须包含 `tenant_schema` tag。
|
||||
|
||||
🟠 **Major T-02**：**R2 文件路径租户隔离命名规范缺失**。
|
||||
- 所有 `file_key TEXT` 字段注释为 "R2 存储路径"，但 TECH_STACK 未约束 key 前缀。
|
||||
- 当前若开发各自约定，可能出现 `photos/{uuid}.jpg`（无租户标识）的隐患——R2 bucket 是共享的。
|
||||
- **责任**：架构师在 `TECH_STACK.md` 增加 R2 路径模板：
|
||||
- 房源照片：`tenants/{schema_name}/property/{property_id}/photos/{uuid}.jpg`
|
||||
- 跟进附件：`tenants/{schema_name}/follow_logs/{log_id}/attachments/{uuid}`
|
||||
- 客户端发布：`releases/{platform}/{version}/{filename}`（共享）
|
||||
- 备份/导出：`platform/backups/{tenant_schema}/{date}/...`、`platform/exports/{task_id}/...`
|
||||
|
||||
🟠 **Major T-03**：**索引清单与查询模式未集中化**。
|
||||
- 各 DATA_MODEL 子文档都有零散索引 DDL，但缺一份"高频查询场景 → 索引"对应表。
|
||||
- 例：客源活跃度筛选、房源多条件搜索（户型/面积/价格/标签/区域）、跟进日志按员工+时间范围——这些查询是否都有覆盖索引？
|
||||
- **责任**：架构师补 `TECH_STACK/索引规范.md` 或在 `DATA_MODEL.md` §六 增加查询索引矩阵。
|
||||
|
||||
🟠 **Major T-04**：**前端构建与资产管线未定义**。
|
||||
- `TECH_STACK.md` 只说 HTMX + Alpine + Tailwind，但未规定：Tailwind JIT 配置位置、CSS 变量加载顺序（UI_SYSTEM 要求 `:root` 中定义所有 token）、HTMX 扩展（`hx-boost`、`hx-ext="response-targets"`）启用清单、Alpine 插件（`x-intersect`、`x-mask`）。
|
||||
- **责任**：架构师补 §「前端构建管线」。
|
||||
|
||||
🟡 **Minor T-05**：**降级方案缺失**（呼应 Review 提示词模板第 348 行）：
|
||||
- R2 不可用时图片上传如何降级？
|
||||
- Redis 不可用时权限/Session 如何降级？
|
||||
- Celery 队列堆积时如何熔断？
|
||||
- 当前文档无任何降级策略。
|
||||
|
||||
🟡 **Minor T-06**：登录技术方案 v2.0 §十 提到滑块 Token 3min Redis TTL，但未说明 Redis 不可用时是否降级到内存（多 worker 不一致）或拒绝登录。
|
||||
|
||||
🟡 **Minor T-07**：发布 PRD 提到 ARM64 按需支持，TECH_STACK 第 8 节同步——但未规定 ARM64 触发条件（用户量阈值？）。
|
||||
|
||||
---
|
||||
|
||||
## 四、UI/UX 完整性审查
|
||||
|
||||
### 4.1 已覆盖（UI_SYSTEM.md / 组件清单.md）
|
||||
|
||||
✅ Design Philosophy（4 条原则 + 9 条反模式，含 `禁 window.alert / 禁无限滚动 / 禁 Generic 错误`）
|
||||
✅ Design Tokens（颜色/间距 4px 网格/圆角/阴影 CSS 变量化）
|
||||
✅ Tailwind fallback 映射表
|
||||
✅ 核心组件：Sortable Data Table、Column Visibility、Pagination、Toolbar、Toggle、Multi-select Tag
|
||||
✅ 焦点环 / disabled / readonly 状态规范
|
||||
|
||||
### 4.2 缺失组件 vs PRD 验收标准
|
||||
|
||||
PRD 的 90+ 验收标准要求以下组件，但组件清单未覆盖：
|
||||
|
||||
🟠 **Major U-01**：**侧边抽屉（Drawer）缺失**。权限 PRD Story 4 明确要求"右侧滑出 Drawer 不覆盖左侧导航"，UI_SYSTEM 仅在 §一开头提及 `--radius-lg → 模态/抽屉/面板`，但组件清单无 Drawer 实现。
|
||||
|
||||
🟠 **Major U-02**：**权限树/复杂表单组件缺失**。权限 PRD 描述：14 个一级模块树 + 每模块多分组 + 分组内 Toggle/Select/Number——这是本期最复杂的页面，组件清单未给原型。
|
||||
|
||||
🟠 **Major U-03**：**Stepper/Wizard 缺失**。房源/客源录入 PRD 强调 ≤ 30s 完成，多 Tab 表单（基本信息/价格/联系人/标签/图片）需要 Stepper 或可折叠分组组件。
|
||||
|
||||
🟡 **Minor U-04**：Toast/Dialog/确认对话框组件未在清单中正式定义（Anti-pattern 中提到"禁 window.alert，使用 Dialog/Toast"）。
|
||||
|
||||
🟡 **Minor U-05**：树形选择器（用于 OrgUnit 选择、Permission Scope 选择）缺失。
|
||||
|
||||
### 4.3 流程缺失
|
||||
|
||||
🟠 **Major U-06**：**Wireframe / 信息架构未启动**。当前进度"UI 未开始"，但 PRD 中存在大量 UI 隐含决策：
|
||||
- 房源列表 21 个核心字段 + 自定义列：默认显示哪些？密度？
|
||||
- 89k 数据列表的视觉层级（卡片 vs 表格——UI_SYSTEM 反模式禁止两者混用）
|
||||
- 多租户登录页面是否需要展示 tenant_logo？（DATA_MODEL_LOGIN 已有 `tenant_logo_url` 字段）
|
||||
- **责任**：UI/UX 设计师在动手前，先输出"页面清单 + 信息架构 + 关键页面 wireframe"。
|
||||
|
||||
🟡 **Minor U-07**：UI_SYSTEM.md 未规定**国际化**策略（虽然本期中文为主，但 `to_tsvector('simple')` 而非 `'chinese'` 已暗示无中文分词，需在 PRD 中确认是否本期不做中文模糊检索）。
|
||||
|
||||
---
|
||||
|
||||
## 五、多租户隔离审查（横切）
|
||||
|
||||
| 隔离维度 | 现状 | 风险 |
|
||||
|----------|------|------|
|
||||
| DB Schema | ✅ django-tenants Schema 隔离，所有租户业务表均在 tenant schema | 已落实 |
|
||||
| public ↔ tenant FK | ✅ 已在 PERMISSION 文档明确禁止跨 schema FK | 已落实 |
|
||||
| 登录认证 | ✅ Tenant 验证在 public，UserAccount 在 tenant schema；username 仅 schema 内唯一 | 已落实 |
|
||||
| Celery Worker schema 切换 | 🟠 仅一例，无统一规范 | T-01 |
|
||||
| R2 文件路径 | 🟠 无租户前缀规范 | T-02 |
|
||||
| Redis Key 命名 | 🟡 登录方案 §十 已包含 `{tenant_id}` 前缀；权限缓存方案 Redis Key 是否带 tenant 待查 | 待确认 |
|
||||
| Sentry / 日志 | 🟡 登录方案提及 `tenant_id` tag，但全局未统一 | T-01 |
|
||||
|
||||
🟠 **Major X-01**：**Redis Key 命名跨模块未统一**。登录方案使用 `login_fail:{tenant_id}:{username}`、`tenant_verify_ip:{ip}`，权限方案使用"员工权限快照"——若不强制 `{tenant_schema_name}:` 前缀，多租户 Redis 共享时存在键冲突。
|
||||
|
||||
🟠 **Major X-02**：**租户注销→数据导出→清除全链路职责模糊**。
|
||||
- PRD/系统管理 §"租户删除"提到：释放子域名、R2 存储桶、License 席位
|
||||
- DATA_MODEL_PUBLIC `export_tasks` 含 24h 下载链接
|
||||
- 缺失：(a) 导出包是否包含 R2 文件实体（PRD §232 提示"含 R2 文件实体"，但 export_tasks 字段是否落地待查）；(b) 清除时序：tenants.status `pending_delete → deleted` 的硬删除窗口（推荐 30 天宽限期，文档未明确）；(c) 备份保留期 vs 清除时序的一致性。
|
||||
- **责任**：PM + 架构师 + 合规共同梳理租户注销 SOP，并在 `DATA_MODEL_PUBLIC.md` §2.4 增加 `tenant_deletion_workflow` 文档化流程图。
|
||||
|
||||
---
|
||||
|
||||
## 六、合规与安全审查
|
||||
|
||||
🟠 **Major S-01**：**手机号加密策略统一性已落实**（AES-256-GCM + SHA-256 哈希），但**密钥管理方案缺失**。
|
||||
- TECH_STACK.md 无 KMS / Vault / 环境变量加密管理说明。
|
||||
- `core.encryption` 仅作为模块名出现，密钥轮换流程未定义。
|
||||
|
||||
🟠 **Major S-02**：**敏感字段访问审计**已通过 `follow_logs.sensitive_view` + `is_deletable=FALSE` 实现，但：
|
||||
- 客源 `phone_hash` 解密查看是否每次都写 `sensitive_view` 跟进？
|
||||
- 业主联系人查看是否同样有"号码方审批"前置流程（PRD 提及 `number_holder_approvals`）？
|
||||
- 这些"敏感数据访问审计"的触发链路在 TECH_STACK 未集中说明。
|
||||
|
||||
🟡 **Minor S-03**：登录失败锁定 30min 是 IP 维度还是账号维度？登录方案 §十 Key 为 `login_fail:{tenant_id}:{username}` 是账号维度，可能被恶意撞库锁定真实用户。建议补 IP 维度限流。
|
||||
|
||||
🟡 **Minor S-04**：MFA 仅强制平台管理员（`admin_mfa_devices`），租户内部 Tenant Admin / 高敏角色（系统管理员）是否需要 MFA 未提及。
|
||||
|
||||
🟡 **Minor S-05**：CORS / CSP / SameSite Cookie 等 Web 安全策略未在 TECH_STACK 中规定。
|
||||
|
||||
---
|
||||
|
||||
## 七、性能与容量审查
|
||||
|
||||
🔴 **Blocker** D-01（Keyset 分页缺失）已在 §二列出。
|
||||
|
||||
🟠 **Major P-08**：**89k 房源筛选的执行计划未验证**。房源 PRD 列表筛选含户型/面积/价格/区域/标签/属性——多条件 AND 查询：
|
||||
- 是否有覆盖索引？（DATA_MODEL_PROPERTY §6 待详查）
|
||||
- 标签筛选（`property_tag_relations` N:M）是否需要 GIN array 索引或物化视图？
|
||||
- 排序字段（更新时间/价格）是否在索引中？
|
||||
|
||||
🟠 **Major P-09**：**`follow_logs` 200 万+/月增长但无分区 DDL**（D-04 已列）。
|
||||
|
||||
🟡 **Minor P-10**：`property_photos` 500 万+ 建议 HASH 分区（DATA_MODEL.md §五），但 DATA_MODEL_PROPERTY §4.14 表 DDL 未给分区子句。
|
||||
|
||||
🟡 **Minor P-11**：智能配房 PRD < 3s 响应，但匹配算法（`client_property_matches`）的索引/计算路径未设计文档。Celery 异步是离线计算还是实时？
|
||||
|
||||
---
|
||||
|
||||
## 八、可维护性与扩展性
|
||||
|
||||
🟡 **Minor M-01**：DATA_MODEL 8 文档 + TECH_STACK 3 文档 + PRD 8 文档版本号不统一（v1.0~v1.4 混用），缺一份 `CHANGELOG.md` 跟踪跨文档变更。
|
||||
|
||||
🟡 **Minor M-02**：`核心文档体系.md` 提到 "AI 指令手册 (`.cursorrules` / `AI_INSTRUCTIONS.md`)"——但仓库未见该文件。AI 协同开发约定缺位。
|
||||
|
||||
🟡 **Minor M-03**：单元测试 / 集成测试约定未在 TECH_STACK 中提及（pytest-django / factory_boy / django-tenants 测试 utility）。
|
||||
|
||||
🟡 **Minor M-04**：Migration 策略未定义。多租户场景下 schema migration 跨数百租户的执行顺序、回滚策略需明文（django-tenants `migrate_schemas` 命令）。
|
||||
|
||||
---
|
||||
|
||||
## 九、行动清单（按责任人 + 优先级）
|
||||
|
||||
### PM（产品经理）
|
||||
|
||||
| ID | 等级 | 任务 |
|
||||
|----|------|------|
|
||||
| PM-1 | 🔴 | **修订权限 PRD v1.2**：锁定数据范围档位（建议五档+DataScope），同步修改 §3 非目标声明，补 DataScope UI 入口 |
|
||||
| PM-2 | 🟠 | 补充 `PRD/系统设置/租户级系统设置模块PRD.md`（lookup_items 配置、字段标签、自定义字段、标签管理） |
|
||||
| PM-3 | 🟠 | 在 `PRD/PRD_MVP.md` 中汇总 NFR 矩阵（性能/可用性/安全/合规） |
|
||||
| PM-4 | 🟠 | 与架构师 + 合规一起梳理租户注销 SOP（导出范围、宽限期、清除时序） |
|
||||
| PM-5 | 🟡 | 房源 status 状态机图、客源公客转换规则文档化 |
|
||||
| PM-6 | 🟡 | 确认是否本期支持中文模糊检索（影响 tsvector 配置） |
|
||||
| PM-7 | 🟡 | MFA 是否扩展到租户内部高权限角色 |
|
||||
|
||||
### 架构师
|
||||
|
||||
| ID | 等级 | 任务 |
|
||||
|----|------|------|
|
||||
| ARCH-1 | 🔴 | **TECH_STACK 新增 §「分页规范」**，给出 Keyset 模板；DATA_MODEL_PROPERTY/CLIENT §6 补查询模式参考 |
|
||||
| ARCH-2 | 🟠 | TECH_STACK 新增 §「异步任务规范」（tenant_schema 必传 + `@with_tenant_context` 装饰器 + Sentry tag） |
|
||||
| ARCH-3 | 🟠 | TECH_STACK 新增 §「R2 路径规范」（每类资源的 key 模板，强制 `tenants/{schema_name}/...` 前缀） |
|
||||
| ARCH-4 | 🟠 | DATA_MODEL 各高写入表（`follow_logs`, `property_photos`, `permission_change_logs`, `login_attempts`, `platform_audit_logs`）补 PARTITION DDL + pg_partman 自动维护脚本 |
|
||||
| ARCH-5 | 🟠 | `properties` / `clients` / `complexes` 主表补 `version` 字段实现乐观锁；楼盘 4 类锁补 `lock_*` 字段 |
|
||||
| ARCH-6 | 🟠 | DATA_MODEL.md §一架构决策补 "**所有 FK 必须同 schema**" 原则；TECH_STACK 补 "**Redis Key 必须 `{schema_name}:` 前缀**" 强制规范 |
|
||||
| ARCH-7 | 🟠 | 补 `TECH_STACK/索引规范.md`（高频查询 → 索引矩阵）；补「前端构建管线」章节（Tailwind JIT/HTMX 扩展/Alpine 插件） |
|
||||
| ARCH-8 | 🟠 | 密钥管理方案：KMS/Vault 选型、轮换 SOP、`core.encryption` 接口契约 |
|
||||
| ARCH-9 | 🟡 | 降级方案：R2/Redis/Celery 不可用时的策略 |
|
||||
| ARCH-10 | 🟡 | Migration 策略（跨 N 租户 migrate_schemas 顺序与回滚） |
|
||||
| ARCH-11 | 🟡 | 测试规范（pytest-django + factory_boy + tenant 测试 utility） |
|
||||
| ARCH-12 | 🟡 | CORS/CSP/SameSite 安全头规范 |
|
||||
| ARCH-13 | 🟡 | 智能配房算法路径文档（实时 vs 异步） |
|
||||
|
||||
### UI/UX 设计师
|
||||
|
||||
| ID | 等级 | 任务 |
|
||||
|----|------|------|
|
||||
| UI-1 | 🟠 | **启动 Wireframe**：先输出页面清单 + 信息架构 + 高频页面（房源列表、房源录入、权限编辑、登录）线框 |
|
||||
| UI-2 | 🟠 | 组件清单补 Drawer / Stepper / Permission Tree / Tree Select |
|
||||
| UI-3 | 🟠 | 89k 列表的视觉密度方案（行高、字段优先级、列宽策略） |
|
||||
| UI-4 | 🟡 | Toast / Dialog / Confirm 组件正式定义 |
|
||||
| UI-5 | 🟡 | 多租户登录页是否展示 `tenant_logo_url` 的视觉规范 |
|
||||
| UI-6 | 🟡 | 国际化策略（v1 是否仅中文） |
|
||||
|
||||
---
|
||||
|
||||
## 十、结论
|
||||
|
||||
### 是否可进入开发阶段
|
||||
|
||||
**部分可以，但需先解决 2 个 Blocker**：
|
||||
|
||||
1. **🔴 P-01 权限范围档位冲突**：必须先在 PRD 锁定，否则 `staff_data_scopes` 表是否生效不明，权限模块无法开工。
|
||||
2. **🔴 D-01 Keyset 分页规范缺失**：必须在 TECH_STACK 落地，否则房源/客源/跟进列表的核心查询路径设计错误，后期返工成本高。
|
||||
|
||||
**可立即并行启动的部分**：
|
||||
- 公共 Schema（`tenants`、`platform_admins`、`audit_logs`）开发——文档完整，无 Blocker
|
||||
- 楼盘/区域/学校 CRUD——DATA_MODEL_COMPLEX 完整，主流程清晰
|
||||
- Electron 客户端框架搭建——TECH_STACK §8 决策已封闭
|
||||
- 登录模块（accounts App）——文档完整，仅需在编码前补 Redis Key 命名前缀规范
|
||||
|
||||
**必须延后的部分**：
|
||||
- 权限模块编码（等待 PM-1 完成）
|
||||
- 房源/客源高基数列表查询（等待 ARCH-1 完成）
|
||||
- 全 UI 实现（等待 UI-1 Wireframe）
|
||||
|
||||
### 文档体系成熟度评分
|
||||
|
||||
| 维度 | 成熟度 | 说明 |
|
||||
|------|--------|------|
|
||||
| PRD 业务清晰度 | 8.5/10 | 用户故事和验收标准充分，仅权限档位有自相矛盾 |
|
||||
| DATA_MODEL 落地深度 | 8/10 | 颗粒度高，但 Keyset/分区 DDL/乐观锁字段缺失 |
|
||||
| TECH_STACK 完整性 | 6/10 | 选型已封闭，但异步/R2/索引/降级等横切规范缺位 |
|
||||
| UI/UX 系统化 | 5/10 | Token/核心组件已成型，但 Wireframe 未启动且复杂组件缺失 |
|
||||
| 跨文档一致性 | 6.5/10 | 整体高，但 6 处裂缝需要修复 |
|
||||
| 多租户隔离严谨度 | 7.5/10 | DB 隔离扎实，但 Celery/R2/Redis 横切层规范不足 |
|
||||
| **整体** | **7/10** | 已达"中等偏上"水平，远超普通 SaaS 项目；解决 Blocker 后可达 8.5+ |
|
||||
|
||||
---
|
||||
|
||||
## 附录 A：本次 Review 涵盖文档
|
||||
|
||||
### PRD（8 份）
|
||||
- `PRD/房源管理/房源管理模块PRD.md` v2.1（1881 行）
|
||||
- `PRD/房源管理/楼盘管理模块PRD.md` v1.0（704 行）
|
||||
- `PRD/客源管理/客源管理模块PRD.md` v1.4（2050 行）
|
||||
- `PRD/权限管理/权限管理模块PRD.md` v1.1（623 行）
|
||||
- `PRD/组织人事管理/组织人事管理模块PRD.md` v1.2（512 行）
|
||||
- `PRD/系统管理/系统管理模块PRD.md` v1.0（594 行）
|
||||
- `PRD/登录管理/用户登录管理模块PRD.md` v1.4（648 行）
|
||||
- `PRD/发布管理/客户端发布管理模块PRD.md` v1.0（407 行）
|
||||
|
||||
### DATA_MODEL（8 份）
|
||||
- `DATA_MODEL/DATA_MODEL.md` v1.3（622 行，索引文档）
|
||||
- `DATA_MODEL/DATA_MODEL_PUBLIC.md` v1.0（599 行）
|
||||
- `DATA_MODEL/DATA_MODEL_ORG.md` v1.0（342 行）
|
||||
- `DATA_MODEL/DATA_MODEL_COMPLEX.md` v1.0（548 行）
|
||||
- `DATA_MODEL/DATA_MODEL_PROPERTY.md` v1.0（1169 行）
|
||||
- `DATA_MODEL/DATA_MODEL_CLIENT.md` v1.0（575 行）
|
||||
- `DATA_MODEL/DATA_MODEL_PERMISSION.md` v1.0（1363 行）
|
||||
- `DATA_MODEL/DATA_MODEL_LOGIN.md` v1.0（470 行）
|
||||
|
||||
### TECH_STACK（3 份）
|
||||
- `TECH_STACK/TECH_STACK.md`（154 行）
|
||||
- `TECH_STACK/登录管理技术方案.md` v2.0（711 行）
|
||||
- `TECH_STACK/权限管理系统技术方案.md` v1.0（677 行）
|
||||
|
||||
### UI/UX（2 份）
|
||||
- `UI&UX/UI_SYSTEM.md`（987 行）
|
||||
- `UI&UX/组件清单.md`（1264 行）
|
||||
|
||||
---
|
||||
|
||||
## 附录 B：问题汇总速查表
|
||||
|
||||
| ID | 等级 | 维度 | 责任人 | 简述 |
|
||||
|----|------|------|--------|------|
|
||||
| P-01 | 🔴 | PRD↔Data | PM | 权限数据范围档位冲突（3 vs 5+叠加） |
|
||||
| D-01 | 🔴 | TECH/Data | 架构师 | Keyset 分页规范全局缺失 |
|
||||
| P-02 | 🟠 | PRD | PM | 租户级系统设置 PRD 缺失 |
|
||||
| P-04 | 🟠 | PRD↔TECH | PM+架构师 | 性能 NFR 矩阵未汇总 |
|
||||
| D-02 | 🟠 | Data | 架构师 | 主表乐观锁/楼盘锁字段缺失 |
|
||||
| D-03 | 🟠 | Data | 架构师 | 跨 schema FK 原则未统一 |
|
||||
| D-04 | 🟠 | Data | 架构师 | 高写入表分区 DDL 未落地 |
|
||||
| D-07 | 🟠 | Data | PM/架构师 | 楼盘价格走势/市场报盘等表缺失待确认 |
|
||||
| T-01 | 🟠 | TECH | 架构师 | Celery 多租户 schema 切换规范缺失 |
|
||||
| T-02 | 🟠 | TECH | 架构师 | R2 文件路径租户隔离规范缺失 |
|
||||
| T-03 | 🟠 | TECH | 架构师 | 索引清单未集中化 |
|
||||
| T-04 | 🟠 | TECH | 架构师 | 前端构建管线未定义 |
|
||||
| U-01 | 🟠 | UI | UI/UX | Drawer 组件缺失 |
|
||||
| U-02 | 🟠 | UI | UI/UX | 权限树/Permission Tree 复杂组件缺失 |
|
||||
| U-03 | 🟠 | UI | UI/UX | Stepper/Wizard 缺失 |
|
||||
| U-06 | 🟠 | UI | UI/UX | Wireframe 未启动 |
|
||||
| X-01 | 🟠 | 多租户 | 架构师 | Redis Key 命名跨模块未统一带 tenant 前缀 |
|
||||
| X-02 | 🟠 | 合规 | PM+架构师 | 租户注销→导出→清除链路职责模糊 |
|
||||
| S-01 | 🟠 | 安全 | 架构师 | 加密密钥管理方案缺失 |
|
||||
| S-02 | 🟠 | 安全 | 架构师 | 敏感字段访问审计触发链未集中说明 |
|
||||
| P-08 | 🟠 | 性能 | 架构师 | 89k 房源筛选执行计划未验证 |
|
||||
| P-03 | 🟡 | PRD | PM | 登录业务规则三处重复 |
|
||||
| P-05 | 🟡 | PRD | PM | 房源 status 状态机图缺失 |
|
||||
| D-05 | 🟡 | Data | PM | 跨平台账号 username 复用未明 |
|
||||
| D-06 | 🟡 | Data | 架构师 | NUMERIC(12,2) 精度选型理由 |
|
||||
| T-05 | 🟡 | TECH | 架构师 | 降级方案缺失 |
|
||||
| T-06 | 🟡 | TECH | 架构师 | Redis 不可用时滑块 Token 行为未定 |
|
||||
| T-07 | 🟡 | TECH | 架构师 | ARM64 触发条件未定 |
|
||||
| U-04 | 🟡 | UI | UI/UX | Toast/Dialog 组件未正式定义 |
|
||||
| U-05 | 🟡 | UI | UI/UX | Tree Select 组件缺失 |
|
||||
| U-07 | 🟡 | UI | PM | 国际化策略未明 |
|
||||
| S-03 | 🟡 | 安全 | 架构师 | 登录限流仅账号维度缺 IP 维度 |
|
||||
| S-04 | 🟡 | 安全 | PM | 租户内 MFA 范围未明 |
|
||||
| S-05 | 🟡 | 安全 | 架构师 | CORS/CSP/SameSite 未规定 |
|
||||
| P-10 | 🟡 | 性能 | 架构师 | property_photos HASH 分区 DDL 缺 |
|
||||
| P-11 | 🟡 | 性能 | 架构师 | 智能配房算法路径未文档化 |
|
||||
| M-01~M-04 | 🟡 | 维护 | 架构师 | CHANGELOG/AI 指令/测试规范/Migration 策略 |
|
||||
|
||||
**合计**：🔴 2 / 🟠 19 / 🟡 17 = **38 项**
|
||||
|
||||
---
|
||||
|
||||
*Report Generated: 2026-04-25 by AI Reviewer*
|
||||
@@ -1,521 +0,0 @@
|
||||
# Fonrey 全局系统设计 Review 报告
|
||||
|
||||
> **Review 类型**：全量 Review（PRD + DATA_MODEL + TECH_STACK + UI/UX + TASK 交叉验证）
|
||||
> **Review 模式**：聚焦 Top 风险 + 与 `REVIEW_全局_2026-04-25.md` 增量对比
|
||||
> **Review 日期**：2026-04-26
|
||||
> **Reviewer**：首席系统设计 Reviewer（AI 辅助）
|
||||
> **当前阶段**：需求 ~85% / 数据模型 ~70% / UI 设计 ~30%（HTML 原型已启动）/ TECH 横切规范 ~50%
|
||||
> **覆盖文档**：8 份 PRD（含 7 份模块 PRD + TASK.md）、8 份 DATA_MODEL、4 份 TECH_STACK、3 份 UI_SYSTEM、UI_DESIGN（10 份原型/设计稿）
|
||||
> **问题分级**：🔴 Blocker（阻塞开发） / 🟠 Major（必须修复但不阻塞） / 🟡 Minor（建议优化）
|
||||
|
||||
---
|
||||
|
||||
## 〇、执行摘要（Executive Summary）
|
||||
|
||||
### 整体评价
|
||||
|
||||
相较 2026-04-25 版本，文档体系出现 **3 项实质性进展**：
|
||||
1. **UI_DESIGN 原型已启动**（昨日 U-06 已修复）：客源列表/详情/新增/编辑、房源列表 5 类核心页面均已产出 HTML + Markdown 双版本，规范深度足够。
|
||||
2. **租户注销 SOP 已落地**（昨日 X-02 已修复）：`PRD/系统管理` 已含导出范围、30 天宽限期、清除时序的完整生命周期。
|
||||
3. **楼盘 4 类锁字段已实现**（昨日 D-02 部分修复）：`complexes.lock_building/lock_room/lock_info/lock_standard_room` DDL 已写入。
|
||||
|
||||
但本次审查也暴露出**昨日 Review 未触及的新一类系统性风险——枚举一致性塌方**：客源状态、客源等级、房源状态、操作日志类型等核心枚举在 PRD ↔ DATA_MODEL ↔ TASK AC 三方均无权威源，且互相矛盾。这是一类比"Keyset 分页缺失"更隐蔽、修复成本更高的债务。
|
||||
|
||||
同时，**昨日列出的 2 个 Blocker（P-01 权限档位 / D-01 Keyset 分页）均未修复**，新增 1 个 Blocker（系统配置 PRD 仍为空骨架但被 9 条 P0/P1 任务引用为权威），共 **3 个 Blocker**，必须在编码启动前清零。
|
||||
|
||||
### 核心问题摘录（Top 10）
|
||||
|
||||
| # | 等级 | 编号 | 问题 | 维度 | 状态 |
|
||||
|---|------|------|------|------|------|
|
||||
| 1 | 🔴 | **B-01** | 系统配置 PRD（`PRD/系统配置/系统配置.md`）仍为 128 行空骨架，但 TASK.md 排期 9 条 P0/P1 任务（US-SETTING-001/010~012/020~023）以其为权威；`lookup_items` 入口、字段必填、活跃度阈值（30 天）等无配置出口 | PRD↔TASK | 🆕 新增 |
|
||||
| 2 | 🔴 | **B-02** | **核心枚举三方不一致**：客源状态（PRD"求购/求租/租购"vs DDL `buying/renting/buy_or_rent`）、等级（PRD A~E vs DDL A_urgent/A/B/C/D）、房源 status（DDL 未含 `sold/unlisted/paused`，但 TASK AC 与查询语句均使用） | PRD↔Data↔TASK | 🆕 新增 |
|
||||
| 3 | 🔴 | **B-03** | 权限 PRD §3 非目标声明"三档"与 §5.6 / Story 3 验收"五档" + DATA_MODEL_PERMISSION 的"DataScope 跨层级叠加"实质性冲突 | PRD↔Data | ⚠️ 昨日 P-01 部分修复，仍 Blocker |
|
||||
| 4 | 🔴 | **B-04** | Keyset 分页规范在 TECH_STACK / DATA_MODEL / 测试规范中**完全缺位**，89k 房源 + 200 万跟进日志的列表查询设计错误 | TECH/Data | ⚠️ 昨日 D-01 未修复 |
|
||||
| 5 | 🟠 | **M-01** | 测试规范 v1.0 与 django-tenants 多租户工作模式冲突：`LocMemCache`/`FileSystemStorage`/`CELERY_TASK_ALWAYS_EAGER=True` 使权限 Redis 快照失效、跨租户隔离、Celery 异步导出三类场景无法验证；`TenantClient`/`tenant fixture` 仅在 §11.2 Prompt 模板中被引用但无实现 | TECH/测试 | 🆕 新增 |
|
||||
| 6 | 🟠 | **M-02** | 主表乐观锁字段缺失：`properties` / `clients` 仍无 `version`；`complexes` 4 类锁已建字段但无解锁权限/过期规则 PRD | Data↔PRD | ⚠️ 昨日 D-02 部分修复 |
|
||||
| 7 | 🟠 | **M-03** | 高写入表分区 DDL 仍未落地：`follow_logs` / `property_photos` / `permission_change_logs` / `login_attempts` / `platform_audit_logs` 均仅文字"建议月度分区"，无 `PARTITION BY` 子句 | Data | ⚠️ 昨日 D-04 未修复 |
|
||||
| 8 | 🟠 | **M-04** | TECH_STACK 三大横切规范仍未补：Celery 多租户 schema 切换（昨 T-01）、R2 文件路径前缀（昨 T-02）、查询索引矩阵（昨 T-03） | TECH/多租户 | ⚠️ 昨日均未修复 |
|
||||
| 9 | 🟠 | **M-05** | 89k 数据 < 2 秒列表查询是 TASK AC 中的硬性 NFR（US-PROPERTY-002 / US-CLIENT-002），但 TECH_STACK + 测试规范无 p95/EXPLAIN/性能基准对应任务，AC 不可测 | NFR↔TECH↔测试 | 🆕 新增 |
|
||||
| 10 | 🟠 | **M-06** | 客户端发布无签名校验/防降级机制：`/api/client/updates/latest/` 与 `download_url` 公开 URL，仅 SHA256 完整性校验，可被中间人投递降级版本 | 安全 | 🆕 新增 |
|
||||
|
||||
### 风险等级分布
|
||||
|
||||
| 等级 | 本次（2026-04-26） | 昨日（2026-04-25） | 净变化 |
|
||||
|------|---|---|---|
|
||||
| 🔴 Blocker | **3** | 2 | +1（系统配置空骨架升级） |
|
||||
| 🟠 Major | **12** | 19 | -7（含 6 项已修复 + 1 项降级） |
|
||||
| 🟡 Minor | **6** | 17 | -11（聚焦 Top 风险，未列入本期） |
|
||||
| 合计 | **21** | 38 | — |
|
||||
|
||||
### 增量对比一览（昨日 P0/P1 → 今日状态）
|
||||
|
||||
| 昨日编号 | 简述 | 状态 | 今日处置 |
|
||||
|----------|------|------|----------|
|
||||
| P-01 | 权限范围档位冲突 | 🟡 部分修复 | 升 B-03 仍 Blocker |
|
||||
| D-01 | Keyset 分页缺失 | 🔴 未修复 | 转 B-04 |
|
||||
| P-02 | 系统配置 PRD 缺失 | 🔴 仍空骨架 | 升 B-01 |
|
||||
| D-02 | 主表乐观锁/楼盘锁 | 🟡 部分修复 | 转 M-02 |
|
||||
| D-04 | 高写入表分区 | 🔴 未修复 | 转 M-03 |
|
||||
| D-07 | 楼盘价格走势/市场报盘 | 🟡 部分修复 | M-08（降级） |
|
||||
| T-01/T-02/T-03 | Celery/R2/索引规范 | 🔴 均未修复 | 合并 M-04 |
|
||||
| U-01/U-02/U-03/U-06 | UI 组件 + Wireframe | 🟡 U-06 已修复，复杂组件仍缺 | 转 M-09 |
|
||||
| X-01 | Redis Key tenant 前缀 | 🟡 部分修复 | M-10（降级） |
|
||||
| X-02 | 租户注销 SOP | ✅ 已修复 | 关闭 |
|
||||
| S-01 | 加密密钥管理 | 🔴 未修复 | M-11 |
|
||||
|
||||
---
|
||||
|
||||
## 一、PRD 一致性审查（PRD ↔ PRD ↔ TASK）
|
||||
|
||||
### 🔴 B-01 系统配置 PRD 空骨架阻塞 9 条 P0/P1 任务
|
||||
|
||||
**文档**：`PRD/系统配置/系统配置.md:1-128`（仅章节标题，无内容）；`PRD/TASK.md:59, 86-88, 107-110`
|
||||
|
||||
**事实**：
|
||||
- 系统配置 PRD 仅 128 行骨架，无任何字段定义、规则说明、UI 描述。
|
||||
- TASK.md 在 Phase 1/2/3 共排期 9 条 Story，全部以"参考 PRD：系统配置.md"为权威：
|
||||
- P0：US-SETTING-001（房源相关设置-字段必填/自定义字段/标签）
|
||||
- P1：US-SETTING-010/011/012（首页/相关方/客源参数）
|
||||
- P2：US-SETTING-020~023（人事 OA/交易/财务/合同模板）
|
||||
- 客源 PRD `DATA_MODEL_CLIENT.md:31` 提到"活跃度新配偶 30 天"阈值由 lookup_items 配置，但配置入口 PRD 不存在。
|
||||
- 房源 PRD 多处提到"字段标签可配置"、"自定义字段"、"标签管理"，全部指向系统配置 PRD。
|
||||
|
||||
**影响**：
|
||||
- US-SETTING-001 是 Phase 1 P0，无 PRD 即无法启动开发。
|
||||
- 所有"可配置"行为（活跃度阈值、自动转公阈值、字段标签）的运营管理入口断链。
|
||||
- DATA_MODEL_PUBLIC `lookup_items` 表已建好，但其管理 UI 与权限设计无规格。
|
||||
|
||||
**责任**：PM 必须在编码 Phase 1 启动前完成 `PRD/系统配置/系统配置.md` 至少 P0 范围（US-SETTING-001 内容），否则 TASK.md 应将 US-SETTING-001 暂时移出 Phase 1。
|
||||
|
||||
---
|
||||
|
||||
### 🔴 B-02 核心枚举三方不一致（PRD ↔ DDL ↔ TASK AC）
|
||||
|
||||
**文档**：多处（见下表）
|
||||
|
||||
| 字段 | PRD 表述 | DATA_MODEL CHECK | TASK AC | 冲突类型 |
|
||||
|------|----------|------------------|---------|----------|
|
||||
| 客源 status | `PRD/客源管理/客源管理模块PRD.md:465,504,1757,1786`：求购/求租/租购/我购/我租/成交/暂缓/无效/公 | `DATA_MODEL_CLIENT.md:471,511`：`buying / renting / buy_or_rent`（仅 3 值，无暂缓/无效/我购/我租/成交/公） | `TASK.md` US-CLIENT-002/010/011/012/013 均按 PRD 中文 | 三方均不同 |
|
||||
| 客源 grade | PRD `1634`：A~E 五级 | `DATA_MODEL_PROPERTY.md:266`（注：客源 grade 字段实际位置待确认）：`A_urgent/A/B/C/D` 5 项无 E | `TASK.md:308` US-CLIENT-009：A 急迫/B 较强/C 一般/D 较弱/E 暂不关注 五级 | DDL 缺 E，混用 `A_urgent` |
|
||||
| 房源 status | PRD 8 态：出售/出租/租售/暂缓/他售/他租/成交/未挂牌 | `DATA_MODEL_PROPERTY.md:197`：`for_sale/for_rent/for_sale_rent/...` | `TASK.md:232` US-PROPERTY-008：在售→暂缓/成交/下架；`TASK.md:1121` 隐式查询 `status NOT IN ('sold','unlisted')` | DDL CHECK 全集未冻结，"sold/unlisted/paused" 出现在查询但未确认在 CHECK 中 |
|
||||
| 操作日志 type | 客源 PRD `668`：明文"系统实际枚举值更多" | DATA_MODEL_CLIENT 未给 CHECK 完整列表 | `TASK.md:578-581` US-CLIENT-024 AC："按操作类型筛选" | 全集未冻结 |
|
||||
| 二级 Tab "暂缓" | 客源 PRD `713-717` Tab：求购/求租/暂缓/全部私客 | `DATA_MODEL_CLIENT.md:471,511` 状态枚举无 `paused/on_hold` | `US-CLIENT-002` AC 含暂缓 Tab | DDL 缺值 |
|
||||
| `tenants.status` | `DATA_MODEL_PUBLIC.md:96`：6 值含 `failed` | 同文件 `488-494` 状态机图：仅 5 态，未含 `failed` | — | 同一文档内自相矛盾 |
|
||||
|
||||
**影响**：
|
||||
- 迁移脚本将凭主观翻译，开发期会出现"经纪人在 PRD 看到的中文"与"DDL CHECK 拒绝的英文"不一致的运行时报错。
|
||||
- 状态机断链导致 transition 校验逻辑（如"暂缓→在售"）无权威实现。
|
||||
- 测试 factory 不知如何取样；form choices 与 query filter 各自定义。
|
||||
|
||||
**责任**：PM + 架构师共同冻结一份《Fonrey 枚举字典 v1.0》（建议放入 `DATA_MODEL/ENUMS.md`），所有 PRD/DDL/TASK 引用统一锚点。
|
||||
|
||||
---
|
||||
|
||||
### 🔴 B-03 权限数据范围档位冲突仍为 Blocker（昨日 P-01 部分修复）
|
||||
|
||||
**文档**：`PRD/权限管理/权限管理模块PRD.md:46`（§3 非目标）vs `:492-499`（§5.6 五档）vs `DATA_MODEL/DATA_MODEL_PERMISSION.md:24,31`（DataScope 跨层级叠加）
|
||||
|
||||
**事实**：
|
||||
- §3 非目标仍写"数据范围控制以**本人 / 本部门 / 全公司**三档为主，本期不含行级权限"。
|
||||
- §5.6 已扩为五档（本人/本组/本门店/本区域/全公司）。
|
||||
- DATA_MODEL_PERMISSION `staff_data_scopes` 已实现"跨层级叠加"（"本组 ∪ 门店B"），实质上为对象级权限的弱化形态，已超出 §3 非目标边界。
|
||||
|
||||
**影响**：
|
||||
- 同一 PRD 内 §3 与 §5.6 自相矛盾，开发无权威依据。
|
||||
- `staff_data_scopes` 是否本期落地不明，权限模块（US-PERMISSION-001~005）无法启动。
|
||||
|
||||
**责任**：PM 在 PRD v1.2 中锁定五档 + 删除 §3 三档表述 + 明确 DataScope 跨层级叠加是否本期范围。
|
||||
|
||||
---
|
||||
|
||||
### 🟠 M-07 楼盘合并/申请流程仅占位
|
||||
|
||||
**文档**：`PRD/房源管理/楼盘管理模块PRD.md:117`
|
||||
|
||||
**事实**：PRD 仅写"小区名称/地址只读 — 合并/申请流程"占位，未定义触发条件、审批人、合并规则、对历史房源/客源的影响。`DATA_MODEL_COMPLEX` 无对应表/工作流。
|
||||
|
||||
**影响**：US-COMPLEX-001 录入与维护流程不闭环；楼盘合并将在生产环境出现"合并后历史房源 complex_id 漂移"问题。
|
||||
|
||||
---
|
||||
|
||||
### 🟠 M-08 客源新房/租房 Tab 字段未冻结
|
||||
|
||||
**文档**：`PRD/客源管理/客源管理模块PRD.md:1663`（明文"字段待截图补充确认"）
|
||||
|
||||
**事实**：US-CLIENT-014（编辑客源 Tab 三：二手/新房/租房）AC 不可测，DATA_MODEL_CLIENT 需求字段 schema 不可冻结。
|
||||
|
||||
---
|
||||
|
||||
## 二、DATA_MODEL 完整性审查
|
||||
|
||||
### 🔴 B-04 Keyset 分页规范全局缺位（昨日 D-01 未修复）
|
||||
|
||||
**文档**：`TECH_STACK/TECH_STACK.md`（全文未现 keyset/分页规范）；`DATA_MODEL_PROPERTY.md:1083` §6 / `DATA_MODEL_CLIENT.md:453` §五 查询模式参考章节均未给 Keyset SQL 模板
|
||||
|
||||
**事实**：
|
||||
- 89k 房源 + 200 万/年跟进日志使用 OFFSET 分页，第 100 页响应将达秒级。
|
||||
- TASK.md US-PROPERTY-002 / US-CLIENT-002 的 AC 均要求"<2 秒"。
|
||||
- 测试规范无性能基准任务覆盖。
|
||||
|
||||
**责任**：架构师在 `TECH_STACK.md` 新增 §「分页规范」，并在 `DATA_MODEL_PROPERTY.md` §6 / `DATA_MODEL_CLIENT.md` §五 给出 Keyset SQL 模板（`WHERE (created_at, id) < (?, ?) ORDER BY created_at DESC, id DESC LIMIT 21`）。
|
||||
|
||||
---
|
||||
|
||||
### 🟠 M-02 主表乐观锁/楼盘锁规则缺失
|
||||
|
||||
**文档**：`DATA_MODEL_COMPLEX.md:194-197`（楼盘 4 类锁字段已建）；`DATA_MODEL_PROPERTY.md` / `DATA_MODEL_CLIENT.md` / `DATA_MODEL_COMPLEX.md` 主表均无 `version` 字段；`PRD/房源管理/楼盘管理模块PRD.md:82` 列出 4 类锁但无解锁/过期规则
|
||||
|
||||
**事实**：
|
||||
- 楼盘 `lock_building/lock_room/lock_info/lock_standard_room` 字段已落库 ✅
|
||||
- 但 PRD 未规定每类锁的解锁权限、自动/手动解锁规则、锁过期机制。
|
||||
- US-COMPLEX-001/002 AC 不含锁状态变更流程。
|
||||
- `properties.version` / `clients.version` 仍未补，多人协作场景的写冲突无防护。
|
||||
|
||||
**责任**：架构师补 `version` 字段；PM 补 4 类锁的业务规则。
|
||||
|
||||
---
|
||||
|
||||
### 🟠 M-03 高写入表分区 DDL 仍未落地（昨日 D-04 未修复）
|
||||
|
||||
**文档**：`DATA_MODEL_PROPERTY.md:523`（follow_logs）/ `:801`（property_photos）；`DATA_MODEL.md:596-604`（仅文字描述）
|
||||
|
||||
**事实**：
|
||||
- `follow_logs`（200 万+/年）、`property_photos`（500 万+）、`permission_change_logs`、`login_attempts`、`platform_audit_logs` 的 DDL 中均无 `PARTITION BY` 子句，仅在文字描述/分区策略表中"建议按月 RANGE 分区"。
|
||||
- 开发期不分区，未来再分区将带来全表数据迁移成本。
|
||||
|
||||
**责任**：架构师在每张高写入表的 DDL 中给出 `PARTITION BY RANGE (created_at)` + `pg_partman` 自动维护脚本。
|
||||
|
||||
---
|
||||
|
||||
### 🟠 M-08 楼盘价格走势已建，市场报盘仍缺（昨日 D-07 部分修复）
|
||||
|
||||
**文档**：`DATA_MODEL_COMPLEX.md:376` `complex_price_trends` ✅；`DATA_MODEL_CLIENT.md:505` §5.4 私客自动转公（仅 SQL 查询示例，非完整 job）；`market_quotes` 表全表缺失
|
||||
|
||||
**事实**：
|
||||
- 楼盘价格走势 ✅ 已建。
|
||||
- "市场报盘"（房源 PRD §）仍无 `market_quotes` 表。
|
||||
- 私客→公客自动转换仅给查询 SQL 示例，无完整 Celery beat 调度任务定义、无幂等保证、无并发控制。
|
||||
|
||||
**影响**：US-CLIENT-016（系统自动将超时无跟进的私客转为公客）AC 无法实现。
|
||||
|
||||
---
|
||||
|
||||
## 三、TECH_STACK 完整性审查
|
||||
|
||||
### 🟠 M-04 三大横切规范全部仍未补（昨日 T-01/T-02/T-03 未修复）
|
||||
|
||||
**文档**：`TECH_STACK/TECH_STACK.md`（全文）
|
||||
|
||||
| 子项 | 现状 | 影响 |
|
||||
|------|------|------|
|
||||
| Celery 多租户 schema 切换 | TECH_STACK.md L30/44/80/90/165 仅零散提及 Celery，无统一规范 | Worker 在错误 schema 下查询、跨租户数据污染 |
|
||||
| R2 文件路径前缀 | TECH_STACK.md L31/105/119/166 仅提及 bucket 名 | 共享 bucket 下 `file_key` 无租户标识，存在隐患 |
|
||||
| 查询索引矩阵 | TECH_STACK/ 目录无"索引规范.md"；DATA_MODEL.md §六 实为 Redis 缓存策略 | 89k 房源多条件筛选无覆盖索引验证依据 |
|
||||
|
||||
**责任**：架构师补 §「异步任务规范」、§「R2 路径规范」、`TECH_STACK/索引规范.md` 三份内容。
|
||||
|
||||
---
|
||||
|
||||
### 🟠 M-01 测试规范与 django-tenants 多租户工作模式冲突
|
||||
|
||||
**文档**：`TECH_STACK/测试规范.md:393-413`（settings_test）/`570-575`（引用未实现的 TenantClient）/`594`（禁用真实外部服务）
|
||||
|
||||
**事实**：
|
||||
- `LocMemCache`（402-406）：django-tenants 需要 `tenant-cache-key` 前缀化的 Redis 行为；权限快照走 Redis（TASK L617 AC "Redis 权限快照失效重载"），切到 LocMem 后跨进程/跨租户隔离测试失效。
|
||||
- `FileSystemStorage`（411-413）：替换 R2 用于测试本身合理，但与 §594 "禁止测试直接调用真实外部服务"叠加后，`tenants/{schema_name}/...` 路径前缀策略（M-04 R2）无 mock 验证手段。
|
||||
- `CELERY_TASK_ALWAYS_EAGER=True`（398-399）：US-CLIENT-003 等要求"Celery 异步处理"的 AC 无法在 EAGER 模式下验证异步语义。
|
||||
- `DATABASES "由环境变量注入"`（393-395）：未约定 `ENGINE=django_tenants.postgresql_backend`，可能落到错误的 backend 上。
|
||||
- `TenantClient` / 多租户 fixture：仅在 §11.2 Prompt 模板中被引用，无实现，§12 又"禁止使用 Django 原生 Client()"，造成所有 integration 测试无统一基类。
|
||||
|
||||
**影响**：
|
||||
- 权限模块 Redis 快照失效场景、跨租户隔离场景的自动化测试不可信。
|
||||
- US-PERMISSION-* / US-CLIENT-003 / US-CLIENT-016 等核心场景的覆盖率指标存在水分。
|
||||
|
||||
**责任**：架构师与测试负责人协同：(a) `tests/conftest.py` 提供 `TenantClient` 与 `tenant_factory` 实现；(b) Redis 改用 fakeredis 而非 LocMemCache；(c) 异步场景使用 `pytest-celery` 的 `celery_app` fixture，区分 EAGER 单元测试与真实 broker 集成测试。
|
||||
|
||||
---
|
||||
|
||||
### 🟠 M-05 性能 NFR 无 SLI/SLO 落地（昨日 P-04 未修复）
|
||||
|
||||
**文档**：`TASK.md:189, 252` US-PROPERTY-002 / US-CLIENT-002 AC 写"89000 条数据下查询 < 2 秒"；`PRD/客源管理/客源管理模块PRD.md:1958-1960` 仅泛述"分页加载"；TECH_STACK 与测试规范无 p95/索引/EXPLAIN 验收
|
||||
|
||||
**事实**：
|
||||
- 性能 AC 散布于 PRD/TASK 多处（录入耗时 ≤ 30s、配房 < 3s、登录成功率、自动更新 ≥ 98%、RTO < 2h），无统一的 NFR 矩阵。
|
||||
- TECH_STACK 未承接为 SLI/SLO；测试规范无性能基准任务（如 pytest-benchmark + EXPLAIN ANALYZE 校验）。
|
||||
|
||||
**影响**：所有性能 AC 不可测，验收阶段将沦为主观判断。
|
||||
|
||||
---
|
||||
|
||||
### 🟠 M-11 加密密钥管理方案缺失（昨日 S-01 未修复）
|
||||
|
||||
**文档**：`TECH_STACK/TECH_STACK.md:96` 仅一句"禁止硬编码密钥"
|
||||
|
||||
**事实**：
|
||||
- 全文未给出 KMS / Vault / 环境变量加密管理说明。
|
||||
- `core.encryption` 仅作为模块名出现，密钥轮换流程未定义。
|
||||
- 手机号、身份证、银行卡号、产证号等多处使用 AES-256-GCM 加密，密钥泄露后批量解密风险高。
|
||||
|
||||
**责任**：架构师补 KMS/Vault 选型 + 轮换 SOP + `core.encryption` 接口契约。
|
||||
|
||||
---
|
||||
|
||||
## 四、UI/UX 完整性审查
|
||||
|
||||
### ✅ U-06 已修复：UI_DESIGN 原型已启动
|
||||
|
||||
**证据**：`UI_DESIGN/preview.html`、`UI_DESIGN/客源列表_UI.html`（81KB）、`客源详情_UI.html`、`房源列表_UI.html`、`新增客源_UI.html`、`编辑客源_UI.html`；`UI_DESIGN/客源管理/`（4 份 md，60KB+/40KB+）；`UI_DESIGN/房源管理/房源列表_UI.md`（69KB）
|
||||
|
||||
**事实**：HTML 原型与对应 Markdown 设计稿均已启动，规范深度足够（含功能范围/页面规范/Data Table/HTMX/权限矩阵）。
|
||||
|
||||
---
|
||||
|
||||
### 🟠 M-09 复杂组件原型仍缺 + 楼盘/客源原型不完整（昨日 U-01/U-02/U-03 未修复）
|
||||
|
||||
**文档**：`UI_SYSTEM/UI_SYSTEM.md`（1742 行）；`UI_SYSTEM/组件清单.md:186`（仅在评论中提到 TreeSelect "建议手写 80~120 行 JS"）；`UI_DESIGN/房源管理/` 仅 1 个文件；`UI_DESIGN/客源管理/` 缺关键浮层
|
||||
|
||||
**事实**：
|
||||
- 组件清单仍缺 4 类关键复杂组件：
|
||||
1. **Stepper / Wizard**（多步表单）：录入私客、转成交浮层
|
||||
2. **PermissionTree**（14 个一级模块树 + 多分组 + Toggle/Select/Number）：US-PERMISSION-001/004 最复杂页面
|
||||
3. **智能配房比对器**（4 分组卡片）：US-CLIENT-020 录客配房
|
||||
4. **审批前置弹层**：US-CLIENT-015 跨团队需店长权限、号码方审批
|
||||
- UI_DESIGN/房源管理/ 仅 `房源列表_UI.md`，TASK.md L152/160 引用 `楼盘列表_UI.md` / `楼盘详情_UI.md` 不存在。
|
||||
- UI_DESIGN/客源管理/ 缺 US-CLIENT-007（带看）/011（转公客）/012（转成交）/015（相关员工）/020（智能配房）独立原型，全部压缩进 `客源详情_UI.md` 单文件，颗粒度不足。
|
||||
|
||||
**责任**：UI/UX 设计师补 4 类复杂组件 + 楼盘 2 份原型 + 客源 5 份关键浮层独立原型。
|
||||
|
||||
---
|
||||
|
||||
## 五、多租户隔离审查（横切）
|
||||
|
||||
### 🟠 M-10 Redis Key 命名跨模块未统一（昨日 X-01 部分修复）
|
||||
|
||||
**文档**：`TECH_STACK/登录管理技术方案.md:553-562`（§九 Redis Key 规范，使用 `{tenant_id}:` 前缀）；`TECH_STACK.md` / `权限管理系统技术方案.md` 未见统一规范引用
|
||||
|
||||
**事实**：
|
||||
- 登录方案已落地 Redis Key 规范表 ✅，但前缀使用 `{tenant_id}:` 而非昨日 Review 建议的 `{tenant_schema_name}:`。
|
||||
- TECH_STACK.md 总纲未引用此规范；权限技术方案未明确 Redis Key 是否带 tenant 前缀。
|
||||
- 跨模块（登录/权限/缓存）若不统一，多租户 Redis 共享时存在键冲突风险。
|
||||
|
||||
**责任**：架构师在 `TECH_STACK.md` 新增 §「Redis Key 规范」，统一 `{tenant_schema_name}:` 前缀（schema_name 比 tenant_id 在日志/调试时更直观），并在登录/权限技术方案中显式引用。
|
||||
|
||||
---
|
||||
|
||||
## 六、合规与安全审查
|
||||
|
||||
### 🟠 M-06 客户端发布无防降级/重放校验
|
||||
|
||||
**文档**：`PRD/发布管理/客户端发布管理模块PRD.md:58`（明文非目标"客户端代码混淆/反逆向"）；`DATA_MODEL_PUBLIC.md:413-416, 419-421`
|
||||
|
||||
**事实**：
|
||||
- `/api/client/updates/latest/` 响应未要求服务端签名。
|
||||
- `client_releases.download_url` 为 R2 公开 URL，无签名/有效期，旧客户端可继续拉取旧版本绕过强制升级。
|
||||
- `min_required_version` 由"应用层比较 SemVer"，未定义服务端签名/客户端公钥校验。
|
||||
- US-RELEASE-011 SHA256 仅校验完整性，不防替换（中间人可投递降级版本）。
|
||||
|
||||
**影响**：中间人攻击可投递已撤销的旧版本（含已修复漏洞的版本），合规与安全双重风险。
|
||||
|
||||
**责任**：架构师与发布团队补：(a) 更新元数据签名（Ed25519/RSA-PSS）；(b) `download_url` 改用签名 URL（短有效期）；(c) 客户端内置公钥校验。
|
||||
|
||||
---
|
||||
|
||||
### 🟠 M-12 RPO 数值缺失，备份计划与 SLA 断链
|
||||
|
||||
**文档**：`PRD/系统管理/系统管理模块PRD.md:58`（仅 RTO < 2h）；`DATA_MODEL_PUBLIC.md:265-280` `backup_schedules` 默认 daily 02:00、retention 10
|
||||
|
||||
**事实**：
|
||||
- daily 频率隐含 RPO ≈ 24h 但 PRD 未声明。
|
||||
- 与"全局/租户级覆盖"组合下，SLA 不可承诺。
|
||||
- `tenants.status` 含 `failed`（DDL）但状态机图（同文 488-494）仅 5 态未含 `failed`，故障租户无 SLA 出口。
|
||||
|
||||
**责任**：PM 补 RPO 数值；架构师补 `failed` 状态的退出路径。
|
||||
|
||||
---
|
||||
|
||||
## 七、性能与容量审查
|
||||
|
||||
### 🟠 M-05 89k 数据 < 2 秒 AC 不可测（已在 §三列出）
|
||||
|
||||
### 🟠 M-13 智能配房算法路径仍未文档化
|
||||
|
||||
**文档**：客源 PRD §5 智能配房；TECH_STACK 与 DATA_MODEL 未有"实时 vs 异步""匹配算法 SQL"说明
|
||||
|
||||
**事实**：
|
||||
- 客源 PRD 要求智能配房响应 < 3s。
|
||||
- 匹配字段（户型/面积/价格/区域）跨 properties/clients 两表的 4-6 维筛选，是否使用物化视图/PostgreSQL 多列 GiST 未定。
|
||||
- `client_property_matches` 表是否本期建（TASK Phase 3 才有 US-CLIENT-020）？若 Phase 1 不需要，PRD 提及的 < 3s 应明确不在 Phase 1 范围。
|
||||
|
||||
---
|
||||
|
||||
## 八、可维护性与扩展性
|
||||
|
||||
### 🟠 M-14 TASK.md PRD 锚点链接断链
|
||||
|
||||
**文档**：`TASK.md:124, 132, 139, 150, 158, 167` 等
|
||||
|
||||
**事实**：TASK.md 大量"参考 PRD：xxx#锚点"形式的链接，但所引锚点（如 `登录管理/用户登录管理模块PRD.md - 账号密码登录` / `楼盘管理模块PRD.md - 楼盘信息管理`）在对应 PRD 中作为标题不存在，开发无法直接跳转。
|
||||
|
||||
**影响**：TASK.md 作为开发入口的"可执行性"打折扣。
|
||||
|
||||
---
|
||||
|
||||
## 九、行动清单（按责任人 + 优先级）
|
||||
|
||||
### PM（产品经理）
|
||||
|
||||
| ID | 等级 | 任务 | 关联编号 |
|
||||
|----|------|------|----------|
|
||||
| PM-1 | 🔴 | **补齐 `PRD/系统配置/系统配置.md` P0 范围**（US-SETTING-001：字段必填/自定义字段/标签管理；lookup_items 配置入口；活跃度/转公阈值配置） | B-01 |
|
||||
| PM-2 | 🔴 | 与架构师协同冻结《Fonrey 枚举字典 v1.0》（建议 `DATA_MODEL/ENUMS.md`），覆盖客源 status/grade、房源 status、操作日志 type、`tenants.status` 等核心枚举 | B-02 |
|
||||
| PM-3 | 🔴 | 修订权限 PRD v1.2：删除 §3 三档表述，锁定五档 + DataScope 跨层级叠加是否本期范围 | B-03 |
|
||||
| PM-4 | 🟠 | 楼盘合并/申请流程详述（触发/审批/对历史房源影响） | M-07 |
|
||||
| PM-5 | 🟠 | 客源新房/租房 Tab 字段冻结 | M-08 |
|
||||
| PM-6 | 🟠 | 在 `PRD_MVP.md` 中汇总 NFR 矩阵（性能/可用性/安全/合规），含 RPO 数值 | M-05, M-12 |
|
||||
| PM-7 | 🟠 | 修复 TASK.md 内所有 PRD 锚点断链 | M-14 |
|
||||
|
||||
### 架构师
|
||||
|
||||
| ID | 等级 | 任务 | 关联编号 |
|
||||
|----|------|------|----------|
|
||||
| ARCH-1 | 🔴 | TECH_STACK 新增 §「分页规范」+ Keyset SQL 模板；DATA_MODEL_PROPERTY/CLIENT §查询模式补 Keyset 示例 | B-04 |
|
||||
| ARCH-2 | 🟠 | 修复测试规范多租户冲突：补 `TenantClient` 实现、`tenant_factory` fixture；Redis 改用 fakeredis；区分 EAGER/真实 broker | M-01 |
|
||||
| ARCH-3 | 🟠 | `properties` / `clients` 主表补 `version`；楼盘 4 类锁补解锁/过期机制（与 PM 协同） | M-02 |
|
||||
| ARCH-4 | 🟠 | 高写入表（follow_logs/property_photos/permission_change_logs/login_attempts/platform_audit_logs）补 `PARTITION BY RANGE` DDL + pg_partman 脚本 | M-03 |
|
||||
| ARCH-5 | 🟠 | TECH_STACK 三大横切规范补：异步任务（Celery tenant_schema 必传 + `@with_tenant_context`）/R2 路径前缀（`tenants/{schema_name}/...`）/查询索引矩阵 | M-04 |
|
||||
| ARCH-6 | 🟠 | TECH_STACK 性能 NFR → SLI/SLO 映射 + 测试规范补 pytest-benchmark/EXPLAIN ANALYZE | M-05 |
|
||||
| ARCH-7 | 🟠 | 客户端发布签名机制：服务端签发 Ed25519/RSA-PSS、客户端内置公钥、`download_url` 改签名 URL | M-06 |
|
||||
| ARCH-8 | 🟠 | `market_quotes` 表 + 私客自动转公完整 Celery beat 任务（含幂等/并发控制） | M-08 |
|
||||
| ARCH-9 | 🟠 | TECH_STACK 新增 §「Redis Key 规范」，统一 `{tenant_schema_name}:` 前缀 | M-10 |
|
||||
| ARCH-10 | 🟠 | 加密密钥管理方案：KMS/Vault 选型、轮换 SOP、`core.encryption` 接口契约 | M-11 |
|
||||
| ARCH-11 | 🟠 | `tenants.status='failed'` 状态机出口补全 | M-12 |
|
||||
| ARCH-12 | 🟠 | 智能配房算法路径文档（实时 vs 异步、索引/物化视图选型） | M-13 |
|
||||
|
||||
### UI/UX 设计师
|
||||
|
||||
| ID | 等级 | 任务 | 关联编号 |
|
||||
|----|------|------|----------|
|
||||
| UI-1 | 🟠 | 补 4 类复杂组件原型：Stepper/Wizard、PermissionTree、智能配房比对器、审批前置弹层 | M-09 |
|
||||
| UI-2 | 🟠 | 补 UI_DESIGN/房源管理/楼盘列表_UI.md / 楼盘详情_UI.md（修复 TASK 引用） | M-09 |
|
||||
| UI-3 | 🟠 | 客源关键浮层独立原型：US-CLIENT-007/011/012/015/020 | M-09 |
|
||||
|
||||
---
|
||||
|
||||
## 十、结论
|
||||
|
||||
### 是否可进入开发阶段
|
||||
|
||||
**否，必须先解决 4 个 Blocker**：
|
||||
|
||||
| 编号 | Blocker | 影响范围 |
|
||||
|------|---------|----------|
|
||||
| B-01 | 系统配置 PRD 空骨架 | 阻塞 9 条 P0/P1 任务（US-SETTING-001/010~012/020~023） |
|
||||
| B-02 | 核心枚举三方不一致 | 阻塞客源/房源全部 CRUD 模块（US-CLIENT-* / US-PROPERTY-*） |
|
||||
| B-03 | 权限档位冲突 | 阻塞权限模块（US-PERMISSION-001~005） |
|
||||
| B-04 | Keyset 分页缺失 | 阻塞高基数列表查询（US-PROPERTY-002 / US-CLIENT-002） |
|
||||
|
||||
**可立即并行启动的部分**：
|
||||
- 公共 Schema（`tenants`、`platform_admins`、`audit_logs`）开发——文档完整，无 Blocker。
|
||||
- 楼盘/区域/学校 CRUD——DATA_MODEL_COMPLEX 完整、4 类锁字段已建，主流程清晰（解锁规则可后置补充）。
|
||||
- Electron 客户端框架搭建——TECH_STACK §8 决策已封闭（但发布签名机制 M-06 需在 RC 前补齐）。
|
||||
- 登录模块（accounts App）——文档完整（含 Redis Key 规范），仅需在多租户场景下补 fakeredis 测试基线。
|
||||
- UI 静态原型实施——核心 5 页已有 HTML 原型可直接转 HTMX 模板。
|
||||
|
||||
**必须延后的部分**：
|
||||
- 系统配置模块（B-01 解除前）。
|
||||
- 权限模块（B-03 解除前）。
|
||||
- 房源/客源高基数列表查询（B-04 解除前；可先实施 OFFSET 分页，但需排期重构）。
|
||||
- 89k 性能 AC 验收（M-05 解除前，AC 不可测）。
|
||||
|
||||
### 文档体系成熟度评分（与昨日对比）
|
||||
|
||||
| 维度 | 今日 | 昨日 | 变化 | 说明 |
|
||||
|------|------|------|------|------|
|
||||
| PRD 业务清晰度 | 7.5/10 | 8.5/10 | ↓ | 新发现枚举塌方 + 系统配置空骨架 |
|
||||
| DATA_MODEL 落地深度 | 8.5/10 | 8/10 | ↑ | 楼盘锁字段、价格走势、租户注销已落地 |
|
||||
| TECH_STACK 完整性 | 6/10 | 6/10 | — | 三大横切规范仍未补 |
|
||||
| UI/UX 系统化 | 7/10 | 5/10 | ↑↑ | UI_DESIGN 原型显著启动 |
|
||||
| 跨文档一致性 | 5.5/10 | 6.5/10 | ↓ | 枚举塌方拉低评分 |
|
||||
| 多租户隔离严谨度 | 7.5/10 | 7.5/10 | — | Redis Key 部分修复但未统一 |
|
||||
| 测试可执行性 | 5/10 | (未评估) | 🆕 | 测试规范与 django-tenants 工作模式冲突 |
|
||||
| **整体** | **6.8/10** | **7/10** | ↓ 0.2 | 新发现风险大于已修复风险，但绝对水平仍属"中等偏上" |
|
||||
|
||||
**结论**：本次 Review 的实质性进展（UI 启动、租户注销 SOP、楼盘锁字段）值得肯定，但**枚举塌方**和**测试规范多租户冲突**是新发现的系统性风险，必须在编码启动前优先治理。在 4 个 Blocker 解除 + 6 项核心 Major 修复后，整体成熟度可重回 8.0+。
|
||||
|
||||
---
|
||||
|
||||
## 附录 A：本次 Review 涵盖文档
|
||||
|
||||
### PRD（8 份）
|
||||
- `PRD/PRD_MVP.md` v1.0
|
||||
- `PRD/TASK.md`（834 行）
|
||||
- `PRD/房源管理/房源管理模块PRD.md` v2.1（1881 行）
|
||||
- `PRD/房源管理/楼盘管理模块PRD.md`
|
||||
- `PRD/客源管理/客源管理模块PRD.md` v1.4（2050 行）
|
||||
- `PRD/权限管理/权限管理模块PRD.md` v1.1
|
||||
- `PRD/组织人事管理/组织人事管理模块PRD.md` v1.2
|
||||
- `PRD/系统管理/系统管理模块PRD.md` v1.0
|
||||
- `PRD/登录管理/用户登录管理模块PRD.md` v1.4
|
||||
- `PRD/发布管理/客户端发布管理模块PRD.md` v1.0
|
||||
- `PRD/系统配置/系统配置.md`（128 行骨架）
|
||||
|
||||
### DATA_MODEL（8 份）
|
||||
- `DATA_MODEL/DATA_MODEL.md` v1.3、`DATA_MODEL_PUBLIC.md`、`DATA_MODEL_ORG.md`、`DATA_MODEL_COMPLEX.md`、`DATA_MODEL_PROPERTY.md`、`DATA_MODEL_CLIENT.md`、`DATA_MODEL_PERMISSION.md`、`DATA_MODEL_LOGIN.md`
|
||||
|
||||
### TECH_STACK（4 份）
|
||||
- `TECH_STACK/TECH_STACK.md` v2.0
|
||||
- `TECH_STACK/登录管理技术方案.md` v2.0
|
||||
- `TECH_STACK/权限管理系统技术方案.md` v1.0
|
||||
- `TECH_STACK/测试规范.md` v1.0
|
||||
|
||||
### UI_SYSTEM（3 份）
|
||||
- `UI_SYSTEM/UI_SYSTEM.md` v1.2（1742 行）
|
||||
- `UI_SYSTEM/组件清单.md`
|
||||
- `UI_SYSTEM/组件规范设计.md`
|
||||
|
||||
### UI_DESIGN（10 份原型/设计稿）
|
||||
- `UI_DESIGN/preview.html`、`客源列表_UI.html`、`客源详情_UI.html`、`房源列表_UI.html`、`新增客源_UI.html`、`编辑客源_UI.html`
|
||||
- `UI_DESIGN/客源管理/客源列表_UI.md`、`客源详情_UI.md`、`新增客源_UI.md`、`编辑客源_UI.md`
|
||||
- `UI_DESIGN/房源管理/房源列表_UI.md`
|
||||
|
||||
---
|
||||
|
||||
## 附录 B：问题汇总速查表
|
||||
|
||||
| ID | 等级 | 维度 | 责任人 | 简述 | vs 昨日 |
|
||||
|----|------|------|--------|------|---------|
|
||||
| B-01 | 🔴 | PRD↔TASK | PM | 系统配置 PRD 空骨架 | ⚠️ P-02 升级 |
|
||||
| B-02 | 🔴 | PRD↔Data↔TASK | PM+架构师 | 核心枚举三方不一致 | 🆕 新增 |
|
||||
| B-03 | 🔴 | PRD↔Data | PM | 权限档位三档/五档冲突 | ⚠️ P-01 部分修复 |
|
||||
| B-04 | 🔴 | TECH/Data | 架构师 | Keyset 分页规范缺失 | ⚠️ D-01 未修复 |
|
||||
| M-01 | 🟠 | TECH/测试 | 架构师 | 测试规范与 django-tenants 冲突 | 🆕 新增 |
|
||||
| M-02 | 🟠 | Data↔PRD | 架构师+PM | 主表乐观锁/楼盘锁规则缺失 | ⚠️ D-02 部分修复 |
|
||||
| M-03 | 🟠 | Data | 架构师 | 高写入表分区 DDL 未落地 | ⚠️ D-04 未修复 |
|
||||
| M-04 | 🟠 | TECH/多租户 | 架构师 | Celery/R2/索引矩阵三大横切规范 | ⚠️ T-01/T-02/T-03 未修复 |
|
||||
| M-05 | 🟠 | NFR↔TECH↔测试 | PM+架构师 | 89k <2 秒 AC 不可测 | 🆕 新增（含昨 P-04） |
|
||||
| M-06 | 🟠 | 安全 | 架构师 | 客户端发布无防降级/签名 | 🆕 新增 |
|
||||
| M-07 | 🟠 | PRD | PM | 楼盘合并/申请流程占位 | 🆕 新增 |
|
||||
| M-08 | 🟠 | PRD/Data | PM+架构师 | 客源新房/租房字段未冻结 + market_quotes 缺 + 转公 job 不完整 | 部分新增 |
|
||||
| M-09 | 🟠 | UI | UI/UX | 复杂组件 + 楼盘/客源浮层原型缺 | ⚠️ U-01/U-02/U-03 未修复 |
|
||||
| M-10 | 🟠 | 多租户 | 架构师 | Redis Key 跨模块未统一前缀 | ⚠️ X-01 部分修复 |
|
||||
| M-11 | 🟠 | 安全 | 架构师 | 加密密钥管理方案缺失 | ⚠️ S-01 未修复 |
|
||||
| M-12 | 🟠 | 合规 | PM+架构师 | RPO 缺 + tenants.status=failed 状态机断链 | 🆕 新增 |
|
||||
| M-13 | 🟠 | 性能 | 架构师 | 智能配房算法路径未文档化 | ⚠️ P-11 未修复（升级） |
|
||||
| M-14 | 🟠 | 维护 | PM | TASK.md PRD 锚点断链 | 🆕 新增 |
|
||||
|
||||
**合计**：🔴 4 / 🟠 14 = **18 项 Top 风险**（聚焦 Top 模式，未列入 Minor）
|
||||
|
||||
### 已关闭项（昨日 → 今日）
|
||||
|
||||
| 昨日编号 | 简述 | 关闭依据 |
|
||||
| --------------- | ----------------------------------------- | ------------------------------------------------- |
|
||||
| X-02 | 租户注销→导出→清除链路 | `PRD/系统管理` §189-227, §552-553 已落地完整生命周期 |
|
||||
| U-06 | Wireframe 未启动 | UI_DESIGN/ 已产出 5 页 HTML + 5 份 md 设计稿 |
|
||||
| 楼盘 4 类锁字段 | DATA_MODEL_COMPLEX 缺 lock_* 字段 | `DATA_MODEL_COMPLEX.md:194-197` 已建（业务规则待补，转 M-02） |
|
||||
| 楼盘价格走势表 | DATA_MODEL_COMPLEX 缺 complex_price_trends | `DATA_MODEL_COMPLEX.md:376` 已建 |
|
||||
| 登录 Redis Key 规范 | 跨模块未统一 | 登录方案 §九已建 ✅；统一 schema 前缀仍待办（转 M-10） |
|
||||
|
||||
---
|
||||
|
||||
*Report Generated: 2026-04-26 by AI Reviewer*
|
||||
*基线版本：REVIEW_全局_2026-04-25.md*
|
||||
@@ -1,421 +0,0 @@
|
||||
# Fonrey 全局系统设计 Review 报告
|
||||
|
||||
> **Review 类型**：全量 Review（PRD + DATA_MODEL + TECH_STACK + UI/UX + TASK 交叉验证）
|
||||
> **Review 模式**：增量对比 `REVIEW_全局_2026-04-26.md`，逐条核对 4 Blocker + 12 Major 状态，识别新增风险
|
||||
> **Review 日期**：2026-04-28
|
||||
> **Reviewer**：首席系统设计 Reviewer（AI 辅助，6 路并行 explore agent 取证）
|
||||
> **当前阶段**：需求 ~90% / 数据模型 ~80% / UI 设计 ~35%（HTML 原型推进中）/ TECH 横切规范 ~70%
|
||||
> **覆盖文档**：9 份 PRD + 10 份 TECH_STACK + 9 份 DATA_MODEL（含新增 ENUMS.md v2.2）+ 3 份 UI_SYSTEM + 11 份 UI_DESIGN + TASK 总表
|
||||
> **问题分级**：🔴 Blocker（阻塞开发） / 🟠 Major（必须修复但不阻塞） / 🟡 Minor（建议优化）
|
||||
> **状态标记**：✅ 已修复 / ⚠️ 部分修复 / ❌ 未修复 / 🆕 新增
|
||||
|
||||
---
|
||||
|
||||
## 〇、执行摘要（Executive Summary）
|
||||
|
||||
### 整体评价
|
||||
|
||||
相较 2026-04-26 版本，本期文档体系出现 **5 项实质性进展**，是历次 Review 以来修复力度最大的一次：
|
||||
|
||||
1. **🟢 B-01 系统配置 PRD 落地**：`PRD/系统配置/系统配置模块PRD.md`（273 行）+ `系统配置参数数据.md`（1100 行）+ `系统配置数据模型设计说明_for_Atlas.md`（275 行）三件套已完整产出，含 US-SETTING-001-A/B/C 三条 P0 Story 与完整验收标准。MVP 范围被显式约束（明确 ❌ 标签/带看/通知/发布平台等推迟到 P1/P2），**降低实施风险**。
|
||||
2. **🟢 B-02 枚举三方不一致根除**：`DATA_MODEL/ENUMS.md` v2.2（780 行，2026-04-28）作为**唯一权威源**已建立，全量统一为 `lower_snake_case`，历史大写值（`SUCCESS`/`A_urgent`/`A_app`）的迁移规则已写明，并要求 DB CHECK / `enum_labels` / 前端三方同步。
|
||||
3. **🟢 B-03 权限档位冲突收敛**：权限 PRD §3 已显式表述"五档基线 + 各权限项可选子集（含三档场景）"；DATA_MODEL_PERMISSION 的 `staff_data_scopes` 表（已建 DDL，含 3 条索引）实现"跨层级并集"模型，PRD ↔ Data ↔ Code 三方对齐。
|
||||
4. **🟢 B-04 Keyset 分页规范化**：`API_CONTRACT.md` §4 已写入 `{"mode":"keyset","cursor":"...","limit":20}` 标准合约 + 不透明游标禁令；客源/房源/楼盘/组织 4 个模块技术方案均已采用此契约。
|
||||
5. **🟢 M-01 测试规范多租户化**：`TECH_STACK.md:208` + `测试规范.md:69,146` 三处明确"必须使用 `TenantClient`，禁止 Django 原生 `Client()`"，4 个模块技术方案同步引用。
|
||||
|
||||
但本期 Review 也确认 **3 类系统性债务未动**，且因其他模块详化反而更显突出：
|
||||
|
||||
- **❌ 主表乐观锁 `version` 字段** —— 全量 DATA_MODEL 0 命中，89k 房源并发编辑场景（PRD 明确双经纪人同时改房）将出现"后写覆盖前写"。
|
||||
- **❌ 高写入表 `PARTITION BY` DDL** —— `follow_logs` / `property_photos` / `permission_change_logs` / `login_attempts` / `platform_audit_logs` 全量 0 命中分区子句，仅文字"建议月度分区"。**200 万跟进日志 + 2 年保留期** 的查询性能未保障。
|
||||
- **❌ KMS / 密钥轮换 SOP** —— 仅见"`R2_ADMIN_KEY` 通过 Docker Secret 注入"，无主密钥轮换流程、无应急吊销、无加密字段重新封装。
|
||||
|
||||
### 核心问题摘录（Top 10）
|
||||
|
||||
| # | 等级 | 编号 | 问题 | 维度 | 状态 |
|
||||
| --- | --- | -------- | ------------------------------------------------------------------------------------------------------------------------------------------------------- | ----------- | ---------------------- |
|
||||
| 1 | 🔴 | **B-05** | 主表乐观锁 `version` 字段全量 0 实现：`properties` / `clients` / `complexes` 在 PRD 多人协作场景中是核心，DDL 无并发控制 | Data↔PRD | 🆕 升 Blocker（持续 3 次未修） |
|
||||
| 2 | 🔴 | **B-06** | 高写入表分区 DDL 仍未落地（M-03 持续 3 次未修）：5 张高频表无 `PARTITION BY RANGE` 子句，仅注释"建议月度分区"，无法在迁移期补加分区 | Data | 🆕 升 Blocker |
|
||||
| 3 | 🟠 | **M-11** | KMS / 密钥轮换 SOP 仍未补：`core/encryption.py` 已声明，但主密钥轮换、密钥版本号、加密字段重新封装、应急吊销四类流程在 TECH_STACK 与系统管理 PRD 中均无对应章节 | 安全 | ❌ 持续未修 |
|
||||
| 4 | 🟠 | **M-12** | Celery 任务 schema 切换缺统一封装：多模块技术方案声明 `tenant_schema_name` 入参，但无 `with_tenant_context` 装饰器或基类抽象，开发期容易漏写导致跨租户脏读 | TECH/多租户 | ⚠️ 部分修复 |
|
||||
| 5 | 🟠 | **M-13** | R2 路径前缀全局规范不一致：系统管理已规范 `backups/{tenant_schema}/...` `exports/{tenant_schema}/...`，但客源/房源/楼盘模块 R2 路径仍写"`property_photos/...`"无 tenant 前缀模板 | TECH/多租户 | ⚠️ 部分修复 |
|
||||
| 6 | 🟠 | **M-06** | 客户端发布无签名校验/防降级：`/api/client/updates/latest/` 与 `download_url` 对外公开，仅 SHA256 完整性校验，可被 MITM 投递降级版本（昨日 M-06 未修） | 安全 | ❌ 持续未修 |
|
||||
| 7 | 🟠 | **M-14** | ORM Manager / QuerySet 数据范围统一封装规范缺失：`DATA_MODEL_PERMISSION.md:143-145` 的 `ScopeQueryBuilder` 只是 helper，未规定"所有业务 QuerySet 必须经过 Scope 包装"的强制约束，开发期容易漏权限校验 | 安全/Data | 🆕 新增 |
|
||||
| 8 | 🟠 | **M-05** | 89k 数据 < 2 秒列表查询 NFR 仍无 p95/EXPLAIN/性能基准测试任务（昨日 M-05 未修） | NFR↔TECH↔测试 | ❌ 持续未修 |
|
||||
| 9 | 🟠 | **M-09** | UI_SYSTEM 复杂组件（虚拟滚动列表、批量操作面板、抽屉表单嵌套规则、文件上传批量、富权限树）规范深度不足；UI_DESIGN 11 份原型仅覆盖客源 + 房源列表，**楼盘/权限/系统配置/组织人事/发布管理 5 大模块全部缺原型** | UI | ⚠️ 部分修复 |
|
||||
| 10 | 🟡 | **N-01** | ENUMS.md v2.2 已统一，但 PRD 文本中仍混用中文枚举（如客源 PRD 仍写"求购/求租"），需要一次全文档"中文枚举 → ENUMS 锚点链接"替换 | PRD↔Data | 🆕 新增 |
|
||||
|
||||
### 风险等级分布
|
||||
|
||||
| 等级 | 本次（2026-04-28） | 上次（2026-04-26） | 净变化 |
|
||||
|------|---|---|---|
|
||||
| 🔴 Blocker | **2** | 4 | -2（B-01/B-02/B-03/B-04 全部清零；B-05/B-06 由历史 Major 升级） |
|
||||
| 🟠 Major | **9** | 12 | -3（M-01 已修复关闭；M-02→B-05、M-03→B-06 升级） |
|
||||
| 🟡 Minor | **3** | 6 | -3（聚焦 Top 风险） |
|
||||
| 合计 | **14** | 22 | -8 |
|
||||
|
||||
### 增量对比一览（昨日 Blocker / Major → 今日状态）
|
||||
|
||||
| 上次编号 | 简述 | 状态 | 今日处置 |
|
||||
|----------|------|------|----------|
|
||||
| **B-01** | 系统配置 PRD 空骨架 | ✅ **已修复**（273+1100+275 行三件套） | 关闭 |
|
||||
| **B-02** | 枚举三方不一致 | ✅ **已修复**（ENUMS.md v2.2 权威源） | 关闭，遗留 N-01（PRD 文本回写） |
|
||||
| **B-03** | 权限三档/五档冲突 | ✅ **已修复**（五档基线 + 子集子句） | 关闭，遗留 M-14（ORM 强制约束） |
|
||||
| **B-04** | Keyset 分页缺失 | ✅ **已修复**（API_CONTRACT §4 + 4 模块） | 关闭 |
|
||||
| M-01 | 测试规范多租户化 | ✅ 已修复（TenantClient 强制） | 关闭 |
|
||||
| M-02 | 主表乐观锁 / 楼盘锁 | ❌ 持续未修 | **升 B-05** |
|
||||
| M-03 | 高写入表分区 | ❌ 持续未修 | **升 B-06** |
|
||||
| M-04 | Celery/R2/索引规范 | ⚠️ 部分修复（拆分） | 拆分为 M-12 / M-13 / 残留索引矩阵 |
|
||||
| M-05 | 89k 数据 < 2s NFR 测试 | ❌ 持续未修 | M-05 |
|
||||
| M-06 | 客户端发布签名 | ❌ 持续未修 | M-06 |
|
||||
| M-07~M-08 | 价格走势 / 楼盘市场报盘 | ⚠️ 降级 | 转 N-02（Minor） |
|
||||
| M-09 | UI 组件 + Wireframe | ⚠️ 部分修复 | M-09 |
|
||||
| M-10 | Redis Key tenant 前缀 | ✅ 已修复（系统设置技术方案 §6.4） | 关闭 |
|
||||
| M-11 | 加密密钥管理 | ❌ 持续未修 | M-11 |
|
||||
| M-12 | 数据保留与归档 | ⚠️ 部分修复 | 转 N-03 |
|
||||
|
||||
---
|
||||
|
||||
## 一、PRD 一致性审查（PRD ↔ PRD ↔ TASK）
|
||||
|
||||
### ✅ 历史 B-01（系统配置 PRD 空骨架）已修复
|
||||
|
||||
**证据**：
|
||||
- `PRD/系统配置/系统配置模块PRD.md`（273 行，v0.1，2026-04-27）：含目标度量、非目标范围、3 条 P0 User Story（US-SETTING-001-A/B/C）与完整 Given-When-Then 验收标准。
|
||||
- `PRD/系统配置/系统配置参数数据.md`（1100 行）：实际参数清单。
|
||||
- `PRD/系统配置/系统配置数据模型设计说明_for_Atlas.md`（275 行）：与 DATA_MODEL_SETTING.md 对齐。
|
||||
|
||||
**剩余风险（降级为 Minor N-04）**：US-SETTING-001 的"配置变更生效时效 ≤ 5 分钟（Redis 缓存 TTL）"与 `DATA_MODEL_SETTING.md` 的 Redis Key TTL = 300s 一致，但**无 Cache Invalidation 主动失效路径**（仅靠 TTL 过期），管理员保存后 5 分钟内一线经纪人仍看不到新选项 —— PRD AC-2 "刷新后立即可见" 与实现存在间隙。
|
||||
|
||||
---
|
||||
|
||||
### ✅ 历史 B-02（核心枚举三方不一致）已修复
|
||||
|
||||
**证据**：
|
||||
- `DATA_MODEL/ENUMS.md` v2.2（780 行，2026-04-28）：作为唯一权威源，覆盖 Public（tenant 生命周期、平台管理员、备份导出、升级发布）+ Tenant（客源/房源/楼盘/权限/组织/系统设置）所有枚举域。
|
||||
- 命名规范统一为 `lower_snake_case`，历史大写值（`SUCCESS`/`FAILED`/`A_urgent`/`A_app`/`B_schema`/`C_feature`/`REPLACE`/`RESTRICT`/`GRANT`）的迁移路径已书面化。
|
||||
|
||||
**剩余风险（N-01 Minor）**：PRD 文本与 TASK AC 中仍存在大量中文/旧值混用（如客源 PRD 仍写"求购/求租"）。需要一次性回写：
|
||||
- 在每份 PRD 顶部加 `> 枚举值参见 DATA_MODEL/ENUMS.md` 锚点
|
||||
- 在 ENUMS.md 中给每个枚举值附"中文展示标签"列（部分已有）
|
||||
- TASK.md 排一条 P0 任务：PRD 中文枚举 → 英文锚点引用
|
||||
|
||||
---
|
||||
|
||||
### ✅ 历史 B-03（权限档位三档/五档冲突）已修复
|
||||
|
||||
**证据**：
|
||||
- `PRD/权限管理/权限管理模块PRD.md:46`：明文"采用**五档模型**：本人 / 本组 / 本门店 / 本区域 / 全公司。注意：并非所有权限项均开放五档，各项的实际可选范围以权限编辑页的下拉配置为准（例如某些权限项仅提供「本人 / 本门店 / 全公司」三个选项）"
|
||||
- `:499`：复述同一约定，闭环。
|
||||
- `DATA_MODEL_PERMISSION.md:24,31,254-275,967-987`：`staff_data_scopes` 表 DDL 完整，索引 3 条到位，"员工数据范围 = 所有 staff_data_scopes 对应 org_units.path 子树并集"实现"跨层级叠加"。
|
||||
|
||||
**剩余风险（M-14 Major）见 §五**：DATA_MODEL_PERMISSION 的 `ScopeQueryBuilder` 只是 helper class，未在 TECH_STACK 中强制规定"所有业务 QuerySet 必须经过 Scope 包装"，开发期容易漏写导致越权读取。
|
||||
|
||||
---
|
||||
|
||||
### 🟡 N-01 PRD 文本回写 ENUMS 锚点（来自 B-02 残留）
|
||||
|
||||
详见上文。**责任**：PM 在 Phase 1 编码启动前完成全量 PRD 文本扫描替换。
|
||||
|
||||
---
|
||||
|
||||
## 二、TECH_STACK / API / 测试规范审查
|
||||
|
||||
### ✅ 历史 B-04（Keyset 分页缺失）已修复
|
||||
|
||||
**证据**：
|
||||
- `API_CONTRACT.md:143,154,157,182`：标准请求/响应契约 `{"mode":"keyset","cursor":"opaque_cursor","limit":20}` + `next_cursor`
|
||||
- `API_CONTRACT.md:165`：`cursor` MUST 为不透明字符串，**禁止暴露内部排序字段组合** ✅ 与 OWASP 推荐一致
|
||||
- `API_CONTRACT.md:378`：drf-spectacular `OpenApiParameter("cursor", ...)` 注解
|
||||
- 客源 / 房源 / 楼盘 / 组织 4 个模块技术方案均同步采用
|
||||
|
||||
**剩余 Minor**：尚无 Keyset SQL 模板（含 `WHERE (sort_key, id) < (?, ?)` 的复合键模板）。建议在 `TECH_STACK.md` 新增 §"分页 SQL 模板"章节，给出含 `tenant_schema` schema-search-path 切换 + 复合键的标准实现。
|
||||
|
||||
---
|
||||
|
||||
### ✅ 历史 M-01（测试规范多租户化）已修复
|
||||
|
||||
**证据**：
|
||||
- `TECH_STACK.md:203,208,211`：明确 TenantClient 强制 + R2/Redis/邮件 Mock
|
||||
- `测试规范.md:55,69,146,230`：`TenantClient` 在测试金字塔图、约束章节、请求路径、Mock 策略 4 处统一表述
|
||||
- 4 个模块技术方案均引用
|
||||
|
||||
---
|
||||
|
||||
### 🟠 M-12 Celery 多租户 schema 切换无统一封装（M-04 拆分）
|
||||
|
||||
**文档**：`TECH_STACK/客源管理技术方案.md` / `房源管理技术方案.md` / `楼盘管理技术方案.md` / `组织人事技术方案.md`（均声明 `tenant_schema_name` 入参）vs `TECH_STACK.md`（无统一规范）
|
||||
|
||||
**事实**：
|
||||
- 模块技术方案约定 Celery 任务签名带 `tenant_schema_name: str` 形参，但**无 `with_tenant_context(schema)` 装饰器或基类**。
|
||||
- 系统设置技术方案虽给出 schema 切换示例（`:28-31`），但未提取为通用规范。
|
||||
- `core/cache.py` 已规范带前缀，但 ORM 查询的 schema 切换由开发者手写 `connection.set_schema(...)`，**漏写不报错**。
|
||||
|
||||
**影响**：
|
||||
- Celery worker 在多租户调度下，相邻任务可能因 search_path 残留导致跨租户脏读/脏写。
|
||||
- 单元测试若忘记设置 schema，测试通过但生产出错。
|
||||
|
||||
**改进方向**：
|
||||
- 在 `TECH_STACK.md` 新增 §"Celery 多租户规范"：
|
||||
- 强制所有业务 task 用 `@tenant_task(schema_arg="tenant_schema_name")` 装饰器
|
||||
- 装饰器职责：进入时 `set_schema_to(schema)`，退出/异常时 `set_schema_to_public()`，记录 task_id 与 schema 到结构化日志
|
||||
- 测试规范补："Celery 任务测试必须显式 mock schema 切换断言"
|
||||
|
||||
---
|
||||
|
||||
### 🟠 M-13 R2 路径前缀全局规范不一致（M-04 拆分）
|
||||
|
||||
**文档**：`TECH_STACK/系统管理技术文档.md:521-525`（已规范）vs 客源/房源/楼盘技术方案（未规范）
|
||||
|
||||
**事实**：
|
||||
- 系统管理已统一：`backups/{tenant_schema}/{record_id}.tar.gz` / `exports/{tenant_schema}/{task_id}.zip` / `releases/system/{version}/...`（无 tenant 前缀，公共资源）✅
|
||||
- 房源 / 客源模块的图片 / 跟进附件 R2 路径**未在文档中给出 key 模板**，仅写"预签名上传"。
|
||||
- 公开发布包路径无 tenant 前缀（合理：客户端尚未登录），但**生产环境 bucket policy 需明示**：哪些 prefix 是 public-read，哪些需要 signed URL。
|
||||
|
||||
**影响**：
|
||||
- 跨租户文件越权访问风险（房源图片如果误用全局 prefix，A 租户能猜到 B 租户对象 key）。
|
||||
- 生命周期策略（如客源跟进附件 90 天清理）无 prefix 难以配置。
|
||||
|
||||
**改进方向**：
|
||||
- 在 `TECH_STACK.md` 新增"R2 路径规范"统一表：
|
||||
```
|
||||
公共资源 → releases/system/{version}/... public-read
|
||||
租户 backup → backups/{tenant_schema}/{rid}.tar.gz signed only
|
||||
租户 export → exports/{tenant_schema}/{tid}.zip signed-24h
|
||||
房源图片 → media/{tenant_schema}/property/{pid}/{photo_id}.{ext}
|
||||
跟进附件 → media/{tenant_schema}/follow/{log_id}/{idx}.{ext}
|
||||
审计归档 → exports/audit/{task_id}.csv signed only
|
||||
```
|
||||
- 路径中**禁止出现 tenant_id（UUID）**，统一用 `tenant_schema_name`，便于跨环境迁移。
|
||||
|
||||
---
|
||||
|
||||
### 🟠 M-05 89k 数据 < 2 秒列表查询 NFR 仍无性能基准（持续未修）
|
||||
|
||||
**文档**：`PRD/客源管理/客源管理模块PRD.md` US-CLIENT-002 / `PRD/房源管理/房源管理模块PRD.md` US-PROPERTY-002 NFR：列表查询 p95 < 2s（89k 数据）
|
||||
|
||||
**事实**：
|
||||
- 测试规范未含 p95 / EXPLAIN ANALYZE / pg_stat_statements 集成。
|
||||
- 无 89k 量级 seed factory（仅有单条 fixture）。
|
||||
- 无 CI 性能回归任务（如 pgTAP / pytest-benchmark）。
|
||||
- AC 中"< 2 秒"实质不可测。
|
||||
|
||||
**改进方向**：
|
||||
- 在 `测试规范.md` 增补 §"性能测试"：
|
||||
- seed: `make seed-perf` 灌入 89k 房源 + 200k 客源 + 200 万 follow_logs
|
||||
- 工具: `pytest-benchmark` + `EXPLAIN (ANALYZE, BUFFERS)` 输出归档
|
||||
- 阈值: 列表 p95 < 2s / 详情 p95 < 500ms / 写入 p95 < 1s
|
||||
- CI: nightly 跑一次，回归 > 20% 标 fail
|
||||
- 优先 instrument 客源/房源列表 API（含 keyset 翻第 100 页场景）
|
||||
|
||||
---
|
||||
|
||||
## 三、DATA_MODEL 审查
|
||||
|
||||
### 🔴 B-05 主表乐观锁 `version` 字段全量未实现（M-02 升级，连续 3 次未修）
|
||||
|
||||
**文档**：`DATA_MODEL/DATA_MODEL_PROPERTY.md` / `DATA_MODEL_CLIENT.md` / `DATA_MODEL_COMPLEX.md`
|
||||
|
||||
**事实**：grep `version|乐观锁|optimistic` 在 properties / clients / complexes 主表 DDL 中 **0 命中**（`version` 仅出现在 ENUMS.md 文档版本号、`SystemVersion` 表名、Permission 缓存 key `:perm:version` 等不相关上下文中）。
|
||||
|
||||
**业务场景**（PRD 多处）：
|
||||
- 房源 PRD：双经纪人同时编辑同一房源（一人改价、一人改状态）
|
||||
- 客源 PRD：录入员 + 跟进经纪人同时编辑客源等级
|
||||
- 楼盘 PRD：4 类锁字段（lock_building/lock_room/lock_info/lock_standard_room）已建，但**无 `lock_version` 触发 update conflict**
|
||||
|
||||
**影响**：后写覆盖前写、数据丢失，**89k 房源体量下必然出现**。
|
||||
|
||||
**改进方向**：
|
||||
- DDL 增加：
|
||||
```sql
|
||||
ALTER TABLE properties ADD COLUMN version INTEGER NOT NULL DEFAULT 1;
|
||||
ALTER TABLE clients ADD COLUMN version INTEGER NOT NULL DEFAULT 1;
|
||||
ALTER TABLE complexes ADD COLUMN version INTEGER NOT NULL DEFAULT 1;
|
||||
```
|
||||
- 应用层：`Model.objects.filter(id=x, version=v_old).update(..., version=F('version')+1)`，受影响行 0 → 抛 `ConflictError`，前端展示"已被他人修改，请刷新后重试"。
|
||||
- API_CONTRACT.md 增补 `If-Match: <version>` Header 规范（PUT/PATCH 必传）。
|
||||
|
||||
---
|
||||
|
||||
### 🔴 B-06 高写入表分区 DDL 仍未落地（M-03 升级，连续 3 次未修）
|
||||
|
||||
**文档**：`DATA_MODEL/DATA_MODEL_CLIENT.md`（follow_logs）/ `DATA_MODEL_PROPERTY.md`（property_photos）/ `DATA_MODEL_PERMISSION.md`（permission_change_logs）/ `DATA_MODEL_LOGIN.md`（login_attempts）/ `DATA_MODEL_PUBLIC.md`（platform_audit_logs）
|
||||
|
||||
**事实**：grep `PARTITION BY|分区表|partition` 在 DATA_MODEL 全量 0 命中。仅文字"建议月度分区"。
|
||||
|
||||
**业务体量**（PRD 与 NFR）：
|
||||
- `follow_logs`：200 万 / 2 年保留期
|
||||
- `property_photos`：89k 房源 × 平均 8 张 ≈ 70 万行
|
||||
- `login_attempts`：所有登录失败/成功审计
|
||||
- `platform_audit_logs`：跨租户全平台审计
|
||||
|
||||
**影响**：单表过大导致索引膨胀、VACUUM 阻塞、按时间窗口归档不可分区操作。**编码后再切分区将停机迁移**，是必须前置的设计决策。
|
||||
|
||||
**改进方向**：
|
||||
- DDL 改为：
|
||||
```sql
|
||||
CREATE TABLE follow_logs (
|
||||
id BIGSERIAL,
|
||||
created_at TIMESTAMPTZ NOT NULL,
|
||||
...
|
||||
PRIMARY KEY (id, created_at) -- 分区键必须含 created_at
|
||||
) PARTITION BY RANGE (created_at);
|
||||
|
||||
CREATE TABLE follow_logs_2026_04 PARTITION OF follow_logs
|
||||
FOR VALUES FROM ('2026-04-01') TO ('2026-05-01');
|
||||
```
|
||||
- 增补 Celery 周期任务 `partition_maintenance_task`（每月最后一天）：自动建下月分区 + drop 超出保留期分区。
|
||||
- DATA_MODEL.md 增 §"分区策略"统一表（保留期 / 分区粒度 / 维护任务）。
|
||||
|
||||
---
|
||||
|
||||
### 🟠 M-14 ORM 数据范围统一封装规范缺失（B-03 残留）
|
||||
|
||||
**文档**：`DATA_MODEL/DATA_MODEL_PERMISSION.md:143-145, 414-455`
|
||||
|
||||
**事实**：
|
||||
- `ScopeQueryBuilder` 已实现（含 `is_protected` 二次 AND 规则）。
|
||||
- 但**未在 TECH_STACK 中强制要求"所有业务 QuerySet 必须经过 ScopeQueryBuilder"**。
|
||||
- 模块技术方案 view 层示例代码直接 `Property.objects.filter(...)`，可绕过权限控制。
|
||||
|
||||
**影响**：开发新功能时漏接 ScopeQueryBuilder → 越权读取，且测试不易发现。
|
||||
|
||||
**改进方向**：
|
||||
- TECH_STACK 增补"ORM 数据范围强制规范"：
|
||||
- 业务 Model 不暴露默认 `objects` Manager，统一暴露 `scoped(staff)` 入口
|
||||
- 写一个 lint rule（pylint plugin / pre-commit）：禁止业务模块直接调用 `<Model>.objects.<query>`
|
||||
- 测试规范补"权限边界测试矩阵"：每个 Model 至少 3 case（own / department / cross_department denied）
|
||||
|
||||
---
|
||||
|
||||
### 🟡 N-02 楼盘价格走势 / 市场报盘（M-07/M-08 降级）
|
||||
|
||||
历史 M-07/M-08 已写明"P2 不做"，本期降为 Minor。
|
||||
|
||||
---
|
||||
|
||||
### 🟡 N-03 数据保留与归档策略（M-12 降级）
|
||||
|
||||
`platform_audit_logs` / `follow_logs` 在 PRD 中提到"2 年保留"，但 DATA_MODEL 无对应 Celery 归档任务定义。建议在 B-06 分区落地后顺带补全 archive job。
|
||||
|
||||
---
|
||||
|
||||
## 四、安全审查（含多租户 / 权限 / 审计）
|
||||
|
||||
### 🟠 M-06 客户端发布无签名校验 / 防降级（持续未修）
|
||||
|
||||
**文档**：`PRD/发布管理/客户端发布管理模块PRD.md:165,198,284,347` / `TECH_STACK.md:119`
|
||||
|
||||
**事实**：
|
||||
- 端点 `GET /api/client/updates/latest/` 公开（合理）。
|
||||
- `download_url` 走 R2 / CDN 公开（合理）。
|
||||
- 完整性校验仅 SHA256（仅防偶然篡改，不防 MITM 主动替换）。
|
||||
- **无代码签名（Authenticode / OS-level codesign）**、**无服务端响应签名**、**无版本号单调递增校验（防降级）**。
|
||||
|
||||
**改进方向**：
|
||||
- 升级响应 JSON 增加 `signature` 字段（HMAC-SHA256，密钥服务端持有），客户端校验签名后再 fetch download_url。
|
||||
- `electron-updater` 配合 `publisherName`（Windows Authenticode），在 CI 用代码签名证书签 EXE。
|
||||
- 客户端本地存"已安装版本 N"，若服务端返回版本 < N，**拒绝降级**（除非配置允许 force_downgrade=true 由后端控制）。
|
||||
|
||||
---
|
||||
|
||||
### 🟠 M-11 加密密钥管理 / 轮换 SOP 仍未补（持续未修）
|
||||
|
||||
**文档**：`TECH_STACK/系统管理技术文档.md:877`（仅"密钥通过 Docker Secret 注入"一句）
|
||||
|
||||
**事实**：
|
||||
- `core/encryption.py` 中央加密 helper 已声明（散见模块技术方案）。
|
||||
- **缺**：主密钥（KEK）轮换流程、字段加密密钥（DEK）版本号设计、密钥泄露应急吊销 + 全量字段重新封装、生产-staging 密钥隔离。
|
||||
|
||||
**改进方向**：
|
||||
- 在 `TECH_STACK.md` 新增 §"密钥管理 SOP"：
|
||||
- 三层结构：环境主密钥（KMS / sops + age） → 应用 KEK（启动时加载） → 字段 DEK（per-row 或 per-column）
|
||||
- DDL 加密字段必须含 `key_version SMALLINT NOT NULL`
|
||||
- 季度轮换 + 异常吊销 runbook
|
||||
- 短期（无 KMS）方案：`age` + `sops` 加密 secrets 文件，运维流程文档化。
|
||||
|
||||
---
|
||||
|
||||
### 🟠 M-14 ORM 数据范围（详见 §三）
|
||||
|
||||
---
|
||||
|
||||
## 五、UI/UX 审查
|
||||
|
||||
### 🟠 M-09 UI 设计覆盖率与复杂组件深度（部分修复）
|
||||
|
||||
**文档**：`UI_DESIGN/`（11 份 markdown + 11 份 HTML 原型 + `UI_设计任务总表.md`）/ `UI_SYSTEM/`（3 份）
|
||||
|
||||
**事实**：
|
||||
- ✅ 已覆盖：客源列表 / 客源详情 / 客源新增 / 客源编辑 / 房源列表 5 类原型（HTML + Markdown 双版本）。
|
||||
- ❌ 仍缺：
|
||||
- 楼盘管理（详情 / 新增 / 锁字段交互）
|
||||
- 权限管理（权限树 / 数据范围编辑器 / 五档下拉子集）
|
||||
- 系统配置（lookup_items 编辑抽屉 / 必填规则配置）
|
||||
- 组织人事（org tree / 员工卡）
|
||||
- 发布管理（升级表单 / 灰度名单 / 进度面板）
|
||||
- ❌ UI_SYSTEM 复杂组件规范深度仍不足：虚拟滚动列表（89k 房源必需）、批量操作面板、抽屉表单嵌套规则、文件上传批量、富权限树。
|
||||
|
||||
**改进方向**：
|
||||
- 把 UI_DESIGN 11 份 task 按 PRD 优先级排进 Phase 1：客源（已齐）→ 房源详情 → 楼盘 → 权限 → 系统配置；
|
||||
- UI_SYSTEM 增补"复杂组件清单"5 项，每项给出 Alpine.js + HTMX 实现范本；
|
||||
- 89k 房源列表必须用虚拟滚动或固定窗口（如 PageDown 分页），HTMX 按 keyset cursor 局部刷新。
|
||||
|
||||
---
|
||||
|
||||
## 六、TASK ↔ 文档交叉验证
|
||||
|
||||
### 🟡 N-04 系统配置 Cache Invalidation（B-01 残留）
|
||||
|
||||
详见 §一。
|
||||
|
||||
---
|
||||
|
||||
## 七、汇总 Action Items
|
||||
|
||||
| # | 等级 | 编号 | Action | 责任 | 建议完成时间 | 编码门禁 |
|
||||
|---|------|------|--------|------|--------------|----------|
|
||||
| 1 | 🔴 | B-05 | 主表 `version` 字段 + `If-Match` API 规范 | 架构师 + 后端 Lead | T+3 天 | **Phase 1 启动前** |
|
||||
| 2 | 🔴 | B-06 | 5 张高写入表 `PARTITION BY RANGE` DDL + 分区维护 Celery 任务 | 架构师 + DBA | T+5 天 | **Phase 1 启动前** |
|
||||
| 3 | 🟠 | M-05 | 性能基准测试方案（seed 89k + pytest-benchmark + p95 阈值） | 测试 Lead | T+7 天 | Phase 1 内 |
|
||||
| 4 | 🟠 | M-06 | 客户端发布签名 + 防降级（HMAC + Authenticode） | 安全 + 发布运维 | T+10 天 | 客户端发布 GA 前 |
|
||||
| 5 | 🟠 | M-09 | 楼盘 / 权限 / 系统配置 / 组织人事 / 发布 5 模块原型 + 5 复杂组件规范 | UX + 前端 Lead | T+14 天 | 各模块编码前 |
|
||||
| 6 | 🟠 | M-11 | 密钥管理 SOP（KEK/DEK/版本/轮换/吊销） | 安全 + 运维 | T+10 天 | 加密字段上线前 |
|
||||
| 7 | 🟠 | M-12 | `@tenant_task` 装饰器 + Celery 多租户规范文档 | 架构师 | T+5 天 | Celery 任务编码前 |
|
||||
| 8 | 🟠 | M-13 | R2 路径前缀统一规范表 + bucket policy | 架构师 + 运维 | T+5 天 | R2 上传任务编码前 |
|
||||
| 9 | 🟠 | M-14 | ORM 数据范围强制 lint + `scoped(staff)` 入口 | 架构师 + 后端 Lead | T+7 天 | Phase 1 内 |
|
||||
| 10 | 🟡 | N-01 | PRD 文本中文枚举 → ENUMS 锚点全量替换 | PM | T+3 天 | 编码前清零 |
|
||||
| 11 | 🟡 | N-04 | 系统配置 Cache Invalidation 主动失效路径补充 | 后端 Lead | T+5 天 | US-SETTING-001 编码前 |
|
||||
| 12 | 🟡 | N-02/N-03 | 楼盘走势/数据归档策略 | PM + DBA | Phase 2 | 非阻塞 |
|
||||
|
||||
---
|
||||
|
||||
## 八、Review 结论
|
||||
|
||||
### 编码门禁（Phase 1 启动前必清零）
|
||||
|
||||
**🔴 Blocker（2 项必修）**：
|
||||
1. **B-05** 主表 `version` 乐观锁字段 —— 涉及 properties / clients / complexes DDL + API_CONTRACT 增补 `If-Match` Header
|
||||
2. **B-06** 5 张高写入表分区 DDL + 分区维护任务 —— 后期切分需停机，必须前置
|
||||
|
||||
### 整体判断
|
||||
|
||||
- **本期是历次 Review 进展最大的一次**：4 个 Blocker 全部清零，1 个 Major（M-01）已修复，文档体系（特别是 PRD ↔ ENUMS ↔ DATA_MODEL）建立了真正的"权威源"。
|
||||
- **剩余 Blocker 集中在 DATA_MODEL 物理设计层面**（并发控制 + 分区），属于"知道但没动"的债务，**修复成本可估、风险可控**，建议本周内打磨完毕。
|
||||
- **Phase 1 编码可在 B-05 / B-06 修复后立即启动**，其余 9 个 Major 可与编码并行推进，但需在对应模块进入开发前清零（参见上表"编码门禁"列）。
|
||||
- **下一次 Review 焦点**：B-05 / B-06 修复后，应重点审查 M-09（UI 覆盖率）+ M-05（性能基准），以及编码启动后第一波集成测试反馈。
|
||||
|
||||
### 文档健康度趋势
|
||||
|
||||
| 维度 | 04-25 | 04-26 | 04-28 | 趋势 |
|
||||
|------|-------|-------|-------|------|
|
||||
| PRD 完整性 | 60% | 75% | **90%** | ↑↑ |
|
||||
| DATA_MODEL 严谨性 | 50% | 65% | **80%** | ↑↑ |
|
||||
| TECH_STACK 横切规范 | 30% | 50% | **70%** | ↑ |
|
||||
| 测试规范多租户化 | 20% | 40% | **85%** | ↑↑↑ |
|
||||
| UI/UX 覆盖率 | 5% | 30% | **35%** | ↑（最弱环节） |
|
||||
| 安全设计 | 40% | 45% | **55%** | ↑（M-06/M-11 持续债务） |
|
||||
|
||||
---
|
||||
|
||||
> **Reviewer 签字**：首席系统设计 Reviewer（AI 辅助）
|
||||
> **下次 Review 建议触发条件**：B-05 + B-06 完成 PR 合并后，或 Phase 1 编码启动后第 1 个 Sprint 结束。
|
||||
448
Project/fonrey/REVIEW/REVIEW_登录管理_平台管理后台_2026-05-02.md
Normal file
448
Project/fonrey/REVIEW/REVIEW_登录管理_平台管理后台_2026-05-02.md
Normal file
@@ -0,0 +1,448 @@
|
||||
# Fonrey 全量 Review — 登录管理 + 平台管理后台
|
||||
|
||||
- **Review 范围**：登录管理模块 + 平台管理后台模块
|
||||
- **Review 日期**：2026-05-02
|
||||
- **Review 维度**：8 维度（PRD质量 / PRD↔TECH一致性 / DATA_MODEL / API / UI_SYSTEM / 安全多租户 / 性能扩展 / 遗漏场景）
|
||||
- **审查文档**：
|
||||
- PRD：`PRD_MVP.md`、`登录管理/用户登录管理模块PRD.md` (v3.0)、`平台管理后台/平台管理后台PRD.md`
|
||||
- TECH：`TECH_STACK.md` (v2.5)、`登录管理技术方案.md` (v4.1)、`平台管理后台技术方案.md`、`API_CONTRACT.md`、`测试规范.md`
|
||||
- DATA_MODEL：`DATA_MODEL.md`、`DATA_MODEL_LOGIN.md`、`DATA_MODEL_PUBLIC.md`、`ENUMS.md` (v2.2)
|
||||
- UI：`UI_SYSTEM.md`、`组件清单.md`、`组件规范设计.md`、`UI_DESIGN/登录管理/登录_UI.md`、`UI_DESIGN/平台管理后台/平台管理后台_UI.md` 及 4 份 HTML 原型
|
||||
- **严重程度图例**：🔴 Blocker（影响交付，需立即处理）/ 🟠 Major（影响质量，需在迭代内处理）/ 🟡 Minor（建议优化）
|
||||
|
||||
---
|
||||
|
||||
## 维度一：PRD 质量
|
||||
|
||||
### 🔴 P-B-1 平台管理后台 PRD 缺失 OpenAPI 与功能编号体系
|
||||
- **位置**：[平台管理后台PRD.md](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0PRD.md) 全文 835 行
|
||||
- **问题**：用户故事未编号（无 PA-001/PA-002 等可追溯标识），与 [用户登录管理模块PRD.md](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E7%99%BB%E5%BD%95%E7%AE%A1%E7%90%86/%E7%94%A8%E6%88%B7%E7%99%BB%E5%BD%95%E7%AE%A1%E7%90%86%E6%A8%A1%E5%9D%97PRD.md) v3.0 已剥离实现细节后的"功能 ID + 验收标准"风格不一致。导致需求→测试用例→Bug 单的可追溯链条断裂。
|
||||
- **改进方向**：按登录 PRD v3.0 风格补齐编号 (PA-001 ~ PA-NN)，每条用户故事附"验收标准 (AC)"小节。
|
||||
- **责任角色**：PM / Product Owner
|
||||
|
||||
### 🟠 P-B-2 平台管理后台 PRD 仍混入实现/字段细节
|
||||
- **位置**：[平台管理后台PRD.md L210-L350](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0PRD.md#L210-L350) 段落
|
||||
- **问题**：租户创建/升级/暂停章节出现 `tenant_code`、`schema_name`、`status` CHECK 值等数据层概念；与 ADR-20260502-003 "PRD 剥离实现细节"原则违背。
|
||||
- **改进方向**：按登录 PRD v3.0 模式将字段/状态描述迁出至 DATA_MODEL_PUBLIC.md，PRD 仅保留业务语义（"租户进入暂停态后所有用户登录被拒绝"）。
|
||||
- **责任角色**：PM + Backend Architect
|
||||
|
||||
### 🟠 P-B-3 平台管理员（PlatformAdmin）"权限矩阵"未量化
|
||||
- **位置**：[平台管理后台PRD.md L40-L60, L520-L560](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0PRD.md#L40-L60) 与 [DATA_MODEL_PUBLIC.md L189-L250](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/DATA_MODEL_PUBLIC.md#L189-L250) `platform_admins` 表
|
||||
- **问题**：PRD 提到 `super_admin` / `ops` / `support` 三种角色，但未列"角色 × 操作 × 资源"矩阵。技术方案与 DATA_MODEL 也未补齐。
|
||||
- **改进方向**：在 PRD 增 §权限矩阵小节，列出每条平台操作（创建租户/暂停租户/查看审计日志/发布客户端版本/等）所需角色，并与 `platform_admins.role` 枚举对应。
|
||||
- **责任角色**：PM + 安全负责人
|
||||
|
||||
### 🟡 P-L-1 登录 PRD 验收标准粒度偏粗
|
||||
- **位置**：[用户登录管理模块PRD.md](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E7%99%BB%E5%BD%95%E7%AE%A1%E7%90%86/%E7%94%A8%E6%88%B7%E7%99%BB%E5%BD%95%E7%AE%A1%E7%90%86%E6%A8%A1%E5%9D%97PRD.md) UC-LOGIN-* 段
|
||||
- **问题**：部分验收标准只描述"成功"路径，未列出所有失败路径覆盖的提示文案（与 [组件规范设计.md](file:///mnt/d/Workspace/nexus/Project/fonrey/UI_SYSTEM/%E7%BB%84%E4%BB%B6%E8%A7%84%E8%8C%83%E8%AE%BE%E8%AE%A1.md) Toast/Form Error 规范脱节）。
|
||||
- **改进方向**：每条 UC 补充"异常路径 ×N → 文案 / 状态码"。
|
||||
- **责任角色**：PM
|
||||
|
||||
---
|
||||
|
||||
## 维度二：PRD ↔ TECH 一致性
|
||||
|
||||
### 🔴 PT-B-1 创建租户事务边界与回滚补偿未在技术方案落地
|
||||
- **位置**：[平台管理后台PRD.md L122-L170](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0PRD.md#L122-L170) vs [平台管理后台技术方案.md](file:///mnt/d/Workspace/nexus/Project/fonrey/TECH_STACK/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0%E6%8A%80%E6%9C%AF%E6%96%B9%E6%A1%88.md)
|
||||
- **问题**：PRD 描述"创建租户 → 建 schema → 跑 migration → 创建 Tenant Admin → 发欢迎邮件"为一体化流程，但技术方案未说明：django-tenants `create_tenant` 与 Django 默认事务无法包裹 `CREATE SCHEMA`；migration 中途失败、欢迎邮件发送失败的补偿/回滚策略缺失。状态机仅定义 `creating`/`failed`，但谁置位 `failed`、是否清理半成品 schema 未规定。
|
||||
- **改进方向**：技术方案补"租户创建 Saga"章节：步骤 1-N、每步失败的补偿动作、`failed` 后的人工/定时清理任务。同步在 PRD 注明"创建过程为异步，最终一致"。
|
||||
- **责任角色**：Backend Architect + Platform Ops
|
||||
|
||||
### 🟠 PT-B-2 客户端发版灰度策略 PRD/TECH 描述不一致
|
||||
- **位置**：[平台管理后台PRD.md L476-L520](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0PRD.md#L476-L520) vs [平台管理后台技术方案.md](file:///mnt/d/Workspace/nexus/Project/fonrey/TECH_STACK/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0%E6%8A%80%E6%9C%AF%E6%96%B9%E6%A1%88.md)
|
||||
- **问题**：PRD 描述"按租户/百分比灰度"，但 [DATA_MODEL_PUBLIC.md client_releases L488-L505](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/DATA_MODEL_PUBLIC.md#L488-L505) 仅有 `channel`、`rollout_percent`，无"租户白名单"字段，技术方案对此未澄清。
|
||||
- **改进方向**：技术方案明确灰度策略对象（租户级 OR 百分比级或两者并存），DATA_MODEL 同步增 `rollout_tenant_codes JSONB` 或 `client_release_tenants` 关联表。
|
||||
- **责任角色**：Backend Architect + DBA
|
||||
|
||||
### 🟠 PT-B-3 平台管理员 MFA 在 PRD 强约束但 TECH 未规定算法/实现
|
||||
- **位置**：[平台管理后台PRD.md L60](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0PRD.md#L60)（"super_admin 必须 MFA"） vs [平台管理后台技术方案.md](file:///mnt/d/Workspace/nexus/Project/fonrey/TECH_STACK/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0%E6%8A%80%E6%9C%AF%E6%96%B9%E6%A1%88.md)、[DATA_MODEL_PUBLIC.md platform_admins L189-L250](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/DATA_MODEL_PUBLIC.md#L189-L250)
|
||||
- **问题**：未定义 MFA 类型（TOTP / SMS / WebAuthn）、密钥存储位置、登录流程接入点、备份码机制。`platform_admins` 表无 `mfa_secret`/`mfa_enabled` 字段。
|
||||
- **改进方向**：技术方案选定 TOTP（pyotp）方案；DATA_MODEL 增 `mfa_enabled BOOL`、`mfa_secret_enc TEXT`、`mfa_recovery_codes_enc TEXT`；PRD 注明备份码生成/重置规则。
|
||||
- **责任角色**：安全负责人 + Backend Architect
|
||||
|
||||
### 🟠 PT-L-1 登录 PRD "首次登录强制改密" 与 TECH 实现路径未对齐
|
||||
- **位置**：[用户登录管理模块PRD.md](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E7%99%BB%E5%BD%95%E7%AE%A1%E7%90%86/%E7%94%A8%E6%88%B7%E7%99%BB%E5%BD%95%E7%AE%A1%E7%90%86%E6%A8%A1%E5%9D%97PRD.md) "is_initial_password" + [DATA_MODEL_LOGIN.md](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/DATA_MODEL_LOGIN.md) `user_accounts.is_initial_password` + [登录管理技术方案.md](file:///mnt/d/Workspace/nexus/Project/fonrey/TECH_STACK/%E7%99%BB%E5%BD%95%E7%AE%A1%E7%90%86%E6%8A%80%E6%9C%AF%E6%96%B9%E6%A1%88.md) v4.1
|
||||
- **问题**：PRD 要求"首次登录必须改密才能进入业务"，但技术方案未明确：登录 API 返回时如何标记需要改密（专用 HTTP code/Header/JSON flag），中间件/装饰器在哪一层拦截，HTMX 前端如何重定向。
|
||||
- **改进方向**：技术方案补 §"初始密码拦截器"：登录响应增 `must_change_password: true` 字段；新增 `RequirePasswordChangeMiddleware`；HTMX 通过 `HX-Redirect` 跳转 `/account/change-password`。
|
||||
- **责任角色**：Backend Architect + Frontend Lead
|
||||
|
||||
### 🟡 PT-L-2 账户锁定阈值 PRD 与 TECH 描述存在微差
|
||||
- **位置**：[用户登录管理模块PRD.md](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E7%99%BB%E5%BD%95%E7%AE%A1%E7%90%86/%E7%94%A8%E6%88%B7%E7%99%BB%E5%BD%95%E7%AE%A1%E7%90%86%E6%A8%A1%E5%9D%97PRD.md) §账户锁定 vs [DATA_MODEL_LOGIN.md L411-L423](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/DATA_MODEL_LOGIN.md#L411-L423)（Redis 计数器说明）
|
||||
- **问题**：PRD 写"连续 5 次失败锁定 30 分钟"，DATA_MODEL 注释也说 5 次，但 [登录管理技术方案.md](file:///mnt/d/Workspace/nexus/Project/fonrey/TECH_STACK/%E7%99%BB%E5%BD%95%E7%AE%A1%E7%90%86%E6%8A%80%E6%9C%AF%E6%96%B9%E6%A1%88.md) 未将阈值/窗口/键名作为可配置项明示，存在硬编码风险。
|
||||
- **改进方向**：技术方案显式列出 settings：`LOGIN_FAIL_THRESHOLD=5`、`LOGIN_LOCK_DURATION_MIN=30`、Redis key `login_fail:{tenant}:{username}`。
|
||||
- **责任角色**：Backend Architect
|
||||
|
||||
---
|
||||
|
||||
## 维度三：DATA_MODEL（数据模型）
|
||||
|
||||
### 🔴 D-1 `public.platform_audit_logs.result` 大小写违反 ENUMS.md 规范
|
||||
- **位置**：[ENUMS.md L70-L72](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/ENUMS.md#L70-L72) vs [DATA_MODEL_PUBLIC.md L274-L275](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/DATA_MODEL_PUBLIC.md#L274-L275)
|
||||
- **问题**：ENUMS.md 规定 `success`/`failed`（lower_snake_case，文件首部 L10 强制要求），DDL 实际为 `CHECK (result IN ('SUCCESS','FAILED'))` DEFAULT `'SUCCESS'`。前端 enum_labels、查询过滤、报表全部错位。
|
||||
- **改进方向**：DDL 改为小写并执行迁移 `UPDATE platform_audit_logs SET result = lower(result)`；同步 enum_labels fixture。
|
||||
- **责任角色**：DBA + Backend Architect
|
||||
|
||||
### 🔴 D-2 `public.export_tasks.status` `done` vs ENUMS `success` 不一致
|
||||
- **位置**：[ENUMS.md L102-L106](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/ENUMS.md#L102-L106) vs [DATA_MODEL_PUBLIC.md L351-L353](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/DATA_MODEL_PUBLIC.md#L351-L353)
|
||||
- **问题**：ENUMS 已迁移为 `success`（注明"原 `done`，已迁移"），DDL 仍为 `CHECK (status IN ('pending','in_progress','done','failed'))`。`backup_records.status` 已用 `success`，与之分裂。
|
||||
- **改进方向**：DDL CHECK + DEFAULT 改为 `success`，迁移脚本 `UPDATE export_tasks SET status='success' WHERE status='done'`。
|
||||
- **责任角色**：DBA
|
||||
|
||||
### 🟠 D-3 ENUMS.md 缺失 `tenants.suspended_reason` 的 `license_expired`
|
||||
- **位置**：[ENUMS.md L55-L59](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/ENUMS.md#L55-L59) vs [DATA_MODEL_PUBLIC.md L115](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/DATA_MODEL_PUBLIC.md#L115)
|
||||
- **问题**：DDL CHECK 已包含 `license_expired`，ENUMS.md 仅列 `overdue/violation/requested/other`。前端文案/报表无法对该值取值。
|
||||
- **改进方向**：ENUMS.md 补 `license_expired`，记录变更 ADR；同步 enum_labels。
|
||||
- **责任角色**：ENUMS.md 维护者（架构 Atlas）
|
||||
|
||||
### 🟠 D-4 ENUMS.md 缺失 `login_attempts.failure_reason` 的 OTP 相关值
|
||||
- **位置**：[ENUMS.md L168-L174](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/ENUMS.md#L168-L174) vs [DATA_MODEL_LOGIN.md L220-L224, L266-L274](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/DATA_MODEL_LOGIN.md#L220-L224)
|
||||
- **问题**：DDL/Model choices 含 `wrong_otp`/`otp_expired`，ENUMS.md 漏写。
|
||||
- **改进方向**：ENUMS.md 补两个值。
|
||||
- **责任角色**：ENUMS.md 维护者
|
||||
|
||||
### 🟠 D-5 `user_accounts.phone_enc / phone_hash` 业务必填但 DB 可空
|
||||
- **位置**：[用户登录管理模块PRD.md L31-L33](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E7%99%BB%E5%BD%95%E7%AE%A1%E7%90%86/%E7%94%A8%E6%88%B7%E7%99%BB%E5%BD%95%E7%AE%A1%E7%90%86%E6%A8%A1%E5%9D%97PRD.md#L31-L33) vs [DATA_MODEL_LOGIN.md L80-L81](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/DATA_MODEL_LOGIN.md#L80-L81)
|
||||
- **问题**：PRD 规定 `username` 等于手机号且必填，但 `phone_enc TEXT NULL`、`phone_hash VARCHAR(64) NULL`。导入/数据修复时可能写入不一致数据，且 `phone_hash` 唯一性索引（若有）可能失效。
|
||||
- **改进方向**：业务确认后将 `phone_hash` 改 NOT NULL（普通员工），或改用 `username` 作 `phone_hash` 计算源在应用层强约束 + 触发器保证一致性。
|
||||
- **责任角色**：accounts 模块负责人 + DBA
|
||||
|
||||
### 🟠 D-6 `client_heartbeats` 无分区策略
|
||||
- **位置**：[DATA_MODEL_PUBLIC.md L568-L609](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/DATA_MODEL_PUBLIC.md#L568-L609)
|
||||
- **问题**：心跳表为高频写表（按 PRD 推算每客户端每 5 分钟一条），无 RANGE 分区，长期会变成 TB 级单表。
|
||||
- **改进方向**：按 `received_at` 月度 RANGE 分区；保留 90 天，老分区归档至冷库或删除。
|
||||
- **责任角色**：DBA
|
||||
|
||||
### 🟠 D-7 `platform_audit_logs.payload_summary` 缺全文索引
|
||||
- **位置**：[DATA_MODEL_PUBLIC.md L260-L290](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/DATA_MODEL_PUBLIC.md#L260-L290)
|
||||
- **问题**：PRD 要求平台审计日志支持关键字搜索，但 `payload_summary` 仅 B-Tree 索引，无 GIN/tsvector 索引。
|
||||
- **改进方向**：增 `CREATE INDEX ON platform_audit_logs USING GIN (to_tsvector('simple', payload_summary))`。
|
||||
- **责任角色**：DBA
|
||||
|
||||
### 🟡 D-8 跨 schema 操作规范未沉淀到 DATA_MODEL.md 头部
|
||||
- **位置**：[DATA_MODEL.md](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/DATA_MODEL.md) 总纲
|
||||
- **问题**：django-tenants 下不可建跨 schema FK，但 README 未明示该约定，新人可能误建外键。
|
||||
- **改进方向**：DATA_MODEL.md 增 §跨 schema 规范小节。
|
||||
- **责任角色**：Data Model Owner
|
||||
|
||||
---
|
||||
|
||||
## 维度四：API 契约
|
||||
|
||||
### 🔴 A-B-1 平台管理后台核心端点未在 API_CONTRACT.md 出现
|
||||
- **位置**：[API_CONTRACT.md](file:///mnt/d/Workspace/nexus/Project/fonrey/TECH_STACK/API_CONTRACT.md) vs [平台管理后台PRD.md](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0PRD.md)
|
||||
- **问题**：平台后台所述创建租户/暂停租户/发布客户端版本/查看审计等 API 均未在契约文档定义；前后端无对接基线。
|
||||
- **改进方向**：API_CONTRACT.md 增 §平台管理后台 端点章节，列出 path/method/请求/响应/错误码。
|
||||
- **责任角色**：Backend Architect + API 文档负责人
|
||||
|
||||
### 🟠 A-L-1 登录端点错误码未对齐 ENUMS `failure_reason`
|
||||
- **位置**：[API_CONTRACT.md](file:///mnt/d/Workspace/nexus/Project/fonrey/TECH_STACK/API_CONTRACT.md) login 端点 vs [ENUMS.md L168-L174](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/ENUMS.md#L168-L174)
|
||||
- **问题**：API 返回错误码与 `login_attempts.failure_reason` 枚举无映射表，客户端无法根据错误码呈现差异化文案。
|
||||
- **改进方向**：增"错误码 ↔ failure_reason ↔ 用户文案"三列对照表。
|
||||
- **责任角色**：Backend Architect + 安全负责人
|
||||
|
||||
### 🟠 A-B-2 客户端心跳/版本检查 API 无幂等与限流约定
|
||||
- **位置**：[API_CONTRACT.md](file:///mnt/d/Workspace/nexus/Project/fonrey/TECH_STACK/API_CONTRACT.md) vs [平台管理后台PRD.md](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0PRD.md) §客户端版本管理
|
||||
- **问题**：心跳/版本检查为高频公网入口，未规定 `Idempotency-Key`、限流阈值（每客户端 N 次/分钟）、签名校验。
|
||||
- **改进方向**：API_CONTRACT 补限流 + HMAC 签名（基于 client_id + secret）。
|
||||
- **责任角色**：Backend Architect + 安全
|
||||
|
||||
### 🟡 A-G-1 全局 API 错误响应结构未统一
|
||||
- **位置**：[API_CONTRACT.md](file:///mnt/d/Workspace/nexus/Project/fonrey/TECH_STACK/API_CONTRACT.md)
|
||||
- **问题**：示例中部分接口错误体为 `{detail: "..."}`、部分为 `{code, message, errors[]}`，前端难以统一错误处理。
|
||||
- **改进方向**：统一为 RFC 7807 Problem Details 或自定义 `{code, message, details}` 结构。
|
||||
- **责任角色**：Backend Architect
|
||||
|
||||
---
|
||||
|
||||
## 维度五：UI / UI_SYSTEM
|
||||
|
||||
### 🟠 U-B-1 平台后台 UI 设计未引用 `组件清单.md` 中已定义组件
|
||||
- **位置**：[UI_DESIGN/平台管理后台/平台管理后台_UI.md](file:///mnt/d/Workspace/nexus/Project/fonrey/UI_DESIGN/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0_UI.md) + [平台管理后台_UI.html](file:///mnt/d/Workspace/nexus/Project/fonrey/UI_DESIGN/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0_UI.html) vs [组件清单.md](file:///mnt/d/Workspace/nexus/Project/fonrey/UI_SYSTEM/%E7%BB%84%E4%BB%B6%E6%B8%85%E5%8D%95.md)
|
||||
- **问题**：HTML 原型直接写裸 Tailwind 类，未与组件清单中的 `<button>` / `<table>` / `<modal>` 规范关联。前端实现易产生样式漂移。
|
||||
- **改进方向**：UI 设计文档每个区块标注引用组件 ID（如 "DataTable v1"）；HTML 原型用占位 class 体现组件契约。
|
||||
- **责任角色**：Frontend Lead + UI Designer
|
||||
|
||||
### 🟠 U-L-1 登录 UI 缺失"账户锁定"与"密码过期"专用状态页
|
||||
- **位置**：[UI_DESIGN/登录管理/登录_UI.md](file:///mnt/d/Workspace/nexus/Project/fonrey/UI_DESIGN/%E7%99%BB%E5%BD%95%E7%AE%A1%E7%90%86/%E7%99%BB%E5%BD%95_UI.md) + 3 份 HTML 原型
|
||||
- **问题**：PRD 描述了账户锁定、密码过期、首次登录强制改密 3 类状态，但 UI 设计仅覆盖登录、账密、重置密码 3 种页面。
|
||||
- **改进方向**：补"已锁定""密码过期"状态页或在登录页定义 inline 提示规范。
|
||||
- **责任角色**：UI Designer + Frontend Lead
|
||||
|
||||
### 🟡 U-G-1 UI_SYSTEM 缺无障碍（A11y）规范
|
||||
- **位置**：[UI_SYSTEM.md](file:///mnt/d/Workspace/nexus/Project/fonrey/UI_SYSTEM/UI_SYSTEM.md)
|
||||
- **问题**：未规定 ARIA、键盘导航、对比度门槛。
|
||||
- **改进方向**：增 §A11y 章节（WCAG 2.1 AA 基线）。
|
||||
- **责任角色**：UI Designer
|
||||
|
||||
### 🟡 U-G-2 HTML 原型与 `组件规范设计.md` 间距/色板未对齐
|
||||
- **位置**：[平台管理后台_UI.html](file:///mnt/d/Workspace/nexus/Project/fonrey/UI_DESIGN/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0_UI.html) vs [组件规范设计.md](file:///mnt/d/Workspace/nexus/Project/fonrey/UI_SYSTEM/%E7%BB%84%E4%BB%B6%E8%A7%84%E8%8C%83%E8%AE%BE%E8%AE%A1.md)
|
||||
- **问题**：原型出现规范外的 `gray-750`、不规则间距 `p-7`。
|
||||
- **改进方向**：原型严格使用 design token（间距 4/8/12/16/24/32）。
|
||||
- **责任角色**：UI Designer
|
||||
|
||||
---
|
||||
|
||||
## 维度六：安全 & 多租户隔离
|
||||
|
||||
### 🔴 S-1 Tenant Admin 初始密码可能明文邮件下发
|
||||
- **位置**：[平台管理后台PRD.md L122-L136](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0PRD.md#L122-L136) + [DATA_MODEL_LOGIN.md L468-L469](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/DATA_MODEL_LOGIN.md#L468-L469)（示例 `Fonrey@2025`）
|
||||
- **问题**：欢迎邮件包含初始密码明文；DATA_MODEL 给出系统统一默认密码示例。一旦邮箱被截获或日志记录到邮件正文，全部新租户首批管理员凭据泄露。
|
||||
- **改进方向**：禁止邮件明文密码；改用一次性安全 reset token（10-30 分钟有效，单次使用），强制首次登录改密；离线场景由平台运维通过加密 PDF 单独交付。
|
||||
- **责任角色**：安全负责人 + PM + accounts 模块
|
||||
|
||||
### 🔴 S-2 平台管理员（PlatformAdmin）会话与租户用户共用 Cookie/Session 边界未明示
|
||||
- **位置**：[平台管理后台PRD.md](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0PRD.md) + [登录管理技术方案.md](file:///mnt/d/Workspace/nexus/Project/fonrey/TECH_STACK/%E7%99%BB%E5%BD%95%E7%AE%A1%E7%90%86%E6%8A%80%E6%9C%AF%E6%96%B9%E6%A1%88.md)
|
||||
- **问题**：未规定平台后台域名/Cookie 域、Session 隔离机制；存在租户 schema 用户登录态与平台后台越权同会话风险。
|
||||
- **改进方向**：平台后台独立子域 `admin.fonrey.com`，Cookie 域分离；中间件根据 `request.tenant.schema_name == 'public'` 强校验只允许 `platform_admins` 表用户进入。
|
||||
- **责任角色**：安全负责人 + Backend Architect
|
||||
|
||||
### 🟠 S-3 短信 OTP 速率/防刷未定量
|
||||
- **位置**：[用户登录管理模块PRD.md](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E7%99%BB%E5%BD%95%E7%AE%A1%E7%90%86/%E7%94%A8%E6%88%B7%E7%99%BB%E5%BD%95%E7%AE%A1%E7%90%86%E6%A8%A1%E5%9D%97PRD.md) §短信验证码 + [DATA_MODEL_LOGIN.md sms_otp_records](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/DATA_MODEL_LOGIN.md)
|
||||
- **问题**：未定义"同号码 X 分钟内最多 N 次"、"同 IP 每日上限"、"图形/滑块校验前置"等防刷规则。
|
||||
- **改进方向**：技术方案补防刷策略；DATA_MODEL 增 IP 维度计数器或在 Redis 中维护。
|
||||
- **责任角色**：安全负责人
|
||||
|
||||
### 🟠 S-4 `phone_enc` 加密密钥轮换与 `phone_hash` salt 策略缺失
|
||||
- **位置**：[DATA_MODEL_LOGIN.md L80-L81](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/DATA_MODEL_LOGIN.md#L80-L81) + [TECH_STACK.md](file:///mnt/d/Workspace/nexus/Project/fonrey/TECH_STACK/TECH_STACK.md)
|
||||
- **问题**：未规定加密算法（AES-GCM？）、密钥来源（KMS？env？）、轮换流程、`phone_hash` 是否加 pepper。
|
||||
- **改进方向**：技术方案补 §字段级加密：算法/Key 来源/轮换 SOP；`phone_hash` 使用 HMAC-SHA256 + 全局 pepper（不入库）。
|
||||
- **责任角色**：安全负责人 + Backend Architect
|
||||
|
||||
### 🟠 S-5 平台后台审计日志覆盖范围未量化
|
||||
- **位置**：[平台管理后台PRD.md](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0PRD.md) + [DATA_MODEL_PUBLIC.md platform_audit_logs](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/DATA_MODEL_PUBLIC.md)
|
||||
- **问题**：哪些操作必须审计、保留期限、防篡改（append-only / 哈希链）未规定。
|
||||
- **改进方向**：列出"必审操作清单"（创建/暂停/删除租户、平台账号增删改、发版、备份恢复）；表加 `prev_hash`/`row_hash` 形成哈希链或转写至追加型存储。
|
||||
- **责任角色**：安全负责人
|
||||
|
||||
### 🟡 S-6 跨 schema 查询风险未在技术方案标注
|
||||
- **位置**：[DATA_MODEL.md](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/DATA_MODEL.md) + [TECH_STACK.md](file:///mnt/d/Workspace/nexus/Project/fonrey/TECH_STACK/TECH_STACK.md)
|
||||
- **问题**：django-tenants 下错误的 schema 切换会导致跨租户数据泄露。
|
||||
- **改进方向**：技术方案增 §"禁止在 view 中直接 `connection.set_schema_to_public()`"，必须经 service 层封装。
|
||||
- **责任角色**：Backend Architect
|
||||
|
||||
---
|
||||
|
||||
## 维度七：性能 & 扩展性
|
||||
|
||||
### 🟠 PF-1 `client_heartbeats` 写入风暴未做缓冲
|
||||
- **位置**：[DATA_MODEL_PUBLIC.md client_heartbeats](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/DATA_MODEL_PUBLIC.md#L568-L609) + [平台管理后台技术方案.md](file:///mnt/d/Workspace/nexus/Project/fonrey/TECH_STACK/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0%E6%8A%80%E6%9C%AF%E6%96%B9%E6%A1%88.md)
|
||||
- **问题**：每条心跳直写 PG，N×M 客户端规模下会成为 IO 瓶颈。
|
||||
- **改进方向**：心跳先写 Redis Stream / Kafka，再批量落表（每分钟一次）；同时配合 D-6 分区。
|
||||
- **责任角色**：Backend Architect
|
||||
|
||||
### 🟠 PF-2 平台后台租户列表 / 审计日志列表 缺分页/索引设计
|
||||
- **位置**：[平台管理后台技术方案.md](file:///mnt/d/Workspace/nexus/Project/fonrey/TECH_STACK/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0%E6%8A%80%E6%9C%AF%E6%96%B9%E6%A1%88.md) + [DATA_MODEL_PUBLIC.md tenants/platform_audit_logs](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/DATA_MODEL_PUBLIC.md)
|
||||
- **问题**：未明示 keyset pagination 或 LIMIT/OFFSET；审计日志按时间倒序查询无 `(created_at DESC, id DESC)` 复合索引。
|
||||
- **改进方向**：明确分页方案；增复合索引；列表 API 默认 page_size 上限。
|
||||
- **责任角色**：Backend Architect + DBA
|
||||
|
||||
### 🟡 PF-3 登录端点未规定 P95 延迟 SLO
|
||||
- **位置**：[用户登录管理模块PRD.md](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E7%99%BB%E5%BD%95%E7%AE%A1%E7%90%86/%E7%94%A8%E6%88%B7%E7%99%BB%E5%BD%95%E7%AE%A1%E7%90%86%E6%A8%A1%E5%9D%97PRD.md) + [测试规范.md](file:///mnt/d/Workspace/nexus/Project/fonrey/TECH_STACK/%E6%B5%8B%E8%AF%95%E8%A7%84%E8%8C%83.md)
|
||||
- **问题**：无性能 SLO（如登录 P95 < 300ms）。
|
||||
- **改进方向**：PRD 增非功能需求小节，列出关键端点 SLO；测试规范补压测脚本。
|
||||
- **责任角色**：PM + QA
|
||||
|
||||
---
|
||||
|
||||
## 维度八：遗漏场景 & 边界
|
||||
|
||||
### 🔴 X-1 租户删除 / 数据导出 / GDPR 类合规流程缺失
|
||||
- **位置**：[平台管理后台PRD.md](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0PRD.md)
|
||||
- **问题**：仅描述 `pending_delete` → `deleted` 状态，未覆盖：删除前数据导出/客户签字回执；平台管理员双人复核；保留期内可撤销。
|
||||
- **改进方向**：新增"租户注销 SOP"用户故事；技术方案补软删除 + 冷却期 + 双人审批流程。
|
||||
- **责任角色**：PM + 安全 + 合规
|
||||
|
||||
### 🟠 X-2 客户端版本回滚 / 灰度中止流程缺失
|
||||
- **位置**：[平台管理后台PRD.md L476-L520](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0PRD.md#L476-L520)
|
||||
- **问题**：发版有，但回滚/灰度中止缺定义。
|
||||
- **改进方向**：补 UC "回滚到上一稳定版本"、"立即停灰度"两条用户故事。
|
||||
- **责任角色**：PM
|
||||
|
||||
### 🟠 X-3 备份恢复演练流程缺失
|
||||
- **位置**：[平台管理后台PRD.md](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0PRD.md) §备份 + [DATA_MODEL_PUBLIC.md backup_records](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/DATA_MODEL_PUBLIC.md)
|
||||
- **问题**：仅描述备份生成，缺"恢复到独立 schema 验证"演练流程；恢复操作风险（覆盖现网）未提示。
|
||||
- **改进方向**：增"恢复到隔离 schema 演练"用户故事；技术方案补恢复执行规范（必须先恢复至 `restore_*` 临时 schema 校验）。
|
||||
- **责任角色**：PM + DBA
|
||||
|
||||
### 🟠 X-4 Tenant Admin 失联 / 转移流程未定义
|
||||
- **位置**：[用户登录管理模块PRD.md](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E7%99%BB%E5%BD%95%E7%AE%A1%E7%90%86/%E7%94%A8%E6%88%B7%E7%99%BB%E5%BD%95%E7%AE%A1%E7%90%86%E6%A8%A1%E5%9D%97PRD.md) + [平台管理后台PRD.md](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0PRD.md)
|
||||
- **问题**：Tenant Admin 离职/换号场景下，由谁/如何重置或迁移管理员未定义。
|
||||
- **改进方向**：PRD 增"平台运维代运营重置 Tenant Admin"流程，需双人复核 + 审计。
|
||||
- **责任角色**：PM + 安全
|
||||
|
||||
### 🟡 X-5 同租户多设备登录策略未明示
|
||||
- **位置**：[用户登录管理模块PRD.md](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E7%99%BB%E5%BD%95%E7%AE%A1%E7%90%86/%E7%94%A8%E6%88%B7%E7%99%BB%E5%BD%95%E7%AE%A1%E7%90%86%E6%A8%A1%E5%9D%97PRD.md)
|
||||
- **问题**：是否允许同账号多端在线、互踢策略未定义。
|
||||
- **改进方向**：PRD 明示"允许并发"或"后登挤掉前登"。
|
||||
- **责任角色**：PM
|
||||
|
||||
### 🟡 X-6 跨场景同义概念差异（租户开通 → Tenant Admin 创建）
|
||||
- **位置**：[平台管理后台PRD.md L122-L136](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0/%E5%B9%B3%E5%8F%B0%E7%AE%A1%E7%90%86%E5%90%8E%E5%8F%B0PRD.md#L122-L136) vs [DATA_MODEL_LOGIN.md L468-L469](file:///mnt/d/Workspace/nexus/Project/fonrey/DATA_MODEL/DATA_MODEL_LOGIN.md#L468-L469) vs [用户登录管理模块PRD.md L31-L33](file:///mnt/d/Workspace/nexus/Project/fonrey/PRD/%E7%99%BB%E5%BD%95%E7%AE%A1%E7%90%86/%E7%94%A8%E6%88%B7%E7%99%BB%E5%BD%95%E7%AE%A1%E7%90%86%E6%A8%A1%E5%9D%97PRD.md#L31-L33)
|
||||
- **问题**：三处对"初始 Tenant Admin"的用户名/初始密码/手机号来源描述用词不统一（PRD 平台后台说"系统生成"，DATA_MODEL 给出 `Fonrey@2025` 默认值，登录 PRD 说 username = `tenants.contact_phone`）。
|
||||
- **改进方向**：以登录 PRD v3.0 描述为准（username = contact_phone；初始密码由系统生成 + 一次性 reset token），其他文档同步修正。
|
||||
- **责任角色**：PM + Data Model Owner
|
||||
|
||||
---
|
||||
|
||||
## 汇总行动列表（按优先级）
|
||||
|
||||
| # | 严重 | 维度 | 问题 | 责任角色 | 建议截止 |
|
||||
| --- | --- | ---------- | -------------------------------------------------------------- | ----------------------- | ---- |
|
||||
| 1 | 🔴 | 安全 | S-1 禁止欢迎邮件明文初始密码，改一次性 reset token | 安全 + PM + accounts | 立即 |
|
||||
| 2 | 🔴 | 安全 | S-2 平台后台独立子域/Cookie 域 + 平台账号会话隔离中间件 | 安全 + Backend Architect | 立即 |
|
||||
| 3 | 🔴 | DATA_MODEL | D-1 `platform_audit_logs.result` 大写改小写 + 数据迁移 | DBA | 当周 |
|
||||
| 4 | 🔴 | DATA_MODEL | D-2 `export_tasks.status` `done`→`success` + 迁移 | DBA | 当周 |
|
||||
| 5 | 🔴 | API | A-B-1 API_CONTRACT 补齐平台后台所有端点 | Backend Architect | 当周 |
|
||||
| 6 | 🔴 | PRD↔TECH | PT-B-1 创建租户 Saga + 补偿事务 | Backend Architect + Ops | 当周 |
|
||||
| 7 | 🔴 | PRD质量 | P-B-1 平台后台 PRD 引入 PA-NNN 编号体系 + AC | PM | 当周 |
|
||||
| 8 | 🔴 | 遗漏场景 | X-1 租户注销 SOP + 双人复核 + 冷却期 | PM + 安全 + 合规 | 下迭代 |
|
||||
| 9 | 🟠 | DATA_MODEL | D-3/D-4 ENUMS.md 补 `license_expired`/`wrong_otp`/`otp_expired` | ENUMS 维护者 | 当周 |
|
||||
| 10 | 🟠 | DATA_MODEL | D-5 `phone_enc/phone_hash` NOT NULL + 数据清洗 | accounts + DBA | 下迭代 |
|
||||
| 11 | 🟠 | DATA_MODEL | D-6 `client_heartbeats` 月度分区 + 90 天保留 | DBA | 下迭代 |
|
||||
| 12 | 🟠 | DATA_MODEL | D-7 `payload_summary` 增 GIN 全文索引 | DBA | 当周 |
|
||||
| 13 | 🟠 | PRD↔TECH | PT-B-2 客户端发版灰度策略澄清 | Backend Architect | 下迭代 |
|
||||
| 14 | 🟠 | PRD↔TECH | PT-B-3 平台 MFA 选型 + DATA_MODEL 字段补充 | 安全 + Architect | 下迭代 |
|
||||
| 15 | 🟠 | PRD↔TECH | PT-L-1 首次登录强制改密拦截器实现路径 | Backend + Frontend | 当周 |
|
||||
| 16 | 🟠 | API | A-L-1 错误码 ↔ failure_reason 对照表 | Backend + 安全 | 当周 |
|
||||
| 17 | 🟠 | API | A-B-2 心跳/版本检查接口限流 + HMAC | Backend + 安全 | 下迭代 |
|
||||
| 18 | 🟠 | UI | U-B-1 平台后台 UI 引用组件清单组件 ID | UI + Frontend | 下迭代 |
|
||||
| 19 | 🟠 | UI | U-L-1 补"账户锁定/密码过期"状态页 | UI + Frontend | 当周 |
|
||||
| 20 | 🟠 | 安全 | S-3 短信 OTP 防刷规则量化 | 安全 | 下迭代 |
|
||||
| 21 | 🟠 | 安全 | S-4 字段级加密 SOP + `phone_hash` pepper | 安全 + Architect | 下迭代 |
|
||||
| 22 | 🟠 | 安全 | S-5 平台审计日志哈希链 + 必审操作清单 | 安全 | 下迭代 |
|
||||
| 23 | 🟠 | 性能 | PF-1 心跳 Redis Stream 缓冲 + 批量落表 | Architect | 下迭代 |
|
||||
| 24 | 🟠 | 性能 | PF-2 列表分页/复合索引 | Architect + DBA | 当周 |
|
||||
| 25 | 🟠 | PRD质量 | P-B-2 平台 PRD 剥离实现细节 | PM + Architect | 下迭代 |
|
||||
| 26 | 🟠 | PRD质量 | P-B-3 PlatformAdmin 权限矩阵 | PM + 安全 | 当周 |
|
||||
| 27 | 🟠 | 遗漏 | X-2 版本回滚 / 中止灰度 UC | PM | 下迭代 |
|
||||
| 28 | 🟠 | 遗漏 | X-3 备份恢复演练流程 | PM + DBA | 下迭代 |
|
||||
| 29 | 🟠 | 遗漏 | X-4 Tenant Admin 重置/迁移流程 | PM + 安全 | 下迭代 |
|
||||
| 30 | 🟡 | DATA_MODEL | D-8 跨 schema 规范沉淀至 README | Data Model Owner | 下迭代 |
|
||||
| 31 | 🟡 | API | A-G-1 错误响应结构统一 | Architect | 下迭代 |
|
||||
| 32 | 🟡 | UI | U-G-1 / U-G-2 A11y 规范 + 设计 token 严控 | UI Designer | 下迭代 |
|
||||
| 33 | 🟡 | 性能 | PF-3 关键端点 SLO + 压测脚本 | PM + QA | 下迭代 |
|
||||
| 34 | 🟡 | PRD↔TECH | PT-L-2 锁定阈值 settings 可配置 | Architect | 当周 |
|
||||
| 35 | 🟡 | PRD质量 | P-L-1 登录 PRD 异常路径细化 | PM | 下迭代 |
|
||||
| 36 | 🟡 | 遗漏 | X-5 多设备登录策略明示 | PM | 下迭代 |
|
||||
| 37 | 🟡 | 遗漏 | X-6 跨文档对"初始 Tenant Admin 创建"用词统一 | PM + Data Model Owner | 当周 |
|
||||
|
||||
**统计**：🔴 Blocker 8 项 / 🟠 Major 21 项 / 🟡 Minor 8 项，合计 37 项。
|
||||
|
||||
---
|
||||
|
||||
## 结论
|
||||
|
||||
**整体评估**：
|
||||
- **登录管理模块**：PRD v3.0 已按 ADR-20260502-003 完成实现细节剥离，整体结构与 DATA_MODEL_LOGIN 大致对齐；主要风险集中在 (a) 手机号字段约束与 PRD 必填语义不一致 (S-1/D-5)、(b) 首次登录改密拦截缺技术方案落地 (PT-L-1)、(c) OTP 防刷规则未量化 (S-3)。
|
||||
- **平台管理后台模块**：处于"PRD 形成中"阶段，存在较多 Blocker：缺端点契约 (A-B-1)、缺创建租户 Saga (PT-B-1)、缺平台账号会话隔离 (S-2)、缺欢迎邮件密码安全策略 (S-1)、PRD 缺编号体系 (P-B-1)、PRD 仍混入实现细节 (P-B-2)。这些项必须在进入开发前解决，否则将引发跨团队反复对齐成本。
|
||||
- **跨文档一致性**：枚举命名规范（lower_snake_case）已被 ENUMS.md 强约束，但 DATA_MODEL_PUBLIC 的 DDL 多处未对齐（D-1/D-2/D-3/D-4），属于"规范已立但落地未跟"，建议立即统一。
|
||||
|
||||
**建议下一步动作**：
|
||||
1. 召开 1 次跨角色 Review 会议（PM + Architect + 安全 + DBA + UI + Frontend），就上表 8 项 Blocker 形成处理排期。
|
||||
2. 由 ENUMS.md 维护者在 1 周内发布 v2.3，吸纳 D-3/D-4，并附数据迁移 SQL 草案（D-1/D-2）。
|
||||
3. 由 PM 在 1 周内完成平台管理后台 PRD 编号化与剥离实现细节的改版，进入下一轮 Review。
|
||||
|
||||
---
|
||||
|
||||
## 附录
|
||||
|
||||
### A. 审查范围内文档清单
|
||||
- PRD：`PRD_MVP.md`、`登录管理/用户登录管理模块PRD.md` (v3.0, 2026-05-02)、`平台管理后台/平台管理后台PRD.md` (835 行)
|
||||
- TECH：`TECH_STACK.md` (v2.5, 2026-04-30)、`登录管理技术方案.md` (v4.1)、`平台管理后台技术方案.md` (986 行)、`API_CONTRACT.md`、`测试规范.md`
|
||||
- DATA_MODEL：`DATA_MODEL.md`、`DATA_MODEL_LOGIN.md` (526 行)、`DATA_MODEL_PUBLIC.md` (1049 行)、`ENUMS.md` (v2.2, 786 行)
|
||||
- UI：`UI_SYSTEM/UI_SYSTEM.md`、`组件清单.md`、`组件规范设计.md`、`UI_DESIGN/登录管理/登录_UI.md`、`UI_DESIGN/平台管理后台/平台管理后台_UI.md`、`登录_UI.html`、`登录_账号密码_UI.html`、`登录_重置密码_UI.html`、`平台管理后台_UI.html`
|
||||
|
||||
### B. 用户原文路径校正
|
||||
- ❌ `Project/fonrey/PRD/系统管理/系统管理模块PRD.md` → ✅ 实际为 `Project/fonrey/PRD/系统配置/系统配置模块PRD.md`（**不在本次 Review 范围**）
|
||||
- ❌ `Project/fonrey/PRD/平台管理后台/平台管理后台PRD` → ✅ 实际带 `.md` 后缀
|
||||
- ❌ `Project/fonrey/UI_DESIGN/登录_UI.md` → ✅ 实际位于 `Project/fonrey/UI_DESIGN/登录管理/登录_UI.md`
|
||||
|
||||
### C. 审查执行说明
|
||||
- 本次 Review 由 4 个并行 explore agent 完成证据收集（登录 8 维度、平台 PRD+TECH、DATA_MODEL_PUBLIC+UI、跨文档枚举/字段/状态机漂移），汇总后按 8 维度去重合并。
|
||||
- 报告坚持"指出问题 + 给出方向"原则，不重写设计文档内容；如某文档段落实际不完整（如 PT-B-3 MFA 字段缺失），按 Major 报告而非自行补全。
|
||||
|
||||
---
|
||||
|
||||
## 进度跟踪表（PM / 架构师填写）
|
||||
|
||||
> **使用说明**：
|
||||
> - **状态**取值：`待启动` / `进行中` / `已完成` / `已验证` / `阻塞` / `搁置`（如搁置请在"备注"列填写理由及决策 ADR ID）
|
||||
> - **处理人**：填写实际负责人姓名/工号
|
||||
> - **修订位置**：填写本次修订涉及的文档路径 + 章节/行号 + Commit/PR 链接
|
||||
> - **关联 ADR**：跨模块规则变更必须创建 ADR（参考 `ADR.md` 治理规则），格式 `ADR-YYYYMMDD-XXX`
|
||||
> - **验证证据**：链接到对应的 PR / Commit / 测试用例 ID（`TC-FON-XXXXXX`）/ Migration 文件
|
||||
> - **每周更新**：建议每周五由 PM 主持 Review 例会更新本表，并在 `REVIEW/` 目录追加增量 Review
|
||||
|
||||
### Blocker 项（🔴 必须立即处理，8 项）
|
||||
|
||||
| # | ID | 问题摘要 | 责任角色 | 处理人 | 状态 | 计划完成日 | 实际完成日 | 修订位置 | 关联 ADR | 验证证据 | 备注 |
|
||||
| --- | ------ | ------------------------------------------- | ----------------------- | ----- | ------- | ---------- | ---------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | ------ | ------------- | ----------------------------------------------------------- |
|
||||
| 1 | S-1 | 禁止欢迎邮件明文初始密码，改一次性 reset token | 安全 + PM + accounts | Nova | **不修复（设计决策）** | 2026-05-02 | 2026-05-02 | — | — | — | PM 决策：保持现有欢迎邮件含初始密码的设计，不改为 reset token 流程。接受此安全取舍。 |
|
||||
| 2 | S-2 | 平台后台独立子域 + 平台账号会话隔离中间件 | 安全 + Backend Architect | Atlas | **已完成** | 2026-05-02 | 2026-05-02 | `TECH_STACK/平台管理后台技术方案.md` v1.1 §7.0（新增"平台后台独立子域与会话隔离"专节，含域名分离表、Cookie 隔离规则、PlatformAdminSessionMiddleware 代码规范、测试矩阵） | — | 技术方案 v1.1 变更历史 | — |
|
||||
| 3 | D-1 | `platform_audit_logs.result` 大写改小写 + 数据迁移 | DBA | Atlas | **已完成** | 2026-05-02 | 2026-05-02 | `DATA_MODEL/DATA_MODEL_PUBLIC.md` v1.7 §2.4 CHECK 约束；附录 M.1 迁移脚本 `migration_D1_audit_logs_result_lowercase.sql` | — | DATA_MODEL v1.7 变更历史 | 迁移脚本须在 Django migration 前执行（见附录 M.1 执行顺序说明） |
|
||||
| 4 | D-2 | `export_tasks.status` `done`→`success` + 迁移 | DBA | Atlas | **已完成** | 2026-05-02 | 2026-05-02 | `DATA_MODEL/DATA_MODEL_PUBLIC.md` v1.7 §2.5 CHECK 约束；附录 M.2 迁移脚本 `migration_D2_export_tasks_done_to_success.sql` | — | DATA_MODEL v1.7 变更历史 | 迁移脚本须在 Django migration 前执行（见附录 M.2 执行顺序说明） |
|
||||
| 5 | A-B-1 | API_CONTRACT 补齐平台后台所有端点 | Backend Architect | Atlas | **已完成** | 2026-05-02 | 2026-05-02 | `TECH_STACK/API_CONTRACT.md` v1.2 §12（新增"平台管理后台端点附录"，涵盖认证 2 项、租户管理 6 项、授权/MFA 3 项、系统配置 3 项、客户端发布 4 项、导出任务 3 项，含请求/响应示例与错误码） | — | API_CONTRACT v1.2 变更历史 | — |
|
||||
| 6 | PT-B-1 | 创建租户 Saga + 补偿事务 | Backend Architect + Ops | Atlas | **已完成** | 2026-05-02 | 2026-05-02 | `TECH_STACK/平台管理后台技术方案.md` v1.1 §6.1.1（新增"创建租户 Saga 与补偿事务"专节，含 7 步流程状态机、各步骤补偿操作、Celery chord 实现规范、幂等性与重试策略、监控告警要求） | — | 技术方案 v1.1 变更历史 | — |
|
||||
| 7 | P-B-1 | 平台后台 PRD 引入 PA-NNN 编号体系 + AC | PM | Nova | **已完成** | 2026-05-02 | 2026-05-02 | `PRD/平台管理后台/平台管理后台PRD.md` v1.1 — 全部 11 条用户故事补齐 PA-001 ~ PA-011 编号，每条附验收标准（AC）小节 | — | PRD v1.1 变更历史 | — |
|
||||
| 8 | X-1 | 租户注销 SOP + 双人复核 + 冷却期 | PM + 安全 + 合规 | Nova | **已完成** | 2026-05-02 | 2026-05-02 | `PRD/平台管理后台/平台管理后台PRD.md` v1.1 — 新增 PA-011 Story E1，含 AC-1 发起申请 / AC-2 数据导出前置确认 / AC-3 双人 MFA 复核 / AC-4 30 天冷静期软删除 / AC-5 硬删除；同步更新 §5.1.4、§5.4、§5.5.1 API 清单、§6 权限矩阵、§7 状态机（新增 Pending Cancellation 节点） | — | PRD v1.1 变更历史 | 技术方案 & DATA_MODEL 联动更新待 Atlas 处理（Pending Cancellation 状态字段） |
|
||||
|
||||
### Major 项（🟠 迭代内处理，21 项）
|
||||
|
||||
| # | ID | 问题摘要 | 责任角色 | 处理人 | 状态 | 计划完成日 | 实际完成日 | 修订位置 | 关联 ADR | 验证证据 | 备注 |
|
||||
| --- | ------ | -------------------------------------- | ------------------ | ----- | --- | ----- | ----- | ---- | ------ | ---- | --- |
|
||||
| 9 | D-3 | ENUMS.md 补 `license_expired` | ENUMS 维护者 | Atlas | 待启动 | | | | | | |
|
||||
| 10 | D-4 | ENUMS.md 补 `wrong_otp` / `otp_expired` | ENUMS 维护者 | Atlas | 待启动 | | | | | | |
|
||||
| 11 | D-5 | `phone_enc/phone_hash` NOT NULL + 数据清洗 | accounts + DBA | Atlas | 待启动 | | | | | | |
|
||||
| 12 | D-6 | `client_heartbeats` 月度分区 + 90 天保留 | DBA | Atlas | 待启动 | | | | | | |
|
||||
| 13 | D-7 | `payload_summary` 增 GIN 全文索引 | DBA | Atlas | 待启动 | | | | | | |
|
||||
| 14 | PT-B-2 | 客户端发版灰度策略澄清（租户/百分比） | Backend Architect | Atlas | 待启动 | | | | | | |
|
||||
| 15 | PT-B-3 | 平台 MFA 选型 + DATA_MODEL 字段补充 | 安全 + Architect | Atlas | 待启动 | | | | | | |
|
||||
| 16 | PT-L-1 | 首次登录强制改密拦截器实现路径 | Backend + Frontend | Atlas | 待启动 | | | | | | |
|
||||
| 17 | A-L-1 | 错误码 ↔ failure_reason 对照表 | Backend + 安全 | Atlas | 待启动 | | | | | | |
|
||||
| 18 | A-B-2 | 心跳/版本检查接口限流 + HMAC 签名 | Backend + 安全 | Atlas | 待启动 | | | | | | |
|
||||
| 19 | U-B-1 | 平台后台 UI 引用组件清单组件 ID | UI + Frontend | Iris | 待启动 | | | | | | |
|
||||
| 20 | U-L-1 | 补"账户锁定/密码过期"状态页 | UI + Frontend | Iris | 待启动 | | | | | | |
|
||||
| 21 | S-3 | 短信 OTP 防刷规则量化 | 安全 | Atlas | 待启动 | | | | | | |
|
||||
| 22 | S-4 | 字段级加密 SOP + `phone_hash` pepper | 安全 + Architect | Atlas | 待启动 | | | | | | |
|
||||
| 23 | S-5 | 平台审计日志哈希链 + 必审操作清单 | 安全 | Atlas | 待启动 | | | | | | |
|
||||
| 24 | PF-1 | 心跳 Redis Stream 缓冲 + 批量落表 | Architect | Atlas | 待启动 | | | | | | |
|
||||
| 25 | PF-2 | 列表分页（keyset）+ 复合索引 | Architect + DBA | Atlas | 待启动 | | | | | | |
|
||||
| 26 | P-B-2 | 平台 PRD 剥离实现细节 | PM + Architect | Nova | 待启动 | | | | | | |
|
||||
| 27 | P-B-3 | PlatformAdmin 权限矩阵 | PM + 安全 | Nova | 待启动 | | | | | | |
|
||||
| 28 | X-2 | 客户端版本回滚 / 中止灰度 UC | PM | Nova | 待启动 | | | | | | |
|
||||
| 29 | X-3 | 备份恢复演练流程（恢复至隔离 schema） | PM + DBA | Nova | 待启动 | | | | | | |
|
||||
| 30 | X-4 | Tenant Admin 重置/迁移流程 | PM + 安全 | Nova | 待启动 | | | | | | |
|
||||
|
||||
### Minor 项（🟡 优化建议，8 项）
|
||||
|
||||
| # | ID | 问题摘要 | 责任角色 | 处理人 | 状态 | 计划完成日 | 实际完成日 | 修订位置 | 关联 ADR | 验证证据 | 备注 |
|
||||
| --- | ------ | ------------------------------------- | --------------------- | ----- | --- | ----- | ----- | ---- | ------ | ---- | --- |
|
||||
| 31 | D-8 | 跨 schema 操作规范沉淀至 DATA_MODEL.md README | Data Model Owner | Atlas | 待启动 | | | | | | |
|
||||
| 32 | A-G-1 | 全局 API 错误响应结构统一 | Architect | Atlas | 待启动 | | | | | | |
|
||||
| 33 | U-G-1 | UI_SYSTEM 增 A11y 规范（WCAG 2.1 AA） | UI Designer | Iris | 待启动 | | | | | | |
|
||||
| 34 | U-G-2 | HTML 原型严控 design token | UI Designer | Iris | 待启动 | | | | | | |
|
||||
| 35 | PF-3 | 关键端点 SLO + 压测脚本 | PM + QA | Nova | 待启动 | | | | | | |
|
||||
| 36 | PT-L-2 | 锁定阈值 settings 可配置（去硬编码） | Architect | Atlas | 待启动 | | | | | | |
|
||||
| 37 | P-L-1 | 登录 PRD 异常路径细化 | PM | Nova | 待启动 | | | | | | |
|
||||
| 38 | X-5 | 多设备登录策略明示 | PM | Nova | 待启动 | | | | | | |
|
||||
| 39 | X-6 | 跨文档对"初始 Tenant Admin 创建"用词统一 | PM + Data Model Owner | Nova | 待启动 | | | | | | |
|
||||
|
||||
### 进度统计（每周更新）
|
||||
|
||||
| 统计日期 | Blocker 已完成 / 总数 | Major 已完成 / 总数 | Minor 已完成 / 总数 | 阻塞项数 | 风险说明 |
|
||||
|---|---|---|---|---|---|
|
||||
| 2026-05-02 | 0 / 8 | 0 / 21 | 0 / 9 | 0 | 初始基线 |
|
||||
| 2026-05-02（更新） | 2 / 8 | 0 / 21 | 0 / 9 | 0 | P-B-1 & X-1 已完成（Nova）；技术方案/DATA_MODEL 联动更新待 Atlas 跟进 |
|
||||
| 2026-05-02（Atlas 修复） | 7 / 8 | 0 / 21 | 0 / 9 | 0 | S-2 / D-1 / D-2 / A-B-1 / PT-B-1 已完成（Atlas）；剩余 Blocker S-1 为 PM 设计决策不修复；全部 Blocker 实质性完结 |
|
||||
| | | | | | |
|
||||
| | | | | | |
|
||||
|
||||
> 注：Minor 项实际为 9 项（编号 31-39），原汇总行动列表 Minor 计为 8 项的差异在于 D-8 与 X-6 是否合并归类，本跟踪表按独立处理项展开计 9 项。
|
||||
|
||||
### Review 闭环规则
|
||||
|
||||
1. 每条 Finding 修订完成后，处理人填写"修订位置 + 验证证据"，状态置 `已完成`。
|
||||
2. PM/架构师在每周例会复核：跨模块/跨文档变更必须先创建 ADR（`ADR.md`）再改文档，并在"关联 ADR"列填写 ID。
|
||||
3. QA 在所有依赖 Finding 全部 `已完成` 后，对涉及业务路径执行回归测试，状态置 `已验证`。
|
||||
4. 任何 Finding 在迭代结束仍未启动需在"备注"列说明原因并在下一次 Review 例会重新评估优先级。
|
||||
5. 本表与 `REVIEW/` 目录后续增量 Review 报告同步：新增 Finding 追加编号（40+）继续延续。
|
||||
Reference in New Issue
Block a user