Restore former add_command functionality

lib/nosedrum/storage.ex

@@ -93,12 +93,27 @@ defmodule Nosedrum.Storage do
   ## Return value
   Returns `:ok` if successful, and `{:error, reason}` otherwise.
   """
-  @callback add_command(
+  @callback queue_command(
               name_or_path :: String.t() | application_command_path,
               command_module :: module,
               name_or_pid
             ) :: :ok | {:error, Nostrum.Error.ApiError.t()}
 
+  @doc """
+  Add a new command under the given name or application command path.
+
+  If the command already exists, it will be overwritten.
+
+  ## Return value
+  Returns `:ok` if successful, and `{:error, reason}` otherwise.
+  """
+  @callback add_command(
+    name_or_path :: String.t() | application_command_path,
+    command_module :: module,
+    scope :: command_scope,
+    name_or_pid
+  ) :: :ok | {:error, Nostrum.Error.ApiError.t()}
+
   @doc """
   Remove the command under the given name or application command path.
 
@@ -124,7 +139,7 @@ defmodule Nosedrum.Storage do
   ## Return value
   Returns `:ok` if successful, and `{:error, reason}` otherwise.
   """
-  @callback update_discord(
+  @callback process_queued_commands(
     scope :: command_scope,
     name_or_pid
   ) :: :ok | {:error, Nostrum.Error.ApiError.t()}

lib/nosedrum/storage/dispatcher.ex

@@ -53,12 +53,12 @@ defmodule Nosedrum.Storage.Dispatcher do
   end
 
   @impl true
-  def update_discord(scope, id \\ __MODULE__) do
+  def process_queued_commands(scope, id \\ __MODULE__) do
     GenServer.call(id, {:submit, scope})
   end
 
   @impl true
-  def add_command(path, command, id \\ __MODULE__) do
+  def queue_command(path, command, id \\ __MODULE__) do
     command_name =
       if is_binary(path) do
         path
@@ -72,6 +72,23 @@ defmodule Nosedrum.Storage.Dispatcher do
     GenServer.call(id, {:add, command_name, command})
   end
 
+  @impl true
+  def add_command(path, command, scope, id \\ __MODULE__) do
+    payload = build_payload(path, command)
+
+    command_name =
+      if is_binary(path) do
+        path
+      else
+        path
+        |> Enum.take(1)
+        |> List.first()
+        |> unwrap_key()
+      end
+
+    GenServer.call(id, {:add, payload, command_name, command, scope})
+  end
+
   @impl true
   def remove_command(name, command_id, scope, id \\ __MODULE__) do
     GenServer.call(id, {:remove, name, command_id, scope})
@@ -116,6 +133,46 @@ defmodule Nosedrum.Storage.Dispatcher do
     {:reply, :ok, Map.put(commands, name, command)}
   end
 
+
+  @impl true
+  def handle_call({:add, payload, name, command, :global}, _from, commands) do
+    case Nostrum.Api.create_global_application_command(payload) do
+      {:ok, _} = response ->
+        {:reply, response, Map.put(commands, name, command)}
+
+      error ->
+        {:reply, {:error, error}, commands}
+    end
+  end
+
+  @impl true
+  def handle_call({:add, payload, name, command, guild_id_list}, _from, commands)
+      when is_list(guild_id_list) do
+    res =
+      Enum.reduce(guild_id_list, {[], []}, fn guild_id, {errors, responses} ->
+        case Nostrum.Api.create_guild_application_command(guild_id, payload) do
+          {:ok, _} = response ->
+            {errors, [response | responses]}
+
+          error ->
+            {[error | errors], responses}
+        end
+      end)
+
+    {:reply, res, Map.put(commands, name, command)}
+  end
+
+  @impl true
+  def handle_call({:add, payload, name, command, guild_id}, _from, commands) do
+    case Nostrum.Api.create_guild_application_command(guild_id, payload) do
+      {:ok, _} = response ->
+        {:reply, response, Map.put(commands, name, command)}
+
+      error ->
+        {:reply, {:error, error}, commands}
+    end
+  end
+
   @impl true
   def handle_call({:remove, name, command_id, :global}, _from, commands) do
     case Nostrum.Api.delete_global_application_command(command_id) do