Reject non-decimal run max values

Author: aiirvizionz

bin/moshcode.mjs

@@ -28,6 +28,9 @@ function readScript(arg) {
 
 function parseMax(value) {
   if (value === undefined) throw new Error("moshcode run: --max requires a positive integer");
+  if (!/^\d+$/.test(value)) {
+    throw new Error(`moshcode run: --max must be a positive integer, got ${JSON.stringify(value)}`);
+  }
   const max = Number(value);
   if (!Number.isInteger(max) || max < 1) {
     throw new Error(`moshcode run: --max must be a positive integer, got ${JSON.stringify(value)}`);

test/run-options.test.mjs

@@ -33,3 +33,12 @@ test("run rejects multiple script files", () => {
   assert.equal(result.status, 1);
   assert.match(result.stderr, /moshcode run: expected one script file/);
 });
+
+test("run rejects non-decimal max values", () => {
+  for (const value of ["0x10", "1e2", "3.5"]) {
+    const result = run(["--max", value, "--dry-run"]);
+
+    assert.equal(result.status, 1);
+    assert.match(result.stderr, /moshcode run: --max must be a positive integer/);
+  }
+});