@adrienlcp/i18n 0.1.1 → 0.1.2

package/dist/translator.js

@@ -106,9 +106,10 @@ const FORMATTED_COUNT = '{?}';
  * That is dictionary text rather than a caller's value, so the rule above is
  * untouched.
  *
- * A value of the wrong type, or one the message never asked for, leaves its
- * placeholder standing rather than throwing: one bad value costs one word, not
- * the whole sentence.
+ * A value of the wrong type, one the message never asked for, or one the
+ * locale cannot print — an invalid `Date`, an infinite count, a code that names
+ * no language — leaves its placeholder standing rather than throwing: one bad
+ * value costs one word, not the whole sentence.
  */
 const substitute = ({ expanding = [], formatters, message, options, values }) => message.replace(PLACEHOLDER, (placeholder, name, type) => {
     const value = values[name];
@@ -117,7 +118,7 @@ const substitute = ({ expanding = [], formatters, message, options, values }) =>
     }
     switch (type) {
         case 'date':
-            return value instanceof Date
+            return isValidDate(value)
                 ? formatters.date(options.date?.[name]).format(value)
                 : placeholder;
         case 'displayname':
@@ -167,7 +168,7 @@ const substitute = ({ expanding = [], formatters, message, options, values }) =>
                 })
                 : placeholder;
         case 'relative':
-            return typeof value === 'number'
+            return typeof value === 'number' && Number.isFinite(value)
                 ? (relativeTime({
                     count: value,
                     formatters,
@@ -220,9 +221,21 @@ const pluralize = ({ count, formatters, forms }) => {
 /**
  * A message declaring `{x:displayname}` must declare the kind of name it wants,
  * so the options are never absent — but the runtime shape stays loose, and
- * `Intl.DisplayNames` throws without a `type`. Nothing rather than a crash.
+ * `Intl.DisplayNames` throws without a `type`, and on a string that is no code
+ * at all (`'not a language'`). Nothing rather than a crash.
  */
-const displayName = ({ formatters, of, options }) => options === undefined ? undefined : formatters.displayname(options).of(of);
+const displayName = ({ formatters, of, options }) => {
+    if (options === undefined) {
+        return undefined;
+    }
+    try {
+        return formatters.displayname(options).of(of);
+    }
+    catch {
+        return undefined;
+    }
+};
+const isValidDate = (value) => value instanceof Date && !Number.isNaN(value.getTime());
 /** Same reasoning: the unit is declared with the message, or there is none. */
 const relativeTime = ({ count, formatters, unit }) => unit === undefined
     ? undefined

package/package.json

@@ -26,7 +26,7 @@
   },
   "sideEffects": false,
   "type": "module",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "scripts": {
     "build": "tsc -p tsconfig.build.json",
     "test": "vitest run",