rails_spotlight 0.4.0 → 0.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 519dd5ff60d4420f62fb8fb3528e6fb5c56f2561508bd87bc9c534d4fbdbe047
-  data.tar.gz: 4520ba3597c02eb7c9b088ed3d9d1d8610836b3276da85ac0151a854e1022be6
+  metadata.gz: e3080195b8e2481d0af07b1c8a7c0e5d59dc7b90d95afc82e722514676003d01
+  data.tar.gz: 472e0580e0e7fceb406bc6810be98ea2945232f673deaaecd28737d3c9bf54ff
 SHA512:
-  metadata.gz: 5a81db70a08bf70c5e0199e80e8984a0cbb6ee0be2946406305747491696debd52e26b009dbe346aa8c28559bc0e61689c7201060aec186806c0bc12ef305e97
-  data.tar.gz: c9b75ea3bb8bae547d2af74e7538e7b781f6c0d2d05ea5afe6bf96e64790cb4fe1f8ed01800c0ae6559bd2a75e92dc341480c217df2f45aaf185a42eff4d9a31
+  metadata.gz: f72425d6484fe1574d05d5f44189265bab421e834fdde23c9e2c96df0911f0b673750395d86e2b45563cac53dc3f0fc2d82794262c7d24ab3787161890259225
+  data.tar.gz: 98cf2fb54e51e960022a2819aabe04f21bfa66d2f7dd34057fb2fb2d5c4a23481c635d3f40deabc1f387d700b6a1dfb152b1ed5be0135c7ec0139e947db9cb78

data/lib/rails_spotlight/log_interceptor.rb

@@ -23,11 +23,42 @@ module RailsSpotlight
       end
       return true if _skip_logging?(message)
 
-      _push_event(SEVERITY_MAP[severity], message, progname)
+      _rails_spotlight_log(SEVERITY_MAP[severity], message, progname, :broadcast)
       super(severity, message, progname) if defined?(super)
       true
     end
 
+
+    def debug(message = nil, *args, &block)
+      _rails_spotlight_log(:debug, message, nil, :event, &block)
+      super
+    end
+
+    def info(message = nil, *args, &block)
+      _rails_spotlight_log(:info, message, nil, :event, &block)
+      super
+    end
+
+    def warn(message = nil, *args, &block)
+      _rails_spotlight_log(:warn, message, nil, :event, &block)
+      super
+    end
+
+    def error(message = nil, *args, &block)
+      _rails_spotlight_log(:error, message, nil, :event, &block)
+      super
+    end
+
+    def fatal(message = nil, *args, &block)
+      _rails_spotlight_log(:fatal, message, nil, :event, &block)
+      super
+    end
+
+    def unknown(message = nil, *args, &block)
+      _rails_spotlight_log(:unknown, message, nil, :event, &block)
+      super
+    end
+
     private
 
     def _skip_logging?(message)
@@ -37,11 +68,23 @@ module RailsSpotlight
       message.include?(::RailsSpotlight::Channels::SPOTLIGHT_CHANNEL)
     end
 
-    def _push_event(level, message, progname = nil) # rubocop:disable Metrics/PerceivedComplexity, Metrics/CyclomaticComplexity, Metrics/AbcSize
+    def _rails_spotlight_log(level, message, progname = nil, output = :event)
       callsite = Utils.dev_callsite(caller.drop(1))
       name = progname.is_a?(String) || progname.is_a?(Symbol) ? progname : nil
-      AppRequest.current.events << Event.new('rsl.notification.log', 0, 0, 0, callsite.merge(message: message, level: level, progname: name)) if AppRequest.current && callsite
+      message = yield if output == :event && message.nil? && block_given?
+      output == :event ? _push_event(level, message, name, callsite) : _broadcast_log(message, level, callsite, name)
+    rescue StandardError => e
+      RailsSpotlight.config.logger.fatal("#{e.message}\n #{e.backtrace.join("\n ")}")
+    end
+
+    def _push_event(level, message, progname = nil, callsite = {})
+      name = progname.is_a?(String) || progname.is_a?(Symbol) ? progname : nil
+      AppRequest.current.events << Event.new('rsl.notification.log', 0, 0, 0, (callsite || {}).merge(message: message, level: level, progname: name)) if AppRequest.current
+    rescue StandardError => e
+      RailsSpotlight.config.logger.fatal("#{e.message}\n #{e.backtrace.join("\n ")}")
+    end
 
+    def _broadcast_log(message, level, callsite = {}, name = nil)
       return unless ::RailsSpotlight.config.use_action_cable?
       return if message.blank?
 

data/lib/rails_spotlight/middlewares/handlers/sql_action_handler.rb

@@ -14,22 +14,31 @@ module RailsSpotlight
 
         private
 
-        def transaction
+        def transactional(&block)
+          return block.call if force_execution?
+
           ActiveRecord::Base.transaction do
-            begin # rubocop:disable Style/RedundantBegin
-              ActiveSupport::Notifications.subscribed(method(:logger), 'sql.active_record', monotonic: true) do
-                run
-              end
-            rescue => e # rubocop:disable Style/RescueStandardError
-              self.error = e
+            begin
+              block.call
             ensure
-              raise ActiveRecord::Rollback unless force_execution?
+              raise ActiveRecord::Rollback
+            end
+          end
+        end
+
+        def transaction
+          begin # rubocop:disable Style/RedundantBegin
+            ActiveSupport::Notifications.subscribed(method(:logger), 'sql.active_record', monotonic: true) do
+              run
             end
+          rescue => e # rubocop:disable Style/RescueStandardError
+            self.error = e
           end
         end
 
         def run # rubocop:disable Metrics/AbcSize
           RailsSpotlight.config.logger && RailsSpotlight.config.logger.info("Executing query: #{query}") # rubocop:disable Style/SafeNavigation
+
           return self.result = ActiveRecord::Base.connection.exec_query(query) if connection_options.blank? || !ActiveRecord::Base.respond_to?(:connects_to)
 
           connections = ActiveRecord::Base.connects_to(**connection_options)
@@ -48,7 +57,7 @@ module RailsSpotlight
             result: result,
             logs: logs,
             error: error.present? ? error.inspect : nil,
-          query_mode: force_execution? ? 'force' : 'default'
+            query_mode: force_execution? ? 'force' : 'default'
           }
         end
 
@@ -80,9 +89,9 @@ module RailsSpotlight
 
         def connection_options
           @connection_options ||= raw_options
-                                  .symbolize_keys
-                                  .slice(:database, :shards)
-                                  .reject { |_, v| v.nil? || (!v.is_a?(TrueClass) && !v.is_a?(FalseClass) && v.empty?) } # TODO: Check for each rails version
+                                    .symbolize_keys
+                                    .slice(:database, :shards)
+                                    .reject { |_, v| v.nil? || (!v.is_a?(TrueClass) && !v.is_a?(FalseClass) && v.empty?) } # TODO: Check for each rails version
         end
 
         def force_execution?

data/lib/rails_spotlight/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module RailsSpotlight
-  VERSION = '0.4.0'
+  VERSION = '0.4.2'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rails_spotlight
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.4.2
 platform: ruby
 authors:
 - Pawel Niemczyk
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-09-14 00:00:00.000000000 Z
+date: 2024-11-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack-contrib
@@ -39,7 +39,7 @@ dependencies:
         version: 3.0.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: '7.3'
+        version: '8.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -49,7 +49,7 @@ dependencies:
         version: 3.0.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: '7.3'
+        version: '8.1'
 - !ruby/object:Gem::Dependency
   name: action-cable-testing
   requirement: !ruby/object:Gem::Requirement