coding-proxy 0.3.1a4__tar.gz → 0.3.1a6__tar.gz

{coding_proxy-0.3.1a4 → coding_proxy-0.3.1a6}/AGENTS.md

@@ -52,6 +52,7 @@
   1. **Python**: 严禁使用 pip/poetry，**必须**统一使用 `uv` 进行包管理与脚本执行（如 `uv run`）；
   2. **JavaScript/TypeScript**: 严禁使用 npm/yarn，**必须**统一使用 `pnpm` 进行包管理与脚本执行。
 - **Database Management**: 谨慎操作，数据迁移、测试等操作严禁将现有数据删除，谨慎操作数据迁移的回滚，防止数据被清理。
+- **In-depth and close to the facts**：系统且全面地进行问题的分析，深入贴近事实，如有疑问，需先发问，不要乱做决定。
 
 ## Documentation Standards (文档规范)
 

{coding_proxy-0.3.1a4 → coding_proxy-0.3.1a6}/CHANGELOG.md

@@ -4,6 +4,7 @@
 
 ## [Unreleased]
 
+- fix(vendor-channels): 为所有 targeting zhipu 的转换通道（zhipu→zhipu、copilot→zhipu、anthropic→zhipu）新增 `tool_result.id` 字段注入，修复 zhipu GLM-5 后端错误访问 `.id` 属性（`'ClaudeContentBlockToolResult' object has no attribute 'id'`）导致的 500 错误，使 zhipu 可完全承接含 tool_result 的会话；
 - fix(vendor-channels): 新增 zhipu 同 vendor 自清理通道，修复 GLM-5 自循环 400 + tool_results 偶发降级；
 - fix(vendor-channels): 修复 `_rewrite_srvtoolu_ids` 块顺序敏感性导致 inline tool_result 漏改名，进而 enforce 阶段 dict key 与 tool_use_ids 错位、anthropic 报 `tool_use ids without tool_result blocks immediately after` 的 cascade failover 问题（改为两遍扫描：先收集 id_map，再统一改写所有 tool_result.tool_use_id 引用）；
 - fix(vendor-channels): `enforce_anthropic_tool_pairing` 增加全局 sanity check pass，主循环边角错位让 dangling tool_use 漏过校验时兜底合成 is_error 占位并打 `pairing_sanity_repaired` 标签，避免 anthropic 二次报错；

{coding_proxy-0.3.1a4 → coding_proxy-0.3.1a6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: coding-proxy
-Version: 0.3.1a4
+Version: 0.3.1a6
 Summary: A High-Availability, Transparent, and Smart Multi-Vendor Proxy for Claude Code. Support Claude Plans, GitHub Copilot, Google Antigravity, ZAI/GLM, MiniMax, Qwen, Xiaomi, Kimi, Doubao...
 Project-URL: Source Code, https://github.com/ThreeFish-AI/coding-proxy
 Project-URL: User Guide, https://github.com/ThreeFish-AI/coding-proxy/blob/master/docs/user-guide.md

{coding_proxy-0.3.1a4 → coding_proxy-0.3.1a6}/docs/issue.md

@@ -141,3 +141,69 @@ adaptations 列表显示 `misplaced_tool_result_relocated` 但**没有** `orphan
 
 - 任何对 messages 进行 ID 重写的转换链 (如 `_rewrite_srvtoolu_ids`、`anthropic_to_openai`、`anthropic_to_gemini`) 都应使用两遍扫描或一次性收集后再批量改写, 以保证 block 顺序无关性。
 - enforce 类校验函数若依赖 dict key 与 list 元素的**等同性**, 必须先确保两者在同一参考系下 (改名前 vs 改名后); 否则错位会以 "看起来 OK 实际有漏" 的方式静默泄漏到下游。
+
+---
+
+## zhipu 500 `'ClaudeContentBlockToolResult' object has no attribute 'id'`
+
+**问题描述**
+
+zhipu GLM-5 在处理含 `tool_result` 块的会话时持续返回 500 错误，每次请求都触发故障转移至 copilot，zhipu 完全无法承接含工具调用的多轮对话：
+
+```
+WARNING zhipu stream error: status=500 body='...message":"\'ClaudeContentBlockToolResult\' object has no attribute \'id\'"}'
+```
+
+**表因**
+
+zhipu 后端在解析 `tool_result` 内容块时错误地访问 `.id` 属性。但 Anthropic API 规范中 `tool_result` 块只有 `tool_use_id` 字段（用于关联对应的 `tool_use`），没有 `id` 字段。
+
+**根因**（2026-04-29 复盘更新）
+
+**初始诊断**（已推翻）：认为 zhipu 后端期望 `tool_result` 有 `id` 字段，通过 `_inject_tool_result_id_for_zhipu` 注入 `id = tool_use_id` 可绕过。
+
+**实际根因**：转换通道本身引入的问题。具体因果链：
+
+1. **转换前**：zhipu 偶发在 assistant 消息中内联输出 `tool_result`（违反 Anthropic 规范），但 zhipu 后端对 assistant 消息中内联的 `tool_result` **不做 `.id` 属性访问**，因此不触发 500。
+2. **转换后**：所有 zhipu 目标通道执行 `enforce_anthropic_tool_pairing`，将 assistant 内联的 `tool_result` 搬迁到紧随的 user 消息。zhipu 后端对 user 消息中的 `tool_result` **执行 `.id` 属性访问**（代码路径不同），触发 `AttributeError` → 500。
+3. **`_inject_tool_result_id_for_zhipu` 无效**：该函数往 JSON dict 注入 `"id": tool_use_id`，但 zhipu 后端的 `ClaudeContentBlockToolResult` Python 类不从 JSON 读取 `id` 字段（类定义中无此属性），注入的值在反序列化时被丢弃。
+
+**实证依据**：用户确认「转换通道之前 zhipu 正常，转换通道之后才出现 500 错误」。
+
+**处理方式**（2026-04-29 更新）
+
+从所有 zhipu 目标转换通道中移除以下三个步骤：
+
+| 移除项 | 原因 |
+|--------|------|
+| `enforce_anthropic_tool_pairing` | 搬迁 `tool_result` 到 user 消息触发 zhipu 500 |
+| `_inject_tool_result_id_for_zhipu` | zhipu 类不读取注入的 `id`，无效且可能干扰 |
+| `_strip_cache_control` | zhipu 原生支持 `cache_control`（cache_read 已实证），剥离反损性能 |
+
+保留的必要步骤：
+
+| 保留项 | 原因 |
+|--------|------|
+| `strip_thinking_blocks` | copilot/anthropic 的 thinking 签名 zhipu 无法验证 |
+| 移除 `thinking`/`extended_thinking` 顶层参数 | zhipu 不支持 |
+| `_remove_vendor_blocks(server_tool_use_delta)` | zhipu 自身流式残块 |
+| `_remove_vendor_blocks(server_tool_use)` | Anthropic beta 块，zhipu 不支持 |
+
+**涉及变更的转换通道**：
+- `prepare_copilot_to_zhipu` — 移除 cache_control / tool pairing / id 注入
+- `prepare_anthropic_to_zhipu` — 移除 cache_control / tool pairing / id 注入
+- `prepare_zhipu_self_cleanup` — 移除 tool pairing / id 注入
+
+**注意**: `prepare_zhipu_to_anthropic` 和 `prepare_zhipu_to_copilot` 不受影响（目标是 anthropic/copilot，不是 zhipu），仍保留 `enforce_anthropic_tool_pairing`。
+
+**后续防范**
+
+- **转换通道的「最小干预」原则**：跨供应商转换应仅清理目标供应商**确认不支持**的特性。未经验证的「预防性清理」（如剥离 cache_control）可能误伤供应商原生支持的功能，甚至引入新的故障。
+- **workaround 须验证有效**：`_inject_tool_result_id_for_zhipu` 虽有注释说明目的，但未经验证其有效性即合入。后续 workaround 须附带验证证据（如 curl 复现、上游确认）。
+- **zhipu 后端 bug 跟踪**：`ClaudeContentBlockToolResult` 类缺少 `id` 属性是 zhipu 上游 bug。若 zhipu 修复此 bug，可考虑恢复 tool pairing 以获得更严格的消息结构校验。
+
+**同类问题影响与处理注意事项**
+
+- `NativeAnthropicVendor` 子类的自清理通道应**精确剪裁**：仅修复 vendor 自身拒绝的产物，不做跨供应商的全量清理。
+- 当 zhipu 后端出现新的 400 拒绝（如 inline tool_result 再次被拒），应优先调查是 zhipu 后端变更还是请求格式问题，而非立即加回 tool pairing（可能重新触发 500）。
+- `_inject_tool_result_id_for_zhipu` 函数暂时保留在代码中（未删除），标记为 deprecated，待确认不需要后清理。

{coding_proxy-0.3.1a4 → coding_proxy-0.3.1a6}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "coding-proxy"
-version = "0.3.1a4"
+version = "0.3.1a6"
 description = "A High-Availability, Transparent, and Smart Multi-Vendor Proxy for Claude Code. Support Claude Plans, GitHub Copilot, Google Antigravity, ZAI/GLM, MiniMax, Qwen, Xiaomi, Kimi, Doubao..."
 readme = "README.md"
 requires-python = ">=3.12"

{coding_proxy-0.3.1a4 → coding_proxy-0.3.1a6}/src/coding/proxy/convert/vendor_channels.py

@@ -322,6 +322,36 @@ def _enforce_pairing_sanity_pass(messages_list: list[Any]) -> list[str]:
     return sanity_synthesized
 
 
+def _inject_tool_result_id_for_zhipu(body: dict[str, Any]) -> int:
+    """为 tool_result 块注入 ``id`` 字段以兼容 zhipu GLM-5 后端.
+
+    zhipu 的 Anthropic 兼容端点在解析 ``tool_result`` 块时会访问 ``.id`` 属性，
+    但 Anthropic API 规范中 ``tool_result`` 只有 ``tool_use_id`` 字段而没有 ``id``。
+    此函数在所有 ``tool_result`` 块上补设 ``id``（值等于 ``tool_use_id``），
+    避免触发 ``'ClaudeContentBlockToolResult' object has no attribute 'id'`` 500 错误。
+
+    Returns:
+        被注入 ``id`` 字段的 tool_result 块数量。
+    """
+    injected = 0
+    for message in body.get("messages", []):
+        if not isinstance(message, dict):
+            continue
+        content = message.get("content")
+        if not isinstance(content, list):
+            continue
+        for block in content:
+            if (
+                isinstance(block, dict)
+                and block.get("type") == "tool_result"
+                and "id" not in block
+                and block.get("tool_use_id")
+            ):
+                block["id"] = block["tool_use_id"]
+                injected += 1
+    return injected
+
+
 def _strip_cache_control(body: dict[str, Any]) -> int:
     """从 system/messages/tools 中移除 cache_control 字段（就地）.
 
@@ -542,13 +572,18 @@ def infer_source_vendor_from_body(body: dict[str, Any]) -> str | None:
 def prepare_copilot_to_zhipu(
     body: dict[str, Any],
 ) -> tuple[dict[str, Any], list[str]]:
-    """copilot → zhipu 转换: 清理 copilot 产物以适配 GLM-5.
+    """copilot → zhipu 转换: 仅清理 copilot 产物中 zhipu 确认不支持的部分.
 
-    GLM-5 的 Anthropic 兼容端点对以下特性支持不完整:
-    - thinking / redacted_thinking 块 (signature 由非 Anthropic 签发)
-    - cache_control 字段
-    - 跨供应商产物 (misplaced tool_result, 非标准 tool_use ID)
-    - 顶层 thinking / extended_thinking 参数
+    GLM-5 的 Anthropic 兼容端点:
+    - ✗ thinking / redacted_thinking 块 (signature 由非 Anthropic 签发)
+    - ✓ cache_control 字段 (cache_read 已在生产实证)
+    - ✓ tool_result 在 assistant 消息中内联 (zhipu 自身偶发产出，可自行消化)
+    - ✗ 顶层 thinking / extended_thinking 参数
+
+    注意: 不再执行 enforce_anthropic_tool_pairing 和 _inject_tool_result_id_for_zhipu。
+    实证表明 tool_result 重定位会触发 zhipu 后端 ``'ClaudeContentBlockToolResult'
+    object has no attribute 'id'`` 500 错误；id 注入对 zhipu 的 Python 类
+    (不读取 JSON 中的 id 字段) 亦无效。详见 docs/issue.md。
 
     Returns:
         (prepared_body, adaptations) — adaptations 为应用的变换描述列表。
@@ -561,22 +596,12 @@ def prepare_copilot_to_zhipu(
     if stripped:
         adaptations.append(f"stripped_{stripped}_thinking_blocks")
 
-    # Step 2: 移除 cache_control 字段
-    removed_cc = _strip_cache_control(prepared)
-    if removed_cc:
-        adaptations.append(f"removed_{removed_cc}_cache_control_fields")
-
-    # Step 3: 移除顶层 thinking/extended_thinking 参数（GLM-5 不支持）
+    # Step 2: 移除顶层 thinking/extended_thinking 参数（GLM-5 不支持）
     for param in ("thinking", "extended_thinking"):
         if param in prepared:
             del prepared[param]
             adaptations.append(f"removed_{param}_param")
 
-    # Step 4: 强制 tool_use/tool_result 配对
-    pairing_fixes = enforce_anthropic_tool_pairing(prepared.get("messages", []))
-    if pairing_fixes:
-        adaptations.extend(pairing_fixes)
-
     return prepared, adaptations
 
 
@@ -597,9 +622,11 @@ def prepare_anthropic_to_zhipu(
     Anthropic API 可能产生的非兼容产物:
     - ``server_tool_use`` blocks（web search / computer use 等 beta 功能）
     - ``thinking`` / ``redacted_thinking`` blocks（含 Anthropic 签发的 signature）
-    - ``cache_control`` 字段
     - 顶层 ``thinking`` / ``extended_thinking`` 参数
 
+    注意: 不再移除 cache_control (GLM-5 支持) ，不再执行 tool pairing 和
+    id 注入。原因同 prepare_copilot_to_zhipu 的 docstring。
+
     Returns:
         (prepared_body, adaptations) — adaptations 为应用的变换描述列表。
     """
@@ -616,22 +643,12 @@ def prepare_anthropic_to_zhipu(
     if stripped:
         adaptations.append(f"stripped_{stripped}_thinking_blocks")
 
-    # Step 3: 移除 cache_control 字段
-    removed_cc = _strip_cache_control(prepared)
-    if removed_cc:
-        adaptations.append(f"removed_{removed_cc}_cache_control_fields")
-
-    # Step 4: 移除顶层 thinking/extended_thinking 参数（GLM-5 不支持）
+    # Step 3: 移除顶层 thinking/extended_thinking 参数（GLM-5 不支持）
     for param in ("thinking", "extended_thinking"):
         if param in prepared:
             del prepared[param]
             adaptations.append(f"removed_{param}_param")
 
-    # Step 5: 强制 tool_use/tool_result 配对
-    pairing_fixes = enforce_anthropic_tool_pairing(prepared.get("messages", []))
-    if pairing_fixes:
-        adaptations.extend(pairing_fixes)
-
     return prepared, adaptations
 
 
@@ -742,24 +759,22 @@ def prepare_zhipu_to_anthropic(
 def prepare_zhipu_self_cleanup(
     body: dict[str, Any],
 ) -> tuple[dict[str, Any], list[str]]:
-    """zhipu → zhipu 自清理: 仅修复 zhipu 自身无法消化的产物.
+    """zhipu → zhipu 自清理: 仅剥离 zhipu 自身的流式残块.
 
-    GLM-5 偶发地在 assistant 消息中输出 ``tool_result`` 块（违反 Anthropic 规范），
-    或在流式响应中暴露 ``server_tool_use_delta`` 私有块。当 Claude Code 将这些
-    产物原样回送下一轮请求时，zhipu 的 Anthropic 兼容端点会以 400 拒绝
-    （表现为 "400 + tool_results" 偶发，进而触发到 copilot 的降级）。
+    GLM-5 在流式响应中偶发暴露 ``server_tool_use_delta`` 私有块。当 Claude Code
+    将这些产物原样回送下一轮请求时，zhipu 的 Anthropic 兼容端点会拒绝。
 
-    本通道仅修复 zhipu 自身拒绝的两类产物，**保留** 所有 zhipu 原生支持的特性:
+    本通道**保留**所有 zhipu 原生支持的特性:
 
     - ✓ ``srvtoolu_*`` ID 与 ``server_tool_use`` 类型（zhipu 原生）
     - ✓ thinking blocks 的 zhipu 自签 signature
     - ✓ ``cache_control`` 字段（GLM Anthropic 端点支持，cache_read 已实证）
     - ✓ 顶层 ``thinking`` / ``extended_thinking`` 参数
+    - ✓ tool_result 在 assistant 消息中内联（zhipu 自身偶发产出，可自行消化）
 
-    清理操作（顺序、就地、幂等）:
-    1. 剥离 ``server_tool_use_delta`` 流式残块
-    2. 强制 tool_use/tool_result 配对（关键: 把 assistant 内联的 tool_result
-       搬迁到紧随的 user 消息）
+    注意: 不再执行 enforce_anthropic_tool_pairing 和 _inject_tool_result_id_for_zhipu。
+    实证表明 tool_result 重定位会触发 zhipu 后端 500 错误。
+    详见 docs/issue.md。
 
     Returns:
         (prepared_body, adaptations) — adaptations 为应用的变换描述列表。
@@ -772,11 +787,6 @@ def prepare_zhipu_self_cleanup(
     if removed_vendor_blocks:
         adaptations.append(f"removed_{removed_vendor_blocks}_zhipu_vendor_blocks")
 
-    # Step 2: 强制 tool_use/tool_result 配对
-    pairing_fixes = enforce_anthropic_tool_pairing(prepared.get("messages", []))
-    if pairing_fixes:
-        adaptations.extend(pairing_fixes)
-
     return prepared, adaptations
 
 

{coding_proxy-0.3.1a4 → coding_proxy-0.3.1a6}/tests/test_router_executor.py

@@ -2034,7 +2034,12 @@ class TestPrepareBodyForTierSelfTransition:
     """验证 zhipu → zhipu 自转换通道在 _prepare_body_for_tier 中的应用行为."""
 
     def test_applies_zhipu_self_cleanup(self):
-        """source=zhipu, target=zhipu → 剥离 server_tool_use_delta + tool pairing."""
+        """source=zhipu, target=zhipu → 仅剥离 server_tool_use_delta.
+
+        不再做 tool pairing（搬迁 tool_result 会触发 zhipu 500），
+        也不做 id 注入（zhipu 类不读取 JSON 中的 id）。
+        inline tool_result 保留在 assistant 消息中，zhipu 可自行消化。
+        """
         tier = MagicMock()
         tier.name = "zhipu"
 
@@ -2067,18 +2072,16 @@ class TestPrepareBodyForTierSelfTransition:
         assert result is not body
         assert len(body["messages"][0]["content"]) == 3
 
-        # delta 块被剥离, tool_result 被搬迁出 assistant
+        # delta 块被剥离
         assistant_content = result["messages"][0]["content"]
-        assert all(
-            b.get("type") not in ("server_tool_use_delta", "tool_result")
-            for b in assistant_content
-        )
-        # tool_result 已搬到下一个 user 消息
-        assert result["messages"][1]["role"] == "user"
+        assert all(b.get("type") != "server_tool_use_delta" for b in assistant_content)
+        # inline tool_result 保留在 assistant 中（不再搬迁）
         assert any(
             b.get("type") == "tool_result" and b.get("tool_use_id") == "srvtoolu_a"
-            for b in result["messages"][1]["content"]
+            for b in assistant_content
         )
+        # 不应插入额外的 user 消息
+        assert len(result["messages"]) == 1
 
     def test_self_cleanup_preserves_srvtoolu_ids(self):
         """回归保护: 自清理通道不得改写 zhipu 原生 srvtoolu_* ID."""