Improve naming consistency

Author: nelsonkopliku

lib/trento/ai/llm_registry.ex

@@ -1,4 +1,4 @@
-defmodule Trento.AI.LlmRegistry do
+defmodule Trento.AI.LLMRegistry do
   @moduledoc """
   This module is responsible for managing the registry of available LLM providers and their models.
   """

lib/trento/ai/user_configuration.ex

@@ -6,14 +6,14 @@ defmodule Trento.AI.UserConfiguration do
 
   alias Trento.Support.Ecto.EncryptedBinary
 
-  alias Trento.AI.LlmRegistry
+  alias Trento.AI.LLMRegistry
 
   @type t :: %__MODULE__{}
 
   @primary_key false
   schema "ai_configurations" do
     field :model, :string
-    field :provider, Ecto.Enum, values: LlmRegistry.providers()
+    field :provider, Ecto.Enum, values: LLMRegistry.providers()
     field :api_key, EncryptedBinary, redact: true
 
     belongs_to :user, Trento.Users.User, primary_key: true
@@ -45,11 +45,11 @@ defmodule Trento.AI.UserConfiguration do
   defp get_model_provider(attrs) do
     attrs
     |> Map.get(:model)
-    |> LlmRegistry.get_model_provider()
+    |> LLMRegistry.get_model_provider()
   end
 
   defp validate_model(_model_field_atom, model) do
-    if LlmRegistry.model_supported?(model) do
+    if LLMRegistry.model_supported?(model) do
       []
     else
       [model: {"is not supported", validation: :ai_model_validity}]

test/support/factory.ex

@@ -182,7 +182,7 @@ defmodule Trento.Factory do
 
   alias TrentoWeb.Auth.PersonalAccessToken, as: PAT
 
-  alias Trento.AI.{LlmRegistry, UserConfiguration}
+  alias Trento.AI.{LLMRegistry, UserConfiguration}
 
   use ExMachina.Ecto, repo: Trento.Repo
 
@@ -1463,7 +1463,7 @@ defmodule Trento.Factory do
 
   def random_ai_model do
     :all
-    |> LlmRegistry.get_provider_models()
+    |> LLMRegistry.get_provider_models()
     |> Enum.random()
   end
 

test/trento/ai/configurations_test.exs

@@ -3,7 +3,7 @@ defmodule Trento.Ai.ConfigurationsTest do
 
   alias Trento.Users.User
 
-  alias Trento.AI.{Configurations, LlmRegistry, UserConfiguration}
+  alias Trento.AI.{Configurations, LLMRegistry, UserConfiguration}
 
   alias Trento.Factory
 
@@ -194,7 +194,7 @@ defmodule Trento.Ai.ConfigurationsTest do
       model = Factory.random_ai_model()
       api_key = Faker.String.base64()
 
-      expected_provider = LlmRegistry.get_model_provider(model)
+      expected_provider = LLMRegistry.get_model_provider(model)
 
       assert {:ok,
               %UserConfiguration{
@@ -354,7 +354,7 @@ defmodule Trento.Ai.ConfigurationsTest do
 
       new_model = Factory.random_ai_model()
 
-      expected_provider = LlmRegistry.get_model_provider(new_model)
+      expected_provider = LLMRegistry.get_model_provider(new_model)
 
       assert {:ok,
               %UserConfiguration{
@@ -378,7 +378,7 @@ defmodule Trento.Ai.ConfigurationsTest do
       new_model = Factory.random_ai_model()
       new_api_key = Faker.String.base64()
 
-      expected_provider = LlmRegistry.get_model_provider(new_model)
+      expected_provider = LLMRegistry.get_model_provider(new_model)
 
       assert {:ok,
               %UserConfiguration{

test/trento/ai/llm_registry_test.exs

@@ -1,7 +1,7 @@
-defmodule Trento.AI.LlmRegistryTest do
+defmodule Trento.AI.LLMRegistryTest do
   use ExUnit.Case, async: true
 
-  alias Trento.AI.LlmRegistry
+  alias Trento.AI.LLMRegistry
 
   setup do
     original_config = Application.get_env(:trento, :ai)
@@ -35,35 +35,35 @@ defmodule Trento.AI.LlmRegistryTest do
 
   describe "providers/0" do
     test "returns the list of configured providers" do
-      assert LlmRegistry.providers() == [:provider1, :provider2, :provider3]
+      assert LLMRegistry.providers() == [:provider1, :provider2, :provider3]
     end
   end
 
   describe "get_provider_models/1" do
     test "returns the list of models for a given provider" do
-      assert LlmRegistry.get_provider_models(:provider1) == [
+      assert LLMRegistry.get_provider_models(:provider1) == [
                "model1",
                "model2"
              ]
 
-      assert LlmRegistry.get_provider_models(:provider2) == [
+      assert LLMRegistry.get_provider_models(:provider2) == [
                "model3",
                "model4"
              ]
 
-      assert LlmRegistry.get_provider_models(:provider3) == [
+      assert LLMRegistry.get_provider_models(:provider3) == [
                "model5",
                "model6"
              ]
     end
 
     test "returns an empty list for an unknown provider" do
-      assert LlmRegistry.get_provider_models(:unknown) == []
-      assert LlmRegistry.get_provider_models("foo") == []
+      assert LLMRegistry.get_provider_models(:unknown) == []
+      assert LLMRegistry.get_provider_models("foo") == []
     end
 
     test "returns all available models" do
-      assert LlmRegistry.get_provider_models(:all) == [
+      assert LLMRegistry.get_provider_models(:all) == [
                "model1",
                "model2",
                "model3",
@@ -76,23 +76,23 @@ defmodule Trento.AI.LlmRegistryTest do
 
   describe "get_model_provider/1" do
     test "returns the provider for a given model" do
-      assert LlmRegistry.get_model_provider("model6") == :provider3
-      assert LlmRegistry.get_model_provider("model1") == :provider1
+      assert LLMRegistry.get_model_provider("model6") == :provider3
+      assert LLMRegistry.get_model_provider("model1") == :provider1
     end
 
     test "returns nil for an unknown model" do
-      assert LlmRegistry.get_model_provider("unknown-model") == nil
+      assert LLMRegistry.get_model_provider("unknown-model") == nil
     end
   end
 
   describe "model_supported?/1" do
     test "returns true for a supported model" do
-      assert LlmRegistry.model_supported?("model1") == true
-      assert LlmRegistry.model_supported?("model6") == true
+      assert LLMRegistry.model_supported?("model1") == true
+      assert LLMRegistry.model_supported?("model6") == true
     end
 
     test "returns false for an unsupported model" do
-      assert LlmRegistry.model_supported?("unknown-model") == false
+      assert LLMRegistry.model_supported?("unknown-model") == false
     end
   end
 end

test/trento/ai_test.exs

@@ -1,4 +1,4 @@
-defmodule Trento.AiTest do
+defmodule Trento.AITest do
   use ExUnit.Case
 
   alias Trento.AI