deps: validate-npm-package-name@4.0.0

 * use latest builtins

3 files changed, 47 insertions, 128 deletions

diff --git a/node_modules/validate-npm-package-name/index.js b/node_modules/validate-npm-package-name/lib/index.js
index eb43fa25e..e7e612bfb 100644
--- a/node_modules/validate-npm-package-name/index.js
+++ b/node_modules/validate-npm-package-name/lib/index.js
@@ -4,10 +4,10 @@ var scopedPackagePattern = new RegExp('^(?:@([^/]+?)[/])?([^/]+?)$')
 var builtins = require('builtins')
 var blacklist = [
   'node_modules',
-  'favicon.ico'
+  'favicon.ico',
 ]
 
-var validate = module.exports = function (name) {
+function validate (name) {
   var warnings = []
   var errors = []
 
@@ -52,14 +52,12 @@ var validate = module.exports = function (name) {
   // Generate warnings for stuff that used to be allowed
 
   // core module names like http, events, util, etc
-  builtins.forEach(function (builtin) {
+  builtins({ version: '*' }).forEach(function (builtin) {
     if (name.toLowerCase() === builtin) {
       warnings.push(builtin + ' is a core module name')
     }
   })
 
-  // really-long-package-names-------------------------------such--length-----many---wow
-  // the thisisareallyreallylongpackagenameitshouldpublishdowenowhavealimittothelengthofpackagenames-poch.
   if (name.length > 214) {
     warnings.push('name can no longer contain more than 214 characters')
   }
@@ -90,16 +88,20 @@ var validate = module.exports = function (name) {
   return done(warnings, errors)
 }
 
-validate.scopedPackagePattern = scopedPackagePattern
-
 var done = function (warnings, errors) {
   var result = {
     validForNewPackages: errors.length === 0 && warnings.length === 0,
     validForOldPackages: errors.length === 0,
     warnings: warnings,
-    errors: errors
+    errors: errors,
+  }
+  if (!result.warnings.length) {
+    delete result.warnings
+  }
+  if (!result.errors.length) {
+    delete result.errors
   }
-  if (!result.warnings.length) delete result.warnings
-  if (!result.errors.length) delete result.errors
   return result
 }
+
+module.exports = validate
diff --git a/node_modules/validate-npm-package-name/package.json b/node_modules/validate-npm-package-name/package.json
index b72e9ef03..fa9a6920d 100644
--- a/node_modules/validate-npm-package-name/package.json
+++ b/node_modules/validate-npm-package-name/package.json
@@ -1,27 +1,37 @@
 {
   "name": "validate-npm-package-name",
-  "version": "3.0.0",
+  "version": "4.0.0",
   "description": "Give me a string and I'll tell you if it's a valid npm package name",
-  "main": "index.js",
+  "main": "lib/",
   "directories": {
     "test": "test"
   },
   "dependencies": {
-    "builtins": "^1.0.3"
+    "builtins": "^5.0.0"
   },
   "devDependencies": {
-    "standard": "^8.6.0",
-    "tap": "^10.0.0"
+    "@npmcli/eslint-config": "^3.0.1",
+    "@npmcli/template-oss": "3.2.1",
+    "tap": "^16.0.1"
   },
   "scripts": {
     "cov:test": "TAP_FLAGS='--cov' npm run test:code",
     "test:code": "tap ${TAP_FLAGS:-'--'} test/*.js",
     "test:style": "standard",
-    "test": "npm run test:code && npm run test:style"
+    "test": "tap",
+    "lint": "eslint \"**/*.js\"",
+    "postlint": "template-oss-check",
+    "template-oss-apply": "template-oss-apply --force",
+    "lintfix": "npm run lint -- --fix",
+    "preversion": "npm test",
+    "postversion": "npm publish",
+    "prepublishOnly": "git push origin --follow-tags",
+    "snap": "tap",
+    "posttest": "npm run lint"
   },
   "repository": {
     "type": "git",
-    "url": "https://github.com/npm/validate-npm-package-name"
+    "url": "https://github.com/npm/validate-npm-package-name.git"
   },
   "keywords": [
     "npm",
@@ -29,10 +39,26 @@
     "names",
     "validation"
   ],
-  "author": "zeke",
+  "author": "GitHub Inc.",
   "license": "ISC",
   "bugs": {
     "url": "https://github.com/npm/validate-npm-package-name/issues"
   },
-  "homepage": "https://github.com/npm/validate-npm-package-name"
+  "homepage": "https://github.com/npm/validate-npm-package-name",
+  "files": [
+    "bin/",
+    "lib/"
+  ],
+  "engines": {
+    "node": "^12.13.0 || ^14.15.0 || >=16.0.0"
+  },
+  "templateOSS": {
+    "//@npmcli/template-oss": "This file is partially managed by @npmcli/template-oss. Edits may be overwritten.",
+    "version": "3.2.1"
+  },
+  "tap": {
+    "statements": 88,
+    "branches": 92,
+    "lines": 88
+  }
 }
diff --git a/node_modules/validate-npm-package-name/test/index.js b/node_modules/validate-npm-package-name/test/index.js
deleted file mode 100644
index 1f0bc6123..000000000
--- a/node_modules/validate-npm-package-name/test/index.js
+++ /dev/null
@@ -1,109 +0,0 @@
-'use strict'
-
-var validate = require('..')
-var test = require('tap').test
-
-test('validate-npm-package-name', function (t) {
-  // Traditional
-
-  t.deepEqual(validate('some-package'), {validForNewPackages: true, validForOldPackages: true})
-  t.deepEqual(validate('example.com'), {validForNewPackages: true, validForOldPackages: true})
-  t.deepEqual(validate('under_score'), {validForNewPackages: true, validForOldPackages: true})
-  t.deepEqual(validate('period.js'), {validForNewPackages: true, validForOldPackages: true})
-  t.deepEqual(validate('123numeric'), {validForNewPackages: true, validForOldPackages: true})
-  t.deepEqual(validate('crazy!'), {
-    validForNewPackages: false,
-    validForOldPackages: true,
-    warnings: ['name can no longer contain special characters ("~\'!()*")']
-  })
-
-  // Scoped (npm 2+)
-
-  t.deepEqual(validate('@npm/thingy'), {validForNewPackages: true, validForOldPackages: true})
-  t.deepEqual(validate('@npm-zors/money!time.js'), {
-    validForNewPackages: false,
-    validForOldPackages: true,
-    warnings: ['name can no longer contain special characters ("~\'!()*")']
-  })
-
-  // Invalid
-
-  t.deepEqual(validate(''), {
-    validForNewPackages: false,
-    validForOldPackages: false,
-    errors: ['name length must be greater than zero']})
-
-  t.deepEqual(validate(''), {
-    validForNewPackages: false,
-    validForOldPackages: false,
-    errors: ['name length must be greater than zero']})
-
-  t.deepEqual(validate('.start-with-period'), {
-    validForNewPackages: false,
-    validForOldPackages: false,
-    errors: ['name cannot start with a period']})
-
-  t.deepEqual(validate('_start-with-underscore'), {
-    validForNewPackages: false,
-    validForOldPackages: false,
-    errors: ['name cannot start with an underscore']})
-
-  t.deepEqual(validate('contain:colons'), {
-    validForNewPackages: false,
-    validForOldPackages: false,
-    errors: ['name can only contain URL-friendly characters']})
-
-  t.deepEqual(validate(' leading-space'), {
-    validForNewPackages: false,
-    validForOldPackages: false,
-    errors: ['name cannot contain leading or trailing spaces', 'name can only contain URL-friendly characters']})
-
-  t.deepEqual(validate('trailing-space '), {
-    validForNewPackages: false,
-    validForOldPackages: false,
-    errors: ['name cannot contain leading or trailing spaces', 'name can only contain URL-friendly characters']})
-
-  t.deepEqual(validate('s/l/a/s/h/e/s'), {
-    validForNewPackages: false,
-    validForOldPackages: false,
-    errors: ['name can only contain URL-friendly characters']})
-
-  t.deepEqual(validate('node_modules'), {
-    validForNewPackages: false,
-    validForOldPackages: false,
-    errors: ['node_modules is a blacklisted name']})
-
-  t.deepEqual(validate('favicon.ico'), {
-    validForNewPackages: false,
-    validForOldPackages: false,
-    errors: ['favicon.ico is a blacklisted name']})
-
-  // Node/IO Core
-
-  t.deepEqual(validate('http'), {
-    validForNewPackages: false,
-    validForOldPackages: true,
-    warnings: ['http is a core module name']})
-
-  // Long Package Names
-
-  t.deepEqual(validate('ifyouwanttogetthesumoftwonumberswherethosetwonumbersarechosenbyfindingthelargestoftwooutofthreenumbersandsquaringthemwhichismultiplyingthembyitselfthenyoushouldinputthreenumbersintothisfunctionanditwilldothatforyou-'), {
-    validForNewPackages: false,
-    validForOldPackages: true,
-    warnings: ['name can no longer contain more than 214 characters']
-  })
-
-  t.deepEqual(validate('ifyouwanttogetthesumoftwonumberswherethosetwonumbersarechosenbyfindingthelargestoftwooutofthreenumbersandsquaringthemwhichismultiplyingthembyitselfthenyoushouldinputthreenumbersintothisfunctionanditwilldothatforyou'), {
-    validForNewPackages: true,
-    validForOldPackages: true
-  })
-
-  // Legacy Mixed-Case
-
-  t.deepEqual(validate('CAPITAL-LETTERS'), {
-    validForNewPackages: false,
-    validForOldPackages: true,
-    warnings: ['name can no longer contain capital letters']})
-
-  t.end()
-})