fix: unknown command error not showing with package.json (#82)

* fix: unknown command error not showing with package.json

* chore: remove debug log

* fix

Author: marvinhagemeister

src/bin.ts

@@ -249,11 +249,10 @@ if (args.length === 0) {
             await runScript(process.cwd(), cmd, { pkgManagerName });
           });
         } else {
-          console.error(kl.red(`Unknown command: ${cmd}`));
-          console.log();
-          printHelp();
-          process.exit(1);
+          throwUnknownCommand(cmd);
         }
+      } else {
+        throwUnknownCommand(cmd);
       }
     }
   }
@@ -278,3 +277,10 @@ async function run(fn: () => Promise<void>) {
     throw err;
   }
 }
+
+function throwUnknownCommand(cmd: string) {
+  console.error(kl.red(`Unknown command: ${cmd}`));
+  console.log();
+  printHelp();
+  process.exit(1);
+}

test/commands.test.ts

@@ -21,6 +21,39 @@ import {
   writeTextFile,
 } from "../src/utils";
 
+describe("general", () => {
+  it("exit 1 on unknown command", async () => {
+    try {
+      await withTempEnv(["foo"], async () => {});
+      assert.fail("no");
+    } catch (err) {
+      if (err instanceof Error) {
+        assert.match(err.message, /Child process/);
+        assert.equal((err as any).code, 1);
+      } else {
+        throw err;
+      }
+    }
+  });
+
+  // See https://github.com/jsr-io/jsr-npm/issues/79
+  it("exit 1 on unknown command in empty folder", async () => {
+    await runInTempDir(async (dir) => {
+      try {
+        await runJsr(["asdf"], dir);
+        assert.fail("no");
+      } catch (err) {
+        if (err instanceof Error) {
+          assert.match(err.message, /Child process/);
+          assert.equal((err as any).code, 1);
+        } else {
+          throw err;
+        }
+      }
+    });
+  });
+});
+
 describe("install", () => {
   it("jsr i @std/encoding - resolve latest version", async () => {
     await withTempEnv(["i", "@std/encoding"], async (dir) => {