feat: support {packageManager: 'npm@...'} and {packageManager:{name: '...'}}

Author: fisker

index.js

@@ -141,12 +141,15 @@ const hasYarnOrPnpmFiles = () => {
  */
 function shouldSortDependenciesLikeNpm(packageJson) {
   const packageManager = packageJson.packageManager
-  if (
-    packageJson.pnpm ||
-    (typeof packageManager === 'string' &&
-      (packageManager.startsWith('yarn@') ||
-        packageManager.startsWith('pnpm@')))
-  ) {
+  if (packageManager) {
+    if (typeof packageManager === 'string') {
+      return packageManager.startsWith('npm@')
+    }
+
+    return packageManager.name === 'npm'
+  }
+
+  if (packageJson.pnpm) {
     return false
   }
 

tests/dependency-sorting.js

@@ -54,6 +54,20 @@ test('npm', (t) => {
     npmSortedResult,
   )
 
+  // packageManager
+  t.deepEqual(
+    getDependencyOrders({ packageManager: 'npm@1.0.0' }, () => {
+      t.fail()
+    }),
+    npmSortedResult,
+  )
+  t.deepEqual(
+    getDependencyOrders({ packageManager: { name: 'npm' } }, () => {
+      t.fail()
+    }),
+    npmSortedResult,
+  )
+
   // Should not call `fs.existsSync()`
   getDependencyOrders({ dependencies: { 'one-dependency': '1.0.0' } }, () => {
     t.fail()
@@ -85,6 +99,12 @@ test('pnpm', (t) => {
     }),
     nonNpmSortedResult,
   )
+  t.deepEqual(
+    getDependencyOrders({ packageManager: { name: 'pnpm' } }, () => {
+      t.fail()
+    }),
+    nonNpmSortedResult,
+  )
 
   // pnpm file exists
   let fsExistsSyncCalled
@@ -106,6 +126,12 @@ test('yarn', (t) => {
     }),
     nonNpmSortedResult,
   )
+  t.deepEqual(
+    getDependencyOrders({ packageManager: { name: 'yarn' } }, () => {
+      t.fail()
+    }),
+    nonNpmSortedResult,
+  )
 
   // yarn file exists
   let fsExistsSyncCalled