fix(review): Code Review C1/C2/I2/M2 修補

C1 drift_interpreter: 寫死 192.168.0.111 → settings.OLLAMA_URL
  違反 feedback_frontend_internal_ip_ban 鐵律（後端 service 層同樣禁止寫死內網 IP）

C2 km_conversion_service: BUG-004 補同步 Redis Working Memory vectorized 欄位
  原修復只更新 DB，Redis incident:{id} JSON 的 vectorized 未同步
  → 審計查 Redis 仍顯示 False，fly-wheel 閉環指標仍不準
  修復：DB 更新後 GET → JSON patch vectorized=True → SET（保留原 TTL）

I2 decision_manager: _ALERTNAME_KEYWORDS HostHighDiskUsage→HostOutOfDiskSpace
  + 補 DockerContainerExited
  + fallback 路徑加 debug log

M2 decision_manager: import json as _json 從 for 迴圈移至方法頂部

docs: ADR-072 新增 Code Review 發現與技術債記錄

2026-04-11 Claude Sonnet 4.6 Asia/Taipei

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

docs/adr/ADR-072-aiops-bug-fixes.md

@@ -96,8 +96,30 @@ BUG-001 (drift_interpreter) → BUG-002 (deployment_name) → BUG-003 (nemotron_
 | BUG-001 | K8s config drift 告警 → Telegram 出現可讀摘要（非錯誤訊息）|
 | BUG-002 | HostHighCpuLoad 告警 → DecisionToken status = completed（非 error）|
 | BUG-003 | 無效 deployment name → nemotron 拒絕並重新分析 |
-| BUG-004 | 新 Incident 完成後 vectorized = True + persisted_to_pg = True |
+| BUG-004 | 新 Incident 完成後 Redis + DB 兩處 vectorized = True（C2 修復：同步 Redis）|
 | BUG-005 | ready decisions → Telegram 發出審核卡片 |
 | BUG-006 | 修復完成後 outcome + verification_result 不為 null |
-| BUG-007 | 新告警 severity label 正確 → P0/P1/P2 分級出現 |
+| BUG-007 | 確認不需修：alerts-unified.yml 全 42 規則均有 severity label |
 | BUG-008 | HostHighCpuLoad → incident_type = "host_cpu"（非 "custom"）|
+
+---
+
+## Code Review 發現（2026-04-11 首席架構師審查）
+
+### 已修復（C1/C2/I2/M2）
+
+| 項目 | 問題 | 修復 |
+|------|------|------|
+| C1 | `drift_interpreter.py` 寫死內網 IP `192.168.0.111` | 改從 `settings.OLLAMA_URL` 讀取 |
+| C2 | BUG-004 只更新 DB，Redis Working Memory `vectorized` 未同步 | 補 Redis JSON patch 同步 |
+| I2 | `_ALERTNAME_KEYWORDS` 用 `HostHighDiskUsage`（與 alerts-unified.yml 不符） | 改為 `HostOutOfDiskSpace` + 補 `DockerContainerExited` + fallback log |
+| M2 | `import json as _json` 在 for 迴圈體內 | 移至方法頂部 |
+
+### 已記錄技術債（不阻塞合併）
+
+| 項目 | 說明 | 後續 |
+|------|------|------|
+| I1 | BUG-008 規格說整合 ADR-064 Rule Engine，實際改用靜態 dict 擴充 | 下 Sprint 整合 ADR-064 YAML 規則動態推斷；靜態 dict 為可接受中間狀態 |
+| I3 | `resend_stale_ready_tokens()` 直接 Redis SCAN + DB 存取，輕微違反積木化 | Phase R 清理；decision_manager 全域已有類似模式，不單獨修 |
+| I4 | BUG-006 outcome 寫入非 atomic | 已足夠改善（從「永遠 null」到「大部分有值」），完整 atomic 待 Phase R |
+| M3 | `alertname_to_type` dict 應抽至 constants 模組 | 下 Sprint 與 ADR-064 整合時一起重構 |