railslove · r7kamura · Oct 9, 2025
diff --git a/Gemfile b/Gemfile
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 source 'https://rubygems.org'
 
 # Specify your gem's dependencies in rack-tracker.gemspec

diff --git a/Rakefile b/Rakefile
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require "bundler/gem_tasks"
 require 'rspec/core/rake_task'
 

diff --git a/lib/rack/tracker.rb b/lib/rack/tracker.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require "rack"
 require "tilt"
 require "active_support/core_ext/class/attribute"

diff --git a/lib/rack/tracker/bing/bing.rb b/lib/rack/tracker/bing/bing.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Rack::Tracker::Bing < Rack::Tracker::Handler
 
   class Conversion < OpenStruct

diff --git a/lib/rack/tracker/controller.rb b/lib/rack/tracker/controller.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Rack
   class Tracker
     module Controller

diff --git a/lib/rack/tracker/criteo/criteo.rb b/lib/rack/tracker/criteo/criteo.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Rack::Tracker::Criteo < Rack::Tracker::Handler
 
   TRACKER_EVENTS = {

diff --git a/lib/rack/tracker/drift/drift.rb b/lib/rack/tracker/drift/drift.rb
@@ -1,2 +1,3 @@
+# frozen_string_literal: true
 class Rack::Tracker::Drift < Rack::Tracker::Handler
 end
diff --git a/lib/rack/tracker/extensions.rb b/lib/rack/tracker/extensions.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'ostruct'
 
 # Backport of 2.0.0 stdlib ostruct#to_h

diff --git a/lib/rack/tracker/facebook/facebook.rb b/lib/rack/tracker/facebook/facebook.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Rack::Tracker::Facebook < Rack::Tracker::Handler
   class Event < OpenStruct
     def write

diff --git a/lib/rack/tracker/facebook_pixel/facebook_pixel.rb b/lib/rack/tracker/facebook_pixel/facebook_pixel.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Rack::Tracker::FacebookPixel < Rack::Tracker::Handler
   self.position = :body
   self.allowed_tracker_options = [:id]

diff --git a/lib/rack/tracker/go_squared/go_squared.rb b/lib/rack/tracker/go_squared/go_squared.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Rack::Tracker::GoSquared < Rack::Tracker::Handler
   class VisitorName < OpenStruct
     def write

diff --git a/lib/rack/tracker/google_adwords_conversion/google_adwords_conversion.rb b/lib/rack/tracker/google_adwords_conversion/google_adwords_conversion.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Rack::Tracker::GoogleAdwordsConversion < Rack::Tracker::Handler
   class Conversion < OpenStruct
   end

diff --git a/lib/rack/tracker/google_analytics/google_analytics.rb b/lib/rack/tracker/google_analytics/google_analytics.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Rack::Tracker::GoogleAnalytics < Rack::Tracker::Handler
 
   self.allowed_tracker_options = [:cookie_domain, :user_id]

diff --git a/lib/rack/tracker/google_global/google_global.rb b/lib/rack/tracker/google_global/google_global.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Rack::Tracker::GoogleGlobal < Rack::Tracker::Handler
   self.allowed_tracker_options = [:cookie_domain, :user_id,
     :link_attribution, :allow_display_features, :anonymize_ip,

diff --git a/lib/rack/tracker/google_tag_manager/google_tag_manager.rb b/lib/rack/tracker/google_tag_manager/google_tag_manager.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Rack::Tracker::GoogleTagManager < Rack::Tracker::Handler
 
   class Push < OpenStruct

diff --git a/lib/rack/tracker/handler.rb b/lib/rack/tracker/handler.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Rack::Tracker::Handler
   class << self
     def process_track(env, method_name, *args, &block)

diff --git a/lib/rack/tracker/handler_delegator.rb b/lib/rack/tracker/handler_delegator.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Rack::Tracker::HandlerDelegator
   class << self
     def handler(method_name)

diff --git a/lib/rack/tracker/heap/heap.rb b/lib/rack/tracker/heap/heap.rb
@@ -1,2 +1,3 @@
+# frozen_string_literal: true
 class Rack::Tracker::Heap < Rack::Tracker::Handler
 end
diff --git a/lib/rack/tracker/hotjar/hotjar.rb b/lib/rack/tracker/hotjar/hotjar.rb
@@ -1,2 +1,3 @@
+# frozen_string_literal: true
 class Rack::Tracker::Hotjar < Rack::Tracker::Handler
 end
diff --git a/lib/rack/tracker/hubspot/hubspot.rb b/lib/rack/tracker/hubspot/hubspot.rb
@@ -1,2 +1,3 @@
+# frozen_string_literal: true
 class Rack::Tracker::Hubspot < Rack::Tracker::Handler
 end
diff --git a/lib/rack/tracker/javascript_helper.rb b/lib/rack/tracker/javascript_helper.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # This module is extracted from Rails to provide reliable javascript escaping.
 #
 # @see https://github.com/rails/rails/blob/master/actionview/lib/action_view/helpers/javascript_helper.rb
@@ -13,8 +14,8 @@ module Rack::Tracker::JavaScriptHelper
       "'" => "\\'"
   }
 
-  JS_ESCAPE_MAP["\342\200\250".force_encoding(Encoding::UTF_8).encode!] = '&#x2028;'
-  JS_ESCAPE_MAP["\342\200\251".force_encoding(Encoding::UTF_8).encode!] = '&#x2029;'
+  JS_ESCAPE_MAP[(+"\342\200\250").force_encoding(Encoding::UTF_8).encode!] = '&#x2028;'
+  JS_ESCAPE_MAP[(+"\342\200\251").force_encoding(Encoding::UTF_8).encode!] = '&#x2029;'
 
   # Escapes carriage returns and single and double quotes for JavaScript segments.
   #

diff --git a/lib/rack/tracker/railtie.rb b/lib/rack/tracker/railtie.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Rack
   class Tracker
     class Railtie < ::Rails::Railtie

diff --git a/lib/rack/tracker/version.rb b/lib/rack/tracker/version.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Rack
   class Tracker
     VERSION = '1.13.0'

diff --git a/lib/rack/tracker/vwo/vwo.rb b/lib/rack/tracker/vwo/vwo.rb
@@ -1,2 +1,3 @@
+# frozen_string_literal: true
 class Rack::Tracker::Vwo < Rack::Tracker::Handler
 end
diff --git a/lib/rack/tracker/zanox/zanox.rb b/lib/rack/tracker/zanox/zanox.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Rack::Tracker::Zanox < Rack::Tracker::Handler
 
   # name of the handler

diff --git a/rack-tracker.gemspec b/rack-tracker.gemspec
@@ -1,4 +1,5 @@
 # coding: utf-8
+# frozen_string_literal: true
 lib = File.expand_path('../lib', __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'rack/tracker/version'

diff --git a/spec/benchmark/tracker_injection_benchmark.rb b/spec/benchmark/tracker_injection_benchmark.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'support/capybara_app_helper'
 require 'benchmark'
 

diff --git a/spec/handler/bing_spec.rb b/spec/handler/bing_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 RSpec.describe Rack::Tracker::Bing do
 
   it 'will be placed in the body' do

diff --git a/spec/handler/criteo_spec.rb b/spec/handler/criteo_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 RSpec.describe Rack::Tracker::Criteo do
 
   describe Rack::Tracker::Criteo::Event do

diff --git a/spec/handler/drift_spec.rb b/spec/handler/drift_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 RSpec.describe Rack::Tracker::Drift do
   def env
     { foo: 'bar' }

diff --git a/spec/handler/facebook_pixel_spec.rb b/spec/handler/facebook_pixel_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 RSpec.describe Rack::Tracker::FacebookPixel do
   def env
     { 'PIXEL_ID' => 'DYNAMIC_PIXEL_ID' }

diff --git a/spec/handler/facebook_spec.rb b/spec/handler/facebook_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 RSpec.describe Rack::Tracker::Facebook do
   describe Rack::Tracker::Facebook::Event do
 

diff --git a/spec/handler/go_squared_spec.rb b/spec/handler/go_squared_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 RSpec.describe Rack::Tracker::GoSquared do
 
   def env

diff --git a/spec/handler/google_adwords_conversion_spec.rb b/spec/handler/google_adwords_conversion_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 RSpec.describe Rack::Tracker::GoogleAdwordsConversion do
 
   def env

diff --git a/spec/handler/google_analytics_spec.rb b/spec/handler/google_analytics_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 RSpec.describe Rack::Tracker::GoogleAnalytics do
   def env
     {

diff --git a/spec/handler/google_global_spec.rb b/spec/handler/google_global_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 RSpec.describe Rack::Tracker::GoogleGlobal do
   def env
     {

diff --git a/spec/handler/google_tag_manager_spec.rb b/spec/handler/google_tag_manager_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 RSpec.describe Rack::Tracker::GoogleTagManager do
 
   def env
@@ -42,7 +43,7 @@ def env
     end
 
     context 'with one line html response' do
-      let(:example_response) { "<html><head></head><body></body></html>" }
+      let(:example_response) { +"<html><head></head><body></body></html>" }
 
       it 'will have render_head content in head tag' do
         expect(subject).to match(%r{<head>.*<script>"HEAD"</script>.*</head>})

diff --git a/spec/handler/handler_spec.rb b/spec/handler/handler_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 RSpec.describe Rack::Tracker::Handler do
   def env
     { misc: 'foobar' }

diff --git a/spec/handler/heap_spec.rb b/spec/handler/heap_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 RSpec.describe Rack::Tracker::Heap do
   def env
     { foo: 'bar' }

diff --git a/spec/handler/hotjar_spec.rb b/spec/handler/hotjar_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 RSpec.describe Rack::Tracker::Hotjar do
   def env
     { foo: 'bar' }

diff --git a/spec/handler/hubspot_spec.rb b/spec/handler/hubspot_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 RSpec.describe Rack::Tracker::Hubspot do
 
   def env

diff --git a/spec/handler/vwo_spec.rb b/spec/handler/vwo_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 RSpec.describe Rack::Tracker::Vwo do
 
   def env

diff --git a/spec/handler/zanox_spec.rb b/spec/handler/zanox_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 RSpec.describe Rack::Tracker::Zanox do
 
   describe Rack::Tracker::Zanox::Sale do

diff --git a/spec/integration/bing_integration_spec.rb b/spec/integration/bing_integration_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'support/capybara_app_helper'
 
 RSpec.describe "Bing Integration" do

diff --git a/spec/integration/criteo_integration_spec.rb b/spec/integration/criteo_integration_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'support/capybara_app_helper'
 
 RSpec.describe "Criteo Integration" do

diff --git a/spec/integration/drift_integration_spec.rb b/spec/integration/drift_integration_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'support/capybara_app_helper'
 
 RSpec.describe 'Drift Integration' do

diff --git a/spec/integration/facebook_integration_spec.rb b/spec/integration/facebook_integration_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'support/capybara_app_helper'
 
 RSpec.describe "Facebook Integration" do

diff --git a/spec/integration/facebook_pixel_integration_spec.rb b/spec/integration/facebook_pixel_integration_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'support/capybara_app_helper'
 
 RSpec.describe "Facebook Pixel Integration" do

diff --git a/spec/integration/go_squared_integration_spec.rb b/spec/integration/go_squared_integration_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'support/capybara_app_helper'
 
 RSpec.describe "GoSquared Integration" do

diff --git a/spec/integration/google_adwords_conversion_integration_spec.rb b/spec/integration/google_adwords_conversion_integration_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'support/capybara_app_helper'
 
 RSpec.describe "Google Adwords Conversion Integration" do

diff --git a/spec/integration/google_analytics_integration_spec.rb b/spec/integration/google_analytics_integration_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'support/capybara_app_helper'
 
 RSpec.describe "Google Analytics Integration" do

diff --git a/spec/integration/google_global_integration_spec.rb b/spec/integration/google_global_integration_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'support/capybara_app_helper'
 
 RSpec.describe "Google Global Integration Integration" do

diff --git a/spec/integration/google_tag_manager_integration_spec.rb b/spec/integration/google_tag_manager_integration_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'support/capybara_app_helper'
 
 RSpec.describe "Google Tag Manager Integration" do

diff --git a/spec/integration/heap_integration_spec.rb b/spec/integration/heap_integration_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'support/capybara_app_helper'
 
 RSpec.describe "Heap Integration" do

diff --git a/spec/integration/hotjar_integration_spec.rb b/spec/integration/hotjar_integration_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'support/capybara_app_helper'
 
 RSpec.describe "Hotjar Integration" do

diff --git a/spec/integration/hubspot_integration_spec.rb b/spec/integration/hubspot_integration_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'support/capybara_app_helper'
 
 RSpec.describe "Hubspot Integration" do

diff --git a/spec/integration/rails_integration_spec.rb b/spec/integration/rails_integration_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'support/capybara_app_helper'
 
 RSpec.describe "Rails Integration" do

diff --git a/spec/integration/vwo_integration_spec.rb b/spec/integration/vwo_integration_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'support/capybara_app_helper'
 
 RSpec.describe "VWO Integration" do

diff --git a/spec/integration/zanox_integration_spec.rb b/spec/integration/zanox_integration_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'support/capybara_app_helper'
 
 RSpec.describe "Zanox Integration" do

diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
 $LOAD_PATH.unshift(File.dirname(__FILE__))
 

diff --git a/spec/support/capybara_app_helper.rb b/spec/support/capybara_app_helper.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'support/metal_controller'
 
 # helper to configure the middleware stack with custom handlers

diff --git a/spec/support/fake_handler.rb b/spec/support/fake_handler.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class TrackAllTheThings < Rack::Tracker::Handler
   def render
     Tilt.new( File.join( File.dirname(__FILE__), '../fixtures/track_all_the_things.erb') ).render(self)

diff --git a/spec/support/metal_controller.rb b/spec/support/metal_controller.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'action_controller'
 
 class MetalController < ActionController::Metal

diff --git a/spec/tracker/controller_spec.rb b/spec/tracker/controller_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 TestController = Struct.new(:env) do
   include Rack::Tracker::Controller
 

diff --git a/spec/tracker/handler_delegator_spec.rb b/spec/tracker/handler_delegator_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Foo = Class.new
 Bar = Class.new
 

diff --git a/spec/tracker/handler_set_spec.rb b/spec/tracker/handler_set_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Dummy < Rack::Tracker::Handler
 end
 

diff --git a/spec/tracker/javascript_helper_spec.rb b/spec/tracker/javascript_helper_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 RSpec.describe Rack::Tracker do
 
   let(:escaper) do
@@ -39,8 +40,8 @@
     it { should escape(%(This "thing" is really\n netos')).to %(This \\"thing\\" is really\\n netos\\') }
     it { should escape(%(backslash\\test)).to %(backslash\\\\test) }
     it { should escape(%(dont </close> tags)).to %(dont <\\/close> tags) }
-    it { should escape(%(unicode \342\200\250 newline).force_encoding(Encoding::UTF_8).encode!).to %(unicode &#x2028; newline) }
-    it { should escape(%(unicode \342\200\251 newline).force_encoding(Encoding::UTF_8).encode!).to %(unicode &#x2029; newline) }
+    it { should escape((+%(unicode \342\200\250 newline)).force_encoding(Encoding::UTF_8).encode!).to %(unicode &#x2028; newline) }
+    it { should escape((+%(unicode \342\200\251 newline)).force_encoding(Encoding::UTF_8).encode!).to %(unicode &#x2029; newline) }
 
     it 'works with symbols' do
       expect(subject).to escape(:dimension1).to 'dimension1'