Merge pull request #63 from pabloh/remove_deprecated_behaviour

Remove deprecated behaviour

Author: pabloh

CHANGELOG.md

@@ -1,13 +1,15 @@
-## [1.3.0] - 2026-04-01
+## [1.3.0] - 2026-03-25
 ### Changed
 - Removed support for `Ruby` versions older than 3.3
 ### Removed
+- Remove support for passing a block to the step method through `DSLMethods#step`
+- Remove support for passing a block to the step method through `DSLMethods#set`
 - Remove support for `DSLMethods#map`
 
 ## [1.2.0] - 2025-11-16
 ### Deprecated
-- Deprecate passing a block to the step method using `DSLMethods#step`
-- Deprecate passing a block to the step method using `DSLMethods#set`
+- Deprecate passing a block to the step method through `DSLMethods#step`
+- Deprecate passing a block to the step method through `DSLMethods#set`
 - Deprecate `DSLMethods#map`
 ### Changed
 - Removed deprecated `:auto_wire_options` option from `:dry_validation` plugin

lib/pathway.rb

@@ -58,7 +58,7 @@ def default_message_for(type)
   class State
     extend Forwardable
 
-    delegate %i([] []= fetch store include? values_at deconstruct_keys) => :@hash
+    delegate %i[[] []= fetch store include? values_at deconstruct_keys] => :@hash
 
     def initialize(operation, values = {})
       @hash = operation.context.merge(values)
@@ -158,27 +158,17 @@ def run(&)
         end
 
         # Execute step and preserve the former state
-        def step(callable, ...)
-          #:nocov:
-          if block_given?
-            warn "[DEPRECATION] Passing a block to the step method using `DSLMethods#step` is deprecated"
-          end
-          #:nocov:
-          bl = _callable(callable)
-          @result = @result.tee { |state| bl.call(state, ...) }
+        def step(callable, *, **, &)
+          bl = _callable(callable, &)
+          @result = @result.tee { |state| bl.call(state, *, **) }
         end
 
         # Execute step and modify the former state setting the key
-        def set(callable, *args, to: @operation.result_key, **kwargs, &bl)
-          #:nocov:
-          if block_given?
-            warn "[DEPRECATION] Passing a block to the step method using `DSLMethods#set` is deprecated"
-          end
-          #:nocov:
-          bl = _callable(callable)
+        def set(callable, *args, to: @operation.result_key, **kwargs, &)
+          bl = _callable(callable, &)
 
           @result = @result.then do |state|
-            wrap(bl.call(state, *args, **kwargs, &bl))
+            wrap(bl.call(state, *args, **kwargs))
               .then { |value| state.update(to => value) }
           end
         end