Impplemented separate silentLogger for browser usage where needed.

Author: BobDickinson

clients/web/src/components/AuthDebugger.tsx

@@ -2,7 +2,7 @@ import { useCallback, useEffect, useState } from "react";
 import { Button } from "@/components/ui/button";
 import { AlertCircle } from "lucide-react";
 import type { InspectorClient } from "@modelcontextprotocol/inspector-core/mcp/index.js";
-import { silentLogger } from "@modelcontextprotocol/inspector-core/logging/node";
+import { silentLogger } from "@modelcontextprotocol/inspector-core/logging/browser";
 import type { AuthGuidedState } from "@modelcontextprotocol/inspector-core/auth/types.js";
 import type { WebEnvironmentResult } from "@/lib/adapters/environmentFactory";
 import { OAuthFlowProgress } from "./OAuthFlowProgress";

clients/web/src/components/OAuthCallback.tsx

@@ -2,7 +2,7 @@ import { useCallback, useEffect, useRef, useState } from "react";
 import type { InspectorClient } from "@modelcontextprotocol/inspector-core/mcp/index.js";
 import type { WebEnvironmentResult } from "@/lib/adapters/environmentFactory";
 import { parseOAuthState } from "@modelcontextprotocol/inspector-core/auth/index.js";
-import { silentLogger } from "@modelcontextprotocol/inspector-core/logging/node";
+import { silentLogger } from "@modelcontextprotocol/inspector-core/logging/browser";
 import useTheme from "@/lib/hooks/useTheme";
 import { useToast } from "@/lib/hooks/useToast";
 import {

core/logging/browser/index.ts

@@ -0,0 +1 @@
+export { silentLogger } from "./logger.js";

core/logging/browser/logger.ts

@@ -0,0 +1,10 @@
+/**
+ * Silent logger for browser contexts. Satisfies pino.Logger; does not output anything.
+ * Use when no logger is passed (e.g. OAuth callback with no client). Web components
+ * that need a fallback import from logging/browser so they don't pull in Node logging.
+ */
+// @ts-expect-error - pino/browser.js exists but TypeScript doesn't have types for the .js extension
+import pino from "pino/browser.js";
+import type { Logger } from "pino";
+
+export const silentLogger: Logger = pino({ level: "silent" });

core/package.json

@@ -18,6 +18,8 @@
     "./mcp/node/*": "./build/mcp/node/*",
     "./auth/browser": "./build/auth/browser/index.js",
     "./auth/browser/*": "./build/auth/browser/*",
+    "./logging/browser": "./build/logging/browser/index.js",
+    "./logging/browser/*": "./build/logging/browser/*",
     "./logging/node": "./build/logging/node/index.js",
     "./logging/node/*": "./build/logging/node/*",
     "./mcp/remote": "./build/mcp/remote/index.js",