Skip to content

Commit 0348137

Browse files
sapphi-redsapphi-red
authored
refactor: use dedicated regex methods (#15228)
1 parent 5b85040 commit 0348137

File tree

18 files changed

+60
-67
lines changed

18 files changed

+60
-67
lines changed

‎packages/vite/src/node/__tests__/plugins/css.spec.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -325,7 +325,7 @@ require("other-module");`
325325
`"require("some-module"),/* empty css */require("other-module");"`,
326326
)
327327
// So there should be no pure css chunk anymore
328-
expect(newCode.match(/pure_css_chunk\.js/)).toBeNull()
328+
expect(newCode).not.toContain('pure_css_chunk.js')
329329
})
330330

331331
test('replaces require call in minified code that uses comma operator followed by assignment', () => {

‎packages/vite/src/node/__tests__/scan.spec.ts‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -106,6 +106,7 @@ describe('optimizer-scan:script-test', () => {
106106
`import type Bar from 'foo'`,
107107
]
108108
shouldFailArray.forEach((str) => {
109+
importsRE.lastIndex = 0
109110
expect(importsRE.test(str)).toBe(false)
110111
})
111112
})

‎packages/vite/src/node/build.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -910,7 +910,7 @@ export function onRollupWarning(
910910
const id = warning.id
911911
const exporter = warning.exporter
912912
// throw unless it's commonjs external...
913-
if (!id || !/\?commonjs-external$/.test(id)) {
913+
if (!id || !id.endsWith('?commonjs-external')) {
914914
throw new Error(
915915
`[vite]: Rollup failed to resolve import "${exporter}" from "${id}".\n` +
916916
`This is most likely unintended because it can break your application at runtime.\n` +

‎packages/vite/src/node/optimizer/index.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -642,7 +642,7 @@ export function runOptimizeDeps(
642642
}
643643

644644
for (const o of Object.keys(meta.outputs)) {
645-
if (!o.match(jsMapExtensionRE)) {
645+
if (!jsMapExtensionRE.test(o)) {
646646
const id = path
647647
.relative(processingCacheDirOutputPath, o)
648648
.replace(jsExtensionRE, '')

‎packages/vite/src/node/plugins/assetImportMetaUrl.ts‎

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -53,15 +53,13 @@ export function assetImportMetaUrlPlugin(config: ResolvedConfig): Plugin {
5353
) {
5454
let s: MagicString | undefined
5555
const assetImportMetaUrlRE =
56-
/\bnew\s+URL\s*\(\s*('[^']+'|"[^"]+"|`[^`]+`)\s*,\s*import\.meta\.url\s*(?:,\s*)?\)/g
56+
// eslint-disable-next-line regexp/no-unused-capturing-group -- https://github.com/ota-meshi/eslint-plugin-regexp/issues/675
57+
/\bnew\s+URL\s*\(\s*('[^']+'|"[^"]+"|`[^`]+`)\s*,\s*import\.meta\.url\s*(?:,\s*)?\)/dg
5758
const cleanString = stripLiteral(code)
5859

5960
let match: RegExpExecArray | null
6061
while ((match = assetImportMetaUrlRE.exec(cleanString))) {
61-
const { 0: exp, 1: emptyUrl, index } = match
62-
63-
const urlStart = cleanString.indexOf(emptyUrl, index)
64-
const urlEnd = urlStart + emptyUrl.length
62+
const [[startIndex, endIndex], [urlStart, urlEnd]] = match.indices!
6563
const rawUrl = code.slice(urlStart, urlEnd)
6664

6765
if (!s) s = new MagicString(code)
@@ -93,8 +91,8 @@ export function assetImportMetaUrlPlugin(config: ResolvedConfig): Plugin {
9391
query: injectQuery(queryString, 'url'),
9492
}
9593
s.update(
96-
index,
97-
index + exp.length,
94+
startIndex,
95+
endIndex,
9896
`new URL((import.meta.glob(${JSON.stringify(
9997
pattern,
10098
)}, ${JSON.stringify(
@@ -141,15 +139,15 @@ export function assetImportMetaUrlPlugin(config: ResolvedConfig): Plugin {
141139
}
142140
}
143141
if (!builtUrl) {
144-
const rawExp = code.slice(index, index + exp.length)
142+
const rawExp = code.slice(startIndex, endIndex)
145143
config.logger.warnOnce(
146144
`\n${rawExp} doesn't exist at build time, it will remain unchanged to be resolved at runtime`,
147145
)
148146
builtUrl = url
149147
}
150148
s.update(
151-
index,
152-
index + exp.length,
149+
startIndex,
150+
endIndex,
153151
`new URL(${JSON.stringify(builtUrl)}, import.meta.url)`,
154152
)
155153
}

‎packages/vite/src/node/plugins/css.ts‎

Lines changed: 12 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1288,8 +1288,6 @@ export async function preprocessCSS(
12881288
return await compileCSS(filename, code, config)
12891289
}
12901290

1291-
const postcssReturnsVirtualFilesRE = /^<.+>$/
1292-
12931291
export async function formatPostcssSourceMap(
12941292
rawMap: ExistingRawSourceMap,
12951293
file: string,
@@ -1299,7 +1297,8 @@ export async function formatPostcssSourceMap(
12991297
const sources = rawMap.sources.map((source) => {
13001298
const cleanSource = cleanUrl(decodeURIComponent(source))
13011299

1302-
if (postcssReturnsVirtualFilesRE.test(cleanSource)) {
1300+
// postcss virtual files
1301+
if (cleanSource[0] === '<' && cleanSource[cleanSource.length - 1] === '>') {
13031302
return `\0${cleanSource}`
13041303
}
13051304

@@ -1373,7 +1372,7 @@ async function resolvePostcssConfig(
13731372
const searchPath =
13741373
typeof inlineOptions === 'string' ? inlineOptions : config.root
13751374
result = postcssrc({}, searchPath).catch((e) => {
1376-
if (!/No PostCSS Config found/.test(e.message)) {
1375+
if (!e.message.includes('No PostCSS Config found')) {
13771376
if (e instanceof Error) {
13781377
const { name, message, stack } = e
13791378
e.name = 'Failed to load PostCSS config'
@@ -1654,6 +1653,11 @@ function resolveMinifyCssEsbuildOptions(
16541653
}
16551654
}
16561655

1656+
const atImportRE =
1657+
/@import(?:\s*(?:url\([^)]*\)|"(?:[^"]|(?<=\\)")*"|'(?:[^']|(?<=\\)')*').*?|[^;]*);/g
1658+
const atCharsetRE =
1659+
/@charset(?:\s*(?:"(?:[^"]|(?<=\\)")*"|'(?:[^']|(?<=\\)')*').*?|[^;]*);/g
1660+
16571661
export async function hoistAtRules(css: string): Promise<string> {
16581662
const s = new MagicString(css)
16591663
const cleanCss = emptyCssComments(css)
@@ -1663,8 +1667,7 @@ export async function hoistAtRules(css: string): Promise<string> {
16631667
// CSS @import can only appear at top of the file. We need to hoist all @import
16641668
// to top when multiple files are concatenated.
16651669
// match until semicolon that's not in quotes
1666-
const atImportRE =
1667-
/@import(?:\s*(?:url\([^)]*\)|"(?:[^"]|(?<=\\)")*"|'(?:[^']|(?<=\\)')*').*?|[^;]*);/g
1670+
atImportRE.lastIndex = 0
16681671
while ((match = atImportRE.exec(cleanCss))) {
16691672
s.remove(match.index, match.index + match[0].length)
16701673
// Use `appendLeft` instead of `prepend` to preserve original @import order
@@ -1673,8 +1676,7 @@ export async function hoistAtRules(css: string): Promise<string> {
16731676

16741677
// #6333
16751678
// CSS @charset must be the top-first in the file, hoist the first to top
1676-
const atCharsetRE =
1677-
/@charset(?:\s*(?:"(?:[^"]|(?<=\\)")*"|'(?:[^']|(?<=\\)')*').*?|[^;]*);/g
1679+
atCharsetRE.lastIndex = 0
16781680
let foundCharset = false
16791681
while ((match = atCharsetRE.exec(cleanCss))) {
16801682
s.remove(match.index, match.index + match[0].length)
@@ -2406,8 +2408,8 @@ export const convertTargets = (
24062408

24072409
for (const entry of entriesWithoutES) {
24082410
if (entry === 'esnext') continue
2409-
const index = entry.match(versionRE)?.index
2410-
if (index) {
2411+
const index = entry.search(versionRE)
2412+
if (index >= 0) {
24112413
const browser = map[entry.slice(0, index)]
24122414
if (browser === false) continue // No mapping available
24132415
if (browser) {

‎packages/vite/src/node/plugins/esbuild.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -320,7 +320,7 @@ export const buildEsbuildPlugin = (config: ResolvedConfig): Plugin => {
320320
const esbuildCode = res.code
321321
const contentIndex =
322322
opts.format === 'iife'
323-
? esbuildCode.match(IIFE_BEGIN_RE)?.index || 0
323+
? Math.max(esbuildCode.search(IIFE_BEGIN_RE), 0)
324324
: opts.format === 'umd'
325325
? esbuildCode.indexOf(`(function(`) // same for minified or not
326326
: 0

‎packages/vite/src/node/plugins/html.ts‎

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,8 @@ const htmlProxyRE =
4848
const inlineCSSRE = /__VITE_INLINE_CSS__([a-z\d]{8}_\d+)__/g
4949
// Do not allow preceding '.', but do allow preceding '...' for spread operations
5050
const inlineImportRE =
51-
/(?<!(?<!\.\.)\.)\bimport\s*\(("(?:[^"]|(?<=\\)")*"|'(?:[^']|(?<=\\)')*')\)/g
51+
// eslint-disable-next-line regexp/no-unused-capturing-group -- https://github.com/ota-meshi/eslint-plugin-regexp/issues/675
52+
/(?<!(?<!\.\.)\.)\bimport\s*\(("(?:[^"]|(?<=\\)")*"|'(?:[^']|(?<=\\)')*')\)/dg
5253
const htmlLangRE = /\.(?:html|htm)$/
5354

5455
const importMapRE =
@@ -926,10 +927,9 @@ export function extractImportExpressionFromClassicScript(
926927
let match: RegExpExecArray | null
927928
inlineImportRE.lastIndex = 0
928929
while ((match = inlineImportRE.exec(cleanCode))) {
929-
const { 1: url, index } = match
930-
const startUrl = cleanCode.indexOf(url, index)
931-
const start = startUrl + 1
932-
const end = start + url.length - 2
930+
const [, [urlStart, urlEnd]] = match.indices!
931+
const start = urlStart + 1
932+
const end = urlEnd - 1
933933
scriptUrls.push({
934934
start: start + startOffset,
935935
end: end + startOffset,
@@ -1004,11 +1004,11 @@ export function preImportMapHook(
10041004
config: ResolvedConfig,
10051005
): IndexHtmlTransformHook {
10061006
return (html, ctx) => {
1007-
const importMapIndex = html.match(importMapRE)?.index
1008-
if (importMapIndex === undefined) return
1007+
const importMapIndex = html.search(importMapRE)
1008+
if (importMapIndex < 0) return
10091009

1010-
const importMapAppendIndex = html.match(importMapAppendRE)?.index
1011-
if (importMapAppendIndex === undefined) return
1010+
const importMapAppendIndex = html.search(importMapAppendRE)
1011+
if (importMapAppendIndex < 0) return
10121012

10131013
if (importMapAppendIndex < importMapIndex) {
10141014
const relativeHtml = normalizePath(

‎packages/vite/src/node/plugins/importAnalysis.ts‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -366,7 +366,7 @@ export function importAnalysisPlugin(config: ResolvedConfig): Plugin {
366366
if (
367367
(isRelative || isSelfImport) &&
368368
!hasImportInQueryParamsRE.test(url) &&
369-
!url.match(DEP_VERSION_RE)
369+
!DEP_VERSION_RE.test(url)
370370
) {
371371
const versionMatch = importer.match(DEP_VERSION_RE)
372372
if (versionMatch) {
@@ -535,7 +535,7 @@ export function importAnalysisPlugin(config: ResolvedConfig): Plugin {
535535
let rewriteDone = false
536536
if (
537537
depsOptimizer?.isOptimizedDepFile(resolvedId) &&
538-
!resolvedId.match(optimizedDepChunkRE)
538+
!optimizedDepChunkRE.test(resolvedId)
539539
) {
540540
// for optimized cjs deps, support named imports by rewriting named imports to const assignments.
541541
// internal optimized chunks don't need es interop and are excluded
@@ -555,7 +555,7 @@ export function importAnalysisPlugin(config: ResolvedConfig): Plugin {
555555
// Non-entry dynamic imports from dependencies will reach here as there isn't
556556
// optimize info for them, but they don't need es interop. If the request isn't
557557
// a dynamic import, then it is an internal Vite error
558-
if (!file.match(optimizedDepDynamicRE)) {
558+
if (!optimizedDepDynamicRE.test(file)) {
559559
config.logger.error(
560560
colors.red(
561561
`Vite Error, ${url} optimized info should be defined`,

‎packages/vite/src/node/plugins/importAnalysisBuild.ts‎

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ export const preloadMarker = `__VITE_PRELOAD__`
4545
export const preloadBaseMarker = `__VITE_PRELOAD_BASE__`
4646

4747
export const preloadHelperId = '\0vite/preload-helper.js'
48-
const preloadMarkerWithQuote = new RegExp(`['"]${preloadMarker}['"]`)
48+
const preloadMarkerWithQuote = new RegExp(`['"]${preloadMarker}['"]`, 'g')
4949

5050
const dynamicImportPrefixRE = /import\s*\(/
5151

@@ -63,13 +63,9 @@ function indexOfMatchInSlice(
6363
reg: RegExp,
6464
pos: number = 0,
6565
): number {
66-
if (pos !== 0) {
67-
str = str.slice(pos)
68-
}
69-
70-
const matcher = str.match(reg)
71-
72-
return matcher?.index !== undefined ? matcher.index + pos : -1
66+
reg.lastIndex = pos
67+
const result = reg.exec(str)
68+
return result?.index ?? -1
7369
}
7470

7571
/**
@@ -351,7 +347,7 @@ export function buildImportAnalysisPlugin(config: ResolvedConfig): Plugin {
351347
if (url !== specifier) {
352348
if (
353349
depsOptimizer.isOptimizedDepFile(resolvedId) &&
354-
!resolvedId.match(optimizedDepChunkRE)
350+
!optimizedDepChunkRE.test(resolvedId)
355351
) {
356352
const file = cleanUrl(resolvedId) // Remove ?v={hash}
357353

@@ -368,7 +364,7 @@ export function buildImportAnalysisPlugin(config: ResolvedConfig): Plugin {
368364
// Non-entry dynamic imports from dependencies will reach here as there isn't
369365
// optimize info for them, but they don't need es interop. If the request isn't
370366
// a dynamic import, then it is an internal Vite error
371-
if (!file.match(optimizedDepDynamicRE)) {
367+
if (!optimizedDepDynamicRE.test(file)) {
372368
config.logger.error(
373369
colors.red(
374370
`Vite Error, ${url} optimized info should be defined`,

0 commit comments

Comments
 (0)