From 8ce33b29da3a79228422adf80a30807c66b7a3d7 Mon Sep 17 00:00:00 2001
From: Prateek <karnalprateek@gmail.com>
Date: Sun, 15 Feb 2026 05:24:25 +0530
Subject: [PATCH] fix: terminal button should link to session detail page, not
 raw xterm URL

The terminal button was fetching from the terminal-web plugin and opening
a raw xterm.js URL (localhost:7801). Changed to a simple link to the
session detail page which has an embedded terminal.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
---
 packages/web/src/components/SessionCard.tsx | 19 +++++--------------
 1 file changed, 5 insertions(+), 14 deletions(-)

diff --git a/packages/web/src/components/SessionCard.tsx b/packages/web/src/components/SessionCard.tsx
index 113852218..940de3ec4 100644
--- a/packages/web/src/components/SessionCard.tsx
+++ b/packages/web/src/components/SessionCard.tsx
@@ -105,22 +105,13 @@ export function SessionCard({ session, onSend, onKill, onMerge, onRestore }: Ses
           </button>
         )}
         {!isTerminal && (
-          <button
-            onClick={(e) => {
-              e.stopPropagation();
-              const port = process.env.NEXT_PUBLIC_TERMINAL_PORT ?? "3001";
-              fetch(`http://localhost:${port}/terminal?session=${encodeURIComponent(session.id)}`)
-                .then((res) => res.json() as Promise<{ url: string }>)
-                .then((data) => window.open(data.url, `terminal-${session.id}`))
-                .catch(() => {
-                  // Fall back to session detail page
-                  window.location.href = `/sessions/${encodeURIComponent(session.id)}`;
-                });
-            }}
-            className="shrink-0 rounded-md border border-[var(--color-border-default)] px-2.5 py-0.5 text-[11px] text-[var(--color-text-secondary)] transition-colors hover:border-[var(--color-accent-blue)] hover:text-[var(--color-accent-blue)]"
+          <a
+            href={`/sessions/${encodeURIComponent(session.id)}`}
+            onClick={(e) => e.stopPropagation()}
+            className="shrink-0 rounded-md border border-[var(--color-border-default)] px-2.5 py-0.5 text-[11px] text-[var(--color-text-secondary)] transition-colors hover:border-[var(--color-accent-blue)] hover:text-[var(--color-accent-blue)] hover:no-underline"
           >
             terminal
-          </button>
+          </a>
         )}
       </div>