chore(react-router): Remove deprecated pre-middleware setup (#7796)

Author: wobsoriano

.changeset/green-humans-yawn.md

@@ -0,0 +1,36 @@
+---
+'@clerk/react-router': major
+---
+
+Usage of `rootAuthLoader` without the `clerkMiddleware()` installed will not throw a runtime error.
+
+**Before (Removed):**
+
+```tsx
+import { rootAuthLoader } from '@clerk/react-router/ssr.server'
+
+export const loader = (args: Route.LoaderArgs) => rootAuthLoader(args)
+```
+
+**After:**
+
+1. Enable the `v8_middleware` future flag:
+
+```ts
+// react-router.config.ts
+export default {
+  future: {
+    v8_middleware: true,
+  },
+} satisfies Config;
+```
+
+2. Use the middleware in your app:
+
+```tsx
+import { clerkMiddleware, rootAuthLoader } from '@clerk/react-router/server'
+
+export const middleware: Route.MiddlewareFunction[] = [clerkMiddleware()]
+
+export const loader = (args: Route.LoaderArgs) => rootAuthLoader(args)
+```

integration/tests/react-router/pre-middleware.test.ts

@@ -4,28 +4,14 @@ import { beforeEach, describe, expect, it, vi } from 'vitest';
 
 import { authFnContext } from '../clerkMiddleware';
 import { getAuth } from '../getAuth';
-import { legacyAuthenticateRequest } from '../legacyAuthenticateRequest';
-
-vi.mock('../legacyAuthenticateRequest', () => {
-  return {
-    legacyAuthenticateRequest: vi.fn().mockResolvedValue({
-      toAuth: vi.fn().mockImplementation(() => ({
-        userId: 'user_xxx',
-        tokenType: TokenType.SessionToken,
-      })),
-      headers: new Headers(),
-      status: 'signed-in',
-    }),
-  };
-});
 
 describe('getAuth', () => {
   beforeEach(() => {
     vi.clearAllMocks();
     process.env.CLERK_SECRET_KEY = 'sk_test_...';
   });
 
-  it('should not call legacyAuthenticateRequest when middleware context exists', async () => {
+  it('should work when middleware context exists', async () => {
     const mockContext = {
       get: vi.fn().mockImplementation(contextKey => {
         if (contextKey === authFnContext) {
@@ -47,12 +33,11 @@ describe('getAuth', () => {
 
     const auth = await getAuth(args);
 
-    expect(legacyAuthenticateRequest).not.toHaveBeenCalled();
     expect(auth.userId).toBe('user_xxx');
     expect(auth.tokenType).toBe('session_token');
   });
 
-  it('should call legacyAuthenticateRequest when middleware context is missing', async () => {
+  it('should throw an error when middleware context is missing', async () => {
     const mockContext = {
       get: vi.fn().mockReturnValue(null),
     };
@@ -62,10 +47,6 @@ describe('getAuth', () => {
       request: new Request('http://clerk.com'),
     } as LoaderFunctionArgs;
 
-    const auth = await getAuth(args);
-
-    expect(legacyAuthenticateRequest).toHaveBeenCalled();
-    expect(auth.userId).toBe('user_xxx');
-    expect(auth.tokenType).toBe('session_token');
+    await expect(getAuth(args)).rejects.toThrow('Clerk: clerkMiddleware() not detected');
   });
 });