feat(codex): 引入 upstream_headers hook 机制,为 Codex 注入 session/conversation/account headers

- 新增 upstream_headers.py:可注册 provider+endpoint 维度的 extra headers 构建 hook
- Codex openai:cli 注入 session_id + conversation_id(由 prompt_cache_key sha256 派生)
- Codex openai:compact 注入 chatgpt-account-id(来自 auth_config)+ session_id,不注入 conversation_id
- 修复 prompt_cache:compact 格式现统一为 codex 策略,不再跳过注入
- chat_handler_base / cli_request_mixin 均在 extra_headers 阶段调用 build_upstream_extra_headers
This commit is contained in:
fawney19
2026-03-18 20:43:14 +08:00
parent 203cd5a9d5
commit 3d5b6141a5
8 changed files with 305 additions and 17 deletions

View File

@@ -1,5 +1,6 @@
from __future__ import annotations
import hashlib
from types import SimpleNamespace
from typing import Any
@@ -17,9 +18,10 @@ from src.services.provider.prompt_cache import build_stable_codex_prompt_cache_k
class _DummyAuthInfo:
auth_header = "Authorization"
auth_value = "Bearer upstream-token"
decrypted_auth_config = None
def __init__(self, decrypted_auth_config: dict[str, Any] | None = None) -> None:
self.auth_header = "Authorization"
self.auth_value = "Bearer upstream-token"
self.decrypted_auth_config = decrypted_auth_config
def as_tuple(self) -> tuple[str, str]:
return self.auth_header, self.auth_value
@@ -86,6 +88,10 @@ def _assert_common_codex_headers(headers: dict[str, str]) -> None:
assert "x-forwarded-scheme" not in headers
def _expected_codex_header_id(prompt_cache_key: str) -> str:
return hashlib.sha256(prompt_cache_key.encode()).hexdigest()[:16]
@pytest.mark.asyncio
async def test_build_upstream_request_codex_cli_injects_prompt_cache_and_forces_stream(
monkeypatch: pytest.MonkeyPatch,
@@ -130,15 +136,18 @@ async def test_build_upstream_request_codex_cli_injects_prompt_cache_and_forces_
assert "max_output_tokens" not in result.payload
assert "temperature" not in result.payload
assert "top_p" not in result.payload
short_id = _expected_codex_header_id(result.payload["prompt_cache_key"])
assert result.headers["session_id"] == short_id
assert result.headers["conversation_id"] == short_id
_assert_common_codex_headers(result.headers)
@pytest.mark.asyncio
async def test_build_upstream_request_codex_compact_drops_stream_and_skips_prompt_cache(
async def test_build_upstream_request_codex_compact_drops_stream_and_injects_prompt_cache(
monkeypatch: pytest.MonkeyPatch,
) -> None:
async def _fake_get_provider_auth(endpoint: Any, key: Any) -> _DummyAuthInfo:
return _DummyAuthInfo()
return _DummyAuthInfo({"account_id": "acc-1"})
monkeypatch.setattr(mixmod, "get_provider_auth", _fake_get_provider_auth)
@@ -164,9 +173,13 @@ async def test_build_upstream_request_codex_compact_drops_stream_and_skips_promp
assert result.url == "https://chatgpt.com/backend-api/codex/responses/compact"
assert result.upstream_is_stream is False
assert "stream" not in result.payload
assert "prompt_cache_key" not in result.payload
assert result.payload["prompt_cache_key"] == build_stable_codex_prompt_cache_key("user-key-123")
assert result.payload["instructions"] == "You are GPT-5."
assert result.payload["store"] is False
short_id = _expected_codex_header_id(result.payload["prompt_cache_key"])
assert result.headers["chatgpt-account-id"] == "acc-1"
assert result.headers["session_id"] == short_id
assert "conversation_id" not in result.headers
_assert_common_codex_headers(result.headers)
@@ -205,8 +218,11 @@ async def test_build_upstream_request_legacy_codex_compact_context_uses_compact_
assert result.url == "https://chatgpt.com/backend-api/codex/responses/compact"
assert result.upstream_is_stream is False
assert "stream" not in result.payload
assert "prompt_cache_key" not in result.payload
assert result.payload["prompt_cache_key"] == build_stable_codex_prompt_cache_key("user-key-123")
assert result.payload["instructions"] == "You are GPT-5."
short_id = _expected_codex_header_id(result.payload["prompt_cache_key"])
assert result.headers["session_id"] == short_id
assert "conversation_id" not in result.headers
_assert_common_codex_headers(result.headers)

View File

@@ -106,6 +106,21 @@ def test_maybe_patch_request_with_prompt_cache_key_for_codex_openai_cli() -> Non
assert out["prompt_cache_key"] == build_stable_codex_prompt_cache_key("user-key-123")
def test_maybe_patch_request_with_prompt_cache_key_for_codex_openai_compact() -> None:
req = {"model": "gpt-5", "input": []}
out = maybe_patch_request_with_prompt_cache_key(
req,
provider_api_format="openai:compact",
provider_type="codex",
base_url="https://chatgpt.com/backend-api/codex",
user_api_key_id="user-key-123",
)
assert out is not req
assert out["prompt_cache_key"] == build_stable_codex_prompt_cache_key("user-key-123")
def test_maybe_patch_request_with_prompt_cache_key_skips_official_compact() -> None:
req = {"model": "gpt-5", "input": []}
@@ -121,7 +136,7 @@ def test_maybe_patch_request_with_prompt_cache_key_skips_official_compact() -> N
assert "prompt_cache_key" not in out
def test_maybe_patch_request_with_prompt_cache_key_skips_legacy_codex_compact_context() -> None:
def test_maybe_patch_request_with_prompt_cache_key_for_legacy_codex_compact_context() -> None:
req = {"model": "gpt-5", "input": []}
try:
@@ -136,8 +151,8 @@ def test_maybe_patch_request_with_prompt_cache_key_skips_legacy_codex_compact_co
finally:
set_codex_request_context(None)
assert out is req
assert "prompt_cache_key" not in out
assert out is not req
assert out["prompt_cache_key"] == build_stable_codex_prompt_cache_key("user-key-123")
def test_maybe_patch_request_with_prompt_cache_key_preserves_existing_key() -> None:

View File

@@ -0,0 +1,95 @@
from __future__ import annotations
import hashlib
from src.services.provider.adapters.codex.context import (
CodexRequestContext,
set_codex_request_context,
)
from src.services.provider.upstream_headers import build_upstream_extra_headers
def test_build_upstream_extra_headers_for_codex_openai_cli() -> None:
request_body = {"model": "gpt-5", "input": [], "prompt_cache_key": "pcache-123"}
headers = build_upstream_extra_headers(
provider_type="codex",
endpoint_sig="openai:cli",
request_body=request_body,
original_headers={},
decrypted_auth_config={"account_id": "acc-1"},
)
short_id = hashlib.sha256(b"pcache-123").hexdigest()[:16]
assert headers == {
"chatgpt-account-id": "acc-1",
"session_id": short_id,
"conversation_id": short_id,
}
def test_build_upstream_extra_headers_respects_existing_session_headers() -> None:
headers = build_upstream_extra_headers(
provider_type="codex",
endpoint_sig="openai:cli",
request_body={"model": "gpt-5", "input": [], "prompt_cache_key": "pcache-123"},
original_headers={
"Session_ID": "client-session",
"Conversation_ID": "client-conversation",
},
decrypted_auth_config={"account_id": "acc-1"},
)
assert headers == {"chatgpt-account-id": "acc-1"}
def test_build_upstream_extra_headers_for_codex_openai_compact() -> None:
request_body = {"model": "gpt-5", "input": [], "prompt_cache_key": "pcache-123"}
headers = build_upstream_extra_headers(
provider_type="codex",
endpoint_sig="openai:compact",
request_body=request_body,
original_headers={},
decrypted_auth_config={"account_id": "acc-1"},
)
short_id = hashlib.sha256(b"pcache-123").hexdigest()[:16]
assert headers == {
"chatgpt-account-id": "acc-1",
"session_id": short_id,
}
def test_build_upstream_extra_headers_for_legacy_codex_compact_context() -> None:
request_body = {"model": "gpt-5", "input": [], "prompt_cache_key": "pcache-123"}
try:
set_codex_request_context(CodexRequestContext(is_compact=True))
headers = build_upstream_extra_headers(
provider_type="codex",
endpoint_sig="openai:cli",
request_body=request_body,
original_headers={},
decrypted_auth_config={"account_id": "acc-1"},
)
finally:
set_codex_request_context(None)
short_id = hashlib.sha256(b"pcache-123").hexdigest()[:16]
assert headers == {
"chatgpt-account-id": "acc-1",
"session_id": short_id,
}
def test_build_upstream_extra_headers_returns_empty_without_match() -> None:
headers = build_upstream_extra_headers(
provider_type="codex",
endpoint_sig="openai:chat",
request_body={"model": "gpt-5", "messages": []},
original_headers={},
decrypted_auth_config={"account_id": "acc-1"},
)
assert headers == {}