Integration#patch method now always returns Hash

lib/datadog/ci/contrib/instrumentation.rb

@@ -26,7 +26,7 @@ def self.instrument(integration_name, options = {}, &block)
           return unless integration.enabled
 
           patch_results = integration.patch
-          if patch_results == true
+          if patch_results[:ok]
             # try to patch dependant integrations (for example knapsack that depends on rspec)
             dependants = integration.dependants
               .map { |name| fetch_integration(name) }
@@ -62,7 +62,7 @@ def self.instrument_on_session_start
             Datadog.logger.debug "#{name} is allowed to be late instrumented"
 
             patch_results = integration.patch
-            if patch_results == true
+            if patch_results[:ok]
               Datadog.logger.debug("#{name} is patched")
             else
               Datadog.logger.debug("#{name} is not patched (#{patch_results})")

lib/datadog/ci/contrib/integration.rb

@@ -97,6 +97,7 @@ def patch
           patcher_klass = patcher
           if !patchable? || patcher_klass.nil?
             return {
+              ok: false,
               available: available?,
               loaded: loaded?,
               compatible: compatible?,
@@ -105,7 +106,7 @@ def patch
           end
 
           patcher_klass.patch
-          true
+          {ok: true}
         end
 
         # Can the patch for this integration be applied automatically?

sig/datadog/ci/contrib/integration.rbs

@@ -22,7 +22,7 @@ module Datadog
 
         def patcher: () -> Datadog::Tracing::Contrib::Patcher?
 
-        def patch: () -> (bool | Hash[Symbol, bool])
+        def patch: () -> Hash[Symbol, bool]
 
         def late_instrument?: () -> bool
 

sig/datadog/ci/contrib/patcher.rbs

@@ -2,12 +2,12 @@ module Datadog
   module CI
     module Contrib
       module Patcher
-        def self.included: (untyped base) -> untyped
+        def self.included: (Class base) -> void
 
         module CommonMethods
-          attr_accessor patch_error_result: untyped
+          attr_accessor patch_error_result: Hash[Symbol, untyped]
 
-          attr_accessor patch_successful: untyped
+          attr_accessor patch_successful: bool
 
           def patch_name: () -> String?
 
@@ -17,13 +17,11 @@ module Datadog
 
           def patch: () -> void
 
-          def on_patch_error: (untyped e) -> untyped
-
-          def default_tags: () -> untyped
+          def on_patch_error: (StandardError e) -> Hash[Symbol, untyped]
 
           private
 
-          def patch_only_once: () -> untyped
+          def patch_only_once: () -> Datadog::Core::Utils::OnlyOnce
 
           @patch_only_once: Datadog::Core::Utils::OnlyOnce
         end