chore: refactor MCP resolution logic by removing caching mechanism and simplifying return values in Next.js and Nuxt.js handlers

Author: regenrek

packages/next/src/route.ts

@@ -75,11 +75,7 @@ function color(level: BrowserLogLevel, msg: string) {
 }
 function dim(s: string) { return c.dim + s + c.reset; }
 
-let __mcpProjectCache: { url: string; routeLogs?: `/${string}` } | null = null;
-
 async function __resolveMcpFromProject(): Promise<{ url: string; routeLogs?: `/${string}` }> {
-  // Only cache positive resolutions; always retry if unresolved/empty
-  if (__mcpProjectCache && __mcpProjectCache.url) return __mcpProjectCache;
   try {
     const { readFileSync, existsSync } = await import('node:fs');
     const { join } = await import('node:path');
@@ -91,13 +87,11 @@ async function __resolveMcpFromProject(): Promise<{ url: string; routeLogs?: `/$
       const url = rawUrl.replace(/\/$/, '').replace(/\/mcp$/i, '');
       const routeLogs = (data?.route ? String(data.route) : '/__client-logs') as `/${string}`;
       if (url && await __pingHealth(`${url}/health`, 250)) {
-        __mcpProjectCache = { url, routeLogs };
-        return __mcpProjectCache;
+        return { url, routeLogs };
       }
     }
   } catch {}
-  __mcpProjectCache = { url: '' } as any;
-  return __mcpProjectCache;
+  return { url: '' } as any;
 }
 
 async function __pingHealth(url: string, timeoutMs: number): Promise<boolean> {

packages/nuxt/src/runtime/server/handler.ts

@@ -76,11 +76,7 @@ function color(level: Level, msg: string) {
 }
 function dim(s: string) { return c.dim + s + c.reset; }
 
-let __mcpProjectCacheNuxt: { url: string; routeLogs?: `/${string}` } | null = null;
-
 async function __resolveMcpFromProjectNuxt(): Promise<{ url: string; routeLogs?: `/${string}` }> {
-  // Only cache positive resolutions; always retry if unresolved/empty
-  if (__mcpProjectCacheNuxt && __mcpProjectCacheNuxt.url) return __mcpProjectCacheNuxt;
   try {
     const { readFileSync, existsSync } = await import('node:fs');
     const { join } = await import('node:path');
@@ -91,12 +87,11 @@ async function __resolveMcpFromProjectNuxt(): Promise<{ url: string; routeLogs?:
       const url = (data?.url ? String(data.url) : '').replace(/\/$/, '').replace(/\/mcp$/i, '');
       const routeLogs = (data?.route ? String(data.route) as `/${string}` : '/__client-logs');
       if (url && await __pingHealthNuxt(`${url}/health`, 300)) {
-        __mcpProjectCacheNuxt = { url, routeLogs };
-        return __mcpProjectCacheNuxt;
+        return { url, routeLogs };
       }
     }
   } catch {}
-  __mcpProjectCacheNuxt = { url: '' } as any; return __mcpProjectCacheNuxt;
+  return { url: '' } as any;
 }
 
 async function __pingHealthNuxt(url: string, timeoutMs: number): Promise<boolean> {

packages/vite/src/index.ts

@@ -1,7 +1,7 @@
 // Avoid exporting Vite types to prevent cross-version type mismatches in consumers
 import ansis from 'ansis';
 import type { BrowserLogLevel } from '@browser-echo/core';
-import { mkdirSync, appendFileSync, existsSync, readFileSync, watch as fsWatch } from 'node:fs';
+import { mkdirSync, appendFileSync, existsSync, readFileSync } from 'node:fs';
 import { join as joinPath, dirname } from 'node:path';
 
 export interface BrowserLogsToTerminalOptions {
@@ -162,17 +162,8 @@ function attachMiddleware(server: any, options: ResolvedOptions) {
     resolvedIngest = '';
   }
 
-  // Resolve once at startup and watch for project json changes to re-resolve
+  // Resolve once at startup
   resolveOnce();
-  try {
-    const cfgPath = joinPath(process.cwd(), '.browser-echo-mcp.json');
-    if (existsSync(cfgPath)) {
-      try { fsWatch(cfgPath, { persistent: false }, () => { try { resolveOnce(); } catch {} }); } catch {}
-    } else {
-      const parent = process.cwd();
-      try { fsWatch(parent, { persistent: false }, (_evt: string, file?: string) => { if (String(file || '') === '.browser-echo-mcp.json') { try { resolveOnce(); } catch {} } }); } catch {}
-    }
-  } catch {}
 
   server.middlewares.use(options.route, (req: import('http').IncomingMessage, res: import('http').ServerResponse, next: Function) => {
     if (req.method !== 'POST') return next();