rails-minify 0.1.1 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: debbde5867be2f422c20756ebf3c08185983f80f1f94b1f9be9ffa5f29283989
-  data.tar.gz: d42eca099bc0278d05c81a868ec03eccd844635799e0d826900680324fd56558
+  metadata.gz: 44f089252dbde289b9e5cfeb25c2613e91544e92a1235369ec7c4457992e477b
+  data.tar.gz: c768e54a78cd0ee553f8732f019dc4eab9fff3c5c72d4ef9d30d23abc68b2994
 SHA512:
-  metadata.gz: 835c2eb9c3a2b1210977d40eaaafc28fe399d7755c3681d6c0adaef25b3063c6f43e699f4924015d404deb083ae9e26324e7f524bd51687b937866fc3d10c08b
-  data.tar.gz: 1a04e068acaaa2991c5e8c8f0ea4ec51145acde7d7285ef2c05d422b05754b733f776e269398449b8b6675f8b647aa2d95df9eb077cc55084851d5844045eae3
+  metadata.gz: da6b92c40d4f1fada0154fb362cd52c8a56e458b67e5d959e20084bb4e37f25a8be93d7d86bc065d6a71ac09dc4763c79fbada65a4f7a73c281c8b9fef511d1b
+  data.tar.gz: 8cca5d4c9850e66798cdb80127907dbfc64cdf5643d1064a7ce01be6bfdd8f46b79bbf63e9e8c80775dd6c3453d9877537bd6951707cb434378049fdb0a92227

data/CHANGELOG.md

@@ -1,5 +1,14 @@
 # Changelog
 
+## 0.1.3 (2026-10-18)
+
+- `config.rails_minify.zjit = true` turns ZJIT on after boot, as `config.yjit` does for YJIT, instead of `RUBY_ZJIT_ENABLE=1` compiling boot-only code too. Japan Voyage, per process: 24% fewer compiled methods, 18 MB less JIT memory, about 25 MB less RSS, the same speed.
+
+## 0.1.2 (2026-10-18)
+
+- `minify:check` also fails on callback chains longer than `callback_limit` (default 5, `ApplicationController`'s callbacks excluded) and on model callbacks that save their own record again; `callback_limit_mode` and `callback_loops` set either to `:warn`. Works without the rest of rails-minify turned on; the manifest check runs only when there is a manifest.
+- Requires activesupport-minimizer 0.1.1.
+
 ## 0.1.1 (2026-10-18)
 
 - `:omniauth` isn't installed when OmniAuth already skips requests that can't be the strategy's (a public `Strategy#passthrough?`, as in the version proposed upstream); its own helper no longer shares that name.

data/README.md

@@ -14,6 +14,19 @@ config.rails_minify.enabled = true
 (`config.ar_minimizer.enums = false`, `config.rails_minify.patches = [:omniauth]`).
 Ruby 3.4+, Rails 8.0, 8.1 and main.
 
+Turn it on in tests too, so a call to a method strict mode removed fails in CI
+rather than in production. In development, strict mode alone is enough: the
+same errors show up while you work, and the compiling parts, which don't
+follow code reloads, stay off.
+
+```ruby
+# config/environments/test.rb
+config.rails_minify.enabled = true
+
+# config/environments/development.rb
+config.ar_minimizer.slim = true # strict mode, without the rest
+```
+
 ## Strict by default
 
 Active Record defines, for every column, a reader, a writer and 19 more
@@ -71,6 +84,22 @@ For parts of Rails and common gems that redo work on every request.
   installed when OmniAuth already does this itself (it then has a public
   `Strategy#passthrough?`, as in the version proposed upstream).
 
+## ZJIT after boot
+
+```ruby
+config.rails_minify.zjit = true # instead of RUBY_ZJIT_ENABLE=1 / --zjit
+```
+
+`RUBY_ZJIT_ENABLE=1` turns ZJIT on before anything loads, so it also compiles
+what runs only while booting: RubyGems, requiring and eager loading, and the
+minimizers' own compiling. Rails turns YJIT on after boot (`config.yjit`) for
+this reason, but has no such setting for ZJIT; this is that one. It works on
+its own, without the rest of rails-minify.
+
+On Japan Voyage, per process after the same 132 pages: 12,612 → 9,557
+compiled methods, JIT code 45.9 → 35.3 MB, ZJIT memory 82.9 → 64.6 MB,
+RSS about 25 MB less, the same CPU time per page.
+
 ## Numbers
 
 Japan Voyage, 132 production pages, with everything on (strict mode after
@@ -81,7 +110,26 @@ loaded for wall-clock times to mean much.
 ## Checks
 
 * `bin/rails minify:census`: how much of the app runs compiled, and why the rest doesn't.
-* `bin/rails minify:check`: callbacks naming methods that don't exist (for CI).
+* `bin/rails minify:check` (for CI; works whether or not the rest of
+  rails-minify is on) fails on:
+  * callbacks naming methods that don't exist;
+  * callback chains with more than `callback_limit` (default 5) of the app's
+    own callbacks (Rails' and gems' don't count; controllers per action).
+    Callbacks inherited from `callback_limit_exclude` (default
+    `["ApplicationController"]`: time zone, locale, current user, the
+    per-request basics) don't count either. A base class over the limit is
+    reported once, with the classes that inherit it;
+  * model callbacks that call `save`, `update`, `touch`, ... on the record
+    they run for, which runs the chain again;
+  * an out-of-date manifest.
+
+  ```ruby
+  config.rails_minify.callback_limit = 8
+  config.rails_minify.callback_limit_exclude = ["ApplicationController", "Admin::BaseController"] # [] counts everything
+  config.rails_minify.callback_limit_mode = :warn # report, don't fail
+  config.rails_minify.callback_loops = :warn
+  config.rails_minify.allow_callback_loops = ["Trip#recalculate"] # guarded on purpose
+  ```
 
 ## Things found that are app config, not patches
 

data/lib/rails_minify/railtie.rb

@@ -42,6 +42,18 @@ module RailsMinify
       end
     end
 
+    # ZJIT from after boot, as Rails does for YJIT (config.yjit): the code
+    # that only runs while booting (requiring, eager loading, the minimizers'
+    # compiling) isn't compiled. Japan Voyage, per process: 24% fewer
+    # compiled methods, 18 MB less JIT memory, the same speed.
+    initializer "rails_minify.zjit" do |app|
+      config.after_initialize do
+        next unless app.config.rails_minify.zjit && defined?(RubyVM::ZJIT.enable) && !RubyVM::ZJIT.enabled?
+
+        RubyVM::ZJIT.enable
+      end
+    end
+
     rake_tasks do
       namespace :minify do
         prepare = lambda do
@@ -69,15 +81,39 @@ module RailsMinify
           end
         end
 
-        desc "Fail on what would break at runtime or compile stale: missing callback targets, an out-of-date manifest"
+        desc "Fail on callback problems (missing targets, chains longer than callback_limit, " \
+             "callbacks that save their own record) and an out-of-date manifest"
         task check: :environment do
           app = prepare.call
-          problems = AsMinimizer::Check.callback_targets(routed: AsMinimizer::Check.routed_actions(app.routes))
-          missing = problems.select { it.status == :missing }
-          missing.each { puts "  #{it}" }
-          manifest = ArMinimizer::Census.manifest_state(app.root, ArMinimizer::Index.from_models)
-          puts "callback targets: #{missing.size} missing; manifest: #{manifest}"
-          abort if missing.any? || manifest.start_with?("out of date")
+          options = app.config.rails_minify
+          routed = AsMinimizer::Check.routed_actions(app.routes)
+          failed = false
+          report = lambda do |title, items, mode|
+            next puts("#{title}: none") if items.empty?
+
+            puts "#{title}: #{items.size}#{" (warning)" if mode == :warn}"
+            items.each { puts "  #{it}" }
+            failed ||= mode != :warn
+          end
+
+          missing = AsMinimizer::Check.callback_targets(routed: routed).select { it.status == :missing }
+          report.("callback targets naming no method", missing, :fail)
+
+          limit = options.callback_limit || 5
+          report.("callback chains with more than #{limit} of the app's callbacks",
+            AsMinimizer::Check.long_chains(limit: limit, root: app.root.to_s, routed: routed,
+              exclude: options.callback_limit_exclude.nil? ? ["ApplicationController"] : Array(options.callback_limit_exclude)),
+            (options.callback_limit_mode || :fail).to_sym)
+
+          report.("callbacks saving their own record",
+            AsMinimizer::Check.callback_loops(root: app.root.to_s, allow: Array(options.allow_callback_loops)), (options.callback_loops || :fail).to_sym)
+
+          if defined?(ArMinimizer::Census) && app.root.join("config/ar_minimizer.json").exist?
+            manifest = ArMinimizer::Census.manifest_state(app.root, ArMinimizer::Index.from_models)
+            puts "manifest: #{manifest}"
+            failed ||= manifest.start_with?("out of date")
+          end
+          abort if failed
         end
       end
     end

data/lib/rails_minify/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module RailsMinify
-  VERSION = "0.1.1"
+  VERSION = "0.1.3"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails-minify
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.3
 platform: ruby
 authors:
 - Sampo Kuokkanen
@@ -44,6 +44,9 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.1'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.1.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -51,6 +54,9 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.1'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.1.1
 description: Turns on activerecord-, activesupport- and actionpack-minimizer, and
   patches parts of Rails and common gems that redo work on every request (Action Text
   plain text, Request#host, OmniAuth strategies).