onload 1.0.5 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 66e3e322ccf2b92d128fb3bf904426cff77ba86e88ec47eb8ba9c5101dd0ae28
-  data.tar.gz: 5495ff350b972c47e6c992f744245d12baa57c0929d7878241f6c9a91922f17c
+  metadata.gz: f3b85b579c018d6457e7bfe46bf77af3c275edff7393131711c49b4cf16c76e4
+  data.tar.gz: 744b42532004241a66de3c6cabf0a7d0013e3924c82c16ae2b7e31b0d8dc3d83
 SHA512:
-  metadata.gz: 1fd824da5c6ee8cd78342b5e0cb945353147970ebe26a4c6e16005bf8245cd5e743223441c76b7ab04216dd1757a8b3925a71b32825402d1f4becf7cbbf488c0
-  data.tar.gz: 8f296f7561c97af7ff6863a87fb73c3ef1c2bccb6379ca4aceb5300c13ccde937d17189de93e877c6016ab76a62f12225ce04e2494e77e229f59172c5dfd0de7
+  metadata.gz: '09dcca366074cce184242321ead06ab87cf46eac0adfb1af585486d1e162bac051f87d323c26b8501bd57ef211f95e4074f29b5a11986541ff1791cdc4ff05dc'
+  data.tar.gz: c61baba2a8de0daf0461643d28f1b3d978898e3104024dc63f6deff37583bc9a02fbb5f1fa4a0b8b92f3f15f61dbd3eed75917d42030300a761d91816f6530be

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+## 1.1.0
+* Automatically add generated files to the specified ignore file, eg. `.gitignore`, when they are written.
+  - Specify an ignore file by setting `Onload.config.ignore_path` to the path to the ignore file you'd like to use.
+* Use appraisal-run to run tests locally for all Ruby and Rails versions.
+
 ## 1.0.5
 * Fix issues with Zeitwerk v2.7.3 and later.
 * Don't attempt to autoload directories.

data/Gemfile

@@ -9,6 +9,7 @@ end
 
 group :development do
   gem 'appraisal'
+  gem 'appraisal-run'
   gem 'benchmark-ips'
 end
 

data/README.md

@@ -46,14 +46,25 @@ Onload.install!
 
 Now, the contents of any file with a .up file extension will be passed to `UpcasePreprocessor.call`. The return value will be written to a separate Ruby file and loaded instead of the original .up file.
 
+## Ignoring Processed Files
+
+It can often be desirable to add processed/generated files to your .gitignore file. Onload comes with a mechanism that can do exactly that. Simply set the config option before running onload:
+
+```ruby
+Onload.config.ignore_path = ".gitignore"
+```
+
+Now whenever files are processed, the generated file will be automatically updated.
+
 ## Running Tests
 
-If you're using [asdf](https://asdf-vm.com/), run `./script/run_appraisal.rb` to run Rails and plain Ruby tests for all supported versions.
+Install Docker, then run `./script/run_appraisals` to run Rails and plain Ruby tests for all supported versions.
 
 Otherwise, use Appraisal to run tests for Rails or plain ruby:
 
 1. Plain ruby: `bundle exec appraisal ruby rake spec:ruby`
 1. Rails: `bundle exec appraisal <version> rake spec:rails`. Run `bundle exec appraisal list` to see the available versions. To run tests for Rails 7.0, try `bundle exec appraisal rails-7.0 rake spec:rails`
+1. Note that you must be running the correct version of Ruby specified in each gemfile.
 
 ## License
 

data/lib/onload/config.rb

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+module Onload
+  class Config
+    attr_accessor :ignore_path
+
+    def initialize
+      @ignore_path = nil
+    end
+
+    def dup
+      self.class.new.tap do |duplicate|
+        duplicate.ignore_path = ignore_path.dup
+      end
+    end
+  end
+end

data/lib/onload/file.rb

@@ -8,14 +8,22 @@ module Onload
       @path = path
     end
 
-    def write
+    def write(ignore_file: nil)
       source = ::File.read(path)
 
       ::File.extname(path).scan(/\.\w+/).each do |ext|
         source = Onload.processors[ext].call(source)
       end
 
-      ::File.write(outfile, source)
+      ::File.write(outfile, source).tap do
+        if ignore_file
+          ignore_file.add(outfile)
+        elsif Onload.config.ignore_path
+          ignore_file = IgnoreFile.load(Onload.config.ignore_path)
+          ignore_file.add(outfile)
+          ignore_file.persist!
+        end
+      end
     end
 
     def outfile

data/lib/onload/ignore_file.rb

@@ -0,0 +1,112 @@
+# frozen_string_literal: true
+
+require "set"
+
+module Onload
+  class MalformedIgnoreFileError < StandardError; end
+
+  class IgnoreFile
+    ONLOAD_SECTION_START = "##### ONLOAD BUILD ARTIFACTS (AUTO-GENERATED) #####"
+    ONLOAD_SECTION_STOP  = "##### END ONLOAD BUILD ARTIFACTS #####"
+
+    class << self
+      def load(manifest_path)
+        manifest_path = ::File.expand_path(manifest_path)
+        lines = ::File.read(manifest_path).split(/\r?\n/)
+        start_idx = lines.index(ONLOAD_SECTION_START)
+        stop_idx = lines.index(ONLOAD_SECTION_STOP)
+
+        # Start and stop indices should either both be numbers or both be nil.
+        # Anything else, and something weird is going on.
+        if start_idx.class != stop_idx.class
+          raise MalformedIgnoreFileError, "the ignore file at #{manifest_path} appears to be malformed and onload doesn't know how to modify it"
+        end
+
+        ignored_paths = if start_idx && stop_idx
+          lines[start_idx..stop_idx].map(&:strip)
+        else
+          []
+        end
+
+        ignored_paths.reject! do |path|
+          path.empty? || path.start_with?("#")
+        end
+
+        new(manifest_path, Set.new(ignored_paths))
+      end
+    end
+
+    attr_reader :manifest_path, :dirty
+
+    alias dirty? dirty
+
+    def initialize(manifest_path, ignored_paths)
+      @manifest_path = manifest_path
+      @ignored_paths = ignored_paths
+      @dirty = false
+    end
+
+    def includes?(path)
+      @ignored_paths.include?(path)
+    end
+
+    alias include? includes?
+
+    def add(path)
+      ignored_path = ::File.expand_path(path)
+      ignored_path_segments = ignored_path.split(::File::SEPARATOR)
+
+      if ignored_path_segments[0...manifest_dirname_segments.size] != manifest_dirname_segments
+        raise "file to ignore #{path} is not relative to the specified ignore file at #{manifest_path}"
+      end
+
+      relative_ignored_path = ignored_path_segments[manifest_dirname_segments.size..-1].join(::File::SEPARATOR)
+      @ignored_paths << relative_ignored_path
+
+      @dirty = true
+
+      nil
+    end
+
+    def persist!
+      return unless dirty?
+
+      lines = ::File.read(manifest_path).split(/\r?\n/)
+      start_idx = lines.index(ONLOAD_SECTION_START)
+      stop_idx = lines.index(ONLOAD_SECTION_STOP)
+
+      if start_idx && stop_idx
+        lines[start_idx..stop_idx] = [
+          ONLOAD_SECTION_START,
+          *@ignored_paths,
+          ONLOAD_SECTION_STOP
+        ]
+      else
+        lines += [
+          "",
+          ONLOAD_SECTION_START,
+          *@ignored_paths,
+          ONLOAD_SECTION_STOP,
+          ""
+        ]
+      end
+
+      contents = lines.join("\n")
+      contents << "\n" unless contents.end_with?("\n")
+
+      ::File.write(manifest_path, contents)
+
+      @dirty = false
+    end
+
+    private
+
+    def manifest_dirname
+      @manifest_dirname ||= ::File.dirname(manifest_path)
+    end
+
+    def manifest_dirname_segments
+      @manifest_diranme_segments ||= manifest_dirname.split(::File::SEPARATOR)
+    end
+  end
+end

data/lib/onload/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Onload
-  VERSION = "1.0.5"
+  VERSION = "1.1.0"
 end

data/lib/onload.rb

@@ -3,7 +3,9 @@
 module Onload
   UNLOADABLE_EXTENSIONS = %w(.bundle .so .dll).freeze
 
-  autoload :File, "onload/file"
+  autoload :Config,     "onload/config"
+  autoload :File,       "onload/file"
+  autoload :IgnoreFile, "onload/ignore_file"
 
   class << self
     attr_accessor :enabled
@@ -105,6 +107,27 @@ module Onload
       @glob ||= "*{#{each_extension.to_a.join(",")}}"
     end
 
+    def config
+      @config ||= Config.new
+    end
+
+    def with_config(override_hash)
+      new_config = config.dup.tap do |new_config|
+        override_hash.each_pair do |k, v|
+          new_config.send("#{k}=", v)
+        end
+      end
+
+      old_config = config
+      @config = new_config
+
+      yield
+
+      nil
+    ensure
+      @config = old_config
+    end
+
     private
 
     def path_cache

data/spec/fixtures/hello.rb

@@ -0,0 +1,5 @@
+class Hello
+  def hello
+    "hello".upcase
+  end
+end

data/spec/rails/controllers/home_controller_spec.rb

@@ -13,6 +13,15 @@ describe HomeController, type: :request do
       )
     end
 
+    it "adds an entry to the specified ignore file" do
+      Onload.with_config(ignore_path: @ignore_path) do
+        get "/"
+
+        ignore_file = Onload::IgnoreFile.load(@ignore_path)
+        expect(ignore_file).to include("fixtures/hello.rb")
+      end
+    end
+
     it "allows hot reloading" do
       get "/"