ask-decisions 0.2.3 → 0.2.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e94c483511119929cad9bbdf2d0925ca498e4b4f98f1119c05ac3c26a239edfc
-  data.tar.gz: e09632e3fbaa5f2dca5ffc8692afe586d1818b985db8afaa600f3498e614ab9f
+  metadata.gz: e64f40bca45e7b9bc9a11f33fabe1e178c30cfd9e61f4460395b77eb8092cc5a
+  data.tar.gz: 4e32792a9c4bdc411731beb94d9a078e98d92f3135aaa4da779090fd6c4f96c9
 SHA512:
-  metadata.gz: a55fb6e583a983afd05acde0048d203d2e81c3098ef3cfa9d278214a3e4cd2de47ba2bd0f78fcb572f078da1c246fc4dc2aa22f2d23380cd79dc7d5c5fdbfe70
-  data.tar.gz: 57e4a24f195eb27e518e88e7340850e27a90499c2a22ac5a3069cca63d787fee3623f350ccf893fb0794f4eb13667656fd82abff369b9ce71c0c62b30f1a0490
+  metadata.gz: e6338ccd1b049f19612e7b33d4e7dbedce5192d4789c29ac62bf6fa075107fa7184764b71c3df9e0edc6284e8d9ed0013225d5ee84a6be9858b5d3ca49d2b874
+  data.tar.gz: f9c69aca54eaae484d182ab78c7265de776fafff15e5b4813908028dac1857e8df4f85428fc9474282dd54b4418a6c9981e8cb5e76673f028593daff8be3102b

data/CHANGELOG.md

@@ -4,6 +4,17 @@ All notable changes to this project will be documented in this file.
 
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
+## [0.2.4] - 2026-09-18
+
+### Fixed
+
+- **`resolve_provider` memoizes the provider instance.** Every call to
+  `resolve_provider` previously created a new `Typesafe` (or other provider)
+  object. In a pipeline with 10+ decision calls per cycle, this meant 10+
+  redundant allocations with identical configuration. The provider is now
+  cached for the lifetime of the process — it holds only read-only state
+  (API key, base URL, model, timeout) set once at boot.
+
 ## [0.2.3] - 2026-09-18
 
 ### Added

data/lib/ask/decisions/compactor.rb

@@ -145,6 +145,8 @@ module Ask
       end
 
       # First message and the most recent +preserve_recent+ messages are pinned.
+      # Pinned messages are never removed, but their tool calls are still
+      # evaluated by Jev — a recent tool result can still be irrelevant.
       def compute_pinned(messages)
         pinned = Set.new([0])
         start = [messages.size - @preserve_recent, 1].max
@@ -152,6 +154,11 @@ module Ask
         pinned
       end
 
+      # Whether a message index is safe to remove (not pinned).
+      def removable?(idx, pinned)
+        !pinned.include?(idx)
+      end
+
       # ── State ────────────────────────────────────────────────────────
 
       # Build the state Jev sees: full conversation with tool results replaced
@@ -162,7 +169,6 @@ module Ask
         pairs.each { |p| result_index[p[:result_msg_idx]] = p }
 
         lines = messages.each_with_index.map do |msg, idx|
-          role = msg[:role] || "unknown"
           content = msg[:content].to_s
 
           if result_index[idx]
@@ -180,7 +186,6 @@ module Ask
           end
         end
 
-        goal_section = @goal ? "\n\nGoal: #{@goal}" : ""
         { conversation: lines, goal: @goal }.compact
       end
 
@@ -195,6 +200,8 @@ module Ask
       def build_questions(pairs, pinned)
         questions = {}
         pairs.each do |pair|
+          # Skip pairs where both the call and result live in pinned messages.
+          # Pinned messages are never touched — Jev does not score them.
           next if pinned.include?(pair[:call_msg_idx]) && pinned.include?(pair[:result_msg_idx])
 
           call_id = pair[:call_id]
@@ -312,8 +319,13 @@ module Ask
 
           case decision[:action]
           when :drop
-            remove_indices.add(pair[:call_msg_idx])
-            remove_indices.add(pair[:result_msg_idx])
+            # Only remove messages that are not pinned.
+            if removable?(pair[:call_msg_idx], pinned)
+              remove_indices.add(pair[:call_msg_idx])
+            end
+            if removable?(pair[:result_msg_idx], pinned)
+              remove_indices.add(pair[:result_msg_idx])
+            end
           when :truncate
             truncate_indices[pair[:result_msg_idx]] = pair[:call]
           end
@@ -342,7 +354,7 @@ module Ask
       # ── Stats ────────────────────────────────────────────────────────
 
       def compute_stats(original, pruned, decisions)
-        action_counts = decisions.values.tally { |d| d[:action] }
+        action_counts = decisions.values.map { |d| d[:action] }.tally
         original_chars = original.sum { |m| m[:content].to_s.length }
         pruned_chars = pruned.sum { |m| m[:content].to_s.length }
 

data/lib/ask/decisions/version.rb

@@ -2,6 +2,6 @@
 
 module Ask
   module Decisions
-    VERSION = "0.2.3"
+    VERSION = "0.2.4"
   end
 end

data/lib/ask-decisions.rb

@@ -122,13 +122,19 @@ module Ask
       end
 
       # Resolve a provider by name, raising on unknown.
+      # Memoized: the provider is stateless after boot (API key,
+      # base URL, model, timeout are all read-only), so one instance
+      # per configuration lifetime is safe and avoids redundant
+      # allocations across the pipeline.
       def resolve_provider(name)
-        klass = Ask::DecisionProvider.resolve(name)
-
-        if configuration.provider_options.any?
-          klass.new(**configuration.provider_options)
-        else
-          klass.new
+        @resolved_provider ||= begin
+          klass = Ask::DecisionProvider.resolve(name)
+
+          if configuration.provider_options.any?
+            klass.new(**configuration.provider_options)
+          else
+            klass.new
+          end
         end
       end
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ask-decisions
 version: !ruby/object:Gem::Version
-  version: 0.2.3
+  version: 0.2.4
 platform: ruby
 authors:
 - Kaka Ruto