Assert non-nullable type instead of using nullish coalesce

There will be undefined `decimal` only if the input to `formatToParts` does not contain a decimal. But there will always be a decimal because the input is hardcoded.

The TS language server is not yet intelligent enough to assert it is non-nullable from the hardcoded input, so we need to pass the type hint.

Using nullish coalesce results in an uncovered branch which is impossible to test, so it should be removed.

diff --git a/src/normalization.js b/src/normalization.js
index d05377c91867aa7f185c59ec2f404462e6a8c13c..b7faa553294747605266ce8619b22916543a0d86 100644 (file)
--- a/src/normalization.js
+++ b/src/normalization.js
@@ -1,5 +1,5 @@
 // User locale's decimal separator
-const DECIMAL_SEPARATOR = Intl.NumberFormat().formatToParts(.1).find(p => p.type === 'decimal')?.value ?? '.';
+const DECIMAL_SEPARATOR = /** @type {Intl.NumberFormatPart!} */(Intl.NumberFormat().formatToParts(.1).find(p => p.type === 'decimal')).value;
 
 /**
  * Convert a string to camelCase.