fix(cli): Improve number parsing

packages/cli/src/commands/adopt.js

@@ -3,6 +3,7 @@ import os from 'os';
 import { E } from '@endo/far';
 import { withEndoAgent } from '../context.js';
 import { parsePetNamePath } from '../pet-name.js';
+import { parseNumber } from '../number-parse.js';
 
 export const adoptCommand = async ({
   messageNumberText,
@@ -11,7 +12,9 @@ export const adoptCommand = async ({
   agentNames,
 }) =>
   withEndoAgent(agentNames, { os, process }, async ({ agent }) => {
-    // TODO less bad number parsing.
-    const messageNumber = Number(messageNumberText);
-    await E(agent).adopt(messageNumber, edgeName, parsePetNamePath(name));
+    await E(agent).adopt(
+      parseNumber(messageNumberText),
+      edgeName,
+      parsePetNamePath(name),
+    );
   });

packages/cli/src/commands/reject.js

@@ -2,14 +2,13 @@
 import os from 'os';
 import { E } from '@endo/far';
 import { withEndoAgent } from '../context.js';
+import { parseNumber } from '../number-parse.js';
 
 export const rejectCommand = async ({
   requestNumberText,
   message,
   agentNames,
 }) =>
   withEndoAgent(agentNames, { os, process }, async ({ agent }) => {
-    // TODO less bad number parsing.
-    const requestNumber = Number(requestNumberText);
-    await E(agent).reject(requestNumber, message);
+    await E(agent).reject(parseNumber(requestNumberText), message);
   });

packages/cli/src/commands/resolve.js

@@ -2,14 +2,13 @@
 import os from 'os';
 import { E } from '@endo/far';
 import { withEndoAgent } from '../context.js';
+import { parseNumber } from '../number-parse.js';
 
 export const resolveCommand = async ({
   requestNumberText,
   resolutionName,
   agentNames,
 }) =>
   withEndoAgent(agentNames, { os, process }, async ({ agent }) => {
-    // TODO less bad number parsing.
-    const requestNumber = Number(requestNumberText);
-    await E(agent).resolve(requestNumber, resolutionName);
+    await E(agent).resolve(parseNumber(requestNumberText), resolutionName);
   });

packages/cli/src/number-parse.js

@@ -0,0 +1,17 @@
+/**
+ * Parses the input and returns it as a number if it's valid, otherwise throws error.
+ *
+ * @param {string | number} input
+ * @returns {number | undefined}
+ */
+export const parseNumber = input => {
+  if (typeof input === 'number' && !Number.isNaN(input)) {
+    return input;
+  }
+
+  if (typeof input === 'string' && /^-?\d+(\.\d+)?$/.test(input.trim())) {
+    return Number(input);
+  }
+
+  throw new Error(`Invalid number: ${input}`);
+};

packages/cli/test/number-parse.test.js

@@ -0,0 +1,53 @@
+import test from 'ava';
+import { parseNumber } from '../src/number-parse.js';
+
+// Test for valid number input
+test('returns the number itself if input is a valid number', t => {
+  t.is(parseNumber(42), 42);
+  t.is(parseNumber(-42), -42);
+  t.is(parseNumber(0), 0);
+  t.is(parseNumber(3.14), 3.14);
+  t.is(parseNumber(-3.14), -3.14);
+});
+
+// Test for valid string input
+test('returns the number when input is a valid numeric string', t => {
+  t.is(parseNumber('42'), 42);
+  t.is(parseNumber('-42'), -42);
+  t.is(parseNumber('3.14'), 3.14);
+  t.is(parseNumber('-3.14'), -3.14);
+  t.is(parseNumber('  42  '), 42);
+});
+
+// Test for invalid string input
+test('throws an error when input is not a valid numeric string', t => {
+  const error = t.throws(() => parseNumber('abc'));
+  t.is(error.message, 'Invalid number: abc');
+
+  const error2 = t.throws(() => parseNumber('42abc'));
+  t.is(error2.message, 'Invalid number: 42abc');
+
+  const error3 = t.throws(() => parseNumber(''));
+  t.is(error3.message, 'Invalid number: ');
+
+  const error4 = t.throws(() => parseNumber('   '));
+  t.is(error4.message, 'Invalid number:    ');
+
+  const error5 = t.throws(() => parseNumber('42..42'));
+  t.is(error5.message, 'Invalid number: 42..42');
+});
+
+// Test for invalid input
+test('throws an error when input is not a valid number', t => {
+  const error1 = t.throws(() => parseNumber(null));
+  t.is(error1.message, 'Invalid number: null');
+
+  const error2 = t.throws(() => parseNumber(undefined));
+  t.is(error2.message, 'Invalid number: undefined');
+
+  const error3 = t.throws(() => parseNumber({}));
+  t.is(error3.message, 'Invalid number: [object Object]');
+
+  const error4 = t.throws(() => parseNumber([]));
+  t.is(error4.message, 'Invalid number: ');
+});