fix(webapp): keep the branches list query string when archiving a branch (#4724)

<!-- ccr-slack-attribution -->
_Requested by **Iss** · [Slack
thread](https://triggerdotdev.slack.com/archives/C045W9WM3E1/p1787161814493949)_

**Before:** archiving a branch dropped the query string on the way back
to the branches list, so the list reset to page 1. Working down a long
list meant re-navigating to the page you were on after every archive.

**After:** you land back on the exact page you archived from, with
`page`, `search` and `showArchived` intact.

The archive action now redirects to the page the request came from
instead of rebuilding a bare branches path.

## How

The archive dialog already submits the page it was opened from as a
hidden `redirectPath` field (`${location.pathname}${location.search}`),
and the failure path already redirected to it — only the success path
ignored it and rebuilt the path with `branchesPath`/`branchesDevPath`,
which have no query string. Both paths now redirect to the submitted
path, run through the existing `sanitizeRedirectPath` helper to keep the
redirect same-origin (the same idiom used by
`resources.batches.$batchId.check-completion`).

## ✅ Checklist

- [x] I have followed every step in the [contributing
guide](https://github.com/triggerdotdev/trigger.dev/blob/main/CONTRIBUTING.md)
- [x] The PR title follows the convention.
- [x] I ran and tested the code works

---

## Testing

Three files change:

- `apps/webapp/app/routes/resources.branches.archive.tsx` — the fix.
- `apps/webapp/test/archiveBranchRedirect.test.ts` — new test that
drives the archive action and asserts the redirect `Location`: the query
string survives on both success and failure, and an off-origin
`redirectPath` falls back to `/`. Reverting the fix makes two of the
three cases fail, so the test covers the regression.
- `.server-changes/archive-branch-keeps-list-page.md` — release-note
entry, since this is a user-facing server-only change.

Also ran `pnpm run typecheck` and `oxlint` for `apps/webapp` — both
clean.

---

## Changelog

Archiving a branch now returns you to the same page of the branches list
instead of resetting it to page 1.

---

## Screenshots

_None — no visual change._

---------

Co-authored-by: Claude <noreply@anthropic.com>

Author: claude[bot]

.server-changes/archive-branch-keeps-list-page.md

@@ -0,0 +1,6 @@
+---
+area: webapp
+type: fix
+---
+
+Archiving a branch now returns you to the same page of the branches list, keeping your place, search and filters instead of resetting to the first page.

apps/webapp/app/routes/resources.branches.archive.tsx

@@ -13,7 +13,7 @@ import { Paragraph } from "~/components/primitives/Paragraph";
 import { redirectWithErrorMessage, redirectWithSuccessMessage } from "~/models/message.server";
 import { ArchiveBranchService } from "~/services/archiveBranch.server";
 import { requireUserId } from "~/services/session.server";
-import { branchesDevPath, branchesPath } from "~/utils/pathBuilder";
+import { sanitizeRedirectPath } from "~/utils";
 
 const ArchiveBranchOptions = z.object({
   environmentId: z.string(),
@@ -35,6 +35,8 @@ export async function action({ request }: ActionFunctionArgs) {
     return redirectWithErrorMessage("/", request, "Invalid form data");
   }
 
+  const redirectPath = sanitizeRedirectPath(submission.value.redirectPath);
+
   const archiveBranchService = new ArchiveBranchService();
 
   const result = await archiveBranchService.call(
@@ -46,15 +48,13 @@ export async function action({ request }: ActionFunctionArgs) {
 
   if (result.success) {
     return redirectWithSuccessMessage(
-      result.branch.type === "DEVELOPMENT"
-        ? branchesDevPath(result.organization, result.project, result.branch)
-        : branchesPath(result.organization, result.project, result.branch),
+      redirectPath,
       request,
       `Branch "${result.branch.branchName}" archived`
     );
   }
 
-  return redirectWithErrorMessage(submission.value.redirectPath, request, result.error);
+  return redirectWithErrorMessage(redirectPath, request, result.error);
 }
 
 export function ArchiveButton({

apps/webapp/test/archiveBranchRedirect.test.ts

@@ -0,0 +1,63 @@
+// The archive dialog submits the page it was opened from, so archiving from a
+// paginated or filtered branches list must land back on that exact page instead
+// of a bare branches path that resets the list to page 1.
+
+import { beforeEach, describe, expect, it, vi } from "vitest";
+import { action } from "~/routes/resources.branches.archive";
+
+vi.mock("~/services/session.server", () => ({
+  requireUserId: vi.fn().mockResolvedValue("user_1"),
+}));
+
+const archiveSucceeds = { value: true };
+
+vi.mock("~/services/archiveBranch.server", () => ({
+  ArchiveBranchService: class {
+    async call() {
+      return archiveSucceeds.value
+        ? { success: true as const, branch: { branchName: "feat/checkout" } }
+        : { success: false as const, error: "Failed to archive branch" };
+    }
+  },
+}));
+
+const LIST_PATH = "/orgs/o/projects/p/env/preview/branches?page=3&search=feat";
+
+async function archive(redirectPath: string) {
+  const body = new URLSearchParams({ environmentId: "env_1", redirectPath });
+
+  return (await (action as any)({
+    request: new Request("https://app.example.com/resources/branches/archive", {
+      method: "POST",
+      body,
+    }),
+    params: {},
+    context: {},
+  })) as Response;
+}
+
+describe("archiving a branch returns to the page it was started from", () => {
+  beforeEach(() => {
+    archiveSucceeds.value = true;
+  });
+
+  it("preserves the query string on success", async () => {
+    const response = await archive(LIST_PATH);
+
+    expect(response.headers.get("Location")).toBe(LIST_PATH);
+  });
+
+  it("preserves the query string on failure", async () => {
+    archiveSucceeds.value = false;
+
+    const response = await archive(LIST_PATH);
+
+    expect(response.headers.get("Location")).toBe(LIST_PATH);
+  });
+
+  it("keeps the redirect same-origin", async () => {
+    const response = await archive("//evil.example.com/branches");
+
+    expect(response.headers.get("Location")).toBe("/");
+  });
+});