diff --git a/plan.md b/plan.md index cb8ae2a..9e34e8d 100644 --- a/plan.md +++ b/plan.md @@ -1,91 +1,80 @@ -# Plan — 调研「场馆删除后编辑商品出现规格重复错误」问题 +# Plan — ShopXO 酷炫前端模板实现方案调研 -> 版本：v1.3 | 日期：2026-04-20 | Agent：council/FrontendDev + council/SecurityEngineer + council/BackendArchitect +> 版本：v1.0 | 日期：2026-04-20 | Agent：council/ProductManager + council/FrontendDev + council/BackendArchitect + council/FirstPrinciples --- ## 任务概述 -当票务商品关联的场馆模板被硬删除后，编辑商品时出现「规格不允许重复」错误。 +vr-shopxo-plugin 项目推进 Phase 3 前端模板调研，聚焦 4 个方向： +- Q1：ShopXO 自定义模板最佳实践 +- Q2：单订单多 SKU 支持（多座位选择前提） +- Q3：第三方无代码构建服务提示词策略 +- Q4：uni-app 兼容性技术栈选型 -**根因调查分工**： -- FrontendDev：前端规格项构建与 fallback 行为 -- BackendArchitect：后端规格去重逻辑、`spec_base_id_map` 解析 -- SecurityEngineer：安全风险评估（P1 vs P2） +**输出目标**：`docs/council-research-output.md` --- -## FrontendDev 任务清单 +## 任务清单 -- [x] [Done: council/FrontendDev] **Task 1**: 读取 `ticket_detail.html`，分析前端构建规格项的过程 -- [x] [Done: council/FrontendDev] **Task 2**: 当模板不存在时，前端如何处理 `template_snapshot` 和 `spec_base_id_map`？ -- [x] [Done: council/FrontendDev] **Task 3**: `loadSoldSeats()` 函数实际实现了吗？soldSeats 数据如何填充？ -- [x] [Done: council/FrontendDev] **Task 4**: 编辑模式下（已有 vr_goods_config），前端是否正确处理已删除场馆的旧规格？ -- [x] [Done: council/FrontendDev] **Task 5**: 给出前端根因分析（含具体文件路径和行号） -- [x] [Done: council/FrontendDev] **Task 6**: 给出修复方案 -- [x] [Done: council/FrontendDev] **Task 7**: 将调研报告写入 `reviews/council-ghost-spec-FrontendDev.md` +### 全体 Round 1（规划并行，限时 2-3 分钟） + +- [ ] [Claimed: council/ProductManager] **Task P1**: ProductManager 创建本 plan.md 并 merge main +- [ ] [ ] **Task F1**: FrontendDev — 分析 `ticket_detail.html` 现有结构，制定 UI 改进方案 +- [ ] **Task B1**: BackendArchitect — 分析 ShopXO 订单模型是否支持单订单多 SKU +- [ ] **Task S1**: FirstPrinciples — 拍板 Q2 结论，识别最大技术风险 + +### 全体 Round 2（执行调研） + +- [ ] [ ] **Task P2**: ProductManager — 综合 Q1/Q3/Q4 结论，输出 `council-research-output.md` +- [ ] **Task F2**: FrontendDev — 输出 H5 模板技术栈选型报告 → `docs/frontend-template-research.md` +- [ ] **Task B2**: BackendArchitect — 输出 ShopXO 多 SKU 调研报告 → `docs/backend-multi-sku-research.md` +- [ ] **Task S2**: FirstPrinciples — 评审所有报告，给出最终拍板结论 + +### 全体 Round 3（收敛） + +- [ ] [ ] **Task P3**: ProductManager — 整合所有输出到 `council-research-output.md`，merge main +- [ ] [ ] 所有 Agent 投票 `[CONSENSUS: YES/NO]` --- -## SecurityEngineer 任务清单 +## 依赖关系 -- [x] [Done: council/SecurityEngineer] **Task S1**: 读取 AdminGoodsSaveHandle.php — 安全审计：保存时是否拒绝脏数据 -- [x] [Done: council/SecurityEngineer] **Task S2**: 读取 SeatSkuService.php — 幽灵 spec 注入路径分析 -- [x] [Done: council/SecurityEngineer] **Task S3**: 读取 AdminGoodsSave.php — ShopXO 入口安全检查 -- [x] [Done: council/SecurityEngineer] **Task S4**: 输出安全审计报告 → `reviews/SecurityEngineer-GHOST_SPEC_SECURITY.md` -- [x] [Done: council/SecurityEngineer] **Task S5**: 更新 `reviews/council-ghost-spec-summary.md` +``` +Q2结论 ──→ Q4是否能做多座位选择 +Q1结论 ──→ Q3/Q4技术栈基础 +Q3/Q4 ──→ 最小可行方案 vs 理想方案 +``` ---- - -## BackendArchitect 任务清单 - -- [x] [Done: council/BackendArchitect] **Task B1**: 读取 AdminGoodsSaveHandle.php，找出 `vr_goods_config` 的读取和解析逻辑 -- [x] [Done: council/BackendArchitect] **Task B2**: 找出 `spec_base_id_map` 如何被转换成规格项 -- [x] [Done: council/BackendArchitect] **Task B3**: 当 `template_id` 指向不存在的场馆时，SeatSkuService.php 的 GetGoodsViewData 如何 fallback？ -- [x] [Done: council/BackendArchitect] **Task B4**: 幽灵 spec 是在哪个环节产生的？是否在保存时过滤？ -- [x] [Done: council/BackendArchitect] **Task B5**: 商品保存时规格去重逻辑在哪里？`vr_goods_config` 中若有多个规格项的 `spec_base_id` 相同会怎样？ -- [x] [Done: council/BackendArchitect] **Task B6**: 给出根因分析（含具体行号）和修复方案 -- [x] [Done: council/BackendArchitect] **Task B7**: 将调研报告写入 `reviews/council-ghost-spec-BackendArchitect.md` +**关键风险**：Q2（多SKU）结论将直接影响 Q4 多座位选择能否落地。 --- ## 阶段划分 -| 阶段 | 状态 | -|------|------| -| **Draft** | ✅ 完成（所有 Agent 完成文件读取和分析）| -| **Review** | ✅ 完成（各 Agent 已提交调研报告）| -| **Finalize** | ✅ 完成（summary.md 写入，含 BackendArchitect 最终报告）| - ---- - -## 关键文件（必须全部检查） - -| 文件 | 关注点 | -|------|--------| -| `shopxo/app/plugins/vr_ticket/view/goods/ticket_detail.html` | 前端规格项构建、template_snapshot fallback | -| `shopxo/app/plugins/vr_ticket/service/SeatSkuService.php` | GetGoodsViewData，模板不存在时的 fallback | -| `shopxo/app/plugins/vr_ticket/hook/AdminGoodsSaveHandle.php` | 商品保存钩子，vr_goods_config 处理 | -| `shopxo/app/plugins/vr_ticket/admin/Admin.php` | VenueDelete 硬删除逻辑 | -| `shopxo/app/admin/hook/AdminGoodsSave.php` | ShopXO 商品保存钩子入口 | +| 阶段 | 状态 | 说明 | +|------|------|------| +| **Draft** | 🔄 进行中 | Round 1 — 各 Agent 并行规划 | +| **Research** | ⬜ 待开始 | Round 2 — 执行调研 | +| **Finalize** | ⬜ 待开始 | Round 3 — 收敛共识 | --- ## 输出文件 -| 文件 | Agent | 状态 | -|------|-------|------| -| `reviews/council-ghost-spec-FrontendDev.md` | FrontendDev | ✅ | -| `reviews/SecurityEngineer-GHOST_SPEC_SECURITY.md` | SecurityEngineer | ✅ | -| `reviews/council-ghost-spec-BackendArchitect.md` | BackendArchitect | ✅ | -| `reviews/council-ghost-spec-summary.md` | SecurityEngineer | ✅ (v2.1 — 含 BackendArchitect 报告) | +| 文件 | Agent | 截止轮次 | +|------|-------|---------| +| `docs/council-research-output.md` | ProductManager | Round 3 | +| `docs/frontend-template-research.md` | FrontendDev | Round 2 | +| `docs/backend-multi-sku-research.md` | BackendArchitect | Round 2 | +| `docs/firstprinciples-verdict.md` | FirstPrinciples | Round 2 | --- -## 根因结论（BackendArchitect 验证） +## 关键文件参考 -| 优先级 | 根因 | 文件:行号 | -|--------|------|-----------| -| **P1** | 无效 config 块未从数组移除，`continue` 后脏数据写回 DB | AdminGoodsSaveHandle.php:88-89 + 148-150 | -| **P2** | GetGoodsViewData 单模板模式，多模板时覆盖有效块 | SeatSkuService.php:368 + 386-388 | -| **P3** | BatchGenerate 对无效 template_id 返回 code=-2，阻断保存 | AdminGoodsSaveHandle.php:164-170 | +- `docs/12_UNIAPP_FRONTEND_RESEARCH.md` — 现有 uni-app 调研（需更新） +- `docs/14_TEMPLATE_RENDER_INVESTIGATION.md` — 现有模板渲染调研 +- `shopxo/app/plugins/vr_ticket/view/goods/ticket_detail.html` — 当前模板 +- `docs/02_FRONTEND_CUSTOMIZATION.md` — 前端定制历史文档