memhealth 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8c0895575f5adc6767b228212ace922f4eb66eb565771c05bde602b4c4b48971
-  data.tar.gz: b44315dc375c9fa4108a5fb889f76dd1a0520883f9c8952a675c8fd840bc981a
+  metadata.gz: 69e8587a6c8a55a3a4e5b50ed6b433c4557f067e867cc9c30520bac3f184a48c
+  data.tar.gz: d7c228db15571b67db8a59adc334aa5611717a61939662bc5abde5464cdfb3e4
 SHA512:
-  metadata.gz: f5e263596880a31d83003ebe56700a6ed94d8bb6eafa60222b7424fbb1f8632b9a534b9952d6c892d41637a43f66df0a4d673d6e7f62c4f35c0279382a8486cc
-  data.tar.gz: 80ad19cf0d82d6a4f543701a82736c6837d265b0daa57ef9a709df99c550452b2fb8e77f131add2e86e2871c0978729d06e84035539b42cc1eb5c40ff6e5f2f5
+  metadata.gz: a82811c70c3ee43ed58dce8f0e8a169d9b7008f2a72303c67e0b654b5b7b653e25c05be27542fc9c168846516299566a763851ac3b98c8a0051908cf813e7fda
+  data.tar.gz: e7f878a2dfffe6c9e4f2ab276c5447889af056be98380e54e94a8783ff099fa8b51a94c63a32986add73eaa3b9417bd6a7c606bbf3d21c967f8fc9ddec641f56

data/README.md

@@ -2,7 +2,45 @@
 
 A Rails engine for monitoring memory health, detecting growth patterns (leaks & bloats) and memory swap operations. Helps you identify requests that consume high amounts of RAM and is compatible with Heroku.
 
-## Features
+## Installation
+
+Add this line to your application's Gemfile:
+
+```ruby
+gem "memhealth"
+```
+
+Use initializer `/config/memhealth.rb` to configure Redis connection:
+
+```ruby
+MemHealth.configure do |config|
+  config.redis_url = ENV.fetch("REDIS_URL", "redis://localhost:6379/0")
+end
+```
+
+Enable memory tracking via ENV variable:
+
+```bash
+MEM_HEALTH_ENABLED=true
+```
+
+### Dashboard setup
+
+Mount the engine in your routes within an authenticated section:
+
+```ruby
+# config/routes.rb
+Rails.application.routes.draw do
+  # ... other routes ...
+
+  # Mount within authenticated admin section
+  authenticate :admin_user do
+    mount MemHealth::Engine, at: "/admin/memhealth"
+  end
+end
+```
+
+# Features
 
 - Real-time memory usage monitoring
 - Track highest memory consuming requests
@@ -23,25 +61,7 @@ If you're getting **R14 - Memory quota exceeded** errors, it means your applicat
 
 MemHealth helps you identify which specific requests are consuming excessive memory, allowing you to pinpoint and fix the root cause of these performance issues.
 
-## Installation
-
-Add this line to your application's Gemfile:
-
-```ruby
-gem "memhealth", git: "https://github.com/topkeyhq/memhealth"
-```
-
-And then execute:
-
-    $ bundle install
-
-## Configuration
-
-```ruby
-MemHealth.configure do |config|
-  config.redis_url = ENV.fetch(ENV.fetch("REDIS_URL", "redis://localhost:6379/0"))
-end
-```
+## ENVIRONMENT VARIABLES
 
 Configure Memhealth using environment variables:
 
@@ -56,28 +76,6 @@ Configure Memhealth using environment variables:
 
 The gem will read the Redis connection URL from the environment variable specified in `MEM_HEALTH_REDIS_KEY`, falling back to `REDIS_URL` if not specified.
 
-## Usage
-
-Mount the engine in your routes within an authenticated section:
-
-```ruby
-# config/routes.rb
-Rails.application.routes.draw do
-  # ... other routes ...
-
-  # Mount within authenticated admin section
-  authenticate :admin_user do
-    mount MemHealth::Engine, at: "/admin/memhealth"
-  end
-end
-```
-
-Enable memory tracking by setting:
-
-```bash
-MEM_HEALTH_ENABLED=true
-```
-
 ### ActiveAdmin Integration
 
 To add Memhealth to your ActiveAdmin Operations menu, add this to your ActiveAdmin initializer:

data/app/views/layouts/memhealth/application.html.erb

@@ -129,6 +129,17 @@
       
       .truncate { overflow: hidden; text-overflow: ellipsis; white-space: nowrap; }
       .whitespace-nowrap { white-space: nowrap; }
+
+      /* URL truncation in table */
+      td.url-cell {
+        max-width: 500px;
+      }
+
+      td.url-cell > div {
+        overflow: hidden;
+        text-overflow: ellipsis;
+        white-space: nowrap;
+      }
       .text-center { text-align: center; }
       .text-left { text-align: left; }
       .uppercase { text-transform: uppercase; }
@@ -158,7 +169,7 @@
       }
       
       /* Container styles */
-      .container { max-width: 1200px; margin: 0 auto; }
+      .container { max-width: 1600px; margin: 0 auto; }
       
       /* Header styles */
       .header { 

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

@@ -15,41 +15,51 @@
           <!-- URL Section -->
           <div class="mb-4">
             <dt class="text-sm font-medium text-gray-500 mb-1">URL</dt>
-            <dd class="text-sm text-gray-900 font-mono bg-gray-100 px-2 py-1 rounded truncate ml-0">
-              <%= @max_memory_url["url"] %>
+            <dd class="text-sm text-gray-900 font-mono bg-gray-100 px-2 py-1 rounded ml-0 flex items-center gap-2" title="<%= @max_memory_url["url"] %>">
+              <% if @max_memory_url["request_method"] %>
+                <span class="inline-flex px-2 py-1 text-xs font-semibold rounded flex-shrink-0 <%= @max_memory_url["request_method"] == 'GET' ? 'bg-blue-100 text-blue-800' : 'bg-green-100 text-green-800' %>">
+                  <%= @max_memory_url["request_method"] %>
+                </span>
+              <% end %>
+              <span class="truncate"><%= @max_memory_url["url"] %></span>
             </dd>
           </div>
 
-          <!-- Four column layout using flex -->
+          <!-- Single row with all metadata -->
           <div class="flex gap-4">
-            <% if @max_memory_url["ram_before"] && @max_memory_url["ram_after"] %>
-              <div class="flex-1 text-left">
-                <dt class="text-sm font-medium text-gray-500 mb-1">RAM Usage</dt>
-                <dd class="text-sm text-gray-600 ml-0">
-                  <%= @max_memory_url["ram_before"] %> MB → <%= @max_memory_url["ram_after"] %> MB
+            <div class="flex-1 text-left">
+              <div class="bg-red-100 border border-red-300 rounded-md px-4 py-3">
+                <dt class="text-sm font-medium text-red-700 mb-1">Memory Diff</dt>
+                <dd class="text-lg font-bold text-red-800 ml-0">
+                  <%= @max_memory_url["memory_diff"] %> MB
                 </dd>
               </div>
-            <% end %>
+            </div>
 
             <div class="flex-1 text-left">
-              <dt class="text-sm font-medium text-gray-500 mb-1">Recorded At</dt>
-              <dd class="text-sm text-gray-600 ml-0"><%= Time.parse(@max_memory_url["recorded_at"]).utc.strftime("%Y-%m-%d %H:%M") %></dd>
+              <dt class="text-sm font-medium text-gray-500 mb-1">RAM Usage</dt>
+              <dd class="text-sm text-gray-600 ml-0">
+                <%= @max_memory_url["ram_before"] %> MB → <%= @max_memory_url["ram_after"] %> MB
+              </dd>
             </div>
 
-            <% if @max_memory_url["account_id"].present? %>
-              <div class="flex-1 text-left">
-                <dt class="text-sm font-medium text-gray-500 mb-1">Account ID</dt>
-                <dd class="text-sm text-gray-600 ml-0"><%= @max_memory_url["account_id"] %></dd>
-              </div>
-            <% end %>
+            <div class="flex-1 text-left">
+              <dt class="text-sm font-medium text-gray-500 mb-1">Execution Time</dt>
+              <dd class="text-sm text-gray-600 ml-0">
+                <%= @max_memory_url["execution_time"] ? "#{@max_memory_url["execution_time"]} s" : "-" %>
+              </dd>
+            </div>
 
             <div class="flex-1 text-left">
-              <div class="bg-red-100 border border-red-300 rounded-md px-4 py-3">
-                <dt class="text-sm font-medium text-red-700 mb-1">Memory Diff</dt>
-                <dd class="text-lg font-bold text-red-800 ml-0">
-                  <%= @max_memory_url["memory_diff"] %> MB
-                </dd>
-              </div>
+              <dt class="text-sm font-medium text-gray-500 mb-1">Info</dt>
+              <dd class="text-sm text-gray-600 ml-0">
+                <% info_parts = [] %>
+                <% info_parts << @max_memory_url["dyno"] if @max_memory_url["dyno"] %>
+                <% info_parts << "T#{@max_memory_url["puma_thread_index"]}" if @max_memory_url["puma_thread_index"] %>
+                <% info_parts << "##{@max_memory_url["account_id"]}" if @max_memory_url["account_id"].present? %>
+                <% info_parts << Time.parse(@max_memory_url["recorded_at"]).utc.strftime("%m/%d %H:%M") %>
+                <%= info_parts.join(" / ") %>
+              </dd>
             </div>
           </div>
         </div>
@@ -100,7 +110,9 @@
               <th class="px-6 py-3 text-left text-xs font-medium text-gray-500 uppercase tracking-wider">Memory Diff</th>
               <th class="px-6 py-3 text-left text-xs font-medium text-gray-500 uppercase tracking-wider">RAM Before</th>
               <th class="px-6 py-3 text-left text-xs font-medium text-gray-500 uppercase tracking-wider">RAM After</th>
+              <th class="px-6 py-3 text-left text-xs font-medium text-gray-500 uppercase tracking-wider">Exec Time</th>
               <th class="px-6 py-3 text-left text-xs font-medium text-gray-500 uppercase tracking-wider">URL</th>
+              <th class="px-6 py-3 text-left text-xs font-medium text-gray-500 uppercase tracking-wider">Dyno/Thread</th>
               <th class="px-6 py-3 text-left text-xs font-medium text-gray-500 uppercase tracking-wider">Account ID</th>
               <th class="px-6 py-3 text-left text-xs font-medium text-gray-500 uppercase tracking-wider">Recorded At</th>
             </tr>
@@ -129,11 +141,25 @@
                 <td class="px-6 py-4 whitespace-nowrap text-sm text-gray-500">
                   <%= url_data["ram_after"] ? "#{url_data["ram_after"]} MB" : "/" %>
                 </td>
-                <td class="px-6 py-4 text-sm text-gray-900">
-                  <div class="max-w-xs truncate">
-                    <code class="bg-gray-100 px-2 py-1 rounded text-xs"><%= url_data["url"] %></code>
+                <td class="px-6 py-4 whitespace-nowrap text-sm text-gray-500">
+                  <%= url_data["execution_time"] ? "#{url_data["execution_time"]} s" : "/" %>
+                </td>
+                <td class="px-6 py-4 text-gray-700 url-cell">
+                  <div title="<%= url_data["url"] %>" style="font-size: 0.75rem; line-height: 1rem;">
+                    <% if url_data["request_method"] %>
+                      <span class="inline-flex px-1 py-0.5 font-semibold rounded <%= url_data["request_method"] == 'GET' ? 'bg-blue-100 text-blue-800' : 'bg-green-100 text-green-800' %>" style="font-size: 0.65rem;">
+                        <%= url_data["request_method"] %>
+                      </span>
+                    <% end %>
+                    <code class="bg-gray-100 px-1 py-0.5 rounded"><%= url_data["url"] %></code>
                   </div>
                 </td>
+                <td class="px-6 py-4 whitespace-nowrap text-sm text-gray-500">
+                  <% dyno_thread = [] %>
+                  <% dyno_thread << url_data["dyno"] if url_data["dyno"] %>
+                  <% dyno_thread << "T#{url_data["puma_thread_index"]}" if url_data["puma_thread_index"] %>
+                  <%= dyno_thread.any? ? dyno_thread.join("/") : "/" %>
+                </td>
                 <td class="px-6 py-4 whitespace-nowrap text-sm text-gray-500">
                   <%= url_data["account_id"].present? ? url_data["account_id"] : "/" %>
                 </td>

data/lib/mem_health/middleware.rb

@@ -1,4 +1,4 @@
-require "get_process_mem"
+require 'get_process_mem'
 
 module MemHealth
   class Middleware
@@ -16,19 +16,38 @@ module MemHealth
     def call(env)
       @@request_count += 1
 
+      # Track execution time
+      start_time = Process.clock_gettime(Process::CLOCK_MONOTONIC)
+
       before = GetProcessMem.new.mb
       status, headers, response = @app.call(env)
       after = GetProcessMem.new.mb
 
+      # Calculate execution time in seconds
+      end_time = Process.clock_gettime(Process::CLOCK_MONOTONIC)
+      execution_time = (end_time - start_time).round(2)
+
       memory_diff = (after - before).round(2)
-      request_url = build_request_url(env)
+      request = Rack::Request.new(env)
+      request_url = request.fullpath
       account_info = extract_account_info(env)
 
+      # Collect additional metadata
+      metadata = {
+        puma_thread_index: Thread.current[:puma_thread_index],
+        dyno: ENV['DYNO'],
+        execution_time: execution_time,
+        request_method: request.request_method
+      }
+
       # Skip the first few requests as they have large memory jumps due to class loading
       if @@request_count > config.skip_requests
         redis.incr(redis_tracked_requests_key)
         should_track = memory_diff > config.memory_threshold_mb && before.round(2) > config.ram_before_threshold_mb
-        track_memory_usage(memory_diff, request_url, before.round(2), after.round(2), account_info) if should_track
+        if should_track
+          track_memory_usage(memory_diff, request_url, before.round(2), after.round(2),
+                             account_info.merge(metadata))
+        end
       else
         @@skipped_requests_count += 1
       end
@@ -80,7 +99,7 @@ module MemHealth
       self.class.redis_tracked_requests_key
     end
 
-    def track_memory_usage(memory_diff, request_url, ram_before, ram_after, account_info = {})
+    def track_memory_usage(memory_diff, request_url, ram_before, ram_after, request_metadata = {})
       # Update max memory diff seen so far
       current_max = redis.get(redis_max_diff_key)&.to_f || 0.0
       if memory_diff > current_max
@@ -94,7 +113,7 @@ module MemHealth
           ram_after: ram_after,
           timestamp: Time.current.to_i,
           recorded_at: Time.current.iso8601
-        }.merge(account_info)
+        }.merge(request_metadata)
         redis.set(redis_max_diff_url_key, max_url_data.to_json)
       end
 
@@ -107,25 +126,17 @@ module MemHealth
         ram_after: ram_after,
         timestamp: timestamp,
         recorded_at: Time.current.iso8601
-      }.merge(account_info)
+      }.merge(request_metadata)
 
       # Add URL to sorted set (score = memory_diff for DESC ordering)
       redis.zadd(redis_high_usage_urls_key, memory_diff, url_data.to_json)
 
       # Keep only top N URLs by removing lowest scores
       current_count = redis.zcard(redis_high_usage_urls_key)
-      if current_count > config.max_stored_urls
-        # Remove the lowest scoring URLs (keep top max_stored_urls)
-        redis.zremrangebyrank(redis_high_usage_urls_key, 0, current_count - config.max_stored_urls - 1)
-      end
-    end
-
-    def build_request_url(env)
-      request = Rack::Request.new(env)
-      url = "#{request.request_method} #{request.fullpath}"
+      return unless current_count > config.max_stored_urls
 
-      # Truncate very long URLs
-      (url.length > 600) ? "#{url[0..650]}..." : url
+      # Remove the lowest scoring URLs (keep top max_stored_urls)
+      redis.zremrangebyrank(redis_high_usage_urls_key, 0, current_count - config.max_stored_urls - 1)
     end
 
     def extract_account_info(env)
@@ -136,13 +147,13 @@ module MemHealth
         if defined?(ActsAsTenant) && ActsAsTenant.current_tenant
           account = ActsAsTenant.current_tenant
           account_info[:account_id] = account.id
-        elsif env["warden"]&.user&.respond_to?(:account)
+        elsif env['warden']&.user&.respond_to?(:account)
           # Try to get from authenticated user
-          account = env["warden"].user.account
+          account = env['warden'].user.account
           account_info[:account_id] = account.id
-        elsif env["HTTP_X_ACCOUNT_ID"]
+        elsif env['HTTP_X_ACCOUNT_ID']
           # Fallback to header if available
-          account_info[:account_id] = env["HTTP_X_ACCOUNT_ID"]
+          account_info[:account_id] = env['HTTP_X_ACCOUNT_ID']
         end
       rescue StandardError => _e
         # Silently fail if account extraction fails

data/lib/mem_health/version.rb

@@ -1,3 +1,3 @@
 module MemHealth
-  VERSION = "0.1.0"
+  VERSION = '0.1.1'
 end

metadata

@@ -1,13 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: memhealth
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Klemen Nagode
+autorequire:
 bindir: bin
 cert_chain: []
-date: 1980-01-02 00:00:00.000000000 Z
+date: 2025-09-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -91,6 +92,7 @@ licenses:
 metadata:
   homepage_uri: https://github.com/topkeyhq/memhealth
   source_code_uri: https://github.com/topkeyhq/memhealth
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -105,7 +107,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.7
+rubygems_version: 3.5.16
+signing_key:
 specification_version: 4
 summary: Rails memory health monitoring and tracking
 test_files: []