chore: include target ID in UncaughtError

Author: szuend

src/PageCollector.ts

@@ -25,10 +25,16 @@ import {
 export class UncaughtError {
   readonly message: string;
   readonly stackTrace?: Protocol.Runtime.StackTrace;
+  readonly targetId: string;
 
-  constructor(message: string, stackTrace?: Protocol.Runtime.StackTrace) {
+  constructor(
+    message: string,
+    stackTrace: Protocol.Runtime.StackTrace | undefined,
+    targetId: string,
+  ) {
     this.message = message;
     this.stackTrace = stackTrace;
+    this.targetId = targetId;
   }
 }
 
@@ -323,7 +329,11 @@ class PageIssueSubscriber {
     const messageWithRest = exception?.description?.split('\n    at ', 2) ?? [];
     const message = text + ' ' + (messageWithRest[0] ?? '');
 
-    this.#page.emit('uncaughtError', new UncaughtError(message, stackTrace));
+    this.#page.emit(
+      'uncaughtError',
+      // @ts-expect-error Use internal Puppeteer API for target ID
+      new UncaughtError(message, stackTrace, this.#session.target()._targetId),
+    );
   };
 
   // On navigation, we reset issue aggregation.