fix: handle u64 conversion for number between MAX_SAFE_INTEGER and i64::MAX

Author: Icemic

src/value/value.rs

@@ -320,6 +320,14 @@ impl OwnedJsValue {
             });
         }
 
+        // numbers between 2^53 -1 and i64::MAX is treated as float in quickjs
+        if self.is_float() {
+            let float = self.to_float()?;
+            return Ok(BigInt {
+                inner: BigIntOrI64::Int(float as i64),
+            });
+        }
+
         if self.is_short_bigint() {
             let int = unsafe { q::JS_Ext_GetShortBigInt(self.value) };
             return Ok(BigInt {

tests/convert.rs

@@ -18,6 +18,10 @@ fn test_try_from_owned_js_value() {
     let value: Option<i32> = js_value.try_into().unwrap();
     assert_eq!(value, Some(42));
 
+    let js_value: OwnedJsValue = context.eval("1754784747637", false).unwrap();
+    let value: Option<u64> = js_value.try_into().unwrap();
+    assert_eq!(value, Some(1754784747637));
+
     let js_value: OwnedJsValue = context.eval("true", false).unwrap();
     let value: bool = js_value.try_into().unwrap();
     assert_eq!(value, true);