From a065eafbe22cc1dba8d6e866092c468204dd940e Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Wed, 18 Dec 2024 09:31:06 +0000
Subject: [PATCH 01/25] Fix bug where the right split point was not being
 picked for the 3rd section onwards

---
 packages/rrweb-snapshot/src/utils.ts     | 3 ++-
 packages/rrweb-snapshot/test/css.test.ts | 6 ++++++
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/packages/rrweb-snapshot/src/utils.ts b/packages/rrweb-snapshot/src/utils.ts
index 42aba4d649..f696bd6cab 100644
--- a/packages/rrweb-snapshot/src/utils.ts
+++ b/packages/rrweb-snapshot/src/utils.ts
@@ -466,7 +466,7 @@ export function splitCssText(
   const childNodes = Array.from(style.childNodes);
   const splits: string[] = [];
   if (childNodes.length > 1 && cssText && typeof cssText === 'string') {
-    const cssTextNorm = normalizeCssString(cssText);
+    let cssTextNorm = normalizeCssString(cssText);
     for (let i = 1; i < childNodes.length; i++) {
       if (
         childNodes[i].textContent &&
@@ -485,6 +485,7 @@ export function splitCssText(
               ) {
                 splits.push(cssText.substring(0, k));
                 cssText = cssText.substring(k);
+                cssTextNorm = cssTextNorm.substring(splitNorm);
                 break;
               }
             }
diff --git a/packages/rrweb-snapshot/test/css.test.ts b/packages/rrweb-snapshot/test/css.test.ts
index 99a5b362e9..d2940a0af5 100644
--- a/packages/rrweb-snapshot/test/css.test.ts
+++ b/packages/rrweb-snapshot/test/css.test.ts
@@ -105,10 +105,16 @@ describe('css splitter', () => {
       // as authored, e.g. no spaces
       style.append('.a{background-color:black;}');
 
+      // test how normalization finds the right sections
+      style.append('.b      {background-color:black;}');
+      style.append('.c{      background-color:                     black}');
+
       // how it is currently stringified (spaces present)
       const expected = [
         '.a { background-color: red; }',
         '.a { background-color: black; }',
+        '.b { background-color: black; }',
+        '.c { background-color: black; }',
       ];
       const browserSheet = expected.join('');
       expect(stringifyStylesheet(style.sheet!)).toEqual(browserSheet);

From e2fe6608207fba4b4bdb4925799877869c17b4eb Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Tue, 17 Dec 2024 23:39:39 +0000
Subject: [PATCH 02/25] Add test to put splitCssText through it's paces with a
 large file

---
 packages/rrweb-snapshot/test/css.test.ts | 30 ++++++++++++++++++++++++
 1 file changed, 30 insertions(+)

diff --git a/packages/rrweb-snapshot/test/css.test.ts b/packages/rrweb-snapshot/test/css.test.ts
index d2940a0af5..30877942ec 100644
--- a/packages/rrweb-snapshot/test/css.test.ts
+++ b/packages/rrweb-snapshot/test/css.test.ts
@@ -7,6 +7,8 @@ import postcss, { type AcceptedPlugin } from 'postcss';
 import { JSDOM } from 'jsdom';
 import { splitCssText, stringifyStylesheet } from './../src/utils';
 import { applyCssSplits } from './../src/rebuild';
+import * as fs from 'fs';
+import * as path from 'path';
 import type {
   serializedElementNodeWithId,
   BuildCache,
@@ -175,6 +177,34 @@ describe('css splitter', () => {
       expect(splitCssText(browserSheet, style)).toEqual(expected);
     }
   });
+
+  it('efficiently finds split points in large files', () => {
+    const cssText = fs.readFileSync(
+      path.resolve(__dirname, './css/benchmark.css'),
+      'utf8',
+    );
+
+    const parts = cssText.split('}');
+    const sections = [];
+    for (let i = 0; i < parts.length - 1; i++) {
+      if (i % 100 === 0) {
+        sections.push(parts[i] + '}');
+      } else {
+        sections[sections.length - 1] += parts[i] + '}';
+      }
+    }
+    sections[sections.length - 1] += parts[parts.length - 1];
+
+    expect(cssText.length).toEqual(sections.join('').length);
+
+    const style = JSDOM.fragment(`<style></style>`).querySelector('style');
+    if (style) {
+      sections.forEach((section) => {
+        style.appendChild(JSDOM.fragment(section));
+      });
+    }
+    expect(splitCssText(cssText, style)).toEqual(sections);
+  });
 });
 
 describe('applyCssSplits css rejoiner', function () {

From 2f663d403054103012088a573a15de5d1d91fea2 Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Wed, 18 Dec 2024 10:41:53 +0000
Subject: [PATCH 03/25] Introduce a limit which causes the 'efficiently' test
 to fail

---
 packages/rrweb-snapshot/src/utils.ts | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/packages/rrweb-snapshot/src/utils.ts b/packages/rrweb-snapshot/src/utils.ts
index f696bd6cab..5788dd8d90 100644
--- a/packages/rrweb-snapshot/src/utils.ts
+++ b/packages/rrweb-snapshot/src/utils.ts
@@ -465,6 +465,7 @@ export function splitCssText(
 ): string[] {
   const childNodes = Array.from(style.childNodes);
   const splits: string[] = [];
+  let iter_limit = 0;
   if (childNodes.length > 1 && cssText && typeof cssText === 'string') {
     let cssTextNorm = normalizeCssString(cssText);
     for (let i = 1; i < childNodes.length; i++) {
@@ -480,6 +481,12 @@ export function splitCssText(
             const splitNorm = cssTextNorm.indexOf(bit);
             // find the split point in the original text
             for (let k = splitNorm; k < cssText.length; k++) {
+              iter_limit += 1;
+              if (iter_limit > 300 * childNodes.length) {
+                // quit for performance purposes
+                splits.push(cssText);
+                return splits;
+              }
               if (
                 normalizeCssString(cssText.substring(0, k)).length === splitNorm
               ) {

From 8ba3b5433657e5975a9ad1f1449812f10c5e2fc3 Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Tue, 17 Dec 2024 16:30:09 +0000
Subject: [PATCH 04/25] Fix that it wasn't able to find a split when both
 halves were identical

---
 packages/rrweb-snapshot/src/utils.ts     | 29 ++++++++++++++++++++----
 packages/rrweb-snapshot/test/css.test.ts | 22 ++++++++++++++++++
 2 files changed, 47 insertions(+), 4 deletions(-)

diff --git a/packages/rrweb-snapshot/src/utils.ts b/packages/rrweb-snapshot/src/utils.ts
index 5788dd8d90..78255b37cd 100644
--- a/packages/rrweb-snapshot/src/utils.ts
+++ b/packages/rrweb-snapshot/src/utils.ts
@@ -474,11 +474,32 @@ export function splitCssText(
         typeof childNodes[i].textContent === 'string'
       ) {
         const textContentNorm = normalizeCssString(childNodes[i].textContent!);
-        for (let j = 3; j < textContentNorm.length; j++) {
-          // find a  substring that appears only once
+        let j = 3;
+        for (; j < textContentNorm.length; j++) {
+          if (
+            textContentNorm.indexOf(textContentNorm.substring(0, j), 1) !== -1
+          ) {
+            // substring needs to be unique to this section
+            continue;
+          }
+          break;
+        }
+        for (; j < textContentNorm.length; j++) {
           const bit = textContentNorm.substring(0, j);
-          if (cssTextNorm.split(bit).length === 2) {
-            const splitNorm = cssTextNorm.indexOf(bit);
+          // this substring should appears only once in overall text too
+          const bits = cssTextNorm.split(bit);
+          let splitNorm = -1;
+          if (bits.length === 2) {
+            splitNorm = cssTextNorm.indexOf(bit);
+          } else if (
+            bits.length > 2 &&
+            bits[0] === '' &&
+            childNodes[i - 1].textContent !== ''
+          ) {
+            // this childNode has same starting content as previous
+            splitNorm = cssTextNorm.indexOf(bit, 1);
+          }
+          if (splitNorm !== -1) {
             // find the split point in the original text
             for (let k = splitNorm; k < cssText.length; k++) {
               iter_limit += 1;
diff --git a/packages/rrweb-snapshot/test/css.test.ts b/packages/rrweb-snapshot/test/css.test.ts
index 30877942ec..a26e0005fd 100644
--- a/packages/rrweb-snapshot/test/css.test.ts
+++ b/packages/rrweb-snapshot/test/css.test.ts
@@ -145,6 +145,28 @@ describe('css splitter', () => {
     }
   });
 
+  it('finds css textElement splits correctly with two identical text nodes', () => {
+    const window = new Window({ url: 'https://localhost:8080' });
+    const document = window.document;
+    // as authored, with comment, missing semicolons
+    const textContent = '.a { color:red; } .b { color:blue; }';
+    document.head.innerHTML = '<style></style>';
+    const style = document.querySelector('style');
+    if (style) {
+      style.append(textContent);
+      style.append(textContent);
+
+      const expected = [textContent, textContent];
+      const browserSheet = expected.join('');
+      expect(splitCssText(browserSheet, style)).toEqual(expected);
+
+      style.append(textContent);
+      const expected3 = [textContent, textContent, textContent];
+      const browserSheet3 = expected3.join('');
+      expect(splitCssText(browserSheet3, style)).toEqual(expected3);
+    }
+  });
+
   it('finds css textElement splits correctly when vendor prefixed rules have been removed', () => {
     const style = JSDOM.fragment(`<style></style>`).querySelector('style');
     if (style) {

From 61e8b5fba2fe3b499a634c50e957666835b435d7 Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Tue, 17 Dec 2024 18:03:34 +0000
Subject: [PATCH 05/25] Fix poor 'crawling' performance in this part of the
 algorithm for large css texts - e.g. for a (doubled) benchmark.css, we were
 running normalizeCssText 9480 times before k got to the right place

---
 packages/rrweb-snapshot/src/utils.ts | 17 +++++++++++++----
 1 file changed, 13 insertions(+), 4 deletions(-)

diff --git a/packages/rrweb-snapshot/src/utils.ts b/packages/rrweb-snapshot/src/utils.ts
index 78255b37cd..dfc77b0cc9 100644
--- a/packages/rrweb-snapshot/src/utils.ts
+++ b/packages/rrweb-snapshot/src/utils.ts
@@ -501,20 +501,29 @@ export function splitCssText(
           }
           if (splitNorm !== -1) {
             // find the split point in the original text
-            for (let k = splitNorm; k < cssText.length; k++) {
+            let k = Math.floor(
+              (cssText.length * splitNorm) / cssTextNorm.length,
+            );
+            let dir = 0;
+            for (; k > 0 && k < cssText.length; k += dir) {
               iter_limit += 1;
               if (iter_limit > 300 * childNodes.length) {
                 // quit for performance purposes
                 splits.push(cssText);
                 return splits;
               }
-              if (
-                normalizeCssString(cssText.substring(0, k)).length === splitNorm
-              ) {
+              let normPart = normalizeCssString(cssText.substring(0, k));
+              if (normPart.length === splitNorm) {
                 splits.push(cssText.substring(0, k));
                 cssText = cssText.substring(k);
                 cssTextNorm = cssTextNorm.substring(splitNorm);
                 break;
+              } else if (dir === 0) {
+                if (normPart.length < splitNorm) {
+                  dir = 1;
+                } else {
+                  dir = -1;
+                }
               }
             }
             break;

From 3474cb27f9449e9a6f784882fe54f07efb129608 Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Tue, 17 Dec 2024 23:03:13 +0000
Subject: [PATCH 06/25] Need to take larger jumps to be efficient; use the
 scaling factor to make better guess at how big a jump to make

 - can reduce iter_limit from 300 to 50 to prove that this approach is better
---
 packages/rrweb-snapshot/src/utils.ts | 31 +++++++++++++++-------------
 1 file changed, 17 insertions(+), 14 deletions(-)

diff --git a/packages/rrweb-snapshot/src/utils.ts b/packages/rrweb-snapshot/src/utils.ts
index dfc77b0cc9..0de5e4daab 100644
--- a/packages/rrweb-snapshot/src/utils.ts
+++ b/packages/rrweb-snapshot/src/utils.ts
@@ -456,8 +456,9 @@ export function normalizeCssString(cssText: string): string {
 
 /**
  * Maps the output of stringifyStylesheet to individual text nodes of a <style> element
- * performance is not considered as this is anticipated to be very much an edge case
- * (javascript is needed to add extra text nodes to a <style>)
+ * which occurs when javascript is used to append to the style element
+ * and may also occur when browsers opt to break up large text nodes
+ * performance needs to be considered, see e.g. #1603
  */
 export function splitCssText(
   cssText: string,
@@ -468,6 +469,7 @@ export function splitCssText(
   let iter_limit = 0;
   if (childNodes.length > 1 && cssText && typeof cssText === 'string') {
     let cssTextNorm = normalizeCssString(cssText);
+    const normFactor = cssTextNorm.length / cssText.length;
     for (let i = 1; i < childNodes.length; i++) {
       if (
         childNodes[i].textContent &&
@@ -501,13 +503,10 @@ export function splitCssText(
           }
           if (splitNorm !== -1) {
             // find the split point in the original text
-            let k = Math.floor(
-              (cssText.length * splitNorm) / cssTextNorm.length,
-            );
-            let dir = 0;
-            for (; k > 0 && k < cssText.length; k += dir) {
+            let k = Math.floor(splitNorm / normFactor);
+            for (; k > 0 && k < cssText.length; ) {
               iter_limit += 1;
-              if (iter_limit > 300 * childNodes.length) {
+              if (iter_limit > 50 * childNodes.length) {
                 // quit for performance purposes
                 splits.push(cssText);
                 return splits;
@@ -518,12 +517,16 @@ export function splitCssText(
                 cssText = cssText.substring(k);
                 cssTextNorm = cssTextNorm.substring(splitNorm);
                 break;
-              } else if (dir === 0) {
-                if (normPart.length < splitNorm) {
-                  dir = 1;
-                } else {
-                  dir = -1;
-                }
+              } else if (normPart.length < splitNorm) {
+                k += Math.max(
+                  1,
+                  Math.floor((splitNorm - normPart.length) / normFactor),
+                );
+              } else {
+                k -= Math.max(
+                  1,
+                  Math.floor((normPart.length - splitNorm) * normFactor),
+                );
               }
             }
             break;

From 906dd3b96f6bd25efa09f94eae84c7679cbf4536 Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Wed, 18 Dec 2024 11:19:37 +0000
Subject: [PATCH 07/25] Add changeset

---
 .changeset/efficiently-splitCssText-1603.md | 6 ++++++
 1 file changed, 6 insertions(+)
 create mode 100644 .changeset/efficiently-splitCssText-1603.md

diff --git a/.changeset/efficiently-splitCssText-1603.md b/.changeset/efficiently-splitCssText-1603.md
new file mode 100644
index 0000000000..57c6d5e6c4
--- /dev/null
+++ b/.changeset/efficiently-splitCssText-1603.md
@@ -0,0 +1,6 @@
+---
+"rrweb-snapshot": patch
+"rrweb": patch
+---
+
+Improve performance of splitCssText for <style> elements with large css content - see #1603

From e0fb33d9f5a26886bd61e58902c9e609f683e25b Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Wed, 18 Dec 2024 11:20:04 +0000
Subject: [PATCH 08/25] Presuming the match on a character is more eficient
 than the indexOf here

---
 packages/rrweb-snapshot/src/utils.ts | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/packages/rrweb-snapshot/src/utils.ts b/packages/rrweb-snapshot/src/utils.ts
index 0de5e4daab..79c11a7cad 100644
--- a/packages/rrweb-snapshot/src/utils.ts
+++ b/packages/rrweb-snapshot/src/utils.ts
@@ -479,9 +479,11 @@ export function splitCssText(
         let j = 3;
         for (; j < textContentNorm.length; j++) {
           if (
+            // keep consuming css identifiers (to get a decent chunk more quickly)
+            textContentNorm[j].match(/[a-zA-Z0-9]/) ||
+            // substring needs to be unique to this section
             textContentNorm.indexOf(textContentNorm.substring(0, j), 1) !== -1
           ) {
-            // substring needs to be unique to this section
             continue;
           }
           break;

From f5fcfa522aabf6e107fe70c66a978bd3e86787a3 Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Wed, 18 Dec 2024 13:23:42 +0000
Subject: [PATCH 09/25] Fix eslint

---
 packages/rrweb-snapshot/src/utils.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/rrweb-snapshot/src/utils.ts b/packages/rrweb-snapshot/src/utils.ts
index 79c11a7cad..7eff11eb4b 100644
--- a/packages/rrweb-snapshot/src/utils.ts
+++ b/packages/rrweb-snapshot/src/utils.ts
@@ -513,7 +513,7 @@ export function splitCssText(
                 splits.push(cssText);
                 return splits;
               }
-              let normPart = normalizeCssString(cssText.substring(0, k));
+              const normPart = normalizeCssString(cssText.substring(0, k));
               if (normPart.length === splitNorm) {
                 splits.push(cssText.substring(0, k));
                 cssText = cssText.substring(k);

From a797e96d409d3fa90b177777b8e3e0c01037f50e Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Thu, 19 Dec 2024 14:13:45 +0000
Subject: [PATCH 10/25] Update puppeteer to try to solve the following issue in
 github actions:

Failed to launch the browser process! [...FATAL:zygote_host_impl_linux.cc(117)] No usable sandbox! Update your kernel ...
---
 packages/rrweb-snapshot/package.json | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/rrweb-snapshot/package.json b/packages/rrweb-snapshot/package.json
index 299384b2de..4db66f8ec5 100644
--- a/packages/rrweb-snapshot/package.json
+++ b/packages/rrweb-snapshot/package.json
@@ -59,7 +59,7 @@
     "@types/jsdom": "^20.0.0",
     "@types/node": "^18.15.11",
     "@types/puppeteer": "^5.4.4",
-    "puppeteer": "^17.1.3",
+    "puppeteer": "^20.9.0",
     "ts-node": "^7.0.1",
     "tslib": "^1.9.3",
     "typescript": "^5.4.5",

From defd363a98b934b4f366d196540c895d4c645f5c Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Thu, 19 Dec 2024 14:28:09 +0000
Subject: [PATCH 11/25] Bump Chrome to see if it solves following issue in
 github actions:

Failed to launch the browser process! [...FATAL:zygote_host_impl_linux.cc(117)] No usable sandbox! Update your kernel ...
---
 .puppeteerrc.cjs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.puppeteerrc.cjs b/.puppeteerrc.cjs
index 0831229fc5..261edd3f59 100644
--- a/.puppeteerrc.cjs
+++ b/.puppeteerrc.cjs
@@ -6,5 +6,5 @@ const { join } = require('path');
 module.exports = {
   // Changes the cache location for Puppeteer.
   cacheDirectory: join(__dirname, '.cache', 'puppeteer'),
-  browserRevision: '115.0.5763.0',
+  browserRevision: '131.0.6778.204',
 };

From ad42cba8158b6dbd67367d5d32bf78eeee989e18 Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Fri, 20 Dec 2024 11:09:40 +0000
Subject: [PATCH 12/25] Both Firefox and the 131 version of Chrome I've
 upgraded to remove default css values from shorthand properties when
 retrieved via `sheet.rules[0].cssText`

---
 packages/rrweb/test/__snapshots__/replayer.test.ts.snap | 4 ++--
 packages/rrweb/test/events/style-sheet-rule-events.ts   | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/packages/rrweb/test/__snapshots__/replayer.test.ts.snap b/packages/rrweb/test/__snapshots__/replayer.test.ts.snap
index c4edd6d540..4d4711ba63 100644
--- a/packages/rrweb/test/__snapshots__/replayer.test.ts.snap
+++ b/packages/rrweb/test/__snapshots__/replayer.test.ts.snap
@@ -72,7 +72,7 @@ file-cid-3
 
 .css-added-at-200 { position: fixed; top: 0px; right: 0px; left: 4rem; z-index: 15; flex-shrink: 0; height: 0.25rem; overflow: hidden; background-color: rgb(17, 171, 209); }
 
-.css-added-at-200.alt { height: 0.25rem; background-color: rgb(190, 232, 242); opacity: 0; transition: opacity 0.5s ease 0s; }
+.css-added-at-200.alt { height: 0.25rem; background-color: rgb(190, 232, 242); opacity: 0; transition: opacity 0.5s ease-in 0.1s; }
 
 .css-added-at-1000-deleted-at-2500 { display: flex; flex-direction: column; min-width: 60rem; min-height: 100vh; color: blue; }
 
@@ -152,7 +152,7 @@ file-cid-3
 
 .css-added-at-200 { position: fixed; top: 0px; right: 0px; left: 4rem; z-index: 15; flex-shrink: 0; height: 0.25rem; overflow: hidden; background-color: rgb(17, 171, 209); }
 
-.css-added-at-200.alt { height: 0.25rem; background-color: rgb(190, 232, 242); opacity: 0; transition: opacity 0.5s ease 0s; }
+.css-added-at-200.alt { height: 0.25rem; background-color: rgb(190, 232, 242); opacity: 0; transition: opacity 0.5s ease-in 0.1s; }
 
 .css-added-at-200.alt2 { padding-left: 4rem; }
 "
diff --git a/packages/rrweb/test/events/style-sheet-rule-events.ts b/packages/rrweb/test/events/style-sheet-rule-events.ts
index 577c0caf41..4835961d32 100644
--- a/packages/rrweb/test/events/style-sheet-rule-events.ts
+++ b/packages/rrweb/test/events/style-sheet-rule-events.ts
@@ -65,7 +65,7 @@ const events: eventWithTime[] = [
                     tagName: 'style',
                     attributes: {
                       _cssText:
-                        '.css-added-at-200 { position: fixed; top: 0px; right: 0px; left: 4rem; z-index: 15; flex-shrink: 0; height: 0.25rem; overflow: hidden; background-color: rgb(17, 171, 209); }.css-added-at-200.alt { height: 0.25rem; background-color: rgb(190, 232, 242); opacity: 0; transition: opacity 0.5s ease 0s; }.css-added-at-200.alt2 { padding-left: 4rem; }',
+                        '.css-added-at-200 { position: fixed; top: 0px; right: 0px; left: 4rem; z-index: 15; flex-shrink: 0; height: 0.25rem; overflow: hidden; background-color: rgb(17, 171, 209); }.css-added-at-200.alt { height: 0.25rem; background-color: rgb(190, 232, 242); opacity: 0; transition: opacity 0.5s ease-in 0.1s; }.css-added-at-200.alt2 { padding-left: 4rem; }',
                       'data-emotion': 'css',
                     },
                     childNodes: [

From dec4ad5e0d1d96ba268905be6209823283584b8d Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Fri, 20 Dec 2024 15:22:51 +0000
Subject: [PATCH 13/25] Drop chrome down again as getting overwhelmed with
 issues so will see if this LTS version also solves:

Failed to launch the browser process! [...FATAL:zygote_host_impl_linux.cc(117)] No usable sandbox! Update your kernel ...
---
 .puppeteerrc.cjs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.puppeteerrc.cjs b/.puppeteerrc.cjs
index 261edd3f59..190ce19726 100644
--- a/.puppeteerrc.cjs
+++ b/.puppeteerrc.cjs
@@ -6,5 +6,5 @@ const { join } = require('path');
 module.exports = {
   // Changes the cache location for Puppeteer.
   cacheDirectory: join(__dirname, '.cache', 'puppeteer'),
-  browserRevision: '131.0.6778.204',
+  browserRevision: '126.0.6478.260',
 };

From 7d1e10045b4297eb24370b195f6f12b4c19cd4ed Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Fri, 20 Dec 2024 15:37:12 +0000
Subject: [PATCH 14/25] Bump all puppeteer to match the LTS version of chrome
 we've just specified - going by
 https://github.com/puppeteer/puppeteer/blob/5d72c7d3c3e9c8cd53bcb4b597a0924dced1b5fc/docs/supported-browsers.md

---
 packages/all/package.json                                 | 2 +-
 packages/plugins/rrweb-plugin-console-record/package.json | 2 +-
 packages/record/package.json                              | 2 +-
 packages/replay/package.json                              | 2 +-
 packages/rrdom-nodejs/package.json                        | 2 +-
 packages/rrdom/package.json                               | 2 +-
 packages/rrweb-snapshot/package.json                      | 2 +-
 packages/rrweb/package.json                               | 2 +-
 8 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/packages/all/package.json b/packages/all/package.json
index 6aa8256457..744643c1f8 100644
--- a/packages/all/package.json
+++ b/packages/all/package.json
@@ -49,7 +49,7 @@
     "package.json"
   ],
   "devDependencies": {
-    "puppeteer": "^20.9.0",
+    "puppeteer": "^22.13.1",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/plugins/rrweb-plugin-console-record/package.json b/packages/plugins/rrweb-plugin-console-record/package.json
index c59352c02c..d0918f48cf 100644
--- a/packages/plugins/rrweb-plugin-console-record/package.json
+++ b/packages/plugins/rrweb-plugin-console-record/package.json
@@ -50,7 +50,7 @@
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
-    "puppeteer": "^20.9.0"
+    "puppeteer": "^22.13.1",
   },
   "peerDependencies": {
     "rrweb": "^2.0.0-alpha.18"
diff --git a/packages/record/package.json b/packages/record/package.json
index 0e58cd5a59..e354b3d4e6 100644
--- a/packages/record/package.json
+++ b/packages/record/package.json
@@ -48,7 +48,7 @@
     "package.json"
   ],
   "devDependencies": {
-    "puppeteer": "^20.9.0",
+    "puppeteer": "^22.13.1",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/replay/package.json b/packages/replay/package.json
index 5c9d3f44e9..3c572ddb9e 100644
--- a/packages/replay/package.json
+++ b/packages/replay/package.json
@@ -49,7 +49,7 @@
     "package.json"
   ],
   "devDependencies": {
-    "puppeteer": "^20.9.0",
+    "puppeteer": "^22.13.1",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/rrdom-nodejs/package.json b/packages/rrdom-nodejs/package.json
index 455c317a7c..37bcedb1dc 100644
--- a/packages/rrdom-nodejs/package.json
+++ b/packages/rrdom-nodejs/package.json
@@ -45,7 +45,7 @@
     "@typescript-eslint/parser": "^5.23.0",
     "compare-versions": "^4.1.3",
     "eslint": "^8.15.0",
-    "puppeteer": "^9.1.1",
+    "puppeteer": "^22.13.1",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/rrdom/package.json b/packages/rrdom/package.json
index 9657f17209..2764ad8564 100644
--- a/packages/rrdom/package.json
+++ b/packages/rrdom/package.json
@@ -46,7 +46,7 @@
     "@typescript-eslint/eslint-plugin": "^5.23.0",
     "@typescript-eslint/parser": "^5.23.0",
     "eslint": "^8.15.0",
-    "puppeteer": "^17.1.3",
+    "puppeteer": "^22.13.1",
     "typescript": "^5.4.5",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1"
diff --git a/packages/rrweb-snapshot/package.json b/packages/rrweb-snapshot/package.json
index 4db66f8ec5..92572b2de4 100644
--- a/packages/rrweb-snapshot/package.json
+++ b/packages/rrweb-snapshot/package.json
@@ -59,7 +59,7 @@
     "@types/jsdom": "^20.0.0",
     "@types/node": "^18.15.11",
     "@types/puppeteer": "^5.4.4",
-    "puppeteer": "^20.9.0",
+    "puppeteer": "^22.13.1",
     "ts-node": "^7.0.1",
     "tslib": "^1.9.3",
     "typescript": "^5.4.5",
diff --git a/packages/rrweb/package.json b/packages/rrweb/package.json
index abb46b258e..0be3d4cfe8 100644
--- a/packages/rrweb/package.json
+++ b/packages/rrweb/package.json
@@ -71,7 +71,7 @@
     "ignore-styles": "^5.0.1",
     "inquirer": "^9.0.0",
     "jest-image-snapshot": "^6.2.0",
-    "puppeteer": "^20.9.0",
+    "puppeteer": "^22.13.1",
     "simple-peer-light": "^9.10.0",
     "ts-node": "^10.9.1",
     "tslib": "^2.3.1",

From 0c809e1e662277cc5faa520ab37e006d1e17b92e Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Fri, 20 Dec 2024 16:10:48 +0000
Subject: [PATCH 15/25] Fix json error

---
 packages/plugins/rrweb-plugin-console-record/package.json | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/plugins/rrweb-plugin-console-record/package.json b/packages/plugins/rrweb-plugin-console-record/package.json
index d0918f48cf..16a3d095b6 100644
--- a/packages/plugins/rrweb-plugin-console-record/package.json
+++ b/packages/plugins/rrweb-plugin-console-record/package.json
@@ -50,7 +50,7 @@
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
-    "puppeteer": "^22.13.1",
+    "puppeteer": "^22.13.1"
   },
   "peerDependencies": {
     "rrweb": "^2.0.0-alpha.18"

From 2bae508ea2594a8bb38b396a24e6ead0227f84a7 Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Fri, 20 Dec 2024 18:37:39 +0000
Subject: [PATCH 16/25] The previous version combo didn't install, trying next
 one up

---
 .puppeteerrc.cjs                                          | 2 +-
 packages/all/package.json                                 | 2 +-
 packages/plugins/rrweb-plugin-console-record/package.json | 2 +-
 packages/record/package.json                              | 2 +-
 packages/replay/package.json                              | 2 +-
 packages/rrdom-nodejs/package.json                        | 2 +-
 packages/rrdom/package.json                               | 2 +-
 packages/rrweb-snapshot/package.json                      | 2 +-
 packages/rrweb/package.json                               | 2 +-
 9 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/.puppeteerrc.cjs b/.puppeteerrc.cjs
index 190ce19726..8706d491ef 100644
--- a/.puppeteerrc.cjs
+++ b/.puppeteerrc.cjs
@@ -6,5 +6,5 @@ const { join } = require('path');
 module.exports = {
   // Changes the cache location for Puppeteer.
   cacheDirectory: join(__dirname, '.cache', 'puppeteer'),
-  browserRevision: '126.0.6478.260',
+  browserRevision: '127.0.6533.88',
 };
diff --git a/packages/all/package.json b/packages/all/package.json
index 744643c1f8..7d10fa4dfa 100644
--- a/packages/all/package.json
+++ b/packages/all/package.json
@@ -49,7 +49,7 @@
     "package.json"
   ],
   "devDependencies": {
-    "puppeteer": "^22.13.1",
+    "puppeteer": "^23.0.0",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/plugins/rrweb-plugin-console-record/package.json b/packages/plugins/rrweb-plugin-console-record/package.json
index 16a3d095b6..44a3f9c635 100644
--- a/packages/plugins/rrweb-plugin-console-record/package.json
+++ b/packages/plugins/rrweb-plugin-console-record/package.json
@@ -50,7 +50,7 @@
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
-    "puppeteer": "^22.13.1"
+    "puppeteer": "^23.0.0"
   },
   "peerDependencies": {
     "rrweb": "^2.0.0-alpha.18"
diff --git a/packages/record/package.json b/packages/record/package.json
index e354b3d4e6..808c74f938 100644
--- a/packages/record/package.json
+++ b/packages/record/package.json
@@ -48,7 +48,7 @@
     "package.json"
   ],
   "devDependencies": {
-    "puppeteer": "^22.13.1",
+    "puppeteer": "^23.0.0",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/replay/package.json b/packages/replay/package.json
index 3c572ddb9e..5b68860f1c 100644
--- a/packages/replay/package.json
+++ b/packages/replay/package.json
@@ -49,7 +49,7 @@
     "package.json"
   ],
   "devDependencies": {
-    "puppeteer": "^22.13.1",
+    "puppeteer": "^23.0.0",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/rrdom-nodejs/package.json b/packages/rrdom-nodejs/package.json
index 37bcedb1dc..cd862aeb97 100644
--- a/packages/rrdom-nodejs/package.json
+++ b/packages/rrdom-nodejs/package.json
@@ -45,7 +45,7 @@
     "@typescript-eslint/parser": "^5.23.0",
     "compare-versions": "^4.1.3",
     "eslint": "^8.15.0",
-    "puppeteer": "^22.13.1",
+    "puppeteer": "^23.0.0",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/rrdom/package.json b/packages/rrdom/package.json
index 2764ad8564..05bfb72a83 100644
--- a/packages/rrdom/package.json
+++ b/packages/rrdom/package.json
@@ -46,7 +46,7 @@
     "@typescript-eslint/eslint-plugin": "^5.23.0",
     "@typescript-eslint/parser": "^5.23.0",
     "eslint": "^8.15.0",
-    "puppeteer": "^22.13.1",
+    "puppeteer": "^23.0.0",
     "typescript": "^5.4.5",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1"
diff --git a/packages/rrweb-snapshot/package.json b/packages/rrweb-snapshot/package.json
index 92572b2de4..3e463b648a 100644
--- a/packages/rrweb-snapshot/package.json
+++ b/packages/rrweb-snapshot/package.json
@@ -59,7 +59,7 @@
     "@types/jsdom": "^20.0.0",
     "@types/node": "^18.15.11",
     "@types/puppeteer": "^5.4.4",
-    "puppeteer": "^22.13.1",
+    "puppeteer": "^23.0.0",
     "ts-node": "^7.0.1",
     "tslib": "^1.9.3",
     "typescript": "^5.4.5",
diff --git a/packages/rrweb/package.json b/packages/rrweb/package.json
index 0be3d4cfe8..1998b5b94d 100644
--- a/packages/rrweb/package.json
+++ b/packages/rrweb/package.json
@@ -71,7 +71,7 @@
     "ignore-styles": "^5.0.1",
     "inquirer": "^9.0.0",
     "jest-image-snapshot": "^6.2.0",
-    "puppeteer": "^22.13.1",
+    "puppeteer": "^23.0.0",
     "simple-peer-light": "^9.10.0",
     "ts-node": "^10.9.1",
     "tslib": "^2.3.1",

From c43d695cff31c2cb9ca35558a01da5fe91a5bbad Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Sun, 22 Dec 2024 18:33:19 +0000
Subject: [PATCH 17/25] Still getting 'zygote_host_impl_linux.cc(128)] No
 usable sandbox!' so iterating through versions

---
 .puppeteerrc.cjs                                          | 2 +-
 packages/all/package.json                                 | 2 +-
 packages/plugins/rrweb-plugin-console-record/package.json | 2 +-
 packages/record/package.json                              | 2 +-
 packages/replay/package.json                              | 2 +-
 packages/rrdom-nodejs/package.json                        | 2 +-
 packages/rrdom/package.json                               | 2 +-
 packages/rrweb-snapshot/package.json                      | 2 +-
 packages/rrweb/package.json                               | 2 +-
 9 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/.puppeteerrc.cjs b/.puppeteerrc.cjs
index 8706d491ef..992b97ecc6 100644
--- a/.puppeteerrc.cjs
+++ b/.puppeteerrc.cjs
@@ -6,5 +6,5 @@ const { join } = require('path');
 module.exports = {
   // Changes the cache location for Puppeteer.
   cacheDirectory: join(__dirname, '.cache', 'puppeteer'),
-  browserRevision: '127.0.6533.88',
+  browserRevision: '128.0.6613.119',
 };
diff --git a/packages/all/package.json b/packages/all/package.json
index 7d10fa4dfa..56f389c525 100644
--- a/packages/all/package.json
+++ b/packages/all/package.json
@@ -49,7 +49,7 @@
     "package.json"
   ],
   "devDependencies": {
-    "puppeteer": "^23.0.0",
+    "puppeteer": "^23.3.0",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/plugins/rrweb-plugin-console-record/package.json b/packages/plugins/rrweb-plugin-console-record/package.json
index 44a3f9c635..68e513f14f 100644
--- a/packages/plugins/rrweb-plugin-console-record/package.json
+++ b/packages/plugins/rrweb-plugin-console-record/package.json
@@ -50,7 +50,7 @@
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
-    "puppeteer": "^23.0.0"
+    "puppeteer": "^23.3.0"
   },
   "peerDependencies": {
     "rrweb": "^2.0.0-alpha.18"
diff --git a/packages/record/package.json b/packages/record/package.json
index 808c74f938..b85fbd6828 100644
--- a/packages/record/package.json
+++ b/packages/record/package.json
@@ -48,7 +48,7 @@
     "package.json"
   ],
   "devDependencies": {
-    "puppeteer": "^23.0.0",
+    "puppeteer": "^23.3.0",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/replay/package.json b/packages/replay/package.json
index 5b68860f1c..0033a8c1c8 100644
--- a/packages/replay/package.json
+++ b/packages/replay/package.json
@@ -49,7 +49,7 @@
     "package.json"
   ],
   "devDependencies": {
-    "puppeteer": "^23.0.0",
+    "puppeteer": "^23.3.0",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/rrdom-nodejs/package.json b/packages/rrdom-nodejs/package.json
index cd862aeb97..50198049cf 100644
--- a/packages/rrdom-nodejs/package.json
+++ b/packages/rrdom-nodejs/package.json
@@ -45,7 +45,7 @@
     "@typescript-eslint/parser": "^5.23.0",
     "compare-versions": "^4.1.3",
     "eslint": "^8.15.0",
-    "puppeteer": "^23.0.0",
+    "puppeteer": "^23.3.0",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/rrdom/package.json b/packages/rrdom/package.json
index 05bfb72a83..01d7885482 100644
--- a/packages/rrdom/package.json
+++ b/packages/rrdom/package.json
@@ -46,7 +46,7 @@
     "@typescript-eslint/eslint-plugin": "^5.23.0",
     "@typescript-eslint/parser": "^5.23.0",
     "eslint": "^8.15.0",
-    "puppeteer": "^23.0.0",
+    "puppeteer": "^23.3.0",
     "typescript": "^5.4.5",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1"
diff --git a/packages/rrweb-snapshot/package.json b/packages/rrweb-snapshot/package.json
index 3e463b648a..f86faa8b1f 100644
--- a/packages/rrweb-snapshot/package.json
+++ b/packages/rrweb-snapshot/package.json
@@ -59,7 +59,7 @@
     "@types/jsdom": "^20.0.0",
     "@types/node": "^18.15.11",
     "@types/puppeteer": "^5.4.4",
-    "puppeteer": "^23.0.0",
+    "puppeteer": "^23.3.0",
     "ts-node": "^7.0.1",
     "tslib": "^1.9.3",
     "typescript": "^5.4.5",
diff --git a/packages/rrweb/package.json b/packages/rrweb/package.json
index 1998b5b94d..24d87def4c 100644
--- a/packages/rrweb/package.json
+++ b/packages/rrweb/package.json
@@ -71,7 +71,7 @@
     "ignore-styles": "^5.0.1",
     "inquirer": "^9.0.0",
     "jest-image-snapshot": "^6.2.0",
-    "puppeteer": "^23.0.0",
+    "puppeteer": "^23.3.0",
     "simple-peer-light": "^9.10.0",
     "ts-node": "^10.9.1",
     "tslib": "^2.3.1",

From e16e2ac20bf1b4338e4bf351a72ed5b800a5f040 Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Sun, 22 Dec 2024 18:38:14 +0000
Subject: [PATCH 18/25] Keep searching through
 https://github.com/puppeteer/puppeteer/blob/5d72c7d3c3e9c8cd53bcb4b597a0924dced1b5fc/docs/supported-browsers.md

---
 .puppeteerrc.cjs                                          | 2 +-
 packages/all/package.json                                 | 2 +-
 packages/plugins/rrweb-plugin-console-record/package.json | 2 +-
 packages/record/package.json                              | 2 +-
 packages/replay/package.json                              | 2 +-
 packages/rrdom-nodejs/package.json                        | 2 +-
 packages/rrdom/package.json                               | 2 +-
 packages/rrweb-snapshot/package.json                      | 2 +-
 packages/rrweb/package.json                               | 2 +-
 9 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/.puppeteerrc.cjs b/.puppeteerrc.cjs
index 992b97ecc6..ec5a806155 100644
--- a/.puppeteerrc.cjs
+++ b/.puppeteerrc.cjs
@@ -6,5 +6,5 @@ const { join } = require('path');
 module.exports = {
   // Changes the cache location for Puppeteer.
   cacheDirectory: join(__dirname, '.cache', 'puppeteer'),
-  browserRevision: '128.0.6613.119',
+  browserRevision: '130.0.6723.58',
 };
diff --git a/packages/all/package.json b/packages/all/package.json
index 56f389c525..74e43e5499 100644
--- a/packages/all/package.json
+++ b/packages/all/package.json
@@ -49,7 +49,7 @@
     "package.json"
   ],
   "devDependencies": {
-    "puppeteer": "^23.3.0",
+    "puppeteer": "^23.6.0",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/plugins/rrweb-plugin-console-record/package.json b/packages/plugins/rrweb-plugin-console-record/package.json
index 68e513f14f..e10e45d663 100644
--- a/packages/plugins/rrweb-plugin-console-record/package.json
+++ b/packages/plugins/rrweb-plugin-console-record/package.json
@@ -50,7 +50,7 @@
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
-    "puppeteer": "^23.3.0"
+    "puppeteer": "^23.6.0"
   },
   "peerDependencies": {
     "rrweb": "^2.0.0-alpha.18"
diff --git a/packages/record/package.json b/packages/record/package.json
index b85fbd6828..1828189d26 100644
--- a/packages/record/package.json
+++ b/packages/record/package.json
@@ -48,7 +48,7 @@
     "package.json"
   ],
   "devDependencies": {
-    "puppeteer": "^23.3.0",
+    "puppeteer": "^23.6.0",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/replay/package.json b/packages/replay/package.json
index 0033a8c1c8..a4f0579d81 100644
--- a/packages/replay/package.json
+++ b/packages/replay/package.json
@@ -49,7 +49,7 @@
     "package.json"
   ],
   "devDependencies": {
-    "puppeteer": "^23.3.0",
+    "puppeteer": "^23.6.0",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/rrdom-nodejs/package.json b/packages/rrdom-nodejs/package.json
index 50198049cf..1fa1484346 100644
--- a/packages/rrdom-nodejs/package.json
+++ b/packages/rrdom-nodejs/package.json
@@ -45,7 +45,7 @@
     "@typescript-eslint/parser": "^5.23.0",
     "compare-versions": "^4.1.3",
     "eslint": "^8.15.0",
-    "puppeteer": "^23.3.0",
+    "puppeteer": "^23.6.0",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/rrdom/package.json b/packages/rrdom/package.json
index 01d7885482..c17e7a330d 100644
--- a/packages/rrdom/package.json
+++ b/packages/rrdom/package.json
@@ -46,7 +46,7 @@
     "@typescript-eslint/eslint-plugin": "^5.23.0",
     "@typescript-eslint/parser": "^5.23.0",
     "eslint": "^8.15.0",
-    "puppeteer": "^23.3.0",
+    "puppeteer": "^23.6.0",
     "typescript": "^5.4.5",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1"
diff --git a/packages/rrweb-snapshot/package.json b/packages/rrweb-snapshot/package.json
index f86faa8b1f..8590aa8098 100644
--- a/packages/rrweb-snapshot/package.json
+++ b/packages/rrweb-snapshot/package.json
@@ -59,7 +59,7 @@
     "@types/jsdom": "^20.0.0",
     "@types/node": "^18.15.11",
     "@types/puppeteer": "^5.4.4",
-    "puppeteer": "^23.3.0",
+    "puppeteer": "^23.6.0",
     "ts-node": "^7.0.1",
     "tslib": "^1.9.3",
     "typescript": "^5.4.5",
diff --git a/packages/rrweb/package.json b/packages/rrweb/package.json
index 24d87def4c..ede0bd9b36 100644
--- a/packages/rrweb/package.json
+++ b/packages/rrweb/package.json
@@ -71,7 +71,7 @@
     "ignore-styles": "^5.0.1",
     "inquirer": "^9.0.0",
     "jest-image-snapshot": "^6.2.0",
-    "puppeteer": "^23.3.0",
+    "puppeteer": "^23.6.0",
     "simple-peer-light": "^9.10.0",
     "ts-node": "^10.9.1",
     "tslib": "^2.3.1",

From c0bf8f8cb245d527f2d0a9a79dced1f5f55554ea Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Sun, 22 Dec 2024 18:50:24 +0000
Subject: [PATCH 19/25] Following
 https://chromium.googlesource.com/chromium/src/+/main/docs/security/apparmor-userns-restrictions.md

---
 packages/rrweb-snapshot/test/integration.test.ts | 1 +
 1 file changed, 1 insertion(+)

diff --git a/packages/rrweb-snapshot/test/integration.test.ts b/packages/rrweb-snapshot/test/integration.test.ts
index 9fa04baf65..a0c8581feb 100644
--- a/packages/rrweb-snapshot/test/integration.test.ts
+++ b/packages/rrweb-snapshot/test/integration.test.ts
@@ -103,6 +103,7 @@ describe('integration tests', function (this: ISuite) {
     serverURL = getServerURL(server);
     browser = await puppeteer.launch({
       // headless: false,
+      args: ['--no-sandbox'],
     });
 
     code = fs.readFileSync(

From 417a83181f5f698af8cb43b39d05348aafdd0780 Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Sun, 22 Dec 2024 19:22:24 +0000
Subject: [PATCH 20/25] Continuing with
 https://chromium.googlesource.com/chromium/src/+/main/docs/security/apparmor-userns-restrictions.md

---
 packages/rrweb-snapshot/test/integration.test.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/rrweb-snapshot/test/integration.test.ts b/packages/rrweb-snapshot/test/integration.test.ts
index a0c8581feb..b93934c696 100644
--- a/packages/rrweb-snapshot/test/integration.test.ts
+++ b/packages/rrweb-snapshot/test/integration.test.ts
@@ -103,7 +103,7 @@ describe('integration tests', function (this: ISuite) {
     serverURL = getServerURL(server);
     browser = await puppeteer.launch({
       // headless: false,
-      args: ['--no-sandbox'],
+      args: ['--no-sandbox', '--disable-setuid-sandbox'],
     });
 
     code = fs.readFileSync(

From a72a28ec25305c417dd86077673e85ae64fd68f5 Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Mon, 30 Dec 2024 11:32:17 +0000
Subject: [PATCH 21/25] Another bump to find minimum version to remove the 'No
 Usable Sandbox' error

---
 .puppeteerrc.cjs                                          | 2 +-
 packages/all/package.json                                 | 2 +-
 packages/plugins/rrweb-plugin-console-record/package.json | 2 +-
 packages/record/package.json                              | 2 +-
 packages/replay/package.json                              | 2 +-
 packages/rrdom-nodejs/package.json                        | 2 +-
 packages/rrdom/package.json                               | 2 +-
 packages/rrweb-snapshot/package.json                      | 2 +-
 packages/rrweb/package.json                               | 2 +-
 9 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/.puppeteerrc.cjs b/.puppeteerrc.cjs
index ec5a806155..148baf21e3 100644
--- a/.puppeteerrc.cjs
+++ b/.puppeteerrc.cjs
@@ -6,5 +6,5 @@ const { join } = require('path');
 module.exports = {
   // Changes the cache location for Puppeteer.
   cacheDirectory: join(__dirname, '.cache', 'puppeteer'),
-  browserRevision: '130.0.6723.58',
+  browserRevision: '131.0.6778.69',
 };
diff --git a/packages/all/package.json b/packages/all/package.json
index 74e43e5499..d01ec4b385 100644
--- a/packages/all/package.json
+++ b/packages/all/package.json
@@ -49,7 +49,7 @@
     "package.json"
   ],
   "devDependencies": {
-    "puppeteer": "^23.6.0",
+    "puppeteer": "^23.8.0",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/plugins/rrweb-plugin-console-record/package.json b/packages/plugins/rrweb-plugin-console-record/package.json
index e10e45d663..56cdfba8a0 100644
--- a/packages/plugins/rrweb-plugin-console-record/package.json
+++ b/packages/plugins/rrweb-plugin-console-record/package.json
@@ -50,7 +50,7 @@
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
-    "puppeteer": "^23.6.0"
+    "puppeteer": "^23.8.0"
   },
   "peerDependencies": {
     "rrweb": "^2.0.0-alpha.18"
diff --git a/packages/record/package.json b/packages/record/package.json
index 1828189d26..85b73c7dd8 100644
--- a/packages/record/package.json
+++ b/packages/record/package.json
@@ -48,7 +48,7 @@
     "package.json"
   ],
   "devDependencies": {
-    "puppeteer": "^23.6.0",
+    "puppeteer": "^23.8.0",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/replay/package.json b/packages/replay/package.json
index a4f0579d81..33080c0034 100644
--- a/packages/replay/package.json
+++ b/packages/replay/package.json
@@ -49,7 +49,7 @@
     "package.json"
   ],
   "devDependencies": {
-    "puppeteer": "^23.6.0",
+    "puppeteer": "^23.8.0",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/rrdom-nodejs/package.json b/packages/rrdom-nodejs/package.json
index 1fa1484346..118a76525d 100644
--- a/packages/rrdom-nodejs/package.json
+++ b/packages/rrdom-nodejs/package.json
@@ -45,7 +45,7 @@
     "@typescript-eslint/parser": "^5.23.0",
     "compare-versions": "^4.1.3",
     "eslint": "^8.15.0",
-    "puppeteer": "^23.6.0",
+    "puppeteer": "^23.8.0",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/rrdom/package.json b/packages/rrdom/package.json
index c17e7a330d..7c39244fbb 100644
--- a/packages/rrdom/package.json
+++ b/packages/rrdom/package.json
@@ -46,7 +46,7 @@
     "@typescript-eslint/eslint-plugin": "^5.23.0",
     "@typescript-eslint/parser": "^5.23.0",
     "eslint": "^8.15.0",
-    "puppeteer": "^23.6.0",
+    "puppeteer": "^23.8.0",
     "typescript": "^5.4.5",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1"
diff --git a/packages/rrweb-snapshot/package.json b/packages/rrweb-snapshot/package.json
index 8590aa8098..27738c7c7a 100644
--- a/packages/rrweb-snapshot/package.json
+++ b/packages/rrweb-snapshot/package.json
@@ -59,7 +59,7 @@
     "@types/jsdom": "^20.0.0",
     "@types/node": "^18.15.11",
     "@types/puppeteer": "^5.4.4",
-    "puppeteer": "^23.6.0",
+    "puppeteer": "^23.8.0",
     "ts-node": "^7.0.1",
     "tslib": "^1.9.3",
     "typescript": "^5.4.5",
diff --git a/packages/rrweb/package.json b/packages/rrweb/package.json
index ede0bd9b36..248cc2d832 100644
--- a/packages/rrweb/package.json
+++ b/packages/rrweb/package.json
@@ -71,7 +71,7 @@
     "ignore-styles": "^5.0.1",
     "inquirer": "^9.0.0",
     "jest-image-snapshot": "^6.2.0",
-    "puppeteer": "^23.6.0",
+    "puppeteer": "^23.8.0",
     "simple-peer-light": "^9.10.0",
     "ts-node": "^10.9.1",
     "tslib": "^2.3.1",

From a677fb6b5d508c34ea974a9e921a29586eb65752 Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Mon, 30 Dec 2024 12:20:14 +0000
Subject: [PATCH 22/25] Final bump to top of
 https://github.com/puppeteer/puppeteer/blob/5d72c7d3c3e9c8cd53bcb4b597a0924dced1b5fc/docs/supported-browsers.md

---
 .puppeteerrc.cjs                                          | 2 +-
 packages/all/package.json                                 | 2 +-
 packages/plugins/rrweb-plugin-console-record/package.json | 2 +-
 packages/record/package.json                              | 2 +-
 packages/replay/package.json                              | 2 +-
 packages/rrdom-nodejs/package.json                        | 2 +-
 packages/rrdom/package.json                               | 2 +-
 packages/rrweb-snapshot/package.json                      | 2 +-
 packages/rrweb/package.json                               | 2 +-
 9 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/.puppeteerrc.cjs b/.puppeteerrc.cjs
index 148baf21e3..e075b5e458 100644
--- a/.puppeteerrc.cjs
+++ b/.puppeteerrc.cjs
@@ -6,5 +6,5 @@ const { join } = require('path');
 module.exports = {
   // Changes the cache location for Puppeteer.
   cacheDirectory: join(__dirname, '.cache', 'puppeteer'),
-  browserRevision: '131.0.6778.69',
+  browserRevision: '131.0.6778.87',
 };
diff --git a/packages/all/package.json b/packages/all/package.json
index d01ec4b385..c6d494722b 100644
--- a/packages/all/package.json
+++ b/packages/all/package.json
@@ -49,7 +49,7 @@
     "package.json"
   ],
   "devDependencies": {
-    "puppeteer": "^23.8.0",
+    "puppeteer": "^23.10.1",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/plugins/rrweb-plugin-console-record/package.json b/packages/plugins/rrweb-plugin-console-record/package.json
index 56cdfba8a0..952326df7e 100644
--- a/packages/plugins/rrweb-plugin-console-record/package.json
+++ b/packages/plugins/rrweb-plugin-console-record/package.json
@@ -50,7 +50,7 @@
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
-    "puppeteer": "^23.8.0"
+    "puppeteer": "^23.10.1"
   },
   "peerDependencies": {
     "rrweb": "^2.0.0-alpha.18"
diff --git a/packages/record/package.json b/packages/record/package.json
index 85b73c7dd8..076578178e 100644
--- a/packages/record/package.json
+++ b/packages/record/package.json
@@ -48,7 +48,7 @@
     "package.json"
   ],
   "devDependencies": {
-    "puppeteer": "^23.8.0",
+    "puppeteer": "^23.10.1",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/replay/package.json b/packages/replay/package.json
index 33080c0034..204a2d38dc 100644
--- a/packages/replay/package.json
+++ b/packages/replay/package.json
@@ -49,7 +49,7 @@
     "package.json"
   ],
   "devDependencies": {
-    "puppeteer": "^23.8.0",
+    "puppeteer": "^23.10.1",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/rrdom-nodejs/package.json b/packages/rrdom-nodejs/package.json
index 118a76525d..53f749a5ab 100644
--- a/packages/rrdom-nodejs/package.json
+++ b/packages/rrdom-nodejs/package.json
@@ -45,7 +45,7 @@
     "@typescript-eslint/parser": "^5.23.0",
     "compare-versions": "^4.1.3",
     "eslint": "^8.15.0",
-    "puppeteer": "^23.8.0",
+    "puppeteer": "^23.10.1",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/rrdom/package.json b/packages/rrdom/package.json
index 7c39244fbb..6a3c66a59b 100644
--- a/packages/rrdom/package.json
+++ b/packages/rrdom/package.json
@@ -46,7 +46,7 @@
     "@typescript-eslint/eslint-plugin": "^5.23.0",
     "@typescript-eslint/parser": "^5.23.0",
     "eslint": "^8.15.0",
-    "puppeteer": "^23.8.0",
+    "puppeteer": "^23.10.1",
     "typescript": "^5.4.5",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1"
diff --git a/packages/rrweb-snapshot/package.json b/packages/rrweb-snapshot/package.json
index 27738c7c7a..19d65be269 100644
--- a/packages/rrweb-snapshot/package.json
+++ b/packages/rrweb-snapshot/package.json
@@ -59,7 +59,7 @@
     "@types/jsdom": "^20.0.0",
     "@types/node": "^18.15.11",
     "@types/puppeteer": "^5.4.4",
-    "puppeteer": "^23.8.0",
+    "puppeteer": "^23.10.1",
     "ts-node": "^7.0.1",
     "tslib": "^1.9.3",
     "typescript": "^5.4.5",
diff --git a/packages/rrweb/package.json b/packages/rrweb/package.json
index 248cc2d832..e43c52a566 100644
--- a/packages/rrweb/package.json
+++ b/packages/rrweb/package.json
@@ -71,7 +71,7 @@
     "ignore-styles": "^5.0.1",
     "inquirer": "^9.0.0",
     "jest-image-snapshot": "^6.2.0",
-    "puppeteer": "^23.8.0",
+    "puppeteer": "^23.10.1",
     "simple-peer-light": "^9.10.0",
     "ts-node": "^10.9.1",
     "tslib": "^2.3.1",

From 9086b94fb78f47a0fac9008247f87c33c8b0234f Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Mon, 30 Dec 2024 12:30:31 +0000
Subject: [PATCH 23/25] Further bump of Chrome as issue still isn't solved -
 this previously seemed to solve things

---
 .puppeteerrc.cjs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.puppeteerrc.cjs b/.puppeteerrc.cjs
index e075b5e458..261edd3f59 100644
--- a/.puppeteerrc.cjs
+++ b/.puppeteerrc.cjs
@@ -6,5 +6,5 @@ const { join } = require('path');
 module.exports = {
   // Changes the cache location for Puppeteer.
   cacheDirectory: join(__dirname, '.cache', 'puppeteer'),
-  browserRevision: '131.0.6778.87',
+  browserRevision: '131.0.6778.204',
 };

From 724a27b255bf2e9920e510b531605fa49d1340a0 Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Thu, 2 Jan 2025 14:03:50 +0000
Subject: [PATCH 24/25] Revert all puppeteer and chrome related changes as the
 problem was fixed with `runs-on: ubuntu-22.04`

---
 .puppeteerrc.cjs                                          | 2 +-
 packages/all/package.json                                 | 2 +-
 packages/plugins/rrweb-plugin-console-record/package.json | 2 +-
 packages/record/package.json                              | 2 +-
 packages/replay/package.json                              | 2 +-
 packages/rrdom-nodejs/package.json                        | 2 +-
 packages/rrdom/package.json                               | 2 +-
 packages/rrweb-snapshot/package.json                      | 2 +-
 packages/rrweb-snapshot/test/integration.test.ts          | 1 -
 packages/rrweb/package.json                               | 2 +-
 10 files changed, 9 insertions(+), 10 deletions(-)

diff --git a/.puppeteerrc.cjs b/.puppeteerrc.cjs
index 261edd3f59..0831229fc5 100644
--- a/.puppeteerrc.cjs
+++ b/.puppeteerrc.cjs
@@ -6,5 +6,5 @@ const { join } = require('path');
 module.exports = {
   // Changes the cache location for Puppeteer.
   cacheDirectory: join(__dirname, '.cache', 'puppeteer'),
-  browserRevision: '131.0.6778.204',
+  browserRevision: '115.0.5763.0',
 };
diff --git a/packages/all/package.json b/packages/all/package.json
index c6d494722b..6aa8256457 100644
--- a/packages/all/package.json
+++ b/packages/all/package.json
@@ -49,7 +49,7 @@
     "package.json"
   ],
   "devDependencies": {
-    "puppeteer": "^23.10.1",
+    "puppeteer": "^20.9.0",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/plugins/rrweb-plugin-console-record/package.json b/packages/plugins/rrweb-plugin-console-record/package.json
index 952326df7e..c59352c02c 100644
--- a/packages/plugins/rrweb-plugin-console-record/package.json
+++ b/packages/plugins/rrweb-plugin-console-record/package.json
@@ -50,7 +50,7 @@
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
-    "puppeteer": "^23.10.1"
+    "puppeteer": "^20.9.0"
   },
   "peerDependencies": {
     "rrweb": "^2.0.0-alpha.18"
diff --git a/packages/record/package.json b/packages/record/package.json
index 076578178e..0e58cd5a59 100644
--- a/packages/record/package.json
+++ b/packages/record/package.json
@@ -48,7 +48,7 @@
     "package.json"
   ],
   "devDependencies": {
-    "puppeteer": "^23.10.1",
+    "puppeteer": "^20.9.0",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/replay/package.json b/packages/replay/package.json
index 204a2d38dc..5c9d3f44e9 100644
--- a/packages/replay/package.json
+++ b/packages/replay/package.json
@@ -49,7 +49,7 @@
     "package.json"
   ],
   "devDependencies": {
-    "puppeteer": "^23.10.1",
+    "puppeteer": "^20.9.0",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/rrdom-nodejs/package.json b/packages/rrdom-nodejs/package.json
index 53f749a5ab..455c317a7c 100644
--- a/packages/rrdom-nodejs/package.json
+++ b/packages/rrdom-nodejs/package.json
@@ -45,7 +45,7 @@
     "@typescript-eslint/parser": "^5.23.0",
     "compare-versions": "^4.1.3",
     "eslint": "^8.15.0",
-    "puppeteer": "^23.10.1",
+    "puppeteer": "^9.1.1",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1",
     "vitest": "^1.4.0",
diff --git a/packages/rrdom/package.json b/packages/rrdom/package.json
index 6a3c66a59b..9657f17209 100644
--- a/packages/rrdom/package.json
+++ b/packages/rrdom/package.json
@@ -46,7 +46,7 @@
     "@typescript-eslint/eslint-plugin": "^5.23.0",
     "@typescript-eslint/parser": "^5.23.0",
     "eslint": "^8.15.0",
-    "puppeteer": "^23.10.1",
+    "puppeteer": "^17.1.3",
     "typescript": "^5.4.5",
     "vite": "^5.3.1",
     "vite-plugin-dts": "^3.9.1"
diff --git a/packages/rrweb-snapshot/package.json b/packages/rrweb-snapshot/package.json
index 19d65be269..299384b2de 100644
--- a/packages/rrweb-snapshot/package.json
+++ b/packages/rrweb-snapshot/package.json
@@ -59,7 +59,7 @@
     "@types/jsdom": "^20.0.0",
     "@types/node": "^18.15.11",
     "@types/puppeteer": "^5.4.4",
-    "puppeteer": "^23.10.1",
+    "puppeteer": "^17.1.3",
     "ts-node": "^7.0.1",
     "tslib": "^1.9.3",
     "typescript": "^5.4.5",
diff --git a/packages/rrweb-snapshot/test/integration.test.ts b/packages/rrweb-snapshot/test/integration.test.ts
index b93934c696..9fa04baf65 100644
--- a/packages/rrweb-snapshot/test/integration.test.ts
+++ b/packages/rrweb-snapshot/test/integration.test.ts
@@ -103,7 +103,6 @@ describe('integration tests', function (this: ISuite) {
     serverURL = getServerURL(server);
     browser = await puppeteer.launch({
       // headless: false,
-      args: ['--no-sandbox', '--disable-setuid-sandbox'],
     });
 
     code = fs.readFileSync(
diff --git a/packages/rrweb/package.json b/packages/rrweb/package.json
index e43c52a566..abb46b258e 100644
--- a/packages/rrweb/package.json
+++ b/packages/rrweb/package.json
@@ -71,7 +71,7 @@
     "ignore-styles": "^5.0.1",
     "inquirer": "^9.0.0",
     "jest-image-snapshot": "^6.2.0",
-    "puppeteer": "^23.10.1",
+    "puppeteer": "^20.9.0",
     "simple-peer-light": "^9.10.0",
     "ts-node": "^10.9.1",
     "tslib": "^2.3.1",

From 0fbf355bdfce7af8ca60f81b6a7fab70ddd40be6 Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Sat, 4 Jan 2025 15:27:04 +0000
Subject: [PATCH 25/25] I keep accidentally forgetting camelCase

---
 packages/rrweb-snapshot/src/utils.ts | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/packages/rrweb-snapshot/src/utils.ts b/packages/rrweb-snapshot/src/utils.ts
index 7eff11eb4b..e1ae7a3ad4 100644
--- a/packages/rrweb-snapshot/src/utils.ts
+++ b/packages/rrweb-snapshot/src/utils.ts
@@ -466,7 +466,7 @@ export function splitCssText(
 ): string[] {
   const childNodes = Array.from(style.childNodes);
   const splits: string[] = [];
-  let iter_limit = 0;
+  let iterLimit = 0;
   if (childNodes.length > 1 && cssText && typeof cssText === 'string') {
     let cssTextNorm = normalizeCssString(cssText);
     const normFactor = cssTextNorm.length / cssText.length;
@@ -507,8 +507,8 @@ export function splitCssText(
             // find the split point in the original text
             let k = Math.floor(splitNorm / normFactor);
             for (; k > 0 && k < cssText.length; ) {
-              iter_limit += 1;
-              if (iter_limit > 50 * childNodes.length) {
+              iterLimit += 1;
+              if (iterLimit > 50 * childNodes.length) {
                 // quit for performance purposes
                 splits.push(cssText);
                 return splits;