Improve meta and error handling for background cover refreshing

lib/music_library/records.ex

@@ -200,9 +200,12 @@ defmodule MusicLibrary.Records do
     end
   end
 
-  def refresh_cover_async(record_id) do
+  def refresh_cover_async(record) do
-    %{"id" => record_id}
+    meta = %{title: record.title, artists: Enum.map(record.artists, & &1.name)}
-    |> Worker.RefreshCover.new()
+    params = %{"id" => record.id}
+
+    params
+    |> Worker.RefreshCover.new(meta: meta)
     |> BackgroundRepo.insert()
   end
 

lib/music_library/worker/refresh_cover.ex

@@ -1,11 +1,12 @@
 defmodule MusicLibrary.Worker.RefreshCover do
-  use Oban.Worker, queue: :default
+  use Oban.Worker, queue: :default, max_attempts: 3
 
   @impl Oban.Worker
   def perform(%Oban.Job{args: %{"id" => record_id}}) do
     record = MusicLibrary.Records.get_record!(record_id)
-    {:ok, new_record} = MusicLibrary.Records.refresh_cover(record)
+
-    MusicLibrary.Records.notify_update(new_record)
+    with {:ok, updated_record} <- MusicLibrary.Records.refresh_cover(record) do
-    :ok
+      MusicLibrary.Records.notify_update(updated_record)
+    end
   end
 end