feat: remove bin/ directory (#948)

lib/cli.js

@@ -1,6 +1,7 @@
+#!/usr/bin/env node
 const yargs = require("yargs");
 
-module.exports = function cli() {
+function cli() {
   // eslint-disable-next-line no-unused-expressions
   yargs
     .usage("$0 <command>")
@@ -9,4 +10,8 @@ module.exports = function cli() {
     .strict()
     .alias("help", "h")
     .alias("version", "v").argv;
-};
+}
+
+cli();
+
+module.exports = cli;

package.json

@@ -12,10 +12,9 @@
     "tool"
   ],
   "main": "lib/index.js",
-  "bin": "bin/cli.js",
+  "bin": "lib/cli.js",
   "types": "types/ybiq.d.ts",
   "files": [
-    "bin",
     "lib",
     "types",
     ".editorconfig",

test/__snapshots__/help.test.js.snap

@@ -33,7 +33,7 @@ Options:
 `;
 
 exports[`with arguments [] 1`] = `
-"Command failed: bin/cli.js
+"Command failed: lib/cli.js
 cli.js <command>
 
 Commands:
@@ -52,7 +52,7 @@ Not enough non-option arguments: got 0, need at least 1
 `;
 
 exports[`with arguments []: stderr 1`] = `
-[Error: Command failed: bin/cli.js
+[Error: Command failed: lib/cli.js
 cli.js <command>
 
 Commands:
@@ -71,7 +71,7 @@ Not enough non-option arguments: got 0, need at least 1
 `;
 
 exports[`with arguments [unknown, xyz] 1`] = `
-"Command failed: bin/cli.js unknown xyz
+"Command failed: lib/cli.js unknown xyz
 cli.js <command>
 
 Commands:
@@ -90,7 +90,7 @@ Unknown arguments: unknown, xyz
 `;
 
 exports[`with arguments [unknown, xyz]: stderr 1`] = `
-[Error: Command failed: bin/cli.js unknown xyz
+[Error: Command failed: lib/cli.js unknown xyz
 cli.js <command>
 
 Commands:
@@ -109,7 +109,7 @@ Unknown arguments: unknown, xyz
 `;
 
 exports[`with arguments [unknown] 1`] = `
-"Command failed: bin/cli.js unknown
+"Command failed: lib/cli.js unknown
 cli.js <command>
 
 Commands:
@@ -128,7 +128,7 @@ Unknown argument: unknown
 `;
 
 exports[`with arguments [unknown]: stderr 1`] = `
-[Error: Command failed: bin/cli.js unknown
+[Error: Command failed: lib/cli.js unknown
 cli.js <command>
 
 Commands:

test/help.test.js

@@ -1,17 +1,18 @@
+const pkg = require("../package.json");
 const exec = require("./helpers/exec");
 
 [[], ["unknown"], ["unknown", "xyz"]].forEach((args) => {
   test(`with arguments [${args.join(", ")}]`, async () => {
-    await expect(exec("bin/cli.js", ...args)).rejects.toThrowErrorMatchingSnapshot();
-    await expect(exec("bin/cli.js", ...args)).rejects.toHaveProperty("code", 1);
-    await expect(exec("bin/cli.js", ...args)).rejects.toHaveProperty("stdout", "");
-    await expect(exec("bin/cli.js", ...args)).rejects.toMatchSnapshot("stderr");
+    await expect(exec(pkg.bin, ...args)).rejects.toThrowErrorMatchingSnapshot();
+    await expect(exec(pkg.bin, ...args)).rejects.toHaveProperty("code", 1);
+    await expect(exec(pkg.bin, ...args)).rejects.toHaveProperty("stdout", "");
+    await expect(exec(pkg.bin, ...args)).rejects.toMatchSnapshot("stderr");
   });
 });
 
 ["--help", "-h"].forEach((option) => {
   test(`with "${option}" option`, async () => {
-    const { stdout, stderr } = await exec("bin/cli.js", option);
+    const { stdout, stderr } = await exec(pkg.bin, option);
     expect(stdout).toMatchSnapshot();
     expect(stderr).toEqual("");
   });

test/init.test.js

@@ -82,7 +82,7 @@ test("throw error if no package.json", () =>
 test("End-to-End via CLI", () =>
   sandbox(async (ctx) => {
     await ctx.fixture("package-normal.json");
-    const { stdout, stderr } = await exec(path.resolve("bin/cli.js"), "init", {
+    const { stdout, stderr } = await exec(path.resolve(pkg.bin), "init", {
       cwd: ctx.initArgs.cwd,
     });
     expect(stdout).toMatchInlineSnapshot(`

test/version.test.js

@@ -3,7 +3,7 @@ const exec = require("./helpers/exec");
 
 ["--version", "-v"].forEach((option) => {
   test(`with "${option}" option`, async () => {
-    const { stdout, stderr } = await exec("bin/cli.js", option);
+    const { stdout, stderr } = await exec(pkg.bin, option);
     expect(stdout).toEqual(`${pkg.version}\n`);
     expect(stderr).toEqual("");
   });