OoO 3b0b4b3d42 fix(ppt): address critic findings on commit 38967ce (HIGH-1, HIGH-2, Medium-4) ...

Critic 審查 38967ce 找出 2 HIGH + 4 medium，本 commit 修最緊急三項：

HIGH-2: cleanup_expired_ppt_cache 預設 dry_run=False 危險
- 函式 default 改 dry_run=True（呼叫方需明確傳 False 才實刪）
- launchd 排程腳本改用 DRY_RUN 環境變數，預設 false→實刪，
  但測試時可 DRY_RUN=true 安全跑
- /cache cleanup Telegram 指令預設乾跑，需加 confirm 才實刪
- /cache cleanup days<1 強制乾跑（防呆）

HIGH-1: matplotlib 三個 helper 缺 try/finally → 渲染失敗洩漏 figure
- _mpl_horiz_bar_png / _mpl_line_chart_png / _mpl_pareto_chart_png
  全部用 try/except/finally 包住，例外時 plt.close() 仍執行
- 同時讓渲染失敗回 None（呼叫端自然 fallback 到原生 chart）

Medium-4: scripts/ppt_cleanup.sh PROJECT_DIR 寫死
- 改用相對路徑解析（cd $SCRIPT_DIR/..），手動執行不再需要設環境變數
- VENV_PY 找不到時 fallback 到系統 python3（容器友善）
- 新增 DRY_RUN 環境變數開關（預設 false）

煙霧測試：
- syntax OK (services/ppt_generator.py + routes/openclaw_bot_routes.py)
- monthly v3.1 重生 10 頁 290KB
- cleanup_expired_ppt_cache dry_run default = True ✓

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

2026-05-02 17:23:02 +08:00

1.8 KiB

Executable File

Raw Permalink Blame History

View Raw