Add HTML support for Telegram notifications and update test coverage

- Enhance `_send_to_telegram` to handle HTML parse mode for compatibility with special characters.
- Introduce `_escape_html` utility to sanitize HTML-specific characters in messages.
- Add `_format_toss_price_recommendation_html` to generate Toss recommendations in HTML format.
- Update `notify_toss_price_recommendation` to utilize HTML formatting for improved message clarity.
- Expand test cases to validate escaping of special characters and correct HTML formatting in recommendations.

Author: robin-watcha

app/monitoring/trade_notifier.py

@@ -312,12 +312,15 @@ def _format_automation_summary(
 
         return "\n".join(parts)
 
-    async def _send_to_telegram(self, message: str) -> bool:
+    async def _send_to_telegram(
+        self, message: str, parse_mode: str = "Markdown"
+    ) -> bool:
         """
         Send message to all configured Telegram chats.
 
         Args:
             message: Message to send
+            parse_mode: Telegram parse mode ("Markdown" or "HTML")
 
         Returns:
             True if at least one message was sent successfully
@@ -335,7 +338,7 @@ async def _send_to_telegram(self, message: str) -> bool:
                     json={
                         "chat_id": chat_id,
                         "text": message,
-                        "parse_mode": "Markdown",
+                        "parse_mode": parse_mode,
                         "disable_web_page_preview": True,
                     },
                 )
@@ -733,7 +736,15 @@ async def notify_toss_sell_recommendation(
             logger.error(f"Failed to send Toss sell recommendation: {e}")
             return False
 
-    def _format_toss_price_recommendation(
+    def _escape_html(self, text: str) -> str:
+        """Escape HTML special characters for Telegram HTML parse mode."""
+        return (
+            text.replace("&", "&")
+            .replace("<", "&lt;")
+            .replace(">", "&gt;")
+        )
+
+    def _format_toss_price_recommendation_html(
         self,
         symbol: str,
         korean_name: str,
@@ -754,7 +765,7 @@ def _format_toss_price_recommendation(
         currency: str = "원",
     ) -> str:
         """
-        Format Toss price recommendation notification with AI analysis.
+        Format Toss price recommendation notification with AI analysis (HTML format).
         """
         is_usd = currency == "$"
         price_fmt = lambda p: f"${p:,.2f}" if is_usd else f"{p:,.0f}{currency}"
@@ -769,27 +780,31 @@ def _format_toss_price_recommendation(
         emoji = decision_emoji.get(decision.lower(), "⚪")
         decision_kr = decision_text.get(decision.lower(), decision)
 
+        # Escape korean_name for HTML
+        safe_name = self._escape_html(korean_name)
+
         parts = [
-            f"📊 *\\[토스\\] {korean_name} ({symbol})*",
+            f"📊 <b>[토스] {safe_name} ({symbol})</b>",
             "",
-            f"*현재가:* {price_fmt(current_price)}",
-            f"*보유:* {toss_quantity}주 (평단가 {price_fmt(toss_avg_price)}, {profit_sign}{profit_percent:.1f}%)",
+            f"<b>현재가:</b> {price_fmt(current_price)}",
+            f"<b>보유:</b> {toss_quantity}주 (평단가 {price_fmt(toss_avg_price)}, {profit_sign}{profit_percent:.1f}%)",
             "",
-            f"{emoji} *AI 판단:* {decision_kr} (신뢰도 {confidence:.0f}%)",
+            f"{emoji} <b>AI 판단:</b> {decision_kr} (신뢰도 {confidence:.0f}%)",
         ]
 
         # 근거 추가
         if reasons:
             parts.append("")
-            parts.append("*근거:*")
+            parts.append("<b>근거:</b>")
             for i, reason in enumerate(reasons[:3], 1):
                 # 긴 근거는 줄임
                 short_reason = reason[:80] + "..." if len(reason) > 80 else reason
-                parts.append(f"  {i}. {short_reason}")
+                safe_reason = self._escape_html(short_reason)
+                parts.append(f"  {i}. {safe_reason}")
 
         # 가격 제안 추가
         parts.append("")
-        parts.append("*가격 제안:*")
+        parts.append("<b>가격 제안:</b>")
 
         if appropriate_buy_min or appropriate_buy_max:
             buy_range = []
@@ -849,6 +864,7 @@ async def notify_toss_price_recommendation(
         Send Toss price recommendation notification with AI analysis.
 
         Always sends regardless of AI decision (buy/hold/sell).
+        Uses HTML parse mode for better compatibility with special characters.
         """
         if not self._enabled:
             return False
@@ -858,7 +874,7 @@ async def notify_toss_price_recommendation(
             return False
 
         try:
-            message = self._format_toss_price_recommendation(
+            message = self._format_toss_price_recommendation_html(
                 symbol=symbol,
                 korean_name=korean_name,
                 current_price=current_price,
@@ -877,7 +893,7 @@ async def notify_toss_price_recommendation(
                 sell_target_max=sell_target_max,
                 currency=currency,
             )
-            return await self._send_to_telegram(message)
+            return await self._send_to_telegram(message, parse_mode="HTML")
         except Exception as e:
             logger.error(f"Failed to send Toss price recommendation: {e}")
             return False

tests/test_kis_manual_holdings_integration.py

@@ -725,3 +725,85 @@ async def get_latest_analysis_by_symbol(self, symbol):
         assert notification_sent[0]["decision"] == "hold"
         assert notification_sent[0]["appropriate_buy_min"] == 23000.0
         assert notification_sent[0]["appropriate_sell_max"] == 28000.0
+
+    def test_format_toss_price_recommendation_html_escapes_special_chars(self):
+        """HTML 포맷 메시지가 특수문자를 올바르게 이스케이프하는지 확인"""
+        from app.monitoring.trade_notifier import TradeNotifier
+
+        notifier = TradeNotifier()
+
+        # 특수문자가 포함된 데이터로 테스트
+        message = notifier._format_toss_price_recommendation_html(
+            symbol="005930",
+            korean_name="삼성전자 <테스트>",  # HTML 특수문자 포함
+            current_price=72000.0,
+            toss_quantity=10,
+            toss_avg_price=70000.0,
+            decision="buy",
+            confidence=75.5,
+            reasons=["RSI < 30 (과매도)", "이평선 & 정배열"],  # 특수문자 포함
+            appropriate_buy_min=68000.0,
+            appropriate_buy_max=70000.0,
+            appropriate_sell_min=75000.0,
+            appropriate_sell_max=78000.0,
+            buy_hope_min=65000.0,
+            buy_hope_max=68000.0,
+            sell_target_min=80000.0,
+            sell_target_max=85000.0,
+            currency="원",
+        )
+
+        # HTML 특수문자가 올바르게 이스케이프 되어야 함
+        assert "&lt;테스트&gt;" in message, "< > 문자가 이스케이프되어야 함"
+        assert "&lt; 30" in message or "RSI &lt; 30" in message, "< 문자가 이스케이프되어야 함"
+        assert "&amp;" in message, "& 문자가 이스케이프되어야 함"
+
+        # <b> 태그는 이스케이프되지 않아야 함 (HTML 포맷팅용)
+        assert "<b>" in message, "볼드 태그는 유지되어야 함"
+        assert "</b>" in message, "볼드 종료 태그는 유지되어야 함"
+
+        # 숫자와 퍼센트 등이 제대로 표시되어야 함
+        assert "72,000원" in message, "현재가가 표시되어야 함"
+        assert "+2.9%" in message, "수익률이 표시되어야 함"
+        assert "76%" in message, "신뢰도가 표시되어야 함 (75.5 -> 76으로 반올림)"
+
+    def test_format_toss_price_recommendation_html_with_parentheses(self):
+        """괄호, 퍼센트 등이 포함된 메시지가 정상적으로 생성되는지 확인"""
+        from app.monitoring.trade_notifier import TradeNotifier
+
+        notifier = TradeNotifier()
+
+        message = notifier._format_toss_price_recommendation_html(
+            symbol="015760",
+            korean_name="한국전력",
+            current_price=25000.0,
+            toss_quantity=10,
+            toss_avg_price=23000.0,
+            decision="sell",
+            confidence=80,
+            reasons=["수익률 8.7% 달성", "목표가(28,000원) 근접"],
+            appropriate_buy_min=22000.0,
+            appropriate_buy_max=23000.0,
+            appropriate_sell_min=26000.0,
+            appropriate_sell_max=28000.0,
+            buy_hope_min=None,
+            buy_hope_max=None,
+            sell_target_min=28000.0,
+            sell_target_max=30000.0,
+            currency="원",
+        )
+
+        # 메시지가 생성되어야 함
+        assert len(message) > 0
+
+        # HTML 태그가 있어야 함
+        assert "<b>" in message
+
+        # 이모지가 있어야 함
+        assert "📊" in message
+        assert "🔴" in message  # sell decision
+
+        # 가격 제안이 있어야 함
+        assert "적정 매수" in message
+        assert "적정 매도" in message
+        assert "매도 목표" in message