twilio-rails 1.0.1 → 1.1.0

data/lib/twilio/rails/formatter.rb

@@ -1,12 +1,10 @@
 # frozen_string_literal: true
+
 module Twilio
   module Rails
     module Formatter
       extend self
 
-      PHONE_NUMBER_REGEX = /\A\+1[0-9]{10}\Z/
-      PHONE_NUMBER_SEGMENTS_REGEX = /\A\+1([0-9]{3})([0-9]{3})([0-9]{4})\Z/
-
       # Takes in a string or a {Twilio::Rails::PhoneNumber} or something that responds to `to_s` and turns it into a
       # consistently formatted valid north american 10 digit phone number prefixed with 1 and plus. It uses the format
       # Twilio expects which is "+15555555555" or returns `nil` if it cannot be coerced.
@@ -14,17 +12,13 @@ module Twilio
       # @param string [String, Twilio::Rails::PhoneNumber, nil, Object] the input to turn into a phone number string.
       # @return [String, nil] the phone number string or nil.
       def coerce_to_valid_phone_number(string)
-        string = string.number if string.is_a?(Twilio::Rails::PhoneNumber)
-        string = string.to_s.presence
-
-        if string
-          string = string.gsub(/[^0-9]/, "")
-          string = "1#{ string }" unless string.starts_with?("1")
-          string = "+#{ string }"
-          string = nil unless valid_north_american_phone_number?(string)
-        end
+        Twilio::Rails.deprecator.warn(<<~DEPRECATION.strip)
+          Twilio::Rails::Formatter#coerce_to_valid_phone_number(s) is deprecated and will be removed in the next major version.
 
-        string
+          Set Twilio::Rails.config.phone_number_formatter = Twilio::Rails::PhoneNumberFormatter::NorthAmerica.new
+          and use Twilio::Rails.config.phone_number_formatter.coerce(s) instead.
+        DEPRECATION
+        north_america_formatter.coerce(string)
       end
 
       # Takes in a string or a {Twilio::Rails::PhoneNumber} or something that responds to `to_s` and validates it
@@ -33,8 +27,13 @@ module Twilio
       # @param phone_number [String, Twilio::Rails::PhoneNumber, nil] the input to validate as a phone number.
       # @return [true, false]
       def valid_north_american_phone_number?(phone_number)
-        phone_number = phone_number.number if phone_number.is_a?(Twilio::Rails::PhoneNumber)
-        !!phone_number&.match?(PHONE_NUMBER_REGEX)
+        Twilio::Rails.deprecator.warn(<<~DEPRECATION.strip)
+          Twilio::Rails::Formatter#valid_north_american_phone_number?(s) is deprecated and will be removed in the next major version.
+
+          Set Twilio::Rails.config.phone_number_formatter = Twilio::Rails::PhoneNumberFormatter::NorthAmerica.new
+          and use Twilio::Rails.config.phone_number_formatter.valid?(s) instead.
+        DEPRECATION
+        north_america_formatter.valid?(phone_number)
       end
 
       # Takes in a string or a {Twilio::Rails::PhoneNumber} or something that responds to `to_s` and turns it into
@@ -44,11 +43,13 @@ module Twilio
       # @param phone_number [String, Twilio::Rails::PhoneNumber, nil] the input to turn into a phone number string.
       # @return [String] the phone number string or empty string if invalid.
       def to_phone_number_url_param(phone_number)
-        phone_number = coerce_to_valid_phone_number(phone_number)
-        return "" unless phone_number
-        matches = phone_number.match(PHONE_NUMBER_SEGMENTS_REGEX)
-        raise Twilio::Rails::Error, "[to_phone_number_url_param] Phone number marked as valid but could not capture. I made a bad regex: #{ phone_number }" unless matches
-        matches.captures.join("-")
+        Twilio::Rails.deprecator.warn(<<~DEPRECATION.strip)
+          Twilio::Rails::Formatter#to_phone_number_url_param(s) is deprecated and will be removed in the next major version.
+
+          Set Twilio::Rails.config.phone_number_formatter = Twilio::Rails::PhoneNumberFormatter::NorthAmerica.new
+          and use Twilio::Rails.config.phone_number_formatter.to_param(s) instead.
+        DEPRECATION
+        north_america_formatter.to_param(phone_number)
       end
 
       # Takes in a string or a {Twilio::Rails::PhoneNumber} or something that responds to `to_s` and turns it into a
@@ -58,14 +59,13 @@ module Twilio
       # @param phone_number [String, Twilio::Rails::PhoneNumber, nil] the input to turn into a phone number string.
       # @return [String, Object] the phone number string or the original object if invalid.
       def display_phone_number(phone_number)
-        coerced_phone_number = coerce_to_valid_phone_number(phone_number)
-        if coerced_phone_number
-          matches = coerced_phone_number.match(PHONE_NUMBER_SEGMENTS_REGEX)
-          raise Twilio::Rails::Error, "[display_phone_number] Phone number marked as valid but could not capture. I made a bad regex: #{ phone_number }" unless matches
-          "(#{ matches.captures[0] }) #{ matches.captures[1] } #{ matches.captures[2] }"
-        else
-          phone_number
-        end
+        Twilio::Rails.deprecator.warn(<<~DEPRECATION.strip)
+          Twilio::Rails::Formatter#display_phone_number(s) is deprecated and will be removed in the next major version.
+
+          Set Twilio::Rails.config.phone_number_formatter = Twilio::Rails::PhoneNumberFormatter::NorthAmerica.new
+          and use Twilio::Rails.config.phone_number_formatter.display(s) instead.
+        DEPRECATION
+        north_america_formatter.display(phone_number)
       end
 
       # Formats a city, province, and country into a single string, correctly handling blanks, and formatting countries.
@@ -85,9 +85,15 @@ module Twilio
         [
           city.presence&.titleize,
           province,
-          country_name,
+          country_name
         ].reject(&:blank?).join(", ")
       end
+
+      private
+
+      def north_america_formatter
+        @north_america_formatter ||= Twilio::Rails::PhoneNumberFormatter::NorthAmerica.new
+      end
     end
   end
 end

data/lib/twilio/rails/models/message.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Twilio
   module Rails
     module Models

data/lib/twilio/rails/models/phone_call.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Twilio
   module Rails
     module Models

data/lib/twilio/rails/models/phone_caller.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Twilio
   module Rails
     module Models
@@ -21,7 +22,7 @@ module Twilio
           # @param phone_number_string [String, Twilio::Rails::PhoneNumber] The phone number to find the record.
           # @return [Twilio::Rails::Models::PhoneCaller, nil] The phone caller record or `nil` if not found.
           def for(phone_number_string)
-            phone_number = Twilio::Rails::Formatter.coerce_to_valid_phone_number(phone_number_string)
+            phone_number = Twilio::Rails::PhoneNumberFormatter.coerce(phone_number_string)
             find_by(phone_number: phone_number) if phone_number.present?
           end
         end
@@ -45,11 +46,11 @@ module Twilio
 
         # @return [Array<Twilio::Rails::Models::SmsConversation>] All SMS conversations for the phone caller.
         def sms_conversations
-          Twilio::Rails.config.sms_conversation_class.phone_number(self.phone_number)
+          Twilio::Rails.config.sms_conversation_class.phone_number(phone_number)
         end
 
         # Returns the digits as a `String` as entered through the keypad during a phone call as `gather:`. Returns
-        #`nil` if the response is not found, if the response has no digits, or if the response was a timeout. Can
+        # `nil` if the response is not found, if the response has no digits, or if the response was a timeout. Can
         # include both `*` and `#` characters if the caller pressed them.
         #
         # @param prompt [String, Symbol] The prompt handle to query.

data/lib/twilio/rails/models/recording.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Twilio
   module Rails
     module Models

data/lib/twilio/rails/models/response.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Twilio
   module Rails
     module Models
@@ -19,11 +20,11 @@ module Twilio
           delegate :phone_caller, to: :phone_call
 
           scope :completed, -> { where(timeout: false) }
-          scope :recent_transcriptions, ->(number=5) { completed.order(created_at: :desc).where.not(transcription: nil).limit(number) }
+          scope :recent_transcriptions, ->(number = 5) { completed.order(created_at: :desc).where.not(transcription: nil).limit(number) }
           scope :final_timeout_check, ->(count:, prompt_handle:) {
             prompt(prompt_handle).order(created_at: :desc).limit(count)
           }
-          scope :tree, ->(name) { joins(:phone_call).where(phone_calls: { tree_name: name }) }
+          scope :tree, ->(name) { joins(:phone_call).where(phone_calls: {tree_name: name}) }
           scope :prompt, ->(prompt_handle) { where(prompt_handle: prompt_handle) }
           scope :in_order, -> { reorder(created_at: :asc) }
           scope :transcribed, -> { where(transcribed: true) }
@@ -37,9 +38,9 @@ module Twilio
         # @param prompt [String, Symbol, Array] The prompt handle or an array of them.
         # @return [true, false] true if the response is for the given prompt and tree.
         def is?(tree:, prompt:)
-          trees = Array(tree).map { |t| t.is_a?(Twilio::Rails::Phone::Tree) ? t.name : t.to_s }
+          Array(tree).map { |t| t.is_a?(Twilio::Rails::Phone::Tree) ? t.name : t.to_s }
 
-          from?(tree: tree) && Array(prompt).map(&:to_s).reject(&:blank?).include?(self.prompt_handle)
+          from?(tree: tree) && Array(prompt).map(&:to_s).reject(&:blank?).include?(prompt_handle)
         end
 
         # Checks if the response is for a given tree or trees or tree names.
@@ -49,7 +50,7 @@ module Twilio
         def from?(tree:)
           trees = Array(tree).map { |t| t.is_a?(Twilio::Rails::Phone::Tree) ? t.name : t.to_s }
 
-          trees.include?(self.phone_call.tree_name)
+          trees.include?(phone_call.tree_name)
         end
 
         # Returns the digits as an `Integer` entered through the keypad during a phone call as `gather:`. Returns `nil`
@@ -59,7 +60,7 @@ module Twilio
         # @return [Integer, nil] The digits as entered by the caller or `nil` if not found or not present.
         def integer_digits
           return nil unless digits.present?
-          return nil unless digits =~ /\A[0-9]+\Z/
+          return nil unless /\A[0-9]+\Z/.match?(digits)
           digits.to_i
         end
 

data/lib/twilio/rails/models/sms_conversation.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Twilio
   module Rails
     module Models

data/lib/twilio/rails/phone/base_tree.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Twilio
   module Rails
     module Phone
@@ -85,7 +86,7 @@ module Twilio
           # @param message [String, Hash, Array, Proc] The message to play to the caller.
           # @param prompt [Symbol, Hash, Proc] The name of the next prompt.
           # @return [nil]
-          def greeting(message: nil, prompt:)
+          def greeting(prompt:, message: nil)
             tree.greeting = Twilio::Rails::Phone::Tree::After.new(message: message, prompt: prompt)
             nil
           end
@@ -153,7 +154,7 @@ module Twilio
           # * `Proc`: A proc that will be called after the message and gather have been called. The proc will receive
           #   the current {Twilio::Rails::Models::Response} instance as an argument. The proc must return one of the
           #   above.
-          def prompt(prompt_name, message: nil, gather: nil, after:)
+          def prompt(prompt_name, after:, message: nil, gather: nil)
             tree.prompts[prompt_name] = Twilio::Rails::Phone::Tree::Prompt.new(name: prompt_name, message: message, gather: gather, after: after)
             nil
           end
@@ -184,10 +185,10 @@ module Twilio
             nil
           end
 
-          # The `message:` object that played to the caller if a call from an invalid phone number is received. The
-          # important case here is a number from outside of North America. This is currently a limitation of the
-          # framework. The default is `nil` and no action is taken. See the documentation for {.prompt} for what a
-          # message object can contain.
+          # The `message:` object that played to the caller if a call from an invalid phone number is received. This can
+          # be an empty or "Unknown" number, but the important case here is a number from outside of North America. This
+          # is currently a limitation of theframework. The default is `nil` and no action is taken. See the
+          # documentation for {.prompt} for what a message object can contain.
           #
           # @param message [String, Hash, Array, Proc] The message to play to the caller.
           def invalid_phone_number(message)
@@ -200,7 +201,7 @@ module Twilio
           #
           # @return [String] the name of the tree.
           def tree_name
-            self.name.demodulize.underscore.sub(/_tree\z/, "")
+            name.demodulize.underscore.sub(/_tree\z/, "")
           end
 
           # The instance of {Twilio::Rails::Phone::Tree} built from the DSL. Should be treated as read-only. Used

data/lib/twilio/rails/phone/tree.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Twilio
   module Rails
     module Phone
@@ -24,14 +25,14 @@ module Twilio
         #
         # @return [String] The outbound URL for the phone tree.
         def outbound_url
-          "#{ ::Twilio::Rails.config.host }#{ ::Twilio::Rails::Engine.routes.url_helpers.phone_outbound_path(tree_name: name, format: :xml) }"
+          "#{::Twilio::Rails.config.host}#{::Twilio::Rails::Engine.routes.url_helpers.phone_outbound_path(tree_name: name, format: :xml)}"
         end
 
         # The fully qualified URL for the tree used by Twilio to be configured in the dashboard.
         #
         # @return [String] The inbound URL for the phone tree.
         def inbound_url
-          "#{ ::Twilio::Rails.config.host }#{ ::Twilio::Rails::Engine.routes.url_helpers.phone_inbound_path(tree_name: name, format: :xml) }"
+          "#{::Twilio::Rails.config.host}#{::Twilio::Rails::Engine.routes.url_helpers.phone_inbound_path(tree_name: name, format: :xml)}"
         end
 
         class Prompt
@@ -136,7 +137,7 @@ module Twilio
         end
 
         class Message
-          attr_reader :value, :voice, :block
+          attr_reader :voice, :block
 
           def initialize(say: nil, play: nil, pause: nil, voice: nil, &block)
             @say = say.presence
@@ -149,8 +150,8 @@ module Twilio
             raise Twilio::Rails::Phone::InvalidTreeError, "must only have one of say: play: pause:" if (@say && @play) || (@say && @pause) || (@play && @pause)
             raise Twilio::Rails::Phone::InvalidTreeError, "say: must be a string or proc" if @say && !(@say.is_a?(String) || @say.is_a?(Proc))
             raise Twilio::Rails::Phone::InvalidTreeError, "play: must be a string or proc" if @play && !(@play.is_a?(String) || @play.is_a?(Proc))
-            raise Twilio::Rails::Phone::InvalidTreeError, "play: be a valid url but is #{ @play }" if @play && @play.is_a?(String) && !@play.match(/^https?:\/\/.+/)
-            raise Twilio::Rails::Phone::InvalidTreeError, "pause: must be over zero but is #{ @pause }" if @pause && @pause <= 0
+            raise Twilio::Rails::Phone::InvalidTreeError, "play: be a valid url but is #{@play}" if @play&.is_a?(String) && !@play.match(/^https?:\/\/.+/)
+            raise Twilio::Rails::Phone::InvalidTreeError, "pause: must be over zero but is #{@pause}" if @pause && @pause <= 0
             raise Twilio::Rails::Phone::InvalidTreeError, "block is only valid for say:" if block_given? && (@play || @pause)
           end
 
@@ -183,7 +184,7 @@ module Twilio
             if set.is_a?(Hash)
               set = set.symbolize_keys
               if set.key?(:message)
-                raise Twilio::Rails::Phone::InvalidTreeError, "MessageSet should never receive a hash with any key other than :message but received #{ set }" if set.keys != [:message]
+                raise Twilio::Rails::Phone::InvalidTreeError, "MessageSet should never receive a hash with any key other than :message but received #{set}" if set.keys != [:message]
                 set = set[:message]
               end
             end
@@ -201,7 +202,7 @@ module Twilio
               elsif message.is_a?(Hash)
                 @messages << Twilio::Rails::Phone::Tree::Message.new(**message.symbolize_keys)
               else
-                raise Twilio::Rails::Phone::InvalidTreeError, "message value #{ message } is not valid"
+                raise Twilio::Rails::Phone::InvalidTreeError, "message value #{message} is not valid"
               end
             end
           end

data/lib/twilio/rails/phone/tree_macros.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Twilio
   module Rails
     module Phone
@@ -21,7 +22,7 @@ module Twilio
             timeout: timeout.to_i.presence || 6,
             number: 1,
             interrupt: true,
-            finish_on_key: "",
+            finish_on_key: ""
           }
         end
 
@@ -34,16 +35,16 @@ module Twilio
         # @return [String] the digits joined with commas.
         def digits(num)
           return "" if num.blank?
-          num.to_s.split("").join(", ")
+          num.to_s.chars.join(", ")
         end
 
         # Pause for a number of seconds, defaults to 1 second. Useful when putting space between segments of speech.
         #
         # @param seconds [Integer] the number of seconds to pause for, defaults to 1 second.
         # @return [Hash] formatted to pass to `message:`.
-        def pause(seconds=nil)
+        def pause(seconds = nil)
           {
-            pause: (seconds.presence || 1),
+            pause: seconds.presence || 1
           }
         end
 
@@ -66,7 +67,7 @@ module Twilio
           raise Twilio::Rails::Phone::Error, "`numbered_choices` macro got an empty array" if choices.empty?
           raise Twilio::Rails::Phone::Error, "`numbered_choices` macro cannot be more than 9" if choices.length > 9
           prefix ||= "For"
-          choices.each_with_index.map { |choice, index| "#{ prefix } #{ choice }, press #{ index + 1 }." }.join(" ")
+          choices.each_with_index.map { |choice, index| "#{prefix} #{choice}, press #{index + 1}." }.join(" ")
         end
 
         # Validates if the response object includes a digit that is within the range of the choices array. This pairs
@@ -115,7 +116,6 @@ module Twilio
           answers_no.include?((string || "").downcase.strip.gsub(/[.,!?]/, ""))
         end
 
-
         # Finds and validates the existence of a file in the `public` folder. Formats that link to include the
         # configured hose from {Twilio::Rails::Configuration#host}, and returns a fully qualified URL to the file. This
         # is useful for playing audio files in a `message:` block. If the file is not found
@@ -128,9 +128,9 @@ module Twilio
           local_path = ::Rails.public_path.join(filename)
 
           if File.exist?(local_path)
-            "#{ ::Twilio::Rails.config.host }/#{ filename }"
+            "#{::Twilio::Rails.config.host}/#{filename}"
           else
-            raise Twilio::Rails::Phone::Error, "Cannot find public file '#{ filename }' at #{ local_path }"
+            raise Twilio::Rails::Phone::Error, "Cannot find public file '#{filename}' at #{local_path}"
           end
         end
 
@@ -139,7 +139,7 @@ module Twilio
         # @param filename [String] the filename of the file to play located in the `public` folder.
         # @return [Hash] formatted to pass to `message:`.
         def play_public_file(filename)
-          { play: public_file(filename) }
+          {play: public_file(filename)}
         end
 
         # Expose a {Twilio::TwiML::Say} node to be used in a `message:` block. This can be used to form Speech Synthesis

data/lib/twilio/rails/phone.rb

@@ -1,12 +1,13 @@
 # frozen_string_literal: true
+
 module Twilio
   module Rails
     module Phone
       # Base error class for errors relating to Twilio phone interactions.
-      class Error < ::Twilio::Rails::Error ; end
+      class Error < ::Twilio::Rails::Error; end
 
       # Error raised when attempting to build a phone tree.
-      class InvalidTreeError < Error ; end
+      class InvalidTreeError < Error; end
     end
   end
 end

data/lib/twilio/rails/phone_number.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Twilio
   module Rails
     # A phone number object that includes the country and some optional metadata.
@@ -10,8 +11,8 @@ module Twilio
       # @param label [String, nil] an optional label for the phone number, such as its source or purpose.
       # @param project [String, nil] an optional project identifier for grouping phone numbers.
       def initialize(number:, country:, label: nil, project: nil)
-        @number = Twilio::Rails::Formatter.coerce_to_valid_phone_number(number)
-        raise Twilio::Rails::Phone::Error, "Invalid phone number '#{ number }'" unless @number
+        @number = Twilio::Rails::PhoneNumberFormatter.coerce(number)
+        raise Twilio::Rails::Phone::Error, "Invalid phone number '#{number}'" unless @number
         @country = country&.upcase
         @label = label
         @project = project.presence&.to_s
@@ -19,9 +20,9 @@ module Twilio
 
       # @return [String] a human readable string representation of the phone number and its metadata.
       def to_s
-        s = "Phone number #{ number } (#{ country })"
-        s = "#{ s } #{ label }" if label.present?
-        s = "#{ s } for #{ project }" if project.present?
+        s = "Phone number #{number} (#{country})"
+        s = "#{s} #{label}" if label.present?
+        s = "#{s} for #{project}" if project.present?
         s
       end
     end

data/lib/twilio/rails/phone_number_formatter/north_america.rb

@@ -0,0 +1,52 @@
+# frozen_string_literal: true
+
+module Twilio
+  module Rails
+    module PhoneNumberFormatter
+      # Formats phone numbers as North American 10 digit numbers only, and treats any other number as invalid.
+      # This is the legacy behavior from 1.0 which will be the default still in 1.1 as an upgrade path.
+      class NorthAmerica
+        PHONE_NUMBER_REGEX = /\A\+1[0-9]{10}\Z/
+        PHONE_NUMBER_SEGMENTS_REGEX = /\A\+1([0-9]{3})([0-9]{3})([0-9]{4})\Z/
+
+        def coerce(string)
+          string = string.number if string.is_a?(Twilio::Rails::PhoneNumber)
+          string = string.to_s.presence
+
+          if string
+            string = string.gsub(/[^0-9]/, "")
+            string = "1#{string}" unless string.starts_with?("1")
+            string = "+#{string}"
+            string = nil unless valid?(string)
+          end
+
+          string
+        end
+
+        def valid?(string)
+          string = string.number if string.is_a?(Twilio::Rails::PhoneNumber)
+          !!string&.match?(PHONE_NUMBER_REGEX)
+        end
+
+        def to_param(string)
+          string = coerce(string)
+          return "" unless string
+          matches = string.match(PHONE_NUMBER_SEGMENTS_REGEX)
+          raise Twilio::Rails::Error, "[to_param] Phone number marked as valid but could not capture. I made a bad regex: #{string}" unless matches
+          matches.captures.join("-")
+        end
+
+        def display(string)
+          coerced_phone_number = coerce(string)
+          if coerced_phone_number
+            matches = coerced_phone_number.match(PHONE_NUMBER_SEGMENTS_REGEX)
+            raise Twilio::Rails::Error, "[display] Phone number marked as valid but could not capture. I made a bad regex: #{string}" unless matches
+            "(#{matches.captures[0]}) #{matches.captures[1]} #{matches.captures[2]}"
+          else
+            string
+          end
+        end
+      end
+    end
+  end
+end

data/lib/twilio/rails/phone_number_formatter.rb

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+module Twilio
+  module Rails
+    # The interface for the phone number validator and formatter that is defined in the `Twilio::Rails.config.phone_number_formatter`
+    # configuration. This delegates the methods to that instance and is used both internally to the gem and by the gem consumer.
+    module PhoneNumberFormatter
+      extend self
+      extend Forwardable
+
+      def_delegators :formatter, :coerce, :valid?, :to_param, :display
+
+      private
+
+      def formatter
+        Twilio::Rails.config.phone_number_formatter
+      end
+    end
+  end
+end

data/lib/twilio/rails/railtie.rb

@@ -1,10 +1,11 @@
 # frozen_string_literal: true
+
 module Twilio
   module Rails
     class Railtie < ::Rails::Railtie
       config.before_initialize do
         ActiveSupport::Inflector.inflections(:en) do |inflect|
-          inflect.acronym 'SMS'
+          inflect.acronym "SMS"
         end
       end
 
@@ -12,6 +13,10 @@ module Twilio
         # TODO: This should work but it does not. I think maybe it happens too late? The same line works if you add it directly to the `application.rb` of the app. It is needed for dev mode.
         # application.config.hosts << Twilio::Rails.config.host_domain
       end
+
+      initializer "twilio_rails.deprecator" do |app|
+        app.deprecators[:twilio_rails] = Twilio::Rails.deprecator
+      end
     end
   end
 end

data/lib/twilio/rails/sms/delegated_responder.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Twilio
   module Rails
     module SMS
@@ -34,7 +35,7 @@ module Twilio
           #
           # @return [String] the name of the responder.
           def responder_name
-            self.name.demodulize.underscore.gsub(/_responder\Z/, "")
+            name.demodulize.underscore.gsub(/_responder\Z/, "")
           end
         end
 
@@ -43,22 +44,22 @@ module Twilio
           @sms_conversation = message.sms_conversation
         end
 
-        # Must be implemented by the subclass otherwise will raise a `NotImplementedError`. Returns true if this
+        # Must be implemented by the subclass otherwise will raise a `NoMethodError`. Returns true if this
         # responder should handle the given message. If true then the {#reply} method will be called to generate the
         # body of the response. It has access to the message and the conversation.
         #
         # @return [true, false] true if this responder should handle the given message.
         def handle?
-          raise NotImplementedError
+          raise NoMethodError, "#{self.class}#handle? must be implemented."
         end
 
-        # Must be implemented by the subclass otherwise will raise a `NotImplementedError`. Returns the body of the
+        # Must be implemented by the subclass otherwise will raise a `NoMethodError`. Returns the body of the
         # message to be sent in response. Will only be called if {#handle?} returns true. It has access to the message
         # and the conversation.
         #
         # @return [String, nil] the body of the response to be sent as SMS, or `nil` if no message should be sent.
         def reply
-          raise NotImplementedError
+          raise NoMethodError, "#{self.class}#reply must be implemented."
         end
 
         protected

data/lib/twilio/rails/sms/responder.rb

@@ -1,5 +1,5 @@
-
 # frozen_string_literal: true
+
 module Twilio
   module Rails
     module SMS
@@ -25,7 +25,8 @@ module Twilio
             return responder.reply if responder.handle?
           end
 
-          raise Twilio::Rails::SMS::InvalidResponderError, "No responder found for message_id=#{ message.id } : #{ message.body }"
+          raise Twilio::Rails::SMS::InvalidResponderError, "No responder found for SMS. message_id=#{message.id} " \
+            "phone_caller_id=#{sms_conversation.phone_caller&.id} from_number=\"#{sms_conversation.from_number}\" body=\"#{message.body}\""
         end
       end
     end

data/lib/twilio/rails/sms.rb

@@ -1,12 +1,13 @@
 # frozen_string_literal: true
+
 module Twilio
   module Rails
     module SMS
       # Base error class for errors relating to Twilio phone interactions.
-      class Error < ::Twilio::Rails::Error ; end
+      class Error < ::Twilio::Rails::Error; end
 
       # Error raised when a responder is unable to handle an SMS message.
-      class InvalidResponderError < Error ; end
+      class InvalidResponderError < Error; end
     end
   end
 end

data/lib/twilio/rails/version.rb

@@ -1,5 +1,5 @@
 module Twilio
   module Rails
-    VERSION = "1.0.1"
+    VERSION = "1.1.0"
   end
 end