| 編號 | 時間 | 呼叫端 | 供應商 | 模型 | 輸入 | 輸出 | 耗時 | 狀態 | 成本 | 標記 |
|---|---|---|---|---|---|---|---|---|---|---|
| {{ r.id }} | {{ r.called_at }} | {{ r.caller }} | {{ obs_label.provider(r.provider) }} | {{ r.model[:25] }} | {{ r.in_tokens }} | {{ r.out_tokens }} | {{ r.duration_ms }} | {{ obs_label.status(r.status, '-') }} | ${{ "%.4f"|format(r.cost) }} | {% if r.cache_hit %}快取{% endif %}{% if r.rag_hit %}RAG{% endif %} |
| 編號 | 時間 | 呼叫端 | 供應商 | 模型 | 輸入 | 輸出 | 耗時 | 狀態 | 成本 | 標記 |
|---|---|---|---|---|---|---|---|---|---|---|
| {{ r.id }} | {{ r.called_at }} | {{ r.caller_display or r.caller }}{% if r.caller_display and r.caller_display != r.caller %}原始：{{ r.caller }}{% endif %} | {{ obs_label.provider(r.provider) }} | {{ r.model[:25] }} | {{ r.in_tokens }} | {{ r.out_tokens }} | {{ r.duration_ms }} | {{ obs_label.status(r.status, '-') }} | ${{ "%.4f"|format(r.cost) }} | {% for badge in r.route_badges %}{{ badge }}{% endfor %}{% if r.cache_hit %}快取{% endif %}{% if r.rag_hit %}RAG{% endif %} |
Ollama 優先策略 v5.0 — AI 流量控制塔
diff --git a/tests/test_admin_observability_routes.py b/tests/test_admin_observability_routes.py index 6752901..991d742 100644 --- a/tests/test_admin_observability_routes.py +++ b/tests/test_admin_observability_routes.py @@ -8,6 +8,7 @@ """ from __future__ import annotations +from datetime import datetime from unittest.mock import MagicMock import pytest @@ -99,6 +100,33 @@ def test_ai_calls_dashboard_db_error_falls_back(client, monkeypatch): assert r.status_code == 200 # 失敗安全：仍 render，不 500 +def test_ai_calls_recent_row_marks_legacy_code_review_gemini_as_backup(): + from routes.admin_observability_routes import _build_ai_call_recent_row + + row = ( + 1452, + datetime(2026, 5, 19, 12, 2, 49), + 'code_review_openclaw', + 'gemini', + 'gemini-2.5-flash', + 181, + 56, + 9158, + 'ok', + 0, + False, + False, + ) + + data = _build_ai_call_recent_row(row) + + assert data['caller'] == 'code_review_openclaw' + assert data['caller_display'] == 'code_review_openclaw_gemini' + assert data['provider'] == 'gemini' + assert 'Gemini 備援' in data['route_badges'] + assert '舊 caller' in data['route_badges'] + + # ────────────────────────────────────────────────────────────────────────── # /observability/promotion_review # ────────────────────────────────────────────────────────────────────────── diff --git a/tests/test_code_review_claude_routing.py b/tests/test_code_review_claude_routing.py index 4f82905..94ca656 100644 --- a/tests/test_code_review_claude_routing.py +++ b/tests/test_code_review_claude_routing.py @@ -77,6 +77,27 @@ def _stub_logger(monkeypatch): monkeypatch.setenv('AI_CALL_LOGGING_ENABLED', 'false') +def _capture_ai_call_states(monkeypatch): + """攔截 ai_call_logger 寫入，保留 caller/provider/fallback_to 供路由斷言。""" + import services.ai_call_logger as logger_mod + + states = [] + + def _capture(state): + states.append({ + "caller": state.caller, + "provider": state.provider, + "model": state.model, + "status": state.status, + "fallback_to": state.fallback_to, + "error": state.error, + }) + + monkeypatch.setattr(logger_mod, '_async_write', _capture) + monkeypatch.setenv('AI_CALL_LOGGING_ENABLED', 'true') + return states + + def _stub_ollama(monkeypatch, *, success: bool = True, content: str = "OLLAMA-RESULT", error: str = "all hosts failed"): @@ -245,6 +266,46 @@ def test_ollama_failure_flag_false_uses_gemini_backup(monkeypatch): fake_elephant.generate.assert_not_called() +def test_gemini_backup_uses_dedicated_caller_in_telemetry(monkeypatch): + """Ollama 失敗後的 Gemini 必須記為 code_review_openclaw_gemini。""" + monkeypatch.setenv('CODE_REVIEW_USE_CLAUDE', 'false') + monkeypatch.setenv('GEMINI_API_KEY', 'test-key') + captured = _capture_ai_call_states(monkeypatch) + + svc_mod = _reload_pipeline() + _stub_ollama(monkeypatch, success=False) + _stub_anthropic(monkeypatch, svc_mod, available=True) + fake_genai, fake_elephant = _stub_gemini_and_elephant(monkeypatch) + + pipeline = _make_pipeline(svc_mod) + result = pipeline._openclaw_assess( + files={"services/foo.py": "def x(): pass"}, + findings=[], + ) + + assert result == "GEMINI-RESULT" + fake_genai.GenerativeModel.assert_called_once() + fake_elephant.generate.assert_not_called() + assert any( + state["caller"] == "code_review_openclaw" + and state["provider"] == "gcp_ollama" + and state["status"] == "fallback" + and state["fallback_to"] == "code_review_openclaw_gemini" + for state in captured + ) + assert any( + state["caller"] == "code_review_openclaw_gemini" + and state["provider"] == "gemini" + and state["status"] == "ok" + for state in captured + ) + assert not any( + state["caller"] == "code_review_openclaw" + and state["provider"] == "gemini" + for state in captured + ) + + def test_ollama_failure_claude_unavailable_uses_gemini_backup(monkeypatch): """Ollama 失敗 + flag=true 但 Claude unavailable → Gemini 才作備援""" monkeypatch.setenv('CODE_REVIEW_USE_CLAUDE', 'true')