fix(cli): return success for help aliases

Author: phucnguyen1707

bin/moshcode.mjs

@@ -496,7 +496,7 @@ async function main() {
   }
 
   help();
-  if (cmd && cmd !== "help") process.exit(1);
+  if (cmd && !["help", "--help", "-h"].includes(cmd)) process.exit(1);
 }
 
 main();

src/cli-schema.mjs

@@ -23,6 +23,8 @@ export const CORE_CLI_COMMANDS = [
   { name: "completion", description: "print a shell completion script" },
   { name: "run", description: "run a moshscript" },
   { name: "help", description: "show command help" },
+  { name: "--help", description: "show command help" },
+  { name: "-h", description: "show command help" },
   { name: "version", description: "show the installed version" },
   { name: "--version", description: "show the installed version" },
   { name: "-v", description: "show the installed version" },

test/cli.test.mjs

@@ -22,6 +22,17 @@ test("moshcode --version prints the package version", () => {
   assert.equal(result.stderr, "");
 });
 
+for (const command of ["help", "--help", "-h"]) {
+  test(`moshcode ${command} prints help successfully`, () => {
+    const result = spawnSync(process.execPath, [BIN, command], {
+      encoding: "utf8",
+    });
+    assert.equal(result.status, 0);
+    assert.match(result.stdout, /^moshcode .*\n\nusage:/);
+    assert.equal(result.stderr, "");
+  });
+}
+
 for (const command of ["engines", "tools"]) {
   test(`moshcode ${command} --json prints machine-readable install status`, () => {
     const result = spawnSync(process.execPath, [BIN, command, "--json"], {

test/completion.test.mjs

@@ -55,6 +55,9 @@ test("completion model derives engines, aliases, and tools from their registries
     assert.ok(top.has(name));
     assert.ok(install.has(name));
   }
+  for (const name of ["help", "--help", "-h"]) {
+    assert.ok(top.has(name));
+  }
 
   const upgrade = new Set(names(model.upgrade));
   for (const { name } of UPGRADE_TARGETS) assert.ok(upgrade.has(name));