react-manifest-rails 0.2.9 → 0.2.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5a26dc7e8929f42fed07d13c8d38d3390d99d8063c81e5887ac171e52c1d9303
-  data.tar.gz: defd50858d68d0e22fdd396d6be214af15edb6eb6c5c030c1047c17f1f2ab0e6
+  metadata.gz: 22469e4a0a5114411640201194bae46bdb549f365a7bb7064eeb95bfd4ddb24e
+  data.tar.gz: 6b85f5dec1cf64f9132fd7cf1267f66d021a00f3058400b1644a7ac8f04bb534
 SHA512:
-  metadata.gz: d94c1392eaaa325b56225b618f318d0af156c4696dc806a2ba238796c10c75f128c4a7369475a455d1e6f851edd8ca66a5a10e86ed246a0d15917ba00a198bab
-  data.tar.gz: 86ddf03428eb1889ea9bcabd463fa761bc5738a724e201197323228ef067cd1d400f85da40d273a69cc06bf7c8016a9bba732ddd36e6c56147d980d14bda74ce
+  metadata.gz: f9501224f4fe519573393612a126f6bedaeea60ece7a44c6adce4e6c143f90db6ec948075ac228b93e1f7ca944195a5645e621d932ec3d1ce7c4890f208a46a4
+  data.tar.gz: e68251a96c1beb886969b2bc479babddd2fedf471d110f861bec99410c3dbbd4a72bf3961bd4088abe887857acdb72f9bd4b84de541c8d108d3f36adcb49c19c

data/CHANGELOG.md

@@ -5,6 +5,17 @@ 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.10] - 2026-04-16
+
+### Fixed
+- `ApplicationMigrator` now removes only `ux`-classified directives and preserves non-UX requires in `application*.js`, preventing accidental removal of root-level assets such as `mini-search`.
+- Re-running migration/setup no longer duplicates the managed header comment block in `application*.js`.
+
+### Added
+- Regression coverage for preserving unknown non-UX requires during migration.
+- Regression coverage for idempotent managed-header behavior on repeated migration runs.
+- Dummy app root-level fixture assets (`axios.min.js`, `mini-search.js`) and corresponding requires in `application.js` / `application_dev.js` to verify setup behavior before release.
+
 ## [0.2.9] - 2026-04-15
 
 ### Added

data/lib/react_manifest/application_migrator.rb

@@ -1,6 +1,6 @@
 module ReactManifest
   # Rewrites application*.js files to remove UX/app code requires,
-  # keeping only vendor lib requires.
+  # while preserving all non-UX directives.
   #
   # Safety:
   #   - Creates a .bak backup before any write; aborts if backup fails
@@ -9,6 +9,12 @@ module ReactManifest
   #   - Adds a managed-by comment at the top
   class ApplicationMigrator
     MANAGED_COMMENT = <<~JS.freeze
+      // Non-UX libraries — loaded on every page.
+      // React app code is now served per-controller via react_bundle_tag.
+      // Managed by react-manifest-rails — do not add require_tree.
+    JS
+
+    LEGACY_MANAGED_COMMENT = <<~JS.freeze
       // Vendor libraries — loaded on every page.
       // React app code is now served per-controller via react_bundle_tag.
       // Managed by react-manifest-rails — do not add require_tree.
@@ -70,12 +76,13 @@ module ReactManifest
 
     def build_new_content(result)
       kept_lines = result.directives
-                         .select do |d|
-        %i[vendor
-           passthrough].include?(d.classification)
+                         .reject do |d|
+        d.classification == :ux_code
       end
                                           .map(&:original_line)
 
+      strip_managed_header!(kept_lines)
+
       # Remove leading blank lines from kept_lines
       kept_lines.shift while kept_lines.first&.strip&.empty?
 
@@ -87,6 +94,25 @@ module ReactManifest
       lines.join("\n")
     end
 
+    def strip_managed_header!(lines)
+      managed_variants = [MANAGED_COMMENT, LEGACY_MANAGED_COMMENT].map { |comment| comment.lines.map(&:chomp) }
+
+      loop do
+        matched = managed_variants.any? { |managed_lines| starts_with_lines?(lines, managed_lines) }
+        break unless matched
+
+        header_len = managed_variants.find { |managed_lines| starts_with_lines?(lines, managed_lines) }.length
+        lines.shift(header_len)
+        lines.shift while lines.first&.strip&.empty?
+      end
+    end
+
+    def starts_with_lines?(lines, prefix)
+      return false if lines.length < prefix.length
+
+      lines.first(prefix.length) == prefix
+    end
+
     def print_diff(file, new_content)
       old_lines = File.readlines(file, encoding: "utf-8").map(&:chomp)
       new_lines = new_content.lines.map(&:chomp)

data/lib/react_manifest/version.rb

@@ -1,3 +1,3 @@
 module ReactManifest
-  VERSION = "0.2.9".freeze
+  VERSION = "0.2.10".freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: react-manifest-rails
 version: !ruby/object:Gem::Version
-  version: 0.2.9
+  version: 0.2.10
 platform: ruby
 authors:
 - Oliver Noonan
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-04-15 00:00:00.000000000 Z
+date: 2026-04-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: railties