fix(tui): reject unknown run options (#25)

Author: aiirvizionz

src/tui.mjs

@@ -277,6 +277,9 @@ async function runFile(args) {
       max = v;
     } else if (a === "--dry-run") {
       dryRun = true;
+    } else if (a.startsWith("-") && !file) {
+      console.log(err(`unknown option ${a}`));
+      return;
     } else if (!file) {
       file = a;
     }

test/tui.test.mjs

@@ -1,8 +1,25 @@
 import assert from "node:assert/strict";
+import { spawn } from "node:child_process";
 import test from "node:test";
+import { fileURLToPath } from "node:url";
 
 import { splitCommandLine } from "../src/tui.mjs";
 
+const BIN = fileURLToPath(new URL("../bin/moshcode.mjs", import.meta.url));
+
+function runTui(input) {
+  return new Promise((resolve, reject) => {
+    const child = spawn(process.execPath, [BIN], { stdio: ["pipe", "pipe", "pipe"] });
+    let stdout = "";
+    let stderr = "";
+    child.stdout.on("data", (chunk) => { stdout += chunk; });
+    child.stderr.on("data", (chunk) => { stderr += chunk; });
+    child.on("error", reject);
+    child.on("close", (status, signal) => resolve({ status, signal, stdout, stderr }));
+    child.stdin.end(input);
+  });
+}
+
 test("TUI command parsing preserves quoted native CLI arguments", () => {
   assert.deepEqual(
     splitCommandLine('/coinpay card pay --description "Fix the build" --note \'ship it\''),
@@ -23,3 +40,11 @@ test("TUI command parsing rejects incomplete quoting", () => {
   assert.throws(() => splitCommandLine('/coinpay --description "unfinished'), /unterminated/);
   assert.throws(() => splitCommandLine("/ugig trailing\\"), /trailing escape/);
 });
+
+test("TUI /run rejects unknown options before reading a script file", async () => {
+  const result = await runTui("/run --dryrun\n/quit\n");
+
+  assert.equal(result.status, 0);
+  assert.match(result.stdout, /unknown option --dryrun/);
+  assert.doesNotMatch(result.stdout, /can't read --dryrun/);
+});