fix(ui): preserve complete Telegram message text

Author: Aethersailor

src/handlers/group_handler.py

@@ -20,16 +20,14 @@ class GroupHandler:
     """群组消息处理器"""
 
     @staticmethod
-    def _escape_inline_code(value: object, max_length: int = 253) -> str:
-        """Keep a dynamic value on one safe legacy-Markdown code line."""
-        text = " ".join(str(value).split())[:max_length]
+    def _escape_inline_code(value: object) -> str:
+        """Keep a complete dynamic value on one safe legacy-Markdown code line."""
+        text = " ".join(str(value).split())
         return text.replace("\\", "\\\\").replace("`", "\\`")
 
-    def _escape_detail(self, value: object, max_length: int = 160) -> str:
-        """Format a short, single-line dynamic explanation."""
+    def _escape_detail(self, value: object) -> str:
+        """Format a complete dynamic explanation and let Telegram wrap it."""
         text = " ".join(str(value).split())
-        if len(text) > max_length:
-            text = f"{text[: max_length - 3]}..."
         text = text.replace("\\", "\\\\")
         return self.handler_manager.escape_markdown(text)
 
@@ -244,7 +242,7 @@ async def _process_domain_request(
                 if user is not None
                 else self.handler_manager.escape_markdown(username)
             )
-            identity = " ".join(str(identity).split())[:80]
+            identity = " ".join(str(identity).split())
             safe_domain = self._escape_inline_code(domain)
             processing_msg = await message.reply_text(
                 "🔍 *正在检查域名*\n\n"

src/handlers/handler_manager.py

@@ -5,7 +5,6 @@
 
 import secrets
 import time
-import unicodedata
 from datetime import datetime, timezone
 from typing import Dict, Any, Optional
 from collections import defaultdict
@@ -73,7 +72,6 @@ def __init__(self, config: Config, data_manager: DataManager, application=None):
         self.MAX_DESCRIPTION_LENGTH = 20
         self.MAX_ADDS_PER_HOUR = 50
         self.MAX_DETAIL_LINES = 4
-        self.MAX_DETAIL_LINE_LENGTH = 56
         self.STATE_TTL = 1800
         self.ACTION_TTL = 900
         self.MAX_USER_STATES = 4096
@@ -565,7 +563,7 @@ def _raw_telegram_identity(user) -> str:
 
     def _format_telegram_identity(self, user) -> str:
         """Format a Telegram identity for Markdown-visible messages."""
-        identity = self._truncate_display(self._raw_telegram_identity(user), 40)
+        identity = self._single_line_display(self._raw_telegram_identity(user))
         identity = self.escape_markdown(identity)
         if getattr(user, "username", None):
             return f"@{identity}"
@@ -610,45 +608,18 @@ def _repo_and_home_keyboard(self) -> InlineKeyboardMarkup:
         )
 
     @staticmethod
-    def _display_width(value: object) -> int:
-        width = 0
-        for char in str(value):
-            if unicodedata.combining(char):
-                continue
-            width += 2 if unicodedata.east_asian_width(char) in {"W", "F"} else 1
-        return width
-
-    @classmethod
-    def _truncate_display(cls, value: object, max_width: int = 56) -> str:
-        """Collapse and bound dynamic text by its approximate rendered width."""
-        text = " ".join(str(value).split())
-        if cls._display_width(text) <= max_width:
-            return text
-        visible = []
-        width = 0
-        for char in text:
-            char_width = (
-                0
-                if unicodedata.combining(char)
-                else 2
-                if unicodedata.east_asian_width(char) in {"W", "F"}
-                else 1
-            )
-            if width + char_width > max_width - 1:
-                break
-            visible.append(char)
-            width += char_width
-        return "".join(visible).rstrip() + "…"
+    def _single_line_display(value: object) -> str:
+        """Normalize dynamic text without discarding visible content."""
+        return " ".join(str(value).split())
 
     def _format_rule_matches(self, matches: list, limit: Optional[int] = None) -> str:
-        """Bound dynamic GitHub matches so Telegram messages remain below limits."""
+        """Format dynamic GitHub matches as complete, independently wrapped rows."""
         if not matches:
             return ""
         limit = limit or getattr(self, "MAX_DETAIL_LINES", 4)
-        line_limit = getattr(self, "MAX_DETAIL_LINE_LENGTH", 56)
         lines = []
         for match in matches[:limit]:
-            rule = self._truncate_display(match.get("rule", ""), line_limit - 12)
+            rule = self._single_line_display(match.get("rule", ""))
             lines.append(f"• 第 {match.get('line', '?')} 行：{self.escape_markdown(rule)}")
         remaining = len(matches) - limit
         if remaining > 0:
@@ -659,7 +630,7 @@ def _format_rule_matches(self, matches: list, limit: Optional[int] = None) -> st
     def _format_value_list(values: list, limit: int = 3) -> str:
         """Format network values one-per-line for narrow Telegram clients."""
         visible = [
-            HandlerManager._truncate_display(value, 56)
+            HandlerManager._single_line_display(value)
             for value in list(values or [])[:limit]
         ]
         remaining = len(values or []) - len(visible)
@@ -704,9 +675,7 @@ def _build_add_review_text(self, domain: str, check_result: dict, user) -> tuple
                 [
                     "",
                     "💡 *判断依据*",
-                    self.escape_markdown(
-                        self._truncate_display(recommendation, 56)
-                    ),
+                    self.escape_markdown(self._single_line_display(recommendation)),
                 ]
             )
         if not should_reject:
@@ -880,7 +849,7 @@ async def _display_message_result(
 
     def _build_main_menu_text(self, username: str) -> str:
         """构建主菜单文案"""
-        username = self.escape_markdown(self._truncate_display(username, 24))
+        username = self.escape_markdown(self._single_line_display(username))
         repo = str(self.config.GITHUB_REPO).strip()
         repo_label = self.escape_markdown(repo)
         return "\n".join(
@@ -1024,9 +993,7 @@ def _format_detail_lines(
         limit = limit or self.MAX_DETAIL_LINES
         lines = []
         for detail in details[:limit]:
-            detail = self._truncate_display(
-                detail, self.MAX_DETAIL_LINE_LENGTH - 2
-            )
+            detail = self._single_line_display(detail)
             lines.append(f"• {self.escape_markdown(detail)}")
 
         remaining = len(details) - limit
@@ -1048,7 +1015,7 @@ def _build_query_summary_text(
         github_exists = bool(github_result.get("exists"))
         check_failed = "error" in check_result
         matches = github_result.get("matches", [])
-        visible_domain = self._truncate_display(domain, 56)
+        visible_domain = self._single_line_display(domain)
 
         if github_unavailable:
             github_status = "暂时无法读取"
@@ -1109,8 +1076,13 @@ def _build_query_summary_text(
             and not github_exists
             and not in_geosite
         ):
-            visible = self._truncate_display(recommendation, 48)
-            lines.append(f"💡 综合判断：{self.escape_markdown(visible)}")
+            lines.extend(
+                [
+                    "",
+                    "💡 *综合判断*",
+                    self.escape_markdown(self._single_line_display(recommendation)),
+                ]
+            )
         return "\n".join(lines)
 
     def _build_query_detail_text(
@@ -1120,7 +1092,7 @@ def _build_query_detail_text(
         check_result: dict,
     ) -> str:
         """Build a bounded technical page without repeating the summary."""
-        visible_domain = self._truncate_display(domain, 56)
+        visible_domain = self._single_line_display(domain)
         lines = ["🔎 *技术详情*", "", f"`{visible_domain}`"]
         github_unavailable = bool(github_result.get("error"))
         matches = github_result.get("matches", [])

src/services/group_service.py

@@ -42,9 +42,9 @@ def _escape_markdown(text: str) -> str:
         return text
 
     @staticmethod
-    def _escape_inline_code(value: object, max_length: int = 253) -> str:
-        """Keep a dynamic value on one safe legacy-Markdown code line."""
-        text = " ".join(str(value).split())[:max_length]
+    def _escape_inline_code(value: object) -> str:
+        """Keep a complete dynamic value on one safe legacy-Markdown code line."""
+        text = " ".join(str(value).split())
         return text.replace("\\", "\\\\").replace("`", "\\`")
 
     async def check_user_in_group(

tests/test_group_handler_ui.py

@@ -33,11 +33,12 @@ def _handler_for_result(self, result, *, is_admin=False):
         )
         return handler
 
-    async def test_error_detail_is_single_line_and_bounded(self):
+    async def test_error_detail_is_single_line_and_complete(self):
+        detail = f"bad\n{'x' * 300}*"
         handler = self._handler_for_result(
             {
                 "action": "error",
-                "message": f"bad\n{'x' * 300}*",
+                "message": detail,
             }
         )
         processing = SimpleNamespace(edit_text=AsyncMock())
@@ -52,7 +53,8 @@ async def test_error_detail_is_single_line_and_bounded(self):
         self.assertNotIn("bad\n", text)
         self.assertIn("请稍后再次 @机器人", text)
         self.assertIn("🛡️ 本次操作未修改任何规则", text)
-        self.assertLess(len(text), 320)
+        self.assertIn(f"bad {'x' * 300}\\*", text)
+        self.assertNotIn("...", text)
 
     async def test_uncertain_write_result_warns_before_any_retry(self):
         handler = self._handler_for_result(

tests/test_ui_ux.py

@@ -21,7 +21,6 @@ def _manager(self, rule_bot_client_enabled=True):
             ANNOUNCEMENT_GROUP_ID=-100123 if rule_bot_client_enabled else None,
         )
         manager.MAX_DETAIL_LINES = 4
-        manager.MAX_DETAIL_LINE_LENGTH = 56
         manager.MAX_DESCRIPTION_LENGTH = 20
         return manager
 
@@ -74,6 +73,54 @@ def test_add_review_discloses_exact_rule_and_public_scope(self):
         self.assertNotIn("结论：", text)
         self.assertNotIn("检查详情", text)
 
+    def test_add_review_preserves_complete_recommendation(self):
+        manager = self._manager()
+        manager.domain_checker = SimpleNamespace(
+            get_target_domain_to_add=MagicMock(return_value="codexradar.com"),
+            should_reject=MagicMock(return_value=True),
+        )
+        recommendation = (
+            "❌ 不建议添加可注册域名 codexradar.com："
+            "域名 IP 和 NS 服务器都不在中国大陆"
+        )
+
+        text, _ = manager._build_add_review_text(
+            "codexradar.com",
+            {"recommendation": recommendation},
+            SimpleNamespace(id=2, username="alice", first_name="Alice"),
+        )
+
+        self.assertIn(f"💡 *判断依据*\n{recommendation}", text)
+        self.assertNotIn("…", text)
+
+    def test_query_pages_preserve_complete_dynamic_text(self):
+        manager = self._manager()
+        recommendation = "综合判断：" + "完整内容" * 20
+        detail = "归属检查：" + "详细结果" * 20
+        rule = "DOMAIN-SUFFIX," + "long-label." * 12 + "example.com"
+        check_result = {
+            "domain_ips": [],
+            "second_level_ips": [],
+            "details": [detail],
+            "recommendation": recommendation,
+        }
+        github_result = {
+            "exists": False,
+            "matches": [{"line": 42, "rule": rule}],
+        }
+
+        summary = manager._build_query_summary_text(
+            "example.com", github_result, False, check_result, "ℹ️ *查询完成*"
+        )
+        technical = manager._build_query_detail_text(
+            "example.com", github_result, check_result
+        )
+
+        self.assertIn(f"💡 *综合判断*\n{recommendation}", summary)
+        self.assertIn(detail, technical)
+        self.assertIn(manager.escape_markdown(rule), technical)
+        self.assertNotIn("…", summary + technical)
+
 
 class TestVisibleFlows(unittest.IsolatedAsyncioTestCase):
     @staticmethod
@@ -100,7 +147,6 @@ def _stateful_manager():
         manager.ACTION_TTL = 900
         manager.MAX_USER_STATES = 4096
         manager.MAX_DETAIL_LINES = 4
-        manager.MAX_DETAIL_LINE_LENGTH = 56
         manager.MAX_DESCRIPTION_LENGTH = 20
         manager.MAX_ADDS_PER_HOUR = 50
         return manager