@optique/core 1.0.0-dev.745 → 1.0.0-dev.757

package/dist/valueparser.cjs

@@ -14,11 +14,31 @@ function isValueParser(object) {
 * Implementation of the choice parser for both string and number types.
 */
 function choice(choices, options = {}) {
+	if (choices.length < 1) throw new TypeError("Expected at least one choice, but got an empty array.");
 	const metavar = options.metavar ?? "TYPE";
 	require_nonempty.ensureNonEmptyString(metavar);
 	const isNumberChoice = choices.length > 0 && typeof choices[0] === "number";
 	if (isNumberChoice) {
-		const numberChoices = choices;
+		const numberChoices = (() => {
+			const seen = /* @__PURE__ */ new Set();
+			let hasPositiveZero = false;
+			let hasNegativeZero = false;
+			const result = [];
+			for (const v of choices) {
+				if (Object.is(v, -0)) {
+					if (hasNegativeZero) continue;
+					hasNegativeZero = true;
+				} else if (Object.is(v, 0)) {
+					if (hasPositiveZero) continue;
+					hasPositiveZero = true;
+				} else {
+					if (seen.has(v)) continue;
+					seen.add(v);
+				}
+				result.push(v);
+			}
+			return result;
+		})();
 		const numberOptions = options;
 		const hasNaN = numberChoices.some((v) => Number.isNaN(v));
 		const validNumberChoices = hasNaN ? numberChoices.filter((v) => !Number.isNaN(v)) : numberChoices;
@@ -88,7 +108,7 @@ function choice(choices, options = {}) {
 			}
 		};
 	}
-	const stringChoices = choices;
+	const stringChoices = [...new Set(choices)];
 	const stringOptions = options;
 	const normalizedValues = stringOptions.caseInsensitive ? stringChoices.map((v) => v.toLowerCase()) : stringChoices;
 	if (stringOptions.caseInsensitive) {
@@ -104,7 +124,7 @@ function choice(choices, options = {}) {
 	return {
 		$mode: "sync",
 		metavar,
-		choices,
+		choices: stringChoices,
 		parse(input) {
 			const normalizedInput = stringOptions.caseInsensitive ? input.toLowerCase() : input;
 			const index = normalizedValues.indexOf(normalizedInput);

package/dist/valueparser.d.cts

@@ -206,6 +206,7 @@ declare function isValueParser<M extends Mode, T>(object: unknown): object is Va
  * @param options Configuration options for the choice parser.
  * @returns A {@link ValueParser} that checks if the input matches one of the
  *          specified values.
+ * @throws {TypeError} If the choices array is empty.
  * @throws {TypeError} If `caseInsensitive` is `true` and multiple choices
  *         normalize to the same lowercase value.
  */
@@ -221,6 +222,7 @@ declare function choice<const T extends string>(choices: readonly T[], options?:
  * @param options Configuration options for the choice parser.
  * @returns A {@link ValueParser} that checks if the input matches one of the
  *          specified values.
+ * @throws {TypeError} If the choices array is empty.
  * @since 0.9.0
  */
 declare function choice<const T extends number>(choices: readonly T[], options?: ChoiceOptionsNumber): ValueParser<"sync", T>;

package/dist/valueparser.d.ts

@@ -206,6 +206,7 @@ declare function isValueParser<M extends Mode, T>(object: unknown): object is Va
  * @param options Configuration options for the choice parser.
  * @returns A {@link ValueParser} that checks if the input matches one of the
  *          specified values.
+ * @throws {TypeError} If the choices array is empty.
  * @throws {TypeError} If `caseInsensitive` is `true` and multiple choices
  *         normalize to the same lowercase value.
  */
@@ -221,6 +222,7 @@ declare function choice<const T extends string>(choices: readonly T[], options?:
  * @param options Configuration options for the choice parser.
  * @returns A {@link ValueParser} that checks if the input matches one of the
  *          specified values.
+ * @throws {TypeError} If the choices array is empty.
  * @since 0.9.0
  */
 declare function choice<const T extends number>(choices: readonly T[], options?: ChoiceOptionsNumber): ValueParser<"sync", T>;

package/dist/valueparser.js

@@ -14,11 +14,31 @@ function isValueParser(object) {
 * Implementation of the choice parser for both string and number types.
 */
 function choice(choices, options = {}) {
+	if (choices.length < 1) throw new TypeError("Expected at least one choice, but got an empty array.");
 	const metavar = options.metavar ?? "TYPE";
 	ensureNonEmptyString(metavar);
 	const isNumberChoice = choices.length > 0 && typeof choices[0] === "number";
 	if (isNumberChoice) {
-		const numberChoices = choices;
+		const numberChoices = (() => {
+			const seen = /* @__PURE__ */ new Set();
+			let hasPositiveZero = false;
+			let hasNegativeZero = false;
+			const result = [];
+			for (const v of choices) {
+				if (Object.is(v, -0)) {
+					if (hasNegativeZero) continue;
+					hasNegativeZero = true;
+				} else if (Object.is(v, 0)) {
+					if (hasPositiveZero) continue;
+					hasPositiveZero = true;
+				} else {
+					if (seen.has(v)) continue;
+					seen.add(v);
+				}
+				result.push(v);
+			}
+			return result;
+		})();
 		const numberOptions = options;
 		const hasNaN = numberChoices.some((v) => Number.isNaN(v));
 		const validNumberChoices = hasNaN ? numberChoices.filter((v) => !Number.isNaN(v)) : numberChoices;
@@ -88,7 +108,7 @@ function choice(choices, options = {}) {
 			}
 		};
 	}
-	const stringChoices = choices;
+	const stringChoices = [...new Set(choices)];
 	const stringOptions = options;
 	const normalizedValues = stringOptions.caseInsensitive ? stringChoices.map((v) => v.toLowerCase()) : stringChoices;
 	if (stringOptions.caseInsensitive) {
@@ -104,7 +124,7 @@ function choice(choices, options = {}) {
 	return {
 		$mode: "sync",
 		metavar,
-		choices,
+		choices: stringChoices,
 		parse(input) {
 			const normalizedInput = stringOptions.caseInsensitive ? input.toLowerCase() : input;
 			const index = normalizedValues.indexOf(normalizedInput);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@optique/core",
-  "version": "1.0.0-dev.745+a12a4fbd",
+  "version": "1.0.0-dev.757+6abd354f",
   "description": "Type-safe combinatorial command-line interface parser",
   "keywords": [
     "CLI",