feat: Collapse multiple newlines to a maximum of two in Telegram HTML conversion and add a corresponding test.

Author: tuvshinjargal

app/Services/Telegram/TelegramMessageService.php

@@ -181,6 +181,9 @@ private function convertMarkdownToHtml(string $markdown): string
         // Strip unsupported tags
         // Telegram supports: <b>, <strong>, <i>, <em>, <u>, <ins>, <s>, <strike>, <del>,
         // <span class="tg-spoiler">, <a>, <code>, <pre>, <blockquote>
-        return trim(strip_tags($html, '<b><strong><i><em><u><ins><s><strike><del><a><code><pre><blockquote>'));
+        $html = strip_tags($html, '<b><strong><i><em><u><ins><s><strike><del><a><code><pre><blockquote>');
+
+        // Collapse multiple newlines into max 2
+        return trim(preg_replace('/\n{3,}/', "\n\n", $html));
     }
 }

tests/Unit/Services/Telegram/TelegramHtmlConversionTest.php

@@ -39,6 +39,18 @@
         ->toContain('• Item 2');
 });
 
+test('handles complex structure without excessive whitespace', function (): void {
+    $service = new TelegramMessageService();
+    $reflection = new ReflectionClass(TelegramMessageService::class);
+    $method = $reflection->getMethod('convertMarkdownToHtml');
+
+    $markdown = "Paragraph 1\n\n- Item 1\n- Item 2\n\nParagraph 2";
+    $html = $method->invoke($service, $markdown);
+
+    // We expect max 2 newlines ideally
+    expect($html)->not->toContain("\n\n\n");
+});
+
 test('safe message length is respected', function (): void {
     $reflection = new ReflectionClass(TelegramMessageService::class);
     $constant = $reflection->getReflectionConstant('SAFE_MESSAGE_LENGTH');