working on redundancy

lib/elixir_ai/data/conversation.ex

@@ -7,7 +7,7 @@ defmodule ElixirAi.Conversation do
   end
 
   def create(name) do
-    case Repo.insert_all("conversations", [[id: Ecto.UUID.generate(), name: name, inserted_at: now(), updated_at: now()]]) do
+    case Repo.insert_all("conversations", [[name: name, inserted_at: now(), updated_at: now()]]) do
       {1, _} -> :ok
       _ -> {:error, :db_error}
     end

lib/elixir_ai/data/message.ex

@@ -6,7 +6,7 @@ defmodule ElixirAi.Message do
     Repo.all(
       from m in "messages",
         where: m.conversation_id == ^conversation_id,
-        order_by: m.position,
+        order_by: m.id,
         select: %{
           role: m.role,
           content: m.content,
@@ -18,17 +18,15 @@ defmodule ElixirAi.Message do
     |> Enum.map(&decode_message/1)
   end
 
-  def insert(conversation_id, message, position) do
+  def insert(conversation_id, message) do
     Repo.insert_all("messages", [
       [
-        id: Ecto.UUID.generate(),
         conversation_id: conversation_id,
         role: to_string(message.role),
         content: message[:content],
         reasoning_content: message[:reasoning_content],
         tool_calls: encode_tool_calls(message[:tool_calls]),
         tool_call_id: message[:tool_call_id],
-        position: position,
         inserted_at: DateTime.truncate(DateTime.utc_now(), :second)
       ]
     ])
@@ -40,9 +38,22 @@ defmodule ElixirAi.Message do
   defp decode_message(row) do
     row
     |> Map.update!(:role, &String.to_existing_atom/1)
+    |> Map.update(:tool_calls, nil, fn
+        nil -> nil
+        json when is_binary(json) ->
+          json |> Jason.decode!() |> Enum.map(&atomize_keys/1)
+        already_decoded -> Enum.map(already_decoded, &atomize_keys/1)
+      end)
     |> drop_nil_fields()
   end
 
+  defp atomize_keys(map) when is_map(map) do
+    Map.new(map, fn
+      {k, v} when is_binary(k) -> {String.to_atom(k), v}
+      {k, v} -> {k, v}
+    end)
+  end
+
   defp drop_nil_fields(map) do
     Map.reject(map, fn {_k, v} -> is_nil(v) end)
   end