隔離 PG sync 整合測試

docs/memory/claude_inventory_validation_20260513.md

@@ -47,6 +47,7 @@
 - `tests/test_requirements_pinning.py` 已鎖住上述被 V2 誤列的 runtime dependencies：套件需留在 `requirements.txt`，且至少一個 runtime import 證據仍存在。
 - `paramiko` 已確認沒有 runtime import；ADR-013 現行實作改以 `utils/ssh_helper.py` 組 CLI `ssh`，因此已從 `requirements.txt` 移除並補測試防止依賴殘影回來。
 - `pgvector` Python package 與 `matplotlib-inline` 已確認不是 runtime 依賴：pgvector 走 PostgreSQL extension + 本地 SQLAlchemy `Vector` type，Jupyter inline backend 不屬 production path；兩者已從 `requirements.txt` 移除並補測試。
+- `tests/test_pg_sync.py` 已改為 opt-in integration test：預設不再連 localhost PostgreSQL 或建立/刪除測試表，需 `RUN_PG_SYNC_INTEGRATION=1` 且提供 `POSTGRES_PASSWORD` 才執行。
 - Telegram `momo:eig:<event_id>` callback 已在 `routes/openclaw_bot_routes.py` 與 `services/telegram_bot_service.py` 實作並有 webhook 測試覆蓋，不是未實作缺口。
 - Telegram `date_*` / `goal_*` 不是死 callback handler：按鈕先送 `await:*` 進入輸入等待狀態，使用者下一則文字才由 pending action 消費；`tests/test_openclaw_bot_menu_keyboards.py` 與 `tests/test_openclaw_bot_routes_webhook.py` 已覆蓋。
 - `services/ai_automation_smoke_service.py` 不是死 service：`run_scheduler.py` 每日 09:10 掛 `run_ai_smoke_daily_summary_task()`，該 task 會呼叫 `send_smoke_daily_summary()`；`tests/test_ai_automation_smoke_service.py` 與 `tests/test_ai_automation_metrics.py` 已覆蓋。

tests/test_pg_sync.py

@@ -1,5 +1,5 @@
 #!/usr/bin/env python3
-"""測試完整 76 欄位同步"""
+"""Opt-in integration smoke for the legacy SQLite → PostgreSQL sync script."""
 
 import os
 import sqlite3
@@ -8,9 +8,18 @@ import pytest
 
 psycopg2 = pytest.importorskip("psycopg2")
 
+pytestmark = pytest.mark.skipif(
+    os.environ.get("RUN_PG_SYNC_INTEGRATION") != "1",
+    reason="PG sync integration touches a local PostgreSQL database; set RUN_PG_SYNC_INTEGRATION=1 to run.",
+)
+
 
 def test_pg_sync():
+    if not os.environ.get("POSTGRES_PASSWORD"):
+        pytest.skip("POSTGRES_PASSWORD is required for PG sync integration")
+
     sqlite_conn = sqlite3.connect("data/momo_database.db")
+    test_table = "test_full76"
     try:
         cursor = sqlite_conn.cursor()
         cursor.execute("PRAGMA table_info(realtime_sales_monthly)")
@@ -30,14 +39,14 @@ def test_pg_sync():
         )
         try:
             pg_cursor = pg_conn.cursor()
-            pg_cursor.execute("DROP TABLE IF EXISTS test_full76 CASCADE")
+            pg_cursor.execute(f'DROP TABLE IF EXISTS "{test_table}" CASCADE')
             cols_def = ", ".join([f'"{c}" TEXT' for c in columns])
-            pg_cursor.execute(f"CREATE TABLE test_full76 (id SERIAL PRIMARY KEY, {cols_def})")
+            pg_cursor.execute(f'CREATE TABLE "{test_table}" (id SERIAL PRIMARY KEY, {cols_def})')
             pg_conn.commit()
 
             cols_sql = ", ".join([f'"{c}"' for c in columns])
             placeholders = ", ".join(["%s"] * len(columns))
-            sql = f"INSERT INTO test_full76 ({cols_sql}) VALUES ({placeholders})"
+            sql = f'INSERT INTO "{test_table}" ({cols_sql}) VALUES ({placeholders})'
 
             success_count = 0
             for row in rows:
@@ -49,7 +58,7 @@ def test_pg_sync():
 
             pg_conn.commit()
             assert success_count >= 0
-            pg_cursor.execute("DROP TABLE test_full76 CASCADE")
+            pg_cursor.execute(f'DROP TABLE IF EXISTS "{test_table}" CASCADE')
             pg_conn.commit()
         finally:
             pg_conn.close()