Update server.js

server.js

@@ -7,23 +7,27 @@ const app = express();
 const PORT = process.env.PORT || 7860;
 
 // --- 1. STRICT CORS CONFIGURATION ---
-// We explicitly define allowed methods and headers to prevent browser blocks
-app.use(cors({
-  origin: '*', // Allow ChatUI/Browsers from anywhere
+// We define the config object
+const corsOptions = {
+  origin: '*', 
   methods: ['GET', 'POST', 'OPTIONS'],
   allowedHeaders: ['Content-Type', 'Authorization', 'X-Requested-With'],
-  credentials: false // Important: must be false if origin is '*'
-}));
+  credentials: false 
+};
 
-// Explicitly handle Preflight (OPTIONS) requests
-app.options('*', cors());
+// Apply Global CORS Middleware
+app.use(cors(corsOptions));
+
+// FIX: Handle Preflight (OPTIONS) using Regex /.*/ instead of '*' 
+// This prevents the "Missing parameter name" crash in Express 5
+app.options(/.*/, cors(corsOptions));
 
 // Increase payload limit
 app.use(express.json({ limit: '10mb' }));
 
 // Logging
 app.use((req, res, next) => {
-  console.log(`[${new Date().toISOString()}] ${req.method} ${req.originalUrl}`);
+  console.log(`[${new Date().toISOString()}] ${req.method} ${req.url}`);
   next();
 });
 
@@ -66,7 +70,6 @@ app.post('/messages', async (req, res) => {
   try {
     const body = req.body;
     // Ensure we always have a valid ID (JSON-RPC requirement)
-    // If body.id is missing, use null.
     const requestId = (body.id !== undefined) ? body.id : null;
 
     console.log(`[MESSAGES] Method: ${body.method} | ID: ${requestId}`);
@@ -78,9 +81,7 @@ app.post('/messages', async (req, res) => {
         id: requestId,
         result: {
           protocolVersion: "2024-11-05",
-          capabilities: { 
-            tools: {} // We announce we have tools capability
-          },
+          capabilities: { tools: {} },
           serverInfo: { name: "github-mcp-hf", version: "1.0.0" }
         }
       };
@@ -176,7 +177,7 @@ app.post('/messages', async (req, res) => {
 });
 
 // --- 4. CATCH-ALL / HEALTH CHECK ---
-// Defined LAST to ensure it doesn't intercept real routes
+// Use standard middleware signature to match everything without crashing
 app.use((req, res) => {
   if (req.method === 'GET' || req.method === 'POST') {
     console.log(`[FALLBACK] Responding 200 OK to: ${req.path}`);