fix(frontend): keep the current branch when opening tasks from the header

The header task-status link derived its `branch` query param from
window.location via constructPath, but nuqs writes the branch into the URL
one render after the branch context updates, so the memoized href carried
the previous branch (none at all on the first switch). Navigating there
dropped the branch and BranchesProvider fell back to the default one.

Derive the param from currentBranch instead, excluding it on the default
branch.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Author: bilalabbad

changelog/+task-status-branch-link.fixed.md

@@ -0,0 +1 @@
+Fixed the task status button in the header sending you to the tasks page on the default branch instead of the branch you were working on.

frontend/app/src/entities/tasks/ui/task-status.test.tsx

@@ -67,6 +67,44 @@ describe("TaskStatus", () => {
     expect(component.getByTestId("pulse").query()).toBeNull();
   });
 
+  test("links to the tasks page with the current branch, not the branch in the URL", async () => {
+    // GIVEN
+    useCurrentBranchMock.mockReturnValue({
+      currentBranch: generateBranch({ name: "branch1", is_default: false }),
+      setCurrentBranch: () => {},
+    });
+    getBranchTaskStatusFromApiMock.mockResolvedValue({
+      data: { InfrahubTaskBranchStatus: { count: 0 } },
+    } as Awaited<ReturnType<typeof getBranchTaskStatusFromApi>>);
+
+    // WHEN
+    const component = await render(<TaskStatus />);
+
+    // THEN
+    await expect
+      .element(component.getByRole("link", { name: "View branch tasks" }))
+      .toHaveAttribute("href", expect.stringContaining("branch=branch1"));
+  });
+
+  test("omits the branch query param when the current branch is the default one", async () => {
+    // GIVEN
+    useCurrentBranchMock.mockReturnValue({
+      currentBranch: generateBranch({ name: "main", is_default: true }),
+      setCurrentBranch: () => {},
+    });
+    getBranchTaskStatusFromApiMock.mockResolvedValue({
+      data: { InfrahubTaskBranchStatus: { count: 0 } },
+    } as Awaited<ReturnType<typeof getBranchTaskStatusFromApi>>);
+
+    // WHEN
+    const component = await render(<TaskStatus />);
+
+    // THEN
+    await expect
+      .element(component.getByRole("link", { name: "View branch tasks" }))
+      .toHaveAttribute("href", expect.not.stringContaining("branch="));
+  });
+
   test("renders error icon with tooltip when query fails", async () => {
     // GIVEN
     useCurrentBranchMock.mockReturnValue({

frontend/app/src/entities/tasks/ui/task-status.tsx

@@ -4,7 +4,7 @@ import { useQuery } from "@tanstack/react-query";
 
 import TasksStatusIcon from "@/assets/icons/tasks-status.svg?react";
 
-import { constructPath } from "@/shared/api/rest/fetch";
+import { constructPath, type overrideQueryParams } from "@/shared/api/rest/fetch";
 import { Pulse } from "@/shared/components/ui/pulse";
 import { QSP } from "@/shared/config/qsp";
 
@@ -28,11 +28,20 @@ export function TaskStatus() {
     value: currentBranch.name,
   };
 
+  // constructPath reads window.location, which nuqs updates one render later than the branch
+  // context, so the branch has to come from currentBranch to avoid linking to the previous one.
+  const branchParam: overrideQueryParams = currentBranch.is_default
+    ? { name: QSP.BRANCH, exclude: true }
+    : { name: QSP.BRANCH, value: currentBranch.name };
+
   const commonButtonProps: LinkButtonProps = {
     shape: "square",
     variant: "outline",
     size: "sm",
-    href: constructPath("/tasks", [{ name: QSP.FILTER, value: JSON.stringify([filter]) }]),
+    href: constructPath("/tasks", [
+      branchParam,
+      { name: QSP.FILTER, value: JSON.stringify([filter]) },
+    ]),
   };
 
   if (error) {