abandon guard, use a manual reloader for the dev server

Author: jneen

Gemfile

@@ -18,8 +18,6 @@ gem 'memory_profiler', require: false
 
 group :development do
   gem 'pry'
-  gem 'guard'
-  gem 'guard-puma'
 
   # Needed for a Rake task
   gem 'git'

config.ru

@@ -1,8 +1,5 @@
 # frozen_string_literal: true
 
-require 'bundler'
-Bundler.require(:default, :development)
-
 require_relative 'spec/visual/app'
 
 run VisualTestApp

spec/visual/app.rb

@@ -3,14 +3,23 @@
 
 $VERBOSE = true
 
+require 'sinatra'
+require 'pry'
+
 # stdlib
 require 'pathname'
 
+require_relative 'rouge_reloader'
+
 class VisualTestApp < Sinatra::Application
   BASE = Pathname.new(__dir__)
   SAMPLES = BASE.join('samples')
   ROOT = BASE.parent.parent
 
+  RougeReloader.install!(:Rouge, ROOT.to_s, 'lib/rouge.rb')
+
+  RELOADER = FeatureReloader.new(:Rouge, ROOT.join('lib').to_s, 'rouge.rb')
+
   DEMOS = ROOT.join('lib/rouge/demos')
 
   def query_string
@@ -58,6 +67,7 @@ def as_boolean(value)
   end
 
   before do
+    RELOADER.reload!
     Rouge::Lexer.enable_debug!
     Rouge::Formatter.enable_escape! if params[:escape]
 

spec/visual/rouge_reloader.rb

@@ -0,0 +1,45 @@
+# [jneen] this reloader is responsible for safely hot-loading Rouge in the dev server.
+# Previously, we were able to simply remove the Rouge constant and load lib/rouge.rb,
+# and all of Rouge would happily be reloaded. However, we have recently switched to
+# require_relative, which maintains its own global cache. Meaning that if we were to
+# remove the Rouge constant and load lib/rouge.rb, the line
+#
+# require_relative 'rouge/lexer'
+#
+# would be a no-op, and hence the constant Rouge::Lexer would not exist.
+#
+# This reloader clears a portion of that cache by mutating $LOADED_FEATURES.
+class FeatureReloader
+  MUTEX = Mutex.new
+
+  def self.install!(const_name, root, path)
+    @instance = new(const_name, root, path)
+    @instance.install!
+  end
+
+  def self.reload!
+    @instance.reload!
+  end
+
+  def initialize(const_name, root, path)
+    @const_name = const_name
+    @root = root
+    @path = File.expand_path(path, root)
+    @cache = {}
+  end
+
+  def reload!
+    STDERR.puts "========= reloading #{@const_name} ========="
+
+    MUTEX.synchronize do
+      # remove the global constant
+      Object.send(:remove_const, @const_name) if Object.const_defined?(@const_name)
+
+      # clear the cache
+      $LOADED_FEATURES.reject! { |f| f.start_with?(@root) }
+
+      # load rouge.rb
+      Kernel.load(@path)
+    end
+  end
+end