Merge pull request #904 from Shopify/query_intent_support

Accept `QueryIntent` from Active Record 8.2 in `execute_intent`

Author: gmcgibbon

lib/semian/activerecord_trilogy_adapter.rb

@@ -80,16 +80,23 @@ def initialize(*options)
       super
     end
 
-    def raw_execute(sql, *)
-      if Semian::ActiveRecordTrilogyAdapter.query_allowlisted?(sql)
-        super
-      else
+    if ActiveRecord.version >= Gem::Version.new("8.2.a")
+      def execute_intent(intent)
+        return super if Semian::ActiveRecordTrilogyAdapter.query_allowlisted?(intent.processed_sql)
+
+        acquire_semian_resource(adapter: :trilogy_adapter, scope: :query) do
+          super
+        end
+      end
+    else
+      def raw_execute(sql, ...)
+        return super if Semian::ActiveRecordTrilogyAdapter.query_allowlisted?(sql)
+
         acquire_semian_resource(adapter: :trilogy_adapter, scope: :query) do
           super
         end
       end
     end
-    ruby2_keywords :raw_execute
 
     def active?
       acquire_semian_resource(adapter: :trilogy_adapter, scope: :ping) do