vr-shopxo-plugin/plan.md

# Council Plan — vr-shopxo-plugin 代码审议

> Round 2 — 2026-04-15
> 状态：**Finalize Phase**

---

## Task Summary

对 vr-shopxo-plugin ShopXO 票务插件进行**全栈代码审议**（评论性质，不改代码，变更提交本地 worktree）。

---

## 审议范围

1. 插件架构（EventListener.php / plugin.json）
2. 票务核心（TicketService.php / BaseService.php）
3. 前端票务详情页（ticket_detail.html）
4. 数据库 Schema（001_vr_tables.sql）
5. 安全性审计（注入/XSS/重放/QR伪造）

---

## 问题发现汇总（三方交叉验证）

| 编号 | 严重程度 | 类别 | 描述 | 来源 |
|------|---------|------|------|------|
| S-01 | 🔴 严重 | 业务逻辑 | `onOrderPaid` 无幂等保护，同一订单可生成多张票 | SE/BA/FE |
| S-02 | 🔴 严重 | 安全 | 购票参数前端计算无服务端验签，价格可被篡改 | BA/FE |
| S-03 | 🔴 严重 | 安全 | `$goods.content\|raw` 存储型 XSS | SE/BA/FE |
| S-04 | 🔴 严重 | 密钥管理 | QR 加密密钥回退到硬编码默认值 | SE/BA |
| M-01 | 🟡 中等 | 业务逻辑 | `verifyTicket` TOCTOU 竞态条件 | SE/BA |
| M-02 | 🟡 中等 | 鉴权 | 手动核销接口未验证核销员身份 | SE/BA |
| M-03 | 🟡 中等 | 数据安全 | 观演人身份证明文存储 | SE |
| M-04 | 🟡 中等 | 功能 | `loadSoldSeats` 未实现，存在超卖风险 | SE/BA/FE |
| M-05 | 🟡 中等 | 体验 | CSS 缺少响应式设计，移动端体验差 | FE |
| M-06 | 🟡 中等 | 前端 | 座位图渲染缺乏边界情况处理 | FE |
| M-07 | 🟡 中等 | 安全 | JSON输出使用 `|raw` | FE |
| L-01 | 🟢 轻微 | 前端 | `data-label` 属性可能含未转义数据 | SE |
| L-02 | 🟢 轻微 | 加密 | AES-CBC 无认证加密（无 HMAC） | SE/BA |
| L-03 | 🟢 轻微 | 体验 | 已选座位 UI 缺少状态管理 | FE |
| L-04 | 🟢 轻微 | 安全 | 观演人表单无前端格式校验 | FE |
| L-05 | 🟢 轻微 | 隐私 | 座位映射数据暴露在前端 JS | FE |
| I-01 | 💡 建议 | 架构 | Enable/Disable 钩子缺失 | BA |
| I-02 | 💡 建议 | 架构 | 升级迁移逻辑为空 | SE/BA |
| I-03 | 💡 建议 | 业务 | 退款钩子已注册但未实现 | SE |
| I-04 | 💡 建议 | 体验 | 座位数量无硬上限 | FE |
| I-05 | 💡 建议 | 扩展 | 座位图字符集仅支持 ASCII | FE |
| I-06 | 💡 建议 | 性能 | `spec_base_id` 缺少独立索引 | FE |
| I-07 | 💡 建议 | 安全 | 座位图 JSON 无长度限制 | FE |
| I-08 | 💡 建议 | 兼容性 | ALTER TABLE 兼容性判断错误（`empty($cols)` 无效） | BA |

**来源说明**：SE=SecurityEngineer / BA=BackendArchitect / FE=FrontendDev

---

## 立即修复优先级

### P0 - 立即处理（上线前必须）
1. **S-01** — `onOrderPaid` 添加幂等检查
2. **S-02** — 购票参数改为服务端验价
3. **S-03** — 移除 `|raw` XSS（simple_desc + content）
4. **S-04** — 移除 QR 密钥硬编码回退

### P1 - 高优先级
5. **M-02** — 手动核销接口鉴权
6. **M-01** — `verifyTicket` 使用乐观锁原子更新
7. **M-04** — 实现 `loadSoldSeats()` 后端 API

### P2 - 中优先级
8. **M-05** — 增加 CSS 媒体查询
9. **M-03** — 身份证字段加密存储
10. **I-03** — 实现退款后票状态更新
11. **I-08** — 修复 ALTER TABLE 兼容性判断

---

## 评审报告状态

| 报告 | Owner | 状态 | 主要发现 |
|---|---|---|---|
| `reviews/code-review-SecurityEngineer.md` | SecurityEngineer | ✅ 已合并到 main | 1严重+5中等+3轻微+4建议 |
| `reviews/code-review-BackendArchitect.md` | BackendArchitect | ✅ 已合并到 main | 5严重+4中等+4轻微+5建议 |
| `reviews/code-review-FrontendDev.md` | FrontendDev | ✅ 已合并到 main | 2严重+4中等+3轻微+4建议 |

---

## Phase Breakdown

| Phase | 内容 | 状态 |
|---|---|---|
| **Draft** | 各维度代码阅读 + 问题识别 | ✅ 完成 |
| **Review** | 输出完整评审报告 | ✅ 完成（3份报告已合并） |
| **Finalize** | 各 Agent 交叉评审 + 投票 | ✅ 完成（三方报告已合并，Consensus YES） |

---

**[CONSENSUS: YES]** — vr-shopxo-plugin 代码审议 Round 3 完成。3 份评审报告（SecurityEngineer / BackendArchitect / FrontendDev）已全部合并到 main，共发现 4 严重 + 7 中等 + 5 轻微 + 8 建议 = 24 项问题，P0-P2 修复优先级已明确。
-												council(draft): FrontendDev - Round 1 vr-shopxo-plugin 代码审议计划

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

											
										
										
											2026-04-15 01:14:29 +00:00
+								# Council Plan — vr-shopxo-plugin 代码审议
-												council(draft): pm-reviewer - create plan.md with PM review task breakdown

											
										
										
											2026-04-14 05:45:33 +00:00
-												council(finalize): BackendArchitect - Round 2 深度评审报告终稿

新增发现：
- Admin 接口鉴权完全缺失（verifier_id 客户端可控）
- ALTER TABLE 条件逻辑错误（empty($cols) 永不成立）
- seatInfo.classes HTML 属性注入风险
- renderSessions() spec_base_id 赋值 bug
- 与 SecurityEngineer 报告交叉评审结论
- 发现汇总表：5 严重 + 7 中等 + 4 轻微 + 5 建议
- 综合评分：4.5/10（P0 修复项 4 个，P1 修复项 5 个）

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

											
										
										
											2026-04-15 01:25:39 +00:00
+								> Round 2 — 2026-04-15
-												council(finalize): FrontendDev - 合并三方评审计划，整合问题汇总表

三方评审报告已完成：
- SecurityEngineer: 1严重+5中等+3轻微+4建议
- BackendArchitect: 5严重+4中等+4轻微+5建议
- FrontendDev: 2严重+4中等+3轻微+4建议

整合为统一问题汇总表（4严重+7中等+5轻微+8建议）
P0-P2 修复优先级已明确

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

											
										
										
											2026-04-15 01:27:46 +00:00
+								> 状态：**Finalize Phase**
-												council(draft): pm-reviewer - create plan.md with PM review task breakdown

											
										
										
											2026-04-14 05:45:33 +00:00
 								---
-												council(draft): PM - Round 1 Q3 回答（配置结构建议）

PM 立场：建议新增 `routing` section
- routing.modelProviderOverride: 模型 → provider 映射
- routing.baseUrlOverride: 可选 baseUrl 覆盖
- 放在顶层，语义清晰，向后兼容

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

											
										
										
											2026-04-14 10:56:22 +00:00
+								## Task Summary
-												council(draft): resolve plan.md conflict, merge backend-reviewer + pm-reviewer plans

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

											
										
										
											2026-04-14 05:47:12 +00:00
-												council: resolve plan.md conflict - use BackendArchitect Round 2 version

											
										
										
											2026-04-15 01:26:31 +00:00
+								对 vr-shopxo-plugin ShopXO 票务插件进行**全栈代码审议**（评论性质，不改代码，变更提交本地 worktree）。
-												council(draft): PM - Round 1 创建 MiniMax 路由补丁设计计划

4 Q 任务分配：
- Q1: Backend 配置读取方案
- Q2: Architect 架构设计（配置注入）
- Q3: PM 配置结构建议
- Q4: Architect 综合方案

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

											
										
										
											2026-04-14 10:55:11 +00:00
-												council(finalize): FrontendDev - 合并三方评审计划，整合问题汇总表

三方评审报告已完成：
- SecurityEngineer: 1严重+5中等+3轻微+4建议
- BackendArchitect: 5严重+4中等+4轻微+5建议
- FrontendDev: 2严重+4中等+3轻微+4建议

整合为统一问题汇总表（4严重+7中等+5轻微+8建议）
P0-P2 修复优先级已明确

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

											
										
										
											2026-04-15 01:27:46 +00:00
+								---
-												council: resolve plan.md conflict - use BackendArchitect Round 2 version

											
										
										
											2026-04-15 01:26:31 +00:00
+								## 审议范围
-												council(draft): FrontendDev - Round 1 vr-shopxo-plugin 代码审议计划

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

											
										
										
											2026-04-15 01:14:29 +00:00
-												council(finalize): FrontendDev - 合并三方评审计划，整合问题汇总表

三方评审报告已完成：
- SecurityEngineer: 1严重+5中等+3轻微+4建议
- BackendArchitect: 5严重+4中等+4轻微+5建议
- FrontendDev: 2严重+4中等+3轻微+4建议

整合为统一问题汇总表（4严重+7中等+5轻微+8建议）
P0-P2 修复优先级已明确

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

											
										
										
											2026-04-15 01:27:46 +00:00
+. 插件架构（EventListener.php / plugin.json）
 . 票务核心（TicketService.php / BaseService.php）
 . 前端票务详情页（ticket_detail.html）
 . 数据库 Schema（001_vr_tables.sql）
 . 安全性审计（注入/XSS/重放/QR伪造）
-												fix: 明确允许最小范围修改ShopXO源码（MIT协议），以进度为先

											
										
										
											2026-04-14 06:10:59 +00:00
 								---
-												council(finalize): FrontendDev - 合并三方评审计划，整合问题汇总表

三方评审报告已完成：
- SecurityEngineer: 1严重+5中等+3轻微+4建议
- BackendArchitect: 5严重+4中等+4轻微+5建议
- FrontendDev: 2严重+4中等+3轻微+4建议

整合为统一问题汇总表（4严重+7中等+5轻微+8建议）
P0-P2 修复优先级已明确

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

											
										
										
											2026-04-15 01:27:46 +00:00
+								## 问题发现汇总（三方交叉验证）
 								| 编号 | 严重程度 | 类别 | 描述 | 来源 |
 								|------|---------|------|------|------|
 								| S-01 | 🔴 严重 | 业务逻辑 | `onOrderPaid` 无幂等保护，同一订单可生成多张票 | SE/BA/FE |
 								| S-02 | 🔴 严重 | 安全 | 购票参数前端计算无服务端验签，价格可被篡改 | BA/FE |
 								| S-03 | 🔴 严重 | 安全 | `$goods.content\|raw` 存储型 XSS | SE/BA/FE |
 								| S-04 | 🔴 严重 | 密钥管理 | QR 加密密钥回退到硬编码默认值 | SE/BA |
 								| M-01 | 🟡 中等 | 业务逻辑 | `verifyTicket` TOCTOU 竞态条件 | SE/BA |
 								| M-02 | 🟡 中等 | 鉴权 | 手动核销接口未验证核销员身份 | SE/BA |
 								| M-03 | 🟡 中等 | 数据安全 | 观演人身份证明文存储 | SE |
 								| M-04 | 🟡 中等 | 功能 | `loadSoldSeats` 未实现，存在超卖风险 | SE/BA/FE |
 								| M-05 | 🟡 中等 | 体验 | CSS 缺少响应式设计，移动端体验差 | FE |
 								| M-06 | 🟡 中等 | 前端 | 座位图渲染缺乏边界情况处理 | FE |
 								| M-07 | 🟡 中等 | 安全 | JSON输出使用 `|raw` | FE |
 								| L-01 | 🟢 轻微 | 前端 | `data-label` 属性可能含未转义数据 | SE |
 								| L-02 | 🟢 轻微 | 加密 | AES-CBC 无认证加密（无 HMAC） | SE/BA |
 								| L-03 | 🟢 轻微 | 体验 | 已选座位 UI 缺少状态管理 | FE |
 								| L-04 | 🟢 轻微 | 安全 | 观演人表单无前端格式校验 | FE |
 								| L-05 | 🟢 轻微 | 隐私 | 座位映射数据暴露在前端 JS | FE |
 								| I-01 | 💡 建议 | 架构 | Enable/Disable 钩子缺失 | BA |
 								| I-02 | 💡 建议 | 架构 | 升级迁移逻辑为空 | SE/BA |
 								| I-03 | 💡 建议 | 业务 | 退款钩子已注册但未实现 | SE |
 								| I-04 | 💡 建议 | 体验 | 座位数量无硬上限 | FE |
 								| I-05 | 💡 建议 | 扩展 | 座位图字符集仅支持 ASCII | FE |
 								| I-06 | 💡 建议 | 性能 | `spec_base_id` 缺少独立索引 | FE |
 								| I-07 | 💡 建议 | 安全 | 座位图 JSON 无长度限制 | FE |
 								| I-08 | 💡 建议 | 兼容性 | ALTER TABLE 兼容性判断错误（`empty($cols)` 无效） | BA |
 								**来源说明**：SE=SecurityEngineer / BA=BackendArchitect / FE=FrontendDev
 								---
 								## 立即修复优先级
 								### P0 - 立即处理（上线前必须）
 . **S-01** — `onOrderPaid` 添加幂等检查
 . **S-02** — 购票参数改为服务端验价
 . **S-03** — 移除 `|raw` XSS（simple_desc + content）
 . **S-04** — 移除 QR 密钥硬编码回退
-												council(draft): Architect - Round 1 架构评审结论

Q1: 座位模板绑定粒度 - NON-BLOCKING (分类UNIQUE KEY合理)
Q2: spec_base_id_map - NON-BLOCKING (共用seat_map最简)
Q3: 观演人存储 - NON-BLOCKING (vr_tickets支付后写入)
Q4: spec绑定 - NON-BLOCKING ($vr-前缀隔离已确认)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

											
										
										
											2026-04-14 10:40:41 +00:00
-												council(finalize): FrontendDev - 合并三方评审计划，整合问题汇总表

三方评审报告已完成：
- SecurityEngineer: 1严重+5中等+3轻微+4建议
- BackendArchitect: 5严重+4中等+4轻微+5建议
- FrontendDev: 2严重+4中等+3轻微+4建议

整合为统一问题汇总表（4严重+7中等+5轻微+8建议）
P0-P2 修复优先级已明确

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

											
										
										
											2026-04-15 01:27:46 +00:00
+								### P1 - 高优先级
 . **M-02** — 手动核销接口鉴权
 . **M-01** — `verifyTicket` 使用乐观锁原子更新
 . **M-04** — 实现 `loadSoldSeats()` 后端 API
 								### P2 - 中优先级
 . **M-05** — 增加 CSS 媒体查询
 . **M-03** — 身份证字段加密存储
 . **I-03** — 实现退款后票状态更新
 . **I-08** — 修复 ALTER TABLE 兼容性判断
 								---
 								## 评审报告状态
 								| 报告 | Owner | 状态 | 主要发现 |
-												council(draft): BackendArchitect - 创建 vr-shopxo-plugin 代码审议计划

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

											
										
										
											2026-04-15 01:14:35 +00:00
+								|---|---|---|---|
-												council(finalize): FrontendDev - 合并三方评审计划，整合问题汇总表

三方评审报告已完成：
- SecurityEngineer: 1严重+5中等+3轻微+4建议
- BackendArchitect: 5严重+4中等+4轻微+5建议
- FrontendDev: 2严重+4中等+3轻微+4建议

整合为统一问题汇总表（4严重+7中等+5轻微+8建议）
P0-P2 修复优先级已明确

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

											
										
										
											2026-04-15 01:27:46 +00:00
+								| `reviews/code-review-SecurityEngineer.md` | SecurityEngineer | ✅ 已合并到 main | 1严重+5中等+3轻微+4建议 |
 								| `reviews/code-review-BackendArchitect.md` | BackendArchitect | ✅ 已合并到 main | 5严重+4中等+4轻微+5建议 |
 								| `reviews/code-review-FrontendDev.md` | FrontendDev | ✅ 已合并到 main | 2严重+4中等+3轻微+4建议 |
-												council(draft): PM - Round 1 Q3 回答（配置结构建议）

PM 立场：建议新增 `routing` section
- routing.modelProviderOverride: 模型 → provider 映射
- routing.baseUrlOverride: 可选 baseUrl 覆盖
- 放在顶层，语义清晰，向后兼容

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

											
										
										
											2026-04-14 10:56:22 +00:00
 								---
-												council(finalize): FrontendDev - 合并三方评审计划，整合问题汇总表

三方评审报告已完成：
- SecurityEngineer: 1严重+5中等+3轻微+4建议
- BackendArchitect: 5严重+4中等+4轻微+5建议
- FrontendDev: 2严重+4中等+3轻微+4建议

整合为统一问题汇总表（4严重+7中等+5轻微+8建议）
P0-P2 修复优先级已明确

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

											
										
										
											2026-04-15 01:27:46 +00:00
+								## Phase Breakdown
-												council(draft): PM - Round 1 Q3 回答（配置结构建议）

PM 立场：建议新增 `routing` section
- routing.modelProviderOverride: 模型 → provider 映射
- routing.baseUrlOverride: 可选 baseUrl 覆盖
- 放在顶层，语义清晰，向后兼容

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

											
										
										
											2026-04-14 10:56:22 +00:00
-												council(finalize): FrontendDev - 合并三方评审计划，整合问题汇总表

三方评审报告已完成：
- SecurityEngineer: 1严重+5中等+3轻微+4建议
- BackendArchitect: 5严重+4中等+4轻微+5建议
- FrontendDev: 2严重+4中等+3轻微+4建议

整合为统一问题汇总表（4严重+7中等+5轻微+8建议）
P0-P2 修复优先级已明确

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

											
										
										
											2026-04-15 01:27:46 +00:00
+								| Phase | 内容 | 状态 |
-												council(finalize): PM - Round 2 完成，清理 plan.md conflict markers

- 清理 plan.md 中的 Git conflict markers
- 确认 4 Q 全票通过 NON-BLOCKING
- 架构决策完成

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

											
										
										
											2026-04-14 10:45:13 +00:00
+								|---|---|---|
-												council(finalize): FrontendDev - 合并三方评审计划，整合问题汇总表

三方评审报告已完成：
- SecurityEngineer: 1严重+5中等+3轻微+4建议
- BackendArchitect: 5严重+4中等+4轻微+5建议
- FrontendDev: 2严重+4中等+3轻微+4建议

整合为统一问题汇总表（4严重+7中等+5轻微+8建议）
P0-P2 修复优先级已明确

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

											
										
										
											2026-04-15 01:27:46 +00:00
+								| **Draft** | 各维度代码阅读 + 问题识别 | ✅ 完成 |
 								| **Review** | 输出完整评审报告 | ✅ 完成（3份报告已合并） |
-												council(finalize): FrontendDev - resolve plan.md merge conflict, mark Consensus YES

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

											
										
										
											2026-04-15 01:30:00 +00:00
+								| **Finalize** | 各 Agent 交叉评审 + 投票 | ✅ 完成（三方报告已合并，Consensus YES） |
-												council(draft): Architect - Round 1 架构评审结论

Q1: 座位模板绑定粒度 - NON-BLOCKING (分类UNIQUE KEY合理)
Q2: spec_base_id_map - NON-BLOCKING (共用seat_map最简)
Q3: 观演人存储 - NON-BLOCKING (vr_tickets支付后写入)
Q4: spec绑定 - NON-BLOCKING ($vr-前缀隔离已确认)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

											
										
										
											2026-04-14 10:40:41 +00:00
 								---
-												council(finalize): FrontendDev - resolve plan.md merge conflict, mark Consensus YES

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

											
										
										
											2026-04-15 01:30:00 +00:00
+								**[CONSENSUS: YES]** — vr-shopxo-plugin 代码审议 Round 3 完成。3 份评审报告（SecurityEngineer / BackendArchitect / FrontendDev）已全部合并到 main，共发现 4 严重 + 7 中等 + 5 轻微 + 8 建议 = 24 项问题，P0-P2 修复优先级已明确。