fix: avoid mutating redirect chain array when formatting

toStringDetailed() and toJSONDetailed() call .reverse() on the array
returned by request.redirectChain(), which mutates the original array
in-place. Subsequent calls to these methods or any code that reads the
redirect chain will see the reversed (incorrect) order.

Copy the array before reversing to avoid mutating shared state.

Author: sumleo

src/formatters/NetworkFormatter.ts

@@ -163,7 +163,7 @@ export class NetworkFormatter {
     if (redirectChain.length) {
       response.push(`### Redirect chain`);
       let indent = 0;
-      for (const request of redirectChain.reverse()) {
+      for (const request of [...redirectChain].reverse()) {
         const id = this.#options.requestIdResolver
           ? this.#options.requestIdResolver(request)
           : undefined;
@@ -191,7 +191,7 @@ export class NetworkFormatter {
 
   toJSONDetailed(): object {
     const redirectChain = this.#request.redirectChain();
-    const formattedRedirectChain = redirectChain.reverse().map(request => {
+    const formattedRedirectChain = [...redirectChain].reverse().map(request => {
       const id = this.#options.requestIdResolver
         ? this.#options.requestIdResolver(request)
         : undefined;