Allow dots in bin names (#7811)

* Fixed bin regex

* Removed unsupported by Windows colon from bin regex

* Updated normalize-manifest tests

* Update fix.js

* Update actual.json

* Update normalize-manifest.js.snap

Co-authored-by: Maël Nison <[email protected]>

Author: valerybugakov

__tests__/__snapshots__/normalize-manifest.js.snap

@@ -62,9 +62,12 @@ Array [
 
 exports[`dangerous bin name: dangerous bin name 1`] = `
 Array [
+  "foo: Invalid bin entry for \\".\\" (in \\"foo\\").",
+  "foo: Invalid bin entry for \\"..\\" (in \\"foo\\").",
   "foo: Invalid bin entry for \\"/tmp/foo\\" (in \\"foo\\").",
   "foo: Invalid bin entry for \\"../tmp/foo\\" (in \\"foo\\").",
   "foo: Invalid bin entry for \\"tmp/../../foo\\" (in \\"foo\\").",
+  "foo: Invalid bin entry for \\"build:cli\\" (in \\"foo\\").",
   "foo: No license field",
 ]
 `;

__tests__/fixtures/normalize-manifest/dangerous bin name/actual.json

@@ -1,9 +1,13 @@
 {
-    "name": "foo",
-    "version": "",
-    "bin": {
-        "/tmp/foo": "main.js",
-        "../tmp/foo": "main.js",
-        "tmp/../../foo": "main.js"
-    }
+  "name": "foo",
+  "version": "",
+  "bin": {
+    ".": "main.js",
+    "..": "main.js",
+    "/tmp/foo": "main.js",
+    "../tmp/foo": "main.js",
+    "tmp/../../foo": "main.js",
+    "build:cli": "main.js",
+    "build.cli": "main.js"
+  }
 }

__tests__/fixtures/normalize-manifest/dangerous bin name/expected.json

@@ -1,5 +1,7 @@
 {
-    "name": "foo",
-    "version": "",
-    "bin": {}
+  "name": "foo",
+  "version": "",
+  "bin": {
+    "build.cli": "main.js"
+  }
 }

src/util/normalize-manifest/fix.js

@@ -12,7 +12,7 @@ const semver = require('semver');
 const path = require('path');
 const url = require('url');
 
-const VALID_BIN_KEYS = /^[a-z0-9_-]+$/i;
+const VALID_BIN_KEYS = /^(?!\.{0,2}$)[a-z0-9._-]+$/i;
 
 const LICENSE_RENAMES: {[key: string]: ?string} = {
   'MIT/X11': 'MIT',