vercel · padmaia · May 23, 2023 · May 18, 2023 · May 22, 2023 · May 23, 2023
@@ -102,7 +102,7 @@ pub async fn get_server_resolve_options_context(
                 enable_node_externals: true,
                 enable_node_native_modules: true,
                 module: true,
-                custom_conditions: vec![mode.node_env().to_string()],
+                custom_conditions: vec![mode.node_env().to_string(), "node".to_string()],
                 import_map: Some(next_server_import_map),
                 plugins: vec![
                     external_cjs_modules_plugin.into(),
@@ -126,7 +126,11 @@ pub async fn get_server_resolve_options_context(
                 enable_node_externals: true,
                 enable_node_native_modules: true,
                 module: true,
-                custom_conditions: vec![mode.node_env().to_string()],
+                custom_conditions: vec![
+                    mode.node_env().to_string(),
+                    // TODO!
+                    "node".to_string(),
+                ],
                 import_map: Some(next_server_import_map),
                 plugins: vec![
                     server_component_externals_plugin.into(),
@@ -150,7 +154,12 @@ pub async fn get_server_resolve_options_context(
                 enable_node_externals: true,
                 enable_node_native_modules: true,
                 module: true,
-                custom_conditions: vec![mode.node_env().to_string(), "react-server".to_string()],
+                custom_conditions: vec![
+                    mode.node_env().to_string(),
+                    "react-server".to_string(),
+                    // TODO
+                    "node".to_string(),
+                ],
                 import_map: Some(next_server_import_map),
                 plugins: vec![
                     server_component_externals_plugin.into(),
@@ -172,7 +181,7 @@ pub async fn get_server_resolve_options_context(
             let resolve_options_context = ResolveOptionsContext {
                 enable_node_modules: Some(root_dir),
                 module: true,
-                custom_conditions: vec![mode.node_env().to_string()],
+                custom_conditions: vec![mode.node_env().to_string(), "node".to_string()],
                 import_map: Some(next_server_import_map),
                 plugins: vec![
                     server_component_externals_plugin.into(),

@@ -0,0 +1,11 @@
+import edgeThenNode from 'edge-then-node'
+import nodeThenEdge from 'node-then-edge'
+
+export const runtime = 'edge'
+
+export default function AppEdge() {
+  return JSON.stringify({
+    edgeThenNode,
+    nodeThenEdge,
+  })
+}
@@ -0,0 +1,11 @@
+import edgeThenNode from 'edge-then-node'
+import nodeThenEdge from 'node-then-edge'
+
+export const runtime = 'nodejs'
+
+export default function AppNodeJs() {
+  return JSON.stringify({
+    edgeThenNode,
+    nodeThenEdge,
+  })
+}
@@ -0,0 +1,7 @@
+export default function RootLayout({ children }: { children: any }) {
+  return (
+    <html>
+      <body>{children}</body>
+    </html>
+  )
+}
@@ -0,0 +1,3 @@
+export default function Loading() {
+  return <>Loading</>
+}
@@ -0,0 +1,9 @@
+import Test from './test'
+
+export default function Page() {
+  return (
+    <div>
+      <Test />
+    </div>
+  )
+}
@@ -0,0 +1,11 @@
+import edgeThenNode from 'edge-then-node'
+import nodeThenEdge from 'node-then-edge'
+
+export const runtime = 'edge'
+
+export default function RouteEdge() {
+  return Response.json({
+    edgeThenNode,
+    nodeThenEdge,
+  })
+}
@@ -0,0 +1,11 @@
+import edgeThenNode from 'edge-then-node'
+import nodeThenEdge from 'node-then-edge'
+
+export const runtime = 'nodejs'
+
+export default function RouteNodeJs() {
+  return Response.json({
+    edgeThenNode,
+    nodeThenEdge,
+  })
+}
@@ -0,0 +1,103 @@
+'use client'
+
+import { useTestHarness } from '@turbo/pack-test-harness'
+
+export default function Test() {
+  useTestHarness(runTests)
+}
+
+async function getJson(url) {
+  const res = await fetch(url)
+  const text = await res.text()
+  const jsonText = /(\{[^}]*\})/.exec(text)
+  return JSON.parse(jsonText[0].replace(/&quot;/g, '"'))
+}
+
+function runTests() {
+  it('page with nodejs runtime should import node conditions', async () => {
+    const json = await getJson('/page-nodejs')
+    expect(json).toMatchObject({
+      edgeThenNode: 'node',
+      nodeThenEdge: 'node',
+    })
+  })
+
+  it('page with edge runtime should import edge conditions', async () => {
+    const json = await getJson('/page-edge')
+    // TODO We don't currently support edge config in page rendering.
+    // When we do, this needs to be updated.
+    expect(json).not.toMatchObject({
+      edgeThenNode: 'edge',
+      nodeThenEdge: 'edge',
+    })
+    // TODO: delete this.
+    expect(json).toMatchObject({
+      edgeThenNode: 'node',
+      nodeThenEdge: 'node',
+    })
+  })
+
+  it('page api with nodejs runtime should import node conditions', async () => {
+    const json = await getJson('/api/api-nodejs')
+    expect(json).toMatchObject({
+      edgeThenNode: 'node',
+      nodeThenEdge: 'node',
+    })
+  })
+
+  it('page api with edge runtime should import edge conditions', async () => {
+    const json = await getJson('/api/api-edge')
+    expect(json).toMatchObject({
+      edgeThenNode: 'edge',
+      nodeThenEdge: 'edge',
+    })
+  })
+
+  it('app with nodejs runtime should import node conditions', async () => {
+    const json = await getJson('/app-nodejs')
+    expect(json).toMatchObject({
+      edgeThenNode: 'node',
+      nodeThenEdge: 'node',
+    })
+  })
+
+  it('app with edge runtime should import edge conditions', async () => {
+    const json = await getJson('/app-edge')
+    // TODO We don't currently support edge config in app rendering.
+    // When we do, this needs to be updated.
+    expect(json).not.toMatchObject({
+      edgeThenNode: 'edge',
+      nodeThenEdge: 'edge',
+    })
+    // TODO: delete this.
+    expect(json).toMatchObject({
+      edgeThenNode: 'node',
+      nodeThenEdge: 'node',
+    })
+  })
+
+  it('app route with nodejs runtime should import node conditions', async () => {
+    const json = await getJson('/route-nodejs')
+    expect(json).toMatchObject({
+      edgeThenNode: 'node',
+      nodeThenEdge: 'node',
+    })
+  })
+
+  it('app route with edge runtime should import edge conditions', async () => {
+    const json = await getJson('/route-edge')
+    expect(json).toMatchObject({
+      edgeThenNode: 'edge',
+      nodeThenEdge: 'edge',
+    })
+  })
+
+  it('middleware should import edge conditions', async () => {
+    const res = await fetch('/middleware')
+    const json = await res.json()
+    expect(json).toMatchObject({
+      edgeThenNode: 'edge',
+      nodeThenEdge: 'edge',
+    })
+  })
+}
@@ -0,0 +1 @@
+module.exports = 'default'
@@ -0,0 +1 @@
+module.exports = 'edge'
@@ -0,0 +1 @@
+module.exports = 'main'
@@ -0,0 +1 @@
+module.exports = 'node'
@@ -0,0 +1,10 @@
+{
+  "main": "main.js",
+  "exports": {
+    ".": {
+      "edge-light": "./edge.js",
+      "node": "./node.js",
+      "default": "./default.js"
+    }
+  }
+}
@@ -0,0 +1,13 @@
+import edgeThenNode from 'edge-then-node'
+import nodeThenEdge from 'node-then-edge'
+
+export const config = {
+  matcher: '/middleware',
+}
+
+export function middleware() {
+  return Response.json({
+    edgeThenNode,
+    nodeThenEdge,
+  })
+}
@@ -0,0 +1 @@
+module.exports = 'default'
@@ -0,0 +1 @@
+module.exports = 'edge'
@@ -0,0 +1 @@
+module.exports = 'main'
@@ -0,0 +1 @@
+module.exports = 'node'
@@ -0,0 +1,10 @@
+{
+  "main": "main.js",
+  "exports": {
+    ".": {
+      "node": "./node.js",
+      "edge-light": "./edge.js",
+      "default": "./default.js"
+    }
+  }
+}
@@ -0,0 +1,6 @@
+{
+  "dependencies": {
+    "edge-then-node": "file:edge-then-node",
+    "node-then-edge": "file:node-then-edge"
+  }
+}
@@ -0,0 +1,14 @@
+import edgeThenNode from 'edge-then-node'
+import nodeThenEdge from 'node-then-edge'
+
+export const config = {
+  runtime: 'edge',
+}
+
+export default function ApiEdge() {
+  return Response.json({
+    NEXT_RUNTIME: process.env.NEXT_RUNTIME,
+    edgeThenNode,
+    nodeThenEdge,
+  })
+}
@@ -0,0 +1,14 @@
+import edgeThenNode from 'edge-then-node'
+import nodeThenEdge from 'node-then-edge'
+
+export const config = {
+  runtime: 'nodejs',
+}
+
+export default function ApiNodeJs(req, res) {
+  res.status(200).json({
+    NEXT_RUNTIME: process.env.NEXT_RUNTIME,
+    edgeThenNode,
+    nodeThenEdge,
+  })
+}
@@ -0,0 +1,13 @@
+import edgeThenNode from 'edge-then-node'
+import nodeThenEdge from 'node-then-edge'
+
+export const config = {
+  runtime: 'experimental-edge',
+}
+
+export default function PageEdge() {
+  return JSON.stringify({
+    edgeThenNode,
+    nodeThenEdge,
+  })
+}
@@ -0,0 +1,13 @@
+import edgeThenNode from 'edge-then-node'
+import nodeThenEdge from 'node-then-edge'
+
+export const config = {
+  runtime: 'nodejs',
+}
+
+export default function PageNodeJs() {
+  return JSON.stringify({
+    edgeThenNode,
+    nodeThenEdge,
+  })
+}
@@ -0,0 +1,35 @@
+PlainIssue {
+    severity: Warning,
+    context: "[project]/packages/next/dist/compiled/edge-runtime/index.js",
+    category: "parse",
+    title: "error TP1003 require.resolve(???*0*, {\"paths\": [???*1*]}) is not statically analyse-able",
+    description: "- *0* arguments[1]\n  ⚠\u{fe0f}  function calls are not analysed yet\n- *1* ???*2*[\"dirname\"](a)\n  ⚠\u{fe0f}  unknown callee object\n- *2* ???(17)\n  ⚠\u{fe0f}  unknown callee",
+    detail: "",
+    documentation_link: "",
+    source: Some(
+        PlainIssueSource {
+            asset: PlainAsset {
+                ident: "[project]/packages/next/dist/compiled/edge-runtime/index.js",
+            },
+            start: SourcePos {
+                line: 0,
+                column: 6730,
+            },
+            end: SourcePos {
+                line: 0,
+                column: 6730,
+            },
+        },
+    ),
+    sub_issues: [],
+    processing_path: Some(
+        [
+            PlainIssueProcessingPathItem {
+                context: Some(
+                    "[project]/packages/next-swc/crates/next-dev-tests/tests/temp/next/import/conditions/input/app",
+                ),
+                description: "Next.js App Route /route-nodejs",
+            },
+        ],
+    ),
+}