ask-agent 0.30.0 → 0.30.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 58b285b250efd49063ba2fb5c078ea3107d3ebd583ab33b4c92c07084280a242
-  data.tar.gz: 1d6df4327981ec16e652cedbb864112721db2a6ae1c1c5d7d74ecacfe4a6aa38
+  metadata.gz: cf67fcc30605c8d4752a97133a13a7fbe500c32a68c47640552bc30cf6dbea5b
+  data.tar.gz: 638931a8c1c2c5ae0ca5535e9e78e9252910106af9134366a455523925b6d895
 SHA512:
-  metadata.gz: 1adec012cf31271f1039068e6329c172981d631c221e5d18ce21fe2f19d2449bceac06a4fb4e8845b762c59b263b20ba947cd70463947b389e70f618beda501e
-  data.tar.gz: 84cdadbbf141991d3c294e06ec9f42e689835407bf3a2577d6ce8a9e173bd3eb21030313e0a59b3663f6718bee88b245deab144aaeda440e443ae176877af4cb
+  metadata.gz: 01ca01b502aa5789eef910f50626085bb6675e86dedd435b189b0e928d592ac27c48c145ff5c159580c2a9854528160c86074031dccd971b033b75b6f42bf3fa
+  data.tar.gz: 6bf3ebc3428cf64c8a1ee7e958e45c349669e4eaba92a8bc5d90b522f0b9a47d4f3743570db235b8d9b4e526d3ea470eb1a81795acc6492fd67afda4b59d9b70

data/CHANGELOG.md

@@ -1,3 +1,19 @@
+## [0.30.1] — 2026-08-06
+
+### Fixed
+
+- **Framework-injected tools are no longer persisted in session metadata.**
+  The built-in `load_skill` tool was saved alongside user tools and could
+  not be auto-instantiated on load (`LoadSkillTool` requires a registry), so
+  the previous 0.30.0 fix skipped it with a broad rescue. Persisted metadata
+  now contains only user-supplied tools (`persisted_tools`); `load_skill` is
+  re-added by `resolve_tools` per session with a proper registry.
+- **`Session.load` no longer swallows tool-restore failures silently.** The
+  safety net now rescues only `NameError` (renamed/removed classes) and
+  `ArgumentError` (constructors with required args), and warns with the
+  tool name instead of failing the whole load — genuine tool bugs surface
+  instead of disappearing.
+
 ## [0.30.0] — 2026-08-06
 
 ### Added

data/lib/ask/agent/session.rb

@@ -314,14 +314,16 @@ module Ask
         session = new(
           id: data[:id],
           model: data.dig(:metadata, :model),
-          # Instantiate saved tool classes defensively: tools that cannot be
-          # auto-constructed (e.g. the built-in LoadSkillTool, which needs a
-          # registry) are skipped — resolve_tools re-adds them with a proper
-          # registry. Callers can also pass their own `tools:` after load.
+          # Restore saved user tools by class name. Tools that cannot be
+          # restored — renamed/removed classes (NameError) or constructors
+          # with required args (ArgumentError) — are skipped with a warning
+          # instead of failing the whole load; resolve_tools re-adds the
+          # framework-injected load_skill tool with a proper registry.
           tools: data.dig(:metadata, :tools).to_a.filter_map do |name|
             begin
               name.constantize.new
-            rescue StandardError
+            rescue NameError, ArgumentError => e
+              warn "[ask-agent] Session.load skipped tool '#{name}': #{e.class}: #{e.message}"
               nil
             end
           end,
@@ -694,6 +696,15 @@ module Ask
         target.instance_variable_set(:@turn_count, data.dig(:metadata, :turn_count) || 0)
       end
 
+      # User-supplied tools only. Framework-injected tools (the built-in
+      # load_skill tool) are re-created by resolve_tools on every session, so
+      # persisting them would leak framework internals into user data — and
+      # they cannot be auto-instantiated on load anyway (LoadSkillTool needs
+      # a registry).
+      def persisted_tools
+        @tools.reject { |t| t.is_a?(Ask::Skills::LoadSkillTool) }
+      end
+
       def persist!
         payload = {
           id: @id,
@@ -707,7 +718,7 @@ module Ask
           },
           metadata: {
             model: @chat.model.respond_to?(:id) ? @chat.model.id : @chat.model,
-            tools: @tools.map { |t| t.class.name },
+            tools: persisted_tools.map { |t| t.class.name },
             max_turns: @max_turns,
             turn_count: @turn_count,
             created_at: @created_at.iso8601,

data/lib/ask/agent/version.rb

@@ -2,6 +2,6 @@
 
 module Ask
   module Agent
-    VERSION = "0.30.0"
+    VERSION = "0.30.1"
   end
 end

metadata

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