active_harness 0.2.20 → 0.2.21

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d1188ca468f31f46c40d118b7e719a849925aa11ddab3a4a87ea4df0fd9bdb84
-  data.tar.gz: 1ed36314efd943e6f4e62bbc43b2ec00d0b0486bdbf015e6b590341d240ffcdc
+  metadata.gz: de49ac66334a5feb91a5d5f0314ab2b9b3460a5d3a549f08b77538677eae619b
+  data.tar.gz: 0bffb6e77d1b8def25121b144dbc956c2c1b0c684826910e3dd0a66ab595a73e
 SHA512:
-  metadata.gz: 37e82033bddd4069f26619d2bfc5faec831c32618216bedb6cb9ce19ca7fbbaac1057e829809ee15f1d88d1a6fc3b3832cd2c02d1f112c850a974a26205d3727
-  data.tar.gz: 3a4fafd8184d55f116689e28c4f411d102f7279d290093c9cc32dafabfc9eafe9aa55abde59a04cec37985e74da94a8ddb14153febce35aa41c2eb0404eed2c6
+  metadata.gz: e51c5ab8c3911d2597f6269f695c2f4992f960d771b4284fdd27121939aca14dfea53e568ecbfc069af26b405725106c18b3beebf0232e751ed8144b3134cea7
+  data.tar.gz: ea95a9f48993b4dc930addeee4260e6056815ec48d3c2534a6bf6f5001168cad2104fcd988c98e442f940f32faa4161f9b59a4692bc6fa3adf7655a221d5d56b

data/lib/active_harness/pipeline/hooks.rb

@@ -0,0 +1,77 @@
+module ActiveHarness
+  class Pipeline
+    VALID_HOOKS      = %i[before_step after_step stopped complete].freeze
+    VALID_STEP_HOOKS = %i[before_step after_step].freeze
+
+    class << self
+      # Register a global or per-step hook.
+      #
+      # Global hooks fire on every step:
+      #   on :before_step do |step_name, payload| ... end
+      #   on :after_step  do |step_name, result|  ... end
+      #   on :stopped     do |step_name, result|  ... end
+      #   on :complete    do |last_result|         ... end
+      #
+      # Per-step hooks fire only for the named step (no step_name passed):
+      #   on :before_step, :translate do |payload| ... end
+      #   on :after_step,  :translate do |result|  ... end
+      def on(event, step_name = nil, &block)
+        if step_name
+          unless VALID_STEP_HOOKS.include?(event)
+            raise ArgumentError,
+              "Per-step hooks support: #{VALID_STEP_HOOKS.join(", ")}. Got :#{event}"
+          end
+          pipeline_config[:step_hooks][step_name] ||= {}
+          pipeline_config[:step_hooks][step_name][event] = block
+        else
+          unless VALID_HOOKS.include?(event)
+            raise ArgumentError,
+              "Unknown Pipeline hook :#{event}. Valid: #{VALID_HOOKS.join(", ")}"
+          end
+          pipeline_config[:hooks][event] = block
+        end
+      end
+
+      # Rails-style aliases for +on+:
+      #
+      # Global:
+      #   before :step                          do |name, payload| end  # → on :before_step
+      #   after  :step                          do |name, result|  end  # → on :after_step
+      #   callback :stopped                     do |name, result|  end  # → on :stopped
+      #   callback :complete                    do |result|        end  # → on :complete
+      #
+      # Per-step:
+      #   after  :step, :translate              do |result| end
+      #   before :step, :translate              do |payload| end
+      def before(event, step_name = nil, &block)
+        on(:"before_#{event}", step_name, &block)
+      end
+
+      def after(event, step_name = nil, &block)
+        on(:"after_#{event}", step_name, &block)
+      end
+
+      def callback(event, &block)
+        on(event, &block)
+      end
+    end
+
+    private
+
+    # Fires global hook AND pipeline_event_stream. Consistent with Agent#fire and Tribunal#fire.
+    def fire(event, step_name, data, config)
+      blk = config[:hooks][event]
+      instance_exec(step_name, data, &blk) if blk
+      @pipeline_event_stream&.call(event, step_name, data)
+    rescue IOError, ActionController::Live::ClientDisconnected
+      nil
+    end
+
+    # Per-step hook: receives (data) only — not forwarded to pipeline_event_stream
+    # (global fire already covers the step event with step_name context).
+    def fire_step(event, step_name, data, config)
+      blk = config[:step_hooks][step_name]&.dig(event)
+      instance_exec(data, &blk) if blk
+    end
+  end
+end

data/lib/active_harness/pipeline.rb

@@ -32,9 +32,6 @@ module ActiveHarness
   #   pipeline.step_results # => { translate: <Result>, ... }
   #
   class Pipeline
-    VALID_HOOKS      = %i[before_step after_step stopped complete].freeze
-    VALID_STEP_HOOKS = %i[before_step after_step].freeze
-
     # -------------------------------------------------------------------------
     # Class-level DSL
     # -------------------------------------------------------------------------
@@ -53,57 +50,6 @@ module ActiveHarness
         pipeline_config[:steps] << Pipeline::Step.new(name, agent_class, &block)
       end
 
-      # Register a global or per-step hook.
-      #
-      # Global hooks fire on every step:
-      #   on :before_step do |step_name, payload| ... end
-      #   on :after_step  do |step_name, result|  ... end
-      #   on :stopped     do |step_name, result|  ... end
-      #   on :complete    do |last_result|         ... end
-      #
-      # Per-step hooks fire only for the named step (no step_name passed):
-      #   on :before_step, :translate do |payload| ... end
-      #   on :after_step,  :translate do |result|  ... end
-      def on(event, step_name = nil, &block)
-        if step_name
-          unless VALID_STEP_HOOKS.include?(event)
-            raise ArgumentError,
-              "Per-step hooks support: #{VALID_STEP_HOOKS.join(", ")}. Got :#{event}"
-          end
-          pipeline_config[:step_hooks][step_name] ||= {}
-          pipeline_config[:step_hooks][step_name][event] = block
-        else
-          unless VALID_HOOKS.include?(event)
-            raise ArgumentError,
-              "Unknown Pipeline hook :#{event}. Valid: #{VALID_HOOKS.join(", ")}"
-          end
-          pipeline_config[:hooks][event] = block
-        end
-      end
-
-      # Rails-style aliases for +on+:
-      #
-      # Global:
-      #   before :step                          do |name, payload| end  # → on :before_step
-      #   after  :step                          do |name, result|  end  # → on :after_step
-      #   callback :stopped                     do |name, result|  end  # → on :stopped
-      #   callback :complete                    do |result|        end  # → on :complete
-      #
-      # Per-step:
-      #   after  :step, :translate              do |result| end
-      #   before :step, :translate              do |payload| end
-      def before(event, step_name = nil, &block)
-        on(:"before_#{event}", step_name, &block)
-      end
-
-      def after(event, step_name = nil, &block)
-        on(:"after_#{event}", step_name, &block)
-      end
-
-      def callback(event, &block)
-        on(event, &block)
-      end
-
       def pipeline_config
         @pipeline_config ||= { steps: [], hooks: {}, step_hooks: {} }
       end
@@ -219,23 +165,8 @@ module ActiveHarness
         ).call.result
       end
     end
-
-    # Fires global hook AND pipeline_event_stream. Consistent with Agent#fire and Tribunal#fire.
-    def fire(event, step_name, data, config)
-      blk = config[:hooks][event]
-      instance_exec(step_name, data, &blk) if blk
-      @pipeline_event_stream&.call(event, step_name, data)
-    rescue IOError, ActionController::Live::ClientDisconnected
-      nil
-    end
-
-    # Per-step hook: receives (data) only — not forwarded to pipeline_event_stream
-    # (global fire already covers the step event with step_name context).
-    def fire_step(event, step_name, data, config)
-      blk = config[:step_hooks][step_name]&.dig(event)
-      instance_exec(data, &blk) if blk
-    end
   end
 end
 
+require_relative "pipeline/hooks"
 require_relative "pipeline/step"

data/lib/active_harness.rb

@@ -29,7 +29,7 @@ require_relative "active_harness/pipeline"
 require_relative "active_harness/railtie" if defined?(Rails::Railtie)
 
 module ActiveHarness
-  VERSION = "0.2.20"
+  VERSION = "0.2.21"
 
   class << self
     # Configure ActiveHarness.

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: active_harness
 version: !ruby/object:Gem::Version
-  version: 0.2.20
+  version: 0.2.21
 platform: ruby
 authors:
 - the-teacher
@@ -51,6 +51,7 @@ files:
 - lib/active_harness/memory/adapter/base.rb
 - lib/active_harness/memory/adapter/file.rb
 - lib/active_harness/pipeline.rb
+- lib/active_harness/pipeline/hooks.rb
 - lib/active_harness/pipeline/step.rb
 - lib/active_harness/providers/PROVIDER_CONTRACT.md
 - lib/active_harness/providers/anthropic.rb