telegram-support-bot 0.1.13 → 0.1.15

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0e2776425f88d7b3255ed8130fbc382325c5c50234963b4cc4d626092e16defd
-  data.tar.gz: 7529852a1df10a48cf1b1381fb0d04822a9fc70875340e0cae3afb36da0430d2
+  metadata.gz: 628391b6587ec2b35dc5ce2984276eb42f4f5d5e992205ef06682f26ad41b8cf
+  data.tar.gz: 9edfd943aee7356088dede4f647609ee9f9c6cc34d9ba82a3bbdb61feef241fa
 SHA512:
-  metadata.gz: cc195cf56e0bdd9299d65342ab7619d9bc091727a09c66f3d4e56a4dde5689b3083cf0cc0531ab7784f85bb600b9b097c5ab056f62f617b2a420b47b980f54b5
-  data.tar.gz: d931f2c70b662d7167cc670a3e2b8eb6f88cfb8b3036dbdcd2f95f50b5c643762f1309921e0c593ce1b68ce38ff579e648bdbd751ad825074690a82a50705a76
+  metadata.gz: 98fb134361de2705a30e239708f317565aaa50728a1a6516516ea6e4728bb216b1c6e2a7195996a8337fe46d85b8ddb77b3185122ada7f9b7088666742d64d0e
+  data.tar.gz: 205c10c6546720fabe6d1167846409f9f08cfd9fa264ee8e2218f973c032556d5742766c069cb3a1fedced5e2db6489a220601bf315a5f91d4841349daa27f45

data/CHANGELOG.md

@@ -4,11 +4,19 @@ All notable changes to this project will be documented in this file.
 
 ## [Unreleased]
 
-## [0.1.13] - 2026-02-26
+## [0.1.15] - 2026-02-26
 
 ### Added
 - Optional `forward_start_to_support` configuration to forward the first user `/start`
   message to support chat, so the team can proactively start the conversation.
+- Update-level deduplication by Telegram `update_id` to prevent repeated replies/forwards
+  when webhook deliveries are retried.
+
+### Fixed
+- Initial `/start` forwarding is now fail-safe: errors in forwarding/persisting first-message
+  marker no longer crash update processing and block subsequent updates.
+- Redis-backed marker storage for update/start dedup now writes JSON objects (not scalar booleans),
+  fixing `JSON::GeneratorError: only generation of JSON objects or arrays allowed` on stricter runtimes.
 
 ## [0.1.12] - 2026-02-26
 

data/README.md

@@ -54,6 +54,9 @@ TelegramSupportBot.configure do |config|
   config.forward_start_to_support = false
   # Optional: block forwarding until contact is shared.
   config.require_contact_for_support = false
+  # Optional: deduplicate repeated Telegram deliveries by update_id.
+  # Keep > 0 (default: 24h) to avoid repeated /start replies/forwards on retries.
+  config.processed_update_ttl_seconds = 24 * 60 * 60
   # Optional callback to persist/lookup user profile in your app.
   config.on_contact_received = ->(profile) { YourUserMatcher.sync_from_telegram(profile) }
   # Optional callback for user-chat commands other than /start.
@@ -245,6 +248,7 @@ TelegramSupportBot.configure do |config|
   # config.mapping_ttl_seconds = 30 * 24 * 60 * 60
   # config.reaction_count_ttl_seconds = 7 * 24 * 60 * 60
   # config.user_profile_ttl_seconds = nil
+  # config.processed_update_ttl_seconds = 24 * 60 * 60
 end
 ```
 

data/lib/telegram_support_bot/configuration.rb

@@ -9,7 +9,7 @@ module TelegramSupportBot
                   :contact_received_message, :contact_invalid_message, :forward_start_to_support, :on_contact_received,
                   :on_user_command,
                   :state_store, :state_store_options, :mapping_ttl_seconds,
-                  :reaction_count_ttl_seconds, :user_profile_ttl_seconds
+                  :reaction_count_ttl_seconds, :user_profile_ttl_seconds, :processed_update_ttl_seconds
 
     def initialize
       @adapter                 = :telegram_bot
@@ -33,6 +33,7 @@ module TelegramSupportBot
       @mapping_ttl_seconds = 30 * 24 * 60 * 60
       @reaction_count_ttl_seconds = 7 * 24 * 60 * 60
       @user_profile_ttl_seconds = nil
+      @processed_update_ttl_seconds = 24 * 60 * 60
     end
   end
 end

data/lib/telegram_support_bot/state_store.rb

@@ -37,6 +37,7 @@ module TelegramSupportBot
           mapping_ttl_seconds: configuration.mapping_ttl_seconds,
           reaction_count_ttl_seconds: configuration.reaction_count_ttl_seconds,
           user_profile_ttl_seconds: configuration.user_profile_ttl_seconds,
+          processed_update_ttl_seconds: configuration.processed_update_ttl_seconds,
           **options
         )
       when :redis
@@ -46,6 +47,7 @@ module TelegramSupportBot
           mapping_ttl_seconds: configuration.mapping_ttl_seconds,
           reaction_count_ttl_seconds: configuration.reaction_count_ttl_seconds,
           user_profile_ttl_seconds: configuration.user_profile_ttl_seconds,
+          processed_update_ttl_seconds: configuration.processed_update_ttl_seconds,
           **options
         )
       else

data/lib/telegram_support_bot/state_stores/memory.rb

@@ -12,6 +12,7 @@ module TelegramSupportBot
         @reaction_count_state = {}
         @user_profiles = {}
         @start_forwarded_users = {}
+        @processed_updates = {}
       end
 
       def message_map
@@ -59,6 +60,15 @@ module TelegramSupportBot
         )
       end
 
+      def processed_updates
+        @processed_updates_proxy ||= StateStore::MapProxy.new(
+          get_proc:   ->(key) { get_processed_update(key) },
+          set_proc:   ->(key, value) { set_processed_update(key, value) },
+          clear_proc: -> { clear_processed_updates },
+          size_proc:  -> { processed_updates_size }
+        )
+      end
+
       def get_message_mapping(key)
         synchronize { @message_map[normalize_key(key)] }
       end
@@ -139,6 +149,22 @@ module TelegramSupportBot
         synchronize { @start_forwarded_users.size }
       end
 
+      def get_processed_update(key)
+        synchronize { @processed_updates[normalize_key(key)] }
+      end
+
+      def set_processed_update(key, value)
+        synchronize { @processed_updates[normalize_key(key)] = value }
+      end
+
+      def clear_processed_updates
+        synchronize { @processed_updates.clear }
+      end
+
+      def processed_updates_size
+        synchronize { @processed_updates.size }
+      end
+
       private
 
       def synchronize(&block)

data/lib/telegram_support_bot/state_stores/redis.rb

@@ -8,7 +8,8 @@ module TelegramSupportBot
       DEFAULT_NAMESPACE = 'telegram_support_bot'
 
       def initialize(url: nil, redis: nil, namespace: DEFAULT_NAMESPACE,
-                     mapping_ttl_seconds: nil, reaction_count_ttl_seconds: nil, user_profile_ttl_seconds: nil, **_options)
+                     mapping_ttl_seconds: nil, reaction_count_ttl_seconds: nil, user_profile_ttl_seconds: nil,
+                     processed_update_ttl_seconds: nil, **_options)
         @redis = redis || begin
           require 'redis'
           ::Redis.new(url: url)
@@ -19,6 +20,7 @@ module TelegramSupportBot
         @mapping_ttl_seconds = mapping_ttl_seconds
         @reaction_count_ttl_seconds = reaction_count_ttl_seconds
         @user_profile_ttl_seconds = user_profile_ttl_seconds
+        @processed_update_ttl_seconds = processed_update_ttl_seconds
       end
 
       def message_map
@@ -66,6 +68,15 @@ module TelegramSupportBot
         )
       end
 
+      def processed_updates
+        @processed_updates_proxy ||= StateStore::MapProxy.new(
+          get_proc:   ->(key) { get_processed_update(key) },
+          set_proc:   ->(key, value) { set_processed_update(key, value) },
+          clear_proc: -> { clear_processed_updates },
+          size_proc:  -> { processed_updates_size }
+        )
+      end
+
       def get_message_mapping(key)
         payload = parse_json(@redis.get(map_key(:message_map, key)))
         symbolize_hash(payload)
@@ -149,6 +160,22 @@ module TelegramSupportBot
         count_by_prefix(prefix(:start_forwarded_users))
       end
 
+      def get_processed_update(key)
+        parse_json(@redis.get(map_key(:processed_updates, key)))
+      end
+
+      def set_processed_update(key, value)
+        write_json(map_key(:processed_updates, key), value, @processed_update_ttl_seconds)
+      end
+
+      def clear_processed_updates
+        delete_by_prefix(prefix(:processed_updates))
+      end
+
+      def processed_updates_size
+        count_by_prefix(prefix(:processed_updates))
+      end
+
       private
 
       def prefix(name)

data/lib/telegram_support_bot/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module TelegramSupportBot
-  VERSION = "0.1.13"
+  VERSION = "0.1.15"
 end

data/lib/telegram_support_bot.rb

@@ -63,6 +63,10 @@ module TelegramSupportBot
       state_store(bot_key).start_forwarded_users
     end
 
+    def processed_updates(bot_key = nil)
+      state_store(bot_key).processed_updates
+    end
+
     def user_profile(chat_id, bot: nil)
       profiles = user_profiles(bot)
       profiles[chat_id] || profiles[chat_id.to_s] || profiles[chat_id.to_i]
@@ -75,6 +79,9 @@ module TelegramSupportBot
 
     def process_update(update, bot: DEFAULT_BOT_KEY)
       with_bot_context(bot) do
+        update_id = update['update_id'] || update[:update_id]
+        return if duplicate_update?(update_id)
+
         # Handle different types of updates
         if update['message']
           # Process standard messages
@@ -91,6 +98,8 @@ module TelegramSupportBot
           # Log or handle unknown update types
           puts "Received an unknown type of update: #{update}"
         end
+
+        mark_update_as_processed(update_id)
       end
     end
 
@@ -517,11 +526,45 @@ module TelegramSupportBot
       return unless should_forward_start_to_support?(chat_id)
       return unless forward_message_to_support_chat(message, chat_id: chat_id)
 
-      start_forwarded_users[chat_id] = true
+      start_forwarded_users[chat_id] = start_forwarded_marker_value
+    rescue StandardError => error
+      warn_start_forwarding_failure(chat_id: chat_id, message_id: message['message_id'], error: error)
     end
 
     def start_forwarded_to_support?(chat_id)
-      !start_forwarded_users[chat_id].nil?
+      marker_present?(start_forwarded_users[chat_id])
+    end
+
+    def duplicate_update?(update_id)
+      return false if update_id.nil?
+
+      marker_present?(processed_updates[update_id])
+    end
+
+    def mark_update_as_processed(update_id)
+      return if update_id.nil?
+
+      processed_updates[update_id] = processed_update_marker_value
+    end
+
+    def marker_present?(value)
+      return false if value.nil?
+      return value if value == true || value == false
+      return value.fetch(:present, value['present']) if value.is_a?(Hash)
+
+      true
+    end
+
+    def start_forwarded_marker_value
+      { present: true }
+    end
+
+    def processed_update_marker_value
+      { present: true }
+    end
+
+    def warn_start_forwarding_failure(chat_id:, message_id:, error:)
+      warn "Failed to forward initial /start for chat_id=#{chat_id} message_id=#{message_id}: #{error.class}: #{error.message}"
     end
 
     def contact_known_for_user?(chat_id)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: telegram-support-bot
 version: !ruby/object:Gem::Version
-  version: 0.1.13
+  version: 0.1.15
 platform: ruby
 authors:
 - Max Buslaev