Remove server sharing feature in favor of chatbox sharing (#2047)

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

Author: chelojimenez

mcpjam-inspector/client/src/App.tsx

@@ -92,10 +92,6 @@ import type {
 import { useViewQueries, useProjectServers } from "./hooks/useViews";
 import { HostedShellGate } from "./components/hosted/HostedShellGate";
 import { resolveHostedShellGateState } from "./components/hosted/hosted-shell-gate-state";
-import {
-  SharedServerChatPage,
-  getSharedPathTokenFromLocation,
-} from "./components/hosted/SharedServerChatPage";
 import {
   ChatboxChatPage,
   getChatboxPathTokenFromLocation,
@@ -157,17 +153,9 @@ import {
   clearChatboxSignInReturnPath,
   readChatboxSession,
   readChatboxSignInReturnPath,
+  slugify,
   writeChatboxSignInReturnPath,
 } from "./lib/chatbox-session";
-import {
-  clearSharedSignInReturnPath,
-  readSharedServerSession,
-  readSharedSignInReturnPath,
-  slugify,
-  writeSharedSignInReturnPath,
-  readPendingServerAdd,
-  clearPendingServerAdd,
-} from "./lib/shared-server-session";
 import {
   sanitizeHostedOAuthErrorMessage,
   clearHostedOAuthResumeMarker,
@@ -428,57 +416,26 @@ export default function App() {
     const callbackContext = getHostedOAuthCallbackContext();
     return callbackContext != null && callbackContext.surface !== "project";
   });
-  const [exitedSharedChat, setExitedSharedChat] = useState(false);
   const [exitedChatboxChat, setExitedChatboxChat] = useState(false);
-  const sharedPathToken = HOSTED_MODE ? getSharedPathTokenFromLocation() : null;
   const chatboxPathToken = HOSTED_MODE
     ? getChatboxPathTokenFromLocation()
     : null;
-  const sharedSession = HOSTED_MODE ? readSharedServerSession() : null;
   const chatboxSession = HOSTED_MODE ? readChatboxSession() : null;
-  const currentHashSlug = window.location.hash
-    .replace(/^#/, "")
-    .replace(/^\/+/, "")
-    .split("/")[0];
   const hostedRouteKind = useMemo(() => {
     if (!HOSTED_MODE) {
       return null;
     }
 
-    if (sharedPathToken) {
-      return "shared" as const;
-    }
     if (chatboxPathToken) {
       return "chatbox" as const;
     }
 
-    if (sharedSession && chatboxSession) {
-      if (currentHashSlug === slugify(sharedSession.payload.serverName)) {
-        return "shared" as const;
-      }
-      if (currentHashSlug === slugify(chatboxSession.payload.name)) {
-        return "chatbox" as const;
-      }
-      return null;
-    }
-
-    if (sharedSession) {
-      return "shared" as const;
-    }
     if (chatboxSession) {
       return "chatbox" as const;
     }
 
     return null;
-  }, [
-    currentHashSlug,
-    chatboxPathToken,
-    chatboxSession,
-    sharedPathToken,
-    sharedSession,
-  ]);
-  const isSharedChatRoute =
-    HOSTED_MODE && !exitedSharedChat && hostedRouteKind === "shared";
+  }, [chatboxPathToken, chatboxSession]);
   const isChatboxChatRoute =
     HOSTED_MODE && !exitedChatboxChat && hostedRouteKind === "chatbox";
 
@@ -489,7 +446,7 @@ export default function App() {
       setEvaluateRunsFlagsLoaded(posthog.featureFlags?.hasLoadedFlags === true);
     });
   }, [posthog]);
-  const isHostedChatRoute = isSharedChatRoute || isChatboxChatRoute;
+  const isHostedChatRoute = isChatboxChatRoute;
   const currentHash = window.location.hash || "#servers";
   const currentHashRoute = useMemo(
     () => resolveHostedNavigation(currentHash, HOSTED_MODE),
@@ -709,18 +666,16 @@ export default function App() {
     // Let AuthKit + Convex auth settle before leaving /callback.
     if (!isAuthLoading && isAuthenticated) {
       const chatboxReturnPath = readChatboxSignInReturnPath();
-      const sharedReturnPath = readSharedSignInReturnPath();
       const persistedCheckoutIntent = readPersistedCheckoutIntent();
       const billingReturnPath = persistedCheckoutIntent
         ? readBillingSignInReturnPath()
         : null;
       clearChatboxSignInReturnPath();
-      clearSharedSignInReturnPath();
       clearBillingSignInReturnPath();
       window.history.replaceState(
         {},
         "",
-        chatboxReturnPath ?? sharedReturnPath ?? billingReturnPath ?? "/"
+        chatboxReturnPath ?? billingReturnPath ?? "/"
       );
       setCallbackCompleted(true);
       setCallbackRecoveryExpired(false);
@@ -893,40 +848,6 @@ export default function App() {
     isHostedDefaultRoute &&
     hostedShellGateState === "auth-loading";
 
-  // Auto-add a shared server when returning from SharedServerChatPage via "Open MCPJam"
-  useEffect(() => {
-    if (isHostedChatRoute) return;
-    if (isLoadingRemoteProjects) return;
-    if (isAuthLoading) return;
-    // In hosted mode, also wait for the project to resolve. handleConnect
-    // would otherwise build a request with `projectId === "none"` and fail.
-    if (HOSTED_MODE && (!activeProjectId || activeProjectId === "none")) return;
-
-    const pending = readPendingServerAdd();
-    if (!pending) return;
-    clearPendingServerAdd();
-
-    if (projectServers[pending.serverName] !== undefined) {
-      return; // Server already exists
-    }
-
-    handleConnect({
-      name: pending.serverName,
-      type: "http",
-      url: pending.serverUrl,
-      useOAuth: pending.useOAuth,
-      clientId: pending.clientId ?? undefined,
-      oauthScopes: pending.oauthScopes ?? undefined,
-    });
-  }, [
-    isHostedChatRoute,
-    isLoadingRemoteProjects,
-    isAuthLoading,
-    activeProjectId,
-    projectServers,
-    handleConnect,
-  ]);
-
   const previousConnectedServersRef = useRef<Set<string> | null>(null);
   useEffect(() => {
     const connectedServers = new Set(
@@ -1209,17 +1130,6 @@ export default function App() {
         preserveCurrentOrganizationOnNonOrgTarget?: boolean;
       }
     ) => {
-      if (isSharedChatRoute) {
-        const storedSession = readSharedServerSession();
-        if (storedSession) {
-          const expectedHash = slugify(storedSession.payload.serverName);
-          if (window.location.hash !== `#${expectedHash}`) {
-            window.location.hash = expectedHash;
-          }
-        }
-        return;
-      }
-
       if (isChatboxChatRoute) {
         const storedSession = readChatboxSession();
         if (storedSession) {
@@ -1287,7 +1197,6 @@ export default function App() {
     [
       effectiveOrganizations,
       isChatboxChatRoute,
-      isSharedChatRoute,
       setSelectedMultipleServersToAllServers,
     ]
   );
@@ -2449,9 +2358,6 @@ export default function App() {
                 : undefined
             }
             onSignIn={() => {
-              if (sharedPathToken) {
-                writeSharedSignInReturnPath(window.location.pathname);
-              }
               if (chatboxPathToken) {
                 writeChatboxSignInReturnPath(window.location.pathname);
               }
@@ -2461,12 +2367,7 @@ export default function App() {
               void signOut();
             }}
           >
-            {isSharedChatRoute ? (
-              <SharedServerChatPage
-                pathToken={sharedPathToken}
-                onExitSharedChat={() => setExitedSharedChat(true)}
-              />
-            ) : isChatboxChatRoute ? (
+            {isChatboxChatRoute ? (
               <ChatboxChatPage
                 pathToken={chatboxPathToken}
                 onExitChatboxChat={() => setExitedChatboxChat(true)}

mcpjam-inspector/client/src/__tests__/App.hosted-oauth.test.tsx

@@ -390,10 +390,6 @@ vi.mock("../components/hosted/HostedShellGate", () => ({
 vi.mock("../components/hosted/hosted-shell-gate-state", () => ({
   resolveHostedShellGateState: () => mockHostedShellGateState.value,
 }));
-vi.mock("../components/hosted/SharedServerChatPage", () => ({
-  SharedServerChatPage: () => <button type="button">Authorize</button>,
-  getSharedPathTokenFromLocation: () => null,
-}));
 vi.mock("../components/hosted/ChatboxChatPage", () => ({
   ChatboxChatPage: () => <button type="button">Authorize</button>,
   getChatboxPathTokenFromLocation: () => null,

mcpjam-inspector/client/src/components/ChatTabV2.tsx

@@ -315,7 +315,6 @@ export function ChatTabV2({
       ),
     [selectedConnectedServerNames, serversByName, appState.servers]
   );
-  const hostedShareToken = hostedContext?.shareToken;
   const hostedChatboxToken = hostedContext?.chatboxToken;
   const hostedChatboxSurface = hostedContext?.chatboxSurface;
   const effectiveHostedProjectId =
@@ -329,7 +328,6 @@ export function ChatTabV2({
     HOSTED_MODE &&
     !isConvexAuthenticated &&
     !effectiveHostedProjectId &&
-    !hostedShareToken &&
     !hostedChatboxToken;
 
   // Use shared chat session hook
@@ -412,7 +410,6 @@ export function ChatTabV2({
   const showHistoryRail =
     HOSTED_MODE &&
     !minimalMode &&
-    !hostedShareToken &&
     !hostedChatboxToken &&
     chatHistoryRailEnabled;
   const {
@@ -1090,7 +1087,6 @@ export function ChatTabV2({
     enableMultiModelChat &&
     !minimalMode &&
     !executionConfig?.modelId &&
-    !hostedShareToken &&
     !hostedChatboxToken &&
     !hostedChatboxSurface &&
     availableModels.length > 1;

mcpjam-inspector/client/src/components/connection/ServerConnectionCard.tsx

@@ -34,7 +34,6 @@ import {
   Cable,
   Trash2,
   AlertCircle,
-  Share2,
   FileText,
 } from "lucide-react";
 import { ServerWithName } from "@/hooks/use-app-state";
@@ -62,7 +61,6 @@ import {
 import { useAuth } from "@workos-inc/authkit-react";
 import { useConvexAuth } from "convex/react";
 import { HOSTED_MODE } from "@/lib/config";
-import { ShareServerDialog } from "./ShareServerDialog";
 import { useExploreCasesPrefetchOnConnect } from "@/hooks/use-explore-cases-prefetch-on-connect";
 import { getOAuthTraceFailureStep } from "@/lib/oauth/oauth-trace";
 
@@ -78,8 +76,6 @@ function isHostedInsecureHttpServer(server: ServerWithName): boolean {
   }
 }
 
-// Temporary hide while chatbox sharing replaces server sharing in the main UI.
-const SERVER_SHARE_UI_ENABLED = false;
 const SERVER_CARD_CONTEXT_MENU_EXEMPT_SELECTOR =
   "[data-server-card-context-menu-exempt]";
 
@@ -139,7 +135,6 @@ export function ServerConnectionCard({
   const [isCreatingTunnel, setIsCreatingTunnel] = useState(false);
   const [isClosingTunnel, setIsClosingTunnel] = useState(false);
   const [showTunnelExplanation, setShowTunnelExplanation] = useState(false);
-  const [isShareDialogOpen, setIsShareDialogOpen] = useState(false);
   const [isActionsMenuOpen, setIsActionsMenuOpen] = useState(false);
 
   const {
@@ -169,25 +164,6 @@ export function ServerConnectionCard({
     server.connectionStatus === "connecting" ||
     server.connectionStatus === "oauth-flow";
   const isReconnectMenuDisabled = isReconnecting || isPendingConnection;
-  const isStdioServer = "command" in server.config;
-  const isInsecureHttpServer =
-    "url" in server.config &&
-    !!server.config.url &&
-    (() => {
-      try {
-        return new URL(server.config.url.toString()).protocol === "http:";
-      } catch {
-        return false;
-      }
-    })();
-  const canShareServer =
-    SERVER_SHARE_UI_ENABLED &&
-    HOSTED_MODE &&
-    !!hostedServerId &&
-    isAuthenticated &&
-    !isStdioServer &&
-    !isInsecureHttpServer;
-
   useEffect(() => {
     if (serverTunnelUrl !== undefined) {
       setTunnelUrl(serverTunnelUrl);
@@ -697,23 +673,6 @@ export function ServerConnectionCard({
               className="flex flex-wrap items-center justify-end gap-2"
               onClick={(e) => e.stopPropagation()}
             >
-              {canShareServer && (
-                <button
-                  data-server-card-context-menu-exempt
-                  onClick={() => {
-                    posthog.capture("share_server_clicked", {
-                      location: "server_connection_card",
-                      platform: detectPlatform(),
-                      environment: detectEnvironment(),
-                    });
-                    setIsShareDialogOpen(true);
-                  }}
-                  className="inline-flex items-center gap-1.5 rounded-full border border-border/70 bg-muted/30 px-2 py-0.5 text-[11px] text-foreground transition-colors hover:bg-accent/60 cursor-pointer"
-                >
-                  <Share2 className="h-3 w-3" />
-                  <span>Share</span>
-                </button>
-              )}
               {showTunnelActions && (
                 <>
                   {hasTunnel ? (
@@ -836,14 +795,6 @@ export function ServerConnectionCard({
         onConfirm={handleConfirmCreateTunnel}
         isCreating={isCreatingTunnel}
       />
-      {canShareServer && hostedServerId && (
-        <ShareServerDialog
-          isOpen={isShareDialogOpen}
-          onClose={() => setIsShareDialogOpen(false)}
-          serverId={hostedServerId}
-          serverName={server.name}
-        />
-      )}
     </>
   );
 }