react-manifest-rails 0.2.30 → 0.2.31

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1331513cb5939f583960098fa86106a9901a0b3ef126b64f5ac9ae545a51f930
-  data.tar.gz: f89ef2eff49327f1e4a4a3aed0657167d5dacabd0bae3735a1a4744c0ca45332
+  metadata.gz: fc0a4a583ea6c0b488c1fc9e4c0af46dec90e4ecdd6b93cd7eae48267ffdbfaa
+  data.tar.gz: 1a00e3b56139db55d5c473e22bc35cd43e2fc1dcb3ee50700602344c09d0c4db
 SHA512:
-  metadata.gz: 6fe469fdebbd0bb563f46c59e01c3931fbcf67485e8d0141cd9628f95969912716f42622454055f758c98e0a4e9482183f79a4a1c025c9e8a9b86a6690dd8f0c
-  data.tar.gz: 36fcaa4b86a562227966018d98355894526ada5f6744427749938dd64b21ed6e5cd4f390c08c309f4500201f220647fe868a7f223ede977fbbafc15960d5ed51
+  metadata.gz: b6db17954b9a91e080c0074ce203d322af67ba58d887be9ae4a1ada51add9fe3d36f7fc256b1587c274c2315c9324caa7576421c19740a6aeab40fa85f3f5606
+  data.tar.gz: 49dc6ae440709098ace838c583f492a516fa5abd0b1b332525cd3803ed3787ed656cd23871b53ad8cb91908a4c0d62e0f9d96b8394180fe950da8ee405068d6c

data/CHANGELOG.md

@@ -7,6 +7,15 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.2.31] - 2026-07-01
+
+### Fixed
+- Controller manifests no longer pull in an unrelated bundle's files just because a component name collides (e.g. a generic `Show` component defined in two different `ux/app/*` dirs). A bundle now always satisfies a symbol from its own files first, instead of attributing it to whichever bundle happened to define that name first.
+- `react_component` no longer silently stops auto-injecting a component's bundle for the rest of the request after `react_bundle_tag` has rendered once. Previously, calling `react_component` for a component in an unrelated bundle (not the current controller's own bundle, e.g. from a haml/erb view mixing controller-based and component-based rendering) would render with no script tag at all if `react_bundle_tag` had already run earlier in the same request.
+
+### Removed
+- The gem version is no longer embedded in the `AUTO-GENERATED` manifest header. Previously every manifest's content digest changed on every gem upgrade, forcing a full regeneration of all `ux_*.js` files for no functional reason.
+
 ## [0.2.28] - 2026-05-11
 
 ### Fixed

data/lib/react_manifest/generator.rb

@@ -30,7 +30,6 @@ module ReactManifest
 
     HEADER = <<~JS.freeze
       // AUTO-GENERATED — DO NOT EDIT
-      // react-manifest-rails %<version>s
       // Run `rails react_manifest:generate` to regenerate.
     JS
 
@@ -130,6 +129,7 @@ module ReactManifest
     def build_controller_context(controller_dirs)
       bundle_files = {}
       symbol_to_bundle = {}
+      bundle_own_symbols = Hash.new { |h, k| h[k] = Set.new }
       external_symbol_to_require = {}
       dependencies = Hash.new { |h, k| h[k] = Set.new }
       external_requires = Hash.new { |h, k| h[k] = Set.new }
@@ -145,6 +145,7 @@ module ReactManifest
             next unless sym.match?(/\A[A-Z][A-Za-z0-9_]*\z/)
 
             symbol_to_bundle[sym] ||= bundle_name
+            bundle_own_symbols[bundle_name] << sym
           end
         end
       end
@@ -170,8 +171,16 @@ module ReactManifest
 
       # Compute per-bundle cross-app and external dependencies
       bundle_files.each do |bundle_name, files|
+        own_symbols = bundle_own_symbols[bundle_name]
+
         files.each do |file_path|
           extract_used_component_symbols(file_path).each do |sym|
+            # A symbol the bundle defines itself (in any of its own files) is
+            # always satisfied locally — never attribute it to another bundle
+            # or an external provider just because that symbol name happens
+            # to collide with something defined elsewhere.
+            next if own_symbols.include?(sym)
+
             dep_bundle = symbol_to_bundle[sym]
             dependencies[bundle_name] << dep_bundle if dep_bundle && dep_bundle != bundle_name
 
@@ -316,10 +325,7 @@ module ReactManifest
     # ----------------------------------------------------------- helpers
 
     def header_lines
-      [
-        format(HEADER, version: ReactManifest::VERSION),
-        ""
-      ].flatten
+      [HEADER, ""]
     end
 
     def js_files_in(dir)

data/lib/react_manifest/version.rb

@@ -1,3 +1,3 @@
 module ReactManifest
-  VERSION = "0.2.30".freeze
+  VERSION = "0.2.31".freeze
 end

data/lib/react_manifest/view_helpers.rb

@@ -26,7 +26,6 @@ module ReactManifest
       return "".html_safe if fresh_bundles.empty?
 
       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)
@@ -38,7 +37,6 @@ 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)
@@ -78,17 +76,5 @@ module ReactManifest
         @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/lib/react_manifest.rb

@@ -146,6 +146,7 @@ module ReactManifest
 
       controller_dirs = TreeClassifier.new(config).classify.controller_dirs
       symbol_to_bundle = {}
+      bundle_own_symbols = Hash.new { |h, k| h[k] = Set.new }
       bundle_files = Hash.new { |h, k| h[k] = [] }
       bundle_dependencies = Hash.new { |h, k| h[k] = Set.new }
 
@@ -160,13 +161,21 @@ module ReactManifest
 
             # Keep first writer to ensure deterministic behavior if a symbol is duplicated.
             symbol_to_bundle[symbol] ||= bundle_name
+            bundle_own_symbols[bundle_name] << symbol
           end
         end
       end
 
       bundle_files.each do |bundle_name, files|
+        own_symbols = bundle_own_symbols[bundle_name]
+
         files.each do |file_path|
           extract_used_component_symbols(file_path).each do |symbol|
+            # A symbol the bundle defines itself is always satisfied locally —
+            # never attribute it to another bundle just because that symbol
+            # name happens to collide with something defined elsewhere.
+            next if own_symbols.include?(symbol)
+
             dep_bundle = symbol_to_bundle[symbol]
             next unless dep_bundle && dep_bundle != bundle_name
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: react-manifest-rails
 version: !ruby/object:Gem::Version
-  version: 0.2.30
+  version: 0.2.31
 platform: ruby
 authors:
 - Oliver Noonan
@@ -160,7 +160,7 @@ licenses:
 metadata:
   homepage_uri: https://github.com/olivernoonan/react-manifest-rails
   source_code_uri: https://github.com/olivernoonan/react-manifest-rails
-  changelog_uri: https://github.com/olivernoonan/react-manifest-rails/blob/main/CHANGELOG.md
+  changelog_uri: https://github.com/olivernoonan/react-manifest-rails/blob/master/CHANGELOG.md
   bug_tracker_uri: https://github.com/olivernoonan/react-manifest-rails/issues
   rubygems_mfa_required: 'true'
 rdoc_options: []