msw 2.14.5 → 2.14.7

package/lib/mockServiceWorker.js

@@ -7,7 +7,7 @@
  * - Please do NOT modify this file.
  */
 
-const PACKAGE_VERSION = '2.14.5'
+const PACKAGE_VERSION = '2.14.7'
 const INTEGRITY_CHECKSUM = '4db4a41e972cec1b64cc569c66952d82'
 const IS_MOCKED_RESPONSE = Symbol('isMockedResponse')
 const activeClientIds = new Set()

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "msw",
-  "version": "2.14.5",
+  "version": "2.14.7",
   "description": "Seamless REST/GraphQL API mocking library for browser and Node.js.",
   "type": "commonjs",
   "main": "./lib/core/index.js",
@@ -248,7 +248,7 @@
     "@fastify/websocket": "^11.2.0",
     "@graphql-typed-document-node/core": "^3.2.0",
     "@open-draft/test-server": "^0.4.2",
-    "@ossjs/release": "^0.10.1",
+    "@ossjs/release": "^0.11.1",
     "@playwright/test": "^1.59.1",
     "@types/express": "^5.0.6",
     "@types/json-bigint": "^1.0.4",
@@ -280,13 +280,12 @@
     "simple-git-hooks": "^2.13.1",
     "tsup": "^8.5.1",
     "typescript": "^5.9.3",
-    "undici": "^8.1.0",
+    "undici": "^7.28.0",
     "url-loader": "^4.1.1",
     "vitest": "^4.1.4",
-    "vitest-environment-miniflare": "^2.14.4",
     "webpack": "^5.106.2",
     "webpack-http-server": "^0.5.0",
-    "msw": "^2.14.5"
+    "msw": "^2.14.7"
   },
   "peerDependencies": {
     "typescript": ">= 4.8.x"

package/src/core/experimental/define-network.ts

@@ -12,6 +12,7 @@ import {
   type AnyHandler,
 } from './handlers-controller'
 import { toReadonlyArray } from '../utils/internal/toReadonlyArray'
+import { Disposable } from '../utils/internal/Disposable'
 
 type UnionToIntersection<U> = (U extends any ? (k: U) => void : never) extends (
   k: infer I,
@@ -112,6 +113,7 @@ export function defineNetwork<Sources extends Array<NetworkSource<any>>>(
 ): NetworkApi<Sources> {
   let readyState: NetworkReadyState = NetworkReadyState.DISABLED
   const events = new Emitter<MergeEventMaps<Sources>>()
+  const disposable = new Disposable()
 
   const deriveHandlersController = (
     handlers: DefineNetworkOptions<Sources>['handlers'],
@@ -130,7 +132,6 @@ export function defineNetwork<Sources extends Array<NetworkSource<any>>>(
    * certain setup APIs, like `setupServer`, don't await `.enable` (`.listen`).
    */
   let handlersController = deriveHandlersController(resolvedOptions.handlers)
-  let listenersController: AbortController
 
   return {
     get readyState() {
@@ -138,7 +139,10 @@ export function defineNetwork<Sources extends Array<NetworkSource<any>>>(
     },
     events,
     configure(options) {
-      invariant(readyState === NetworkReadyState.DISABLED, '')
+      invariant(
+        readyState === NetworkReadyState.DISABLED,
+        'Failed to call "configure()" on the network: cannot configure an already enabled network.',
+      )
 
       if (
         options.handlers &&
@@ -158,9 +162,18 @@ export function defineNetwork<Sources extends Array<NetworkSource<any>>>(
         'Failed to call "enable" on the network: already enabled',
       )
 
-      listenersController = new AbortController()
       readyState = NetworkReadyState.ENABLED
 
+      /**
+       * @note Use a session object scoped to the current "enable()"
+       * to prevent "frame.events" listeners from surviving across enable/disable cycles.
+       * @see The note about `AbortController` below.
+       */
+      const session = { active: true }
+      disposable['subscriptions'].push(() => {
+        session.active = false
+      })
+
       const result = resolvedOptions.sources.map((source) => {
         /**
          * @note Preemptively disable the network source before enabling.
@@ -171,8 +184,19 @@ export function defineNetwork<Sources extends Array<NetworkSource<any>>>(
         NetworkSource.prototype.disable.call(source)
 
         source.on('frame', async ({ frame }) => {
-          frame.events.on('*', (event) => events.emit(event), {
-            signal: listenersController.signal,
+          frame.events.on('*', (event) => {
+            /**
+             * @note Prevent event forwarding manually and not via an AbortController
+             * because certain runtimes, like Cloudflare, throw when referencing an
+             * AbortController created in a different context. Bear in mind that the frame
+             * events run in the patched request client context while the AbortController
+             * is created outside, in the "defineNetwork" closure, which is a test context.
+             */
+            if (!session.active) {
+              return
+            }
+
+            events.emit(event)
           })
 
           const handlers = frame.getHandlers(handlersController)
@@ -197,8 +221,8 @@ export function defineNetwork<Sources extends Array<NetworkSource<any>>>(
         'Failed to call "disable" on the network: already disabled',
       )
 
-      listenersController.abort()
       readyState = NetworkReadyState.DISABLED
+      disposable.dispose()
 
       return colorlessPromiseAll(
         resolvedOptions.sources.map((source) => source.disable()),

package/src/core/sse.ts

@@ -80,7 +80,7 @@ const SSE_RESPONSE_INIT: ResponseInit = {
 class ServerSentEventHandler<
   EventMap extends EventMapConstraint,
 > extends HttpHandler {
-  #emitter: Emitter<ServerSentEventClientEventMap>
+  #loggableRequests: WeakSet<Request>
 
   constructor(path: Path, resolver: ServerSentEventResolver<EventMap, any>) {
     invariant(
@@ -94,7 +94,16 @@ class ServerSentEventHandler<
         info.request,
       )
 
-      client[kClientEmitter] = this.#emitter
+      /**
+       * @note Scope the emitter to this connection so its logging
+       * listeners don't accumulate on the handler across connections.
+       */
+      const emitter = new Emitter<ServerSentEventClientEventMap>()
+      client[kClientEmitter] = emitter
+
+      if (this.#loggableRequests.has(info.request)) {
+        this.#attachClientLogger(info.request, emitter)
+      }
 
       await resolver({
         ...info,
@@ -105,7 +114,7 @@ class ServerSentEventHandler<
       return response
     })
 
-    this.#emitter = new Emitter<ServerSentEventClientEventMap>()
+    this.#loggableRequests = new WeakSet()
   }
 
   async predicate(args: {
@@ -134,7 +143,9 @@ class ServerSentEventHandler<
         response: new Response('[streaming]', SSE_RESPONSE_INIT),
       })
 
-      this.#attachClientLogger(args.request, this.#emitter)
+      // Defer attaching the client logger until the resolver runs
+      // and creates a connection-scoped emitter for this request.
+      this.#loggableRequests.add(args.request)
     }
 
     return matches

package/src/core/utils/internal/Disposable.ts

@@ -1,3 +1,5 @@
+import { devUtils } from './devUtils'
+
 export type DisposableSubscription = () => void
 
 export class Disposable {
@@ -5,8 +7,27 @@ export class Disposable {
 
   public dispose() {
     let subscription: DisposableSubscription | undefined
+    const errors: Array<Error> = []
+
     while ((subscription = this.subscriptions.shift())) {
-      subscription()
+      try {
+        subscription()
+      } catch (error) {
+        if (error instanceof Error) {
+          errors.push(error)
+        }
+      }
+    }
+
+    if (errors.length > 0) {
+      console.error(
+        new AggregateError(
+          errors,
+          devUtils.formatMessage(
+            'Failed to dispose of some side effects. This is likely an issue with MSW, please report it on GitHub: https://github.com/mswjs/msw/issues',
+          ),
+        ),
+      )
     }
   }
 }