apm_bro 0.1.13 → 0.1.15

data/lib/apm_bro/view_rendering_subscriber.rb

@@ -5,17 +5,17 @@ require "active_support/notifications"
 module ApmBro
   class ViewRenderingSubscriber
     # Rails view rendering events
-    RENDER_TEMPLATE_EVENT = "render_template.action_view".freeze
-    RENDER_PARTIAL_EVENT = "render_partial.action_view".freeze
-    RENDER_COLLECTION_EVENT = "render_collection.action_view".freeze
-    
+    RENDER_TEMPLATE_EVENT = "render_template.action_view"
+    RENDER_PARTIAL_EVENT = "render_partial.action_view"
+    RENDER_COLLECTION_EVENT = "render_collection.action_view"
+
     THREAD_LOCAL_KEY = :apm_bro_view_events
 
     def self.subscribe!(client: Client.new)
       # Track template rendering
       ActiveSupport::Notifications.subscribe(RENDER_TEMPLATE_EVENT) do |name, started, finished, _unique_id, data|
         duration_ms = ((finished - started) * 1000.0).round(2)
-        
+
         view_info = {
           type: "template",
           identifier: safe_identifier(data[:identifier]),
@@ -24,14 +24,14 @@ module ApmBro
           virtual_path: data[:virtual_path],
           rendered_at: Time.now.utc.to_i
         }
-        
+
         add_view_event(view_info)
       end
 
       # Track partial rendering
       ActiveSupport::Notifications.subscribe(RENDER_PARTIAL_EVENT) do |name, started, finished, _unique_id, data|
         duration_ms = ((finished - started) * 1000.0).round(2)
-        
+
         view_info = {
           type: "partial",
           identifier: safe_identifier(data[:identifier]),
@@ -41,14 +41,14 @@ module ApmBro
           cache_key: data[:cache_key],
           rendered_at: Time.now.utc.to_i
         }
-        
+
         add_view_event(view_info)
       end
 
       # Track collection rendering (for partials rendered in loops)
       ActiveSupport::Notifications.subscribe(RENDER_COLLECTION_EVENT) do |name, started, finished, _unique_id, data|
         duration_ms = ((finished - started) * 1000.0).round(2)
-        
+
         view_info = {
           type: "collection",
           identifier: safe_identifier(data[:identifier]),
@@ -60,10 +60,10 @@ module ApmBro
           cached_count: data[:cached_count],
           rendered_at: Time.now.utc.to_i
         }
-        
+
         add_view_event(view_info)
       end
-    rescue StandardError
+    rescue
       # Never raise from instrumentation install
     end
 
@@ -85,11 +85,11 @@ module ApmBro
 
     def self.safe_identifier(identifier)
       return "" unless identifier.is_a?(String)
-      
+
       # Extract meaningful parts of the file path
       # e.g., "/app/views/users/show.html.erb" -> "users/show.html.erb"
       identifier.split("/").last(3).join("/")
-    rescue StandardError
+    rescue
       identifier.to_s
     end
 
@@ -98,29 +98,29 @@ module ApmBro
       return {} if view_events.empty?
 
       total_duration = view_events.sum { |event| event[:duration_ms] }
-      
+
       # Group by view type
       by_type = view_events.group_by { |event| event[:type] }
-      
+
       # Find slowest views
       slowest_views = view_events.sort_by { |event| -event[:duration_ms] }.first(5)
-      
+
       # Find most frequently rendered views
       view_frequency = view_events.group_by { |event| event[:identifier] }
-                                 .transform_values(&:count)
-                                 .sort_by { |_, count| -count }
-                                 .first(5)
-      
+        .transform_values(&:count)
+        .sort_by { |_, count| -count }
+        .first(5)
+
       # Calculate cache hit rates for partials
       partials = view_events.select { |event| event[:type] == "partial" }
       cache_hits = partials.count { |event| event[:cache_key] }
       cache_hit_rate = partials.any? ? (cache_hits.to_f / partials.count * 100).round(2) : 0
-      
+
       # Collection rendering analysis
       collections = view_events.select { |event| event[:type] == "collection" }
       total_collection_items = collections.sum { |event| event[:count] || 0 }
       total_cached_items = collections.sum { |event| event[:cached_count] || 0 }
-      collection_cache_hit_rate = total_collection_items > 0 ? 
+      collection_cache_hit_rate = (total_collection_items > 0) ?
         (total_cached_items.to_f / total_collection_items * 100).round(2) : 0
 
       {
@@ -128,7 +128,7 @@ module ApmBro
         total_view_duration_ms: total_duration.round(2),
         average_view_duration_ms: (total_duration / view_events.count).round(2),
         by_type: by_type.transform_values(&:count),
-        slowest_views: slowest_views.map { |view| 
+        slowest_views: slowest_views.map { |view|
           {
             identifier: view[:identifier],
             duration_ms: view[:duration_ms],

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: apm_bro
 version: !ruby/object:Gem::Version
-  version: 0.1.13
+  version: 0.1.15
 platform: ruby
 authors:
 - Emanuel Comsa
@@ -40,11 +40,11 @@ files:
 - lib/apm_bro/subscriber.rb
 - lib/apm_bro/version.rb
 - lib/apm_bro/view_rendering_subscriber.rb
-homepage: https://github.com/rubydev/apm_bro
+homepage: https://www.deadbro.com
 licenses: []
 metadata:
-  homepage_uri: https://github.com/rubydev/apm_bro
-  source_code_uri: https://github.com/rubydev/apm_bro
+  homepage_uri: https://www.deadbro.com
+  source_code_uri: https://github.com/rubydevro/apm_bro
 rdoc_options: []
 require_paths:
 - lib