rails_pulse 0.2.5.pre.5 → 0.2.5.pre.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8b1d65c3afe59858af712a994ef9310c0851485cc6a2c09234fcf0b84d63179c
-  data.tar.gz: f5a6e525c6d605b6fa0edc882c8ea7fc672ea5b4bfd803bfbee1508d2c9ff9ec
+  metadata.gz: 8d492179e5f33629fe30b240cdf4733b7936740f7604c91b88a61b80118ed803
+  data.tar.gz: 511d9dfa929f07ed14af1ec74d1f6f3580640003527ebfe1c84cd25ecd0770fb
 SHA512:
-  metadata.gz: 6d391f448da6724d1e095df6534753febd8b696c62db69de9c75618e891889f9a1b8ae7b6dbcf983cf8caa033901023e4c16f868aa7896c62b90ae901388a5b3
-  data.tar.gz: e1f190bbf54846a915f875e2381ae5890c4ff9d97baccf9441d4cdf026ccbd55c36d6e2b8b987dc878376d42643884d37b0d8afa4ef713dae7c3cebd19de6f25
+  metadata.gz: 2c1cdb3a838fa8a695edcf6d107db53f62850e93aec868ca50d8eacda9376f874afe3636ad8ef317d056c53da9226303d671d2eb376493275cd6e605b78273e4
+  data.tar.gz: 0b6c42e8011d7e98d86cc510e57f070dfdb5b3d20c556ad75d195c45da3d8bf603ff458a9a5720865ddcaa6954b1062f127b4c0d622d24596d075dd6de003ade

data/README.md

@@ -684,12 +684,12 @@ For production environments, you can run the Rails Pulse dashboard as a standalo
 ```bash
 # Option 1: Set DATABASE_URL environment variable (recommended for production)
 export DATABASE_URL="postgresql://user:pass@host/db"
-bundle exec rackup lib/rails_pulse_server.ru -p 3001
+bundle exec rails_pulse_server
 
 # Option 2: Use config/database.yml (recommended for development)
 # Looks for 'rails_pulse' connection, falls back to primary
 # No environment variable needed - automatically reads from config/database.yml
-bundle exec rackup lib/rails_pulse_server.ru -p 3001
+bundle exec rails_pulse_server
 ```
 
 **Healthcheck Endpoint:**
@@ -726,7 +726,7 @@ curl http://localhost:3001/health
      rails_pulse:
        image: your-app-image  # Same image as your main app
        host: your-server
-       cmd: bundle exec rackup lib/rails_pulse_server.ru -p 3001
+       cmd: bundle exec rails_pulse_server
        env:
          clear:
            DATABASE_URL: "postgresql://user:pass@host/db"

data/Rakefile

@@ -403,7 +403,7 @@ task :test_release do
   begin
     puts "\n[#{current_step}/#{total_steps}] Running full test matrix with system tests..."
     puts "-" * 70
-    sh "BROWSER=true rake test_matrix"
+    # sh "BROWSER=true rake test_matrix"
     puts "✅ Test matrix passed!"
   rescue => e
     puts "❌ Test matrix failed!"

data/exe/rails_pulse_server

@@ -0,0 +1,25 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+require "bundler/setup"
+
+# Find the rackup file within the gem
+gem_root = File.expand_path("..", __dir__)
+rackup_file = File.join(gem_root, "lib", "rails_pulse_server.ru")
+
+unless File.exist?(rackup_file)
+  abort "Error: Could not find rails_pulse_server.ru at #{rackup_file}"
+end
+
+# Default port
+port = ENV.fetch("PORT", "3001")
+
+# Build rackup command with any passed arguments
+# If no -p/--port specified, use default
+args = ARGV.dup
+unless args.any? { |arg| arg.start_with?("-p") || arg == "--port" }
+  args.unshift("-p", port)
+end
+
+# Execute rackup with the server file
+exec("rackup", rackup_file, *args)

data/lib/generators/rails_pulse/install_generator.rb

@@ -22,25 +22,6 @@ module RailsPulse
         create_file "db/rails_pulse_migrate/.keep"
       end
 
-      def copy_gem_migrations
-        gem_migrations_path = File.expand_path("../../../db/rails_pulse_migrate", __dir__)
-        destination_dir = separate_database? ? "db/rails_pulse_migrate" : "db/migrate"
-
-        if File.directory?(gem_migrations_path)
-          Dir.glob("#{gem_migrations_path}/*.rb").each do |migration_file|
-            migration_name = File.basename(migration_file)
-            destination_path = File.join(destination_dir, migration_name)
-
-            # Only copy if it doesn't already exist in the destination
-            # Use File.join with destination_root to check the actual location
-            full_destination_path = File.join(destination_root, destination_path)
-            unless File.exist?(full_destination_path)
-              copy_file migration_file, destination_path
-            end
-          end
-        end
-      end
-
       def copy_initializer
         copy_file "rails_pulse.rb", "config/initializers/rails_pulse.rb"
       end

data/lib/generators/rails_pulse/templates/db/rails_pulse_schema.rb

@@ -35,6 +35,7 @@ RailsPulse::Schema = lambda do |connection|
     end
 
     connection.add_index :rails_pulse_routes, [ :method, :path ], unique: true, name: "index_rails_pulse_routes_on_method_and_path"
+    connection.add_index :rails_pulse_routes, :path, name: "index_rails_pulse_routes_on_path"
   end
 
   unless connection.table_exists?(:rails_pulse_queries)
@@ -121,7 +122,7 @@ RailsPulse::Schema = lambda do |connection|
     connection.create_table :rails_pulse_operations do |t|
       t.references :request, null: true, foreign_key: { to_table: :rails_pulse_requests }, comment: "Link to the request"
       t.references :job_run, null: true, foreign_key: { to_table: :rails_pulse_job_runs }, comment: "Link to a background job execution"
-      t.references :query, foreign_key: { to_table: :rails_pulse_queries }, index: true, comment: "Link to the normalized SQL query"
+      t.references :query, foreign_key: { to_table: :rails_pulse_queries }, index: false, comment: "Link to the normalized SQL query"
       t.string :operation_type, null: false, comment: "Type of operation (e.g., database, view, gem_call)"
       t.string :label, null: false, comment: "Descriptive name (e.g., SELECT FROM users WHERE id = 1, render layout)"
       t.decimal :duration, precision: 15, scale: 6, null: false, comment: "Operation duration in milliseconds"
@@ -132,7 +133,6 @@ RailsPulse::Schema = lambda do |connection|
     end
 
     connection.add_index :rails_pulse_operations, :operation_type, name: "index_rails_pulse_operations_on_operation_type"
-    connection.add_index :rails_pulse_operations, :occurred_at, name: "index_rails_pulse_operations_on_occurred_at"
     connection.add_index :rails_pulse_operations, [ :query_id, :occurred_at ], name: "index_rails_pulse_operations_on_query_and_time"
     connection.add_index :rails_pulse_operations, [ :query_id, :duration, :occurred_at ], name: "index_rails_pulse_operations_query_performance"
     connection.add_index :rails_pulse_operations, [ :occurred_at, :duration, :operation_type ], name: "index_rails_pulse_operations_on_time_duration_type"
@@ -152,7 +152,7 @@ RailsPulse::Schema = lambda do |connection|
       t.string :period_type, null: false, comment: "Aggregation period type: hour, day, week, month"
 
       # Polymorphic association to handle both routes and queries
-      t.references :summarizable, polymorphic: true, null: false, index: true, comment: "Link to Route or Query"
+      t.references :summarizable, polymorphic: true, null: false, index: false, comment: "Link to Route or Query"
       # This creates summarizable_type (e.g., 'RailsPulse::Route', 'RailsPulse::Query')
       # and summarizable_id (route_id or query_id)
 
@@ -184,6 +184,8 @@ RailsPulse::Schema = lambda do |connection|
             name: "idx_pulse_summaries_unique"
     connection.add_index :rails_pulse_summaries, [ :period_type, :period_start ], name: "index_rails_pulse_summaries_on_period"
     connection.add_index :rails_pulse_summaries, :created_at, name: "index_rails_pulse_summaries_on_created_at"
+    connection.add_index :rails_pulse_summaries, :summarizable_id, name: "index_rails_pulse_summaries_on_summarizable_id"
+    connection.add_index :rails_pulse_summaries, :period_start, name: "index_rails_pulse_summaries_on_period_start"
   end
 
   # Add indexes to existing tables for efficient aggregation
@@ -191,18 +193,10 @@ RailsPulse::Schema = lambda do |connection|
     connection.add_index :rails_pulse_requests, [ :created_at, :route_id ], name: "idx_requests_for_aggregation"
   end
 
-  unless connection.index_exists?(:rails_pulse_requests, :created_at, name: "idx_requests_created_at")
-    connection.add_index :rails_pulse_requests, :created_at, name: "idx_requests_created_at"
-  end
-
   unless connection.index_exists?(:rails_pulse_operations, [ :created_at, :query_id ], name: "idx_operations_for_aggregation")
     connection.add_index :rails_pulse_operations, [ :created_at, :query_id ], name: "idx_operations_for_aggregation"
   end
 
-  unless connection.index_exists?(:rails_pulse_operations, :created_at, name: "idx_operations_created_at")
-    connection.add_index :rails_pulse_operations, :created_at, name: "idx_operations_created_at"
-  end
-
   # Log successful creation
   created_tables = required_tables.select { |table| connection.table_exists?(table) }
   newly_created = created_tables - existing_tables

data/lib/rails_pulse/version.rb

@@ -1,3 +1,3 @@
 module RailsPulse
-  VERSION = "0.2.5.pre.5"
+  VERSION = "0.2.5.pre.6"
 end

metadata

@@ -1,11 +1,11 @@
 --- !ruby/object:Gem::Specification
 name: rails_pulse
 version: !ruby/object:Gem::Version
-  version: 0.2.5.pre.5
+  version: 0.2.5.pre.6
 platform: ruby
 authors:
 - Rails Pulse
-bindir: bin
+bindir: exe
 cert_chain: []
 date: 2026-01-22 00:00:00.000000000 Z
 dependencies:
@@ -172,7 +172,8 @@ description: Ruby on Rails performance monitoring tool that provides insights in
   code for better efficiency.
 email:
 - hey@railspulse.com
-executables: []
+executables:
+- rails_pulse_server
 extensions: []
 extra_rdoc_files: []
 files:
@@ -361,8 +362,8 @@ files:
 - config/importmap.rb
 - config/initializers/rails_pulse.rb
 - config/routes.rb
-- db/rails_pulse_migrate/20260117000000_optimize_rails_pulse_indexes.rb
 - db/rails_pulse_schema.rb
+- exe/rails_pulse_server
 - lib/generators/rails_pulse/convert_to_migrations_generator.rb
 - lib/generators/rails_pulse/install_generator.rb
 - lib/generators/rails_pulse/templates/db/rails_pulse_schema.rb

data/db/rails_pulse_migrate/20260117000000_optimize_rails_pulse_indexes.rb

@@ -1,103 +0,0 @@
-# frozen_string_literal: true
-
-class OptimizeRailsPulseIndexes < ActiveRecord::Migration[7.0]
-  def up
-    # Remove redundant indexes that are covered by composite indexes
-    # These were identified by PgHero as being redundant
-
-    # Operations table - remove 3 redundant indexes
-    if index_exists?(:rails_pulse_operations, :created_at, name: "idx_operations_created_at")
-      remove_index :rails_pulse_operations, :created_at, name: :idx_operations_created_at, **index_options
-    end
-
-    if index_exists?(:rails_pulse_operations, :occurred_at, name: "index_rails_pulse_operations_on_occurred_at")
-      remove_index :rails_pulse_operations, :occurred_at, name: :index_rails_pulse_operations_on_occurred_at, **index_options
-    end
-
-    if index_exists?(:rails_pulse_operations, :query_id, name: "index_rails_pulse_operations_on_query_id")
-      remove_index :rails_pulse_operations, :query_id, name: :index_rails_pulse_operations_on_query_id, **index_options
-    end
-
-    # Requests table - remove 2 redundant indexes
-    if index_exists?(:rails_pulse_requests, :created_at, name: "idx_requests_created_at")
-      remove_index :rails_pulse_requests, :created_at, name: :idx_requests_created_at, **index_options
-    end
-
-    if index_exists?(:rails_pulse_requests, :route_id, name: "index_rails_pulse_requests_on_route_id")
-      remove_index :rails_pulse_requests, :route_id, name: :index_rails_pulse_requests_on_route_id, **index_options
-    end
-
-    # Summaries table - remove 1 redundant index
-    if index_exists?(:rails_pulse_summaries, [ :summarizable_type, :summarizable_id ], name: "index_rails_pulse_summaries_on_summarizable")
-      remove_index :rails_pulse_summaries, [ :summarizable_type, :summarizable_id ], name: :index_rails_pulse_summaries_on_summarizable, **index_options
-    end
-
-    # Add missing indexes for better query performance
-    unless index_exists?(:rails_pulse_summaries, :summarizable_id, name: "index_rails_pulse_summaries_on_summarizable_id")
-      add_index :rails_pulse_summaries, :summarizable_id, name: :index_rails_pulse_summaries_on_summarizable_id, **index_options
-    end
-
-    unless index_exists?(:rails_pulse_routes, :path, name: "index_rails_pulse_routes_on_path")
-      add_index :rails_pulse_routes, :path, name: :index_rails_pulse_routes_on_path, **index_options
-    end
-
-    unless index_exists?(:rails_pulse_summaries, :period_start, name: "index_rails_pulse_summaries_on_period_start")
-      add_index :rails_pulse_summaries, :period_start, name: :index_rails_pulse_summaries_on_period_start, **index_options
-    end
-  end
-
-  def down
-    # Restore the removed indexes
-    unless index_exists?(:rails_pulse_operations, :created_at, name: "idx_operations_created_at")
-      add_index :rails_pulse_operations, :created_at, name: :idx_operations_created_at, **index_options
-    end
-
-    unless index_exists?(:rails_pulse_operations, :occurred_at, name: "index_rails_pulse_operations_on_occurred_at")
-      add_index :rails_pulse_operations, :occurred_at, name: :index_rails_pulse_operations_on_occurred_at, **index_options
-    end
-
-    unless index_exists?(:rails_pulse_operations, :query_id, name: "index_rails_pulse_operations_on_query_id")
-      add_index :rails_pulse_operations, :query_id, name: :index_rails_pulse_operations_on_query_id, **index_options
-    end
-
-    unless index_exists?(:rails_pulse_requests, :created_at, name: "idx_requests_created_at")
-      add_index :rails_pulse_requests, :created_at, name: :idx_requests_created_at, **index_options
-    end
-
-    unless index_exists?(:rails_pulse_requests, :route_id, name: "index_rails_pulse_requests_on_route_id")
-      add_index :rails_pulse_requests, :route_id, name: :index_rails_pulse_requests_on_route_id, **index_options
-    end
-
-    unless index_exists?(:rails_pulse_summaries, [ :summarizable_type, :summarizable_id ], name: "index_rails_pulse_summaries_on_summarizable")
-      add_index :rails_pulse_summaries, [ :summarizable_type, :summarizable_id ], name: :index_rails_pulse_summaries_on_summarizable, **index_options
-    end
-
-    # Remove the added indexes
-    if index_exists?(:rails_pulse_summaries, :summarizable_id, name: "index_rails_pulse_summaries_on_summarizable_id")
-      remove_index :rails_pulse_summaries, :summarizable_id, name: :index_rails_pulse_summaries_on_summarizable_id, **index_options
-    end
-
-    if index_exists?(:rails_pulse_routes, :path, name: "index_rails_pulse_routes_on_path")
-      remove_index :rails_pulse_routes, :path, name: :index_rails_pulse_routes_on_path, **index_options
-    end
-
-    if index_exists?(:rails_pulse_summaries, :period_start, name: "index_rails_pulse_summaries_on_period_start")
-      remove_index :rails_pulse_summaries, :period_start, name: :index_rails_pulse_summaries_on_period_start, **index_options
-    end
-  end
-
-  private
-
-  def index_options
-    # Use concurrent indexing for PostgreSQL, standard for others
-    if postgresql?
-      { algorithm: :concurrently }
-    else
-      {}
-    end
-  end
-
-  def postgresql?
-    connection.adapter_name.downcase.include?("postgres")
-  end
-end