foobara-ai 1.0.3 → 1.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a394ba18a1ef53f37bfe3fd850907042103c0e50a34daac9faf4a2ecaab7b76d
-  data.tar.gz: bcc1f6b814d116fe46e7ffbc7f427223e6040c3e53c12d84956f7830ef0a8c18
+  metadata.gz: 97027ea41c3a72b3bc242c6aeae9e91a4d8631320c1f43e5c42cc9830df2eb04
+  data.tar.gz: 7d991b95c1ea8cf2db63fb46934d3516b23c0271dd9f452b557032916ae8b14e
 SHA512:
-  metadata.gz: 87174a0b4fb67958c07c5b783d985e3298507c64fc272fc5de3967a39edbd9f85fd3bc7fa6e5b322671f95ed90a11405c361821165c9f320091799f0a1de1b8a
-  data.tar.gz: 639426e8589599c55b382ba7cbfeee0ffca83c993b3a057ed7eacfbf8d8fa64176e08cbae66fcf35a01c677df471aecd1f38b5c6ad8ac5fe902c3c5032a856d1
+  metadata.gz: 86b3ee6a3f145e2a76580cbd96ced441d4060083126f420334e08d51f27ce22cfaea96adeba1eb39970087132dca22beb6950a1e168b44ef641d1823f0df4f93
+  data.tar.gz: 18f3281c42615b4b4f5b4c6948fd36d323e07f3e0cb1afd8a451f4439d1f17c6c5885f796a24a12ed63320b0cf0c9e45a8d4bba256ebe3cad2b75d05f3eaa746

data/CHANGELOG.md

@@ -1,3 +1,12 @@
+## [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
+
 ## [1.0.3] - 2025-10-30
 
 - Sort models by id instead of service,id for demo convenience

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

@@ -32,18 +32,18 @@ module Foobara
           @ai_services[AnswerBot::Types::ServiceEnum::OLLAMA] = OllamaApi
         end
 
-        if @ai_services.empty?
-          # :nocov:
+        if @ai_services.empty? && ENV["SKIP_AI_SERVICES"] != "true"
+          # simplecov:disable
           require "foobara/anthropic_api"
           @ai_services = { AnswerBot::Types::ServiceEnum::ANTHROPIC => AnthropicApi }
-          # :nocov:
+          # simplecov:enable
         end
 
         @ai_services
       end
 
       def default_llm_model
-        @default_llm_model ||= installed_ai_services.values.first.default_llm_model
+        @default_llm_model ||= installed_ai_services&.values&.first&.default_llm_model
       end
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foobara-ai
 version: !ruby/object:Gem::Version
-  version: 1.0.3
+  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: []