active_harness 0.2.38 → 0.2.40

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '064709a78ef949aadd741237f4cdd5a2f36056d211fa2038048a2911fc45a5be'
-  data.tar.gz: 298318704ce083cf46b87dc5dbaebbae15df127a313685ea32d5c82b03191072
+  metadata.gz: 4246d31550942e31c26ec9a708f49aae5b2e0f385a32aeaa06d6d6a75f2849ee
+  data.tar.gz: d7f12a75e7f72e3d004bfd3b64a094e99a082992fd0077e69253bddf97b919f0
 SHA512:
-  metadata.gz: b7c4e5a06e8f8a66e89a0f0b2289c7f461e76cd942a374abb5e6a3845f35e348b8b5fc63182e384d825863ef4c8698c59b64d606d11b5eb27bcada47b1d57642
-  data.tar.gz: 3528828f40f1a11a0b62b45ff193b49e05cc202e09e754a578e94c372279cdb760622d5e33de3f61aeef62d7cf15166197a4e0d2a5637d0c4cb8e5d5cf332112
+  metadata.gz: 81cf0f7cb7f5f54c2a996097e34461f724bd66da920db0cee57cc84f558f8430bc83ad1e914c62a56f3493e890c993c9f2dd9408c161563ae8c161f5ac03e4d5
+  data.tar.gz: e144ed04675d274af6d66157a2f733e1cb65342adc7312b10428e9afc3c135b64af616218d6bb9ddbfb36948efd0e83b817f36654523f910c51d4797e07674e6

data/lib/active_harness/agent/image.rb

@@ -7,6 +7,10 @@ module ActiveHarness
       #     image true
       #     size  "1024x1024"
       #
+      #     # Optional: base style/formatting guidance, prepended to @input
+      #     # to form the final image prompt (see Agent#build_image_prompt).
+      #     system_prompt "Watercolor style, soft pastel colors, no text overlays."
+      #
       #     model do
       #       use      provider: :openrouter, model: "openai/gpt-5-image-mini"
       #       fallback provider: :openai,     model: "gpt-image-1"

data/lib/active_harness/agent/providers.rb

@@ -47,7 +47,7 @@ module ActiveHarness
 
     def attempt_model(entry, system_prompt)
       return attempt_via_custom_llm(entry, system_prompt) if @config[:custom_llm_backend]
-      return attempt_image_model(entry)                   if @config[:image]
+      return attempt_image_model(entry, system_prompt)     if @config[:image]
 
       provider = resolve_provider(entry[:provider])
       messages = build_messages(system_prompt, @input)
@@ -58,17 +58,26 @@ module ActiveHarness
       provider.call(**opts)
     end
 
-    def attempt_image_model(entry)
+    def attempt_image_model(entry, system_prompt)
       factory = IMAGE_PROVIDERS[entry[:provider].to_sym]
       raise ArgumentError, "Provider #{entry[:provider].inspect} does not support image generation. " \
                            "Supported image providers: #{IMAGE_PROVIDERS.keys.join(', ')}" unless factory
 
       size = entry[:size] || @config[:image_size] || "1024x1024"
-      opts = { model: entry[:model], prompt: @input.to_s, size: size }
+      opts = { model: entry[:model], prompt: build_image_prompt(system_prompt, @input.to_s), size: size }
       opts[:quality] = entry[:quality] if entry[:quality]
       factory.call.call(**opts)
     end
 
+    # Image APIs take a single prompt string (no system/user role split), so the
+    # system prompt — e.g. base style/formatting guidance — is prepended to the
+    # user's input rather than sent as a separate message.
+    def build_image_prompt(system_prompt, input)
+      return input if system_prompt.nil? || system_prompt.to_s.strip.empty?
+
+      "#{system_prompt}\n\n#{input}"
+    end
+
     def resolve_provider(name)
       factory = PROVIDERS[name.to_sym]
       raise ArgumentError, "Unknown provider: #{name.inspect}. Supported: #{PROVIDERS.keys.join(', ')}" unless factory

data/lib/active_harness/memory.rb

@@ -10,14 +10,21 @@ module ActiveHarness
   # It does NOT automatically inject history into LLM messages.
   # Injection is always manual — you control when and how context is used.
   #
-  # --- Recording (automatic) ---
+  # --- Recording ---
   #
-  # When a Memory object is passed to an agent, the agent automatically
-  # saves each successful turn (request + response) after the call.
+  # Passing a Memory object to Agent.call(memory:) does NOT by itself save
+  # anything — Agent never calls #load or #record on it. Loading and
+  # recording turns for a bare agent call is entirely manual (e.g. via
+  # before_call/after_call hooks calling @memory.load / @memory.record).
+  #
+  # Pipeline is the one place recording is automatic: when a Memory is
+  # passed to Pipeline#call, the pipeline itself calls #load before running
+  # steps and #record after a successful run — independent of any hooks the
+  # individual agents define.
   #
   #   memory = ActiveHarness::Memory.new(session_id: "u42", depth: 8)
   #   SupportAgent.call(input: "Hello", memory: memory)
-  #   # => turn is saved to storage/ai/memory/u42.json
+  #   # => nothing is saved unless SupportAgent's own hooks record it
   #
   # --- Manual injection patterns ---
   #

data/lib/active_harness/pipeline/README.md

@@ -25,7 +25,7 @@ pipeline.call
 
 pipeline.output       # => final payload string (nil if stopped)
 pipeline.stopped?     # => false
-pipeline.step_results # => { translate: <Result>, injection_guard: <Result>, ... }
+pipeline.steps.to_a   # => [[:translate, <TranslationAgent>, <Result>], [:injection_guard, <InjectionGuardAgent>, <Result>], ...]
 ```
 
 ## Step types

data/lib/active_harness/pipeline/step.rb

@@ -1,18 +1,18 @@
 module ActiveHarness
   class Pipeline
     class Step
-      attr_reader :name, :agent_class
+      attr_reader :name, :executor
 
-      def initialize(name, agent_class = nil, &block)
-        @name        = name
-        @agent_class = agent_class
-        @stop_if     = nil
+      def initialize(name, executor = nil, &block)
+        @name     = name
+        @executor = executor
+        @stop_if  = nil
         instance_eval(&block) if block_given?
       end
 
-      # DSL: use TranslationAgent
+      # DSL: use TranslationAgent / SafetyTribunal / NestedPipeline
       def use(klass)
-        @agent_class = klass
+        @executor = klass
       end
 
       # DSL (inside block): stop_if ->(result) { ... }
@@ -35,9 +35,13 @@ module ActiveHarness
         @transform_block
       end
 
-      # True if agent_class is a Tribunal subclass — tribunal steps do not update payload.
+      def lambda?
+        @executor.is_a?(Proc)
+      end
+
+      # True if the executor is a Tribunal subclass — tribunal steps do not update payload.
       def tribunal?
-        @agent_class.is_a?(Class) && @agent_class <= ActiveHarness::Tribunal
+        @executor.is_a?(Class) && @executor <= ActiveHarness::Tribunal
       end
 
       # Returns true when this step should update the pipeline payload after execution.

data/lib/active_harness/pipeline.rb

@@ -29,7 +29,7 @@ module ActiveHarness
   #   pipeline.call
   #   pipeline.output       # => final payload string (nil if stopped)
   #   pipeline.stopped?     # => false
-  #   pipeline.step_results # => { translate: <Result>, ... }
+  #   pipeline.steps { |name, result| ... }  # iterate over completed steps
   #
   class Pipeline
     # -------------------------------------------------------------------------
@@ -46,8 +46,8 @@ module ActiveHarness
       #     use InjectionGuardAgent
       #     stop_if ->(result) { result.processed["detected"] == true }
       #   end
-      def step(name, agent_class = nil, &block)
-        pipeline_config[:steps] << Pipeline::Step.new(name, agent_class, &block)
+      def step(name, executor = nil, &block)
+        pipeline_config[:steps] << Pipeline::Step.new(name, executor, &block)
       end
 
       def pipeline_config
@@ -100,7 +100,6 @@ module ActiveHarness
                   :stopped_at,
                   :stop_reason,
                   :execution_time,
-                  :step_results,
                   :context
     attr_writer   :context
     attr_accessor :params
@@ -126,6 +125,7 @@ module ActiveHarness
       @token          = token
       class_streams   = self.class.pipeline_config[:streams] || {}
       @stream         = merge_stream(stream, class_streams)
+      @executors      = build_executors
       @step_results   = {}
       @stopped        = false
       @stopped_at     = nil
@@ -134,10 +134,44 @@ module ActiveHarness
       @output         = nil
     end
 
+    # Returns a hash of pre-created executor instances keyed by step name.
+    # Available immediately after new — before call — so instances can be
+    # configured (model lists, params, etc.) before execution begins.
+    #
+    #   pipeline = SupportPipeline.new(input: "...")
+    #   pipeline.executors[:translate].models.prepend(provider: :openai, model: "gpt-4.1")
+    #   pipeline.executors[:guard].params = { strict: true }
+    #   pipeline.call
+    def executors
+      @executors
+    end
+
     def stopped?
       @stopped
     end
 
+    # Iterates over completed steps as (name, executor, result) tuples.
+    # Steps that did not run (pipeline stopped before reaching them) are skipped.
+    # Returns an Enumerator when called without a block.
+    #
+    #   pipeline.steps { |name, executor, result| }
+    #
+    #   name     — step name symbol          (:translate, :guard, …)
+    #   executor — the instance that ran     (TranslationAgent instance, …)
+    #   result   — Result struct             (output, processed, usage, model, …)
+    #
+    #   pipeline.steps.map { |name, executor, result| [name, result.output] }
+    #   pipeline.steps.to_a
+    def steps
+      return enum_for(:steps) unless block_given?
+      self.class.pipeline_config[:steps].each do |step|
+        result = @step_results[step.name]
+        next unless result
+        yield step.name, @executors[step.name], result
+      end
+      self
+    end
+
     # Wraps pipeline outcome into a Result so a pipeline can be used as a step
     # inside another pipeline, matching the same interface as Agent and Tribunal.
     #
@@ -153,7 +187,15 @@ module ActiveHarness
     end
 
     # Execute all steps sequentially. Returns self for chaining.
-    def call
+    # Accepts optional input, token, stream to match the Agent/Tribunal call interface.
+    def call(input = nil, token: nil, stream: nil)
+      if input
+        @original_input = input
+        @payload        = input
+      end
+      @token  = token  if token
+      @stream = stream if stream
+
       config = self.class.pipeline_config
       t0     = Process.clock_gettime(Process::CLOCK_MONOTONIC)
 
@@ -231,13 +273,40 @@ module ActiveHarness
     end
 
     def execute_step(step)
-      step.agent_class.new(
-        input:   @payload,
-        context: @context.dup,
-        params:  @params,
-        token:   @token,
-        stream:  @stream
-      ).call.result
+      if step.lambda?
+        execute_lambda_step(step)
+      else
+        instance         = @executors[step.name]
+        instance.context = @context.dup
+        instance.call(@payload).result
+      end
+    end
+
+    def execute_lambda_step(step)
+      lam    = step.executor
+      has_kw = lam.parameters.any? { |type, _| [:key, :keyreq, :keyrest].include?(type) }
+      result = has_kw ? lam.call(@payload, context: @context.dup, params: @params) : lam.call(@payload)
+
+      unless result.is_a?(Result)
+        raise ArgumentError,
+          "Lambda step :#{step.name} must return ActiveHarness::Result, got #{result.class}"
+      end
+
+      result
+    end
+
+    def build_executors
+      self.class.pipeline_config[:steps].each_with_object({}) do |step, hash|
+        next if step.lambda?
+
+        hash[step.name] = step.executor.new(
+          input:   @payload,
+          context: @context.dup,
+          params:  @params,
+          token:   @token,
+          stream:  @stream
+        )
+      end
     end
   end
 end

data/lib/active_harness/tribunal.rb

@@ -105,11 +105,17 @@ module ActiveHarness
     # Run all agents in parallel, then compute the verdict.
     # Returns self so calls can be chained: tribunal.call.verdict
     #
+    # Accepts an optional input to update payload before running — matches
+    # the Agent#call(input) interface so tribunals work as pipeline executors.
+    #
     # Behaviour on failure:
     #   - If some agents fail/timeout, their errors are in #errors and
     #     #results contains only successful results.
     #   - If ALL agents fail/timeout, raises Errors::AllAgentsFailed.
-    def call
+    def call(input = nil, token: nil, stream: nil)
+      @input  = input  if input
+      @token  = token  if token
+      @stream = stream if stream
       agents = resolve_agents
       fire(:before_call)
 

data/lib/active_harness.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: active_harness
 version: !ruby/object:Gem::Version
-  version: 0.2.38
+  version: 0.2.40
 platform: ruby
 authors:
 - the-teacher
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2026-06-14 00:00:00.000000000 Z
+date: 2026-07-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: concurrent-ruby