From 5c5310065160babd3b185eaf8e96fd08d75be2b8 Mon Sep 17 00:00:00 2001
From: Haileyesus <something@gmail.com>
Date: Thu, 16 Apr 2026 20:58:16 +0300
Subject: [PATCH] refactor: put /api/providers in index.js and remove
 /providers prefix from provider.routes.ts

---
 server/index.js                             |  4 ++++
 server/modules/providers/provider.routes.ts | 10 +++++-----
 2 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/server/index.js b/server/index.js
index 5f7e7a4e..28b9e184 100755
--- a/server/index.js
+++ b/server/index.js
@@ -69,6 +69,7 @@ import codexRoutes from './routes/codex.js';
 import geminiRoutes from './routes/gemini.js';
 import pluginsRoutes from './routes/plugins.js';
 import messagesRoutes from './routes/messages.js';
+import providerRoutes from './modules/providers/provider.routes.js';
 import { createNormalizedMessage } from './providers/types.js';
 import { startEnabledPluginServers, stopAllPlugins, getPluginPort } from './utils/plugin-process-manager.js';
 import { initializeDatabase, sessionNamesDb, applyCustomSessionNames } from './database/db.js';
@@ -404,6 +405,9 @@ app.use('/api/plugins', authenticateToken, pluginsRoutes);
 // Unified session messages route (protected)
 app.use('/api/sessions', authenticateToken, messagesRoutes);
 
+// Unified provider MCP routes (protected)
+app.use('/api/providers', authenticateToken, providerRoutes);
+
 // Agent API Routes (uses API key authentication)
 app.use('/api/agent', agentRoutes);
 
diff --git a/server/modules/providers/provider.routes.ts b/server/modules/providers/provider.routes.ts
index 0e376523..22da9658 100644
--- a/server/modules/providers/provider.routes.ts
+++ b/server/modules/providers/provider.routes.ts
@@ -143,7 +143,7 @@ const parseProvider = (value: unknown): LLMProvider => {
 };
 
 router.get(
-  '/providers/:provider/mcp/servers',
+  '/:provider/mcp/servers',
   asyncHandler(async (req: Request, res: Response) => {
     const provider = parseProvider(req.params.provider);
     const workspacePath = readOptionalQueryString(req.query.workspacePath);
@@ -161,7 +161,7 @@ router.get(
 );
 
 router.post(
-  '/providers/:provider/mcp/servers',
+  '/:provider/mcp/servers',
   asyncHandler(async (req: Request, res: Response) => {
     const provider = parseProvider(req.params.provider);
     const payload = parseMcpUpsertPayload(req.body);
@@ -171,7 +171,7 @@ router.post(
 );
 
 router.put(
-  '/providers/:provider/mcp/servers/:name',
+  '/:provider/mcp/servers/:name',
   asyncHandler(async (req: Request, res: Response) => {
     const provider = parseProvider(req.params.provider);
     const payload = parseMcpUpsertPayload({
@@ -184,7 +184,7 @@ router.put(
 );
 
 router.delete(
-  '/providers/:provider/mcp/servers/:name',
+  '/:provider/mcp/servers/:name',
   asyncHandler(async (req: Request, res: Response) => {
     const provider = parseProvider(req.params.provider);
     const scope = parseMcpScope(req.query.scope);
@@ -199,7 +199,7 @@ router.delete(
 );
 
 router.post(
-  '/providers/:provider/mcp/servers/:name/run',
+  '/:provider/mcp/servers/:name/run',
   asyncHandler(async (req: Request, res: Response) => {
     const provider = parseProvider(req.params.provider);
     const body = (req.body as Record<string, unknown> | undefined) ?? {};