fix: comments that are on the same line as a regexp:

vitejs · Apr 23, 2022 · 2ec1ff1 · 2ec1ff1
1 parent 33db886
commit 2ec1ff1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/packages/vite/src/node/__tests__/cleanString.spec.ts b/packages/vite/src/node/__tests__/cleanString.spec.ts
@@ -79,6 +79,8 @@ test('regexp', () => {
     "'''1'''"
     /"''''1''''"/
     "''''1''''"
+    /'1'/  // '1'
+    /'1'/  /* '1' */
   `)
   expect(clean).not.toMatch('1')
 })

diff --git a/packages/vite/src/node/cleanString.ts b/packages/vite/src/node/cleanString.ts
@@ -3,7 +3,7 @@ import type { RollupError } from 'rollup'
 // /`([^`\$\{\}]|\$\{(`|\g<1>)*\})*`/g can match nested string template
 // but js not support match expression(\g<0>). so clean string template(`...`) in other ways.
 const cleanerRE =
-  /"([^"]|(?<=\\)")*"|'([^']|(?<=\\)')*'|\/\*(.|[\r\n])*?\*\/|\/\/.*|\/.*\//g
+  /"([^"]|(?<=\\)")*"|'([^']|(?<=\\)')*'|\/\*(.|[\r\n])*?\*\/|\/\/.*|\/([^\/]|(?<=\/)\/)*\//g
 
 const blankReplacer = (s: string) => ' '.repeat(s.length)
 const stringBlankReplacer = (s: string) =>