Remove check about feature being enabled in AI entrypoint module

Author: nelsonkopliku

lib/trento/ai.ex

@@ -19,26 +19,16 @@ defmodule Trento.AI do
 
   See `Trento.AI.Configurations.create_user_configuration/2` for more details.
   """
-  def create_user_configuration(user, attrs) do
-    execute_if_enabled(fn -> configurations().create_user_configuration(user, attrs) end)
-  end
+  def create_user_configuration(user, attrs),
+    do: configurations().create_user_configuration(user, attrs)
 
   @doc """
   Updates a user configuration for AI.
 
   See `Trento.AI.Configurations.update_user_configuration/2` for more details.
   """
-  def update_user_configuration(user, attrs) do
-    execute_if_enabled(fn -> configurations().update_user_configuration(user, attrs) end)
-  end
-
-  defp execute_if_enabled(function) do
-    if enabled?() do
-      function.()
-    else
-      {:error, :ai_features_disabled}
-    end
-  end
+  def update_user_configuration(user, attrs),
+    do: configurations().update_user_configuration(user, attrs)
 
   defp configurations,
     do: Keyword.get(ApplicationConfigLoader.load(), :configurations, Configurations)

lib/trento/ai/user_configuration.ex

@@ -35,18 +35,10 @@ defmodule Trento.AI.UserConfiguration do
     |> foreign_key_constraint(:user_id, message: "User does not exist")
   end
 
-  defp maybe_update_provider(attrs) do
-    case Map.has_key?(attrs, :model) do
-      true -> Map.put(attrs, :provider, get_model_provider(attrs))
-      false -> attrs
-    end
-  end
+  defp maybe_update_provider(%{model: model} = attrs),
+    do: Map.put(attrs, :provider, LLMRegistry.get_model_provider(model))
 
-  defp get_model_provider(attrs) do
-    attrs
-    |> Map.get(:model)
-    |> LLMRegistry.get_model_provider()
-  end
+  defp maybe_update_provider(attrs), do: attrs
 
   defp validate_model(_model_field_atom, model) do
     if LLMRegistry.model_supported?(model) do

test/trento/ai_test.exs

@@ -27,22 +27,9 @@ defmodule Trento.AITest do
     end
   end
 
-  describe "enabling/disabling features" do
-    test "features are disabled" do
-      expect(Trento.AI.ApplicationConfigLoader.Mock, :load_config, 2, fn -> [enabled: false] end)
-      user = build(:user)
-
-      attrs = %{
-        model: "foo",
-        api_key: "bar"
-      }
-
-      assert AI.create_user_configuration(user, attrs) == {:error, :ai_features_disabled}
-      assert AI.update_user_configuration(user, attrs) == {:error, :ai_features_disabled}
-    end
-
-    test "features are enabled" do
-      expect(Trento.AI.ApplicationConfigLoader.Mock, :load_config, 4, fn ->
+  describe "delegating creation and update to configurations module" do
+    test "should delegate to configurations module" do
+      expect(Trento.AI.ApplicationConfigLoader.Mock, :load_config, 2, fn ->
         [
           enabled: true,
           configurations: DummyConfigurations