From b4e671d83fc01d75629e30c9a7593318fb1f986d Mon Sep 17 00:00:00 2001 From: Viren070 Date: Tue, 6 May 2025 20:36:34 +0100 Subject: [PATCH] perf: don't bypass cache and use source instead of toString --- packages/addon/src/addon.ts | 6 +++--- packages/utils/src/regex.ts | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/packages/addon/src/addon.ts b/packages/addon/src/addon.ts index 77dcca01..12dc36be 100644 --- a/packages/addon/src/addon.ts +++ b/packages/addon/src/addon.ts @@ -135,7 +135,7 @@ export class AIOStreams { Settings.DEFAULT_REGEX_EXCLUDE_PATTERN : undefined; const excludeRegex = excludeRegexPattern - ? compileRegex(excludeRegexPattern, 'i', true) + ? compileRegex(excludeRegexPattern, 'i') : undefined; const excludeKeywordsRegex = this.config.excludeFilters @@ -147,7 +147,7 @@ export class AIOStreams { Settings.DEFAULT_REGEX_INCLUDE_PATTERN : undefined; const requiredRegex = requiredRegexPattern - ? compileRegex(requiredRegexPattern, 'i', true) + ? compileRegex(requiredRegexPattern, 'i') : undefined; const requiredKeywordsRegex = this.config.strictIncludeFilters @@ -162,7 +162,7 @@ export class AIOStreams { ? sortRegexPatterns .split(/\s+/) .filter(Boolean) - .map((pattern) => compileRegex(pattern, 'i', true)) + .map((pattern) => compileRegex(pattern, 'i')) : undefined; excludeRegex || diff --git a/packages/utils/src/regex.ts b/packages/utils/src/regex.ts index ed59c00f..85a90ce8 100644 --- a/packages/utils/src/regex.ts +++ b/packages/utils/src/regex.ts @@ -30,7 +30,7 @@ export function safeRegexTest( try { return resultCache.wrap( (p: RegExp, s: string) => isMatch(p, s, { timeout: timeoutMs }), - getTextHash(`${compiledPattern.toString()}|${str}`), + getTextHash(`${compiledPattern.source}|${str}`), 100, compiledPattern, str