query_owl 0.4.0 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9dd0e38c57b1b0c94a99a0ffb22f6ecc71f0e28c6d4d6d041dac57bc6e5bf626
-  data.tar.gz: dd6c94c94314dc38da3a757d0e4147dc349ae8842f10e0e29f9fa7fc2e0727ed
+  metadata.gz: 2654c621778910c1a0b1926398480e21b967f86c63ff45b642e635d80bfebcd0
+  data.tar.gz: 54245db5c8aa3a64869f3ce18b9c95b662d01ba58a042150ea551f19e1bf7299
 SHA512:
-  metadata.gz: be09a0350d981ffcd6e2b5b4e57825daf3d4aca7a0747d0f3579188df22ddc5f26babeab619c24e1d47c0e86ebda1dc1dce99ee65ce32cc01ff1cef1f997a95e
-  data.tar.gz: 6d499e2d7972b3252f6eccaae8e415f742e2e28b579c08d9e53f2c7377a998a6c5b7be1a46bc3f4189d353258fe7dc0dd01f5df29eb4986d7851c07ad1179fca
+  metadata.gz: 5d1a2a46359810b4f5e482097264c6d1d402c4344f5cf29b672d89a9c834c04e81936f07f3111454abf549e12c5a30614e6829c5d62563860de6b625220ce234
+  data.tar.gz: 41561e881c73aac6ed2ce8b7d3fc4f1a9f83d35fe92f4473713d274978451d9d7cd4ae7fada1af25739d3222457216ae13fdfb015b41bc8cf87116ee8aaa9b1a

data/app/assets/stylesheets/query_owl/_03_table.css

@@ -26,4 +26,9 @@
 }
 
 .qo-table tr:last-child td { border-bottom: none; }
-.qo-table tr:hover td      { background: #fafafa; }
+.qo-table tr:hover td      { background: #fafafa; }
+
+.qo-table td:nth-child(2) { max-width: 320px; overflow: hidden; text-overflow: ellipsis; white-space: nowrap; }
+
+.qo-request  { white-space: nowrap; }
+.qo-path     { font-family: ui-monospace, "SFMono-Regular", Menlo, monospace; font-size: 11px; opacity: .7; }

data/app/views/query_owl/slow_queries/index.html.erb

@@ -12,6 +12,7 @@
         <th>Type</th>
         <th>SQL / Details</th>
         <th>Info</th>
+        <th>Request</th>
         <th>Recorded At</th>
         <th>Backtrace</th>
       </tr>
@@ -25,6 +26,14 @@
             <% if event[:count] %>count: <%= event[:count] %><% end %>
             <% if event[:duration_ms] %><%= event[:duration_ms] %>ms<% end %>
           </td>
+          <td class="qo-request qo-muted">
+            <% if event[:controller] || event[:action] %>
+              <span class="qo-monospace"><%= [event[:controller], event[:action]].compact.join("#") %></span>
+            <% end %>
+            <% if event[:path] %>
+              <br><span class="qo-path"><%= event[:path] %></span>
+            <% end %>
+          </td>
           <td class="qo-muted"><%= event[:recorded_at]&.strftime("%H:%M:%S") %></td>
           <td class="qo-monospace qo-muted"><%= Array(event[:backtrace]).first %></td>
         </tr>

data/lib/generators/query_owl/install/templates/initializer.rb

@@ -34,6 +34,13 @@ QueryOwl.configure do |config|
   # Disabled by default (nil). Useful for persistence across restarts.
   # config.log_file = Rails.root.join("log/query_owl.log").to_s
 
+  # Paths to skip entirely — accepts strings (prefix match) or regexes.
+  # Useful for health check endpoints and other high-frequency low-value paths.
+  # config.ignore_paths = ["/up", "/healthz", %r{^/assets/}]
+
+  # Controllers to skip — matched against the Rails controller name (e.g. "rails/health").
+  # config.ignore_controllers = ["rails/health", "admin/metrics"]
+
   # Notifiers receive each detected event via #call(event).
   # Defaults to [QueryOwl::Notifiers::Logger] which writes to Rails.logger.
   # Use Console for TTY-aware colorized output (yellow: N+1, red: slow query).

data/lib/query_owl/configuration.rb

@@ -5,13 +5,19 @@ module QueryOwl
 
     attr_reader :log_level, :backtrace_filter
     attr_accessor :enabled, :slow_query_threshold_ms, :n_plus_one_threshold, :backtrace_lines,
-                  :raise_on_n_plus_one, :event_store_size, :dashboard_enabled, :log_file
+                  :raise_on_n_plus_one, :event_store_size, :dashboard_enabled, :log_file,
+                  :ignore_paths, :ignore_controllers
 
     def notifiers
       @notifiers ||= [Notifiers::Logger.new]
     end
 
     def notifiers=(arr)
+      arr.each do |notifier|
+        unless notifier.respond_to?(:call)
+          raise ArgumentError, "notifiers must respond to #call (#{notifier.class} does not)"
+        end
+      end
       @notifiers = arr
     end
 
@@ -26,6 +32,8 @@ module QueryOwl
       @event_store_size        = 100
       @dashboard_enabled       = Rails.env.development?
       @log_file                = nil
+      @ignore_paths            = []
+      @ignore_controllers      = []
     end
 
     def log_level=(level)

data/lib/query_owl/event_store.rb

@@ -4,7 +4,7 @@ module QueryOwl
       def push(event)
         mutex.synchronize do
           ensure_buffer_size
-          buffer[@write_pos] = event.merge(recorded_at: Time.now)
+          buffer[@write_pos] = event.merge(recorded_at: Time.current)
           @write_pos = (@write_pos + 1) % capacity
           @stored    = [@stored + 1, capacity].min
         end

data/lib/query_owl/file_logger.rb

@@ -1,4 +1,5 @@
 require "json"
+require "fileutils"
 
 module QueryOwl
   class FileLogger
@@ -9,9 +10,12 @@ module QueryOwl
         path = QueryOwl.config.log_file
         return unless path
 
+        FileUtils.mkdir_p(File.dirname(path))
         File.open(path, "a") do |f|
           events.each { |e| f.puts(JSON.generate(serializable(e))) }
         end
+      rescue => e
+        Rails.logger.error "[QueryOwl] FileLogger failed: #{e.message}"
       end
 
       private

data/lib/query_owl/middleware.rb

@@ -4,35 +4,63 @@ module QueryOwl
       @app = app
     end
 
-    def raise_on_n_plus_one!(events)
-      event = events.find { |e| e[:type] == :n_plus_one }
-      return unless event
-
-      raise NPlusOneError, "N+1 detected: #{event[:sql]} (#{event[:count]} times) #{event[:backtrace].first}"
-    end
-
     def call(env)
+      tracking = false
       return @app.call(env) unless QueryOwl.config.enabled
+      return @app.call(env) if ignored_path?(env["PATH_INFO"])
 
+      tracking = true
       QueryTracker.start!
       EagerLoadTracker.start!
       @app.call(env)
     ensure
-      params  = env["action_dispatch.request.path_parameters"] || {}
-      RequestContext.set(controller: params[:controller], action: params[:action], path: env["PATH_INFO"])
-      queries    = QueryTracker.stop!
-      eager_data = EagerLoadTracker.stop!
-      context    = RequestContext.current
-      RequestContext.clear
-      events     = (Detector.detect_n_plus_one(queries) +
+      if tracking
+        params     = env["action_dispatch.request.path_parameters"] || {}
+        RequestContext.set(controller: params[:controller], action: params[:action], path: env["PATH_INFO"])
+        queries    = QueryTracker.stop!
+        eager_data = EagerLoadTracker.stop!
+        context    = RequestContext.current
+        RequestContext.clear
+
+        unless ignored_controller?(context[:controller])
+          events = (Detector.detect_n_plus_one(queries) +
                     Detector.detect_slow_queries(queries) +
                     Detector.detect_unused_eager_loads(eager_data))
                    .map { |e| e.merge(context) }
-      events.each { |event| QueryOwl.config.notifiers.each { |notifier| notifier.call(event) } }
-      Logger.log_summary(events)
-      events.each { |e| EventStore.push(e) }
-      FileLogger.append(events)
-      raise_on_n_plus_one!(events) if QueryOwl.config.raise_on_n_plus_one
+          events.each do |event|
+            QueryOwl.config.notifiers.each do |notifier|
+              notifier.call(event)
+            rescue => e
+              Rails.logger.error "[QueryOwl] Notifier #{notifier.class} raised: #{e.message}"
+            end
+          end
+          Logger.log_summary(events)
+          events.each { |e| EventStore.push(e) }
+          FileLogger.append(events)
+          raise_on_n_plus_one!(events) if QueryOwl.config.raise_on_n_plus_one
+        end
+      end
     end
+
+    def raise_on_n_plus_one!(events)
+      event = events.find { |e| e[:type] == :n_plus_one }
+      return unless event
+
+      raise NPlusOneError, "N+1 detected: #{event[:sql]} (#{event[:count]} times) #{event[:backtrace].first}"
+    end
+
+    private
+
+      def ignored_path?(path)
+        QueryOwl.config.ignore_paths.any? do |pattern|
+          pattern.is_a?(Regexp) ? pattern.match?(path) : path.start_with?(pattern.to_s)
+        end
+      end
+
+      def ignored_controller?(controller)
+        return false unless controller
+
+        QueryOwl.config.ignore_controllers.any? { |name| name.to_s == controller }
+      end
   end
 end

data/lib/query_owl/version.rb

@@ -1,3 +1,3 @@
 module QueryOwl
-  VERSION = "0.4.0"
+  VERSION = "0.5.0"
 end

data/lib/tasks/query_owl_tasks.rake

@@ -1,4 +1,8 @@
-# desc "Explaining what the task does"
-# task :query_owl do
-#   # Task goes here
-# end
+namespace :query_owl do
+  desc "Clear all events from the QueryOwl in-memory event store"
+  task clear: :environment do
+    count = QueryOwl::EventStore.size
+    QueryOwl::EventStore.clear
+    puts "[QueryOwl] Event store cleared (#{count} event#{"s" if count != 1} removed)."
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: query_owl
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.5.0
 platform: ruby
 authors:
 - Chuck Smith