inner_performance 0.1.0 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5b0f7620adf9d09e4c9a58a007fdf054d5a89760ad099e8d699bc3deff12f00d
-  data.tar.gz: 271e25fe6de8f021a05ac78ac295cbcf3b8c69ebde3b798151526143776eed86
+  metadata.gz: c6fb86a53413ef53d7256b526e4b75dd7d910473fc8d047b97976285ad82a91f
+  data.tar.gz: 60147dd06fb7069a809c402e3cd9aeebf18df7d4921d1df52f0733159390ec22
 SHA512:
-  metadata.gz: 1cdcbbef0725de7a886b2c10ad2a392d0bfe77c9591bdebc4d26219bba832caf25694a7a79d4ef9476748db9e576e48214db5edf896e0688906e6f36bedb4e2d
-  data.tar.gz: 3321bac2c9273787a3b3d0a3c6ed84e8a0a702be9cb180fada9bd5188eca7412ca5e2275702e20b043da06699a945ea8da82cc296be8145521bc4bb036b1b212
+  metadata.gz: 71ffe8d954c92f5736d9a57d161657dbca3cbbea5303fb42f31a5f007af962096f8a9709a190eb21bb10cfd78004ff628f7548317fc90505971d2f0893a3477b
+  data.tar.gz: cc4a9176b3d7c787f0bd9636c294ffc554471fd3466c1a10c92eb267dd67d273152df7e00fbdb21eee520157b7cdf11a40fa1738b9762eeed8d551fc2648e293

data/README.md

@@ -1,4 +1,4 @@
-# InnerPerformance
+# ☯️ InnerPerformance [![Gem Version](https://badge.fury.io/rb/inner_performance.svg)](https://badge.fury.io/rb/inner_performance)
 Database-backed modest performance monitoring tool for your Rails app.
 
 <img width="1188" alt="image" src="https://github.com/user-attachments/assets/cb659c76-8139-4ee0-b683-d6acef227dc6">
@@ -62,7 +62,7 @@ InnerPerformance.configure do |config|
   #   the job that saves the events because that leeds to infinite loop.
   #   Better not remove this rule as it will lead to stack overflow.
   config.ignore_rules.push(
-    proc { |event| event.is_a?(ActiveSupport::Notifications::Event) }
+    proc { |event| !event.is_a?(ActiveSupport::Notifications::Event) }
   )
 end
 ```

data/app/controllers/inner_performance/dashboard_controller.rb

@@ -1,10 +1,7 @@
 module InnerPerformance
   class DashboardController < ApplicationController
     def index
-      @q = InnerPerformance::Event.all.ransack(params[:q])
-      @q.sorts = "created_at desc" if @q.sorts.empty?
-      @recent_events = @q.result.limit(100)
-
+      @recent_events = InnerPerformance::Event.all.order(id: :desc).limit(25)
       @average_req_duration = InnerPerformance::Events::ProcessActionActionController.all.average(:duration)
       @average_job_duration = InnerPerformance::Events::PerformActiveJob.average(:duration) || 0
       @biggest_events = \

data/app/controllers/inner_performance/events_controller.rb

@@ -1,7 +1,6 @@
 module InnerPerformance
   class EventsController < ApplicationController
     include Pagy::Backend
-    include Pagy::Frontend
 
     def index
       @q = InnerPerformance::Event.all.ransack(params[:q])

data/app/helpers/inner_performance/application_helper.rb

@@ -1,5 +1,7 @@
 module InnerPerformance
   module ApplicationHelper
+    include Pagy::Frontend
+
     # Based on https://stackoverflow.com/a/45428183/552936 and
     # https://datadome.co/learning-center/how-to-reduce-server-response-time/
     def row_class_from_duration(duration)

data/app/views/inner_performance/dashboard/index.html.erb

@@ -41,14 +41,12 @@
 <h4 class="mt-4">Recent events</h4>
 <table class="table table-hover">
   <thead>
-    <th><%= sort_link @q, :name, hide_indicator: true %></th>
-    <th><%= sort_link @q, :duration, hide_indicator: true %></th>
-    <th><%= sort_link @q, :db_runtime, hide_indicator: true %></th>
-    <th class="text-end"><%= sort_link @q, :created_at, hide_indicator: true %></th>
+    <th>Name</th>
+    <th>Duration</th>
+    <th>DB</th>
+    <th class="text-end">When</th>
   </thead>
   <tbody>
-    <% @recent_events.each do |event| %>
-      <%= render event.becomes(InnerPerformance::Event) %>
-    <% end %>
+    <%= render collection: @recent_events, partial: 'inner_performance/events/event', cached: true %>
   </tbody>
 </table>

data/app/views/inner_performance/events/index.html.erb

@@ -21,9 +21,7 @@
     <th class="text-end"><%= sort_link @q, :created_at, hide_indicator: true %></th>
   </thead>
   <tbody>
-    <% @events.each do |event| %>
-      <%= render event.becomes(InnerPerformance::Event) %>
-    <% end %>
+    <%= render collection: @events, partial: 'inner_performance/events/event', cached: true %>
   </tbody>
 </table>
 

data/db/migrate/20241124111458_add_type_to_inner_performance_events.rb

@@ -1,4 +1,4 @@
-class AddTypeToInnerPerformanceEvents < ActiveRecord::Migration[8.0]
+class AddTypeToInnerPerformanceEvents < ActiveRecord::Migration[7.1]
   def change
     add_column :inner_performance_events, :type, :string
   end

data/lib/inner_performance/configuration.rb

@@ -13,8 +13,8 @@ module InnerPerformance
       @events_retention = 1.week
       @medium_duration_range = [200, 999]
       @ignore_rules = [
-        proc { |event| rand(100) <= InnerPerformance.configuration.sample_rates[event.name.to_s] },
-        proc { |event| (event.payload[:job]&.class&.name || '').exclude?('InnerPerformance') }
+        proc { |event| rand(100) > InnerPerformance.configuration.sample_rates[event.name.to_s] },
+        proc { |event| (event.payload[:job]&.class&.name || '').include?('InnerPerformance') }
       ]
     end
   end

data/lib/inner_performance/version.rb

@@ -1,3 +1,3 @@
 module InnerPerformance
-  VERSION = "0.1.0"
+  VERSION = "0.1.2"
 end

data/lib/inner_performance.rb

@@ -46,10 +46,12 @@ module InnerPerformance
       end
     end
 
+    # Iterate throug the ignored_rules array and return false on first rule
+    # that returns `true`
     def save_event?(event)
       InnerPerformance.configuration.ignore_rules.each_with_object([]) do |rule, arr|
         arr << rule.call(event)
-      end.all? { |r| r == true }
+      end.find { |r| r == true }
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: inner_performance
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.2
 platform: ruby
 authors:
 - mbajur