chore: deprecate yaml mention extractor compatibility path

Author: gqy20

src/issuelab/response_processor.py

@@ -10,6 +10,7 @@
 import os
 import re
 import subprocess
+import warnings
 from pathlib import Path
 from typing import Any, cast
 
@@ -163,6 +164,15 @@ def _extract_sources_from_parsed_yaml(parsed: Any) -> list[str]:
 
 
 def extract_mentions_from_yaml(response_text: str) -> list[str]:
+    """Deprecated compatibility helper.
+
+    Main trigger pipeline now parses mentions from controlled markdown sections.
+    """
+    warnings.warn(
+        "extract_mentions_from_yaml is deprecated; use extract_controlled_mentions instead",
+        DeprecationWarning,
+        stacklevel=2,
+    )
     yaml_text = extract_yaml_block(response_text)
     if not yaml_text:
         return []
@@ -357,7 +367,7 @@ def _normalize_agent_output(response_text: str, agent_name: str | None) -> tuple
             parsed_confidence = str(parsed.get("confidence", "")).lower()
             if parsed_confidence in {"high", "medium", "low"}:
                 confidence = parsed_confidence
-            parsed_mentions = extract_mentions_from_yaml(response_text)
+            parsed_mentions = extract_controlled_mentions(response_text)
             parsed_sources = _extract_sources_from_parsed_yaml(parsed)
 
     def _yaml_escape(value: str) -> str:

tests/test_response_processor.py

@@ -2,9 +2,11 @@
 
 from unittest.mock import patch
 
+import pytest
+
 
 class TestExtractMentionsFromYaml:
-    """测试从 YAML 提取 mentions"""
+    """测试 YAML mentions 兼容层（已废弃）"""
 
     def test_extract_mentions_list(self):
         from issuelab.response_processor import extract_mentions_from_yaml
@@ -20,7 +22,8 @@ def test_extract_mentions_list(self):
   - bob
 confidence: "high"
 ```"""
-        assert extract_mentions_from_yaml(text) == ["alice", "bob"]
+        with pytest.warns(DeprecationWarning):
+            assert extract_mentions_from_yaml(text) == ["alice", "bob"]
 
     def test_extract_mentions_with_at_prefix(self):
         from issuelab.response_processor import extract_mentions_from_yaml
@@ -34,7 +37,8 @@ def test_extract_mentions_with_at_prefix(self):
   - "@delta"
 confidence: "medium"
 ```"""
-        assert extract_mentions_from_yaml(text) == ["charlie", "delta"]
+        with pytest.warns(DeprecationWarning):
+            assert extract_mentions_from_yaml(text) == ["charlie", "delta"]
 
     def test_extract_mentions_invalid_items_filtered(self):
         from issuelab.response_processor import extract_mentions_from_yaml
@@ -49,12 +53,14 @@ def test_extract_mentions_invalid_items_filtered(self):
   - "ok_user"
 confidence: "low"
 ```"""
-        assert extract_mentions_from_yaml(text) == ["ok_user"]
+        with pytest.warns(DeprecationWarning):
+            assert extract_mentions_from_yaml(text) == ["ok_user"]
 
     def test_no_yaml_mentions(self):
         from issuelab.response_processor import extract_mentions_from_yaml
 
-        assert extract_mentions_from_yaml("No mentions here") == []
+        with pytest.warns(DeprecationWarning):
+            assert extract_mentions_from_yaml("No mentions here") == []
 
 
 class TestResponseFormatConfig: