@whatwg-node/node-fetch 0.2.0-alpha-20230210095043-408a558 → 0.2.0-alpha-20230210105523-366e74c

package/Headers.d.ts

@@ -1,7 +1,13 @@
 export type PonyfillHeadersInit = [string, string][] | Record<string, string | string[] | undefined> | Headers;
 export declare class PonyfillHeaders implements Headers {
+    private headersInit?;
     private map;
-    constructor(headersInit?: PonyfillHeadersInit);
+    private mapIsBuilt;
+    private objectNormalizedKeysOfHeadersInit;
+    private objectOriginalKeysOfHeadersInit;
+    constructor(headersInit?: PonyfillHeadersInit | undefined);
+    private _get;
+    private getMap;
     append(name: string, value: string): void;
     get(name: string): string | null;
     has(name: string): boolean;

package/index.js

@@ -840,19 +840,66 @@ function isHeadersLike(headers) {
 }
 class PonyfillHeaders {
     constructor(headersInit) {
+        this.headersInit = headersInit;
         this.map = new Map();
-        if (headersInit != null) {
-            if (Array.isArray(headersInit)) {
-                this.map = new Map(headersInit);
+        this.mapIsBuilt = false;
+        this.objectNormalizedKeysOfHeadersInit = [];
+        this.objectOriginalKeysOfHeadersInit = [];
+    }
+    // perf: we don't need to build `this.map` for Requests, as we can access the headers directly
+    _get(key) {
+        // If the map is built, reuse it
+        if (this.mapIsBuilt) {
+            return this.map.get(key.toLowerCase()) || null;
+        }
+        // If the map is not built, try to get the value from the this.headersInit
+        if (this.headersInit == null) {
+            return null;
+        }
+        const normalized = key.toLowerCase();
+        if (Array.isArray(this.headersInit)) {
+            return this.headersInit.find(header => header[0] === normalized);
+        }
+        else if (isHeadersLike(this.headersInit)) {
+            return this.headersInit.get(normalized);
+        }
+        else {
+            const initValue = this.headersInit[key] || this.headersInit[normalized];
+            if (initValue != null) {
+                return initValue;
+            }
+            if (!this.objectNormalizedKeysOfHeadersInit.length) {
+                Object.keys(this.headersInit).forEach(k => {
+                    this.objectOriginalKeysOfHeadersInit.push(k);
+                    this.objectNormalizedKeysOfHeadersInit.push(k.toLowerCase());
+                });
+            }
+            const index = this.objectNormalizedKeysOfHeadersInit.indexOf(normalized);
+            if (index === -1) {
+                return null;
             }
-            else if (isHeadersLike(headersInit)) {
-                headersInit.forEach((value, key) => {
+            const originalKey = this.objectOriginalKeysOfHeadersInit[index];
+            return this.headersInit[originalKey];
+        }
+    }
+    // perf: Build the map of headers lazily, only when we need to access all headers or write to it.
+    // I could do a getter here, but I'm too lazy to type `getter`.
+    getMap() {
+        if (this.mapIsBuilt) {
+            return this.map;
+        }
+        if (this.headersInit != null) {
+            if (Array.isArray(this.headersInit)) {
+                this.map = new Map(this.headersInit);
+            }
+            else if (isHeadersLike(this.headersInit)) {
+                this.headersInit.forEach((value, key) => {
                     this.map.set(key, value);
                 });
             }
             else {
-                for (const initKey in headersInit) {
-                    const initValue = headersInit[initKey];
+                for (const initKey in this.headersInit) {
+                    const initValue = this.headersInit[initKey];
                     if (initValue != null) {
                         const normalizedValue = Array.isArray(initValue) ? initValue.join(', ') : initValue;
                         const normalizedKey = initKey.toLowerCase();
@@ -861,39 +908,48 @@ class PonyfillHeaders {
                 }
             }
         }
+        this.mapIsBuilt = true;
+        return this.map;
     }
     append(name, value) {
         const key = name.toLowerCase();
-        const existingValue = this.map.get(key);
+        const existingValue = this.getMap().get(key);
         const finalValue = existingValue ? `${existingValue}, ${value}` : value;
-        this.map.set(key, finalValue);
+        this.getMap().set(key, finalValue);
     }
     get(name) {
         const key = name.toLowerCase();
-        return this.map.get(key) || null;
+        const value = this._get(key);
+        if (value == null) {
+            return null;
+        }
+        if (Array.isArray(value)) {
+            return value.join(', ');
+        }
+        return value;
     }
     has(name) {
         const key = name.toLowerCase();
-        return this.map.has(key);
+        return !!this._get(key); // we might need to check if header exists and not just check if it's not nullable
     }
     set(name, value) {
         const key = name.toLowerCase();
-        this.map.set(key, value);
+        this.getMap().set(key, value);
     }
     delete(name) {
         const key = name.toLowerCase();
-        this.map.delete(key);
+        this.getMap().delete(key);
     }
     forEach(callback) {
-        this.map.forEach((value, key) => {
+        this.getMap().forEach((value, key) => {
             callback(value, key, this);
         });
     }
     entries() {
-        return this.map.entries();
+        return this.getMap().entries();
     }
     [Symbol.iterator]() {
-        return this.map.entries();
+        return this.getMap().entries();
     }
 }
 

package/index.mjs

@@ -834,19 +834,66 @@ function isHeadersLike(headers) {
 }
 class PonyfillHeaders {
     constructor(headersInit) {
+        this.headersInit = headersInit;
         this.map = new Map();
-        if (headersInit != null) {
-            if (Array.isArray(headersInit)) {
-                this.map = new Map(headersInit);
+        this.mapIsBuilt = false;
+        this.objectNormalizedKeysOfHeadersInit = [];
+        this.objectOriginalKeysOfHeadersInit = [];
+    }
+    // perf: we don't need to build `this.map` for Requests, as we can access the headers directly
+    _get(key) {
+        // If the map is built, reuse it
+        if (this.mapIsBuilt) {
+            return this.map.get(key.toLowerCase()) || null;
+        }
+        // If the map is not built, try to get the value from the this.headersInit
+        if (this.headersInit == null) {
+            return null;
+        }
+        const normalized = key.toLowerCase();
+        if (Array.isArray(this.headersInit)) {
+            return this.headersInit.find(header => header[0] === normalized);
+        }
+        else if (isHeadersLike(this.headersInit)) {
+            return this.headersInit.get(normalized);
+        }
+        else {
+            const initValue = this.headersInit[key] || this.headersInit[normalized];
+            if (initValue != null) {
+                return initValue;
+            }
+            if (!this.objectNormalizedKeysOfHeadersInit.length) {
+                Object.keys(this.headersInit).forEach(k => {
+                    this.objectOriginalKeysOfHeadersInit.push(k);
+                    this.objectNormalizedKeysOfHeadersInit.push(k.toLowerCase());
+                });
+            }
+            const index = this.objectNormalizedKeysOfHeadersInit.indexOf(normalized);
+            if (index === -1) {
+                return null;
             }
-            else if (isHeadersLike(headersInit)) {
-                headersInit.forEach((value, key) => {
+            const originalKey = this.objectOriginalKeysOfHeadersInit[index];
+            return this.headersInit[originalKey];
+        }
+    }
+    // perf: Build the map of headers lazily, only when we need to access all headers or write to it.
+    // I could do a getter here, but I'm too lazy to type `getter`.
+    getMap() {
+        if (this.mapIsBuilt) {
+            return this.map;
+        }
+        if (this.headersInit != null) {
+            if (Array.isArray(this.headersInit)) {
+                this.map = new Map(this.headersInit);
+            }
+            else if (isHeadersLike(this.headersInit)) {
+                this.headersInit.forEach((value, key) => {
                     this.map.set(key, value);
                 });
             }
             else {
-                for (const initKey in headersInit) {
-                    const initValue = headersInit[initKey];
+                for (const initKey in this.headersInit) {
+                    const initValue = this.headersInit[initKey];
                     if (initValue != null) {
                         const normalizedValue = Array.isArray(initValue) ? initValue.join(', ') : initValue;
                         const normalizedKey = initKey.toLowerCase();
@@ -855,39 +902,48 @@ class PonyfillHeaders {
                 }
             }
         }
+        this.mapIsBuilt = true;
+        return this.map;
     }
     append(name, value) {
         const key = name.toLowerCase();
-        const existingValue = this.map.get(key);
+        const existingValue = this.getMap().get(key);
         const finalValue = existingValue ? `${existingValue}, ${value}` : value;
-        this.map.set(key, finalValue);
+        this.getMap().set(key, finalValue);
     }
     get(name) {
         const key = name.toLowerCase();
-        return this.map.get(key) || null;
+        const value = this._get(key);
+        if (value == null) {
+            return null;
+        }
+        if (Array.isArray(value)) {
+            return value.join(', ');
+        }
+        return value;
     }
     has(name) {
         const key = name.toLowerCase();
-        return this.map.has(key);
+        return !!this._get(key); // we might need to check if header exists and not just check if it's not nullable
     }
     set(name, value) {
         const key = name.toLowerCase();
-        this.map.set(key, value);
+        this.getMap().set(key, value);
     }
     delete(name) {
         const key = name.toLowerCase();
-        this.map.delete(key);
+        this.getMap().delete(key);
     }
     forEach(callback) {
-        this.map.forEach((value, key) => {
+        this.getMap().forEach((value, key) => {
             callback(value, key, this);
         });
     }
     entries() {
-        return this.map.entries();
+        return this.getMap().entries();
     }
     [Symbol.iterator]() {
-        return this.map.entries();
+        return this.getMap().entries();
     }
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@whatwg-node/node-fetch",
-  "version": "0.2.0-alpha-20230210095043-408a558",
+  "version": "0.2.0-alpha-20230210105523-366e74c",
   "description": "Fetch API implementation for Node",
   "sideEffects": false,
   "peerDependencies": {