Skip to content

Commit 77ce5a7

Browse files
committed
chore: escape more
1 parent f80e212 commit 77ce5a7

File tree

1 file changed

+4
-6
lines changed

1 file changed

+4
-6
lines changed

shared/utils/dev-dependency.ts

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
import { regExpEscape } from '@li/regexp-escape-polyfill'
2+
13
export type DevDependencySuggestionReason = 'known-package' | 'readme-hint'
24

35
export interface DevDependencySuggestion {
@@ -59,15 +61,11 @@ function isKnownDevDependencyPackage(packageName: string): boolean {
5961
)
6062
}
6163

62-
function escapeRegExp(text: string): string {
63-
return text.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')
64-
}
65-
6664
function hasReadmeDevInstallHint(packageName: string, readmeContent?: string | null): boolean {
6765
if (!readmeContent) return false
6866

69-
const escapedName = escapeRegExp(packageName)
70-
const escapedNpmName = escapeRegExp(`npm:${packageName}`)
67+
const escapedName = regExpEscape(packageName)
68+
const escapedNpmName = regExpEscape(`npm:${packageName}`)
7169
const packageSpec = `(?:${escapedName}|${escapedNpmName})(?:@[\\w.-]+)?`
7270

7371
const patterns = [

0 commit comments

Comments
 (0)