From 560f28ecf04f8498071058f294cf75cd9c98bf58 Mon Sep 17 00:00:00 2001
From: Cy Zheng <congyao119@gmail.com>
Date: Tue, 5 Nov 2024 17:16:43 +0100
Subject: [PATCH] =?UTF-8?q?=F0=9F=91=B7=E2=80=8D=E2=99=80=EF=B8=8F=20[RUM-?=
 =?UTF-8?q?6424]=20Change=20webpack=20chunk=20config=20to=20speed=20up=20l?=
 =?UTF-8?q?ocal=20unit=20tests=20(#3057)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* 👷‍♀️ Remove split Chunk false for webpack

* 👷‍♀️ only remove splitChunk with scope  command

* Remove splitchunk by default locally

* remove `splitChunks` everywhere

Somehow, even if we exclude `developer-extension` sources in
browser-stack karma configuration, they still end up in the bundle. In
those sources, we used a Regexp syntax unsupported in older browsers.

This commit work around this issue by using the `new RegExp` constructor
instead of the regexp syntax.

---------

Co-authored-by: Benoît Zugmeyer <benoit.zugmeyer@datadoghq.com>
---
 developer-extension/src/panel/hooks/useEvents/eventFilters.ts | 2 +-
 test/unit/karma.base.conf.js                                  | 1 -
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/developer-extension/src/panel/hooks/useEvents/eventFilters.ts b/developer-extension/src/panel/hooks/useEvents/eventFilters.ts
index 74f9fd91e6..f81ccec225 100644
--- a/developer-extension/src/panel/hooks/useEvents/eventFilters.ts
+++ b/developer-extension/src/panel/hooks/useEvents/eventFilters.ts
@@ -81,7 +81,7 @@ function filterOutdatedVersions(events: SdkEvent[]): SdkEvent[] {
 
 export function parseQuery(query: string) {
   const queryParts = query
-    .split(/(?<!\\)\s/g) // Hack it to escape whitespace with backslashes
+    .split(new RegExp('(?<!\\\\)\\s', 'g')) // Hack it to escape whitespace with backslashes. Use a regex constructor to avoid syntax errors in older browsers
     .filter((queryPart) => queryPart)
     .map((queryPart) => queryPart.split(':'))
 
diff --git a/test/unit/karma.base.conf.js b/test/unit/karma.base.conf.js
index 6fdccfc90a..bb5109bd5d 100644
--- a/test/unit/karma.base.conf.js
+++ b/test/unit/karma.base.conf.js
@@ -56,7 +56,6 @@ module.exports = {
       // with all dependencies shared.  Our test suite does not support sharing dependencies, each
       // spec bundle should include its own copy of dependencies.
       runtimeChunk: false,
-      splitChunks: false,
     },
     ignoreWarnings: [
       // we will see warnings about missing exports in some files