perf: improve stringify benchmarks

Author: anonrig

benchmark/stringify.mjs

@@ -8,7 +8,10 @@ const value = {
   frappucino: "muffin",
   goat: "scone",
   pond: "moose",
-  foo: ["bar", "baz"],
+  foo: ["bar", "baz", "bal"],
+  bool: true,
+  bigIntKey: BigInt(100),
+  numberKey: 256,
 };
 
 await benchmark(

lib/internals/querystring.js

@@ -71,11 +71,6 @@ function encodeString(str) {
     // Surrogate pair
     ++i;
 
-    // This branch should never happen because all URLSearchParams entries
-    // should already be converted to USVString. But, included for
-    // completion's sake anyway.
-    if (i >= len) throw new Error("Invalid URI");
-
     const c2 = str.charCodeAt(i) & 0x3FF;
 
     lastPos = i + 1;

lib/stringify.js

@@ -8,13 +8,13 @@ function getAsPrimitive(value) {
   if (type === "string") {
     // Length check is handled inside encodeString function
     return encodeString(value);
-  } else if (type === "number" && Number.isFinite(value)) {
-    if (Math.abs(value) < 1e21) return value + "";
-    return encodeString(value);
   } else if (type === "bigint") {
     return value + "";
   } else if (type === "boolean") {
     return value ? "true" : "false";
+  } else if (type === "number" && Number.isFinite(value)) {
+    if (Math.abs(value) < 1e21) return value + "";
+    return encodeString(value + "");
   }
 
   return "";