rails_pulse 0.3.3.pre.2 → 0.3.3.pre.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 70f6de9d79a6321df43145819c47aae97aeb6cf3c184c76b66aa8707eea1f80c
-  data.tar.gz: a7c524c7a77b5f39c44eaa20a820c221d1f9e32c30baeda10d6dae78cd720ded
+  metadata.gz: 80612b2461699451e01b4efa5ef5143eeb6c24d042eaba019e82d1aed02145d2
+  data.tar.gz: 119e0dd5913f3f6930bd1fe64c4234d9196c58f2388d818a26db7c85519f350d
 SHA512:
-  metadata.gz: 5e4486641f354374863b138553539cb9cebbcd92ee7190910af0fe5c339c459839a51b14e07cb114d90228542bc9e16a074d1404a5325ee48aa92e1557d7ea9f
-  data.tar.gz: be64b7d1c67259124a69c20bab07fc9697f90f76ca9096f62d01a8a52663e00032833e392645c76ea4ccf0be3fa70103ab170648adcca0cf24cbd6b9c906038b
+  metadata.gz: 60f5e66f20e1cf0cdbeee2a1f543509a227121c758248236adc8bc1d49690f968b64335842dba4192ed0c4b960fb9cdb783b003946e60278731282d520975b5b
+  data.tar.gz: 9a25108617038b5ac69646fceee12a14fb9f18fc278a83f104a020ee245c74e7450111d31feb37d2357fd621e010c021ee1f77fbd74c735c24ef5ac60981faa6

data/app/models/rails_pulse/job.rb

@@ -60,6 +60,8 @@ module RailsPulse
 
         update!(updates)
       end
+    rescue ActiveRecord::RecordNotFound
+      RailsPulse.logger.warn "RailsPulse::Job #{id} was deleted concurrently; skipping cache update for run #{run.id}"
     end
 
     def failure_rate

data/app/models/rails_pulse/route.rb

@@ -13,7 +13,21 @@ module RailsPulse
     validates :path, presence: true
 
     def self.by_method_and_path(method, path)
-      find_or_create_by!(method: method, path: path)
+      # Fast path: route already exists (the common case for recurring requests).
+      route = find_by(method: method, path: path)
+      return route if route
+
+      # Use INSERT ... ON CONFLICT DO NOTHING rather than create_or_find_by so that
+      # concurrent inserts for the same route are silently skipped at the database level.
+      # create_or_find_by rescues RecordNotUnique at the Ruby level, but PostgreSQL still
+      # logs a constraint violation ERROR before raising — causing noisy logs.
+      # unique_by is omitted for cross-database compatibility (MySQL rejects it); there is
+      # only one unique constraint on this table so DO NOTHING resolves unambiguously.
+      # Tags must be set explicitly here because insert bypasses before_save callbacks.
+      insert(
+        { method: method, path: path, tags: "[]", created_at: Time.current, updated_at: Time.current }
+      )
+      find_by!(method: method, path: path)
     end
 
     def self.ransackable_attributes(auth_object = nil)

data/lib/rails_pulse/tracker.rb

@@ -26,8 +26,7 @@ module RailsPulse
         RailsPulse::ApplicationRecord.connection_pool.with_connection do
           RequestStore.store[:skip_recording_rails_pulse_activity] = true
 
-          route = RailsPulse::Route.find_by(method: data[:method], path: data[:path]) ||
-                  RailsPulse::Route.create_or_find_by(method: data[:method], path: data[:path])
+          route = RailsPulse::Route.by_method_and_path(data[:method], data[:path])
 
           request = RailsPulse::Request.create!(
             route: route,

data/lib/rails_pulse/version.rb

@@ -1,3 +1,3 @@
 module RailsPulse
-  VERSION = "0.3.3.pre.2"
+  VERSION = "0.3.3.pre.3"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails_pulse
 version: !ruby/object:Gem::Version
-  version: 0.3.3.pre.2
+  version: 0.3.3.pre.3
 platform: ruby
 authors:
 - Scott Harvey
@@ -520,7 +520,7 @@ metadata:
   changelog_uri: https://github.com/railspulse/rails_pulse/blob/main/CHANGELOG.md
   documentation_uri: https://railspulse.com/documentation/installation
 post_install_message: |
-  Rails Pulse 0.3.3.pre.2 installed. If upgrading, run:
+  Rails Pulse 0.3.3.pre.3 installed. If upgrading, run:
     rails generate rails_pulse:upgrade && rails db:migrate
 rdoc_options: []
 require_paths: