active_record_compose 0.8.1 → 0.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9b1f795843fd29ef6dcd9cf17e897938ea97fc8f32db35f0b444b36455e619a3
-  data.tar.gz: 584e44a2c49c6861ad232f0c52c75b6d73abb550f39165fabe14690a71575c81
+  metadata.gz: dba7d13bf8f2dc9ef449ead00c1a7ae3fff83e216f07b5d11606659a3c1f4650
+  data.tar.gz: 883a9480ff3e6bd4c99246c12200a7b5a6e44103c0d1c967ce97c4536aa478b1
 SHA512:
-  metadata.gz: 04bc96e1479134c02ad09ae5d014ad5ad93c6d6ae6914a39724d42152f691120ccfaa76c681e3790bb1b436b49cc39d57eb8bfe7f6fc73d064818e22738ce1c9
-  data.tar.gz: e757a94164248e188c46413a706ec2418eb99c97f28bc5631305620c064b8fd7b04a6621d4809e405f95ac4d2228c22509700169f0049b581a1998ceba364acd
+  metadata.gz: 905bf7969733f738e00726f0dc7ed1442bce1f598485a72a0478fd67069c556b5d8299fdc5485636b1194e095649df550ba9f3c01ffb42c107a8a69afcde85ae
+  data.tar.gz: 22f44a63d515d31682f22641e112c3bf720c5b3ff29c8fb0a30ceb777b23847a74f053eb3591211c35bfe2e2f96de345523236d08b30365059fc3a7421437bf4

data/CHANGELOG.md

@@ -1,9 +1,11 @@
 ## [Unreleased]
 
-## [0.8.1] - 2025-06-21
+## [0.9.0] - 2025-03-16
 
-- Add migration guide message.
-- Add upper version.
+- removed `persisted_flag_callback_control` support.
+- Omitted `:private` option from `delegate_attribute` because, assuming the
+  behavior and use cases of `ActiveModel::Attributes.attribute`, making it private is unnecessary.
+- Added the URL for the sample application to the README
 
 ## [0.8.0] - 2025-02-22
 

data/README.md

@@ -286,18 +286,11 @@ end
 
 ### Callback ordering by `#persisted?`
 
-The behavior of `(before|after|around)_create` and `(before|after|around)_update` hooks depends on
-the state of the `persisted_flag_callback_control` setting.
-default value is `true`, behavior when set to `false` will be removed in the next release.
-
-When `persisted_flag_callback_control` is set to `true`, it behaves almost like callback control in ActiveRecord.
-Depending on the evaluation result of `#persisted?`,
+The behavior of `(before|after|around)_create` and `(before|after|around)_update` hooks depending on the evaluation result of `#persisted?`,
 either the create-related callbacks or the update-related callbacks will be triggered.
 
 ```ruby
 class ComposedModel < ActiveRecordCompose::Model
-  self.persisted_flag_callback_control = true # In the future, true will be the default and false will no longer be supported.
-
   # ...
 
   before_save { puts 'before_save called!' }
@@ -345,48 +338,10 @@ model.save # or `model.update` (the same callbacks will be triggered in all case
 # after_save called!
 ```
 
-When `persisted_flag_callback_control` is set to false,
-the execution of `#create`, `#update`, or `#save` determines which callbacks will be triggered.
-This behavior will no longer be supported in the next release.
-
-```ruby
-class ComposedModel < ActiveRecordCompose::Model
-  self.persisted_flag_callback_control = false # Currently defaults to false, but will no longer be supported in the future.
-
-  # ...
-
-  before_save { puts 'before_save called!' }
-  before_create { puts 'before_create called!' }
-  before_update { puts 'before_update called!' }
-  after_save { puts 'after_save called!' }
-  after_create { puts 'after_create called!' }
-  after_update { puts 'after_update called!' }
-end
-```
-
-```ruby
-model = ComposedModel.new
-
-model.save
-# before_save called!
-# after_save called!
-
-model.create
-# before_save called!
-# before_create called!
-# after_create called!
-# after_save called!
-
-model.update
-# before_save called!
-# before_update called!
-# after_update called!
-# after_save called!
-```
-
 ## Links
 
 - [Smart way to update multiple models simultaneously in Rails](https://dev.to/hamajyotan/smart-way-to-update-multiple-models-simultaneously-in-rails-51b6)
+- [Sample application as an example](https://github.com/hamajyotan/active_record_compose-example)
 
 ## Development
 

data/lib/active_record_compose/delegate_attribute.rb

@@ -45,7 +45,7 @@ module ActiveRecordCompose
     module ClassMethods
       # Defines the reader and writer for the specified attribute.
       #
-      def delegate_attribute(*attributes, to:, allow_nil: nil, private: nil)
+      def delegate_attribute(*attributes, to:, allow_nil: nil)
         delegates = attributes.flat_map do |attribute|
           reader = attribute.to_s
           writer = "#{attribute}="
@@ -53,7 +53,7 @@ module ActiveRecordCompose
           [reader, writer]
         end
 
-        delegate(*delegates, to:, allow_nil:, private:)
+        delegate(*delegates, to:, allow_nil:)
         self.delegated_attributes = delegated_attributes.to_a + attributes.map { _1.to_s }
       end
     end

data/lib/active_record_compose/model.rb

@@ -15,46 +15,6 @@ module ActiveRecordCompose
     include ActiveRecordCompose::DelegateAttribute
     include ActiveRecordCompose::TransactionSupport
 
-    # This flag controls the callback sequence for models.
-    # The current default value is `true`, behavior when set to `false` will be removed in the next release.
-    #
-    # When `persisted_flag_callback_control` is set to `true`,
-    # the occurrence of callbacks depends on the evaluation result of `#persisted?`.
-    # Additionally, the definition of `#persisted?` itself can be appropriately overridden in subclasses.
-    #
-    # if `#persisted?` returns `false`:
-    # * before_save
-    # * before_create
-    # * after_create
-    # * after_save
-    #
-    # if `#persisted?` returns `true`:
-    # * before_save
-    # * before_update
-    # * after_update
-    # * after_save
-    #
-    # On the other hand, when `persisted_flag_callback_control` is set to `false`,
-    # the invoked methods during saving operations vary depending on the method used.
-    #
-    # when performing `#save` or `#save!`:
-    # * before_save
-    # * after_save
-    #
-    # when performing `#update` or `#update!`:
-    # * before_save
-    # * before_update
-    # * after_update
-    # * after_save
-    #
-    # when performing `#create` or `#create!`:
-    # * before_save
-    # * before_create
-    # * after_create
-    # * after_save
-    #
-    class_attribute :persisted_flag_callback_control, instance_accessor: false, default: true
-
     define_model_callbacks :save
     define_model_callbacks :create
     define_model_callbacks :update
@@ -70,24 +30,16 @@ module ActiveRecordCompose
     #
     # The save is performed within a single transaction.
     #
+    # Options like `:validate` and `:context` are not accepted as arguments.
+    # The need for such values indicates that operations from multiple contexts are being handled.
+    # However, if the contexts are different, it is recommended to separate them into different model definitions.
+    #
     # @return [Boolean] returns true on success, false on failure.
     def save
       return false if invalid?
 
       with_transaction_returning_status do
-        if self.class.persisted_flag_callback_control
-          with_callbacks { save_models(bang: false) }
-        else
-          # steep:ignore:start
-          deprecator.warn(
-            'The behavior with `persisted_flag_callback_control` set to `false` will be removed in 0.9.0. ' \
-            'Use `self.persisted_flag_callback_control = true` set to `true`. ' \
-            '(Alternatively, exclude statements that set `false`) ' \
-            'cf. https://github.com/hamajyotan/active_record_compose/blob/v0.8.1/UPGRADE.md ',
-          )
-          # steep:ignore:end
-          run_callbacks(:save) { save_models(bang: false) }
-        end
+        with_callbacks { save_models(bang: false) }
       rescue ActiveRecord::RecordInvalid
         false
       end
@@ -98,170 +50,31 @@ module ActiveRecordCompose
     #
     # Saving, like `#save`, is performed within a single transaction.
     #
-    def save!
-      valid? || raise_validation_error
-
-      with_transaction_returning_status do
-        if self.class.persisted_flag_callback_control
-          with_callbacks { save_models(bang: true) }
-        else
-          # steep:ignore:start
-          deprecator.warn(
-            'The behavior with `persisted_flag_callback_control` set to `false` will be removed in 0.9.0. ' \
-            'Use `self.persisted_flag_callback_control = true` set to `true`. ' \
-            '(Alternatively, exclude statements that set `false`) ' \
-            'cf. https://github.com/hamajyotan/active_record_compose/blob/v0.8.1/UPGRADE.md ',
-          )
-          # steep:ignore:end
-          run_callbacks(:save) { save_models(bang: true) }
-        end
-      end || raise_on_save_error
-    end
-
-    # Behavior is same to `#save`, but `before_create` and `after_create` hooks fires.
-    #
-    #   class ComposedModel < ActiveRecordCompose::Model
-    #     # ...
-    #
-    #     before_save { puts 'before_save called!' }
-    #     before_create { puts 'before_create called!' }
-    #     before_update { puts 'before_update called!' }
-    #     after_save { puts 'after_save called!' }
-    #     after_create { puts 'after_create called!' }
-    #     after_update { puts 'after_update called!' }
-    #   end
-    #
-    #   model = ComposedModel.new
+    # Options like `:validate` and `:context` are not accepted as arguments.
+    # The need for such values indicates that operations from multiple contexts are being handled.
+    # However, if the contexts are different, it is recommended to separate them into different model definitions.
     #
-    #   model.save
-    #   # before_save called!
-    #   # after_save called!
-    #
-    #   model.create
-    #   # before_save called!
-    #   # before_create called!
-    #   # after_create called!
-    #   # after_save called!
-    #
-    # @deprecated
-    def create(attributes = {})
-      if self.class.persisted_flag_callback_control
-        raise '`#create` cannot be called. The context for creation or update is determined by the `#persisted` flag.'
-      end
-
-      # steep:ignore:start
-      deprecator.warn(
-        'The behavior with `persisted_flag_callback_control` set to `false` will be removed in 0.9.0. ' \
-        'Use `self.persisted_flag_callback_control = true` set to `true`. ' \
-        '(Alternatively, exclude statements that set `false`) ' \
-        'cf. https://github.com/hamajyotan/active_record_compose/blob/v0.8.1/UPGRADE.md ',
-      )
-      # steep:ignore:end
-
-      assign_attributes(attributes)
-      return false if invalid?
-
-      with_transaction_returning_status do
-        with_callbacks(context: :create) { save_models(bang: false) }
-      rescue ActiveRecord::RecordInvalid
-        false
-      end
-    end
-
-    # Behavior is same to `#create`, but raises an exception prematurely on failure.
-    #
-    # @deprecated
-    def create!(attributes = {})
-      if self.class.persisted_flag_callback_control
-        raise '`#create` cannot be called. The context for creation or update is determined by the `#persisted` flag.'
-      end
-
-      # steep:ignore:start
-      deprecator.warn(
-        'The behavior with `persisted_flag_callback_control` set to `false` will be removed in 0.9.0. ' \
-        'Use `self.persisted_flag_callback_control = true` set to `true`. ' \
-        '(Alternatively, exclude statements that set `false`) ' \
-        'cf. https://github.com/hamajyotan/active_record_compose/blob/v0.8.1/UPGRADE.md ',
-      )
-      # steep:ignore:end
-
-      assign_attributes(attributes)
+    def save!
       valid? || raise_validation_error
 
       with_transaction_returning_status do
-        with_callbacks(context: :create) { save_models(bang: true) }
+        with_callbacks { save_models(bang: true) }
       end || raise_on_save_error
     end
 
-    # Behavior is same to `#save`, but `before_update` and `after_update` hooks fires.
-    #
-    #   class ComposedModel < ActiveRecordCompose::Model
-    #     # ...
-    #
-    #     before_save { puts 'before_save called!' }
-    #     before_create { puts 'before_create called!' }
-    #     before_update { puts 'before_update called!' }
-    #     after_save { puts 'after_save called!' }
-    #     after_create { puts 'after_create called!' }
-    #     after_update { puts 'after_update called!' }
-    #   end
-    #
-    #   model = ComposedModel.new
-    #
-    #   model.save
-    #   # before_save called!
-    #   # after_save called!
-    #
-    #   model.update
-    #   # before_save called!
-    #   # before_update called!
-    #   # after_update called!
-    #   # after_save called!
+    # Assign attributes and save.
     #
+    # @return [Boolean] returns true on success, false on failure.
     def update(attributes = {})
       assign_attributes(attributes)
-      return false if invalid?
-
-      with_transaction_returning_status do
-        if self.class.persisted_flag_callback_control
-          with_callbacks { save_models(bang: false) }
-        else
-          # steep:ignore:start
-          deprecator.warn(
-            'The behavior with `persisted_flag_callback_control` set to `false` will be removed in 0.9.0. ' \
-            'Use `self.persisted_flag_callback_control = true` set to `true`. ' \
-            '(Alternatively, exclude statements that set `false`) ' \
-            'cf. https://github.com/hamajyotan/active_record_compose/blob/v0.8.1/UPGRADE.md ',
-          )
-          # steep:ignore:end
-          with_callbacks(context: :update) { save_models(bang: false) }
-        end
-      rescue ActiveRecord::RecordInvalid
-        false
-      end
+      save
     end
 
     # Behavior is same to `#update`, but raises an exception prematurely on failure.
     #
     def update!(attributes = {})
       assign_attributes(attributes)
-      valid? || raise_validation_error
-
-      with_transaction_returning_status do
-        if self.class.persisted_flag_callback_control
-          with_callbacks { save_models(bang: true) }
-        else
-          # steep:ignore:start
-          deprecator.warn(
-            'The behavior with `persisted_flag_callback_control` set to `false` will be removed in 0.9.0. ' \
-            'Use `self.persisted_flag_callback_control = true` set to `true`. ' \
-            '(Alternatively, exclude statements that set `false`) ' \
-            'cf. https://github.com/hamajyotan/active_record_compose/blob/v0.8.1/UPGRADE.md ',
-          )
-          # steep:ignore:end
-          with_callbacks(context: :update) { save_models(bang: true) }
-        end
-      end || raise_on_save_error
+      save!
     end
 
     # Returns true if model is persisted.
@@ -281,13 +94,9 @@ module ActiveRecordCompose
       models.__wrapped_models.lazy.select { _1.invalid? }.each { errors.merge!(_1) }
     end
 
-    def with_callbacks(context: nil, &block)
-      run_callbacks(:save) { run_callbacks(callback_context(context:), &block) }
-    end
+    def with_callbacks(&block) = run_callbacks(:save) { run_callbacks(callback_context, &block) }
 
-    def callback_context(context: nil)
-      context || (persisted? ? :update : :create)
-    end
+    def callback_context = persisted? ? :update : :create
 
     def save_models(bang:)
       models.__wrapped_models.all? { bang ? _1.save! : _1.save }
@@ -298,15 +107,5 @@ module ActiveRecordCompose
     def raise_on_save_error = raise ActiveRecord::RecordNotSaved.new(raise_on_save_error_message, self)
 
     def raise_on_save_error_message = 'Failed to save the model.'
-
-    # steep:ignore:start
-    def deprecator
-      if ActiveRecord.respond_to?(:deprecator)
-        ActiveRecord.deprecator
-      else # for rails 7.0.x or lower
-        ActiveSupport::Deprecation
-      end
-    end
-    # steep:ignore:end
   end
 end

data/lib/active_record_compose/transaction_support.rb

@@ -18,7 +18,7 @@ module ActiveRecordCompose
 
       def with_connection(&) = ar_class.with_connection(&) # steep:ignore
 
-      def composite_primary_key? = false # steep:ignore
+      def composite_primary_key? = false
 
       private
 

data/lib/active_record_compose/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ActiveRecordCompose
-  VERSION = '0.8.1'
+  VERSION = '0.9.0'
 end

data/lib/active_record_compose.rb

@@ -7,15 +7,3 @@ require_relative 'active_record_compose/model'
 
 module ActiveRecordCompose
 end
-
-if ActiveRecordCompose::VERSION == '0.8.1'
-  unless ENV['ACTIVE_RECORD_COMPOSE_SILENCE_DEPRECATION'] # rubocop:disable Style/SoleNestedConditional
-    warn <<~WARN
-
-      [DEPRECATION] You are using active_record_compose version 0.8.1, which is deprecated.
-      Please upgrade to the latest version.
-      See: https://github.com/hamajyotan/active_record_compose/blob/v0.8.1/UPGRADE.md
-
-    WARN
-  end
-end

data/sig/_internal/package_private.rbs

@@ -24,7 +24,7 @@ module ActiveRecordCompose
     def delegated_attributes: () -> Array[String]
 
     module ClassMethods : Module
-      def delegate_attribute: (*untyped methods, to: untyped, ?allow_nil: untyped?, ?private: untyped?) -> untyped
+      def delegate_attribute: (*untyped methods, to: untyped, ?allow_nil: untyped?) -> untyped
       def delegated_attributes: () -> Array[String]
       def delegated_attributes=: (Array[String]) -> untyped
     end

data/sig/active_record_compose.rbs

@@ -69,10 +69,7 @@ module ActiveRecordCompose
     def self.after_commit: (*callback[instance], ?if: condition[instance], ?unless: condition[instance], **untyped) ?{ () [self: instance] -> void } -> void
     def self.after_rollback: (*callback[instance], ?if: condition[instance], ?unless: condition[instance], **untyped) ?{ () [self: instance] -> void } -> void
 
-    def self.persisted_flag_callback_control: () -> boolish
-    def self.persisted_flag_callback_control=: (boolish) -> untyped
-
-    def self.delegate_attribute: (*untyped methods, to: untyped, ?allow_nil: untyped?, ?private: untyped?) -> untyped
+    def self.delegate_attribute: (*untyped methods, to: untyped, ?allow_nil: untyped?) -> untyped
     def self.connection: -> ActiveRecord::ConnectionAdapters::AbstractAdapter
     def self.lease_connection: -> ActiveRecord::ConnectionAdapters::AbstractAdapter
     def self.with_connection: [T] () { () -> T } -> T
@@ -88,8 +85,8 @@ module ActiveRecordCompose
 
     private
     def models: -> ComposedCollection
-    def with_callbacks: (?context: (nil | :create | :update)) { () -> bool } -> bool
-    def callback_context: (?context: (nil | :create | :update)) -> (:create | :update)
+    def with_callbacks: { () -> bool } -> bool
+    def callback_context: -> (:create | :update)
     def validate_models: -> void
     def save_models: (bang: bool) -> bool
     def raise_validation_error: -> bot

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: active_record_compose
 version: !ruby/object:Gem::Version
-  version: 0.8.1
+  version: 0.9.0
 platform: ruby
 authors:
 - hamajyotan
 bindir: bin
 cert_chain: []
-date: 2025-06-21 00:00:00.000000000 Z
+date: 2025-03-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -16,9 +16,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '7.0'
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '8.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -26,9 +23,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '7.0'
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '8.1'
 description: activemodel form object pattern. it embraces multiple AR models and provides
   a transparent interface as if they were a single model.
 email:
@@ -42,7 +36,6 @@ files:
 - CODE_OF_CONDUCT.md
 - LICENSE.txt
 - README.md
-- UPGRADE.md
 - lib/active_record_compose.rb
 - lib/active_record_compose/composed_collection.rb
 - lib/active_record_compose/delegate_attribute.rb
@@ -59,17 +52,8 @@ metadata:
   homepage_uri: https://github.com/hamajyotan/active_record_compose
   source_code_uri: https://github.com/hamajyotan/active_record_compose
   changelog_uri: https://github.com/hamajyotan/active_record_compose/blob/main/CHANGELOG.md
-  documentation_uri: https://www.rubydoc.info/gems/active_record_compose/0.8.1
+  documentation_uri: https://www.rubydoc.info/gems/active_record_compose/0.9.0
   rubygems_mfa_required: 'true'
-post_install_message: |2+
-
-  Notice: This version (0.8.1) is deprecated.
-
-  Please upgrade to the latest version of `active_record_compose` as soon as possible.
-  Future releases will not support this version.
-
-  See https://github.com/hamajyotan/active_record_compose/blob/v0.8.1/UPGRADE.md for migration steps.
-
 rdoc_options: []
 require_paths:
 - lib
@@ -88,4 +72,3 @@ rubygems_version: 3.6.2
 specification_version: 4
 summary: activemodel form object pattern
 test_files: []
-...

data/UPGRADE.md

@@ -1,113 +0,0 @@
-# Migration Guide: `active_record_compose` 0.8.x to 0.9.0+
-
-⚠️ **Breaking Change**
-
-This guide explains how to migrate from version **0.8.x** to **0.9.0 or later**, assuming that your codebase includes:
-
-```ruby
-self.persisted_flag_callback_control = false
-```
-
-This setting was officially **deprecated** in 0.9.0 and is **no longer supported**. Migration is required to adopt the current default behavior.
-
-## Background
-
-In version **0.7.x**, the default value of `persisted_flag_callback_control` was `false`.
-
-In **0.8.x**, the default was changed to `true`.
-If you are still using `false` in your models, it means you have **explicitly overridden the default**, likely to preserve backward-compatible behavior.
-
-This guide helps you safely migrate to the new behavior while preserving intended callback semantics.
-
-## Goal of the Migration
-
-- Remove any use of `persisted_flag_callback_control = false`
-- Adjust callback definitions to align with the semantics of `#persisted?`
-
-## Step 1 – Remove Deprecated Flag
-
-Find and remove all instances of:
-
-```diff
--self.persisted_flag_callback_control = false
-```
-
-## Step 2 – Understand Callback Behavior Changes
-
-With `persisted_flag_callback_control = true`, whether a callback is fired depends on the return value of `#persisted?`, not the method used (`create` or `update`).
-
-### When saving with `#update`:
-
-If `persisted?` returns `false`, then:
-
-- `before_update`
-- `after_update`
-- `around_update`
-
-will **not** be triggered.
-
-✅ Recommended fix:
-
-If you don’t differentiate between creation and update phases, switch to `*_save` callbacks:
-
-```diff
-- before_update :track_change
-+ before_save :track_change
-```
-
-### When saving with `#create`:
-
-If `persisted?` returns `true`, then:
-
-- `before_create`
-- `after_create`
-- `around_create`
-
-will **not** be triggered.
-
-✅ Recommended fix:
-
-Again, prefer `*_save` if you're using shared logic across creation and update:
-
-```diff
-- after_create :send_notification
-+ after_save :send_notification
-```
-
-## Step 3 – Override `#persisted?` if Needed
-
-If your composed model wraps an ActiveRecord instance and delegates its persistence logic, be sure to override `#persisted?` to reflect the correct state:
-
-```ruby
-class Foo < ActiveRecordCompose::Model
-  def initialize(bar = Bar.new)
-    super()
-    @bar = bar
-  end
-
-  def persisted? = bar.persisted?
-
-  private
-
-  attr_reader :bar
-end
-```
-
-## Migration Checklist ✅
-
-- [ ] All models now `persisted_flag_callback_control` omit it entirely
-- [ ] All callback definitions have been reviewed and updated
-- [ ] Any `*_create` or `*_update` callbacks have been replaced with `*_save` where applicable
-- [ ] `#persisted?` is correctly overridden where needed
-- [ ] All tests pass and expected callbacks are fired
-
-## Why This Change?
-
-By aligning callback behavior with `persisted?`, you gain:
-
-- Clearer intent and semantics
-- More accurate behavior when composing or wrapping persisted models
-- Improved compatibility with Rails conventions
-- Less surprising callback triggering
-
-If you have questions or run into edge cases, feel free to [open an issue](https://github.com/hamajyotan/active_record_compose/issues) or start a discussion.