fix: unable to clear runs from history & url params rewrite (#4423)

* fix: url paths for runs

* tests: update

* feat: convert params between legacy & collab editor

* fix: remove ?a support

* tests: update

* chore: update changelog

---------

Co-authored-by: Frank Midigo <midigofrank@gmail.com>

Author: doc-han

CHANGELOG.md

@@ -28,6 +28,8 @@ and this project adheres to
 - Enforce `external_id` uniqueness on credentials per user and per project to
   prevent ambiguous keychain resolution
   [#4170](https://github.com/OpenFn/lightning/issues/4170)
+- unable to clear runs from history & url params rewrite
+  [#4397](https://github.com/OpenFn/lightning/issues/4397)
 
 ## [2.15.14] - 2026-02-13
 

assets/js/collaborative-editor/components/ide/FullScreenIDE.tsx

@@ -124,8 +124,7 @@ export function FullScreenIDE({
 }: FullScreenIDEProps) {
   const { params, updateSearchParams } = useURLState();
   const jobIdFromURL = params.job ?? null;
-  // Support both 'run' (collaborative) and 'a' (classical) parameter for run ID
-  const runIdFromURL = params.run ?? params.a ?? null;
+  const runIdFromURL = params.run ?? null;
   const stepIdFromURL = params.step ?? null;
   const { selectJob, saveWorkflow } = useWorkflowActions();
   const { selectStep } = useHistoryCommands();
@@ -338,7 +337,7 @@ export function FullScreenIDE({
       return;
     }
 
-    const runId = params.run ?? params.a ?? null;
+    const runId = params.run ?? null;
     if (!runId) {
       return;
     }

lib/lightning_web/live/run_live/show.ex

@@ -107,9 +107,9 @@ defmodule LightningWeb.RunLive.Show do
                       navigate={
                         # Only include version param if snapshot differs from current workflow version
                         if run.snapshot.lock_version == @workflow.lock_version do
-                          ~p"/projects/#{@project}/w/#{@workflow.id}?a=#{run.id}"
+                          ~p"/projects/#{@project}/w/#{@workflow.id}?run=#{run.id}"
                         else
-                          ~p"/projects/#{@project}/w/#{@workflow.id}?a=#{run.id}&v=#{run.snapshot.lock_version}"
+                          ~p"/projects/#{@project}/w/#{@workflow.id}?run=#{run.id}&v=#{run.snapshot.lock_version}"
                         end
                       }
                       class="link text-ellipsis"

lib/lightning_web/live/run_live/workorder_component.ex

@@ -183,9 +183,9 @@ defmodule LightningWeb.RunLive.WorkOrderComponent do
                 # Only include version param if snapshot differs from current workflow version
                 if @work_order.workflow.lock_version ==
                      @work_order.snapshot.lock_version do
-                  ~p"/projects/#{@project}/w/#{@work_order.workflow.id}?a=#{@last_run.id}"
+                  ~p"/projects/#{@project}/w/#{@work_order.workflow.id}?run=#{@last_run.id}"
                 else
-                  ~p"/projects/#{@project}/w/#{@work_order.workflow.id}?a=#{@last_run.id}&v=#{@work_order.snapshot.lock_version}"
+                  ~p"/projects/#{@project}/w/#{@work_order.workflow.id}?run=#{@last_run.id}&v=#{@work_order.snapshot.lock_version}"
                 end
               }
               class="inline-block"

lib/lightning_web/live/workflow_live/helpers.ex

@@ -326,12 +326,15 @@ defmodule LightningWeb.WorkflowLive.Helpers do
 
   @param_mappings %{
     direct: %{
-      "a" => "run"
+      "a" => "run",
+      "run" => "a"
     },
     mode_to_panel: %{
       "expand" => "editor",
       "workflow_input" => "run",
-      "settings" => "settings"
+      "settings" => "settings",
+      "editor" => "expand",
+      "run" => "workflow_input"
     },
     preserved: ["v", "method", "w-chat", "j-chat", "code"],
     collaborative_only: ["panel"]
@@ -340,9 +343,14 @@ defmodule LightningWeb.WorkflowLive.Helpers do
   def legacy_editor_url(params, live_action) do
     base_url = legacy_base_url(params, live_action)
 
-    query_params = Map.drop(params, ["id", "project_id"])
+    final_params =
+      params
+      |> Map.drop(["id", "project_id"])
+      |> Enum.reduce(%{}, fn {key, value}, acc ->
+        convert_param(key, value, acc, params)
+      end)
 
-    build_url_with_params(base_url, query_params)
+    build_url_with_params(base_url, final_params)
   end
 
   defp legacy_base_url(%{"project_id" => project_id}, :new) do
@@ -428,18 +436,41 @@ defmodule LightningWeb.WorkflowLive.Helpers do
     Map.put(acc, @param_mappings.direct["a"], value)
   end
 
+  defp convert_param("run", value, acc, _assigns) do
+    Map.put(acc, @param_mappings.direct["run"], value)
+  end
+
   defp convert_param("s", value, acc, assigns) do
     selection_type = determine_selection_type(value, assigns)
     Map.put(acc, selection_type, value)
   end
 
+  defp convert_param("job", value, acc, _assigns) do
+    Map.put(acc, "s", value)
+  end
+
+  defp convert_param("trigger", value, acc, _assigns) do
+    Map.put(acc, "s", value)
+  end
+
+  defp convert_param("edge", value, acc, _assigns) do
+    Map.put(acc, "s", value)
+  end
+
   defp convert_param("m", value, acc, _assigns) do
     case Map.get(@param_mappings.mode_to_panel, value) do
       nil -> acc
       panel -> Map.put(acc, "panel", panel)
     end
   end
 
+  defp convert_param("panel", value, acc, _assigns) do
+    case Map.get(@param_mappings.mode_to_panel, value) do
+      nil -> acc
+      panel -> Map.put(acc, "m", panel)
+    end
+  end
+
   defp convert_param(key, value, acc, _assigns) do
     cond do
       key in @param_mappings.collaborative_only ->

test/lightning_web/live/run_live/show_test.exs

@@ -256,7 +256,7 @@ defmodule LightningWeb.RunLive.ShowTest do
 
       # Find the workflow link - should NOT include version param
       assert html =~
-               ~r/href="\/projects\/#{project.id}\/w\/#{workflow.id}\?a=#{run_id}"/
+               ~r/href="\/projects\/#{project.id}\/w\/#{workflow.id}\?run=#{run_id}"/
 
       refute html =~ ~r/&v=/
     end
@@ -292,7 +292,7 @@ defmodule LightningWeb.RunLive.ShowTest do
       # Find the workflow link - should include version param
       # Note: & is HTML-escaped as & in rendered output
       assert html =~
-               ~r/href="\/projects\/#{project.id}\/w\/#{workflow.id}\?a=#{run_id}&v=#{snapshot.lock_version}"/
+               ~r/href="\/projects\/#{project.id}\/w\/#{workflow.id}\?run=#{run_id}&v=#{snapshot.lock_version}"/
     end
   end
 

test/lightning_web/live/workflow_live/collaborate_test.exs

@@ -2260,7 +2260,9 @@ defmodule LightningWeb.WorkflowLive.CollaborateTest do
                "/projects/#{project.id}/w/new/legacy?method=template"
     end
 
-    test "redirects to legacy editor with query params preserved", %{conn: conn} do
+    test "redirects to legacy editor with query params transformed", %{
+      conn: conn
+    } do
       user = insert(:user)
 
       project =
@@ -2272,7 +2274,6 @@ defmodule LightningWeb.WorkflowLive.CollaborateTest do
       workflow = workflow_fixture(project_id: project.id)
       job = insert(:job, workflow: workflow)
 
-      # Set user preference to prefer legacy editor
       user_with_prefs =
         user
         |> Ecto.Changeset.change(%{
@@ -2290,14 +2291,14 @@ defmodule LightningWeb.WorkflowLive.CollaborateTest do
           ~p"/projects/#{project.id}/w/#{workflow.id}?job=#{job.id}&panel=editor"
         )
 
-      # Should redirect to legacy editor with query params preserved
+      # Should redirect to legacy editor with query params transformed
       assert String.starts_with?(
                redirect_path,
                "/projects/#{project.id}/w/#{workflow.id}/legacy"
              )
 
-      assert redirect_path =~ "job=#{job.id}"
-      assert redirect_path =~ "panel=editor"
+      assert redirect_path =~ "s=#{job.id}"
+      assert redirect_path =~ "m=expand"
     end
 
     test "does not redirect when user does not prefer legacy editor", %{

test/lightning_web/live/workflow_live/helpers_test.exs

@@ -174,7 +174,7 @@ defmodule LightningWeb.WorkflowLive.HelpersTest do
       }
 
       result = Helpers.collaborative_editor_url(params, :edit)
-      assert result == "/projects/proj-1/w/wf-1"
+      assert result == "/projects/proj-1/w/wf-1?m=workflow_input"
     end
 
     test "preserves unknown parameters for future compatibility" do