fix: rename frontend redirection route

Author: Vincent Molinié

packages/mcp-server/src/forest-oauth-provider.test.ts

@@ -263,7 +263,7 @@ describe('ForestAdminOAuthProvider', () => {
       );
 
       expect(mockResponse.redirect).toHaveBeenCalledWith(
-        expect.stringContaining('https://app.forestadmin.com/authentication/mcp-login'),
+        expect.stringContaining('https://app.forestadmin.com/oauth/authorize'),
       );
     });
 
@@ -284,7 +284,7 @@ describe('ForestAdminOAuthProvider', () => {
       const url = new URL(redirectCall);
 
       expect(url.hostname).toBe('app.forestadmin.com');
-      expect(url.pathname).toBe('/authentication/mcp-login');
+      expect(url.pathname).toBe('/oauth/authorize');
       expect(url.searchParams.get('redirect_uri')).toBe('https://example.com/callback');
       expect(url.searchParams.get('code_challenge')).toBe('test-code-challenge');
       expect(url.searchParams.get('client_id')).toBe('test-client-id');

packages/mcp-server/src/forest-oauth-provider.ts

@@ -86,7 +86,7 @@ export default class ForestAdminOAuthProvider implements OAuthServerProvider {
     try {
       // Redirect to Forest Admin agent for actual authentication
       const agentAuthUrl = new URL(
-        '/authentication/mcp-login',
+        '/oauth/authorize',
         process.env.FOREST_FRONTEND_HOSTNAME || 'https://app.forestadmin.com',
       );
 

packages/mcp-server/src/server.test.ts

@@ -337,9 +337,7 @@ describe('ForestAdminMCPServer Instance', () => {
         });
 
         expect(response.status).toBe(302);
-        expect(response.headers.location).toContain(
-          'https://app.forestadmin.com/authentication/mcp-login',
-        );
+        expect(response.headers.location).toContain('https://app.forestadmin.com/oauth/authorize');
 
         const redirectUrl = new URL(response.headers.location);
         expect(redirectUrl.searchParams.get('redirect_uri')).toBe('https://example.com/callback');
@@ -362,9 +360,7 @@ describe('ForestAdminMCPServer Instance', () => {
         });
 
         expect(response.status).toBe(302);
-        expect(response.headers.location).toContain(
-          'https://app.forestadmin.com/authentication/mcp-login',
-        );
+        expect(response.headers.location).toContain('https://app.forestadmin.com/oauth/authorize');
       });
 
       it('should handle POST method for authorize', async () => {
@@ -382,7 +378,7 @@ describe('ForestAdminMCPServer Instance', () => {
 
         expect(response.status).toBe(302);
         expect(response.headers.location).toStrictEqual(
-          `https://app.forestadmin.com/authentication/mcp-login?redirect_uri=${encodeURIComponent(
+          `https://app.forestadmin.com/oauth/authorize?redirect_uri=${encodeURIComponent(
             'https://example.com/callback',
           )}&code_challenge=test-challenge&client_id=registered-client&state=test-state&scope=${encodeURIComponent(
             'mcp:read',