react-manifest-rails 0.2.5 → 0.2.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 91ed3c8b3020fda5a5d811b059fdc654dd22c220878c42e76a42011ee899713e
-  data.tar.gz: a3ea26760e50e90e85d35a790fac7df24fbbaa38c717e56f071b61dc7003dcfd
+  metadata.gz: d9635ab7ddcec793c091f9ba04da6196f6b86b130ede696a7431f85e9b9e43a1
+  data.tar.gz: e56d0d2981e699e9113f189e981b9d37e7d0664f2c72cc63c153c03bd1d60558
 SHA512:
-  metadata.gz: f6648f2f5dbe48b37a3a96db8a472308be09af4ca652a1b5cb70132ec82d7a6b423451298c5c58721baf566a8b798df33e05a6da7a399036d5b589385bf3ba0a
-  data.tar.gz: 25ec716c8cb0c806152762e0e70c93468a68b3d07cb68a002180be303be892953ae2f2b4b400343c1607e821b44928a7303f264ec2129366f200230d71e523e2
+  metadata.gz: 9142794e512c65c87b23580503fcfb270cfb139ad12a725fc96990052d43ba71c3fc2203a75603a4204078332d629d207e03b8b6e15ec017e416062f8b2288e9
+  data.tar.gz: 57406840f71a57bf614bd591bcdefd8a0de16ded215cfa6d181831e5720487aa350ea06fab874acce6b1607c4dcb5f1cd6d4aca79ccda4faf91016a39a75ec92

data/CHANGELOG.md

@@ -5,6 +5,23 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.2.7] - 2026-04-15
+
+### Added
+- Generated manifests now live in a dedicated folder (`app/assets/javascripts/ux_manifests` by default) to keep `app/assets/javascripts` tidy.
+- Automatic migration of legacy `ux_*.js` files from `output_dir` root into the manifest folder during generation.
+
+### Changed
+- Added `config.manifest_subdir` (default: `ux_manifests`) for explicit control over generated manifest placement.
+- `react_manifest:clean` now removes auto-generated manifests from both the dedicated manifest folder and legacy root location.
+- README and inline configuration docs updated to explain the clean manifest directory behavior.
+
+## [0.2.6] - 2026-04-15
+
+### Fixed
+- Resolved Railtie boot error caused by calling private class method `missing_manifest_bundles` with an explicit receiver.
+- Development server boot no longer raises `private method missing_manifest_bundles called for ReactManifest::Railtie:Class`.
+
 ## [0.2.5] - 2026-04-15
 
 ### Fixed

data/README.md

@@ -42,6 +42,7 @@ ReactManifest.configure do |config|
   config.ux_root = "app/assets/javascripts/ux"
   config.app_dir = "app"
   config.output_dir = "app/assets/javascripts"
+  config.manifest_subdir = "ux_manifests"
   config.shared_bundle = "ux_shared"
   config.always_include = []
   config.ignore = []
@@ -107,6 +108,8 @@ In development:
 
 - `ux_shared.js`: shared files outside `ux/app/`
 - `ux_<controller>.js`: one bundle per directory under `ux/app/`
+- Generated manifests are written to `app/assets/javascripts/ux_manifests/` by default.
+- Existing legacy `app/assets/javascripts/ux_*.js` files are moved automatically on generation.
 
 Example:
 - `ux/app/users/...` -> `ux_users.js`

data/lib/react_manifest/configuration.rb

@@ -17,6 +17,10 @@ module ReactManifest
     # Where generated ux_*.js manifests are written (relative to Rails.root)
     attr_accessor :output_dir
 
+    # Subdirectory under output_dir that holds generated ux_*.js manifests.
+    # Keeping generated files out of output_dir root avoids clutter.
+    attr_accessor :manifest_subdir
+
     # Bundle name for auto-generated shared bundle (all non-app/ dirs)
     attr_accessor :shared_bundle
 
@@ -54,6 +58,7 @@ module ReactManifest
       @ux_root           = "app/assets/javascripts/ux"
       @app_dir           = "app"
       @output_dir        = "app/assets/javascripts"
+      @manifest_subdir   = "ux_manifests"
       @shared_bundle     = "ux_shared"
       @always_include    = []
       @ignore            = []
@@ -99,5 +104,16 @@ module ReactManifest
     def abs_output_dir
       Rails.root.join(output_dir).to_s
     end
+
+    def abs_manifest_dir
+      subdir = normalized_manifest_subdir
+      return abs_output_dir if subdir.empty?
+
+      File.join(abs_output_dir, subdir)
+    end
+
+    def normalized_manifest_subdir
+      manifest_subdir.to_s.gsub(%r{\A/+|/+\z}, "")
+    end
   end
 end

data/lib/react_manifest/generator.rb

@@ -48,6 +48,8 @@ module ReactManifest
       manifests << build_shared(classification.shared_dirs)
       classification.controller_dirs.each { |ctrl| manifests << build_controller(ctrl) }
 
+      migrate_legacy_manifests!
+
       # Phase 2: write — each write is atomic (tmp + rename).
       results = manifests.map { |m| write_manifest(m[:filename], m[:content]) }
 
@@ -96,7 +98,7 @@ module ReactManifest
     # --------------------------------------------------------------- write
 
     def write_manifest(filename, content)
-      dest = File.join(@config.abs_output_dir, filename)
+      dest = File.join(@config.abs_manifest_dir, filename)
 
       # Safety: never touch files not bearing our AUTO-GENERATED header
       # (unless they don't exist yet)
@@ -131,6 +133,31 @@ module ReactManifest
       { path: dest, status: :written }
     end
 
+    def migrate_legacy_manifests!
+      legacy_dir   = @config.abs_output_dir
+      manifest_dir = @config.abs_manifest_dir
+      return if legacy_dir == manifest_dir
+
+      legacy_files = Dir.glob(File.join(legacy_dir, "ux_*.js"))
+      return if legacy_files.empty?
+
+      if @config.dry_run?
+        legacy_files.each do |legacy|
+          target = File.join(manifest_dir, File.basename(legacy))
+          $stdout.puts "[ReactManifest] DRY-RUN: would move #{legacy} -> #{target}"
+        end
+        return
+      end
+
+      FileUtils.mkdir_p(manifest_dir)
+      legacy_files.each do |legacy|
+        target = File.join(manifest_dir, File.basename(legacy))
+        next if File.exist?(target)
+
+        FileUtils.mv(legacy, target)
+      end
+    end
+
     # ----------------------------------------------------------- helpers
 
     def header_lines

data/lib/react_manifest/railtie.rb

@@ -10,7 +10,8 @@ module ReactManifest
       next unless Rails.env.development?
 
       config = ReactManifest.configuration
-      missing = self.class.missing_manifest_bundles(config)
+      # Private class method: call via send from the initializer instance context.
+      missing = self.class.send(:missing_manifest_bundles, config)
       next if missing.empty?
 
       message = "[ReactManifest] Missing manifests on boot: #{missing.join(', ')}. Generating now..."
@@ -34,6 +35,12 @@ module ReactManifest
     # Expose config as Rails.application.config.react_manifest
     config.react_manifest = ReactManifest.configuration
 
+    # Keep generated manifests in a dedicated folder while preserving logical
+    # asset names (ux_shared, ux_users, etc.) via an explicit Sprockets path.
+    initializer "react_manifest.assets_path" do |app|
+      app.config.assets.paths << ReactManifest.configuration.abs_manifest_dir if app.config.respond_to?(:assets)
+    end
+
     # ----------------------------------------------------------------
     # Start the file watcher in development
     # ----------------------------------------------------------------
@@ -77,9 +84,9 @@ module ReactManifest
       private
 
       def missing_manifest_bundles(config)
-        output_dir = config.abs_output_dir
         expected_manifest_bundles(config).reject do |bundle_name|
-          File.exist?(File.join(output_dir, "#{bundle_name}.js"))
+          File.exist?(File.join(config.abs_manifest_dir, "#{bundle_name}.js")) ||
+            File.exist?(File.join(config.abs_output_dir, "#{bundle_name}.js"))
         end
       end
 

data/lib/react_manifest/version.rb

@@ -1,3 +1,3 @@
 module ReactManifest
-  VERSION = "0.2.5".freeze
+  VERSION = "0.2.7".freeze
 end

data/lib/react_manifest.rb

@@ -30,23 +30,25 @@ module ReactManifest
     # Used by the react_bundle_tag view helper.
     def resolve_bundles(ctrl_name)
       config  = configuration
-      output  = config.abs_output_dir
       bundles = []
 
       # 1. Shared bundle always first
-      bundles << config.shared_bundle if bundle_exists?(output, config.shared_bundle)
+      shared = resolve_bundle_reference(config, config.shared_bundle)
+      bundles << shared if shared
 
       # 2. always_include bundles (e.g. ux_main)
       config.always_include.each do |b|
-        bundles << b if bundle_exists?(output, b) && !bundles.include?(b)
+        resolved = resolve_bundle_reference(config, b)
+        bundles << resolved if resolved && !bundles.include?(resolved)
       end
 
       # 3. Controller-specific bundle
       # Try fully-namespaced first: admin/users → ux_admin_users
       # Then drop segments: ux_admin
       controller_candidates(ctrl_name).each do |candidate|
-        if bundle_exists?(output, candidate) && !bundles.include?(candidate)
-          bundles << candidate
+        resolved = resolve_bundle_reference(config, candidate)
+        if resolved && !bundles.include?(resolved)
+          bundles << resolved
           break
         end
       end
@@ -56,8 +58,15 @@ module ReactManifest
 
     private
 
-    def bundle_exists?(output_dir, bundle_name)
-      File.exist?(File.join(output_dir, "#{bundle_name}.js"))
+    def resolve_bundle_reference(config, bundle_name)
+      manifest_path = File.join(config.abs_manifest_dir, "#{bundle_name}.js")
+      return bundle_name if File.exist?(manifest_path)
+
+      # Backward compatibility for apps still holding legacy manifests in output_dir root.
+      legacy_path = File.join(config.abs_output_dir, "#{bundle_name}.js")
+      return bundle_name if File.exist?(legacy_path)
+
+      nil
     end
 
     def controller_candidates(ctrl_name)

data/tasks/react_manifest.rake

@@ -68,19 +68,21 @@ namespace :react_manifest do
   desc "Remove all AUTO-GENERATED ux_*.js manifests"
   task clean: :environment do
     config    = ReactManifest.configuration
-    output    = config.abs_output_dir
+    targets   = [config.abs_manifest_dir, config.abs_output_dir].uniq
     removed   = 0
     skipped   = 0
 
-    Dir.glob(File.join(output, "ux_*.js")).each do |file|
-      first_line = File.foreach(file).first.to_s
-      if first_line.include?("AUTO-GENERATED")
-        File.delete(file)
-        puts "[ReactManifest] Removed: #{file.sub("#{Rails.root}/", '')}"
-        removed += 1
-      else
-        puts "[ReactManifest] Skipped (not auto-generated): #{file.sub("#{Rails.root}/", '')}"
-        skipped += 1
+    targets.each do |dir|
+      Dir.glob(File.join(dir, "ux_*.js")).each do |file|
+        first_line = File.foreach(file).first.to_s
+        if first_line.include?("AUTO-GENERATED")
+          File.delete(file)
+          puts "[ReactManifest] Removed: #{file.sub("#{Rails.root}/", '')}"
+          removed += 1
+        else
+          puts "[ReactManifest] Skipped (not auto-generated): #{file.sub("#{Rails.root}/", '')}"
+          skipped += 1
+        end
       end
     end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: react-manifest-rails
 version: !ruby/object:Gem::Version
-  version: 0.2.5
+  version: 0.2.7
 platform: ruby
 authors:
 - Oliver Noonan