Skip to content

fix(telegram): remove duplicate heading value in table rendering#29333

Closed
Deserett wants to merge 1 commit into
NousResearch:mainfrom
Deserett:main
Closed

fix(telegram): remove duplicate heading value in table rendering#29333
Deserett wants to merge 1 commit into
NousResearch:mainfrom
Deserett:main

Conversation

@Deserett

Copy link
Copy Markdown

When converting markdown tables to Telegram-friendly bullet lists, the first cell was used as the row heading but remained in data_cells, causing duplicate output like:

Evening (18:00–21:00)
• Time: Evening (18:00–21:00) ← duplicate
• Temp: +17°C

Now the first cell is removed from data_cells after being extracted as the heading, eliminating the duplication.

Fixes: _render_table_block_for_telegram() in gateway/platforms/telegram.py

When converting markdown tables to Telegram-friendly bullet lists,
the first cell was used as the row heading but remained in data_cells,
causing duplicate output like:

  **Вечер (18:00–21:00)**
  • Время: Вечер (18:00–21:00)  ← duplicate
  • Температура: +17°C

Now the first cell is removed from data_cells after being extracted
as the heading, eliminating the duplication.

Fixes: _render_table_block_for_telegram() in gateway/platforms/telegram.py
@alt-glitch alt-glitch added type/bug Something isn't working P3 Low — cosmetic, nice to have platform/telegram Telegram bot adapter comp/gateway Gateway runner, session dispatch, delivery labels May 20, 2026
@Deserett Deserett closed this May 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/gateway Gateway runner, session dispatch, delivery P3 Low — cosmetic, nice to have platform/telegram Telegram bot adapter type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants