fix: TRAE body 归一 developer 角色 + tool_calls 转 function_call

用 DUMP_REQUEST_BODIES 抓到的 PI 真实请求体逐项排查 TRAE 失败：

1. tool_calls 历史格式：OpenAI 是 tool_calls[].function{name,arguments}，
   TRAE SOLO 要求 tool_calls[].function_call——原样透传导致上游
   protobuf 解析错（code=2001 required field Name is not set）
   → prepare_body 转换 function→function_call，无 name 剔除，全空删除
2. developer 角色在 TRAE 同样不被接受，但不报错而是静默空流
  （code=3003 llm response empty stream）→ 同 CB 归一为 system

试验矩阵定位：function_call 转换修掉 2001 后仍 3003，逐层裁剪
（tool 历史/消息数/内容长度）最终二分到 developer 角色——
实测 developer=3003、system=ok，同一 body 其余不变。

端到端验证：PI 真实请求体（developer+33消息+7工具+tool历史）
→ 200 正常生成 reasoning 与 tool_calls。后端 643 测试 / 覆盖 100%。

Author: RobbsLuo

src/provider/trae/client.py

@@ -83,6 +83,12 @@ def prepare_body(payload: dict[str, Any], model: str) -> dict[str, Any]:
     body["model"] = model
 
     messages = body.get("messages")
+    if isinstance(messages, list):
+        for message in messages:
+            # OpenAI developer 角色（PI 等客户端对 reasoning 模型使用）
+            # TRAE 上游不认：静默返回空流（3003）→ 归一为 system
+            if isinstance(message, dict) and message.get("role") == "developer":
+                message["role"] = "system"
     if isinstance(messages, list):
         rewritten: list[Any] = []
         for message in messages:
@@ -97,12 +103,25 @@ def prepare_body(payload: dict[str, Any], model: str) -> dict[str, Any]:
                 pass  # 已是数组 → 透传
             tool_calls = item.get("tool_calls")
             if isinstance(tool_calls, list):
-                kept = [tc for tc in tool_calls if isinstance(tc, dict)]
+                kept = []
+                for tc in tool_calls:
+                    if not isinstance(tc, dict):
+                        continue
+                    fn = tc.get("function")
+                    if not isinstance(fn, dict):
+                        continue
+                    # OpenAI function{name,arguments} → SOLO function_call；
+                    # 上游要求 FunctionCall.Name 必填，无 name 的剔除
+                    if not str(fn.get("name") or "").strip():
+                        continue
+                    tc = dict(tc)
+                    tc["function_call"] = fn
+                    del tc["function"]
+                    kept.append(tc)
                 if kept:
                     item["tool_calls"] = kept
                 else:
                     item.pop("tool_calls", None)
-                item.pop("content", None) if not content else None
             rewritten.append(item)
         body["messages"] = rewritten
 

tests/test_m1a_trae.py

@@ -252,13 +252,31 @@ def test_prepare_body_drops_empty_tool_calls_and_content():
     body = prepare_body({"messages": [{"role": "assistant", "content": None,
                                        "tool_calls": []}]}, "m")
     assert "tool_calls" not in body["messages"][0]
-    assert "content" not in body["messages"][0]
+
+
+def test_prepare_body_skips_non_dict_tool_call_entries():
+    """非对象 tool_call 条目跳过（102-103 分支）。"""
+    body = prepare_body({"messages": [{"role": "assistant", "content": None,
+        "tool_calls": ["junk", {"id": "c", "type": "function",
+                                "function": {"name": "bash", "arguments": "{}"}}]}]}, "m")
+    tcs = body["messages"][0]["tool_calls"]
+    assert len(tcs) == 1 and tcs[0]["function_call"]["name"] == "bash"
+    assert body["messages"][0]["content"] is None   # 原实现保留 nil content
 
 
 def test_prepare_body_keeps_valid_tool_calls():
+    """有 name 的 function → function_call；无 function/name 的剔除。"""
     body = prepare_body({"messages": [{"role": "assistant", "content": "x",
-                                       "tool_calls": [{"id": "c"}]}]}, "m")
-    assert body["messages"][0]["tool_calls"] == [{"id": "c"}]
+        "tool_calls": [
+            {"id": "c1", "type": "function",
+             "function": {"name": "bash", "arguments": "{\"a\":1}"}},
+            {"id": "c2", "type": "function", "function": {"arguments": "{}"}},
+            {"id": "c3"},
+        ]}]}, "m")
+    tcs = body["messages"][0]["tool_calls"]
+    assert len(tcs) == 1 and tcs[0]["id"] == "c1"
+    assert tcs[0]["function_call"] == {"name": "bash", "arguments": "{\"a\":1}"}
+    assert "function" not in tcs[0]
 
 
 # ------------------------------------------------------------ 模型解析
@@ -955,3 +973,24 @@ async def stream_chat(self, cred, payload, model):
     waited.clear()
     _ = [e async for e in provider2.stream_chat({"accessToken": "a"}, {}, "m")]
     assert not waited
+
+
+def test_prepare_body_drops_unnamed_function_call():
+    """function_call 无 name 剔除后若全空 → 整个 tool_calls 删除（102-103）。"""
+    body = prepare_body({"messages": [
+        {"role": "assistant", "content": None,
+         "tool_calls": [{"id": "c", "type": "function",
+                         "function": {"arguments": "{}"}}]},
+    ]}, "m")
+    assert "tool_calls" not in body["messages"][0]
+
+
+
+def test_prepare_body_normalizes_developer_role():
+    """TRAE 不认 developer 角色（静默空流 3003）→ 归一 system。"""
+    body = prepare_body({"messages": [
+        {"role": "developer", "content": "You are PI."},
+        {"role": "user", "content": "hi"},
+    ]}, "m")
+    roles = [m["role"] for m in body["messages"]]
+    assert roles == ["system", "user"]