foobara-ai 1.0.4 → 1.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 11ae5fa1e4405619dbdd89d232509e5ef06926f295320337d2c609943b3b7763
-  data.tar.gz: d0e7922f19cfcdf327fc99ae2a7bd8f4913a6ba3b2e2cd86fa6bad6197acf4c5
+  metadata.gz: 97027ea41c3a72b3bc242c6aeae9e91a4d8631320c1f43e5c42cc9830df2eb04
+  data.tar.gz: 7d991b95c1ea8cf2db63fb46934d3516b23c0271dd9f452b557032916ae8b14e
 SHA512:
-  metadata.gz: a377b038639fa8386826e4299ed20e21f6a03d3fd554cf063c6ce19f85f85692176e6c8fe9481ebc3914927f1da3c3b1b159bc036f4c1d048f8fac501aa28dd0
-  data.tar.gz: 0e449e9c8d756045309af013bdf7e73065859885e343fd38919d32940d6259f461028333f80b44331c928313dd9a3e2bfb8ac7695a0a93841fde77f4da5a03d7
+  metadata.gz: 86b3ee6a3f145e2a76580cbd96ced441d4060083126f420334e08d51f27ce22cfaea96adeba1eb39970087132dca22beb6950a1e168b44ef641d1823f0df4f93
+  data.tar.gz: 18f3281c42615b4b4f5b4c6948fd36d323e07f3e0cb1afd8a451f4439d1f17c6c5885f796a24a12ed63320b0cf0c9e45a8d4bba256ebe3cad2b75d05f3eaa746

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+## [1.0.5] - 2026-09-22
+
+- Fix generate next message domain mapper selection problems using domain-specific model_enum
+- Make temperature optional/removed to deal with API changes/complications added to the API by Anthropic and OpenAI
+
 ## [1.0.4] - 2025-11-10
 
 - Allow disabling automatic loading of AI services

data/src/foobara/ai/answer_bot/domain_mappers/anthropic_api/chat_to_create_message.rb

@@ -6,10 +6,13 @@ module Foobara
       module DomainMappers
         module AnthropicApi
           class ChatToCreateMessage < Foobara::DomainMapper
+            # TODO: why doesn't this have a model?
             from do
               chat Types::Chat, :required
-              temperature :float
-              model :model_enum
+              # NOTE: temperature is not supported by Anthropic these days and it is ignored
+              temperature :float, :allow_nil, "temperature is not supported by Anthropic these days and it is ignored"
+              # TODO: shouldn't this be in Types prefix??
+              model :"Foobara::Ai::AnthropicApi::model_enum"
             end
             to Foobara::Ai::AnthropicApi::CreateMessage
 
@@ -26,10 +29,6 @@ module Foobara
                 inputs[:system] = system_messages.map(&:content).join("\n")
               end
 
-              if temperature
-                inputs[:temperature] = temperature
-              end
-
               if model
                 inputs[:model] = model
               end
@@ -41,10 +40,6 @@ module Foobara
               from[:chat]
             end
 
-            def temperature
-              from[:temperature]
-            end
-
             def model
               from[:model]
             end

data/src/foobara/ai/answer_bot/domain_mappers/model_to_ai_service.rb

@@ -16,9 +16,9 @@ module Foobara
               return service if domain::Types::ModelEnum.value?(symbol)
             end
 
-            # :nocov:
+            # simplecov:disable
             raise "Unknown model: #{model}"
-            # :nocov:
+            # simplecov:enable
           end
 
           alias model from

data/src/foobara/ai/answer_bot/domain_mappers/ollama_api/chat_to_generate_chat_completion.rb

@@ -8,8 +8,9 @@ module Foobara
           class ChatToGenerateChatCompletion < Foobara::DomainMapper
             from do
               chat Types::Chat, :required
-              temperature :float
-              model :model_enum
+              temperature :float, :allow_nil
+              # TODO: shouldn't this be in Types prefix??
+              model :"Foobara::Ai::OllamaApi::model_enum"
             end
             to Foobara::Ai::OllamaApi::GenerateChatCompletion
 

data/src/foobara/ai/answer_bot/domain_mappers/open_ai_api/chat_to_generate_chat_completion.rb

@@ -8,8 +8,12 @@ module Foobara
           class ChatToGenerateChatCompletion < Foobara::DomainMapper
             from do
               chat Types::Chat, :required
-              temperature :float
-              model :model_enum
+              # NOTE: ignoring temperature for now to prevent breakage
+              # TODO: either figure out a deterministic way to know if temperature should be dropped
+              # or a way to react to the error with a retry when it happens.
+              temperature :float, :allow_nil
+              # TODO: shouldn't this be in Types prefix??
+              model :"Foobara::Ai::OpenAiApi::model_enum"
             end
             to Foobara::Ai::OpenAiApi::CreateChatCompletion
 
@@ -20,11 +24,11 @@ module Foobara
                 end
               }
 
-              if temperature
-                # NOTE: some models don't support temperature, like o1. Instead of possibly receiving an error
-                # we will set the temperature to 1 which prevents a 400. Maybe print a warning instead?
-                inputs[:temperature] = model_supports_temperature? ? temperature : 1
-              end
+              # if temperature
+              #   # NOTE: some models don't support temperature, like o1. Instead of possibly receiving an error
+              #   # we will set the temperature to 1 which prevents a 400. Maybe print a warning instead?
+              #   inputs[:temperature] = model_supports_temperature? ? temperature : 1
+              # end
 
               if model
                 inputs[:model] = model
@@ -37,17 +41,17 @@ module Foobara
               from[:chat]
             end
 
-            def temperature
-              from[:temperature]
-            end
+            # def temperature
+            #   from[:temperature]
+            # end
 
             def model
               from[:model]
             end
 
-            def model_supports_temperature?
-              model && model.to_s !~ /\Ao[13](-|$)/
-            end
+            # def model_supports_temperature?
+            #   model && model.to_s !~ /\Ao[13](-|$)/
+            # end
           end
         end
       end

data/src/foobara/ai/answer_bot/domain_mappers/service_to_command.rb

@@ -18,9 +18,9 @@ module Foobara
             when Types::ServiceEnum::OLLAMA
               Ai::OllamaApi::GenerateChatCompletion
             else
-              # :nocov:
+              # simplecov:disable
               raise "Unknown service: #{service}"
-              # :nocov:
+              # simplecov:enable
             end
           end
 

data/src/foobara/ai/answer_bot/domain_mappers/service_to_list_models_command.rb

@@ -16,9 +16,9 @@ module Foobara
             when Types::ServiceEnum::OLLAMA
               Ai::OllamaApi::ListLocalModels
             else
-              # :nocov:
+              # simplecov:disable
               raise "Unknown service: #{service}"
-              # :nocov:
+              # simplecov:enable
             end
           end
 

data/src/foobara/ai/answer_bot/generate_next_message.rb

@@ -8,7 +8,7 @@ module Foobara
           chat Types::Chat, :required
           service :service_enum, "If two services expose the same model, you can specify which one to use."
           model :model_enum, default: Ai.default_llm_model, description: "The model to use."
-          temperature :float
+          temperature :float, :allow_nil
         end
 
         result Types::Message

data/src/foobara/ai/answer_bot/types/model_enum.rb

@@ -28,11 +28,11 @@ module Foobara
         #         models = threads.each(&:join).map(&:value).flatten
 
         unless models.uniq == models
-          # :nocov:
+          # simplecov:disable
           # TODO: we should return something more complicated than a bare list of strings. Either a more
           # robust object or prefix strings with the relevant service to handle collisions.
           raise "Duplicate models detected"
-          # :nocov:
+          # simplecov:enable
         end
 
         ModelEnum = Foobara::Enumerated.make_module(models)

data/src/foobara/ai.rb

@@ -33,10 +33,10 @@ module Foobara
         end
 
         if @ai_services.empty? && ENV["SKIP_AI_SERVICES"] != "true"
-          # :nocov:
+          # simplecov:disable
           require "foobara/anthropic_api"
           @ai_services = { AnswerBot::Types::ServiceEnum::ANTHROPIC => AnthropicApi }
-          # :nocov:
+          # simplecov:enable
         end
 
         @ai_services

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foobara-ai
 version: !ruby/object:Gem::Version
-  version: 1.0.4
+  version: 1.0.5
 platform: ruby
 authors:
 - Miles Georgi
@@ -72,7 +72,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.9
+rubygems_version: 4.0.20
 specification_version: 4
 summary: No description. Add one.
 test_files: []