chewy 8.4.0 → 8.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bbef8e814a251ebcf189865c37c82476a8e3397983c4ede20694df87a7a61bd9
-  data.tar.gz: e673c1ab4ae490dd3ae716bcb3e617a0661f87c3f43143f59237a46228777375
+  metadata.gz: c7ffb0b83cbd361f40168edad592083762af5035dc02ca60f205f16c6224a9a6
+  data.tar.gz: ca56cddc2b59646fbaa9188c707c1d9f23d9dabf2166e1a1ff168d83adf6e6ef
 SHA512:
-  metadata.gz: '04927525ee47f4f5591ed3f826d2477a87f267be465b739070c82cf1c54d558fe77d56bc88868d35be5fa6c27c8a0302ac31b6fd1f7c79c18b0759eb3e14eabf'
-  data.tar.gz: 62c30c3643ed78e661409929e5a075f02cbac789978f84d0a89531e00ab90764782dc085a7a44243b53b5613466794baa4b881eb22c55abdbde7b46bbb583966
+  metadata.gz: 91751c0575275edd79e08a945fed07b775a6cd0cd216c3fcba95fbab4085b22dd476d23485c63ee93d740b3f2b8e08bc742789e6f60313fcc49d43013bd48ebe
+  data.tar.gz: 4de8630df6346a7eccbbd14ca0cc2b5d84d636e314c58840cae605433a570e6a9b6c07bf8d2f27b695808efcdc491b41006fd4a18038412ee485fe36669711bb

data/CHANGELOG.md

@@ -8,6 +8,16 @@
 
 ### Changes
 
+## 8.4.1 (2026-06-19)
+
+### New Features
+
+### Bug Fixes
+
+* [#1043](https://github.com/toptal/chewy/pull/1043)Fixed `field ..., value: proc(&:method)` (and other splat-declaring procs) raising `ArgumentError: wrong number of arguments` during import. The compiled compose path introduced in 8.4.0 forwarded `crutches`/`context` as extra positional arguments to splat procs; `Symbol#to_proc` then passed them on to the method. Splat procs are now called with the object alone, matching the plain compose path. ([@AlfonsoUceda][])
+
+### Changes
+
 ## 8.4.0 (2026-06-17)
 
 ### New Features

data/lib/chewy/index/compiled.rb

@@ -187,17 +187,24 @@ module Chewy
             idx = @procs.size
             @procs << v
             procs_ref = "compiled_procs[#{idx}]"
-            param_count = positional_param_count(v)
-            case v.arity
-            when 0
+            if v.arity.zero?
               "#{obj_var}.instance_exec(&#{procs_ref})"
+            elsif v.parameters.any? { |type, _| type == :rest }
+              # Procs that declare a splat — Symbol#to_proc (`proc(&:method)`,
+              # parameters `[[:req], [:rest]]`) and `->(*args)` — must NOT
+              # receive crutches/context as extra positional args: a symbol
+              # proc would forward them to the method (`obj.method(crutches,
+              # context)` => "wrong number of arguments"). Mirror the plain
+              # Base#value_by_proc negative-arity branch (`value.call(*object)`)
+              # so the compiled and fallback paths behave identically.
+              "#{procs_ref}.call(*#{obj_var})"
             else
               # Pass only as many of (object, crutches, context) as the
               # proc actually declares. This keeps lambdas with optional
               # args (negative arity like `->(o, c=nil)`) from being
               # called with too many arguments. Anything beyond context
               # truncates to all three.
-              args = [obj_var, 'crutches', 'context'].first(param_count)
+              args = [obj_var, 'crutches', 'context'].first(positional_param_count(v))
               "#{procs_ref}.call(#{args.join(', ')})"
             end
           else
@@ -212,14 +219,13 @@ module Chewy
           end
         end
 
-        # Number of positional parameters declared by `proc`, counting
-        # required + optional. Splats and keyword args contribute one
-        # bucket each so the caller still passes all three context args.
+        # Number of (object, crutches, context) args to pass to a splat-free
+        # proc: its required + optional positional parameters, capped at the
+        # three available. Splat procs are handled by the caller and never
+        # reach here.
         def positional_param_count(proc)
-          params = proc.parameters
-          required = params.count { |type, _| %i[req opt].include?(type) }
-          has_splat = params.any? { |type, _| type == :rest }
-          has_splat ? 3 : [required, 3].min
+          required = proc.parameters.count { |type, _| %i[req opt].include?(type) }
+          [required, 3].min
         end
 
         def safe_identifier?(name)

data/lib/chewy/version.rb

@@ -1,3 +1,3 @@
 module Chewy
-  VERSION = '8.4.0'.freeze
+  VERSION = '8.4.1'.freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: chewy
 version: !ruby/object:Gem::Version
-  version: 8.4.0
+  version: 8.4.1
 platform: ruby
 authors:
 - Toptal, LLC