ask-agent 0.25.2 → 0.25.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d6d5dd31344228811bd4c7993547c9e0f9d7c39c779f58250d82ad95f271bd7d
-  data.tar.gz: ba08c57e9fc7dfc410e6a835ffab1076d1f4b5704bee866f7a38e158da712d34
+  metadata.gz: 3703e22ceaf07962231c1cc08751e635d202a3dcb2bf7a0134ee3c1acbeeb975
+  data.tar.gz: d013b9b6a4d188dc71dd6c267388c1b36d83be5b43c0967d73b8cb10843d2123
 SHA512:
-  metadata.gz: 5d46b653b22b50dee9597238cc65ce5fe72c3840322e1ecd03efccfe6494a4494bff473f3bc3c4eff1ae56df849142a45f52bc77e23c9a93a9c240f465d7db68
-  data.tar.gz: 9c158ab0aa0be906723681458ea54d90b15cd7289647e7acb8527faf367ead4490919a84b0a2e71186b4b93945ac065db063a33ead26b2532d0d50efd2dd8bf3
+  metadata.gz: 7bf19be96f3344b84b0c8ee7c2079e3eb1ad0a2655c26e5075652a863f66c1c47816d6b4418ba3f29b79b3722e8cf924bedac9aa9965da4099009b4993877b7b
+  data.tar.gz: f214e8356c0e55be0aba092d81c0a0ac79d57b79abf41ce1f2e59db45a73a8ac51fcd0566d6335931a4d68292cba894b3240d4dbda4175c9238ba6f7fc7d4639

data/CHANGELOG.md

@@ -1,3 +1,33 @@
+## [0.25.4] - 2026-08-05
+
+### Added
+
+- **`skills_disclosure` opt-out for progressive skill disclosure.** Sessions
+  auto-inject the `load_skill` tool by default; agents with a fixed tool
+  surface (e.g. voice receptionists) can declare `skills_disclosure false`
+  in their definition (or pass `skills_disclosure: false` to
+  `Session.new`) to keep the tool payload minimal and deterministic.
+
+## [0.25.3] - 2026-08-04
+
+### Fixed
+
+- **Tools now respect the session's `parallel_tools` setting.** The agent
+  loop dispatched tool calls straight to `execute_parallel`, so tools always
+  ran in worker threads — even with `parallel_tools: false`. Sequential
+  sessions now run tools in the caller thread, which is what frameworks
+  like Rails rely on for per-request context (`CurrentAttributes` are
+  thread-local). The loop calls `ToolExecutor#execute`, which honors the
+  executor's `parallel` flag.
+- **Parallel tool threads inherit the caller's thread-local state.** When
+  tools do run in threads (parallel mode), `execute_parallel` copies the
+  caller's `Thread.current` locals into each worker thread first, so
+  per-request context (Rails `CurrentAttributes`, log tags, etc.) reaches
+  the tools instead of being nil.
+- `ToolExecutor#execute` accepts a `result_callback:` kwarg (invoked per
+  completed tool in both sequential and parallel modes); sequential
+  execution reports results through it too, matching parallel behavior.
+
 ## [0.25.2] - 2026-08-03
 
 ### Fixed

data/lib/ask/agent/context_source.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+require "time"
 
 module Ask
   module Agent

data/lib/ask/agent/context_sources.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+require "time"
 
 module Ask
   module Agent

data/lib/ask/agent/definition.rb

@@ -92,6 +92,18 @@ module Ask
           end
         end
 
+        # Set or get the progressive skill disclosure flag. When enabled
+        # (default), sessions include the load_skill tool so the model can
+        # pull in skills on demand. Voice/UI agents with a fixed tool
+        # surface can disable it to keep the payload minimal.
+        def skills_disclosure(value = :__no_value__)
+          if value == :__no_value__
+            _config.key?(:skills_disclosure) ? _config[:skills_disclosure] : true
+          else
+            _config[:skills_disclosure] = value
+          end
+        end
+
         # Set an arbitrary Session option. Accepts any key that
         # Ask::Agent::Session.new understands.
         #

data/lib/ask/agent/extensions/audit_log/active_record_writer.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+require "time"
 
 module Ask
   module Agent

data/lib/ask/agent/extensions/audit_log.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require "json"
+require "time"
 
 module Ask
   module Agent

data/lib/ask/agent/loop.rb

@@ -79,12 +79,17 @@ module Ask
 
         if user_tool_calls.any?
           # Execute user tool calls locally
-          user_results = tool_executor.execute_parallel(
-            user_tool_calls, tools, hooks, event_emitter, ToolAbortController.new
-          ) do |tool_call_id, result|
-            tc = user_tool_calls[tool_call_id]
-            chat.add_message(role: :tool, content: result[:message].to_s, tool_call_id: tool_call_id) if tc
-          end
+          # Respect the session's parallel_tools setting: parallel tools run
+          # in threads (with the caller's thread-local context inherited);
+          # sequential tools run in the caller thread so per-request context
+          # (e.g. Rails CurrentAttributes) is visible without any copying.
+          user_results = tool_executor.execute(
+            user_tool_calls, tools, hooks: hooks, event_emitter: event_emitter,
+            result_callback: lambda do |tool_call_id, result|
+              tc = user_tool_calls[tool_call_id]
+              chat.add_message(role: :tool, content: result[:message].to_s, tool_call_id: tool_call_id) if tc
+            end
+          )
           all_tool_results.concat(user_results)
         end
 

data/lib/ask/agent/meta_agent.rb

@@ -2,6 +2,7 @@
 
 require "set"
 require "json"
+require "time"
 
 module Ask
   module Agent

data/lib/ask/agent/session.rb

@@ -21,12 +21,14 @@ module Ask
                      compactor: nil, hooks: {}, state: nil, persistence: nil,
                      id: nil, system_prompt: nil, parallel_tools: true,
                      reflector: nil, telemetry: true, meta_agent: nil,
-                     agent_dir: nil, evaluator: nil, audit_log: nil, **chat_options)
+                     agent_dir: nil, evaluator: nil, audit_log: nil,
+                     skills_disclosure: true, **chat_options)
         @id = id || SecureRandom.uuid
         @agent_dir = agent_dir
         @max_turns = max_turns
         @max_tool_retries = max_tool_retries
         @parallel_tools = parallel_tools
+        @skills_disclosure = skills_disclosure
         @event_handlers = { all: [] }
         @running = false
         @deleted = false
@@ -356,13 +358,20 @@ module Ask
           tool.is_a?(Class) ? tool.new : tool
         end
         # Always include the load_skill tool for progressive skill disclosure,
-        # unless the test framework is loaded (test mode keeps tools deterministic)
-        unless defined?(Ask::Agent::Test) && Ask::Agent::Test
+        # unless disabled by the agent (voice agents keep a minimal tool
+        # surface) or the test framework is loaded (test mode keeps tools
+        # deterministic)
+        if skills_disclosure_enabled?
           resolved << Skills::LoadSkillTool.new(registry: @skills_registry) unless resolved.any? { |t| t.name == "load_skill" }
         end
         resolved
       end
 
+      # Whether progressive skill disclosure is active for this session.
+      def skills_disclosure_enabled?
+        @skills_disclosure && !(defined?(Ask::Agent::Test) && Ask::Agent::Test)
+      end
+
       def build_compactor(config)
         global = Ask::Agent.configuration
         compactor = Compactor.new(

data/lib/ask/agent/telemetry.rb

@@ -3,6 +3,7 @@
 require "fileutils"
 require "json"
 require "securerandom"
+require "time"
 
 module Ask
   module Agent

data/lib/ask/agent/tool_executor.rb

@@ -19,7 +19,7 @@ module Ask
 
       attr_writer :telemetry
 
-      def execute(tool_calls, tools, hooks:, event_emitter:, session_id: nil)
+      def execute(tool_calls, tools, hooks:, event_emitter:, session_id: nil, result_callback: nil)
         return [] if tool_calls.empty?
 
         @total_executions = 0
@@ -27,9 +27,11 @@ module Ask
         sibling_abort = ToolAbortController.new
 
         if @parallel
-          execute_parallel(tool_calls, tools, hooks, event_emitter, sibling_abort)
+          execute_parallel(tool_calls, tools, hooks, event_emitter, sibling_abort, &result_callback)
         else
-          execute_sequential(tool_calls, tools, hooks, event_emitter, sibling_abort)
+          execute_sequential(tool_calls, tools, hooks, event_emitter, sibling_abort) do |id, result|
+            result_callback&.call(id, result)
+          end
         end
       end
 
@@ -38,8 +40,15 @@ module Ask
         mutex = Mutex.new
         results = {}
 
+        # Inherit the caller's thread-local state (Rails CurrentAttributes
+        # and similar frameworks store per-request context in Thread.current)
+        # so tools see the same context they would in a sequential run.
+        inherited_locals = {}
+        Thread.current.keys.each { |key| inherited_locals[key] = Thread.current[key] }
+
         tool_calls.each do |id, tool_call|
           threads << Thread.new do
+            inherited_locals.each { |key, value| Thread.current[key] = value }
             begin
               if sibling_abort.aborted?
                 mutex.synchronize { results[id] = aborted_result(tool_call) }
@@ -75,13 +84,14 @@ module Ask
         tool_calls.keys.map { |id| results[id] }.compact
       end
 
-      def execute_sequential(tool_calls, tools, hooks, event_emitter, sibling_abort)
+      def execute_sequential(tool_calls, tools, hooks, event_emitter, sibling_abort, &result_callback)
         results = []
         tool_calls.each do |id, tool_call|
           break if sibling_abort.aborted?
 
           result = execute_single_tool(tool_call, tools, hooks, event_emitter, sibling_abort)
           results << result
+          result_callback&.call(id, result)
           break if result[:critical_failure]
           break if result[:halted]
         end

data/lib/ask/agent/version.rb

@@ -2,6 +2,6 @@
 
 module Ask
   module Agent
-    VERSION = "0.25.2"
+    VERSION = "0.25.4"
   end
 end

data/lib/ask/agent.rb

@@ -170,6 +170,7 @@ module Ask
         session_opts[:provider] = config[:provider] if config[:provider]
         session_opts[:max_turns] = config[:max_turns] if config[:max_turns]
         session_opts[:parallel_tools] = config[:parallel_tools] if config.key?(:parallel_tools)
+        session_opts[:skills_disclosure] = config[:skills_disclosure] if config.key?(:skills_disclosure)
 
         # Pass arbitrary session options
         if config[:options]

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ask-agent
 version: !ruby/object:Gem::Version
-  version: 0.25.2
+  version: 0.25.4
 platform: ruby
 authors:
 - Kaka Ruto