feat: better support for network.provideResponse (#2065)

TODO: Update comment is the code

src/bidiMapper/domains/network/NetworkProcessor.ts

@@ -197,7 +197,7 @@ export class NetworkProcessor {
   ): Promise<EmptyResult> {
     const {
       statusCode,
-      reasonPhrase,
+      reasonPhrase: responsePhrase,
       headers,
       body,
       request: networkId,
@@ -218,11 +218,38 @@ export class NetworkProcessor {
       Network.InterceptPhase.AuthRequired,
     ]);
 
+    // We need to pass through if the request is already in
+    // AuthRequired phase
+    if (request.interceptPhase === Network.InterceptPhase.AuthRequired) {
+      // We need to use `ProvideCredentials`
+      // As `Default` may cancel the request
+      await request.continueWithAuth({
+        response: 'ProvideCredentials',
+      });
+      return {};
+    }
+
+    // If we con't modify the response
+    // Just continue the request
+    if (!body && !headers) {
+      await request.continueRequest();
+      return {};
+    }
+
+    const responseCode = statusCode ?? request.statusCode ?? 200;
+
+    let parsedBody: string | undefined;
+    if (body?.type === 'string') {
+      parsedBody = btoa(body.value);
+    } else if (body?.type === 'base64') {
+      parsedBody = body.value;
+    }
+
     await request.provideResponse({
-      responseCode: statusCode ?? request.statusCode,
-      responsePhrase: reasonPhrase,
+      responseCode,
+      responsePhrase,
       responseHeaders,
-      body: body?.value, // TODO: Differ base64 / string
+      body: parsedBody,
     });
 
     return {};

src/bidiMapper/domains/network/NetworkRequest.ts

@@ -457,12 +457,11 @@ export class NetworkRequest {
   }
 
   /** Returns the HTTP status code associated with this request if any. */
-  get statusCode(): number {
+  get statusCode(): number | undefined {
     return (
       this.#response.paused?.responseStatusCode ??
       this.#response.extraInfo?.statusCode ??
-      this.#response.info?.status ??
-      -1 // TODO: Throw an exception or use some other status code?
+      this.#response.info?.status
     );
   }
 
@@ -544,7 +543,7 @@ export class NetworkRequest {
     return {
       url: this.url,
       protocol: this.#response.info?.protocol ?? '',
-      status: this.statusCode,
+      status: this.statusCode ?? -1, // TODO: Throw an exception or use some other status code?
       statusText:
         this.#response.info?.statusText ||
         this.#response.paused?.responseStatusText ||