From 53e6d159c472d96bdfb8b3f1c30c941c9e514a1b Mon Sep 17 00:00:00 2001 From: Mark Erikson Date: Sun, 6 Apr 2025 22:08:07 -0400 Subject: [PATCH] Improve duplicate middleware error and save build output --- errors.json | 3 ++- packages/toolkit/src/configureStore.ts | 2 +- packages/toolkit/src/query/tests/injectEndpoints.test.tsx | 2 +- packages/toolkit/src/tests/configureStore.test.ts | 2 +- 4 files changed, 5 insertions(+), 4 deletions(-) diff --git a/errors.json b/errors.json index 082897b010..8169973800 100644 --- a/errors.json +++ b/errors.json @@ -40,5 +40,6 @@ "38": "Cannot refetch a query that has not been started yet.", "39": "called \\`injectEndpoints\\` to override already-existing endpointName without specifying \\`overrideExisting: true\\`", "40": "maxPages for endpoint '' must be a number greater than 0", - "41": "getPreviousPageParam for endpoint '' must be a function if maxPages is used" + "41": "getPreviousPageParam for endpoint '' must be a function if maxPages is used", + "42": "Duplicate middleware references found when creating the store. Ensure that each middleware is only included once." } \ No newline at end of file diff --git a/packages/toolkit/src/configureStore.ts b/packages/toolkit/src/configureStore.ts index 70529fabd2..c367609897 100644 --- a/packages/toolkit/src/configureStore.ts +++ b/packages/toolkit/src/configureStore.ts @@ -187,7 +187,7 @@ export function configureStore< finalMiddleware.forEach((middleware) => { if (middlewareReferences.has(middleware)) { throw new Error( - 'Duplicate middleware found. Ensure that each middleware is only included once', + 'Duplicate middleware references found when creating the store. Ensure that each middleware is only included once.', ) } middlewareReferences.add(middleware) diff --git a/packages/toolkit/src/query/tests/injectEndpoints.test.tsx b/packages/toolkit/src/query/tests/injectEndpoints.test.tsx index 89404bff8e..2f00e37cec 100644 --- a/packages/toolkit/src/query/tests/injectEndpoints.test.tsx +++ b/packages/toolkit/src/query/tests/injectEndpoints.test.tsx @@ -115,7 +115,7 @@ describe('injectEndpoints', () => { }) expect(makeStore).toThrowError( - 'Duplicate middleware found. Ensure that each middleware is only included once', + 'Duplicate middleware references found when creating the store. Ensure that each middleware is only included once.', ) }) }) diff --git a/packages/toolkit/src/tests/configureStore.test.ts b/packages/toolkit/src/tests/configureStore.test.ts index de78a80700..c5d75c3b8c 100644 --- a/packages/toolkit/src/tests/configureStore.test.ts +++ b/packages/toolkit/src/tests/configureStore.test.ts @@ -143,7 +143,7 @@ describe('configureStore', async () => { } expect(makeStore).toThrowError( - 'Duplicate middleware found. Ensure that each middleware is only included once', + 'Duplicate middleware references found when creating the store. Ensure that each middleware is only included once.', ) })