From 90f3811b833a7ab87b580a073f9be65e6305b6b4 Mon Sep 17 00:00:00 2001 From: Shen Wei Date: Sat, 2 May 2026 16:21:46 +0800 Subject: [PATCH] =?UTF-8?q?review=20=E7=BB=93=E6=9E=9C?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../fonrey/DATA_MODEL/DATA_MODEL_PUBLIC.md | 67 ++- .../PRD/平台管理后台/平台管理后台PRD.md | 123 ++++- .../fonrey/REVIEW/REVIEW_全局_2026-04-25.md | 441 --------------- .../fonrey/REVIEW/REVIEW_全局_2026-04-26.md | 521 ------------------ .../fonrey/REVIEW/REVIEW_全局_2026-04-28.md | 421 -------------- ...REVIEW_登录管理_平台管理后台_2026-05-02.md | 448 +++++++++++++++ Project/fonrey/TECH_STACK/API_CONTRACT.md | 115 +++- .../fonrey/TECH_STACK/平台管理后台技术方案.md | 336 ++++++++++- Project/fonrey/UI_DESIGN/UI_设计任务总表.md | 63 --- .../REVIEW - 系统设计Review -20260502.md | 397 +++++++++++++ 10 files changed, 1448 insertions(+), 1484 deletions(-) delete mode 100644 Project/fonrey/REVIEW/REVIEW_全局_2026-04-25.md delete mode 100644 Project/fonrey/REVIEW/REVIEW_全局_2026-04-26.md delete mode 100644 Project/fonrey/REVIEW/REVIEW_全局_2026-04-28.md create mode 100644 Project/fonrey/REVIEW/REVIEW_登录管理_平台管理后台_2026-05-02.md delete mode 100644 Project/fonrey/UI_DESIGN/UI_设计任务总表.md create mode 100644 Project/fonrey/prompt/REVIEW - 系统设计Review -20260502.md diff --git a/Project/fonrey/DATA_MODEL/DATA_MODEL_PUBLIC.md b/Project/fonrey/DATA_MODEL/DATA_MODEL_PUBLIC.md index f4d201ee..d7ec7e8c 100644 --- a/Project/fonrey/DATA_MODEL/DATA_MODEL_PUBLIC.md +++ b/Project/fonrey/DATA_MODEL/DATA_MODEL_PUBLIC.md @@ -2,7 +2,7 @@ # Fonrey — Public Schema 数据模型 > **作者**: Backend Architect -> **版本**: v1.6 +> **版本**: v1.7 > **日期**: 2026-05-02 > **权威源**: 本文件是 `public` schema 所有表的唯一权威定义 > **设计依据**: 平台管理后台 PRD（`PRD/平台管理后台/平台管理后台PRD.md`） @@ -15,6 +15,7 @@ | 日期 | 变更人 | 变更内容 | |---|---|---| | 2026-05-02 | Atlas | 对齐平台管理后台 PRD v1.0：`tenants` 增加 License 授权用户数上限字段 `license_user_limit`（含约束、索引、查询示例）；`suspended_reason` 增加 `license_expired`；审计动作补充 `UPDATE_LICENSE_USER_LIMIT` 与 `AUTO_SUSPEND_LICENSE_EXPIRED`；`domains` 标注为统一 `tenant=` 路由下的兼容保留；同步更新元信息与版本历史。 | +| 2026-05-02 | Atlas | v1.7 架构师审核修复：`platform_audit_logs.result` 枚举值改小写；`export_tasks.status` 枚举值 `done`→`success`；补充附录 M 数据迁移脚本。 | | 2026-04-30 | Atlas | 补充“变更历史”章节（文档治理） | ## 一、概览 @@ -271,8 +272,8 @@ CREATE TABLE public.platform_audit_logs ( target_id VARCHAR(255), -- 操作对象 ID（UUID 或其他） target_name VARCHAR(255), -- 操作对象可读名称（快照） payload_summary TEXT, -- 操作内容摘要（非敏感字段） - result VARCHAR(10) NOT NULL DEFAULT 'SUCCESS' - CHECK (result IN ('SUCCESS','FAILED')), + result VARCHAR(10) NOT NULL DEFAULT 'success' + CHECK (result IN ('success','failed')), error_message TEXT, ip_address INET, -- 无 deleted_at，无 UPDATE；按月 RANGE 分区 @@ -350,7 +351,7 @@ CREATE TABLE public.export_tasks ( format VARCHAR(10) NOT NULL CHECK (format IN ('csv','json','sql_dump')), status VARCHAR(15) NOT NULL DEFAULT 'pending' - CHECK (status IN ('pending','in_progress','done','failed')), + CHECK (status IN ('pending','in_progress','success','failed')), storage_path TEXT, -- R2 临时目录路径 download_url TEXT, -- 带签名下载链接 expires_at TIMESTAMPTZ, -- 下载链接有效期（默认 24 小时） @@ -1046,4 +1047,60 @@ ORDER BY created_at DESC; | v1.3 | 2026-04-30 | 配合登录管理 PRD v2.0 / 系统管理 PRD：§2.1 `tenants` 新增 `tenant_code` (CHAR(12), 全局唯一对外识别码) 与 `contact_phone` (CHAR(11)，Tenant Admin 账号创建数据来源)，`contact_email` 改为可 NULL；同步更新约束、查询模式 | | v1.4 | 2026-04-30 | 配合客户端发布管理 PRD v1.1 §5.5 Story 5：§2.6 新增 `client_heartbeats` 表（启动时 Upsert by `tenant_id + device_id`，活跃定义为最近 24h 内心跳）；同步更新表清单、§三 约束（Upsert 锚点 / 启动上报 / 不可下沉到租户 schema）、§5.1 查询模式（Upsert 模板 + 版本活跃分布 JOIN + 平台总活跃数 + 租户维度排查）；MVP 不做升级趋势图（v2 规划） | | v1.5 | 2026-04-30 | 配合客户端发布管理 PRD v1.2 §5.5 Story 5 验收标准追加（按租户统计安装数）：① §2.6 `client_heartbeats.tenant_id` 注释强化，明确为"按租户统计安装数 / 活跃数 / 版本分布"的核心维度字段；② §5.1 新增"租户维度安装/活跃统计"查询专区（4 个查询：某租户活跃安装数 / 历史装机总数 / 租户内版本分布 / 全平台租户活跃榜） | -| v1.6 | 2026-05-02 | 对齐平台管理后台 PRD v1.0：① `tenants` 增加 `license_user_limit`（`INTEGER NOT NULL DEFAULT 50 CHECK (>0)`）并新增 `idx_tenants_license_user_limit`；② `suspended_reason` 增加 `license_expired` 枚举；③ 新增 `idx_tenants_paid_until_active` 并将“即将到期”查询窗口由 7 天调整为 15 天；④ 新增“用户数已满/超限”查询示例（`v_tenant_user_counts` 联表）；⑤ 审计动作补充 `UPDATE_LICENSE_USER_LIMIT`、`AUTO_SUSPEND_LICENSE_EXPIRED`；⑥ `domains` 标记为统一 `tenant=` 路由下的兼容保留。 | +| v1.6 | 2026-05-02 | 对齐平台管理后台 PRD v1.0：① `tenants` 增加 `license_user_limit`（`INTEGER NOT NULL DEFAULT 50 CHECK (>0)`）并新增 `idx_tenants_license_user_limit`；② `suspended_reason` 增加 `license_expired` 枚举；③ 新增 `idx_tenants_paid_until_active` 并将"即将到期"查询窗口由 7 天调整为 15 天；④ 新增"用户数已满/超限"查询示例（`v_tenant_user_counts` 联表）；⑤ 审计动作补充 `UPDATE_LICENSE_USER_LIMIT`、`AUTO_SUSPEND_LICENSE_EXPIRED`；⑥ `domains` 标记为统一 `tenant=` 路由下的兼容保留。 | +| v1.7 | 2026-05-02 | 架构师审核 D-1/D-2 修复：① `platform_audit_logs.result` CHECK 枚举由大写（`SUCCESS/FAILED/PARTIAL`）改为小写（`success/failed/partial`）；② `export_tasks.status` 枚举值 `done` 统一改为 `success`；③ 补充两项对应数据迁移脚本（`migration_D1_audit_logs_result_lowercase.sql` / `migration_D2_export_tasks_done_to_success.sql`）。 | + +--- + +## 附录 M：数据迁移脚本 + +### M.1 D-1 `platform_audit_logs.result` 大写改小写 + +> **背景**：审核报告 D-1 要求 `result` 值统一为小写（`success`, `failed`, `partial`），对应 DDL CHECK 约束已由 v1.7 更新。存量大写数据须一次性迁移。 +> +> **执行条件**：仅在首次部署（或已有存量大写数据的环境）执行，幂等安全（`LOWER()` 结果与小写值相同）。 + +```sql +-- migration_D1_audit_logs_result_lowercase.sql +-- 幂等：多次执行结果一致 +BEGIN; + +UPDATE public.platform_audit_logs +SET result = LOWER(result) +WHERE result ~ '[A-Z]'; -- 仅更新含大写字母的行 + +-- 验证：应返回 0 行 +SELECT COUNT(*) AS remaining_uppercase +FROM public.platform_audit_logs +WHERE result NOT IN ('success', 'failed', 'partial'); + +COMMIT; +``` + +> **回滚脚本**（若需要逆向）： +> ```sql +> -- 仅在需要回滚到旧约束时执行 +> UPDATE public.platform_audit_logs SET result = UPPER(result); +> ``` + +### M.2 D-2 `export_tasks.status` `done` → `success` + +> **背景**：审核报告 D-2 要求 `status` 终态值 `done` 统一改为 `success`，对应 DDL CHECK 约束已由 v1.7 更新。存量 `done` 数据须一次性迁移。 + +```sql +-- migration_D2_export_tasks_done_to_success.sql +BEGIN; + +UPDATE public.export_tasks +SET status = 'success' +WHERE status = 'done'; + +-- 验证：应返回 0 行 +SELECT COUNT(*) AS remaining_done +FROM public.export_tasks +WHERE status = 'done'; + +COMMIT; +``` + +> **执行顺序**：先执行本脚本（数据迁移），再部署含新 DDL 的 Django migration（该 migration 会 `ALTER TABLE ... DROP CONSTRAINT ... ADD CONSTRAINT ...` 更新 CHECK）。若顺序颠倒，`done` 存量数据会触发 CHECK 约束违反。 diff --git a/Project/fonrey/PRD/平台管理后台/平台管理后台PRD.md b/Project/fonrey/PRD/平台管理后台/平台管理后台PRD.md index d11187de..f1fec538 100644 --- a/Project/fonrey/PRD/平台管理后台/平台管理后台PRD.md +++ b/Project/fonrey/PRD/平台管理后台/平台管理后台PRD.md @@ -3,8 +3,7 @@ **状态**：Draft **作者**：产品经理 **最后更新**：2026-05-02 -**版本**：v1.0 -**所属系统**：Fonrey 房产经纪管理系统 +**版本**：v1.1 **关联模块**：权限管理（仅平台侧管理员账号体系）、登录管理（管理员登录与 MFA）、所有租户业务模块（间接，仅作运营/监控对象） **利益相关方**：工程负责人、运营团队、安全合规、客户成功团队、IT 运维 @@ -15,6 +14,7 @@ | 版本 | 日期 | 作者 | 变更说明 | |------|------|------|---------| | v1.0 | 2026-05-02 | 产品经理 | 由 `PRD/系统管理/系统管理模块PRD.md` (v1.3) 与 `PRD/发布管理/客户端发布管理模块PRD.md` (v1.2) 合并而来；统一面向平台管理员视角，统一规划页面路由与业务 API 操作清单。原两份 PRD 文件同步删除，参见 `ADR-20260502-001`。 | +| v1.1 | 2026-05-02 | 产品经理 | **修复 P-B-1**：全部用户故事补齐功能编号 PA-001 ~ PA-011，与登录管理 PRD v3.0「功能 ID + 验收标准」风格对齐，建立需求→测试用例→Bug 单可追溯链条。**修复 X-1**：新增 PA-011「租户注销 SOP」用户故事（Persona E），覆盖删除前数据导出确认、双人复核审批流程、30 天冷静期可撤销机制；同步更新租户状态机（新增 Pending Cancellation 状态节点）、权限矩阵（新增注销审批相关操作行）、§5.1.4 删除章节（引用 PA-011 SOP）、§5.4 页面导航逻辑（注销审批待办流程）、租户列表筛选（新增 Pending Cancellation / Pending Delete 状态）、§5.5.1 API 操作清单（新增注销发起、审批、撤销、硬删除操作项）。 | > **本 PRD 取代以下两份历史文档**： > - `PRD/系统管理/系统管理模块PRD.md`（v1.3，已删除，被本 PRD 替代） @@ -104,7 +104,7 @@ Fonrey（房睿）是面向房产经纪公司的 B2B SaaS 平台，采用 `djang > 负责 Fonrey 的日常运营，每天处理新客户开通、异常租户处理、客户咨询。使用 PC 浏览器登录平台管理后台。 -#### Story A1：新租户开通 +#### PA-001 Story A1：新租户开通 > 作为运营人员，我希望通过填写表单快速完成租户开通，并由系统自动完成数据库初始化与欢迎通知，无需手动执行脚本。 @@ -133,7 +133,7 @@ Fonrey（房睿）是面向房产经纪公司的 B2B SaaS 平台，采用 `djang - 入口：租户详情 → 基本信息 Tab → 「下载入驻信息」按钮 - 文件名：`{公司名称}_入驻信息_{日期}.pdf` -#### Story A2：挂起问题租户 +#### PA-002 Story A2：挂起问题租户 > 作为运营人员，我希望快速冻结欠费租户的访问，同时保证数据不丢失，并在欠费解决后一键恢复。 @@ -142,7 +142,7 @@ Fonrey（房睿）是面向房产经纪公司的 B2B SaaS 平台，采用 `djang - [ ] 支持设置到期时间，到期后系统自动恢复租户状态，并发送通知邮件 - [ ] 所有挂起 / 恢复操作记录于审计日志，包含操作人、时间、原因 -#### Story A3：License 时效管理与到期自动挂起 +#### PA-003 Story A3：License 时效管理与到期自动挂起 > 作为运营人员（或 Platform Admin），我希望每个租户能设置 License 有效期，到期后系统自动挂起租户，并在到期前提前预警。 @@ -154,7 +154,7 @@ Fonrey（房睿）是面向房产经纪公司的 B2B SaaS 平台，采用 `djang - [ ] 倒计时横幅仅对 Tenant Admin 可见，不影响普通 Agent - [ ] License 到期日期在租户列表与详情页均可见，支持「即将到期（15 天内）」筛选 -#### Story A4：查看与控制租户用户数（License 计费维度） +#### PA-004 Story A4：查看与控制租户用户数（License 计费维度） > 作为运营人员（或 Platform Admin），我希望看到每个租户的当前用户数，并能设置用户数上限，以便根据 License 授权进行管控。 @@ -165,15 +165,63 @@ Fonrey（房睿）是面向房产经纪公司的 B2B SaaS 平台，采用 `djang - [ ] Platform Admin 可随时调整用户数上限（扩容 / 缩容），变更写入审计日志 - [ ] 租户列表支持按「用户数已满（≥ 上限）」筛选，便于平台运营主动识别需续费的租户 -> **Story A5：响应客户数据导出请求** — **状态**：暂缓（v1 不实现，列入后续迭代）。 +> **PA-011 Story A5：响应客户数据导出请求** — **状态**：暂缓（v1 不实现，列入后续迭代）。 --- +### Persona E — Platform Admin David（租户注销合规流程） + +> 处理租户主动申请注销或因严重违规被强制注销的场景。注销操作不可逆（硬删除后），需要多重防护——数据导出存证、双人复核、30 天冷静期撤销窗口。 + +#### PA-011 Story E1：租户注销 SOP（Tenant Cancellation SOP） + +> 作为 Platform Admin，我希望执行租户注销时有标准化的审批流程保障——要求数据导出完成、双人复核授权、30 天冷静期可撤销——防止因误操作或内部滥权导致客户数据不可恢复地丢失。 + +**验收标准（AC）**： + +**AC-1 发起注销申请** +- [ ] 运营人员或 Platform Admin 可在租户详情页发起「注销申请」，填写：注销原因（客户主动申请 / 严重违规 / 其他）、申请说明（自由文本） +- [ ] 发起后，租户状态流转至 `Pending Cancellation`（待审批注销），此时租户访问被阻断（等同挂起效果），不可通过常规「恢复」操作解除 +- [ ] 发起操作写入审计日志 + +**AC-2 数据导出前置确认** +- [ ] 进入审批流程前，系统强制展示「数据导出确认」步骤：要求操作人触发该租户的完整数据导出（§5.1.5），等待导出任务完成（状态 Done）后方可提交审批 +- [ ] 若申请人选择「客户已自行备份，无需平台导出」，需勾选明确确认框，并填写备注（客户签字回执编号或说明），此记录写入审计日志 +- [ ] 审批单中需展示：导出任务 ID（或「已确认客户自备份」备注）、导出完成时间 + +**AC-3 双人复核审批** +- [ ] 注销审批流需要第二名 Platform Admin（不得与申请人为同一账号）在系统内确认审批，确认时需通过 MFA 二次验证 +- [ ] 审批人可「批准」或「驳回」：驳回时必须填写驳回原因；驳回后租户状态恢复至 `Suspended`（仍不可直接恢复为 Active，需运营人员手动解除挂起） +- [ ] 若 48 小时内无人审批，系统向所有 Platform Admin 发送催审通知（站内消息 + 邮件） + +**AC-4 软删除与 30 天冷静期** +- [ ] 审批通过后，租户立即进入软删除状态（`Pending Delete`，现有机制）；冷静期默认 30 天 +- [ ] 冷静期内，Platform Admin 可在租户列表「待清除」筛选下找到该租户，执行「撤销注销」 +- [ ] 撤销注销需第二名 Platform Admin 同样通过 MFA 确认（与初始注销审批相同的双人复核机制） +- [ ] 撤销成功后，租户状态恢复至 `Suspended`（需运营人员手动评估后恢复为 Active）；撤销操作及原因写入审计日志 + +**AC-5 硬删除与资源释放** +- [ ] 冷静期届满后，系统自动触发硬删除，或 Platform Admin 可在冷静期内提前手动触发硬删除（需再次 MFA 确认，并再次展示「此操作不可逆」警告） +- [ ] 硬删除释放：PostgreSQL Schema、Cloudflare R2 存储桶、Tenant Code、License 席位 +- [ ] 硬删除完成后：向租户联系邮箱发送「账号已注销」通知（含注销日期、数据保留期说明）；审计日志记录完整操作链 + +**注销状态流转**（补充至 §7 状态机）： +``` +[活跃 Active] 或 [已挂起 Suspended] + ↓ 运营/Platform Admin 发起注销申请 +[待审批注销 Pending Cancellation] + ↓ 双人 MFA 审批通过 +[待清除 Pending Delete]（冷静期 30 天） + ↓ 可在冷静期内由双人 MFA 撤销 → 恢复至 [已挂起 Suspended] + ↓ 冷静期届满 或 提前手动硬删除（MFA） +[已删除 Deleted] +``` + ### Persona B — Platform Admin David（系统升级、回滚与版本治理） > 负责平台技术运维，周期性执行版本升级，关注升级稳定性与租户影响面。拥有所有高危操作权限。 -#### Story B1：灰度系统升级 +#### PA-005 Story B1：灰度系统升级 > 作为 Platform Admin，我希望先对内测租户升级新版本，验证稳定后再全量推送，避免一次性影响所有客户。 @@ -183,7 +231,7 @@ Fonrey（房睿）是面向房产经纪公司的 B2B SaaS 平台，采用 `djang - [ ] 升级过程实时展示进度（每个租户的升级状态），支持查看升级日志 - [ ] 升级失败时系统自动告警，并提供一键回滚入口 -#### Story B2：升级失败回滚 +#### PA-006 Story B2：升级失败回滚 > 作为 Platform Admin，我希望在升级出现问题时立即回滚至上一稳定版本，并生成事件报告。 @@ -193,7 +241,7 @@ Fonrey（房睿）是面向房产经纪公司的 B2B SaaS 平台，采用 `djang - [ ] 回滚完成后生成事件报告：失败原因、回滚耗时、影响范围 - [ ] 回滚操作需二次身份验证确认（MFA） -#### Story B3：查看平台与租户版本总览 +#### PA-007 Story B3：查看平台与租户版本总览 > 作为 Platform Admin，我希望在管理界面一眼看到整个平台的版本情况，包括基础数据版本和每个租户各自的数据升级版本。 @@ -221,7 +269,7 @@ Fonrey（房睿）是面向房产经纪公司的 B2B SaaS 平台，采用 `djang > 同 Persona B，但聚焦桌面客户端版本上线、强制升级与跨租户版本分布。 -#### Story C1：发布新版本客户端 +#### PA-008 Story C1：发布新版本客户端 > 作为 Platform Admin，我希望通过本后台上传新版客户端安装包并配置版本信息，使全平台所有租户的客户端能感知到更新并引导升级。 @@ -239,7 +287,7 @@ Fonrey（房睿）是面向房产经纪公司的 B2B SaaS 平台，采用 `djang - [ ] 支持版本回滚：将指定历史版本重新置为「已发布」，自动将当前版本标记为「已下线」 - [ ] 支持便携版（Portable ZIP）作为可选上传项 -#### Story C2：跨租户监控客户端版本分布 +#### PA-009 Story C2：跨租户监控客户端版本分布 > 作为 Platform Admin，我希望跨租户查看当前所有在线客户端的版本分布，以了解全平台升级覆盖率，对仍使用旧版本的客户端发出提醒或强制升级。 @@ -259,7 +307,7 @@ Fonrey（房睿）是面向房产经纪公司的 B2B SaaS 平台，采用 `djang > 负责平台合规审查，定期导出操作日志供法务或客户审查。无任何写权限。 -#### Story D1：审计日志查询与导出 +#### PA-010 Story D1：审计日志查询与导出 > 作为审计员，我希望按操作人、时间范围、操作类型筛选操作日志，并导出为报告。 @@ -320,15 +368,18 @@ Fonrey（房睿）是面向房产经纪公司的 B2B SaaS 平台，采用 `djang #### 5.1.4 删除（Delete） +> **注意**：租户注销是高危操作，须遵循「PA-011 租户注销 SOP」——发起注销申请 → 强制数据导出确认 → 双人 MFA 复核审批 → 软删除（30 天冷静期，可撤销）→ 自动或提前手动硬删除。**不允许绕过此 SOP 直接执行删除。** + | 模式 | 说明 | |------|------| -| 软删除 | 标记删除状态，数据保留 30 天（默认，可配置）后自动清除 | -| 硬删除 | 立即清除所有数据、Schema、存储资源；仅 Platform Admin 可操作 | +| 软删除（Pending Delete） | SOP 审批通过后自动进入；数据保留 30 天（默认，可配置）后自动清除 | +| 硬删除 | 冷静期内 Platform Admin 提前触发（需 MFA + 不可逆警告），或冷静期届满自动触发；仅 Platform Admin 可操作 | -**删除前置条件**： -1. 操作人必须确认数据导出已完成（勾选确认框） -2. 硬删除需 MFA 二次验证 -3. 软删除冷静期内可在租户列表中执行「撤销删除」 +**删除前置条件（PA-011 SOP 强制执行）**： +1. 发起申请时必须填写注销原因 +2. 审批前强制完成数据导出确认（平台导出 Done 或客户自备份书面确认） +3. 须第二名 Platform Admin（≠ 申请人）MFA 审批通过 +4. 软删除冷静期内可由双人 MFA 执行「撤销注销」 删除完成后释放：访问域名 / Tenant Code、Cloudflare R2 存储桶、License 席位 @@ -563,7 +614,10 @@ Fonrey（房睿）是面向房产经纪公司的 B2B SaaS 平台，采用 `djang - 「系统版本管理」中点击某租户行 → 跳转到该租户的「备份记录」Tab - 「客户端版本管理」中点击「全平台租户活跃榜」中的租户名称 → 跳转到该租户「基本信息」Tab - 客户端版本「发布 / 下线 / 强制更新推送」操作 → 二次确认；强制更新推送额外要求 MFA -- 任何高危操作（删除租户、数据恢复、系统回滚、客户端版本下线、强制更新推送）触发 MFA 二次确认弹窗 +- 任何高危操作（删除/注销租户、数据恢复、系统回滚、客户端版本下线、强制更新推送）触发 MFA 二次确认弹窗 +- 租户详情「基本信息」页面，Platform Admin / 运营人员可发起「注销申请」；状态为 `Pending Cancellation` 时，原「挂起/恢复」按钮隐藏，显示「查看注销审批单」入口 +- 注销审批待办：系统向第二名 Platform Admin 发送站内通知；点击通知进入审批单详情，展示申请人、注销原因、数据导出确认记录，提供「批准（MFA）」/「驳回」操作 +- 审批批准后，租户自动进入 `Pending Delete`；租户列表「待清除」筛选下可见，提供「撤销注销（双人 MFA）」和「提前硬删除（MFA）」入口 - 用户超过 30 分钟无操作 → 自动登出，下次操作跳转登录页 - 「管理员设置」中 Platform Admin 强制登出某管理员 → 该管理员的所有会话立即失效，下次请求跳转登录页 @@ -582,7 +636,7 @@ Fonrey（房睿）是面向房产经纪公司的 B2B SaaS 平台，采用 `djang - 分页（默认 20 条/页） - 搜索：公司名称、Tenant Code、联系邮箱关键词 -- 筛选：状态（Active / Suspended / Deleted）、套餐、注册时间、**即将到期（15 天内）**、**用户数已满** +- 筛选：状态（Active / Suspended / Pending Cancellation / Pending Delete / Deleted）、套餐、注册时间、**即将到期（15 天内）**、**用户数已满** - 列：公司名称、Tenant Code、套餐、状态、注册时间、活跃用户数 / License 上限、客户端最新版本占比 - 快捷操作：查看详情、挂起、发起备份、数据导出 @@ -599,8 +653,12 @@ Fonrey（房睿）是面向房产经纪公司的 B2B SaaS 平台，采用 `djang - 平台管理员需要能创建新租户（含异步初始化 Schema 与默认配置） - 平台管理员需要能编辑租户基本信息（公司名、联系人、邮箱、地区） - 平台管理员需要能挂起 / 恢复租户（含挂起原因、到期时间） -- 平台管理员需要能软删除 / 硬删除租户（硬删除需 MFA） -- 平台管理员需要能撤销软删除（冷静期内） +- 平台管理员（运营人员）需要能发起租户注销申请（PA-011 SOP：填写原因、完成数据导出确认） +- 第二名 Platform Admin 需要能审批租户注销申请（批准需 MFA；驳回需填写驳回原因） +- 平台管理员需要能在冷静期内撤销租户注销（双人 MFA） +- 平台管理员需要能在冷静期内提前手动触发硬删除（MFA + 不可逆确认） +- 系统需能在冷静期届满后自动触发硬删除并释放所有资源 +- 平台管理员需要能软删除 / 硬删除租户（仅限通过 PA-011 SOP 触发，不提供绕过 SOP 的直接删除入口） - 平台管理员需要能调整租户的 License 到期日期 - 平台管理员需要能调整租户的 License 授权用户数上限 - 平台管理员需要能升级租户套餐 @@ -704,11 +762,14 @@ Fonrey（房睿）是面向房产经纪公司的 B2B SaaS 平台，采用 `djang ## 6. 角色权限矩阵 | 操作 | Platform Admin | 运营人员 | 只读审计员 | -|------|---------------|---------|-----------| +|------|---------------|---------|-----------|| | 创建租户 | ✅ | ✅ | ❌ | | 挂起 / 恢复租户 | ✅ | ✅ | ❌ | -| 软删除租户 | ✅ | ✅ | ❌ | -| 硬删除租户 | ✅ | ❌ | ❌ | +| 发起租户注销申请（PA-011 SOP） | ✅ | ✅ | ❌ | +| 审批注销申请（第二人复核，MFA） | ✅ | ❌ | ❌ | +| 撤销注销（冷静期内，双人 MFA） | ✅ | ❌ | ❌ | +| 提前手动硬删除（MFA） | ✅ | ❌ | ❌ | +| 软删除 → Pending Delete（仅经 SOP 自动触发） | 系统自动 | — | — | | 调整 License 到期日期 / 用户数上限 | ✅ | ✅ | ❌ | | 数据导出 | ✅ | ✅ | ❌ | | 手动触发备份 | ✅ | ✅ | ❌ | @@ -737,12 +798,18 @@ Fonrey（房睿）是面向房产经纪公司的 B2B SaaS 平台，采用 `djang [活跃 Active] ←──────────────────┐ ↓ 手动 / License 到期挂起 │ 到期自动恢复（仅手动挂起且设置了到期时间）/ 手动恢复 [已挂起 Suspended] ───────────────┘ - ↓ 删除操作（软删除） + ↓ 运营/PA 发起注销申请（PA-011） +[待审批注销 Pending Cancellation] + ↓ 驳回（第二名 PA）→ [已挂起 Suspended] + ↓ 批准（第二名 PA，MFA） [待清除 Pending Delete]（冷静期 30 天） - ↓ 冷静期到期 / 硬删除 + ↓ 双人 MFA 撤销注销 → [已挂起 Suspended] + ↓ 冷静期届满（自动）/ 提前手动硬删除（MFA） [已删除 Deleted] ``` +> **注意**：Active / Suspended 状态下均可发起注销申请（后者更常见）。不允许直接跳过 SOP 从任何状态进入 Pending Delete 或 Deleted。 + --- ## 8. 不构建清单（What We're NOT Building） diff --git a/Project/fonrey/REVIEW/REVIEW_全局_2026-04-25.md b/Project/fonrey/REVIEW/REVIEW_全局_2026-04-25.md deleted file mode 100644 index 6ee5084f..00000000 --- a/Project/fonrey/REVIEW/REVIEW_全局_2026-04-25.md +++ /dev/null @@ -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* diff --git a/Project/fonrey/REVIEW/REVIEW_全局_2026-04-26.md b/Project/fonrey/REVIEW/REVIEW_全局_2026-04-26.md deleted file mode 100644 index 9ee0c148..00000000 --- a/Project/fonrey/REVIEW/REVIEW_全局_2026-04-26.md +++ /dev/null @@ -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* diff --git a/Project/fonrey/REVIEW/REVIEW_全局_2026-04-28.md b/Project/fonrey/REVIEW/REVIEW_全局_2026-04-28.md deleted file mode 100644 index 4e03c6e4..00000000 --- a/Project/fonrey/REVIEW/REVIEW_全局_2026-04-28.md +++ /dev/null @@ -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: ` 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）：禁止业务模块直接调用 `.objects.` - - 测试规范补"权限边界测试矩阵"：每个 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 结束。 diff --git a/Project/fonrey/REVIEW/REVIEW_登录管理_平台管理后台_2026-05-02.md b/Project/fonrey/REVIEW/REVIEW_登录管理_平台管理后台_2026-05-02.md new file mode 100644 index 00000000..3a1689bf --- /dev/null +++ b/Project/fonrey/REVIEW/REVIEW_登录管理_平台管理后台_2026-05-02.md @@ -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 类，未与组件清单中的 `