fix: print a warning if a package.json uses workspaces field (#5547)

close #5363
close #5413

.changeset/twelve-bikes-serve.md

@@ -0,0 +1,6 @@
+---
+"@pnpm/config": patch
+"pnpm": patch
+---
+
+Print a warning if a package.json has a workspaces field but there is no pnpm-workspace.yaml file [#5363](https://github.com/pnpm/pnpm/issues/5363).

packages/config/package.json

@@ -54,6 +54,7 @@
   "devDependencies": {
     "@pnpm/config": "workspace:*",
     "@pnpm/prepare": "workspace:*",
+    "@pnpm/test-fixtures": "workspace:*",
     "@types/is-windows": "^1.0.0",
     "@types/ramda": "0.28.15",
     "@types/which": "^2.0.1",

packages/config/src/index.ts

@@ -524,7 +524,10 @@ export async function getConfig (
   if (!pnpmConfig.ignorePnpmfile) {
     pnpmConfig.hooks = requireHooks(pnpmConfig.lockfileDir ?? pnpmConfig.dir, pnpmConfig)
   }
-  pnpmConfig.rootProjectManifest = await safeReadProjectManifestOnly(pnpmConfig.lockfileDir ?? pnpmConfig.dir) ?? undefined
+  pnpmConfig.rootProjectManifest = await safeReadProjectManifestOnly(pnpmConfig.lockfileDir ?? pnpmConfig.workspaceDir ?? pnpmConfig.dir) ?? undefined
+  if (pnpmConfig.rootProjectManifest?.['workspaces']?.length && !pnpmConfig.workspaceDir) {
+    warnings.push('The "workspaces" field in package.json is not supported by pnpm. Create a "pnpm-workspace.yaml" file instead.')
+  }
 
   pnpmConfig.failedToLoadBuiltInConfig = failedToLoadBuiltInConfig
 

packages/config/test/fixtures/pkg-using-workspaces/package.json

@@ -0,0 +1,3 @@
+{
+  "workspaces": ["packages/*"]
+}

packages/config/test/index.ts

@@ -7,6 +7,7 @@ import { getConfig } from '@pnpm/config'
 import { PnpmError } from '@pnpm/error'
 import loadNpmConf from '@pnpm/npm-conf'
 import { prepare, prepareEmpty } from '@pnpm/prepare'
+import { fixtures } from '@pnpm/test-fixtures'
 
 import symlinkDir from 'symlink-dir'
 
@@ -26,6 +27,7 @@ const env = {
   PNPM_HOME: __dirname,
   [PATH]: __dirname,
 }
+const f = fixtures(__dirname)
 
 test('getConfig()', async () => {
   const { config } = await getConfig({
@@ -941,3 +943,31 @@ test('getConfig() returns failedToLoadBuiltInConfig', async () => {
 
   expect(config.failedToLoadBuiltInConfig).toBeDefined()
 })
+
+test('return a warning if a package.json has workspaces field but there is no pnpm-workspaces.yaml file', async () => {
+  const prefix = f.find('pkg-using-workspaces')
+  const { warnings } = await getConfig({
+    cliOptions: { dir: prefix },
+    packageManager: {
+      name: 'pnpm',
+      version: '1.0.0',
+    },
+  })
+
+  expect(warnings).toStrictEqual([
+    'The "workspaces" field in package.json is not supported by pnpm. Create a "pnpm-workspace.yaml" file instead.',
+  ])
+})
+
+test('do not return a warning if a package.json has workspaces field and there is a pnpm-workspace.yaml file', async () => {
+  const prefix = f.find('pkg-using-workspaces')
+  const { warnings } = await getConfig({
+    cliOptions: { dir: prefix },
+    workspaceDir: prefix,
+    packageManager: {
+      name: 'pnpm',
+      version: '1.0.0',
+    },
+  })
+  expect(warnings).toStrictEqual([])
+})

packages/config/tsconfig.json

@@ -12,6 +12,9 @@
     {
       "path": "../../privatePackages/prepare"
     },
+    {
+      "path": "../../privatePackages/test-fixtures"
+    },
     {
       "path": "../constants"
     },