From c6c218970c680126685fd48866c298d5ea28c5a1 Mon Sep 17 00:00:00 2001
From: Schuwi <schuwi4@gmail.com>
Date: Sat, 20 Sep 2025 11:52:43 +0200
Subject: [PATCH] style: format codebase

---
 config/dev.exs                                |   1 +
 lib/components_elixir/apriltag.ex             |  24 +-
 lib/components_elixir/apriltag/tag36h11.ex    |  84 +++--
 lib/components_elixir/datasheet_downloader.ex |  38 ++-
 lib/components_elixir/inventory.ex            |  40 ++-
 lib/components_elixir/inventory/category.ex   |   2 +-
 lib/components_elixir/inventory/component.ex  |  13 +-
 .../inventory/hierarchical.ex                 |  54 +++-
 .../inventory/storage_location.ex             |  19 +-
 .../components/core_components.ex             |  16 +-
 .../controllers/file_controller.ex            |  12 +-
 .../live/categories_live.ex                   | 137 ++++----
 .../live/components_live.ex                   |  81 +++--
 lib/components_elixir_web/live/login_live.ex  |   2 +-
 .../live/storage_locations_live.ex            | 302 +++++++++++-------
 lib/mix/tasks/apriltag.generate_all.ex        |   6 +-
 .../20250914160354_migrate_qr_to_apriltag.exs |   4 +-
 priv/repo/seeds.exs                           | 265 ++++++++++-----
 .../controllers/error_html_test.exs           |   3 +-
 .../controllers/error_json_test.exs           |   4 +-
 20 files changed, 722 insertions(+), 385 deletions(-)

diff --git a/config/dev.exs b/config/dev.exs
index 263019c..c932b91 100644
--- a/config/dev.exs
+++ b/config/dev.exs
@@ -16,6 +16,7 @@ config :components_elixir, ComponentsElixir.Repo,
 # For development, use a local uploads directory
 config :components_elixir,
   uploads_dir: "./uploads"
+
 #
 # The watchers configuration can be used to run external
 # watchers to your application. For example, we can use it
diff --git a/lib/components_elixir/apriltag.ex b/lib/components_elixir/apriltag.ex
index ef4a110..9ab25e1 100644
--- a/lib/components_elixir/apriltag.ex
+++ b/lib/components_elixir/apriltag.ex
@@ -31,6 +31,7 @@ defmodule ComponentsElixir.AprilTag do
   def valid_apriltag_id?(id) when is_integer(id) do
     id >= 0 and id < @tag36h11_count
   end
+
   def valid_apriltag_id?(_), do: false
 
   @doc """
@@ -78,8 +79,8 @@ defmodule ComponentsElixir.AprilTag do
   def used_apriltag_ids do
     ComponentsElixir.Repo.all(
       from sl in ComponentsElixir.Inventory.StorageLocation,
-      where: not is_nil(sl.apriltag_id),
-      select: sl.apriltag_id
+        where: not is_nil(sl.apriltag_id),
+        select: sl.apriltag_id
     )
   end
 
@@ -130,10 +131,11 @@ defmodule ComponentsElixir.AprilTag do
   This should be run once during setup to pre-generate all AprilTag images.
   """
   def generate_all_apriltag_svgs(opts \\ []) do
-    static_dir = Path.join([
-      Application.app_dir(:components_elixir, "priv/static"),
-      "apriltags"
-    ])
+    static_dir =
+      Path.join([
+        Application.app_dir(:components_elixir, "priv/static"),
+        "apriltags"
+      ])
 
     # Ensure directory exists
     File.mkdir_p!(static_dir)
@@ -187,10 +189,12 @@ defmodule ComponentsElixir.AprilTag do
   """
   def cleanup_apriltag_svg(apriltag_id) do
     filename = "tag36h11_id_#{String.pad_leading(to_string(apriltag_id), 3, "0")}.svg"
-    file_path = Path.join([
-      Application.app_dir(:components_elixir, "priv/static/apriltags"),
-      filename
-    ])
+
+    file_path =
+      Path.join([
+        Application.app_dir(:components_elixir, "priv/static/apriltags"),
+        filename
+      ])
 
     if File.exists?(file_path) do
       File.rm(file_path)
diff --git a/lib/components_elixir/apriltag/tag36h11.ex b/lib/components_elixir/apriltag/tag36h11.ex
index d73fa64..8ecbbb3 100644
--- a/lib/components_elixir/apriltag/tag36h11.ex
+++ b/lib/components_elixir/apriltag/tag36h11.ex
@@ -16,6 +16,7 @@ defmodule ComponentsElixir.AprilTag.Tag36h11 do
     case Regex.run(~r/\(april\.tag\.Tag36h11, id = (\d+)\) <([0-9a-fA-F]+)> maketag/, line) do
       [_, id_str, hex_pattern] ->
         {String.to_integer(id_str), String.downcase(hex_pattern)}
+
       _ ->
         nil
     end
@@ -23,26 +24,30 @@ defmodule ComponentsElixir.AprilTag.Tag36h11 do
 
   # Extract patterns from PostScript file at compile time
   @all_patterns (
-    path = Path.join([File.cwd!(), "apriltags.ps"])
+                  path = Path.join([File.cwd!(), "apriltags.ps"])
 
-    if File.exists?(path) do
-      File.read!(path)
-      |> String.split("\n")
-      |> Enum.filter(&String.contains?(&1, "april.tag.Tag36h11"))
-      |> Enum.map(fn line ->
-        case Regex.run(~r/\(april\.tag\.Tag36h11, id = (\d+)\) <([0-9a-fA-F]+)> maketag/, line) do
-          [_, id_str, hex_pattern] ->
-            {String.to_integer(id_str), String.downcase(hex_pattern)}
-          _ ->
-            nil
-        end
-      end)
-      |> Enum.reject(&is_nil/1)
-      |> Map.new()
-    else
-      raise "Error: apriltags.ps file not found in project root. This file is required for AprilTag pattern generation."
-    end
-  )
+                  if File.exists?(path) do
+                    File.read!(path)
+                    |> String.split("\n")
+                    |> Enum.filter(&String.contains?(&1, "april.tag.Tag36h11"))
+                    |> Enum.map(fn line ->
+                      case Regex.run(
+                             ~r/\(april\.tag\.Tag36h11, id = (\d+)\) <([0-9a-fA-F]+)> maketag/,
+                             line
+                           ) do
+                        [_, id_str, hex_pattern] ->
+                          {String.to_integer(id_str), String.downcase(hex_pattern)}
+
+                        _ ->
+                          nil
+                      end
+                    end)
+                    |> Enum.reject(&is_nil/1)
+                    |> Map.new()
+                  else
+                    raise "Error: apriltags.ps file not found in project root. This file is required for AprilTag pattern generation."
+                  end
+                )
 
   # Sample of real tag36h11 hex patterns from AprilRobotics repository
   # This will be populated with patterns extracted from the PostScript file
@@ -64,7 +69,8 @@ defmodule ComponentsElixir.AprilTag.Tag36h11 do
       |> Enum.reject(&is_nil/1)
       |> Map.new()
     else
-      %{} # Return empty map if file not found, will fall back to hardcoded patterns
+      # Return empty map if file not found, will fall back to hardcoded patterns
+      %{}
     end
   end
 
@@ -76,6 +82,7 @@ defmodule ComponentsElixir.AprilTag.Tag36h11 do
   def get_hex_pattern(id) when is_integer(id) and id >= 0 and id < 587 do
     Map.get(@tag36h11_patterns, id)
   end
+
   def get_hex_pattern(_), do: nil
 
   @doc """
@@ -97,6 +104,7 @@ defmodule ComponentsElixir.AprilTag.Tag36h11 do
 
     # Each scanline is padded to a byte boundary: 3 bytes per 10 pixels (2 bpp)
     row_bytes = 3
+
     rows =
       for row <- 0..9 do
         <<_::binary-size(row * row_bytes), r::binary-size(row_bytes), _::binary>> = bytes
@@ -104,12 +112,23 @@ defmodule ComponentsElixir.AprilTag.Tag36h11 do
 
         samples =
           [
-            b0 >>> 6 &&& 0x3, b0 >>> 4 &&& 0x3, b0 >>> 2 &&& 0x3, b0 &&& 0x3,
-            b1 >>> 6 &&& 0x3, b1 >>> 4 &&& 0x3, b1 >>> 2 &&& 0x3, b1 &&& 0x3,
-            b2 >>> 6 &&& 0x3, b2 >>> 4 &&& 0x3, b2 >>> 2 &&& 0x3, b2 &&& 0x3
+            b0 >>> 6 &&& 0x3,
+            b0 >>> 4 &&& 0x3,
+            b0 >>> 2 &&& 0x3,
+            b0 &&& 0x3,
+            b1 >>> 6 &&& 0x3,
+            b1 >>> 4 &&& 0x3,
+            b1 >>> 2 &&& 0x3,
+            b1 &&& 0x3,
+            b2 >>> 6 &&& 0x3,
+            b2 >>> 4 &&& 0x3,
+            b2 >>> 2 &&& 0x3,
+            b2 &&& 0x3
           ]
-          |> Enum.take(10)              # drop the 2 padding samples at end of row
-          |> Enum.map(&(&1 == 0))       # 0 = black, 3 = white → boolean
+          # drop the 2 padding samples at end of row
+          |> Enum.take(10)
+          # 0 = black, 3 = white → boolean
+          |> Enum.map(&(&1 == 0))
 
         samples
       end
@@ -133,7 +152,8 @@ defmodule ComponentsElixir.AprilTag.Tag36h11 do
   Only black modules are drawn over a white background.
   """
   def binary_matrix_to_svg(binary_matrix, opts \\ []) do
-    size = Keyword.get(opts, :size, 200)  # final CSS size in px
+    # final CSS size in px
+    size = Keyword.get(opts, :size, 200)
     id_text = Keyword.get(opts, :id_text, "")
 
     # binary_matrix is 10x10 of booleans: true=black, false=white
@@ -172,9 +192,11 @@ defmodule ComponentsElixir.AprilTag.Tag36h11 do
 
       <!-- caption -->
       #{if id_text != "" do
-          ~s(<text x="#{modules_w/2}" y="#{modules_h + 1.4}" text-anchor="middle"
+      ~s(<text x="#{modules_w / 2}" y="#{modules_h + 1.4}" text-anchor="middle"
                    font-family="Arial" font-size="0.9">#{id_text}</text>)
-        else "" end}
+    else
+      ""
+    end}
     </svg>
     """
   end
@@ -197,11 +219,13 @@ defmodule ComponentsElixir.AprilTag.Tag36h11 do
         opts_with_id = Keyword.put(opts, :id_text, id_text)
         binary_matrix_to_svg(binary_matrix, opts_with_id)
     end
-  end  # Generate a placeholder pattern for IDs we don't have real data for yet
+  end
+
+  # Generate a placeholder pattern for IDs we don't have real data for yet
   defp generate_placeholder_svg(id, opts) do
     size = Keyword.get(opts, :size, 200)
     margin = Keyword.get(opts, :margin, div(size, 10))
-    square_size = size - (2 * margin)
+    square_size = size - 2 * margin
 
     """
     <svg width="#{size}" height="#{size + 30}" xmlns="http://www.w3.org/2000/svg">
diff --git a/lib/components_elixir/datasheet_downloader.ex b/lib/components_elixir/datasheet_downloader.ex
index e8368d6..e8b7b00 100644
--- a/lib/components_elixir/datasheet_downloader.ex
+++ b/lib/components_elixir/datasheet_downloader.ex
@@ -28,6 +28,7 @@ defmodule ComponentsElixir.DatasheetDownloader do
     case URI.parse(url) do
       %URI{scheme: scheme} when scheme in ["http", "https"] ->
         {:ok, URI.parse(url)}
+
       _ ->
         {:error, "Invalid URL scheme. Only HTTP and HTTPS are supported."}
     end
@@ -36,9 +37,12 @@ defmodule ComponentsElixir.DatasheetDownloader do
   defp generate_filename(url) do
     # Try to extract a meaningful filename from the URL
     uri = URI.parse(url)
+
     original_filename =
       case Path.basename(uri.path || "") do
-        "" -> "datasheet"
+        "" ->
+          "datasheet"
+
         basename ->
           # Remove extension and sanitize
           basename
@@ -54,10 +58,14 @@ defmodule ComponentsElixir.DatasheetDownloader do
 
   defp sanitize_filename(filename) do
     filename
-    |> String.replace(~r/[^\w\-_]/, "_")  # Replace non-word chars with underscores
-    |> String.replace(~r/_+/, "_")        # Replace multiple underscores with single
-    |> String.trim("_")                   # Remove leading/trailing underscores
-    |> String.slice(0, 50)                # Limit length
+    # Replace non-word chars with underscores
+    |> String.replace(~r/[^\w\-_]/, "_")
+    # Replace multiple underscores with single
+    |> String.replace(~r/_+/, "_")
+    # Remove leading/trailing underscores
+    |> String.trim("_")
+    # Limit length
+    |> String.slice(0, 50)
     |> case do
       "" -> "datasheet"
       name -> name
@@ -66,17 +74,19 @@ defmodule ComponentsElixir.DatasheetDownloader do
 
   defp fetch_pdf(url) do
     case Req.get(url,
-      redirect: true,
-      max_redirects: 5,
-      receive_timeout: 30_000,
-      headers: [
-        {"User-Agent", "ComponentSystem/1.0 DatasheetDownloader"}
-      ]
-    ) do
+           redirect: true,
+           max_redirects: 5,
+           receive_timeout: 30_000,
+           headers: [
+             {"User-Agent", "ComponentSystem/1.0 DatasheetDownloader"}
+           ]
+         ) do
       {:ok, %Req.Response{status: 200} = response} ->
         {:ok, response}
+
       {:ok, %Req.Response{status: status}} ->
         {:error, "HTTP error: #{status}"}
+
       {:error, reason} ->
         Logger.error("Failed to download PDF from #{url}: #{inspect(reason)}")
         {:error, "Download failed: #{inspect(reason)}"}
@@ -88,6 +98,7 @@ defmodule ComponentsElixir.DatasheetDownloader do
     case body do
       <<"%PDF", _rest::binary>> ->
         :ok
+
       _ ->
         {:error, "Downloaded content is not a valid PDF file"}
     end
@@ -105,10 +116,12 @@ defmodule ComponentsElixir.DatasheetDownloader do
           :ok ->
             Logger.info("Successfully saved datasheet: #{filename}")
             :ok
+
           {:error, reason} ->
             Logger.error("Failed to save datasheet file: #{inspect(reason)}")
             {:error, "Failed to save file: #{inspect(reason)}"}
         end
+
       {:error, reason} ->
         Logger.error("Failed to create datasheets directory: #{inspect(reason)}")
         {:error, "Failed to create directory: #{inspect(reason)}"}
@@ -129,6 +142,7 @@ defmodule ComponentsElixir.DatasheetDownloader do
       :ok ->
         Logger.info("Deleted datasheet file: #{filename}")
         :ok
+
       {:error, reason} ->
         Logger.warning("Failed to delete datasheet file #{filename}: #{inspect(reason)}")
         {:error, reason}
diff --git a/lib/components_elixir/inventory.ex b/lib/components_elixir/inventory.ex
index 9545fb1..ade789f 100644
--- a/lib/components_elixir/inventory.ex
+++ b/lib/components_elixir/inventory.ex
@@ -19,7 +19,7 @@ defmodule ComponentsElixir.Inventory do
     locations =
       StorageLocation
       |> order_by([sl], asc: sl.name)
-      |> preload([parent: [parent: [parent: [parent: :parent]]]])
+      |> preload(parent: [parent: [parent: [parent: :parent]]])
       |> Repo.all()
 
     # Ensure AprilTag SVGs exist for all locations
@@ -162,7 +162,7 @@ defmodule ComponentsElixir.Inventory do
   """
   def list_categories do
     Category
-    |> preload([parent: [parent: [parent: [parent: :parent]]]])
+    |> preload(parent: [parent: [parent: [parent: :parent]]])
     |> Repo.all()
   end
 
@@ -217,8 +217,15 @@ defmodule ComponentsElixir.Inventory do
 
     # Verify the category exists before getting descendants
     case Enum.find(categories, &(&1.id == category_id)) do
-      nil -> []
-      _category -> ComponentsElixir.Inventory.Hierarchical.descendant_ids(categories, category_id, &(&1.parent_id))
+      nil ->
+        []
+
+      _category ->
+        ComponentsElixir.Inventory.Hierarchical.descendant_ids(
+          categories,
+          category_id,
+          & &1.parent_id
+        )
     end
   end
 
@@ -233,13 +240,21 @@ defmodule ComponentsElixir.Inventory do
   for typical storage location tree sizes (hundreds of locations). For very large storage location trees,
   a recursive CTE query could be used instead.
   """
-  def get_storage_location_and_descendant_ids(storage_location_id) when is_integer(storage_location_id) do
+  def get_storage_location_and_descendant_ids(storage_location_id)
+      when is_integer(storage_location_id) do
     storage_locations = list_storage_locations()
 
     # Verify the storage location exists before getting descendants
     case Enum.find(storage_locations, &(&1.id == storage_location_id)) do
-      nil -> []
-      _storage_location -> ComponentsElixir.Inventory.Hierarchical.descendant_ids(storage_locations, storage_location_id, &(&1.parent_id))
+      nil ->
+        []
+
+      _storage_location ->
+        ComponentsElixir.Inventory.Hierarchical.descendant_ids(
+          storage_locations,
+          storage_location_id,
+          & &1.parent_id
+        )
     end
   end
 
@@ -306,7 +321,7 @@ defmodule ComponentsElixir.Inventory do
   }
 
   defp get_sort_order(criteria) do
-    Map.get(@sort_orders, criteria, [asc: :name, asc: :id])
+    Map.get(@sort_orders, criteria, asc: :name, asc: :id)
   end
 
   @doc """
@@ -338,10 +353,12 @@ defmodule ComponentsElixir.Inventory do
           case ComponentsElixir.DatasheetDownloader.download_pdf_from_url(url) do
             {:ok, filename} ->
               Map.put(attrs, "datasheet_filename", filename)
+
             {:error, _reason} ->
               # Continue without datasheet file if download fails
               attrs
           end
+
         _ ->
           attrs
       end
@@ -372,13 +389,18 @@ defmodule ComponentsElixir.Inventory do
             {:ok, filename} ->
               # Delete old datasheet file if it exists
               if component.datasheet_filename do
-                ComponentsElixir.DatasheetDownloader.delete_datasheet_file(component.datasheet_filename)
+                ComponentsElixir.DatasheetDownloader.delete_datasheet_file(
+                  component.datasheet_filename
+                )
               end
+
               Map.put(attrs, "datasheet_filename", filename)
+
             {:error, _reason} ->
               # Keep existing filename if download fails
               attrs
           end
+
         _ ->
           attrs
       end
diff --git a/lib/components_elixir/inventory/category.ex b/lib/components_elixir/inventory/category.ex
index 06a09e0..4ea8a2d 100644
--- a/lib/components_elixir/inventory/category.ex
+++ b/lib/components_elixir/inventory/category.ex
@@ -37,7 +37,7 @@ defmodule ComponentsElixir.Inventory.Category do
   """
   @impl true
   def full_path(%Category{} = category) do
-    Hierarchical.full_path(category, &(&1.parent), path_separator())
+    Hierarchical.full_path(category, & &1.parent, path_separator())
   end
 
   @impl true
diff --git a/lib/components_elixir/inventory/component.ex b/lib/components_elixir/inventory/component.ex
index 1c656ba..72435b2 100644
--- a/lib/components_elixir/inventory/component.ex
+++ b/lib/components_elixir/inventory/component.ex
@@ -30,7 +30,18 @@ defmodule ComponentsElixir.Inventory.Component do
   @doc false
   def changeset(component, attrs) do
     component
-    |> cast(attrs, [:name, :description, :keywords, :position, :count, :datasheet_url, :datasheet_filename, :image_filename, :category_id, :storage_location_id])
+    |> cast(attrs, [
+      :name,
+      :description,
+      :keywords,
+      :position,
+      :count,
+      :datasheet_url,
+      :datasheet_filename,
+      :image_filename,
+      :category_id,
+      :storage_location_id
+    ])
     |> validate_required([:name, :category_id])
     |> validate_length(:name, min: 1, max: 255)
     |> validate_length(:description, max: 2000)
diff --git a/lib/components_elixir/inventory/hierarchical.ex b/lib/components_elixir/inventory/hierarchical.ex
index 02473d9..56c6c4a 100644
--- a/lib/components_elixir/inventory/hierarchical.ex
+++ b/lib/components_elixir/inventory/hierarchical.ex
@@ -29,10 +29,12 @@ defmodule ComponentsElixir.Inventory.Hierarchical do
     case parent_accessor_fn.(entity) do
       nil ->
         entity.name
+
       %Ecto.Association.NotLoaded{} ->
         # Parent not loaded - fall back to database lookup
         # This is a fallback and should be rare if preloading is done correctly
         build_path_with_db_lookup(entity, separator)
+
       parent ->
         "#{full_path(parent, parent_accessor_fn, separator)}#{separator}#{entity.name}"
     end
@@ -52,12 +54,14 @@ defmodule ComponentsElixir.Inventory.Hierarchical do
       nil ->
         # This is a root entity, add its name and return the complete path
         [entity.name | path_so_far]
+
       parent_id ->
         # Load parent from database
         case load_parent_entity(entity, parent_id) do
           nil ->
             # Parent not found (orphaned record), treat this as root
             [entity.name | path_so_far]
+
           parent ->
             # Recursively get the path from the parent, then add current entity
             collect_path_from_root(parent, [entity.name | path_so_far])
@@ -93,9 +97,9 @@ defmodule ComponentsElixir.Inventory.Hierarchical do
       entity_id = id_accessor_fn.(entity)
 
       # Remove self-reference
-      entity_id == editing_entity_id ||
       # Remove descendants (they would create a cycle)
-      descendant?(entities, entity_id, editing_entity_id, parent_id_accessor_fn)
+      entity_id == editing_entity_id ||
+        descendant?(entities, entity_id, editing_entity_id, parent_id_accessor_fn)
     end)
   end
 
@@ -114,13 +118,21 @@ defmodule ComponentsElixir.Inventory.Hierarchical do
 
   defp descendant_recursive?(entities, entity, ancestor_id, parent_id_accessor_fn) do
     case parent_id_accessor_fn.(entity) do
-      nil -> false
-      ^ancestor_id -> true
+      nil ->
+        false
+
+      ^ancestor_id ->
+        true
+
       parent_id ->
         parent = Enum.find(entities, fn e -> e.id == parent_id end)
+
         case parent do
-          nil -> false
-          parent_entity -> descendant_recursive?(entities, parent_entity, ancestor_id, parent_id_accessor_fn)
+          nil ->
+            false
+
+          parent_entity ->
+            descendant_recursive?(entities, parent_entity, ancestor_id, parent_id_accessor_fn)
         end
     end
   end
@@ -182,15 +194,20 @@ defmodule ComponentsElixir.Inventory.Hierarchical do
   Includes proper filtering to prevent cycles and formatted display names.
   Results are sorted hierarchically for intuitive navigation.
   """
-  def parent_select_options(entities, editing_entity_id, parent_accessor_fn, nil_option_text \\ "No parent") do
+  def parent_select_options(
+        entities,
+        editing_entity_id,
+        parent_accessor_fn,
+        nil_option_text \\ "No parent"
+      ) do
     available_entities =
       filter_parent_options(
         entities,
         editing_entity_id,
-        &(&1.id),
-        &(&1.parent_id)
+        & &1.id,
+        & &1.parent_id
       )
-      |> sort_hierarchically(&(&1.parent_id))
+      |> sort_hierarchically(& &1.parent_id)
       |> Enum.map(fn entity ->
         {display_name(entity, parent_accessor_fn), entity.id}
       end)
@@ -205,7 +222,7 @@ defmodule ComponentsElixir.Inventory.Hierarchical do
   def select_options(entities, parent_accessor_fn, nil_option_text \\ nil) do
     sorted_entities =
       entities
-      |> sort_hierarchically(&(&1.parent_id))
+      |> sort_hierarchically(& &1.parent_id)
       |> Enum.map(fn entity ->
         {display_name(entity, parent_accessor_fn), entity.id}
       end)
@@ -300,9 +317,10 @@ defmodule ComponentsElixir.Inventory.Hierarchical do
     descendant_ids_only = List.delete(all_descendant_ids, entity_id)
 
     # Sum counts for all descendants
-    children_count = Enum.reduce(descendant_ids_only, 0, fn id, acc ->
-      acc + count_fn.(id)
-    end)
+    children_count =
+      Enum.reduce(descendant_ids_only, 0, fn id, acc ->
+        acc + count_fn.(id)
+      end)
 
     {self_count, children_count, self_count + children_count}
   end
@@ -320,7 +338,13 @@ defmodule ComponentsElixir.Inventory.Hierarchical do
     - singular_noun: What to call a single item (default: "component")
     - plural_noun: What to call multiple items (default: "components")
   """
-  def format_count_display(self_count, children_count, is_expanded, singular_noun \\ "component", plural_noun \\ "components") do
+  def format_count_display(
+        self_count,
+        children_count,
+        is_expanded,
+        singular_noun \\ "component",
+        plural_noun \\ "components"
+      ) do
     total_count = self_count + children_count
 
     count_noun = if total_count == 1, do: singular_noun, else: plural_noun
diff --git a/lib/components_elixir/inventory/storage_location.ex b/lib/components_elixir/inventory/storage_location.ex
index 3421b4f..9d853d8 100644
--- a/lib/components_elixir/inventory/storage_location.ex
+++ b/lib/components_elixir/inventory/storage_location.ex
@@ -25,7 +25,7 @@ defmodule ComponentsElixir.Inventory.StorageLocation do
     timestamps(type: :naive_datetime_usec)
   end
 
-    @doc false
+  @doc false
   def changeset(storage_location, attrs) do
     storage_location
     |> cast(attrs, [:name, :description, :parent_id, :apriltag_id])
@@ -40,7 +40,7 @@ defmodule ComponentsElixir.Inventory.StorageLocation do
   # HierarchicalSchema implementations
   @impl true
   def full_path(%StorageLocation{} = storage_location) do
-    Hierarchical.full_path(storage_location, &(&1.parent), path_separator())
+    Hierarchical.full_path(storage_location, & &1.parent, path_separator())
   end
 
   @impl true
@@ -80,11 +80,12 @@ defmodule ComponentsElixir.Inventory.StorageLocation do
 
   defp get_next_available_apriltag_id do
     # Get all used AprilTag IDs
-    used_ids = ComponentsElixir.Repo.all(
-      from sl in ComponentsElixir.Inventory.StorageLocation,
-      where: not is_nil(sl.apriltag_id),
-      select: sl.apriltag_id
-    )
+    used_ids =
+      ComponentsElixir.Repo.all(
+        from sl in ComponentsElixir.Inventory.StorageLocation,
+          where: not is_nil(sl.apriltag_id),
+          select: sl.apriltag_id
+      )
 
     # Find the first available ID (0-586)
     0..586
@@ -93,7 +94,9 @@ defmodule ComponentsElixir.Inventory.StorageLocation do
       nil ->
         # All IDs are used - this should be handled at the application level
         raise "All AprilTag IDs are in use"
-      id -> id
+
+      id ->
+        id
     end
   end
 end
diff --git a/lib/components_elixir_web/components/core_components.ex b/lib/components_elixir_web/components/core_components.ex
index 83a6c00..fa98d40 100644
--- a/lib/components_elixir_web/components/core_components.ex
+++ b/lib/components_elixir_web/components/core_components.ex
@@ -195,7 +195,10 @@ defmodule ComponentsElixirWeb.CoreComponents do
             name={@name}
             value="true"
             checked={@checked}
-            class={@class || "checkbox checkbox-sm border-base-300 checked:bg-primary checked:border-primary"}
+            class={
+              @class ||
+                "checkbox checkbox-sm border-base-300 checked:bg-primary checked:border-primary"
+            }
             {@rest}
           />{@label}
         </span>
@@ -213,7 +216,10 @@ defmodule ComponentsElixirWeb.CoreComponents do
         <select
           id={@id}
           name={@name}
-          class={[@class || "w-full select bg-base-100 border-base-300 text-base-content", @errors != [] && (@error_class || "select-error border-error")]}
+          class={[
+            @class || "w-full select bg-base-100 border-base-300 text-base-content",
+            @errors != [] && (@error_class || "select-error border-error")
+          ]}
           multiple={@multiple}
           {@rest}
         >
@@ -235,7 +241,8 @@ defmodule ComponentsElixirWeb.CoreComponents do
           id={@id}
           name={@name}
           class={[
-            @class || "w-full textarea bg-base-100 border-base-300 text-base-content placeholder:text-base-content/50",
+            @class ||
+              "w-full textarea bg-base-100 border-base-300 text-base-content placeholder:text-base-content/50",
             @errors != [] && (@error_class || "textarea-error border-error")
           ]}
           {@rest}
@@ -258,7 +265,8 @@ defmodule ComponentsElixirWeb.CoreComponents do
           id={@id}
           value={Phoenix.HTML.Form.normalize_value(@type, @value)}
           class={[
-            @class || "w-full input bg-base-100 border-base-300 text-base-content placeholder:text-base-content/50",
+            @class ||
+              "w-full input bg-base-100 border-base-300 text-base-content placeholder:text-base-content/50",
             @errors != [] && (@error_class || "input-error border-error")
           ]}
           {@rest}
diff --git a/lib/components_elixir_web/controllers/file_controller.ex b/lib/components_elixir_web/controllers/file_controller.ex
index a2fa0b0..aba36bc 100644
--- a/lib/components_elixir_web/controllers/file_controller.ex
+++ b/lib/components_elixir_web/controllers/file_controller.ex
@@ -13,7 +13,8 @@ defmodule ComponentsElixirWeb.FileController do
 
           conn
           |> put_resp_content_type(mime_type)
-          |> put_resp_header("cache-control", "public, max-age=86400") # Cache for 1 day
+          # Cache for 1 day
+          |> put_resp_header("cache-control", "public, max-age=86400")
           |> send_file(200, file_path)
         else
           conn
@@ -40,7 +41,8 @@ defmodule ComponentsElixirWeb.FileController do
 
           conn
           |> put_resp_content_type(mime_type)
-          |> put_resp_header("cache-control", "public, max-age=86400") # Cache for 1 day
+          # Cache for 1 day
+          |> put_resp_header("cache-control", "public, max-age=86400")
           |> put_resp_header("content-disposition", "inline; filename=\"#{filename}\"")
           |> send_file(200, file_path)
         else
@@ -64,9 +66,9 @@ defmodule ComponentsElixirWeb.FileController do
       # Security validation: prevent directory traversal and only allow safe characters
       # Allow letters, numbers, spaces, dots, dashes, underscores, parentheses, and basic punctuation
       if String.match?(decoded_filename, ~r/^[a-zA-Z0-9\s_\-\.\(\)\[\]]+$/) and
-         not String.contains?(decoded_filename, "..") and
-         not String.starts_with?(decoded_filename, "/") and
-         not String.contains?(decoded_filename, "\\") do
+           not String.contains?(decoded_filename, "..") and
+           not String.starts_with?(decoded_filename, "/") and
+           not String.contains?(decoded_filename, "\\") do
         {:ok, decoded_filename}
       else
         {:error, "Invalid filename: contains unsafe characters"}
diff --git a/lib/components_elixir_web/live/categories_live.ex b/lib/components_elixir_web/live/categories_live.ex
index 801ed73..9c897ff 100644
--- a/lib/components_elixir_web/live/categories_live.ex
+++ b/lib/components_elixir_web/live/categories_live.ex
@@ -46,12 +46,13 @@ defmodule ComponentsElixirWeb.CategoriesLive do
   def handle_event("show_edit_form", %{"id" => id}, socket) do
     category = Inventory.get_category!(id)
     # Create a changeset with current values forced into changes for proper form display
-    changeset = Inventory.change_category(category, %{
-      name: category.name,
-      description: category.description,
-      parent_id: category.parent_id
-    })
-    |> Ecto.Changeset.force_change(:parent_id, category.parent_id)
+    changeset =
+      Inventory.change_category(category, %{
+        name: category.name,
+        description: category.description,
+        parent_id: category.parent_id
+      })
+      |> Ecto.Changeset.force_change(:parent_id, category.parent_id)
 
     form = to_form(changeset)
 
@@ -112,7 +113,12 @@ defmodule ComponentsElixirWeb.CategoriesLive do
          |> reload_categories()}
 
       {:error, _changeset} ->
-        {:noreply, put_flash(socket, :error, "Cannot delete category - it may have components assigned or child categories")}
+        {:noreply,
+         put_flash(
+           socket,
+           :error,
+           "Cannot delete category - it may have components assigned or child categories"
+         )}
     end
   end
 
@@ -120,11 +126,12 @@ defmodule ComponentsElixirWeb.CategoriesLive do
     category_id = String.to_integer(id)
     expanded_categories = socket.assigns.expanded_categories
 
-    new_expanded = if MapSet.member?(expanded_categories, category_id) do
-      MapSet.delete(expanded_categories, category_id)
-    else
-      MapSet.put(expanded_categories, category_id)
-    end
+    new_expanded =
+      if MapSet.member?(expanded_categories, category_id) do
+        MapSet.delete(expanded_categories, category_id)
+      else
+        MapSet.put(expanded_categories, category_id)
+      end
 
     {:noreply, assign(socket, :expanded_categories, new_expanded)}
   end
@@ -138,17 +145,17 @@ defmodule ComponentsElixirWeb.CategoriesLive do
     Hierarchical.parent_select_options(
       categories,
       editing_category_id,
-      &(&1.parent),
+      & &1.parent,
       "No parent (Root category)"
     )
   end
 
   defp root_categories(categories) do
-    Hierarchical.root_entities(categories, &(&1.parent_id))
+    Hierarchical.root_entities(categories, & &1.parent_id)
   end
 
   defp child_categories(categories, parent_id) do
-    Hierarchical.child_entities(categories, parent_id, &(&1.parent_id))
+    Hierarchical.child_entities(categories, parent_id, & &1.parent_id)
   end
 
   defp count_components_in_category(category_id) do
@@ -164,38 +171,41 @@ defmodule ComponentsElixirWeb.CategoriesLive do
     border_class = if assigns.depth > 0, do: "border-l-2 border-base-300 pl-6", else: ""
 
     # Icon size and button size based on depth
-    {icon_size, button_size, text_size, title_tag} = case assigns.depth do
-      0 -> {"w-5 h-5", "p-2", "text-lg", "h3"}
-      1 -> {"w-4 h-4", "p-1.5", "text-base", "h4"}
-      _ -> {"w-3 h-3", "p-1", "text-sm", "h5"}
-    end
+    {icon_size, button_size, text_size, title_tag} =
+      case assigns.depth do
+        0 -> {"w-5 h-5", "p-2", "text-lg", "h3"}
+        1 -> {"w-4 h-4", "p-1.5", "text-base", "h4"}
+        _ -> {"w-3 h-3", "p-1", "text-sm", "h5"}
+      end
 
     children = child_categories(assigns.categories, assigns.category.id)
     has_children = !Enum.empty?(children)
     is_expanded = MapSet.member?(assigns.expanded_categories, assigns.category.id)
 
     # Calculate component counts including descendants
-    {self_count, children_count, _total_count} = Hierarchical.count_with_descendants(
-      assigns.category.id,
-      assigns.categories,
-      &(&1.parent_id),
-      &count_components_in_category/1
-    )
+    {self_count, children_count, _total_count} =
+      Hierarchical.count_with_descendants(
+        assigns.category.id,
+        assigns.categories,
+        & &1.parent_id,
+        &count_components_in_category/1
+      )
 
     # Format count display
     count_display = Hierarchical.format_count_display(self_count, children_count, is_expanded)
 
-    assigns = assigns
-    |> assign(:margin_left, margin_left)
-    |> assign(:border_class, border_class)
-    |> assign(:icon_size, icon_size)
-    |> assign(:button_size, button_size)
-    |> assign(:text_size, text_size)
-    |> assign(:title_tag, title_tag)
-    |> assign(:children, children)
-    |> assign(:has_children, has_children)
-    |> assign(:is_expanded, is_expanded)
-    |> assign(:count_display, count_display)
+    assigns =
+      assigns
+      |> assign(:margin_left, margin_left)
+      |> assign(:border_class, border_class)
+      |> assign(:icon_size, icon_size)
+      |> assign(:button_size, button_size)
+      |> assign(:text_size, text_size)
+      |> assign(:title_tag, title_tag)
+      |> assign(:children, children)
+      |> assign(:has_children, has_children)
+      |> assign(:is_expanded, is_expanded)
+      |> assign(:count_display, count_display)
 
     ~H"""
     <div class={[@border_class, @depth > 0 && "mt-4"]} style={"margin-left: #{@margin_left}px"}>
@@ -215,12 +225,16 @@ defmodule ComponentsElixirWeb.CategoriesLive do
               <% end %>
             </button>
           <% else %>
-            <div class="w-6"></div> <!-- Spacer for alignment -->
+            <div class="w-6"></div>
+            <!-- Spacer for alignment -->
           <% end %>
 
-          <.icon name="hero-folder" class={"#{@icon_size} #{if @depth == 0, do: "text-primary", else: "text-base-content/60"} mt-0.5"} />
-
-          <!-- Content area - always starts at same vertical position -->
+          <.icon
+            name="hero-folder"
+            class={"#{@icon_size} #{if @depth == 0, do: "text-primary", else: "text-base-content/60"} mt-0.5"}
+          />
+          
+    <!-- Content area - always starts at same vertical position -->
           <div class="flex-1">
             <!-- Minimized view (default) -->
             <%= unless @is_expanded do %>
@@ -268,8 +282,8 @@ defmodule ComponentsElixirWeb.CategoriesLive do
                 </div>
               </div>
             <% end %>
-
-            <!-- Expanded view -->
+            
+    <!-- Expanded view -->
             <%= if @is_expanded do %>
               <div class="flex items-start justify-between">
                 <div class="flex-1">
@@ -321,11 +335,16 @@ defmodule ComponentsElixirWeb.CategoriesLive do
           </div>
         </div>
       </div>
-
-      <!-- Render children recursively (only when expanded) -->
+      
+    <!-- Render children recursively (only when expanded) -->
       <%= if @is_expanded do %>
         <%= for child <- @children do %>
-          <.category_item category={child} categories={@categories} expanded_categories={@expanded_categories} depth={@depth + 1} />
+          <.category_item
+            category={child}
+            categories={@categories}
+            expanded_categories={@expanded_categories}
+            depth={@depth + 1}
+          />
         <% end %>
       <% end %>
     </div>
@@ -368,8 +387,8 @@ defmodule ComponentsElixirWeb.CategoriesLive do
           </div>
         </div>
       </div>
-
-      <!-- Add Category Modal -->
+      
+    <!-- Add Category Modal -->
       <%= if @show_add_form do %>
         <div class="fixed inset-0 bg-base-content/50 overflow-y-auto h-full w-full z-50">
           <div class="relative top-20 mx-auto p-5 border border-base-300 w-11/12 md:w-1/2 shadow-lg rounded-md bg-base-100">
@@ -424,8 +443,8 @@ defmodule ComponentsElixirWeb.CategoriesLive do
           </div>
         </div>
       <% end %>
-
-      <!-- Edit Category Modal -->
+      
+    <!-- Edit Category Modal -->
       <%= if @show_edit_form do %>
         <div class="fixed inset-0 bg-base-content/50 overflow-y-auto h-full w-full z-50">
           <div class="relative top-20 mx-auto p-5 border border-base-300 w-11/12 md:w-1/2 shadow-lg rounded-md bg-base-100">
@@ -480,13 +499,15 @@ defmodule ComponentsElixirWeb.CategoriesLive do
           </div>
         </div>
       <% end %>
-
-      <!-- Categories List -->
+      
+    <!-- Categories List -->
       <div class="max-w-7xl mx-auto px-4 sm:px-6 lg:px-8 py-6">
         <div class="bg-base-100 shadow overflow-hidden sm:rounded-md">
           <div class="px-6 py-4 border-b border-base-300">
             <h2 class="text-lg font-medium text-base-content">Category Hierarchy</h2>
-            <p class="text-sm text-base-content/60 mt-1">Manage your component categories and subcategories</p>
+            <p class="text-sm text-base-content/60 mt-1">
+              Manage your component categories and subcategories
+            </p>
           </div>
 
           <%= if Enum.empty?(@categories) do %>
@@ -501,8 +522,7 @@ defmodule ComponentsElixirWeb.CategoriesLive do
                   phx-click="show_add_form"
                   class="inline-flex items-center px-4 py-2 border border-transparent shadow-sm text-sm font-medium rounded-md text-white bg-indigo-600 hover:bg-indigo-700"
                 >
-                  <.icon name="hero-plus" class="w-4 h-4 mr-2" />
-                  Add Category
+                  <.icon name="hero-plus" class="w-4 h-4 mr-2" /> Add Category
                 </button>
               </div>
             </div>
@@ -511,7 +531,12 @@ defmodule ComponentsElixirWeb.CategoriesLive do
               <!-- Recursive Category Tree -->
               <%= for category <- root_categories(@categories) do %>
                 <div class="px-6 py-4">
-                  <.category_item category={category} categories={@categories} expanded_categories={@expanded_categories} depth={0} />
+                  <.category_item
+                    category={category}
+                    categories={@categories}
+                    expanded_categories={@expanded_categories}
+                    depth={0}
+                  />
                 </div>
               <% end %>
             </div>
diff --git a/lib/components_elixir_web/live/components_live.ex b/lib/components_elixir_web/live/components_live.ex
index f18bd5f..67d604f 100644
--- a/lib/components_elixir_web/live/components_live.ex
+++ b/lib/components_elixir_web/live/components_live.ex
@@ -139,7 +139,11 @@ defmodule ComponentsElixirWeb.ComponentsLive do
      |> push_patch(to: path)}
   end
 
-  def handle_event("storage_location_filter", %{"storage_location_id" => storage_location_id}, socket) do
+  def handle_event(
+        "storage_location_filter",
+        %{"storage_location_id" => storage_location_id},
+        socket
+      ) do
     storage_location_id = String.to_integer(storage_location_id)
     query_string = build_query_params_with_storage_location(socket, storage_location_id)
     path = if query_string == "", do: "/", else: "/?" <> query_string
@@ -387,7 +391,10 @@ defmodule ComponentsElixirWeb.ComponentsLive do
       |> save_uploaded_image(component_params)
       |> save_uploaded_datasheet(socket)
 
-    case Inventory.update_component_with_datasheet(socket.assigns.editing_component, updated_params) do
+    case Inventory.update_component_with_datasheet(
+           socket.assigns.editing_component,
+           updated_params
+         ) do
       {:ok, _component} ->
         {:noreply,
          socket
@@ -496,7 +503,8 @@ defmodule ComponentsElixirWeb.ComponentsLive do
       search: Map.get(overrides, :search, socket.assigns.search),
       criteria: Map.get(overrides, :criteria, socket.assigns.sort_criteria),
       category_id: Map.get(overrides, :category_id, socket.assigns.selected_category),
-      storage_location_id: Map.get(overrides, :storage_location_id, socket.assigns.selected_storage_location)
+      storage_location_id:
+        Map.get(overrides, :storage_location_id, socket.assigns.selected_storage_location)
     }
 
     params
@@ -506,12 +514,14 @@ defmodule ComponentsElixirWeb.ComponentsLive do
 
   defp parse_filter_id(nil), do: nil
   defp parse_filter_id(""), do: nil
+
   defp parse_filter_id(id) when is_binary(id) do
     case Integer.parse(id) do
       {int_id, ""} -> int_id
       _ -> nil
     end
   end
+
   defp parse_filter_id(id) when is_integer(id), do: id
 
   defp build_query_params_with_category(socket, category_id) do
@@ -553,7 +563,7 @@ defmodule ComponentsElixirWeb.ComponentsLive do
   end
 
   defp category_options(categories) do
-    Hierarchical.select_options(categories, &(&1.parent), "Select a category")
+    Hierarchical.select_options(categories, & &1.parent, "Select a category")
   end
 
   defp storage_location_display_name(location) do
@@ -561,7 +571,7 @@ defmodule ComponentsElixirWeb.ComponentsLive do
   end
 
   defp storage_location_options(storage_locations) do
-    Hierarchical.select_options(storage_locations, &(&1.parent), "No storage location")
+    Hierarchical.select_options(storage_locations, & &1.parent, "No storage location")
   end
 
   @impl true
@@ -610,7 +620,7 @@ defmodule ComponentsElixirWeb.ComponentsLive do
           </div>
         </div>
       </div>
-
+      
     <!-- Filters -->
       <div class="max-w-7xl mx-auto px-4 sm:px-6 lg:px-8 py-6">
         <div class="flex flex-col sm:flex-row gap-4">
@@ -687,23 +697,27 @@ defmodule ComponentsElixirWeb.ComponentsLive do
               class="inline-flex items-center px-3 py-2 border border-base-300 text-sm font-medium rounded-md text-base-content bg-base-100 hover:bg-base-200 focus:outline-none focus:ring-2 focus:ring-offset-2 focus:ring-primary"
             >
               <.icon name="hero-adjustments-horizontal" class="w-4 h-4 mr-2" />
-              <%= if @show_advanced_filters, do: "Hide", else: "More" %> Filters
+              {if @show_advanced_filters, do: "Hide", else: "More"} Filters
             </button>
           </div>
         </div>
-
-        <!-- Advanced Filters (Collapsible) -->
+        
+    <!-- Advanced Filters (Collapsible) -->
         <%= if @show_advanced_filters do %>
           <div class="mt-4 p-4 bg-base-100 border border-base-300 rounded-md">
             <div class="flex flex-col sm:flex-row gap-4">
               <div>
-                <label class="block text-sm font-medium text-base-content mb-2">Storage Location</label>
+                <label class="block text-sm font-medium text-base-content mb-2">
+                  Storage Location
+                </label>
                 <form phx-change="storage_location_filter">
                   <select
                     name="storage_location_id"
                     class="block w-full px-3 py-2 border border-base-300 rounded-md shadow-sm bg-base-100 text-base-content focus:outline-none focus:ring-primary focus:border-primary sm:text-sm"
                   >
-                    <option value="" selected={is_nil(@selected_storage_location)}>All Storage Locations</option>
+                    <option value="" selected={is_nil(@selected_storage_location)}>
+                      All Storage Locations
+                    </option>
                     <%= for {location_name, location_id} <- Hierarchical.select_options(@storage_locations, &(&1.parent)) do %>
                       <option value={location_id} selected={@selected_storage_location == location_id}>
                         {location_name}
@@ -716,7 +730,7 @@ defmodule ComponentsElixirWeb.ComponentsLive do
           </div>
         <% end %>
       </div>
-
+      
     <!-- Add Component Modal -->
       <%= if @show_add_form do %>
         <div class="fixed inset-0 bg-base-content/50 overflow-y-auto h-full w-full z-50">
@@ -895,7 +909,7 @@ defmodule ComponentsElixirWeb.ComponentsLive do
           </div>
         </div>
       <% end %>
-
+      
     <!-- Edit Component Modal -->
       <%= if @show_edit_form do %>
         <div class="fixed inset-0 bg-base-content/50 overflow-y-auto h-full w-full z-50">
@@ -1029,8 +1043,7 @@ defmodule ComponentsElixirWeb.ComponentsLive do
                         target="_blank"
                         class="inline-flex items-center text-primary hover:text-primary/80"
                       >
-                        <.icon name="hero-document-text" class="w-4 h-4 mr-1" />
-                        View PDF
+                        <.icon name="hero-document-text" class="w-4 h-4 mr-1" /> View PDF
                       </a>
                     </div>
                   <% end %>
@@ -1097,7 +1110,7 @@ defmodule ComponentsElixirWeb.ComponentsLive do
           </div>
         </div>
       <% end %>
-
+      
     <!-- Components List -->
       <div class="max-w-7xl mx-auto px-4 sm:px-6 lg:px-8 pb-6">
         <div class="bg-base-100 shadow overflow-hidden sm:rounded-md">
@@ -1166,7 +1179,7 @@ defmodule ComponentsElixirWeb.ComponentsLive do
                         </button>
                       </div>
                     </div>
-
+                    
     <!-- Content area with image and details -->
                     <div class="flex gap-6">
                       <!-- Large Image -->
@@ -1192,7 +1205,7 @@ defmodule ComponentsElixirWeb.ComponentsLive do
                           </div>
                         <% end %>
                       </div>
-
+                      
     <!-- Details -->
                       <div class="flex-1 space-y-4 select-text">
                         <!-- Full Description -->
@@ -1201,10 +1214,13 @@ defmodule ComponentsElixirWeb.ComponentsLive do
                             <h4 class="text-sm font-medium text-base-content mb-2">Description</h4>
                             <%!-- Keep interpolation inline to prevent whitespace-pre-wrap from preserving template indentation.
                                  Use phx-no-format so the formatter won't break the layout. --%>
-                            <p phx-no-format class="text-sm text-base-content/70 leading-relaxed whitespace-pre-wrap">{component.description}</p>
+                            <p
+                              phx-no-format
+                              class="text-sm text-base-content/70 leading-relaxed whitespace-pre-wrap"
+                            >{component.description}</p>
                           </div>
                         <% end %>
-
+                        
     <!-- Metadata Grid -->
                         <div class="grid grid-cols-1 md:grid-cols-2 gap-4 text-sm">
                           <%= if component.storage_location do %>
@@ -1261,7 +1277,10 @@ defmodule ComponentsElixirWeb.ComponentsLive do
 
                           <%= if component.datasheet_filename || component.datasheet_url do %>
                             <div class="flex items-start">
-                              <.icon name="hero-document-text" class="w-4 h-4 mr-2 text-base-content/50 mt-0.5 flex-shrink-0" />
+                              <.icon
+                                name="hero-document-text"
+                                class="w-4 h-4 mr-2 text-base-content/50 mt-0.5 flex-shrink-0"
+                              />
                               <div>
                                 <span class="font-medium text-base-content">Datasheet:</span>
                                 <div class="space-y-1 mt-1">
@@ -1284,8 +1303,7 @@ defmodule ComponentsElixirWeb.ComponentsLive do
                                         target="_blank"
                                         class="inline-flex items-center text-base-content/70 hover:text-primary text-sm"
                                       >
-                                        <.icon name="hero-link" class="w-4 h-4 mr-1" />
-                                        Original URL
+                                        <.icon name="hero-link" class="w-4 h-4 mr-1" /> Original URL
                                       </a>
                                     </div>
                                   <% end %>
@@ -1296,7 +1314,7 @@ defmodule ComponentsElixirWeb.ComponentsLive do
                         </div>
                       </div>
                     </div>
-
+                    
     <!-- Action Buttons -->
                     <div class="flex justify-end items-center space-x-2 pt-4 border-t border-base-300">
                       <button
@@ -1390,16 +1408,19 @@ defmodule ComponentsElixirWeb.ComponentsLive do
                           </span>
                         </div>
                       </div>
-
+                      
     <!-- Middle row: Description -->
                       <%= if component.description do %>
                         <div class="mt-1">
                           <%!-- Keep interpolation inline to prevent whitespace-pre-wrap from preserving template indentation.
                                Use phx-no-format so the formatter won't break the layout. --%>
-                          <p phx-no-format class="text-sm text-base-content/70 line-clamp-2 whitespace-pre-wrap">{component.description}</p>
+                          <p
+                            phx-no-format
+                            class="text-sm text-base-content/70 line-clamp-2 whitespace-pre-wrap"
+                          >{component.description}</p>
                         </div>
                       <% end %>
-
+                      
     <!-- Bottom row: Metadata -->
                       <div class="mt-2 grid grid-cols-1 sm:grid-cols-3 gap-x-4 gap-y-1 text-sm text-base-content/60">
                         <%= if component.storage_location do %>
@@ -1427,7 +1448,7 @@ defmodule ComponentsElixirWeb.ComponentsLive do
                           </div>
                         <% end %>
                       </div>
-
+                      
     <!-- Keywords row -->
                       <%= if component.keywords do %>
                         <div class="mt-2">
@@ -1510,7 +1531,7 @@ defmodule ComponentsElixirWeb.ComponentsLive do
       >
         <!-- Background overlay -->
         <div class="absolute inset-0 bg-black bg-opacity-75"></div>
-
+        
     <!-- Modal content -->
         <div
           class="relative bg-base-100 rounded-lg shadow-xl max-w-4xl w-full max-h-full overflow-auto"
@@ -1528,7 +1549,7 @@ defmodule ComponentsElixirWeb.ComponentsLive do
               ×
             </button>
           </div>
-
+          
     <!-- Content -->
           <div class="p-6 bg-base-100 rounded-b-lg">
             <div class="text-center">
diff --git a/lib/components_elixir_web/live/login_live.ex b/lib/components_elixir_web/live/login_live.ex
index 65c4035..8a365fc 100644
--- a/lib/components_elixir_web/live/login_live.ex
+++ b/lib/components_elixir_web/live/login_live.ex
@@ -66,7 +66,7 @@ defmodule ComponentsElixirWeb.LoginLive do
 
           <%= if @error_message do %>
             <div class="text-red-600 text-sm text-center">
-              <%= @error_message %>
+              {@error_message}
             </div>
           <% end %>
 
diff --git a/lib/components_elixir_web/live/storage_locations_live.ex b/lib/components_elixir_web/live/storage_locations_live.ex
index 3ed048f..7fe95d3 100644
--- a/lib/components_elixir_web/live/storage_locations_live.ex
+++ b/lib/components_elixir_web/live/storage_locations_live.ex
@@ -54,12 +54,13 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
   def handle_event("show_edit_form", %{"id" => id}, socket) do
     location = Inventory.get_storage_location!(id)
     # Create a changeset with current values forced into changes for proper form display
-    changeset = Inventory.change_storage_location(location, %{
-      name: location.name,
-      description: location.description,
-      parent_id: location.parent_id
-    })
-    |> Ecto.Changeset.force_change(:parent_id, location.parent_id)
+    changeset =
+      Inventory.change_storage_location(location, %{
+        name: location.name,
+        description: location.description,
+        parent_id: location.parent_id
+      })
+      |> Ecto.Changeset.force_change(:parent_id, location.parent_id)
 
     form = to_form(changeset)
 
@@ -82,29 +83,31 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
 
   def handle_event("save_location", %{"storage_location" => location_params}, socket) do
     # Process AprilTag assignment based on mode
-    processed_params = case socket.assigns.apriltag_mode do
-      "none" ->
-        # Remove any apriltag_id from params to ensure it's nil
-        Map.delete(location_params, "apriltag_id")
+    processed_params =
+      case socket.assigns.apriltag_mode do
+        "none" ->
+          # Remove any apriltag_id from params to ensure it's nil
+          Map.delete(location_params, "apriltag_id")
 
-      "auto" ->
-        # Auto-assign next available AprilTag ID
-        case AprilTag.next_available_apriltag_id() do
-          nil ->
-            # No available IDs, proceed without AprilTag
-            Map.delete(location_params, "apriltag_id")
-          apriltag_id ->
-            Map.put(location_params, "apriltag_id", apriltag_id)
-        end
+        "auto" ->
+          # Auto-assign next available AprilTag ID
+          case AprilTag.next_available_apriltag_id() do
+            nil ->
+              # No available IDs, proceed without AprilTag
+              Map.delete(location_params, "apriltag_id")
 
-      "manual" ->
-        # Use the manually entered apriltag_id (validation will be handled by changeset)
-        location_params
+            apriltag_id ->
+              Map.put(location_params, "apriltag_id", apriltag_id)
+          end
 
-      _ ->
-        # Fallback: remove apriltag_id
-        Map.delete(location_params, "apriltag_id")
-    end
+        "manual" ->
+          # Use the manually entered apriltag_id (validation will be handled by changeset)
+          location_params
+
+        _ ->
+          # Fallback: remove apriltag_id
+          Map.delete(location_params, "apriltag_id")
+      end
 
     case Inventory.create_storage_location(processed_params) do
       {:ok, _location} ->
@@ -147,7 +150,12 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
          |> reload_storage_locations()}
 
       {:error, _changeset} ->
-        {:noreply, put_flash(socket, :error, "Cannot delete storage location - it may have components assigned or child locations")}
+        {:noreply,
+         put_flash(
+           socket,
+           :error,
+           "Cannot delete storage location - it may have components assigned or child locations"
+         )}
     end
   end
 
@@ -164,10 +172,17 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
       {apriltag_id, ""} when apriltag_id >= 0 and apriltag_id <= 586 ->
         case Inventory.get_storage_location_by_apriltag_id(apriltag_id) do
           nil ->
-            {:noreply, put_flash(socket, :error, "Storage location not found for AprilTag ID: #{apriltag_id}")}
+            {:noreply,
+             put_flash(
+               socket,
+               :error,
+               "Storage location not found for AprilTag ID: #{apriltag_id}"
+             )}
 
           location ->
-            scanned_tags = [%{apriltag_id: apriltag_id, location: location} | socket.assigns.scanned_tags]
+            scanned_tags = [
+              %{apriltag_id: apriltag_id, location: location} | socket.assigns.scanned_tags
+            ]
 
             {:noreply,
              socket
@@ -188,11 +203,12 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
     location_id = String.to_integer(id)
     expanded_locations = socket.assigns.expanded_locations
 
-    new_expanded = if MapSet.member?(expanded_locations, location_id) do
-      MapSet.delete(expanded_locations, location_id)
-    else
-      MapSet.put(expanded_locations, location_id)
-    end
+    new_expanded =
+      if MapSet.member?(expanded_locations, location_id) do
+        MapSet.delete(expanded_locations, location_id)
+      else
+        MapSet.put(expanded_locations, location_id)
+      end
 
     {:noreply, assign(socket, :expanded_locations, new_expanded)}
   end
@@ -203,19 +219,26 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
 
   def handle_event("set_edit_apriltag_mode", %{"mode" => mode}, socket) do
     # Clear the apriltag_id field when switching modes
-    form = case mode do
-      "remove" ->
-        socket.assigns.form
-        |> Phoenix.Component.to_form()
-        |> Map.put(:params, Map.put(socket.assigns.form.params || %{}, "apriltag_id", nil))
-      "keep" ->
-        current_id = socket.assigns.editing_location.apriltag_id
-        socket.assigns.form
-        |> Phoenix.Component.to_form()
-        |> Map.put(:params, Map.put(socket.assigns.form.params || %{}, "apriltag_id", current_id))
-      _ ->
-        socket.assigns.form
-    end
+    form =
+      case mode do
+        "remove" ->
+          socket.assigns.form
+          |> Phoenix.Component.to_form()
+          |> Map.put(:params, Map.put(socket.assigns.form.params || %{}, "apriltag_id", nil))
+
+        "keep" ->
+          current_id = socket.assigns.editing_location.apriltag_id
+
+          socket.assigns.form
+          |> Phoenix.Component.to_form()
+          |> Map.put(
+            :params,
+            Map.put(socket.assigns.form.params || %{}, "apriltag_id", current_id)
+          )
+
+        _ ->
+          socket.assigns.form
+      end
 
     {:noreply,
      socket
@@ -234,7 +257,8 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
             {:noreply, put_flash(socket, :error, "Failed to get AprilTag URL")}
 
           apriltag_url ->
-            filename = "#{location.name |> String.replace(" ", "_")}_AprilTag_#{location.apriltag_id}.svg"
+            filename =
+              "#{location.name |> String.replace(" ", "_")}_AprilTag_#{location.apriltag_id}.svg"
 
             # Send file download to browser
             {:noreply,
@@ -257,7 +281,7 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
     Hierarchical.parent_select_options(
       storage_locations,
       editing_location_id,
-      &(&1.parent),
+      & &1.parent,
       "No parent (Root location)"
     )
   end
@@ -267,11 +291,11 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
   end
 
   defp root_storage_locations(storage_locations) do
-    Hierarchical.root_entities(storage_locations, &(&1.parent_id))
+    Hierarchical.root_entities(storage_locations, & &1.parent_id)
   end
 
   defp child_storage_locations(storage_locations, parent_id) do
-    Hierarchical.child_entities(storage_locations, parent_id, &(&1.parent_id))
+    Hierarchical.child_entities(storage_locations, parent_id, & &1.parent_id)
   end
 
   defp count_components_in_location(location_id) do
@@ -291,46 +315,53 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
     border_class = if assigns.depth > 0, do: "border-l-2 border-base-300 pl-6", else: ""
 
     # Icon size and button size based on depth
-    {icon_size, button_size, text_size, title_tag} = case assigns.depth do
-      0 -> {"w-5 h-5", "p-2", "text-lg", "h3"}
-      1 -> {"w-4 h-4", "p-1.5", "text-base", "h4"}
-      _ -> {"w-3 h-3", "p-1", "text-sm", "h5"}
-    end
+    {icon_size, button_size, text_size, title_tag} =
+      case assigns.depth do
+        0 -> {"w-5 h-5", "p-2", "text-lg", "h3"}
+        1 -> {"w-4 h-4", "p-1.5", "text-base", "h4"}
+        _ -> {"w-3 h-3", "p-1", "text-sm", "h5"}
+      end
 
     # Different icons based on level - QR code is always present for storage locations
-    icon_name = case assigns.depth do
-      0 -> "hero-building-office"  # Shelf/Room
-      1 -> "hero-archive-box"      # Drawer/Cabinet
-      _ -> "hero-cube"             # Box/Container
-    end
+    icon_name =
+      case assigns.depth do
+        # Shelf/Room
+        0 -> "hero-building-office"
+        # Drawer/Cabinet
+        1 -> "hero-archive-box"
+        # Box/Container
+        _ -> "hero-cube"
+      end
 
     children = child_storage_locations(assigns.storage_locations, assigns.location.id)
     has_children = !Enum.empty?(children)
     is_expanded = MapSet.member?(assigns.expanded_locations, assigns.location.id)
 
     # Calculate component counts including descendants
-    {self_count, children_count, _total_count} = Hierarchical.count_with_descendants(
-      assigns.location.id,
-      assigns.storage_locations,
-      &(&1.parent_id),
-      &count_components_in_location/1
-    )
+    {self_count, children_count, _total_count} =
+      Hierarchical.count_with_descendants(
+        assigns.location.id,
+        assigns.storage_locations,
+        & &1.parent_id,
+        &count_components_in_location/1
+      )
 
     # Format count display
     count_display = Hierarchical.format_count_display(self_count, children_count, is_expanded)
 
-    assigns = assigns
-    |> assign(:margin_left, margin_left)
-    |> assign(:border_class, border_class)
-    |> assign(:icon_size, icon_size)
-    |> assign(:button_size, button_size)
-    |> assign(:text_size, text_size)
-    |> assign(:title_tag, title_tag)
-    |> assign(:icon_name, icon_name)
-    |> assign(:children, children)
-    |> assign(:has_children, has_children)
-    |> assign(:is_expanded, is_expanded)
-    |> assign(:count_display, count_display)
+    assigns =
+      assigns
+      |> assign(:margin_left, margin_left)
+      |> assign(:border_class, border_class)
+      |> assign(:icon_size, icon_size)
+      |> assign(:button_size, button_size)
+      |> assign(:text_size, text_size)
+      |> assign(:title_tag, title_tag)
+      |> assign(:icon_name, icon_name)
+      |> assign(:children, children)
+      |> assign(:has_children, has_children)
+      |> assign(:is_expanded, is_expanded)
+      |> assign(:count_display, count_display)
 
     ~H"""
     <div class={[@border_class, @depth > 0 && "mt-4"]} style={"margin-left: #{@margin_left}px"}>
@@ -350,12 +381,16 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
               <% end %>
             </button>
           <% else %>
-            <div class="w-6"></div> <!-- Spacer for alignment -->
+            <div class="w-6"></div>
+            <!-- Spacer for alignment -->
           <% end %>
 
-          <.icon name={@icon_name} class={"#{@icon_size} #{if @depth == 0, do: "text-primary", else: "text-base-content/60"} mt-0.5"} />
-
-          <!-- Content area - always starts at same vertical position -->
+          <.icon
+            name={@icon_name}
+            class={"#{@icon_size} #{if @depth == 0, do: "text-primary", else: "text-base-content/60"} mt-0.5"}
+          />
+          
+    <!-- Content area - always starts at same vertical position -->
           <div class="flex-1">
             <!-- Minimized view (default) -->
             <%= unless @is_expanded do %>
@@ -408,8 +443,8 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
                 </div>
               </div>
             <% end %>
-
-            <!-- Expanded view -->
+            
+    <!-- Expanded view -->
             <%= if @is_expanded do %>
               <div class="flex items-start justify-between">
                 <div class="flex-1">
@@ -468,8 +503,7 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
                       class="inline-flex items-center px-3 py-1.5 border border-base-300 rounded-md shadow-sm text-sm font-medium text-base-content bg-base-100 hover:bg-base-200 flex-shrink-0"
                       title="Download AprilTag"
                     >
-                      <.icon name="hero-arrow-down-tray" class="w-4 h-4 mr-1.5" />
-                      Download
+                      <.icon name="hero-arrow-down-tray" class="w-4 h-4 mr-1.5" /> Download
                     </button>
                   <% end %>
                   <div class="flex items-center space-x-2">
@@ -495,11 +529,16 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
           </div>
         </div>
       </div>
-
-      <!-- Render children recursively (only when expanded) -->
+      
+    <!-- Render children recursively (only when expanded) -->
       <%= if @is_expanded do %>
         <%= for child <- @children do %>
-          <.location_item location={child} storage_locations={@storage_locations} expanded_locations={@expanded_locations} depth={@depth + 1} />
+          <.location_item
+            location={child}
+            storage_locations={@storage_locations}
+            expanded_locations={@expanded_locations}
+            depth={@depth + 1}
+          />
         <% end %>
       <% end %>
     </div>
@@ -552,8 +591,8 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
           </div>
         </div>
       </div>
-
-      <!-- Add Location Modal -->
+      
+    <!-- Add Location Modal -->
       <%= if @show_add_form do %>
         <div class="fixed inset-0 bg-base-content/50 overflow-y-auto h-full w-full z-50">
           <div class="relative top-20 mx-auto p-5 border border-base-300 w-11/12 md:w-1/2 shadow-lg rounded-md bg-base-100">
@@ -589,7 +628,9 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
                 </div>
 
                 <div>
-                  <label class="block text-sm font-medium text-base-content">AprilTag ID (Optional)</label>
+                  <label class="block text-sm font-medium text-base-content">
+                    AprilTag ID (Optional)
+                  </label>
                   <div class="space-y-2">
                     <div class="flex items-center space-x-2">
                       <input
@@ -647,10 +688,12 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
                           class="w-32"
                         />
                         <div class="text-xs text-base-content/60">
-                          Available IDs: <%= length(@available_apriltag_ids) %> of 587
+                          Available IDs: {length(@available_apriltag_ids)} of 587
                           <%= if length(@available_apriltag_ids) < 20 do %>
-                            <br/>Next available: <%= @available_apriltag_ids |> Enum.take(10) |> Enum.join(", ") %>
-                            <%= if length(@available_apriltag_ids) > 10, do: "..." %>
+                            <br />Next available: {@available_apriltag_ids
+                            |> Enum.take(10)
+                            |> Enum.join(", ")}
+                            {if length(@available_apriltag_ids) > 10, do: "..."}
                           <% end %>
                         </div>
                       </div>
@@ -678,8 +721,8 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
           </div>
         </div>
       <% end %>
-
-      <!-- Edit Location Modal -->
+      
+    <!-- Edit Location Modal -->
       <%= if @show_edit_form do %>
         <div class="fixed inset-0 bg-base-content/50 overflow-y-auto h-full w-full z-50">
           <div class="relative top-20 mx-auto p-5 border border-base-300 w-11/12 md:w-1/2 shadow-lg rounded-md bg-base-100">
@@ -773,12 +816,14 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
                           class="w-32"
                         />
                         <div class="text-xs text-base-content/60">
-                          Available IDs: <%= length(@available_apriltag_ids) %> of 587
+                          Available IDs: {length(@available_apriltag_ids)} of 587
                         </div>
                       </div>
                     <% end %>
                     <p class="text-xs text-base-content/60 mt-1">
-                      Current: <%= if @editing_location.apriltag_id, do: "ID #{@editing_location.apriltag_id}", else: "None" %>
+                      Current: {if @editing_location.apriltag_id,
+                        do: "ID #{@editing_location.apriltag_id}",
+                        else: "None"}
                     </p>
                   </div>
                 </div>
@@ -803,8 +848,8 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
           </div>
         </div>
       <% end %>
-
-      <!-- AprilTag Scanner Modal -->
+      
+    <!-- AprilTag Scanner Modal -->
       <%= if @apriltag_scanner_open do %>
         <div class="fixed inset-0 bg-base-content/30 bg-opacity-50 overflow-y-auto h-full w-full z-50">
           <div class="relative top-10 mx-auto p-5 border border-base-300 w-11/12 md:w-1/2 shadow-lg rounded-md bg-base-100">
@@ -818,16 +863,20 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
                   <.icon name="hero-x-mark" class="w-6 h-6" />
                 </button>
               </div>
-
-              <!-- AprilTag Scanner Interface -->
+              
+    <!-- AprilTag Scanner Interface -->
               <div class="border-2 border-dashed border-base-300 rounded-lg p-6 text-center">
                 <.icon name="hero-qr-code" class="mx-auto h-12 w-12 text-base-content/50" />
                 <p class="mt-2 text-sm text-base-content/70">Camera AprilTag scanner would go here</p>
-                <p class="text-xs text-base-content/60 mt-1">In a real implementation, this would use JavaScript AprilTag detection</p>
-
-                <!-- Test buttons for demo -->
+                <p class="text-xs text-base-content/60 mt-1">
+                  In a real implementation, this would use JavaScript AprilTag detection
+                </p>
+                
+    <!-- Test buttons for demo -->
                 <div class="mt-4 space-y-2">
-                  <p class="text-sm font-medium text-base-content/80">Test with sample AprilTag IDs:</p>
+                  <p class="text-sm font-medium text-base-content/80">
+                    Test with sample AprilTag IDs:
+                  </p>
                   <button
                     phx-click="apriltag_scanned"
                     phx-value-apriltag_id="0"
@@ -848,8 +897,8 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
           </div>
         </div>
       <% end %>
-
-      <!-- Scanned Tags Display -->
+      
+    <!-- Scanned Tags Display -->
       <%= if length(@scanned_tags) > 0 do %>
         <div class="max-w-7xl mx-auto px-4 sm:px-6 lg:px-8 py-4">
           <div class="bg-green-50 border border-green-200 rounded-lg p-4">
@@ -863,26 +912,35 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
               </button>
             </div>
             <div class="space-y-2">
-              <div :for={scan <- @scanned_tags} class="flex items-center justify-between bg-base-100 p-2 rounded border border-base-300">
+              <div
+                :for={scan <- @scanned_tags}
+                class="flex items-center justify-between bg-base-100 p-2 rounded border border-base-300"
+              >
                 <div>
-                  <span class="font-medium text-base-content">{location_display_name(scan.location)}</span>
-                  <span class="text-sm text-base-content/70 ml-2">(AprilTag ID {scan.apriltag_id})</span>
+                  <span class="font-medium text-base-content">
+                    {location_display_name(scan.location)}
+                  </span>
+                  <span class="text-sm text-base-content/70 ml-2">
+                    (AprilTag ID {scan.apriltag_id})
+                  </span>
                 </div>
                 <span class="text-xs text-green-600 bg-green-100 px-2 py-1 rounded">
-                  Level <%= Hierarchical.compute_level(scan.location, &(&1.parent)) %>
+                  Level {Hierarchical.compute_level(scan.location, & &1.parent)}
                 </span>
               </div>
             </div>
           </div>
         </div>
       <% end %>
-
-      <!-- Storage Locations List -->
+      
+    <!-- Storage Locations List -->
       <div class="max-w-7xl mx-auto px-4 sm:px-6 lg:px-8 py-6">
         <div class="bg-base-100 shadow overflow-hidden sm:rounded-md">
           <div class="px-6 py-4 border-b border-base-300">
             <h2 class="text-lg font-medium text-base-content">Storage Location Hierarchy</h2>
-            <p class="text-sm text-base-content/60 mt-1">Manage your physical storage locations and AprilTags</p>
+            <p class="text-sm text-base-content/60 mt-1">
+              Manage your physical storage locations and AprilTags
+            </p>
           </div>
 
           <%= if Enum.empty?(@storage_locations) do %>
@@ -897,8 +955,7 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
                   phx-click="show_add_form"
                   class="inline-flex items-center px-4 py-2 border border-transparent shadow-sm text-sm font-medium rounded-md text-white bg-indigo-600 hover:bg-indigo-700"
                 >
-                  <.icon name="hero-plus" class="w-4 h-4 mr-2" />
-                  Add Location
+                  <.icon name="hero-plus" class="w-4 h-4 mr-2" /> Add Location
                 </button>
               </div>
             </div>
@@ -907,7 +964,12 @@ defmodule ComponentsElixirWeb.StorageLocationsLive do
               <!-- Recursive Storage Location Tree -->
               <%= for location <- root_storage_locations(@storage_locations) do %>
                 <div class="px-6 py-4">
-                  <.location_item location={location} storage_locations={@storage_locations} expanded_locations={@expanded_locations} depth={0} />
+                  <.location_item
+                    location={location}
+                    storage_locations={@storage_locations}
+                    expanded_locations={@expanded_locations}
+                    depth={0}
+                  />
                 </div>
               <% end %>
             </div>
diff --git a/lib/mix/tasks/apriltag.generate_all.ex b/lib/mix/tasks/apriltag.generate_all.ex
index e69a7ae..5f0b886 100644
--- a/lib/mix/tasks/apriltag.generate_all.ex
+++ b/lib/mix/tasks/apriltag.generate_all.ex
@@ -25,9 +25,8 @@ defmodule Mix.Tasks.Apriltag.GenerateAll do
 
     start_time = System.monotonic_time(:millisecond)
 
-    result = ComponentsElixir.AprilTag.generate_all_apriltag_svgs(
-      force_regenerate: force_regenerate
-    )
+    result =
+      ComponentsElixir.AprilTag.generate_all_apriltag_svgs(force_regenerate: force_regenerate)
 
     end_time = System.monotonic_time(:millisecond)
     duration = end_time - start_time
@@ -39,6 +38,7 @@ defmodule Mix.Tasks.Apriltag.GenerateAll do
 
     if result.errors > 0 do
       IO.puts("\nErrors encountered:")
+
       result.results
       |> Enum.filter(&match?({:error, _, _}, &1))
       |> Enum.each(fn {:error, id, reason} ->
diff --git a/priv/repo/migrations/20250914160354_migrate_qr_to_apriltag.exs b/priv/repo/migrations/20250914160354_migrate_qr_to_apriltag.exs
index 3a55848..baa1b2e 100644
--- a/priv/repo/migrations/20250914160354_migrate_qr_to_apriltag.exs
+++ b/priv/repo/migrations/20250914160354_migrate_qr_to_apriltag.exs
@@ -14,7 +14,9 @@ defmodule ComponentsElixir.Repo.Migrations.MigrateQrToApriltag do
     create unique_index(:storage_locations, [:apriltag_id])
 
     # Add constraint to ensure apriltag_id is in valid range (0-586 for tag36h11)
-    create constraint(:storage_locations, :apriltag_id_range, check: "apriltag_id >= 0 AND apriltag_id <= 586")
+    create constraint(:storage_locations, :apriltag_id_range,
+             check: "apriltag_id >= 0 AND apriltag_id <= 586"
+           )
 
     # Note: We keep qr_code_old for now in case we need to rollback
     # It can be removed in a future migration after confirming everything works
diff --git a/priv/repo/seeds.exs b/priv/repo/seeds.exs
index 81a75a5..269e4c4 100644
--- a/priv/repo/seeds.exs
+++ b/priv/repo/seeds.exs
@@ -25,102 +25,208 @@ Repo.delete_all(Category)
 Repo.delete_all(StorageLocation)
 
 # Create categories
-{:ok, resistors} = Inventory.create_category(%{name: "Resistors", description: "Various types of resistors"})
-{:ok, capacitors} = Inventory.create_category(%{name: "Capacitors", description: "Electrolytic, ceramic, and film capacitors"})
-{:ok, semiconductors} = Inventory.create_category(%{name: "Semiconductors", description: "ICs, transistors, diodes"})
-{:ok, connectors} = Inventory.create_category(%{name: "Connectors", description: "Headers, terminals, plugs"})
+{:ok, resistors} =
+  Inventory.create_category(%{name: "Resistors", description: "Various types of resistors"})
+
+{:ok, capacitors} =
+  Inventory.create_category(%{
+    name: "Capacitors",
+    description: "Electrolytic, ceramic, and film capacitors"
+  })
+
+{:ok, semiconductors} =
+  Inventory.create_category(%{name: "Semiconductors", description: "ICs, transistors, diodes"})
+
+{:ok, connectors} =
+  Inventory.create_category(%{name: "Connectors", description: "Headers, terminals, plugs"})
 
 # Create subcategories
-{:ok, _through_hole_resistors} = Inventory.create_category(%{
-  name: "Through-hole",
-  description: "Traditional leaded resistors",
-  parent_id: resistors.id
-})
+{:ok, _through_hole_resistors} =
+  Inventory.create_category(%{
+    name: "Through-hole",
+    description: "Traditional leaded resistors",
+    parent_id: resistors.id
+  })
 
-{:ok, _smd_resistors} = Inventory.create_category(%{
-  name: "SMD/SMT",
-  description: "Surface mount resistors",
-  parent_id: resistors.id
-})
+{:ok, _smd_resistors} =
+  Inventory.create_category(%{
+    name: "SMD/SMT",
+    description: "Surface mount resistors",
+    parent_id: resistors.id
+  })
 
-{:ok, _ceramic_caps} = Inventory.create_category(%{
-  name: "Ceramic",
-  description: "Ceramic disc and multilayer capacitors",
-  parent_id: capacitors.id
-})
+{:ok, _ceramic_caps} =
+  Inventory.create_category(%{
+    name: "Ceramic",
+    description: "Ceramic disc and multilayer capacitors",
+    parent_id: capacitors.id
+  })
 
-{:ok, _electrolytic_caps} = Inventory.create_category(%{
-  name: "Electrolytic",
-  description: "Polarized electrolytic capacitors",
-  parent_id: capacitors.id
-})
+{:ok, _electrolytic_caps} =
+  Inventory.create_category(%{
+    name: "Electrolytic",
+    description: "Polarized electrolytic capacitors",
+    parent_id: capacitors.id
+  })
 
 # Create a DEEP category hierarchy to test fallback path (7+ levels)
-{:ok, deep_cat_1} = Inventory.create_category(%{name: "Level 1", description: "Deep hierarchy test", parent_id: resistors.id})
-{:ok, deep_cat_2} = Inventory.create_category(%{name: "Level 2", description: "Deep hierarchy test", parent_id: deep_cat_1.id})
-{:ok, deep_cat_3} = Inventory.create_category(%{name: "Level 3", description: "Deep hierarchy test", parent_id: deep_cat_2.id})
-{:ok, deep_cat_4} = Inventory.create_category(%{name: "Level 4", description: "Deep hierarchy test", parent_id: deep_cat_3.id})
-{:ok, deep_cat_5} = Inventory.create_category(%{name: "Level 5", description: "Deep hierarchy test", parent_id: deep_cat_4.id})
-{:ok, deep_cat_6} = Inventory.create_category(%{name: "Level 6", description: "Deep hierarchy test", parent_id: deep_cat_5.id})
-{:ok, deep_cat_7} = Inventory.create_category(%{name: "Level 7", description: "Deep hierarchy test - triggers fallback", parent_id: deep_cat_6.id})
+{:ok, deep_cat_1} =
+  Inventory.create_category(%{
+    name: "Level 1",
+    description: "Deep hierarchy test",
+    parent_id: resistors.id
+  })
+
+{:ok, deep_cat_2} =
+  Inventory.create_category(%{
+    name: "Level 2",
+    description: "Deep hierarchy test",
+    parent_id: deep_cat_1.id
+  })
+
+{:ok, deep_cat_3} =
+  Inventory.create_category(%{
+    name: "Level 3",
+    description: "Deep hierarchy test",
+    parent_id: deep_cat_2.id
+  })
+
+{:ok, deep_cat_4} =
+  Inventory.create_category(%{
+    name: "Level 4",
+    description: "Deep hierarchy test",
+    parent_id: deep_cat_3.id
+  })
+
+{:ok, deep_cat_5} =
+  Inventory.create_category(%{
+    name: "Level 5",
+    description: "Deep hierarchy test",
+    parent_id: deep_cat_4.id
+  })
+
+{:ok, deep_cat_6} =
+  Inventory.create_category(%{
+    name: "Level 6",
+    description: "Deep hierarchy test",
+    parent_id: deep_cat_5.id
+  })
+
+{:ok, deep_cat_7} =
+  Inventory.create_category(%{
+    name: "Level 7",
+    description: "Deep hierarchy test - triggers fallback",
+    parent_id: deep_cat_6.id
+  })
 
 # Create storage locations
-{:ok, shelf_a} = Inventory.create_storage_location(%{name: "Shelf A", description: "Main electronics shelf"})
-{:ok, _shelf_b} = Inventory.create_storage_location(%{name: "Shelf B", description: "Components overflow shelf"})
+{:ok, shelf_a} =
+  Inventory.create_storage_location(%{name: "Shelf A", description: "Main electronics shelf"})
+
+{:ok, _shelf_b} =
+  Inventory.create_storage_location(%{name: "Shelf B", description: "Components overflow shelf"})
 
 # Create drawers on Shelf A
-{:ok, drawer_a1} = Inventory.create_storage_location(%{
-  name: "Drawer 1",
-  description: "Resistors and capacitors",
-  parent_id: shelf_a.id
-})
+{:ok, drawer_a1} =
+  Inventory.create_storage_location(%{
+    name: "Drawer 1",
+    description: "Resistors and capacitors",
+    parent_id: shelf_a.id
+  })
 
-{:ok, drawer_a2} = Inventory.create_storage_location(%{
-  name: "Drawer 2",
-  description: "Semiconductors and ICs",
-  parent_id: shelf_a.id
-})
+{:ok, drawer_a2} =
+  Inventory.create_storage_location(%{
+    name: "Drawer 2",
+    description: "Semiconductors and ICs",
+    parent_id: shelf_a.id
+  })
 
 # Create boxes in Drawer A1
-{:ok, box_a1_1} = Inventory.create_storage_location(%{
-  name: "Box 1",
-  description: "Through-hole resistors",
-  parent_id: drawer_a1.id
-})
+{:ok, box_a1_1} =
+  Inventory.create_storage_location(%{
+    name: "Box 1",
+    description: "Through-hole resistors",
+    parent_id: drawer_a1.id
+  })
 
-{:ok, _box_a1_2} = Inventory.create_storage_location(%{
-  name: "Box 2",
-  description: "SMD resistors",
-  parent_id: drawer_a1.id
-})
+{:ok, _box_a1_2} =
+  Inventory.create_storage_location(%{
+    name: "Box 2",
+    description: "SMD resistors",
+    parent_id: drawer_a1.id
+  })
 
-{:ok, box_a1_3} = Inventory.create_storage_location(%{
-  name: "Box 3",
-  description: "Ceramic capacitors",
-  parent_id: drawer_a1.id
-})
+{:ok, box_a1_3} =
+  Inventory.create_storage_location(%{
+    name: "Box 3",
+    description: "Ceramic capacitors",
+    parent_id: drawer_a1.id
+  })
 
 # Create boxes in Drawer A2
-{:ok, box_a2_1} = Inventory.create_storage_location(%{
-  name: "Box 1",
-  description: "Microcontrollers",
-  parent_id: drawer_a2.id
-})
+{:ok, box_a2_1} =
+  Inventory.create_storage_location(%{
+    name: "Box 1",
+    description: "Microcontrollers",
+    parent_id: drawer_a2.id
+  })
 
-{:ok, _box_a2_2} = Inventory.create_storage_location(%{
-  name: "Box 2",
-  description: "Transistors and diodes",
-  parent_id: drawer_a2.id
-})
+{:ok, _box_a2_2} =
+  Inventory.create_storage_location(%{
+    name: "Box 2",
+    description: "Transistors and diodes",
+    parent_id: drawer_a2.id
+  })
 
 # Create a DEEP storage location hierarchy to test fallback path (7+ levels)
-{:ok, deep_loc_1} = Inventory.create_storage_location(%{name: "Deep Level 1", description: "Deep hierarchy test", parent_id: box_a1_3.id})
-{:ok, deep_loc_2} = Inventory.create_storage_location(%{name: "Deep Level 2", description: "Deep hierarchy test", parent_id: deep_loc_1.id})
-{:ok, deep_loc_3} = Inventory.create_storage_location(%{name: "Deep Level 3", description: "Deep hierarchy test", parent_id: deep_loc_2.id})
-{:ok, deep_loc_4} = Inventory.create_storage_location(%{name: "Deep Level 4", description: "Deep hierarchy test", parent_id: deep_loc_3.id})
-{:ok, deep_loc_5} = Inventory.create_storage_location(%{name: "Deep Level 5", description: "Deep hierarchy test", parent_id: deep_loc_4.id})
-{:ok, deep_loc_6} = Inventory.create_storage_location(%{name: "Deep Level 6", description: "Deep hierarchy test", parent_id: deep_loc_5.id})
-{:ok, deep_loc_7} = Inventory.create_storage_location(%{name: "Deep Level 7", description: "Deep hierarchy test - triggers fallback", parent_id: deep_loc_6.id})
+{:ok, deep_loc_1} =
+  Inventory.create_storage_location(%{
+    name: "Deep Level 1",
+    description: "Deep hierarchy test",
+    parent_id: box_a1_3.id
+  })
+
+{:ok, deep_loc_2} =
+  Inventory.create_storage_location(%{
+    name: "Deep Level 2",
+    description: "Deep hierarchy test",
+    parent_id: deep_loc_1.id
+  })
+
+{:ok, deep_loc_3} =
+  Inventory.create_storage_location(%{
+    name: "Deep Level 3",
+    description: "Deep hierarchy test",
+    parent_id: deep_loc_2.id
+  })
+
+{:ok, deep_loc_4} =
+  Inventory.create_storage_location(%{
+    name: "Deep Level 4",
+    description: "Deep hierarchy test",
+    parent_id: deep_loc_3.id
+  })
+
+{:ok, deep_loc_5} =
+  Inventory.create_storage_location(%{
+    name: "Deep Level 5",
+    description: "Deep hierarchy test",
+    parent_id: deep_loc_4.id
+  })
+
+{:ok, deep_loc_6} =
+  Inventory.create_storage_location(%{
+    name: "Deep Level 6",
+    description: "Deep hierarchy test",
+    parent_id: deep_loc_5.id
+  })
+
+{:ok, deep_loc_7} =
+  Inventory.create_storage_location(%{
+    name: "Deep Level 7",
+    description: "Deep hierarchy test - triggers fallback",
+    parent_id: deep_loc_6.id
+  })
 
 # Create sample components
 sample_components = [
@@ -162,7 +268,8 @@ sample_components = [
     keywords: "microcontroller avr atmega328 arduino",
     storage_location_id: box_a2_1.id,
     count: 10,
-    datasheet_url: "https://ww1.microchip.com/downloads/en/DeviceDoc/Atmel-7810-Automotive-Microcontrollers-ATmega328P_Datasheet.pdf",
+    datasheet_url:
+      "https://ww1.microchip.com/downloads/en/DeviceDoc/Atmel-7810-Automotive-Microcontrollers-ATmega328P_Datasheet.pdf",
     category_id: semiconductors.id
   },
   %{
@@ -264,7 +371,11 @@ IO.puts("")
 IO.puts("🎉 Database seeded successfully!")
 IO.puts("📊 Summary:")
 IO.puts("  Categories: #{length(Inventory.list_categories())}")
-IO.puts("  Storage Locations: #{length(Inventory.list_storage_locations())} (with auto-assigned AprilTags)")
+
+IO.puts(
+  "  Storage Locations: #{length(Inventory.list_storage_locations())} (with auto-assigned AprilTags)"
+)
+
 IO.puts("  Components: #{length(Inventory.list_components())}")
 IO.puts("")
 IO.puts("🏷️  AprilTag System:")
diff --git a/test/components_elixir_web/controllers/error_html_test.exs b/test/components_elixir_web/controllers/error_html_test.exs
index f98a9d0..44a8c86 100644
--- a/test/components_elixir_web/controllers/error_html_test.exs
+++ b/test/components_elixir_web/controllers/error_html_test.exs
@@ -9,6 +9,7 @@ defmodule ComponentsElixirWeb.ErrorHTMLTest do
   end
 
   test "renders 500.html" do
-    assert render_to_string(ComponentsElixirWeb.ErrorHTML, "500", "html", []) == "Internal Server Error"
+    assert render_to_string(ComponentsElixirWeb.ErrorHTML, "500", "html", []) ==
+             "Internal Server Error"
   end
 end
diff --git a/test/components_elixir_web/controllers/error_json_test.exs b/test/components_elixir_web/controllers/error_json_test.exs
index ca3abdb..deb11b5 100644
--- a/test/components_elixir_web/controllers/error_json_test.exs
+++ b/test/components_elixir_web/controllers/error_json_test.exs
@@ -2,7 +2,9 @@ defmodule ComponentsElixirWeb.ErrorJSONTest do
   use ComponentsElixirWeb.ConnCase, async: true
 
   test "renders 404" do
-    assert ComponentsElixirWeb.ErrorJSON.render("404.json", %{}) == %{errors: %{detail: "Not Found"}}
+    assert ComponentsElixirWeb.ErrorJSON.render("404.json", %{}) == %{
+             errors: %{detail: "Not Found"}
+           }
   end
 
   test "renders 500" do