fix: 移除 HistoryItem 响应 Schema 中 sentiment_score 的… (#942) (#962)

* fix(issue-942): [bug]

* fix(review-feedback-962): address latest review comments

Author: ZhuLinsen

api/v1/schemas/history.py

@@ -23,10 +23,8 @@ class HistoryItem(BaseModel):
     stock_name: Optional[str] = Field(None, description="股票名称")
     report_type: Optional[str] = Field(None, description="报告类型")
     sentiment_score: Optional[int] = Field(
-        None, 
-        description="情绪评分 (0-100)",
-        ge=0,
-        le=100
+        None,
+        description="情绪评分（历史数据可能超出 0-100 范围，读取时不做约束）",
     )
     operation_advice: Optional[str] = Field(None, description="操作建议")
     created_at: Optional[str] = Field(None, description="创建时间")
@@ -133,10 +131,8 @@ class ReportSummary(BaseModel):
     operation_advice: Optional[str] = Field(None, description="操作建议")
     trend_prediction: Optional[str] = Field(None, description="趋势预测")
     sentiment_score: Optional[int] = Field(
-        None, 
-        description="情绪评分 (0-100)",
-        ge=0,
-        le=100
+        None,
+        description="情绪评分（历史数据可能超出 0-100 范围，读取时不做约束）",
     )
     sentiment_label: Optional[str] = Field(None, description="情绪标签")
 

docs/CHANGELOG.md

@@ -12,6 +12,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/).
 <!-- 新条目格式：- [类型] 描述（类型取值：新功能/改进/修复/文档/测试/chore）-->
 <!-- 每条独立一行追加到本段末尾，无需分类标题，合并时冲突最小 -->
 - [修复] **MiniMax-M2.7 模型连接测试支持** — 修复 LLM 通道连接测试在 MiniMax-M2.7 模型下返回 "Empty response" 的问题；增加了 `max_tokens` 上限（8→256）以容纳 MiniMax 思考过程，并添加 `content_blocks` 格式解析逻辑统一处理 MiniMax 响应格式差异。
+- [修复] 移除 `HistoryItem` 与 `ReportSummary` 响应 Schema 中 `sentiment_score` 的 `ge=0/le=100` 约束（fixes #942）——历史库中存储的超范围负值或大于 100 的情绪评分不再触发 Pydantic ValidationError，历史列表与详情接口恢复正常返回。
 
 ## [3.12.0] - 2026-04-01
 

tests/test_analysis_history.py

@@ -643,5 +643,69 @@ def test_delete_history_api_deletes_selected_records(self, mock_auth) -> None:
             self.assertIsNotNone(session.query(AnalysisHistory).filter(AnalysisHistory.id == record_id_2).first())
 
 
+class HistoryItemSchemaNegativeSentimentTest(unittest.TestCase):
+    """Regression: HistoryItem / ReportSummary must accept out-of-range sentiment_score from DB rows."""
+
+    @classmethod
+    def setUpClass(cls) -> None:
+        """Import schema classes once for all tests, skipping gracefully when deps are missing."""
+        try:
+            from api.v1.schemas.history import HistoryItem, ReportSummary  # type: ignore
+        except ModuleNotFoundError:
+            cls.HistoryItem = None
+            cls.ReportSummary = None
+        else:
+            cls.HistoryItem = HistoryItem
+            cls.ReportSummary = ReportSummary
+
+    def test_negative_sentiment_score_does_not_raise(self) -> None:
+        """Bug #942: sentiment_score=-22 in DB should not cause Pydantic ValidationError."""
+        if self.HistoryItem is None:
+            self.skipTest("fastapi / pydantic not installed in this test environment")
+
+        item = self.HistoryItem(query_id="q1", stock_code="600519", sentiment_score=-22)
+        self.assertEqual(item.sentiment_score, -22)
+
+    def test_out_of_range_high_sentiment_score_does_not_raise(self) -> None:
+        """HistoryItem should also accept scores above 100 from legacy data."""
+        if self.HistoryItem is None:
+            self.skipTest("fastapi / pydantic not installed in this test environment")
+
+        item = self.HistoryItem(query_id="q2", stock_code="600519", sentiment_score=150)
+        self.assertEqual(item.sentiment_score, 150)
+
+    def test_none_sentiment_score_is_allowed(self) -> None:
+        """HistoryItem.sentiment_score=None should still be valid (optional field)."""
+        if self.HistoryItem is None:
+            self.skipTest("fastapi / pydantic not installed in this test environment")
+
+        item = self.HistoryItem(query_id="q3", stock_code="600519", sentiment_score=None)
+        self.assertIsNone(item.sentiment_score)
+
+    def test_report_summary_negative_sentiment_score_does_not_raise(self) -> None:
+        """ReportSummary.sentiment_score should also accept negative values from legacy DB rows."""
+        if self.ReportSummary is None:
+            self.skipTest("fastapi / pydantic not installed in this test environment")
+
+        summary = self.ReportSummary(sentiment_score=-22)
+        self.assertEqual(summary.sentiment_score, -22)
+
+    def test_report_summary_out_of_range_high_sentiment_score_does_not_raise(self) -> None:
+        """ReportSummary.sentiment_score should also accept scores above 100 from legacy data."""
+        if self.ReportSummary is None:
+            self.skipTest("fastapi / pydantic not installed in this test environment")
+
+        summary = self.ReportSummary(sentiment_score=150)
+        self.assertEqual(summary.sentiment_score, 150)
+
+    def test_report_summary_none_sentiment_score_is_allowed(self) -> None:
+        """ReportSummary.sentiment_score=None should still be valid (optional field)."""
+        if self.ReportSummary is None:
+            self.skipTest("fastapi / pydantic not installed in this test environment")
+
+        summary = self.ReportSummary(sentiment_score=None)
+        self.assertIsNone(summary.sentiment_score)
+
+
 if __name__ == "__main__":
     unittest.main()