From b4c6a2b59ee663712e4f1d529d2089d0fb8d1a7e Mon Sep 17 00:00:00 2001
From: Gautam Tayal <gautamtayal65@gmail.com>
Date: Sun, 12 Apr 2026 18:25:15 +0530
Subject: [PATCH] Fix orchestrator self-link on session detail

---
 packages/web/src/components/SessionDetail.tsx |  2 +-
 .../__tests__/SessionDetail.desktop.test.tsx  | 25 +++++++++++++++++++
 2 files changed, 26 insertions(+), 1 deletion(-)

diff --git a/packages/web/src/components/SessionDetail.tsx b/packages/web/src/components/SessionDetail.tsx
index b9e7be44b..2a6dcf517 100644
--- a/packages/web/src/components/SessionDetail.tsx
+++ b/packages/web/src/components/SessionDetail.tsx
@@ -517,7 +517,7 @@ export function SessionDetail({
           ) : null}
           <div className="dashboard-app-header__spacer" />
           <div className="dashboard-app-header__actions">
-            {orchestratorHref ? (
+            {!isOrchestrator && orchestratorHref ? (
               <a
                 href={orchestratorHref}
                 className="dashboard-app-btn dashboard-app-btn--amber"
diff --git a/packages/web/src/components/__tests__/SessionDetail.desktop.test.tsx b/packages/web/src/components/__tests__/SessionDetail.desktop.test.tsx
index bed6a584b..6ffe4a46b 100644
--- a/packages/web/src/components/__tests__/SessionDetail.desktop.test.tsx
+++ b/packages/web/src/components/__tests__/SessionDetail.desktop.test.tsx
@@ -186,4 +186,29 @@ describe("SessionDetail desktop layout", () => {
     expect(screen.getByText(/Terminal session has ended/i)).toBeInTheDocument();
     expect(screen.queryByTestId("direct-terminal")).not.toBeInTheDocument();
   });
+
+  it("hides the desktop orchestrator button on orchestrator session pages", () => {
+    render(
+      <SessionDetail
+        session={makeSession({
+          id: "my-app-orchestrator",
+          summary: "Project orchestrator",
+        })}
+        isOrchestrator
+        orchestratorZones={{
+          merge: 1,
+          respond: 0,
+          review: 0,
+          pending: 0,
+          working: 2,
+          done: 3,
+        }}
+        projectOrchestratorId="my-app-orchestrator"
+        projects={[{ id: "my-app", name: "My App", path: "/tmp/my-app" }]}
+      />,
+    );
+
+    expect(screen.queryByRole("link", { name: "Orchestrator" })).not.toBeInTheDocument();
+    expect(screen.getByText("orchestrator")).toBeInTheDocument();
+  });
 });