@fsvreddit/bot-bouncer-evaluation 0.7.41 → 0.7.43

package/dist/allEvaluators.d.ts

@@ -1,6 +1,6 @@
 import { EvaluatePinnedPostTitles } from "./userEvaluation/EvaluatePinnedPostTitle.js";
+import { EvaluateSocialLinks } from "./userEvaluation/EvaluateSocialLinks.js";
 import { EvaluateBotGroupAdvanced } from "./userEvaluation/EvaluateBotGroupAdvanced.js";
-import { EvaluateWarmupBot } from "./userEvaluation/EvaluateWarmupBot.js";
 import { EvaluateBotGroupAdvancedInternal } from "./userEvaluation/EvaluateBotGroupAdvancedInternal.js";
 import { EvaluateLinkReuse } from "./userEvaluation/EvaluateLinkReuse.js";
 /**
@@ -12,5 +12,5 @@ import { EvaluateLinkReuse } from "./userEvaluation/EvaluateLinkReuse.js";
  * As a result, order entries to prioritise faster evaluators that do not need to
  * reference social links or account history first.
  */
-export declare const ALL_EVALUATORS: (typeof EvaluatePinnedPostTitles | typeof EvaluateBotGroupAdvanced | typeof EvaluateWarmupBot | typeof EvaluateLinkReuse)[];
+export declare const ALL_EVALUATORS: (typeof EvaluatePinnedPostTitles | typeof EvaluateSocialLinks | typeof EvaluateBotGroupAdvanced | typeof EvaluateLinkReuse)[];
 export declare const ALL_INTERNAL_EVALUATORS: (typeof EvaluateBotGroupAdvancedInternal)[];

package/dist/userEvaluation/EvaluateBadDisplayname.js

@@ -15,11 +15,7 @@ class EvaluateBadDisplayName extends UserEvaluatorBase_js_1.UserEvaluatorBase {
         this.banContentThreshold = 0;
     }
     getBadDisplayNameRegexes() {
-        if (this.badDisplayNameRegexes === undefined) {
-            const bannableDisplayNameRegexes = this.getVariable("regexes", []);
-            const ignoredDisplayNameRegexes = this.getVariable("ignoredRegexes", []);
-            this.badDisplayNameRegexes = bannableDisplayNameRegexes.filter(regex => !ignoredDisplayNameRegexes.includes(regex));
-        }
+        this.badDisplayNameRegexes ?? (this.badDisplayNameRegexes = this.getVariable("regexes", []));
         return this.badDisplayNameRegexes;
     }
     isBadDisplayName(displayName) {

package/dist/userEvaluation/EvaluateBadDisplaynameDefinedHandles.d.ts

@@ -8,6 +8,8 @@ export declare class EvaluateBadDisplayNameDefinedHandles extends UserEvaluatorB
     banContentThreshold: number;
     private isBadDisplayName;
     gatherRegexes(): EvaluatorRegex[];
+    private compiledRegexes;
+    private getCompiledRegexes;
     validateVariables(): ValidationIssue[];
     preEvaluateComment(_: CommentCreate): boolean;
     preEvaluatePost(_: Post): boolean;

package/dist/userEvaluation/EvaluateBadDisplaynameDefinedHandles.js

@@ -15,12 +15,11 @@ class EvaluateBadDisplayNameDefinedHandles extends UserEvaluatorBase_js_1.UserEv
         if (!displayName) {
             return false;
         }
-        const regexes = this.gatherRegexes().map(r => r.regex);
-        const matchedRegexes = regexes.filter(regex => new RegExp(regex, "u").test(displayName));
+        const matchedRegexes = this.getCompiledRegexes().filter(regex => regex.test(displayName));
         if (matchedRegexes.length === 0) {
             return false;
         }
-        this.addHitReason(`Display name matches regexes: ${matchedRegexes.map(r => `\`${r}\``).join(", ")}`);
+        this.addHitReason(`Display name matches regexes: ${matchedRegexes.map(r => `\`${r.source}\``).join(", ")}`);
         return true;
     }
     gatherRegexes() {
@@ -41,6 +40,10 @@ class EvaluateBadDisplayNameDefinedHandles extends UserEvaluatorBase_js_1.UserEv
             flags: "u",
         })));
     }
+    getCompiledRegexes() {
+        this.compiledRegexes ?? (this.compiledRegexes = this.gatherRegexes().map(r => new RegExp(r.regex, r.flags)));
+        return this.compiledRegexes;
+    }
     validateVariables() {
         const results = [];
         let regexes;

package/dist/userEvaluation/EvaluateBadUsername.d.ts

@@ -9,6 +9,8 @@ export declare class EvaluateBadUsername extends UserEvaluatorBase {
     private isBadUsername;
     validateVariables(): ValidationIssue[];
     gatherRegexes(): EvaluatorRegex[];
+    private compiledRegexes;
+    private getCompiledRegexes;
     preEvaluateComment(event: CommentCreate): boolean;
     preEvaluatePost(post: Post): boolean;
     preEvaluateUser(user: UserExtended): boolean;

package/dist/userEvaluation/EvaluateBadUsername.js

@@ -19,10 +19,9 @@ class EvaluateBadUsername extends UserEvaluatorBase_js_1.UserEvaluatorBase {
         if (!username) {
             return false;
         }
-        const regexes = this.getVariable("regexes", []);
-        const matchedRegex = regexes.find(regex => new RegExp(regex).test(username));
+        const matchedRegex = this.getCompiledRegexes().find(regex => regex.test(username));
         if (matchedRegex) {
-            this.addHitReason(`Username matches regex: ${(0, markdown_escape_1.default)(matchedRegex)}`);
+            this.addHitReason(`Username matches regex: ${(0, markdown_escape_1.default)(matchedRegex.source)}`);
         }
         return matchedRegex !== undefined;
     }
@@ -51,6 +50,10 @@ class EvaluateBadUsername extends UserEvaluatorBase_js_1.UserEvaluatorBase {
             regex,
         })));
     }
+    getCompiledRegexes() {
+        this.compiledRegexes ?? (this.compiledRegexes = this.getVariable("regexes", []).map(regex => new RegExp(regex)));
+        return this.compiledRegexes;
+    }
     preEvaluateComment(event) {
         return this.isBadUsername(event.author?.name);
     }

package/dist/userEvaluation/EvaluateBioText.d.ts

@@ -8,10 +8,12 @@ export declare class EvaluateBioText extends UserEvaluatorBase {
     banContentThreshold: number;
     private bioText;
     private getBioText;
-    preEvaluateComment(event: CommentCreate): boolean;
     validateVariables(): ValidationIssue[];
     getVariableOverrides(): Record<string, unknown>;
     gatherRegexes(): EvaluatorRegex[];
+    private compiledRegexes;
+    private getCompiledRegexes;
+    preEvaluateComment(event: CommentCreate): boolean;
     preEvaluatePost(_: Post): boolean;
     preEvaluateUser(user: UserExtended): boolean;
     evaluate(user: UserExtended): boolean;

package/dist/userEvaluation/EvaluateBioText.js

@@ -14,17 +14,9 @@ class EvaluateBioText extends UserEvaluatorBase_js_1.UserEvaluatorBase {
         this.banContentThreshold = 0;
     }
     getBioText() {
-        if (this.bioText === undefined) {
-            const bannableBioText = this.getVariable("bantext", []);
-            const ignoredBanText = this.getVariable("ignoredBanText", []);
-            this.bioText = bannableBioText.filter(bioText => !ignoredBanText.includes(bioText));
-        }
+        this.bioText ?? (this.bioText = this.getVariable("bantext", []));
         return this.bioText;
     }
-    preEvaluateComment(event) {
-        const problematicBioText = this.getBioText();
-        return problematicBioText.some(bioText => event.author?.description && new RegExp(bioText, "u").test(event.author.description));
-    }
     validateVariables() {
         const results = [];
         const regexes = this.getVariable("bantext", []);
@@ -62,30 +54,35 @@ class EvaluateBioText extends UserEvaluatorBase_js_1.UserEvaluatorBase {
             flags: "u",
         }));
     }
+    getCompiledRegexes() {
+        this.compiledRegexes ?? (this.compiledRegexes = this.getBioText().map(bio => new RegExp(bio, "u")));
+        return this.compiledRegexes;
+    }
+    preEvaluateComment(event) {
+        if (!event.author?.description) {
+            return false;
+        }
+        return this.getCompiledRegexes().some(regex => event.author?.description && regex.test(event.author.description));
+    }
     // eslint-disable-next-line @typescript-eslint/no-unused-vars
     preEvaluatePost(_) {
         const bannableBioText = this.getBioText();
         return bannableBioText.length > 0;
     }
     preEvaluateUser(user) {
-        const bannableBioText = this.getBioText();
-        if (bannableBioText.length === 0) {
+        if (!user.userDescription) {
             return false;
         }
         if (user.commentKarma > 2000 && user.linkKarma > 2000) {
             return false;
         }
-        return bannableBioText.some(bioText => user.userDescription && new RegExp(bioText, "u").test(user.userDescription));
+        return this.getCompiledRegexes().some(regex => user.userDescription && regex.test(user.userDescription));
     }
     evaluate(user) {
-        const bannableBioText = this.getBioText();
-        if (bannableBioText.length === 0) {
-            return false;
-        }
-        const bannableBioTextFound = bannableBioText.find(bio => user.userDescription && new RegExp(bio, "u").test(user.userDescription));
-        if (bannableBioTextFound) {
+        const matchedRegex = this.getCompiledRegexes().find(regex => user.userDescription && regex.test(user.userDescription));
+        if (matchedRegex) {
             this.canAutoBan = true;
-            this.addHitReason(`Bio text matched regex: ${(0, markdown_escape_1.default)(bannableBioTextFound)}`);
+            this.addHitReason(`Bio text matched regex: ${(0, markdown_escape_1.default)(matchedRegex.source)}`);
         }
         else {
             return false;

package/dist/userEvaluation/EvaluateBioTextDefinedHandles.d.ts

@@ -7,6 +7,8 @@ export declare class EvaluateBioTextDefinedHandles extends UserEvaluatorBase {
     shortname: string;
     banContentThreshold: number;
     private getHandles;
+    private compiledRegexes;
+    private getCompiledRegexes;
     preEvaluateComment(event: CommentCreate): boolean;
     validateVariables(): ValidationIssue[];
     gatherRegexes(): EvaluatorRegex[];

package/dist/userEvaluation/EvaluateBioTextDefinedHandles.js

@@ -22,9 +22,15 @@ class EvaluateBioTextDefinedHandles extends UserEvaluatorBase_js_1.UserEvaluator
         }
         return parsed.body.map(part => part.raw);
     }
+    getCompiledRegexes() {
+        this.compiledRegexes ?? (this.compiledRegexes = this.gatherRegexes().map(r => new RegExp(r.regex, "u")));
+        return this.compiledRegexes;
+    }
     preEvaluateComment(event) {
-        const problematicBioText = this.gatherRegexes().map(r => r.regex);
-        return problematicBioText.some(bioText => event.author?.description && new RegExp(bioText, "u").test(event.author.description));
+        if (!event.author?.description) {
+            return false;
+        }
+        return this.getCompiledRegexes().some(regex => event.author?.description && regex.test(event.author.description));
     }
     validateVariables() {
         const results = [];
@@ -70,29 +76,21 @@ class EvaluateBioTextDefinedHandles extends UserEvaluatorBase_js_1.UserEvaluator
         if (!user.userDescription) {
             return false;
         }
-        const regexes = this.gatherRegexes().map(r => r.regex);
-        if (regexes.length === 0) {
-            return false;
-        }
         if (user.commentKarma > 2000 && user.linkKarma > 2000) {
             return false;
         }
-        return regexes.some(bioText => user.userDescription && new RegExp(bioText, "u").test(user.userDescription));
+        return this.getCompiledRegexes().some(regex => user.userDescription && regex.test(user.userDescription));
     }
     evaluate(user) {
         if (!user.userDescription) {
             return false;
         }
-        const regexes = this.gatherRegexes().map(r => r.regex);
-        if (regexes.length === 0) {
-            return false;
-        }
-        const bannableBioTextFound = regexes.filter(bio => user.userDescription && new RegExp(bio, "u").test(user.userDescription));
+        const bannableBioTextFound = this.getCompiledRegexes().filter(regex => user.userDescription && regex.test(user.userDescription));
         if (bannableBioTextFound.length === 0) {
             return false;
         }
         this.canAutoBan = true;
-        this.addHitReason(`Bio text matched regexes: ${bannableBioTextFound.map(bio => `\`${bio}\``).join(", ")}`);
+        this.addHitReason(`Bio text matched regexes: ${bannableBioTextFound.map(regex => `\`${regex.source}\``).join(", ")}`);
         return user.nsfw || this.getPosts().some(post => post.isNsfw());
     }
 }

package/dist/userEvaluation/EvaluateBotGroupAdvanced.d.ts

@@ -85,11 +85,12 @@ export declare class EvaluateBotGroupAdvanced extends UserEvaluatorBase {
     name: string;
     shortname: string;
     banContentThreshold: number;
-    private verboseLogging;
     private redis;
     allowNewFeatures: boolean;
     private evaluationError;
     constructor(context: TriggerContext, history: (Post | Comment)[], socialLinks: UserSocialLink[] | undefined, variables: Record<string, unknown>);
+    private compiledRegexes;
+    private getCompiledRegex;
     private anyRegexMatches;
     getBotGroups(): BotGroup[];
     validateVariables(): ValidationIssue[];

package/dist/userEvaluation/EvaluateBotGroupAdvanced.js

@@ -457,8 +457,8 @@ class EvaluateBotGroupAdvanced extends UserEvaluatorBase_js_1.UserEvaluatorBase
         this.name = "Bot Group Advanced";
         this.shortname = "botgroupadvanced";
         this.banContentThreshold = 0; // No content ban threshold for this evaluator to support account properties only checks
-        this.verboseLogging = false;
         this.allowNewFeatures = false;
+        this.compiledRegexes = new Map();
         this.cachedPostProperties = {};
         this.postPropertiesStatistics = {
             totalTime: 0,
@@ -467,11 +467,19 @@ class EvaluateBotGroupAdvanced extends UserEvaluatorBase_js_1.UserEvaluatorBase
             cacheMisses: 0,
             errors: 0,
         };
-        this.verboseLogging = this.getVariable("verboseLogging", false);
         this.redis = context.appSlug === constants_js_1.MAIN_APP_NAME ? context.redis.global : context.redis;
     }
+    getCompiledRegex(regex) {
+        const result = this.compiledRegexes.get(regex);
+        if (result) {
+            return result;
+        }
+        const compiledRegex = new RegExp(regex, "u");
+        this.compiledRegexes.set(regex, compiledRegex);
+        return compiledRegex;
+    }
     anyRegexMatches(input, regexes) {
-        return regexes.some(regex => new RegExp(regex, "u").test(input));
+        return regexes.some(regex => this.getCompiledRegex(regex).test(input));
     }
     getBotGroups() {
         const groups = this.getAllVariables("group");
@@ -746,9 +754,6 @@ class EvaluateBotGroupAdvanced extends UserEvaluatorBase_js_1.UserEvaluatorBase
                 continue;
             }
             if (!group.criteria) {
-                if (this.verboseLogging) {
-                    console.log(`Pre-evaluation: Comment ${comment.comment.id} for ${comment.author.name} matches bot group with no criteria ${group.name}`);
-                }
                 return true;
             }
             ;

package/dist/userEvaluation/EvaluateCommentPhrase.d.ts

@@ -8,6 +8,8 @@ export declare class EvaluateCommentPhrase extends UserEvaluatorBase {
     banContentThreshold: number;
     validateVariables(): ValidationIssue[];
     gatherRegexes(): EvaluatorRegex[];
+    private compiledRegexes;
+    private getCompiledRegexes;
     private eligibleComment;
     preEvaluateComment(event: CommentCreate): boolean;
     preEvaluatePost(_: Post): boolean;

package/dist/userEvaluation/EvaluateCommentPhrase.js

@@ -20,11 +20,11 @@ class EvaluateCommentPhrase extends UserEvaluatorBase_js_1.UserEvaluatorBase {
                 regex = new RegExp(regexVal);
             }
             catch {
-                results.push({ severity: "error", message: `Invalid regex in biotext: ${regexVal}` });
+                results.push({ severity: "error", message: `Invalid regex in comment phrase: ${regexVal}` });
                 continue;
             }
             if (regex.test("")) {
-                results.push({ severity: "error", message: `Bio Text regex is too greedy: ${regexVal}` });
+                results.push({ severity: "error", message: `Comment phrase regex is too greedy: ${regexVal}` });
             }
         }
         return results;
@@ -37,14 +37,16 @@ class EvaluateCommentPhrase extends UserEvaluatorBase_js_1.UserEvaluatorBase {
             regex: phrase,
         })));
     }
+    getCompiledRegexes() {
+        this.compiledRegexes ?? (this.compiledRegexes = this.gatherRegexes().map(r => new RegExp(r.regex)));
+        return this.compiledRegexes;
+    }
     eligibleComment(comment) {
-        const phrases = this.getVariable("phrases", []);
         const maxCommentAgeInDays = this.getVariable("maxcommentageindays", 30);
-        if (phrases.length === 0) {
+        if (comment.createdAt <= (0, date_fns_1.subDays)(new Date(), maxCommentAgeInDays)) {
             return false;
         }
-        return comment.createdAt > (0, date_fns_1.subDays)(new Date(), maxCommentAgeInDays)
-            && phrases.some(phrase => new RegExp(phrase).test(comment.body));
+        return this.getCompiledRegexes().some(regex => regex.test(comment.body));
     }
     preEvaluateComment(event) {
         if (!event.comment) {
@@ -73,8 +75,7 @@ class EvaluateCommentPhrase extends UserEvaluatorBase_js_1.UserEvaluatorBase {
             return false;
         }
         const matchingComment = matchingComments[0];
-        const phrases = this.getVariable("phrases", []);
-        const matchedPhrase = phrases.find(phrase => new RegExp(phrase).test(matchingComment.body));
+        const matchedPhrase = this.getCompiledRegexes().find(regex => regex.test(matchingComment.body))?.source;
         if (!matchedPhrase) {
             // Impossible to reach this point.
             return false;

package/dist/userEvaluation/EvaluatePostTitle.d.ts

@@ -10,6 +10,8 @@ export declare class EvaluatePostTitle extends UserEvaluatorBase {
     gatherRegexes(): EvaluatorRegex[];
     preEvaluateComment(_: CommentCreate): boolean;
     private getTitles;
+    private titleRegexes;
+    private getTitleRegexes;
     preEvaluatePost(post: Post): boolean;
     preEvaluateUser(user: UserExtended): boolean;
     evaluate(_: UserExtended): boolean;

package/dist/userEvaluation/EvaluatePostTitle.js

@@ -46,15 +46,17 @@ class EvaluatePostTitle extends UserEvaluatorBase_js_1.UserEvaluatorBase {
         return false;
     }
     getTitles() {
-        const prefixesToIgnore = this.getVariable("ignoredRegexPrefixes", []);
-        return this.getVariable("bantext", []).filter(title => !prefixesToIgnore.some(prefix => title.startsWith(prefix)));
+        return this.getVariable("bantext", []);
+    }
+    getTitleRegexes() {
+        this.titleRegexes ?? (this.titleRegexes = this.getTitles().map(title => new RegExp(title, "u")));
+        return this.titleRegexes;
     }
     preEvaluatePost(post) {
         if (post.crosspostParentId) {
             return false;
         }
-        const bannableTitles = this.getTitles();
-        return bannableTitles.some(title => new RegExp(title, "u").test(post.title));
+        return this.getTitleRegexes().some(title => title.test(post.title));
     }
     preEvaluateUser(user) {
         const maxCommentKarma = this.getVariable("maxCommentKarma", 2000);
@@ -70,18 +72,18 @@ class EvaluatePostTitle extends UserEvaluatorBase_js_1.UserEvaluatorBase {
         if (userPosts.length === 0) {
             return false;
         }
-        const bannableTitles = this.gatherRegexes().map(title => ({ pattern: title.regex, regex: new RegExp(title.regex, title.flags) }));
         const nonMatchingTitles = new Set();
+        const regexes = this.getTitleRegexes();
         for (const title of userPosts.map(post => post.title)) {
             if (nonMatchingTitles.has(title)) {
                 continue;
             }
-            const matchedBanRegex = bannableTitles.find(bannable => bannable.regex.test(title));
+            const matchedBanRegex = regexes.find(regex => regex.test(title));
             if (!matchedBanRegex) {
                 nonMatchingTitles.add(title);
                 continue;
             }
-            this.addHitReason(`Post title "${title}" matched bannable regex: ${(0, markdown_escape_1.default)(matchedBanRegex.pattern)}`);
+            this.addHitReason(`Post title "${title}" matched bannable regex: ${(0, markdown_escape_1.default)(matchedBanRegex.source)}`);
             this.canAutoBan = true;
             return true;
         }

package/dist/userEvaluation/EvaluatePostTitleDefinedHandles.d.ts

@@ -8,6 +8,8 @@ export declare class EvaluatePostTitleDefinedHandles extends UserEvaluatorBase {
     banContentThreshold: number;
     validateVariables(): ValidationIssue[];
     gatherRegexes(): EvaluatorRegex[];
+    private compiledRegexes;
+    private getCompiledRegexes;
     preEvaluateComment(_: CommentCreate): boolean;
     preEvaluatePost(post: Post): boolean;
     preEvaluateUser(user: UserExtended): boolean;

package/dist/userEvaluation/EvaluatePostTitleDefinedHandles.js

@@ -60,6 +60,10 @@ class EvaluatePostTitleDefinedHandles extends UserEvaluatorBase_js_1.UserEvaluat
             flags: "u",
         })));
     }
+    getCompiledRegexes() {
+        this.compiledRegexes ?? (this.compiledRegexes = this.gatherRegexes().map(r => new RegExp(r.regex, r.flags)));
+        return this.compiledRegexes;
+    }
     // eslint-disable-next-line @typescript-eslint/no-unused-vars
     preEvaluateComment(_) {
         return false;
@@ -68,8 +72,7 @@ class EvaluatePostTitleDefinedHandles extends UserEvaluatorBase_js_1.UserEvaluat
         if (post.crosspostParentId) {
             return false;
         }
-        const problematicTitles = this.gatherRegexes().map(r => r.regex);
-        return problematicTitles.some(title => new RegExp(title, "u").test(post.title));
+        return this.getCompiledRegexes().some(regex => regex.test(post.title));
     }
     preEvaluateUser(user) {
         const maxCommentKarma = this.getVariable("maxCommentKarma", 2000);
@@ -85,7 +88,7 @@ class EvaluatePostTitleDefinedHandles extends UserEvaluatorBase_js_1.UserEvaluat
         if (userPosts.length === 0) {
             return false;
         }
-        const regexes = this.gatherRegexes().map(r => ({ pattern: r.regex, regex: new RegExp(r.regex, r.flags) }));
+        const regexes = this.getCompiledRegexes();
         if (regexes.length === 0) {
             return false;
         }
@@ -94,12 +97,12 @@ class EvaluatePostTitleDefinedHandles extends UserEvaluatorBase_js_1.UserEvaluat
             if (nonMatchingTitles.has(title)) {
                 continue;
             }
-            const matchedRegex = regexes.find(r => r.regex.test(title));
+            const matchedRegex = regexes.find(r => r.test(title));
             if (!matchedRegex) {
                 nonMatchingTitles.add(title);
                 continue;
             }
-            this.addHitReason(`Post title "${title}" matched bannable regex: ${(0, markdown_escape_1.default)(matchedRegex.pattern)}`);
+            this.addHitReason(`Post title "${title}" matched bannable regex: ${(0, markdown_escape_1.default)(matchedRegex.source)}`);
             this.canAutoBan = true;
             return true;
         }

package/dist/userEvaluation/EvaluatePostTitleMulti.d.ts

@@ -9,6 +9,8 @@ export declare class EvaluatePostTitleMulti extends UserEvaluatorBase {
     validateVariables(): ValidationIssue[];
     getVariableOverrides(): Record<string, unknown>;
     gatherRegexes(): EvaluatorRegex[];
+    private compiledRegexes;
+    private getCompiledRegexes;
     preEvaluateComment(_: CommentCreate): boolean;
     preEvaluatePost(post: Post): boolean;
     preEvaluateUser(user: UserExtended): boolean;

package/dist/userEvaluation/EvaluatePostTitleMulti.js

@@ -52,6 +52,10 @@ class EvaluatePostTitleMulti extends UserEvaluatorBase_js_1.UserEvaluatorBase {
             flags: "u",
         }));
     }
+    getCompiledRegexes() {
+        this.compiledRegexes ?? (this.compiledRegexes = this.gatherRegexes().map(r => new RegExp(r.regex, r.flags)));
+        return this.compiledRegexes;
+    }
     // eslint-disable-next-line @typescript-eslint/no-unused-vars
     preEvaluateComment(_) {
         return false;
@@ -60,8 +64,8 @@ class EvaluatePostTitleMulti extends UserEvaluatorBase_js_1.UserEvaluatorBase {
         if (post.crosspostParentId) {
             return false;
         }
-        const regexes = this.gatherRegexes();
-        return regexes.some(regexObj => new RegExp(regexObj.regex, regexObj.flags).test(post.title));
+        const regexes = this.getCompiledRegexes();
+        return regexes.some(regex => regex.test(post.title));
     }
     preEvaluateUser(user) {
         return user.nsfw;
@@ -74,14 +78,14 @@ class EvaluatePostTitleMulti extends UserEvaluatorBase_js_1.UserEvaluatorBase {
             return false;
         }
         const distinctTitles = (0, lodash_1.uniq)(userPosts.map(post => post.title));
-        const regexes = this.gatherRegexes();
-        const matchedRegexes = regexes.filter(regexObj => distinctTitles.some(postTitle => new RegExp(regexObj.regex, regexObj.flags).test(postTitle)));
+        const regexes = this.getCompiledRegexes();
+        const matchedRegexes = regexes.filter(regex => distinctTitles.some(postTitle => regex.test(postTitle)));
         const matchesNeeded = this.getVariable("matchesNeeded", 4);
         if (matchedRegexes.length < matchesNeeded) {
             return false;
         }
         const regexesInOutput = this.getVariable("regexesInOutput", 5);
-        this.addHitReason(`User has ${matchedRegexes.length} bad post titles: ${matchedRegexes.slice(0, regexesInOutput).map(r => `\`${r.regex}\``).join(", ")}`);
+        this.addHitReason(`User has ${matchedRegexes.length} bad post titles: ${matchedRegexes.slice(0, regexesInOutput).map(r => `\`${r.source}\``).join(", ")}`);
         return true;
     }
 }

package/dist/userEvaluation/EvaluateSocialLinks.d.ts

@@ -8,7 +8,6 @@ export declare class EvaluateSocialLinks extends UserEvaluatorBase {
     banContentThreshold: number;
     validateVariables(): ValidationIssue[];
     getVariableOverrides(): Record<string, unknown>;
-    private getDomains;
     preEvaluateComment(_: CommentCreate): boolean;
     preEvaluatePost(post: Post): boolean;
     preEvaluateUser(user: UserExtended): boolean;

package/dist/userEvaluation/EvaluateSocialLinks.js

@@ -36,25 +36,6 @@ class EvaluateSocialLinks extends UserEvaluatorBase_js_1.UserEvaluatorBase {
             return {};
         }
     }
-    getDomains() {
-        const domains = new Set(this.getGenericVariable("redditdomains", []));
-        domains.add("redgifs.com");
-        domains.add("instagram.com");
-        domains.add("i.redd.it");
-        const badLinks = this.getVariable("badlinks", []);
-        for (const link of badLinks) {
-            try {
-                const domain = (0, evaluatorHelpers_js_1.domainFromUrl)(link);
-                if (domain) {
-                    domains.add(domain);
-                }
-            }
-            catch {
-                //
-            }
-        }
-        return Array.from(domains);
-    }
     // eslint-disable-next-line @typescript-eslint/no-unused-vars
     preEvaluateComment(_) {
         if (this.socialLinks) {

package/dist/userEvaluation/EvaluateWorldTraveller.d.ts

@@ -8,7 +8,9 @@ export declare class EvaluateWorldTraveller extends UserEvaluatorBase {
     canAutoBan: boolean;
     banContentThreshold: number;
     validateVariables(): ValidationIssue[];
+    private subGroups;
     private getWTSubGroups;
+    private subList;
     private getSubList;
     private isInEligibleSubreddit;
     preEvaluateComment(_: CommentCreate): boolean;

package/dist/userEvaluation/EvaluateWorldTraveller.js

@@ -37,17 +37,18 @@ class EvaluateWorldTraveller extends UserEvaluatorBase_js_1.UserEvaluatorBase {
         return results;
     }
     getWTSubGroups() {
-        const subGroups = this.getVariable("subgroups", []);
-        return subGroups.map(group => ({ group, subs: group.split(",").map(s => s.trim()) }));
+        this.subGroups ?? (this.subGroups = this.getVariable("subgroups", []).map(group => ({ group, subs: new Set(group.split(",").map(s => s.trim())) })));
+        return this.subGroups;
     }
     getSubList() {
-        return this.getWTSubGroups().map(group => group.subs).flat();
+        this.subList ?? (this.subList = new Set(this.getWTSubGroups().map(group => Array.from(group.subs)).flat()));
+        return this.subList;
     }
     isInEligibleSubreddit() {
         if (!this.context.subredditName) {
             return false;
         }
-        return this.context.subredditName.startsWith(constants_js_1.CONTROL_SUBREDDIT) || this.getSubList().includes(this.context.subredditName);
+        return this.context.subredditName.startsWith(constants_js_1.CONTROL_SUBREDDIT) || this.getSubList().has(this.context.subredditName);
     }
     // eslint-disable-next-line @typescript-eslint/no-unused-vars
     preEvaluateComment(_) {
@@ -57,7 +58,7 @@ class EvaluateWorldTraveller extends UserEvaluatorBase_js_1.UserEvaluatorBase {
         if (!post.nsfw) {
             return false;
         }
-        return this.getSubList().includes(post.subredditName);
+        return this.getSubList().has(post.subredditName);
     }
     preEvaluateUser(user) {
         return user.nsfw && this.isInEligibleSubreddit();
@@ -67,10 +68,10 @@ class EvaluateWorldTraveller extends UserEvaluatorBase_js_1.UserEvaluatorBase {
         const nsfwOnly = this.getVariable("nsfwonly", false);
         const relevantHistory = this.getPosts({ since: (0, date_fns_1.subMonths)(new Date(), 1) })
             .filter(post => nsfwOnly ? post.nsfw : true)
-            .filter(post => this.getSubList().includes(post.subredditName));
+            .filter(post => this.getSubList().has(post.subredditName));
         const distinctSubreddits = (0, lodash_1.uniq)(relevantHistory.map(item => item.subredditName));
         const distinctSubGroups = (0, lodash_1.uniq)(this.getWTSubGroups()
-            .filter(group => distinctSubreddits.some(sub => group.subs.includes(sub)))
+            .filter(group => distinctSubreddits.some(sub => group.subs.has(sub)))
             .map(group => group.group));
         const requiredSubCount = this.getVariable("distinctgroups", 5);
         return distinctSubGroups.length >= requiredSubCount;

package/dist/userEvaluation/UserEvaluatorBase.d.ts

@@ -38,6 +38,8 @@ export declare abstract class UserEvaluatorBase {
     private userPosts;
     private userComments;
     protected ignoredSubs: Set<string>;
+    protected verboseLogging: boolean;
+    protected regexWarnThreshold: number;
     constructor(context: TriggerContext, history: (Post | Comment)[], socialLinks: UserSocialLink[] | undefined, variables: Record<string, unknown>);
     setHistory(history: (Post | Comment)[]): void;
     setSocialLinks(socialLinks: UserSocialLink[]): void;

package/dist/userEvaluation/UserEvaluatorBase.js

@@ -3,6 +3,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.UserEvaluatorBase = void 0;
 const tid_js_1 = require("@devvit/public-api/types/tid.js");
 const evaluatorHelpers_js_1 = require("./evaluatorHelpers.js");
+const constants_js_1 = require("../constants.js");
 class UserEvaluatorBase {
     constructor(context, history, socialLinks, variables) {
         this.variables = {};
@@ -13,6 +14,8 @@ class UserEvaluatorBase {
         this.context = context;
         this.socialLinks = socialLinks;
         this.variables = variables;
+        this.verboseLogging = context.appSlug !== constants_js_1.MAIN_APP_NAME;
+        this.regexWarnThreshold = this.getVariable("regexWarnThreshold", 100);
         this.ignoredSubs = new Set(this.getGenericVariable("ignoredsubs", []));
         this.history = history;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@fsvreddit/bot-bouncer-evaluation",
-    "version": "0.7.41",
+    "version": "0.7.43",
     "description": "Evaluation code for Bot Bouncer, a Dev Platform app designed to hunt bots on Reddit",
     "main": "dist/index.js",
     "scripts": {
@@ -37,14 +37,14 @@
         "@types/markdown-escape": "^1.1.3",
         "eslint": "^10.0.2",
         "typescript": "^5.8.3",
-        "vitest": "^4.0.15"
+        "vitest": "^5.0.0"
     },
     "keywords": [
         "reddit",
         "devvit"
     ],
     "peerDependencies": {
-        "@devvit/protos": "^0.14.1",
-        "@devvit/public-api": "^0.14.1"
+        "@devvit/protos": "^0.14.3",
+        "@devvit/public-api": "^0.14.3"
     }
 }