fix: LEAP-509: Optimize regular expression for improved performance (H…

…umanSignal#5310) * fix: LEAP-509: Optimize regular expression for improved performance * ci: Build frontend Workflow run: https://github.com/HumanSignal/label-studio/actions/runs/7585393196 * fix regex to match the scope * ci: Build frontend Workflow run: https://github.com/HumanSignal/label-studio/actions/runs/7586535924 * Update web/libs/editor/src/utils/data.js Co-authored-by: hlomzik <[email protected]> * ci: Build frontend Workflow run: https://github.com/HumanSignal/label-studio/actions/runs/7611143402 * [submodules] Copy src HumanSignal/label-studio-frontend Workflow run: https://github.com/HumanSignal/label-studio/actions/runs/7611551866 * [submodules] Copy src HumanSignal/label-studio-frontend Workflow run: https://github.com/HumanSignal/label-studio/actions/runs/7643711911 * ci: Build frontend Workflow run: https://github.com/HumanSignal/label-studio/actions/runs/7643735748 * [submodules] Copy src HumanSignal/label-studio-frontend Workflow run: https://github.com/HumanSignal/label-studio/actions/runs/7646041615 --------- Co-authored-by: robot-ci-heartex <[email protected]> Co-authored-by: hlomzik <[email protected]> Co-authored-by: juliosgarbi <[email protected]>
tourdedave · Jan 24, 2024 · 59fc751 · 59fc751
1 parent 9c28615
commit 59fc751
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/web/dist/libs/editor/main.js b/web/dist/libs/editor/main.js
diff --git a/web/dist/libs/editor/main.js.map b/web/dist/libs/editor/main.js.map
diff --git a/web/libs/editor/src/utils/data.js b/web/libs/editor/src/utils/data.js
@@ -72,9 +72,9 @@ export const parseCSV = (text, separator = 'auto') => {
 
   const re = new RegExp(
     [
-      '"(""|[^"]+)*"', // quoted text with possible quoted quotes inside it ("not a ""value""")
+      '"(?:""|[^"])*"', // quoted text with possible quoted quotes inside it ("not a ""value""")
       `[^"${separator}]+`, // usual value, no quotes, between separators
-      `(?=${separator}(${separator}|$))`, // empty value in the middle or at the end of string
+      `(?=${separator}(?:${separator}|$))`, // empty value in the middle or at the end of string
       `^(?=${separator})`, // empty value at the start of the string
     ].join('|'),
     'g',