ask-agent 0.29.1 → 0.30.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7b6eca46c976555990a13366053665b9340f4a97d578a332a8cbf9615cd5647d
-  data.tar.gz: 7c16e54008a44cc0eafb8ebfea51c752e0d6879749635e606142f84cdd03d682
+  metadata.gz: cf67fcc30605c8d4752a97133a13a7fbe500c32a68c47640552bc30cf6dbea5b
+  data.tar.gz: 638931a8c1c2c5ae0ca5535e9e78e9252910106af9134366a455523925b6d895
 SHA512:
-  metadata.gz: 05eb5c72881fcb3730dab2f89d51444b3f86f41413f454090331d5c9bb240b6ca4751c40f0fdc3ef10f2ea718989aa7e68e716e13570e247a094f76ae0702051
-  data.tar.gz: 712139e5f89850fe706268b74a08854c2f1f0be7276e4717039c73192a872d6f4a3c2566954ae4133ce10917d51ea4ae5687995b828ef024bead95a8222ac68d
+  metadata.gz: 01ca01b502aa5789eef910f50626085bb6675e86dedd435b189b0e928d592ac27c48c145ff5c159580c2a9854528160c86074031dccd971b033b75b6f42bf3fa
+  data.tar.gz: 6bf3ebc3428cf64c8a1ee7e958e45c349669e4eaba92a8bc5d90b522f0b9a47d4f3743570db235b8d9b4e526d3ea470eb1a81795acc6492fd67afda4b59d9b70

data/CHANGELOG.md

@@ -1,3 +1,52 @@
+## [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
+
+- **Session checkpoints: fork, rollback, resume.** Versioned, durable
+  checkpoints on any state adapter. Enable with
+  `Session.new(state: store, checkpoints: true)` — every turn is snapshotted
+  under a sequential key, and the session gains time travel:
+  - **`Session#rollback!(seq: / turn:)`** — rewind messages and turn count
+    to an earlier checkpoint. Later checkpoints are kept, so the session
+    can roll forward again. The legacy blob stays consistent.
+  - **`Session#fork(at_seq: / at_turn:)`** — a new session (new id, same
+    model/tools) whose history is everything up to that point, backed by
+    its own checkpoint chain; continue the branch with `run`.
+  - **`Session#checkpoint_history` / `Session#load_checkpoint(seq:)`** —
+    inspect the timeline. `Session.load` re-enables checkpointing
+    automatically when the stored session has checkpoints.
+  - `Ask::Agent::CheckpointStore` — the store itself, usable standalone.
+    It needs only the minimal KV contract (`get`/`set`/`delete`), so it
+    works with every state provider (SQLite, Redis, Postgres, MySQL) and
+    custom adapters — no list primitives required, no provider mandated.
+  - `Events::SessionRolledBack` / `Events::SessionForked` fire on rollback
+    and fork. `Session#delete` cleans up checkpoint keys too.
+
+### Fixed
+
+- **`Session.load` could not restore sessions saved with tools.** The
+  built-in `LoadSkillTool` (auto-added to every session) cannot be
+  instantiated without a registry, so any saved session with tools failed
+  to load with `ArgumentError: missing keyword: :registry`. Load now
+  instantiates saved tool classes defensively — un-instantiable classes are
+  skipped and re-added by `resolve_tools` with a proper registry.
+
 ## [0.29.1] — 2026-08-06
 
 ### Fixed

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.29.1"
+    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.29.1
+  version: 0.30.1
 platform: ruby
 authors:
 - Kaka Ruto