time_range_uniqueness 1.0.2 → 1.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: dc737c44b6dbbf13c6781eef86f089832aaf04cda0a50c0d513f91e19536f600
-  data.tar.gz: 1be53b987bdc21f730b2ace9ea9aa9478eef92387ecdc6508a6bfad68f90d4ac
+  metadata.gz: c9c0a85c6dc8bcf5a87ec0bc0bc5ba71cd33e74431ae8b614ee3c317f53137e2
+  data.tar.gz: 9e783861728a33574463cd47748bacbbba82141ff642b60d29a5af419fcd8954
 SHA512:
-  metadata.gz: d6d32eb4c17f95d7d0815e692f09eed56d62aeb259a9c3b05e70fa2c78dfc59ac89fbd8127dc9bf33e29e8a5c5d2382ae21fdf07dd6ae906d9b36be3a878c4e5
-  data.tar.gz: 889b032899cbdc7cc3fc2cb73d05b593d438ecf112561dca2926a715c713d26cf92e5224cc85a9470b55c3404426484a5c66633ad8e716f7a13365e17c910bda
+  metadata.gz: 125a522c47ec9c1a985b7ebb573190e42561b1f96adc560ed0b5733aa54ea5cbab4026ff4c4eb5eee04331aaeb895276105b3719a0d940855c4c643cbf3c5aeb
+  data.tar.gz: c80909df16e9628c5fe0013847db4207abf0b906ea109a0df0878aba6be80f03fa51d1442604e6e54b0a9da14b08635b04f962dfa07f5a06cdf0425cd1e12c28

data/.rubocop.yml

@@ -13,6 +13,8 @@ Style/Documentation:
       - Exclude
   Exclude:
     - lib/time_range_uniqueness.rb
+    - lib/time_range_uniqueness/constraint_naming.rb
+    - lib/time_range_uniqueness/model_additions.rb
 
 RSpec/BeforeAfterAll:
   Enabled: false

data/CHANGELOG.md

@@ -1,5 +1,14 @@
 ## [Unreleased]
 
+## [1.0.3] - 2026-06-05
+
+- Translate the database-level exclusion-constraint violation into a validation error instead of
+  letting it surface as an unhandled `ActiveRecord::StatementInvalid`. This closes the gap left by
+  the model validation's check-then-insert: a conflict introduced by a concurrent write or by
+  `save(validate: false)` now adds the overlap error to the time range column (`save` returns
+  `false`, `save!` raises `ActiveRecord::RecordInvalid`). Unrelated database errors still propagate.
+- Run the test suite against ActiveRecord 7.1, 7.2, and 8.0 in CI to back the supported version range.
+
 ## [1.0.2] - 2026-05-31
 
 - Update the README to match the current behavior: document the Ruby >= 3.2 and

data/README.md

@@ -4,6 +4,7 @@
 
 It adds support for creating exclusion constraints on PostgreSQL `tstzrange` columns and validates the uniqueness of time ranges in models.
 
+[![Gem Version](https://badge.fury.io/rb/time_range_uniqueness.svg?icon=si%3Arubygems)](https://badge.fury.io/rb/time_range_uniqueness)
 [![rspec](https://github.com/j-boers-13/time_range_uniqueness/actions/workflows/ci.yml/badge.svg)](https://github.com/j-boers-13/time_range_uniqueness/actions/workflows/ci.yml)
 
 ## Features
@@ -12,6 +13,7 @@ It adds support for creating exclusion constraints on PostgreSQL `tstzrange` col
 - **Model Additions**: Adds validation to ensure time ranges do not overlap with existing records.
 - Supports optional scoping to ensure time ranges are unique within specified contexts (e.g., unique per event name).
 - Honors the time range's bound inclusivity (`..` vs `...`) so the model validation agrees with the database-level exclusion constraint.
+- Translates a database-level exclusion-constraint violation (e.g. from a concurrent write or `save(validate: false)`) into a validation error instead of an unhandled exception.
 - Treats a `NULL` scope value as never-conflicting, matching PostgreSQL's exclusion-constraint semantics (`NULL = NULL` is never true).
 - Works with models that use a composite primary key.
 - Keeps generated constraint names within PostgreSQL's 63-character identifier limit, and raises if a custom `:name` exceeds it.

data/lib/time_range_uniqueness/constraint_naming.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+require 'digest'
+
+module TimeRangeUniqueness
+  MAX_IDENTIFIER_LENGTH = 63
+
+  module ConstraintNaming
+    module_function
+
+    def default_constraint_name(table, scope_columns, time_range_column)
+      name = "exclude_#{table}_on_#{[scope_columns, time_range_column].flatten.join('_')}"
+      return name if name.length <= MAX_IDENTIFIER_LENGTH
+
+      digest = Digest::SHA256.hexdigest(name)[0, 10]
+      "#{name[0, MAX_IDENTIFIER_LENGTH - digest.length - 1]}_#{digest}"
+    end
+  end
+end

data/lib/time_range_uniqueness/migration_additions.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require 'digest'
-
 module TimeRangeUniqueness
   # This module provides methods for adding and managing time range uniqueness
   # constraints in ActiveRecord migrations.
@@ -34,7 +32,7 @@ module TimeRangeUniqueness
     COLUMN_TYPE = :tstzrange
 
     # PostgreSQL truncates identifiers to 63 bytes (NAMEDATALEN - 1).
-    MAX_IDENTIFIER_LENGTH = 63
+    MAX_IDENTIFIER_LENGTH = TimeRangeUniqueness::MAX_IDENTIFIER_LENGTH
 
     # Adds a time range column and an exclusion constraint to the specified table.
     #
@@ -104,13 +102,7 @@ module TimeRangeUniqueness
     # @param time_range_column [Symbol] The time range column name.
     # @return [String] The generated constraint name.
     def generate_constraint_name(table, scope_columns, time_range_column)
-      name = "exclude_#{table}_on_#{[scope_columns, time_range_column].flatten.join('_')}"
-      return name if name.length <= MAX_IDENTIFIER_LENGTH
-
-      # Keep the name deterministic and within PostgreSQL's limit so the up and down
-      # migrations refer to the same constraint. A digest avoids collisions after truncation.
-      digest = Digest::SHA256.hexdigest(name)[0, 10]
-      "#{name[0, MAX_IDENTIFIER_LENGTH - digest.length - 1]}_#{digest}"
+      TimeRangeUniqueness::ConstraintNaming.default_constraint_name(table, scope_columns, time_range_column)
     end
 
     # Ensures the btree_gist extension is enabled.

data/lib/time_range_uniqueness/model_additions.rb

@@ -54,12 +54,76 @@ module TimeRangeUniqueness
       scope_columns = Array(options[:scope])
       message = options[:message] || 'overlaps with an existing record'
 
+      TimeRangeUniqueness::ModelAdditions.register_constraint(self, time_range_column, scope_columns, message,
+                                                              options[:name])
+
       validate do
         overlapping = TimeRangeUniqueness::ModelAdditions.overlapping?(self, time_range_column, scope_columns)
         errors.add(time_range_column, message) if overlapping
       end
     end
 
+    def self.register_constraint(model, time_range_column, scope_columns, message, name)
+      unless model.respond_to?(:time_range_uniqueness_constraints)
+        model.class_attribute :time_range_uniqueness_constraints, instance_accessor: false, default: []
+        model.prepend(ViolationHandling)
+      end
+
+      model.time_range_uniqueness_constraints += [
+        { column: time_range_column, scope_columns: scope_columns, message: message, name: name }
+      ]
+    end
+
+    def self.violation_constraint_name(error)
+      cause = error.cause
+      return nil unless defined?(PG::ExclusionViolation) && cause.is_a?(PG::ExclusionViolation)
+
+      result = cause.respond_to?(:result) ? cause.result : nil
+      result&.error_field(PG::Result::PG_DIAG_CONSTRAINT_NAME)
+    end
+
+    def self.constraint_name_for(model, config)
+      return config[:name].to_s if config[:name]
+
+      TimeRangeUniqueness::ConstraintNaming.default_constraint_name(
+        model.table_name, config[:scope_columns], config[:column]
+      )
+    end
+
+    def self.matched_constraint(record, error)
+      name = violation_constraint_name(error)
+      return unless name
+
+      record.class.time_range_uniqueness_constraints.find do |config|
+        constraint_name_for(record.class, config) == name
+      end
+    end
+
+    module ViolationHandling
+      def save(...)
+        super
+      rescue ActiveRecord::StatementInvalid => e
+        apply_time_range_uniqueness_error(e)
+        false
+      end
+
+      def save!(...)
+        super
+      rescue ActiveRecord::StatementInvalid => e
+        apply_time_range_uniqueness_error(e)
+        raise ActiveRecord::RecordInvalid, self
+      end
+
+      private
+
+      def apply_time_range_uniqueness_error(error)
+        config = TimeRangeUniqueness::ModelAdditions.matched_constraint(self, error)
+        raise error unless config
+
+        errors.add(config[:column], config[:message])
+      end
+    end
+
     # Checks whether the record's time range overlaps any other record, optionally scoped.
     #
     # @param record [ActiveRecord::Base] The record being validated.

data/lib/time_range_uniqueness/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module TimeRangeUniqueness
-  VERSION = '1.0.2'
+  VERSION = '1.0.3'
 end

data/lib/time_range_uniqueness.rb

@@ -2,6 +2,7 @@
 
 require_relative 'time_range_uniqueness/version'
 require 'active_record'
+require_relative 'time_range_uniqueness/constraint_naming'
 require_relative 'time_range_uniqueness/migration_additions'
 require_relative 'time_range_uniqueness/model_additions'
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: time_range_uniqueness
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.0.3
 platform: ruby
 authors:
 - j-boers-13
@@ -62,6 +62,7 @@ files:
 - README.md
 - Rakefile
 - lib/time_range_uniqueness.rb
+- lib/time_range_uniqueness/constraint_naming.rb
 - lib/time_range_uniqueness/migration_additions.rb
 - lib/time_range_uniqueness/model_additions.rb
 - lib/time_range_uniqueness/version.rb