Fix: handle empty bin strings

CHANGELOG.md

@@ -4,6 +4,10 @@ Please add one entry in this file for each change in Yarn's behavior. Use the sa
 
 ## Master
 
+- Fixes handling of empty string entries for bin in package.json
+
+  [#6515](https://github.com/yarnpkg/yarn/pull/6515) - [**Ryan Burrows**](https://github.com/rhburrows)
+
 - Adds support for basic auth for registries with paths, such as artifactory
 
   [#5322](https://github.com/yarnpkg/yarn/pull/5322) - [**Karolis Narkevicius**](https://twitter.com/KidkArolis)

__tests__/__snapshots__/normalize-manifest.js.snap

@@ -90,6 +90,12 @@ Array [
 ]
 `;
 
+exports[`empty bin string: empty bin string 1`] = `
+Array [
+  "foo: No license field",
+]
+`;
+
 exports[`engines array: engines array 1`] = `
 Array [
   "No license field",

__tests__/fixtures/normalize-manifest/empty bin string/actual.json

@@ -0,0 +1,4 @@
+{
+  "name": "foo",
+  "bin": ""
+}

__tests__/fixtures/normalize-manifest/empty bin string/expected.json

@@ -0,0 +1,5 @@
+{
+  "name": "foo",
+  "version": "",
+  "bin": ""
+}

src/util/normalize-manifest/fix.js

@@ -153,7 +153,7 @@ export default (async function(
   // if the `bin` field is as string then expand it to an object with a single property
   // based on the original `bin` field and `name field`
   // { name: "foo", bin: "cli.js" } -> { name: "foo", bin: { foo: "cli.js" } }
-  if (typeof info.name === 'string' && typeof info.bin === 'string') {
+  if (typeof info.name === 'string' && typeof info.bin === 'string' && info.bin.length > 0) {
     // Remove scoped package name for consistency with NPM's bin field fixing behaviour
     const name = info.name.replace(/^@[^\/]+\//, '');
     info.bin = {[name]: info.bin};