BREAKING CHANGE: do not allow "~'!()*" in package names

Author: chrisdickinson

index.js

@@ -68,6 +68,10 @@ var validate = module.exports = function(name) {
     warnings.push("name can no longer contain capital letters")
   }
 
+  if (/[~'!()*]/.test(name.split('/').slice(-1)[0])) {
+    warnings.push('name can no longer contain special characters ("~\'!()*")')
+  }
+
   if (encodeURIComponent(name) !== name) {
 
     // Maybe it's a scoped package name, like @user/package

test/index.js

@@ -12,12 +12,16 @@ test("validate-npm-package-name", function (t) {
   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: 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: true, validForOldPackages: true})
+  t.deepEqual(validate("@npm-zors/money!time.js"), {validForNewPackages: false, validForOldPackages: true, warnings: [
+    'name can no longer contain special characters ("~\'!()*")'
+  ]})
 
   // Invalid