delete unsued exports files

vercel · Feb 6, 2024 · 0ba1c20 · 0ba1c20
1 parent 138b73e
commit 0ba1c20
Show file tree

Hide file tree

Showing 13 changed files with 15 additions and 33 deletions.
diff --git a/packages/next/server.js b/packages/next/server.js
@@ -9,11 +9,8 @@ const serverExports = {
     .userAgentFromString,
   userAgent: require('next/dist/server/web/spec-extension/user-agent')
     .userAgent,
-}
-
-if (typeof URLPattern !== 'undefined') {
-  // eslint-disable-next-line no-undef
-  serverExports.URLPattern = URLPattern
+  URLPattern: require('next/dist/server/web/spec-extension/url-pattern')
+    .URLPattern,
 }
 
 // https://nodejs.org/api/esm.html#commonjs-namespaces

diff --git a/packages/next/src/server/web/exports/image-response.ts b/packages/next/src/server/web/exports/image-response.ts
diff --git a/packages/next/src/server/web/exports/index.ts b/packages/next/src/server/web/exports/index.ts
@@ -1,8 +1,7 @@
 // Alias index file of next/server for edge runtime for tree-shaking purpose
 
-export { default as ImageResponse } from './image-response'
-export { default as NextRequest } from './next-request'
-export { default as NextResponse } from './next-response'
-export { default as userAgent } from './user-agent'
-export { default as userAgentFromString } from './user-agent-from-string'
-export { default as URLPattern } from './url-pattern'
+export { ImageResponse } from '../spec-extension/image-response'
+export { NextRequest } from '../spec-extension/request'
+export { NextResponse } from '../spec-extension/response'
+export { userAgent, userAgentFromString } from '../spec-extension/user-agent'
+export { URLPattern } from '../spec-extension/url-pattern'
diff --git a/packages/next/src/server/web/exports/next-request.ts b/packages/next/src/server/web/exports/next-request.ts
diff --git a/packages/next/src/server/web/exports/next-response.ts b/packages/next/src/server/web/exports/next-response.ts
diff --git a/packages/next/src/server/web/exports/revalidate-path.ts b/packages/next/src/server/web/exports/revalidate-path.ts
diff --git a/packages/next/src/server/web/exports/revalidate-tag.ts b/packages/next/src/server/web/exports/revalidate-tag.ts
diff --git a/packages/next/src/server/web/exports/unstable-cache.ts b/packages/next/src/server/web/exports/unstable-cache.ts
diff --git a/packages/next/src/server/web/exports/unstable-no-store.ts b/packages/next/src/server/web/exports/unstable-no-store.ts
diff --git a/packages/next/src/server/web/exports/user-agent-from-string.ts b/packages/next/src/server/web/exports/user-agent-from-string.ts
diff --git a/packages/next/src/server/web/exports/user-agent.ts b/packages/next/src/server/web/exports/user-agent.ts
diff --git a/...ext/src/server/web/exports/url-pattern.ts → .../server/web/spec-extension/url-pattern.ts b/...ext/src/server/web/exports/url-pattern.ts → .../server/web/spec-extension/url-pattern.ts
@@ -2,4 +2,4 @@ const GlobalURLPattern =
   // @ts-expect-error: URLPattern is not available in Node.js
   typeof URLPattern === 'undefined' ? undefined : URLPattern
 
-export default GlobalURLPattern
+export { GlobalURLPattern as URLPattern }
diff --git a/test/e2e/app-dir/app-external/app-external.test.ts b/test/e2e/app-dir/app-external/app-external.test.ts
@@ -37,7 +37,7 @@ createNextDescribe(
     buildCommand: 'yarn build',
     skipDeployment: true,
   },
-  ({ next }) => {
+  ({ next, isNextStart }) => {
     it('should be able to opt-out 3rd party packages being bundled in server components', async () => {
       await next.fetch('/react-server/optout').then(async (response) => {
         const result = await resolveStreamResponse(response)
@@ -249,8 +249,12 @@ createNextDescribe(
       const html = await next.render('/test-middleware')
       expect(html).toContain('it works')
 
-      const middlewareBundle = await next.readFile('.next/server/middleware.js')
-      expect(middlewareBundle).not.toContain('image-response')
+      if (isNextStart) {
+        const middlewareBundle = await next.readFile(
+          '.next/server/middleware.js'
+        )
+        expect(middlewareBundle).not.toContain('image-response')
+      }
     })
 
     it('should use the same async storages if imported directly', async () => {