show proper 404 page for missing custom pages instead of redirecting

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

lib/berrypod_web/live/shop/custom_page.ex

@@ -15,23 +15,13 @@ defmodule BerrypodWeb.Shop.CustomPage do
     cond do
       is_nil(page) ->
         record_broken_url("/#{slug}")
-
+        raise BerrypodWeb.NotFoundError
-        {:noreply,
-         socket
-         |> put_flash(:error, "Page not found")
-         |> push_navigate(to: ~p"/")}
 
       page.type != "custom" ->
-        {:noreply,
+        raise BerrypodWeb.NotFoundError
-         socket
-         |> put_flash(:error, "Page not found")
-         |> push_navigate(to: ~p"/")}
 
       page.published != true and not socket.assigns.is_admin ->
-        {:noreply,
+        raise BerrypodWeb.NotFoundError
-         socket
-         |> put_flash(:error, "Page not found")
-         |> push_navigate(to: ~p"/")}
 
       true ->
         extra = Pages.load_block_data(page.blocks, socket.assigns)

lib/berrypod_web/not_found_error.ex

@@ -0,0 +1,10 @@
+defmodule BerrypodWeb.NotFoundError do
+  @moduledoc """
+  Raised in LiveViews when a resource can't be found.
+
+  Implements `Plug.Exception` so Phoenix renders the 404 error page
+  instead of a 500.
+  """
+
+  defexception message: "not found", plug_status: 404
+end

test/berrypod_web/live/shop/custom_page_test.exs

@@ -49,8 +49,10 @@ defmodule BerrypodWeb.Shop.CustomPageTest do
       :ok
     end
 
-    test "redirects anonymous users to home", %{conn: conn} do
+    test "returns 404 for anonymous users", %{conn: conn} do
-      {:error, {:live_redirect, %{to: "/"}}} = live(conn, "/draft-page")
+      assert_raise BerrypodWeb.NotFoundError, fn ->
+        live(conn, "/draft-page")
+      end
     end
 
     test "renders for admin users", %{conn: conn, user: user} do
@@ -61,8 +63,10 @@ defmodule BerrypodWeb.Shop.CustomPageTest do
   end
 
   describe "nonexistent page" do
-    test "redirects to home with flash", %{conn: conn} do
+    test "returns 404", %{conn: conn} do
-      {:error, {:live_redirect, %{to: "/"}}} = live(conn, "/does-not-exist")
+      assert_raise BerrypodWeb.NotFoundError, fn ->
+        live(conn, "/does-not-exist")
+      end
     end
   end