Chore: Bump required liveview version to v1.1.7 (#974)

* remove old process api

* adjust for 1.1 telemetry

* remove versions.ex

* update workflow mix.lock

* format

* remove unused api

* update deps

* cr suggestions

* replace backward-compatibility with matrix

* Remove alias

* fix formatting

* update mix.lock

* Bump required version of LiveView to 1.1.7

* Change phoenix to 1.7.0 in backward compatibility test

---------

Co-authored-by: Krzysztof Nalepa <56135216+kraleppa@users.noreply.github.com>
Co-authored-by: kraleppa <krzysztof.nalepa@swmansion.com>

Author: srzeszut

.github/workflows/assets/mix.lock

@@ -23,9 +23,21 @@ concurrency:
 
 jobs:
   build:
-    name: Build and test
+    name: Build and test (Elixir ${{ matrix.elixir-version }})
     runs-on: ubuntu-24.04
 
+    strategy:
+      fail-fast: false
+      matrix:
+        include:
+          - elixir-version: "1.19.1"
+            otp-version: "28.1.1"
+            backward-compatibility: false
+
+          - elixir-version: "1.16.0"
+            otp-version: "26.0.2"
+            backward-compatibility: true
+
     steps:
       - uses: actions/checkout@v4
 
@@ -35,17 +47,21 @@ jobs:
       - name: Set up Elixir
         uses: erlef/setup-beam@v1.20.4
         with:
-          elixir-version: "1.19.1" # [Required] Define the Elixir version
-          otp-version: "28.1.1" # [Required] Define the Erlang/OTP version
+          elixir-version: ${{ matrix.elixir-version }}
+          otp-version: ${{ matrix.otp-version }}
+
+      - name: Copy mix.lock for backward compatibility
+        if: ${{ matrix.backward-compatibility }}
+        run: cp ./.github/workflows/assets/mix.lock mix.lock
 
       - name: Cache Mix
         uses: actions/cache@v4
         with:
           path: |
             deps
             _build
-          key: ${{ runner.os }}-mix-${{ hashFiles('**/mix.lock') }}
-          restore-keys: ${{ runner.os }}-mix-
+          key: ${{ runner.os }}-mix-${{ matrix.elixir-version }}-${{ matrix.otp-version }}-${{ hashFiles('**/mix.lock') }}
+          restore-keys: ${{ runner.os }}-mix-${{ matrix.elixir-version }}-${{ matrix.otp-version }}-
 
       - name: Install Dependencies
         run: mix deps.get

.github/workflows/backward-compatibility-ci.yaml

@@ -50,75 +50,11 @@ defmodule LiveDebugger.API.LiveViewDebug do
     @moduledoc false
     @behaviour LiveDebugger.API.LiveViewDebug
 
-    if LiveDebugger.API.System.Module.loaded?(Phoenix.LiveView.Debug) do
-      @impl true
-      defdelegate list_liveviews(), to: Phoenix.LiveView.Debug
-      @impl true
-      defdelegate socket(pid), to: Phoenix.LiveView.Debug
-      @impl true
-      defdelegate live_components(pid), to: Phoenix.LiveView.Debug
-    else
-      alias LiveDebugger.API.System.Process, as: ProcessAPI
-
-      @impl true
-      def list_liveviews() do
-        ProcessAPI.list()
-        |> Enum.filter(&liveview?/1)
-        |> Enum.map(fn pid ->
-          case ProcessAPI.state(pid) do
-            {:ok, %{socket: socket, topic: topic}} ->
-              %{
-                pid: pid,
-                view: socket.view,
-                topic: topic,
-                transport_pid: socket.transport_pid
-              }
-
-            {:error, _} ->
-              nil
-          end
-        end)
-        |> Enum.reject(&is_nil/1)
-      end
-
-      @impl true
-      def socket(pid) do
-        case ProcessAPI.state(pid) do
-          {:ok, %{socket: %Phoenix.LiveView.Socket{} = socket}} -> {:ok, socket}
-          _ -> {:error, :not_alive_or_not_a_liveview}
-        end
-      end
-
-      @impl true
-      def live_components(pid) do
-        case ProcessAPI.state(pid) do
-          {:ok, %{components: {components, _, _}}} ->
-            component_info =
-              Enum.map(components, fn {cid, {mod, id, assigns, private, _prints}} ->
-                %{
-                  id: id,
-                  cid: cid,
-                  module: mod,
-                  assigns: assigns,
-                  private: private,
-                  children_cids: private.children_cids
-                }
-              end)
-
-            {:ok, component_info}
-
-          _ ->
-            {:error, :not_alive_or_not_a_liveview}
-        end
-      end
-
-      @spec liveview?(pid :: pid()) :: boolean()
-      defp liveview?(pid) do
-        case ProcessAPI.initial_call(pid) do
-          {:ok, {_module, :mount, _arity}} -> true
-          _ -> false
-        end
-      end
-    end
+    @impl true
+    defdelegate list_liveviews(), to: Phoenix.LiveView.Debug
+    @impl true
+    defdelegate socket(pid), to: Phoenix.LiveView.Debug
+    @impl true
+    defdelegate live_components(pid), to: Phoenix.LiveView.Debug
   end
 end

.github/workflows/elixir-ci.yaml

@@ -324,10 +324,7 @@ defmodule LiveDebugger.App.Debugger.CallbackTracing.Web.Components.Trace do
       </:header>
 
       <div class="relative">
-        <div
-          :if={is_nil(@displayed_trace.error)}
-          class="absolute right-4 top-0 z-10"
-        >
+        <div :if={is_nil(@displayed_trace.error)} class="absolute right-4 top-0 z-10">
           <.open_in_editor_button
             id={@id <> "-fullscreen"}
             elixir_editor={@elixir_editor}

lib/live_debugger/api/live_view_debug.ex

@@ -3,7 +3,6 @@ defmodule LiveDebugger.App.Debugger.Streams.StreamUtils do
   Utilities for extracting Phoenix.LiveView.Stream diffs
   from render traces and mapping them into a list of functions.
   """
-  alias LiveDebugger.Utils.Versions
 
   @type live_stream_item :: %Phoenix.LiveView.LiveStream{
           name: atom(),
@@ -237,9 +236,5 @@ defmodule LiveDebugger.App.Debugger.Streams.StreamUtils do
     end)
   end
 
-  if Versions.live_view_streams_order_changed?() do
-    defp adjust_inserts(inserts), do: Enum.reverse(inserts)
-  else
-    defp adjust_inserts(inserts), do: inserts
-  end
+  defp adjust_inserts(inserts), do: Enum.reverse(inserts)
 end

lib/live_debugger/api/system/process.ex

@@ -89,10 +89,7 @@ defmodule LiveDebugger.App.Debugger.Web.DebuggerLive do
               <Navbar.garbage_collection_warning />
 
               <Navbar.settings_button return_to={@url} />
-              <span
-                :if={@show_sidebar_icon?}
-                class="h-5 border-r border-default-border md_ct:hidden"
-              >
+              <span :if={@show_sidebar_icon?} class="h-5 border-r border-default-border md_ct:hidden">
               </span>
 
               <.nav_icon

lib/live_debugger/app/debugger/callback_tracing/web/components/trace.ex

@@ -32,11 +32,7 @@ defmodule LiveDebugger.App.Discovery.Web.DiscoveryLive do
     ~H"""
     <div class="h-full flex-1 min-w-[25rem] grid grid-rows-[auto_1fr]">
       <div>
-        <.live_component
-          module={TracerStatus}
-          id="tracer-status"
-          tracer_started?={@tracer_started?}
-        />
+        <.live_component module={TracerStatus} id="tracer-status" tracer_started?={@tracer_started?} />
         <NavbarComponents.navbar class="flex justify-between">
           <NavbarComponents.live_debugger_logo />
           <div class="flex items-center gap-2">

lib/live_debugger/app/debugger/streams/stream_utils.ex

@@ -46,11 +46,7 @@ defmodule LiveDebugger.App.Settings.Web.SettingsLive do
     ~H"""
     <div class="flex-1 min-w-[25rem] grid grid-rows-[auto_1fr]">
       <div>
-        <.live_component
-          module={TracerStatus}
-          id="tracer-status"
-          tracer_started?={@tracer_started?}
-        />
+        <.live_component module={TracerStatus} id="tracer-status" tracer_started?={@tracer_started?} />
         <NavbarComponents.navbar class="flex pl-2 justify-between">
           <div class="flex items-center gap-2">
             <NavbarComponents.return_link return_link={@return_to || RoutesHelper.discovery()} />