fix(ci): 拒绝截断的审查正文

只有 finish_reason=stop 且正文有效时才发布；length 截断即重试，避免 sticky 评论包含半条 finding。

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: mirror29

scripts/deepseek_review.py

@@ -141,10 +141,10 @@ def _call_deepseek(api_key: str, title: str, diff: str, rules: str) -> str:
             if attempt + 1 < MAX_ATTEMPTS:
                 continue
             raise
-        content = body["choices"][0]["message"].get("content")
-        if _is_valid_review(content):
-            return content.strip()
         choice = body["choices"][0]
+        content = choice["message"].get("content")
+        if choice.get("finish_reason") == "stop" and _is_valid_review(content):
+            return content.strip()
         usage = body.get("usage", {})
         reasoning_tokens = usage.get("completion_tokens_details", {}).get(
             "reasoning_tokens", "unknown"

tests/test_deepseek_review.py

@@ -24,8 +24,9 @@ def _load_module():
 
 
 class _FakeResponse:
-    def __init__(self, content: str = "LGTM") -> None:
+    def __init__(self, content: str = "LGTM", finish_reason: str = "stop") -> None:
         self._content = content
+        self._finish_reason = finish_reason
 
     def __enter__(self):
         return self
@@ -35,7 +36,14 @@ def __exit__(self, *_args):
 
     def read(self) -> bytes:
         return json.dumps(
-            {"choices": [{"message": {"content": self._content}}]}
+            {
+                "choices": [
+                    {
+                        "finish_reason": self._finish_reason,
+                        "message": {"content": self._content},
+                    }
+                ]
+            }
         ).encode()
 
 
@@ -74,6 +82,25 @@ def fake_urlopen(request, *, timeout):
         self.assertIn("## 项目规则（CLAUDE.md）\nproject rules", payload["messages"][1]["content"])
         self.assertEqual(captured["timeout"], module.TIMEOUT_S)
 
+    def test_truncated_content_retries_instead_of_publishing_partial_review(self) -> None:
+        module = _load_module()
+        responses = [
+            _FakeResponse("未完成的 finding", finish_reason="length"),
+            _FakeResponse("完整 review"),
+        ]
+
+        with patch.object(
+            module.urllib.request,
+            "urlopen",
+            side_effect=responses,
+        ) as urlopen:
+            result = module._call_deepseek(
+                "secret-value", "PR title", "diff body", "project rules"
+            )
+
+        self.assertEqual(result, "完整 review")
+        self.assertEqual(urlopen.call_count, 2)
+
     def test_empty_content_retries_then_returns_review(self) -> None:
         module = _load_module()
         responses = [_FakeResponse(""), _FakeResponse("最终 review")]