fix(routes): 修复 count_tokens 路由对 target_vendor.name 的错误属性访问; (#235)

根因: BaseVendor 仅暴露抽象方法 get_name(), 所有 Vendor 子类均无 name 实例属性. routes.py:153/160 错误访问 target_vendor.name 导致 AttributeError 返回 500.

间歇性原因: 仅当 infer_source_vendor_from_body() 从请求体推断出非空 source (含 zhipu 私有产物 srvtoolu_* / server_tool_use) 时才走错误分支, 因此日志中 200/500 共存.

变更要点:
- routes.py 两处 .name 改为 .get_name() 并提取局部变量 target_name 复用, 避免重复方法调用与日志/调用点不一致风险
- 新增回归测试 test_count_tokens_triggers_zhipu_to_target_channel: 注入 server_tool_use + srvtoolu_* 触发 zhipu→anthropic 通道, 断言 200 且 debug 日志含 channel adaptation, 弥补此前 6 个 count_tokens 测试均未触达该分支的盲区
- docs/issue.md 追加档案, 总结根因/防范/同类问题影响

🤖 Generated with [Claude Code](https://github.com/claude), [CodeX](https://openai.com), [Gemini](https://github.com/apps/gemini-code-assist)

Author: ThreeFish-AI

docs/issue.md

@@ -45,3 +45,45 @@ if "usage" in data:        # 仅判断 key 存在
 
 - 本仓库内 `parse_usage_from_chunk` 的 Gemini `usageMetadata` 分支 (line ~219) 已经使用 `isinstance(um, dict)` 防御, 不受影响, 可作为参考实现。
 - 检查其他解析器 (如 routing / vendor adapter 层) 是否还有 `if "key" in data: v = data["key"]; v.get(...)` 这种模式, 必要时同步加固。
+
+---
+
+## count_tokens 路由 `AttributeError: 'ZhipuVendor' object has no attribute 'name'`
+
+**问题描述**
+
+后台日志反复出现 `POST /v1/messages/count_tokens?beta=true 500 Internal Server Error`，并伴随：
+
+```
+File ".../coding/proxy/server/routes.py", line 153, in count_tokens
+    channel_fn = get_transition_channel(source, target_vendor.name)
+AttributeError: 'ZhipuVendor' object has no attribute 'name'
+```
+
+同一时间窗口内大量请求 200 OK、少量请求 500，呈"间歇性"故障特征。
+
+**表因**
+
+`src/coding/proxy/server/routes.py` 的 `count_tokens` 在 153 / 160 两处访问 `target_vendor.name`，触发 `AttributeError` 被 ASGI 中间件捕获返回 500。
+
+**根因**
+
+`BaseVendor` 仅暴露**抽象方法** `get_name() -> str`（`src/coding/proxy/vendors/base.py:75-77`），所有派生类（`AnthropicVendor`、`ZhipuVendor`、`CopilotVendor`、`MinimaxVendor`、`DoubaoVendor`、`KimiVendor` 等）均通过 `_vendor_name` 类属性配合 `get_name()` 返回名称 —— **并无 `name` 实例属性**。该错误访问在 lint/类型检查阶段无告警（因 `BaseVendor` 未在类型系统中约束 `name` 字段），仅在运行时触发。
+
+间歇性原因：第 152 行 `if source:` 是守卫；`source` 由 `infer_source_vendor_from_body(body)`（`src/coding/proxy/convert/vendor_channels.py:357-394`）从请求体启发式推断，仅当出现 zhipu 私有产物（`srvtoolu_*` 形式的 `tool_use.id` 或 `server_tool_use` / `server_tool_use_delta` 类型 content block）时返回 `"zhipu"`，否则 `None`。纯净的首轮 count_tokens 请求 `source is None` 自然绕过 153 行，因此 200/500 共存。
+
+**处理方式**
+
+1. `routes.py:153,160` 将 `target_vendor.name` 改为 `target_vendor.get_name()`，并将结果提取到局部变量 `target_name` 复用，避免重复方法调用与日志/调用点不一致风险。
+2. `tests/test_app_routes.py` 新增 `test_count_tokens_triggers_zhipu_to_target_channel`：通过注入 `server_tool_use` + `srvtoolu_*` 让 `infer_source_vendor_from_body` 返回 `"zhipu"`，断言返回 200 且 debug 日志含 `"count_tokens channel zhipu → anthropic"`，证明通道被实际触发。此前 6 个 count_tokens 测试的请求体都是纯净的、未触达该分支，是 bug 长期漏过的根因。
+
+**后续防范**
+
+- 跨模块引用 Vendor 实例字段时，**统一通过 `BaseVendor` 暴露的方法**（`get_name()`、`map_model()` 等），避免直接访问派生类未定义的"假属性"。
+- 长期演进可考虑在 `BaseVendor` 增加 `@property name` 指向 `get_name()`，将契约前移到类型系统由 mypy / pyright 拦截 —— 该重构属"演进式设计"范畴，不在本次最小干预范围内。
+- 测试覆盖原则：路由层涉及"内容感知"分支（如 `infer_source_vendor_from_body`）时，至少补一个让分支命中的最小用例，避免守卫掩盖代码缺陷。
+
+**同类问题影响与处理注意事项**
+
+- 已 `grep -rn "vendor\.name\b" src/` 全仓扫描，确认 `target_vendor.name | vendor.name` 误用仅 routes.py 的这两处，已随本次修复一并消除。`/v1/messages` 主链路在 executor 中调用 `tier.name`（`Tier` 对象的合法 dataclass 属性），与 vendor 实例 `name` 无关，不受影响。
+- 若未来新增 Vendor 子类，仍只需实现 `get_name()` 抽象方法；外部调用方应遵循同一契约，本档案的修复模式可作为参考。

src/coding/proxy/server/routes.py

@@ -150,14 +150,15 @@ async def count_tokens(request: Request) -> Response:
 
         source = infer_source_vendor_from_body(body)
         if source:
-            channel_fn = get_transition_channel(source, target_vendor.name)
+            target_name = target_vendor.get_name()
+            channel_fn = get_transition_channel(source, target_name)
             if channel_fn is not None:
                 body, adaptations = channel_fn(body)
                 if adaptations:
                     logger.debug(
                         "count_tokens channel %s → %s: %s",
                         source,
-                        target_vendor.name,
+                        target_name,
                         ", ".join(adaptations),
                     )
 

tests/test_app_routes.py

@@ -286,6 +286,76 @@ def test_count_tokens_falls_back_to_tiers0_on_cold_start():
             assert resp.json()["input_tokens"] == 88
 
 
+def test_count_tokens_triggers_zhipu_to_target_channel(caplog):
+    """count_tokens 请求体含 zhipu 私有产物时，应触发跨供应商通道并返回 200.
+
+    回归测试：routes.py 历史上错误访问 target_vendor.name（BaseVendor 仅暴露 get_name()
+    方法，并无 name 属性），当 infer_source_vendor_from_body() 推断出非空 source 时
+    会抛 AttributeError 返回 500。本用例通过注入 zhipu 私有产物（srvtoolu_* id 与
+    server_tool_use 块）触发该路径，断言 200 且 adaptations 日志被打印。
+    """
+    config = ProxyConfig(
+        tiers=[
+            {"vendor": "anthropic", "enabled": True, "api_key": "sk-ant-test"},
+        ],
+        database={"path": "/tmp/test-count-tokens-zhipu-channel.db"},
+    )
+    app = create_app(config)
+
+    mock_response = MagicMock()
+    mock_response.content = b'{"input_tokens": 99}'
+    mock_response.status_code = 200
+
+    body_with_zhipu_artifact = {
+        "model": "claude-sonnet-4-20250514",
+        "messages": [
+            {"role": "user", "content": "Hello"},
+            {
+                "role": "assistant",
+                "content": [
+                    {
+                        "type": "server_tool_use",
+                        "id": "srvtoolu_abc123",
+                        "name": "web_search",
+                        "input": {"query": "test"},
+                    },
+                ],
+            },
+            {
+                "role": "user",
+                "content": [
+                    {
+                        "type": "tool_result",
+                        "tool_use_id": "srvtoolu_abc123",
+                        "content": "result",
+                    },
+                ],
+            },
+        ],
+    }
+
+    with TestClient(app) as client:
+        with patch.object(
+            httpx.AsyncClient,
+            "post",
+            new_callable=AsyncMock,
+            return_value=mock_response,
+        ):
+            with caplog.at_level(logging.DEBUG, logger="coding.proxy.server.routes"):
+                resp = client.post(
+                    "/v1/messages/count_tokens?beta=true",
+                    json=body_with_zhipu_artifact,
+                    headers={"authorization": "Bearer sk-test"},
+                )
+            assert resp.status_code == 200
+            assert resp.json()["input_tokens"] == 99
+            # 通道被实际触发的证据：debug 日志含 "count_tokens channel zhipu → anthropic"
+            assert any(
+                "count_tokens channel zhipu" in record.message
+                for record in caplog.records
+            ), "expected zhipu→anthropic channel adaptation log"
+
+
 def test_status_exposes_vendor_diagnostics():
     """状态接口暴露供应商诊断信息，便于排查凭证交换异常."""
     config = ProxyConfig(