chore: bump supported msw version to v2.3.0 (#66351)

### What?

This bumps the supported `msw` version when using the `playwright` test
mode. There's no need for mocking requests and responses anymore since
[they adopted](mswjs/msw#1436) the global fetch
API in `msw`.

### Why?

Users can extend `test` from `playwright` manually and create the `msw`
fixtures, but it's better if it works out of the box with newer versions
of `msw`.

Fixes #63338.

Co-authored-by: Sam Ko <[email protected]>

Author: 2 people

packages/next/package.json

@@ -259,7 +259,7 @@
     "lodash.curry": "4.1.1",
     "lru-cache": "5.1.1",
     "mini-css-extract-plugin": "2.4.4",
-    "msw": "1.3.0",
+    "msw": "2.3.0",
     "nanoid": "3.1.32",
     "native-url": "0.3.4",
     "neo-async": "2.6.1",

packages/next/src/experimental/testmode/playwright/msw.ts

@@ -1,12 +1,7 @@
 import { test as base, defineConfig } from './index'
 import type { NextFixture } from './next-fixture'
 // eslint-disable-next-line import/no-extraneous-dependencies
-import {
-  type RequestHandler,
-  type MockedResponse,
-  MockedRequest,
-  handleRequest,
-} from 'msw'
+import { type RequestHandler, handleRequest } from 'msw'
 // eslint-disable-next-line import/no-extraneous-dependencies
 import { Emitter } from 'strict-event-emitter'
 
@@ -33,39 +28,13 @@ export const test = base.extend<{
       const emitter = new Emitter()
 
       next.onFetch(async (request) => {
-        const {
-          body,
-          method,
-          headers,
-          credentials,
-          cache,
-          redirect,
-          integrity,
-          keepalive,
-          mode,
-          destination,
-          referrer,
-          referrerPolicy,
-        } = request
-        const mockedRequest = new MockedRequest(new URL(request.url), {
-          body: body ? await request.arrayBuffer() : undefined,
-          method,
-          headers: Object.fromEntries(headers),
-          credentials,
-          cache,
-          redirect,
-          integrity,
-          keepalive,
-          mode,
-          destination,
-          referrer,
-          referrerPolicy,
-        })
+        const requestId = Math.random().toString(16).slice(2)
         let isUnhandled = false
         let isPassthrough = false
-        let mockedResponse: MockedResponse | undefined
+        let mockedResponse
         await handleRequest(
-          mockedRequest,
+          request.clone(),
+          requestId,
           handlers.slice(0),
           {
             onUnhandledRequest: () => {
@@ -91,19 +60,7 @@ export const test = base.extend<{
         }
 
         if (mockedResponse) {
-          const {
-            status,
-            headers: responseHeaders,
-            body: responseBody,
-            delay,
-          } = mockedResponse
-          if (delay) {
-            await new Promise((resolve) => setTimeout(resolve, delay))
-          }
-          return new Response(responseBody, {
-            status,
-            headers: new Headers(responseHeaders),
-          })
+          return mockedResponse
         }
 
         return 'abort'