mcp-remote 0.0.5 → 0.0.6

package/README.md

@@ -11,10 +11,7 @@ E.g: Claude Desktop or Windsurf
   "mcpServers": {
     "remote-example": {
       "command": "npx",
-      "args": [
-        "mcp-remote",
-        "https://remote.mcp.server/sse"
-      ]
+      "args": ["mcp-remote", "https://remote.mcp.server/sse"]
     }
   }
 }
@@ -23,4 +20,3 @@ E.g: Claude Desktop or Windsurf
 Cursor:
 
 ![image](https://github.com/user-attachments/assets/14338bfa-a779-4e8a-a477-71f72cc5d99d)
-

package/dist/react/index.js

@@ -300,11 +300,6 @@ function useMcp(options) {
           callbackUrl
         });
       }
-      if (!metadataRef.current) {
-        addLog("info", "Discovering OAuth metadata...");
-        metadataRef.current = await discoverOAuthMetadata(url);
-        addLog("debug", `OAuth metadata: ${metadataRef.current ? "Found" : "Not available"}`);
-      }
       clientRef.current = new Client(
         {
           name: clientConfig.name || "mcp-react-client",
@@ -316,7 +311,6 @@ function useMcp(options) {
           }
         }
       );
-      const tokens = await authProviderRef.current.tokens();
       setState("connecting");
       addLog("info", "Creating transport...");
       const serverUrl = new URL(url);
@@ -330,13 +324,7 @@ function useMcp(options) {
       transportRef.current.onerror = (err) => {
         addLog("error", `Transport error: ${err.message}`);
         if (err.message.includes("Unauthorized")) {
-          setState("authenticating");
-          handleAuthentication().catch((authErr) => {
-            addLog("error", `Authentication error: ${authErr.message}`);
-            setState("failed");
-            setError(`Authentication failed: ${authErr.message}`);
-            connectingRef.current = false;
-          });
+          discoverOAuthAndAuthenticate(err);
         } else {
           setState("failed");
           setError(`Connection error: ${err.message}`);
@@ -353,14 +341,35 @@ function useMcp(options) {
           }, delay);
         }
       };
+      const discoverOAuthAndAuthenticate = async (error2) => {
+        try {
+          if (!metadataRef.current) {
+            addLog("info", "Discovering OAuth metadata...");
+            metadataRef.current = await discoverOAuthMetadata(url);
+            addLog("debug", `OAuth metadata: ${metadataRef.current ? "Found" : "Not available"}`);
+          }
+          if (metadataRef.current) {
+            setState("authenticating");
+            await handleAuthentication();
+            return connect();
+          } else {
+            setState("failed");
+            setError(`Authentication required but no OAuth metadata found: ${error2.message}`);
+            connectingRef.current = false;
+          }
+        } catch (oauthErr) {
+          addLog("error", `OAuth discovery error: ${oauthErr instanceof Error ? oauthErr.message : String(oauthErr)}`);
+          setState("failed");
+          setError(`Authentication setup failed: ${oauthErr instanceof Error ? oauthErr.message : String(oauthErr)}`);
+          connectingRef.current = false;
+        }
+      };
       try {
         addLog("info", "Starting transport...");
       } catch (err) {
         addLog("error", `Transport start error: ${err instanceof Error ? err.message : String(err)}`);
         if (err instanceof Error && err.message.includes("Unauthorized")) {
-          setState("authenticating");
-          await handleAuthentication();
-          return connect();
+          await discoverOAuthAndAuthenticate(err);
         } else {
           setState("failed");
           setError(`Connection error: ${err instanceof Error ? err.message : String(err)}`);
@@ -387,9 +396,13 @@ function useMcp(options) {
         }
       } catch (connectErr) {
         addLog("error", `Client connect error: ${connectErr instanceof Error ? connectErr.message : String(connectErr)}`);
-        setState("failed");
-        setError(`Connection error: ${connectErr instanceof Error ? connectErr.message : String(connectErr)}`);
-        connectingRef.current = false;
+        if (connectErr instanceof Error && connectErr.message.includes("Unauthorized")) {
+          await discoverOAuthAndAuthenticate(connectErr);
+        } else {
+          setState("failed");
+          setError(`Connection error: ${connectErr instanceof Error ? connectErr.message : String(connectErr)}`);
+          connectingRef.current = false;
+        }
       }
     } catch (err) {
       addLog("error", `Unexpected error: ${err instanceof Error ? err.message : String(err)}`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mcp-remote",
-  "version": "0.0.5",
+  "version": "0.0.6",
   "type": "module",
   "bin": {
     "mcp-remote": "dist/cli/proxy.js"
@@ -18,7 +18,8 @@
     }
   },
   "scripts": {
-    "build": "tsup"
+    "build": "tsup",
+    "check": "prettier --check . && tsc"
   },
   "dependencies": {
     "express": "^4.21.2",