active_harness 0.3.9 → 0.3.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5549eddee812779586e86bbe67c2e72d077b7770b98421b0a43dbcb6dca7ab00
-  data.tar.gz: 9249bc587c75dfe7f0d5141b311f46ba0dd4b90b308873b31535044bd7f37ef2
+  metadata.gz: a3cdd2394f02350ed7562926bffc2107880d7830feef4280b2e4e438b60253bd
+  data.tar.gz: e585c386a7ad6725f8608c6356b1ca8c76789a02656f2f70d2cdc38de379929a
 SHA512:
-  metadata.gz: 6aebf953bbb38bb9c7f3ea943b2b1f10efefaa7371ba34b8aad0e8f921a42749731af2473723fc870a4581f84cbf364315435ce18d7f5f78809c175de86607dd
-  data.tar.gz: ee32ce687ef93c1616b659ed39ca49e0ac9c98705a7f2768e00bf5ba207cdae92938eaad8012cd8b9bad6413b489419cc7f3c2e3dc6138d8ad5af91dd8bc5ec1
+  metadata.gz: 869af984c65a9519d3f3122f582233f7296aff8646ca5df573d0d3260b6abec2315b536a690493f9253361f953280b55ae435fa6a9a166e647d08df04820b53b
+  data.tar.gz: acf051bd40ba5e6e10c1bd3e9222673e85ebd07cc89c89f9822559db53b65b4421d75605387060e50d70d756c9613ca66e33acce17d50997bc501e0bbadd45ff

data/lib/active_harness/agent.rb

@@ -263,7 +263,7 @@ module ActiveHarness
         body << "        answer: #{res.output.to_s[0, 80].inspect}" if turn.tool_calls.empty? && res
         ([head] + body).join("\n")
       end
-      lines << format("stop    %s   %d turns, %d tool calls (%d denied), $%.4f",
+      lines << format("stop    %s   %d turns, %d tool calls (%d refused or failed), $%.4f",
                       @state.stop_reason, @turns.size, tool_calls.size, denied_calls.size,
                       @state.usage&.cost&.total.to_f)
       lines.join("\n")

data/lib/active_harness/tool.rb

@@ -43,7 +43,8 @@ module ActiveHarness
   # call passes through your checks first.
   #
   #   class LsTool < ActiveHarness::Tool
-  #     scope       :os
+  #     tool_scope  :os
+  #     tool_name   "ls"
   #     description "List files in a directory"
   #     param :path, "Directory to list"
   #
@@ -94,20 +95,22 @@ module ActiveHarness
     class << self
       # --- what the model sees ---
 
-      # Required namespace: :os, :web, :ruby ... Full name is "scope::name".
-      def scope(value = nil)
-        value ? @scope = value.to_sym : @scope
+      # Required namespace: :os, :web, :ruby ... Full name is "tool_scope::tool_name".
+      # Both are explicit on purpose: the name the model sees must not change when a class is renamed.
+      def tool_scope(value = nil)
+        value ? @tool_scope = value.to_sym : @tool_scope
       end
 
-      # Defaults to the class name without "Tool": LsTool => "ls", FetchUrlTool => "fetch_url".
+      # Required: "ls", "fetch_url" ...
       def tool_name(value = nil)
-        value ? @tool_name = value.to_s : (@tool_name || default_name)
+        value ? @tool_name = value.to_s : @tool_name
       end
 
       def full_name
-        raise ArgumentError, "#{name || 'Tool'} must declare a scope (e.g. `scope :os`)" unless scope
+        raise ArgumentError, "#{name || 'Tool'} must declare a tool_scope (e.g. `tool_scope :os`)" unless tool_scope
+        raise ArgumentError, "#{name || 'Tool'} must declare a tool_name (e.g. `tool_name \"ls\"`)" unless tool_name
 
-        "#{scope}::#{tool_name}"
+        "#{tool_scope}::#{tool_name}"
       end
 
       # Name on the wire: providers allow only [a-zA-Z0-9_-] in function names.
@@ -227,7 +230,7 @@ module ActiveHarness
 
       def inherited(subclass)
         super
-        subclass.instance_variable_set(:@scope, @scope)
+        subclass.instance_variable_set(:@tool_scope, @tool_scope)
         subclass.instance_variable_set(:@steps, steps.dup)
         subclass.instance_variable_set(:@params, params.map(&:dup))
         subclass.instance_variable_set(:@hooks, hooks.each_with_object({}) { |(k, v), h| h[k] = v.dup })
@@ -238,15 +241,6 @@ module ActiveHarness
       def call(input:, context: {})
         new(context: context).call(input)
       end
-
-      private
-
-      def default_name
-        return nil unless name
-
-        base = name.split("::").last.sub(/Tool\z/, "")
-        base.gsub(/([A-Z]+)([A-Z][a-z])/, '\1_\2').gsub(/([a-z\d])([A-Z])/, '\1_\2').downcase
-      end
     end
 
     include Core::HookRunner

data/lib/active_harness.rb

@@ -44,7 +44,7 @@ require_relative "active_harness/agent"
 require_relative "active_harness/railtie" if defined?(Rails::Railtie)
 
 module ActiveHarness
-  VERSION = "0.3.9"
+  VERSION = "0.3.10"
 
   class << self
     # Configure ActiveHarness.

data/lib/generators/active_harness/install/templates/concerns/agent_tracing.rb

@@ -29,7 +29,7 @@ module AgentTracing
 
     base.on(:complete) do |result|
       usage = result.usage
-      trace_log "#{status}/#{stop_reason} — #{turns.size} turns, #{tool_calls.size} tool calls (#{denied_calls.size} refused), " \
+      trace_log "#{status}/#{stop_reason} — #{turns.size} turns, #{tool_calls.size} tool calls (#{denied_calls.size} refused or failed), " \
                 "#{usage&.tokens&.total.to_i} tokens#{format(", $%.6f", usage.cost.total) if usage&.cost}, #{result.execution_time}s"
     end
   end

data/lib/generators/active_harness/install/templates/tools/ls_tool.rb

@@ -5,9 +5,10 @@ class LsTool < ActiveHarness::Tool
   include ActiveHarness::Tool::PathPolicy   # safe_path!: resolves symlinks and ".." and checks ALLOWED_PATHS
   include ToolTracing                        # logs every step; see app/ai/concerns/
 
-  # `scope` namespaces the tool (os::ls vs web::ls) and lets you allow or log a whole group.
-  # The tool name comes from the class (LsTool => "ls"); override with `tool_name "..."`.
-  scope :os
+  # `tool_scope` namespaces the tool (os::ls vs web::ls) and lets you allow or log a whole group;
+  # `tool_name` is the name the model sees. Both are explicit: renaming the class changes nothing for the model.
+  tool_scope :os
+  tool_name  "ls"
 
   # What the model reads when it decides whether to call this tool — write it for the model.
   description "List files in a directory inside the workspace"

data/lib/generators/active_harness/install/templates/tools/read_file_tool.rb

@@ -6,7 +6,8 @@ class ReadFileTool < ActiveHarness::Tool
   include ActiveHarness::Tool::PathPolicy
   include ToolTracing
 
-  scope       :os                          # full name: os::read_file (tool name comes from the class)
+  tool_scope  :os                          # full name: os::read_file
+  tool_name   "read_file"
   description "Read the first lines of a text file inside the workspace"
 
   param :path,  "Absolute path of the file"

data/lib/generators/active_harness/tool/templates/tool.rb.tt

@@ -1,6 +1,8 @@
 class <%= class_name %>Tool < ActiveHarness::Tool
-  # Namespaces the tool: full name is app::<%= file_name %>. Use :os, :web, :ruby ... per area.
-  scope :app
+  # Namespace + name the model sees: <%= file_name %> becomes app::<%= file_name %> (wire name app__<%= file_name %>).
+  # Both are explicit, so renaming this class never changes what the model calls. Use :os, :web, :ruby ... per area.
+  tool_scope :app
+  tool_name  "<%= file_name %>"
 
   # Written for the model: when to call this tool.
   description "Describe what this tool does for the model"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: active_harness
 version: !ruby/object:Gem::Version
-  version: 0.3.9
+  version: 0.3.10
 platform: ruby
 authors:
 - the-teacher