From 157f0f98be241ec25e7bd12aee293ef062bdbf71 Mon Sep 17 00:00:00 2001 From: ogt Date: Fri, 3 Jul 2026 08:30:17 +0800 Subject: [PATCH] feat(smoke): monitor pchome decision closeout --- config.py | 2 +- docs/AI_INTELLIGENCE_MODULE_SOT.md | 6 +- .../pchome_ai_automation_priority_backlog.md | 16 +- services/ai_automation_smoke_service.py | 224 ++++++++++++++++++ tests/test_ai_automation_metrics.py | 11 +- tests/test_ai_automation_smoke_service.py | 151 +++++++++++- 6 files changed, 393 insertions(+), 17 deletions(-) diff --git a/config.py b/config.py index 4cddfe9..8daccd4 100644 --- a/config.py +++ b/config.py @@ -402,7 +402,7 @@ YOUTUBE_API_KEY = os.getenv('YOUTUBE_API_KEY', '') # ========================================== # 系統版本與路徑 # ========================================== -SYSTEM_VERSION = "V10.727" +SYSTEM_VERSION = "V10.728" LOG_FILE_PATH = os.path.join(BASE_DIR, 'logs/system.log') public_url = PUBLIC_URL # 用於模板顯示 diff --git a/docs/AI_INTELLIGENCE_MODULE_SOT.md b/docs/AI_INTELLIGENCE_MODULE_SOT.md index b1ffb82..9e5b063 100644 --- a/docs/AI_INTELLIGENCE_MODULE_SOT.md +++ b/docs/AI_INTELLIGENCE_MODULE_SOT.md @@ -1,8 +1,8 @@ # PChome 業績成長自動化作戰系統 — AI 競價情報模組 Single Source of Truth > **最後更新**: 2026-07-03 (台北時間) -> **狀態**: 🟢 四 AI Agent 自動化閉環已落地；LLM 路由紅線升級為 Ollama-first 三主機級聯；PChome 後台業績匯入韌性已補強；產品定位正名為「PChome 業績成長自動化作戰系統」；外部市場來源正規化層、自動同步、作戰清單與價格參考表優先讀取、CSV 備援預檢、前台操作入口、高可見頁面繁中化守門、比價/作戰 UI 工作台化、AI 密集工作台文字密度守門、跨平台來源治理、商品身份 UI 契約、sitewide visual QA runtime readback、PChome auto-policy authorization guard monitoring 與 decision preflight machine evidence monitoring 已建立，GCP embedding 熔斷延後處理、110 proxy rescue 與 direct host health skip 已建立 -> **適用版本**: V10.727 +> **狀態**: 🟢 四 AI Agent 自動化閉環已落地；LLM 路由紅線升級為 Ollama-first 三主機級聯；PChome 後台業績匯入韌性已補強；產品定位正名為「PChome 業績成長自動化作戰系統」；外部市場來源正規化層、自動同步、作戰清單與價格參考表優先讀取、CSV 備援預檢、前台操作入口、高可見頁面繁中化守門、比價/作戰 UI 工作台化、AI 密集工作台文字密度守門、跨平台來源治理、商品身份 UI 契約、sitewide visual QA runtime readback、PChome auto-policy authorization guard monitoring、decision preflight machine evidence monitoring 與 decision closeout monitoring 已建立，GCP embedding 熔斷延後處理、110 proxy rescue 與 direct host health skip 已建立 +> **適用版本**: V10.728 --- @@ -243,6 +243,7 @@ SQL漏斗(~300筆) - `/api/ai-automation/smoke` 提供 read-only JSON 狀態，不做外部網路呼叫。 - `/api/ai-automation/smoke` 必須包含 `PChome auto-policy authorization guard`：read-only 組裝 PChome auto-policy DB apply guard lane，檢查 no-secret、no-SQL、no-DB-write、machine-verifiable contract、`manual_review_mode=exception_only` 與 `primary_human_gate_count=0`。此 check 不做外部 fetch、不寫 DB、不發 authorization；`/api/ai-automation/scheduled-health-summary` 需輸出 `pchome_auto_policy_authorization_guard` family，`/metrics` 需暴露對應 `momo_ai_automation_scheduled_health_family_status`。 - `/api/ai-automation/smoke` 必須包含 `PChome auto-policy decision preflight`：以本機 deterministic machine evidence adapter 推進 `execute_fetch=True` 的 no-write decision preflight ready path，並明確回報 `payload_source=local_contract_fixture`、`machine_fetch_evidence_source=local_schema_fixture`、`outbound_network=false`、`business_data_source=false`、`writes_database_count=0`、`primary_human_gate_count=0`、`ready_for_database_apply_now=false`、`issues_database_apply_authorization=false`。此 check 不讀 secret、不執行 shell/SQL、不寫 DB、不使用外部 PChome 網路、不把 fixture 當商業資料；`/api/ai-automation/scheduled-health-summary` 需輸出 `pchome_auto_policy_decision_preflight` family，`/metrics` 需暴露對應 `momo_ai_automation_scheduled_health_family_status`。 +- `/api/ai-automation/smoke` 必須包含 `PChome auto-policy decision closeout`：以本機 deterministic contract fixture 與 machine evidence adapter 推進 no-write decision closeout ready path，並明確回報 `post_apply_verifier_required_count=1`、`same_run_truth_required_count=1`、`payload_source=local_contract_fixture`、`outbound_network=false`、`business_data_source=false`、`writes_database_count=0`、`primary_human_gate_count=0`、`ready_for_database_apply_now=false`、`issues_database_apply_authorization=false`。此 check 不讀 secret、不執行 shell/SQL、不寫 DB、不發 apply authorization；`/api/ai-automation/scheduled-health-summary` 需輸出 `pchome_auto_policy_decision_closeout` family，`/metrics` 需暴露對應 `momo_ai_automation_scheduled_health_family_status`。 - 健康檢查 API 會將最近檢查結果保存到 JSONL，頁面顯示最近狀態趨勢。 - 健康檢查歷史支援 JSONL 匯出、清理與每日「正常 / 注意 / 嚴重」摘要。 - 健康檢查每日摘要支援手動 Telegram 推播，並由 `momo-scheduler` 每日 09:10 呼叫 `run_ai_smoke_daily_summary_task()`。 @@ -846,6 +847,7 @@ POSTGRES_HOST=momo-db | 2026-06-27 | AI 挑品必須直接顯示商品證據與賣場入口 | V10.724 起商品看板 AI 挑品清單會在 selection 階段合併最新 PChome 同款證據，卡片需顯示 MOMO 商品ID、PChome 商品ID、同款信心與兩邊賣場入口；商品圖需使用 PChome 圖片作為 fallback，不得只給一段建議理由。 | | 2026-06-27 | Google Drive 自動匯入不得在背景排程啟動瀏覽器授權 | V10.725 起正式容器與排程服務即使誤設 `GOOGLE_DRIVE_ALLOW_INTERACTIVE_AUTH=true` 也會硬阻擋互動 OAuth；匯入前會檢查 JSON token 是否存在、可刷新且可寫回，Drive 測試與檔案列表 API 會 fail-closed 回傳可處理的授權狀態。 | | 2026-07-03 | PChome auto-policy decision preflight 必須有 machine evidence runtime monitoring | V10.727 起 `/api/ai-automation/smoke`、`/api/ai-automation/scheduled-health-summary` 與 `/metrics` 必須輸出 `PChome auto-policy decision preflight` / `pchome_auto_policy_decision_preflight`；此 runtime check 使用本機 deterministic schema evidence adapter 與 `payload_source=local_contract_fixture` 自動驗證 `execute_fetch=True` 的 decision preflight ready path，但明確標記 `outbound_network=false`、`business_data_source=false`、`writes_database_count=0`、`primary_human_gate_count=0`，不讀 secret、不執行 SQL、不寫 DB、不發 apply authorization。 | +| 2026-07-03 | PChome auto-policy decision closeout 必須有 runtime monitoring | V10.728 起 `/api/ai-automation/smoke`、`/api/ai-automation/scheduled-health-summary` 與 `/metrics` 必須輸出 `PChome auto-policy decision closeout` / `pchome_auto_policy_decision_closeout`；此 runtime check 自動驗證 decision closeout ready path、post-apply verifier requirement、same-run production truth requirement 與 future authorization decision package，但明確標記 `writes_database_count=0`、`primary_human_gate_count=0`、`ready_for_database_apply_now=false`、`issues_database_apply_authorization=false`，不讀 secret、不執行 SQL、不寫 DB、不發 apply authorization。 | | 2026-06-29 | PChome DB apply 授權 lane 必須先通過 no-write guard / decision preflight / decision closeout / issuer gate / signing-decision preflight / signing-decision closeout / signing-issuer guard | V10.725 的 PChome mapping backlog auto-policy 已新增 `/api/ai/pchome-growth/mapping-backlog/auto-policy-db-apply-authorization-lane-guard`、`/api/ai/pchome-growth/mapping-backlog/auto-policy-db-apply-authorization-decision-preflight`、`/api/ai/pchome-growth/mapping-backlog/auto-policy-db-apply-authorization-decision-closeout`、`/api/ai/pchome-growth/mapping-backlog/auto-policy-db-apply-authorization-issuer-gate`、`/api/ai/pchome-growth/mapping-backlog/auto-policy-db-apply-authorization-signing-decision-preflight`、`/api/ai/pchome-growth/mapping-backlog/auto-policy-db-apply-authorization-signing-decision-closeout` 與 `/api/ai/pchome-growth/mapping-backlog/auto-policy-db-apply-authorization-signing-issuer-guard`；這些 endpoint 只驗證 final exact request package、same-run production truth requirement、secret rejection、rollback boundary、lane entry requirements、decision input requirements、rejection policy、post-apply verifier、future authorization decision package、final nonsecret authorization envelope、signing decision preflight inputs、unsigned signing decision package 與 signable request boundary，不讀 secret、不執行 shell/SQL、不寫 DB，也不簽發 database apply authorization。 | | 2026-06-29 | PChome DB apply 授權簽署發行者 lane 必須先產出 final signable request package | V10.725 的 PChome mapping backlog auto-policy 新增 `/api/ai/pchome-growth/mapping-backlog/auto-policy-db-apply-authorization-signing-issuer-closeout`；此 endpoint 只把 signing-issuer guard 的 signable request boundary 收斂成 final signable request package 與 closeout contract，確認 fresh production truth、post-apply verifier、migration hash、secret boundary 與 no-side-effect checks，不讀 secret、不簽發 authorization、不執行 shell/SQL、不寫 DB，也不代表正式 DB apply 已授權。 | | 2026-06-29 | PChome DB apply 授權簽署執行 lane 必須先通過 operator-held secret boundary preflight | V10.725 的 PChome mapping backlog auto-policy 新增 `/api/ai/pchome-growth/mapping-backlog/auto-policy-db-apply-authorization-signing-execution-preflight`；此 endpoint 只把 final signable request package 轉成 future signing execution preflight package、operator-held secret boundary contract、nonsecret signing inputs、command-shape preview、rollback boundary 與 abort conditions，不讀 secret、不接受 plaintext secret、不簽發 authorization、不執行 shell/SQL、不寫 DB，也不代表正式 DB apply 已授權。 | diff --git a/docs/guides/pchome_ai_automation_priority_backlog.md b/docs/guides/pchome_ai_automation_priority_backlog.md index 9e70a3c..06dbb36 100644 --- a/docs/guides/pchome_ai_automation_priority_backlog.md +++ b/docs/guides/pchome_ai_automation_priority_backlog.md @@ -9,7 +9,7 @@ 以下是使用者在主線推進期間插入、且必須保留在完整工作項目裡的要求。後續不得再把這些需求漏掉或變成口頭提醒。 1. 正式環境才是最新版本真相；不得用本機檔案、舊筆記、舊分支蓋過 production truth。 -2. 版本不得搞錯；目前正式環境 `/health` 版本維持 `V10.727`，除非有明確版本 bump、部署與 readback。 +2. 版本不得搞錯；目前正式環境 `/health` 版本維持 `V10.728`，除非有明確版本 bump、部署與 readback。 3. GitHub 全面 freeze；不得使用 GitHub、`gh`、GitHub API、GitHub Actions、PR、issue、mirror 或 read-only GitHub 流程。 4. 實作結果必須推到 Gitea `dev` 與 `main`；若改到 runtime 行為，還必須部署到正式環境並回讀。 5. 主方向是 AI 自動化，不是人工審核；人工欄位只能當 evidence / ledger / UI truth，不得阻擋低爆炸半徑、可驗證、可回滾的 controlled apply。 @@ -41,7 +41,7 @@ 已完成: -- 正式環境版本真相 guard：`/health` 是最高真相，目前維持 `V10.727`。 +- 正式環境版本真相 guard：`/health` 是最高真相，目前維持 `V10.728`。 - Gitea `dev` / `main` source truth 對齊。 - Retry exception controlled apply executor 已落地，目標表為 `pchome_product_matches`。 - 正式 DB 已 controlled apply 4 個 selectors: @@ -99,10 +99,17 @@ - `/api/ai-automation/scheduled-health-summary` 已輸出 `pchome_auto_policy_decision_preflight` family - `/metrics` 已輸出 `momo_ai_automation_scheduled_health_family_status{family="pchome_auto_policy_decision_preflight",...}` - 不讀 secret、不執行 SQL、不寫 DB、不發 apply authorization，下一個機器動作是 authorization decision closeout lane +- PChome auto-policy decision closeout runtime monitoring 已完成: + - `/api/ai-automation/smoke` 已加入 `PChome auto-policy decision closeout` + - 以本機 contract fixture + machine evidence adapter 自動推進 decision closeout ready path + - 明確標記 `post_apply_verifier_required_count=1`、`same_run_truth_required_count=1`、`writes_database_count=0`、`primary_human_gate_count=0` + - `/api/ai-automation/scheduled-health-summary` 已輸出 `pchome_auto_policy_decision_closeout` family + - `/metrics` 已輸出 `momo_ai_automation_scheduled_health_family_status{family="pchome_auto_policy_decision_closeout",...}` + - 不讀 secret、不執行 SQL、不寫 DB、不發 apply authorization，下一個機器動作是 authorization issuer gate lane 進行中 / 下一步，必須照順序: -1. 將 auto-policy decision preflight 往 authorization decision closeout lane 推進。 +1. 將 auto-policy decision closeout 往 authorization issuer gate lane 推進。 完成標準: @@ -269,7 +276,7 @@ 進行中 / 下一步，必須照順序: -1. 將 auto-policy decision preflight 推進到 authorization decision closeout lane。 +1. 將 auto-policy decision closeout 推進到 authorization issuer gate lane。 2. 對其他 safe mapping backlog lanes 套用 receipt / replay / drift verifier 模式。 3. 為每一類 controlled apply 自動產生 rollback evidence package。 4. 把 AI automation metrics 持續整合進既有 observability surfaces。 @@ -343,6 +350,7 @@ | P3.4 | Observability metrics integration | 已完成 | `/metrics` exports scheduled health summary gauges + focused tests; AI surface HTML readback now feeds smoke and scheduled health | P3.5 auto-policy guard monitoring | | P3.5 | PChome auto-policy authorization guard monitoring | 已完成 | AI smoke + scheduled family + metrics family read no-write/no-secret/exception-only guard; focused tests | machine fetch evidence / authorization decision preflight lane | | P3.6 | PChome auto-policy machine evidence decision preflight monitoring | 已完成 | AI smoke + scheduled family + metrics family prove local contract fixture, no-network machine evidence, decision preflight ready, no DB write, no human primary gate | authorization decision closeout lane | +| P3.7 | PChome auto-policy authorization decision closeout monitoring | 已完成 | AI smoke + scheduled family + metrics family prove decision closeout ready, post-apply verifier bound, same-run truth required, no DB write, no apply authorization | authorization issuer gate lane | | P4.1 | Source / deployment / runtime truth package | 已完成 | `scripts/ops/report_source_deploy_runtime_truth.py` + focused tests | 每次 Gitea push / production deploy 後執行 P4 report | | P2.2 | Benchmark guardrails applied to core AI surfaces | 已完成 | `/ai_intelligence` + `/observability/overview` golden-signal strips + focused tests | 後續 safe automation lanes 套同樣 first-viewport summary | diff --git a/services/ai_automation_smoke_service.py b/services/ai_automation_smoke_service.py index 0b09391..f5439ca 100644 --- a/services/ai_automation_smoke_service.py +++ b/services/ai_automation_smoke_service.py @@ -239,6 +239,11 @@ def build_scheduled_automation_health_summary( if not auto_policy_preflight or not auto_policy_preflight_details: auto_policy_preflight = _pchome_auto_policy_decision_preflight_check() auto_policy_preflight_details = auto_policy_preflight.get("details") or {} + auto_policy_closeout = _find_check(source_result, "PChome auto-policy decision closeout") + auto_policy_closeout_details = auto_policy_closeout.get("details") or {} + if not auto_policy_closeout or not auto_policy_closeout_details: + auto_policy_closeout = _pchome_auto_policy_decision_closeout_check() + auto_policy_closeout_details = auto_policy_closeout.get("details") or {} surface_readback = _find_check(source_result, "AI surface HTML readback") surface_details = surface_readback.get("details") or {} sitewide_readback = _find_check(source_result, "Sitewide UI/UX Agent readback") @@ -414,6 +419,69 @@ def build_scheduled_automation_health_summary( "writes_database": False, }, }, + { + "key": "pchome_auto_policy_decision_closeout", + "label": "PChome auto-policy decision closeout", + "status": auto_policy_closeout.get("status") or "warning", + "summary": ( + auto_policy_closeout.get("summary") + or "PChome auto-policy decision closeout has no latest check." + ), + "next_machine_action": auto_policy_closeout_details.get("next_machine_action") + or ( + "continue_pchome_auto_policy_decision_closeout_monitoring" + if auto_policy_closeout.get("status") == "ok" + else "refresh_pchome_auto_policy_decision_closeout" + ), + "details": { + "policy": auto_policy_closeout_details.get("policy"), + "result": auto_policy_closeout_details.get("result"), + "decision_closeout_ready_count": int( + auto_policy_closeout_details.get("decision_closeout_ready_count") or 0 + ), + "decision_preflight_ready_count": int( + auto_policy_closeout_details.get("decision_preflight_ready_count") or 0 + ), + "decision_closeout_check_count": int( + auto_policy_closeout_details.get("decision_closeout_check_count") or 0 + ), + "decision_closeout_pass_count": int( + auto_policy_closeout_details.get("decision_closeout_pass_count") or 0 + ), + "decision_closeout_waiting_count": int( + auto_policy_closeout_details.get("decision_closeout_waiting_count") or 0 + ), + "post_apply_verifier_required_count": int( + auto_policy_closeout_details.get("post_apply_verifier_required_count") or 0 + ), + "same_run_truth_required_count": int( + auto_policy_closeout_details.get("same_run_truth_required_count") or 0 + ), + "reads_secret_count": int(auto_policy_closeout_details.get("reads_secret_count") or 0), + "executes_sql_count": int(auto_policy_closeout_details.get("executes_sql_count") or 0), + "writes_database_count": int( + auto_policy_closeout_details.get("writes_database_count") or 0 + ), + "primary_human_gate_count": int( + auto_policy_closeout_details.get("primary_human_gate_count") or 0 + ), + "manual_review_mode": auto_policy_closeout_details.get("manual_review_mode"), + "payload_source": auto_policy_closeout_details.get("payload_source"), + "execute_fetch": bool(auto_policy_closeout_details.get("execute_fetch")), + "outbound_network": bool(auto_policy_closeout_details.get("outbound_network")), + "business_data_source": bool(auto_policy_closeout_details.get("business_data_source")), + "ready_for_database_apply_now": bool( + auto_policy_closeout_details.get("ready_for_database_apply_now") + ), + "issues_database_apply_authorization": bool( + auto_policy_closeout_details.get("issues_database_apply_authorization") + ), + "permits_future_authorization_decision_lane": bool( + auto_policy_closeout_details.get("permits_future_authorization_decision_lane") + ), + "writes_database": False, + }, + }, { "key": "ai_surface_html_readback", "label": "AI surface HTML readback", @@ -1400,6 +1468,161 @@ def _pchome_auto_policy_decision_preflight_check() -> Dict[str, Any]: ) +def _pchome_auto_policy_decision_closeout_check() -> Dict[str, Any]: + """Read-only monitor for the future authorization decision closeout package.""" + try: + from services import pchome_mapping_backlog_service as backlog + from services.ai_exception_contract import ( + LEGACY_REVIEW_REQUIRED_COUNT_KEY, + PRIMARY_HUMAN_GATE_COUNT_KEY, + ) + + payload = _pchome_auto_policy_decision_preflight_contract_payload() + closeout = backlog.build_pchome_auto_policy_db_apply_authorization_decision_closeout( + payload, + batch_size=max(1, min(_PCHOME_AUTO_POLICY_DECISION_PREFLIGHT_BATCH_SIZE, 50)), + execute_fetch=True, + timeout_seconds=max(1, min(_PCHOME_AUTO_POLICY_DECISION_PREFLIGHT_TIMEOUT_SECONDS, 30)), + http_get=_pchome_auto_policy_machine_fetch_evidence_get, + ) + summary = closeout.get("summary") or {} + decision = closeout.get("future_authorization_decision_closeout") or {} + package = closeout.get("future_authorization_decision_package") or {} + contract = closeout.get("decision_closeout_contract") or {} + safety = closeout.get("safety") or {} + result = str(closeout.get("result") or "UNKNOWN") + + closeout_ready_count = int(summary.get("authorization_decision_closeout_ready_count") or 0) + preflight_ready_count = int( + summary.get("authorization_decision_preflight_ready_count") or 0 + ) + check_count = int(summary.get("decision_closeout_check_count") or 0) + pass_count = int(summary.get("decision_closeout_pass_count") or 0) + waiting_count = int(summary.get("decision_closeout_waiting_count") or 0) + input_requirement_count = int(summary.get("decision_input_requirement_count") or 0) + rejection_reason_count = int(summary.get("decision_rejection_reason_count") or 0) + verifier_required_count = int(summary.get("post_apply_verifier_required_count") or 0) + same_run_truth_count = int(summary.get("same_run_truth_required_count") or 0) + reads_secret_count = int(summary.get("reads_secret_count") or 0) + executes_sql_count = int(summary.get("executes_sql_count") or 0) + writes_database_count = int(summary.get("writes_database_count") or 0) + primary_human_gate_count = int(summary.get(PRIMARY_HUMAN_GATE_COUNT_KEY) or 0) + manual_review_required_count = int(summary.get(LEGACY_REVIEW_REQUIRED_COUNT_KEY) or 0) + side_effect_count = reads_secret_count + executes_sql_count + writes_database_count + risk_detected = any( + [ + side_effect_count, + primary_human_gate_count, + manual_review_required_count, + safety.get("reads_secret_in_preview") is not False, + safety.get("executes_sql") is not False, + safety.get("writes_database") is not False, + contract.get("issues_database_apply_authorization") is not False, + contract.get("ready_for_database_apply_now") is not False, + contract.get("writes_database") is not False, + package.get("ready_for_database_apply_now") is not False, + package.get("issues_database_apply_authorization") is not False, + decision.get("ready_for_database_apply_now") is not False, + decision.get("issues_database_apply_authorization") is not False, + ] + ) + closeout_contract_present = ( + closeout_ready_count == 1 + and preflight_ready_count == 1 + and check_count >= 12 + and pass_count == check_count + and waiting_count == 0 + and input_requirement_count == 8 + and rejection_reason_count == 10 + and verifier_required_count == 1 + and same_run_truth_count == 1 + and decision.get("ready_for_future_authorization_decision_closeout") is True + and package.get("ready_for_future_authorization_decision_package") is True + and package.get("requires_post_apply_verifier") is True + and package.get("requires_fresh_production_truth_in_same_run") is True + and contract.get("machine_verifiable") is True + and contract.get("permits_future_authorization_decision_lane") is True + and safety.get("manual_review_mode") == "exception_only" + ) + + if risk_detected: + status = "critical" + summary_text = "PChome auto-policy decision closeout 偵測到 secret、SQL、DB write 或人工主 gate 風險" + next_machine_action = "halt_pchome_auto_policy_decision_closeout_and_inspect_risk" + elif closeout_contract_present: + status = "ok" + summary_text = ( + "PChome auto-policy decision closeout 已自動完成 " + f"{pass_count}/{check_count} checks，post-apply verifier 已綁定，DB apply authorization=0" + ) + next_machine_action = "continue_to_pchome_auto_policy_authorization_issuer_gate" + else: + status = "warning" + summary_text = "PChome auto-policy decision closeout 尚未達自動監控契約" + next_machine_action = "refresh_pchome_auto_policy_decision_closeout" + + return _check( + "PChome auto-policy decision closeout", + status, + summary_text, + { + "policy": closeout.get("policy"), + "result": result, + "source_policy": closeout.get("source_policy"), + "decision_closeout_ready_count": closeout_ready_count, + "decision_preflight_ready_count": preflight_ready_count, + "decision_closeout_check_count": check_count, + "decision_closeout_pass_count": pass_count, + "decision_closeout_waiting_count": waiting_count, + "decision_input_requirement_count": input_requirement_count, + "decision_rejection_reason_count": rejection_reason_count, + "post_apply_verifier_required_count": verifier_required_count, + "same_run_truth_required_count": same_run_truth_count, + "reads_secret_count": reads_secret_count, + "executes_sql_count": executes_sql_count, + "writes_database_count": writes_database_count, + "primary_human_gate_count": primary_human_gate_count, + "manual_review_required_count": manual_review_required_count, + "manual_review_mode": safety.get("manual_review_mode"), + "payload_source": "local_contract_fixture", + "machine_fetch_evidence_source": "local_schema_fixture", + "http_get_adapter": "_pchome_auto_policy_machine_fetch_evidence_get", + "execute_fetch": True, + "outbound_network": False, + "business_data_source": False, + "ready_for_database_apply_now": bool(package.get("ready_for_database_apply_now")), + "issues_database_apply_authorization": bool( + package.get("issues_database_apply_authorization") + ), + "permits_future_authorization_decision_lane": bool( + contract.get("permits_future_authorization_decision_lane") + ), + "requires_post_apply_verifier": bool(package.get("requires_post_apply_verifier")), + "requires_fresh_production_truth": bool( + package.get("requires_fresh_production_truth_in_same_run") + ), + "writes_database": False, + "next_machine_action": next_machine_action, + }, + ) + except Exception as exc: + return _check( + "PChome auto-policy decision closeout", + "warning", + f"PChome auto-policy decision closeout 例行監控暫時無法讀取：{exc}", + { + "writes_database": False, + "writes_database_count": 0, + "primary_human_gate_count": 0, + "execute_fetch": True, + "outbound_network": False, + "business_data_source": False, + "payload_source": "local_contract_fixture", + "next_machine_action": "refresh_pchome_auto_policy_decision_closeout", + }, + ) + + def _ai_surface_html_readback_check() -> Dict[str, Any]: """Read-only AI product surface guardrail readback.""" try: @@ -1629,6 +1852,7 @@ def collect_ai_automation_smoke(*, record_history: bool = True, history_limit: i _pchome_controlled_apply_drift_monitor_check(), _pchome_auto_policy_authorization_guard_check(), _pchome_auto_policy_decision_preflight_check(), + _pchome_auto_policy_decision_closeout_check(), _ai_surface_html_readback_check(), _sitewide_ui_ux_agent_check(), _sitewide_visual_qa_check(), diff --git a/tests/test_ai_automation_metrics.py b/tests/test_ai_automation_metrics.py index ff02e21..0c8c1d4 100644 --- a/tests/test_ai_automation_metrics.py +++ b/tests/test_ai_automation_metrics.py @@ -107,10 +107,10 @@ def test_system_metrics_exports_scheduled_health_summary(): Gauge, { "summary": { - "ok": 6, + "ok": 7, "warning": 1, "critical": 0, - "total": 7, + "total": 8, "primary_human_gate_count": 0, "writes_database_count": 0, }, @@ -119,6 +119,7 @@ def test_system_metrics_exports_scheduled_health_summary(): {"key": "pchome_controlled_apply_drift_monitor", "status": "warning"}, {"key": "pchome_auto_policy_authorization_guard", "status": "ok"}, {"key": "pchome_auto_policy_decision_preflight", "status": "ok"}, + {"key": "pchome_auto_policy_decision_closeout", "status": "ok"}, {"key": "sitewide_ui_ux_agent", "status": "ok"}, {"key": "sitewide_visual_qa", "status": "ok"}, ], @@ -126,7 +127,7 @@ def test_system_metrics_exports_scheduled_health_summary(): ) output = generate_latest(registry).decode("utf-8") - assert 'momo_ai_automation_scheduled_health_summary_total{status="ok"} 6.0' in output + assert 'momo_ai_automation_scheduled_health_summary_total{status="ok"} 7.0' in output assert 'momo_ai_automation_scheduled_health_summary_total{status="warning"} 1.0' in output assert ( 'momo_ai_automation_scheduled_health_family_status{family="ai_automation_smoke",status="ok"} 1.0' @@ -144,6 +145,10 @@ def test_system_metrics_exports_scheduled_health_summary(): 'momo_ai_automation_scheduled_health_family_status{family="pchome_auto_policy_decision_preflight",status="ok"} 1.0' in output ) + assert ( + 'momo_ai_automation_scheduled_health_family_status{family="pchome_auto_policy_decision_closeout",status="ok"} 1.0' + in output + ) assert ( 'momo_ai_automation_scheduled_health_family_status{family="sitewide_ui_ux_agent",status="ok"} 1.0' in output diff --git a/tests/test_ai_automation_smoke_service.py b/tests/test_ai_automation_smoke_service.py index f717177..a5b89c3 100644 --- a/tests/test_ai_automation_smoke_service.py +++ b/tests/test_ai_automation_smoke_service.py @@ -62,6 +62,38 @@ def _auto_policy_decision_preflight_history_check(status="ok"): } +def _auto_policy_decision_closeout_history_check(status="ok"): + return { + "name": "PChome auto-policy decision closeout", + "status": status, + "summary": "PChome auto-policy decision closeout 已自動完成", + "details": { + "policy": "read_only_pchome_growth_auto_policy_db_apply_authorization_decision_closeout", + "result": "DB_APPLY_AUTHORIZATION_DECISION_CLOSEOUT_READY", + "decision_closeout_ready_count": 1, + "decision_preflight_ready_count": 1, + "decision_closeout_check_count": 12, + "decision_closeout_pass_count": 12, + "decision_closeout_waiting_count": 0, + "post_apply_verifier_required_count": 1, + "same_run_truth_required_count": 1, + "reads_secret_count": 0, + "executes_sql_count": 0, + "writes_database_count": 0, + "primary_human_gate_count": 0, + "manual_review_mode": "exception_only", + "payload_source": "local_contract_fixture", + "execute_fetch": True, + "outbound_network": False, + "business_data_source": False, + "ready_for_database_apply_now": False, + "issues_database_apply_authorization": False, + "permits_future_authorization_decision_lane": True, + "next_machine_action": "continue_to_pchome_auto_policy_authorization_issuer_gate", + }, + } + + def test_event_router_smoke_reports_queued_deliveries(tmp_path, monkeypatch): from services import ai_automation_metrics as metrics from services import ai_automation_smoke_service as smoke @@ -99,6 +131,7 @@ def test_collect_ai_automation_smoke_uses_worst_status(monkeypatch): monkeypatch.setattr(smoke, "_pchome_controlled_apply_drift_monitor_check", lambda: smoke._check("pchome", "ok", "ok")) monkeypatch.setattr(smoke, "_pchome_auto_policy_authorization_guard_check", lambda: smoke._check("auto-policy", "ok", "ok")) monkeypatch.setattr(smoke, "_pchome_auto_policy_decision_preflight_check", lambda: smoke._check("auto-policy preflight", "ok", "ok")) + monkeypatch.setattr(smoke, "_pchome_auto_policy_decision_closeout_check", lambda: smoke._check("auto-policy closeout", "ok", "ok")) monkeypatch.setattr(smoke, "_ai_surface_html_readback_check", lambda: smoke._check("surface", "ok", "ok")) monkeypatch.setattr(smoke, "_sitewide_ui_ux_agent_check", lambda: smoke._check("sitewide", "ok", "ok")) monkeypatch.setattr(smoke, "_sitewide_visual_qa_check", lambda: smoke._check("visual", "ok", "ok")) @@ -106,7 +139,7 @@ def test_collect_ai_automation_smoke_uses_worst_status(monkeypatch): result = smoke.collect_ai_automation_smoke(record_history=False) assert result["status"] == "critical" - assert result["summary"] == {"ok": 10, "warning": 1, "critical": 1, "total": 12} + assert result["summary"] == {"ok": 11, "warning": 1, "critical": 1, "total": 13} def test_pchome_controlled_apply_drift_monitor_reports_verified_zero_drift(monkeypatch): @@ -294,6 +327,83 @@ def test_pchome_auto_policy_decision_preflight_reports_machine_fetch_no_write(mo assert result["details"]["next_machine_action"] == "continue_to_pchome_auto_policy_decision_closeout_lane" +def test_pchome_auto_policy_decision_closeout_reports_no_write_package(monkeypatch): + from services import ai_automation_smoke_service as smoke + from services import pchome_mapping_backlog_service as backlog + captured = {} + + def fake_closeout(*_args, **kwargs): + captured.update(kwargs) + return { + "policy": "read_only_pchome_growth_auto_policy_db_apply_authorization_decision_closeout", + "result": "DB_APPLY_AUTHORIZATION_DECISION_CLOSEOUT_READY", + "source_policy": "read_only_pchome_growth_auto_policy_db_apply_authorization_decision_preflight", + "summary": { + "authorization_decision_closeout_ready_count": 1, + "authorization_decision_preflight_ready_count": 1, + "decision_closeout_check_count": 12, + "decision_closeout_pass_count": 12, + "decision_closeout_waiting_count": 0, + "decision_input_requirement_count": 8, + "decision_rejection_reason_count": 10, + "post_apply_verifier_required_count": 1, + "same_run_truth_required_count": 1, + "reads_secret_count": 0, + "executes_sql_count": 0, + "writes_database_count": 0, + "primary_human_gate_count": 0, + "manual_review_required_count": 0, + }, + "future_authorization_decision_closeout": { + "ready_for_future_authorization_decision_closeout": True, + "ready_for_database_apply_now": False, + "issues_database_apply_authorization": False, + }, + "future_authorization_decision_package": { + "ready_for_future_authorization_decision_package": True, + "ready_for_database_apply_now": False, + "issues_database_apply_authorization": False, + "requires_post_apply_verifier": True, + "requires_fresh_production_truth_in_same_run": True, + }, + "decision_closeout_contract": { + "machine_verifiable": True, + "permits_future_authorization_decision_lane": True, + "issues_database_apply_authorization": False, + "ready_for_database_apply_now": False, + "writes_database": False, + }, + "safety": { + "reads_secret_in_preview": False, + "executes_sql": False, + "writes_database": False, + "manual_review_mode": "exception_only", + }, + } + + monkeypatch.setattr( + backlog, + "build_pchome_auto_policy_db_apply_authorization_decision_closeout", + fake_closeout, + ) + + result = smoke._pchome_auto_policy_decision_closeout_check() + + assert result["status"] == "ok" + assert captured["execute_fetch"] is True + assert captured["http_get"] is smoke._pchome_auto_policy_machine_fetch_evidence_get + assert result["details"]["decision_closeout_pass_count"] == 12 + assert result["details"]["post_apply_verifier_required_count"] == 1 + assert result["details"]["same_run_truth_required_count"] == 1 + assert result["details"]["payload_source"] == "local_contract_fixture" + assert result["details"]["outbound_network"] is False + assert result["details"]["business_data_source"] is False + assert result["details"]["primary_human_gate_count"] == 0 + assert result["details"]["writes_database_count"] == 0 + assert result["details"]["issues_database_apply_authorization"] is False + assert result["details"]["next_machine_action"] == "continue_to_pchome_auto_policy_authorization_issuer_gate" + + def test_collect_ai_automation_smoke_persists_recent_history(tmp_path, monkeypatch): from services import ai_automation_smoke_service as smoke @@ -309,6 +419,7 @@ def test_collect_ai_automation_smoke_persists_recent_history(tmp_path, monkeypat monkeypatch.setattr(smoke, "_pchome_controlled_apply_drift_monitor_check", lambda: smoke._check("pchome", "ok", "ok")) monkeypatch.setattr(smoke, "_pchome_auto_policy_authorization_guard_check", lambda: smoke._check("auto-policy", "ok", "ok")) monkeypatch.setattr(smoke, "_pchome_auto_policy_decision_preflight_check", lambda: smoke._check("auto-policy preflight", "ok", "ok")) + monkeypatch.setattr(smoke, "_pchome_auto_policy_decision_closeout_check", lambda: smoke._check("auto-policy closeout", "ok", "ok")) monkeypatch.setattr(smoke, "_ai_surface_html_readback_check", lambda: smoke._check("surface", "ok", "ok")) monkeypatch.setattr(smoke, "_sitewide_ui_ux_agent_check", lambda: smoke._check("sitewide", "ok", "ok")) monkeypatch.setattr(smoke, "_sitewide_visual_qa_check", lambda: smoke._check("visual", "ok", "ok")) @@ -367,7 +478,7 @@ def test_scheduled_automation_health_summary_reads_history_without_side_effects( json.dumps({ "generated_at": datetime.now().isoformat(timespec="seconds"), "status": "ok", - "summary": {"ok": 12, "warning": 0, "critical": 0, "total": 12}, + "summary": {"ok": 13, "warning": 0, "critical": 0, "total": 13}, "checks": [ { "name": "PChome 受控落地 drift monitor", @@ -383,6 +494,7 @@ def test_scheduled_automation_health_summary_reads_history_without_side_effects( }, _auto_policy_guard_history_check(), _auto_policy_decision_preflight_history_check(), + _auto_policy_decision_closeout_history_check(), { "name": "AI surface HTML readback", "status": "ok", @@ -444,7 +556,7 @@ def test_scheduled_automation_health_summary_reads_history_without_side_effects( ) assert summary["policy"] == "read_only_ai_automation_scheduled_health_summary" assert summary["status"] == "ok" - assert summary["summary"]["total"] == 9 + assert summary["summary"]["total"] == 10 assert summary["summary"]["primary_human_gate_count"] == 0 assert summary["summary"]["writes_database_count"] == 0 assert pchome_family["status"] == "ok" @@ -469,6 +581,19 @@ def test_scheduled_automation_health_summary_reads_history_without_side_effects( assert preflight_family["details"]["business_data_source"] is False assert preflight_family["details"]["primary_human_gate_count"] == 0 assert preflight_family["details"]["writes_database_count"] == 0 + closeout_family = next( + item for item in summary["families"] + if item["key"] == "pchome_auto_policy_decision_closeout" + ) + assert closeout_family["status"] == "ok" + assert closeout_family["details"]["decision_closeout_pass_count"] == 12 + assert closeout_family["details"]["post_apply_verifier_required_count"] == 1 + assert closeout_family["details"]["same_run_truth_required_count"] == 1 + assert closeout_family["details"]["payload_source"] == "local_contract_fixture" + assert closeout_family["details"]["outbound_network"] is False + assert closeout_family["details"]["business_data_source"] is False + assert closeout_family["details"]["primary_human_gate_count"] == 0 + assert closeout_family["details"]["writes_database_count"] == 0 surface_family = next( item for item in summary["families"] if item["key"] == "ai_surface_html_readback" @@ -503,7 +628,7 @@ def test_scheduled_automation_health_summary_can_use_current_smoke_without_recor current_smoke = { "generated_at": "2026-07-02T12:00:00", "status": "critical", - "summary": {"ok": 11, "warning": 0, "critical": 1, "total": 12}, + "summary": {"ok": 12, "warning": 0, "critical": 1, "total": 13}, "checks": [ { "name": "PChome 受控落地 drift monitor", @@ -513,6 +638,7 @@ def test_scheduled_automation_health_summary_can_use_current_smoke_without_recor }, _auto_policy_guard_history_check(), _auto_policy_decision_preflight_history_check(), + _auto_policy_decision_closeout_history_check(), { "name": "AI surface HTML readback", "status": "ok", @@ -568,7 +694,7 @@ def test_scheduled_automation_health_summary_falls_back_to_visual_qa_artifact(tm json.dumps({ "generated_at": datetime.now().isoformat(timespec="seconds"), "status": "ok", - "summary": {"ok": 10, "warning": 0, "critical": 0, "total": 10}, + "summary": {"ok": 11, "warning": 0, "critical": 0, "total": 11}, "checks": [], }, ensure_ascii=False) + "\n", encoding="utf-8", @@ -584,6 +710,11 @@ def test_scheduled_automation_health_summary_falls_back_to_visual_qa_artifact(tm "_pchome_auto_policy_decision_preflight_check", lambda: _auto_policy_decision_preflight_history_check(), ) + monkeypatch.setattr( + smoke, + "_pchome_auto_policy_decision_closeout_check", + lambda: _auto_policy_decision_closeout_history_check(), + ) monkeypatch.setattr( surface_service, "build_sitewide_visual_qa_readback", @@ -632,7 +763,7 @@ def test_scheduled_automation_health_summary_route_returns_compact_payload(tmp_p json.dumps({ "generated_at": datetime.now().isoformat(timespec="seconds"), "status": "ok", - "summary": {"ok": 11, "warning": 0, "critical": 0, "total": 11}, + "summary": {"ok": 12, "warning": 0, "critical": 0, "total": 12}, "checks": [], }, ensure_ascii=False) + "\n", encoding="utf-8", @@ -648,6 +779,11 @@ def test_scheduled_automation_health_summary_route_returns_compact_payload(tmp_p "_pchome_auto_policy_decision_preflight_check", lambda: _auto_policy_decision_preflight_history_check(), ) + monkeypatch.setattr( + smoke, + "_pchome_auto_policy_decision_closeout_check", + lambda: _auto_policy_decision_closeout_history_check(), + ) app = Flask(__name__) with app.test_request_context( @@ -873,6 +1009,7 @@ def test_surface_html_readback_check_is_part_of_ai_smoke(monkeypatch): monkeypatch.setattr(smoke, "_pchome_controlled_apply_drift_monitor_check", lambda: smoke._check("pchome", "ok", "ok")) monkeypatch.setattr(smoke, "_pchome_auto_policy_authorization_guard_check", lambda: smoke._check("auto-policy", "ok", "ok")) monkeypatch.setattr(smoke, "_pchome_auto_policy_decision_preflight_check", lambda: smoke._check("auto-policy preflight", "ok", "ok")) + monkeypatch.setattr(smoke, "_pchome_auto_policy_decision_closeout_check", lambda: smoke._check("auto-policy closeout", "ok", "ok")) monkeypatch.setattr(smoke, "_sitewide_visual_qa_check", lambda: smoke._check( "Sitewide visual QA readback", "ok", @@ -894,7 +1031,7 @@ def test_surface_html_readback_check_is_part_of_ai_smoke(monkeypatch): item for item in result["checks"] if item["name"] == "Sitewide visual QA readback" ) - assert result["summary"]["total"] == 12 + assert result["summary"]["total"] == 13 assert surface_check["status"] == "ok" assert surface_check["details"]["checked_surface_count"] == 10 assert sitewide_check["status"] == "ok"