From 62b8ec7afdd3d741fd28b3aeea0ba7af50ab7703 Mon Sep 17 00:00:00 2001
From: Graham McIntire <graham@mcintire.me>
Date: Fri, 13 Feb 2026 08:58:19 -0600
Subject: [PATCH] add contextual insights to device preseem tab

---
 lib/towerops_web/live/device_live/show.ex     | 21 ++++++-
 .../live/device_live/show.html.heex           | 57 +++++++++++++++++++
 2 files changed, 75 insertions(+), 3 deletions(-)

diff --git a/lib/towerops_web/live/device_live/show.ex b/lib/towerops_web/live/device_live/show.ex
index 9a93f179..905c69c3 100644
--- a/lib/towerops_web/live/device_live/show.ex
+++ b/lib/towerops_web/live/device_live/show.ex
@@ -436,16 +436,20 @@ defmodule ToweropsWeb.DeviceLive.Show do
     device = socket.assigns.device
     preseem_ap = Towerops.Preseem.get_access_point_for_device(device.id)
 
-    metrics =
+    {metrics, insights} =
       if preseem_ap do
-        Towerops.Preseem.list_subscriber_metrics(preseem_ap.id, limit: 50)
+        {
+          Towerops.Preseem.list_subscriber_metrics(preseem_ap.id, limit: 50),
+          Towerops.Preseem.list_device_insights(device.id)
+        }
       else
-        []
+        {[], []}
       end
 
     socket
     |> assign(:preseem_access_point, preseem_ap)
     |> assign(:preseem_metrics, metrics)
+    |> assign(:preseem_insights, insights)
   end
 
   defp get_available_firmware(nil), do: nil
@@ -1222,6 +1226,17 @@ defmodule ToweropsWeb.DeviceLive.Show do
     {:noreply, assign(socket, :selected_backup_ids, MapSet.new())}
   end
 
+  @impl true
+  def handle_event("dismiss_insight", %{"id" => insight_id}, socket) do
+    case Towerops.Preseem.dismiss_insight(insight_id) do
+      {:ok, _} ->
+        {:noreply, assign_preseem_data(socket)}
+
+      {:error, _} ->
+        {:noreply, put_flash(socket, :error, "Failed to dismiss insight")}
+    end
+  end
+
   @impl true
   def handle_event("delete_backup", %{"id" => backup_id}, socket) do
     import ToweropsWeb.Permissions
diff --git a/lib/towerops_web/live/device_live/show.html.heex b/lib/towerops_web/live/device_live/show.html.heex
index a0c643d8..7ce92a16 100644
--- a/lib/towerops_web/live/device_live/show.html.heex
+++ b/lib/towerops_web/live/device_live/show.html.heex
@@ -2206,6 +2206,63 @@
       <% "preseem" -> %>
         <%= if @preseem_access_point do %>
           <div class="space-y-6">
+            <%!-- Insights section --%>
+            <%= if @preseem_insights != [] do %>
+              <div class="space-y-3">
+                <%= for insight <- @preseem_insights do %>
+                  <div class={[
+                    "rounded-lg border p-4 flex items-start justify-between",
+                    case insight.urgency do
+                      "critical" ->
+                        "bg-red-50 border-red-200 dark:bg-red-900/20 dark:border-red-800"
+
+                      "warning" ->
+                        "bg-yellow-50 border-yellow-200 dark:bg-yellow-900/20 dark:border-yellow-800"
+
+                      _ ->
+                        "bg-blue-50 border-blue-200 dark:bg-blue-900/20 dark:border-blue-800"
+                    end
+                  ]}>
+                    <div class="flex items-start gap-3">
+                      <div class={[
+                        "mt-0.5",
+                        case insight.urgency do
+                          "critical" -> "text-red-500"
+                          "warning" -> "text-yellow-500"
+                          _ -> "text-blue-500"
+                        end
+                      ]}>
+                        <%= case insight.urgency do %>
+                          <% "critical" -> %>
+                            <.icon name="hero-exclamation-triangle" class="h-5 w-5" />
+                          <% "warning" -> %>
+                            <.icon name="hero-exclamation-circle" class="h-5 w-5" />
+                          <% _ -> %>
+                            <.icon name="hero-information-circle" class="h-5 w-5" />
+                        <% end %>
+                      </div>
+                      <div>
+                        <p class="text-sm font-medium text-gray-900 dark:text-white">
+                          {insight.title}
+                        </p>
+                        <p class="mt-1 text-sm text-gray-600 dark:text-gray-300">
+                          {insight.description}
+                        </p>
+                      </div>
+                    </div>
+                    <button
+                      phx-click="dismiss_insight"
+                      phx-value-id={insight.id}
+                      class="ml-4 text-gray-400 hover:text-gray-600 dark:hover:text-gray-300 flex-shrink-0"
+                      title="Dismiss"
+                    >
+                      <.icon name="hero-x-mark" class="h-4 w-4" />
+                    </button>
+                  </div>
+                <% end %>
+              </div>
+            <% end %>
+
             <%!-- Score cards --%>
             <div class="grid grid-cols-1 sm:grid-cols-3 gap-4">
               <div class="bg-white dark:bg-gray-800/50 rounded-lg border border-gray-200 dark:border-white/10 p-4">