fix!: rejects NaN in Fraction (#428)

NaN passed both the upper-bound and lower-bound comparisons because every
ordered comparison against NaN is false, so an invalid value reached the
constructor. Add an explicit is_nan check and a NotANumber variant so the
value is rejected at the boundary instead of relying on downstream consumers
to special-case NaN.

https://claude.ai/code/session_01SEknK8pBtZzArSLx7wvrLj

Co-authored-by: Claude <noreply@anthropic.com>

Author: KSXGitHub

src/args/fraction.rs

@@ -18,12 +18,18 @@ pub enum ConversionError {
     /// Provided value is less than 0.
     #[display("less than 0")]
     LowerBound,
+    /// Provided value is `NaN`.
+    #[display("not a number")]
+    NotANumber,
 }
 
 impl Fraction {
     /// Create a [`Fraction`].
     pub fn new(value: f32) -> Result<Self, ConversionError> {
         use ConversionError::*;
+        if value.is_nan() {
+            return Err(NotANumber);
+        }
         if value >= 1.0 {
             return Err(UpperBound);
         }

tests/args_fraction.rs

@@ -48,6 +48,18 @@ fn equal_to_one() {
     );
 }
 
+#[test]
+fn not_a_number() {
+    let actual_error = "NaN".parse::<Fraction>().expect_err("cause nan error");
+    let actual_message = actual_error.to_string();
+    let expected_error = Conversion(NotANumber);
+    let expected_message = "not a number".to_string();
+    assert_eq!(
+        (actual_error, actual_message),
+        (expected_error, expected_message),
+    );
+}
+
 #[test]
 fn invalid_float_literal() {
     let actual = "a"