Merge branch 'canary' into styfle/next-1591-next-dev-on-windows-error

.cargo/config.toml

@@ -35,8 +35,4 @@ rustflags = [
   "-Zshare-generics=y",
   "-Csymbol-mangling-version=v0",
   "-Aclippy::too_many_arguments",
-  # Clippy's needless mut lint is buggy: https://github.com/rust-lang/rust-clippy/issues/11299
-  "-Aclippy::needless_pass_by_ref_mut",
-  # Clippy's partial_eq lint is buggy: https://github.com/rust-lang/rust-clippy/issues/11178
-  "-Aclippy::non_canonical_partial_ord_impl",
 ]

.eslintrc.json

@@ -213,6 +213,18 @@
         ],
         "eslint-plugin/require-meta-docs-url": "error"
       }
+    },
+    {
+      "files": ["packages/**/*.tsx", "packages/**/*.ts"],
+      "rules": {
+        "@typescript-eslint/consistent-type-imports": [
+          "error",
+          {
+            "disallowTypeAnnotations": false
+          }
+        ],
+        "@typescript-eslint/no-import-type-side-effects": "error"
+      }
     }
   ],
   "rules": {

.github/actions/next-stats-action/package.json

@@ -20,7 +20,7 @@
   },
   "engines": {
     "node": ">=16.14.0",
-    "pnpm": "8.7.1"
+    "pnpm": "8.9.0"
   },
-  "packageManager": "pnpm@8.7.1"
+  "packageManager": "pnpm@8.9.0"
 }

.github/actions/next-stats-action/src/prepare/repo-setup.js

@@ -1,5 +1,7 @@
 const path = require('path')
-const fs = require('fs-extra')
+const fse = require('fs-extra')
+const fs = require('fs')
+const fsp = require('fs/promises')
 const exec = require('../util/exec')
 const { remove } = require('fs-extra')
 const logger = require('../util/logger')
@@ -54,11 +56,23 @@ module.exports = (actionInfo) => {
     },
     async linkPackages({ repoDir, nextSwcVersion }) {
       const pkgPaths = new Map()
+
+      /**
+       * @typedef {Object} PkgData
+       * @property {string} pkgDataPath Where the package.json file is located
+       * @property {string} pkg The folder name of the package
+       * @property {string} pkgPath The path to the package folder
+       * @property {any} pkgData The content of package.json
+       * @property {string} packedPkgPath The npm pack output .tgz file path
+       */
+
+      /** @type {Map<string, PkgData>} */
       const pkgDatas = new Map()
+
       let pkgs
 
       try {
-        pkgs = await fs.readdir(path.join(repoDir, 'packages'))
+        pkgs = await fsp.readdir(path.join(repoDir, 'packages'))
       } catch (err) {
         if (err.code === 'ENOENT') {
           require('console').log('no packages to link')
@@ -67,65 +81,81 @@ module.exports = (actionInfo) => {
         throw err
       }
 
-      for (const pkg of pkgs) {
-        const pkgPath = path.join(repoDir, 'packages', pkg)
-        const packedPkgPath = path.join(pkgPath, `${pkg}-packed.tgz`)
-
-        const pkgDataPath = path.join(pkgPath, 'package.json')
-        if (!fs.existsSync(pkgDataPath)) {
-          require('console').log(`Skipping ${pkgDataPath}`)
-          continue
-        }
-        const pkgData = require(pkgDataPath)
-        const { name } = pkgData
-
-        pkgDatas.set(name, {
-          pkgDataPath,
-          pkg,
-          pkgPath,
-          pkgData,
-          packedPkgPath,
+      await Promise.all(
+        pkgs.map(async (pkg) => {
+          const pkgPath = path.join(repoDir, 'packages', pkg)
+          const packedPkgPath = path.join(pkgPath, `${pkg}-packed.tgz`)
+
+          const pkgDataPath = path.join(pkgPath, 'package.json')
+          if (fs.existsSync(pkgDataPath)) {
+            const pkgData = JSON.parse(await fsp.readFile(pkgDataPath))
+            const { name } = pkgData
+
+            pkgDatas.set(name, {
+              pkgDataPath,
+              pkg,
+              pkgPath,
+              pkgData,
+              packedPkgPath,
+            })
+            pkgPaths.set(name, packedPkgPath)
+          } else {
+            require('console').log(`Skipping ${pkgDataPath}`)
+          }
         })
-        pkgPaths.set(name, packedPkgPath)
-      }
-
-      for (const pkg of pkgDatas.keys()) {
-        const { pkgDataPath, pkgData } = pkgDatas.get(pkg)
+      )
 
-        for (const pkg of pkgDatas.keys()) {
-          const { packedPkgPath } = pkgDatas.get(pkg)
+      for (const [
+        pkg,
+        { pkgDataPath, pkgData, pkgPath },
+      ] of pkgDatas.entries()) {
+        // update the current package dependencies to point to packed tgz path
+        for (const [pkg, { packedPkgPath }] of pkgDatas.entries()) {
           if (!pkgData.dependencies || !pkgData.dependencies[pkg]) continue
           pkgData.dependencies[pkg] = packedPkgPath
         }
 
         // make sure native binaries are included in local linking
         if (pkg === '@next/swc') {
-          if (!pkgData.files) {
-            pkgData.files = []
-          }
+          pkgData.files ||= []
+
           pkgData.files.push('native')
-        }
 
-        if (pkg === 'next') {
+          try {
+            const swcBinariesDirContents = await fsp.readdir(
+              path.join(pkgPath, 'native')
+            )
+            require('console').log(
+              'using swc binaries: ',
+              swcBinariesDirContents.join(', ')
+            )
+          } catch (err) {
+            if (err.code === 'ENOENT') {
+              require('console').log('swc binaries dir is missing!')
+            }
+            throw err
+          }
+        } else if (pkg === 'next') {
+          const nextSwcPkg = pkgDatas.get('@next/swc')
+
           console.log('using swc dep', {
             nextSwcVersion,
-            nextSwcPkg: pkgDatas.get('@next/swc'),
+            nextSwcPkg,
           })
           if (nextSwcVersion) {
             Object.assign(pkgData.dependencies, {
               '@next/swc-linux-x64-gnu': nextSwcVersion,
             })
           } else {
-            if (pkgDatas.get('@next/swc')) {
-              pkgData.dependencies['@next/swc'] =
-                pkgDatas.get('@next/swc').packedPkgPath
+            if (nextSwcPkg) {
+              pkgData.dependencies['@next/swc'] = nextSwcPkg.packedPkgPath
             } else {
               pkgData.files.push('native')
             }
           }
         }
 
-        await fs.writeFile(
+        await fsp.writeFile(
           pkgDataPath,
           JSON.stringify(pkgData, null, 2),
           'utf8'
@@ -135,41 +165,47 @@ module.exports = (actionInfo) => {
       // wait to pack packages until after dependency paths have been updated
       // to the correct versions
       await Promise.all(
-        Array.from(pkgDatas.keys()).map(async (pkgName) => {
-          const { pkgPath, packedPkgPath } = pkgDatas.get(pkgName)
-
-          let cleanup = null
-
-          if (pkgName === '@next/swc') {
-            // next-swc uses a gitignore to prevent the committing of native builds but it doesn't
-            // use files in package.json because it publishes to individual packages based on architecture.
-            // When we used yarn to pack these packages the gitignore was ignored so the native builds were packed
-            // however npm does respect gitignore when packing so we need to remove it in this specific case
-            // to ensure the native builds are packed for use in gh actions and related scripts
-            await fs.rename(
-              path.join(pkgPath, 'native/.gitignore'),
-              path.join(pkgPath, 'disabled-native-gitignore')
-            )
-            cleanup = async () => {
-              await fs.rename(
-                path.join(pkgPath, 'disabled-native-gitignore'),
-                path.join(pkgPath, 'native/.gitignore')
+        Array.from(pkgDatas.entries()).map(
+          async ([pkgName, { pkgPath, packedPkgPath }]) => {
+            /** @type {null | () => Promise<void>} */
+            let cleanup = null
+
+            if (pkgName === '@next/swc') {
+              // next-swc uses a gitignore to prevent the committing of native builds but it doesn't
+              // use files in package.json because it publishes to individual packages based on architecture.
+              // When we used yarn to pack these packages the gitignore was ignored so the native builds were packed
+              // however npm does respect gitignore when packing so we need to remove it in this specific case
+              // to ensure the native builds are packed for use in gh actions and related scripts
+
+              const nativeGitignorePath = path.join(
+                pkgPath,
+                'native/.gitignore'
+              )
+              const renamedGitignorePath = path.join(
+                pkgPath,
+                'disabled-native-gitignore'
               )
+
+              await fsp.rename(nativeGitignorePath, renamedGitignorePath)
+              cleanup = async () => {
+                await fsp.rename(renamedGitignorePath, nativeGitignorePath)
+              }
             }
-          }
 
-          const { stdout } = await execa('npm', ['pack'], {
-            cwd: pkgPath,
-            env: {
-              ...process.env,
-              COREPACK_ENABLE_STRICT: '0',
-            },
-          })
-          await fs.rename(path.resolve(pkgPath, stdout.trim()), packedPkgPath)
-          if (cleanup) {
-            await cleanup()
+            const { stdout } = await execa('pnpm', ['pack'], {
+              cwd: pkgPath,
+              env: {
+                ...process.env,
+                COREPACK_ENABLE_STRICT: '0',
+              },
+            })
+
+            return Promise.all([
+              fsp.rename(path.resolve(pkgPath, stdout.trim()), packedPkgPath),
+              cleanup?.(),
+            ])
           }
-        })
+        )
       )
       return pkgPaths
     },

.github/workflows/build_and_deploy.yml

@@ -254,6 +254,7 @@ jobs:
         with:
           node-version: ${{ env.NODE_LTS_VERSION }}
           check-latest: true
+      - run: corepack enable
 
       - name: Install Rust
         uses: ./.github/actions/setup-rust

.github/workflows/build_and_test.yml

@@ -58,6 +58,7 @@ jobs:
       - uses: actions/setup-node@v3
         with:
           node-version: 18
+      - run: corepack enable
       - name: 'Run link checker'
         run: node ./.github/actions/validate-docs-links/lib
         env:

.github/workflows/code_freeze.yml

@@ -32,6 +32,7 @@ jobs:
         with:
           node-version: 18
           check-latest: true
+      - run: corepack enable
 
       - run: git clone https://github.com/vercel/next.js.git --depth=1 .
 

.github/workflows/pull_request_approved.yml

@@ -14,6 +14,7 @@ jobs:
       - uses: actions/setup-node@v3
         with:
           node-version: 18
+      - run: corepack enable
       - name: 'Send notification to Slack'
         run: node ./.github/actions/pr-approved-open/index.mjs
         env: