multi_json 1.21.0 → 1.21.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6eb65ee97a46816e1bfe43fbeff173a83846bb1ff85e3c5e4fe97c80cec4dacf
-  data.tar.gz: 9eedd9bd17375a5540dc4878613a9165e49925d1997c24f4d9776f41efb1d7f0
+  metadata.gz: 7cffb1170fe976efac86851b47b4db7208fb1eefc745ea73e6cf4aaeb4aaa281
+  data.tar.gz: d5b3f218ad72f318eb5c86152c87005f27f7fc3b4b265561f1156092cbbd6c4b
 SHA512:
-  metadata.gz: c727bce3f895367c1092319c4725dedde9252706cc6da7a5d8b1ef4bef5abb025869b7a04dc44218dfe53198b8da0c0c36e1d61037824a34a59dcfd78a030f68
-  data.tar.gz: 2dc0b84560382d05f58549875227be1dbc32c274172f2c67e4c395f4baede84217061a10fcf22ebd078ce7e89fb25a88e067f7424b4bdd830c75081e96d2938f
+  metadata.gz: a61eb7a6f29720708c0944b8288ed8a9ce1917f76d9934f3f417935a606fa3da0c8e6e1b8158a57420c49b97ba617222ff3a5e1ed28bbfc23c7146f46e32d0ce
+  data.tar.gz: fc6f0c205a0c6a05740a08de6c3a27d59e482e64f179591b47a718a84d02b7d509e1ffd368d28c6d5edc88a454bb8ce4c64cd784d7283cc4353c82f7d1f2919d

data/lib/multi_json/version.rb

@@ -10,7 +10,7 @@ module MultiJSON
     # Minor version number
     MINOR = 21 unless defined? MultiJSON::Version::MINOR
     # Patch version number
-    PATCH = 0 unless defined? MultiJSON::Version::PATCH
+    PATCH = 1 unless defined? MultiJSON::Version::PATCH
     # Pre-release version suffix
     PRE = nil unless defined? MultiJSON::Version::PRE
 

data/lib/multi_json.rb

@@ -272,52 +272,34 @@ require_relative "multi_json/deprecated"
 #
 # Downstream code that still writes ``MultiJson.parse(...)`` or
 # ``rescue MultiJson::ParseError`` continues to work, but emits a
-# one-time deprecation warning pointing at ``MultiJSON``. The module
-# forwards every method call to {MultiJSON} via {.method_missing} and
-# resolves constant access via {.const_missing}, so both dotted calls
-# and ``::`` constant lookups (including rescue clauses) route through
-# the canonical module.
+# one-time deprecation warning pointing at ``MultiJSON``. Each public
+# method on {MultiJSON} gets an explicit forwarder defined on this
+# module, and constant access resolves via {.const_missing}, so both
+# dotted calls and ``::`` constant lookups (including rescue clauses)
+# route through the canonical module.
 #
 # @api public
 # @deprecated Use {MultiJSON} (all-caps) instead. Will be removed in v2.0.
 module MultiJson
-  class << self
-    # Forward method calls to {MultiJSON}, emitting a one-time warning
-    #
-    # Uses explicit ``*args, **kwargs, &block`` forwarding because
-    # mutant's AST structure table on the current parser version does
-    # not yet recognize ``...`` forwarding nodes, so the module would
-    # crash mutation analysis. The rubocop exclusions below document
-    # that intent.
-    #
-    # @api public
-    # @return [Object] the delegated call's return value
-    # @example
-    #   MultiJson.parse('{"a":1}')   # delegates to MultiJSON.parse
-    # rubocop:disable Naming/BlockForwarding, Style/ArgumentsForwarding
-    def method_missing(name, *args, **kwargs, &block)
+  # Forward every public method MultiJSON exposes through an explicit
+  # singleton method on the legacy MultiJson module, so callers that
+  # capture the method as a Method object (``MultiJson.method(:load)``)
+  # find this forwarder instead of falling back to inherited methods like
+  # ``Kernel#load``. The earlier ``method_missing``-based shim left
+  # ``MultiJson.method(:load)`` resolving to ``Kernel#load`` (because
+  # ``Module#method`` doesn't consult ``method_missing``) and broke
+  # libraries (Sawyer, Octokit, Danger) that capture decoders as Method
+  # objects. Forwarding eagerly fixes the capture path while preserving
+  # the one-time deprecation warning each call emits.
+  (::MultiJSON.public_methods - ::Module.public_methods).each do |forwarded|
+    define_singleton_method(forwarded) do |*args, **kwargs, &block|
       ::MultiJSON.warn_deprecation_once(:multi_json_constant,
         "The MultiJson constant is deprecated and will be removed in v2.0. Use MultiJSON instead.")
-      if ::MultiJSON.respond_to?(name)
-        ::MultiJSON.public_send(name, *args, **kwargs, &block)
-      else
-        super
-      end
-    end
-    # rubocop:enable Naming/BlockForwarding, Style/ArgumentsForwarding
-
-    # Respond to any method {MultiJSON} responds to
-    #
-    # @api public
-    # @param name [Symbol] method name
-    # @param include_private [Boolean] include private methods
-    # @return [Boolean] true if {MultiJSON} responds to the method
-    # @example
-    #   MultiJson.respond_to?(:parse)  #=> true
-    def respond_to_missing?(name, include_private)
-      ::MultiJSON.respond_to?(name, include_private)
+      ::MultiJSON.public_send(forwarded, *args, **kwargs, &block)
     end
+  end
 
+  class << self
     # Resolve missing constants to their {MultiJSON} counterparts
     #
     # Enables ``rescue MultiJson::ParseError`` and

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: multi_json
 version: !ruby/object:Gem::Version
-  version: 1.21.0
+  version: 1.21.1
 platform: ruby
 authors:
 - Michael Bleigh
@@ -43,10 +43,10 @@ licenses:
 - MIT
 metadata:
   bug_tracker_uri: https://github.com/sferik/multi_json/issues
-  changelog_uri: https://github.com/sferik/multi_json/blob/v1.21.0/CHANGELOG.md
-  documentation_uri: https://www.rubydoc.info/gems/multi_json/1.21.0
+  changelog_uri: https://github.com/sferik/multi_json/blob/v1.21.1/CHANGELOG.md
+  documentation_uri: https://www.rubydoc.info/gems/multi_json/1.21.1
   rubygems_mfa_required: 'true'
-  source_code_uri: https://github.com/sferik/multi_json/tree/v1.21.0
+  source_code_uri: https://github.com/sferik/multi_json/tree/v1.21.1
   wiki_uri: https://github.com/sferik/multi_json/wiki
 rdoc_options: []
 require_paths: