Enable rubocop-thread_safety plugin

Author: horgh

.rubocop.yml

@@ -2,6 +2,7 @@ plugins:
     - rubocop-performance
     - rubocop-rake
     - rubocop-rspec
+    - rubocop-thread_safety
 
 AllCops:
     TargetRubyVersion: '3.2'

lib/minfraud.rb

@@ -32,6 +32,9 @@
 # for the gem's classes.
 module Minfraud
   class << self
+    # rubocop:disable ThreadSafety/ClassAndModuleAttributes
+    # This is a false positive - these configuration attributes are set during initialization
+
     # The MaxMind account ID that is used for authorization.
     #
     # @return [Integer, nil]
@@ -57,6 +60,8 @@ class << self
     # @return [String, nil]
     attr_accessor :license_key
 
+    # rubocop:enable ThreadSafety/ClassAndModuleAttributes
+
     # @!visibility private
     attr_reader :connection_pool
 
@@ -67,19 +72,25 @@ def configure
       yield self
 
       pool_size        = 5
+      # rubocop:disable ThreadSafety/ClassInstanceVariable
+      # This is a false positive - this configuration is set during initialization
       host             = @host.nil? ? 'minfraud.maxmind.com' : @host
       @connection_pool = ConnectionPool.new(size: pool_size) do
+        # rubocop:enable ThreadSafety/ClassInstanceVariable
         make_http_client.persistent("https://#{host}")
       end
     end
 
     private
 
     def make_http_client
+      # rubocop:disable ThreadSafety/ClassInstanceVariable
+      # This is a false positive - this configuration is set during initialization
       HTTP.basic_auth(
         user: @account_id,
         pass: @license_key,
       ).headers(
+        # rubocop:enable ThreadSafety/ClassInstanceVariable
         accept:     'application/json',
         user_agent: "minfraud-api-ruby/#{Minfraud::VERSION} ruby/#{RUBY_VERSION} http/#{HTTP::VERSION}",
       )

lib/minfraud/enum.rb

@@ -15,7 +15,10 @@ module ClassMethods
       #
       # @return [Hash]
       def mapping
+        # rubocop:disable ThreadSafety/ClassInstanceVariable
+        # This is a false positive - this is set during class definition and then only read
         @mapping ||= {}
+        # rubocop:enable ThreadSafety/ClassInstanceVariable
       end
 
       # Create a set of methods for enum-like behavior of the attribute.

minfraud.gemspec

@@ -34,6 +34,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'rubocop-performance'
   spec.add_development_dependency 'rubocop-rake'
   spec.add_development_dependency 'rubocop-rspec'
+  spec.add_development_dependency 'rubocop-thread_safety'
   spec.add_development_dependency 'webmock', '~> 3.14'
   spec.metadata = {
     'rubygems_mfa_required' => 'true'