fix(mf2): Do not fallback on all :number and :datetime option resolution errors

Author: XM5jDcsHTyGJtQqlCi

mf2/fluent/src/fluent.test.ts

@@ -91,7 +91,7 @@ const testCases: Record<string, TestCase> = {
       {
         msg: 'num-fraction-bad',
         scope: { arg: 1234 },
-        exp: '{$arg}',
+        exp: '1,234',
         errors: ['bad-option']
       },
       { msg: 'num-style', scope: { arg: 1234 }, exp: '123,400%' },

mf2/messageformat/src/functions/datetime.ts

@@ -68,7 +68,7 @@ export const datetime = (
         }
       } catch {
         const msg = `Value ${value} is not valid for :datetime option ${name}`;
-        throw new MessageResolutionError('bad-option', msg, ctx.source);
+        ctx.onError(new MessageResolutionError('bad-option', msg, ctx.source));
       }
     }
 

mf2/messageformat/src/functions/number.test.ts

@@ -0,0 +1,8 @@
+import { MessageFormat } from '../index.js';
+
+test('soft fail for integer options', () => {
+  const mf = new MessageFormat('en', '{42 :number minimumFractionDigits=foo}');
+  const onError = jest.fn();
+  expect(mf.format(undefined, onError)).toEqual('42');
+  expect(onError.mock.calls).toMatchObject([[{ type: 'bad-option' }]]);
+});

mf2/messageformat/src/functions/number.ts

@@ -105,7 +105,7 @@ export function number(
       }
     } catch {
       const msg = `Value ${optval} is not valid for :number option ${name}`;
-      throw new MessageResolutionError('bad-option', msg, source);
+      ctx.onError(new MessageResolutionError('bad-option', msg, source));
     }
   }