fix: stop retrying unchanged telegram edits

Handle BadRequest before NetworkError in the message queue.

Treat Telegram's 'message is not modified' response as a successful no-op edit instead of a retryable network failure. This prevents pointless retries and dead-letter churn for unchanged status updates.

Author: deadman96385

dumpyarabot/message_queue.py

@@ -9,7 +9,7 @@
 from pydantic import BaseModel, Field, model_validator
 from rich.console import Console
 from telegram import Bot
-from telegram.error import RetryAfter, TelegramError, NetworkError
+from telegram.error import RetryAfter, TelegramError, NetworkError, BadRequest
 import telegram
 
 from dumpyarabot.config import settings
@@ -505,13 +505,22 @@ async def _process_message(self, message: QueuedMessage) -> bool:
             console.print(f"[green]Successfully processed {message.type.value} message[/green]")
             return True
 
-        except RetryAfter as e:
-            console.print(f"[yellow]Rate limited by Telegram API. Retry after {e.retry_after} seconds[/yellow]")
-            # Re-queue the message with a delay
-            message.scheduled_for = datetime.now(timezone.utc) + timedelta(seconds=e.retry_after)
-            await self._requeue_message(message)
-            return True  # Don't increment retry count for rate limits
-
+        except RetryAfter as e:
+            console.print(f"[yellow]Rate limited by Telegram API. Retry after {e.retry_after} seconds[/yellow]")
+            # Re-queue the message with a delay
+            message.scheduled_for = datetime.now(timezone.utc) + timedelta(seconds=e.retry_after)
+            await self._requeue_message(message)
+            return True  # Don't increment retry count for rate limits
+
+        except BadRequest as e:
+            error_text = str(e)
+            if "message is not modified" in error_text.lower():
+                console.print("[yellow]Skipping no-op edit: message content is unchanged[/yellow]")
+                return True
+
+            console.print(f"[red]Telegram bad request processing message: {e}[/red]")
+            return False
+
         except NetworkError as e:
             console.print(f"[yellow]Network error processing message: {e}[/yellow]")
             message.retry_count += 1