fix: runIfWaitingForDebugger when targets are reloaded after crashing

packages/server/lib/browsers/browser-cri-client.ts

@@ -263,6 +263,12 @@ export class BrowserCriClient {
       this._onTargetDestroyed({ browserClient, browserCriClient, browserName, event, onAsynchronousError })
     })
 
+    browserClient.on('Inspector.targetReloadedAfterCrash', async (event, sessionId) => {
+      // Things like service workers will effectively crash in terms of CDP when the page is reloaded in the middle of things
+      // We will still auto attach in this case, but we need to runIfWaitingForDebugger to get the page back to a running state
+      await browserClient.send('Runtime.runIfWaitingForDebugger', undefined, sessionId)
+    })
+
     await Promise.all(promises)
   }
 

packages/server/lib/browsers/cdp_automation.ts

@@ -142,7 +142,7 @@ export const normalizeResourceType = (resourceType: string | undefined): Resourc
 
 export type SendDebuggerCommand = <T extends CdpCommand>(message: T, data?: ProtocolMapping.Commands[T]['paramsType'][0], sessionId?: string) => Promise<ProtocolMapping.Commands[T]['returnType']>
 
-export type OnFn = <T extends CdpEvent>(eventName: T, cb: (data: ProtocolMapping.Events[T][0]) => void) => void
+export type OnFn = <T extends CdpEvent>(eventName: T, cb: (data: ProtocolMapping.Events[T][0], sessionId?: string) => void) => void
 
 export type OffFn = (eventName: string, cb: (data: any) => void) => void
 

system-tests/projects/e2e/cypress/e2e/web_worker.cy.js

@@ -32,3 +32,12 @@ it('loads web workers', { defaultCommandTimeout: 1900 }, () => {
   .then(webWorker)
   .then(sharedWorker)
 })
+
+// Timeout of 1900 will ensure that the proxy correlation timeout is not hit
+it('reloads web workers', { defaultCommandTimeout: 1900 }, () => {
+  cy.visit('https://localhost:1515/web_worker.html')
+
+  cy.reload()
+  .then(webWorker)
+  .then(sharedWorker)
+})

system-tests/test/web_worker_spec.js

@@ -46,8 +46,8 @@ describe('e2e web worker', () => {
     spec: 'web_worker.cy.js',
     onRun: async (exec, browser) => {
       await exec()
-      expect(requestsForWebWorker).to.eq(1)
-      expect(requestsForSharedWorker).to.eq(1)
+      expect(requestsForWebWorker).to.eq(2)
+      expect(requestsForSharedWorker).to.eq(2)
     },
   })
 })