jsc: use saturating coerce::<i32> for Map/Set size display

src/jsc/ConsoleObject.rs

@@ -4790,7 +4790,7 @@ pub mod formatter {
             let length_value = value
                 .get(self.global_this, "size")?
                 .unwrap_or_else(|| JSValue::js_number_from_int32(0));
-            let length = length_value.coerce_to_i32(self.global_this)?;
+            let length = length_value.coerce::<i32>(self.global_this)?;
 
             let prev_quote_strings = self.quote_strings;
             self.quote_strings = true;
@@ -4932,7 +4932,7 @@ pub mod formatter {
             let length_value = value
                 .get(self.global_this, "size")?
                 .unwrap_or_else(|| JSValue::js_number_from_int32(0));
-            let length = length_value.coerce_to_i32(self.global_this)?;
+            let length = length_value.coerce::<i32>(self.global_this)?;
 
             let prev_quote_strings = self.quote_strings;
             self.quote_strings = true;

src/runtime/crypto/PasswordObject.rs

@@ -93,7 +93,7 @@ impl AlgorithmValue {
                                     .throw_invalid_argument_type("hash", "cost", "number"));
                             }
 
-                            let rounds = rounds_value.coerce_to_i32(global_object)?;
+                            let rounds = rounds_value.coerce::<i32>(global_object)?;
 
                             if rounds < 4 || rounds > 31 {
                                 return Err(global_object.throw_invalid_arguments(format_args!(
@@ -117,7 +117,7 @@ impl AlgorithmValue {
                                     .throw_invalid_argument_type("hash", "timeCost", "number"));
                             }
 
-                            let time_cost = time_value.coerce_to_i32(global_object)?;
+                            let time_cost = time_value.coerce::<i32>(global_object)?;
 
                             if time_cost < 1 {
                                 return Err(global_object.throw_invalid_arguments(format_args!(
@@ -137,7 +137,7 @@ impl AlgorithmValue {
                                 ));
                             }
 
-                            let memory_cost = memory_value.coerce_to_i32(global_object)?;
+                            let memory_cost = memory_value.coerce::<i32>(global_object)?;
 
                             // argon2 requires `memoryCost >= 8 * parallelism`;
                             // Bun hard-codes `parallelism = 1` (see

src/runtime/server/ServerWebSocket.rs

@@ -1339,7 +1339,7 @@ impl ServerWebSocket {
                 )));
             }
 
-            break 'brk args.ptr[0].coerce_to_i32(global_this)?;
+            break 'brk args.ptr[0].coerce::<i32>(global_this)?;
         };
 
         let message_value: ZigStringSlice = 'brk: {

src/sourcemap_jsc/JSSourceMap.rs

@@ -329,8 +329,8 @@ fn get_line_column(global: &JSGlobalObject, frame: &CallFrame) -> JsResult<[i32;
 
     Ok([
         // Node.js does no validations.
-        line_number_value.coerce_to_i32(global)?,
-        column_number_value.coerce_to_i32(global)?,
+        line_number_value.coerce::<i32>(global)?,
+        column_number_value.coerce::<i32>(global)?,
     ])
 }
 

test/js/bun/util/inspect.test.js

@@ -364,6 +364,23 @@ it("inspect", () => {
     value: "not a number",
   });
   expect(Bun.inspect(mapWithOverriddenSize)).toBe("Map {}");
+
+  // Size values >= 2^31 saturate to i32::MAX instead of wrapping negative.
+  const mapWithHugeSize = new Map([[1, 1]]);
+  Object.defineProperty(mapWithHugeSize, "size", { value: 3000000000 });
+  expect(Bun.inspect(mapWithHugeSize)).toBe("Map(2147483647) {\n  1: 1,\n}");
+
+  const setWithHugeSize = new Set([1]);
+  Object.defineProperty(setWithHugeSize, "size", { value: 3000000000 });
+  expect(Bun.inspect(setWithHugeSize)).toBe("Set(2147483647) {\n  1,\n}");
+
+  const mapWithInfiniteSize = new Map([[1, 1]]);
+  Object.defineProperty(mapWithInfiniteSize, "size", { value: Infinity });
+  expect(Bun.inspect(mapWithInfiniteSize)).toBe("Map(2147483647) {\n  1: 1,\n}");
+
+  const setWithInfiniteSize = new Set([1]);
+  Object.defineProperty(setWithInfiniteSize, "size", { value: Infinity });
+  expect(Bun.inspect(setWithInfiniteSize)).toBe("Set(2147483647) {\n  1,\n}");
   expect(Bun.inspect(<div>foo</div>)).toBe("<div>foo</div>");
   expect(Bun.inspect(<div hello>foo</div>)).toBe("<div hello=true>foo</div>");
   expect(Bun.inspect(<div hello={1}>foo</div>)).toBe("<div hello=1>foo</div>");