feat: rootless workspace init provides suggestion

Author: Nathan Fritz

lib/commands/init.js

@@ -54,7 +54,12 @@ class Init extends BaseCommand {
     // reads package.json for the top-level folder first, by doing this we
     // ensure the command throw if no package.json is found before trying
     // to create a workspace package.json file or its folders
-    const pkg = await rpj(resolve(this.npm.localPrefix, 'package.json'))
+    const pkg = await rpj(resolve(this.npm.localPrefix, 'package.json')).catch((err) => {
+      if (err.code === 'ENOENT') {
+        log.warn('Missing package.json. Try with `--include-workspace-root`.')
+      }
+      throw err
+    })
 
     // these are workspaces that are being created, so we cant use
     // this.setWorkspaces()

tap-snapshots/test/lib/commands/init.js.test.cjs

@@ -22,29 +22,3 @@ exports[`test/lib/commands/init.js TAP workspaces no args -- yes > should print
 
 added 1 package in {TIME}
 `
-
-exports[`test/lib/commands/init.js TAP workspaces no args -- yes > should reify tree on init ws complete 1`] = `
-{
-  "name": "top-level",
-  "lockfileVersion": 3,
-  "requires": true,
-  "packages": {
-    "": {
-      "name": "top-level",
-      "workspaces": [
-        "a"
-      ]
-    },
-    "a": {
-      "version": "1.0.0",
-      "license": "ISC",
-      "devDependencies": {}
-    },
-    "node_modules/a": {
-      "resolved": "a",
-      "link": true
-    }
-  }
-}
-
-`

test/lib/commands/init.js

@@ -335,7 +335,7 @@ t.test('workspaces', async t => {
   })
 
   await t.test('missing top-level package.json when settting workspace', async t => {
-    const { npm } = await mockNpm(t, {
+    const { npm, logs } = await mockNpm(t, {
       config: { workspace: 'a' },
     })
 
@@ -344,6 +344,25 @@ t.test('workspaces', async t => {
       { code: 'ENOENT' },
       'should exit with missing package.json file error'
     )
+
+    t.equal(logs.warn[0][0], 'Missing package.json. Try with `--include-workspace-root`.')
+  })
+
+  await t.test('bad package.json when settting workspace', async t => {
+    const { npm, logs } = await mockNpm(t, {
+      prefixDir: {
+        'package.json': '{{{{',
+      },
+      config: { workspace: 'a' },
+    })
+
+    await t.rejects(
+      npm.exec('init', []),
+      { code: 'EJSONPARSE' },
+      'should exit with parse file error'
+    )
+
+    t.strictSame(logs.warn, [])
   })
 
   await t.test('using args - no package.json', async t => {