Remove the requirejs:precompile:disable_js_compressor task and put its functionality in the engine's before_initialize hook

Author: carsomyr

lib/requirejs/rails/engine.rb

@@ -20,6 +20,15 @@ class Engine < ::Rails::Engine
 
         config.assets.precompile += config.requirejs.precompile
 
+        # Check for the existence of the requirejs:precompile:all top-level Rake task and disable asset pipeline
+        # compression to ensure that `r.js` gets uncompressed assets.
+        Rake.application.top_level_tasks.each do |task_name|
+          case task_name
+            when "requirejs:precompile:all"
+              config.assets.js_compressor = false
+          end
+        end if defined?(Rake.application)
+
         manifest_directory = config.assets.manifest || File.join(::Rails.public_path, config.assets.prefix)
         manifest_path = File.join(manifest_directory, "rjs_manifest.yml")
         config.requirejs.manifest_path = Pathname.new(manifest_path)

lib/tasks/requirejs-rails_tasks.rake

@@ -77,19 +77,6 @@ OS X Homebrew users can use 'brew install node'.
                   "requirejs:precompile:run_rjs",
                   "requirejs:precompile:digestify_and_compress"]
 
-    task :disable_js_compressor do
-      # Ensure that Sprockets doesn't try to compress assets before they hit
-      # r.js.  Failure to do this can cause a build which works in dev, but
-      # emits require.js "notloaded" errors, etc. in production.
-      #
-      # Note that a configuration block is used here to ensure that it runs
-      # after the environment ("config/application.rb",
-      # "config/environments/*.rb") has been set up.
-      Rails.application.config.assets.configure do |env|
-        env.js_compressor = false
-      end
-    end
-
     # Invoke another ruby process if we're called from inside
     # assets:precompile so we don't clobber the environment
     #
@@ -160,6 +147,3 @@ OS X Homebrew users can use 'brew install node'.
 end
 
 task "assets:precompile" => ["requirejs:precompile:external"]
-if ARGV[0] == "requirejs:precompile:all"
-  task "assets:environment" => ["requirejs:precompile:disable_js_compressor"]
-end