Add new option. Review.

Author: omeH

lib/table_sync/receiving/config.rb

@@ -58,20 +58,15 @@ def add_option(name, value_setter_wrapper:, value_as_proc_setter_wrapper:, defau
       def add_hook_option(name, hook_class:)
         ivar = :"@#{name}"
 
-        default_conditions = { columns: %i[] }
-        default_handler = proc { |**_| }
-
         @default_values_for_options ||= {}
-        @default_values_for_options[ivar] = proc do
-          hook_class.new(
-            conditions: default_conditions,
-            handler: default_handler,
-          )
-        end
+        @default_values_for_options[ivar] = proc { [] }
 
         define_method(name) do |conditions, &handler|
-          hook = hook_class.new(conditions:, handler:)
-          instance_variable_set(ivar, hook)
+          hooks = instance_variable_get(ivar)
+          hooks ||= []
+
+          hooks << hook_class.new(conditions:, handler:)
+          instance_variable_set(ivar, hooks)
         end
       end
     end

lib/table_sync/receiving/handler.rb

@@ -161,8 +161,9 @@ def perform(config, params) # rubocop:disable Metrics/MethodLength
         model.after_commit do
           config.option(:after_commit_on_update, **params, results:)
 
-          hook = config.option(:on_first_sync)
-          hook.perform(config:, targets: results) if hook.enabled?
+          Array(config.option(:on_first_sync)).each do |hook|
+            hook.perform(config:, targets: results) if hook.enabled?
+          end
         end
       else
         model.after_commit { config.option(:after_commit_on_destroy, **params, results:) }

spec/receiving/config_spec.rb

@@ -351,23 +351,20 @@
 
     describe "#on_first_sync" do
       it "returns correct default value" do
-        hook = config.option(:on_first_sync)
-        expect(hook).not_to be_nil
-        expect(hook).not_to be_enabled
-        expect(hook.conditions[:columns]).to be_empty
-        expect(hook.lookup_code).to eq("")
+        hooks = config.option(:on_first_sync)
+        expect(hooks).to be_empty
       end
 
       it "processes a value" do
         config.on_first_sync(columns: %i[test], test: "value") do |**_|
           # Some hook work here
         end
 
-        hook = config.option(:on_first_sync)
-        expect(hook).not_to be_nil
-        expect(hook).to be_enabled
-        expect(hook.conditions[:columns]).not_to be_empty
-        expect(hook.lookup_code).to eq("test-value")
+        hooks = config.option(:on_first_sync)
+        expect(hooks).not_to be_nil
+        expect(hooks.first).to be_enabled
+        expect(hooks.first.conditions[:columns]).not_to be_empty
+        expect(hooks.first.lookup_code).to eq("test-value")
       end
     end