action_policy 0.7.7 → 0.7.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: dd9504fed879c71d0ebb7025a8bf8ced503b3ed6dfd2d41b1585db47ca2d5548
-  data.tar.gz: 0c494eaae0d4283a604494af6f3e61326a21402e9ef98321840e755b585a0273
+  metadata.gz: 4a3a3b344c3b13973dae50a7a0400d1de1bc6547bddbf5870d2547b701a222f6
+  data.tar.gz: d9293952684089a73a27678e54b4a5c2b981117cc49b079821249c9af2d4feae
 SHA512:
-  metadata.gz: acdb59593707dc6f0f28b6f3c75f770386a1db35662eea95ae1eb0932fc1900e8a384c10c4e88eb70e3fde853e88414e3442f4994c59a364f94e50efb6c97cf7
-  data.tar.gz: a2deccf57e2c97b5c3dae5730ae7deecf487b49bb08cfcf488034ad3ff94e80495f3d76ad4553d76cf3f7b8f1322157c3c8f9b45179779af14212a4c9d18ce51
+  metadata.gz: ae6665c68d31d55c44e730624bc6dd4cf1e460f9ed26bbdda249189aec7c89258589079f2646f9c5ce9265f57668e046354cd49ae801db0bdd129937f9fab916
+  data.tar.gz: 9dbd344bc5f8bad378bdd40d899726a1dcf1136990a586be4a2a3db0bb9a061f56b19825387c7028743fce0e9d5d70d3497e373b40a96edf53a29f4d30bd38e9

data/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## master
 
+## 0.7.8 (2026-11-03)
+
+- Improve policy caching (less key calculations, fixed collisions). ([@palkan][])
+
+- Fix `ActiveModel::MissingAttributeError` when authorizing a partially selected Active Record model from a memoized context. ([@czj][])
+
 ## 0.7.7 (2026-09-03)
 
 - Fix caching non-persistent Active Record objects as the same entry ([@palkan][])
@@ -566,3 +572,4 @@ This value is now stored in a cache (if any) instead of just the call result (`t
 [@stephannv]: https://github.com/stephannv
 [@sedubois]: https://github.com/sedubois
 [@Pagehey]: https://github.com/Pagehey
+[@czj]: https://github.com/czj

data/lib/.rbnext/3.0/action_policy/behaviours/policy_for.rb

@@ -8,14 +8,14 @@ module ActionPolicy
       using ActionPolicy::Ext::PolicyCacheKey
 
       # Returns policy instance for the record.
-      def policy_for(record:, with: nil, namespace: authorization_namespace, context: nil, allow_nil: false, default: default_authorization_policy_class, strict_namespace: authorization_strict_namespace)
-        context = context ? build_authorization_context.merge(context) : authorization_context
+      def policy_for(record:, with: nil, namespace: authorization_namespace, context: nil, allow_nil: false, default: default_authorization_policy_class, strict_namespace: authorization_strict_namespace, cache_key: nil, resolved_context: nil)
+        resolved_context ||= policy_for_context(context)
 
         policy_class = with || ::ActionPolicy.lookup(
           record,
-          namespace: namespace, context: context, allow_nil: allow_nil, default: default, strict_namespace: strict_namespace
+          namespace: namespace, context: resolved_context, allow_nil: allow_nil, default: default, strict_namespace: strict_namespace
         )
-        policy_class&.new(record, **context)
+        policy_class&.new(record, **resolved_context)
       end
 
       def authorization_context ;  @authorization_context ||= build_authorization_context; end
@@ -59,12 +59,16 @@ module ActionPolicy
         )
       end
 
-      def policy_for_cache_key(record:, with: nil, namespace: nil, context: authorization_context, **__kwrest__)
+      def policy_for_cache_key(record:, context:, with: nil, namespace: authorization_namespace, **opts)
         record_key = record._policy_cache_key(use_object_id: true)
-        context_key = context.values.map { |it| it._policy_cache_key(use_object_id: true) }.join(".")
+        context_key = context.map { |k, v| "#{k}:#{v._policy_cache_key(use_object_id: true)}" }.join(".")
 
         "#{namespace}/#{with}/#{context_key}/#{record_key}"
       end
+
+      def policy_for_context(context)
+        context ? build_authorization_context.merge(context) : authorization_context
+      end
     end
   end
 end

data/lib/.rbnext/3.0/action_policy/behaviours/thread_memoized.rb

@@ -44,14 +44,15 @@ module ActionPolicy
       end
 
       module InstanceMethods # :nodoc:
-        def policy_for(record:, **opts)
-          __policy_thread_memoize__(record, **opts) { super }
+        def policy_for(record:, context: nil, resolved_context: nil, cache_key: nil, **opts)
+          resolved_context ||= policy_for_context(context)
+          cache_key ||= policy_for_cache_key(record: record, context: resolved_context, **opts)
+
+          __policy_thread_memoize__(cache_key) { super(record: record, **opts, resolved_context: resolved_context, cache_key: cache_key) }
         end
       end
 
-      def __policy_thread_memoize__(record, **options)
-        cache_key = policy_for_cache_key(record: record, **options)
-
+      def __policy_thread_memoize__(cache_key)
         ActionPolicy::PerThreadCache.fetch(cache_key) { yield }
       end
     end

data/lib/.rbnext/3.0/action_policy/ext/policy_cache_key.rb

@@ -10,7 +10,7 @@ module ActionPolicy
     #
     # Raises ArgumentError otherwise.
     module PolicyCacheKey # :nodoc: all
-      module ObjectExt
+      refine Object do
         def _policy_cache_key(use_object_id: false)
           return policy_cache_key if respond_to?(:policy_cache_key)
           return cache_key_with_version if respond_to?(:cache_key_with_version)
@@ -22,10 +22,6 @@ module ActionPolicy
         end
       end
 
-      refine Object do
-        ::RubyNext::Core.import_methods(ObjectExt, binding)
-      end
-
       refine NilClass do
         def _policy_cache_key(*__rest__) ;  ""; end
       end

data/lib/.rbnext/3.1/action_policy/behaviours/memoized.rb

@@ -0,0 +1,52 @@
+# frozen_string_literal: true
+
+module ActionPolicy
+  module Behaviours
+    # Per-instance memoization for policies.
+    #
+    # Used by `policy_for` to re-use policy object for records.
+    #
+    # Example:
+    #
+    #   include ActionPolicy::Behaviour
+    #   include ActionPolicy::Memoized
+    #
+    #   record = User.first
+    #   policy = policy_for(record)
+    #   policy2 = policy_for(record)
+    #
+    #   policy.equal?(policy) #=> true
+    #
+    #   policy.equal?(policy_for(record, with: CustomPolicy)) #=> false
+    module Memoized
+      class << self
+        def prepended(base)
+          base.prepend InstanceMethods
+        end
+
+        alias_method :included, :prepended
+      end
+
+      module InstanceMethods # :nodoc:
+        def policy_for(record:, context: nil, resolved_context: nil, cache_key: nil, **opts)
+          resolved_context ||= policy_for_context(context)
+          cache_key ||= policy_for_cache_key(record: record, context: resolved_context, **opts)
+          __policy_memoize__(cache_key) { super(record: record, **opts, resolved_context: resolved_context, cache_key: cache_key) }
+        end
+      end
+
+      def __policy_memoize__(cache_key)
+        return __policies_cache__[cache_key] if
+          __policies_cache__.key?(cache_key)
+
+        policy = yield
+
+        __policies_cache__[cache_key] = policy
+      end
+
+      def __policies_cache__
+        @__policies_cache__ ||= {}
+      end
+    end
+  end
+end

data/lib/.rbnext/3.1/action_policy/behaviours/policy_for.rb

@@ -8,14 +8,14 @@ module ActionPolicy
       using ActionPolicy::Ext::PolicyCacheKey
 
       # Returns policy instance for the record.
-      def policy_for(record:, with: nil, namespace: authorization_namespace, context: nil, allow_nil: false, default: default_authorization_policy_class, strict_namespace: authorization_strict_namespace)
-        context = context ? build_authorization_context.merge(context) : authorization_context
+      def policy_for(record:, with: nil, namespace: authorization_namespace, context: nil, allow_nil: false, default: default_authorization_policy_class, strict_namespace: authorization_strict_namespace, cache_key: nil, resolved_context: nil)
+        resolved_context ||= policy_for_context(context)
 
         policy_class = with || ::ActionPolicy.lookup(
           record,
-          namespace: namespace, context: context, allow_nil: allow_nil, default: default, strict_namespace: strict_namespace
+          namespace: namespace, context: resolved_context, allow_nil: allow_nil, default: default, strict_namespace: strict_namespace
         )
-        policy_class&.new(record, **context)
+        policy_class&.new(record, **resolved_context)
       end
 
       def authorization_context = @authorization_context ||= build_authorization_context
@@ -59,12 +59,16 @@ module ActionPolicy
         )
       end
 
-      def policy_for_cache_key(record:, with: nil, namespace: nil, context: authorization_context, **__kwrest__)
+      def policy_for_cache_key(record:, context:, with: nil, namespace: authorization_namespace, **opts)
         record_key = record._policy_cache_key(use_object_id: true)
-        context_key = context.values.map { |it| it._policy_cache_key(use_object_id: true) }.join(".")
+        context_key = context.map { |k, v| "#{k}:#{v._policy_cache_key(use_object_id: true)}" }.join(".")
 
         "#{namespace}/#{with}/#{context_key}/#{record_key}"
       end
+
+      def policy_for_context(context)
+        context ? build_authorization_context.merge(context) : authorization_context
+      end
     end
   end
 end

data/lib/.rbnext/3.1/action_policy/behaviours/thread_memoized.rb

@@ -0,0 +1,60 @@
+# frozen_string_literal: true
+
+module ActionPolicy
+  module PerThreadCache # :nodoc:
+    CACHE_KEY = "action_policy.per_thread_cache"
+
+    class << self
+      attr_writer :enabled
+
+      def enabled?() = @enabled == true
+
+      def fetch(key)
+        return yield unless enabled?
+
+        store = (Thread.current[CACHE_KEY] ||= {})
+
+        return store[key] if store.key?(key)
+
+        store[key] = yield
+      end
+
+      def clear_all
+        Thread.current[CACHE_KEY] = {}
+      end
+    end
+
+    # Turn off by default in test env
+    self.enabled = !(ENV["RAILS_ENV"] == "test" || ENV["RACK_ENV"] == "test")
+  end
+
+  module Behaviours
+    # Per-thread memoization for policies.
+    #
+    # Used by `policy_for` to re-use policy object for records.
+    #
+    # NOTE: don't forget to clear thread cache with ActionPolicy::PerThreadCache.clear_all
+    module ThreadMemoized
+      class << self
+        def prepended(base)
+          base.prepend InstanceMethods
+        end
+
+        alias_method :included, :prepended
+      end
+
+      module InstanceMethods # :nodoc:
+        def policy_for(record:, context: nil, resolved_context: nil, cache_key: nil, **opts)
+          resolved_context ||= policy_for_context(context)
+          cache_key ||= policy_for_cache_key(record: record, context: resolved_context, **opts)
+
+          __policy_thread_memoize__(cache_key) { super(record: record, **opts, resolved_context: resolved_context, cache_key: cache_key) }
+        end
+      end
+
+      def __policy_thread_memoize__(cache_key)
+        ActionPolicy::PerThreadCache.fetch(cache_key) { yield }
+      end
+    end
+  end
+end

data/lib/.rbnext/3.2/action_policy/ext/policy_cache_key.rb

@@ -10,7 +10,7 @@ module ActionPolicy
     #
     # Raises ArgumentError otherwise.
     module PolicyCacheKey # :nodoc: all
-      module ObjectExt
+      refine Object do
         def _policy_cache_key(use_object_id: false)
           return policy_cache_key if respond_to?(:policy_cache_key)
           return cache_key_with_version if respond_to?(:cache_key_with_version)
@@ -22,10 +22,6 @@ module ActionPolicy
         end
       end
 
-      refine Object do
-        import_methods ObjectExt
-      end
-
       refine NilClass do
         def _policy_cache_key(*__rest__) = ""
       end

data/lib/action_policy/behaviours/memoized.rb

@@ -28,14 +28,14 @@ module ActionPolicy
       end
 
       module InstanceMethods # :nodoc:
-        def policy_for(record:, **opts)
-          __policy_memoize__(record, **opts) { super }
+        def policy_for(record:, context: nil, resolved_context: nil, cache_key: nil, **opts)
+          resolved_context ||= policy_for_context(context)
+          cache_key ||= policy_for_cache_key(record:, context: resolved_context, **opts)
+          __policy_memoize__(cache_key) { super(record:, **opts, resolved_context:, cache_key:) }
         end
       end
 
-      def __policy_memoize__(record, **options)
-        cache_key = policy_for_cache_key(record: record, **options)
-
+      def __policy_memoize__(cache_key)
         return __policies_cache__[cache_key] if
           __policies_cache__.key?(cache_key)
 

data/lib/action_policy/behaviours/policy_for.rb

@@ -8,14 +8,14 @@ module ActionPolicy
       using ActionPolicy::Ext::PolicyCacheKey
 
       # Returns policy instance for the record.
-      def policy_for(record:, with: nil, namespace: authorization_namespace, context: nil, allow_nil: false, default: default_authorization_policy_class, strict_namespace: authorization_strict_namespace)
-        context = context ? build_authorization_context.merge(context) : authorization_context
+      def policy_for(record:, with: nil, namespace: authorization_namespace, context: nil, allow_nil: false, default: default_authorization_policy_class, strict_namespace: authorization_strict_namespace, cache_key: nil, resolved_context: nil)
+        resolved_context ||= policy_for_context(context)
 
         policy_class = with || ::ActionPolicy.lookup(
           record,
-          namespace:, context:, allow_nil:, default:, strict_namespace:
+          namespace:, context: resolved_context, allow_nil:, default:, strict_namespace:
         )
-        policy_class&.new(record, **context)
+        policy_class&.new(record, **resolved_context)
       end
 
       def authorization_context = @authorization_context ||= build_authorization_context
@@ -59,12 +59,16 @@ module ActionPolicy
         )
       end
 
-      def policy_for_cache_key(record:, with: nil, namespace: nil, context: authorization_context, **)
+      def policy_for_cache_key(record:, context:, with: nil, namespace: authorization_namespace, **opts)
         record_key = record._policy_cache_key(use_object_id: true)
-        context_key = context.values.map { it._policy_cache_key(use_object_id: true) }.join(".")
+        context_key = context.map { |k, v| "#{k}:#{v._policy_cache_key(use_object_id: true)}" }.join(".")
 
         "#{namespace}/#{with}/#{context_key}/#{record_key}"
       end
+
+      def policy_for_context(context)
+        context ? build_authorization_context.merge(context) : authorization_context
+      end
     end
   end
 end

data/lib/action_policy/behaviours/thread_memoized.rb

@@ -44,14 +44,15 @@ module ActionPolicy
       end
 
       module InstanceMethods # :nodoc:
-        def policy_for(record:, **opts)
-          __policy_thread_memoize__(record, **opts) { super }
+        def policy_for(record:, context: nil, resolved_context: nil, cache_key: nil, **opts)
+          resolved_context ||= policy_for_context(context)
+          cache_key ||= policy_for_cache_key(record:, context: resolved_context, **opts)
+
+          __policy_thread_memoize__(cache_key) { super(record:, **opts, resolved_context:, cache_key:) }
         end
       end
 
-      def __policy_thread_memoize__(record, **options)
-        cache_key = policy_for_cache_key(record: record, **options)
-
+      def __policy_thread_memoize__(cache_key)
         ActionPolicy::PerThreadCache.fetch(cache_key) { yield }
       end
     end

data/lib/action_policy/ext/policy_cache_key.rb

@@ -10,7 +10,7 @@ module ActionPolicy
     #
     # Raises ArgumentError otherwise.
     module PolicyCacheKey # :nodoc: all
-      module ObjectExt
+      refine Object do
         def _policy_cache_key(use_object_id: false)
           return policy_cache_key if respond_to?(:policy_cache_key)
           return cache_key_with_version if respond_to?(:cache_key_with_version)
@@ -22,10 +22,6 @@ module ActionPolicy
         end
       end
 
-      refine Object do
-        import_methods ObjectExt
-      end
-
       refine NilClass do
         def _policy_cache_key(*) = ""
       end

data/lib/action_policy/rails/ext/active_record.rb

@@ -19,10 +19,10 @@ end)
 # Ensure non-persistent models are not cached as a single entry
 ActiveRecord::Base.include(Module.new do
   def policy_cache_key
-    if persisted?
-      cache_key_with_version
-    else
-      object_id.to_s
-    end
+    return object_id.to_s unless persisted?
+    # A select leaving updated_at out makes cache_version raise MissingAttributeError
+    return object_id.to_s if !has_attribute?("updated_at") && self.class.attribute_types.key?("updated_at")
+
+    cache_key_with_version
   end
 end)

data/lib/action_policy/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ActionPolicy
-  VERSION = "0.7.7"
+  VERSION = "0.7.8"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: action_policy
 version: !ruby/object:Gem::Version
-  version: 0.7.7
+  version: 0.7.8
 platform: ruby
 authors:
 - Vladimir Dementyev
@@ -149,19 +149,18 @@ files:
 - lib/.rbnext/3.0/action_policy/rspec/have_authorized_scope.rb
 - lib/.rbnext/3.0/action_policy/utils/pretty_print.rb
 - lib/.rbnext/3.0/action_policy/utils/suggest_message.rb
+- lib/.rbnext/3.1/action_policy/behaviours/memoized.rb
 - lib/.rbnext/3.1/action_policy/behaviours/policy_for.rb
 - lib/.rbnext/3.1/action_policy/behaviours/scoping.rb
+- lib/.rbnext/3.1/action_policy/behaviours/thread_memoized.rb
 - lib/.rbnext/3.1/action_policy/ext/module_namespace.rb
-- lib/.rbnext/3.1/action_policy/ext/policy_cache_key.rb
 - lib/.rbnext/3.1/action_policy/policy/authorization.rb
-- lib/.rbnext/3.2/action_policy/behaviours/policy_for.rb
 - lib/.rbnext/3.2/action_policy/ext/policy_cache_key.rb
 - lib/.rbnext/3.2/action_policy/lookup_chain.rb
 - lib/.rbnext/3.2/action_policy/policy/core.rb
 - lib/.rbnext/3.2/action_policy/rspec/be_authorized_to.rb
 - lib/.rbnext/3.2/action_policy/rspec/have_authorized_scope.rb
 - lib/.rbnext/3.2/action_policy/utils/suggest_message.rb
-- lib/.rbnext/3.4/action_policy/behaviours/policy_for.rb
 - lib/.rbnext/3.4/action_policy/i18n.rb
 - lib/.rbnext/3.4/action_policy/policy/cache.rb
 - lib/.rbnext/3.4/action_policy/policy/pre_check.rb

data/lib/.rbnext/3.1/action_policy/ext/policy_cache_key.rb

@@ -1,72 +0,0 @@
-# frozen_string_literal: true
-
-module ActionPolicy
-  module Ext
-    # Adds #_policy_cache_key method to Object,
-    # which just call #policy_cache_key or #cache_key
-    # or #object_id (if `use_object_id` parameter is set to true).
-    #
-    # For other core classes returns string representation.
-    #
-    # Raises ArgumentError otherwise.
-    module PolicyCacheKey # :nodoc: all
-      module ObjectExt
-        def _policy_cache_key(use_object_id: false)
-          return policy_cache_key if respond_to?(:policy_cache_key)
-          return cache_key_with_version if respond_to?(:cache_key_with_version)
-          return cache_key if respond_to?(:cache_key)
-
-          return object_id.to_s if use_object_id == true
-
-          raise ArgumentError, "object is not cacheable"
-        end
-      end
-
-      refine Object do
-        ::RubyNext::Core.import_methods(ObjectExt, binding)
-      end
-
-      refine NilClass do
-        def _policy_cache_key(*__rest__) = ""
-      end
-
-      refine TrueClass do
-        def _policy_cache_key(*__rest__) = "t"
-      end
-
-      refine FalseClass do
-        def _policy_cache_key(*__rest__) = "f"
-      end
-
-      refine String do
-        def _policy_cache_key(*__rest__) = self
-      end
-
-      refine Symbol do
-        def _policy_cache_key(*__rest__) = to_s
-      end
-
-      if RUBY_PLATFORM.match?(/java/i)
-        refine Integer do
-          def _policy_cache_key(*__rest__) = to_s
-        end
-
-        refine Float do
-          def _policy_cache_key(*__rest__) = to_s
-        end
-      else
-        refine Numeric do
-          def _policy_cache_key(*__rest__) = to_s
-        end
-      end
-
-      refine Time do
-        def _policy_cache_key(*__rest__) = to_s
-      end
-
-      refine Module do
-        def _policy_cache_key(*__rest__) = name
-      end
-    end
-  end
-end

data/lib/.rbnext/3.2/action_policy/behaviours/policy_for.rb

@@ -1,70 +0,0 @@
-# frozen_string_literal: true
-
-module ActionPolicy
-  module Behaviours
-    # Adds `policy_for` method
-    module PolicyFor
-      require "action_policy/ext/policy_cache_key"
-      using ActionPolicy::Ext::PolicyCacheKey
-
-      # Returns policy instance for the record.
-      def policy_for(record:, with: nil, namespace: authorization_namespace, context: nil, allow_nil: false, default: default_authorization_policy_class, strict_namespace: authorization_strict_namespace)
-        context = context ? build_authorization_context.merge(context) : authorization_context
-
-        policy_class = with || ::ActionPolicy.lookup(
-          record,
-          namespace:, context:, allow_nil:, default:, strict_namespace:
-        )
-        policy_class&.new(record, **context)
-      end
-
-      def authorization_context = @authorization_context ||= build_authorization_context
-
-      def build_authorization_context
-        Kernel.raise NotImplementedError, "Please, define `build_authorization_context` method!"
-      end
-
-      def authorization_namespace
-        # override to provide specific authorization namespace
-      end
-
-      def default_authorization_policy_class
-        # override to provide a policy class use when no policy found
-      end
-
-      def authorization_strict_namespace
-        # override to provide strict namespace lookup option
-      end
-
-      # Override this method to provide implicit authorization target
-      # that would be used in case `record` is not specified in
-      # `authorize!` and `allowed_to?` call.
-      #
-      # It is also used to infer a policy for scoping (in `authorized_scope` method).
-      def implicit_authorization_target
-        # no-op
-      end
-
-      # Return implicit authorization target or raises an exception if it's nil
-      def implicit_authorization_target!
-        implicit_authorization_target || Kernel.raise(
-          NotFound,
-          [
-            self,
-            "Couldn't find implicit authorization target " \
-            "for #{self.class}. " \
-            "Please, provide policy class explicitly using `with` option or " \
-            "define the `implicit_authorization_target` method."
-          ]
-        )
-      end
-
-      def policy_for_cache_key(record:, with: nil, namespace: nil, context: authorization_context, **__kwrest__)
-        record_key = record._policy_cache_key(use_object_id: true)
-        context_key = context.values.map { |it| it._policy_cache_key(use_object_id: true) }.join(".")
-
-        "#{namespace}/#{with}/#{context_key}/#{record_key}"
-      end
-    end
-  end
-end

data/lib/.rbnext/3.4/action_policy/behaviours/policy_for.rb

@@ -1,70 +0,0 @@
-# frozen_string_literal: true
-
-module ActionPolicy
-  module Behaviours
-    # Adds `policy_for` method
-    module PolicyFor
-      require "action_policy/ext/policy_cache_key"
-      using ActionPolicy::Ext::PolicyCacheKey
-
-      # Returns policy instance for the record.
-      def policy_for(record:, with: nil, namespace: authorization_namespace, context: nil, allow_nil: false, default: default_authorization_policy_class, strict_namespace: authorization_strict_namespace)
-        context = context ? build_authorization_context.merge(context) : authorization_context
-
-        policy_class = with || ::ActionPolicy.lookup(
-          record,
-          namespace:, context:, allow_nil:, default:, strict_namespace:
-        )
-        policy_class&.new(record, **context)
-      end
-
-      def authorization_context = @authorization_context ||= build_authorization_context
-
-      def build_authorization_context
-        Kernel.raise NotImplementedError, "Please, define `build_authorization_context` method!"
-      end
-
-      def authorization_namespace
-        # override to provide specific authorization namespace
-      end
-
-      def default_authorization_policy_class
-        # override to provide a policy class use when no policy found
-      end
-
-      def authorization_strict_namespace
-        # override to provide strict namespace lookup option
-      end
-
-      # Override this method to provide implicit authorization target
-      # that would be used in case `record` is not specified in
-      # `authorize!` and `allowed_to?` call.
-      #
-      # It is also used to infer a policy for scoping (in `authorized_scope` method).
-      def implicit_authorization_target
-        # no-op
-      end
-
-      # Return implicit authorization target or raises an exception if it's nil
-      def implicit_authorization_target!
-        implicit_authorization_target || Kernel.raise(
-          NotFound,
-          [
-            self,
-            "Couldn't find implicit authorization target " \
-            "for #{self.class}. " \
-            "Please, provide policy class explicitly using `with` option or " \
-            "define the `implicit_authorization_target` method."
-          ]
-        )
-      end
-
-      def policy_for_cache_key(record:, with: nil, namespace: nil, context: authorization_context, **)
-        record_key = record._policy_cache_key(use_object_id: true)
-        context_key = context.values.map { |it| it._policy_cache_key(use_object_id: true) }.join(".")
-
-        "#{namespace}/#{with}/#{context_key}/#{record_key}"
-      end
-    end
-  end
-end