fix: support using API's media type if present in batch scanner

Author: jfro

lib/fuzzy_catalog_web/controllers/book_controller.ex

@@ -390,11 +390,14 @@ defmodule FuzzyCatalogWeb.BookController do
                 case Catalog.find_book_by_title_and_author(title, author) do
                   nil ->
                     # Step 4: Book doesn't exist, create it
-                    # Determine media type from suggested types or use the provided one
+                    # Determine media type based on user selection
                     determined_media_type =
-                      case determine_media_type(book_params) do
-                        "unspecified" -> media_type
-                        suggested_type -> suggested_type
+                      if media_type == "automatic" do
+                        # Use API-suggested media type, or "unspecified" if not available
+                        determine_media_type(book_params)
+                      else
+                        # Use the user's explicit selection
+                        media_type
                       end
 
                     final_params =
@@ -411,7 +414,7 @@ defmodule FuzzyCatalogWeb.BookController do
                             title: book.title,
                             author: book.author,
                             isbn: isbn,
-                            media_type: media_type
+                            media_type: determined_media_type
                           }
                         })
 

lib/fuzzy_catalog_web/controllers/book_html/batch_scanner.html.heex

@@ -45,12 +45,18 @@
             <span class="label-text-alt text-xs">Applied to new books</span>
           </label>
           <select id="default-media-type" class="select select-bordered w-full">
+            <option value="automatic" selected>Automatic (from API)</option>
             <option value="unspecified">Unspecified</option>
-            <option value="hardcover" selected>Hardcover</option>
+            <option value="hardcover">Hardcover</option>
             <option value="paperback">Paperback</option>
             <option value="audiobook">Audiobook</option>
             <option value="ebook">eBook</option>
           </select>
+          <label class="label">
+            <span class="label-text-alt text-xs text-base-content/60">
+              "Automatic" uses API-detected format or "Unspecified" if not found
+            </span>
+          </label>
         </div>
 
 <!-- Control Buttons -->

test/fuzzy_catalog/catalog/providers/hardcover_provider_test.exs

@@ -203,19 +203,23 @@ defmodule FuzzyCatalog.Catalog.Providers.HardcoverProviderTest do
       assert nil == HardcoverProvider.__format_series__([])
     end
 
-    test "extracts first series name from map" do
-      series = [%{"name" => "The Stormlight Archive"}]
+    test "extracts first series name from nested structure" do
+      series = [%{"series" => %{"name" => "The Stormlight Archive"}}]
       assert "The Stormlight Archive" == HardcoverProvider.__format_series__(series)
     end
 
     test "extracts first series name when multiple exist" do
-      series = [%{"name" => "The Stormlight Archive"}, %{"name" => "Another Series"}]
+      series = [
+        %{"series" => %{"name" => "The Stormlight Archive"}},
+        %{"series" => %{"name" => "Another Series"}}
+      ]
+
       assert "The Stormlight Archive" == HardcoverProvider.__format_series__(series)
     end
 
-    test "extracts first series name from string list" do
-      series = ["The Stormlight Archive", "Another Series"]
-      assert "The Stormlight Archive" == HardcoverProvider.__format_series__(series)
+    test "handles malformed series data gracefully" do
+      series = [%{"name" => "The Stormlight Archive"}]
+      assert nil == HardcoverProvider.__format_series__(series)
     end
 
     test "returns nil for unexpected format" do