council(review): SecurityEngineer - merge Task 13 audit report into main plan

Task 13 complete: reviews/SecurityEngineer-AUDIT.md
- Confirms BackendArchitect root cause findings (P0: Line 77, P1: Line 71)
- Adds PHP 8 compatibility note on null[key] TypeError
- Provides complete fix code

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

plan.md

@@ -32,15 +32,23 @@ Undefined array key "id"
 - [x] [Done: council/DebugAgent] **Task 11**: Round 2 — 编写 DebugAgent 最终根因报告 → `reviews/DebugAgent-ROOT_CAUSE.md`
 - [x] [Done: council/BackendArchitect] **Task 12**: Round 2 — 评审 DebugAgent ROOT_CAUSE 报告 → `reviews/BackendArchitect-on-DebugAgent-ROOT_CAUSE.md`
 
+- [x] [Done: council/SecurityEngineer] **Task 13**: Round 2 — 独立安全审计（6项子任务）→ `reviews/SecurityEngineer-AUDIT.md`
+  - Q1: "Undefined array key 'id'" 最可能出现的行 → Primary: Line 77
+  - Q2: Db::name() 表前缀行为 → 等价，排除
+  - Q3: find() 返回 null 处理 → Secondary: Line 71
+  - Q4: $configs JSON 解码类型安全 → 部分安全
+  - Q5: selected_rooms 数据结构 → 类型正确但无空安全
+  - Q6: BatchGenerate + item_type → 安全
+
 ---
 
 ## 阶段划分
 
 | 阶段 | 内容 |
 |------|------|
-| **Draft** | ✅ Task 1-6（BackendArchitect）+ Task 9（DebugAgent）|
+| **Draft** | ✅ Task 1-6（BackendArchitect）+ Task 9（DebugAgent）+ Task 13（SecurityEngineer）|
 | **Review** | ✅ Task 7（BackendArchitect）+ Task 11（DebugAgent）+ Task 12（BackendArchitect）|
-| **Finalize** | ✅ Task 8 + Task 12：所有评审报告输出完毕 |
+| **Finalize** | ✅ Task 8 + Task 12 + Task 13：所有评审报告输出完毕 |
 
 ---
 
@@ -51,6 +59,7 @@ Undefined array key "id"
 3. **Tertiary（静默）**: `AdminGoodsSaveHandle.php:77` — `selected_rooms` 类型不匹配，`in_array` 永远 false
 4. **已排除**: 表前缀问题 — `Db::name()` 和 `BaseService::table()` 均查询 `vrt_vr_seat_templates`，等价
 5. **已排除**: SeatSkuService::BatchGenerate — 第 100 行已有 `!empty()` 空安全 fallback
+6. **SecurityEngineer 补充**: PHP 8+ 中 `null['key']` 抛出 `TypeError`（非 Warning）；`$configs` JSON 解码有 `is_array` 防御；`item_type` 有 `?? ''` 兜底；修复建议已在 `reviews/SecurityEngineer-AUDIT.md`
 
 ## DebugAgent 补充结论（Round 1）