[bugfix] no more proxy in "/v1/model" and "/v1/shields" path (#2324)

* fixed a bug where /v1/models was being proxied (adding userId) at the end.

Signed-off-by: Lucas <lyoon@redhat.com>

* skipping /models and /shields completely for history length and user id

Signed-off-by: Lucas <lyoon@redhat.com>

---------

Signed-off-by: Lucas <lyoon@redhat.com>

Author: JslYoon

workspaces/lightspeed/plugins/lightspeed-backend/src/service/router.ts

@@ -37,6 +37,8 @@ import {
 } from './types';
 import { validateCompletionsRequest } from './validation';
 
+const SKIP_USER_ID_ENDPOINTS = new Set(['/v1/models', '/v1/shields']);
+
 /**
  * @public
  * The lightspeed backend router
@@ -105,19 +107,33 @@ export async function createRouter(
       target: `http://0.0.0.0:${port}`,
       changeOrigin: true,
       pathRewrite: (path, _) => {
-        // Add user query parameter from the authenticated user
+        const isSkippable = Array.from(SKIP_USER_ID_ENDPOINTS).some(endpoint =>
+          path.startsWith(endpoint),
+        );
+
+        if (isSkippable) {
+          return path;
+        }
+
+        let newPath = path;
+
+        // Add user_id
         const userQueryParam = `user_id=${encodeURIComponent(userEntity)}`;
-        // Check if there are already query parameters
-        let newPath = path.includes('?')
+        newPath = path.includes('?')
           ? `${path}&${userQueryParam}`
           : `${path}?${userQueryParam}`;
+
+        // Add history_length if needed
         if (
           !path.includes('history_length') &&
           path.includes('conversation_id')
         ) {
           const historyLengthQuery = `history_length=${DEFAULT_HISTORY_LENGTH}`;
-          newPath = `${newPath}&${historyLengthQuery}`;
+          newPath = newPath.includes('?')
+            ? `${newPath}&${historyLengthQuery}`
+            : `${newPath}?${historyLengthQuery}`;
         }
+
         logger.info(`Rewriting path from ${path} to ${newPath}`);
         return newPath;
       },