Micro perf improvement

Author: drwpow

docs/openapi-fetch/api.md

@@ -74,18 +74,19 @@ Sometimes your backend doesn’t use one of the standard serialization methods,
 
 ```ts
 const client = createClient({
-  querySerializer(queryParam) {
-    let s = [];
-    for (const [k, v] of Object.entries(queryParam)) {
-      if (Array.isArray(v)) {
-        for (const i of v) {
-          s.push(`${k}[]=${encodeURIComponent(i)}`);
+  querySerializer(queryParams) {
+    const search = [];
+    for (const name in queryParams) {
+      const value = queryParams[name];
+      if (Array.isArray(value)) {
+        for (const item of value) {
+          s.push(`${name}[]=${encodeURIComponent(item)}`);
         }
       } else {
-        s.push(`${k}=${encodeURLComponent(v)}`);
+        s.push(`${name}=${encodeURLComponent(value)}`);
       }
     }
-    return encodeURI(s.join(",")); // ?tags[]=food,tags[]=california,tags[]=healthy
+    return search.join(","); // ?tags[]=food,tags[]=california,tags[]=healthy
   },
 });
 ```
@@ -126,8 +127,8 @@ const { data, error } = await PUT("/submit", {
   },
   bodySerializer(body) {
     const fd = new FormData();
-    for (const [k, v] of Object.entries(body)) {
-      fd.append(k, v);
+    for (const name in body) {
+      fd.append(name, body[name]);
     }
     return fd;
   },

packages/openapi-fetch/src/index.js

@@ -3,7 +3,6 @@ const DEFAULT_HEADERS = {
   "Content-Type": "application/json",
 };
 
-const LEADING_QUESTION_RE = /^\?+/;
 const PATH_PARAM_RE = /\{[^{}]+\}/g;
 
 /**
@@ -19,7 +18,7 @@ export default function createClient(clientOptions) {
   } = clientOptions ?? {};
   let baseUrl = baseOptions.baseUrl ?? "";
   if (baseUrl.endsWith("/")) {
-    baseUrl = baseUrl.slice(0, -1); // remove trailing slash
+    baseUrl = baseUrl.substring(0, baseUrl.length - 1);
   }
 
   /**
@@ -331,12 +330,8 @@ export function createQuerySerializer(options) {
  * @see https://swagger.io/docs/specification/serialization/#path
  */
 export function defaultPathSerializer(pathname, pathParams) {
-  const matches = pathname.match(PATH_PARAM_RE);
-  if (!matches || !matches.length) {
-    return undefined;
-  }
   let nextURL = pathname;
-  for (const match of matches) {
+  for (const match of pathname.match(PATH_PARAM_RE) ?? []) {
     let paramName = match.substring(1, match.length - 1);
     let explode = false;
     let style = "simple";
@@ -411,9 +406,10 @@ export function createFinalURL(pathname, options) {
   if (options.params?.path) {
     finalURL = defaultPathSerializer(finalURL, options.params.path);
   }
-  const search = options
-    .querySerializer(options.params.query ?? {})
-    .replace(LEADING_QUESTION_RE, "");
+  let search = options.querySerializer(options.params.query ?? {});
+  if (search.startsWith("?")) {
+    search = search.substring(1);
+  }
   if (search) {
     finalURL += `?${search}`;
   }

packages/openapi-fetch/test/index.test.ts

@@ -684,8 +684,8 @@ describe("client", () => {
         },
         bodySerializer(body) {
           const fd = new FormData();
-          for (const [k, v] of Object.entries(body)) {
-            fd.append(k, v);
+          for (const name in body) {
+            fd.append(name, body[name as keyof typeof body]);
           }
           return fd;
         },

packages/openapi-fetch/test/v7-beta.test.ts

@@ -693,8 +693,8 @@ describe("client", () => {
         },
         bodySerializer(body) {
           const fd = new FormData();
-          for (const [k, v] of Object.entries(body)) {
-            fd.append(k, v);
+          for (const name in body) {
+            fd.append(name, body[name as keyof typeof body]);
           }
           return fd;
         },