feat!: remove default license for npm init

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

@@ -76,15 +76,15 @@ exports[`test/lib/commands/config.js TAP config list --json > output matches sna
   "init-author-email": "",
   "init-author-name": "",
   "init-author-url": "",
-  "init-license": "ISC",
+  "init-license": "",
   "init-module": "{CWD}/home/.npm-init.js",
   "init-type": "commonjs",
   "init-version": "1.0.0",
   "init-private": false,
   "init.author.email": "",
   "init.author.name": "",
   "init.author.url": "",
-  "init.license": "ISC",
+  "init.license": "",
   "init.module": "{CWD}/home/.npm-init.js",
   "init.version": "1.0.0",
   "install-links": false,
@@ -255,15 +255,15 @@ include-workspace-root = false
 init-author-email = ""
 init-author-name = ""
 init-author-url = ""
-init-license = "ISC"
+init-license = ""
 init-module = "{CWD}/home/.npm-init.js"
 init-private = false
 init-type = "commonjs"
 init-version = "1.0.0"
 init.author.email = ""
 init.author.name = ""
 init.author.url = ""
-init.license = "ISC"
+init.license = ""
 init.module = "{CWD}/home/.npm-init.js"
 init.version = "1.0.0"
 install-links = false

tap-snapshots/test/lib/docs.js.test.cjs

@@ -888,10 +888,11 @@ homepage.
 
 #### \`init-license\`
 
-* Default: "ISC"
+* Default: ""
 * Type: String
 
-The value \`npm init\` should use by default for the package license.
+The value \`npm init\` should use by default for the package license. If not
+set, the license field will be omitted from new packages.
 
 
 
@@ -2145,7 +2146,7 @@ Alias for \`--init-author-url\`
 
 #### \`init.license\`
 
-* Default: "ISC"
+* Default: ""
 * Type: String
 * DEPRECATED: Use \`--init-license\` instead.
 

test/lib/commands/init.js

@@ -43,7 +43,7 @@
 
   const pkg = require(resolve(prefix, 'package.json'))
   t.equal(pkg.version, '1.0.0')
-  t.equal(pkg.license, 'ISC')
+  t.equal(pkg.license, undefined, 'license is omitted by default')
 })
 
 t.test('classic interactive npm init', async t => {
@@ -273,7 +273,7 @@
     const pkg = require(resolve(prefix, 'a/package.json'))
     t.equal(pkg.name, 'a')
     t.equal(pkg.version, '1.0.0')
-    t.equal(pkg.license, 'ISC')
+    t.equal(pkg.license, undefined, 'license is omitted by default')
 
     t.matchSnapshot(joinedOutput(), 'should print helper info')
 
@@ -306,7 +306,7 @@
     const pkg = require(resolve(prefix, 'packages/a/package.json'))
     t.equal(pkg.name, 'a')
     t.equal(pkg.version, '2.0.0')
-    t.equal(pkg.license, 'ISC')
+    t.equal(pkg.license, undefined, 'license is omitted by default')
   })
 
   await t.test('fail parsing top-level package.json to set workspace', async t => {
@@ -422,12 +422,12 @@
 
     const pkg = require(resolve(npm.localPrefix, 'package.json'))
     t.equal(pkg.version, '1.0.0')
-    t.equal(pkg.license, 'ISC')
+    t.equal(pkg.license, undefined, 'license is omitted by default')
     t.strictSame(pkg.workspaces, ['packages/a'])
 
     const ws = require(resolve(npm.localPrefix, 'packages/a/package.json'))
     t.equal(ws.version, '1.0.0')
-    t.equal(ws.license, 'ISC')
+    t.equal(ws.license, undefined, 'license is omitted by default')
   })
   t.test('init pkg - installed workspace package', async t => {
     const { npm } = await mockNpm(t, {

workspaces/config/lib/definitions/definitions.js

@@ -983,11 +983,12 @@ const definitions = {
     `,
   }),
   'init-license': new Definition('init-license', {
-    default: 'ISC',
+    default: '',
     hint: '<license>',
     type: String,
     description: `
       The value \`npm init\` should use by default for the package license.
+      If not set, the license field will be omitted from new packages.
     `,
   }),
   'init-module': new Definition('init-module', {
@@ -1058,7 +1059,7 @@ const definitions = {
     `,
   }),
   'init.license': new Definition('init.license', {
-    default: 'ISC',
+    default: '',
     type: String,
     deprecated: `
       Use \`--init-license\` instead.