fix: address PR review — fix chunk ordering, extract lang from info string

- Fix chunk_blocks empty-acc clause to pass hard_split result as
  initial accumulator instead of concatenating reversed lists
- Extract only first word from code block info string as language
  (handles "python title=..." style info strings)
- Pin formatted HTML in webhook handler test instead of raw text
- Add tests for both fixes

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: T0ha

lib/bodhi/telegram/formatter.ex

@@ -85,8 +85,13 @@ defmodule Bodhi.Telegram.Formatter do
   end
 
   defp render_node(%MDEx.CodeBlock{info: info, literal: text}) do
-    if info != "" do
-      "<pre><code class=\"language-#{escape(info)}\">" <>
+    lang =
+      info
+      |> String.split(" ", parts: 2)
+      |> List.first("")
+
+    if lang != "" do
+      "<pre><code class=\"language-#{escape(lang)}\">" <>
         escape(text) <> "</code></pre>"
     else
       "<pre><code>" <> escape(text) <> "</code></pre>"
@@ -183,7 +188,7 @@ defmodule Bodhi.Telegram.Formatter do
     if String.length(block) <= @max_length do
       chunk_blocks(rest, [block])
     else
-      hard_split(block) ++ chunk_blocks(rest, [])
+      chunk_blocks(rest, hard_split(block))
     end
   end
 

test/bodhi/telegram/formatter_test.exs

@@ -42,6 +42,14 @@ defmodule Bodhi.Telegram.FormatterTest do
                Formatter.format(input)
     end
 
+    test "code block info string uses only first word as language" do
+      input = "```python title=\"example.py\"\nprint(1)\n```"
+
+      {html, _} = Formatter.format(input)
+      assert html =~ "language-python"
+      refute html =~ "title="
+    end
+
     test "link" do
       assert {"<a href=\"http://ex.com\">text</a>", _} =
                Formatter.format("[text](http://ex.com)")
@@ -185,6 +193,17 @@ defmodule Bodhi.Telegram.FormatterTest do
       assert Enum.join(chunks, "") == line
     end
 
+    test "first block hard-split preserves order with subsequent blocks" do
+      # First block > 4096 chars, followed by a normal block
+      big = String.duplicate("a", 3000) <> "\n" <> String.duplicate("b", 3000)
+      small = "tail"
+      text = big <> "\n\n" <> small
+      chunks = Formatter.split(text)
+
+      assert List.last(chunks) =~ "tail"
+      assert Enum.all?(chunks, &(String.length(&1) <= 4096))
+    end
+
     test "empty text returns single empty chunk" do
       assert Formatter.split("") == [""]
     end

test/bodhi/tg_webhook_handler_test.exs

@@ -113,10 +113,11 @@ defmodule Bodhi.TgWebhookHandlerTest do
   describe "send_message/2" do
     test "Sends and saves message correctly", %{chat: chat, bot_user: bot_user} do
       text = Faker.Lorem.paragraph()
+      {expected_html, _} = Bodhi.Telegram.Formatter.format(text)
       chat_id = chat.id
 
       # Expect send_message to be called once with these exact arguments
-      expect(Bodhi.TelegramMock, :send_message, fn ^chat_id, ^text, _opts ->
+      expect(Bodhi.TelegramMock, :send_message, fn ^chat_id, ^expected_html, _opts ->
         {:ok,
          %Telegex.Type.Message{
            from: %Telegex.Type.User{