react-manifest-rails 0.2.22 → 0.2.24

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c87f68d7ed8248c19732ce3b946a1207ed85907a92bd39ab5a2eda665f5b001e
-  data.tar.gz: 7ee42c73f25048a31d8ca713d0cdfc23b344eb99ec68d1f72c58b5cfda3b635d
+  metadata.gz: 786d53e91fe8a65339def86bc62a94179762364cf13d1abcc8427ea9585f0c38
+  data.tar.gz: 90c58cd92bc06eb2911deaf829aca60ed8533e6dc58e535759e9f6d60eefdcd2
 SHA512:
-  metadata.gz: d71e1c8b255678a3b1a72df766134b80dadd8bf7553d4380e240d3a8b526dff5e6f6ef848574b93c86f6778feef896cc04b945a0f020d03d0bccf034acfe7d74
-  data.tar.gz: 874538d0cd06e8b8a121d9343a3846649b6612cd3ad4bb8c0c28a7c1589c1e05bf2f48b80ae0fe6a8af478d07a3d4309d9a5bc692536f4cdebeeed90b89ca74c
+  metadata.gz: 30c7f0633646f06cfc497a617c76dbbb50f74e84104fd590f3054bff5c228a42e08aa6683dfdc96702656a52cec57533917b2438d4bb39ede667ef6c0686f219
+  data.tar.gz: 0ea044f3d55bb64be2ce9a725cc90cf9b33da8f343fde043707c098995ce6e88840e609947867203efe739623ce5dba3a32e4d444dfb9b3f8b252b56b991adbb

data/CHANGELOG.md

@@ -7,10 +7,23 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.2.24] - 2026-04-22
+
+### Fixed
+- Controller manifests now inline files from bundles listed in `always_include` (for example `ux_main`), so runtime symbol availability no longer depends on cross-bundle script execution order in production.
+- Scanner analysis no longer emits warnings for ux/app file naming convention mismatches, reducing noise for apps that intentionally use custom filename patterns.
+
+### Changed
+- Updated `Gemfile.lock` to keep lockfile state aligned with the released codebase.
+
+## [0.2.23] - 2026-04-21
+
 ### Fixed
 - `react_component` now emits only `ux_shared` plus the direct owning bundle for the requested component symbol, instead of also emitting transitive controller manifests as separate script tags. This prevents unnecessary network requests for additional `ux_*.js` manifests while preserving dependency loading through generated manifest `require` directives.
 - Generator now skips `external_roots` and `external_providers` entries that resolve to files already included in `ux_shared`, preventing duplicate runtime declarations from overlapping shared/external includes.
 - View helper bundle deduplication now canonicalizes bundle names (e.g. `ux_shared` vs `ux_manifests/ux_shared`) to avoid re-emitting equivalent script tags.
+- Controller manifests now include scanner-detected shared dependencies after `ux_shared` removal, including transitive shared dependencies needed by shared components (for example `DataTable` -> `SortHeader`).
+- Shared `ux/lib` utility files are now included in controller manifests, restoring runtime availability for global helper functions such as `formatDate` and `formatCurrency`.
 
 ## [0.2.10] - 2026-04-16
 

data/lib/react_manifest/generator.rb

@@ -42,12 +42,12 @@ module ReactManifest
     # written and others stale/missing.
     def run!
       classification = @classifier.classify
-      controller_context = build_controller_context(classification.controller_dirs, classification.shared_dirs)
+      scan_result = Scanner.new(@config).scan(classification)
+      controller_context = build_controller_context(classification.controller_dirs, classification.shared_dirs,
+                                                    scan_result)
 
       # Phase 1: build all content in memory — no I/O.
-      manifests = []
-      manifests << build_shared(classification.shared_dirs)
-      classification.controller_dirs.each { |ctrl| manifests << build_controller(ctrl, controller_context) }
+      manifests = classification.controller_dirs.map { |ctrl| build_controller(ctrl, controller_context) }
 
       migrate_legacy_manifests!
 
@@ -62,33 +62,19 @@ module ReactManifest
 
     # ------------------------------------------------------------------ shared
 
-    def build_shared(shared_dirs)
-      lines     = header_lines
-      any_files = false
-
-      shared_dirs.each do |shared_dir|
-        files = js_files_in(shared_dir[:path])
-        next if files.empty?
-
-        any_files = true
-        files.each { |f| lines << "//= require #{relative_require_path(f)}" }
-      end
-
-      lines << "// (no shared files found)" unless any_files
-
-      { filename: "#{@config.shared_bundle}.js", content: "#{lines.join("\n")}\n" }
-    end
-
     # --------------------------------------------------------------- controller
 
     def build_controller(ctrl, controller_context)
       lines = header_lines
+      always_include_reqs = controller_context[:always_include_requires].fetch(ctrl[:bundle_name], [])
       dep_requires = controller_dependency_requires(ctrl[:bundle_name], controller_context)
+      lib_reqs = controller_context[:shared_lib_requires]
+      shared_reqs = controller_context[:shared_requires].fetch(ctrl[:bundle_name], Set.new).to_a.sort
       ext_reqs = controller_context[:external_requires].fetch(ctrl[:bundle_name], Set.new).to_a.sort
 
       files = js_files_in(ctrl[:path])
       own_requires = files.map { |f| relative_require_path(f) }
-      all_requires = (dep_requires + ext_reqs + own_requires).uniq
+      all_requires = (always_include_reqs + dep_requires + lib_reqs + shared_reqs + ext_reqs + own_requires).uniq
 
       if all_requires.empty?
         lines << "// (no JSX files found in #{ctrl[:name]}/)"
@@ -99,16 +85,26 @@ module ReactManifest
       { filename: "#{ctrl[:bundle_name]}.js", content: "#{lines.join("\n")}\n" }
     end
 
-    def build_controller_context(controller_dirs, shared_dirs)
+    # rubocop:disable Metrics/AbcSize
+    def build_controller_context(controller_dirs, shared_dirs, scan_result)
       bundle_files = {}
       symbol_to_bundle = {}
       external_symbol_to_require = {}
       dependencies = Hash.new { |h, k| h[k] = Set.new }
       external_requires = Hash.new { |h, k| h[k] = Set.new }
       shared_require_paths = shared_require_path_set(shared_dirs)
+      shared_requires = Hash.new { |h, k| h[k] = Set.new }
+      shared_dependency_map = build_shared_dependency_map(shared_dirs, shared_require_paths, scan_result)
+      shared_lib_requires = shared_lib_require_paths(shared_dirs)
 
-      # Index controller-defined symbols for cross-app detection
       controller_dirs.each do |ctrl|
+        scan_result.controller_usages.fetch(ctrl[:name], []).each do |req_path|
+          shared_requires[ctrl[:bundle_name]] << req_path
+        end
+        shared_requires[ctrl[:bundle_name]] = expand_shared_requires(shared_requires[ctrl[:bundle_name]],
+                                                                     shared_dependency_map)
+
+        # Index controller-defined symbols for cross-app detection
         bundle_name = ctrl[:bundle_name]
         files = js_files_in(ctrl[:path])
         bundle_files[bundle_name] = files
@@ -127,10 +123,6 @@ module ReactManifest
         abs_root = abs_external_root(root_path)
         external_js_files_in(abs_root).each do |file_path|
           req_path = relative_require_path(file_path)
-          if shared_require_paths.include?(normalize_require_path(req_path))
-            warn "[ReactManifest] Skipping external_roots file already provided by shared bundle: #{req_path}"
-            next
-          end
 
           warn_on_external_controller_references(file_path, symbol_to_bundle)
 
@@ -142,12 +134,6 @@ module ReactManifest
 
       # Explicit external_providers win over scanned roots on symbol conflicts
       @config.external_providers.each do |sym, req_path|
-        if shared_require_paths.include?(normalize_require_path(req_path))
-          warn "[ReactManifest] Skipping external provider '#{sym}' because it is already " \
-               "provided by shared bundle: #{req_path}"
-          next
-        end
-
         external_symbol_to_require[sym] = req_path
       end
 
@@ -164,12 +150,18 @@ module ReactManifest
         end
       end
 
+      always_include_requires = build_always_include_requires(bundle_files, dependencies)
+
       {
         bundle_files: bundle_files,
         dependencies: dependencies,
+        always_include_requires: always_include_requires,
+        shared_lib_requires: shared_lib_requires,
+        shared_requires: shared_requires,
         external_requires: external_requires
       }
     end
+    # rubocop:enable Metrics/AbcSize
 
     def controller_dependency_requires(bundle_name, controller_context)
       deps = transitive_dependencies(bundle_name, controller_context[:dependencies])
@@ -199,6 +191,32 @@ module ReactManifest
       ordered
     end
 
+    def build_always_include_requires(bundle_files, dependencies)
+      bundles = @config.always_include.map(&:to_s).reject(&:empty?).uniq
+      return Hash.new { |h, k| h[k] = [] } if bundles.empty?
+
+      requires_by_bundle = Hash.new { |h, k| h[k] = [] }
+
+      bundle_files.each_key do |bundle_name|
+        requires = Set.new
+
+        bundles.each do |always_bundle|
+          next if always_bundle == bundle_name
+
+          transitive = [always_bundle] + transitive_dependencies(always_bundle, dependencies)
+          transitive.each do |dep_bundle|
+            bundle_files.fetch(dep_bundle, []).each do |abs_path|
+              requires << relative_require_path(abs_path)
+            end
+          end
+        end
+
+        requires_by_bundle[bundle_name] = requires.to_a.sort
+      end
+
+      requires_by_bundle
+    end
+
     # --------------------------------------------------------------- write
 
     def write_manifest(filename, content)
@@ -370,6 +388,58 @@ module ReactManifest
       end
     end
 
+    def shared_lib_require_paths(shared_dirs)
+      shared_dirs.each_with_object([]) do |shared_dir, paths|
+        next unless File.basename(shared_dir[:path]) == "lib"
+
+        js_files_in(shared_dir[:path]).each do |file_path|
+          paths << normalize_require_path(relative_require_path(file_path))
+        end
+      end.sort.uniq
+    end
+
+    def build_shared_dependency_map(shared_dirs, shared_require_paths, scan_result)
+      dependency_map = Hash.new { |h, k| h[k] = Set.new }
+
+      shared_symbol_index = scan_result.symbol_index.each_with_object({}) do |(sym, req_path), index|
+        normalized = normalize_require_path(req_path)
+        next unless shared_require_paths.include?(normalized)
+
+        index[sym] = normalized
+      end
+
+      shared_dirs.each do |shared_dir|
+        js_files_in(shared_dir[:path]).each do |file_path|
+          from_req = normalize_require_path(relative_require_path(file_path))
+          extract_used_component_symbols(file_path).each do |sym|
+            to_req = shared_symbol_index[sym]
+            next if to_req.nil? || to_req == from_req
+
+            dependency_map[from_req] << to_req
+          end
+        end
+      end
+
+      dependency_map
+    end
+
+    def expand_shared_requires(initial_requires, dependency_map)
+      expanded = Set.new(initial_requires)
+      queue = initial_requires.to_a
+
+      until queue.empty?
+        req = queue.shift
+        dependency_map.fetch(req, Set.new).each do |dep_req|
+          next if expanded.include?(dep_req)
+
+          expanded << dep_req
+          queue << dep_req
+        end
+      end
+
+      expanded
+    end
+
     def normalize_require_path(path)
       path.to_s.sub(/\.js\.jsx$/, "").sub(/\.jsx$/, "").sub(/\.js$/, "")
     end

data/lib/react_manifest/scanner.rb

@@ -9,7 +9,6 @@ module ReactManifest
   # Phase 1 — builds a symbol index from shared dirs:
   #   "PrimaryButton" => "ux/components/buttons/primary_button"
   #   "useFetch"      => "ux/hooks/use_fetch"
-  #   "formatDate"    => "ux/lib/format_date"
   #
   # Phase 2 — scans controller files for usage of those symbols
   #   and produces per-controller lists of referenced shared files.
@@ -21,22 +20,19 @@ module ReactManifest
     DEFINITION_PATTERNS = [
       # CommonJS / variable-assignment style
       /(?:const|let|var)\s+([A-Z][A-Za-z0-9_]*)\s*=/, # const FooBar =
-      /function\s+([A-Z][A-Za-z0-9_]*)\s*\(/,                                       # function FooBar(
-      /class\s+([A-Z][A-Za-z0-9_]*)\s*(?:extends|\{)/,                              # class FooBar
+      /function\s+([A-Z][A-Za-z0-9_]*)\s*\(/,         # function FooBar(
+      /class\s+([A-Z][A-Za-z0-9_]*)\s*(?:extends|\{)/, # class FooBar
       /(?:const|let|var)\s+(use[A-Z][A-Za-z0-9_]*)\s*=/, # const useFoo = (hooks)
       /function\s+(use[A-Z][A-Za-z0-9_]*)\s*\(/, # function useFoo(
-      /(?:const|let|var)\s+([a-z][A-Za-z0-9_]{2,})\s*=\s*(?:function|\()/, # const formatDate = function/arrow
-      /^function\s+([a-z][A-Za-z0-9_]{2,})\s*\(/, # function formatDate( at line start
 
       # ES module style (export default / named exports)
-      /^export\s+default\s+(?:function|class)\s+([A-Z][A-Za-z0-9_]*)/,             # export default function Foo
-      /^export\s+default\s+(?:function|class)\s+(use[A-Z][A-Za-z0-9_]*)/,          # export default function useFoo
-      /^export\s+(?:const|let|var)\s+([A-Z][A-Za-z0-9_]*)\s*=/,                    # export const Foo =
-      /^export\s+(?:const|let|var)\s+(use[A-Z][A-Za-z0-9_]*)\s*=/,                 # export const useFoo =
-      /^export\s+(?:const|let|var)\s+([a-z][A-Za-z0-9_]{2,})\s*=\s*(?:function|\()/, # export const formatDate =
-      /^export\s+function\s+([A-Z][A-Za-z0-9_]*)\s*\(/,                             # export function Foo(
-      /^export\s+function\s+(use[A-Z][A-Za-z0-9_]*)\s*\(/,                          # export function useFoo(
-      /^export\s+class\s+([A-Z][A-Za-z0-9_]*)\s*(?:extends|\{)/ # export class Foo
+      /^export\s+default\s+(?:function|class)\s+([A-Z][A-Za-z0-9_]*)/,
+      /^export\s+default\s+(?:function|class)\s+(use[A-Z][A-Za-z0-9_]*)/,
+      /^export\s+(?:const|let|var)\s+([A-Z][A-Za-z0-9_]*)\s*=/,
+      /^export\s+(?:const|let|var)\s+(use[A-Z][A-Za-z0-9_]*)\s*=/,
+      /^export\s+function\s+([A-Z][A-Za-z0-9_]*)\s*\(/,
+      /^export\s+function\s+(use[A-Z][A-Za-z0-9_]*)\s*\(/,
+      /^export\s+class\s+([A-Z][A-Za-z0-9_]*)\s*(?:extends|\{)/
     ].freeze
 
     # Patterns to detect usage in controller files.
@@ -66,7 +62,7 @@ module ReactManifest
 
     # rubocop:disable Metrics/MethodLength,Metrics/AbcSize,Metrics/PerceivedComplexity
     def scan(classification)
-      warnings      = []
+      warnings      = Set.new
       symbol_index  = {}
       external_file_paths = {} # file_path => relative_require_path for external_roots files
 
@@ -79,7 +75,7 @@ module ReactManifest
           symbols = extract_definitions(file_path)
           symbols.each do |sym|
             if symbol_index.key?(sym)
-              warnings << "Duplicate symbol '#{sym}' in #{relative} (already from #{symbol_index[sym]})"
+              warnings.add("Duplicate symbol '#{sym}' in #{relative} (already from #{symbol_index[sym]})")
             else
               symbol_index[sym] = relative
             end
@@ -131,10 +127,9 @@ module ReactManifest
         files   = js_files_in(ctrl[:path])
         used    = Set.new
 
-        warnings << "Controller dir '#{ctrl[:name]}' has no JS/JSX files" if files.empty? && @config.verbose?
+        warnings.add("Controller dir '#{ctrl[:name]}' has no JS/JSX files") if files.empty? && @config.verbose?
 
         files.each do |file_path|
-          validate_naming(file_path, ctrl[:name], warnings)
           content = read_controller_file(file_path, warnings)
           next unless content
 
@@ -150,7 +145,7 @@ module ReactManifest
       Result.new(
         symbol_index: symbol_index,
         controller_usages: controller_usages,
-        warnings: warnings,
+        warnings: warnings.to_a,
         shared_violations: shared_violations,
         external_violations: external_violations
       )
@@ -195,15 +190,6 @@ module ReactManifest
       rel.sub(/\.js\.jsx$/, "").sub(/\.jsx$/, "").sub(/\.js$/, "")
     end
 
-    def validate_naming(file_path, ctrl_name, warnings)
-      basename = File.basename(file_path, ".*").sub(/\.js$/, "")
-      # Expected: <controller>_index, <controller>_show, <controller>_form, etc.
-      return if basename.start_with?("#{ctrl_name}_") || basename == ctrl_name
-
-      warnings << "File '#{File.basename(file_path)}' in '#{ctrl_name}' does not follow " \
-                  "'#{ctrl_name}_<action>.js.jsx' naming convention"
-    end
-
     def detect_shared_violations(shared_file_paths, controller_symbol_index, warnings)
       violations = []
       shared_file_paths.each do |file_path, relative|
@@ -225,9 +211,9 @@ module ReactManifest
             info = controller_symbol_index[sym]
             violations << { shared_file: relative, symbol: sym,
                             controller: info[:controller], app_file: info[:file] }
-            warnings << "Shared file '#{relative}' uses app-dir symbol '#{sym}' " \
-                        "(from ux/app/#{info[:controller]}). " \
-                        "Move '#{sym}' to a shared dir or the shared file will be incomplete."
+            warnings.add("Shared file '#{relative}' uses app-dir symbol '#{sym}' " \
+                         "(from ux/app/#{info[:controller]}). " \
+                         "Move '#{sym}' to a shared dir or the shared file will be incomplete.")
           end
         end
       end
@@ -255,9 +241,9 @@ module ReactManifest
             info = controller_symbol_index[sym]
             violations << { external_file: relative, symbol: sym,
                             controller: info[:controller], app_file: info[:file] }
-            warnings << "External file '#{relative}' uses app-dir symbol '#{sym}' " \
-                        "(from ux/app/#{info[:controller]}). " \
-                        "Move '#{sym}' into a shared ux dir to avoid duplicate runtime declarations."
+            warnings.add("External file '#{relative}' uses app-dir symbol '#{sym}' " \
+                         "(from ux/app/#{info[:controller]}). " \
+                         "Move '#{sym}' into a shared ux dir to avoid duplicate runtime declarations.")
           end
         end
       end
@@ -272,20 +258,17 @@ module ReactManifest
       end
 
       fanout.each do |file, count|
-        if count > 3
-          warnings << "High fan-out: '#{file}' is used by #{count} controllers " \
-                      "(consider ensuring it's in the shared bundle)"
-        end
+        warnings.add("High fan-out: '#{file}' is used by #{count} controllers") if count > 3
       end
     end
 
     def read_controller_file(file_path, warnings)
       File.read(file_path, encoding: "utf-8")
     rescue Errno::ENOENT, Errno::EACCES => e
-      warnings << "Skipping #{file_path}: #{e.message}"
+      warnings.add("Skipping #{file_path}: #{e.message}")
       nil
     rescue Encoding::InvalidByteSequenceError
-      warnings << "Skipping #{file_path}: not valid UTF-8"
+      warnings.add("Skipping #{file_path}: not valid UTF-8")
       nil
     end
 

data/lib/react_manifest/version.rb

@@ -1,3 +1,3 @@
 module ReactManifest
-  VERSION = "0.2.22".freeze
+  VERSION = "0.2.24".freeze
 end

data/lib/react_manifest/view_helpers.rb

@@ -5,11 +5,10 @@ module ReactManifest
   #   <%= react_bundle_tag %>
   #
   # Resolves which ux_*.js bundles to include based on controller_path:
-  #   1. Always includes config.shared_bundle (e.g. "ux_shared")
-  #   2. Always appends config.always_include (e.g. ["ux_main"])
-  #   3. Appends "ux_<controller_path>" if that bundle file exists
-  #   4. For namespaced controllers (admin/users): checks ux_admin_users, then ux_admin
-  #   5. Returns "" for pure ERB/HAML pages with no matching bundle, or when
+  #   1. Appends config.always_include (e.g. ["ux_main"])
+  #   2. Appends "ux_<controller_path>" if that bundle file exists
+  #   3. For namespaced controllers (admin/users): checks ux_admin_users, then ux_admin
+  #   4. Returns "" for pure ERB/HAML pages with no matching bundle, or when
   #      called outside a controller context (mailers, engines, etc.)
   module ViewHelpers
     def react_bundle_tag(**html_options)
@@ -22,10 +21,14 @@ module ReactManifest
       return "".html_safe if bundles.empty?
 
       # Record emitted bundles so react_component doesn't re-emit them.
-      emitted = (@_react_manifest_emitted_bundles ||= [])
-      bundles.each { |b| emitted << b unless emitted_bundle?(emitted, b) }
+      emitted = emitted_bundles
+      fresh_bundles = bundles.reject { |b| emitted_bundle?(emitted, b) }
+      return "".html_safe if fresh_bundles.empty?
 
-      asset_names = bundles.map { |bundle| "#{bundle}.js" }
+      fresh_bundles.each { |b| emitted << b }
+      mark_bundle_tag_rendered
+
+      asset_names = fresh_bundles.map { |bundle| "#{bundle}.js" }
       javascript_include_tag(*asset_names, extname: false, **html_options)
     end
 
@@ -35,12 +38,13 @@ module ReactManifest
     # This avoids strict dependence on controller_path -> bundle naming alignment.
     def react_component(*args, **kwargs, &block)
       html = super
+      return html if bundle_tag_rendered?
 
       component_name = args.first
       bundles = ReactManifest.resolve_bundles_for_component_direct(component_name)
       return html if bundles.empty?
 
-      emitted = (@_react_manifest_emitted_bundles ||= [])
+      emitted = emitted_bundles
 
       new_tags = bundles.filter_map do |bundle|
         next if emitted_bundle?(emitted, bundle)
@@ -64,5 +68,27 @@ module ReactManifest
     def canonical_bundle_name(bundle)
       bundle.to_s.split("/").last
     end
+
+    def emitted_bundles
+      # ActionView can instantiate multiple helper contexts during one request.
+      # Store emitted bundles in request env so layout + template helpers dedupe.
+      if respond_to?(:request, true) && request
+        request.env["react_manifest.emitted_bundles"] ||= []
+      else
+        @emitted_bundles ||= []
+      end
+    end
+
+    def mark_bundle_tag_rendered
+      return unless respond_to?(:request, true) && request
+
+      request.env["react_manifest.bundle_tag_rendered"] = true
+    end
+
+    def bundle_tag_rendered?
+      return false unless respond_to?(:request, true) && request
+
+      request.env["react_manifest.bundle_tag_rendered"] == true
+    end
   end
 end

data/tasks/react_manifest.rake

@@ -130,12 +130,6 @@ namespace :react_manifest do
     scan_result    = scanner.scan(classification)
     dep_map        = ReactManifest::DependencyMap.new(scan_result)
     dep_map.print_report
-
-    unless scan_result.warnings.empty?
-      puts "Warnings (#{scan_result.warnings.size}):"
-      scan_result.warnings.each { |w| puts "  ⚠  #{w}" }
-      puts
-    end
   end
 
   desc "Analyze application*.js files — show what migrate_application would change"

metadata

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